Git repo doesn't merge well with current (rather old) Debian version

I want to package the latest git version, as it includes the fix for background scaling, as well as some other nice options, but I am unable to do so.  2.2.2 from the jwm homepage works brilliantly.  however, I am wondering if something changed (the recent change today did not fix my issue) that I am unaware of... or if there is something remarkably different in the releases that is not present in the current git repo.  Any sort of help tracking down this issue would be most appreciated, as I want to have the latest JWM available, and have to option of continually updating it very easily.  Thanks so much!

What doesn't merge well?
If it helps, I have a list of changes from v2.2.2 here: http://joewing.net/projects/jwm/release-2.3.shtml
Also, I just updated the snapshot on joewing.net to have the head revision.

There is some issue in the autoconf that I am not understanding.
building it using autoreconf && ./configure && make
works fine here.
However I get errors trying to build with pbuilder.  I tried using the original setup, but it failed.  So I am wondering what could have changed.  The debian/ubuntu version is 2.1.0+svn579
I figure the changes form 2.1.0 (s579) to 2.2.2 were not major enough to break autotools being used in the Debian process, but something has diverged significantly since then, as I am no longer able to simply merge things and rebuild.
Thanks for taking the time to reply to me, I really appreciate all you do!!

I just brute-forced everything to work by tar-ing the directory and using it as upstream source.  I have been trying to contact the Debian maintainer for months to get a more recent version in.  Hopefully I can get in contact and get things updated, as the more recent version supports some very good features.  Thanks for humouring me.  Sometimes it just helps to get it out so I can find the solution.


application of a positional force seems wrong

The Body.applyForce method https://github.com/liabru/matter-js/blob/081645474c4aa798e5b1ede5d5230b0ecb8835d2/src/body/Body.js#L690-L695 appears to simply add the force to the bodies center of mass, while also applying a torque.
If the force is inline with the center of mass I can accept this behavior, but if it's off center it should translate the body less and rotate it more... right?
for example, the rudder (at the back of a boat or airplane) applies a lift force sideways, and mainly rotates the craft. It would impart some sideways force on the object, depending on the distribution of mass in the object.
If that rudder was on the side, it would push through the center of mass and only translate the craft, but in the matter-js model it still translates the craft by the same amount when it's on the tail as well as causing rotation. That's free energy! you are getting the same amount of linear kinetic energy, plus rotational energy too.
I've been trying to understand how forces are applied to a free rigid body. I'm interested in simulating objects like airplanes and boats that have multiple forces (lift and drag) operating on them in different places. I've been struggling to find something that explains how this should work and that lead me to read your code.

Okay, eventually I figured out I was wrong.
The most helpful were physics stack overflow questions about it
https://physics.stackexchange.com/questions/88887/2d-force-applied-at-angle-to-body-where-translational-vector-will-be-directed?rq=1
and then i found this:
https://www.youtube.com/watch?v=vWVZ6APXM4w&list=UUHnyfMqiRRG1u-2MsSQLbXA
The real answer is that actually it's quite a complicated situation, but presented unrealistically simply in a physics simulator or the equations. An arbitrary force can be applied on paper, but on the airplane, the force from the tail rudder starts rotating, and then moves, as the plane rotates, and that changes the force also. The extra energy can be expended as drag, as maybe the bullet doesn't enter as far into the block. So, if the spin was created via another object hitting it, instead of just creating free energy, it's could slow that object a bit more.

Good job


Implement Session.copy_object()

This PR implements C_CopyObject(). It takes a template as an optional argument (i.e. Option<&[Attribute]>).
The PR comes along with rustdoc and dedicated test case.

The CI script fails for the reason described in #216, can this be looked at? I guess it will block all PRs going forward.
Thank you!


[Update Request]: DindeGDPS Launcher

What type of update are you requesting?
A new version of an existing package
Current Package Identifier
DimisAIO.DindeGDPS
Package Version
2<IP_ADDRESS>
Please describe the changes you would like to see
I updated DindeGDPS to 2<IP_ADDRESS>
Download link still is https://cdn-dinde.141412.xyz/DindeGDPS.exe

Thanks!


Add "kudoctl delete" command

What type of PR is this?

/component kudoctl

What this PR does / why we need it:
This PR adds a "kudoctl delete" command that removes a given framework instance.
Which issue(s) this PR fixes:
Fixes #226
Does this PR introduce a user-facing change?:
This PR adds a "kudoctl delete" command that can be used to uninstall a framework instance.
``

Yes. Closing for now.


Multi-line function arguments

Anyone else bothered by the closing parentheses being on the same indentation level as the function and end keywords? Ideally I would prefer everything between these two keywords to be indented.
Currently this is recommended, note the ) at the same indent level as function and end, visually interrupting/splitting this block.
# currently recommended
function foobar(
    df::DataFrame,
    id::Symbol,
    variable::Symbol,
    value::AbstractString,
    prefix::AbstractString="",
)
    # code
end

On the other hand, I do not like the idea of just indenting  the ) on more level since then there is no visual separation between the function argument list and the code. I.e.
# NO
function foobar(
    df::DataFrame,
    id::Symbol,
    variable::Symbol,
    value::AbstractString,
    prefix::AbstractString="",
    )
    # code
end

I also much prefer the recommended style over multiple arguments per line and arguments in the first line. I.e.
# NO
function foobar(df::DataFrame, id::Symbol, variable::Symbol,
    value::AbstractString, prefix::AbstractString="",)
    # code
end

My suggestion would be:

indent the arguments twice. This should be fine since there is only one argument per line.
indent the closing parenthesis once
E.g.

# YES
function foobar(
        df::DataFrame,
        id::Symbol,
        variable::Symbol,
        value::AbstractString,
        prefix::AbstractString="",
    )
    # code
end

This would be breaking though. Thoughts?

I prefer the currently recommended version, since the trailing lines are an extension of the top line, which is at the left-most indentation level. I think this provides a good separation between the arguments and the first line of code without requiring a blank line (with the increased indentation level, I have seen people instinctively add another blank line before the code in order to provide visual separation).
Why do you prefer the increased indentation?

Yeah, it is a matter of preference. And FWIW I prefer the approach here over that in the other styles. It was not the increased indentation that I prefer, but the absence of anything else between the function and end at the first indentation level. Just the visual interruption of that block. But might be just me.


Wrong url for token service in sample code

In the sample call code for acquiring a MR token the address for the GET request should be:
https://sts.region.mixedreality.azure.com/Accounts/...
instead of:
https://sts.mixedreality.azure.com/Accounts/...
Took me a long time to figure that one out.

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

ID: 7d43bc57-798b-d3a4-5f31-a22f9ef99b2d
Version Independent ID: dff6c910-70a8-6441-c67a-c66ec5e01cbb
Content: Authentication and authorization - Azure Spatial Anchors
Content Source: articles/spatial-anchors/concepts/authentication.md
Service: azure-spatial-anchors
GitHub Login: @craigktreasure
Microsoft Alias: crtreasu

Hello @tomgeene, Thank you for your feedback! We have assigned this issue to the author to review further and take the next course of action.

Thanks for reporting the issue. We obviously missed this in our last big update. I'll get a fix out soon.

@ChaitanyaNaykodi-MSFT I've merged a fix for this. This issue can now be close. Thanks!

Thank you @craigktreasure.


Update worker-variables.md fix broken function example

The email variable is passed to the digestMessage() function. The  text variable would be undefined.
As requested here: https://discord.com/channels/595317990191398933/917505178016579605/1093902711231086632

Confirming this fix is correct!


Add docker to react storybook for quick setup

Looks alright to me. Did you test it on your machine?


Change Forcing.h basin_id to catchment_id with a default of 0 in the constructor

Change Forcing.h basin_id to catchment_id with a default of 0 in constructor with a todo to return to this
Testing

Passes all unit tests

Notes
Fixes #142
Checklist

[x] PR has an informative and human-readable title
[x] Changes are limited to a single goal (no scope creep)
[x] Code can be automatically merged (no conflicts)
[x] Code follows project standards (link if applicable)
[x] Passes all existing automated tests
[x] Any change in functionality is tested
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged / future todos are captured in comments
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
[ ] Reviewers requested with the Reviewers tool :arrow_right:

Testing checklist (automated report can be put here)

Target Environment support

[ ] Linux

Haha thanks!
On Wed, Aug 19, 2020 at 8:27 AM Chris Tubbs<EMAIL_ADDRESS>wrote:

@christophertubbs approved this pull request.
I don't know, David. That looks like too complicated of a change ;)
That looks great to me.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NOAA-OWP/ngen/pull/146#pullrequestreview-470455729,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ3MJYVKCK3ASJ34FAXR6JDSBPHNBANCNFSM4QEKV37Q
.

--
John "David" Mattern
Lynker https://www.lynker.com/
Analysis and Prediction Division | NOAA National Water Center, Tuscaloosa,
AL
<EMAIL_ADDRESS>|<PHONE_NUMBER>


Replace internal release script with existing tool

This is ready to go:
https://circleci.com/gh/apprenda/kismatic/367

I like the task names, LGTM!


remove unnecessary :needs_server tags with html in watirspecs

Having html files in watir-webdriver project requires the server to display based on the file implementation. I don't see a problem with putting them in the watirspec project so we can run these specs without the server. This PR is in conjunction with https://github.com/watir/watirspec/pull/73

See my comments in watir/watirspec#73.

To be clear, you are ok with moving the 4 html files into watirspec?

:+1:


NestedDiagnosticsLogicalContext.Pop should return string just like NDC

PopObject() already exists.
For NLog 5.0

Since ScopeContext has taken over, then no need to adjust obsolete methods.

Since ScopeContext has taken over, then no need to adjust obsolete methods.


Add information about John/Jane Doe as other defendant?

I searched pending cases in St. Louis County for John Doe and there were a number of eviction cases.  Do we want them to enter John Doe as a defendant, or give special instructions about what to do if John Doe is listed as a defendant?

User is advised to not add John or Jane Doe


Page Transitions

Please could we add Page Transitions?
https://www.youtube.com/watch?v=JCJUPJ_zDQ4
Early days but I would love to track the progress of this spec.

I believe it might be because the spec is still a draft, and hasn't been filed to W3C yet: https://tabatkins.github.io/specs/css-shared-element-transitions/
But regardless, I would love to track support for this feature too 👍

There is information on their github repo

It is now called the View Transition API.
Explainer: https://developer.chrome.com/docs/web-platform/view-transitions/

Its now shipped in chrome 🥳
https://developer.chrome.com/blog/new-in-chrome-111/

Now available at https://caniuse.com/view-transitions


on_messages racecondition

