Unneeded argument

Base.isBackendValidationError() uses _.invoke() to call error.getResponse(). However, it also passes in an argument of false. ResponseError.getResponse() doesn't have any parameters, so that argument is unnecessary.

Good call. Removed in 753a274


bug(Material 3 > Button): When button is last child element in mat-dialog-content it causes a vertical scrollbar to appear

Is this a regression?

[X] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
v17 (Material 2)
Description
When button is last child element in mat-dialog-content it causes a vertical scrollbar to appear.
Reproduction
StackBlitz link
Steps to reproduce:

Create Angular v18 project + add Angular material v18
Create a dialog component
In dialog component: Add a mat-button in mat-dialog-content
Open dialog

Expected Behavior
There should be no vertical scrollbar in the dialog
Actual Behavior
There is a vertical scroll bar in the dialog
Environment

Angular: "@angular/core": "18.0.0"
CDK/Material: "@angular/material": "18.0.0"
Browser(s): Edge
Operating System: Windows

Angular CLI: 18.0.0
Node: 22.2.0
Package Manager: npm 10.7.0
OS: win32 x64
Angular: 18.0.0
... animations, cdk, cli, common, compiler, compiler-cli, core
... forms, language-service, material, material-luxon-adapter
... platform-browser, platform-browser-dynamic, platform-server
... router
Package                         Version
@angular-devkit/architect       0.1800.0
@angular-devkit/build-angular   18.0.0
@angular-devkit/core            18.0.0
@angular-devkit/schematics      18.0.0
@schematics/angular             18.0.0
rxjs                            7.8.1
typescript                      5.4.5
zone.js                         0.14.6

This is due to the inner touch target element being a larger size than it's container, causing the dialog content to be a height shorter than it should be. A workaround is to add a margin-bottom: 4px to the button. The fix would need some thought since the button is supposed to have a 40px height visually and layout-wise, but for accessibility reasons it needs to be a minimum of 48px.


[couchpatato] deprecate and remove CouchPatato

Details
CouchPatato has been abandonware for years now.
I tried to track why it actually got added last year as a chart, but due to all restructuring git wasn't happy about it. It does show there needs to be more quality vetting of the underlaying applications behind charts.
Running years old webui's on python and such, is a very bad idea and shouldn't be supported in any way, shape or form (besides completely forking couchpatato)
Helm chart name:
couchpatato
Describe the solution you'd like:
Preferably someone moves the chart to a fork or forks the couchpatato project.
If thats not to happen:
At the very least the chart needs to be deprecated.
either directly, or in the long run, followed by complete removal
Anything else you would like to add:
To be clear: i don't want to be "that guy" when it comes to deprication and deleting charts, but there has to be some line drawn in the sand.
No mater where you place that line: years of inactivity would almost always cross it, leading to deprecation/removal
Additional Information:

I am actually okay with this, not only is the app in a unsupported state in terms of commits but the maintainer has shut off every area of people asking for support. GH issues on the repo are disabled and he closed down the couchpotato forums about 6-8 months ago.

Yeah, not a fan of cutting this as we all have a history with this project.
But... yeah...  i'll look into it this week!


Join Late Switch Sync

If you join late, after switches have already been hit, then their state is not synced correctly.

Fixed in 4a9d90a3972af8742d91d0cb4be1d5c458bb578b


Problems with transcription into a specific language

Hi!
I have installed the package and run the Low-Level sample code but I encountered a problem with transcription to specific language when explicitly specifying the language in the parameters ->withLanguage('uk'). But if I remove ->withLanguage('uk') from the parameters the transcription works fine with automatic language detection.
I also tried running High-Level sample code, the problem did not go away when the language was explicitly specified
The log for whisper shows this:
[2024-12-21 12:45:37] whisper.error: whisper_lang_id: unknown language ;h���' []
My environment:
Laravel 11.31 inside docker container, image php:8.3.3-fpm
Inside the container libffi-dev is installed, also the ffi extension is installed
I ran the code with the medium and small models, the result is the same

Can you provide a full example of the script?

I investigated the problem via xdebug, everything is probably fine on the php side, the encoding problem occurs at the FFI stage and beyond.
Here is my code, it runs inside a laravel command

    // Initialize context with a model
    $modelsStoragePath = Storage::disk('models_storage')->path('');
    $audioPath = Storage::disk('uploads')->path('/1/input.mp3');

    Whisper::setLogger(new WhisperLogger(STDOUT));
    $modelPath = ModelLoader::loadModel('medium', $modelsStoragePath);
    $contextParams = WhisperContextParameters::default();
    $ctx = new WhisperContext($modelPath, $contextParams);

    // Create state and set parameters
    $state = $ctx->createState();
    $fullParams = WhisperFullParams::default()
        ->withNThreads(4)
        ->withLanguage('en');

    // Transcribe audio
    $pcm = readAudio($audioPath);
    $state->full($pcm, $fullParams);

    // Process segments
    $numSegments = $state->nSegments();
    for ($i = 0; $i < $numSegments; $i++) {
        $segment = $state->getSegmentText($i);
        $startTimestamp = $state->getSegmentStartTime($i);
        $endTimestamp = $state->getSegmentEndTime($i);

        printf(
            "[%s - %s]: %s\n",
            toTimestamp($startTimestamp),
            toTimestamp($endTimestamp),
            $segment
        );
    }

I found the cause of the problem.
There is a method \Codewithkyrian\Whisper\WhisperFullParams::toCStruct() that collects data into a structure. Everything is fine up to the moment of return $params;, but then in the \Codewithkyrian\Whisper\WhisperState::full() method, where the toCStruct method is called, for some reason I have a broken encoding in the parameters - char* language: b“ÿÿ”.
Most likely the problem is in the garbage collector, I set the parameter owned = false in line 326 and everything worked fine: $language = $ffi->new(“char[$len]”, false);
It doesn't solve my problem, but at least I found the cause

Hi @DimaSmirnoff27,
Thanks so much for taking the time to dig into this issue and sharing the details. You've saved me tons of work, for real.
I’ve been trying to replicate the error on my end but haven’t had any luck so far. I tested on my Mac, a Linux aarch64 VM, an Intel Linux VM, and even a PHP container (serversideup), but everything seems to work fine in those environments. It’s been a bit frustrating since not being able to reproduce the issue makes it tough to track down and fix.
Would you mind sharing the base Docker image you’re using? It’d help me recreate your exact setup and hopefully uncover what’s going wrong. Thanks again for your patience and for flagging this.

Also, the workaround you found with making the cdata owned does hint at this being a GC issue. But even so, the error message in the logs doesn’t seem to reflect the correct string passed in, so it feels like there might be more going on here.
One thing to keep in mind is that making it owned means you’ll have to manually free the memory allocated by that function call. Since the params are used until the transcription process ends, one possible workaround could be maintaining a list of references in the class and then manually disposing of them in __destruct().
That said, without being able to replicate the issue myself, it’s hard to pin this down completely or test a proper fix.

Here is a copy of my project, there is only a test command for transcription. In README I described how to run the docker and the command.
https://gitlab.com/dima.smirnoff27/stt-converter


[Question] how "Custom File Picker" work in iOS

Thanks for the awesome library, I implemented "Custom file picker" and its work fine in android platform with the demo code.

let extensions = [];
if (app.ios) {
extensions = [kUTTypePDF, kUTTypeText]; // you can get more types from here: https://developer.apple.com/documentation/mobilecoreservices/uttype
} else {
extensions = ['txt', 'pdf'];
}
let options: FilePickerOptions = {
android: {
extensions: extensions,
maxNumberFiles: 2
},
ios: {
extensions: extensions,
multipleSelection: true
}
};
let mediafilepicker = new Mediafilepicker();
mediafilepicker.openFilePicker(options);
mediafilepicker.on("getFiles", function (res) {
let results = res.object.get('results');
console.dir(results);
});
mediafilepicker.on("error", function (res) {
let msg = res.object.get('msg');
console.log(msg);
});
mediafilepicker.on("cancel", function (res) {
let msg = res.object.get('msg');
console.log(msg);
});

However when I test with iOS, picker only available to display iCloud file and the file is not selectable. I notice that you are stated the in the limitation that iCloud is not supported.
So may I know what is the usage for pick file in iOS? how can I attached file in iOS platform ?

@marcuschoong if you are using custom file picker in iOS then this limitation won't be applicable because I have used UIDocumentPickerViewController which will copy iCloud file temporary to your app directory.

@jibon57 since the iCould file has temporary copy to the app directory, do you have any plan to open the file in iOS in the future release ?

@marcuschoong the purpose of this plugin isn't to open a file. For opening file you will have to use different extensions.

Hi,
When i use custom file picker... i get an error as 'NSErrorWrapper: You don’t have permission to save the file “com.appid-Inbox” in the folder “tmp”.'
Can you please help me with the same?

@harshitjain0803 see my solution here: https://github.com/jibon57/nativescript-mediafilepicker/issues/103#issuecomment-694159938


Kotlin gRPC service with blocking execution context: not returning result with multiple concurrent calls

When I execute multiple concurrent calls to a service, which is based on Kotlin coroutines and uses blockingExecutionContext, some calls return OK status but no response. In this case client reports an error (I tried both java & kotlin clients - error message is slightly different, but result is the same). With the non-blocking context there's no issue.
Steps to reproduce:
In armeria grpc-kotlin examples do the following changes:

