IStringLocalizer where T is part of library assembly and not web application assembly

Hi,
I am having some serious issue setting up localization for our new web application and am forced to admit that this has gotten the best of me.
Background
I have multiple web applications and multiple libraries, all which needs to support localisation. The idea is to inject IStringLocalizer instances into classes located in the libraries from a web application. A lot of string resources will be shared between the UI layer in the web applications and the business logic in the libraries. Therefor it is desired to share resource files and the idea is to use IStringLocalizer where ASharedClass is defined in a library.
Now, when I try to setup a simple solution to support this, I can't get the IStringLocalizer instances to use the correct resources files unless I override the ResourceManagerStringLocalizerFactory and clean it from "assembly.Resources". Interestingly, using ResourceManager works, although it picks resource files from unexpected locations.
Demo
I have a simple demo repo for this, in which the web application is setup for localisation and setup to use the "Resources" folder for resource files. It has a vast amount of resource files in various places with various namings to try to get a resource loaded. Each resource file have a different value for the string "Test".
Repo URL: https://github.com/xandhen/NetCoreLocalizationPlayground
Current output
IStringLocalizer (Class1): Test
Culture: sv:sv
ResourceManager: Localized string for value "Test" classlibrary root (en)
IStringLocalizer (LocalClass): Test
Expected output
IStringLocalizer (Class1): Localized string for value "Test" web resource folder Namespace.class (sv)
Culture: sv:sv
ResourceManager: Localized string for value "Test" web resource folder Namespace.class (sv)
IStringLocalizer (LocalClass): Localized string for value "Test" web resource folder assembly local (sv)
Analysis
As one can se, no resource file is used for looking up the string except for in the case where the ResourceManager class is used directly, but then it picks the resource file from the root of the library, not the resource folder of the library or the web application.
It is also visible the even though the default culture is set to Swedish, the UI culture gets set to English. This is also the case when the Accept-Language request header includes sv as the first language in the list.

@ryanbrandenburg can you take a look at this? I think this is an area where others have run into problems. Assuming this all works, but is tricky, do we have a sample for this? Or missing docs?

