401 Invalid_token The issuer is invalid - Asp.Net Core 3.1, Angular, Identity Server 4, Azure App Service, Linux

Question
I have created a web app https://www.alpacabiriba.com using the Visual Studio 2019 Template (Create New Project | Asp.Net Core Web Application | Angular + Inidividual User Accounts). I published this to an Azure Linux App Service using the VS2019 wizzards.
It works fine in Azure except when I click on Fetch Data when I get WWW-Authenticate: Bearer error="invalid_token", error_description="The issuer 'https://www.alpacabiriba.com' is invalid"
Please could some kind person help?
Minimal working example
I changed as little as I could to try & get it working before I went on to change it to do Biriba things ...

I changed it to load the p12 SSL certificate from /var/ssl/private the Azure App Service put it in. This included my attempt to inline the AddApiAuthorization extension method to get round it insisting on accessing the certificate key from the appsettings.json rather than /var/ssl/private.

I added some code to void Startup.Configure(IApplicationBuilder app, IWebHostEnvironment env) to apply database migrations when run in Azure.

var identityServicesServicesColection = services.AddIdentityServer();
identityServicesServicesColection
  .AddAspNetIdentity < ApplicationUser > ()
  .AddOperationalStore < ApplicationDbContext > ()
  //.ConfigureReplacedServices()
  .AddIdentityResources()
  .AddApiResources()
  .AddClients();
var environment = Environment.GetEnvironmentVariable("ASPNETCORE_ENVIRONMENT");
if (environment == Environments.Development)
{
  identityServicesServicesColection.AddSigningCredentials();
}
else
{
  var p12 = LoadP12Certificate();
  identityServicesServicesColection.AddSigningCredential(p12);
}

Relevant parts of the log file
WWW-Authenticate: Bearer error="invalid_token", error_description="The issuer 'https://www.alpacabiriba.com' is invalid"

The template comes from Microsoft. Please ask them over in the ASP.NET Core repo.

The template comes from Microsoft. Please ask them over in the ASP.NET Core repo.

Thanks @leastprivilege (Dominick). Do you know of  another template that works when deployed to a production environment?

I'm not 100% sure but I think I had the exact same problem deploying on a Linux Azure Webapp. The solution was setting the PublicOrigin-property with the correct domain origin. But if you are using the latest version (IdentityServer4 4.0+) that property has now been removed. I haven't yet tried to upgrade to the latest version (still using 3.1.3) and tried it out yet, but I suspect this will be a breaking change for my environment.

Thanks Mohammad (@imt-s1897). I set the PublicOrigin as follows bit sadly with no luck. If you could share your StartUp class of one that works when deployed to Azure I'd be immensley grateful.
            services.AddIdentityServer(options =>                     options.PublicOrigin = "https://alpacabiriba.com";             })

https://alpacabiriba.com

It that the place where IdentityServer4 is deployed? When one tries to access the domain you provided you get the welcome screen for Azure deployment. It's not under a subdomain? In my startup the PublicOrigin-property is for example set to https://login.my-domain.com.
Have you tried the templates provided by IdentityServer4? Just to see if you can get it running. You can always add a Frontend-project later on.
https://github.com/IdentityServer/IdentityServer4.Templates

Apologies Mohammad . A temporary unrelated deployment problem. It's back up again now. Yes it's deployed to https://alpacabiriba.com.
Is the "login" subdomain something that is needed by Identity Server 4 or something you've decided to use in your specific application?
I also tried https://www.alpacabiriba.com without luck. The razor pages work fine (login, register, amend user details) - it's the angular component calling the weather forecast web-api that doesn't work.
Thanks for the suggestion about using the templates provided by IdentityServer4 - I'll give them a go.

Is the "login" subdomain something that is needed by Identity Server 4 or something you've decided to use in your specific application?

No, not needed.

I also tried https://www.alpacabiriba.com without luck. The razor pages work fine (login, register, amend user details) - it's the angular component calling the weather forecast web-api that doesn't work.

So maybe it's not a PublicOrigin-issue in Azure Linux webb app environments, and instead something specific to the template used. You could head over to the ASP.NET Core repo as suggested by Dominick.

I've got it working. I found an info message in the logs saying "Failed to determine the https port for redirect."
and set ASPNETCORE_HTTPS_PORT to 5001 in the App Service configuration. Thanks for your help Mohammad.

Failed to determine the https port for redirect."
and set ASPNETCORE_HTTPS_PORT to 500

Could you please try it without the PublicOrigin-property being set and see if that in fact made a difference?

Hi, I've been experiencing this problem for a long time. But I'm using blazor, in this case.
I tried this configuration: ASPNETCORE_HTTPS_PORT: 5001, but there was no effect (which I think is strange, because there is no port configured in azure).


crash on plutovg_canvas_add_text

I'm getting a crash on the final line below. I am loading a valid OTF font which I have tested with STB's TTF engine many times over the years, after modifying it to stream. (My forked header is 95% compatible with the original, is tested as part of the LVGL test suite)
plutovg_font_face_t* face = plutovg_font_face_load_from_stream(*info.ttf_font,(int)info.ttf_font_face,nullptr,nullptr);
if(face==nullptr) {
    return gfx_result::invalid_format; // TODO: might be out of memory
}
plutovg_canvas_set_font_face(CHND,face);
plutovg_canvas_add_text(CHND,info.text,info.text_byte_count,info.encoding,location.x,location.y);

Curiously, this function isn't getting past the if(face->glyphs{msb][lsb]) which evaluates to true without ever running the code further down below the if:
static glyph_t* get_glyph(const plutovg_font_face_t* face, plutovg_codepoint_t codepoint)
{
    unsigned int msb = (codepoint >> 8) & 0xFF;
    if(face->glyphs[msb] == NULL) {
        ((plutovg_font_face_t*)face)->glyphs[msb] = (glyph_t**)calloc(GLYPH_CACHE_SIZE, sizeof(glyph_t*));
    }

    unsigned int lsb = codepoint & 0xFF;
    if(face->glyphs[msb][lsb]) {
        return face->glyphs[msb][lsb];
    }

    glyph_t* glyph = (glyph_t*)malloc(sizeof(glyph_t));
    glyph->index = stbtt_FindGlyphIndex(&face->info, codepoint);
    glyph->nvertices = stbtt_GetGlyphShape(&face->info, glyph->index, &glyph->vertices);
    stbtt_GetGlyphHMetrics(&face->info, glyph->index, &glyph->advance_width, &glyph->left_side_bearing);
    if(!stbtt_GetGlyphBox(&face->info, glyph->index, &glyph->x1, &glyph->y1, &glyph->x2, &glyph->y2))
        glyph->x1 = glyph->y1 = glyph->x2 = glyph->y2 = 0.f;
    return (face->glyphs[msb][lsb] = glyph);
}

Am I doing something wrong?

The code itself looks fine, and I don't see anything immediately wrong with it. It's possible that the issue might be happening elsewhere, potentially causing a segmentation fault or another problem before the function is called. I would recommend using a debugger to step through and identify if there's a memory-related issue or if something unexpected is happening just before this section of the code.

I am in a debugger, and it looks the glyph is never getting initialized. That code after the if NEVER runs, as I said. And face-glyphs[msb][lsb] seems to hold a null but it's hard to tell since it's multi-dimmed and i forget how C/C++ lay those out in memory:
This is what I'm seeing

When I get into get_glyph() things get suspicious: It calloc's so obv it's the first iteration

But then it goes here. Always here.

Here's some debug at the point in the last screen grab:

It almost feels like it's walking off the edge, like the calloc wasn't big enough. Any ideas?

Thanks for sharing all the details! It looks like the issue might be with the size of the cache. For this function to work correctly, GLYPH_CACHE_SIZE must be 256. Could you check if that's the case in your code? Let me know if this helps!

It's 16. I may have lowered it. It could be a problem on embedded systems.