in service function HelloServiceImpl.blockingHello decrease the sleep timeout value (e.g. make it Thread.sleep(10)). (Interesting, that all works OK with bigger timeouts (eg 500))
Change test method HelloServiceTest.replyFromServerSideBlockingCall to the following:

    @ParameterizedTest
    @MethodSource("uris")
    fun replyFromServerSideBlockingCall(uri: String) {
        runBlocking {
            val helloService = Clients.newClient(uri, HelloServiceCoroutineStub::class.java)
            repeat(30) {
                launch {
                    assertThat(helloService.blockingHello(HelloRequest.newBuilder().setName("Armeria").build()).message)
                            .isEqualTo("Hello, Armeria!")
                }
            }
        }
    }

After running this test, I get a consistent exception:
INTERNAL: Expected one request for MethodDescriptor{fullMethodName=example.grpc.hello.HelloService/BlockingHello, type=UNARY, idempotent=false, safe=false, sampledToLocalTracing=true, requestMarshaller=io.grpc.protobuf.lite.ProtoLiteUtils$MessageMarshaller@5298be71, responseMarshaller=io.grpc.protobuf.lite.ProtoLiteUtils$MessageMarshaller@29f896c1, schemaDescriptor=example.armeria.grpc.kotlin.HelloServiceGrpc$HelloServiceMethodDescriptorSupplier@72c26864} but received none
io.grpc.StatusException: INTERNAL: Expected one request for MethodDescriptor{fullMethodName=example.grpc.hello.HelloService/BlockingHello, type=UNARY, idempotent=false, safe=false, sampledToLocalTracing=true, requestMarshaller=io.grpc.protobuf.lite.ProtoLiteUtils$MessageMarshaller@5298be71, responseMarshaller=io.grpc.protobuf.lite.ProtoLiteUtils$MessageMarshaller@29f896c1, schemaDescriptor=example.armeria.grpc.kotlin.HelloServiceGrpc$HelloServiceMethodDescriptorSupplier@72c26864} but received none
	at io.grpc.kotlin.HelpersKt$singleOrStatusFlow$1.invokeSuspend(Helpers.kt:73)
	at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33)
	at kotlinx.coroutines.internal.ScopeCoroutine.afterResume(Scopes.kt:32)
	at kotlinx.coroutines.AbstractCoroutine.resumeWith(AbstractCoroutine.kt:113)
	at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:46)
.....etc...

I am not sure if this is a problem in Armeria itself or in kotlin-grpc...

Hi @smax48, I wonder how you implement helloService.blockingHello.
Do you block Armeria's event loop inside blockingHello by any chance such as Thread.sleep(...)

If you used Armeria's grpc-kotlin example, let me reproduce it.

I've run your test case locally and it works without any exceptions or error messages.

Filed an issue(https://github.com/grpc/grpc-kotlin/issues/151) about the unexpected behavior of ServerCall.Listener.onHalfClose() in gRPC-Kotlin


Is it possible to check if a task + arguments have already been scheduled?

If so - how?

This doesn't seem to work in version 4.1.0. Always get an empty array even with pending jobs. Any clue?


Update to latest pinning API

PR https://github.com/rust-lang/rust/pull/53877 change the pinning API, breaking futures as well. Is someone working on fixing this? Or should I make an attempt?

I was planning to wait on this until https://github.com/rust-lang/rust/pull/54339 landed, which includes more breaking changes.

@cramertj I believe this can be closed now that #1268 and #1273 have been merged and #1266 closed.


Create material brush tool

New tool to simplify applying materials.  Click and drag to assign current material.

Hello! Will this tool also allow to paint materials on specific faces?

Yes.  If you click on the Individual face checkbox in the tool properties, it will paint individual faces.  The erase option has not been implemented yet.

The paint material brush is committed now.


suggest a design principle: VM should behave like a native workload

Not only do we keep native workloads intact, we aim to have VM-based workloads behave just like native ones.
What this PR does / why we need it:
I understand that this design principle is well know and agreed upon. However it is quite implicit, and its importance was not clear to me. I would like it to be better understood and followed.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Special notes for your reviewer:
@fabiand @rmohr @simon3z
Release note:
NONE

My point is that the architecture must ALLOW the cluster owner to say "let anyone run VMs" without compromising their security, without a need to grant further rights to users. The cluster owner can still say that users starting with f* are not allowed to use VMs, but that's besides my point.
For example, if an Operator wants to start a VM with sriov access, he must not need any more rights than a user willing to run sriov in a Pod. This is, currently, not the case, and we need to fix it because it contradict a basic design principle.

@booxter please take a look. For you this was obvious - but not for me. That's the reason we must make sure sriov in VMs does not require any more user permission relative to sriov in Pods.

Thanks for clarification and adjustment.

Awaiting @booxter feedback, then we can merge it.

@dankenigsberg the wording is fine but I think we have a disconnect after all about sriov specifics. As it stands now, a user does NOT need additional privileges allocated to them to run a SR-IOV attached VMI. The issue with privileges with SR-IOV is NOT that users need additional RBAC rules but that launcher pods are running privileged (while there are ways to avoid it now with fresh SR-IOV DP), so anyone who can start a VMI and attach it to a SR-IOV network can effectively get a privileged container available to them. So, again, the issue is NOT that a user can't do something but that the user CAN do more with the SR-IOV enabled VMI than with a regular VMI (perhaps more than they should be able to).

Thanks for approving my text.
I suspect that I don't understand something basic, so please break it down to me.
As an unprivileged user:

may I receive a Pod with a VF in it? (I hope I do!)
may I receive a VMI with a VF in it? (I hope I don't, because currently it means that I have gained access to a privileged Pod and I can pwn the host)

IOW:
If a user uses a pod with SRIOV. the container is unprivileged.
If a user uses a VMI with SRIOV, the launcher container is privileged.
If a user uses a VMI without SRIOV, the launcher container is not privileged.
This is probably because we need extra privileges inside the container to permit libvirt to consume the device.

What @fabiand said. The only thing to add is that we allocate privileges to the launcher pod so that libvirt can configure the device for vfio consumption. With the latest builds of SR-IOV device plugin*, we should be able to avoid privilege escalation by 1) pre-registering all devices with vfio as pre-deployment step; and 2) removing the privileged=true from launcher pods; and 3) making sure that libvirt doesn't do anything silly like assuming that it should do the registration itself and crashing, or unregistering vfio devices on exit. This is future work.

the plugin now can register vfio device inodes against container device plugin

I've merged the PR since it makes general sense even if it came from a wrong place (specifically, from misunderstanding of the issues we have with SR-IOV privileges - it's not that users may not have enough but that they may have too much). If we would follow up on this particular SR-IOV case here we would probably also state that no feature should escalate privileges for virt-* pods owned by users beyond minimally viable. This would be a separate discussion and a separate PR though.

@booxter can you please file an issue to track the privilege removal for sriov-ed VMs?


