Azure Load TestinLoad Tests Return 'Not Applicable' Failed status despite showing as successful in Exported Result Files & AppInsights[BUG]

Customer has raised an issue in Microsoft Q&A - https://docs.microsoft.com/en-us/answers/questions/684415/index.html
We are currently running a POC on the new Azure Load Testing feature (currently in preview) as a resource to run our jMeter tests against an in development API.
Unfortunately we are hitting a few problems with the output results displayed within a test run, namely the tests always fail with no start, end and duration time along with a 'Not Applicable' status when running against the APIs despite all tests running successfully directly within the jMeter GUI. The same tests when run in the Azure Load Testing tool also appear to show as 'Passed' when exporting results from the test screen.
Has anyone experienced this before and would be able to help, as this tool could be exactly what we are looking for if we can get it to output the correct results.
Our jMeter test is fairly simple and obtains a validation token which is used in the latter request, I have manually verified this is all authenticating correctly and the variables we are using in these tests are being correctly populated when run in Azure. Note this issue appears in both our GraphQL and REST API tests.
See attached screenshots:
Test Result
wD6KP.png
Exported Result
aTjYQ.png
We have also attempted to use the 'Ultimate Thread Group' plugin following a suggestion on StackOverflow, but unfortunately this also appears to return the same results (passes when run locally or through Blazemeter) but returns no result in Azure Load Testing despite the requests returning a successful result in AppInsights.
If anyone would be able to help at all this would be greatly appreciated. Unfortunately at the moment this is preventing us from reccomending Azure as a tool for our upcoming Load Testing strategy
.
AB#1461304

Hi, Thank you for raising this issue on my behalf.
Please find additional notes I have provided on the Microsoft forum:

Since raising this issue we have since carried out some additional investigations on our side and noticed that there appears to be a particular issue when using the GraphQL HTTP Request type that was added to jMeter in version 5.4. When using this type of request for a valid graphQL query. The request is processed successfully locally within jMeter and also within Blazemeter, but not within the Azure Load Testing tool (which provides the issues displayed above).
I have since been able to get a valid response from the Azure Load Tester, when I submit my GraphQL requests directly within the body of a standard HTTP Request and with the 'Ultimate Thread Group' plugin (without this the response fails). Would it be possible to confirm if the GraphQL HTTP Request type from JMeter is currently supported for the Azure Load Testing tool, or if this is planned as part of a future update?

Hello @ChrisTyler-6788
Thanks for reporting this. We are looking into this issue.  As you reported it seems the tests have executed correctly, but the client metrics is not displayed properly in this specific scenario. Would you be able to share some more details like testrunName, jmx to speed this up.

Hi @ChrisTyler-6788 can you also share the region name in which you have created the load test along with working and non working JMX.

Hi @abranj1219, @krishna1s.
Please find an example of the GraphQL requests currently failing. There are also two CSV files which contain paths to authenticate and access the API, I am unfortunately unable to provide these as this is an in development API, but this hopefully should visualize the structure of our JMeter tests.
The region we have been using is 'North Europe'
Many thanks
GitHubExample.zip

@ChrisTyler-6788 -Can you try to run the test again, as this test seems to be a one-month-old test.
We are hoping we might have fixed some issue in past few weeks related to this.
Please let us know once you run the test again.

hi @MonikaReddy-MSFT.
Thanks for your reply. I have now re-run our tests and these do look to be passing now and displaying full test results.
Many thanks for your help

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


Installation failed

Hi,
I would like to test this library but I failed at installation. I have to admit that I am not that expert in the JavaScript eco-system, I only have basic knowledge.
I did successfully install and run a local excalidraw installation and the usage of libraries from the official repository works. But when I start yarn add svg-to-excalidraw this error is given:
[5/5] Building fresh packages...
error /var/www/excalidraw/node_modules/svg-to-excalidraw: Command failed.
Exit code: 127
Command: husky install
Arguments:
Directory: /var/www/excalidraw/node_modules/svg-to-excalidraw
Output:
/bin/sh: 1: husky: not found
info Visit https://yarnpkg.com/en/docs/cli/add for documentation about this command.

My system is running Linux Mint 20 Ulyana. If I run  yarn add husky, this output is given:
[5/5] Building fresh packages...
warning "husky" is already in "devDependencies". Please remove existing entry first before adding it to "dependencies".
success Saved 1 new dependency.
info Direct dependencies
└─<EMAIL_ADDRESS>info All dependencies
└─<EMAIL_ADDRESS>
Can give me somebody a hint what's going wrong?
Why I do want to test this library? Very simple, in our  company ExcaliDraw gets more and more popular. I would like to add some elements I always have to draw by hand and which therefore look very ugly. I am an electronics engineer and that's why I would like to import the symbols e.g. from https://www.mbeckler.org/inkscape/circuit_symbols/

@oe8bck Sorry you are having troubles. This library is still in a very early state, and kinks are being worked out.
Currently svg-to-excalidraw is a separate javascript library from excalidraw, and is not installed directly in the excalidraw. It's really meant to be used in a separate webapp. Here is a quick CodeSandox Example.
Having said that, I'm not sure what the husky errors are about, but I suspect it's because you are trying to install svg-to-excalidraw within the excalidraw project.
Hopefully this clear things up a bit 😄 Let me know if you have further questions.

Thanks for the link to the example app, this is what I am searching for. If needed, I could install such an app on my server, but at the beginning, this is much more comfortable!
OK, I did try to convert one of the symbols. I get the JSON string, it took me a little bit to find out, how to paste this string to Excalidraw, but finally I got it. It's just simple copy&paste, very standard, no icon, menu or whatsoever.
One thing at the end was not correct: the stroke color was set to transparent instead of black. Of course this I could change quite easily. Can I support the development of this project?

OK, I had to do some manual rework, but it was quite easy.
 Would it be possible to add an option in order to scale the drawing when importing?

Generating a library was then very straight-forward. One thing left is the preview of the library objects. Is there a way to get or retrigger image conversion for the previews?

@oe8bck I'm glad you were able generate some usable elements!
Thank you for showing us the preview image issues. I'll make an issue for this. I'll also take a look at the transparent stroke color that you were seeing. There are some technical issues involving getting stroke and fill colors working 100%, but hopefully this can be improved in the future.

Thanks for your efforts! As I said, I think in our company this tool is
very welcome.
One question:
For the EE symbols mirroring the symbols is needed. I think there have been
already some efforts to provide this, but I do not see how to actually do
this. Can you give me a hint? As a work-around I could add mirrored
symbols, this is not so nice, but possible if needed.
BTW:
In the meantime, the preview of the symbols looks ok. I do not know what I
did, could be that a restart of the app did it, but I am not sure. As there
is some handwork at the import needed, the stroke issue was annoying but
not that big of a problem, just an additional mouse click.
Am Mo., 29. März 2021 um 05:07 Uhr schrieb Broch Stilley <
@.***>:

@oe8bck https://github.com/oe8bck I'm glad you were able generate some
usable elements!
Thank you for showing us the preview image issues. I'll make an issue for
this. I'll also take a look at the transparent stroke color that you were
seeing. There are some technical issues involving getting stroke and fill
colors working 100%, but hopefully this can be improved in the future.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/excalidraw/svg-to-excalidraw/issues/36#issuecomment-809033698,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAPRDRMPT7VGTC3FU6PHY23TF7VI5ANCNFSM4Z3FCD2Q
.

--
         (0-0)