Summary
racecondition between message objects
Reproduction Steps
get two users or a webhook to post something inside the same time window, doesn't matter if its locally or global across other servers.
its necesarry thats the posts collide, old and new both at the same time. (should be no news for bigger bots):  https://i.imgur.com/GCjgaMu.png
you'll see the oldest message object gets overwritten by the youngest: https://i.imgur.com/ud3JaxW.png
the posts in question: https://i.imgur.com/CnUaxHo.png
Minimal Reproducible Code
No response
Expected Results
user #1 should recieve a reply with the post displaying '1' as number.
expected results: https://i.imgur.com/0vobRFq.png
Actual Results
user #1 gets the post from user #2 replied instead, which seems to be mixed up.
Intents
messages
System Information

Python v3.10.4-final
nextcord v2.0.0-alpha

nextcord pkg_resources: v2.0.0a10

aiohttp v3.8.1
system info: Windows 10 10.0.22000

Checklist

[X] I have searched the open issues for duplicates.
[X] I have shown the entire traceback, if possible.
[X] I have removed my token from display, if visible.

Additional Context
No response

You need to send a minimal amount of code so we can reproduce the bug

You need to send a minimal amount of code so we can reproduce the bug

not entirely sure how to handle a bug report like this. the issue appears on the stock on_message event as well, without third party code being included.
after trying to debug and reproduce the issue for more than a month i came to the conclusion my code has nothing to do with it.
all i can do is pointing my finger at the event and be like 'yeah the issue is there, i just don't know WHERE' or if its a bug, on purpose, or an issue on discord's end.

Reproducible code isn't entirely centred on pointing out issues with it, it's so others can confirm the issue and attempt a fix

i'll close this for now.
once the issue re-appears i'll open a new case and provide a proper code snippet.


Restore test projects prior to running SDK tests

https://github.com/aspnet/AspNetCore/issues/12494

🆙 📅


cpid builtin for child PID

As a follow on from the -c option #297, this should work:
bpftrace -e 'k:vfs_read /pid == cpid/ { printf("%s == ls\n", comm); }' -c ls
...
ls == ls
ls == ls
ls == ls

This will involve adding cpid as a builtin. It should return the child PID of the process launched in -c.