From: Samuel Ugochukwu @.>
Sent: Friday, October 11, 2024 8:29 AM
To: sammycage/plutovg @.>
Cc: honey the codewitch @.>; Author @.>
Subject: Re: [sammycage/plutovg] crash on plutovg_canvas_add_text (Issue #42)
Thanks for sharing all the details! It looks like the issue might be with the size of the cache. For this function to work correctly, GLYPH_CACHE_SIZE must be 256. Could you check if that's the case in your code? Let me know if this helps!
—
Reply to this email directly, view it on GitHubhttps://github.com/sammycage/plutovg/issues/42#issuecomment-2407652771, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AD5ICVO7RFGG7X5XYSXMBRLZ27VEBAVCNFSM6AAAAABPYYDKB2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMBXGY2TENZXGE.
You are receiving this because you authored the thread.Message ID: @.***>

If you change it to 16, that means the algorithm in the get_glyph function will need to be adjusted accordingly. You might want to consider using a map to handle the glyph caching more dynamically in this case.

Well I changed that to 256, and now the issue is it's looping forever traversing the glyph paths. It could be a bug in my STB code if you're using functions I haven't exercised but I don't know enough about your glyph traversal function to figure out why it's hanging. Like, where.
I just checked in the hanging version


[5.4] Adds ability to test raw content in post, put and patch requests

Adds optional $content variables to post, put, and patch so raw data can be passed through to the Symfony Request during integration testing.
This is a 5.4 pull request of https://github.com/laravel/framework/pull/17130

I'm trying to avoid adding parameters to these simple get, put, post methods. I prefer people just use call if they need more customization than these offer. Thanks.


#isAbsolute()

Method to indicate absolute paths:
https://nodejs.org/api/path.html#path_path_isabsolute_path

isAbsolute() is not needed if we have #35


?via parameters aren't preserved in "Accept invite" button

None of the accept/copy invite buttons keep the ?via parameters, which makes the whole thing rather useless (you won't be able to join the room unless your server already knows about it)
e.g. going to https://matrix.to/#/!ekJXCOhsmAWQUMhDgn:riot.ovh?via=riot.ovh&via=matrix.org&via=kaplu.mba, the Element button links to https://app.element.io/#/room/!ekJXCOhsmAWQUMhDgn:riot.ovh

Yeah, that's a serious one. I'll prioritise this


RangeError in /clan/d-e-s-t-n-y-uju2vgc0

Error in ClashLeaders
RangeError in /clan/d-e-s-t-n-y-uju2vgc0
Maximum call stack size exceeded
View on Bugsnag
Stacktrace
webpack:///node_modules/vue/dist/vue.esm.js:520 - isServerRendering

View full stacktrace
Created automatically via Bugsnag

An error linked to this issue has been marked as fixed in Bugsnag
RangeError in /clan/htt2-l0jpcuv


Build API test generator with MVVM

Built first version of test generator with core MVVM components
Baseline test

I have finished my pass of the PR.

PR created for GAX related change: https://github.com/googleapis/gax-java/pull/92

PTAL

PTAL

LGTM


Add interface flux calculation in arbitrary directions for curved mesh 3D

The calculation of the interface flux is now implemented for arbitrary directions for Euler (FluxRotated) and LSA.

As discussed in #532, we added tests for free stream preservation.

As discussed in #532, we added tests for free stream preservation.

And they work now? Do you also have a convergence test on the curved mesh with expected EOC?

We now have L2 errors under 2e-14 and Linf under 8e-13 for the free stream preservation tests.
We added a convergence test for 2D here, similar to the TreeMesh test.
Should we add a 3D test as well? There is none for TreeMesh, though.
https://github.com/trixi-framework/Trixi.jl/blob/0c3dd3e3bdf6bd0049c7c5581b780c5954b43008/test/test_special_elixirs.jl#L19-L26

Should we add a 3D test as well? There is none for TreeMesh, though.

Yes, I think that would be useful. In contrast to the TreeMesh, 3D curved is qualitatively different from 2D, thus a convergence test is very useful (especially if we keep tinkering with the implementation, add new features etc.)


Running demo installer results in a stacktrace

Downloaded and ran the demo installer on Windows 10. Result is:
java.lang.NullPointerException
at no.tornado.FxlDemo.start(FxlDemo.java:31)
at fxlauncher.Launcher.lambda$launchAppFromManifest$2(Launcher.java:104)
at com.sun.javafx.application.PlatformImpl.lambda$null$173(PlatformImpl.java:295)
at java.security.AccessController.doPrivileged(Native Method)
at com.sun.javafx.application.PlatformImpl.lambda$runLater$174(PlatformImpl.java:294)
at com.sun.glass.ui.InvokeLaterDispatcher$Future.run(InvokeLaterDispatcher.java:95)
at com.sun.glass.ui.win.WinApplication._runLoop(Native Method)
at com.sun.glass.ui.win.WinApplication.lambda$null$148(WinApplication.java:191)
at java.lang.Thread.run(Thread.java:745)

Thanks, I have fixed the Windows installer now, will fix the Mac and Linux installers shortly. Can you confirm it is working for you now? You need to uninstall the old and download/reinstall.
The reason for the error is that I changed the FXLauncher version in the deployment, but didn't update the installer. There were some major changes a few versions back, so the installer also needed updating.

Linux installer updated as well, only Mac installer left.

Yes, that works now. Thanks!

Thanks for reporting it :) Mac installer is also updated.


Make reason for Mig* resource unreadiness more obvious for MigPlan and MigMigration

Copied over from 1003 since this is a separate issue.

Problem
When Mig* resource goes unready, the reason for unreadiness isn't exposed. User has to dig through the related resources to find out what stopped the migration.
Suggested solution with MigPlan as example
If MigPlan status is unready, show specific problem e.g. "Plan is unready due to Replication Repo xyz being unready.", or equivalent warning for Clusters being unready. This can be accomplished by checking MigPlan for conditions

SourceClusterNotReady
DestinationClusterNotReady
StorageNotReady

User should be pointed to this specific information instead of generic "Plan Unready" when this more specific info is available.

Full list of condition messages for migplan:
https://github.com/konveyor/mig-controller/blob/master/pkg/controller/migplan/validation.go#L29
Full list of condition messages for migmigration:
https://github.com/konveyor/mig-controller/blob/master/pkg/controller/migmigration/validation.go#L46


Applying custom functions on the data before parsing

I want to apply custom functions to clean the data before spark-csv starts to parse the data. Is there a way to do that with current versions?

@Sree2015 Maybe, you could use CsvParser.csvRdd(...) API. After loading it into RDD[String] and cleaning up the data, then you could use this API to load it into DataFrame.

CsvParser.csvRdd calls CsvRelation which parses the file. I want to do the cleaning before parsing.

Could I please ask what you mean by "custom functions to clean the data"? I thought you meant you want some transformation before converting it into DataFrame.

For example for decimals we get signs at the end as they are treated as strings. I will have to write a custom function to move the sign to the beginning such that it can be parsed while being loaded into data frame with custom schema.

I suggest you load that field as string, and then apply a simple transformation (or UDF) to get a new DataFrame (or table) with desired column.

In this case i will have to use inferSchema option and then apply UDFs for specific columns and then load into another dataframe with custom schema. Is this right?

Yes. Or you can do everything with DataFrames to avoid the last step.

I would want to do everything within the same data frame, however the issue is if the data is not transformed before applying the custom schema, it fails parsing. Is there a way to overcome this and still do in single dataframe?


Collect service information for Spring Boot applications

Service information (loaded libraries) are not collected for Spring Boot applications except for cases where they are packaged as war files and are loaded on a container (such as Tomcat).
A lot of Spring Boot applications are run as an executable jar (or war), so it would be useful to collect these information in these cases as well.

+1, nice feature. as many projects prefer spring boot for integration.


[BEAM-3425] Get the range start & end as String

Follow this checklist to help us incorporate your contribution quickly and easily:

[X] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[X] Each commit in the pull request should have a meaningful subject line and body.
[X] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.
[X] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[X] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[X] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Hi @jbonofre! Do you know what is the state of spliting at the moment? I see it is still the same in master, do you need any help with that?

@kubum This PR was closed and there is another PR #5124 which is supposed to fix these 3 CassandraIO issues. The work is in progress but we expect that it will be merged soon.


Will this API be unavailable in 2020

To continuously improve Instagram users' privacy and security, we are accelerating the deprecation of Instagram API Platform, making the following changes effective immediately. We understand that this may affect your business or services, and we appreciate your support in keeping our platform secure.
These capabilities will be disabled immediately (previously set for July 31, 2018 or December 11, 2018 deprecation). The following will be deprecated according to the timeline we shared previously:
Public Content - all remaining capabilities to read public media on a user's behalf on December 11, 2018
Basic - to read a user's own profile info and media in early 2020
For your reference, information on the new Instagram Graph API.
Will this API be unavailable in 2020

This is for official Instagram API,
this library using unofficial Instagram API. this api extracted from instagram app


⚠ GitLab Server has degraded performance

In a58084b, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 11519 ms

Resolved: GitLab Server performance has improved in 447a572.


[security]: automatically add nonce or hash to script-src-elem, style-src-attr & style-src-elem csp directive if necessary

This PR adds functionality for automatically adding CSP-directives that are required only if they're used:

script-src-elem
style-src-attr
style-src-elem

error thrown because of missing nonce/hash when script-src-elem is used:

error thrown because of missing nonce/hash if style-src-attr and/or style-src-elem is used:

style-src-attr and style-src-elem is also automatically added during dev so things work locally.
Please don't delete this checklist! Before submitting the PR, please make sure you do the following:

[ ] 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

[x] 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:.

thank you!


Sony X1 - snap 855 - error while Run axie_tflite

Hi,
When I try to run the test, I get this error and then my phone stops working (I have to force reboot it). After searching for relevant information but finding nothing, I don't know what to do next.
Thanks for your time.
 255|SOV40:/ $ axie_tflite /audio/output_audio_mono.wav tiny
Compiled Oct 31 202419:35:52
AXIE TFLite Runner
Argmax, Inc.

lib dir: /data/local/tmp/lib
cache dir: /data/local/tmp/cache
INFO: Initialized TensorFlow Lite runtime.
WARNING: [QNN Delegate] HTP precision is set as kHtpFp16 but the platform doesn't support kHtpFp16. Ignored the setting.
INFO: [QNN Delegate] Caching: cache_filename of /data/local/tmp/cache/qnn_binary_6604284145099505954.bin isn't available, caching in SAVE MODE.
ERROR: Failed to get HTP platform info. Error 1100
ERROR: Restored original execution plan after delegate application failure.

Thanks for the report @nongthanhhong! We are currently in alpha release so I salute your enthusiasm for testing our alpha codebase. 🫡  As part of building up to the beta release, we are actively extending our testing device coverage. Could you please share a link to the product page for your device?(I assume you are talking about Sony Xperia 1 but want to make sure)

Our current Snapdragon test coverage includes devices with:

Snapdragon 888
Snapdragon 8 Gen 1
Snapdragon 8 Gen 2
Snapdragon 8 Gen 3
so 855 will be a good addition

cc: @keith4ever @a2they  for visibility

@atiorh Sorry for the confusion. Yes, it's Sony Xperia 1 mark 1 . My device is running on Android 11, and although I've checked that the Snapdragon 855 is supported by QAI, maybe it needs some extra steps in my case?

Hello @nongthanhhong , thanks for your interest.
Thanks for the link, and it seems your phone has SM8150 architecture which is not currently supported by QCOM's AI SDK.
We're currently working with key stake holders to possibly support as many AP as possible, so please check back here, or get a more modern AP. These are currently supported via HTP (NPU) or GPU:
SM8650
SM8550
SM8450
SM8350

@keith4ever Thank you for sharing your hard work. I will try again with a newer, supported device.

Hi, @keith4ever, just come back here with S23u - SM8550. on 8m length audio - I got a well correct transcript and:
Average model latency:
Audio Input:   108.710 ms
Melspectro:    76.060 ms
Encoder:       326.060 ms
Decoder:       17.680 ms
Postproc:      1.080 ms
Total Duration:  48382.320 ms
, but checking the log seems like it failed to delegate on NPU/GPU. How can I check where I'm wrong?
Thank you and have a good day 😊
LOG:
AXIE TFLite Runner
Argmax, Inc.
lib dir: /data/local/tmp/lib
cache dir: /data/local/tmp/cache
Creating cache directory: /data/local/tmp/cache
INFO: Initialized TensorFlow Lite runtime.
INFO: [QNN Delegate] Caching: cache_filename of /data/local/tmp/cache/qnn_binary_8316569388647923909.bin isn't available, caching in SAVE MODE.
INFO: Wrote serialized data for model mel_spectrogram2.20.0 (88 B) to /data/local/tmp/cache/mel_spectrogram2.20.0_4183742775783677416.bin
INFO: TfLiteQnnDelegate delegate: 21 nodes delegated out of 21 nodes with 1 partitions.
VERBOSE: Replacing 21 out of 21 node(s) with delegate (TfLiteQnnDelegate) node, yielding 1 partitions for the whole graph.
INFO: Wrote serialized data for model mel_spectrogram2.20.0 (20 B) to /data/local/tmp/cache/mel_spectrogram2.20.0_8577268126971921961.bin
INFO: [QNN Delegate] Caching: cache_filename of /data/local/tmp/cache/qnn_binary_12912538988619984752.bin isn't available, caching in SAVE MODE.
INFO: Wrote serialized data for model whisper_encoder_base2.20.0 (1680 B) to /data/local/tmp/cache/whisper_encoder_base2.20.0_4809695520788379349.bin
VERBOSE: Replacing 419 out of 419 node(s) with delegate (TfLiteQnnDelegate) node, yielding 1 partitions for the whole graph.
INFO: Wrote serialized data for model whisper_encoder_base2.20.0 (20 B) to /data/local/tmp/cache/whisper_encoder_base2.20.0_14245224574025941496.bin
INFO: [QNN Delegate] Caching: cache_filename of /data/local/tmp/cache/qnn_binary_792083847501657808.bin isn't available, caching in SAVE MODE.
INFO: Wrote serialized data for model whisper_decoder_base2.20.0 (3936 B) to /data/local/tmp/cache/whisper_decoder_base2.20.0_14718706179135460764.bin
VERBOSE: Replacing 983 out of 983 node(s) with delegate (TfLiteQnnDelegate) node, yielding 1 partitions for the whole graph.
INFO: Wrote serialized data for model whisper_decoder_base2.20.0 (20 B) to /data/local/tmp/cache/whisper_decoder_base2.20.0_15366836946474653222.bin
INFO: [QNN Delegate] Caching: cache_filename of /data/local/tmp/cache/qnn_binary_18177980569190481050.bin isn't available, caching in SAVE MODE.
INFO: Wrote serialized data for model audio_input2.20.0 (24 B) to /data/local/tmp/cache/audio_input2.20.0_8184959232207587746.bin
VERBOSE: Replacing 5 out of 5 node(s) with delegate (TfLiteQnnDelegate) node, yielding 1 partitions for the whole graph.
INFO: Wrote serialized data for model audio_input2.20.0 (20 B) to /data/local/tmp/cache/audio_input2.20.0_13172141440965276315.bin
postproc vocab size: 51864
INFO: Created TensorFlow Lite delegate for GPU.
INFO: Created TensorFlow Lite XNNPACK delegate for CPU.
VERBOSE: Replacing 27 out of 42 node(s) with delegate (TfLiteXNNPackDelegate) node, yielding 12 partitions for the whole graph.
ERROR: Following operations are not supported by GPU delegate:
DELEGATE TfLiteXNNPackDelegate: Operation is not supported.
LOG_SOFTMAX: Operation is not supported.
SCATTER_ND: Operation is not supported.
ZEROS_LIKE: Operation is not supported.
3 operations will run on the GPU, and the remaining 18 operations will run on the CPU.
VERBOSE: Replacing 3 out of 21 node(s) with delegate (TfLiteGpuDelegateV2) node, yielding 3 partitions for the whole graph.
ERROR: TfLiteGpuDelegate Init: Tensor "model/tf.math.reduce_logsumexp/ReduceLogSumExp/Log" has bad input dims size: 0.
INFO: Created 0 GPU delegate kernels.
ERROR: TfLiteGpuDelegate Prepare: delegate is not initialized
ERROR: Node number 48 (TfLiteGpuDelegateV2) failed to prepare.
ERROR: Restored original execution plan after delegate application failure.
Falling back to default TFLite GPU backend..tflite_init done..

@nongthanhhong,
I'm glad you got it working.
The 3 errors are from this line of code, I believe:
https://github.com/argmaxinc/WhisperKitAndroid/blob/main/src/tflite_runner.cpp#L84
which is not a concern, as it may have delegated to another computing resource (NPU or CPU) and therefore you got the right result.
If you want to try again with NPU (HTP for QCOM), replace the line with this:
    TFLITE_INIT_CHECK(postproc->initialize(postproc_model, lib_dir, cache_dir, kHtpBackend));

@keith4ever, thanks for your time. Can I utilize and integrate this repo as part of the Android app but not just run it as an execution? Are there any sharable documents from our team about using the built .so library, cpp native code, etc in an Android app through JNI?
Even not, I still really appreciate your and your team's efforts. Looking forward to hearing from you soon 😄

Hi @nongthanhhong ,
You're welcome to develop as it suits your needs.. absolutely you can develop JNI layer around this.
As we speak, we are also developing JNI layer + sample Android app but we're not yet 100% certain if we can include it in v0.1 Beta release.


Add German translation

As the title says.
I also made Yannick's translation actually a French translation - or should it be fr_CH? I am half French myself and I couldn't find any Swiss specialities in his translation. That's why I used fr and not fr_CH.

Well spotted for the mistake in fr.po! Can you tell I've been copying and pasting stuff manually? :P
Can you please update the PR by adding de to the list of supported locales in lib/src/app.dart, around line 52?
Thanks in advance!

Can you tell I've been copying and pasting stuff manually?

Nooo, not at all! I would never do such things :innocent:
I rebased my PR and added de to the file!
Ready to merge as far as I am concerned.

Thank you!!


Add a query string to bust cache on deploys.

Changes
Adds a query string to CSS files (e.g. app.css?v1.0.32) so that they properly invalidate cache on deploys. References #4867.
For review: @sergii-tkachenko

:+1:


Add slight shimmer to the path the mobs will take to reach the destination

Helps the player to visualize the maze

Completed, can currently be toggled using "ui_left" and "ui_right" (left and right arrow keys).


yiiunit\framework\mutex\RetryAcquireTraitTest::testRetryAcquire test fail by ci.appveyor.com

What steps will reproduce the problem?
https://ci.appveyor.com/project/samdark/yii2/builds/26985116#L161
What is the expected result?
accept 49.97
What do you get instead?
1) yiiunit\framework\mutex\RetryAcquireTraitTest::testRetryAcquire
Failed asserting that 49.96800422668457 is equal to 50 or is greater than 50.