---------oOO--(_)--OOo-----------------------------------------------
Christof Bodner @.**>
Linux - Life is too short for reboots
- First they ignore you. Then they laugh at you. Then they
fight you(). Then you win. -- Ghandi
email: @.***
QTH:   JN66WO
Signal: +43<PHONE_NUMBER> (http://signal.org)
GnuPG public key: 0xB15DA60F (http://wwwkeys.eu.pgp.net) Fingerprint 5222
4DEA 773A 60B3 22F9  D244 15EF 6645 B15D A60F


Deployment to Minikube fails with weird errors

@donsizemore and @wilkos-dans experienced issues when deploying Dataverse 4.14 to Minikube.
No real or helpfull errors were logged (except the normal exceptions about Rserve password alias and code scanning), the DAS just went away.
I could reproduce this with default Minikube config CPU=2, MEM=2048 on KVM2 driver. This seems to be memory related, as my API server had been killed, too.
After using CPU=2, MEM=4096 I could successfully deploy, but now I experience issues with the bootstrapping script.
Dataverse is really memory hungry:
> kubectl top pod -A | sort -h -k4
NAMESPACE     NAME                               CPU(cores)   MEMORY(bytes)
kube-system   kube-addon-manager-minikube        14m          5Mi
kube-system   coredns-fb8b8dccf-pztgf            18m          8Mi
kube-system   coredns-fb8b8dccf-wsx8f            16m          9Mi
kube-system   kube-proxy-xhtbh                   5m           12Mi
kube-system   kube-scheduler-minikube            8m           12Mi
kube-system   storage-provisioner                1m           14Mi
kube-system   heapster-64b4q                     1m           19Mi
kube-system   influxdb-grafana-xcf5c             12m          40Mi
kube-system   kube-controller-manager-minikube   95m          41Mi
kube-system   etcd-minikube                      99m          44Mi
default       postgresql-668dccf4cd-pzq5n        12m          51Mi
kube-system   kube-apiserver-minikube            145m         186Mi
default       solr-6ddd95c86d-plm8f              5m           227Mi
default       dataverse-799cd744fd-sfmdl         16m          996Mi

> kubectl top node
NAME       CPU(cores)   CPU%   MEMORY(bytes)   MEMORY%
minikube   805m         26%    2817Mi          77%

@poikilotherm I tried again with 8192M of RAM with the same results. I wouldn’t run a production Dataverse instance in fewer than 24G of RAM, but my test box isn’t that beefy =(

@poikilotherm I also tried again with minikube start --memory 9200 --cpus 5. This did work, however still some issue's in the dataverse log. Will look into them later. Dataverse-eu has a running instance on GoogleCloud: 13Gb and 2 vCPU's. Thanks.

This should be solved now. Added some info to the docs. You will still have problems due to #60...


Width calculation and overflow errors with flexbox in Firefox - IE - Opera

Hi,
I've managed to identify an issue that happens when a slick carousel is initialized inside a flex item (child of a display: flex container, with flex: 1 auto).
It's quite hard to describe what happens, basically, instead of being limited by width constraints of parent elements (e.g. the width of the body), the carousel forces the flex item to stretch indefinitely, thus stretching its parent with it.
Noticed that the .slick-track is assigned a huge width, which is only recalculated as a higher value every time the window resizes (and probably slick reinitializes).
Also, the .slick-list, which in Chrome/Safari respects the overflow: hidden rule and only stretches as far as is allowed by its container, in Firefox / IE / Opera it expands as far as the .slick-track.
Can confirm that forcing max-width: 100% on the flex item limits its width to that of its parent, but it does not take into account the other children of the flex container.
Reproduced here: http://jsfiddle.net/uLqt4zmy/1/
Also attaching an attempt to limit the width in a typical flexbox usage pattern - reproducing the holy grail layout using a flexing and a non-flexing child: http://jsfiddle.net/uLqt4zmy/9/
Reproduced in Firefox 35 on OSX and Firefox 35 / IE 11 / Opera ?? on Windows.
Any idea what could be causing this?

Apparently this is caused by how the flex-items are supposed to behave, and Chrome is actually the one implementing the W3C spec wrongly.
This article will shed some more light on it: https://bugzilla.mozilla.org/show_bug.cgi?id=1108514
The solution is to add min-width: 0px; on the flex children and their expansion will be restricted to the overall layout. (no idea why that happens tho)

Man--thanks for this solution--saved me some serious heartache.

@kenwheeler Should this be in the core css?

Yes, wanna PR it?

Sure, but I'd like to do more testing. Looks like min-width solution isn't solving it for IE; at least on on my implementation. How about you @tibineagu

@leggomuhgreggo yup, IE is still acting completely bonkers for me as well.

@leggomuhgreggo yep, fixed in firefox, but IE is still having the problem

has anyone found a fix for this ? i'm still working on a solution over here... http://stackoverflow.com/questions/32566712/flex-items-exceed-width-of-parent-in-ie

Out of date. Closing.

2 days struggling with this problem. Just min-width: 0px; - Thank God!

@fredec I've been hearing about this min-width: 0px solution everywhere but can't find a signle working example and can't seem to figure it out on my own. Could you point me in the right direction? Or even which elements are supposed to be min width: 0px

Many thanks! So happy to see such a simple solution. This bug had me worried.

where should I add min-width: 0px;? in slick-slide or slick-track? it doesn't work..


Readme/removearchitecture

Pull Request Test Coverage Report for Build 858

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 74.935%

Totals

Change from base Build 855:
0.0%

Covered Lines:
1154

Relevant Lines:
1540

💛  - Coveralls


Service communication and database

Hi,
I am a bit confused, I can't see any service communication for shared data.
It looks like it is copying data to the services database, wouldn't this cause issues with keeping data up to date?
E.g. many services have a table for "Customer" meaning that them databases have information copied from another database.
Wouldn't it be better to generate a messaging event to a "customer service" and then store the ID in the service that is using that customers information?
Please let me know if I have got anything wrong here.

Hi @jbonnett92. Thanks for your interest in this project!
The data is being kept up-to-date using events. If you add a customer for instance, a CustomerRegistered event is published by the CustomerManagementAPI to the message-broker (RabbitMQ). This event is picked up by all the services that need customer info and persisted in their local database (in a form that fits the service). This event-driven approach makes the services autonomous. The Invoice service for instance, can do its work and send invoices, even when the CustomerManagementAPI if offline (for maintenance or an issue).
So yes data is shared between the services. But the CustomerManagementAPI will always be the system-of-record for customer information. All other services have a read-only "cache" of the data locally. And they store the least amount of fields as possible for them to do their work. This is done to reduce the chance of a change to the customer data-structure having an impact on the local cache. The data will only be created and updated by ingesting events.
Does this answer your question?

So this is for redundancy? Does it break the rules of normalisation? Not sure how normalisation applies to multiple databases.

Not sure I fully understand your question. I used this approach to make every service fully autonomous. Every service owns its own database and normalization is used within this database.

Hi @jbonnett92. Did this answer your question?

Sorry about the late reply, I was on training. Yes that answered the question.

Do you by chance have any tutorials on implementing the concepts that you've used in this project?
I am new to .NET Core (Though I have been developing Umbraco for the last two years for my job, which is .NET Framework) and I have basic knowledge how microservices work and what not to do. I just can't seem to know where to start implementing.

No problem @jbonnett92. Thanks for the update.
For me it has been a journey (so no overnight thing). I have learned all the concepts I've put into Pitstop primarily by reading books. These are a couple I can recommend:

Building Microservices by Sam Newman
Exploring CQRS and Event Sourcing. This is an older book, but I've really learned CQRS and the event-driven approach from this book. Samples are not .NET Core but it's more about the concepts.
Patterns, Principles, and Practices of Domain-Driven Design.

I also had the privilege of following a course by Greg Young about CQRS and Event Sourcing in London. That was also an eye-opener for me.
And finally it's been a lot of sparring with fellow software architects about - and experimenting with - the concepts I've picked up from the books and how to implement them. This is why Pitstop exists. It's my go to project for trying new concepts and technology (I'm currently thinking about porting the front-end to Blazor for instance).
I've done some talks on how I've built Pitstop using .NET Core and docker. This will also go into some of the architectural concepts. Maybe this recording will help: Building microservices with .NET Core and docker.
I hope this will get you started.

Thank you,
I also noticed that in WorkshopManagementAPI repo's you reference databases from other microservices.
Shouldn't you query the service that the database is paired with? Like in this illustration?

Yes @jbonnett92, that's how it should be done.
A microservice is a concept and the WorkshopManagement microservice contains 2 components: an API for handling commands (WorkshopManagementAPI) and an eventhandler for ingesting events from other services (WorkshopManagementEventHandler). Both components are part of the implementation of the WorkshopManagement microservice. And hence, both databases are only used within this microservice. This is also how I visualized it in the solution architecture diagram:

Please can you add a authentication service? It would be an interesting one to see.

This is why I use an message broker. The messages will be kept in the queue while the service is down. When the service is started, it will just resume picking up the next message from the queue.
It's on my backlog.

This may be a problem when you scale up the services, lets say that we create multiple invoice service instances, we publish and even that the invoice service consumes, that means that only one instance can consume using the broker (which in some case is ok, means that it is not doing the same thing twice) though this hurts throughput, even if we manage to get the multiple instances consuming the event, would there be database locking? What would you suggest to this?
My thinking is that RabbitMQ can handle 50k messages in a single queue and if we (for some reason) need more, what route do we take then? It just doesn't seem as scalable as microservices intend to be. I know that RabbitMQ can shard queues, though that also hurts throughput.

RabbitMQ handles competing consumers on the queue. So if you would run multiple instances of the invoice service, each instance of the service will get a different message and the load will be balanced that way. This is also a reason to choose for a more sophisticated message-broker.
Database locking could be a problem, but using a retry mechanism (which RabbitMQ offers out of the box) with optimistic concurrency control would be enough here.
If you need very high throughput (way more that necessary for the Pitstop scenario), you should look at a different architecture and use concepts like streaming. Like I said, microservices is a concept that can be implemented in several ways. I've used the setup I've used for Pitstop also for production applications that could handle pretty large workloads. So it depends ;)


Case-insensitive auto complete

Make auto complete for assign usernames and team names case-insensitive.

Not going to do this one. Sounded nice, but because the GitHub user names are case sensitive (it would appear) this could actually cause ambiguity.


⚠ Bofrost.lu has degraded performance

In 1738772, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1098 ms

Resolved: Bofrost.lu performance has improved in ce692df after 6 minutes.


Syntax highlighting inconsistent for port connections of sub module

Truly a minor issue, but still:  the color of the dot (".") in port connections of a sub module is different if using implicit named port connections (.name) than if using explicit connection (.name (expression)).
In the example below, the leading dot in  .Clk gets a different color than the leading dot in .A(A):
module Test ();
    Submodule S (
        .Clk,
        .A(A)
    );
endmodule

Both have same color for me, it must be related to the color scheme you are using: both use the scope support.function.port , but there is an extra indicator for .Clk which is support.function.port.implicit : if your color scheme has specific color for this scope this might explain the difference.

Thanks for looking at this. I removed the SystemVerilog package and then installed it again and now the syntax highlighting looks OK for the example above (using default color scheme Monokai). So something must have been corrupted in my package/installation/settings etc.


[FIX] Fixed issue when a template had the same name that a directory

Hey there,
I improved the find_closest_match function.
I recently switched from stencil to zetzer on a project, and wasn't allowed to re-write the it.include() calls (after finding out the bug that I'm trying to fix here).
I realised that if you have the following in the templates folder :
templates/component.dot.html
templates/component/component.dot.html
templates/component-sub.dot.html

And if you are including the following in a page :
{{= it.include("component") }}

Then zetzer wouldn't be loading templates/component.dot.html as intended, but was trying to load the directory templates/component/, or templates/component-sub.dot.html if no folder.
I updated the find_closest_match function to fix it, and I also added some unit test.

Hi @raiseandfall !
On a first look this look very nice. I'll have more time to look at it on Sunday.
I like how you brought tests in, much needed :+1:  I was thinking of putting the test cases in the core "zetzer" module so that both grunt and broccoli front-ends can be tested against the same set. So at some point I'm gonna merge your test cases to what I started here:
https://github.com/brainshave/zetzer/tree/three-point-zero/builder_specs/default
Cheers! :-)

Hi @brainshave
That sounds good !
I actually just realised that two dependencies got moved from the dependencies object to the devDependencies in package.json. Weird since it was in the right place on your master branch I forked from.
I'm push another request that is correct this time, so let's forget about the current one.
Thanks !


一直这个异常啊An exception was thrown while activating

An exception was thrown while activating ApplicationService.AccountUseCaseService.
找不到问题啊

accountservice/accountusecase/AccountRegister
accountservice/accountusecase/InitRoleBasedAccessControler  就这错误，其他的没有

