IdentityServer .Net Core 3.0 & Owin/Katana Token validation

Issue / Steps to reproduce the problem
Hello,
I started own project with IdentityServer4 and .Net Core 3.0. Everything works good. I'm able to generate token and consume via other .Net Core Api.
The problem occured when I trying to "consume" token via .Net Framework Api (OWIN/Katana).
I always had: { "message": "Authorization has been denied for this request." }
I enabled system.diagnostics in my Owin/Katana project and I found interesting thing:
Microsoft.Owin.Security.OAuth.OAuthBearerAuthenticationMiddleware Error: 0 : Authentication failed System.ArgumentException: IDX10703: Unable to decode the 'header': ...  as Base64url encoded string. jwtEncodedString: .... ---> System.IdentityModel.Tokens.SecurityTokenException: IDX10702: Jwt header type specified, must be 'JWT' or 'http://openid.net/specs/jwt/1.0'.  Type received: 'at+jwt'.    at System.IdentityModel.Tokens.JwtSecurityToken.Decode(String jwtEncodedString) in c:\workspace\WilsonForDotNet45Release\src\System.IdentityModel.Tokens.Jwt\JwtSecurityToken.cs:line 387    --- End of inner exception stack trace ---    at System.IdentityModel.Tokens.JwtSecurityToken.Decode(String jwtEncodedString) in c:\workspace\WilsonForDotNet45Release\src\System.IdentityModel.Tokens.Jwt\JwtSecurityToken.cs:line 376    at System.IdentityModel.Tokens.JwtSecurityTokenHandler.ReadToken(String tokenString) in c:\workspace\WilsonForDotNet45Release\src\System.IdentityModel.Tokens.Jwt\JwtSecurityTokenHandler.cs:line 627    at Microsoft.Owin.Security.Jwt.JwtFormat.Unprotect(String protectedText)    at Microsoft.Owin.Security.OAuth.OAuthBearerAuthenticationHandler.<AuthenticateCoreAsync>d__0.MoveNext()     ProcessId=20692     DateTime=2019-10-07T11:26:01.1811172Z
So I tried to decode my token. I used jwt.io and it is the result:

The type is: 'at+jwt' <----- so it's why my Owin project didnt understand it.
The problem is in Owin and System.IdentityModel.Tokens.Jwt
I cannot use version 5+ of System.IdentityModel.Tokens.Jwt because of Owin (Owin doesnt support it) so my token cannot be parsed through api. Only this version is able to read token with type: 'at + jwt'.
I created test project (simple .NET Framework library + xunit). I added System.IdentityModel.Tokens.Jwt 5.5.0 and eveyrhing was read correctly!
So now it is not possible to use IdentityServer4 in .Net Core 3 and consume through Owin/Kata Api but..... I repeated eveyrhing with .Net Core 2.2 -> I used IdentityServer4 with .Net core 2.2. I generated token and I checked again on jwt.io and here is the result:

Now type is 'JWT' and my Owin/Katana Api authenticated correctly so the summary is:
When I generate token with IdentityServer4 and .Net Core 3.0 it doesn't work
When I generate token with IdentityServer4 and .Net core 2.2 it works

Explicit typing of JWTs is recommended these days, that's why we made that change:
https://leastprivilege.com/2019/09/13/use-explicit-typing-for-your-jwts/
You can revert back to the old behavior by setting IdentityServerOptions.AccessTokenJwtType to empty string or JWT.

https://github.com/IdentityServer/IdentityServer4/issues/3706

Dominik - thanks. But it not resolve my problem.

in old version - "aud" has additional  URL,
"jti" - it looks like it use another algorithm for hash

Yes - the/resources aud has been removed in favour of a typed JWT. If that causes problems you can add it back manually.

Hi - I do not know - I need to check
Also JTI looks different - in old it looks like guid, in new it is some hash.
I think problem could be in IdentityModel - so .net 4.6 can convert IdentityModel 4.0 to IdentityModel 3.10.6.
in 3.0.0-preview3.4 - IdentityModel (>= 3.10.6), in newer one IdentityModel (>= 4.0.0-preview.3).

/resources

Yes - the/resources aud has been removed in favour of a typed JWT. If that causes problems you can add it back manually.

how you can add it back?
thank you

https://identityserver4.readthedocs.io/en/latest/search.html?q=EmitLegacyResourceAudienceClaim&check_keywords=yes&area=default


Properly find natvis file for UE5.3

For some reason, they added another file that breaks the current logic, so I changed it to use a for loop. I don't have the ability to test on previous versions of Unreal, but it should work fine there too.
I'm not too sure about it, there may be some perf impact since we are then looping through stuff and doing string checks but I think that should be okay? Feel free to suggest any changes!

Hi @AWildErin, thanks for point this out. I'm going to investigate this issue.

perfomance won't be an issue. Iterated collections are very small in terms of elements.
Refactored code + apply some code formatting
Tested on UE v5.3.1
Regression Tested on UE v5.2.1,  v5.0.3, v4.27.2
close #22


Add get_delegee_scheduler query.

Adds the get_delegee_scheduler receiver query to implement issue #294. get_delegee_scheduler is forwarded independently from get_scheduler and forwards through the on algorithm. This allows a forward progress delegation target, like sync_wait's scheduler, to forward through to schedulers and algorithms that may use it to support forward progress delegation.
For example, in:
sync_wait(on(system_thread_pool, just() | sort(...)));

the sort implementation and/or system_thread_pool scheduler may delegate work to the blocked thread to ensure forward progress in the same fashion that we see now with the standard parallel algorithms.

Can you update the reference implementation to match?

I did. Somehow I just forgot to add it to git.

Oops, a test failure.


Move from bash to zsh

bash vs zsh