As an update, I noticed that there seems to be some connection with the assembly name and the root namespace of the library project. If I rename the project (only .csproj file, directory doesn't matter) to be the same as the root namespace of the library project, then it can successfully load the rest file from the Resources directory of the library project. If this is also done for the web application, then the assembly local res file can be found.
Output with .csproj files renamed to be equal to the root namespace.
IStringLocalizer (Class1): Localized string for value "Test" classlibrary resource dir (en)
Culture: sv:en
ResourceManager: Localized string for value "Test" classlibrary root (en)
IStringLocalizer (LocalClass): Localized string for value "Test" web resource folder assembly local (en)

You are correct, renaming your csproj files fixes the issue. I've filed dotnet/sdk#895 to find out if this is expected behavior. If it is we'll update the documentation to reflect that.

@ryanbrandenburg I'm putting this on your plate for tracking until we get final confirmation from the SDK team.

In the mean time @xandhen you can work around this if you set both AssemblyName and RootNamespace to the same value and use that value as the root of your class names.

@ryanbrandenburg Thanks for the tip. Explicitly setting the AssemblyName removes the need for renaming the csproj file and enables a "logical" naming of the project, not tied to the output assembly name.
I'm still puzzled about the culture since only CurrentCulture is set to Swedish while CurrentUiCulture is being set to English. The default request culture is set to "sv-SE" in the application startup and even though the HTTP Accept-Language request header has sv as the first language it is ignored for the UiCulture. This is really blocking me as the web application in development is a primarily Swedish application. I would like UiCulture to be set to english only if Swedish is completely missing from the localisation cookie, query string and request header.
Some additional food for thought though.

When injecting resources through IStringLocalizer the resource directory within the class library is dictated by the localisation service configuration services.AddLocalization((LocalizationOptions options) => options.ResourcesPath = ResourcePath);
When getting resources through the ResourceManager the resources are expected to be located in the assembly root folder new ResourceManager(typeof(Class1)).GetString("Test")

I'm not sure this should be aligned or even how, but it could be worth mentioning this in the docs as it forces an alignment of resource file location within all the assemblies of an application.
A potential mitigation for this, which could be very useful in general, is to allow the executable assembly to override resource files in the referenced assemblies. This would allow one to place resource files in a directory of choice in the executable assembly therefor not be dependent on location in the referenced libraries (even though you have to copy all the resource files with a scoped file name). It would also allow an application to override resources in referenced libraries and provide more application tailored resources.
With this approach, the resource location would be resolved in this order

Resource directory in executable assembly (full namespace naming of resource files)
Resource directory in the assembly that owns the class

Another issue seems to be that the csproj file name can not include '.'.
For example a library project with a project file named x.y.csproj and assembly and root namespace name of 'x.y' will not work.
But a library project with a project file named xy.csproj and assembly and root namespace name of 'x.y' will work.

I'm not sure I entirely follow your scenario in the second paragraph, maybe you could clarify with expected vs actual examples? But I do note that your SupportedCulture objects are different from eachother here and that supportedCulture only has the Culture "se", which is the code for "Sami", not swedish.
We don't advise that people manually create instances of ResourceManager (if you know of somewhere where we do please let me know so we can fix it) and the behavior of a raw ResourceManager is not within the scope of the localization project or its documentation. If you DO create one then you'll have to learn the intricacies of how Resources are generated and retrieved. You can learn some about that by reading the code for ResourceManagerStringLocalizer.
Your second comment seems worth noting on dotnet/sdk#895 since it creates a limit on Resource names. I'll bring it up there.

@ryanbrandenburg Thanks for the tip and forwarding the resource name limits.
I tried however aligning the  SupportedCulture objects to the following
protected static List<CultureInfo> supportedCultures = new List<CultureInfo>
{
	new CultureInfo("sv-SE")
};
protected static List<CultureInfo> supportedUICultures = new List<CultureInfo>
{
	new CultureInfo("sv-SE"),
	new CultureInfo("en-US"),
	new CultureInfo("nn-NO")
};

This doesn't change anything however and the CurrentUICulture is still set to english. The only way to get Swedish UICulture is to provide an empty list of RequestCultureProviders to the RequestLocalizationOptions instance that is supplied as an argument to the app.UseRequestLocalization() call.
As this is my AcceptLanguage header, I would assume that the AcceptLanguageHeaderRequestCultureProvider provider would set Swedish as the UI language.
Accept-Language:sv,en-US;q=0.8,en;q=0.6,el;q=0.4
The expected outcome, for any request where a language cookie is not set and a query string parameter is not supplied, would be that if Swedish "sv" is supplied in the Accept-Language header property or if neither any of the supported UI cultures ("sv", "nn" or "en") are supplied in the Accept-Language header, that the CurrentUICulture is set to Swedish.
I would also expect that "en" or "nn" would only be set as the CurrentUICulture if they are supplied in the the Accept-Language header and "sv" is not supplied therein.
The current outcome is that the English is always set as the CurrentUICulture if it is present in the Accept-Language header, even if it is specified with a lower priority than other supported UI cultures. I've fiddled around with some combinations of RequestCultureProviders and Accept-Language header values and found that the core issue seems to be a bug in the prioritisation in the AcceptLanguageHeaderRequestCultureProvider class. It can be noted that my system (OS X) is set to English so maybe the AcceptLanguageHeaderRequestCultureProvider ranks the system culture higher than the language list supplied in the Accept-Language header value?

You're welcome. Sorry for the confusion but I requested a little more information regarding the csproj "." stuff in an edit (I accidentally pressed comment while still composing my response).
You're requesting "general" swedish in your acceptLanguage, but you've told your program that you only accept Sweden swedish. Since your headers only accept the more general sv it won't give you the more specific sv-SE. The reverse would work (If your AcceptLanguage is sv-SE and your supportedUICultures includes sv you would be returned sv) but you can only go from more specific locals to less specific.
One way to solve this is to include sv in supportedUICultures, but the "correct" course depends on the specifics of that language tree, which I'll leave to you.

I'm a bit puzzled about the relationship between SupportedCulture and SupportedUICulture in this case. To my understanding, the UICulture is for language purposes, while Culture is mainly for formatting etc. I would therefor believe that language would be the primary selector and one could therefor go from less to more specifics. In my case I also only have a single supported culture, so regardless of the specifics of the header value, it will always be set to "sv-SE".
Unfortunately to go form more to less doesn't work well with modern browsers sine e.g. adding an extra language in Chrome most of the time only gives you a less specific choice, unless the language selected comes in many cultures. In my case I only want to support multiple languages, but not cultures so being less specific in the SupportedUICulture list will not be a problem.
E.g. English has 7 different options in Chrome while Swedish only have one option, which does not include culture ("sv"). On OS X Safari is even worse, it will only send the primary OS language.
Perhaps these details of the AcceptLanguageHeaderRequestCultureProvider could be added to the docs?

@ryanbrandenburg regarding the csproj "." stuff I think you can ignore that for now. I had that issue in two applications in different repositories and now I can't reproduce it in any off them.

You understand the difference between SupportedCulture and SupportedUICulture correctly, but I think your suggestion to go from less specific to more specific Cultures would be more confusing than helpful. If your application supported both es-ES and es-MX, and the AcceptHeader of a request was for "es" which one would be chosen under your system? Under the current system in this scenario if you wanted to return es-MX in this scenario you can simply create two resource sets, "es" which would be returned for a request of either es or es-MX and es-ES. @DamianEdwards does that all sound right?
That said, you're correct that we don't talk about this in the docs, I filed aspnet/Docs#2846.

Seems like this is wrapped up so I'm closing it out.


[CI] Remove Multi-Platform Build and Release workflow

Current Behavior

This workflow: https://github.com/layer5io/sistent/blob/master/.github/workflows/multi-platform.yml is disabled
Desired Behavior

Remove workflow as this is no longer needed
Implementation

Acceptance Tests

Contributor Guides and Resources

🛠 Meshery Build & Release Strategy
🎨 Wireframes and designs for Sistent site in Figma (open invite)
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

@nebula-aac If it is confirmed to be disabled, I would like to raise a PR to remove this workflow file from the repository.
So please assign it to me.


cmd/graph-vulcan-assets: add command

This PR adds the command graph-vulcan-assets.
The current implementation requires a lot of requests against the Asset
Inventory API every time an asset is updated. We need to evaluate if the
throughput is acceptable in the context of our production environment.
This PR depends on adevinta/graph-asset-inventory-api#10. It also
expects the new release of graph-asset-inventory-api to be tagged as
v0.6.0.

@manelmontilla PTAL
You can use this link to review the last changes, I did it in multiple commits and the GitHub GUI does not allow to select a wide range of commits...


[ICS20-2] Final cleanup for multi denom

This issue contains misc. points that have come up and need to be addressed before this feature can be shipped.

[x] GetEscrowAddress uses a hardcoded string with value ics20-1. Reason if this could be changed without breaking backwards compatibility. (See which test(s) failed when changing this constant)
[x] Need to make decision about: adding a new MsgTransferV2 with tokens (and no token field) or use existing MsgTransfer , add the tokens field and deprecate (and later remove) field token.
[x] Maybe for the ICS20V1 and ICS20V2 const strings we could have an enum ICS20Versions with two values V1 and V2.
[x] Add validation for the Token trace in packet v2 ValidateBasic

Addressing the listed bullets:

As per @colin-axner's comment, we can leave as "ics20-1", I will add a comment to make this more explicit.
We will stick with using MsgTransfer and add a Tokens array instead of a MsgTransferV2
Still open for debate, but thinking more about it, just keeping two string constants is fine by me. "ics20-1" and "ics20-2"
Token trace validation has already been included here.

closed by #6345


small bug fix with unpermitted params

in the method workout_params included params.require(:workout) to ensure the hash contains a workout key

Why are we deleting lines 15 - 19 and 40 - 42 in workouts_controller? Is this logic being rendered elsewhere or replaced, otherwise how are we fetching the leaderboard?

youre, right, lines 15-19 & update_leaderboard method shouldnt be removed

Can you fix, and then push again?


Refactor file validation to handle spin and repoint file extensions

Change Summary
closes #100
Overview
Added paths for spin and repointing data in spice/ directory on s3.
Updated Files

imap_data_access/file_validation.py

Added spin and repoint updates

Testing

tests/test_file_validation.py

pre-commit.ci autofix


Enable ppc64le support

This PR adds support for hyperstart on ppc64le architecture.
runv hot plugs the NIC after VM is booted. In ppc64le RTAS daemon enables this feature for the VM. This patch adds RTAS related libs in initrd for ppc64le architecture.
Signed-off-by: Harshal Patil<EMAIL_ADDRESS>

Can one of the admins verify this patch?

@laijs I have updated  PR as per our discussion.

Thanks!


Registration

Just installed V1.8. Logged in as Admin ok. Logged out and attempted to register as a new member. On clicking the "Register" button two boxes appear. A green one saying "You are now registered" and a red one saying "An error occurred".  Logging in as admin and looking at the Membership > Manage Users section there is only the Admin account showing.
Logs show:-
Validation failed for one or more entities. See 'EntityValidationErrors' property for more details. The validation errors are: The field Body must be a string or array type with a maximum length of '4000'.
Cheers,
Craig

Good catch. It's a SQLCE issue and purely down to the email body field being too long to store in SQLCE. I've just pushed up a stripped down email (Not ideal but it has to work on both SQLCE and MSSQL).


FlashyWrappers on Android devices with high resolution

Hi,
We exploring some issue with the screen blinking using FlashyWrappers on Android devices with high resolution (e.g. Samsung Galaxy 7, Motorola XT1254).
Here you can watch video taken from those devices:
https://drive.google.com/open?id=0B8R1h4LnVnSFZHRQN1diOEU2OUE
https://drive.google.com/open?id=0B8R1h4LnVnSFVnlIZDNtXy0wVFU
Can you please help us? It's an emergency.
P.S. We use GPU render mode.

Here is a photo of the issue. We should also note that we tested this on a less complex animation with only FlashWrapper contained in the build.

Update: when we use "direct" render mode screen blinking disappears but then we have problems with Sound Mixer.

Never got to this, the project ceased to be commerical as I was unable to do fulltime support for free. If somebody still has this issue let me know - I'll be willing to do paid support for urgent issues.


BugFix: Import Action Look at File System Disk Driver for s3

Description
When a filesystems disk name is not s3 the import action fails with fopen regardless of FILAMENT_FILESYSTEM_DISK and FILESYSTEM_DISK being set to the s3 driver. This PR changes the lookup to check the driver of the configured disks vs the name.
Visual changes
N/A
Functional changes

[x] Code style has been fixed by running the composer cs command.
[x] Changes have been tested to not break existing functionality.
[x] Documentation is up-to-date.

Happy to see this fix.  We ran into this a few weeks ago and just haven't had the bandwidth to try to address it via PR.
Thx, @mmccall10  and @danharrin !  🤓


Invalidate Query

Add the option to invalidate queries by key on the frontend when a mutation is submitted (like trpc does).
For now, you could use a subscription to work around the lack of this although that's not gonna be the best DX.
Also, add typesafe invalidate hook to the frontend.

Maybe something like
<Router>::new()
    .query("version", |ctx, arg: ()| {
        ctx.invalidate_key(..); // This would return a result and would fail if the key or arguments are invalid.
        invalidate_key!(ctx, ..); // Similar to Spacedrive's `invalidate_query!` this would be able to check the argument/key is valid on startup using ctor.
    })

#113 just linking this because it discusses this issue.

One huge concern here is that we need to be able to only invalidate queries that are actively rendered on the frontend.
For future me's reference: keep it scoped to a single procedure and return the keys to invalidate in the response for it. This way this isn't a concern.

Why is the backend concerned with query invalidation, shouldn't that be client-only?
Sorry if there's something obvious I'm missing. I was just checking out the library so I'm not familiar with it.

For many applications, all of the changes to data are coming from the active user and in this case, you only need to deal with invalidation on the frontend because it's fine to assume the active user is only going to have a single instance of your app open at a time.
In a practical example: The user makes a change to some data, then the frontend updates its own cache and shows the user the updated data. If the user jumps to another device we assume they will open/reload the page to get the most up-to-date data.
For Spacedrive we have a whole job system that runs in the background and can scan & organise your files. We also sync data over a peer-to-peer system so all of your devices have their own local copy of your data. The challenge here is that many of the data changes come from the backend instead of the user/frontend. A backend invalidation system allows the backend to ask the frontend to update a piece of data because it knows it has changed.
In a practical example: A job in Spacedrive scans a new file and the "total storage used" for a library is updated. The invalidation system then tells the frontend to refetch the value from the backend so it is showing the most accurate value. Without a backend invalidation system the frontend wouldn't know the data had changed and it would require the user to refresh the page for an updated value which isn't a great UX for an application as interactive as Spacedrive.
Many applications poll for new data every x seconds/minutes. This is super easy to implement but it is either slow to receive updates or has a large network and server overhead. Backend-powered invalidation is hard but it is much more efficient and can lead to near-instant updates for new data.
Hopefully that all makes sense, let me know if you need any clarification.

That was very clear! Thanks for the detailed response :)