Although this device also do the receive operation from other device, receiving operation also stop when this issue happen.
Exception or Stack Trace
com.azure.messaging.servicebus.ServiceBusException: Did not observe any item or terminal signal within 60000ms in 'takeUntil' (and no fallback has been configured)
        at com.azure.messaging.servicebus.ServiceBusSenderAsyncClient.mapError(ServiceBusSenderAsyncClient.java:734)
        at com.azure.messaging.servicebus.ServiceBusSenderAsyncClient.lambda$sdnkNTfOSsjt71Sge2CDNDwGsp8(Unknown Source:0)
        at com.azure.messaging.servicebus.-$$Lambda$ServiceBusSenderAsyncClient$sdnkNTfOSsjt71Sge2CDNDwGsp8.apply(Unknown Source:4)
        at reactor.core.publisher.Mono.lambda$onErrorMap$30(Mono.java:3286)
        at reactor.core.publisher.-$$Lambda$Mono$XLTmz-olhTxwhVTbz1w4HO4CnSc.apply(Unknown Source:4)
        at reactor.core.publisher.FluxOnErrorResume$ResumeSubscriber.onError(FluxOnErrorResume.java:88)
        at reactor.core.publisher.MonoFlatMap$FlatMapMain.onError(MonoFlatMap.java:165)
        at reactor.core.publisher.FluxPeekFuseable$PeekFuseableSubscriber.onError(FluxPeekFuseable.java:227)
        at reactor.core.publisher.MonoFlatMap$FlatMapMain.secondError(MonoFlatMap.java:185)
        at reactor.core.publisher.MonoFlatMap$FlatMapInner.onError(MonoFlatMap.java:251)
        at reactor.core.publisher.MonoFlatMap$FlatMapMain.secondError(MonoFlatMap.java:185)
        at reactor.core.publisher.MonoFlatMap$FlatMapInner.onError(MonoFlatMap.java:251)
        at reactor.core.publisher.FluxMapFuseable$MapFuseableSubscriber.onError(FluxMapFuseable.java:134)
        at reactor.core.publisher.Operators$MonoSubscriber.onError(Operators.java:1829)
        at reactor.core.publisher.MonoIgnoreThen$ThenIgnoreInner.onError(MonoIgnoreThen.java:235)
        at reactor.core.publisher.SerializedSubscriber.onError(SerializedSubscriber.java:124)
        at reactor.core.publisher.FluxRetryWhen$RetryWhenMainSubscriber.whenError(FluxRetryWhen.java:213)
        at reactor.core.publisher.FluxRetryWhen$RetryWhenOtherSubscriber.onError(FluxRetryWhen.java:255)
        at reactor.core.publisher.FluxFlatMap$FlatMapMain.checkTerminated(FluxFlatMap.java:834)
        at reactor.core.publisher.FluxFlatMap$FlatMapMain.drainLoop(FluxFlatMap.java:600)
        at reactor.core.publisher.FluxFlatMap$FlatMapMain.drain(FluxFlatMap.java:580)
        at reactor.core.publisher.FluxFlatMap$FlatMapMain.onError(FluxFlatMap.java:443)
        at reactor.core.publisher.FluxZip$ZipCoordinator.drain(FluxZip.java:747)
        at reactor.core.publisher.FluxZip$ZipInner.onNext(FluxZip.java:911)
        at reactor.core.publisher.FluxMap$MapSubscriber.onNext(FluxMap.java:114)
        at reactor.core.publisher.DirectProcessor$DirectInner.onNext(DirectProcessor.java:339)
        at reactor.core.publisher.DirectProcessor.onNext(DirectProcessor.java:148)
        at reactor.core.publisher.SerializedSubscriber.onNext(SerializedSubscriber.java:99)
        at reactor.core.publisher.FluxRetryWhen$RetryWhenMainSubscriber.onError(FluxRetryWhen.java:180)
        at reactor.core.publisher.SerializedSubscriber.onError(SerializedSubscriber.java:124)
        at reactor.core.publisher.FluxTimeout$TimeoutMainSubscriber.handleTimeout(FluxTimeout.java:288)
        at reactor.core.publisher.FluxTimeout$TimeoutMainSubscriber.doTimeout(FluxTimeout.java:273)
        at reactor.core.publisher.FluxTimeout$TimeoutTimeoutSubscriber.onNext(FluxTimeout.java:395)
        at reactor.core.publisher.StrictSubscriber.onNext(StrictSubscriber.java:89)
        at reactor.core.publisher.FluxOnErrorResume$ResumeSubscriber.onNext(FluxOnErrorResume.java:73)
        at reactor.core.publisher.MonoDelay$MonoDelayRunnable.run(MonoDelay.java:117)
        at reactor.core.scheduler.SchedulerTask.call(SchedulerTask.java:68)
        at reactor.core.scheduler.SchedulerTask.call(SchedulerTask.java:28)
        at java.util.concurrent.FutureTask.run(FutureTask.java:266)
        at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:301)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1167)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:641)
        at java.lang.Thread.run(Thread.java:764)
     Caused by: java.util.concurrent.TimeoutException: Did not observe any item or terminal signal within 60000ms in 'takeUntil' (and no fallback has

java.util.concurrent.TimeoutException: Did not observe any item or terminal signal within 3000ms in 'onErrorResume' (and no fallback has been configured)
        at reactor.core.publisher.FluxTimeout$TimeoutMainSubscriber.handleTimeout(FluxTimeout.java:288)
        at reactor.core.publisher.FluxTimeout$TimeoutMainSubscriber.doTimeout(FluxTimeout.java:273)
        at reactor.core.publisher.FluxTimeout$TimeoutTimeoutSubscriber.onNext(FluxTimeout.java:395)
        at reactor.core.publisher.StrictSubscriber.onNext(StrictSubscriber.java:89)
        at reactor.core.publisher.FluxOnErrorResume$ResumeSubscriber.onNext(FluxOnErrorResume.java:73)
        at reactor.core.publisher.MonoDelay$MonoDelayRunnable.run(MonoDelay.java:117)
        at reactor.core.scheduler.SchedulerTask.call(SchedulerTask.java:68)
        at reactor.core.scheduler.SchedulerTask.call(SchedulerTask.java:28)
        at java.util.concurrent.FutureTask.run(FutureTask.java:266)
        at java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask.run(ScheduledThreadPoolExecutor.java:301)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1167)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:641)
        at java.lang.Thread.run(Thread.java:764)

To Reproduce
1.Sending a message (async) to the specified topic every 3 seconds.
2.The message is initially sent as expected.
3.After a while (2-5 minutes), sending message will stop.
Just wait a few minutes and we'll be able to send the message again.
(It has not been resolved. Sending will stop after a certain period of time.)
The sending function seems to be executed, but it looks like that  the completion flow doesn't run, we guess.
Code Snippet
1.Creat Client
    public ServiceBusSenderAsyncClient getSenderClient(String topicName) {
        return new ServiceBusClientBuilder()
                .connectionString(rootConnectionString)
                .sender()
                .topicName(topicName)
                .buildAsyncClient();
    }