[x] add `chsh -s /bin/zsh or /bin/bash to instructions
[x] check ohmyzsh
[x] fix terminal fonts for vscode

done


Improve performance of CheckSuspiciousContent

PR Summary
By commit:

Use MIN_LEN and MAX_LEN constants for better readability.
Skip hash searches for length < MIN_LEN.

Before we searched for strings with lengths 1, 2, and 3 but shortest pattern (Emit) has length 4.

Use Span<> instead of array

Exclude array allocations
Exclude boundary checks in loops

Replace multiplication with more efficient operations

Multiplying a value by 31 is multiplying by 32 minus one value. Then change the multiplication to a shift. (The compiler knows how to replace multiplication by a power of two with a shift, but we do this explicitly to be safe.)

Improve to-lower-case

before we do 4-8 operations, 6 in most cases
after we do 3-5 operations, 3 in most cases

Results for Match("aaaaaaaEmitaaaaaaa"):

BenchmarkDotNet=v0.13.2, OS=Windows 10 (10.0.19044.2251/21H2/November2021Update)
Intel Core i5-2410M CPU 2.30GHz (Sandy Bridge), 1 CPU, 4 logical and 2 physical cores
.NET SDK=7.0.100
  [Host]     : .NET 7.0.0 (<IP_ADDRESS>805), X64 RyuJIT AVX
  DefaultJob : .NET 7.0.0 (<IP_ADDRESS>805), X64 RyuJIT AVX

Method
text
Mean
Ratio
Code Size
Allocated
Alloc Ratio

Orig
aaaaaaaEmitaaaaaaa
423.8 ns
1.00
4,445 B
144 B
1.00

Fast
aaaaaaaEmitaaaaaaa
266.0 ns
0.63
4,493 B
-
0.00

PR Context

PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[x] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[ ] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and opened an issue in the relevant tool repository. This may include:

[ ] Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode
(which runs in a different PS Host).

[ ] Issue filed:

[ ] Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.

[ ] Issue filed:

[ ] Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).

[ ] Issue filed:

[ ] Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

[ ] Issue filed:

LookUpHash() uses a large switch statement, would it be faster to change to a dictionary?

LookUpHash() uses a large switch statement, would it be faster to change to a dictionary?

I tested this and found the switch statement is more faster (looking asm code I guess C# compiler tries to do a binary search).

@iSazonov @TravisEz13's comments were not addressed. Also, I doubt if Travis still has his original tests available, but will let Travis to confirm.

LOG const remove at all.


fix: give to var.handler priority for python runtime

What does this PR do?
This pull request wants to address the following issue: https://github.com/DataDog/terraform-aws-lambda-datadog/issues/19
In other words, we want to give priority to the var.handler variable over the default values set by DataDog.
Motivation
When we migrate from aws_lambda_function to lambda-datadog we do not need to change the handler name for the already existing functions.
In fact, we do not want to delete and recreate the existing functions, we want to migrate them to the new lambda-datadog one.
Additional Notes
Describe how to test/QA your changes
Try to deploy a lambda function with:
   runtime          = "python3.9"

@duncanpharvey can I have please a feedback on this issue/pr?
We would like to migrate to lambda-datadog in our stack without the need to change the existing lambda functions.
Can this change be accepted or it goes against any particular design choices on DD?

Thanks for the explanation @duncanpharvey
Closing the pr since it is not needed.


[popups] don't break old editor when going back from builder

close https://github.com/CartoDB/cartodb/issues/8312
added the missing "none" templates, and some other details
I'm don't know why it is needed the mustache-templates/infowindows/tooltip_light something related to the migrator seems not OK (find trace below)
https://rollbar.com/vizzuality/CartoDB/items/19219/
also, when going back to old editor it's needed a cartodb/table/views/tooltip/custom_templates/infowindow_light (reported https://github.com/CartoDB/cartodb/issues/8333)
added them both and I created an issue for the later, but it's smelly at least
CR @xavijam /cc @juanignaciosl

Maybe not because of the migrator but because of vizjson 3 or any new presenter. If you think that something should be fixed in the backend let me know, please.

@juanignaciosl  what about the trace?

@matallo damn, I've seen the error where backend is mixing infowindows and tooltip paths. On it. #8327

🚀


Entrypoint does not support arguments

Unless I'm missing something obvious, entrypoint does not seem to support arguments. For example if I configure the plugin like the following:
<configuration>
  <images>
    <image>
      <name>ubuntu</name>
      <alias>test-container</alias>
      <run>
        <entrypoint>/bin/echo hello world</entrypoint>
      </run>
    </image>
  </images>
</configuration>

The build fails with:
[ERROR] Failed to execute goal org.jolokia:docker-maven-plugin:0.11.0:start (start-containers) on project docker-maven-plugin-entrypoint: Error while calling docker: Starting container with id 3e9420f4ff87b0213147eb165ab021b2889943ecf4fe0bb4d8f114b616b946fa (Not Found: 404)

However , the build runs fine if I remove the arguments from entrypoint. Not sure if this is intentional but entrypoint should support arguments just like the normal command configuration.
I've fixed this on my fork in this commit 63f8165200a2b5d596366d3be00395674c315e40. If you're happy to accept a PR, let me know.

Yes, you are right. I will going to fix that for the next release and copy over your fix directly into the integration branch (if you don't mind) . So no need for a PR.
Thanks ...

Yes, no problem. Thanks!


Queries are getting executed sequentially even when using Interactive transactions API using Promise.all().

Bug description
I want to execute some updateMany/createMany calls in different tables concurrently, basically i want prisma to call them all in 1 go and commit all of them if all succeeds, or else rollback all of them if any one fails.
I thought of using interactive transactions API for executing this, and then call Promise.all() with all those transaction API calls, waiting for their result.
but when i console.log the way prisma is executing these queries, what i see in console is that prisma is executing these queries sequentially only, i see that it prints ->
rest-api:dev: Query: UPDATE "public"."contacts" SET "last_name" = $1, "description" = $2, "image_source" = $3, "starred" = $4, "company" = $5, "next_reminder_at" = $6, "last_seen_at" = $7, "job_title" = $8 WHERE ("public"."contacts"."id" = $9 AND 1=1) RETURNING "public"."contacts"."id", "public"."contacts"."user_id", "public"."contacts"."first_name", "public"."contacts"."last_name", "public"."contacts"."description", "public"."contacts"."birthday", "public"."contacts"."legacy_location", "public"."contacts"."education", "public"."contacts"."job_title", "public"."contacts"."company", "public"."contacts"."image_source", "public"."contacts"."image_url", "public"."contacts"."starred", "public"."contacts"."last_seen_at", "public"."contacts"."last_reminder_at", "public"."contacts"."facebook", "public"."contacts"."linkedin", "public"."contacts"."twitter", "public"."contacts"."instagram", "public"."contacts"."created_at", "public"."contacts"."updated_at", "public"."contacts"."never_keep_in_touch", "public"."contacts"."telegram", "public"."contacts"."website", "public"."contacts"."next_reminder_at", "public"."contacts"."source", "public"."contacts"."is_archived", "public"."contacts"."birthday_year", "public"."contacts"."linkedin_data", "public"."contacts"."linkedin_enhance_date", "public"."contacts"."ignore_merge", "public"."contacts"."full_name", "public"."contacts"."business_card_url", "public"."contacts"."linkedin_companies", "public"."contacts"."linkedin_education", "public"."contacts"."linkedin_last_message_at", "public"."contacts"."linkedin_last_message_snippet", "public"."contacts"."linkedin_message_link"
rest-api:dev: Params: ["Sun","
new description

---

founder at Dex

---

Lebra-, Co-Founder
",null,false,"Dex-Lightning",null,"2024-06-04 21:06:45.757 UTC","Dex-Lightning Founder","f48f2d60-df41-4daf-a4bc-4319ee6ba93b"]
rest-api:dev: Duration: 563ms
rest-api:dev: Query: INSERT INTO "public"."contact_phone_numbers" ("user_id","created_at","contact_id","phone_number","label","ranking","updated_at") VALUES ($1,$2,$3,$4,$5,$6,$7) ON CONFLICT DO NOTHING
rest-api:dev: Params: ["gXkM2Ax2FbdiyR1WVQoe5ZFwouD2","2024-06-04 21:09:25.504 UTC","f48f2d60-df41-4daf-a4bc-4319ee6ba93b","8888888888",null,-1,"2024-06-04 21:09:25.504 UTC"]
rest-api:dev: Duration: 547ms
rest-api:dev: Query: INSERT INTO "public"."contact_emails" ("label","email","user_id","ranking","updated_at","created_at","contact_id") VALUES ($1,$2,$3,$4,$5,$6,$7) ON CONFLICT DO NOTHING
rest-api:dev: Params: [null,"dex-lightning@gmail.com","gXkM2Ax2FbdiyR1WVQoe5ZFwouD2",-1,"2024-06-04 21:09:25.504 UTC","2024-06-04 21:09:25.504 UTC","f48f2d60-df41-4daf-a4bc-4319ee6ba93b"]
rest-api:dev: Duration: 558ms
rest-api:dev: Query: INSERT INTO "public"."contacts_custom_fields" ("contact_id","user_id","custom_field_id") VALUES ($1,$2,$3) ON CONFLICT DO NOTHING
rest-api:dev: Params: ["f48f2d60-df41-4daf-a4bc-4319ee6ba93b","gXkM2Ax2FbdiyR1WVQoe5ZFwouD2","8a5ff124-a8dc-446f-bf39-ec9848b1983e"]
rest-api:dev: Duration: 543ms
rest-api:dev: Query: INSERT INTO "public"."groups_contacts" ("contact_id","group_id","user_id") VALUES ($1,$2,$3) ON CONFLICT DO NOTHING
rest-api:dev: Params: ["f48f2d60-df41-4daf-a4bc-4319ee6ba93b","11c7b912-1316-409d-9309-0f3e47948ff2","gXkM2Ax2FbdiyR1WVQoe5ZFwouD2"]
rest-api:dev: Duration: 553ms

which clearly indicates that it is getting executed sequentially 1 after the other and in total the API returns the reponse in around ~5 seconds whereas it should have returned it in ~1 second if it'd have executed them in parallel.
any guide/help here would be really useful.
How to reproduce
use prisma.$transaction() interactive transaction API with Promise.all() inside it.
Expected behavior
No response
Prisma information
await prisma.$transaction(
      async (tx: Prisma.TransactionClient) => {
        ...
          await performUpdates(tx, targetId, userId, coalesceUpdates, missingEmails, missingPhoneNumbers, missingLegacyContactAddresses, missingCustomFields, missingGroupsContacts, uniqueRelations, mergeIds);
          },
          { timeout: 30000, maxWait: 60000 }
        );

          async function performUpdates(
              tx: Prisma.TransactionClient,
              targetId: string,
              userId: string,
              coalesceUpdates: Prisma.contactsUncheckedUpdateInput,
              missingEmails: ContactEmail[],
              missingPhoneNumbers: ContactPhoneNumber[],
              missingLegacyContactAddresses: LegacyContactAddress[],
              missingCustomFields: ContactsCustomField[],
              missingGroupsContacts: GroupsContact[],
              uniqueRelations: ContactRelation[],
              mergeIds: string[]
            ) {
              await Promise.all([
                tx.contacts.update({ where: { id: targetId }, data: coalesceUpdates }),
                missingPhoneNumbers.length ? tx.contact_phone_numbers.createMany({ data: missingPhoneNumbers.map((l) => ({ user_id: userId, contact_id: targetId, phone_number: l.phone_number, label: l.label })), skipDuplicates: true }) : Promise.resolve([]),
                missingEmails.length ? tx.contact_emails.createMany({ data: missingEmails.map((e) => ({ user_id: userId, contact_id: targetId, email: e.email.trim().substring(0, 319), label: e.label })), skipDuplicates: true }) : Promise.resolve([]),
                missingLegacyContactAddresses.length ? tx.legacy_contact_addresses.createMany({ data: missingLegacyContactAddresses.map((address) => ({ user_id: userId, contact_id: targetId, formatted: address.formatted, street: address.street, city: address.city, region: address.region, country: address.country, postal_code: address.postal_code, address_type: address.address_type })), skipDuplicates: true }) : Promise.resolve([]),
                missingCustomFields.length ? tx.contacts_custom_fields.createMany({ data: missingCustomFields.map((customField) => ({ user_id: userId, contact_id: targetId, custom_field_id: customField.custom_field_id })), skipDuplicates: true }) : Promise.resolve([]),
                missingGroupsContacts.length ? tx.groups_contacts.createMany({ data: missingGroupsContacts.map((group) => ({ user_id: userId, contact_id: targetId, group_id: group.group_id })), skipDuplicates: true }) : Promise.resolve([]),
                tx.timeline_items_contacts.updateMany({ where: { contact_id: { in: mergeIds }, user_id: userId }, data: { contact_id: targetId } }),
                tx.reminders_contacts.updateMany({ where: { contact_id: { in: mergeIds }, user_id: userId }, data: { contact_id: targetId } }),
                tx.contact_relations.createMany({
                  data: uniqueRelations,
                  skipDuplicates: true
                })
            ]);
}

Environment & setup

OS: macOS
Database: PostgreSQL
Node.js version: 19.1.0

Prisma Version

@janpio any help/guidance here would be really helpful.

Please don't open discussions and issues at the same time.
=> https://github.com/prisma/prisma/discussions/24452


One page checkout creating new address in profile every time.

I have latest version of bagisto with default theme.
during checkout option, Selected address from my list of address and selected use same as shipping address for billing address. But during checkout I found it create new address every time.

Hi @pbhavesh45
This bug is already raised https://github.com/bagisto/bagisto/issues/2987

Hi @pbhavesh45
This bug is already raised #2987

PHP
PHP 7.4.5 (cli) (built: Apr 16 2020 17:13:20) ( NTS )
Copyright (c) The PHP Group
Zend Engine v3.4.0, Copyright (c) Zend Technologies
with Zend OPcache v7.4.5, Copyright (c), by Zend Technologies


Feature request: inspect $refs

It would be nice to be able to inspect this.$refs
Just like there is $route in a component, maybe it could have $refs below it?

can you elaborate a bit on why it would be nice?

Sometimes I need to see is inside a certain ref. Sometimes a ref is undefined and I want to know what refs a component has and confirm that it is actually undefined or that it was due to the component lifecycle.
Being able to see the $el inside a ref can help debugging as well.

Personally, I think it could be useful to highlight content as you hover them (pretty much like in the components panel) because you sometimes need to locate refs in the page and, as you said, know if they exist (undefined or not)

Yea, the ability to go from the vue devtools page to the Elements page would be really cool.
So for example, I click on a ref called video, it would go to the Elements tab (in Chrome dev tools) and highlight that video element.

I think I could implement it over the weekend, let me try and get a PR going.

I've sat with this problems for a couple of hours and I think it might not be possible to this functionality now. I've encountered the same problem when trying to implement context menu's Inspect element button. Object are serialized when sending between shell and DevTools panel, thus we're losing its identity, and inspection API requires to call it with object we want to serialize.
// this is now we call inspect API
chrome.devtools.inspectedWindow.eval(someHTMLElement);

// but during inspection of component we have to call 'stringify(getIntanceDetails)'
// which extract data we want but serializes it to JSON, in case of HTMLElement we're
// left with it's string representation
bridge.on('select-instance', id => {
  ...
  bridge.send('instance-details', stringify(getInstanceDetails(id)))
})

Anyone has idea how to work around it?

@posva is it possible as a first step for now to at least show the available refs under each component ?

I don't think listing them is enough to actually be useful 🙁  You'll end up just clicking on the element in the devtools and writing $vm.$refs in the console to properly inspect

Would like to see the list of $refs as well.

Hi, I've managed to make this feature but with some dirty hacks (from my point of view). Also I'm not very good at styling things so for now this feature looks like this http://nimb.ws/Ykq09X.
So is there some one that can look at screenshot and help me to make look it more pretty ?

Would be nice to see it land in the next version. 👏 @1099511627776

@1099511627776 @Akryum when inspecting the component with the select btn the console will start logging msgs each time u move the mouse, can u dbl check ?
version 5.0.0.b3


Instructions for Mac

Add some instructions for installing and running the exercises in Mac OS.

@sr-murthy could you add some basic instructions for running in a Mac and verify. Thanks

Updated README on a branch
https://github.com/OasisLMF/Workshop2019/blob/fix/readme/README.md
I'm not able to test the instructions on my system as it would involve potential side effects on the system libraries, and I don't have another Mac available to test on, but the instructions are fairly simple and mainly use brew (equivalent of apt on Debian/Linux) and the Xcode command line tools, and installing both of these is also included in the README.

@mpinkerton-oasis The README has been updated with OS X instructions in this branch.


Build using Python 3.13

Build Python 3.13 wheels.

Looks like we're missing Python 3.13:
https://discourse.vtk.org/t/vtk-9-4-0-release/14577/19


Grouped Headers using Hypergrid Schema

Im having a problem with the plugin, beacuse im using the schema option in hypergrid instantiation.
In the demo its not used like this so my quesiton are:

is there a demo with schema?
the specific problem I have right now its when i set the headers, the parent header text is not centered.
is there a function like:
grid.setSchema({
field:'header1', name:'header1',
field:'header2', name:'header2',
field:'header3', name:'subrow|header3',
field:'header4', name:'subrow|header4',
field:'header5', name:'subrow|header5',
})

Thanks!!

@joneit

@eibertek I'm assuming your referring to v2.0.2 using the grouped header demo here
You can pass in the the schema in an options object with any setData [call])(https://github.com/fin-hypergrid/core/blob/master/src/Hypergrid.js#L814) or directly with or assigning to grid.behavior.dataModel.schema explicitly (which is a setter


Are colors supposed to auto update whenever the theme is changed?

I noticed that the readme says
"a Neovim colorscheme plugin that automatically fetches and applies the colors that are auto generated by Pywal."
I assume this means that the plugin watches for changes in ~/.cache/wallust/colors_neopywal.vim/~/.cache/wal/colors-wal.vim and automatically reapply the theme with the updates colors.  However, I have to manually run :coloscheme neopywal each time I change my colorscheme.  I was wondering if this was the intended behavior or not.

Actually i am not sure if such thing would even be possible in the first place.
By "automatically fetches" what i meant was "you change your pywal theme, neopywal theme is updated", but that would still require you to either close and reopen neovim or run :colorscheme neopywal.
Despite saying that, maybe there's a way to detect changes on those files that i am not aware of. It would be a neat feature to have if it's actually possible so i may take a look at that in the future, but i won't promise anything.

Update: Never mind turns out doing such thing was extremely easy to implement.
Expect that feature to be available on  v2.5.0 :) (unless i find some critical bug with my implementation cannot be fixed ...).

In the meantime I got it working with fwatch.
    -- Use fwatch to watch for changes in wallust theme
    local fwatch = require('fwatch')
    fwatch.watch(vim.env.HOME .. "/.cache/wallust/colors_neopywal.vim", "colorscheme neopywal")


Add to CocoaPods repo

SwiftMonkey and SwiftMonkeyPaws should be properly added to CocoaPods. I have no idea how yet, so if anyone wants to do it quickly, please go ahead!

I recommend you doing it. Otherwise it will be in the other person's account
pod trunk push
https://guides.cocoapods.org/making/getting-setup-with-trunk.html


Edit/a3g

@MikeWasson,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Implement useable subset of USB Audio Class 2 (Audio + MIDI)

MIDI already works (see https://github.com/greatscottgadgets/luna/pull/94).
Unfortunately I cannot test Audio because of https://github.com/greatscottgadgets/luna/issues/86
But the descriptors build well in examples and since MIDI works I think it is definitely
worthy of sharing.
I will make amends to the audio parts as soon as the mentioned issue is resolved.

Awesome -- appreciate this a ton! Merged. :)

Thank you so much!


Add support for multiple implementations in operationId

To make it possible to use the same swagger.yaml file, make it possible to specify several operationId entry points. Like:
operationId:
  connexion:
    app.get_users
  swagger1st:
    get_users

Or maybe even to somehow pin the project to enable not per-swagger-first-library, but per-project distinction.

According to Swagger Spec operationId has to be a string.
Connexion supports Jinja2 templating and Selmer for Clojure seems similar, so maybe it's possible to change the operationId based on the implementation:
operationId: {% if using_connexion %}app.get_users {% else %}get_users{% endif%}

how a JSON or YAML is not string? :D

The templated version looks really horrible.
operationId: '{"connexion": "app.get_users", "swagger1st": "get_users"}'

though it is not quite clear, if this version with JSON as a string is better :)

@jmcs is right, "operationId" must be a single string.
We could implement some name mangling, i.e. using one operationId for both Connexion and Swagger1st and automatically finding the correct function:

Connexion could use a configurable base module to prepend
Swagger1st could use a configurable base namespace to prepend and additionally replace underscores with hyphens

Small introduction how the operationId is handled in https://github.com/sarnowski/swagger1st :
The configured executor has a generic way of plugging in any operationId evaluation you need. The default implementation resolves the operationId string to a full qualified Clojure function.
However, this can be configured as you like. The resolver takes a function that takes the argument operationId (actually the whole request definition) and returns a function that should be called upon request. This generic functional approach allows you to plug in every logic you might want. Some examples:
(s1st/executor
  :resolver (fn [request-definition]
               (let [operationId (get request-definition "operationId")]
                 (function-by-name operationId))))

You can also think of completely different implementations like a SQL executor:
(s1st/executor
  :resolver (fn [request-definition]
               (let [sql-query (get request-definition "operationId")]
                 (fn [request]
                   (jdbc/execture sql-query)))))

For this issue, there are only 2 clean ways in my opinion:
Way 1: Abstract Idenfitier
The Swagger Spec also specifies the operationId as an identifier for the operation. There is no specification that this has to translate to a language specific construct. One way to tackle the language-independent integration is to make this identifier compeltely abstract and let the implementations resolve it.
operationId: "foo"

Using a mapping table in the implementation, one is able to determine the correct action. Luckily, Clojure's map is also a function, so a map can directly be used as a resolver in swagger1st:
(s1st/executor
  :resolver {"foo" api/my-foo-fn
             "bar" api/my-bar-fn})

I consider this the cleanest solution as it makes no assumption at all about the implementation.
Way 2: Declarative Configuration
If you really want to encapsulate implementation specific information per implementation, then I would go for embedded JSON as a declarative way:
operationId: '{"connexion": "my-py-fn", "swagger1st": "my-clj-fn"}'

This way, it is trivial for any implementation to get to is implementation specific value by just parsing JSON (which is the easiest format in any language):
(s1st/executor
  :resolver (fn [request-definition]
               (let [operationId (get request-definition "operationId")
                     s1st-operationId (-> operationId json/parse :swagger1st)
                 (function-by-name s1st-operationId))))

I would discourage any idea of trying to use a turing complete templating engine to "generate" a correct string!

As I understand, way2 can be considered as one of the default implementations of the way1.

I'm completely against having way 2 as default on connexion, not only it's hacky, it would also break existing code. Adding support for a custom mapper seems cleaner.

@valgog in case of swagger1st, this would be a custom handler anyway yes

@jmcs one should not really break anything, one could implement the resolver with a fallback possibility.
Anyway, even if we would decide to go with the Way2, we have to have a solution for the Way1 in connexion. Do we have it already?

@valgog no, not yet, I would propose adding some "operation resolver" configuration to the Api class (default implementation like it is now).

@hjacobs I agree, I will assign this issue to my self to implement this approach.


duplicate connection created when send multiple request to one server

when sending multiple request(the same host), duplicate connection may be created due to the synchronized method in function findConnection().
That there were several synchronized (connectionPool) blocks, and 2 request may entry the code of findConnection() method, and create 2 connections.
the deduplicate() method could remove the duplicated connection, but actually it was done after the connection operator, so it will waste server resource and client resource.
so anyone can provide some suggestion to optimize it?

to avoid duplicated connection creating, one way is to use one synchronized (connectionPool) in findConnection() method or use synchronized block to call the method.
another way is to use named lock for each server, specified by the address(host and port).

One problem is that we don't know whether the connection will support HTTP/2 until after we’re connected. Locking would slow down requests that don't have it.

@swankjesse thanks.
We are porting okhttp3 to support our customized protocol (connections reusing over tcp). so we are trying to use named lock (for each server) to avoid create duplicated connection and reduce the server pressure, since the server need to handle hundreds of thousands connection.

Sounds good. No action for us to take here.


Are FastRCNN models from TorchVision supported in TRTorch?

❓ Question
Tried FastRCNN and MaskRCNN models from TorchVision. The model fails to compile with error "RuntimeError: tuple appears in op that does not forward tuples, unsupported kind: aten::append"
What you have already tried
code to reproduce:
import torch
print(torch.version)
import trtorch
print(trtorch.version)
import torchvision
fastrcnn = torchvision.models.detection.fasterrcnn_resnet50_fpn(pretrained=True)
model = fastrcnn.eval().to("cuda")
scripted_model = torch.jit.script(model)
compile_settings = {
"input_shapes": [ [3, 300, 400],[3, 300, 400] ],
"op_precision": torch.float
}
trt_model = trtorch.compile(scripted_model, compile_settings)
Environment

Build information about the TRTorch compiler can be found by turning on debug messages

PyTorch Version (e.g., 1.0): 1.8.1
CPU Architecture:
OS (e.g., Linux): Ubuntu
How you installed PyTorch (conda, pip, libtorch, source): pip
Build command you used (if compiling from source): N/A
Are you using local sources or building from archives:  N/A
Python version: python3.7
CUDA version: 11.1
GPU models and configuration:
Any other relevant information: TRTorch - 0.3.0

Additional context

Can you share some more information ?
Is the error coming from torch.jit.script () call or from trtorch.compile ?
If it's a TRTorch error, can you enable debug log by adding trtorch.logging.set_reportable_log_level(trtorch.logging.Level.Debug) to your python script and run ? This can help to identify which layer this might be coming from.

RuntimeError: tuple appears in op that does not forward tuples, unsupported kind: aten::append is from trtorch.compile.
torch.jit.script() works fine.
I made the changes to the script to enable debug log but didn't get any additional information.

I'm running into the exact same problem.  torch.jit.script works just fine, but trtorch.compile throws an exception.
Here's a Dockerfile to reproduce the issue:
FROM pytorch/pytorch:1.8.1-cuda11.1-cudnn8-devel

# Have to download and copy over the TensorRT .deb package
COPY nv-tensorrt-repo-ubuntu1804-cuda11.1-trt<IP_ADDRESS>-ga-20210226_1-1_amd64.deb ./
RUN dpkg -i nv-tensorrt-repo-ubuntu1804-cuda11.1-trt<IP_ADDRESS>-ga-20210226_1-1_amd64.deb

RUN apt update && apt install -y libnvinfer-dev=7.2.3-1+cuda11.1 \
    libnvinfer-plugin-dev=7.2.3-1+cuda11.1 \
    libnvparsers-dev=7.2.3-1+cuda11.1 \
    libnvonnxparsers-dev=7.2.3-1+cuda11.1 \
    libnvinfer-samples=7.2.3-1+cuda11.1 \
    tensorrt

RUN pip install https://github.com/NVIDIA/TRTorch/releases/download/v0.3.0/trtorch-0.3.0-cp38-cp38-linux_x86_64.whl

Running the following script gives the same error as mentioned earlier:
import torch
from torchvision.models.detection import maskrcnn_resnet50_fpn
import trtorch

model = maskrcnn_resnet50_fpn()
scripted = torch.jit.script(model)
trt = trtorch.compile(scripted, {"input_shapes": [(1, 3, 512, 512)]})

Output:
RuntimeError                              Traceback (most recent call last)
<ipython-input-1-bf759ff71e00> in <module>
      5 model = maskrcnn_resnet50_fpn()
      6 scripted = torch.jit.script(model)
----> 7 trt = trtorch.compile(scripted, {"input_shapes": [(1, 3, 512, 512)]})

/opt/conda/lib/python3.8/site-packages/trtorch/_compiler.py in compile(module, compile_spec)
     71             "torch.jit.ScriptFunction currently is not directly supported, wrap the function in a module to compile")
     72
---> 73     compiled_cpp_mod = trtorch._C.compile_graph(module._c, _parse_compile_spec(compile_spec))
     74     compiled_module = torch.jit._recursive.wrap_cpp_module(compiled_cpp_mod)
     75     return compiled_module

RuntimeError: tuple appears in op that does not forward tuples, unsupported kind: aten::append

Same Problem.
Please tell me how to resolve the error!!
Error converting detectron2 to TensorRT
I'm converting the Detectron 2 Faster-RCNN FPN Model to Tensor RT.
(1)Convert the Model of Detectron2 to Torchscript and save it in a file
(2)Read Torchscript from the converted file and compile it to TensorRT.
The following error has occurred in the compilation of (2) above.
Environment
ubuntu18.04
CUDA 11.3
torch 1.10.1 + cu113
torchvision 0.11.2 + cu113
torch-tensorrt 1.0.0
Error
[W lower_tuples.cpp:209] Warning: tuple appears in op inputs, but this op does not forward tuples, unsupported kind: aten::append (function flattenInputs)
[W lower_tuples.cpp:209] Warning: tuple appears in op inputs, but this op does not forward tuples, unsupported kind: prim::SetAttr (function flattenInputs)
[W lower_tuples.cpp:209] Warning: tuple appears in op inputs, but this op does not forward tuples, unsupported kind: prim::SetAttr (function flattenInputs)
[W lower_tuples.cpp:248] Warning: tuple appears in the op outputs, but this op does not forward tuples, unsupported kind: prim::GetAttr (function flattenOutputs)
Traceback (most recent call last):
  File "tools/compile_torchscript.py", line 61, in <module>
    main()
  File "tools/compile_torchscript.py", line 56, in main
    trt_ts_module = torch_tensorrt.compile(script_model,
  File "/home/ubuntu/detectron2_v0.5_aws/venv/lib/python3.8/site-packages/torch_tensorrt/_compile.py", line 97, in compile
    return torch_tensorrt.ts.compile(ts_mod, inputs=inputs, enabled_precisions=enabled_precisions, **kwargs)
  File "/home/ubuntu/detectron2_v0.5_aws/venv/lib/python3.8/site-packages/torch_tensorrt/ts/_compiler.py", line 119, in compile
    compiled_cpp_mod = _C.compile_graph(module._c, _parse_compile_spec(spec))
RuntimeError: tuple use not matched to tuple construct. Instead found: aten::append

Code
    data_loader = build_detection_test_loader(cfg, cfg.DATASETS.TEST[0])
    first_batch = next(iter(data_loader))
    inputs = [{"image": first_batch[0]["image"]}]
    with torch.no_grad():
        fields = {
            "proposal_boxes": Boxes,
            "objectness_logits": Tensor,
            "pred_boxes": Boxes,
            "scores": Tensor,
            "pred_classes": Tensor,
            "pred_masks": Tensor,
        }
        model = torch.jit.load("model_final.pt")
        script_model = scripting_with_instances(model, fields)
        print(script_model)
        start = perf_counter()
        scripted_instance = \
        script_model.inference(inputs, do_postprocess=False)[0]
        end = perf_counter()
        print(end - start)
        compile_settings = {
            "inputs": [torch_tensorrt.Input(
                min_shape=[1, 3, 224, 224],
                opt_shape=[1, 3, 512, 512],
                max_shape=[1, 3, 1024, 1024],
                dtype=torch.half
                # Datatype of input tensor. Allowed options torch.(float|half|int8|int32|bool)
            )],
            "enabled_precisions": {torch.half},  # Run with FP16
        }
        trt_ts_module = torch_tensorrt.compile(script_model,
                                               **compile_settings)

Same issue here, debug a little deeper, the error is thrown from torch::jit::LowerAllTuples(g); in torch_tensorrt::core::lowering::LowerGraph.
You do have evaluator for aten::append, however, that seems to be too late for this model. Do you have any plan to fix this issue?

Besides, as for now, Torch-TensorRT doesn't seem to support models with Tensor[] as input. For example, for FasterRCNN model, the model is defined as:
graph(%self : __torch__.torchvision.models.detection.faster_rcnn.___torch_mangle_47.FasterRCNN,
      %images.1 : Tensor[],
      %targets.44 : Dict(str, Tensor)[]?):

And Torch-TensorRT requires the inputs to be Union(torch_tensorrt.Input, torch.Tensor).

@bowang007 has been working on these models so he might have some insight


Gp Error when creating a claim

The error code will show up instead of saying the minimum amount of blocks the claim must be in order to be claimed

Yeah it's a known issue, I think the plugin developer is working on it.

Should work fine now, we've updated. Closing


USB: Add usb fs driver for userspace transfer

Summary
Add usb_fs driver so that userspace can directly transfer USB packets through the EP node.
ADB, Fastboot, MTP will use usb_fs, these class driver only need to provide the descriptor information and register the usb_fs device.
This PR depends on https://github.com/spiriou/microADB/pull/36.
Impact
microADB device change from /dev/adb0 to /dev/adb0/ep0 and dev/adb0/ep1
Testing
sim:usbdev and bes board usb adb

@spiriou please take a look!

@spiriou please take a look!

I'm not sure to understand how this PR would change the way EP are exposed to apps.
I can only see USB descriptors refactoring here. Can you share more details?

@spiriou please take a look!

I'm not sure to understand how this PR would change the way EP are exposed to apps. I can only see USB descriptors refactoring here. Can you share more details?

The patch has been updated, please take a look.

@spiriou please take a look!

I'm not sure to understand how this PR would change the way EP are exposed to apps. I can only see USB descriptors refactoring here. Can you share more details?

@spiriou the major change is:

Move the common code to usbdev_fs, so all pass-through driver(e.g. adb, fastboot, mtp) can share the implementation.
Split dev/adb0 to /dev/adb0/ep0 and /dev/adb0/ep1 since many usb driver has more than two data endpoint.

@spiriou please take a look!

I'm not sure to understand how this PR would change the way EP are exposed to apps. I can only see USB descriptors refactoring here. Can you share more details?

@spiriou the major change is:

Move the common code to usbdev_fs, so all pass-through driver(e.g. adb, fastboot, mtp) can share the implementation.
Split dev/adb0 to /dev/adb0/ep0 and /dev/adb0/ep1 since usb interface may have more than two data endpoints.

@spiriou please take a look!

I'm not sure to understand how this PR would change the way EP are exposed to apps. I can only see USB descriptors refactoring here. Can you share more details?

@spiriou the major change is:

Move the common code to usbdev_fs, so all pass-through driver(e.g. adb, fastboot, mtp) can share the implementation.
Split dev/adb0 to /dev/adb0/ep0 and /dev/adb0/ep1 since many usb driver has more than two data endpoint.

I'm not against this change although I feel like a char device with a single file descriptor makes more sense to interact with the ADB device endpoints, compared to 2 (ie. 1 file descriptor for read operations and another one for write operations). The ADB driver is very similar to the CDC ACM in terms of "userspace" interactions.
I'll merge the change https://github.com/spiriou/microADB/pull/36 once this PR is in.


uAction causing DelegateDrawer error

I have an uAction happen when a projectile knocks into something. I added the action the first time and it worked as expected.  Restarting the editor didn't fix it. Nor did reinstalling the Vexe unitypackage.
Here are some of the code involved (it depends on so much I didn't want to put it all here):
using UnityEngine;
using System.Collections;
using System.Collections.Generic;
using System;
using UnityEngine.Events;
using Vexe.Runtime.Types;

    public class ProjectileShell : BetterBehaviour
    {
        public uAction<Projectile, Collision2D, CombatCharacter> Collided;

        public void Collide(Projectile projectile, Collision2D collision, CombatCharacter shotBy)
        {
            Collided.Invoke(projectile, collision, shotBy);
            OnCollided(projectile, collision, shotBy);
        }

        protected virtual void OnCollided(Projectile projectile, Collision2D collision,
            CombatCharacter shotBy) { }
    }

    public class SimpleProjectileGenerator : ProjectileGenerator
    {
        [SerializeField] GameObject projectile_;

        public void GenerateFromProjectileCollision(Projectile projectile, Collision2D collision, CombatCharacter shotBy)
        {
            print("generating from collision");

            Projectile p = Instantiate(projectile_).GetComponent<Projectile>();
            p.Init(projectile.transform.position, projectile.transform.rotation, shotBy);
        }
    }

This is really low priority because you made UnityEvent work so I can just use that until this is fixed. If you get stuck trying to reproduce this I could send you the code for my game.

I see what it's causing it to happen (a method is null from one of the handlers). But I don't see why would that happen. Yes send me a small project replicating this please.

I haven't been able to reproduce it on a smal scale yet. I'll work on it.

I found one of the errors. Really simple but just not what I expected. I just have to initialize the uAction like this in the Awake function:
void Awake()
{
    Collided = new uAction<Projectile, Collision2D, CombatCharacter>()
}

Is there any way I could just have the uAction auto initialized so I don't have to worry about it?

You don't need to do it in Awake, in fact you shouldn't do it like that. Cause then it will be reinitialized when you run the game overwriting whatever values you set in the editor. You should check if it's null instead. A better place would be to initialize it immediately when you declare it. If you leave it uninitialized, delegate drawer will initialize it for you but only when the drawer is active. i.e. if you add this script in runtime from code, well, there's no drawers so it will be null.

I can't reproduce the bug since I removed the component and added it again. I must've done something funky. Anyway, I'm happy enough to say that this was probably something strange I was doing rather than a framework problem. I was messing around with different versions of VFW so maybe I had some files from an old version mixed with a new one or something even though I thought I clean reinstalled. I'm happy to close this issue.


Add support to judge whether desired version is available or not

Now no matter what desired version provides, always give info "Helm ${TAG} is available.
Changing from version ${version}". It's obviously wrong.
This patch check whether desired version is actually available or not
by compare desired vesion with all available version in
https://github.com/helm/helm/releases
Signed-off-by: Ma Xinjian<EMAIL_ADDRESS>fix #8647

What this PR does / why we need it:
Special notes for your reviewer:
If applicable:

[ ] this PR contains documentation
[ ] this PR contains unit tests
[ ] this PR has been tested for backwards compatibility

Per policy, this needs an associated issue number. Please create an issue that clearly explains the problem and link it to this PR in the "What this PR does..." field. After that, we can begin the process of reviewing this PR

Needs a rebase before we can merge this. Otherwise LGTM.

ping？

thanks for the ping.

Hi, this is causing issues with our workflow - it seems like it doesn't take into account pagination of the releases page on Github. It's just reading the first page therefore when specifying an older version not on the first page it exits with "Helm $DESIRED_VERSION is unavailable"

reverted in https://github.com/helm/helm/pull/9056.

Hi, this is causing issues with our workflow - it seems like it doesn't take into account pagination of the releases page on Github. It's just reading the first page therefore when specifying an older version not on the first page it exits with "Helm $DESIRED_VERSION is unavailable"
Issue described in #9055

Sorry for the trouble.
I found someone others also met this issue. They solved by add ?per_page=100 to the end of url. I have tried but unfortunately it didn't work with our url.......


question: day_of_year why not return u16?

#[inline]
    pub fn day_of_year(self) -> i16 {
        type DayOfYear = ri16<1, 366>;

        let days = C(1) + self.since_days_ranged(self.first_of_year());
        DayOfYear::rfrom(days).get()
    }

Maybe other function can also return unsigned number.

Duplicate of #35, #102.


Factor out algorithms that have complicated dependencies

Gonna try moving those to their own packages, and make an umbrella package to pull in everything.

Hi All, after speaking with @sschmidt23 and following up on discussions at the collaboration meeting I’m going to propose some restructuring of rail in order to better manage the dependencies.

We’d like to split out three of the estimators that bring in more dependencies into their own packages.  This will make it easier for people developing the various degraders.   Specifically we would like to have rail_delight , rail_bpz, and rail_flexzboost , each of which just has the relevant estimator and test code.
We’d like to add an umbrella package to collect all of the various packages, for the uses who do want everything.   This would be rail_top
We’d like to add two packages to collect various analysis pipelines to make it easier to run productions jobs at NERSC, these would be rail_pipelines and rail_pipe_configs.  This first would have python code and notebeooks to run the pipelines iteractively or in notebooks, i.e., for development.  The second would have production configurations to run rail at NERSC under ceci.
Making this work will require a few changes to the RAIL package.   The most important is that rail would become a python namespace instead of a module.   What that means is that import rail wouldn’t really do much (right now it pulls in pretty much everything).
I'll make the new packages follow a more modern python package structure.  I.e., using pyproject.toml and having the code live in src/rail/<whatever>

One knock-on effect is that using rail.__file__ to find various test data wont work anymore.   Instead, I've added a variable in rail.core.utils called RAILDIR.   I.e., you can do:

from rail.core.utils import RAILDIR
datafile = os.path.join(RAILDIR, 'examples', 'some_data.fits')

I'll fix that anywhere I find it in the test scripts and example notebooks.

Things to add to install docs:

you need to conda install h5py alongside RAIL. The best way to do this is to just install it when creating the environment:, e.g. conda create -n rail -c conda-forge -y python=3.10 h5py
A list of each of the repos and what they're for
I assume there won't be a required install order once everything is on pypi/conda-forge? But if there still is, we should note this
a description of how this looks from the user side of things. Just something like "when you install these other packages, you access them like you access anything else in RAIL from rail.something.something import .... Otherwise, people might think you have to do stuff like from rail_pipelines import ...

Done in #233


feat: multi-select w/ shift

resolves https://github.com/transmissions11/flux/issues/17

@h0tw4t3r this should be what you what you need, tho sadly as with many things in react flow its a little inconsistent (i believe of cuz of this https://github.com/wbkd/react-flow/issues/2826)


fix: data-viewer-theme attribute would leak at top level document

I noticed that the UserSettings will apply the data-viewer-theme attributes on the top level document. When reusing the window after the reader was used, that attributes remains, but it should not.
this PR will make use of a #root element if exists instead, and else keep the existing behaviour for backward compatibility

@gotson i haven't tried this yet, but wouldn't this mean i have to have an element with id #root otherwise all of it will just be ignored?

@gotson i haven't tried this yet, but wouldn't this mean i have to have an element with id #root otherwise all of it will just be ignored?

never mind i see what you are doing, will have it reviewed shortly

i still question though when would there be an element with id root?

it needs to be set by the application using the library. Setting the root element allows for controlled application of the theme onto other elements.
I do this in Komga by setting the top level component of that specific view as root, so the theme is applied on the whole view, not just the reader.

@gotson ok, i'll accept and merge this PR. since it doesn't break any backwards compatibility,


Missing dash license tool argument

The PR #88 includes a 'dashLicenseCheck' task. It is missing an argument as per TRG 7.04: '-project automotive.tractusx'. This must be added for the task to be compliant.

Resolved by #106


Removed MS Defender for EndPoint record types as per Insights workitem

https://office.visualstudio.com/MAX/_workitems/edit/5789721

Docs Build status updates of commit bdf547a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

office-365-management-api/office-365-management-activity-api-schema.md
:bulb:Suggestion
View
Details

office-365-management-api/office-365-management-activity-api-schema.md

Line 273, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 503, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 531, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 605, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 674, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 692, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 708, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 718, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 768, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 1220, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 1287, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.
Line 1326, Column 2: [Suggestion: column-header-missing - See documentation] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.

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 the docs.microsoft.com contributor guidesPost your question in the Docs support channel


'google/protobuf/port_def.inc' file not found

I used tulsi generate a HandTrackingLibray,  and embedded this lib to my Xcode project. when i common + B  to build , the Xcode tips  'google/protobuf/port_def.inc' file not found in 'calculator.pb.h'.  how to fix this program?

456CGD


build error and runtime error

Device: NVIDIA GeForce RTX 4090 D,
Cuda compilation tools, release 12.2, V12.2.91
Build cuda_12.2.r12.2/compiler.32965470_0
gcc version 13.2.0 (Ubuntu 13.2.0-23ubuntu4)
torch                             2.3.0
Python                          3.10
In the execution of the example code where f = host.predict_async(prompt_tokens, 32), an error occurred: RuntimeError: CUDA error: CUBLAS_STATUS_NOT_SUPPORTED when calling cublasGemmEx(handle, opa, opb, m, n, k, &falpha, a, CUDA_R_16F, lda, b, CUDA_R_16F, ldb, &fbeta, c, CUDA_R_16F, ldc, CUDA_R_32F, CUBLAS_GEMM_DEFAULT_TENSOR_OP)`