I took a look at this, but there's a problem with ordering: when we build the BPF program and process cpid, the child process hasn't launched yet from main.cpp->run().
If the code could be modified to do the fork() earlier, and the execve() later (somehow! we're now in a different process, so it might involve some IPC call), then the child PID would be available for when we need it.

Novice user of bpftrace here. I have a tentative patch that does that in https://github.com/kazuho/bpftrace/commit/2ca62f9d7376d568b83c6396a6a49bec3e35d07d, using  pipe (2), though it is my understanding that it becomes impossible to set USDT probes if we delay calling execve.
So to me it seems that the right solution is to use ptrace to trap the execution of the child process after execve is being called.
Does that sound correct to you? If it does, maybe I can find some time to work on a PR. I am interested in using USDT from the moment the process is being launched.


[BUG] Get-TwitterDM is not allowed

Describe the bug
Running Get-TwitterDM produces an exception:
Line |
1063 |          Invoke-TwitterCursorRequest -OAuthParameters $OAuthParameters …
     |          ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
     | This application is not allowed to access or delete your direct messages.get-t

To Reproduce
Install module v0.1.1
Configure with secrets and export resources
Run Get-TwitterDM
I get the same error if I try to use Count.
Expected behavior
I should get 30 most DMs per the help.
PowerShell Environment (please complete the following information):

OS: Microsoft Windows 10.0.19042
Edition Core
Version 7.1.0

Additional context
Exception             :
    Type       : Microsoft.PowerShell.Commands.HttpResponseException
    Response   : StatusCode: 403, ReasonPhrase: 'Forbidden', Version: 1.1, Content: System.Net.Http.HttpConnectionResponseContent, Headers:
                 {
                 Cache-Control: no-store, must-revalidate, no-cache, pre-check=0, post-check=0
                 Date: Tue, 24 Nov 2020 14:31:13 GMT
                 Pragma: no-cache
                 Server: tsa_b
                 Set-Cookie: personalization_id="v1_I75MF2AboHJIvuDKAS0UHg=="; Max-Age=63072000; Expires=Thu, 24 Nov 2022 14:31:13 GMT; Path=/; Domain=.twitter.com; Secure; SameSite=None
                 Set-Cookie: lang=en; Path=/
                 Set-Cookie: guest_id=v1%3A160622827308660937; Max-Age=63072000; Expires=Thu, 24 Nov 2022 14:31:13 GMT; Path=/; Domain=.twitter.com; Secure; SameSite=None
                 status: 403 Forbidden
                 Strict-Transport-Security: max-age=631138519
                 x-connection-hash: 3444663e8eaa0856782c4517920a5764
                 X-Content-Type-Options: nosniff
                 X-Frame-Options: SAMEORIGIN
                 x-rate-limit-limit: 15
                 x-rate-limit-remaining: 13
                 x-rate-limit-reset:<PHONE_NUMBER>
                 x-response-time: 17
                 x-transaction: 00fbefa400b9179c
                 x-twitter-response-tags: BouncerCompliant
                 X-XSS-Protection: 0
                 Content-Disposition: attachment; filename=json.json
                 Content-Length: 110
                 Content-Type: application/json; charset=utf-8
                 Expires: Tue, 31 Mar 1981 05:00:00 GMT
                 Last-Modified: Tue, 24 Nov 2020 14:31:13 GMT
                 }
    TargetSite :
        Name          : ThrowTerminatingError
        DeclaringType : System.Management.Automation.MshCommandRuntime, System.Management.Automation, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35
        MemberType    : Method
        Module        : System.Management.Automation.dll
    StackTrace :
   at System.Management.Automation.MshCommandRuntime.ThrowTerminatingError(ErrorRecord errorRecord)
    Message    : This application is not allowed to access or delete your direct messages.
    Source     : System.Management.Automation
    HResult    : -2146233088
TargetObject          : https://api.twitter.com/1.1/direct_messages/events/list.json
CategoryInfo          : PermissionDenied: (https://api.twitterΓÇªes/events/list.json:String) [Invoke-TwitterCursorRequest], HttpResponseException
FullyQualifiedErrorId : Get-TwitterDM,Invoke-TwitterCursorRequest
InvocationInfo        :
    MyCommand        : Invoke-TwitterCursorRequest
    ScriptLineNumber : 1063
    OffsetInLine     : 9
    HistoryId        : 257
    ScriptName       : C:\Users\Jeff\Documents\PowerShell\Modules\bluebirdps\0.1.1\BluebirdPS.psm1
    Line             : Invoke-TwitterCursorRequest -OAuthParameters $OAuthParameters -ReturnValue events

    PositionMessage  : At C:\Users\Jeff\Documents\PowerShell\Modules\bluebirdps\0.1.1\BluebirdPS.psm1:1063 char:9
                       +         Invoke-TwitterCursorRequest -OAuthParameters $OAuthParameters ΓÇª
                       +         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    PSScriptRoot     : C:\Users\Jeff\Documents\PowerShell\Modules\bluebirdps\0.1.1
    PSCommandPath    : C:\Users\Jeff\Documents\PowerShell\Modules\bluebirdps\0.1.1\BluebirdPS.psm1
    InvocationName   : Invoke-TwitterCursorRequest
    CommandOrigin    : Internal
ScriptStackTrace      : at Invoke-TwitterCursorRequest, C:\Users\Jeff\Documents\PowerShell\Modules\bluebirdps\0.1.1\BluebirdPS.psm1: line 418
                        at Get-TwitterDM, C:\Users\Jeff\Documents\PowerShell\Modules\bluebirdps\0.1.1\BluebirdPS.psm1: line 1063
                        at <ScriptBlock>, <No file>: line 1

Could you verify that your app has the correct permissions (obviously a gap in documentation)?
You can see this in Details > Permissions on the Apps dashboard.
You should see something like the following:

A quick way to see what permissions you've granted to your application is to do the following:
Get-TwitterUser -ScreenName thedavecarroll | Out-Null
$AccessLevel = Get-TwitterHistory -Last 1
$AccessLevel.Response.'x-access-level'

read-write-directmessages

I'll create a feature request for a Get-TwitterAccessPermissions command.
Thanks again.

I didn't have DM permissions set. Do you know if I have to regenerate anything for the new permissions to take effect?

Yes I did.

I have added a command Get-TwitterPermissions which reveals what access has been configured for the access key and secret.
This will be in the upcoming release.


Firefox-specific scrolling issue

Opening any tutorial in sandbox.bio appears to dramatically increase the page width to the right, to the point where I can scroll all the actual content off the screen. The extra width persists when I leave the tutorial and return to the main page. I've tested the site in Google Chrome and Firefox, and the issue only crops up in Firefox right after the in-browser terminal loads.

Oh wow, that's pretty strange behavior, thanks for the issue!

No problem! I don’t have a lot of web design experience, but let me know if there’s any other testing I can do or information I can send over to make things easier for you.On Feb 1, 2024, at 7:20 PM, Robert Aboukhalil @.***> wrote:
Oh wow, that's pretty strange behavior, thanks for the issue!
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>


Start removing io dependencies

Ex:

req.session
req.headers
req.connectionInformation

This will need some sort of general abstraction before adapting Angel for package:hypertext.

This can (finally) be closed! package:mock_request allows us to manufacture dart:io requests, so we could even embed into shelf over hypertext.


api bad design:x.Consumer(()=>get_my_consumer());

Consumer(Func factory) where T:IConsumer;
I wasted two hours on this api this afternoon.
how this api works(I guess)
the factory will be called when message received.
so at the beginning when we config this consumer.
api don't care about the actual factory return.
it cares about the generic type T.
this api analysis the type T to build rabbitmq exchange queue and binding stuff.
question
but what if my factory return an interface?
this api can't analysis the generic type as a interface.
this consumer just simply ignore the operation.
no exception thrown, no message feedback?
suggest

change this api to Consumer(Func factory) where T:class,IConsumer;
throw exceptions when T is not match your requirement

end
thanks for your contribution

Yeah, I'd suggest reading the documentation.

I had read the doc,
maybe this api works well in demo, but it can be better(in other different sicuations).

There are several overloads for .Consumer<T>, including support for most major containers, and one that just accepts an object, and a Func factory method. Every use case is covered, unless you can be more specific about what you find lacking (beyond understanding).

check if this works well
public class MyConumer:IComsumer<Message>{}
Func<IConsumer> factory=()=>new MyConsumer();
x.Consumer<IConsumer>(()=>factory());

I tried again
using System;
using MassTransit.RabbitMqTransport;
using MassTransit;
using System.Threading.Tasks;

namespace BlueSky.Framework.MessageBus
{
    public class BadConsumerRegTestMessage { }

    public class BadConsumerRegTest : IConsumer<BadConsumerRegTestMessage>
    {
        public async Task Consume(ConsumeContext<BadConsumerRegTestMessage> context)
        {
            Console.WriteLine("bad message");
        }
    }

    public static class BadConsumerRegTestExtension
    {
        public static void TestBadConsumerReg(this IRabbitMqBusFactoryConfigurator config, IRabbitMqHost host)
        {
            config.ReceiveEndpoint(host, "bad_consumer_reg_test", endpoint =>
            {
                Func<IConsumer> factory = () => new BadConsumerRegTest();

                endpoint.Consumer(() => factory.Invoke());
            });
        }

        public static async Task SendBadTestMessage(this IBusControl bus)
        {
            await bus.Publish(new BadConsumerRegTestMessage() { });
        }
    }
}

Again, reading the documentation does wonders.
    public static void TestBadConsumerReg(this IRabbitMqBusFactoryConfigurator config, IRabbitMqHost host)
    {
        config.ReceiveEndpoint(host, "bad_consumer_reg_test", endpoint =>
        {
            Func<BadConsumerRegTest> factory = () => new BadConsumerRegTest();

            endpoint.Consumer<BadConsumerRegTest>(factory);
        });
    }

The documentation specifically states that you must register the concrete consumer type, not IConsumer.

@phatboyg
yes i know how to make this api work.
i never say its a bug,i said its bad design.

you should let users know they are making mistake when they try to build(or run).


Wrong link definition in page for managing Groups and System groups

Hello,
I found a bug in Version <IP_ADDRESS>5, in page for managing groups (same in page for managing system groups), the link on the "Edit" icon button is not good regarding the link on the name for the same group (see screenshot below)
On name link is "http://nas:5301/settings/groups/46f46366-cded-49fd-b57e-eb5da69a55a0"
On edit icon button link is "http://nas:5301/settings/dashboards/46f46366-cded-49fd-b57e-eb5da69a55a0"

This may be related to #178.  When you click the edit button on the right with the bad link it causes the page to not respond which you then have to refresh the browser window to get it working again.

fixed

This issue seems to have popped back up.  I was experiencing  this in the 23.08 (docker) version.  The page would freeze in browser (MS Edge)  after trying to delete a group (System or user)  ,No link would work until a refresh was preformed.  This does not seem to be an issue if running Chrome.
The console error I was receiving was:
There was an unhandled exception on the current circuit, so this circuit will be terminated. For more details turn on detailed exceptions by setting 'DetailedErrors: true' in 'appSettings.Development.json' or set 'CircuitOptions.DetailedErrors'.
After fiddling around with it a little while I decided to update to the  'revenz/fenrus:develop' (24.04) docker image, and it also was having issues with MS EDGE.          Chrome is working fine.


Fix two issues in cherry_pick_pull.sh

Ignore errexit in trap handler
Check for branch (since two people have complained now..)

Over to @thockin for complaining. :)

One nit then LGTM

Fixed.

Tool, merging.

LGTM


attachment size

Hi,
First of all you made a great library.
In the attachment class I would like a method that returns the size of an attachment.
Let me know what you think.

Hello,
Yes it's possible. Do you have already the function ?

Not yet. I hope I have some time tomorrow

I didn't test it but I think this does the trick
    /**
     * Calculate the size of the attachment
     * @return int
     */
    public function getSize()
    {
        return mb_strlen($this->getContent());
    }

@thierryve i don't think this lib has dependency of mb_.

I didn't had a function because you can used filesize from PHP.
I updated Readme with an example.


using $file causes syntax highlighting issues

The third line in this stanza does not get highlighted correctly:
resource "aws_key_pair" "deployer" {
key_name = "deployer-key"
public_key = "$file("ssh_keys/production_${var.aws_region}.pem.pub")"
}
It seems to not process the quotes properly when nested inside other parens or curly braces.

According to the documentation you should use curly braces:

Built-in Functions
Terraform ships with built-in functions. Functions are called with the syntax name(arg, arg2, ...). For example, to read a file: ${file("path.txt")}. The built-in functions are documented below.

This will get highlighted properly:
resource "aws_key_pair" "deployer" {
key_name = "deployer-key"
public_key = "${file("ssh_keys/production_${var.aws_region}.pem.pub")}"
}

Thank you


Do more prep for the 2.0.0 release

Also adds support for configurable config filenames (#14).

Coverage remained the same at 100.0% when pulling a1e654bdfb3a7a8321ef31616e4885e8fd68a72d on jdh-2.0-prep into bc08384445e47e52c4bf381d89ff2dcc8d37aedd on master.

Coverage remained the same at 100.0% when pulling 9d4893ef7aa094871b1e39dc8a5920d253569504 on jdh-2.0-prep into bc08384445e47e52c4bf381d89ff2dcc8d37aedd on master.

Coverage remained the same at 100.0% when pulling d8a1a7916868265200df6bea050b73a7a63a9a57 on jdh-2.0-prep into bc08384445e47e52c4bf381d89ff2dcc8d37aedd on master.

Coverage remained the same at 100.0% when pulling 90d217341e2678baf482331692143b95de387aed on jdh-2.0-prep into bc08384445e47e52c4bf381d89ff2dcc8d37aedd on master.

Coverage remained the same at 100.0% when pulling 947cbe23cc2501605a5951ecc0a9daf695e5b884 on jdh-2.0-prep into bc08384445e47e52c4bf381d89ff2dcc8d37aedd on master.

Coverage remained the same at 100.0% when pulling 947cbe23cc2501605a5951ecc0a9daf695e5b884 on jdh-2.0-prep into bc08384445e47e52c4bf381d89ff2dcc8d37aedd on master.

Coverage remained the same at 100.0% when pulling 6747cf22c8519cbb246db3d8068ef586edbca606 on jdh-2.0-prep into bc08384445e47e52c4bf381d89ff2dcc8d37aedd on master.


Improved DashedVMobject

Overview: What does this pull request change?

DashedVMobject used to create stretched and uneven dashes on most plotted curves. Now the dash lengths are equalized. An option is reserved to use the old method.
New feature added to DashedVMobject: dash_offset. This parameter shifts the starting point. It can be used with ValueTracker and what not to create cool looping animations.

Motivation and Explanation: Why and how do your changes improve the library?

Even length dashes are just better...?
The dash_offset is inspired by the 'Dashes' effect I knew from Adobe After Effects. It opens a few interesting possibilities, like a single line quickly running along a path, like a roller-coaster.
Links to added or changed documentation pages

Further Information and Comments

I did a small refactoring on the way curve length is calculated.
I also updated the Example, though it may need more work.
I haven't added tests yet.

Reviewer Checklist

[ ] The PR title is descriptive enough for the changelog, and the PR is labeled correctly
[ ] If applicable: newly added non-private functions and classes have a docstring including a short summary and a PARAMETERS section
[ ] If applicable: newly added functions and classes are tested

I'm struggling with this git process, but I think for the last commit it's finally good, with Ben's review comments fixed.

So, is this going to get merged? What else do I need to do?

We usually leave approved PRs open for a few days to give other developers the opportunity to have a look as well -- but I agree, this has been open long enough, I'm merging it. Thanks again for your contribution!


Can't fetch the source map while using the experimental debugger

Description
I'm trying to use the new experimental debugger in our RN project, but in the DevTool window it says this:

Failed to fetch source map http://<IP_ADDRESS>:8081/index.map?platform=ios&dev=true&lazy=true&minify=false&inlineSourceMap=false&modulesOnly=false&runModule=true&app=com.teslamotors.enterpriseapp: remote fetches not permitted

and can't fetch the source map, thus no source code is shown.
Here're some interesting I've found:

I can access this URL in Safari and in CLI using wget and curl, but I got this when I open it in Chrome:

I noticed this using Chrome's dev tools:

https://github.com/user-attachments/assets/54f9a1d7-cc13-4b0f-a2ce-30dd38225ce6
The "content download" always gets killed after around 3.5s, and then it becomes "Aw Snap!"

I tried to use the experimental debugger on the Expo tutorial's starter app, which works fine (I think it's using the experimental debugger, instead of the old one?)
Our source map has ~200MB size

My assumption is that because our source map is so big, Chrome isn't able to download it within a time limit (so the download is always canceled after 3.5s) but I have no idea of how to verify that. Not sure if this is a Chrome bug or RN bug but I'll post it here anyway.
Steps to reproduce
Just launch the experimental debugger on some huge app with ~200MB source map, I guess?
React Native Version
0.73.6
Affected Platforms
Build - MacOS
Output of npx react-native info
npx react-native info

  WARNING: You should run npx react-native@latest to ensure you're always using the most current version of the CLI. NPX has cached version (0.73.6) != current release (0.75.2)

info Fetching system and libraries information...
System:
  OS: macOS 14.5
  CPU: (12) arm64 Apple M3 Pro
  Memory: 807.53 MB / 36.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 18.12.0
    path: /usr/local/bin/node
  Yarn:
    version: 1.22.22
    path: /opt/homebrew/bin/yarn
  npm:
    version: 8.19.2
    path: ~/.nvm/versions/node/v18.12.0/bin/npm
  Watchman:
    version: 2<IP_ADDRESS>
    path: /usr/local/bin/watchman
Managers:
  CocoaPods:
    version: 1.15.2
    path: /Users/kainzhong/.rbenv/shims/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 23.2
      - iOS 17.2
      - macOS 14.2
      - tvOS 17.2
      - watchOS 10.2
  Android SDK: Not Found
IDEs:
  Android Studio: 2024.1 AI-241.15989.150.2411.11948838
  Xcode:
    version: 15.1/15C65
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 17.0.11
    path: /usr/bin/javac
  Ruby:
    version: 2.7.5
    path: /Users/kainzhong/.rbenv/shims/ruby
npmPackages:
  "@react-native-community/cli": Not Found
  react:
    installed: 18.2.0
    wanted: 18.2.0
  react-native:
    installed: 0.73.6
    wanted: 0.73.6
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: false
iOS:
  hermesEnabled: true
  newArchEnabled: false

info React Native v0.75.2 is now available (your project is running on v0.73.6).
info Changelog: https://github.com/facebook/react-native/releases/tag/v0.75.2
info Diff: https://react-native-community.github.io/upgrade-helper/?from=0.75.2
info For more info, check out "https://reactnative.dev/docs/upgrading?os=macos".

Stacktrace or Logs
> Failed to fetch source map http://<IP_ADDRESS>:8081/index.map?platform=ios&dev=true&lazy=true&minify=false&inlineSourceMap=false&modulesOnly=false&runModule=true&app=com.teslamotors.enterpriseapp: remote fetches not permitted

Reproducer
I really don't know how to reproduce this on smaller projects
Screenshots and Videos
Included some in the description.

I'm trying to use the new experimental debugger in our RN project, but in the DevTool window it says this:

Please update to 0.76.x and try React Native DevTools over there. We don't offer support for the experimental debugger for previous versions of React Native.

I'm trying to use the new experimental debugger in our RN project, but in the DevTool window it says this:

Please update to 0.76.x and try React Native DevTools over there. We don't offer support for the experimental debugger for previous versions of React Native.

OK let me try that (it might take some time because it's a huge project but I'll try to get back to you ASAP)


issue connecting to external auth cluster for the first time

We have configured external_auth_cluster as
 - name: external_auth_cluster
    connect_timeout: 0.25s
    type: EDS
    eds_cluster_config:
      eds_config: {ads: {}}
      service_name: extauthservice
    lb_policy: LEAST_REQUEST
    http2_protocol_options: {}

Please note that extauthservice is deployed locally for testing on the same machine as envoy is running.
With this configuration, it consistently fails for the first request with a "gateway-time-out" error.  I increased the connect_timeout to 1s (not sure why it is required for this cluster though, we have other service clusters deployed on the same host which works fine with 0.25 s timeout), it works for few requests.
To make it consistently work for the first request, we also had to set the --drain-time-s less than the connect time out.  Also this is only for the initial request - subsequent requests work fine even with out these settings.
This seems to be a bug to me in the ext_auth filter connection management with EDS type of clusters.
Any suggestion?

@saumoh @htuch any ideas on this would be very useful.

Could you please share the entire configuration file? Does Authorization work fine with a static cluster without EDS?
I'm not sure if it's possible right now to configure Envoy to use the embedded gRPC client to send Authorization requests and EndpointDiscoveryService requests at the same time.

Static Cluster with out EDS works fine IIRC. Also please note with EDS also this comes only for first few requests. Are you suspecting that for first few requests it is not able to load EDS? The error we are getting is "gateway timeout" - if we increase connect timeout it works some time.
Here is the full config
node:
  id: id_1
  cluster: test
admin:
  access_log_path: /tmp/admin_access.log
  address:
    socket_address: { address: <IP_ADDRESS>, port_value: 9901 }
dynamic_resources:
  ads_config:
    api_type: GRPC
    grpc_services: {envoy_grpc: {cluster_name: xds_cluster}}
  lds_config: {ads: {}}
  cds_config: {ads: {}}
static_resources:
  clusters:
  - name: xds_cluster
    connect_timeout: 0.25s
    type: STRICT_DNS
    lb_policy: ROUND_ROBIN
    http2_protocol_options: {}
    hosts:
      socket_address:
        address: <IP_ADDRESS>
        port_value: 15001
  - name: zipkin
    connect_timeout: 0.25s
    type: STRICT_DNS
    lb_policy: ROUND_ROBIN
    hosts:
      socket_address:
        address: <IP_ADDRESS>
        port_value: 9411
  - name: external_auth_cluster
    connect_timeout: 1s
    type: EDS
    eds_cluster_config:
      eds_config: {ads: {}}
      service_name: extauth
    lb_policy: LEAST_REQUEST
    http2_protocol_options: {}

@saumoh @htuch @mattklein123 I think the problem is because of this default value https://github.com/envoyproxy/envoy/blob/master/source/extensions/filters/http/ext_authz/config.cc#L25. In our integration test environments, for first few requests it is failing. We have configured it with a much higher value and it seems to fix the issue.
Is this the right default that we use for all other clusters? or should we increase it here?

@ramaraochavali 200ms seems fine as a default. But if it's enough or not kind of depends on what all the gRPC server side has to do when it get's a request. If it has to invoke other services and/or do a bunch of initialization then it may not be large enough to get things going.
Does the gRPC server get invoked at all?

@saumoh No. It is not going the gRPC server at all. It returns gateway timeout.

Closing due to lack of activity. @ramaraochavali if this is still an issue or if you think there is a code/doc change required please open a PR.


[Question] Naming ELB

Hello, is it possible to set LB name via annotations, or somehow...?
I tried with adding Name tag, but no success, my ELBs are always created with name similar to this
5048fede-default-vault-c063
Or at least... how to know in advance, with what name will be created? Because I'm using terraform for deploying helm charts and AWS resources, and I need to reference other resources to this LB

/kind feature
Hi, it's currently not possible to set the name.
The name is generated here: https://github.com/kubernetes-sigs/aws-alb-ingress-controller/blob/master/internal/alb/generator/name.go#L23

Thx for the response.
It would be nice to have that feature.
Usually AWS resourcess are named with the Tag "Name".
Maybe this naming feature can be overridden if Name tag exist in annotations.
Then in terraform with data resource we can reference all other resources, like CloudFront is.

Thx for the response.
It would be nice to have that feature.
Usually AWS resourcess are named with the Tag "Name".
Maybe this naming function can be overridden if Name tag exist in annotations.

It seemed not work for ALB. The Name did not get updated, after I added the Name tag for it.

Feature request https://github.com/kubernetes-sigs/aws-load-balancer-controller/issues/1483


How can I save array of entities to repository?

Hi, I'm able to use createAndSave()to save one entity at a time, but I have an array of objects which has approximately 5000 elements. How can I store such data in the repository so that I could perform query operations later?
module.exports.redisTest = async (req, res, next) => {
    const repo = await personRepository;

    repo.createIndex();

    let sampleObj = [
        {
            name: "John",
            id: "1",
        },
        {
            raw_text: "Mary",
            id: "2",
         }

    ];

    const result= await repo.save(JSON.stringify(sampleObj )); // TypeError: entity.toRedisJson is not a function
    const result = await repo.createAndSave(JSON.stringify(sampleObj )); // all filed are null

    res.status(200).json({
        status: "success",
        result
    });
};

I got it working by using Promise.all()
 await Promise.all(
                    JSON.parse(normalizedData).map(async (data) => {
                        await client.createAndSave(data);
                    })
                );

I'm not sure if this is the best way or the correct way to do this, so if there are any better ways please let me know. Thank you!


Feature Request: Side-by-Side for services

Hello, I don't have a real problem, but a feature request and hope that I'm in the right place.
I would like to be able to open two (or more) windows side by side within the app, so to speak side-by-side.
Would something like that be conceivable and, above all, also viable?
I am looking forward to a reply.
Best regards
Man-in-Black

+1 for this


Search Binding

We'll need to complete the search binding. However, in PISA, query processing is done via one of two binaries -- queries or evaluate_queries -- perhaps we need to consider creating a PISA-side "search" interface to simplify the binding here? Otherwise, there might be some difficulty with the templating of indexes (although I haven't tried yet). Any thoughts or comments?
@amallia
@elshize