2.Sending message
protected void sendMessage(ServiceBusSenderAsyncClient sender, String messageText, String userName) {
        try {
            String deviceId = "xxxx-xxxx-xxxx-xxxx";
            ServiceBusMessage m = new ServiceBusMessage(messageText.getBytes())
                    .setSubject(userName)
                    .setCorrelationId(deviceId)
                    .setContentType("application/json");
            // Sending message
            sender.sendMessage(m).subscribe(unused -> {
                }, error -> {
                    Log.e("[Failed to send message]", Log.getStackTraceString(error));
                }, () -> {
                    Log.i("[Completed]", "Completed to send message");
            });
    }

3.Sending message on a regular basis.
We use the startService(new Intent(packageContext, serviceClassName)) for starting the service.
    public int onStartCommand(Intent intent, int flags, int startId) {
        super.onStartCommand( intent, flags, startId );
        // executed another thred
        schedule = Executors.newSingleThreadScheduledExecutor();
        schedule.scheduleAtFixedRate(new Runnable() {
            @Override
            public void run() {
                // send message
                String message = "";
                try {
                    sendMessage(sender, message);
                } catch (IllegalStateException e) {
                    Log.e("UnSent", Log.getStackTraceString(e));
                    sender.close();
                    sender = getSenderClient(topic);
                }
            }
        }, 0, interval, TimeUnit.MILLISECONDS);
        return START_STICKY_COMPATIBILITY;
    }

Expected behavior
We can send the message without stopping
Setup (please complete the following information):

OS: Android 9（Lenovo TB X606X）
Version of the Library used
azure-messaging-servicebus : 7.0.0
minSdkVersion : 26

@kekatahi  I am looking into it and will update you after I try to replicate it. Meanwhile I have a question.
Have you seen this issue on Windows/Linux using Java 9 or Java 11 ? Or your use case is to run on  Mobile App using "Android 9" ?

@kekatahi  I am looking into it and will update you after I try to replicate it. Meanwhile I have a question.
Have you seen this issue on Windows/Linux using Java 9 or Java 11 ? Or your use case is to run on  Mobile App using "Android 9" ?

I haven't seen similar issue Java 9 ,11. Yes , true. Customer use Android 9 on their mobile app.
Customer agreed to close the case because they use the rest api and this issue never happened so far.
Before the case close , I hope I could follow up the issue.
Thanks,
Kentaro
From: Hemant Tanwar<EMAIL_ADDRESS>Sent: Thursday, January 14, 2021 8:46 AM
To: Azure/azure-sdk-for-java<EMAIL_ADDRESS>Cc: Kentaro Katahira<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Azure/azure-sdk-for-java] [BUG] Sending message stop after certain period of time (#18293)
@kekatahihttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fkekatahi&data=04|01|Kentaro.Katahira%40microsoft.com|6ee69e4b6b5943a6330608d8b81d6bb7|72f988bf86f141af91ab2d7cd011db47|1|0|637461783791534806|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=0A5his6RdABf9PazQridur%2BUyG9s89Onp4IB06EdHM4%3D&reserved=0 I am looking into it and will update you after I try to replicate it. Meanwhile I have a question.
Have you seen this issue on Windows/Linux using Java 9 or Java 11 ? Or your use case is to run on Mobile App using "Android 9" ?

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-sdk-for-java%2Fissues%2F18293%23issuecomment-759829524&data=04|01|Kentaro.Katahira%40microsoft.com|6ee69e4b6b5943a6330608d8b81d6bb7|72f988bf86f141af91ab2d7cd011db47|1|0|637461783791544751|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=QUKwcKJ2NeqLflxQyqXo5gK%2FqDTmd3P%2B8ym1WBfHje8%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FASHEO2E3FOP545O5NKXWJG3SZYWERANCNFSM4VEH5YKA&data=04|01|Kentaro.Katahira%40microsoft.com|6ee69e4b6b5943a6330608d8b81d6bb7|72f988bf86f141af91ab2d7cd011db47|1|0|637461783791544751|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=B5%2BXBkhwb8GcIcyiHRVEHtBgxzbPch9XgklvkfXIXBo%3D&reserved=0.

I haven't seen similar issue Java 9 ,11. Yes , true. Customer use Android 9 on their mobile app.
Customer agreed to close the case because they use the rest api and this issue never happened so far.
Before the case close , I hope I could follow up the issue.
Thanks,
Kentaro
From: Hemant Tanwar<EMAIL_ADDRESS>Sent: Thursday, January 14, 2021 8:46 AM
To: Azure/azure-sdk-for-java<EMAIL_ADDRESS>Cc: Kentaro Katahira<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [Azure/azure-sdk-for-java] [BUG] Sending message stop after certain period of time (#18293)
@kekatahihttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fkekatahi&data=04|01|Kentaro.Katahira%40microsoft.com|6ee69e4b6b5943a6330608d8b81d6bb7|72f988bf86f141af91ab2d7cd011db47|1|0|637461783791534806|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=0A5his6RdABf9PazQridur%2BUyG9s89Onp4IB06EdHM4%3D&reserved=0 I am looking into it and will update you after I try to replicate it. Meanwhile I have a question.
Have you seen this issue on Windows/Linux using Java 9 or Java 11 ? Or your use case is to run on Mobile App using "Android 9" ?

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-sdk-for-java%2Fissues%2F18293%23issuecomment-759829524&data=04|01|Kentaro.Katahira%40microsoft.com|6ee69e4b6b5943a6330608d8b81d6bb7|72f988bf86f141af91ab2d7cd011db47|1|0|637461783791544751|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=QUKwcKJ2NeqLflxQyqXo5gK%2FqDTmd3P%2B8ym1WBfHje8%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FASHEO2E3FOP545O5NKXWJG3SZYWERANCNFSM4VEH5YKA&data=04|01|Kentaro.Katahira%40microsoft.com|6ee69e4b6b5943a6330608d8b81d6bb7|72f988bf86f141af91ab2d7cd011db47|1|0|637461783791544751|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=B5%2BXBkhwb8GcIcyiHRVEHtBgxzbPch9XgklvkfXIXBo%3D&reserved=0.

@kekatahi I was able to run a sample android application  which ran for 10 minutes and sent one message every 5 seconds  and 122 messages in-total.   I am not able to replicate the issue.
My setting is following.
minSdkVersion 26
JDK 1.8
Android Studio 4.1
I am not android expert, but talking to someone, the scheduling of thread which run your service is controlled and android OS controls it and it can stop it as well.  You might want to implement following call back to understand what is happening.
   @Override
    public void onDestroy() {
      // Log this.
    }

From SDK point of view: By default if there is no activity by sender for 60 seconds , it will close underlaying connections, that is what I see in your exception. You can follow that discussion here also https://github.com/Azure/azure-sdk-for-java/issues/18123

@kekatahi I was able to run a sample android application  which ran for 10 minutes and sent one message every 5 seconds  and 122 messages in-total.   I am not able to replicate the issue.
My setting is following.
minSdkVersion 26
JDK 1.8
Android Studio 4.1
I am not android expert, but talking to someone, the scheduling of thread which run your service is controlled and android OS controls it and it can stop it as well.  You might want to implement following call back to understand what is happening.
   @Override
    public void onDestroy() {
      // Log this.
    }

From SDK point of view: By default if there is no activity by sender for 60 seconds , it will close underlaying connections, that is what I see in your exception. You can follow that discussion here also https://github.com/Azure/azure-sdk-for-java/issues/18123

Since we found no issue, I am closing it. If you need more clarification, you can reopen it or create new issue

Since we found no issue, I am closing it. If you need more clarification, you can reopen it or create new issue


Sometime bot stuck, no processing battle

when run this bot, and start today, i see stuck process battle.

likely due to some maintenance on server or connection issue. Rarely I restart it twice to make sure all battles completed.

Ya, i always restart this bot when stuck.. but this is always happen.. is this from my connection or metamon connection ya?  i already speedtest our connection is stable, and i turn off my antivirus and firewall too, and still same, bot like stuck in only 2-3 battle. never completed all, this happen already 2 days ago

might be IPv6 issue, I had to disable linking in network settings as I wass getting network changed screen on web pages from time to time.


Added a working paginator

Waiting for author of this PR ( @e-xe ).

Waiting for author of this PR ( @e-xe ) to address remaining requested changes and merge master into this branch in order to get it up to date with the current version of the repository.

It is being worked on, and there will be a completely new paginator coming up.

If this is only a WIP PR, please convert it to a draft PR to avoid confusion, Open pull requests are the ones waiting for review or waiting for the author to address new requested changes, not for adding new things which are still unfinished


your system has not been modified

when i try to install twitch leecher i got a message "your system has not been modified" and i tried all the solutions provided but always the same issue.
i need help as soon as possible

When you create an issue there is the following hint:
"Please read
https://github.com/Franiac/TwitchLeecher#support--issues
before posting anything!"
So do that please...

Closed because stale


Report encodings

The following encodings should be collected and reported in the sys module section:

sys.stdout.encoding
sys.stderr.encoding
sys.stdin.encoding
sys.getfilesystemencoding()

Originally suggested in #10


Doesn't recognize custom benches/bins

Only seems to pull default bins/benches from RCT2. Won't recognize any custom benches or bins.

@cruzy80 When you say "custom" what do you mean? Like stuff you've downloaded?
They might not actually be registering as benches or bins. We are querying OpenRCT2 for all of the footpath_addition object types that are loaded into the game right here, which is how we populate the dropdowns and allow you to choose a bench and bin.
 If not, they may just be scenery additions and not actually functional.

Hi, they are custom benches that can be sat on.  Is it where they are
saved? My custom bench scenery is saved under the Open RCT Object Directory
while the default scenery is under the Steam RCT2 directory. Also the
benches that Benchwarmer does recognize has the extension .json, while
everything custom I;ve downloaded is .dat  also I've tried copying the
bench .dats to the Steam RCT2 folder, but it doesn't work.
,
[image: image.png]
[image: image.png]
On Mon, May 23, 2022 at 2:54 PM Tom Scott @.***> wrote:

@cruzy80 https://github.com/cruzy80 When you say "custom" what do you
mean? Like stuff you've downloaded?
They might not actually be registering as benches or bins. We are
querying OpenRCT2 for all of the footpath_addition object types that are
loaded into the game right here
https://github.com/tubbo/openrct2-benchwarmer/blob/master/src/index.ts#L10-L11,
which is how we populate the dropdowns and allow you to choose a bench and
bin.
 If not, they may just be scenery
additions and not actually functional.
—
Reply to this email directly, view it on GitHub
https://github.com/tubbo/openrct2-benchwarmer/issues/57#issuecomment-1135027633,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZERVZ6IRZIXI7DWTWMOAZLVLPH63ANCNFSM5WFQVQUA
.
You are receiving this because you were mentioned.Message ID:
@.***>

@cruzy80 Whatever images you sent did not come through, unfortunately. The benches/bins that Benchwarmer is using come from this line of code, which calls into OpenRCT2 to see what objects we have available. If your scenery isn't making it into that list, the game itself is not recognizing it as a "footpath addition", so we won't be able to know it's there.


"Leave behind list items" lag/jump at edge of screen

This was reported internally b/77967901, I can reproduce in the Flutter Gallery on my iPhone 6s.  I suspect it occurs on Android too, but I haven't tried.

Closing as dup of https://github.com/flutter/flutter/issues/16529


LibraryEngine: Handling of binaryTargets in error messages missing

BinaryEngine has this whole concept of incorrectlyPinnedBinaryTarget, which is used to output a "helpful" error message:
https://github.com/prisma/prisma/blob/fe3d63cc11678d756732ad14887eea62b2590c36/packages/engine-core/src/binary/BinaryEngine.ts#L398-L402
https://github.com/prisma/prisma/blob/fe3d63cc11678d756732ad14887eea62b2590c36/packages/engine-core/src/binary/BinaryEngine.ts#L474-L484
For LibraryEngine the same error message is used without the actual check if it is incorrectly pinned via binary target, leading to even more misleading error messages: https://github.com/prisma/prisma/blob/fe3d63cc11678d756732ad14887eea62b2590c36/packages/engine-core/src/library/LibraryEngine.ts#L585-L590 (it will tell you you pinned wrong just because the file is missing)
This whole concept seems untested as well.

Hi there,
As part of our effort to clean up our GitHub repo, we’re closing this issue to help focus on features and improvements that align with our focus areas and those of our community.
This issue was originally created for internal tracking purposes, but if you feel this is important to you, feel free to reopen the issue and get like-minded users to upvote as we will prioritize fixing issues based on community interest.
For more insight into our priorities and how we’re thinking about the future of Prisma, please see our blog post: https://www.prisma.io/blog/prisma-orm-manifesto.
Thank you for your understanding and support!


Change description/usage order

When calling via command line the help shows usage then description (see official argparse example).
The html-docs show description before usage which may be odd depending on how one uses description.
It would be nice if either the order is changed in general or a flag like :usage_first: existed.
I can open a PR for either one of the solutions if you tell me which one is preferable.

Let's add a flag 👍


refactor: padding of result text

Resolves #11

applied hover effect on 'Rules' Button

hover effect is already present with background-color: rgb(203, 153, 228);

Added the required changes. Please review the PR


Alias + Filter not generating correct json

NEST/Elasticsearch.Net version: 6.3.1/6.3.1
Elasticsearch version: N/A
Description of the problem including expected versus actual behavior:
When creating a filter on an alias the expected JSON is not generated to send to ElasticSearch.
Steps to reproduce:
public class MyObject
{
	public DateTimeOffset StartDate { get; set; }
	public DateTimeOffset EndDate { get; set; }
}

var createdIndexResponse = client.CreateIndex(_myIndexName, index =>
	index.Mappings(mappings => mappings
			.Map<MyObject>(obj => obj
				.AutoMap()
			)
		).Aliases(aliases => aliases
			.Alias(_myAliasName, a => a
				.Filter<BoolQueryDescriptor<DateRange>>(filterSelector => filterSelector
					.Bool(filter => filter
						.Must(query => query
							.DateRange(selector => selector
								.Name("startDate")
								.LessThan(DateMath.Now.Add("1d"))
							)
						).Must(query => query
							.DateRange(selector => selector
								.Name("endDate")
								.GreaterThan(DateMath.Now)
							)
						)
					)
				)
			)
		)
	);

This generates the following json (roughly - adapted from my actual example):
{
"aliases": {
        "aliasName": {}
    },
    "mappings": {
        "defaultObjectName": {
            "properties": {
                "startDate": {
                    "type": "date"
                },
                "endDate": {
                    "type": "date"
                },
            }
        }
    }
}

When I expected something like this instead:
"aliases": {
        "aliasName": {
              "filter": {
                    "bool": {
                        "must": [
                            {
                                "range": {
                                    "startDate": {
                                        "lt": "now+1d/d"
                                    }
                                }
                            },
                            {
                                "range": {
                                    "endDate": {
                                        "gt": "now/d"
                                    }
                                }
                            }
                        ]
                    }
                }
      }
    },
    "mappings": {
        "defaultObjectName": {
            "properties": {
                "startDate": {
                    "type": "date"
                },
                "endDate": {
                    "type": "date"
                },
            }
        }
    }
}

Also tried BoolQueryDescriptor, BoolQueryDescriptor, BoolQueryDescriptor BoolQuery, BooleanPropertyDescriptor, BooleanPropertyDescriptor, BooleanPropertyDescriptor and a host of other combinations

The issue is that both DateRange queries don't specify a field to execute against; .Name() is to name a query so that in cases of compound queries, you can ascertain which queries a document was a match for. You want to use .Field() here instead, which is the field to target. Here's an example
var client = new ElasticClient();

var _myAliasName = "my-alias";
var _myIndexName = "myIndexName";

var createdIndexResponse = client.CreateIndex(_myIndexName, index => index
    .Mappings(mappings => mappings
        .Map<MyObject>(obj => obj
            .AutoMap()
        )
    )
    .Aliases(aliases => aliases
        .Alias(_myAliasName, a => a
            .Filter<MyObject>(f => f
                .DateRange(selector => selector
                    .Field("startDate")
                    .LessThan(DateMath.Now.Add("1d"))
                    .GreaterThan(DateMath.Now)
                )
            )
        )
    )
);

which emits the following request JSON
{
  "aliases": {
    "my-alias": {
      "filter": {
        "range": {
          "startDate": {
            "gt": "now",
            "lt": "now+1d"
          }
        }
      }
    }
  },
  "mappings": {
    "_doc": {
      "properties": {
        "startDate": {
          "type": "date"
        },
        "endDate": {
          "type": "date"
        }
      }
    }
  }
}

I've combined both range operators into one query too as it's semantically identical and easier to write :)
Hope this helps. I'm going to close this 👍