For my personal reference this is an idea Brendan came up with for Spacedrive.

just a quick solution for @rspc/react / @rspc/client / @tanstack/react-query users
export function invalidateQuery(queryKey: Procedures["queries"]["key"]) {
  queryClient.invalidateQueries({ queryKey: [queryKey] });
};


Ensuring that the random number generator is TRNG.

closes #1120

@bjoernQ - I've tried to do some "interference" tests (some basic cases, like just use both ADC and RNG peripherals) and seems to work fine...
At least if there'll be some error in those drivers we know where's the first suspect 😅

I think that the trng module should not exist, and that code should probably live in the soc module.

@jessebraham agree. Will be redesigned

Rebased onto main, ready for merge


Change timescaleDB to use the new database model and migrate if necessary

timescaleDB should create the new database model when no database exists. If there is already a database, it should automatically migrate.

Superseded by https://github.com/united-manufacturing-hub/united-manufacturing-hub/issues/1286


安装问题

您好，我按照您的教程安装，但是始终不成功，ImportError: No module named stockwell
我的环境是：ubuntu16.04,python2.7
请问您可以解决下吗

Refer to #1 It has been ported to python3 now, use
pip install -U stockwell

to install new version.

Which repository and version does "pip install -U stockwell" install?
thanks!
rich
to 27. kesäk. 2019 klo 14.38 ixaxaar<EMAIL_ADDRESS>kirjoitti:

Closed #3 https://github.com/synergetics/stockwell_transform/issues/3.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/synergetics/stockwell_transform/issues/3?email_source=notifications&email_token=ABEBKULQ27C36IAX5IITHBTP4UCJTA5CNFSM4GY4XRPKYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOSG7P6MQ#event-2445213490,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABEBKUP6Y6WB4DDKQPVUIQLP4UCJTANCNFSM4GY4XRPA
.

--
\  Rich Hammett       http://blog.arichneuraltapestry.com
/<EMAIL_ADDRESS>\ Nothing great is achieved without passion, but underneath
/ the passion there should always be that large impersonal
\ survey which sets limits to actions that our passions inspire.
/ - Bertrand Russell

It installs v1.0.0 of the code in this repository, as tagged in setup.py.

Hello, I have successfully installed Stockwell, but the error is: ModuleNotFoundError: No module named 'stockwell.smt'，I don't understand why.
Thanks！

Hey the stockwell package on pypi is no longer this one, it is a different project maintained by someone else


Catch TypeError together with OSError when inspect source

close #111

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 98.93%. Comparing base (a950dba) to head (5544a95).

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@             Coverage Diff             @@
##            master     #115      +/-   ##
===========================================
- Coverage   100.00%   98.93%   -1.07%
===========================================
  Files            5        5
  Lines          374      374
===========================================
- Hits           374      370       -4
- Misses           0        4       +4

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


Player stats coming back as null/NaN

Hi,
Great work on this project by the way, really helpful and fun tool. I am having an issue where I am receiving the following response when calling ow.playerStats() and wondering if you know what might be causing it:
  stats: {
    competitive_rank: { tank: null, damage: null, support: null },
    endorsement_level: NaN,
    games_won: NaN,
    achievements: [],
    quickplay: {},
    competitive: {}
  }

This account is at level 154 and has played many competitive and quickplay games. Is there something I'm missing?
Cheers!

Hey, I figured it out.
ow.player() is not case sensitive
ow.playerStats() is case sensitive
:)


"over-writing existing variable" log message needs to be set as [debug] log level

over-writing existing variable '{}' with new value: {} is currently set at WARN log level.
10:07:33.439 [main] WARN  com.intuit.karate - over-writing existing variable 'user' with new value: 'NEW VALUE'
It's not really a warning and it increases the log size especially when executing performance test where we only want to check for actionable warnings. I have discussed with @ptrthomas about this issue and creating PR for the same.

1.0.1 released


Context-Menu COPY is not working in vscode webview, BUT ctrl+c is working fine

Issue Type: Bug
Use Case 1:

Install an extension that uses a Webview (Oracle Developer Tool for VS Code)
Connect to a Database
Run any SELECT query
In Result Tab, we can see the SQL Query that we just ran + Query Result
Select the SQL Query Text >> Right Click >> COPY
Paste in any text editor

Expected Result: Text is getting copied to text editor
Actual Result: Text is not getting copied to text editor
Use Case 2:

Install an extension that uses a Webview (Oracle Developer Tool for VS Code)
Connect to a Database
Run any SELECT query
In Result Tab, we can see the SQL Query that we just ran + Query Result
Select the SQL Query Text >> Press Ctrl+C
Paste in any text editor

Expected Result: Text is getting copied to text editor
Actual Result: Text is getting copied to text editor
VS Code version: Code 1.63.2 (899d46d82c4c95423fb7e10e68eba52050e30ba3, 2021-12-15T09:40:02.816Z)
OS version: Windows_NT x64 10.0.18363
Restricted Mode: No

System Info

Item
Value

CPUs
Intel Xeon E3-12xx v2 (Ivy Bridge, IBRS) (2 x 2693)

GPU Status
2d_canvas: unavailable_softwaregpu_compositing: disabled_softwaremultiple_raster_threads: disabled_offoop_rasterization: disabled_offopengl: disabled_offrasterization: disabled_softwareskia_renderer: enabled_onvideo_decode: disabled_softwarevulkan: disabled_offwebgl: unavailable_softwarewebgl2: unavailable_software

Load (avg)
undefined

Memory (System)
14.65GB (8.33GB free)

Process Argv
. --crash-reporter-id 699fa828-d5c2-42a4-95b5-df472e05dbaf

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

oracledevtools
Ora
21.3.0

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyl392:30422343
pythontb:30283811
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
pythondataviewer:30285071
vscod805:30301674
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
vsc1dst:30424896
pythonvs932:30410667
vscop804:30404766
vscop453:30404998
vsrem710cf:30416617

Please put together a minimal example that doesn't rely on an complex extension or connecting to a database
You can build on the webview sample for this: https://github.com/microsoft/vscode-extension-samples/tree/main/webview-sample

@mjbvz , we have already looked into the sample code. In fact, we made the below change in the html returned from _getHtmlForWebview method like below:
Original Code:

0

Replaced with

This is a Heading

This is a paragraph.

After making the change, we tried to copy the text using context menu from the Webview but text is not getting copied.

Also, please let us know if there is any way to disable/greyed-out CUT/PASTE item in context menu. We just need the COPY item in the context menu.

Logan and I ran into this in the hex editor today as well.