I should have found the reason, it should be that I am a non-root user of the server, and there is a higher version of gcc on the root of the server, but it does not match my cuda toolkit, which may cause the error, but I have configured a lower version of gcc under my account and set the environment variables, and my own version of gcc is still not recognized when the project is build, and I will discuss it when I solve this error.

I should have found the reason, it should be that I am a non-root user of the server, and there is a higher version of gcc on the root of the server, but it does not match my cuda toolkit, which may cause the error, but I have configured a lower version of gcc under my account and set the environment variables, and my own version of gcc is still not recognized when the project is build, and I will discuss it when I solve this error.

Sorry for the late reply. Many dependent packages have released new versions since we published the paper and code. So compliation errors are common. Recently we verified the build again with new pacakge versions. Please follow the guide to compile and test.
https://github.com/microsoft/chunk-attention#gpu-kernel-blascuda


OpenCubes C++ implementation cleanup.

Hello!
I did some work on top of nsch0e/opencubes/cpp branch:
I don't intend this to be merged as is,
and I will likely make better branch/version of this cleanup.
I wanted to point out how the single core performance can be massively improved,
and I didn't like to keep the changes to my self.
Fix multitude of problems and code smells:

Cleanup the "using namespace std;" fallout. I don't know where you got the idea for this... 90% of this diff is fixing up this mess.
Add proper preprocessor if/endif guards to headers.
Fix-up the hasher functors mess, no need to put it into namespace std
Replace unordered_set with CubeSet typedef as it is used in few places.
This also solves the hasher namespacing.
"return std::move(thing);" is never correct, this prevents the copy-ellision.