We're not going to link to either of this binaries. These are not really part of the PISA library in that sense. I think we just need to add a function (with options) or two functions in binder.cpp that expose template-independent API. Ultimately, all template instantiations will be done when compiling the binder.cpp compilation unit.
We might need to either duplicate some code or extract some code into a header.
On a side note, any moder linker shouldn't have problems linking two instantiations of the same function, I believe. The real problem would be if two symbols aren't pointing at the same exact code.

Yeah that's exactly what I meant. We don't want to link to either binary, but we will need a "search" (header?) in PISA which we can then directly expose here. We may have to think carefully about what to expose, but it shouldn't be too difficult. I guess this is another issue better suited for the PISA repo itself?

I think https://github.com/pisa-engine/pisa/pull/439 is what would be nice to have here. I haven't had time to look at it again. When I find some time, I'll first try to figure out https://github.com/pisa-engine/pisa/pull/461, then rebase and try again the former.


A few search box bugs

After testing #2136

when there are no suggestions returned, but there are products being found, the input value should appear as a suggestion
you're supposed to be able to click on the search suggestion but it wasn't working
in wcms > component > search for 'search', choose 'searchbox' for spa-electronics online,
set to false any of these: display product images, display products, display suggestions
no effect
changing the max/min fields works

QA:

when there are no suggestions returned from the backend, but there are products being found, the input value should appear as a suggestion. verify that you can click on such a suggestion and land at the PLP
verify that changing display product images, display products, display suggestions will have effect in the UI

input value appears as suggestion 🆗
input value redirects to product listing page. 🆗
wcms modifications to search component work as expected. 🆗


Consider compressing all payloads en route to the collector

Currently NewRelic::Agent::NewRelicService#compress_request_if_needed will only perform compression on a payload if it is found to be greater than 64KiB (64 * 1024 bytes) in size. The source code comment for that method states that compressing smaller payloads constitutes aggressive compression behavior that will run afoul of related bugs present in some versions of Ruby.
We do not currently know:

If the choice of 64KiB was arbitrary or based on some code or testing
What the related Ruby bugs are
Which Ruby versions are thought to have the bugs (NOTE: the comment was written in 2012)
If there are bugs present for both our raw DEFLATE stream and GZIP stream compression approaches (configurable per agent instance)

We should research these things and conduct testing with an optimal goal of having the Ruby agent compress data in all cases without condition. If that goal can not be achieved, and any part of the 10 year old source comment still holds true for currently supported Ruby versions, we should update the comment and any relevant internal knowledge bases.

The .NET agent has a 20 byte minimum before compression is applied, owing to that being the smallest size a payload can reach and not have the compressed size come out larger.
I did some testing with the Ruby agent's compression routine (zlib gzip).
Here are the results for the smallest byte sizes I could get to for each type of string that would result in a smaller size after compression.

24 byte string comprised entirely of the same single byte character => 23 bytes compressed
201 byte string made up of random characters (1-3 bytes each) (with repeats) => 200 bytes compressed
201 byte string made up of 67 triple byte characters (with repeats) => 200 bytes compressed
246 byte string made up of 123 double byte characters (with repeats) => 245 bytes compressed
253 byte string made up of single byte characters (with repeats) => 252 bytes compressed

This findings have me leaning towards imposing at least a 253 byte payload size minimum before applying compression. Given that compressing at that size only yields a 0.4% size decrease, perhaps we should go even higher for the minimum and factor in a minimum compression percentage.

There is a school of thought that it is wasteful to compress content that is to be transmitted over a network if that content is much smaller than 1500 bytes, given that the internet MTU is 1500 bytes. This article recommends setting a threshold of 1400 bytes, to allow for 100 bytes worth of padding. I've seen 1000 bytes thrown around a lot in discussions as well, without any cited reasons.
In my testing...

I have not observed any issues with zlib/gzip compressing of short strings with Ruby 2.2
Using random ascii bytes to build a string (which will not be comprised entirely of unique characters), I determined a minimum value of 535 bytes to obtain at least a 10% shrinkage result with compression.


Announcement line-wrapping refinement needed.

Season announcements are processed in order to keep paragraphs ("\n\n") and replace standard newlines ("\n") with a single space to enforce line-wrapping in the announcement embed.
Currently this isn't a perfect solution due to us needing to occasionally force newlines once such as in lists.
The current situation forces us to add an extra newline between each list item to enforce separation which isn't ideal.
Hopefully this can be refined before Pride seasonal event to allow for line-wrapping without the list issues.