I'm also seeing this issue with a web view on our extension.
It's really weird, Ctrl+C works perfectly fine, but right-click -> copy does not copy to the clipboard.
I found this, and I think this might be related to the issue:
https://stackoverflow.com/questions/70788421/context-menu-copy-is-not-working-in-vscode-webview-but-ctrlc-is-working-fine
A user replicated this issue while debugging vscode and found out that CTRL+C and right-click->Copy have different implementations.
Does anyone know if there is a workaround for this? Or if there'll be a resolution soon?
Thanks!

I have the exact inverse problem on Mac os (right click copy is working fine, but cmd+c does not work).

@mjbvz this issue happens on both Windows as well as on Ubuntu. Text is not copied to clipboard.
It would be a really breeze to use on cases where it's easier to read and copy data from a markdown table for use in an another tool.
For example, I want to copy docker image sizes from a markdown file, while previewing it. Image attached below:

Bumping this issue, it's a bit annoying for users when interfacing with a GUI-based extension.

Also, please let us know if there is any way to disable/greyed-out CUT/PASTE item in context menu. We just need the COPY item in the context menu.

https://stackoverflow.com/a/73247563/819417 should fix that and also the copy issue for your extension.

My team is running into this same issue with the Thunder Client extension, as well as the extensions market place. We cannot cut/copy/paste from these.

The context menu copy function is not working
Does this issue occur when all extensions are disabled?: Yes/No
Yes
VS Code Version: 1.78.2
OS Version: Windows 11 Pro 22H2
Steps to Reproduce:

I copy using the context menu.
I paste, but nothing is copied.
I copy using the keyboard shortcut.
I paste, and everything is copied.

Please watch the video.
https://github.com/microsoft/vscode/assets/109512933/7cedd1c7-0b0d-47e8-87b3-e82fe9cefec2

I'm using a codegpt extension and im not able to copy. what's the point of codegpt then lol

Should we remove the "copy" context menu action until it's working? It's rather misleading at the moment.

+1 here on being not able to copy logs from kubernetes addon

I am having the issue as well in Thunderclient and markdown extensions.
Thank you for any work put forward to resolve this.

ref: https://github.com/microsoft/vscode/issues/205624, Copy Image command in webview is not working neither.

@mjbvz I found the cause of the problem. When we right-click on the webview, the right-click menu that appears is actually provided by vscode (not webview), and webview mistakenly regarded this situation as losing focus. When the callback function of the copy command is executed, the activeWebview value cannot be obtained, so no operation is done.
When the right-click menu appears, onBlur is executed.

The error caused vscode to think that there was no webview in the active state at this time.
https://github.com/microsoft/vscode/blob/00abefa3e27ee1866cf6a6ac825578c1ddafd32c/src/vs/workbench/contrib/webview/browser/webview.contribution.ts#L22~L38

@mjbvz I found the cause of the problem. When we right-click on the webview, the right-click menu that appears is actually provided by vscode (not webview), and webview mistakenly regarded this situation as losing focus. When the callback function of the copy command is executed, the activeWebview value cannot be obtained, so no operation is done.我找到了问题的原因。当我们右键单击webview时，出现的右键菜单实际上是由vscode (而不是webview) 提供的，webview错误地将这种情况视为失去焦点。执行copy命令的回调函数时，无法获取activeWebview值，因此不进行任何操作。
When the right-click menu appears, onBlur is executed.出现右键单击菜单时，将执行onBlur。
The error caused vscode to think that there was no webview in the active state at this time.该错误导致vscode认为此时没有处于活动状态的webview。
https://github.com/microsoft/vscode/blob/00abefa3e27ee1866cf6a6ac825578c1ddafd32c/src/vs/workbench/contrib/webview/browser/webview.contribution.ts#L22~L38

I tried to fix the problem in #206529 : only need to judge whether the right-click menu currently appears when judging whether the webview is focused.


[FLINK-7638] [flip6] Port CurrentJobsOverviewHandler to new REST endpoint

What is the purpose of the change
Ports the CurrentJobsOverviewHandler to the new REST endpoint by letting it implement
the LegacyRestHandler interface. This commit changes the JobDetails JSON such that it
now contains the number of tasks for each ExecutionState, including SCHEDULED,
DEPLOYING, CREATED and RECONCILING. These state will now also be displayed in the
web frontend.
Verifying this change

Have tested manually that the old web frontend is still working
Added JobDetailsTest and MultipleJobsDetailsTest which test the un/marshalling

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

Thanks for the review @tzulitai. I've addressed your comments. Concerning the custom serialization logic, I think it does not hurt to have it, because it should be more efficient than the generic serialization logic. Moreover it's an implementation detail which could be changed later on as well. Given that, I would like to keep it at the moment like this.

@tillrohrmann Yes that makes sense, lets do that separately in the future then.
+1.

Thanks for your review @tzulitai. Travis passed. Will merge the PR.


Data model now draw.io based

Data model now draw.io based :)

Cf. https://github.com/AMWA-TV/is-11/blob/v1.0-dev/docs/images/is-11-data-model.png which is also a diagrams.net PNG.
It would be nice if the two diagrams ended up identical apart from the introduction of Input and Output in the IS-11 one.

A relationship has changed:
was Sender 1 -> 0..* Receiver and is now Sender 0..* -> 1 Receiver
I think that the former was the right one.

In the old diagram the arrowhead from a flow was pointing towards the source whereas in the new one the direction is reversed which might be wrong?

This PR should address the comment about the diagram in #176.


Error in Mobaxterm