Then the code improvements:

Fixups to code where Cube can be moved instead of copied with std::move() large speed up
Pre-allocate memory in few critical places.
Add begin(),end(),size(),reserve(),emplace_back() for Cube This simplifies the code everywhere else.
Make Rotations::rotate() just return copy of Cube, and shuffle it around with std::move().

I thank you publishing C++ code for OpenCubes. :-)
This commit is just bashed together in one evening.
N=12  From ~12000 to ~15000 baseCubes/s with single core on my machine.
PS: If you can verify I didn't break the output, that would be good.

I made less crusty pull request: https://github.com/nsch0e/opencubes/pull/5 that replaces this.


Avoid pushing fybrik-template chart to public registry.

With the recent move of the implicit copy charts to the  data-movement-operator repo only the fybrik-template chart is left as a chart we push into the public registry. As this chart is used for testing there is no point in pushing it to the public registry thus this PR removes the use of helm-push-public.

/lgtm

@revit13 Did you replace all occurrences of ghcr.io/fybrik/fybrik-template:0.1.0 in the test yamls?


fix: deprecate an obsolete trie_tree.

Description of Change
This file does not follow any good coding practices and if we were to resolve it, we would end up with a version very similar to what has already been implemented in trie_modern.cpp. Moreover, it causes the only LGTM alert from the entire repo:
https://lgtm.com/projects/g/TheAlgorithms/C-Plus-Plus/alerts/?mode=list
Checklist