Additional info
Maybe the float vs integer is the problem.

Q
A

Yii version
2.0.26-dev

PHP version
?

Operating system
windows

That is an indication that it fails with certain conditions. Any idea on what conditions these may be?

At first glance, it cannot be less than 50.0, unless it is time traveler.

That is the issue with concurrency issues. They are very similar to time travelers.

There is a bigger difference:
1) yiiunit\framework\mutex\RetryAcquireTraitTest::testRetryAcquire
Failed asserting that 48.61307144165039 is equal to 50 or is greater than 50.
C:\projects\yii2\tests\framework\mutex\RetryAcquireTraitTest.php:45

I don't think it is correct to ignore it in tests. I've reproduced it. Trying to understand what actually happens.

OMG, seems windows usleep() precision is awful: https://ci.appveyor.com/project/samdark/yii2/builds/27850232. It finished 2ms earlier than specified.

Aha. That is normal on Windows: https://docs.microsoft.com/en-us/windows/win32/api/synchapi/nf-synchapi-sleep?redirectedfrom=MSDN

If dwMilliseconds is less than the resolution of the system clock, the thread may sleep for less than the specified length of time.


Issue when running the Unsupervised Segmentation when fitting the NP Bayesian Models.

Convert data to list by ID
dat.list <- dat.disc %>%

df_to_list(., "id")

Only retain id and discretized data streams
dat.list.sub <- map(dat.list, subset, select = c(id, SL, TA, Disp))
Convert the id column to integer for each data frame in the list
dat.list.sub <- map(dat.list.sub, ~ .x %>% mutate(id = as.integer(id)))
Check the structure to confirm the change
glimpse(dat.list.sub)
List of 3
$ 6009:'data.frame':	1174 obs. of  4 variables:
..$ id  : int [1:1174] 6009 6009 6009 6009 6009 6009 6009 6009 6009 6009 ...
..$ SL  : int [1:1174] 1 2 2 2 5 5 2 5 4 4 ...
..$ TA  : int [1:1174] 3 3 3 2 2 2 4 3 3 3 ...
..$ Disp: int [1:1174] 1 1 1 1 1 2 2 2 2 1 ...
$ 6011:'data.frame':	1174 obs. of  4 variables:
..$ id  : int [1:1174] 6011 6011 6011 6011 6011 6011 6011 6011 6011 6011 ...
..$ SL  : int [1:1174] 2 1 3 3 5 2 1 2 2 1 ...
..$ TA  : int [1:1174] 3 3 3 3 3 1 1 3 3 3 ...
..$ Disp: int [1:1174] 1 1 1 1 1 1 1 1 1 1 ...
$ 6018:'data.frame':	1175 obs. of  4 variables:
..$ id  : int [1:1175] 6018 6018 6018 6018 6018 6018 6018 6018 6018 6018 ...
..$ SL  : int [1:1175] 1 2 3 5 2 1 2 2 2 1 ...
..$ TA  : int [1:1175] 3 3 3 3 3 1 4 3 4 3 ...
..$ Disp: int [1:1175] 1 1 1 1 1 1 1 1 1 1 ...
Run the segmentation model (unsupervised)
set.seed(123)
alpha <- 1  # hyperparameter for prior (Dirichlet) distribution
ngibbs <- 10000  # number of iterations for Gibbs sampler
nbins <- c(3, 6, 8)  # define number of bins per data stream (SL, TA, Disp)
progressr::handlers(progressr::handler_progress(clear = FALSE))  # to initialize progress bar
Set the number of workers to a minimum of 1
n_workers <- max(1, availableCores() - 2)
future::plan(multisession, workers = n_workers)
dat.res.seg1 <- segment_behavior(data = dat.list.sub, ngibbs = ngibbs, nbins = nbins, alpha = alpha)
\ [............................................................................]   0% :message
Progress interrupted by FutureError condition: MultisessionFuture () failed to receive message results from cluster RichSOCKnode #1 (PID 14356 on localhost ‘localhost’). The reason reported was ‘error reading from connection’. Post-mortem diagnostic: No process exists with this PID, i.e. the localhost worker is no longer alive. The total size of the 16 globals exported is 112.61 KiB. The three largest globals are ‘...furrr_fn’ (68.24 KiB of class ‘function’), ‘...furrr_chunk_args’ (18.62 KiB of class ‘list’) and ‘behav_gibbs_sampler’ (11.89 KiB of class ‘function’)
Error in unserialize(node$con) :
MultisessionFuture () failed to receive message results from cluster RichSOCKnode #1 (PID 14356 on localhost ‘localhost’). The reason reported was ‘error reading from connection’. Post-mortem diagnostic: No process exists with this PID, i.e. the localhost worker is no longer alive. The total size of the 16 globals exported is 112.61 KiB. The three largest globals are ‘...furrr_fn’ (68.24 KiB of class ‘function’), ‘...furrr_chunk_args’ (18.62 KiB of class ‘list’) and ‘behav_gibbs_sampler’ (11.89 KiB of class ‘function’)