Also forgot to say; Latest NEST 6.x nuget package is tested and supported to work with all versions of Elasticsearch 6.x. Newer package versions may contain newer APIs not available in the Elasticsearch version you are targeting, and we try our best to call these out in XML docs. But more importantly, newer package versions also include bug fixes, so recommend updating to newer versions in the same major, whenever you get a chance.

Thank you thank you thank you!


Rotation modifies z-index on each zoom

Hello!
I am using leaflet-django with leaflet 1.6.0.
I have polylines and many markers on my map. Some markers should be visible all the time and I used z-index of markerPane for that. But after adding RotatedMarker I have styles like following:
element.style {
    margin-left: -16px;
    margin-top: -16px;
    width: 32px;
    height: 32px;
    transform: rotate(102deg);
    z-index: 324;
    transform-origin: center center;
}

And z-index forcibly changes on every zoom or update to some random value.
I want to have some control over z-index.

For now I use CSS-style for all markers with !important.

is that code still working on the current version


Doesn’t compile on 32 bit platforms

It doesn’t compile on 32 bit platforms.
src/alloc.rs:5:32: 5:39 error: bitshift exceeds the type's number of bits, #[deny(exceeding_bitshifts)] on by default
src/alloc.rs:5 const MAX_ALLOC_SIZE: usize = (1 << 32) - 1;
                                              ^~~~~~~

This should be fixed on master! Thanks.


How to remove the Success trext from QuickAlertType.success

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Hi @AshwinKumar021 👋
You can use the title parameter in the QuickAlert.show() method to set the title.
You can set it to empty string to remove the title.
Example:
QuickAlert.show(
 context: context,
 type: QuickAlertType.success,
 title: ' ',
);

ok let me try... thanks


Unhandled Rejection (TypeError): Cannot read property 'data' of undefined

Just clone the repo, install modules and run develop.
When press on login button with the credentials it throw:
Unhandled Rejection (TypeError): Cannot read property 'data' of undefined
_callee$
src/components/Login.js:11
8 | const [password, setPassword] = useState('')
9 | const [error, setError] = useState('')
10 |