[X] Added description of change
[X] Added file name matches File name guidelines
[X] Added tests and example, test must pass
[X] Added documentation so that the program is self-explanatory and educational - Doxygen guidelines
[X] Relevant documentation/comments is changed or added
[X] PR title follows semantic commit guidelines
[X] Search previous suggestions before making a new one, as yours may be a duplicate.
[X] I acknowledge that all my contributions will be made under the project's license.

Notes:

@Anmol3299 This will lead to removal of your contribution. Please review.

@fhlasek See #1059


Add Windows test-run

#354
#351

https://github.com/fredrikhgrelland/vagrant-hashistack/pull/351#issuecomment-698208516

Virtualization is not enabled on windows virtual machines that github provides so currently it isn't possible to run the tests on windowsIs there a plan to enable virtualization on the windows-2019 image?
cd template_example; SSL_CERT_FILE= CURL_CA_BUNDLE= vagrant up --provision
Bringing machine 'default' up with 'virtualbox' provider...
==> default: Box 'fredrikhgrelland/hashistack' could not be found.  Once the master VM is prepared,
    default: it will be used as a base for linked clones, making the creation
    default: of new VMs take milliseconds on a modern system.
==> default: Importing base box 'fredrikhgrelland/hashistack'...

Progress: 10%
Progress: 20%
Progress: 40%
Progress: 50%
Progress: 60%
Progress: 90%
==> default: Cloning VM...