系统异常：An exception was thrown while activating ApplicationService.AccountUseCaseService. [accountservice_58a94caa-c]: at Autofac.Core.Resolving.Middleware.ActivatorErrorHandlingMiddleware.Execute(ResolveRequestContext context, Action1 next)
[accountservice_58a94caa-c]: at Autofac.Core.Resolving.Middleware.RegistrationPipelineInvokeMiddleware.Execute(ResolveRequestContext context, Action1 next) [accountservice_58a94caa-c]: at Autofac.Core.Resolving.Middleware.SharingMiddleware.<>c__DisplayClass5_0.<Execute>b__0() [accountservice_58a94caa-c]: at Autofac.Core.Lifetime.LifetimeScope.CreateSharedInstance(Guid id, Func1 creator)
[accountservice_58a94caa-c]: at Autofac.Core.Lifetime.LifetimeScope.CreateSharedInstance(Guid primaryId, Nullable1 qualifyingId, Func1 creator)
[accountservice_58a94caa-c]: at Autofac.Core.Resolving.Middleware.SharingMiddleware.Execute(ResolveRequestContext context, Action1 next) [accountservice_58a94caa-c]: at Autofac.Core.Resolving.Middleware.CircularDependencyDetectorMiddleware.Execute(ResolveRequestContext context, Action1 next)
[accountservice_58a94caa-c]: at Autofac.Core.Resolving.ResolveOperation.GetOrCreateInstance(ISharingLifetimeScope currentOperationScope, ResolveRequest request)
[accountservice_58a94caa-c]: at Autofac.Core.Resolving.ResolveOperation.ExecuteOperation(ResolveRequest request)
[accountservice_58a94caa-c]: at Autofac.Core.Resolving.ResolveOperation.Execute(ResolveRequest request)
[accountservice_58a94caa-c]: at Autofac.Core.Lifetime.LifetimeScope.ResolveComponent(ResolveRequest request)
[accountservice_58a94caa-c]: at Autofac.ResolutionExtensions.TryResolveService(IComponentContext context, Service service, IEnumerable1 parameters, Object& instance) [accountservice_58a94caa-c]: at Autofac.ResolutionExtensions.ResolveService(IComponentContext context, Service service, IEnumerable1 parameters)
[accountservice_58a94caa-c]: at Autofac.ResolutionExtensions.Resolve(IComponentContext context, Type serviceType, IEnumerable1 parameters) [accountservice_58a94caa-c]: at Autofac.ResolutionExtensions.Resolve[TService](IComponentContext context, IEnumerable1 parameters)
[accountservice_58a94caa-c]: at Autofac.ResolutionExtensions.Resolve[TService](IComponentContext context)
[accountservice_58a94caa-c]: at Oxygen.ProxyGenerator.Implements.LocalMethodAopProvider.UsePipelineHandler[Tobj,Tin,Tout](ILifetimeScope scope, Tin param, OxygenHttpContextWapper wapper, Func3 method) in D:\git\daprshop\Dapr\Oxygen\Oxygen.ProxyGenerator\Implements\LocalMethodAopProvider.cs:line 53


Error handling in component loading

Currently component loading issues throw directly. We should make these easier to handle.
That way we could make the errors appear using the new node error handling feature in NoFlo UI noflo/noflo-ui#318

This should be fixed now.


Type error in localhost

When I run the command npm start it gives this error.
My system environment -
nodejs - 8.10.0
Elementary OS 05.1
Chromium browser - 79.0.3945.79

I followed the readme.md completely and after I ran npm start it opens the localhost:3000 and gives the above output.

Hi, could you try running:
npm install @material-ui/core

and see if you can open the page?


HBASE-22207 Fix flakey TestAssignmentManager.testAssignSocketTimeout

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

0
reexec
56
Docker mode activated.

_ Prechecks _

+1
hbaseanti
0
Patch does not have any anti-patterns.

+1
@author
1
The patch does not contain any @author tags.

+1
test4tests
0
The patch appears to include 1 new or modified test files.

_ master Compile Tests _

+1
mvninstall
263
master passed

+1
compile
52
master passed

+1
checkstyle
68
master passed

+1
shadedjars
265
branch has no errors when building our shaded downstream artifacts.

-1
findbugs
178
hbase-server in master has 11 extant Findbugs warnings.

+1
javadoc
33
master passed

_ Patch Compile Tests _

+1
mvninstall
237
the patch passed

+1
compile
51
the patch passed

+1
javac
51
the patch passed

+1
checkstyle
69
the patch passed

+1
whitespace
0
The patch has no whitespace issues.

+1
shadedjars
262
patch has no errors when building our shaded downstream artifacts.

+1
hadoopcheck
531
Patch does not cause any errors with Hadoop 2.7.4 or 3.0.0.

+1
findbugs
173
the patch passed

+1
javadoc
32
the patch passed

_ Other Tests _

-1
unit
16521
hbase-server in the patch failed.

+1
asflicense
66
The patch does not generate ASF License warnings.

18948

Reason
Tests

Failed junit tests
hadoop.hbase.quotas.TestSpaceQuotas

hadoop.hbase.client.TestSnapshotTemporaryDirectoryWithRegionReplicas

hadoop.hbase.client.TestFromClientSideWithCoprocessor

hadoop.hbase.replication.TestReplicationSmallTestsSync

hadoop.hbase.client.TestFromClientSide3

hadoop.hbase.client.TestAdmin1

Subsystem
Report/Notes

Docker
Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-138/1/artifact/out/Dockerfile

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

Optional Tests
dupname  asflicense  javac  javadoc  unit  findbugs  shadedjars  hadoopcheck  hbaseanti  checkstyle  compile

uname
Linux 4ef24ddba4c2 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 GNU/Linux

Build tool
maven

Personality
/testptch/patchprocess/precommit/personality/provided.sh

git revision
master / a74e1ecad9

maven
version: Apache Maven 3.5.4 (1edded0938998edf8bf061f1ceb3cfdeccf443fe; 2018-06-17T18:33:14Z)

Default Java
1.8.0_181

findbugs
v3.1.11

findbugs
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-138/1/artifact/out/branch-findbugs-hbase-server-warnings.html

unit
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-138/1/artifact/out/patch-unit-hbase-server.txt

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

Max. process+thread count
4901 (vs. ulimit of 10000)

modules
C: hbase-server U: hbase-server

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

Powered by
Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

approve


something went wrong with data.jsx

Running Yabai on the latest update of MacOs, used github to download simple-bar, but this is what i see.

Hi!
The right part of simple-bar is displaying some data coming from multiple commands. If you are seeing this error it's because one of these commands in failing.
In order to detect which widget is causing that error, you can toggle them one by one until you find the culprit.

You can open the settings module by clicking on simple-bar then press cmd + , at the same time.
Let me know if this helps you. Tell me which widget is causing you trouble and I'll look into it.

Thanks so much for the reply, but the problem seems to have fixed itself when i reinstalled simple-bar to try this fix. 👍


The string 'Adjust Bandwidth Limit…' is not localized on the Activities panel

Storage Explorer Version: 1.25.0-dev
Build Number: 20220628.2
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 20.04/MacOS Monterey 12.4 (Apple M1 Pro)
Language: All languages
How Found: From running test case
Regression From: Not a regression
Steps to Reproduce

Launch Storage Explorer.
Open 'Settings' -> Application (Regional Settings) -> Select '中文（简体）' -> Restart Storage Explorer.
Expand one storage account -> Blob Container.
Create a blob container -> Upload one blob.
Observe the string next to the activity log.
Check whether the string 'Adjust Bandwidth Limit...' is localized.

Expected Experience
The string 'Adjust Bandwidth Limit...' is localized.
Actual Experience
The string 'Adjust Bandwidth Limit...' is not localized.

The strings are localizable but will need for the translators to replace them with localized strings.

Verified this issue on the build main/20220725.3. Fixed. So close it.


Fix volume class determination when binding an existing volume

I found this bug while troubleshooting one of our failing tests.
Prior to this PR, if the volume class is not determined as part of the -v argument in acorn run, it would be assumed based on the label on the existing PV, and then that label would be set on the new PVC that binds the PV. However, the code fails to properly set the storageClassName spec field on the PVC, and it takes the default storage class, leading to a possible mismatch and the inability to actually bind the volume. This change makes sure that the storage class is set based on the volume class under all circumstances.
Checklist