Automated season announcements are going away so we don't need to do this.


The result of Array slice calculation does not match that of direct calculation with date type bfloat16

Description
w = np.random.randn(2048, 2048).astype(np.float32)
x = np.random.randn(2048, 2048).astype(np.float32)

res1 = jnp.asarray(x, dtype=jnp.bfloat16) @ jnp.asarray(w, dtype=jnp.bfloat16)
res2 = jnp.asarray(x[0:1,:], dtype=jnp.bfloat16) @ jnp.asarray(w, dtype=jnp.bfloat16)
error = jnp.max(jnp.abs(res1[0:1,:] - res2))
print(f"Max error: {error} ")

#The error is 0.25 and will increase with the increase of matrix size.

Maybe it's due to bfloat32 precision? But they have the same date type and the same operation @ , I wonder why the results are inconsistent. Looking forward to your answer, thanks.
What jax/jaxlib version are you using?
jax 0.4.13, jaxlib 0.4.13+cuda12.cudnn89
Which accelerator(s) are you using?
No response
Additional system info
No response
NVIDIA GPU info
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 525.105.17   Driver Version: 525.105.17   CUDA Version: 12.2     |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|                               |                      |               MIG M. |
|===============================+======================+======================|
|   0  NVIDIA A100 80G...  Off  | 00000000:9D:00.0 Off |                   On |
| N/A   38C    P0    76W / 300W |                  N/A |     N/A      Default |
|                               |                      |              Enabled |
+-------------------------------+----------------------+----------------------+

+-----------------------------------------------------------------------------+
| MIG devices:                                                                |
+------------------+----------------------+-----------+-----------------------+
| GPU  GI  CI  MIG |         Memory-Usage |        Vol|         Shared        |
|      ID  ID  Dev |           BAR1-Usage | SM     Unc| CE  ENC  DEC  OFA  JPG|
|                  |                      |        ECC|                       |
|==================+======================+===========+=======================|
|  0    3   0   0  |     13MiB / 19968MiB | 28      0 |  2   0    1    0    0 |
|                  |      0MiB / 32767MiB |           |                       |
+------------------+----------------------+-----------+-----------------------+

+-----------------------------------------------------------------------------+
| Processes:                                                                  |
|  GPU   GI   CI        PID   Type   Process name                  GPU Memory |
|        ID   ID                                                   Usage      |
|=============================================================================|
|  No running processes found                                                 |
+-----------------------------------------------------------------------------+

Hi - this is working as expected.
Any floating point computation will accumulate rounding error, and in the case of bfloat16 there are only eight mantissa bits, meaning that rounding error will generally be about 1 part in $2^8$, or roughly 0.004 relative error.
You might expect in this particular case that the results would be identical, because the first operation is a subset of the second operation. This would be the case if the backend were executing the floating point operations in the same order in both cases, but you generally can't depend on this being the case when performing operations between matrices of different shape.

Thanks for your reply. However, in the example above,  I modify the code to output the result as shown below:
np.random.seed(13)
w = np.random.randn(2048, 2048).astype(np.float32)
x = np.random.randn(2048, 2048).astype(np.float32)

res1 = jnp.asarray(x, dtype=jnp.bfloat16) @ jnp.asarray(w, dtype=jnp.bfloat16)
res2 = jnp.asarray(x[0:1,:], dtype=jnp.bfloat16) @ jnp.asarray(w, dtype=jnp.bfloat16)
errors = jnp.max(jnp.abs(res1[0:1,:] - res2))
index = jnp.argmax(jnp.abs(res1[0:1,:] - res2))
res1_fp32 = jnp.asarray(x, dtype=jnp.float32) @ jnp.asarray(w, dtype=jnp.float32)
res2_fp32 = jnp.asarray(x[0:1,:], dtype=jnp.float32) @ jnp.asarray(w, dtype=jnp.float32)

print(f"res1: {res1[0][index]}, res1_fp32: {res1_fp32[0][index]}, relative error: {(res1_fp32[0][index] - res1[0][index])/res1_fp32[0][index]},\n\
        res2: {res2[0][index]}, res2_fp32: {res2_fp32[0][index]}, relative error: {(res2_fp32[0][index] - res2[0][index])/res2_fp32[0][index]}")
print(f"Max error: {errors} ")

#output is:
res1: -34.5, res1_fp32: -34.54495620727539, relative error: 0.0013013827847316861,
res2: -34.75, res2_fp32: -34.55058670043945, relative error: -0.005771632771939039
Max error: 0.25

I think the absolute error and relative error are too large，it's greater than 0.004. Is it still as expected? Looking forward to your answer, thanks.

That looks right: the relative error is on order 0.004 (0.001 and 0.005 are not inconsistent with the expected approximate relative error of 0.004) and given the size of the entries, this translates to an absolute error of about 0.2 to 0.3.

Thanks very much. By the way, why  JAX got different result from Torch's on GPU?
res1 = jnp.asarray(x, dtype=jnp.float32) @ jnp.asarray(w, dtype=jnp.float32)
res2 = torch.tensor(x).to(torch.float32).cuda() @ torch.tensor(w).to(torch.float32).cuda()
res2 = res2.cpu().float().numpy()
err = np.max(np.abs(np.asarray(res1,dtype=np.float32) - res2))

# err: 0.125

I suspect this indicates that JAX dot products and pytorch dot projects are lowering to different kernels. It looks like the error here is consistent for what we'd expect from floating point roundoff error in bfloat16 precision.

OK, thanks. So is it an inherent feature of JAX that causes the deviation compared with PyTorch? Or what can we do to eliminate the deviation?

Neither JAX nor PyTorch is incorrect here. Both are as close as can be expected to the true, real-valued answer given the inherent inaccuracies of bfloat16 arithmetic. The only way to eliminate the deviation between two bfloat16 computations is to ensure that the operations are computed identically at a low level - and generally when using high level libraries like PyTorch or JAX, you don’t have a lot of control over those low-level details.

Thanks a lot.


It's not WORKING.

Traceback (most recent call last):
File "D:\softwares\Sublime Text 3\sublime_plugin.py", line 818, in run_
return self.run(edit)
File "Timenow in C:\Users\SSZ\AppData\Roaming\Sublime Text 3\Installed Packages\Timenow.sublime-package", line 27, in run
TypeError: must be str, not None

I'm working to fix it. Thanks for your feedback, @casey1991.

Duplicated: #3


query.lua:219: query: invalid node type at position 2765 for language vim

Describe the bug
../nvim/0.8.1/share/nvim/runtime/lua/vim/treesitter/query.lua:219: query: invalid node type at position 2765 for language vim
Which version of Neovim are you using?
neovim v0.8.1
To Reproduce
Steps to reproduce the behavior:

install
press :
after typing error shown

Expected Behavior
no error
Other details
I am on WSL, this doesn't happen though on macOS

That's treesitter. Try running TSUpdate and if that doesn't help check further in the nvim-treesitter docs. Not caused by Noice

for future reference https://github.com/nvim-treesitter/nvim-treesitter/issues/3934
in short, remove your ../lib/nvim/parser


add async

Thank you very much for developing the new features for this library. I really appreciate it. Man, you're awesome!

kimi and open ai is awesome.for example  I  dont know why we need this line
await worker.work_queue.join()


Analyze which table have name but no geometry

Create table which shows which table have the geometry and name in same table
Example: Boundaries <=> Country Label
(Geometry is in admin layer and name is in country_label layer => join between tables is needed)

List of all layers / tables which has "name", "name_en", etc. and whether we have bounding box: a) in the same table b) in a different table - and which one c) not imported by imposm3 mapping d) completely missing in OSM

List of all layers / tables which has "name", "name_en", etc. and whether we have bounding box: a) in the same table b) in a different table - and which one c) not imported by imposm3 mapping d) completely missing in OSM

a) layers / tables with bounding box (geometry) in the same table:

road / osm_roads (no polygon only linestring)
road_label / osm_roads (no polygon only linestring)
water / osm_water_polygons
water_label / osm_water_polygons
waterway / osm_water_lines (no polygon only linestring)
waterway_label / osm_water_lines (no polygon only linestring)
marine_label / custom_seas (no polygon only linestring and point)

b) layers / tables with bounding box (geometry) in a different table:

poi_label / osm_poi_points => aeroway, landuse, landuse_overlay, building

c) layers / tables with bounding box (geometry) not imported by imposm3 mapping:

place_label => admin*
state_label / custom_states => admin*
country_label / custom_countries=> admin*

d) layers / tables with bounding box (geometry) completely missing in OSM:
(place_label, state_label, country_label)
*Administrative boundaries are currently stored as linestrings to convert them to polygons a new mapping would be required.
Boundaries are represented as linestrings in OSM that are linked together with a relation, therefore imposm3 would have to convert these relations to polygons.
We decided against this last year because not all linestrings could be converted to polygons, this resulted in missing boundaries for some countries and admin areas.
The open question c) and d) is still how to map the label to the polygon. See Stefan's post about relations => https://github.com/geometalab/OSMNames/issues/1
Conclusion
Because simply joining the tables is not possible in all cases, a new imposm mapping needs to be done for the layers place_label, country_label, poi_label.


Explore merging with other iTerm2 Night Owl themes

Twitter revealed one other iTerm2 Night Owl theme:

https://github.com/jsit/night-owl-iterm2-theme

@jsit Are you interested in collaborating on a single version? (I wasn't aware of your version when creating mine — sorry!)

There are quite a few differences but we could work to unify them (click to enlarge image).

ANSI bright colours. This version lifts them from the source here: https://github.com/sdras/night-owl-vscode-theme/blob/9d36c7624b85fe02f36c772f572a55225fc54d85/themes/Night Owl-color-theme.json#L182-L189. I think it produces slightly better legibility, visible in the tig screenshot above.
Cursor color. (I chose the same purple as the VS Code cursor.)
Links — open to interpretation; mine are from the link color when alt-clicking a function name to jump to the definition in VS Code.
Bold color. I felt the green better suits this because it stands out and is a dominant colour in the theme, but I'm open to ideas.
Foreground color. I took this from https://github.com/sdras/night-owl-vscode-theme/blob/9d36c7624b85fe02f36c772f572a55225fc54d85/themes/Night Owl-color-theme.json#L8.

Another here: https://github.com/florianeckerstorfer/night-owl-itermcolors


What is push-pull?

Hey 👋,
This is more a question than an issue.
In the readme, you say "I continued to research the Push-Pull reactivity system here" but does that mean? What is related to the "Push" in your current implementation? Do you think you could explain more?

@Barbapapazes
I can't explain it to you in detail, because I don't fully understand the alien-signals source code yet.
However, my current understanding of the source code for alien-signals to implement Push-Pull Reactive System is as follows:
Push

When the signal value is updated, look up the dependency through the subscriber list, and mark the dependency marked as needing to be propagated as needing to be checked for dirtiness

At the end of the propagate function, if the batch depth is 0, run the effect notification

Pull
The most important thing is that Computed is lazy, so combined with Push, the rest is:

Execute the queued effects

At this time, the effect reads the value of computed

computed is a lazy getter because it is marked in Push.2, and the prev value and current value are checked to decide whether to rebuild the value.

@Barbapapazes See:

#19

@Barbapapazes See:

feat: push-pull-push model 🚀 #19

Thanks! This is very interresting!

I've seen a wiki that explains the related concepts in great detail, but I can't find it.
Here's another related one (I just learned that the push-pull-push method is the same algorithmic concept as Reactively!):
https://milomg.dev/2022-12-01/reactivity/

I've seen a wiki that explains the related concepts in great detail, but I can't find it.

I found it!
https://en.wikipedia.org/wiki/Reactive_programming
The concepts are explained very clearly in it and I think we can close this one. :)