[*******************************************************o............................]  67%

I tried the example code for running this model from the 04_Fit NP Bayesian models in bayesmove.R script and didn't receive an error. Since I'm not sure what dataset you're using and how you've discretized the data streams (since this doesn't appear to be the dataset from the example), I can't replicate your error.
Based on the error message you included, the only thing I can tell is that there's a problem when trying to run this model in parallel using the {furrr} and {future} packages. I would suggest running the model not in parallel (i.e., sequentially) to see if there's a specific issue you're having with the segment_behavior() function from the {bayesmove} package. It also may be worth checking that the exact example (linked above) from the repo works as expected on your computer.
It's also worth mentioning that none of the bins created when discretizing can be empty. So if you run apply(dat.disc[,c('SL','TA','Disp')], 2, table), make sure that none of these bins have 0 observations assigned. If so, you'll need to reconfigure how you discretize this data stream.

It turns out this is an issue with memory size allocation on my mac it has low memory because i am using a large dataset, any recommendations on how to run the code if i have a large dataset?

If it works on your computer, I would suggest just running the segmentation model on a single core (instead of in parallel). So, just running dat.res.seg1 <- segment_behavior(data = dat.list.sub, ngibbs = ngibbs, nbins = nbins, alpha = alpha) without future::plan(multisession, workers = n_workers) before it.
If that still poses a problem, I would try splitting your dataset (dat.list.sub) up into chunks. So maybe start by defining one R object that contains the first half of that list and another that contains the second half. Each ID (or list element) is evaluated independently, so this won't affect the results.
Hope that helps.

Thank you so much we give you feedback if it works

I found the solution to the problem. I was not a memory issue but rather as mismatch in the number of bins that was causing it. Thank you for the help, I was only about to identify it when i reduced the chunk the chunk size to 1 id.

Glad you were able to figure out the problem and that it was simple to fix.


Make document

Make wiki or something

Since it is my first time to make package wiki, I guess I must refer to other npm package wikis.
If someone can help me making wiki, that would be so much better.

Imtolazyplzwait

Ok I have a plan
I'll write the wiki in Korean first, and then translate it in English


Group index page lost default ordering in 5.2

Issue Summary
The IndexView for groups defines a default_ordering but it is not being used in Wagtail 5.2. Reverting to Wagtail 5.1 shows the expected ordering by name without ?ordering=name in the url.
Steps to Reproduce

Use your favorite method to spin up Bakery Demo
Go to http://localhost:8000/admin/groups/ and add a new Group with a name starting with something after "M". The new group should appear after "Moderators" in the listing page but instead it will appear at the top of the list.

Any other relevant information.
Explicit sorting is working fine (http://localhost:8000/admin/groups/?ordering=name and http://localhost:8000/admin/groups/?ordering=-name sort by name ascending and descending). It is only the default_ordering that is getting lost.

I have confirmed that this issue can be reproduced as described on a fresh Wagtail project: yes

Technical details

Python version: 3.8.18
Django version: 4.0.10
Wagtail version: wagtail source at bdc454d3cb

Thanks for the report! I can confirm this. I'm not sure if it qualifies as a bug but it definitely wasn't intended.


Improve InfoSQLDatabaseTool because it doesn't work well with qwen

Remove single quotation marks from the string, because qwen may return a string within single quotation marks.
Make the splition more robust, because qwen may return a list without blank after comma.

Apologies for the slow review! Pr has some merge conflicts, happy to re-review if you'd like to resolve

Apologies we haven't managed to review this on time. I'm going to close this PR for now since it accumulated merge conflicts. Feel free to re-open if this is still an issue or relevant for you.


GitHub CSS change fixes

GitHub has changed their CSS class tags.
The previous BeautifulSoup looks were not working either. Maybe a version difference...

It looks like a Facebook check is failing too.  Is there a reason you chose to parse the document instead of just looking for an expected string?

Opening a different pull request with Facebook fix included


Sample request: Do you have example to use Fresco as BitmapProvider?

Hi, the title is obvious. Can you help me with the example?

no, i don't have a sample, but you can probably find a way.  you just want the Bitmap, nothing else - TileView will manage when it's displayed and when it's memory is reclaimed (a lot of the same things Fresco does); you probably won't be able to take advantage of most of Fresco's features, as they're occupying a lot of the same responsibilities the TileView does...  Note that TileView does not use any ImageView instances - Bitmaps are drawn to the Canvas of a single View.
Honestly, unless you have a real reason for doing so, I don't think you're going to get much out of Fresco in conjunction with TileView.  If you're loading from assets, just use the provided provider - if you're getting images from a network, both Glide and Picasso offer very simple APIs to get a Bitmap from an image on a server.
Good luck, post back if you come up with something.


Added simple phyloreference state system

This allows each phyloreference to be in one of the states laid out in #25. Changes in state for each phyloreference are recorded along with a timestamp, allowing the previous history of the phyloreference to be documented. In the future, this could be extended to archive previous versions of the phyloreference or to identify who changed its state (#59).
Since this pull request requires new prefixes in the PHYX file, I've also included a fix for #46 so that we include a copy of the context we need in this repository.
Do not merge: this pull request is complete, but hasn't yet been fully tested.

this pull request is complete, but hasn't yet been fully tested.

Is this still true?

Yes, still true! I've tested this in the Curation Tool by itself, but I'd like to make sure that the status information can actually be used by the Clade Ontology. I've gotten it to the point where the status information is read and retained by the Clade Ontology scripts (phyloref/clade-ontology#32), but I still need to modify JPhyloref to take it into account when testing (phyloref/jphyloref#8).
I'll submit all three pull requests for review once I have the Travis report to show that they all worked!

Nope, nothing controversial! Thanks for the review!


I want to improve the styling

The fonts arent properly visible.

The header can be much more creative and appealing

Please assign to me and i ll make sure it looks good.

Please check and revert back to me. And if it's okay then merge it.


Azure Core ResourceOperationStatusResponse create a model with multiple @key

This is making the duplicate key validation fail. There is a workaround fixing that https://github.com/microsoft/cadl/pull/1215
but this might need to be rethought
Playground Link
^ without the workaround this code would fail with

Created in the wrong repo https://github.com/Azure/cadl-azure/issues/2162


[bug] Multi Row Replacement

If the code(expression) is written in multirow(for readability or some other things), then one more non-exsist space will be added in each of the line in the replacement.
I test different languages and different multirow expression. This always happens when something is in front of the multirow exp (such as 't = '). But it will not happen if the line started directly with that multirow exp.
I'm not sure whether it is bug, or it's just a feature.
The example is in this link.
Playground Link
💻 Code

Code:
t = [
    1,
    2,
    3
]

Pattern:
$X1 = $X2

Replace:
$X1 = $X2

🙁 Actual behavior
An unexpected space is added in each of the line.
🙂 Expected behavior
As the replace is the same with the match pattern, the expected replacement code should be same with the input.

Hi I cannot reproduce the issue. I see the playground diff tab is a noop.

would you like to provide more info?

Emm, maybe I'm not explain it clearly. You can copy the text in the 'diff' back to the 'source', and see it again. Repeat this, there will be more and more indent in front of the line(line 2 ~ 5).

Hi I cannot reproduce the issue. I see the playground diff tab is a noop.

would you like to provide more info?

In source code, line 2-5 have only 2 space in front of the code. But in diff, it has 3. I dont' know where is the extra space comes from.

Oh, gotcha. Thanks for the explanation.

Just an update, I'm fixing this now


remove onKernelRequest listener from LoggableListener

Can we remove the request listener now that there is a LoggableListener::prePersistLogEntry() function?  The retrieval of the user could be put there, no?
ref: https://github.com/l3pp4rd/DoctrineExtensions/commit/8ab7c9d3f971cfcba1a974e1fcab04139527bb62

Closing as stale.


fix: [#9517][CSS] PRO | COMPOSER | In .triggers[0].actions[0]: Text: alwaysPrompt the return type does not match

Description
This PR fixes the issue by adding a new way of indexing the values in the cache object, thus avoiding overwriting the data due to its typing.
Task Item
Fixes # 9517
#minor
Screenshots
These images show the user sample and a Microsoft template working successfully with the new indexing.

/promoted 9538


eq-205 Multi-choice multi-answer

Changes

Adds multiple choice/multiple answer question type (checkboxes)
radios/checkboxes can optionally be directed to use 2 column layout (via schema)

How to test

npm run compile
complete star wars survey, checking that responses are maintained between pages (eg. on submission of errors) and formatted correctly on submission page

Who can test

anyone but @hamishtaplin and @weapdiv-david

@weapdiv-david I'm only getting the first response on the submission page—need to be an array so I can list them out.

LGTM - Happy to merge, can we squash or rewrite the commits that don't have much description? Makes the commit log a lot easier to work with. 👍

@dhilton Squash away.


CSS output is html encoded!

CSS produced by Fontello is HTML entity encoded. And I cannot use it in React.
Fontello Link: https://fontello.com/82445ee7629e593fdf6d2c4428e9458a

Use config.json to generate any CSS you need, if defaults not suitable for you needs. Or report bug to your bundle, because css is valid.


Cannot push to mirror

My friend as a git server. (gogs)
I also have a server running gogs.
When I setup a mirror of one of his repos (on which I am a collaborator) on to my server, I can pull the mirrored repo perfectly fine, but cannot push and have it mirrored back to his server.
The error I get is a 403 with error 'remote: mirror repository is read-only.'
Surely it should be possible to setup bidirectional mirrors; or is the gogs functionality limited to pull-only?

As far as I know, in general the mirror functionality (of git) is intended in one way only.
I just finished to setup a bidirectional mirror with gogs and other remote repositories and I did in this way:

create a normal bare repository
add the other repository as remote, eg. git remote add origin repo_url
access the repository directory with ssh
modify the config file in the root of the repo, adding the mirror = true option under the section of the remote, modify also the fetch option to include all the refs, in the end it should like

[remote "origin"]
 url    = remote_repo_url
 fetch  = +refs/*:refs/*
 mirror = true

add a post-receive hook in the repository's settings page, like this

git remote prune origin
git push --quiet origin

In this way, everytime you push something into your repository, the script will update (push) in the remote repository.
If you do exactly the same for the remote repository, just setting your repository as remote, it will work in the other direction, updating your repository everytime the other repository is updated.

This behavior is designated and expected...


We need to give node some breathing room every once in a while.

Otherwise we will end up Maximum Call Stack Size exceeded errors for big datasets. Fixes #2

Seems like this could be better if we use the asap module. Relying on setTimeout in this way is a bit subpar.

Thanks for pointing me in the right direction. I do appreciate this PR, it's just that the solution I came up with (to process synchronously if possible) is a hardier solution to the problem as a whole and doesn't rely on setTimeout or some other arbitrary stopgap.


HBASE-24428 : Update compaction priority for recently split daughter …

…regions

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  3s
Docker mode activated.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 45s
the patch passed

+1 :green_heart:
compile
1m 14s
the patch passed

+1 :green_heart:
javac
1m 14s
the patch passed

+1 :green_heart:
shadedjars
6m 52s
patch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 50s
hbase-server in the patch failed.

_ Other Tests _

+1 :green_heart:
unit
212m  3s
hbase-server in the patch passed.

243m  0s

Subsystem
Report/Notes

Docker
Client=19.03.9 Server=19.03.9 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1784/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1784

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux 18a8f13552b3 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 852150a23f

Default Java
2020-01-14

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1784/1/artifact/yetus-jdk11-hadoop3-check/output/branch-javadoc-hbase-server.txt

javadoc
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1784/1/artifact/yetus-jdk11-hadoop3-check/output/patch-javadoc-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1784/1/testReport/

Max. process+thread count
3194 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1784/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 32s
Docker mode activated.

-0 :warning:
yetus
0m  3s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ master Compile Tests _

+1 :green_heart:
mvninstall
4m 18s
master passed

+1 :green_heart:
compile
1m  0s
master passed

+1 :green_heart:
shadedjars
6m  9s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 39s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m  0s
the patch passed

+1 :green_heart:
compile
1m  0s
the patch passed

+1 :green_heart:
javac
1m  0s
the patch passed

+1 :green_heart:
shadedjars
6m  8s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 38s
the patch passed

_ Other Tests _

-1 :x:
unit
223m  9s
hbase-server in the patch failed.

250m 27s

Subsystem
Report/Notes

Docker
Client=19.03.9 Server=19.03.9 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1784/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1784

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux e7d9b90c2753 4.15.0-101-generic #102-Ubuntu SMP Mon May 11 10:07:26 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 852150a23f

Default Java
1.8.0_232

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1784/1/artifact/yetus-jdk8-hadoop3-check/output/patch-unit-hbase-server.txt

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1784/1/testReport/

Max. process+thread count
2858 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1784/1/console

versions
git=2.17.1 maven=(cecedd343002696d0abb50b32b541b8a6ba2883f)

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 32s
Docker mode activated.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m  7s
the patch passed

+1 :green_heart:
compile
1m  4s
the patch passed

+1 :green_heart:
javac
1m  4s
the patch passed

+1 :green_heart:
shadedjars
6m 14s
patch has no errors when building our shaded downstream artifacts.

-0 :warning:
javadoc
0m 50s
hbase-server in the patch failed.

Test failed in the recent build is flaky. It's passing locally.

Thanks @apurtell @anoopsjohn for the review. Let me merge this.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m  9s
Docker mode activated.

-1 :x:
patch
0m  3s
https://github.com/apache/hbase/pull/1784 does not apply to master. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help.

Subsystem
Report/Notes

Docker
Client=19.03.9 Server=19.03.9 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1784/3/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1784

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1784/3/console

versions
git=2.17.1

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 34s
Docker mode activated.

-1 :x:
patch
0m  3s
https://github.com/apache/hbase/pull/1784 does not apply to master. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help.

Subsystem
Report/Notes

Docker
Client=19.03.9 Server=19.03.9 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1784/3/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1784

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1784/3/console

versions
git=2.17.1

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 35s
Docker mode activated.

-1 :x:
patch
0m  3s
https://github.com/apache/hbase/pull/1784 does not apply to master. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help.

Subsystem
Report/Notes

Docker
Client=19.03.9 Server=19.03.9 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1784/3/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/1784

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-1784/3/console

versions
git=2.17.1

Powered by
Apache Yetus 0.11.1 https://yetus.apache.org

This message was automatically generated.


Trimmed video first seconds sometimes stuck

Hi!
First of all, I love this new package,
While I was using it, I realized that sometimes the video is stuck in the first few seconds after it was trimmed (but the audio is still playing).
I think that it is because of the FFmpeg command, but I'm not sure and couldn't find the exact reason.

@sbis04 Hi,
Did you find anything?

was using FFmpeg before this. It was also stucking. Anybody finds a better solution for this

Any new...

Hi @omercn18 @braysonjohn148, I took a look at the issue and found out that giving the time in the formats like, HH:MM:SS.xx or SS.xx is causing this issue.
Upon further investigation, I found out that removing these decimal places present after the seconds is resolving this issue.
So, I have updated this in the version 0.2.4
Let me know if it fixes the issue.

Well, it seems like 0.2.4 is not public yet.
I'll update when it will be.

Now the v0.2.4 is public, do give it a try @omercn18

Changed the command like you did in v0.2.4, the first frames still freezing :/ . Please help

@sbis04 Well same as what @vercorro said, first frames are freezing..

Hi @vercorro @omercn18 Can you please cross-check that you are using the version 0.2.4?
In your pubspec.yaml file import it like:
video_trimmer: 0.2.4

That is, without the carat sign.
Also, go to pubspec.lock file and make sure that the version is correct.
Just let me know about this?

Hi @vercorro @omercn18 Can you please cross-check that you are using the version 0.2.4?
In your pubspec.yaml file import it like:
video_trimmer: 0.2.4

That is, without the carat sign.
Also, go to pubspec.lock file and make sure that the version is correct.
Just let me know about this?

Hi sbis04, thank you for your reply and your contribution into flutter community.  I actually imported your whole library files into my projet so I dont use it as a dependency. It works well besides those first frames. I looked into your update for the 0.2.4 and the only modification is the command line (correct me if I'm wrong). So I replaced the line exactly like you did and still have freezing frames.

Hi @vercorro @omercn18 Can you please cross-check that you are using the version 0.2.4?
In your pubspec.yaml file import it like:
video_trimmer: 0.2.4

That is, without the carat sign.
Also, go to pubspec.lock file and make sure that the version is correct.
Just let me know about this?

Yes, version is 0.2.4

I also faced this issue in version 0.2.4, is there any update?

The issue with the starting and ending video frames freezing still persists. Changing the FFmpeg trim video start & end position to seconds does not fix the issue, so I will be reverting it to milliseconds as before in the v0.2.5.
As per this thread, the freezing issue occurs due to the video not being cut from an I-frame. But, I couldn’t yet figure out a proper method for finding an I-frame from a video.
This can be fixed by re-encoding the video as per PR #34, but this will take significantly more time.

This problem is solved by changing this line -i "$ _videoPath" -ss $ startPoint -t $ {endPoint - startPoint} by this -i "$ _videoPath" -ss $ startPoint -to $ {endPoint - startPoint} and add -copyts to the line -c copy

@tinnit1 I tried with the following changes, but still the video freezes for some time at the starting point:
String _trimLengthCommand = '-i "$_videoPath" -ss $startPoint -to ${endPoint - startPoint}';

_command = '$_trimLengthCommand -c copy -copyts ';

Try this, in this order. if not work maybe it can be an instance ffmpeg that it have conflict.
String _trimLengthCommand = '-ss $startPoint -i "$_videoPath" -to ${endPoint - startPoint}' -c copy -copyts $OutputPath;

Using this is causing an unusual timeline, i.e. the start time doesn't begin from 0. It starts from the start time in which position the video is cut from the original one. Check the screen recording:

A temporary fix has been deployed in v0.2.6: Added a new property to saveTrimmedVideo() method called applyVideoEncoding which will let you specify whether to re-encode the trimmed video.
NOTE: Applying this will take a significantly greater amount of time to process the output video.
Refer to: 30f6015afcc33dbd2586d07718f9c8f0a33ab5ef

Thanks for the temporary fix in v0.2.6.
After setting applyVideoEncoding = true, the video quality becomes bad.
Is it expected?

@sbis04 Thanks for the temporary fix in v0.2.6.
After setting applyVideoEncoding = true, the video quality becomes bad.
Is it expected?

it becomes way worse than normal quality and the drop is quite noticeable when the original video is of 720p resolution

No solution yet?

did anyone find any solution to this?

@sbis04 Hi, do you see any good fix for this issue coming soon?
applyVideoEncoding takes a lot of time and destroys the video.

Hey @omercn18 and @sbis04 I found a really easy fix.
Right how you have this:
String _trimLengthCommand = '-i "$_videoPath" -ss $startPoint -t ${endPoint - startPoint}';
And you need to change it to this: String _trimLengthCommand =         ' -ss $startPoint -i "$_videoPath" -t ${endPoint - startPoint}';

Awesome, it works!
Thanks 👍

@courtneypresto Thanks a lot for finding this amazing solution! I created a PR with this change. The only problem being that video duration gets shortened by exactly 600ms.
So I changed the command to
   String _trimLengthCommand = ' -ss $startPoint -i "$_videoPath" -t ${endPoint - startPoint + durationCorrection}';
@sbis04  There is also another problem, however, this one existed all the time - When starting time is any non zero value, the video still gets stuck, most likely because it is waiting for the nearest keyframe, we have yet to find the solution for this problem. WhatsApp tackles this by starting and ending the trimmed video with the nearest keyframe timestamp. That's why the trimmed video in WhatsApp starts and ends with different timestamps than we intended to. I think we can too implement this in the plugin.

@ArjanAswal @kasyrm @courtneypresto This solution had been presented long before by @tinnit1, I had tested this at that time, but this results in a much worse scenario because if the video doesn't start from the initial starting point then the timeline gets distorted, and sometimes that weird trimmed video isn't even playable. Refer: https://github.com/sbis04/video_trimmer/issues/22#issuecomment-663434391
I would suggest avoiding this solution at this point.
And, yes @ArjanAswal this problem can be solved if the video can be trimmed from a keyframe. But still, I haven't found any efficient approach to get that, refer: https://github.com/sbis04/video_trimmer/issues/22#issuecomment-660572809
I would appreciate a PR if someone can retrieve the keyframes of the video and use them for trimming, rather than trimming from any random point.

@sbis04 Retrieving keyframes of the video can be implemented by using the ffprobe -show_frames command. It can take a bit of time to complete although I've never tried it.  That's one way. I'll look into it.
Also, courtneypresto's solution still stands good if initial position of video is zero, atleast we can implement it in that special case.  We can have best of both worlds until we find the optimal solution for this issue.
     String _trimLengthCommand = '-i "$_videoPath" -ss $startPoint -t ${endPoint - startPoint}';

     if(startValue == 0){
       _trimLengthCommand = ' -ss $startPoint -i "$_videoPath" -t ${endPoint - startPoint + durationCorrection}';
     }

Hmm okay I haven't actually tried it yet but I just found this solution which distinguishes a normal cut (nc) with a keyframe cut (kc)... and doesn't need the cut to actually start at 00:00.
Someone provided an example here.
Seems promising! @ArjanAswal @sbis04

@courtneypresto Adding -avoid_negative_ts make_zero rectifies the problem that @sbis04 mentioned here.
I think we have now solved the issue. The only problem being that the starting point of the trimmed video picks the keyframe that is earlier instead of the later one.
For example, we want to trim a video from 10s to 50s timestamp. Let's say the 2 keyframes nearest to the 10s timestamp are 9s and 11s. Now instead of picking the 11s, it picks the 9s timestamp which makes the video 41 seconds long instead of 39.
That is a trade-off we can make, for now, at least it is better than the video getting frozen for 1-2 seconds in the beginning.
Now, it is up to @sbis04 to make the final decision.


[Feature request] Add a way to come back after starting a search through a tag

https://user-images.githubusercontent.com/75091899/207057501-947607e2-d6ba-4ef1-b4d0-4366b67ad06c.mp4
after starting a search through the results of the first search (a tag of a single post of the first search), i would like to come back to the post where the second search started. If i was not clear enough just ask me questions.

I would say that this feature is very important and, if implemented, would mean an huge improvement in the QoL of the user

Sorry, with current app architecture, it's impossible to mimick the behaviour of web browser.

It is really a pity then

This might be a breaking point for some users pushing them to other apps that do that thing instead. well, if you don't plan to earn from the app ,praticing your coding skills or just hobby then it is fine

This might be a breaking point for some users pushing them to other apps that do that thing instead. well, if you don't plan to earn from the app ,praticing your coding skills or just hobby then it is fine

Sorry, it just I'm not interested to implement multi back stack entry because it requires a lot of data flow redesign, not to mention about performance consideration since user can have literally thousands of content scrolled in back stack being held in memory.
This also applied to things like tabs feature (just in case someone thinks about that).


Wallhaven official api

First of all, thank you for this package.
Wallhaven is now out of beta and an official api is now exposed
See (pre release) link : http://stest39.wallhaven.cc/help/api

Awesome, I will deprecate this project later.


Cannot set object to null in v1

In my model, I have a nested object, which is optional.  This nested object can be added or removed from the model at any time.  I have tried to do model.set('obj', null) - but this gives no change in the object before or after saving.
I've made a demo repo here: https://github.com/peterchoo/astronomy-v1-set-object-null
Is there a way of removing a nested object?  Thanks!

Bug fixed in 1.2.11, however I would strongly recommend switching to version 2.0

Thank you, I'll upgrade to 1.2.11 now.  We are intending to upgrade to 2.0 ASAP, we just have some blocks right now, it will be upgraded within a few weeks.

Ok good


Specify the port with annotation

Hello,
With the class rule EmbeddedKafkaRule this is possible to specify the port like that:
public static EmbeddedKafkaRule embeddedKafka = new EmbeddedKafkaRule(1, true, "topic").kafkaPorts(9092);

How can I achieve the same with the annotation ? (I use Junit 5)
@EmbeddedKafka(controlledShutdown = true, topics = {"topic"})

I think this is not too bad to support on the annotation level:
/**
 * @return ports for brokers.
 */
int[] ports() default { };

So, treating this as an  improvement.
And that's not too late yet to implement in the current version.


Custom dropdown cannot be scrolled on Internet Explorer

Are you requesting a feature or reporting a bug?
Reporting a Bug
What is the current behavior?
Internet Explorer fails to scroll on a custom angular dropdown
What is the expected behavior?
Internet Explorer should scroll our custom dropdown before clicking/performing actions
How would you reproduce the current behavior (if this is a bug)?
Click the last element of our custom dropdown. Chrome and Firefox scroll fine.
Provide the test code and the tested page URL (if applicable)
I created a code pen and a test case.
This is a simplified version of our angular component, but the dom element and the style is the same.
Tested page URL:
https://codepen.io/anon/pen/rKmRNo
Test code
import { Selector } from 'testcafe';

fixture `IE BUG`
    .page('https://codepen.io/anon/pen/rKmRNo');

test(`Cannot click on the last element`, async t => {
    let iframe = Selector('iframe.result-iframe');
    let last = Selector('p').withText('Ten');

    await t
        .wait(10000)
        .switchToIframe(iframe)
        .expect(last.exists).ok('The element exists')
        .click(last);
});

Specify your

operating system: Windows 10. Internet Explorer 11.48.17134,0
testcafe version: 0.20.1
node.js version: v8.9.4

Hi @berrutti,
I've reproduced this problem and will investigate it.

The hasScroll function works incorrectly in IE11 if an element has the overflow-x: auto CSS attribute only. In other browsers, computedStyle.overflowX and computedStyle.overflowY equal auto by default. In IE11, computedStyle.overflowY equals visible. Test is passed if I change the overflow-x: auto line to overflow-y: auto in example higher.

Thank you very much. We added the changes and can confirm it now scrolls properly. Thanks a lot.

I'm working on it

Test cases:

[ ] element
[ ] <body>
[ ] <html>


docs: update switch-per-node diagram to include gateway details and PDF version

@shettyg @rajatchopra

Do we need all 3? Should we delete the odg one?

@shettyg the ODG should probably be the source if we need to update it in the future.  I removed the SVG one, and instead posted a PDF one which is easier to view I think.

@dcbw
I still see the svg one.

@shettyg fixed


Update CI

Update to actions/checkout v4 from v2.
Switch to dtolnay/rust-toolchain. The old actions-rs actions haven't been maintained in years.
Stop using xargo for no_std builds and use cargo instead. Also, use better targets that have no std library available for them.

@sebcrozet This should fix the CI builds for this crate (and some other stuff).


Equalent go benchmark

The header setting thing only slows hello function and actually does nothing. Also the is no need to declare those temporary text and status variables on every request.

I updated the article in two plaes with the following text:

As user @heppu points out Go’s stdlib HTTP server can be 12% faster than the graph shows when written more carefully. Also there is the awesome fasthttp server for Go that apparently is only 18% slower than Japronto in this particular benchmark. Awesome! For details see https://github.com/squeaky-pl/japronto/pull/12 and https://github.com/squeaky-pl/japronto/pull/14


⚠ Hartman Foundation has degraded performance

In 75e9b53, Hartman Foundation (https://gordonhartman.com/) experienced degraded performance:

HTTP code: 200
Response time: 2987 ms

Resolved: Hartman Foundation performance has improved in e7d631f.


Revert "Limit CI concurrency to one per PR"

Reverts cloudberrydb/cloudberrydb#233

it has been concluded the incident happened this morning is due to routing issue in our aws configurations. the concurrency limitation will be brought back in #261 once the CI workflow is stable again.


i18n(zh-cn): Update solid-js.mdx

Description (required)
Update solid-js.mdx
Related issues & labels (optional)

#5978
#6005
#6109

@huyikai @liruifengv - This file had additions today (new section at the bottom). Would you like to add those changes here, too?

@huyikai @liruifengv - This file had additions today (new section at the bottom). Would you like to add those changes here, too?

Yes, We'll let you know when it's updated.

@liruifengv Updated, please review

@sarah11918 ready


Misleading variables in databuilder readme

Hi,
I would like to fix examples in readme of databuilder. Maybe it has some reason and I just don't see it, but probably that is just copy-paste issue.
There are some variables in readme https://github.com/lyft/amundsendatabuilder/blob/master/README.md for example in Elasticsearch publisher
tmp_folder = '/var/tmp/amundsen/dummy_metadata'
node_files_folder = '{tmp_folder}/nodes/'.format(tmp_folder=tmp_folder)
relationship_files_folder = '{tmp_folder}/relationships/'.format(tmp_folder=tmp_folder)

but those variables are not used in the example code. It was a bit misleading for me at the beginning. I would fix it by removing not used variables and adding those used in the example code.

@jhettler Good catch and apologies for the confusion that this caused. Feel free to post PR on this.
Thanks!


fixed display of english data logger site

Added english translation for the data logger

@Thiemann96 site empty and disappears from the navbar after clicking on EN


ros_msg_parser: 1.0.0-1 in 'melodic/distribution.yaml' [bloom]

Increasing version of package(s) in repository ros_msg_parser to 1.0.0-1:

upstream repository: https://github.com/facontidavide/ros_msg_parser.git
release repository: https://github.com/facontidavide/ros_msg_parser-release.git
distro file: melodic/distribution.yaml
bloom version: 0.10.7
previous version for package: null

Closing this due to no response. @facontidavide please comment back when this mismatch has been resolved.


Longhorn Helm Chart (usable with helm cli)

Right now longhorn has a helm chart which can be installed using the catalog (gui).
It would be nice to get a official helm chart, which can be used with the normal helm cli tool.
I want to use this for a reproducable cluster deployment (with rke / rancher / longhorn).
Maybe that could be published in a repo like https://releases.rancher.com/longhorn-charts/stable same as rancher 2 has its own chart repo https://releases.rancher.com/server-charts/stable.

@PhilippHeuer I was thinking about this, now I am curious why you prefer Helm? Longhorn currently can be installed/upgraded by a single line of kubectl, uninstalled by two lines of kubectl. It doesn't like other applications that needs many configurations since we spent quite some efforts on detecting the user environment and configure Longhorn automatically.
Could you share your ideas on why you prefer Helm for Longhorn?

You do already supported helm over the rancher catalog, just not as vanilla helm chart because of the different directory structure.
The reason is pretty simple, i know how to install any app with helm since it's always the same command with a different chart. But everytime i need to use kubectl apply i need to check the docs for my current version on what to do (multiple commands for setup / delete job / ..).
For example the first 2 times i uninstalled longhorn i left my cluster in a broken state since i didn't know about the uninstall job - which needed to run first. Helm takes care of such things too using hooks.
It's basically just quality-of-life - it's the same commands for every kind of service i deploy and it takes care of everything for me. At least thats the reason for me personally ;)

@PhilippHeuer Thanks for elaborating.
Yeah, we will include an official Helm chart in the Longhorn (besides the one using by Rancher) in the next release.


TSS is broken

The TSS related tests currently fail. This means tss is non-functional and disabled by default for now.

I believe this has been fixed and the issue can be closed

Let's close it for now. Please reopen, if needed.

As it has been enabled again. We are seeing the test failures again.


Include exception to full linkage requirement.

Full linkage is not required when relationship fields are
excluded via sparse fieldsets.
[Closes #670]

Works for me. :+1:


Request: Add isMaximized and isMinimized

isMaximized has added to linux, macOS, windows.
isMinimized  has added to macOS, windows.
I haven’t found how to minimize it in linux, because gtk_window_minimize, gtk_window_unminimize only exists in gtk4.

This can be closed I guess?


(WIP) Add witness role to Raft

Fundamental implementation to support witness role in the core raft library. Unit tests & integration tests are still in progress.

Codecov Report

Merging #92 into master will decrease coverage by 0.09%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #92      +/-   ##
==========================================
- Coverage   80.43%   80.35%   -0.08%
==========================================
  Files          12       12
  Lines        3500     3500
==========================================
- Hits         2815     2812       -3
  Misses        454      454
- Partials      231      234       +3

Impacted Files
Coverage Δ

execengine.go
82.2% <0%> (-0.52%)
:arrow_down:

node.go
78.33% <0%> (-0.22%)
:arrow_down:

event.go
38.71% <0%> (+1.62%)
:arrow_up:

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 13013c5...ddbfa45. Read the comment docs.

Hi abbccdda, thanks for the PR.
Any chance you can email me your WeChat ID? My email is<EMAIL_ADDRESS>thanks!

just figured out that I already have @abbccdda's contact details. we two will talk in details soon to make a plan to get this fully implemented, tested and merged.

@lni Thanks Lei!

As briefly mentioned to @abbccdda yesterday, this PR can just focus on the witness role implementation for the raft protocol layer.
I'd like to discuss the following items -

the current implementation requires it to be added via Config Change. it seems to me that users should be allowed to specify witness nodes straightaway when starting a new Raft cluster.

witness nodes have nil logdb provided to them, how they are going to get index/term values for those persistent entries?

broadcastHeartbeatMessageWithHint() should send heartbeat messages with concrete pb.SystemCtx to all voting nodes.

in makeReplicateMessage(), MsgReplicate messages sent to witness nodes have all Cmd payload set to nil. in place changing the fields of returned entries is not allowed.

for MetadataEntry, what fields should be kept? we probably only need to keep the index/term/type values.

how InstallSnapshot messages are handled for witness nodes? we should only send dummy snapshot to witness nodes. a dummy snapshot has index/term and cluster membership info, but it doesn't contain any user state machine data.

handleWitnessReadIndex() re-routes received ReadIndex message to handleFollowerReadIndex(). witness nodes do not support read-only query, so it won't see ReadIndex message in the first place.

for simplicity, the same can probably also be done for Propose messages.

test plan.

From my sync with @lni , we have reached some initial conclusions on the points 1~6, and will discuss further details for 7~8 in the coming weeks.

We will support config level initialization for witness node. This could be done on the upper layer of raft, which will be introduced in a separate PR.

Witness node should persist metadata logs into disk to avoid potential time travel bugs, where a restart of the witness could cause unnecessary downtime. Furthermore a nil logdb will be an intrusive change to the storage layer logic handling.

Will fix this issue in next commit.

When sending log entries to witness, leader will rebuild the entries from scratch to avoid race access caused by in-placing field swapping. A little allocation overhead will be incurred but should be acceptable.

Witness will only persist entries with metadata fields such as type/index/term for storage efficiency.

Since witness naturally couldn't support linear read query, upper layer will disable read query towards witness, which means we could discard this logic in raft layer.


DDoS attack against anaconda.org

Anaconda has been notified by our CDN provider of an DDoS attack against anaconda.org; at this time, it appears that downloads from larger channels like conda-forge & bioconda are unaffected, but package uploads, downloads from smaller channels, and other API requests are failing.
For updates, please check: https://anaconda.statuspage.io/

Thanks Cheng! 🙏
Appreciate keeping us updated and the work to resolve the issue 🙂
Should we close this out now?

CDN DDoS protection measures have been put into place, and the various anaconda.org services have recovered to normal operating status. Closing this issue.

Think we can now unpin this one as well


Add support for budget tracking across burdock module executions

Added in PR https://github.com/opendifferentialprivacy/whitenoise-system/pull/155


[BUG] App Configuration SDK returns (No Label) label incorrectly.

Describe the bug
When the App Configuration SDK returns a configuration with the (No Label) value it sets the label to null. Where is should be \0.
This results in this unexpected result:
        ConfigurationAsyncClient client = new ConfigurationClientBuilder().connectionString(CONNECTION_STRING)
            .buildAsyncClient();

        SettingSelector selector = new SettingSelector().setKeyFilter("/application/config.message").setLabelFilter("\0");

        client.listConfigurationSettings(selector).subscribe(setting ->
        System.out.printf("Key: %s, Value: %s, Label: %s%n", setting.getKey(), setting.getValue(), setting.getLabel()));

        Thread.sleep(10000);

        ConfigurationSetting value = client.listConfigurationSettings(selector).blockFirst();

        selector = new SettingSelector().setKeyFilter(value.getKey()).setLabelFilter(value.getLabel());

        client.listConfigurationSettings(selector).subscribe(setting ->
        System.out.printf("Second Key: %s, Value: %s, Label: %s%n", setting.getKey(), setting.getValue(), setting.getLabel()));

The first list only prints out a single result.
Next the one result is retrieved and it's key and label are used to call list again.
This list returns all labels, as setting null as the label returns all results.
Exception or Stack Trace
N/A
To Reproduce
Steps to reproduce the behavior:
See above code.
Code Snippet
See above code.
Expected behavior
(No Label) keys should have the label \0 not null.
Screenshots
Setup (please complete the following information):

OS: [e.g. iOS] Windows 10
IDE: [e.g. IntelliJ] Eclipse
Library/Libraries: [e.g. com.azure:azure-core:1.16.0 (groupId:artifactId:version)]  com.azure:azure-data-appconfiguration:1.2.1
Java version: [e.g. 8] 11
App Server/Environment: [e.g. Tomcat, WildFly, Azure Function, Apache Spark, Databricks, IDE plugin or anything special] Local
Frameworks: [e.g. Spring Boot, Micronaut, Quarkus, etc] SpringBoot

If you suspect a dependency version mismatch (e.g. you see NoClassDefFoundError, NoSuchMethodError or similar), please provide

dependency tree (mvn dependency:tree -Dverbose)

Additional context
Add any other context about the problem here.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[ ] Bug Description Added
[ ] Repro Steps Added
[ ] Setup information Added

Closing this issue as service decided to keep as it is.


Feature/allow explicit api creation

@BrandonHoffman I think you'll appreciate the switch to the metaclass powered module-level singleton, which was inspired by your question about the current approach. This allows any one two subclass the base API object then use it by doing
class MyCustomAPI(hug.API):
    # my overrides

api = MyCustomAPI(__name__)

which simply wasn't possible before


Improve chart labels edge cases behaviour/look

As charts will have the labels inside the charts (ref: #129), currently there was no work done, to make sure that the charts will look fine if there will be not enough space for the label to fully be visible in the chart
My current idea is to place the label outside the chart if there is not enough space for it, and if there will be too many labels, which could overlap between each other, we could think about maybe introducing legend or something else.
Overall, there needs to be done some work to improve this.

Triage: @bartsmykla can you elaborate? Is the issue still present?


fix(a11y): fix focus trap ignoring element with disabled=false

General summary
Elements with disabled=false or aria-disabled=false at the end or start of a dialog or lightbox were skipped in the focus trap.
This PR fixes that and adds tests on the code detecting the first and last focusable elements in dialogs and lightbox.
Test:

Go to the dialog focus trap story

Check that the focus is contained inside the dialog
Every focusable item is reachable
Every non focusable item are skipped

Test OK
Button with disabled=false at the end of a dialog is focusable


Failed to load XYZ - try again later.

Prerequisites

[x] Put an X between the brackets on this line if you have done all of the following:

[x] Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
[x] Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/
[x] Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
[x] Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
[x] Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
I'm behind a corporate firewall and using this settings inside .atom/.apmrc file:
proxy=http://user:password@host:port/
https-proxy=http://user:password@host:port/
strict-ssl=false

Steps to Reproduce

Set up a proxy with self-signed certificate
Install clean version of latest atom
Go to "Install +" setting and click on any community package

Expected behavior:
View package description
Actual behavior:
Getting error: Failed to load XYZ - try again later.
Reproduces how often:
Every community package
Versions
atom --version
Atom    : 1.20.0
Electron: 1.6.9
Chrome  : 56.0.2924.87
Node    : 7.4.0

apm --version
apm  1.18.4
npm  3.10.10
node 6.9.5 x64
python 2.7.13
git 2.14.1.windows.1

cmd /K ver
Microsoft Windows [Version 6.1.7601]

Additional Information
If I hit CTRL+SHIFT+F5 on package description page, info message appears: Loading XYZ...
and then I get package description.
After I loaded it this way, I also get description of this package after restarting Atom
Also in Developers tools pane inside Network tab when I click on package in packag list I can't see any activity, it appears only when I hit refresh window keybinding

Hi, I was able to reproduce your issue but I believe it belongs in the settings-view repository.

I guess this error was gone


Prepare bumper for golang 1.22/OCP 4.17

/test all

We'll hold this here until OCP is branched. We may also have to do what @grokspawn suggested:

We would need to update here before ci-operator could compile.
There is no go1.22 builder for ocp 4.16, but there is already one for 4.17 (registry.ci.openshift.org/ocp/builder:rhel-9-golang-1.22-openshift-4.17) if we want to 'jump the gun' and start relying on it.

/unhold 4.16 branch is cut

This looks good to me, and we are beyond branch cut now. Should we merge?


Virtual screen error in IQOO NEO 6

I am using IQOO NEO 6, with scrcpy windows latest release on windows 11.
I am on android 14 and the normal mirroring works fine, but the virtual display option brings up just a white screen, as seen in the video below.
Also one thing to mention, when using the virtual display mode to launch a specific app from the terminal, that works just fine.
https://github.com/user-attachments/assets/37a0f70b-510e-4da8-b12d-3ddcc8801ba1

Some devices have no "launcher" for virtual displays. In that case, you must start an app in the virtual display (that app mah be a custom launcher).

What launcher would you recommend

You may use your default launcher (find it in scrcpy --list-apps).
Or you can use for example this open source launcher (Fossify Launcher, available on F-Droid): https://f-droid.org/en/packages/org.fossify.home/

You may use your default launcher (find it in scrcpy --list-apps).您可以使用默认启动器（在scrcpy --list-apps中找到它）。
Or you can use for example this open source launcher: Fossify Launcher, available on F-Droid (https://f-droid.org/en/packages/org.fossify.home/)或者您可以使用例如这个开源启动器：Fossify Launcher，可在 F-Droid 上使用（ https://f-droid.org/en/packages/org.fossify.home/ ）

OS: Windows 23h2
Scrcpy version: 3.0
Device model: realme GT Neo5 SE
Android version: 14
No matter whether I use Fossify Launcher or set it as the default home screen, the display is still like this:

Even with:
scrcpy --no-vd-system-decorations --start-app=org.fossify.home

?

Even with:
scrcpy --no-vd-system-decorations --start-app=org.fossify.home

?

This works now.


About yii\helpers\ArrayHelper::getColumn and array_column

    public static function getColumn($array, $name, $keepKeys = true)
    {
        $result = [];
        if ($keepKeys) {
            foreach ($array as $k => $element) {
                $result[$k] = static::getValue($element, $name);
            }
        } else {
            foreach ($array as $element) {
                $result[] = static::getValue($element, $name);
            }
        }

        return $result;
    }

PHP 5.5+ have a function is array_column. What is the difference and yii\helpers\ArrayHelper::getColumn

The difference is in calling getValue method, that support closures. And one more reason: we support 5.4 :)


Create docker file

Create docker-compose and docker files, to run inside a container.

Because of a problem with NPM, that can find it here, if dependency installing from git+https, package-lock always registers it as git+https.


Question about RemovePipelines

Hello,
I just found your repo and I'm learning Statiq from it.
I have two questions about this code :
https://github.com/AnkurSheel/StatiqHelpers/blob/01a669baaae0e5ddf6289b5e061cab66a54ed91c/src/StatiqHelpers/Extensions/BootstrapperExtensions.cs#L60-L78

Why are you adding a new pipeline in the RemovePipelines method ?

What does ReplaceDocuments module do in this case ?

Thanks a lot.

Hey,
1 can be moved out from RemovePipelines and into AddPiplines. Its an oversight. Thanks for pointing that out,
ReplaceDocuments replaces the input documents for the AnalyzeContent pipeline with  the output documents of all of the pipelines passed as a parameter to it.
Hope that helps

Perfect, thanks a lot !


[DRAFT] RFC: Api change for mdx/md file parsing

note: this is early draft
Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Description
Problem
In current architecture there is no way to change parser/implementation of parser

each of plugins require its own implementation of md/mdx parser
there is no way to change default behavior of parser
md files are not properly supported
there is no way to author plugin that changes how markdown files are parsed across website

Solution

remove custom parsing code from each plugin and all remark/mdx configurations out of them
create new package @docosaurus/plugin-mdx that is used as "fallback" md/x file parsing

related issues: #5757 #4029
Has this been requested on Canny?
No response
Motivation
support for simple md files
API design
No response
Have you tried building it?
No response
Self-service

[ ] I'd be willing to contribute this feature to Docusaurus myself.

Yeah, our Markdown infra is quite messy (also #4625)... We have an mdx-loader to load the MD files, a fallback loader when importing Markdown from outside the plugin, and a linkifier in each plugin because only the plugin knows how the MD file is mapped to URL paths. I think we should have a global protocol so plugins can communicate about the file -> URL mapping. I'm aware of the challenges the ts-eslint website is facing

each of plugins require its own implementation of md/mdx parser

Yes, there's a lot of boilerplate atm for plugin authors atm and we should provide better utils to factorize that

there is no way to change default behavior of parser

Yes, we should decouple more from MDX. For example, if your parser emits some kind of MD AST and you want to be able to render it with a runtime lib, this must be possible.
We should:

not assume that the content prop is a React component
allow to plugin a renderer of content
provide a custom parsing hook with unknown return type.

md files are not properly supported

MDX 2 seems to improve on CommonMark compatibility, if this is what you mean:
https://mdxjs.com/blog/v2/#architectural-improvements

there is no way to author plugin that changes how markdown files are parsed across website

Related to https://github.com/facebook/docusaurus/issues/4625

I think we should have a global protocol so plugins can communicate about the file -> URL mapping.

Yes, we need some kind of reverse URL mapping. Currently, one plugin cannot link to another by using md file paths, which is confusing.

MDX 2 seems to improve on CommonMark compatibility, if this is what you mean: https://mdxjs.com/blog/v2/#architectural-improvements

yes its better but MDX 2 removed support for html comments <!-- -->

i just relalased that this is a duplicate of #3018

Well, I feel like this is a mix of #4625, #3018, and what I'm going to implement in #5568


Feature suggestion: configurable base directory

It's relatively easy to configure dn's base directory, as it reads DN_HOME (and the script can of course just be adapted).
Maybe it makes sense to either read DN_HOME as well or to allow configuring the base directory (potentially before building)?

Thank you for your suggestion. The feature has been added.
The notes' storage location can be set with the environment variable DN_HOME, which is expected to be an absolute path.
The feature was added in commit 31d9809d232c44771cb55fea53774d5a56650e37.


DTS: Convert SAM UART/USART to DT_INST macros (plus pinctrl support)

Convert Atmel SAM UART & USART to the new DT_INST macros and as part of this conversion we introduce basic pinctrl support for the ATMEL SAM.

All Atmel drivers need use nodelabel. I'll test this PR on top of #24110.

All Atmel drivers need use nodelabel. I'll test this PR on top of #24110.

Why?

All Atmel drivers need use nodelabel. I'll test this PR on top of #24110.

Why?

static const struct uart_sam_dev_cfg uart##n##_sam_config = {		\
	.regs = (Uart *)DT_REG_ADDR(DT_NODELABEL(uart##n)),		\
	.periph_id = UART_PROP(n, peripheral_id),			\
	.pin_rx = DT_ATMEL_SAM_PIN(DT_NODELABEL(uart##n), 0),		\
	.pin_tx = DT_ATMEL_SAM_PIN(DT_NODELABEL(uart##n), 1),		\
	UART_SAM_IRQ_CALLBACK(n)					\
};									\

The above was tested and it works.
This is an old discussion, people questioned about it on the devicetree slack channel at 2020-04-17, inclusive. Atmel drivers don't work using DT_INST without a property to handle the HW instance. It can't instantiate the proper HW instance using the node instance itself, it simple don't match with HW. If you can't follow, try create a non regular case like disabling USART0 and then try enable only USART1 on SAM_V71_XULT, for instance, then you will see the problem. When you disable USART0 the USART1 will be DT_INST___0 and HW will be wrongly configured.
I already sent a patch that can address this but was declined in favor to use DT_NODELABEL without a technical answer at #23335. The #23338, #23332, #23359 are examples how to use HW instance + DT_INST based on #23335 but were closed in favor to use DT_NODELABEL.
I'm adding people that questioned on slack to see this case. I think is important everyone have this pretty clear.
@pabigot @mnkp @mbolivar
That said, I've been using since than what seems to be the right solution and sent the #24110 as first PR to collect feedback to continue Atmel drivers conversion.

@nandojve This PR adds pinctrl support and uses that to retrieve pin numbers from the device tree of each U(S)ART instance, so no need to use node label anymore.

@nandojve This PR adds pinctrl support and uses that to retrieve pin numbers from the device tree of each U(S)ART instance, so no need to use node label anymore.
#24110 (review)

Ohh boy! I was stuck on that and didn't realize!
I tested again and it worked with sam_v71_xult.


Documentation links are broken

The links in the online documentation (https://facebook.github.io/immutable-js/docs/) are broken. When clicking on "List" for example, I get redirected to https://facebook.github.io/List

I'm seeing this too... DevDocs to the rescue!!

This makes it very hard to use immutable when you can't even see what functions are available. @flarnie: As the last contributor to the repo can you have someone take a look at this

For anyone who sees this and needs access to the docs, I pulled in https://github.com/facebook/immutable-js/pull/1444 and uploaded the files here: https://immutable-docs.herokuapp.com/docs/

Hi - I honestly don't know how the docs for Immutable.js are built and updated.
Also when I visit the docs the links are working for me. Tips on reproducing this issue?
@leebyron do you have ideas about this? And for the future, can you point me at any docs about the docs?

Thanks @flarnie for taking a look.
TL;DR: Looks like a falseish alarm.
I was still able to repro, but then on a whim decided to try on incognito mode... and it started working. I figured out that one of extensions blocks jsdelivr.net (which is loading the react runtime).
However, instead of breaking in any of the usual ways, it does the whole redirect to https://facebook.github.io/List instead of https://facebook.github.io/#List.
Allowing that call through has the docs working again.
So, there is still a bug in that the HTML that is generated without javascript has the incorrect links. However, even if you manually put in the correct URL, the docs don't load because the website is really a react app. If I were you folks I would either close this as a won't-fix, or add some kind of  warning if loading react from the CDN fails. OR move the javascript file to a 1st party URL so it's not blocked by default.


chore: replace platformdirs with xdg-base-dirs

Currently we use the platformdirs package to automatically pick the platform specific system directories that are used by instructlab. While this works fine on linux with XDG dirs, On macOS this means usage of /Users/<user>/Library/Application Support/... which is awkward and hard to use
This PR replaces platformdirs with xdg-base-dirs to standardize the dirs across linux and macOS
acc to https://pypi.org/project/xdg-base-dirs/ xdg-base-dirs functions take XDG env vars into account so we get env var support for free with this package
There is a consideration to be made that we are making a change to the project's dependencies by making this switch so that should be taken into account
Issue resolved by this Pull Request:
Resolves #1747
Checklist:

[x] Commit Message Formatting: Commit titles and messages follow guidelines in the
conventional commits.
[x] Changelog updated with breaking and/or notable changes for the next minor release.
[x] Documentation has been updated, if necessary.
[ ] Unit tests have been added, if necessary.
[ ] Integration tests have been added, if necessary.

Not sure if this will cause problems on windows, but we are not really actively focussed on windows at the moment

Not sure if this will cause problems on windows, but we are not really actively focussed on windows at the moment

We don't support Windows at all AFAIK

I assume we will need similar PRs for the other projects like sdg?

oh yeah, hadn't thought of that - good point
sdg does seem to also be using platformdirs - although IMO the library shouldn't be doing any FS manipulation, all the FS interaction should ideally be happening in the CLI

I assume we will need similar PRs for the other projects like sdg?

oh yeah, hadn't thought of that - good point sdg does seem to also be using platformdirs - although IMO the library shouldn't be doing any FS manipulation, all the FS interaction should ideally be happening in the CLI

Agree very much with that


🐛 Fixed members import unsubscribing members when subscribe_to_emails is blank

fixes ENG-611

Previously, if an existing member with newsletter subscriptions was imported, and subscribe_to_emails was blank/empty, the member would be unsubscribed from all newsletters, which is not the expected behavior.
This PR changes the behavior so if subscribe_to_emails is blank, it will not unsubscribe existing members.

Hard agree!