==> default: Matching MAC address for NAT networking...
==> default: Checking if box 'fredrikhgrelland/hashistack' version '0.4.3' is up to date...
==> default: Setting the name of the VM: template_example_default_1600964656119_57263
Vagrant is currently configured to create VirtualBox synced folders with
the `SharedFoldersEnableSymlinksCreate` option enabled. If the Vagrant
guest is not trusted, you may want to disable this option. For more
information on this option, please refer to the VirtualBox manual:

  https://www.virtualbox.org/manual/ch04.html#sharedfolders

This option can be disabled globally with an environment variable:

  VAGRANT_DISABLE_VBOXSYMLINKCREATE=1

or on a per folder basis within the Vagrantfile:

  config.vm.synced_folder '/host/path', '/guest/path', SharedFoldersEnableSymlinksCreate: false
==> default: Clearing any previously set network interfaces...
==> default: Preparing network interfaces based on configuration...
    default: Adapter 1: nat
    default: Adapter 2: hostonly
==> default: Forwarding ports...
    default: 8500 (guest) => 8500 (host) (adapter 1)
    default: 4646 (guest) => 4646 (host) (adapter 1)
    default: 8200 (guest) => 8200 (host) (adapter 1)
    default: 22 (guest) => 2222 (host) (adapter 1)
==> default: Running 'pre-boot' VM customizations...
==> default: Booting VM...
There was an error while executing `VBoxManage`, a CLI used by Vagrant
for controlling VirtualBox. The command and stderr is shown below.
mingw32-make[1]: Leaving directory 'D:/a/vagrant-hashistack/vagrant-hashistack/template'

Command: ["startvm", "a28a62c3-39c1-4de5-9426-6b4ee423dacc", "--type", "headless"]

Stderr: VBoxManage.exe: error: WHvCapabilityCodeHypervisorPresent is FALSE! Make sure you have enabled the 'Windows Hypervisor Platform' feature. (VERR_NEM_NOT_AVAILABLE).

VBoxManage.exe: error: VT-x is not available (VERR_VMX_NO_VMX)

VBoxManage.exe: error: Details: code E_FAIL (0x80004005), component ConsoleWrap, interface IConsole

mingw32-make[1]: *** [Makefile:46: template_example] Error 1
mingw32-make: *** [Makefile:21: wintest] Error 2
Error: Process completed with exit code 1.

https://github.com/Neha-Sinha2305/vagrant-hashistack/runs/1161512513?check_suite_focus=true

Closing, not planning to enable virtualization https://github.com/actions/virtual-environments/issues/1481#issuecomment-681898396


Mistake detected in void Styles::addDxfFormat

On line 301 and 311, you should write m_dxf_formatsHash instead of m_xf_formatsHash to avoid a crash.
Thanks for your xlsx library !

Dear @akkauser
Thanks for reporting.
I have reflected that code. I think we need more tests.


fix: add scrollable area check and scrollable area attribute

fixes part of #2733
Adds a check to see if html or body is scrollable.
Otherwise, it looks for an element with the attribute data-sveltekit-main-scrollable to determine which element to scroll during navigation.
Looking for feedback as this solution still feels quite rough. @dummdidumm
TODO

[ ] add tests
[ ] docs
[ ] changeset

Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[x] It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
[x] This message body should clearly illustrate what problems it solves.
[ ] Ideally, include a test that fails without this PR but passes with it.

Tests

[x] Run the tests with pnpm test and lint the project with pnpm lint and pnpm check

Changesets

[ ] If your PR makes a change that should be noted in one or more packages' changelogs, generate a changeset by running pnpm changeset and following the prompts. Changesets that add features should be minor and those that fix bugs should be patch. Please prefix changeset messages with feat:, fix:, or chore:.

The idea is tempting, although I'm wondering how much code people really save by doing that. I'm also wondering if they would always prefer the explicitly set scrollable area to be used for scrolling. Also, how often will be html element be non-scrollable while the body will be? Feels like both won't have that if someone does that.

I agree. The thread showed that many people were able to solve it with their own workarounds.
It might be enough to document the behaviour and/or provide a warning when html isn't scrollable.
The common patterns I've gathered from the issue thread are:

html is not scrollable and its height is set to the window height.
body is the scrolling area.
html and/or body are not scrollable and both have height set to window height.
div or main is the scrolling area. The element location varies.

I think warning + documenting is the most important thing - it is very non-obvious that this will happen, mainly because as a user you're not thinking about how the magic happens.

The thread showed that many people were able to solve it with their own workarounds.

But I really like the idea of providing a proper solution - as a mortal Sveltekit user, I don't feel like I understand the workaround we have done enough to know that I've done it right. Like, I added the check for when navigation.type === "link"... because otherwise goto didn't work properly.... I want to be able to just say, "hey Sveltekit, treat this as the scrollable area" and have it work in the same way as it does for the html element.
Please let me know if I can help testing this - and if so, how. 🙏

But I really like the idea of providing a proper solution - as a mortal Sveltekit user, I don't feel like I understand the workaround we have done enough to know that I've done it right. Like, I added the check for when navigation.type === "link"... because otherwise goto didn't work properly.... I want to be able to just say, "hey Sveltekit, treat this as the scrollable area" and have it work in the same way as it does for the html element.

In this case, it would be preferable to always require the explicit scrolling area set. I hesitated earlier about this when I recalled a conversation about the issues of having too many framework specific attributes.

Please let me know if I can help testing this - and if so, how. 🙏

I had the idea to add tests for the new attribute following the procedure:

Scroll down.
Click a link.
Check scroll position of the main scrolling element.

However, I'm waiting for more feedback from everyone before continuing this PR.

I wonder if #8710 is an adequate solution to this, if properly documented? It's a tiny bit more work than adding an attribute, but it uses existing API surface area and teaches people a technique that they can then generalise to other problems.

It would also need to address the issue of correctly scrolling to the top when navigating to a new page.
My current understanding of snapshots is that it only restores scroll when navigating backwards / forwards to visited pages.

It also doesn't stop the mistake being made, I don't think? We only discovered it late in a cycle when testing on real mobile devices, so the buggy scrolling went into production. So we need something like this DEV check / warn.
As an aside, I'm not sure I've seen an explanation here of what the relevant difference is in mobile browsers that causes this behaviour to manifest (or - rather - not manifest reliably desktop). 🙏

@Rich-Harris said:

Snapshots are restored immediately after afterNavigate callbacks

Unfortunately I'm seeing afterNavigate being called after snapshot restore, so this doesn't work.

I was very confused that this was not in the sveltekit docs.


New order email is missing

Expected behavior
After you placed an new order you should receive an new order e-mail to your mail that was assigned in the module 'Mail alerts'
Actual behavior
Only the payment accepted mails are send to the customer but the new order e-mail is missing.
Steps to reproduce the behavior
Place a new order and you should receive an new order mail.
Environment
Prestashop <IP_ADDRESS>
Mollie 4..4.2
PHP-version: 7.3.31

Hello,
As there are duplicates of this ticket, we will keep the information regarding email case in the ticket - #444
Closing this one.
--
Best regards,
Invertus Support team

Hi @Maikel726 @pantoffelke!
The issue has been resolved in the new release, which is out now, and can be downloaded here - https://github.com/mollie/PrestaShop/releases/tag/4.4.3
Let us know if you have any other questions, and any feedback regarding the release!
--
Best regards,
Invertus Support team


Mb 14733 customer can see closeout office on ppm shipment review card

Jira ticket for this change
Summary
Is there anything you would like reviewers to give additional scrutiny?
this article explains more about the approach used.
Setup to Run Your Code

💻 You will need to use three separate terminals to test this locally.
Terminal 1
Start the Storybook locally.
make storybook

Terminal 2
Start the UI locally.
make client_run

Terminal 3
Start the Go server locally.
make server_run

Additional steps

Access the
Login as a

Verification Steps for Author
These are to be checked by the author.