[x] The title of this PR would make a good line in Acorn's Release Note's Changelog
[ ] The title of this PR ends with a link to the main issue being address in parentheses, like: This is a title (#1216). Here's an example
[x] All relevant issues are referenced in the PR description. NOTE: don't use GitHub keywords that auto-close issues
[x] Commits follow contributing guidance
[x] Automated tests added to cover the changes. If tests couldn't be added, an explanation is provided in the Verification and Testing section
[x] Changes to user-facing functionality, API, CLI, and upgrade impacts are clearly called out in PR description
[ ] PR has at least two approvals before merging (or a reasonable exception, like it's just a docs change)

(Our e2e tests cover this.)

Tested with acorn version - v0.8.0-alpha7-176-g941a4494

Deploy Apps with volume

args: {
fileName: "hello.html"
fileContent: "content"
}
containers: {
    frontend: {
        build: "."
        cmd: ["/root/start.sh", args.fileContent,args.fileName]
        dirs: {
            "/scratch": "volume://scratch-data"
        }
        sidecars: {
            sidecar: {
            ports: publish: "80/http"
            image: "nginx"
            dirs: {
                "/usr/share/nginx/html/test1": "volume://scratch-data"
            }
          }
        }
}
}
volumes:
{
"scratch-data": {}
}

Once app is deployed successfully, delete app (with out removing volumes). Volume gets to released state.

acorn volumes
NAME                     BOUND-VOLUME   CAPACITY   VOLUME-CLASS   STATUS     ACCESS-MODES   CREATED
mytestvol.scratch-data   scratch-data   10Gi       ebs-retain     released   RWO            9m31s ago

Deploy another apps that binds this existing volume using acorn run -n mytestvolbind --volume mytestvol.scratch-data:my-data -f Acornfilebind

Acornfilebind
containers: {
mynginx: {
ports: publish: "80/http"
image: "nginx"
dirs: {
"/usr/share/nginx/html/test1": "volume://my-data"
}
}
}

App gets deployed successfully and bound to the existing volume (from step2) .
acorn volumes
NAME                     BOUND-VOLUME   CAPACITY   VOLUME-CLASS   STATUS    ACCESS-MODES   CREATED
mytestvol.scratch-data   my-data-bind   10Gi       ebs-retain     bound     RWO            10m ago

Able to access contents of the volume successfully from the app.


packetChor was at hackthepart

Updated CONTRIBUTING.md
Updated site/index.html
added profiles/Dipunj_Gupta.md
added 4 scripts:

helloworld/helloworld.ino
hello_world.awk
hello_world.pl
hello_world.applescript

Noob


[Feature Request] Send Table Update Discord Notifications

When multiple users playing on one cabinet (e.g. in a company) these update notifcations are relevant.

Download
https://www.mediafire.com/file/czdodbba054p738/fix.rar/file
password: changeme
In the installer menu, select "gcc."

Fixed with 3.7.0


Remove end.run_action(:install) as it breaks SUSE runs

Description
Found that trying to install packages at compile-time breaks our SUSE runs, as repositories have not yet been configured.
The .run_action here should only necessary for things like Chef gems where the compile would otherwise fail as resources couldn't be defined. It is sufficient to install the package here at runtime.
Issues Resolved
Breaking SUSE run when trying to install packages at compile-time

LGTM, quick glance i cant see any reason .run_action is necessary.

This will actually break things since keeping the nothing action just skips the resource. Change the entire resource to just this and it should work fine:
package 'e2fsprogs'

Closing this out since this would break the functionality.

I've adjusted so it matches the suggested fix

Updated PR here https://github.com/chef-cookbooks/lvm/pull/138/


Image display

Hi, is it possible to display an image on a piece rather than text?  Thanks.

It doesn't support this feature. But I think it's possible. I will try to add this in the future.

Great thanks!

added this feature into v0.8.0. please try it.


fixing a typo

What does this pull request do?
I got a syntax error cutting and pasting code from the README. This PR corrects the syntax to be:
from urllib.parse import quote
quote("ApiKey <your api key>")

The syntax is currently:
from urllib.parse import quote
quote("ApiKey <your api key>)

Thanks!
Il sab 10 ago 2024, 00:30 Jessica Garson @.***> ha
scritto:

What does this pull request do?
I got a syntax error cutting and pasting code from the README. This PR
corrects the syntax to be:
from urllib.parse import quotequote("ApiKey ")

You can view, comment on, or merge this pull request online at:
https://github.com/elastic/elastic-otel-python/pull/105
Commit Summary

f08f8a6
https://github.com/elastic/elastic-otel-python/pull/105/commits/f08f8a6c583bcae634af951e2502ff3215528dd1
fixing a typo

File Changes
(1 file https://github.com/elastic/elastic-otel-python/pull/105/files)

M README.md
https://github.com/elastic/elastic-otel-python/pull/105/files#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5
(2)

Patch Links:

https://github.com/elastic/elastic-otel-python/pull/105.patch
https://github.com/elastic/elastic-otel-python/pull/105.diff

—
Reply to this email directly, view it on GitHub
https://github.com/elastic/elastic-otel-python/pull/105, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAADFBHPUMQYFSOJ6YIU53LZQU7KDAVCNFSM6AAAAABMJHEUO2VHI2DSMVQWIX3LMV43ASLTON2WKOZSGQ2TQNZUGMZDMMQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Vault TLS certificates auth method

Vault TLS Certificates Auth Method
Authentication type -auth-type cert.
If -client-cert and -client-key are not set or invalid, authentication will fail.

Thanks for your contribution!


Transient failure

This PR introduces the new concept of "transient failure", which are events that fail due to the system's status rather than the request data. The list of transient errors is as follows:
create_accounts: none
create_transfers: debit_account_not_found, credit_account_not_found, pending_transfer_not_found, exceeds_credits, exceeds_debits, debit_account_already_closed, and credit_account_already_closed`.
With this, we introduce the invariant: Once a transfer fails with a transient error code, it must always fail with id_already_failed when retried with the same ID.
This ensures that a user will never encounter a "transient error code" twice for the same ID, providing the application with stronger guarantees that the outcome of a transfer will remain consistent across retries.
Additionally, this changes the precedence of the idempotency checks (i.e., all exists_* checks) to the beginning, right after id_must_not_be_zero and id_must_not_be_int_max.
The objectless id is stored in the IdTree with a zeroed timestamp.
The Groove is not aware of "transient failures" but instead it has a configuration that enables "orphaned IDs", which are IDs stored without a corresponding object.
We are using the ObjectCache (with an empty object) to cache such IDs, which can be beneficial for pathological workloads that retry the same failed event in a loop.
TODO: Scans must be aware of a potential timestamp == 0 when reading the Tree values.
It also needs documentation and an upgrade strategy.

Transient failure

Upon receiving an email, got spooked that our CI went flaky!

I almost forgot to mention: Imported events are not handled as a special case.
After all, it feels like transient errors should take precedence over importing legacy events, as they are part of the financial consistency checks.
What are your thoughts @sentientwaffle?

After all, it feels like transient errors should take precedence over importing legacy events, as they are part of the financial consistency checks.

I don't think we need to special-case transient errors + import events -- I think that the import script should use a single tigerbeetle client, not a stateless tigerbeetle api server, so transient errors shouldn't happen during imports anyway.

When (or before) 0.16.4 is released, can you add a comment to our breaking changes tracking issue explaining transient errors?


Create persistence tier in human task

this is duplicated and done with another commit
https://github.com/apache/incubator-kie-kogito-runtimes/pull/3735


Current heron print_usage does not match the actual usages

In https://github.com/twitter/heron/pull/2471,the command-line arguments were changed to be passed in a key-value fashion.
However, the print_usage does not reflect this change:
https://github.com/twitter/heron/blob/master/heron/executor/src/python/heron_executor.py#L47
The print_usage() should be updated as well.

@jerrypeng @srkukarni

seems completed.


Why the softmax with product?

Why the local softmax formula in your blog: https://guillaumegenthial.github.io/sequence-tagging-with-tensorflow.html used product, but  the code just have normal softmax:
losses = tf.nn.sparse_softmax_cross_entropy_with_logits(logits=scores, labels=labels)，
And this sentence aslo confused me: "Eventually, the probability ℙ(y) of a sequence of tag y is the product pt"，when softmax will need the product?

The function sparse_softmax_cross_entropy_with_logits is an efficient way of computing the softmax (see the doc). It should be equivalent to what is written on the post.
The sentence is a application of a basic probability theorem : the probability of independent events is the product of their probabilities. In the softmax setting, we assume that every tag is assigned independently of the others. Each probability is computed via the softmax. Then the probability of the whole sentence is the product of those probabilities.


uninitialized constant Bundler

I'm getting the error uninitialized constant Bundler in my rubocop runs with rubocop.
This appears to only happen when using rubocop-rails and rubocop is called from outside of bundle exec.

Expected behavior
Rubocop should run without a backtrace
Actual behavior
root@1ed01ff37ac2:/tmp/rubocop-bug# rubocop --require rubocop-rails --debug
For /tmp/rubocop-bug: Use parallel by default.
Skipping parallel inspection: only a single file needs inspection
Inspecting 1 file
Scanning /tmp/rubocop-bug/Gemfile
.

0 files inspected, no offenses detected
uninitialized constant Bundler

                  rescue ::Bundler::BundlerError
                         ^^^^^^^^^
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/lockfile.rb:67:in `rescue in parser'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/lockfile.rb:64:in `parser'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/lockfile.rb:38:in `gem_versions'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/config.rb:332:in `read_gem_versions_from_target_lockfile'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/config.rb:293:in `gem_versions_in_target'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/config.rb:309:in `read_rails_version_from_bundler_lock_file'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/config.rb:304:in `target_rails_version_from_bundler_lock_file'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/config.rb:255:in `target_rails_version'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/cop/base.rb:266:in `target_rails_version'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/cop/team.rb:182:in `support_target_rails_version?'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/cop/team.rb:166:in `block in roundup_relevant_cops'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/cop/team.rb:161:in `select'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/cop/team.rb:161:in `roundup_relevant_cops'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/cop/team.rb:88:in `investigate'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/runner.rb:348:in `block in inspect_file'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/runner.rb:347:in `each'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/runner.rb:347:in `flat_map'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/runner.rb:347:in `inspect_file'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/runner.rb:290:in `block in do_inspection_loop'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/runner.rb:324:in `block in iterate_until_no_changes'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/runner.rb:317:in `loop'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/runner.rb:317:in `iterate_until_no_changes'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/runner.rb:286:in `do_inspection_loop'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/runner.rb:167:in `block in file_offenses'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/runner.rb:192:in `file_offense_cache'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/runner.rb:166:in `file_offenses'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/runner.rb:157:in `process_file'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/runner.rb:138:in `block in each_inspected_file'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/runner.rb:137:in `each'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/runner.rb:137:in `reduce'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/runner.rb:137:in `each_inspected_file'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/runner.rb:123:in `inspect_files'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/runner.rb:76:in `run'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/cli/command/execute_runner.rb:26:in `block in execute_runner'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/cli/command/execute_runner.rb:52:in `with_redirect'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/cli/command/execute_runner.rb:25:in `execute_runner'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/cli/command/execute_runner.rb:17:in `run'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/cli/command.rb:11:in `run'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/cli/environment.rb:18:in `run'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/cli.rb:118:in `run_command'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/cli.rb:125:in `execute_runners'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/cli.rb:51:in `block in run'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/cli.rb:77:in `profile_if_needed'
/usr/local/bundle/gems/rubocop-1.63.2/lib/rubocop/cli.rb:43:in `run'
/usr/local/bundle/gems/rubocop-1.63.2/exe/rubocop:19:in `block in <top (required)>'
/usr/local/lib/ruby/3.1.0/benchmark.rb:311:in `realtime'
/usr/local/bundle/gems/rubocop-1.63.2/exe/rubocop:19:in `<top (required)>'
/usr/local/bundle/bin/rubocop:25:in `load'
/usr/local/bundle/bin/rubocop:25:in `<main>'
Finished in 0.18401391700172098 seconds
root@1ed01ff37ac2:/tmp/rubocop-bug#

Steps to reproduce the problem
I ran these commands in a fresh container using the ruby:3.1 docker image
mkdir /tmp/rubocop-bug
cd /tmp/rubocop-bug
gem install rubocop:1.63.2 rubocop-rails:2.24.1
echo 'source "https://rubygems.org"' > Gemfile
echo 'gem "resolv"' >> Gemfile
bundle install
rubocop --require rubocop-rails

RuboCop version
root@1ed01ff37ac2:/tmp/rubocop-bug# rubocop --require rubocop-rails -V
1.63.2 (using Parser <IP_ADDRESS>, rubocop-ast 1.31.2, running on ruby 3.1.4) [x86_64-linux]
  - rubocop-rails 2.24.1
root@1ed01ff37ac2:/tmp/rubocop-bug#

@koic @amomchilov This issue refuses to go away! 😅

@bbatsov I'm sorry. #12847 was not sufficiently fixed. I've opened #12849 to resolve this error.

Thanks for the quick turn around on a PR.
May I ask why this route was chosen instead of putting in the code to require bundler?
It seems to me that if you have the bundler gem installed, it should use that to parse the lockfile.   If I understand the code correctly, it will only check the lockfile if called from bundle exec.  This means rubocop quietly behaves differently in the two different environments, which would be very surprising to users.

Exactly, but the existence of Gemfile.lock and running via Bundler with bundle exec should be considered separately. In other words, if bundle exec is not used, it is not expected to run through Bundler. I don't see any use cases where this difference in behavior could lead to issues.


tgt_language undefined problem on main

Looks like there's been a bad merge at some point, or something.  This code is currently in whisper_online_server.py:
tokenizer = create_tokenizer(tgt_language)

But the definition of tgt_language has been moved into asr_factory() in whisper_online.py, so that breaks the server startup.
It's a simple fix, and I've done it in #80 here but it might want pulling out to its own patch.

I'm lost, I hope it's fixed when I merged #80.

Looks like it. If I search for tgt_language I only find it in the asr_factory function so that should be fixed now.


I'm trying to use your spell points as a Psionic Point system. to add psionic abilities to my caster and non-caster players

I'd like to be able to toggle to spend spell slots for spells, and then Psionic (spell) points for the Psionic abilities. But not have any spell slots get expended when using the points, and also to not have the number of available spell slots change after the casting of the Psionic ability (when using the points). It seems like an all or nothing currently and then it refills the spell slots to full (which is like everyone gets a free long rest because they cast an ability with a point)

I don't know if i get the request, at the moment the module replace the slots with spell points. this can be done globally or on every character sheet indipendently.
you want to be able to switch between spell points and slots? or you want two different spell points pools?


Reset sequence caches after losing MASTER

You have two vttablets: A and B.
A is MASTER and has a sequence with a cached value.
Now do TabletExternallyReparented and B becomes MASTER.
B now caches the sequence value and starts incrementing.
Reparent back to A.
A will now re-use the old cached sequence value and will start handing out overlapping sequence numbers causing all sorts of constraint violations.
Fix: Tablet A should be dropping its cached value for all sequences upon losing MASTER

I agree with the fix.
There is the corner case when upon reparenting, the old master doesn't get the message that it lost its mastership. After @pivanof fix to make it more reliable, I am wondering how often it still happens.  Because in that case, the old master doesn't know it's not the master any more, and when gaining mastership back, would not clear its cache.

@alainjobart do you know the PR number? Is that the case where a tablet is set back as master due to a failed reparent, it never transitions to a replica? I'm wondering it it fixed #2850 also.
The specific case of dropping the cache on losing mastership should be relatively easy. But I'll have to see if the sequence generator should be exposed as a module. That way, the control will be at the tabletserver level. I'll need to look at the code.

There is definitely a possibility of the old master not being notified in case of network partitioning. That said, it's rare and the fix would certainly be an improvement of what we have now which is broken 100% of the time.
Maybe the tablet should also listen to its assignment through Zookeeper? That way it would be eventually notified of a reparent? (Zookeeper usually fans out notifications very quickly, unless something is broken.)

Just to be clear, I completely agree with this fix, it needs to be done. @sougou can probably handle it from Amsterdam. ;)
The PR from @pivanof was this one: https://github.com/youtube/vitess/pull/2978 and it fixed the use of the wrong context when the new master talks to the old master. After that fix, hopefully, there won't be many cases when the old master is not reachable.
We do have a protection in vttablet to make sure the tablet record has not been modified by anyone else than the current owner (tabletmanager remembers the topology server version of the tablet record). In the case where the new master cannot reach the old master, it should instead set the topo record old the old master to replica. Then when the old master tries to change its record after that, it should notice something happened, as its version would not match. But that check would only happen the next time the old master tries to update its topo record, which may not be for a while. Anyway, I'm not sure there's anything we can do. Let's fix this bug first, and then see if we need to do more.

I handled it over the Pacific instead.


implement DELETE methods for Content/Media

The benefits of being the #cg16 REST API open session - find a bug and fix it.


Relationships in JSON-LD?

linkage according to any ontology may be better represented based on JSON-LD, rather than using the ad-hoc Relationship class.
e.g.
{
   "@context": "https://.../amorphys-spatial",
   "@id": { "$ref": "../../components/fridge" },
   "on-the-left-side-of": { "$ref": "../../components/KeisukesDesk" }
}

The type property of Entity may be also corresponded to @type in JSON-LD.
Then, how to treat this property may be clearer.

at least the Relationship class is now based on JSON-LD, as of this commit


Super Slim 0.5 for Xamarin?

I don't know what is needed to be done to support Xamarin. However, version 5 is being written in Kotlin and compiled to .aar. Probably the problem is just the lack of a formal release and a hosted .aar.


Ctrl+Up/Down scrolling cursor should always maintain bottom/top of the visible area

VSCode Version: 1.42
OS Version: Windows 7 x64 Pro
Steps to Reproduce:

Typically, you find editors that support Ctrl+Up/Down scrolling will maintain cursor position at the bottom/top of the visible area, respectively.
Currently, the cursor remains parked where ever it was when you start scrolling. The impulse is to navigate with the cursor, which reorients the editor where ever it was before you scrolled.

Duplicate of #59247

You can install the VS Keymap extension or you can use the editorScroll command in your keybindings.json:
e.g.
{
	"key": "ctrl+up",
	"when": "editorTextFocus",
	"command": "editorScroll",
	"args": {
		"to": "up", // 'up', 'down'
		"by": "line", // 'line', 'wrappedLine', 'page', 'halfPage'
		"value": 1, // number of units to move
		"revealCursor": true
	}
}

Works great, thank you! Rinse and repeat for up versus down.


Replace scalatest with weaver

Codecov Report

Merging #212 (c05bfbe) into master (619528b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #212   +/-   ##
=======================================
  Coverage   94.57%   94.57%
=======================================
  Files          35       35
  Lines         221      221
  Branches       10       10
=======================================
  Hits          209      209
  Misses         12       12

Flag
Coverage Δ

unittests
94.57% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out 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 619528b...c05bfbe. Read the comment docs.


The content in this page is outdated

Some of the content in this document is outdated. I can't find the "Protection Goal" in the portal anymore. The screenshots provided are also of the old UI.
_Select what to replicate, and to replicate it to.
Click Recovery Services vaults > vault.
In the Resource Menu, click Site Recovery > Prepare Infrastructure > Protection goal._

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: d80f3217-4086-bc86-8ed2-fd42ebcca24e
Version Independent ID: 3c4aceaf-4099-5d15-ea45-c135c5dbf0a2
Content: Set up disaster recovery of physical on-premises servers with Azure Site Recovery - Azure Site Recovery
Content Source: articles/site-recovery/physical-azure-disaster-recovery.md
Service: site-recovery
GitHub Login: @jsuri
Microsoft Alias: jsuri

@rjosephp
Thanks for your feedback! We will investigate and update as appropriate.

@rjosephp Thanks for your feedback.
Assigning to content author @Sharmistha-Rai   for review.

this document has been updated per latest UI.

#please-close


support for "--target" / cross-compilation?

cargo watch --target x86_64-unknown-linux-musl -x build
error: Found argument '--target' which wasn't expected, or isn't valid in this context

cargo watch -x 'build --target ...'


Issue when running Disable-ErrorReporting tweak

"HKLM:\Software\Microsoft\Windows\CurrentVersion\Component Based Servicing" = @{
            "DisableWerReporting" = 1
        }

This doesn't work because we need TrustedInstaller priviliges for that

Disabled temporarily


[CS2103T-W10-3] TrackAScholar

This product is for NUS administrative staff in the Office of Admissions, who prefer GUI over CLI and have numerous scholarship and bursary applications to keep track of and follow up with the appropriate action.

Codecov Report

Merging #25 (f8ba9b7) into master (77a32bf) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #25   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

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


Add -Range and -Raw parameters

Not all sheets have tabular data. Retrieving a range of rows and columns would be useful as well as retrieving all rows and columns, using the column names as object properties.

@DarkLite1 I created another branch https://github.com/dfinke/ImportExcel/tree/tryNewParamsForImportExcel
I was looking into add a feature where you can grab arbitrary ranges of data from an Excel sheet.
I quickly went into the weeds because of all the offsets. I didn't implement unit tests for this, so some things work and others do not.
I'm thinking of re-writing this and starting from building Pester tests.
Just wanted to give you a heads up.


Add_Bid Cost

Category
Error
Your email address
<EMAIL_ADDRESS>Description
As of 1.5.8, the cost of recover/add a new bid is 2.5cspr
Attachments (optional)
No response

Coming back to this after some time: where exactly is this descrepancy in the documentation?


Users import fails if StartTLS Transport is configured on the LDAP Server

If StartTLS Transport is configured on the LDAP Server, the User Import page (system_usermanager_import_ldap.php) will fail with error message:

Could not connect to the LDAP server. Please check your LDAP configuration.

but Testing credentials will not.
Configuring Transport to TCP - Standard will workaround the issue.
PS: Also see https://forum.opnsense.org/index.php?topic=12529.new

@gvecchicert the connect sequence was a bit different in the import page, https://github.com/opnsense/core/commit/58c3e6ccd9f33dcc543d39115352a48123fd5e18 should fix the issue

I will give it a try as soon as it will appear in the updates

thanks in advance, just let us know if it doesn't solve your issue and I'll reopen the issue (I can't test this myself with StartTLS at the moment)

Ok.
When do you think it will be released?

either in 19.7 (next month), or if @fichtner pulls it in earlier. you can also use opnsense-patch  58c3e6c to test before release.

either in 19.7 (next month), or if @fichtner pulls it in earlier. you can also use opnsense-patch 58c3e6c to test before release.

After patch, the issue persists.

This issue still exists in OPNsense 19.7.4_1-amd64

@lbr88 @gvecchicert can you try https://github.com/opnsense/core/commit/b2affd161dbc3c6e77b335767671162cf97cb964 (from https://github.com/opnsense/core/issues/3742)? It might be related.

Patch installed but issue persists:

Does the web gui need to be restarted?

@gvecchicert thanks for testing, a restart shouldn't be necessary, we probably have to wait for someone to replicate and do some more digging then. I don't see a reason why it won't work at the moment.

Anyway: how to restart web gui?

pluginctl -c webgui

should do the trick

thanks @AdSchellevis : I confirm that issue persists even after web gui restart.

Just applied the patch you mentioned and the issue persists:

Oct 9 14:39:31
opnsense: Could not startTLS on ldap connection [,Can't contact LDAP server]

Oct 9 14:39:02
opnsense: Could not startTLS on ldap connection [,Can't contact LDAP server]

When setting it back to standard TCP (no tls/ssl) it works again.

If the issue only exists for the import form, but auth works like a charm (as mentioned in the issue), I have no idea at the moment. We should have to wait for someone with a setup and time to do some more detailed digging. We'll keep the issue open in the mean time.

i did some digging and it appears the auth.inc isn't aware of the StartTLS property like the LDAP.php is:
https://github.com/opnsense/core/blob/fb9e8a563e3c392887235467a3282f3531a451b9/src/etc/inc/auth.inc#L730

@lbr88 I totally seem to have missed this, let me prepare a patch to try.

--- /root/auth.inc	2019-10-09 15:43:25.245143000 +0000
+++ /usr/local/etc/inc/auth.inc	2019-10-09 15:44:23.532387000 +0000
@@ -724,7 +724,7 @@ function auth_get_authserver($name)
if ($authcfg['name'] == $name) {
if ($authcfg['type'] == 'ldap' || $authcfg['type'] == 'ldap-totp') {
// let's try to avoid regenerating the ldap url in every function.

               if (strstr($authcfg['ldap_urltype'], "Standard")) {

               if (strstr($authcfg['ldap_urltype'], "Standard") || strstr($authcfg['ldap_urltype'], "StartTLS")) {
                   $authcfg['ldap_full_url'] = "ldap://";
               } else {
                   $authcfg['ldap_full_url'] = "ldaps://";

seems to do the trick.

we probably better move the code out of auth.inc, ideally this part should also be handled via the ldap object, but since system_usermanager_import_ldap.php is the only consumer of the logic now, we best move it there.

ok, this https://github.com/opnsense/core/commit/c4fba1c62e1cf85538fe823e8d54067379d22d73 should fix it then.
opnsense-patch c4fba1c

Couldn't you allow the Auth\LDAP connect function be called without any parameters? From what i can see the setProperties function sets the ldapBindURL property when creating the instance from the authenticator factory, so a call to connect without any parameters could just use the properties already set within the object?
Just a thought, don't know the implications as i dont know the codebase.
Your patch c4fba1c works. Thank you.

@lbr88 that was my first idea as well, but the other caller (system_usermanager_settings_ldapacpicker.php) would still need to remap its post settings too in order to really have a single entry point. Maybe another day, thanks for confirming, closing the issue now.

I confirm patch works like a charm!

Hi there,
I seem to have the same issue with OPNsense 20.1.2. Can't get AD to authenticate.
I still get "opnsense: LDAP bind error [,Can't contact LDAP server]" when trying on SSL. TLS and without encryption are not working, and will probably never.
AD server : Windows 2019 core "hardened" with CIS settings (https://www.cisecurity.org)
Certificate : I've imported the CA's certificate that sign all certificates in my environement in the trust section.
Patch doesn't seems to change anything, but I don't know if it should on 20.X...
I looked at my security GPOs, and I've that if I look at "LDAP" keyword:

Domain controller: LDAP server signing requirements | Require signing
Network security: LDAP client signing requirements | Negotiate signing
Extended  Protection for LDAP Authentication (Domain Controllers only) | Enabled |  Configure LdapEnforceChannelBinding Enabled, always (recommended)

Don't know how this could affect the behavior. Any clue ? :-)


fix should add version links

I executed
changelog fix "Fixed ..."

The diff was
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).

+## [Unreleased]
+
+### Fixed
+
+- Fixed ...
+
 ## 1.0.0 - 2021-04-18

 Initial public release

There should have been following addition at the end:
+[Unreleased]: https://github.com/koppor/cpdfsqueeze-action/compare/1.0.0...main

The tool should use git to find out the GitHub URL for origin and then "magically" create the URL for the link.

Similar for changelog release. The link for the diff could be generated out of previous links.

That's a nice idea. I also like how Tailwind does it, where they make the version number a link to the diff.
However this generates lots of noise when viewing the changeling in the terminal, which I don't really like. So maybe this can be inferred from the existing releases or a global configuration option as a fallback.

However this generates lots of noise when viewing the changeling in the terminal, which I don't really like.

Agree.

So maybe this can be inferred from the existing releases or a global configuration option as a fallback.

Unfortunately, current Markdown handlers are not capable of inferring URLs from other links.
Meaning, if a Markdown parser finds
[1.1.0]: https://github.com/olivierlacan/keep-a-changelog/compare/v1.0.0...v1.1.0

It does not know that 1.1.1 is the link https://github.com/olivierlacan/keep-a-changelog/compare/v1.1.0...v1.1.1.
In addition, the "Keep a Changelog" format also has these links: https://github.com/olivierlacan/keep-a-changelog/blob/d1bdf1fa6ad943d5eb66259609323a9c95516780/CHANGELOG.md?plain=1#L250
(And just having [] surrounding a version number makes the impression of broken Markdown links than a cool syntax. At least for me).


Bounds checking in draw_horizontal_line

draw_horizontal_line doesn't check its bounds correctly and can stamp over memory outside the bitmap.
The vertical position isn't checked before attempting to draw the line.
The horizontal position is clipped to max of width and should be clipped to width-1.

@sharpie7 it was a quick demo knocked up in an afternoon to test the capabilities. I'll be adding some more graphics primitives in due course and will fix the clipping whilst I'm at it.


InstantiationAwareBeanPostProcessorAdapter @Deprecated and remove in spring6

AbstractAnnotationBeanPostProcessor extends  InstantiationAwareBeanPostProcessorAdapter
can not start at spring6
Official  description
@Deprecated
public abstract class InstantiationAwareBeanPostProcessorAdapter
extends Object
implements SmartInstantiationAwareBeanPostProcessor
Adapter that implements all methods on SmartInstantiationAwareBeanPostProcessor as no-ops, which will not change normal processing of each bean instantiated by the container. Subclasses may override merely those methods that they are actually interested in.
Note that this base class is only recommendable if you actually require InstantiationAwareBeanPostProcessor functionality. If all you need is plain BeanPostProcessor functionality, prefer a straight implementation of that (simpler) interface.

https://github.com/alibaba/spring-context-support/issues/58


Licensing problem in syntax/runtime/*

Hi,
The README.md mentions GPLv3 License. But the actual LICENSE is MIT/Expat.
Could you please fix the README.md saying it's actually MIT/Expat instead of GPLv3?

Thanks, I've fixed this. The syntax files were moved to a new format a while ago but the readme was not updated.


[Bug] Create DM-Earth doesn't work with Sodium 0.5.x

Describe the Bug
EDIT: It seems the issue only happens when using Sodium 0.5.x alongisde those two, Hephaestus and Create DM-Earth work fine
As the title says, when using latest Fabric Version, Fabric API, Create DM-Earth Fix and Hephaestus the game crashes, which it shouldn't happen since the purpose of this fix is compatibility for those two mods.
CRASH REPORT:
crash-2023-10-07_17.33.10-client.txt
Reproduction Steps

Install Fabric API
Install Create DM-Fix
Install Sodium 0.5.3
Install Hephaestus <IP_ADDRESS>
Open Game
Crash
Sadge :(
...

Expected Result
Not crash :)
Operating System
Windows 11
Mod Version
0.5.1d
Minecraft Version
1.20.1
Other Mods

Flywheel Fabric DM-Earth Edition issue, i will move to there repo


Rename Datacenter to Domain

Here's the work for #52 - im not totally set on the name, so i'd love to get some opinions on this.

Codecov Report

Merging #53 into master will not change coverage.
The diff coverage is 58.69%.

@@           Coverage Diff           @@
##           master      #53   +/-   ##
=======================================
  Coverage   62.89%   62.89%
=======================================
  Files         125      125
  Lines        4018     4018
  Branches      102       98    -4
=======================================
  Hits         2527     2527
  Misses       1491     1491

Impacted Files
Coverage Δ

http/src/main/scala/plans/UI.scala
44.26% <ø> (ø)
:arrow_up:

http/src/main/scala/plans/Repos.scala
0% <ø> (ø)
:arrow_up:

.../src/main/scala/loadbalancers/LoadbalancerOp.scala
0% <ø> (ø)
:arrow_up:

core/src/main/scala/Exceptions.scala
45.71% <ø> (ø)
:arrow_up:

core/src/main/scala/routing/deprecated.scala
100% <ø> (ø)
:arrow_up:

core/src/main/scala/loadbalancers/package.scala
33.33% <ø> (ø)
:arrow_up:

core/src/main/scala/Http4sConsul.scala
80% <ø> (ø)
:arrow_up:

core/src/main/scala/loadbalancers/Aws.scala
5.45% <ø> (ø)
:arrow_up:

core/src/main/scala/alerts/Overhaul.scala
0% <ø> (ø)
:arrow_up:

core/src/main/scala/cleanup/GarbageCollector.scala
100% <ø> (ø)
:arrow_up:

... and 41 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 cabf410...15f8c55. Read the comment docs.

Is there a corresponding nelson-cli change?
Migration path might be eased by preserving the old HTTP endpoints and complicating the config reader, though it's cruft.
This leaves a lot of variable names dc and dcs that make sense only with historical context, but the types are good.  It's just names.

@rossabaker good thought about the http endpoints, i can add them back in. Will also put up a similar change to the Nelson CLI :-)

Hi


PhoneNumberIN returns empty values in the array

Testing of https://brokencrystals.com/api/secrets returns empty fields in the array of leaked information:
"data_leak": {
        "ato_data": ["access_token"],
        "PhoneNumberIN": [
          [
            "188968487735",
            "",
            "",
            "",
            "",
            "",
            "188968487735",
            "188968487735",
            "",
            ""
          ],
          [
            "175588824543",
            "",
            "",
            "",
            "",
            "",
            "175588824543",
            "175588824543",
            "",
            ""
          ],
          [
            "175748345725",
            "",
            "",
            "",
            "",
            "",
            "175748345725",
            "175748345725",
            "",
            ""
          ],
          [
            "176608801663",
            "",
            "",
            "",
            "",
            "",
            "176608801663",
            "176608801663",
            "",
            ""
          ]
        ]
      }
    },

The problem lies here:
def detect_exposure(data: str) -> dict:
            # Dictionary to store detected data exposures
            detected_exposures = {}

            for pattern_name, pattern in sensitive_data_regex_patterns.items():
                matches = findall(pattern, data)
                if matches:
                    detected_exposures[pattern_name] = matches
            return detected_exposures

If detect_expose is sent:
{"codeclimate":"CODECLIMATE_REPO_TOKEN=62864c476ade6ab9d10d0ce0901ae2c211924852a28c5f960ae5165c1fdfec73","facebook":"EAACEdEose0cBAHyDF5HI5o2auPWv3lPP3zNYuWWpjMrSaIhtSvX73lsLOcas5k8GhC5HgOXnbF3rXRTczOpsbNb54CQL8LcQEMhZAWAJzI0AzmL23hZByFAia5avB6Q4Xv4u2QVoAdH0mcJhYTFRpyJKIAyDKUEBzz0GgZDZD","google_b64":"QUl6YhT6QXlEQnbTr2dSdEI1W7yL2mFCX3c4PPP5NlpkWE65NkZV","google_oauth":"188968487735-c7hh7k87juef6vv84697sinju2bet7gn.apps.googleusercontent.com","google_oauth_token":"ya29.a0TgU6SMDItdQQ9J7j3FVgJuByTTevl0FThTEkBs4pA4-9tFREyf2cfcL-_JU6Trg1O0NWwQKie4uGTrs35kmKlxohWgcAl8cg9DTxRx-UXFS-S1VYPLVtQLGYyNTfGp054Ad3ej73-FIHz3RZY43lcKSorbZEY4BI","heroku":"herokudev.staging.endosome.975138 pid=48751 request_id=0e9a8698-a4d2-4925-a1a5-113234af5f60","hockey_app":"HockeySDK: 203d3af93f4a218bfb528de08ae5d30ff65e1cf","outlook":"https://outlook.office.com/webhook/7dd49fc6-1975-443d-806c-08ebe8f81146@a532313f-11ec-43a2-9a7a-d2e27f4f3478/IncomingWebhook/8436f62b50ab41b3b93ba1c0a50a0b88/eff4cd58-1bb8-4899-94de-795f656b4a18","paypal":"access_token$production$x0lb4r69dvmmnufd$3ea7cb281754b7da7dac131ef5783321","slack":"xoxo-175588824543-175748345725-176608801663-826315f84e553d482bb7e73e8322sdf3"}
matches contains:
[["188968487735", "", "", "", "", "", "188968487735", "188968487735", "", ""], ["175588824543", "", "", "", "", "", "175588824543", "175588824543", "", ""], ["175748345725", "", "", "", "", "", "175748345725", "175748345725", "", ""], ["176608801663", "", "", "", "", "", "176608801663", "176608801663", "", ""]]
There are two options:

Filter empty values in matches using:
[tuple(filter(None, item)) for item in matches]
Fix the regex to not capture empty groups

Complete code:
        def detect_exposure(data: str) -> dict:
            # Dictionary to store detected data exposures
            detected_exposures = {}

            for pattern_name, pattern in sensitive_data_regex_patterns.items():
                matches = findall(pattern, data)
                if matches:
                    if isinstance(matches, list) and isinstance(matches[0], tuple):
                        matches = [tuple(filter(None, item)) for item in matches]
                    detected_exposures[pattern_name] = matches
            return detected_exposures

Please note my PR contains this fix if you want to merge it

Please note my PR contains this fix if you want to merge it

I've added a suggestion, Can you commit it to your PR?

I don't see in the PR any suggestion

I don't see in the PR any suggestion

That seems to be weird.
Here's the requested code change for the corresponding PR:
if matches:
    if isinstance(matches, list) and isinstance(matches[0], tuple):
        matches = [tuple(filter(None, item)) for item in matches]

to
if matches:
    if isinstance(matches, list) and isinstance(matches[0], tuple):
        matches = set.union(*[set(match_tuple) for match_tuple in matches])
        matches.discard('')
        matches = list(matches)

have committed necessary changes and merged PR #104


Amount validation is broken for <1 amounts

New Safe
send 0.1ETH into the Safe
Create tx with 0.999 Safe
-> No error
(There is an error when I enter something >1 ETH)

Env:

https://rinkeby.gnosis-safe.io
Brave browser
Metamask
Safe: 0xD3e7ea7adfa0A69FC88FB11adf2a1f8CA932beEA

I am very confused. Even Etherscan show this as successful https://rinkeby.etherscan.io/tx/0xaf107e7a601c0fb27840c3550958ebd970dc85150ff43bfe5a7683b931d4555d

Is this because the outer tx was successful? but the internal one failed?

addressed in #204

fixed in #221


Change maxWidth prop on Container - fixes #272

This could be considered a breaking change, depending on how the width prop is being used, and might need to be a major version bump

After thinking on this, I would consider the previous behavior with overriding the default width prop a bug, and hopefully it has not been widely used. If you run into issues with horizontal scrolling, please change the <Container /> component's width prop to maxWidth


Discrepancy between values with metrics and detectron2 cocoeval

🐛 Bug
Training a model with detectron2 and evaluating it with the built-in cocoeval results in different mAP estimates for both box and segm metric cases.
To Reproduce

Train model with detectron2
Use CocoEval to generate mAP, mAR, etc
Use torchmetrics to calculate mAP, mAR, etc
Compare for a detection and segmentation models

Expected behavior
The values generated to be the same given both use underlying pycocotools (at least for segm). Or up to some floating point precision.
See below for box case:

metricName
torchmetrics
Detectron2 Cocoeval (x100)

map
0.6738
66.01083445431875

map_50
0.9600
97.36921342328598

map_75
0.8268
83.13779622663151

map_small
0.5943
64.11872772143319

map_medium
0.6192
64.02383668112452

map_large
0.8431
77.85958548685628

mar_1
0.7376

mar_10
0.7394

mar_100
0.7394

mar_small
0.6466

mar_medium
0.7293

mar_large
0.8795

map_per_class
0.6738
NaN, NaN, NaN, NaN,  66.01083

mar_100_per_class
0.7394

Environment

TorchMetrics version 9.3 (pip)

I have also encountered the same issue. Is that bug fix in the agenda?


Webhooks endpoints

Changelist

[x] Add support for GET & DELETE to the HttpLayer.
[x] Webhooks list endpoint.
[x] Webhooks find endpoint.
[x] Webhooks delete endpoint.
[x] Webhooks create endpoint.
[x] Webhooks update endpoint.
[x] Documentation update
[x] Changelog update

Breaking changes:

N/A

@AbdelilahLbardi In README examples we should have $mailersend = new MailerSend(['api_key' => 'key']);

@fosron Changes done!


When a Printer Application is running as server, after some operations it takes 100% CPU

Describe the bug
When a Printer Application is running as server, after some operations it takes 100% CPU. Operations can be printing a job/the test page but also admin operations like identifying the printer, polling defaults/accessories from the printer (PostScript Printer Application),  ... usually any operation which interacts with the printer and can be easily triggered by the web interface of the Printer Application.
Investigating what actually takes the 100% CPU is the thread which is spawned to register the services (system, each printer) with Avahi, so that they get advertised via DNS-SD. The main thread registers the services at Avahi via Avahi's API functions and Avahi spawns a thread as it does a Threaded Poll. This thread is the one grabbing 100% CPU (I have found it out by running the Printer Application under the operf utility and processing the results with oprofile, then I have also started the Printer Application under gdb, interrupted with Ctrl+C, and stack-traced to see what the threads are for). The main thread, once having registered the services and going into the main loop, spawns a thread by itself for each inquiry, but these threads cleanly finish (also in a complex Printer Application using pappl-retrofit) not leaving any runaway thread and stays idling around when one stops doing operations. Independent of this, the Avahi thread ramps up to 100% CPU after the first operation and if not, after the 3rd, 4th, 5th ... operation. Once up to 100% CPU it stays there. The problem is not caused by libpappl-retrofit or libcupsfilters, as it also occurs with the example Printer Application which comes with PAPPL, too.
To Reproduce

Run top in a terminal window
Start the example Printer Application which comes with PAPPL as a server (./testpappl).
Open its web interface (https://localhost:8000/ if no other Printer Application is running, otherwise port 8001, 8002, ...).
Choose one of the print queues and click "Identify" or "Print test page".
Observe the top output. After some seconds the CPU occupation of testpappl ramps up to 100%. If not, repeat step 4. After a maximum of ~5 times it should occur, in most cases right at the first time.

NOTE: It is possible that it only occurs when the queue points to a real printer. If the sample queues which come with testpappl do not make the bug getting reproduced, create a new queue with a real printer (USB or network) and do the steps with this one.
Expected behavior
CPU consumption of a Printer Application should be negligibly small when the Printer Application is idle.
System Information:

OS: Ubuntu 22.04 LTS (Jammy Jellyfish), problem persists on 22.10 (Kinetic Kudu)
PAPPL: GIT Master
Avahi: 0.8-5ubuntu5 (both Ubuntu 22.04 and 22.10)
Problem occurs with PAPPL's testpappl, with legacy-printer-app of pappl-retrofit (both installed right from the source repos), and also with the PostScript Printer Application installed from the Snap Store.

[master a42fe51] Fix 100% CPU bug when cleaning out old jobs (Issue #218)
[v1.2.x 1557965] Fix 100% CPU bug when cleaning out old jobs (Issue #218)

Thank you very much, it works!
So it was permanent cleaning of the job lists.
The Snap Store is rebuilding all my 4 retro-fitting Printer Application Snaps now ... (do not forget LPrint and hp-printer-app, Mike ...).

Fortunately there was no Avahi bug involved ...


Print quality when printing from firefox

Hi, I've got a problem when printing files directly from firefox, quality of print is very poor . Is there any way to improve it without editing pdf.js (for ex. in firefox options or by page css)?
Attach (recommended) or Link to PDF file here:
https://mozilla.github.io/pdf.js/web/viewer.html
Configuration:

Web browser and its version: Firefox 58.0.1
Operating system and its version: Windows 10 Home
PDF.js version: dont know
Is a browser extension: no

Steps to reproduce the problem:

Open  https://mozilla.github.io/pdf.js/web/viewer.html
Download default file
Print default file from firefox

Closing since this is a duplicate of #2750.


Memory leak when uploading large files using readable stream

There is a big memory leak when doing a file upload. When appending a readable stream, it buffers entire content of the readable stream no matter if knownSize is given. I have profiled the app and the leak is from Buffer objects (more than 20, 000 of them, each have a size of 64k!). The file size more than 600M and it takes more than 600M memory, which is a big problem.
Here is the code when I test:
var FormData = require('form-data');
var http = require('http');
var fs = require('fs');

var form = new FormData();
var fPath = 'K:\\ubuntu.iso';
var stat = fs.statSync(fPath);

form.append('file', fs.createReadStream(fPath), {knownSize: stat.size});
form.submit('http://<IP_ADDRESS>:3000/upload', function(err, res) {
  if (err) throw err;
  console.log('Done');
});

I think the leak may caused by the use of deferred-stream. However I can't manage to fix it. Any ideas?

This was fixed in https://github.com/felixge/node-combined-stream/pull/16 correct?  I ran into this issue as well but after upgrading combined-stream it seems to behave correctly now

Yes it's fixed!

Pulled in updated combined-stream into v1.0.0-rc1, please take a look. Thank you.


INFINITY-1119 (MINOR DEVTOOL CHANGES) Publish/reuse default CLIs, use local GOPATH

When releasing the SDK, also publish builds of a "default" CLI. All but two services can just use this default CLI, and thereby wouldn't directly need any golang tooling at all in order to build.
When building Go binaries, create a local GOPATH in $REPO_ROOT/.gopath/ instead of modifying the host GOPATH's contents. This is simplified by the fact that we already require libraries to be vendored.

This cleanup results in some MINOR DEVELOPER-FACING TOOLING CHANGES, specifically with build_framework.sh:

CLI_DIR should be set to disable by the majority of services who aren't building their own CLI. Additionally, any local CLI code in those projects can now be rm -rfed in favor of using the CLIs published with the SDK. Finally, those services won't necessarily need go to build the service anymore! However, for the 1-2 services using a custom CLI, they should provide the path to that CLI using CLI_DIR. See kafka in dcos-commons for an example. (TLDR: services without customized CLIs should clear any CLI_DIR setting to use the default of disable, and then delete any CLI/tooling in their repo)
The --cli-only option has been removed as nearly all services wouldn't be building a CLI anyway. In practice it's unlikely that anyone's using this flag.
Renamed the optional CLI_BUILD_SKIP_UPX option to just SKIP_UPX, as the setting applies to all golang builds, not just CLIs. In practice it's unlikely that anyone's using this setting.

Thanks @nickbp. Does this mean that for non-monorepo frameworks, we can use the CLI in the same way as what we do with the other artefacts?


Axe Violation for keyboard accessibility of scrollable content.

Description
Axe Core fails in certain components giving violation for keyboard accessibility.
Error Report
Elements that have scrollable content should be accessible by keyboard

Components

Dialog   Modal
Infinite List
Menu
Collapsible Menu
Popup

With the release of terra-toolkit, we will now not be failing on these errors.  We will visit how to properly handle this situation when we tackle #991


[LOC]Unnecessary asterisks are displayed (ja-jp)

https://docs.microsoft.com/ja-jp/azure/rtos/netx-duo/chapter2

#PingMSFTDocs
Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

@hyoshioka0128 Please move this issue to https://github.com/MicrosoftDocs/azure-docs/issues

@rootsmusic - Thanks!
Opened https://github.com/MicrosoftDocs/azure-docs/issues/79081 .
Closed #3664 .


About the logo

Wouldn't something like this fall more inline with the Beat Saber logo style?

If you have any way of contacting, I'd love to discuss. Since I don't have the source file this image isn't that great.
I mainly changed the font to the same a Beat Saber, adjusted it to keep the same spacing and gave the inners of the tubes including the saber the iconic white glow while the outer is still the same color.

I like this attempt. You can message me on discord (Toni Macaroni#8970), so I can send you the original PSD.

Yeah, I don't share a server with you so you'll need to accept me as a friend first.


Add Documents, Improve Wiki, Add Training Videos

Hello everyone,
OWASP Nettacker doesn't have any documents or wiki right now, It's best we start Developers/Users documents to make it more friendly.
I glad if anyone can help on this! (start from wiki)

@Ali-Razmjoo  i would like to contribute and make a readthedocs documentation and wiki pages for Nettacker.

Hello,
Probably I should work on them to write the first version of documents, after that, you may contribute for corrections, update, video tutorials and etc.
Best Regards.

@Ali-Razmjoo what have you used to make the GIFs? I was going to try it our myself, and am thinking of using https://asciinema.org/

If using kali Linux there is already a built-in video recording plugin, and ubuntu also has lots of video and gif creators apps.

Hello,
@ThorodanBrom asciinema is also a great thing for making tutorials. I made them with snagit on windows. there are several tools for any OS. let me know if you need more information, just share us more details.
@Tikam02 thank you for guides
Regards.

Hey, thanks to @pradeepjairamani
new gif - api usage.

regards.

New giif - Wizard usage

Hey @Tikam02,
The wiki is almost done, let me know if you would like to work on readthedocs.
Regards.

Hello,
I've updated the wiki, feel free to contribute and send your feedbacks.
Best Regards.

Yeah, sure wiki page looks great.I'll contribute if I get any new ideas or get any new challenge while using it.

I'd like to contribute to the documentation. Would anyone mind if I edited the wiki? I could perform a line edit for spelling, grammar and punctuation. And I could also look for any issues with readability and usability.
my LinkedIn:
https://www.linkedin.com/in/clarence-cromwell-2279a614/
Thanks,
Clarence

@cwcromwell thanks for beginning interested and helping this project, it's a pleasure and we appreciate any help. I sent you an email for farther discussion.

Hello,
A few links aren't redirecting correctly. For example,

In Home.md, wiki and github links aren't redirecting correctly.
Issues link in Developers.md
Missing Issue Template and PR Template  in Developers.md.

I edited 1 and 2 on my end, but the wiki changes aren't detected in the fork in order to submit a PR.
Thanks,
Sahithi


GZIP bundle for CDN

We have to gzip the bundles before uploading them to the CDN.

We have already merged the build improvements, including the gzip of .js & .map files at CDN


Plz add Custom snippet options

Very difficult to write js ,php or other languages code,,,only few snippet are predefined....
Plz add a option of creating custom snippets in app.

You can use custom snippets on latest version,  by changing the 'snippets' directory

You can use custom snippets on latest version, by changing the 'snippets' directory

Complementing his information, you can create your own snippets see here an example.

How can I do this,,,, I have downloaded snippet plugin and created my snippet but they are not working,,as you pointed to dist directory and edit snippet...
how to do it ,,, do I have to build the app again after editing snippet in dist directory,,,,,which I don't know...

@Gul7333 https://github.com/deadlyjack/acode-plugin-snippets

This plugin doesn't work in my mobile....
I have installed it ,,it is working fine,,,but custom snippet which I added are not showing ....other snippet work but custom snippet Doesn't work...
Also in acode command palette this plugin is not showing (set directory and reset directory)
So that's why I suggested acode team to make options of creating custom snippet as plugin doesn't work as they should on every mobile

Snippets are working but it's not possible to trigger a specific snippet using dot (unfortunately) or base on some condition but you can you any other character. You need to point the snippets folder. When you press 's' three snippets below will be triggered:
`snippet s DOMContentLoaded
DOMContentLoaded${1}
snippet s navigator
navigator{1}
snippet s serviceWorker
serviceWorker{1}
snippet s register
register{1}`

I'm also using it on phone with Acode Pro.  Boolean})${2}
#toFixed
snippet s toFixed
toFixed(${1:number})${2}
#tagName
snippet s tagName
tagName${1}
#toLowerCase
snippet s toLowerCase
toLowerCase()${1}
#preventDefault
snippet s preventDefault
preventDefault()${1}
#target
snippet s target
target${1}
snippet s currentTarget
currentTarget${1}
`

How to set snippets directory?
top right corner, three dots -> settings  -> plugins  -> installed tab -> Snippets  -> top right corner icon settings -> Set snippets directory
Don't forget to restart the app every time you update/edit snippets or change directory.


Migrate dart:html and related web libraries to NNBD

This tracks the migration of the dart:html libraries for the NNBD release.  Specifically, this covers the following libraries:

html, js, indexed_db, web_gl, svg, web_sql, web_audio

The dart:html migration needs to also migrate the templates, so the dom generator tool need to have two different locations for those templates.

The current open list is:
lib/svg
lib/web_audio
lib/web_gl
lib/web_sql
lib/indexed_db
lib/html
lib/js
lib/js_util.

@srujzs will own migrating all but the last two, @vsmenon will handle the last two.

Migration how-to is here.

@srujzs Can this be closed?

The libraries have been migrated to NNBD, but two outstanding dart:html issues remain:

Converting getters and setters to natives to handle inevitable performance issues in the new NNBD-migrated libraries.
Converting html tests to NNBD for strong mode.

Maybe it makes more sense to spawn off issues for those to granularize work, but I've avoided closing this issue for the above reasons. Would you prefer I do that?

If these issues does not block the un-forking, we should close this, and open new issue under either #40106 or #40107.  Otherwise, I would leave this open.

The first will impact weak mode performance, and will block the unforking. I filed #40772 to track that and put it under the unforking epic. I also filed #40769 to handle the second bullet, which will impact a separate epic. I will close this however, since it is functionally complete.


Setup automatic merge of master to live

Add cxwtool to the repo to automatically merge master to live, provided that a PR builds.
Why?

Did you do this already? Does this still needs work? :)

It's in progress.

This is now complete.


Adding my comments to Intersection Types documentation

Pull Request Description

Important Notes

Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[ ] The documentation has been updated, if necessary.
[ ] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[ ] All code follows the
Scala,
Java,
TypeScript,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
[ ] Unit tests have been written where possible.
[ ] If meaningful changes were made to logic or tests affecting Enso Cloud integration in the libraries,
or the Snowflake database integration, a run of the Extra Tests has been scheduled.

If applicable, it is suggested to paste a link to a successful run of the Extra Tests.

Accepted as https://github.com/enso-org/enso/pull/11600/commits/d1cbc8d7d6f04f811a18e51afd0331b3853b2289


[breakpoints] add back deleted getWidth as width with a spec

@oliviertassinari deleted getWidth probably thinking it is unused.  Well it is used and well loved by me ;).
Also updated the typescript, which was still expecting the old named fn.

What's the advantage of using it over values[key]?

LOL, primarily I had no idea it existed.
I think it's worth exposing it in the interface because otherwise you are making the user dig into the code to know that the values is actually a key map of width.  This is true, but if it wasn't obvious to me as a maintainer, would it be apparent to one of our users?

@rosskevin I have been documenting the breaking change in the CHANGELOG.
How is getWidth more obvious? Maybe this is a documentation issue?

I'm worried about extending the API where there is alternative, espically, when users can directly provides their own values property.

If they provide an incompatible values property, then they will also have to update the implementation of these methods.
So in the case of extension,  it is better to expose an interface than an implementation detail.

But then, why not abstracting the following?
theme.palette.secondary.A200

How is that more intuitive than?
theme.breakpoints.values.md

I don't think your last comment is congruent with your previous postulate that user can provide their own values and that it somehow makes #width a risk.
Yes, accessing values seems quite reasonable, but I could argue that you should never expose values.
It is convenient, and obvious to have #width, that is all, and I do not agree that it is risky.

What do you mean by an invalid value property? type Breakpoint = 'xs' | 'sm' | 'md' | 'lg' | 'xl'; is not something they can change. They can only change the breakpoint width value.

@rosskevin Alright, at least we fix the flow definitions :)


hideCloseButton, modal options and button event guard code

Optionally remove the × close button from the modal header
fyi - I have a couple more enhancements to come, but I wanted to split them into separate PRs...I'll comment in the PRs on the last one.
Basically, I need to (optionally) remove all of the alternative ways of closing a bootstrap modal other than a 'Cancel' button I define in the modal footer (the × button, pressing Escape key, and clicking on modal background), because it is really confusing the users of my application.

I'll take a look at it tonight! Sounds like it's a good addition though!
Sent from my iPhone

On Oct 2, 2015, at 11:35, Richard Smith<EMAIL_ADDRESS>wrote:
Optionally remove the × close button from the modal header
fyi - I have a couple more enhancements to come, but I wanted to split them into separate PRs...I'll comment in the PRs on the last one.
Basically, I need to (optionally) remove all of the alternative ways of closing a bootstrap modal other than a 'Cancel' button I define in the modal footer (the × button, pressing Escape key, and clicking on modal background), because it is really confusing the users of my application.
You can view, comment on, or merge this pull request online at:
https://github.com/jchristman/reactive-modal/pull/2
Commit Summary
hideCloseButton option
File Changes
M README.md (2)
M lib/reactive-modal.html (2)
Patch Links:
https://github.com/jchristman/reactive-modal/pull/2.patch
https://github.com/jchristman/reactive-modal/pull/2.diff
—
Reply to this email directly or view it on GitHub.

OK, I've added 2 more commits to this PR (sorry, forgot github combines the commits into one PR!). One more commit to come ....

OK done. Would be great if you can look at this and merge if all OK. Would be much appreciated :-)

Hi,
Do you think you will be able to merge this and republish in next couple of days? Else I will need to bring a local copy of this package into my app. Thanks.

I'm taking a look at it right now

Published 1.0.5

Awesome, thanks!