Whatsapp crashing on image upload.

I expected Whatsapp to work exactly like the desktop client. However, its not. The issue is that on the desktop client I used to drag and drop images directly from the websites to the person I want to sent it to. Unfortunately, Franz crashes when I try to drag an image from Safari straight into Franz's Whatsapp tab of my friend.

Expected Behavior

Ideally, dragging an image from Safari, it should upload the image normally.

Current Behavior

Franz crashes on image being dragged and dropped from Safari.
However, its working normally when I drag an image that is saved on my desktop onto Whatsapp. But dragging from Safari creates the problem.

Screenshots (if appropriate):
Possible Solution

Steps to Reproduce (for bugs)

Context

This is extremely important for me, as sometimes I just want to send an image without saving it on my laptop.

Your Environment

Franz Version used:
Operating System and version:

I`m having the same issue in ElementaryOS. When I drag a file from a folder to any app in Franz, it crashes and reloads.


Pp 4304 extract auth gateway req interface

WHAT
Extract AuthorisationGatewayRequest into an interface so the actual common implementation can be replaced with Stripe specific implementation in a different PR.

Not required in the current context, therefore closing.


Listing Stake With MAX and EEL pool under different pool groups.

Description
Separating Stake With MAX and EEL pool. As they do not belong to a pool group and are not using same pool resources.
Where should the reviewer start?
There are only two file which are changed

singlepooloperators.json
pool_clusters.json

Motivation and context
As both are showing as under same pool group. We wanted to change that.
Which issue it fixes?
How has this been tested?

What's the background here? These were same pools before and now the operator runs them under two brands?

So - it has been run by two different owners, but we were sharing relays earlier. Wanted to separate the relays too now.

sure - but the operator seems to still be the same?

sure - but the operator seems to still be the same?

how? No they aren't same.

Without the pool IDs changing, the pool operation keys are not modified (it is not design to be handed off) - so there is little sense in trying to make a claim that the pools are now independent - from security pov

The sensitive pool keys can not modified without changing pool IDs, only owners/KES/op cert/etc can (it is not design to be handed off) - so there is little sense in trying to make a claim that the pools are now independent - from security pov, when they werent operated independently before

We are operating these pools for our two different clients. Earlier we shared same relays for both, but now we are using different relays for both. So the two pools shouldn't be grouped.

You might have missed to read definition:
https://github.com/cardano-community/pool_groups/blob/a3d79a4d9c8834fb76a98d64f33670ad104b96cb/README.md?plain=1#L2
Ofcourse such services are expected to be part of cluster

@rdlrt
hi - we do not share any keys nor servers between both pools. As a technical person, I just maintain the servers of the pool. Both are different servers, different marketing and different people. Can you separate them?

Essentially a pool that may be sharing Core/Relays, Marketing, provisioning pool as a part of a service - are all eligible to be reported to addendum.json file on this repository.

Falls under "provisioning pool as a part of a service", so not really


import/ignore does not handle symlinked path?

My project creates a symlink called app that links to a directory in our monorepo to allow for easier module imports. Unfortunately, this seems to trip the import/no-extraneous-dependencies rule.
I want to ignore this rule on any line that imports from app/**.
I have added the following to our eslintrc.js:
    settings: {
      "import/ignore": [0, [
        /.*app.*/
      ]],
    }

Maybe I am missing something about how this setting works, but it is not disabling these warnings.

settings aren't eslint rules, you don't need to include a severity. import/ignore should just be an array of string patterns. https://github.com/benmosher/eslint-plugin-import#importignore

Ah yes, my mistake. I've removed the 0 and flattened the array, but I still get the following warning
'app' should be listed in the project's dependencies

for the line
import * as utils from 'app/lib/utils';

You're using a regex, but the setting calls for a string - try passing that instead?

i catch similar bug, if  symlinks in projects paths
tempoporary fix -- npm i<EMAIL_ADDRESS>--save-exact, i don't know how it works...

Actually, I think there is a real issue here. We're using resolve v1, which preserves symlinks by default, but node (and resolve v2) follows them by default.
What happens if you set your import/resolver settings so the node resolver is node: { preserveSymlinks: false }?


fix generics templates

This PR fixes the templates for MicroMapperInterface and MapperInterface.
regression tests have been added.
as an aside, is the $toClass param in MapperInterface::populate(...) redundant?
It seems we will only ever have concrete implementations here and will just new up a new class, which will match the class provided in the AsMapper attribute.

@weaverryan i don't think these are my failures

Hmm, actually, I think they are... it's related to deprecations in PHPStan - so my guess is from the new test. But these are "Other deprecation notices"... are those direct? Indirect? I'm actually not sure - we might need to tweak our SYMFONY_DEPRECATIONS_HELPER... or figure out the underlying cause of this.

I guess they are not classified properly because the deprecation is triggered from a class defined inside a phar, so it path has starts with phar://, which means it does not start with the absolute path of the vendor folder. I don't find it unexpected that we did not though of the case of phars in the vendor folder in that detection logic.

I wonder if changing the phpstan dependency to the stable 1.10 in composer.json would help (I can't remember why I have 1.11-dev - probably by accident).

doesn't help. but i tried using phpunit directly instead of (not so) simple-phpunit, and there are no deprecations.

done

Still not happy - not sure why - phpunit-bridge can be pesky to work with and configure. BUT, I have changed the settings so that if you push, the CI will run automatically vs waiting for my approval 🙃

oh yeah, it's hard coded paths in the allowed deprations

can't see the issue atm. there are deprecations in ci that i don't get locally.


Refactored Flight class to throw exception on instantiation failur…

Refactored Flight class to throw exception on instantiation failure, 2) Added helper class for consistent shared functionality, 3) modified the error log handling in XmlParser

Good job man!


updated alt text and updated application-specific captions

should be good to go with that last commit.


Fix URLs in vite-plugin-ruby package

Description 📖
This pull request fixes some broken URLs in vite-plugin-ruby/package.json.
Specifically, it changes ElMasssimo (3 "s"s) to ElMassimo (2 "s"s) and changes the branch name master to main (since there is no master branch and the default branch is main).
Background 📜
I noticed this because when dependabot created this PR, the links in the PR description (pointing here and here) were 404ing.
The Fix 🔨
I updated the URLs in vite-plugin-ruby/package.json to the appropriate URLs.
Screenshots 📷
N/A

I wasn't sure if I should add an entry to vite-plugin-ruby/CHANGELOG.md (?).

Hi David! Thanks for the fix, I'll release a new version shortly.

You're welcome. Thanks for this great library (including the slick documentation site)! ✨

Thanks, appreciate it!
1.0.12 has the corrected URLs!


Improvement

remove unnecessary manu
missing form & list view for model field (click "new" link but page not showing)
add feature to config email from server.conf[email]
  host: smtp.mailgun.org
  password: <your_password>
  port: 465
  type: mailgun
  user: netforce.com
  security: ssl

please run this command before testing : ./run.py -uf -d db_name because I set required field maibox_id to False (old version is True)


"add" support for withdrawcoin

seems like thats all that is needed - i'm lazy. is that right?

The change is in v0.5.2.


Set Text method removes border/boxes of text field

Set Text method removes border/boxes of the text field

If it is not an issue or duplicated please let me know the solution

You can make transparent Texfield by setting  backgroundColor: undefined and borderColor: undefined at the time of adding textfield to pdf.

You can make transparent Texfield by setting backgroundColor: undefined and borderColor: undefined at the time of adding textfield to pdf.

what I found these are the properties for creating a new field how can I apply these to the existing field?

@Hopding please look at the issue

@vbgsmanzoor   Did you find a way to apply properties to existing field?

You have to get the widgets of your field and set the property BS (border style).

Please fill out the new bug report form if you're still having trouble with this.


Unknown prop onItemClick on  tag

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

onItemClick should be triggered when I click on a menu item
Current Behavior

I got the following warning:

Warning: Unknown prop onItemClick on  tag. Remove this prop from the element. For details, see https://fb.me/react-unknown-prop

And click handler is not triggered
Context

  <Menu
    ref="autocompleteMenu"
    autoWidth={false}
    disableAutoFocus={this.state.focusTextField}
    initiallyKeyboardFocused={true}
    style={styles.menu}
    listStyle={styles.list}
    onItemClick={this.onItemClick}
    onMouseDown={this.handleMouseDown}
  >
    {this.props.data.map((v, i) => <MenuItem innerDivStyle={styles.innerDiv} key={i}>{this.props.displayText(v)}</MenuItem>)}
  </Menu>

We have been porting the component on the v1-beta branch. We have reimplemented it from the ground-up.
We encourage people starting with Material-UI to go with v1-beta. The master branch is in a low maintenance mode. We don't plan any work on it. However, we still do our best to review and merge pull requests that fix bugs on this branch.


Add Arweave support for NFT Metadata

Problem
Can we get Arweave stored metadata files indexed and displaying properly on HashScan.
Right now metadata stored in areweave will not show.
Example:
https://hashscan.io/mainnet/token/0.0.1518294/2?p=1&p1=1&k1=1672434637.410991003
Metadata URL: https://arweave.net/accqwldSOZL84IGcrnPTIt6FgNQTOlsSyEXS4-u8kk4
See UI:

Solution
Can we index or retrieve arweave urls and display accordingly?
Alternatives
There really is only one possible option to accept arweave as a storage solution for Token Metadata.

I see in the metadata that the ar:// URI scheme is used to locate the image.
I am wondering why such URI scheme is not used to locate the metadata JSON file as well (i.e. the decoded metadata field of the token would be ar://accqwldSOZL84IGcrnPTIt6FgNQTOlsSyEXS4). This would greatly help determine for sure this is stored on Arweave. Explorer might otherwise query the Arweave server with a hash which is not an Arweave hash.

I see in the metadata that the ar:// URI scheme is used to locate the image.
I am wondering why such URI scheme is not used to locate the metadata JSON file as well (i.e. the decoded metadata field of the token would be ar://accqwldSOZL84IGcrnPTIt6FgNQTOlsSyEXS4). This would greatly help determine for sure this is stored on Arweave. Explorer might otherwise query the Arweave server with a hash which is not an Arweave hash.

Hi there
This article can solve the problem
https://davincigraph.medium.com/how-dapps-can-utilize-arweave-data-storage-for-nfts-on-hedera-hashgraph-404086170c6c

https://docs.opensea.io/docs/metadata-standards#ipfs-and-arweave-uris
IPFS and Arweave URIs
OpenSea supports the storage of NFT metadata in decentralized file networks, so that they can’t be modified by a central party.
If you use IPFS to host your metadata, your URL should be in the format ipfs://. For example, ipfs://QmTy8w65yBXgyfG2ZBg5TrfB2hPjrDQH3RCQFJGkARStJb. If you plan to store on IPFS, we recommend Pinata for easily storing data. Here's a tutorial by Chainlink for getting started: https://blog.chain.link/build-deploy-and-sell-your-own-dynamic-nft/.
Arweave's equivalent is ar://. For example, ar://jK9sR4OrYvODj7PD3czIAyNJalub0-vdV_JAg1NqQ-o.
https://davincigraph.medium.com/how-dapps-can-utilize-arweave-data-storage-for-nfts-on-hedera-hashgraph-404086170c6c


Use ? operator in process_read

What have you changed? (mandatory)
This is a PR extracted from the Mvcc refinement PR. It introduced a function called process_read_impl -> Result<..> so that we can use ? operator to handle errors.
What are the type of the changes? (mandatory)

Improvement (non-breaking change which is an improvement to an existing feature)

How has this PR been tested? (mandatory)
CI
Does this PR affect documentation (docs/docs-cn) update? (mandatory)
No
Does this PR affect tidb-ansible update? (mandatory)
No
Refer to a related PR or issue link (optional)
Benchmark result if necessary (optional)
Add a few positive/negative examples (optional)

This PR is very simple. Please kindly give an approval. Thanks! @MyonKeminta @zhangjinpeng1987

/run-integration-tests

@BusyJay PTAL again, thanks!

@MyonKeminta Please click approve again, thanks!


Use enter, update, exit semantics

Should use enter, update and exit semantics instead of appear, update and remove.  Better for people who have worked with d3 previously.
Types should go from...
APPEAR, UPDATE, REMOVE
to
ENTER, UPDATE and EXIT
Component methods should be onEnter, onUpdate and onExit then.

Done in 0.6.0.


Node bridge

I'm very interested in a node bridge and was wondering if that was currently being worked on. I would be interested in contributing.

Hi @hugentoblerd , Node bridge is not developed yet.
I'm very interested to have this for the project (i was thinking in expressJS)
So, if you are interested, you can propose a pull request to add to the project.
For backend side, Me in particular, Im very busy right now with the project frontend side in order to support/handle multiple files and drag and drop features.
Regards!

Hi @hugentoblerd, in my project I include a Node.js bridge (using Express). I intent to publish it as an own project in the next days.

Hey @fkoester, that's awesome! Will you let me know when you publish it?

@hugentoblerd of course!

@hugentoblerd @joni2back Checkout my initial prototype for a Node.js bridge here:
https://github.com/fkoester/angular-filemanager-nodejs-bridge


Couple of fixes / improvements

Hey!
I added a npm start script that will compile the ES6 code to ES5, I removed an unused dependency and fixed the following warning:
Warning: You are manually calling a React.PropTypes validation function for the `value` prop on `BezierEditor`. This is deprecated and will not work in production with the next major version. You may be seeing this warning due to a third-party PropTypes library. See https://fb.me/react-warning-dont-call-proptypes for details.

Hope you like it =)

Hmmz, the reason I added the lib folder is so I can import the module in my project.
I am using Browserify + Babel but by default, node_modules will not get "babelified".
Currently this module only makes ES6 files available through the NPM package, which is probably fine when you use something like Rollup, but not with most other packagers.
I think that lib should be included and "lib/BezierEditor.js" should even be the "main" in the package.json

I agree for the problem you are raising and also that the the "main" should point to lib/, however, lib/ should not be versioned,
here is an example:
https://github.com/gre/react-json2d/blob/master/package.json#L5-L8

nowaday, I also tend to define a "files" field, that explicitely specify what should be pushed to NPM.
e.g.
"main": "lib/index.js",
  "files": [
    "src",
    "lib",
    "README.md"
  ],

Oh I see what you mean, sure, I guess you can pull it in and change it on your end right?

(NB: prepublish script is implicitely called when typing npm publish)

as you prefer, i can refactor that project conf tonight ;)

Hi @vespakoen ,
A new version was released (0.6.0) using latest lib versions.
The npm packages is published without need to transform anything, lib/index.js is the entry, and also have the appropriate sourcemap files.

Example is also more up to date, using create-react-app bootstrap.
Cheers


[Feature request] Implement OT-TRPO

Hi,
we developed and tested our algorithm OT-TRPO (published at the upcoming NeurIPS2022, you can find the preprint here) using stable baselines.
Is there an interest in integrating it with the existing package? We would be happy to discuss the implementation and integration of such a feature. You can find the available code here. It should need only minor effort to (i) integrate the continuous spaces implementation with stable baselines, and (ii) automatically switch between discrete and continuous case, with the same interface. We already have the tuning parameters for many environments (found with the hyperparameter tuning functionality of stable baselines).
Cheers,
Antonio

Hi,
thank you for your answer!

Two of the key features of optimal transport discrepancies are the following (please see the paper for more):

they allow us to compare probability measures (and thus policies) not sharing the same support (for which the KL divergence is infinity);
they encapsulate the geometry encoded by the transport cost in the action space: the discrepancy between two actions coincides with the discrepancy between the corresponding deterministic policies (whereas the KL divergence is again infinity).

These are reported in Table 5. In some instances, OT-TRPO results slower than both, in others faster the TRPO, but slower than PPO. However, ours is research code: we did not spend a lot of time optimizing it, but we still used the stable-baselines implementation for TRPO/PPO. In fact, one of the goals of integrating our code in stable baselines is to reach a “production-ready” version. Moreover, as mentioned in the paper, it is not a completely fair comparison. The theoretical benefits enabled by optimal transport discrepancies compared to the KL divergence come at the price of a “much harder” constraint in the trust region. A fairer comparison is then one that compares the proposed algorithm to other methods using similar “high-quality” notions of closeness, e.g. BGPG and WNPG. These are much slower than OT-TRPO.

We have not investigated this question much, but we can certainly try. Is this the case with TRPO/PPO? As far as we know, they also have a different set of hyperparameters for each of the discussed environments. This can definitely be something to look into it, but for which it would be great to have a community-reviewed optimized code.

We never tuned TRPO/PPO, we used the values in the repo mentioned in the paper. We acknowledge the results might need to be updated if better hyperparameters have been found between the time we wrote the paper and today. Nonetheless, our results are in line with what we could find on the internet (e.g., see https://spinningup.openai.com/en/latest/spinningup/bench.html for Swimmer). Finally, I think it is worth mentioning that we believe the community of stable baselines plays also a big role in the game when it comes to finding better hyperparameters and optimizing the code/spotting bugs. We did not spend much time optimizing our hyperparams either. Indeed, our main focus was to provide a class of theoretically justified algorithms. We are also looking forward to seeing what the community can achieve by selecting specific transportation costs, etc. In this sense, our experimental results are more proof of concept. We foresee much better results once a “production-ready” version of the code will be available (hence the request for starting this integration, with code review, etc.)

Not yet. We have results for some Mujoco environments, but we would be really happy to extend the studies to more environments.

No particular reason, we were working with that version when the project started. Also, if such issues affect the performances, we definitely did not look into them for our algorithm. Our implementation at the moment is a proof of concept, hence offering a wide range of improvements.

Please let us know if we can clarify any other points. But in general, I think that implementation/performances related questions are something that we would like to look at together with you and the community, as we did not have the capacity to push too much on the software engineering side, and we believe our results can be further improved.
Thank you!
All the best,
Antonio
PS: Your side note is the kind of polishing we look forward :)

We have not investigated this question much, but we can certainly try. Is this the case with TRPO/PPO?

yes, there are good defaults that work reasonably well on a wide range of problems (they are not optimal but they do work).

Nonetheless, our results are in line with what we could find on the internet (e.g., see https://spinningup.openai.com/en/latest/spinningup/bench.html for Swimmer).

It is known that most Swimmer results are under-tuned:

We have not investigated this question much, but we can certainly try. Is this the case with TRPO/PPO?

yes, there are good defaults that work reasonably well on a wide range of problems (they are not optimal but they do work).

Nonetheless, our results are in line with what we could find on the internet (e.g., see https://spinningup.openai.com/en/latest/spinningup/bench.html for Swimmer).

It is known that most Swimmer results are under-tuned: https://arxiv.org/abs/2208.07587
getting 300+ score in 20k steps: https://twitter.com/araffin2/status/1582452208033214464

In this sense, our experimental results are more proof of concept.

then I guess as for REDQ, I would prefer to wait a bit (for instance, I waited until DroQ came, which is a practical extension of REDQ).
In the meantime, you can add your repo to the SB3 project list that we have in our doc (please open a PR for that).
And feel free to share new experimental results if you have some.


CommandCallback should be removed

CommandCallback should be removed from the framework. CommandBus should return CompletableFuture instead of dealing with callback.

I believe this work ties in with #751.

Removing CommandCallback will cause problems in cases where we want to have some kind of general behaviour for the result processing. Hence closing.


Fix incomplete docstring for BuilderConfig

Fixes #5820
Also fixed a couple of typos I spotted

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


[FEAT] add i18n spanish, french and english

A simple POC of translation app with ngx-translate.
Fixes: #52
Add i18n for Spanish, English and French.
The translation is made with JSON files by sections and by key and values.
For example, for component side-bar.
Feel free to upload the rest of the translations.
For the moment only applied in side-bar.

[ ] translate all the app

[ ] side-bar

[x] ES
[x] FR
[x] EN
[x] AR
[x] RU

[ ] Find a good place to put the default language for the app for the moment is hardcoded in the file locale.service.ts:23 app.module.ts:160

don't hesitate to tell me if I say something wrong in Spanish @sanchis  :)

still in WIP

i need to fix locale for edit vuln and add vulns and it will be ok


Aggresive optimizers for CofEnv/CofThy

Minimum requirements:

It should render Cof.reduce completely useless.
[(phi \/ psi); (phi \/ psi); (phi \/ psi)] should be simplified to just [(phi \/ psi)].

Current design decisions:

Disjunctive normal forms?

@cangiuli @jonsterling Okay I will do the normal form...


bump operator-frawork-api

Description
bump operator-frawork-api to get its latest changes and fixes (https://github.com/operator-framework/api/compare/v0.10.7...f579141ce61d8bb7c4dc7c06169bc436ac4b9703)

@camilamacedo86 you might have a hard time finding another LGTM this week. Since this is pretty trivial, I'm comfortable if you want to merge when CI passes.


Fix #255

Fixes #255 using the same solution adopted in the Go std lib (see https://github.com/golang/go/issues/8415)

@Diggs thank you for the PR! If you add a test I'll merge and release.
Thanks again!


support RequestWrapper and ResponseWrapper annotation

maybe it will directely work but need test to validate.

start to work on it on branch wrapperAnnotation

remain todo: add unit test to validate that class in not generated and check that native work with this behaviour


Error 404 on download attempt

I have tried your program and I can't seem to get it to download bookmarks and/or likes. I have tried downloading channels and that works so it can crawl the site but if i ask it to download bookmarks and once i give it the access token it 404s

Hi,
it seems that they removed v2/users/me endpoint that I was using. Thankfully, this is easy to fix. I'll soon release v0.11.

Try this release @aeroFBW
https://github.com/Bukk94/CoubDownloader/releases/tag/v0.11

Hi,
I have downloaded the new version and it seems that for this version in particular the GUI does not want to pop up when i run it. I have tried both the basic and full releases

it seems to be working i just had to download .net 8.0, Thank you for your help and for making this program


Add channeled output with custom splitters adding ability

Update to latest binary from off site
Add NewServerCh/CommandCh to be able to run with output to channels

This improvement is needed in case of batch working with for example recursive scan of big library.
With original implementation it fails with buffer overloaded even on 50-200 files max.
With current let's say unlimited.
Idea of splitters is sending data to channel file by file and then + report footer.
Added splitters for -j option and also for default output.

I'm not sure I understand the purpose of this yet.
Two notes:

I only update ExifTool to the latest "stable", which is still 12.76
this package is based on the ideas behind cpp_exiftool

So I'm not sure I want to extend it further. I'll look into it though.

The main purpose of this –  ability to run some big actions which ExifTool itself can do easily.
For example I can run recursive scan of library with many thousands of content and all of them will be published to stdout or file.
But with this tool output will be overloaded even on library of about 100 photos.
So I've just added channel for output with ability to split it by some logic. And also made default splitFunc based on default output format.
In this case stdout will not be overloaded with any amount of items.
And sure there is entrypoint to put your own preferred SplitFunc

And Also I found there some other issues.
For Server:

by default -echo4 "some" - post it to stderr also, but you're not catching it, as result it returns error without proper tokens even if they was there

you're not managing PIDs of instances, so there is not ability to manage Server instances (with command there is at least context)

You're using only one "boundary" token, but for parallel running would be better to generate unique for each server instance.

The same issue with big output, like after recursive scan of a lot of mediafiles

Not PID returned to be able to manage it.

Reverted versions to LTS, agree. Sorry for that ))


Pass custom renderer via cli

Looking at https://github.com/miyuchina/mistletoe/blob/master/mistletoe/convert.py#L19
It appears that running mistletoe CLI parses to markdown by default.  It would be helpful to provide custom renderers at runtime.
Cheers,
-Tommy.

Feature added in 427c7d3.
Usage:
mistletoe foo.md --renderer custom_renderer.CustomRenderer

Thanks @lastcoolnameleft !

You’re welcome. And thank you as this made my debugging and experimenting a lot easier.

You would be happy to learn that if you just pass in the custom renderer flag without a file name, mistletoe will open interactive mode with that renderer. Just added earlier today in bac53767 !

Good to know. Thanks!
On Sun, Dec 24, 2017 at 8:27 AM Mi Yu<EMAIL_ADDRESS>wrote:

You would be happy to learn that if you just pass in the custom renderer
flag without a file name, mistletoe will open interactive mode with that
renderer. Just added earlier today in bac5376
https://github.com/miyuchina/mistletoe/commit/bac53767fe9e8e6b730ffe33abab4e6355371ef7
!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/miyuchina/mistletoe/issues/9#issuecomment-353787177,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAp7l7lG_hfAhv1W9P65Sqxs-p5L5zkMks5tDl9BgaJpZM4RKLaU
.


.Net Framework 4.8

For reasons I cannot explain (2 days of continuous debugging), when I upgraded two of my running project to .Net 4.8, I started to get he message with Action 'http://carpark.company.com/services/internal/ILookupService/GetRequestTypes' cannot be processed at the receiver, due to a ContractFilter mismatch at the EndpointDispatcher. This may be because of either a contract mismatch (mismatched Actions between sender and receiver) or a binding/security mismatch between the sender and the receiver. Check that sender and receiver have the same contract and the same binding (including security requirements, e.g. Message, Transport, None).
Reverting back to 4.7.2 resolves this.
Could you find time to validate this?

I may get a chance over the next few days to look more into this, but I couldn't recreate your issue on my first attempt. Which version are you using? You can try giving the 2.3.0-alpha4 release a chance and see if that fixes things. It's been seeing significant production use but I just haven't had the opportunity to fully release it.

Sorry, yes I'm on the latest alpha. And I'm still trying to figure this one out.

I have come to the conclusion that this issue is isolated to me and over a few projects only. I have rebuilt a project from scratch and got it working fine. I'm closing this issue now.


WM SVG generator

Should be good to go... no issues that I see on my end.

You need to delete the .svg files from your repo so they disappear from the pull request

You need to delete the .svg files from your repo so they disappear from the pull request

Ah yes, I forgot about that. It should be good now! Sent from Mail for Windows 10 From: Jim BowringSent: Tuesday, January 19, 2021 4:01 PMTo: CIRDLES/SquidCc: Noah Drake; AuthorSubject: Re: [CIRDLES/Squid] WM SVG generator (#545) You need to delete the .svg files from your repo so they disappear from the pull request—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or unsubscribe.

Ah yes, I forgot about that. It should be good now! Sent from Mail for Windows 10 From: Jim BowringSent: Tuesday, January 19, 2021 4:01 PMTo: CIRDLES/SquidCc: Noah Drake; AuthorSubject: Re: [CIRDLES/Squid] WM SVG generator (#545) You need to delete the .svg files from your repo so they disappear from the pull request—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or unsubscribe.

@drakene - still need to remove mouse menu ;-) Great job!

@drakene - still need to remove mouse menu ;-) Great job!

Of course I forgot! It's gone now... should be all good now.

Of course I forgot! It's gone now... should be all good now.


Update to openjdk8u31b13

If anybody happens to know how to keep track of openjdk releases, it would be great if they could tell me how.  Anyway until I manage to find a better way I'll just track the arch pkgs.

Travis failure is unrelated.

I added a fix for i686 builds. Currently they fail.

I also added the patch many distros use to get nonreparenting wms to work better.

I think the travis failure is due to a timeout.

I assume you've tested that this builds and everything?

Yeah I'm actually using it right now.


The package in the registry unexpectedly contains optionalDependencies

Description

The latest version (3.1.0) of this package in the Yarn registry (https://registry.yarnpkg.com/typesafe-actions/-/typesafe-actions-3.1.0.tgz) contains an optionalDependencies as well.
{
  "optionalDependencies": {
    "typescript": "3.3.3"
  }
}

Steps to Reproduce

Download the package directly from https://registry.yarnpkg.com/typesafe-actions/-/typesafe-actions-3.1.0.tgz or run yarn add typesafe-actions and checkout the downloaded package in node_modules.
Checkout package.json in the package.

CodeSandbox to help you reproduce issue in isolation

Please use this sandbox as an isolated environment to reproduce the bug: https://codesandbox.io/s/github/piotrwitek/typesafe-actions/tree/master/codesandbox

Expected behavior

package.json in the master branch (https://github.com/piotrwitek/typesafe-actions/blob/master/package.json#L55) doesn't have optionalDependencies. The published one should be the same.
Suggested solution(s)

Not sure how did this happen. Re-publish the package might help?
Project Dependencies

Typesafe-Actions Version: 3.1.0
TypeScript Version: 3.3.3333
tsconfig.json:

{
  "compilerOptions": {
    /* Basic Options */
    "target": "es5",
    "module": "commonjs",
    "lib": ["es2017", "dom"],
    "jsx": "react",
    "sourceMap": true,
    "removeComments": true,
    "noEmit": true,

    /* Strict Type-Checking Options */
    "strict": true,

    /* Additional Checks */
    "noUnusedLocals": true,
    "noUnusedParameters": true,
    "noImplicitReturns": true,
    "noFallthroughCasesInSwitch": true,

    /* Module Resolution Options */
    "moduleResolution": "node",
    "typeRoots": ["./node_modules/@types", "./typings"],
    "esModuleInterop": true,

    /* Experimental Options */
    "experimentalDecorators": true,

    /* Advanced Options */
    "skipLibCheck": true,
    "forceConsistentCasingInFileNames": true
  },
  "exclude": ["dist", ".cache", "node_modules"]
}

Environment

Browser and Version: Google Chrome 72.0.3626.109
OS: macOS Mojave 10.14.3
Node Version: 10.13.0
Package Manager and Version: Yarn 1.13.0

@lukyth thanks for the report
It's not a bug, TS was optional dep in v3.1.0, I removed it recently and it'll be release with<EMAIL_ADDRESS>Is there any specific issue that you're experiencing right now because of this?

It doesn't cause any project issue. I just notice that there're two TypeScript versions in my dependencies and it bugs me 😅 So I investigated until I found this.
Thank you for taking care of this issue though! 😄

Closed by ca8468cfe2f9935f59901381bdc02cf67d0526f6


⚠ ClamAV has degraded performance

In 5e8252d, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 777 ms

Resolved: ClamAV performance has improved in 8daff16.


Parking Lot Managements System

#3292 [NEW PROJECT] Parking Lot Management System

Close #3292

Checklist:

[✅] I have mentioned the issue number in my Pull Request.
[✅] I have commented my code, particularly in hard-to-understand areas

@MohitGupta121 Kindly,review this

@MohitGupta121 I have completed requested changes. Kindly,review and merge the project.

@MohitGupta121  Kindly, review and merge the project file pleasee.

@MohitGupta121  @TusharKesarwani  Requested changes had been made. Kindly, review the file and merge the PR 🎉


Fix typo in arXiv link

A very small fix, but when clicking the link to your paper in the README, I noticed the link was broken. This fixes it

With small fixes comes great discoverability. Thanks!