[ ] Tested in the Experimental environment (for changes to containers, app startup, or connection to data stores)
[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

Verification Steps for Reviewers
These are to be checked by a reviewer.

Frontend

[ ] User facing changes have been reviewed by design.
[ ] There are no aXe warnings for UI.
[ ] This works in Supported Browsers and their phone views (Chrome, Firefox, IE, Edge).
[ ] There are no new console errors in the browser devtools
[ ] There are no new console errors in the test output
[ ] If this PR adds a new component to Storybook, it ensures the component is fully responsive, OR if it is intentionally not, a wrapping div using the officeApp class or custom min-width styling is used to hide any states the would not be visible to the user.

Backend

[ ] Code follows the guidelines for Logging
[ ] The requirements listed in Querying the Database Safely have been satisfied.

Database
Any new migrations/schema changes:

[ ] Follows our guidelines for Zero-Downtime Deploys
[ ] Have been communicated to #g-database
[ ] Secure migrations have been tested following the instructions in our docs

Screenshots

Warnings

:warning:
Please add the JIRA issue key to the PR title (e.g. MB-123)

:warning:
This PR does not include changes to unit tests, even though it affects app code.

  Generated by :no_entry_sign: dangerJS against fd328be701f805c0d7835a38a5a9d55baa596e47

Took a look and I'm not seeing the Closeout office in the shipment card on the review page when the customer enters an Army or Air Force move.
This is working as expected for Coast Guard (no Closeout office listed on review page).

Could you check these accounts to see if the closeout office field populates<EMAIL_ADDRESS>and<EMAIL_ADDRESS>

After a long pair session with Patrick, we figured out Closeout_Office is dropped from the API payload between the review and submission of PPMShipments (lines 96-98). Further action will need to be taken to ensure the closeout_office is included in the API response after submission of  PPMShipments.

After a long pair session with Patrick, we figured out Closeout_Office is dropped from the API payload between the review and submission of PPMShipments (lines 96-98). Further action will need to be taken to ensure the closeout_office is included in the API response after submission of PPMShipments.

That's a great catch, sorry that's a fetcher I missed but does show the value of our Cypress tests.  I know you're OOO today, I could quickly add in the fetcher work seems like an easy eager loading tweak.


how to detect my own picture

this project can not detect my own picture and my  custum class, could you release the code ?meanshile, it cannot train your fsod data.

I am fighting for CVPR... After the DDL, I will add this feature. If you can not wait, you can reference the detectron2 tutorials to  detect your picture: https://github.com/facebookresearch/detectron2/blob/master/demo/README.md


[color theme][chart] Series in the chart is starting in a random order of color

Series in chart is starting in a random order of color
How to reproduce the bug

Go to explore, make a gauge chart with multiple series
Click on color theme and observe the color number
See error

Expected results
Series will follow the color order
Actual results
Series seems appear in random color order
Screenshots

Environment
(please complete the following information):

browser type and version: Chrome
superset version: master
python version: python --version
node.js version: node -v
any feature flags active:

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[ ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
Add any other context about the problem here.

This issue seems exist in other chart types
for example:

Better to check all Popular charts and Echarts


Sample nonfunctional for all pytorch frameworks

Compilation with 1,6.0 works, but deployment fails, no model.pth file created in the sample dir /folder structure anticipated by Huggingface 2.3.0

I have found a series of fixes. I will update and merge changes

@aarsanjani update here on the fixes specifically related to deploying the endpoint?


Fix display of default user svg icon in Messages

Description of what you did:

My PR is a:

[x] 🐛 Bug fix
[ ] 💥 Breaking change
[ ] 💅 Enhancement
[ ] 🚀 New feature

Main update on the:

[ ] api
[x] frontend
[ ] lib

Hi @leventdeniz thank you for contributing. I can't merge this PR, because renderUserImage is duplicated in MessagesChat and MessageDetailHeading components. Since we are using user's image with this functionality in multiple places, ( Header, Sidebar, Messages ... ) I think it will be better if we'll create one reusable component that will receive image size as a prop and then we'll use that one component in all places.


Wierd AppBar title bug

Hello, I have a project in Flutter.
return MaterialApp(       title: "Simple To-Do List",       home: Scaffold(           appBar: AppBar(title: Text("Simple To-Do List")),       ),     );
This code will work fine, but in the active app display menu, on the phone virtual phone (Pixel 2API 24) it will display a random "." in the app list, under the name.
Also, if I replace the word "Simple" from the AppBar Title Text, it will work fine.
The image with the code from the top:

The image with the first word modified (no dot "."):

Why that "." randomly appears? I think this is a bug.

@CommanderPaladin
Can you provide flutter doctor -v ? Do you see the same behavior on physical device too ?
Can you try below properly formatted code and see if it works ?
return MaterialApp(
      title: "Simple To-Do List",
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: Scaffold(
        appBar: AppBar(title: Text("Simple To-Do List"),),

@CommanderPaladin
Can you provide flutter doctor -v ? Do you see the same behavior on physical device too ?
Can you try below properly formatted code and see if it works ?
return MaterialApp(
      title: "Simple To-Do List",
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: Scaffold(
        appBar: AppBar(title: Text("Simple To-Do List"),),

Even with this, is still the same, on a new raw app.
I haven't test it on a physical device yet, meanwhile
Here is my Flutter Doctor:
`C:\Users\Razvan>flutter doctor -v
[√] Flutter (Channel stable, 2.2.3, on Microsoft Windows [Version 10.0.19043.1165], locale en-US)
• Flutter version 2.2.3 at C:\Users\Razvan\Desktop\flutter
• Framework revision f4abaa0735 (9 weeks ago), 2021-07-01 12:46:11 -0700
• Engine revision 241c87ad80
• Dart version 2.13.4
[√] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
• Android SDK at C:\Users\Razvan\AppData\Local\Android\Sdk
• Platform android-31, build-tools 31.0.0
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6842174)
• All Android licenses accepted.
[X] Chrome - develop for the web (Cannot find Chrome executable at .\Google\Chrome\Application\chrome.exe)
! Cannot find Chrome. Try setting CHROME_EXECUTABLE to a Chrome executable.
[√] Android Studio
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin can be installed from:
https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
https://plugins.jetbrains.com/plugin/6351-dart
• android-studio-dir = C:\Program Files\Android\Android Studio
• Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6842174)
[√] VS Code, 64-bit edition (version 1.57.1)
• VS Code at C:\Program Files\Microsoft VS Code
• Flutter extension can be installed from:
https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter
[√] Connected device (2 available)
• Android SDK built for x86 (mobile) • emulator-5554 • android-x86    • Android 7.0 (API 24) (emulator)
• Edge (web)                         • edge          • web-javascript • Microsoft Edge 92.0.902.84
! Doctor found issues in 1 category.`

This seems to be specific to Pixel emulator or device, as I see the same using Pixel 4 emulator.

But don't see the same behavior on physical Android Samsung device.
Keeping it open for further insights.

 master flutter doctor -v

C:\Users\Win>flutter doctor -v
Could not load custom device from config index 0: Expected enabled to be a boolean.
[√] Flutter (Channel master, 2.6.0-1.0.pre.136, on Microsoft Windows [Version 10.0.19042.1165], locale en-US)
    • Flutter version 2.6.0-1.0.pre.136 at D:\FlutterSDK\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 96f977f4df (2 hours ago), 2021-08-31 02:16:01 -0400
    • Engine revision fda0ac4520
    • Dart version 2.15.0 (build 2.15.0-66.0.dev)

[√] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at C:\Users\Win\AppData\Local\Android\sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files (x86)\Google\Chrome\Application\chrome.exe

[X] Visual Studio - develop for Windows
    X Visual Studio not installed; this is necessary for Windows development.
      Download at https://visualstudio.microsoft.com/downloads/.
      Please install the "Desktop development with C++" workload, including all of its default components

[√] Android Studio (version 2020.3)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[√] Connected device (4 available)
    • Android SDK built for x86 (mobile) • emulator-5554 • android-x86    • Android 6.0 (API 23) (emulator)
    • Windows (desktop)                  • windows       • windows-x64    • Microsoft Windows [Version 10.0.19042.1165]
    • Chrome (web)                       • chrome        • web-javascript • Google Chrome 92.0.4515.159
    • Edge (web)                         • edge          • web-javascript • Microsoft Edge 92.0.902.84

! Doctor found issues in 1 category.

C:\Users\Win>

Isn't that just the leg of the p showing? seems like a portion of the appbar is showing but not enough to show the whole text, might be resolution/dpi dependant

Indeed. Increasing the font size of the text, we can get this:


is there a way for an error to not stop redun from running non-dependent tasks?

e.g.
pipeline is
task1->task2->task3
task1->task4->task5
If task2 fails and returns an error I still want redun to run task5 if task4 was successful

You could look into using catch() for catching an error and returning a intermediate value that allows the workflow to continue. See https://github.com/insitro/redun/blob/367033e87ad17ad8a1bdb09bec472f846ffe4b4e/examples/07_control_flow/workflow.py#L36-L42
So your example would look something like:
@task
def task1():
    x = catch(task2(task3(x)), ValueError, None)
    y = task4(task5(y))
    return [x, y]

@mattrasmus Great! Thanks!


Walk fix

This fixes the walker to use the latest version of ESTree.
I went through the whole ESTree spec, including experimental features.
I added the following types: SwitchCase, ClassBody, Decorator  (requires acorn-es7 plugin), TemplateElement.
I added the following abstract types: Declaration, ModuleSpecifier, ModuleDeclaration, ExportSpecifier, AssignmentProperty.
I fixed the following discrepancies with the spec:

SpreadElement and Super are not instances of Expression
Identifier and MemberExpression are instances of Expression
ObjectPattern.properties are instances of Property not Pattern
Class.id and Function.id is an Identifier not a Pattern
ForOfStatement and ForInStatement left can be a Pattern or a VariableDeclaration, not an Expression
AwaitExpression.argument is not optional
VariablePattern and MemberPattern do not exist

Thanks for working on this! I added a comment for a small issue (not using ES5+ feature). And though we don't have a test suite for the walker, I've been using the Tern tests as a crude smoke test, since Tern intensively uses the walk. When I use a version of Acorn with these changes, a lot of tests start failing, suggesting something about the behavior of the walker was changed in an incompatible way. Could you look into that?

I fixed Array.includes().
I tried to clone Tern, using modified version of acorn. I ran the test and got: Ran 598 tests from 101 files..
Maybe fixing Array.includes() solved the problem?

Are you sure you built Acorn before running the tests? I'm getting 61 failures with your code.

I managed to reproduce the 61 failures you are getting.
However, I am having issues understanding what's going on, for two reasons:

understanding the tests requires understanding Tern, which I am unfamiliar with.
the tests use a quite custom setup and set of helpers which makes it hard for me to get a grasp of what's failing.

Closing this due to inactivity. (Feel free to reopen to continue discussion.)

@marijnh inactive because I was waiting for your reply :)


http2 cleartext support

This PR is in reference to this thread in the tus-js-client repository issues. @Acconut invited us to create a lightweight PR to solve this issue, on the tusd side.
PROBLEM
In GCP Cloud Run, HTTP/2 traffic is accepted but TLS is terminated at the load balancer and traffic is forwarded to the tusd container as cleartext (non-encrypted) http. This is problematic because tusd expects TLS for HTTP/2 traffic.
SOLUTION
Golang's net/http2 library  allows for a drop in support of HTTP/2 in cleartext (h2c), with simultaneous capability for HTTP/1.1 cleartext. Meaning, if traffic is identified to be HTTP/1.1, the h2c handler will bypass HTTP/2 and serve based on HTTP/1.1.
This PR uses the newHandler() method to wrap the existing mux handler, such that no additional configuration is needed  to handle either HTTP version. HTTP/2 encrypted traffic can also be handled as usual by making calls directly to tusd via https. If behind a load balancer such as Cloud Run, all that would be required in terms of configuration is the -behind-proxy flag.
TESTING
We built an image based on this PR and tested it for file uploads on Cloud Run with tus-js-client and it performed successfully.

Thank you for this PR! The use of h2c on its own looks good, but I would prefer if it sits behind a feature flag and is not enabled by default. The caveat about memory consumption from https://github.com/tus/tus-js-client/issues/700#issuecomment-2208270992 is concerning enough to let user opt into this behavior. I can imagine that in the future we have multiple flags to enable different HTTP versions, e.g. --enable-h2c and --enable-h3.

Thanks @Acconut good point - pushed a new commit. Let me know if it addresses your concern.

Thanks for the updates! I will try them out on my own in about two weeks

Merged in https://github.com/tus/tusd/commit/dfaf9f76332b477fa4be87748315bab61f1b15fd, thank you very much for this!

This is released in https://github.com/tus/tusd/releases/tag/v2.5.0.


ENH: Add scheduler_cycles

Codecov Report
Base: 95.50% // Head: 95.51% // Increases project coverage by +0.00% :tada:

Coverage data is based on head (8afb843) compared to base (9e3539b).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #106   +/-   ##
=======================================
  Coverage   95.50%   95.51%
=======================================
  Files          88       88
  Lines        4604     4612    +8
=======================================
+ Hits         4397     4405    +8
  Misses        207      207

Impacted Files
Coverage Δ

rocketry/conditions/scheduler.py
100.00% <ø> (ø)

rocketry/conds/__init__.py
100.00% <ø> (ø)

rocketry/conditions/api.py
98.68% <100.00%> (+0.06%)
:arrow_up:

rocketry/core/task.py
93.34% <0.00%> (-0.03%)
:arrow_down:

rocketry/conditions/func.py
89.58% <0.00%> (ø)

rocketry/core/time/anchor.py
92.43% <0.00%> (+0.12%)
:arrow_up:

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


EIT: max repetition time of P/F sections

Hi @lelegard ,
Here's another suggestion based on incorrect table repetition values. (I'm sorry, if I'm very insistent with this kind of problems, but even if they can't be solved I prefer to comment on their existence.)
After testing some outputs from TSDuck with a TR 101 290 Analyser, I observed that one Priority 3 error appears. The error is related to the use of the psimerge plugin. And the problem is quite clear: the repetition rate of the P/F tables (Sections with Table ID 0x4E in the PID 0x12) in the composed EIT is not repeated with a maximum interval of 2 seconds.
So, I suggest to add the parameter --enforce-repeat-pf to the psimerge plugin to overcome this problem and generate conformant MPTS.
You agree?
Regards.

Hi @lelegard ,
Using the tsanalyze tool I get:
|--------------------------------------------------------------------------------------------|
|  0x4E (78, EIT p/f Actual), TID ext: 0x0064 (100)                                          |
|         Repetition: ........ 6146  ms  Section cnt: .......... 220                         |
|         Min repet.: ........ 1540  ms  Table cnt: ............ 102                         |
|         Max repet.: ....... 27174  ms  Version: ............... 10                         |
|--------------------------------------------------------------------------------------------|
|  0x4E (78, EIT p/f Actual), TID ext: 0x0065 (101)                                          |
|         Repetition: ........ 7199  ms  Section cnt: .......... 217                         |
|         Min repet.: ........ 1390  ms  Table cnt: ............. 86                         |
|         Max repet.: ....... 32437  ms  Version: ................ 7                         |
|--------------------------------------------------------------------------------------------|
|  0x4E (78, EIT p/f Actual), TID ext: 0x0066 (102)                                          |
|         Repetition: ........ 6725  ms  Section cnt: .......... 162                         |
|         Min repet.: ........ 1469  ms  Table cnt: ............. 92                         |
|         Max repet.: ....... 26982  ms  Version: ................ 3                         |
|--------------------------------------------------------------------------------------------|
|  0x4E (78, EIT p/f Actual), TID ext: 0x0067 (103)                                          |
|         Repetition: ........ 5595  ms  Section cnt: .......... 177                         |
|         Min repet.: ........ 1555  ms  Table cnt: ............ 112                         |
|         Max repet.: ....... 35910  ms  Version: ............... 18                         |
|--------------------------------------------------------------------------------------------|
|  0x4E (78, EIT p/f Actual), TID ext: 0x0068 (104)                                          |
|         Repetition: ........ 3429  ms  Section cnt: .......... 434  First version: .... 7  |
|         Min repet.: ........ 1033  ms  Table cnt: ............ 181  Last version: ..... 9  |
|         Max repet.: ....... 25100  ms  Versions: ......... 7, 8, 9                         |
|--------------------------------------------------------------------------------------------|
|  0x4E (78, EIT p/f Actual), TID ext: 0x0069 (105)                                          |
|         Repetition: ........ 2656  ms  Section cnt: .......... 469                         |
|         Min repet.: ........ 1424  ms  Table cnt: ............ 236                         |
|         Max repet.: ....... 10616  ms  Version: ............... 23                         |
|--------------------------------------------------------------------------------------------|
|  0x4E (78, EIT p/f Actual), TID ext: 0x006A (106)                                          |
|         Repetition: ........ 3651  ms  Section cnt: .......... 395  First version: .... 7  |
|         Min repet.: ........ 1127  ms  Table cnt: ............ 171  Last version: ..... 9  |
|         Max repet.: ....... 41169  ms  Versions: ......... 7, 8, 9                         |
|--------------------------------------------------------------------------------------------|
|  0x4E (78, EIT p/f Actual), TID ext: 0x006B (107)                                          |
|         Repetition: ........ 2565  ms  Section cnt: .......... 512                         |
|         Min repet.: ........ 1441  ms  Table cnt: ............ 245                         |
|         Max repet.: ....... 10867  ms  Version: ............... 18                         |
|--------------------------------------------------------------------------------------------|

And you can see that the "Max repet" is far from the repetition target. So I want to found a solution.
With other PSI/SI tables I can smooth them with a two-pass process based on your suggestion: first with -P tables I save in a file the binary representation of the table. After in a second-pass I do -P filter -p 100 -n -s for the corresponding PID and a -P inject -p 100 --binary file.pmt.100=450 to reinject at the correct interval. And it works. Thank you for the tip!
However, with the EIT I can't do that.
Any idea about how to smooth or reinject some specific sections (the target is 0x4E) with explicit interval?
For example, I'm thinking on something like -P smooth -p 18 --section 0x4E=1500.
What you think?
I really appreciate your ideas.
Thank you!

Hi @lelegard ,
I need your help to accomplish this task. I feel that you know a method to achieve it. Please help me.
I discovered that I can extract all parts of the EIT tables individually:
tsp ... -P tables --all-once --include-next -p 18 -m -b eit ...

So, I can obtain the files eit_p0012_t4E_e####_v##_s## and eit_p0012_t50_e####_v##_s##. Then it will be possible to reinject these tables with different intervals. For the schedule tables (t50) I don't care, but for the P/F data (t4E) I want to repeat each one with a 1500ms interval.
What would be the best way to do that?
Thank you very much (as usual)!

Hi @lelegard ,
Please, can you explain why the psimerge implementation uses a CyclingPacketizer class for PAT, CAT, NIT and SDT/BAT, but only a Packetizer class for the EIT?
And another question: It will be useful, from your point of view, to include in the psimerge plugin the option to set the repetition interval of the different tables, instead of use the default values everytime?
This will solve the problem explained at #370. As using the psimerge plugin with the main tables and a fake (aka empty) merge label, all tables will be recalculated using the new repetition interval. The trick is to mark the entire stream with the --main-label and use an empty label as --merge-label.
Please, comment it.
Thank you!

Hi @lelegard ,
I started to see that the plugin sections can also be used to fine-grade processing the EIT tables. So, I request if it's possible to include the option to control the repetition interval inside this plugin. Perhaps using the CyclingPacketizer instead of the simple Packetizer class.
You think this makes sense?
Please, comment it.
Thank you!


Contraint Layout Problem

Whenever i try to add a widget or a text box etc on the screen the following error occurs
This view is not constrained. It only has designtime positions, so it will jump to (0,0) at runtime unless you add the constraints  The layout editor allows you to place widgets anywhere on the canvas, and it records the current position with designtime attributes (such as layout_editor_absoluteX). These attributes are not applied at runtime, so if you push your layout on a device, the widgets may appear in a different location than shown in the editor. To fix this, make sure a widget has both horizontal and vertical constraints by dragging from the edge connections.

This sounds like an Android-specific issue, and not related to HAXM virtualization. Unless your issue only occurs with HAXM and not with Linux+KVM, this issue should be closed.
For Android SDK related issues, please refer to https://developer.android.com/support or ask any questions you might have on Stack Overflow.

@AlexAltea Thanks for helping out :-)
@kaushik199501 I hope you understand that this is not an Android Studio support forum. HAXM is a driver that makes x86/x86_64 AVDs run faster, but it can't be responsible for all Android Emulator (not to mention Android Studio IDE) issues. Please take a moment to read Google's official instructions on how to report an Android Studio or Emulator issue:
https://source.android.com/setup/contribute/report-bugs#developer-tools
https://developer.android.com/studio/report-bug


desktop sign up frame

created the desktop sign up frame, added the color style, added input fields, button, and all the necessary content.

https://www.figma.com/file/LEKdq0JqyuzzOGxv0tZWjh/WalletFriendly?node-id=1245%3A2782


Migration helper for secrets

Fixes issues with related name collisions

As per our discussion, I think it might be better to use a "proxy" model in the migration directly instead.


Performance Difference Compared With Original Paper？

Thank you for your implementation!!!
Can you share Performance Difference Compared With Original Paper and checkpoint?

I am still running experiments.

Is there any update of the performance?

Does Pix2Seq converge slower than DETR?

Due to different learning schedules, it's hard to compare the convergence speed between DETR and Pix2Seq.

We released the pretrained model at https://github.com/gaopengcuhk/Pretrained-Pix2Seq.