Here is the Error Log when i run the code in Mobaxterm.
2024-11-05 19:59:58.279712: E external/local_xla/xla/stream_executor/cuda/cuda_fft.cc:477] Unable to register cuFFT factory: Attempting to register factory for plugin cuFFT when one has already been registered
WARNING: All log messages before absl::InitializeLog() is called are written to STDERR
E0000 00:00:1730807998.300203  429218 cuda_dnn.cc:8310] Unable to register cuDNN factory: Attempting to register factory for plugin cuDNN when one has already been registered
E0000 00:00:1730807998.306628  429218 cuda_blas.cc:1418] Unable to register cuBLAS factory: Attempting to register factory for plugin cuBLAS when one has already been registered
2024-11-05 19:59:58.328525: I tensorflow/core/platform/cpu_feature_guard.cc:210] This TensorFlow binary is optimized to use available CPU instructions in performance-critical operations.
To enable the following instructions: AVX2 AVX512F FMA, in other operations, rebuild TensorFlow with the appropriate compiler flags.
Framework not specified. Using pt to export the model.
Loading TensorFlow model in PyTorch before exporting.
W0000 00:00:1730808000.512014  429218 gpu_device.cc:2344] Cannot dlopen some GPU libraries. Please make sure the missing libraries mentioned above are installed properly if you would like to use GPU. Follow the guide at https://www.tensorflow.org/install/gpu for how to download and setup the required libraries for your platform.
Skipping registering GPU devices...
Traceback (most recent call last):
File "/home/gymao/openvino_build_deploy/ai_ref_kits/custom_ai_assistant/venv/lib/python3.11/site-packages/transformers/modeling_utils.py", line 575, in load_state_dict
return torch.load(
^^^^^^^^^^^
File "/home/gymao/openvino_build_deploy/ai_ref_kits/custom_ai_assistant/venv/lib/python3.11/site-packages/torch/serialization.py", line 1039, in load
raise pickle.UnpicklingError(UNSAFE_MESSAGE + str(e)) from None
_pickle.UnpicklingError: Weights only load failed. Re-running torch.load with weights_only set to False will likely succeed, but it can result in arbitrary code execution.Do it only if you get the file from a trusted source. WeightsUnpickler error: Unsupported operand 118
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/gymao/openvino_build_deploy/ai_ref_kits/custom_ai_assistant/venv/lib/python3.11/site-packages/optimum/exporters/tasks.py", line 2115, in get_model_from_task
model = model_class.from_pretrained(model_name_or_path, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/gymao/openvino_build_deploy/ai_ref_kits/custom_ai_assistant/venv/lib/python3.11/site-packages/transformers/models/auto/auto_factory.py", line 564, in from_pretrained
return model_class.from_pretrained(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/gymao/openvino_build_deploy/ai_ref_kits/custom_ai_assistant/venv/lib/python3.11/site-packages/transformers/modeling_utils.py", line 3726, in from_pretrained
state_dict = load_state_dict(resolved_archive_file)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/gymao/openvino_build_deploy/ai_ref_kits/custom_ai_assistant/venv/lib/python3.11/site-packages/transformers/modeling_utils.py", line 585, in load_state_dict
raise OSError(
OSError: You seem to have cloned a repository without having git-lfs installed. Please install git-lfs and run git lfs install followed by git lfs pull in the folder you cloned.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/gymao/openvino_build_deploy/ai_ref_kits/custom_ai_assistant/convert_and_optimize_asr.py", line 57, in
convert_asr_model(args.asr_model_type, args.precision, Path(args.model_dir))
File "/home/gymao/openvino_build_deploy/ai_ref_kits/custom_ai_assistant/convert_and_optimize_asr.py", line 37, in convert_asr_model
ov_model = OVModelForSpeechSeq2Seq.from_pretrained("/home/gymao/openvino_build_deploy/ai_ref_kits/custom_ai_assistant/model/Belle-distilwhisper-large-v2-zh/", export=True, compile=False, load_in_8bit=False)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/gymao/openvino_build_deploy/ai_ref_kits/custom_ai_assistant/venv/lib/python3.11/site-packages/optimum/intel/openvino/modeling_base.py", line 373, in from_pretrained
return super().from_pretrained(
^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/gymao/openvino_build_deploy/ai_ref_kits/custom_ai_assistant/venv/lib/python3.11/site-packages/optimum/modeling_base.py", line 424, in from_pretrained
return from_pretrained_method(
^^^^^^^^^^^^^^^^^^^^^^^
File "/home/gymao/openvino_build_deploy/ai_ref_kits/custom_ai_assistant/venv/lib/python3.11/site-packages/optimum/intel/openvino/modeling_base_seq2seq.py", line 297, in _from_transformers
main_export(
File "/home/gymao/openvino_build_deploy/ai_ref_kits/custom_ai_assistant/venv/lib/python3.11/site-packages/optimum/exporters/openvino/main.py", line 308, in main_export
model = TasksManager.get_model_from_task(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/gymao/openvino_build_deploy/ai_ref_kits/custom_ai_assistant/venv/lib/python3.11/site-packages/optimum/exporters/tasks.py", line 2124, in get_model_from_task
model = model_class.from_pretrained(model_name_or_path, **kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/gymao/openvino_build_deploy/ai_ref_kits/custom_ai_assistant/venv/lib/python3.11/site-packages/transformers/models/auto/auto_factory.py", line 564, in from_pretrained
return model_class.from_pretrained(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/gymao/openvino_build_deploy/ai_ref_kits/custom_ai_assistant/venv/lib/python3.11/site-packages/transformers/modeling_utils.py", line 3892, in from_pretrained
model, loading_info = load_tf2_checkpoint_in_pytorch_model(
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/gymao/openvino_build_deploy/ai_ref_kits/custom_ai_assistant/venv/lib/python3.11/site-packages/transformers/modeling_tf_pytorch_utils.py", line 532, in load_tf2_checkpoint_in_pytorch_model
load_tf_weights(tf_model, tf_checkpoint_path)
File "/home/gymao/openvino_build_deploy/ai_ref_kits/custom_ai_assistant/venv/lib/python3.11/site-packages/transformers/modeling_tf_utils.py", line 952, in load_tf_weights
return load_function(
^^^^^^^^^^^^^^
File "/home/gymao/openvino_build_deploy/ai_ref_kits/custom_ai_assistant/venv/lib/python3.11/site-packages/transformers/modeling_tf_utils.py", line 961, in load_tf_weights_from_h5
with h5py.File(resolved_archive_file, "r") as sharded_checkpoint_file:
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/gymao/openvino_build_deploy/ai_ref_kits/custom_ai_assistant/venv/lib/python3.11/site-packages/h5py/_hl/files.py", line 561, in init
fid = make_fid(name, mode, userblock_size, fapl, fcpl, swmr=swmr)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/home/gymao/openvino_build_deploy/ai_ref_kits/custom_ai_assistant/venv/lib/python3.11/site-packages/h5py/_hl/files.py", line 235, in make_fid
fid = h5f.open(name, flags, fapl=fapl)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "h5py/_objects.pyx", line 54, in h5py._objects.with_phil.wrapper
File "h5py/_objects.pyx", line 55, in h5py._objects.with_phil.wrapper
File "h5py/h5f.pyx", line 102, in h5py.h5f.open
OSError: Unable to synchronously open file (file signature not found)

Possible to share the pip freeze?

Also the environment and so.


Error of "TypeError: object of type 'NoneType' has no len()" when doing ancestry classification

What you did:
We want to use the ancestry classification tool to predict genetic ancestry of UKBB samples. (https://gnomad.broadinstitute.org/news/2021-09-using-the-gnomad-ancestry-principal-components-analysis-loadings-and-random-forest-classifier-on-your-dataset/)
What happened:
There is an error of "TypeError: object of type 'NoneType' has no len()" when I finally did the prediction.
Below I attach all my codes and their outputs.

Python 3.9.10 (main, Feb 22 2022, 16:34:24)
[GCC 4.8.5 20150623 (Red Hat 4.8.5-44)] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>>
>>> # Two measurements at the initial visit (*_0_0 and *_0_1) are averaged
>>> import pandas as pd

## Import imputed genotype data
mt_to_project = hl.import_bgen(bgen_path,
                    entry_fields=['GT'],
                    sample_file=sample_path,
                    index_file_map={bgen_path: bgenidx_path})

import pickle
from gnomad.sample_qc.ancestry import assign_population_pcs

# Load MatrixTable for projection and gnomAD loadings Hail Table
loadings_ht = hl.read_table(path_to_gnomad_loadings)

# Project new genotypes onto loadings
ht = hl.experimental.pc_project(
    mt_to_project.GT,
    loadings_ht.loadings,
    loadings_ht.pca_af,
)

>>> import os
>>> import sys
>>> os.environ["PYSPARK_SUBMIT_ARGS"]="--total-executor-cores 6 --executor-memory 5g --driver-memory 10g pyspark-shell"
>>>
>>> import hail as hl

# Assign global ancestry using the gnomAD RF model and PC project scores
# Loading of the v2 RF model requires an older version of scikit-learn, this can be installed using pip install -U scikit-learn==0.21.3
with hl.hadoop_open(path_to_gnomad_rf, "rb") as f:
    fit = pickle.load(f)

# Reduce the scores to only those used in the RF model, this was 6 for v2 and 16 for v3.1
num_pcs = fit.n_features_
ht = ht.annotate(scores=ht.scores[:num_pcs])
htt, rf_model = assign_population_pcs(
    ht,
    pc_cols=ht.scores,
    fit=fit,
)

>>> hl.init()
2022-07-01 11:01:00 WARN  NativeCodeLoader:60 - Unable to load native-hadoop library for your platform... using builtin-java classes where applicable
Setting default log level to "WARN".
To adjust logging level use sc.setLogLevel(newLevel). For SparkR, use setLogLevel(newLevel).
2022-07-01 11:01:01 WARN  Hail:43 - This Hail JAR was compiled for Spark 3.1.2, running with Spark 3.1.3.
  Compatibility is not guaranteed.
Running on Apache Spark version 3.1.3
SparkUI available at http://compute-106.cm.cluster:4040
Welcome to
     __  __     <>__
    / /_/ /__  __/ /
   / __  / _ `/ / /
  /_/ /_/\_,_/_/_/   version 0.2.96-39909e0a396f
LOGGING: writing to /users/jzhang2/hail-20220701-1100-0.2.96-39909e0a396f.log
>>>
>>> ## Path to input files
>>> bgen_path = "./chrall.bgen"
>>> sample_path = "./chrall.sample"
>>> bgenidx_path = "./chrall.idx2"
>>> path_to_gnomad_loadings = "./gnomad/gnomad.v3.1.pca_loadings.ht"
>>> path_to_gnomad_rf = "./gnomad/gnomad.v3.1.RF_fit.pkl"
>>>
>>>
>>> ## Import imputed genotype data
>>> mt_to_project = hl.import_bgen(bgen_path,
...                     entry_fields=['GT'],
...                     sample_file=sample_path,
...                     index_file_map={bgen_path: bgenidx_path})
2022-07-01 11:01:08 Hail: INFO: Number of BGEN files parsed: 1
2022-07-01 11:01:08 Hail: INFO: Number of samples in BGEN files: 78575
2022-07-01 11:01:08 Hail: INFO: Number of variants across all BGEN files: 75866
>>>
>>> import pickle
>>> from gnomad.sample_qc.ancestry import assign_population_pcs
>>>
>>> # Load MatrixTable for projection and gnomAD loadings Hail Table
>>> loadings_ht = hl.read_table(path_to_gnomad_loadings)
>>>
>>> # Project new genotypes onto loadings
>>> ht = hl.experimental.pc_project(
...     mt_to_project.GT,
...     loadings_ht.loadings,
...     loadings_ht.pca_af,
... )
2022-07-01 11:01:10 Hail: WARN: cols(): Resulting column table is sorted by 'col_key'.
    To preserve matrix table column order, first unkey columns with 'key_cols_by()'
>>>
>>>
>>> # Assign global ancestry using the gnomAD RF model and PC project scores
>>> # Loading of the v2 RF model requires an older version of scikit-learn, this can be installed using pip install -U scikit-learn==0.21.3
>>> with hl.hadoop_open(path_to_gnomad_rf, "rb") as f:
...     fit = pickle.load(f)
...
/jhpce/shared/jhpce/core/python/3.9.10/lib/python3.9/site-packages/sklearn/base.py:329: UserWarning: Trying to unpickle estimator DecisionTreeClassifier from version 0.23.2 when using version 1.0.2. This might lead to breaking code or invalid results. Use at your own risk. For more info please refer to:
https://scikit-learn.org/stable/modules/model_persistence.html#security-maintainability-limitations
  warnings.warn(
/jhpce/shared/jhpce/core/python/3.9.10/lib/python3.9/site-packages/sklearn/base.py:329: UserWarning: Trying to unpickle estimator RandomForestClassifier from version 0.23.2 when using version 1.0.2. This might lead to breaking code or invalid results. Use at your own risk. For more info please refer to:
https://scikit-learn.org/stable/modules/model_persistence.html#security-maintainability-limitations
  warnings.warn(
>>> # Reduce the scores to only those used in the RF model, this was 6 for v2 and 16 for v3.1
>>> num_pcs = fit.n_features_
/jhpce/shared/jhpce/core/python/3.9.10/lib/python3.9/site-packages/sklearn/utils/deprecation.py:103: FutureWarning: Attribute `n_features_` was deprecated in version 1.0 and will be removed in 1.2. Use `n_features_in_` instead.
  warnings.warn(msg, category=FutureWarning)
>>> ht = ht.annotate(scores=ht.scores[:num_pcs])
>>> htt, rf_model = assign_population_pcs(
...     ht,
...     pc_cols=ht.scores,
...     fit=fit,
... )
2022-07-01 11:04:02 Hail: INFO: Coerced sorted dataset            (12 + 2) / 16]
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/users/jzhang2/.local/lib/python3.9/site-packages/gnomad/sample_qc/ancestry.py", line 174, in assign_population_pcs
    num_out_cols = min([len(x) for x in pop_pc_pd["pca_scores"].values.tolist()])
  File "/users/jzhang2/.local/lib/python3.9/site-packages/gnomad/sample_qc/ancestry.py", line 174, in <listcomp>
    num_out_cols = min([len(x) for x in pop_pc_pd["pca_scores"].values.tolist()])
TypeError: object of type 'NoneType' has no len()
>>>
>>>

Hi @Jingning-Zhang Thank you for reaching out, and apologies for the long delay in responding. Unfortunately, I wasn't able to reproduce your error with our test dataset.
If you are still having trouble with this and would like assistance, I have some follow-up questions.

Were you trying to use the v2 or v3 loadings and RF?
What version of gnomad_methods were you using?

Additionally, we have updated that function since the blog post went out and this issue helped us notice that the current blog post code will not work with the updates. It doesn't look like that was the issue here though because the line indicated in your error num_out_cols = min([len(x) for x in pop_pc_pd["pca_scores"].values.tolist()]) no longer exists in the new code, so your error message would be different. I have put in a PR to modify the new code so the blog post example can work.
If you try this again with the newest version of gnomad_methods (v0.6.3), you would need to modify the code in the following way:
This:
htt, rf_model = assign_population_pcs(
    ht,
    pc_cols=ht.scores,
    fit=fit,
)

should change to:
htt, rf_model = assign_population_pcs(
    ht,
    pc_cols=range(num_pcs),
    fit=fit,
)

This should be fixed in the next release so the original blog post code can work as expected.
Thanks,
Julia

The newest release v0.6.4 has fixed the assign_population_pcs function so that it still works with the code in the blog post. Closing the issue since I think this is now fixed, but please reach out if it's still a problem.


App icon in manifest

Description of the new feature/enhancement
The icon of the app in question should be in the manifest. For example as base64 encoded png.
I believe this will greatly simplify and practicly enable visually beautiful winget-integrations in other systems such as Software Center, Company Portal and the new Windows 11 Store. Or any other third party "app store" for Windows.
Another solution could be to have the icons externally and have url to icon in the manifest. But given the relatively small file size of an icon, I would lean towards having it directly in the manifest for simplicity's sake.

Proposed technical implementation details (optional)
The packaging tools could take file path or url to png as parameters, convert to base64 and insert the string in the yaml.
To not bloat the repo too much, I suggest a resolution cap of 512x512. This cap could also be tested with winget validate.

Company Portal/Software Center/etc just grab an .ico file out of the installer, right? Any reason why they can't just keep doing that when bringing in an installer from a URL?
(I'm not trying to be down on your idea because icons are nice, it's just I'm afraid of bumping up the size of the manifests and having a big binary blob in there that we have to worry about moderating, because it's just a matter of time before someone puts something not kid friendly in the icon field).

Software Center can use the exe installer as source for the icon during packaging in ConfigMgr, yes. But then the extracted png/ico gets stored in the database as base64 as a property of the app. Company Portal needs you to supply the png yourself and upload it to intune.
The intent was in my mind to make the icon available before actually downloading the installer. Just like in any app store :+1:
I see your concern about the moderation, I did not think about that.

Related: https://github.com/microsoft/winget-cli/issues/1253 (and the place to post this feature idea)

Thanks @khaffner.  I agree with OfficialEsco.  Would you mind thumbs upping the related PR and adding your suggestions there?


Vimeo Player player.js not loading on mount in NextJS

Current Behavior
We have implemented a Vimeo React Player in our NextJS application, however we noticed the player behaving differently in our production environment. When navigating from a page without a video player to a page with a video player, player.js is not being requested which prevents several of our controls from functioning. We are seeing these warnings in the console when player.js hasn't been loaded:
ReactPlayer: Vimeo player could not call setPlaybackRate – The method was not available
ReactPlayer: Vimeo player could not call setQuality – The method was not available

We have a business account, so both of these methods are available to us via the SDK.
If we refresh the page, player.js is fetched and the controls function as we expect them to. Additionally, if we then navigate to another page with a video player, the controls do not work again until the page is refreshed.
Expected Behavior
player.js should be available as soon as the player is ready.
Steps to Reproduce

Go to https://mrionline.com/course/radiology-emergency-imaging-chest/
Click on any link with a video icon
Attempt to change the playback speed or quality
Refresh the page
Attempt to change the playback speed or quality again

It's working fine for me 🤔

We were able to implement a work around by adding a Next script element to our component and ensuring that player.js is fetched before trying to load the player.
<Script
    src="https://player.vimeo.com/api/player.js"
    onLoad={() => setSdkLoaded(true)}
/>

We're still not sure why this happened in the first place, when investigating we were not able to reproduce it in our development or test environments. Our test environment is set up to be identical to our production environment (running in a container that is started up with the same command). We haven't fully ruled out that it was something in our prod config, but we weren't able to pinpoint anything there or find anything in react player's, load-script's, or next's documentation that would explain why the script was fetched in the first place.


Nothing happens :-/

I just tried your extension for Alfred, hoping to be able to remember that my tea was done, but it didn't work, and my tea is ruined! Ruined I tell you, ruined!!
Well not really, but still, I can't get the extension to work. I type "timer 5:30" into alfred, it recognizes the alias but when I hit ENTER nothing happens. I tried making it non-silent and the terminal showed python: can't open file 'timer.py': [Errno 2] No such file or directory
Please let me know if I can help figuring this issue out :)

1.search noti in alfred
2.open notification switch
3.try again, and it works


Using golang 1.12+

If you are using golang 1.12 and up, which supports go modules, you don't need to have the project in your GOPATH.
Here are the steps that you can follow for trying out the project with golang 1.12+
git clone https://github.com/kubernetes/sample-controller.git
cd sample-controller/
go mod vendor
go build -o sample-controller .
./sample-controller -kubeconfig=$HOME/.kube/config

@devdattakulkarni Thanks! Closing since it'll be covered in https://github.com/kubernetes/kubernetes/pull/79081

@nikhita 👍


Fixed prediction_time being read as a string

Can we do this in __init__.py instead?

Should, to make it universal and not converted on-the-fly.

Would that go in this block? https://github.com/channemann/openaps-predict/blob/wip-bg_momentum/openapscontrib/predict/init.py#L109-L114

Or here? https://github.com/channemann/openaps-predict/blob/wip-bg_momentum/openapscontrib/predict/init.py#L150

The latter (line 150). Since the argparse type is Int, let's use int()
there too.
On Tue, Oct 27, 2015 at 2:55 PM Chris Hannemann<EMAIL_ADDRESS>wrote:

Or here?
https://github.com/channemann/openaps-predict/blob/wip-bg_momentum/openapscontrib/predict/__init__.py#L150
—
Reply to this email directly or view it on GitHub
https://github.com/loudnate/openaps-predict/pull/26#issuecomment-151657171
.


One-time events

Adding fromEventOnce(eventName) method to the socket service, to support one-time events and async/await operations.
e.g.
async () => {
const response = await this.socket.fromEventOnce("getResponse");
console.log(response);
}

okey thanks I will merged it after adding a test for it. thanks :)

thanks for the PR.


Bump rest-client version

Hope this is all it takes to resolve this https://github.com/simsicon/weibo_2/issues/20

Thank you for pulling this.


CSS

CSS
User Story
AS A marketing agency
I WANT a codebase that follows accessibility standards
SO THAT our own site is optimized for search engines
Acceptance Criteria
GIVEN I’m a marketing agency user
AND the codebase follows accessibility standards
WHEN a search engine trawls our website
THEN Application's CSS selectors and properties are consolidated and organized to follow semantic structure.
AND Application's CSS file is properly commented.
AND Application resembles (at least 90%) screenshots provided in the Challenge instructions.

CSS selectors and properties are consolidated and organized to follow semantic structure.
CSS file is properly commented.
Application resembles (at least 90%) screenshots provided in the Challenge instructions.


Submit/newm

Motivation for this change
Initially package the Wayland compositor newm and its abstraction layer pywm
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Is there anything holding this back? I've been keeping my eye on this for a while now, can't wait to see it get submitted :)

Closing this for now - going with a flakes-installation seems like a more straight-forward way.

OK then. I will pick this later.


Feature - Update Wiki page

Is your feature request related to a problem? Please describe.
Update Wiki page to illustrate the contribution of the task and meeting detail
Describe the solution you'd like
Update the Wiki using the md format
Additional context
Should be done by 26/09

Done. Update the Wiki page for release 2.


fix: update operation not working from codemirror

Signed-off-by: Abhishek-kumar09<EMAIL_ADDRESS>Description
Update-operation fix from the code-mirror window.
Submit handler is expecting type instead of action here.
handleSubmit( { data, id, name, type })

This PR fixes #
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Codecov Report

Merging #5398 (9996c9d) into master (84c477b) will decrease coverage by 4.31%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5398      +/-   ##
==========================================
- Coverage   17.47%   13.16%   -4.32%
==========================================
  Files         107      107
  Lines        9882     9882
==========================================
- Hits         1727     1301     -426
- Misses       7733     8304     +571
+ Partials      422      277     -145

Flag
Coverage Δ

gointegrationtests
?

unittests
13.16% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

mesheryctl/internal/cli/root/system/update.go
1.05% <0.00%> (-58.95%)
:arrow_down:

mesheryctl/internal/cli/root/system/start.go
1.88% <0.00%> (-55.19%)
:arrow_down:

mesheryctl/internal/cli/root/system/status.go
1.14% <0.00%> (-55.18%)
:arrow_down:

mesheryctl/internal/cli/root/system/restart.go
1.92% <0.00%> (-48.08%)
:arrow_down:

mesheryctl/internal/cli/root/system/stop.go
2.15% <0.00%> (-44.61%)
:arrow_down:

mesheryctl/internal/cli/root/system/logs.go
3.26% <0.00%> (-36.96%)
:arrow_down:

mesheryctl/internal/cli/root/system/check.go
1.03% <0.00%> (-23.03%)
:arrow_down:

mesheryctl/internal/cli/root/system/dashboard.go
1.70% <0.00%> (-9.41%)
:arrow_down:

mesheryctl/pkg/utils/healthcheck.go
8.82% <0.00%> (-4.42%)
:arrow_down:

Continue to review full report at Codecov.

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


[React] [Thirdweb v5] Account Switch Listener and useSiweAuth Hook Causing Broken State

Client Code:
<ConnectButton
  client={client}
  chain={chain}
  chains={[chain]}
  auth={{
    isLoggedIn: async (address) => {
      const loggedIn = await isLoggedIn(address);
      if (!loggedIn) {
        await logout();
      }
      return loggedIn;
    },
    doLogin: async (params) => {
      await login(params);
    },
    getLoginPayload: async ({ address }) => generatePayload({ address }),
    doLogout: async () => {
      await logout();
    },
  }}
/>

Backend Code:
const privateKey = env.THIRDWEB_ADMIN_PRIVATE_KEY;

if (!privateKey) {
  throw new Error('Missing THIRDWEB_ADMIN_PRIVATE_KEY in .env file.');
}

const thirdwebAuth = createAuth({
  domain: env.NEXT_PUBLIC_APP_URL || '',
  adminAccount: privateKeyToAccount({ client, privateKey }),
});

export const { generatePayload } = thirdwebAuth;

export async function login(payload: VerifyLoginPayloadParams) {
  const verifiedPayload = await thirdwebAuth.verifyPayload(payload);
  if (verifiedPayload.valid) {
    const jwt = await thirdwebAuth.generateJWT({
      payload: verifiedPayload.payload,
    });
    cookies().set('jwt', jwt);
  }
}

export async function isLoggedIn(address: string) {
  const jwt = cookies().get('jwt');
  if (!jwt?.value) {
    return false;
  }

  const authResult = await thirdwebAuth.verifyJWT({ jwt: jwt.value });
  if (!authResult.valid || authResult.parsedJWT.sub !== address) {
    return false;
  }
  return true;
}

export async function logout() {
  cookies().delete('jwt');
}

Problem Description:
When connecting using a wallet like MetaMask and switching between two accounts ( using the Metamask account switcher not the modal provided by ThirdWeb ), the application ends up in a broken state. Specifically, the ConnectButton indicates that the user is connected when, in fact, they actually need to sign in again to obtain a valid JWT. This inconsistency is caused by the account switch not properly triggering the authentication state to update, resulting in the UI reflecting an incorrect state.
Expected Behavior:
Upon switching accounts from a wallet extension, the ConnectButton should correctly prompt the user to sign in again to obtain a new JWT, ensuring the authentication state is accurately maintained.

was able to reproduce on: https://playground.thirdweb.com/connect/auth
@0xboga we'll take a look, this seems like a legit bug on our end. We'll get it fixed ASAP thanks for the report!

@0xboga this is fixed in 5.44.1 - please feel free to re-open if you can still reproduce it after upgrading


chore: release main

:robot: I have created a release beep boop
echo: 1.0.0
1.0.0 (2024-03-13)
Features

add simple echo protocol (#2439) (e1798aa)

multidim-interop: 2.0.22
2.0.22 (2024-03-13)
Dependencies

The following workspace dependencies were updated

dependencies

@libp2p/webrtc bumped from ^4.0.21 to ^4.0.22

webrtc: 4.0.22
4.0.22 (2024-03-13)
Documentation

update webrtc readme to show difference between transports (#2441) (fad3074)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/libp2p/js-libp2p/releases/tag/multidim-interop-v2.0.22 :sunflower:

:robot: Release is at https://github.com/libp2p/js-libp2p/releases/tag/echo-v1.0.0 :sunflower:

:robot: Release is at https://github.com/libp2p/js-libp2p/releases/tag/webrtc-v4.0.22 :sunflower:


fixed ctbs

Codecov Report

Merging #18 (91e7c2d) into main (e964202) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #18   +/-   ##
=======================================
  Coverage   25.13%   25.13%
=======================================
  Files           1        1
  Lines         187      187
=======================================
  Hits           47       47
  Misses        134      134
  Partials        6        6

Continue to review full report at Codecov.

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


Fix flaky TestHuberLoss

Fix #7696.
Repeated 100K times the tests without failures locally.

Jenkins & flexCI, test this please.

Jenkins CI test (for commit 7b62f43f0a286c2aa40abb3f383d7cbecc45f242, target branch master) succeeded!


Fix the import for fromEvent function.

- relates to https://github.com/ar90n/serverless-s3-local/issues/83

Hi @pratik12
Thanks for your great contribution! I merged it and released v0.5.4.
Please check it.


Set Kyuubi server thrift client socket timeout to inf

Why are the changes needed?

We can detect engine dead cases after https://github.com/apache/incubator-kyuubi/pull/2122, so set thrift client socket timeout to inf to avoid socket timeout in blocking case, e.g. retrieve data except to 1st partition in increment collection mode, spark scale mode.
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[x] Run test locally before make a pull request

update AllKyuubiConfigurationSuite


Terminate on retrun an error

When i use http or  tcp type and  return (err,res) where err is not null, service terminate!
Is it a normal behavior?
If yes, how we can return an error with err-res pattern?
 var Seneca = require('seneca');
 Seneca({log: 'test', timeout: '1000'})
        .use('seneca-transport')
        .use('mesh', {
            isbase: false,
            listen: [
                {pin: 'cat:test,cmd:run1', timeout: 1000, type: 'tcp'}
            ]
        })
        .add('cat:test,cmd:run1',function(msg,callback){
            callback(new Error('my error'));
        });

And here is act result that call from another service :

Seneca Fatal Error
==================
Message: seneca: Action cat:test,cmd:run1 failed: my error.
Code: act_execute
Details: { message: 'my error',
pattern: 'cat:test,cmd:run1',
instance: 'Seneca/fglbckoc6k4a/1479906487908/2721/3.2.2/-',
'orig$': {},
'message$': 'my error',
plugin: {} }
Stack:
at internals.Utils.handle_response (./node_modules/seneca-transport/lib/transport-utils.js:70:11)
at Duplex.messager._write (./node_modules/seneca-transport/lib/tcp.js:186:19)
...
...
SENECA TERMINATED at 2016-11-23T13:08:30.022Z. See above for error report.

@AdrianRossouw @mcollina @rjrodger @geek @darsee No answer?

I am personally not maintaining seneca. Sorry.

@AdrianRossouw  @rjrodger @geek @darsee No answer?

@mohammadrezaprw I'm also not involved in the maintenance of this repository.

Same here.

@mohammadrezaprw got it! this is now fixed- thanks for the bug report - this was a nasty one.
New release out shortly.

Hello, the issue still seems to exists.it worked fine with seneca-mesh 0.9.0. updating to seneca-mesh 0.10.0 has caused the same issue for me.  Also setting seneca,fixedargs.fatal = false is not helping


Fix metadata service block

Description of the Change
This fixes a potential bypass allowing users to get to the real AWS metadata service.

Pull Request Test Coverage Report for Build 2063

0 of 2 (0.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 24.519%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

vpc/bpf/filter/filter.go
0
1
0.0%

vpc/setup/setup_linux.go
0
1
0.0%

Totals

Change from base Build 2023:
0.0%

Covered Lines:
2433

Relevant Lines:
9923

💛  - Coveralls

Waiting on @rgulewich's changes before this will be merged / pass.


Improve console output

Includes:

show a summary as the last line e.g. >> 0 failures, 5 warnings

uses tabs to list the issues to make the output easier to read if the line numbers are longer
old
 1: [W] SelectorFormat:  ...
 12: [W] PropertySortOrder: ...

new
 1:	[W] SelectorFormat:  ...
 12:	[W] PropertySortOrder: ...

Thanks for this. I'd rather have spaces than tabs, because of display problems with how terminals interrupt a tab size.

Can you fix the tests?

Any news?

Any news?


Global roles vs Organisation roles

So far roles have been defined in a global way, through a top level claim. However, their format of caseworker-<orgid>-... implies they should be organisation specific rather than global. This would more adequately fit a multi-tenant system.
What about QuickCase reserved roles for functions like definition import?

Pushing back to milestone 1.1.0 as organisation-level are not a requested feature at this time.

Pushing back to milestone 1.1.0 as organisation-level are not a requested feature at this time.

Outcome of internal discussions - 2021-01-27
QuickCase current implementation relies on global roles defined as part of user metadata. However, when consumed to be applied, filtering is applied to filter out roles in the form caseworker-<orgId>* for which the organisation identified differs from the current organisation. In effect, this makes these roles organisation-specific. Roles not following the caseworker- pattern however are used and applied regardless of organisation, making them global.
By being implicit, the rule for selecting relevant roles is non-intuitive and favours mistakes in access control configuration. In addition, in a multi-tenant context, non-namespaced global roles could result in conflicts across tenants with the risk of access being granted beyond the tenant's organisation.
As such, the preferred route is to phase out global roles and entirely replace them by organisation-specific roles instead. As this is a breaking change, a transition period should be respected with both deprecated global roles and organisation roles being available and supported in parallel.

Outcome of internal discussions - 2021-01-27
QuickCase current implementation relies on global roles defined as part of user metadata. However, when consumed to be applied, filtering is applied to filter out roles in the form caseworker-<orgId>* for which the organisation identified differs from the current organisation. In effect, this makes these roles organisation-specific. Roles not following the caseworker- pattern however are used and applied regardless of organisation, making them global.
By being implicit, the rule for selecting relevant roles is non-intuitive and favours mistakes in access control configuration. In addition, in a multi-tenant context, non-namespaced global roles could result in conflicts across tenants with the risk of access being granted beyond the tenant's organisation.
As such, the preferred route is to phase out global roles and entirely replace them by organisation-specific roles instead. As this is a breaking change, a transition period should be respected with both deprecated global roles and organisation roles being available and supported in parallel.


pretrained model

Hi can you provide a pre trained model?

Hello this is really good repository
But I am struggling when I set it up and download data

I've used Google colab for the training process. It's worked well for me.

Hi can you provide a pre trained model?
Hi, I've added them to the pre-trained models folder. Please check.

Hi can you provide a pre trained model?
Hi, I've added them to the pre-trained models folder. Please check.

@sanjay235 Cool thanks. But I was able to train your code and get the weights. It worked great. Thanks for the awesome tutorial!