11 | const handleSubmit = async (event) => {
12 |   event.preventDefault()
13 |   try{
14 |     await login({ identifier, password })

You can use the chrome debugging tool to figure out what's wrong. I would put my money on strapi url.

Check the '.env.development' file; spelling, content, etc.
I'm running both strapi and gatsby in docker images, so I had to set the ip address to my physical machine

Hope this helps

Hey, I just ran into this issue too. Referencing the Strapi docs, I changed a line in useAuth.js:
${apiURL}/auth/local needs to change to ${apiURL}/api/auth/local
Hope that helps!


Proposal: Resolve-Path with -Relative parameter should allow the use of a reference path

Summary of the new feature/enhancement
As a PowerShell developer, I want to be able to obtain the path of an item relative to a path that is not my current directory so that I can avoid changing my current location context for a relative directory lookup.
Proposed technical implementation details
Assuming I have a directory like this:
/MyProject
    |-/dir
        |-test_1.txt
        |-test_2.txt
        |-/subdir
            |-test_3.txt

Currently, if I want to do a resolve a path relative to a different directory, I have to use set-location before calling Resolve-Path. For example, if I wanted to get all my .txt file locations relative to dir, I would have to do something like this:
/MyProject> cd dir
/MyProject/dir> Get-ChildItem -Recurse -Include "*.txt" | Resolve-Path -Relative
./subdir/test_3.txt
./test_1.txt
./test_2.txt
/MyProject/dir> cd ..

Ideally, I would like to do something like this:
/MyProject> Get-ChildItem .\dir -Recurse -Include "*.txt" | Resolve-Path -Relative -ReferencePath .\dir
./subdir/test_3.txt
./test_1.txt
./test_2.txt

Totally agree. We shouldn't have to Push-Location just to use this feature.

You could use Join-Path -Resolve for this:
PS> Join-Path $home\Downloads ../Documents -Resolve
C:\Users\Keith\Documents

This also has the benefit of correcting the directory sep char for the platform.

@rkeithhill  I don't think that'll work for this particular problem. It appears that the -Resolve switch on Join-Path is only for checking whether the resulting joined path actually exists. At least on the surface, it doesn't appear to help with relative resolution. From the docs:
-Resolve
Indicates that this cmdlet should attempt to resolve the joined path from the current provider.

* If wildcards are used, the cmdlet returns all paths that match the joined path.
* If no wildcards are used, the cmdlet will error if the path does not exist.

This is my workaround:
function Resolve-RelativePath {
	[CmdletBinding()]
	param(
		[parameter(Mandatory, ValueFromPipeline)]$Path,
		$ReferencePath="."
	)
	$Path = (Resolve-Path $Path).Path
	$ReferencePath = (Resolve-Path $ReferencePath).Path

	Write-Verbose "path: $path, reference: $ReferencePath"
	$PSCmdlet.SessionState.Path.NormalizeRelativePath($Path, $ReferencePath)
}

This would help mitigate #10509 .  Would love to see some traction on this, as it seems pretty simple.

What if we add new parameter for base path Resolve-Path -BasePath as optional for Relative parameter?


Would Microsoft's PowerQuery fit?

Hi,
Thanks for this list.
Not sure if you have considered Microsoft's Power Query?
I love Power Query and regard it a relational programmer's Swiss Army knife.
It includes query-folding where it can translate queries into SQL and offload them to data sources.
All the best,
David

Interesting, thanks for the pointer! I'll have to read up on Power Query and query folding to see if it fits.


IDEA: Prettier tip - formatting all files

Including the code below in the Readme may help people who are looking to format their entire application.
prettier --write . or prettier --write .

Hey @ACPK, thanks for your suggestion! I think we're going to leave this out for now. We already link to the Prettier website and if people are unfamiliar with Prettier they can read the documentation there 👍


[JavaScript] Different widths on identical grandchild nodes

Report
See the example code below.  My guess is that this is a rounding error, since the discrepancy doesn't occur when the root node is given a slightly different fixed width or when the root node is not given justify-content: space-around.

[x] I have searched existing issues and this is not a duplicate

Issues and Steps to Reproduce
Run the following code in a browser:
import * as yoga from "yoga-layout";

const root = yoga.Node.create();
root.setJustifyContent(yoga.JUSTIFY_SPACE_AROUND);
root.setFlexDirection(yoga.FLEX_DIRECTION_ROW);
root.setAlignItems(yoga.ALIGN_FLEX_START);
root.setWidth(200);
root.insertChild(makeChildNode(), 0);
root.insertChild(makeChildNode(), 1);
root.insertChild(makeChildNode(), 2);

function makeChildNode() {
    const node = yoga.Node.create();
    node.setAlignItems(yoga.ALIGN_FLEX_START);
    node.setWidth(50);
    const child = yoga.Node.create();
    child.setMeasureFunc(() => new yoga.Size(1, 1));
    node.insertChild(child, 0);
    return node;
};

root.calculateLayout(yoga.UNDEFINED, yoga.UNDEFINED, yoga.DIRECTION_LTR);
(function printRecursive(node, indent) {
    const {left, top, width, height} = node.getComputedLayout();
    const info = `left:${left} top:${top} width:${width} height:${height}`;
    console.log(indent + info);
    for (let i = 0; i < node.getChildCount(); i++) {
        printRecursive(node.getChild(i), indent + "    ");
    }
})(root, "");

Expected Behavior
All of the leaf nodes have the same dimensions (1px by 1px).
left:0 top:0 width:200 height:1
    left:8 top:0 width:50 height:1
        left:0 top:0 width:1 height:1
    left:75 top:0 width:50 height:1
        left:0 top:0 width:1 height:1
    left:142 top:0 width:50 height:1
        left:0 top:0 width:1 height:1

Actual Behavior
Leaf nodes have different dimensions (one is 1px by 1px, and two are 2px by 1px).
left:0 top:0 width:200 height:1
    left:8 top:0 width:50 height:1
        left:0 top:0 width:2 height:1
    left:75 top:0 width:50 height:1
        left:0 top:0 width:1 height:1
    left:142 top:0 width:50 height:1
        left:0 top:0 width:2 height:1

Link to Code
This fiddle demonstrates the correct behavior in a browser.  To reproduce the bug in Yoga, replace the leaf nodes with custom-measured nodes like in the sample code above.

This looks like #567 and #580. Do you use the latest master version or npm release? If npm please compile it yourself to asm and try if this issue is resolved on the master head.

Thanks for the quick response. I'm using the pre-built asm.js release from
npm. I'll give latest master a try tomorrow and see if the issue is fixed.
On Jul 10, 2017 12:56 AM, "Lukas Wöhrl"<EMAIL_ADDRESS>wrote:

This looks like #567 https://github.com/facebook/yoga/issues/567 and
#580 https://github.com/facebook/yoga/issues/580. Do you use the latest
master version or npm release? If npm please compile it yourself to asm and
try if this issue is resolved on the master head.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/facebook/yoga/issues/592#issuecomment-314032789, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AC5iNSFNOrOxQvdCnp1L4_BZkWpi0MwJks5sMdlAgaJpZM4OSQA3
.

Hey, just a quick update---I'm having some trouble building master.  Buck seems to make some assumptions about the length of file paths that causes it to not work on Windows.  I'll try again from Linux and see if I can get it working.

I figured out how to do the ASM.js build, and the problem is indeed fixed in master.  Thanks.


can't implement pathOr function

flow.org/try
const pathOr = (val, path, obj) => {
  if (path.length) {
    return pathOr(val, path.slice(1), obj[path[0]])
  } else {
    return obj || val
  }
}

console.log(pathOr(9, ['meta', 'pagination', 'total'], {
  meta: { pagination: { total: 2 } }
}))

should not this work? (it does work but flow does not like it)
this is what i get
4:     return pathOr(val, path.slice(1), obj[path[0]])
                                         ^ Cannot get `obj[path[0]]` because an indexer property is missing in `Number` [1].
References:
11:   meta: { pagination: { total: 2 } }
                                   ^ [1]
4:     return pathOr(val, path.slice(1), obj[path[0]])
                                             ^ Cannot get `obj[path[0]]` because property `meta` is missing in object literal [1].
References:
11:   meta: { pagination: { total: 2 } }            ^ [1]
4:     return pathOr(val, path.slice(1), obj[path[0]])
                                             ^ Cannot get `obj[path[0]]` because property `meta` is missing in object literal [1].
References:
11:   meta: { pagination: { total: 2 } }
                          ^ [1]
4:     return pathOr(val, path.slice(1), obj[path[0]])
                                             ^ Cannot get `obj[path[0]]` because property `pagination` is missing in object literal [1].
References:
10: console.log(pathOr(9, ['meta', 'pagination', 'total'], {                                                           ^ [1]
4:     return pathOr(val, path.slice(1), obj[path[0]])
                                             ^ Cannot get `obj[path[0]]` because property `pagination` is missing in object literal [1].
References:
11:   meta: { pagination: { total: 2 } }
                          ^ [1]
4:     return pathOr(val, path.slice(1), obj[path[0]])
                                             ^ Cannot get `obj[path[0]]` because property `total` is missing in object literal [1].
References:
10: console.log(pathOr(9, ['meta', 'pagination', 'total'], {                                                           ^ [1]
4:     return pathOr(val, path.slice(1), obj[path[0]])
                                             ^ Cannot get `obj[path[0]]` because property `total` is missing in object literal [1].
References:
11:   meta: { pagination: { total: 2 } }            ^ [1]

When flow has issues inferring what you are trying to do I suggest explicitly writing the types you need. In this case, it's noticing your array has 3 different strings but the objects it could apply them to don't have all three properties, each one has only one (hence I'm assuming why you're seeing the error twice per property).
Moreover, the first error is because you can't index into a number, and one of the nested properties is a number (2, in this case). Remember flow's inference is doing guesswork more so than "running" the code, and tries to be general 👍

Oh hmmm. Opening this issue to keep track of this bug.

@mrkev i think we need to create separate issue for RecursiveObject and close this one (title and description of this issue is non related and misleading for purpose why this issue is still open...)

Sounds good! Feel free to create the new, separate issue. Closing this one.


CUSTOMIZED CURSOR AND SCROLL BAR

Describe the feature

Customized cursors and scroll bars might seem like minor design elements, but they can significantly enhance the user experience of a website .
A well-designed custom cursor and scroll bar can add visual interest and attractiveness to a website.
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Hi @Its-Aman-Yadav ,Kindly consider this issue and please assign this to me under GSSOC'24.

Hey @Nih1tGupta we don't need scroll bar, just modify the cursor design
Thank you

@Its-Aman-Yadav  Thank you.
I will work upon cursor

Hello @Its-Aman-Yadav, I am a graphic designer as well. May i know what type of cursors you are lookin for ?  What illustration you want the cursor to have. Something related to farming ?

@Its-Aman-Yadav Please assign this issue to me under GSSoC24

My name is Harshwardhan Singh Panwar and I'm a contributor at GSSOC'24 .
idea:- By incorporating div CSS and animations, I can design a great animated cursor trail
Please assign this task to me, GSSOC'24 .

Please assign this task to me. I am really interested in working on this

I would like  to contribute in this project , so please assign this issue to me .


Reflect.getOwnMetadata gets metadata for parents

Reflect.getOwnMetadata lacks a check for target.hasOwnProperty(metadataContainerKey) and so will erroneously return the metadata from the parent prototype.
I discovered this when switching to the use this polyfill instead of core-js and one of my test cases on metadata exposure through class inheritance failed.

We would love it if you could submit a PR to fix the issue :)


MH-13399, Update Elasticsearch Configuration

This patch updates the meachanism of how Opencast's internal
Elasticsearch is configured. It moves most of the configuration into the
bundle resources and mostly unifies the externalapi and adminui index
configuration.
Reasons for the change are:

A lot of the exposed configuration options were actually fixed
settings and changing them could cause errors in Opencast. In other
words, there was nothing to configure at all.

The current configuration was mixing node and index configuration
which is no longer allowed starting with Elasticsearch 5.0. This
patch properly separates these settings.

All relevant configuration setting for both indexes were just
duplicated. Even worse, they had to be since the main node and some
index settings would only be loaded from one configuration location
and not defining it twice could lead to inconsistent behaviour of
Opencast after restarts.

Fixed merge conflict

I'm not terribly familiar with the guts of ES, but if I wanted to run it as a separate service external to Opencast, is that possible with this new PR?  Otherwise, looks good to me!

[…] if I wanted to run it as a separate service external to Opencast, is that possible with this new PR?

Yes, you can. This configuration does not change at all:
https://docs.opencast.org/develop/admin/modules/searchindex/elasticsearch/
How this works is basically, that a connection to ES is established (that is still configurable) and then Opencast will initialize the index if it does not already exist. This works with both the internal and external Elasticsearch.
For the configuration, you now have:

The connection configuration in custom.properties
The ES node configuration in etc/elasticsearch.yml. This is what you would use to configure an external ES as well
The index configuration which is tied to our code and is now in the search module's resources since you  should not change that without changing the code

Fixed the space and re-based onto develop

Thanks for the review!
ES5 will follow soon.


describe the parameters

[x] <IP_ADDRESS>	Maximum stomatal conductance and R50
[x] <IP_ADDRESS>	Stage I evaporation (U)
[x] <IP_ADDRESS>	Stage II evaporation (ConA)
[x] 	3.8.1	Leaf area expansion rate
[x] 3.8.2	Radiation use efficiency
[x] 3.8.3	Phyollochron
[x] 3.8.4	Flowering
[x] 3.8.5	Height

simpleleaf uses Penman-Monteith

done and close for now


ci: changed identifier label and increased timeout from 10 to 30 seconds

Changed identifier label and increased timeout from 10 to 30 seconds
Made this change in hopes that the UI tests will start passing.
Checklist
Before raising your pull request:

[x] Commit messages that conform to conventional commit messages
[x] Ran the app locally ensuring it builds
[x] Ran the tests locally ensuring they pass on Build
[x] Pull request has a clear title with a short description about the feature or update
[x] Created a draft pull request if it is not yet ready for review

Before your pull request can be reviewed:
~- [ ] Met all of the acceptance criteria specified in the user story on Jira~

[x] Reviewed your own code to ensure you are following the style guidelines
[x] Ran the app and tested the feature on a range of device sizes
Please include iPod Touch, iPhone SE and iPhone 11 as a minimum.
~- [ ] Written Unit and Integration tests if needed~

~- [ ] Met all accessibility requirements?
- [ ] Checked dynamic type sizes are applied
- [ ] Checked VoiceOver can navigate your new code
- [ ] Checked a user can navigate only using a keyboard around your new code~
Before merging your pull request:

[ ] Ensure that the code coverage and SonarCloud checks have passed
[ ] Actioned and resolved all comments, reaching out to reviewers for clarifications if necessary.
[ ] Ran the app to ensure that no regressions have been caused by changes during code review.

It's great that this has worked. It would be good to identify which of these changes was the one that fixed it and revert the one that wasn't required. The benefit of that is that we then learn about what the actual fix was too :)

Ah, the runner is only doing Unit Tests on commit. I think it would be worth considering setting up some other workflows for manually testing UITests when required. It would need to be a separate ticket, but something to talk to Gabby and Dylan about.


CSS Templates

Once #7 is completed, we should be able to pick from multiple CSS templates easily

Solved by #34


fix(time-picker): vertically center caret

Closes #2094
Closes #2098
This PR adds style rules to position the time picker <select> carets properly after https://github.com/IBM/carbon-components/pull/1978
Changelog
New

documentation examples for invalid and disabled states

Changed

Carbon classes for text input and <select>s to share styling
type styles

Removed

redundant and unused styles

Testing / Reviewing
Ensure the component is not broken

all labels in the library are using label-01 (https://github.com/IBM/carbon-components/pull/1859) and all error messages are using caption-01 (https://github.com/IBM/carbon-components/pull/2064)
the other changes should be addressed, but just to clarify: the hh:mm text is a placeholder, and if you start typing the actual color is text-03


Isolating Background Music

Does anyone know if this can be used to isolate the background instrumental music (instead of vocals)? I tried multiplying attenuation by -1 but that doesn't work.

The default functionality of AUSoundIsolation will isolate vocals using a neural network shipping with macOS/ (specifically aufx-vois-appl-nnet-vi-v0.plist within AudioDSP.component)! If you comment out the audio unit properties setting the paths to the custom music-oriented model (plist/base/dereverb), this should be used by default. You may want to tinker some with the denoising/tuning properties, or remove them entirely.
(However, in my experience, it's not good at fully removing loud background music. This same audio unit (or something very similar?) appears to be used for the "Voice Isolation" microphone setting, and that itself isn't always the best :)

Got it, Thanks for your reply! The quality might not be the best, but I love that it works with 0 latency. I have tried other vocal-remover solutions like librosa (Python library) and multiple deep learning solutions, where a 3 min. song can take upto 30 seconds to process depending on the CPU/GPU. (But maybe there are other better solutions I don't know of?)
So, apparently while testing it is crashing on my iPhone 11 because it cannot find the file "aufx-nnet-appl.plist" but working on my iPhone 12 Pro (Both iPhones on iOS 17+). I don't know why that is happening because Apple says that voice isolation should work on Xr and later devices (https://support.apple.com/en-in/101993#:~:text=You can use Voice Isolation,iPad Pro 11-inch models). Maybe they are using a different audio unit for this.

Glad to hear this tool has helped :) unfortunately not familiar with other solutions, but wishing you best of luck with the search!
Both your iPhone 11 and 12 Pro should work! I've pushed some commits recently regarding model discovery under iOS 17 as the model location changed (especially 72a8734bb4464a63a64aa8b5d22dd6b04719c839 and 09839ccc295f904ce90600e57761ad4be7258b24) - with these pulled, are you able to run as expected?
If not, could you please send the logging of what paths it attempted to search for the model?

Great! 72a8734 fixed it.
So while testing it more, it is working for most files but I encountered an error where the plist file threw an exception for a file in .m4a format (karaoke recording with background music + vocals). Was wondering if you have any idea why this could happen? Maybe because the recording quality is bad, and background or vocals are perceived as noise?

Assertion failed: ((numInternalIOChannels == 1 || (numInternalIOChannels == numIOChannels && internalBatchSize == 1)) && "internal format must be one channel or the same number of IO channels (when internal batch size = 1)"), function CreateProcessingGraph, file SoundIsolationGraphAdapter.cpp, line 335

Apologies for taking a little while to get back! I'm able to reproduce this with any audio that has a single channel, but unfortunately am not familiar enough to determine why.

Ok np. Thanks for all your help :) I am really enjoying this project!!

Got it, Thanks for your reply! The quality might not be the best, but I love that it works with 0 latency. I have tried other vocal-remover solutions like librosa (Python library) and multiple deep learning solutions, where a 3 min. song can take upto 30 seconds to process depending on the CPU/GPU. (But maybe there are other better solutions I don't know of?)
So, apparently while testing it is crashing on my iPhone 11 because it cannot find the file "aufx-nnet-appl.plist" but working on my iPhone 12 Pro (Both iPhones on iOS 17+). I don't know why that is happening because Apple says that voice isolation should work on Xr and later devices (https://support.apple.com/en-in/101993#:~:text=You can use Voice Isolation,iPad Pro 11-inch models). Maybe they are using a different audio unit for this.

Thank you for your time and effort. I would like to ask you something I don't understand. What exactly does the file and AudioDSP in the following sentence refer to? I've been doing some work on that lately.
specifically aufx-vois-appl-nnet-vi-v0.plist within AudioDSP.component

@eric-gitta-moore

What exactly does the file and AudioDSP in the following sentence refer to? I've been doing some work on that lately.

Absolutely! From my understanding (and please note I very well could be wrong), AudioDSP is a Carbon component providing the functionality behind Audio Units. AUSoundIsolation seems to leverage another private audio unit (known as AUNeuralNet) for its functionality.
That plist configures the neural network used for sound isolation, and under macOS, can be found under /System/Library/Components/AudioDSP.component/Contents/Resources/Tunings/Generic/AU/SoundIsolation/aufx-vois-appl-nnet-vi-voice.plist.
(Note that this is its intended functionality, so you shouldn't need to configure anything else beyond the wet/dry mix parameter to leverage the audio unit for voice/sound isolation! QuietNow only has to specify their custom, secondary neural network for music isolation. You can take this project and strip out all of the neural net-related parameters to achieve its default usage.)


Add name property to Metric class

IMPORTANT: Please do not create a Pull Request without creating an issue first.
Before submitting (must do checklist)

[x] Did you read the contribution guide?
[x] Did you update the docs? We use Numpy format for all the methods and classes.
[x] Did you write any new necessary tests?
[ ] Did you update the CHANGELOG?

Type of Change

[ ] Examples / docs / tutorials / contributors update
[ ] Bug fix (non-breaking change which fixes an issue)
[x] Improvement (non-breaking change which improves an existing feature)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Proposed Changes
Look #446
Related Issue
#446
Closing issues
Closes #446

Codecov Report

Merging #454 (35ab373) into master (4659990) will decrease coverage by 34.78%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           master     #454       +/-   ##
===========================================
- Coverage   87.79%   53.00%   -34.79%
===========================================
  Files         114      114
  Lines        5351     5354        +3
===========================================
- Hits         4698     2838     -1860
- Misses        653     2516     +1863

Impacted Files
Coverage Δ

etna/metrics/base.py
91.52% <100.00%> (-4.91%)
:arrow_down:

etna/pipeline/pipeline.py
83.33% <100.00%> (-14.07%)
:arrow_down:

etna/commands/__init__.py
0.00% <0.00%> (-100.00%)
:arrow_down:

etna/commands/backtest_command.py
0.00% <0.00%> (-96.43%)
:arrow_down:

etna/commands/forecast_command.py
0.00% <0.00%> (-92.00%)
:arrow_down:

etna/commands/__main__.py
0.00% <0.00%> (-87.50%)
:arrow_down:

etna/commands/resolvers.py
0.00% <0.00%> (-80.00%)
:arrow_down:

etna/analysis/outliers/density_outliers.py
22.44% <0.00%> (-75.52%)
:arrow_down:

etna/transforms/timestamp/time_flags.py
26.02% <0.00%> (-73.98%)
:arrow_down:

etna/transforms/timestamp/fourier.py
27.08% <0.00%> (-72.92%)
:arrow_down:

... and 67 more

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 4659990...35ab373. Read the comment docs.


Some links are broken

Thank you for maintaining ratatui and its great documents! 😃
I found some broken links in this https://ratatui.rs/tutorial/json-editor/ui.html.

These 2 links are broken.
Applicable codes are here.
I thought this codes are itented to link somewhere in https://ratatui.rs/concepts/layout/index.html, but I don't know the relevant part.😓

Thanks for reporting!

Thank you for very quick fix too! :)

I'm surprised that linkcheck didn't catch this for us.
https://github.com/ratatui-org/ratatui-book/blob/ebf79f8d88bf3ee2a3d1a32fc0a462fe579000e6/book.toml#L26
This is something I have to look into.


Changes for v2.5.1 release

This is a pull request for changes to land in v2.5.1.
Included changes - https://github.com/StackStorm/st2/pulls?q=is%3Apr+is%3Aclosed+milestone%3A2.5.1.

Also need to cherry pick this into v2.5 (st2docs) once this PR has been merged - https://github.com/StackStorm/st2docs/pull/666


[Bug Report]: Microsoft.Network/vpnGateways/connections] ConnectionBandwidth property has been deprecated

Describe the bug
It appears the ConnectionBandwidth property has been deprecated, but the ARM docs haven't been updated here: https://docs.microsoft.com/en-us/azure/templates/microsoft.network/vpngateways/vpnconnections?tabs=bicep#vpnconnectionproperties
This means deployments fail and there is no way to use the module as it has a default value in the bicep file :(
To reproduce
https://github.com/Azure/ResourceModules/blob/80c29bd5fef1d522354b801b6bb59106e74ad67c/modules/Microsoft.Network/vpnGateways/connections/deploy.bicep#L38
https://github.com/Azure/ResourceModules/blob/80c29bd5fef1d522354b801b6bb59106e74ad67c/modules/Microsoft.Network/vpnGateways/connections/deploy.bicep#L76
Code snippet
module vpnc 'main/modules/Microsoft.Network/vpnGateways/connections/deploy.bicep' = {
  name: '${deployment().name}_vpnc'
  params: {
    name: 'vpnGatewayConnection'
    vpnGatewayName: 'vpnGateway'
    remoteVpnSiteResourceId: 'remoteVpnSiteResourceId'
    vpnLinkConnections: [...]
  }
}

Relevant log output
{
    "status": "Failed",
    "error": {
        "code": "VpnConnectionPropertyIsDeprecated",
        "message": "Update request for VpnConnection /subscriptions/....../resourceGroups/..../providers/Microsoft.Network/vpnGateways/..../vpnConnections/.... contains deprecated property ConnectionBandwidth. Please use corresponding VpnSiteLinkConnection property instead.",
        "details": []
    }
}

Hey @fabmas,
I just migrated this issue from CARML. Please take a look an triage. Maybe it's not relevant anymore :)

Hey @fabmas, please take a moment to triage this issue if you can 💪

Hey, sorry for the delay. I'm currently restructuring the entire module introducing new UDTs. I'll take care of this issue in the next release of the module.

Hey @fabmas,
thanks. Just note that the CI's automation (bot) will continue to comment and ping you until the labels Needs: Triage and Status: Response Overdue are removed 😉

This issue is no more relevant.


Rework command enqueing to minimize swallowing

Is your feature request related to a problem? Please describe.

It is currently not possible to call m_sig.emit(button_press{*}); more than once before m_inputdata is cleared in the next iteration of the eventqueue worker loop. This is a problem, for example, if more than one action need to be triggered in the same method call.
Why does polybar need this feature?

This feature would make the core system underlying polybar considerably more flexible, and would allow for easier and faster implementation of some features.
Describe the solution you'd like

Instead of m_inputdata there could be a queue of input data strings.
Describe alternatives you've considered

One alternative is to not emit many signals one after the other, and instead store the signals and call them after a short delay, but this increases complexity and makes everything more prone to bugs.
Additional context

Results in

Hello I am new to polybar, I would like to contribute to this issue, is there any place where I could start?

@madhavpcm Yes, the pending action string is stored in inputdata in notifications_t in controller.hpp. Making that a queue would probably be the most straightforward way to solve this.
Thanks :)

I have did some changes to  notifier_handler() and trigger_action() as these were the places where I noticed enqueuing and dequeuing takes place. I believe the notifier_handler() loop runs separately and checks if inputdata is available and call process_inputdata(). However, in order to test I'm confused not sure how to try out m_sig.emit(button_press{*}) multiple times

Yeah testing might be tricky here since it relies on the button clicks (or ipc actions) arriving very quickly after each other. I don't think there is a way to trigger this by hand.
I also don't think you can write any automated tests for this since the controller relies on an X server running which we can't handle yet in our testing infrastructure.
But if you want to just quickly verify things work for you, you could add something like this:
m_loop->timer_handle([this]() { trigger_action("notify-send A"); trigger_action("notify-send B"); })->start(500, 0);

This will trigger two actions directly after one another (they should each create a notification popup) after about half a second. Add this line directly before we set m_loop_ready to true.

I hope this is the desired behaviour

Yes, that looks alright :)

Im terribly sorry for the messy commits in the PR, I keep forgetting one thing or the other before committing. Kind of a beginner with git and pull requests. Thanks alot for the help!

Don't worry about that one bit. That's why we have PR reviews and formatting tools ;)
I appreciate the effort

Thanks Alot! I will work on my approach, looking forward to contribute more in the near future :).


Require react-native instead of reaching into its private modules

The new version of the packager doesn't let you reach into the private module namespace of react-native so go through the public interface.

Perfect :)


Addon Name

Addon Manifest URL
https://tv.egydead.live/a3/
Addon Description
mm
Language of Content
Arabic (ar-SA)
Choose Labels

[X] anime
[ ] asian drama
[ ] bollywood
[ ] cartoons
[ ] debrid support
[ ] http streams
[ ] live tv
[ ] metadata
[ ] misc
[X] movies
[X] music
[ ] nsfw
[ ] podcasts
[ ] radios
[X] subtitles
[ ] torrents
[X] tv shows
[ ] video games

i don't know what this is, but it's not a new addon, closing it


#1384 : Updated contact us .html file by adding form validations

Fixes #1384
Changes proposed

I added the form tag which was the main part of the validations in the contact us form.
I also changed the type of the phone number and email to their appropriate types so that only those values can be added for better usability.
✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

I have made the requested changes . Please revive it and do the needful. Let me know if further changes required.
Thank you !


Only initialise LDAP if enabled

.ldap.initialise was called twice meaning that regardless if ldap was enabled it was still called.
Start a test TorQ process with ldap enabled, and qcon in using correct user name and password
.z.u returns username:

The q process returns authenticated message (if .ldap.debug is set to 2i in default.q):

The cache is updated:

qcon using wrong credentials:
.z.u doesn't return a value:

The q process returns a failed to authenticate message

Continued attempts will block the user

Fixed bug in .ldap.cache where attempts counter was wrong for authenticated users

Replaced by #596


Malformed XML in "vmu.ldefs".

https://github.com/RossMeikleham/Ghidra-VMU-Processor/blob/main/LC8670/data/languages/vmu.ldefs
Line 12 reads:
<description>LC8670/description>
...but should read:
<description>LC8670</description>
Malformed XML is causing Ghidra to throw an error on launch.

🤦 Thanks for the report, I didn't realise I hadn't pushed up the latest changes. https://github.com/RossMeikleham/Ghidra-VMU-Processor/commit/a519abf1754e90157118654b0de12d94548f7d69 should fix this issue.


[Bug]: Exception Unknown scheme for Frame.waitForEventInfo

Playwright version
1.14.1
Operating system
Windows
What browsers are you seeing the problem on?
Chromium, Firefox, WebKit
Other information
No response
What happened? / Describe the bug
An unobserved task exception with the message 'Unknown scheme for Frame.waitForEventInfo' occurs when executing RunAndWaitForNavigationAsync.
Code snippet to reproduce your bug
using Microsoft.Playwright;
using System;
using System.Linq;
using System.Threading.Tasks;
class Bug
{
    public static async Task Main()
    {
        using var playwright = await Playwright.CreateAsync();
        await using var browser = await playwright.Chromium.LaunchAsync(new BrowserTypeLaunchOptions
        {
            Headless = false,
        });

        TaskScheduler.UnobservedTaskException += (_, args) =>
        {
            Console.Error.WriteLine(args.Exception.InnerExceptions.Single().Message);
        };

        var context = await browser.NewContextAsync();
        // Open new page
        var page = await context.NewPageAsync();
        await page.GotoAsync("http://www.example.com");
        await page.MainFrame.SetContentAsync(@"
<!DOCTYPE html>
<html>
<body>
<form action='http://example.com'>
  <input type='text' id='fname' name='fname' value='John'><br>
  <input type='submit' value='Submit'>
</form>
</body>
</html>
");
        await Task.Delay(TimeSpan.FromSeconds(3));
        await page.RunAndWaitForNavigationAsync(async () =>
        {
            await page.ClickAsync("text=Submit");
            await Task.Delay(TimeSpan.FromSeconds(3));
        });
    }
}

Relevant log output
No response

Can you share the Playwright driver version as well? You can get it by going to the project root directory and executing
playwright --version

Thanks!

Sure
GAC    Version        Location
---    -------        --------
False  v4.0.30319     C:\dev\PlayGround\MyPlaywright\bin\Debug\net461/Microsoft.Playwright.dll
Version 1.14.0-1629718845000
0

Hm, that looks fine. What .NET version are you targeting?

4.6.1

No, I could not. And 4.6 is not actually supported - our minimum is 4.8.
Can you add this line as the first line of your application?
global::Microsoft.Playwright.Program.Main(new string[1] { "--version" });

Does it print the same version as above?

Yes it's the same: Version 1.14.0-1629718845000
But I just tried it with 4.8 and also got the errors:
C:/Dev/PlayGround/MyPlaywright/bin/Debug/net48/MyPlaywright.exe
Unknown scheme for Frame.waitForEventInfo
Unknown scheme for Frame.waitForEventInfo

Hm, so I can repro with 4.8! Thanks for this. I'll have a detailed look.


Migration to Swift

Swift is the long-term plan of all Objective-C projects. Unfortunatelly, it seems like it’s not possible to make KeepLayout feel good in both languages at once. Transition to complex numbers as priorities broke Swift compatibility and adopting protocols (a proposed improvement) would break Objective-C compatibility.
I can imagine parallel development of Objective-C branch and Swift branch during transitioning period (years). This way, the Swift part would be free to diverge far into an arcane world of structs, protocols and crazy operators without carrying the Objective-C baggage.
I don’t plan to invest my time to this in near future, so I encourage users of KeepLayout to fork the project and experiment with Swift, as you wish. I would be happy to merge good ideas back to this main repo.

The optional priorities could be improved using protocols:
protocol KeepValue {
    var keepPrioritizedValue: KeepPrioritizedValue { get }
}

extension Double: KeepValue {
    var keepPrioritizedValue: KeepPrioritizedValue {
        return KeepPrioritizedValue(value: self, priority: 1000)
    }
}

struct KeepPrioritizedValue: KeepValue {
    let value: Double
    let priority: UILayoutPriority

    var keepPrioritizedValue: KeepPrioritizedValue { return self }
}

func KeepHigh(value: Double) -> KeepValue {
    return KeepPrioritizedValue(value: value, priority: UILayoutPriorityDefaultHigh)
}

Usage would then be:
var constraint: KeepValue
constraint = 10
constraint = KeepHigh(10)

I made the first step towards Swift future. I brought the Swift API on par with Objective-C API using the above approach. The usage is now identical, but uses completely different type systems.
// Objective-C
view.keepWidth.equal = 100;

// Swift
view.keepWidth.equal = 100

Any way that you do a swift 3.0 branch @Tricertops ?
I tried but I couldn't figure out one error given by the compiler "value use within it's own type"

@AncAinu I looked into this and it seems like a bug in Swift. However, I found a workaround and it’s now available on swift-3 branch.

@Tricertops awesome workaround lol