Is RIO going to be used in Kestrel?

Is RIO going to be integrated into Kestrel anytime soon?
Why shouldn't it be the standard, instead of libuv? I mean, it is faster in most cases... isn't it?

@halter73

https://github.com/aspnet/KestrelHttpServer/issues/828 Is work that should help us enable that. There are some trade-offs with RIO, so we will probably have people opt in to using RIO and allow developers to configure RIO-specific settings.

What are the trade-offs with RIO? As far as I've seen in some tests, it works faster and better than plain socket and simple IOCP. We all want people to start talking about ASP.NET Core and .NET core in general. Performance is a great tool to get them started. If RIO still has some pitfalls (which I'm not yet aware of), I think MSFT should concentrate on making it flawless so they could market .NET Core as a fast, reliable, lean and cross platform (also cross ISA) framework.
BTW, if you ask developers why they switched to NodeJS (and many have) they will almost automatically shout the "Asynchronous Non Blocking I/O" as if its Node's slogan. Why shouldn't Microsoft attract developers with even a faster, leaner approach?
Kestrel can use netmap for Linux and RIO for Windows.

I'm not an expert on RIO, so the short answer to the new title is that I don't know if RIO will be made the default. We'll be able to make a more informed decision once we build the RIO transport.
@mnns Are aware of any other production HTTP server that uses RIO and/or netmap (which seems to be a kernel module you need to manually install on Linux)? If so, that could certainly help our investigation. It's interesting you bring up NodeJS which, as I'm sure you are aware, is what libuv was built for.
@benaadams worked on an experimental RIO server a while back that I've taken a quick look at. One big issue seems to be is that every connection needs to have a unique (as in not shared with other concurrent connections) buffer constantly registered. This could lead the RIO server to consume a lot more memory than the equivalent libuv/socket based server given a lot of idle connections.
Without knowing ahead of time whether or not a given connection would be saturated or idle, it could be difficult to determine the optimal size of the registered buffer in order not to waste too much memory on idle connections while maintaining perf for active ones.
It's entirely possible we could work around the memory usage with some smart heuristics, but I'm not able to commit to making RIO the default networking engine without further investigation.

@halter73 Yep, NodeJS is using libuv and that's exactly why I think we should make ASP.NET Core better and faster than NodeJS, especially if we have the tools and technology (RIO), as we do right now.
Also, like you said, I'm sure that constantly registered connection buffers memory consumption can be easily mitigated with some simple heuristics. I'm no expert on RIO, but I've ran some tests and the numbers are extremely impressive, even unusual for a Windows machine.
Imagine the market impact if MSFT would announce a cross platform, cross ISA, modular, easy to deploy, extremely fast web server that utilizes cutting edge networking methodologies, both on Linux and on Windows. Add some PHP/ASP-like simple scriptable HTML web pages and you've got the best server in the market (only then add the bloated MVC overhead as an option, but that's another issue).

Any thoughts guys?

The transport refactoring has already happened to a degree https://github.com/aspnet/KestrelHttpServer/tree/dev/src
Kestrel.Transport.Abstractions
Kestrel.Transport.Sockets
Kestrel.Transport.Libuv

So RIO would become just another Transport https://github.com/aspnet/KestrelHttpServer/pull/1630
However there are bigger changes for the whole stack that means it will be easier for everyone to take advantage of these changes https://github.com/aspnet/KestrelHttpServer/issues/1980

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain spinthemoose.com has exceeded the max emails per hour (244/200 (122%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from github-smtp2-ext6.iad.github.net ([<IP_ADDRESS>]:57380 helo=github-smtp2b-ext-cp1-prd.iad.github.net)
by cp1164.blacksun.ca with esmtps (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256)
(Exim 4.89_1)
(envelope-from<EMAIL_ADDRESS>id 1eVQjW-0000Zk-Lm
for<EMAIL_ADDRESS>Sat, 30 Dec 2017 17:40:06 -0600
Date: Sat, 30 Dec 2017 15:39:46 -0800
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com;
s=pf2014; t=1514677186;
bh=2lPaGbLZA3ngd0g5gg33XgTk8n569OEgx8amgQoMi24=;
h=From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID:
List-Archive:List-Post:List-Unsubscribe:From;
b=ym4ba8HEfhfmGz/pztUqknXqrLA4QF9s+XS1eK7juZXZVTMFLBh6gptgBDo5syut9
tMCwRhu4+pI1eT+uwFjtv/C2dPZFp6FEy5Xixj49inGsQOwDkjVu0Y8didwJdMwlkj
/O9iWoN9q7FuWu1jzzWD9cG+nUy12uVtRRKHNdmk=
From: "ASP.NET Hello Bot"<EMAIL_ADDRESS>Reply-To: aspnet/Home<EMAIL_ADDRESS>To: aspnet/Home<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>In-Reply-To<EMAIL_ADDRESS>References<EMAIL_ADDRESS>Subject: Re: [aspnet/Home] Is RIO going to be Kestrel's default networking
engine? (#1903)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_5a4823c2e5025_1ff23fe9ff896f383731017";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: aspnet-hello
X-GitHub-Recipient: davidalpert
X-GitHub-Reason: subscribed
List-ID: aspnet/Home <Home.aspnet.github.com>
List-Archive: https://github.com/aspnet/Home
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/AACpROC4qvIh3wEf3pcoU2XP-8F_F9Ijks5tFsnCgaJpZM4LmTep
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>----==_mimepart_5a4823c2e5025_1ff23fe9ff896f383731017
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #1903.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/aspnet/Home/issues/1903#event-1405208826
----==_mimepart_5a4823c2e5025_1ff23fe9ff896f383731017
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #1903.
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.

----==_mimepart_5a4823c2e5025_1ff23fe9ff896f383731017--

This message was created automatically by mail delivery software.
A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain spinthemoose.com has exceeded the max emails per hour (245/200 (122%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from o1.sgmail.github.com ([<IP_ADDRESS>]:7067)
by cp1164.blacksun.ca with esmtps (TLSv1.2:ECDHE-RSA-AES128-GCM-SHA256:128)
(Exim 4.89_1)
(envelope-from<EMAIL_ADDRESS>id 1eVQjY-0000Zl-2B
for<EMAIL_ADDRESS>Sat, 30 Dec 2017 17:40:08 -0600
DKIM-Signature: v=1; a=rsa-sha1; c=relaxed/relaxed; d=github.com;
h=from:reply-to:to:cc:in-reply-to:references:subject:mime-version:content-type:content-transfer-encoding:list-id:list-archive:list-post:list-unsubscribe;
s=s20150108; bh=CU3EJPjU7P2Px0Dyv8ZwK0I4W1Q=; b=kliA1e64dBxZ+2q+
d7DNq+vD05QZcx0Wlm8LEJY/J5F+wos+8MZTNjnZ2YHNNkvKTKd/YGsLdglU2BjF
ECSelfvvJ/Kgaxor91B4rC22JTmSnC6HxJV8htRngYrMuIkSv5ukNQiYiwZ3oNRo
DPgRSchmrAY4urquZqSz6Z+aoSY=
Received: by filter0114p1las1.sendgrid.net with SMTP id filter0114p1las1-5707-5A4823C2-28
2017-12-30 23:39:46.962212382 +0000 UTC
Received: from github-smtp2a-ext-cp1-prd.iad.github.net (github-smtp2a-ext-cp1-prd.iad.github.net [<IP_ADDRESS>])
by ismtpd0008p1iad1.sendgrid.net (SG) with ESMTP id DPKVoy3nQie5cVf_fxG91Q
for<EMAIL_ADDRESS>Sat, 30 Dec 2017 23:39:46.811 +0000 (UTC)
Date: Sat, 30 Dec 2017 23:39:46 +0000 (UTC)
From: "ASP.NET Hello Bot"<EMAIL_ADDRESS>Reply-To: aspnet/Home<EMAIL_ADDRESS>To: aspnet/Home<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>In-Reply-To<EMAIL_ADDRESS>References<EMAIL_ADDRESS>Subject: Re: [aspnet/Home] Is RIO going to be Kestrel's default networking
engine? (#1903)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_5a4823c2afbe5_86ca3fa9fc52cf3428208d";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: aspnet-hello
X-GitHub-Recipient: davidalpert
X-GitHub-Reason: subscribed
List-ID: aspnet/Home <Home.aspnet.github.com>
List-Archive: https://github.com/aspnet/Home
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/AACpROC4qvIh3wEf3pcoU2XP-8F_F9Ijks5tFsnCgaJpZM4LmTep
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>X-SG-EID: rlj3t9ahcX0BZUzAOlajQ3VmBXo43xFJc4M1ImIDJhGAP+VHJvN/jLI2pKB98A2vCBGvWGjfR8pJR/
2J1pPvOco1hcf2oRDmUL+6RSod2eYKCzqcwP7fTYhrNmEcJwAGJdlbFi8VLrT2wVRtHlCBSkdN8deH
1qZSeySndz7dxeo/lvpkphcjWvzijaSNJPMAm/vV8PNYPYrklyvTtq+7yH9xPa8jG/32XwLKexkddj
M=
----==_mimepart_5a4823c2afbe5_86ca3fa9fc52cf3428208d
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/aspnet/Home/issues/1903#issuecomment-354575091
----==_mimepart_5a4823c2afbe5_86ca3fa9fc52cf3428208d
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: quoted-printable
This issue is being closed because it has not been updated in 3 months.<=
/p>
We apologize if this causes any inconvenience. We ask that if you are st=
ill encountering this issue, please log a new issue with updated informatio=
n and we will investigate.
&mda=
sh;You are receiving this because you are subscribed to this thread.<=
br />Reply to this email directly, view it on GitHub, or mute the thread.

=
----==_mimepart_5a4823c2afbe5_86ca3fa9fc52cf3428208d--


Fix race condition in EvalLoader

When using swoole coroutines with eval loader, where is a possibility to get an error Cannot declare class {class}, because the name is already in use, because coroutines share memory and declared classes.
Can be tested like this
require 'vendor/autoload.php';

readonly class Dto
{
    public function __construct(
        public int $id,
    ) {}
}

$mapper = \AutoMapper\AutoMapper::create();
$data = ['id' => 1];

\Co\run(static function () use ($mapper, $data) {
    for ($i=1; $i < 100_000; $i++) {
        \Co\go(static fn () => $mapper->map($data, Dto::class));
    }
});

Indeed, we do have done something similar for the FileLoader here https://github.com/jolicode/automapper/pull/174
Can you use the LockFactory instead of a while loop, like done on this PR ? This way i believe you could use a specific implementation that can handle an event loop for the LockFactory (like one that use this https://www.php.net/manual/fr/class.swoole-lock.php)
Then it will avoid loop + sleep and just block in an async way (allowing you to execute other coroutines)


Hard dependency on diff-highlight from diff-so-fancy

Maybe we should just depend on diff-highlight from inside of diff-so-fancy.
That would mean people instead configure it with git diff | diff-so-fancy directly.
If diff-highlight isn't found, we could just pass through the stdin and perhaps issue a warning that says "hey you're missing diff-highlight bud"
Seems a little more user-friendly. wdyt?

:+1: Ideally I don't wanna maintain diff-highlight. Wish we could reference that dependency from the official git repo somehow.

yeah understood.
i dont know of an easy way.
I just explored trying to use diff-highlight from the machine, but i'm not sure how to do that on non-homebrew installs.
tbh, the simplest looks to be just writing a stupid script to pull the latest diff-highlight from the git repo every once in a while. it doesnt change a lot so no biggie.

here's my proposal in #13

Found a problem in #13 (sorry didn't find it earlier). We assume the diff_highlight path is ./third_party/diff-highlight/diff-highlight which might not be the case. npm link won't work. Documentation

Grab the two scripts (diff-highlight and diff-so-fancy) via either downloading or cloning the repo.
Place them in a location that is in your PATH.

Is incorrect.
We don't really know the path of our hard dependency diff-highlight

Maybe we just use which diff-highlight?

Why not just let people install and pipe themselves?
git diff --color | diff-highlight | diff-so-fancy

We want to load diff-hight internally in diff-so-fancy.


Fix url contains /%2F can't set path correctly

Issues associated with this PR
#1651
Solutions
You should show your solutions about the issues in your PR, including the overall solutions, details and the changes. At this time, Chinese is allowed to describe these.
UT result
Unit Test is needed if the code is changed, your unit test should cover boundary cases, corner cases, and some exceptional cases. And you need to show the UT result.
Benchmark
If your code involves the processing of every request, you should give the Benchmark Result.
Code Style

Make sure Goimports has run
Show Golint result

@antJack PTAL

@antJack PTAL again

@nejisama @antJack PTAL   squash下commit


Allow rate limits to be applied across APIs.

Summary
This PR adds the capabilities requested in #2347. A new attribute has been added to the schema to allow the caller to specify that they want the rate-limiting plugin applied across APIs. The handler has been updated to read this attribute and use a constant api_id whenever it is applied across APIs so that all requests will be allocated to the same bucket.
Full changelog

Add new "cross_api" schema attribute
Update handler to honor the new attribute

Issues resolved
Fix #2347

I know this is incomplete per your guidelines, but I'm having local environment issues when I try to execute the test suite so I thought I'd go on and at least get this out here for review.

Hi @smitchelus,
Thanks for the PR. I've marked this as WIP since we'll need tests for this before we can review. Thank you!

@smitchelus any update based on the comments above?

Hi @p0pr0ck5, thanks for the reminder. I've been pulled off on other things. I will try to get back to this and incorporate your feedback ASAP.

Hi @smitchelus,
Thanks for the initial contribution. At this time, we don't think this is the appropriate approach here. The plugins development ecosystem, as well as the underlying logic behind ratelimiting plugins, is due for a revamp, so this work would likely be undone in the near future; given this, we're going to close this PR. We do appreciate the contribution and feedback. Please let us know if we can clarify anything here. Thank you very much!


Use path/filepath instead of path

The path is a package to manipulate pseudo paths. use path/filepath for physical paths. For example, backslash and slash is contained both in --path usage.

Thank you


Certificates are not renewed even though logs says otherwise

Checklist

Have you pulled and found the error with jc21/nginx-proxy-manager:latest docker image?

Yes

Are you sure you're not using someone else's docker image?

Yes

  app:
    image: 'jc21/nginx-proxy-manager:latest'

Have you searched for similar issues (both open and closed)?

Yes

Describe the bug
I just got notification from LE that my certificates are about to "expire in 19 days". From what I know, certbot should renew them 30 days prior.
Logs says, that NPE tries to renew them and it completes it (automatically), but certificates are not updated. Manual Force certificate update results in certificate being updated.
Nginx Proxy Manager Version
v2.9.18
To Reproduce
There is no error nowhere nor actual steps to reproduce as it should be done via automatic task
Expected behavior
Certificates should be updated automatically. If not, there should by an error why it's not updated.
 You should use "a.[b, c]" instead of "[a.b, a.c]". This will cause an error in objection 2.0
app_1  | 2022-10-26T09:55:38.633275944Z [10/26/2022] [9:55:38 AM] [Nginx    ] › ℹ  info      Reloading Nginx

Operating System
Running on RPi via docker (docker-compose)
Additional context
I read that sometimes NPM fails to renvew certificates if Force Redirect ies enabled but my understanding is that it fails because it can't reach the proper challange in that case. I disable the option still but it seems it didn't have any effect (and my certificates are still valid).
I can provide more logs/test things but I don't know where to dig for more :-)

A few points that may be useful

Yes force SSL does break renewal at this time.
Letsencrypt will not normally renew unless inside that 30 day to expiry so you not seeing renewals for those 9th December certs is expected
Forcing a renew of a cert should work at any time (letsencrypt has restrictions for over use). i.e. use the staging url if you are testing it has less restrictive quotas
The email from letsencrypt may be for certs no longer used but created by your account, if the dates are not the same as your current ones you can just ignore the emails. If NPM is working correctly you will not receive these emails since at 30 days from expiry your certs would auto renew which means letsencrypt forgets about the old cert being renewed with the same private key. You get emails when the private key changes i.e. you reinstall NPM with a fresh /data volume and create new certs from scratch.

Hi,
thank you for the comments and indeed (4) hit the nail on the head - I checked again the email from LE and while it matches the domains it's about different certificates ((on 15 Nov 22 08:16 +0000)). My bad for not paying more attention, sorry :-)


Refactor metal backend for memory efficiency and fix codegen error

I hope this one is more clear than the previews PR.
changes made are minimal

There's still mixed whitespace/typing and functional changes.

Changes made in tinygrad/:
------------------------------------------------------------
files                             insertions       deletions
------------------------------------------------------------
tinygrad/runtime/ops_metal.py             10               8
------------------------------------------------------------
lines added in the tinygrad folder: 2

I'm going to close this. I still don't understand what it gains, there's no Metal CI tests yet, and I don't like passing None to the no copy, it just seems like the wrong constructor.


What about network encryption?

Let's say I've got REST microservices running on AKS communicating together over HTTP and Kevin Mitnick wants to read my traffics, enters your data center and connects to Brocade network devices. How is network protected? It's mainly SDN with AES? Can I send sensitive data over HTTP without a need for HTTPs?

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

ID: 16c7901b-497c-38bd-3615-defab17b572b
Version Independent ID: 9855d796-3469-8be5-c9a7-75516d9b8856
Content: Azure encryption overview
Content Source: articles/security/fundamentals/encryption-overview.md
Service: security
Sub-service: security-fundamentals
GitHub Login: @barclayn
Microsoft Alias: barclayn

@kubaseai Thank you for your query . We will investigate and update this thread

@kubaseai Thank you for reaching out. Unfortunately, at this time we have been unable to review your issue in a timely manner. If you feel that this issue is still a concern, please respond and let us know. If you have a suggested update to our documentation, please don’t hesitate to reach out again. #please-close


RenderRepaintBoundary toImage() only returns visible area data when working with ScrollView

I have a SingleChildScrollView, and it can scrolls horizontally. The child widget of the SingleChildScrollView is a Container widget with height: 125.0, width: 3000.0
the child widget of the Container is a Stack widget with children are all Draggable widgets wrapped by Positioned widget, so I can place these Draggable widgets at my will. But when I use RenderRepaintBoundary toImage() method to capture the content of ScrollView, only the visible area can be captured, not the full content of ScrollView.

It's my bad to mess up the Widgets hierarchy. I made a mistake to let RepaintBoundary be the parent of SingleChildScrollView, but it really needs to put RepaintBoundary as the child of the SingleChildScrollView to work properly.
Closed.


Short notes with controls like CKEditor possible?

Hi Ronny,
A customer would like to have some additional functions for editing his notes, which he already knows from the CKEditor, e.g.

Colorize text or background (highlite)
Normal style
Undo function
checkbox (that would be awesome)

Perhaps a function would be possible, where you can define the actions yourself.
Many greetings!
Matthias

Development Done. Will be included in next Release


fix: channel id outputs coming due to discord.js v updates

Fixes #
Proposed Changes

Fixes the problem of id mentions added in the new discord.js version

Screenshots

This will now break for the other version. We should lock down the version to be consistent and make sure we lock it down to the latest version

I mean we can upgrade to latest on both platforms

We should not need to upgrade on both not even sure how we could do that, we should upgrade in the repo and make sure that everywhere is using the package.lock to be consistent

Do we really need to mention them again?
I think 2 mentions for a simple test command, is a bit too much

Do we really need to mention them again?
I think 2 mentions for a simple test command, is a bit too much

We don't exactly use the mention again, we keep the mention. If we don't want to keep the mention I can introduce changes?


Document how to install PyTorch with CUDA

There's no way we can automatically install the CUDA version of PyTorch on Windows. We must explain in the README how to install the CUDA version.

:tada: This issue has been resolved in version 0.24.0 :tada:
The release is available on:

v0.24.0
GitHub release

Your semantic-release bot :package::rocket:


update SICF files

also please upgrade to latest abapGit version and push the SICF files again
related: https://github.com/larshp/abapGit/pull/2453

I am at version 1.84.0 at the moment. What is the latest version? I am currently getting an error message when trying to PULL the latest version of abapGit

the message about texts are okay, if you have pulled the latest it is the latest


Permissions dont work when shading

CommandAPI version: 6.3.1
Minecraft version: 1.17
When shading commandapi, i couldnt get the permissions to work. It just says no permissions.
I created clean server with clean plugin which had only one command.
Permissions works if the plugin doesnt use shaded version of command api.
Even if i added command .withPermission("test") it wouldnt work.
new CommandAPICommand("test1")                 .executesPlayer((sender, args) -> {                     sender.sendMessage("test");                 }).register();

@Zoltus
Did you run CommandAPI.onEnable(plugin) in your shaded plugin's onEnable() method? The CommandAPI.onEnable(plugin) method schedules the CommandAPI's permission linker which runs when the server finishes starting up.

@Zoltus Did you run CommandAPI.onEnable(plugin) in your shaded plugin's onEnable() method? The CommandAPI.onEnable(plugin) method schedules the CommandAPI's permission linker which runs when the server finishes starting up.


Added the move keyword

You also need to generate the .tmLanguage file. See the README.md

Done it.

Thanks :)
Btw: You can squash commits using git commit --amend. Also try to use imperative present tense in the commit message, see http://git-scm.com/book/ch5-2.html

Great, maybe next time! Thanks :)

On May 28, 2015, at 6:43 PM, Jan Niklas Hasse<EMAIL_ADDRESS>wrote:
Thanks :)
Btw: You can squash commits using git commit --amend. Also try to use imperative present tense in the commit message, see http://git-scm.com/book/ch5-2.html http://git-scm.com/book/ch5-2.html
—
Reply to this email directly or view it on GitHub https://github.com/jhasse/sublime-rust/pull/53#issuecomment-106476787.


Error on WEB login

Hi @Mahmoudz,
I'm testing the last release, when I'm trying to login on admin.hello.dev/login and submit the form, the response is:
{
  "message": "Type error: Argument 2 passed to App\\Containers\\Debugger\\Objects\\Output::__construct() must be an instance of Illuminate\\Http\\Response, instance of Illuminate\\Http\\RedirectResponse given, called in \/home\/vagrant\/Code\/hello\/app\/Containers\/Debugger\/Middlewares\/RequestsMonitorMiddleware.php on line 29",
  "status_code":500,
  "debug":{
    "line":49,
    "file": "\/home\/vagrant\/Code\/hello\/app\/Containers\/Debugger\/Objects\/Output.php",
    "class": "Symfony\\Component\\Debug\\Exception\\FatalThrowableError",
    "trace": [...]
  }
}

The RequestsMonitorMiddleware should be used on API routes only, or not?

Find other error related to WEB Login, when submitting the login form, the user credential check is ok and the user is successful authenticated, so, in the redirection to de dashboard page the user appears to not being authenticated and is redirected to the login page again.

Looks like no web middleware are executed, but can't find why...


Update Bolt

[x] verified that the CMS I'm adding is still maintained.
[x] read CONTRIBUTING.md.
[x] did not generate README.md.

Thank you!

Thank you!

Thank you as well :-)

Thank you as well :-)


add timezone config

Hello.
If you are using wordpress in a foreigntime zone, it seems that the output dates will not match.
e.g.
If I write article in the morning to wordpress running on Asia/Tokyo and export it.

expect
2020-01-05-my-morning-post
actual
2020-01-04-my-morning-post

This will change the url of the ported blog.
http://mysite.jp/2020/01/04/my-morning-post
It will be more useful if i can config it.

Thank you! They code was refactored since you created this PR, so I had to move some things around, but this PR is merged now.


TLS for goldstone server

Adds a self-signed cert to the web server, changes port to 8443.  Currently no redirect in place.  There are plans to support BYO certs and redirection from insecure port in the future.
Also, included detailed versioning in docker images, and HTML page version references.  Updated the various lifecycle scripts to deal...

Current coverage is 68.49%

Merging #430 into master will decrease coverage by -0.07% as of f2ac002

@@            master    #430   diff @@
======================================
  Files           63      63
  Stmts         2780    2787     +7
  Branches         0       0
  Methods          0       0
======================================
+ Hit           1906    1909     +3
  Partial          0       0
- Missed         874     878     +4

Review entire Coverage Diff as of f2ac002

Uncovered Suggestions

+0.62% via goldstone/views.py#30...46
+0.59% via ...stone/drfes/views.py#134...149
+0.41% via ...tone/drfes/models.py#165...175
See 7 more...

Powered by Codecov. Updated on successful CI builds.

closing, will resubmit.  need to update the circle.yml to use the right image version.

ready for review again...


Update Rust information and add JSDoc

Rust is now well past its 1.0 release, so I updated that and added further details about the rustdoc tool (also in the code generation section). I also added JSDoc, which seems like the sort of tool that should be on this kind of list.

Hey @sjsyrek. Thanks for the PR and sorry for taking so long to get back to you. I did a manual merge of the JSDoc addition to master. I'm all for the Rust details being accurate but your addition is a bit too long. I'm going to leave it out for now but send me another PR if you can find a way to get it down to ~2 lines.


Added word to documentation

Improved documentation by adding word for better sentence structure.
Signed-off-by: Mike Casas<EMAIL_ADDRESS>
- What I did
- How I did it
- How to verify it
- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

LGTM


Request body only "kind" is supported why not all others (proxyUser,...heartbeatTimeoutInSecond) as input ?

Only 'kind' is allowed as parameter, what about all other request body inputs supported by livy rest api.
Am I missing something here?
proxyUser | User to impersonate when starting the session | string
jars | jars to be used in this session | List of string
pyFiles | Python files to be used in this session | List of string
files | files to be used in this session | List of string
driverMemory | Amount of memory to use for the driver process | string
driverCores | Number of cores to use for the driver process | int
executorMemory | Amount of memory to use per executor process | string
executorCores | Number of cores to use for each executor | int
numExecutors | Number of executors to launch for this session | int
archives | Archives to be used in this session | List of string
queue | The name of the YARN queue to which submitted | string
name | The name of this session | string
conf | Spark configuration properties | Map of key=val
heartbeatTimeoutInSecond | Timeout in second to which session be orphaned | int

I just didn't get around to implementing them yet. Are there any in particular you need? I'm happy to extend the code to support them if it would be of use to you.

We are using most of these parameters in our calls.
It would be great if we have these in the library.
If you can create any branch for this, I can help on test/fix issues and/or implement some.
Ramana

Sorry for not getting to this sooner - I've just merged PR #40 supporting the proxyUser and hope to implement others soon.

Hi
Looks to me that would be great if all those param would be available.
Maybe by passing a dict ?
Thanks

This has been implemented in release 0.5.0. Thanks for the suggestion!


add info on how to use a specific config

add information about how to install and use this package in different configurations

Is there a reason for not merging this pull request?

@Arcanemagus could you see what’s up with circleci? I don’t have admin access anymore.

It looks like when this PR was created they had based it on a commit from before the CircleCI 2.0 configuration was added.
@oupala are you still interested in updating this PR? If so it will need to be rebased on master (and there are several changes I would say it needs, but I'm not going to do a full review if it's not going to be worked on).

This PR contains only documentation changes. Do you really think a rebase is mandatory?
Is a rebase possible with GitHub? I'm asking as my origin repo has been deleted since the time the PR is waiting.

This PR contains only documentation changes. Do you really think a rebase is mandatory?
Is a rebase possible with GitHub?

I could bypass the protections in place and merge this, but that's reserved for when there is no other way, which isn't the case here.

I'm asking as my origin repo has been deleted since the time the PR is waiting.

That.... definitely poses some issues. If you want to create a fork again you could cherry-pick the commit from here and start a new PR where you can make changes.

I had to create a new pull request #211.
Feel free to merge it now!


Did you forget to signal async completion?

Before you open this issue, please complete the following tasks:

[ ] use the search bar at the top of the page to search this repository for similar issues or discussions that have already been opened.
[ ] if you are looking for help from the gulp team or community, open a discussion.
[ ] if you think there is a problem with the plugin you're using, open a discussion.
[ ] if you think there is a bug in our code, open this issue.

What were you expecting to happen?
What actually happened?
Please give us a sample of your gulpfile

'use strict';

import gulp from 'gulp';
import sourcemaps from'gulp-sourcemaps';
import stylus from 'gulp-stylus';
import coffee from 'gulp-coffee';
import babel from 'gulp-babel';
import minify from 'gulp-babel-minify';

gulp.task('compile-styl',async function(){
  gulp.src('./www/dist/private/*.styl')
    .pipe(sourcemaps.init())
      .pipe(stylus())
      .pipe(sourcemaps.write('../mapping/styles'))
      .pipe(gulp.dest('./www/dist/private/temp/styles'));
});

gulp.task('compile-coffee',async function(){
  gulp.src('./www/dist/private/*.coffee')
    .pipe(coffee({bare: true}))
    .pipe(sourcemaps.init())
      .pipe(coffee())
      .pipe(sourcemaps.write('../mapping/scripts'))
      .pipe(gulp.dest('./www/dist/private/temp/scripts'));
});

gulp.task('minify-css',async function(){
  gulp.src('./www/dist/private/temp/styles/*.css')
    .pipe(stylus({
			debug: true,
			compress: true,
		}, (details) => {
      console.log(`tamanho: ${details.stats.originalSize}`);
      console.log(`tamanho minificado: ${details.stats.minifiedSize}`);
      console.log(`time: ${details.stats.timeSpent}`);
    }))
    .pipe(sourcemaps.init())
    .pipe(sourcemaps.write('../mapping'))
    .pipe(gulp.dest('./www/dist/public/styles'));
});

gulp.task('transpile-babel',async function(){
  gulp.src('./www/dist/private/temp/scripts/*.js')
    .pipe(sourcemaps.init())
    .pipe(babel({
            presets: ['@babel/env']
        }))
      .pipe(sourcemaps.write('../mapping'))
    .pipe(gulp.dest('./www/dist/private/temp/scripts/compat'));
});

gulp.task('minify-js',() =>{
  gulp.src('./www/dist/private/temp/scripts/compat/*.js')
    .pipe(sourcemaps.init())
    .pipe(minify({
      mangle: {
        keepClassName: true
      }
      }))
      .pipe(sourcemaps.write('../mapping'))
    .pipe(gulp.dest('./www/dist/public/scripts'));
});

Terminal output / screenshots

$ gulp compile-styl
[19:44:33] Requiring external module @babel/register
[19:44:37] Using gulpfile ~\Desktop\projects\projectX\gulpfile.babel.js
[19:44:37] Starting 'compile-styl'...
[19:44:37] The following tasks did not complete: compile-styl
[19:44:37] Did you forget to signal async completion?

Please provide the following information:

OS & version [e.g. MacOS Catalina 10.15.4]: windows 10-latest
node version (run node -v):14.17.4
npm version (run npm -v):6.14.14
gulp version (run gulp -v):
CLI version: 2.3.0
Local version: 4.0.2

Additional information

solved

solution
  gulp.src('./www/dist/private/*.styl')
    .pipe(sourcemaps.init())
    .pipe(stylus())
    .pipe(sourcemaps.write('../mapping/styles'))
    .pipe(gulp.dest('./www/dist/private/temp/styles'));
		done();
});

gulp.task('compile-coffee',done =>{
  gulp.src('./www/dist/private/*.coffee')
    .pipe(coffee({bare: true}))
    .pipe(sourcemaps.init())
    .pipe(coffee())
    .pipe(sourcemaps.write('../mapping/scripts'))
    .pipe(gulp.dest('./www/dist/private/temp/scripts'));
		done();
});

gulp.task('minify-css',done =>{
  gulp.src('./www/dist/private/temp/styles/*.css')
    .pipe(stylus({
			debug: true,
			compress: true,
		}, (details) => {
      console.log(`tamanho: ${details.stats.originalSize}`);
      console.log(`tamanho minificado: ${details.stats.minifiedSize}`);
      console.log(`time: ${details.stats.timeSpent}`);
    }))
    .pipe(sourcemaps.init())
    .pipe(sourcemaps.write('../mapping'))
    .pipe(gulp.dest('./www/dist/public/styles'));
		done();
});

gulp.task('transpile-babel',done =>{
  gulp.src('./www/dist/private/temp/scripts/*.js')
    .pipe(sourcemaps.init())
    .pipe(babel({
            presets: ['@babel/env']
        }))
      .pipe(sourcemaps.write('../mapping'))
    .pipe(gulp.dest('./www/dist/private/temp/scripts/compat'));
		done();
});

gulp.task('minify-js',done =>{
  gulp.src('./www/dist/private/temp/scripts/compat/*.js')
    .pipe(sourcemaps.init())
    .pipe(minify({
      mangle: {
        keepClassName: true
      }
      }))
      .pipe(sourcemaps.write('../mapping'))
    .pipe(gulp.dest('./www/dist/public/scripts'));
		done();
});

gulp.task("default", gulp.series("compile-styl","compile-coffee"))```

Your "solution" is completely incorrect. Read the async completion docs on the website.

i try as docs but doesn't work

https://gulpjs.com/docs/en/getting-started/async-completion#returning-a-stream - you must always return gulp streams from a task.

exports.default give me a error

This is not a support forum. Please direct your question to github discussions or stack overflow.


Let extensions use the blue loading bar at the top

The jupyter extension would like to use the blue loading bar when pressing 'restart kernel' on native notebooks.

Did you look at vscode.window.withProgress() ?

Did you look at vscode.window.withProgress() ?

We did, but that shows progress either in the status bar or with a notification, the blue loading bar is more visible.

We did, but that shows progress either in the status bar or with a notification, the blue loading bar is more visible.


Lower dependency constraints to match dio

Pull Request Template
Your checklist for this pull request
Please review the guidelines for contributing to this repository.

[x] Make sure you are requesting to pull a feature/bugfix branch (right side). Don't request your main!
[x] Make sure you are making a pull request against the main branch (left side). Also, you should start your branch off our main.
[ ] Make sure that you add reviewers, assignees, labels, projects, milestones and linkes issues to the pull request as you see fit.
[ ] Check the commit's or even all commits' message styles to match our requested structure.
[ ] Check your code additions will fail neither code linting checks nor unit test.

Tip: Delete everything but the following section that contains the description of the pull request.

Description

fixes #143
Lower dependency constraints to match dio
Run pipeline on min/stable SDK

running on different platforms is not required, there is no platform specific code involved in this package

CC @SebastianKlaiber @chunghha
@LukaGiorgadze The platform tests make no sense for the package, instead testing for min/stable SDK is more important. If that works for you please adjust the required workflows.


New extension: Mochi

Description

Quickly create Mochi cards
Screencast

https://user-images.githubusercontent.com/11070996/213101113-0b51650c-c1b0-4ead-b893-1c96ebc462f4.mp4
Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Thank you for your review.
I'll fix your suggestion.

@pernielsentikaer
Hi, I have added an error message view. This view is displayed when an API Error occurs.

Add validation so the user can't submit empty fields; you can use useForm

Sorry, I can't suggest the above. because this extension's form field is created dynamically.
So I can't know the required field in advance.

Hi 👋
Looks good to me, approved 🔥


Add Feature - Eth Money Velocity widget

Rationale
A money metric frequently used in trad finance regarding currency.  Gives good indicator about the relative change in economic activity across time.
General formula
V = Velocity of money
GDP = total value of economic transactions
M1 = Monetary Base
$V = \frac  {GDP}{M1}$
Eth relevant inputs
Velocity
The idea here, is to include all transfers of eth in Ethereum economy, this formula should cover it:
$GDP_{eth}$ = eth_pure_transfers +  tx_msg.value + weth.transfer + weth.transferFrom
NOTE: This does not include L2s + sidechains
M1 - coins and currency in circulation
M1 should include coins and currency in circulation. "in circulation" could be debatable. To be consistant with other graph we could use the liquid ETH from "scarcity" widget

Thus:
M1 = totalSupply - ("staking" +"defi (stale)" + "burn")
Resources
velocity
Money supply

I totally missed this @markoinether . Sorry. Regardless, still hunkered down in the current dev cycle, when we get back to the roadmap I'll make sure to discuss and evaluate this with the team. Thanks for the suggestion!

@alextes I'll be really happy to pick this up. Let me know if I can, thanks :)

@ameya-deshmukh I think this one would need more design thinking but would you be interested in contributing to another widget?
For example I've been wanting a better accounting of how much ETH is in DeFi. Maybe using DefiLlama, would you be interested in helping with that?  It would require some Rust backend work.

If you prefer TypeScript frontend we have this design which needs implementation:

lmk, and let's continue in a separate issue!

For sure! Keen on both issues but more excited about the ETH accounting in DeFi one! Can you open a separate issue please? LFG!

@ameya-deshmukh #274


I have a problème when i install the python-binance in pycharm i have python 3.8 and pycharm 2020.1 . can you help me pleas

Try:
Install "Build Tools for Visual Studio": https://visualstudio.microsoft.com/downloads/
py -m pip install setuptools --upgrade
py -m pip install wheel --upgrade
If it doesnt help, please post the log file mentioned in your error msg.

I had the same problem when i tried to install it the first time. It seems that Twisted caused the error. I fixed it by installing the "C++ Universal Windows Platform runtime for v142" component from the visual studio build tools

Hello,
For those with solution above not working.
I tried several solutions like upgrade of pip, setuptools or Wheel. Nothing work. I reinstalled Microsoft VC tools. And it doesn't work.
I found solution by downgrade of my python version. I was on python 3.9
I changed to python 3. 7.9, then it works perfectly.
Hope it will help


Make spectral-axis a SpectralCoord

https://docs.astropy.org/en/stable/api/astropy.coordinates.SpectralCoord.html

(this may be a subset of "inherit from specutils")

https://github.com/astropy/specutils/blob/main/specutils/spectra/spectral_axis.py


Fix code highlighting

"Command" isn't part of the command so should be outside the highlight.

❤ thank you !


router: fix slash path trimming

Describe the pull request
Fix the wrong path trimming behavior, which makes route /{**: **} can get the correct parameter value when there is a slash at the end of the path.
Link to the issue: https://github.com/flamego/flamego/issues/75
Checklist

[x] I agree to follow the Code of Conduct by submitting this pull request.
[x] I have read and acknowledge the Contributing guide.
[x] I have added test cases to cover the new code.

https://github.com/flamego/flamego/releases/tag/v1.0.1 has been tagged for this merge.


DCOS-38065, DCOS-37579: feat: implement table component

Wrap react-virtualized in a new component called Table
exposing only the props necessary to display a barebones
table.
Closes DCOS-38064
chore(deps): add packages
This adds the following packages:

react virtualized
This is the library we are using for our virtual list it is providing
us with the multiGrid component which we use for our table.
memoize one
This is a function which will cache return values based on the
parameters, so it will only execute if the parameters change
enzyme-to-json
This will generate a json for snapshot testing from enzyme objects.
Unfortunately testing the table is cumbersome with the react
renderer.
jest-emotion
This will provide the possibility to render the css created by
emotion instead of just the css class names. By that we can also test
the resulting css. Which is important for the table.
These packages are needed for the
Table component.

Commit:
3ed0db9c92ed1bb06dc3822ec9981f27d40fad08
feat(table): add column component
This introduces the Column component to the library. The Column will take 3 properties:

cellRenderer
The cellRenderer will expect a function which returns a JSX.Element which will be
used to render the Datum cell.
width
The width will expect a function to calculate the width it should return a number.
As a parameter the total count of columns, total width, remaining width and current
column index are provided in a single object.
label
The label expects a JSX Element or a string which will be used to render the header.

closes DCOS-38064

Commit:
39fadd5b83fd4341916966ca47b6ddab3fd76683
feat(table): this introduces the table wrapping component
This introduces a Table component which expects data as props and Columns as children. Data is a
Array of any. It will use the props from the Column Component to render a proper table. This also
introduces the horizontal scrolling for the table if the content is bigger then the total width.
Although the first column always stays fixed.
close DCOS-38065, DCOS-37579

Commit:
1a80d30301071704beda1e23169d4e9b2e4e3a35
test: add initial tests

Commit:
62b60b75a4397235d8d7bfc5107cee08b4b91b87

This needs more

[ ] Tests
[ ] Documentation

Flicts!

Thanks, @nLight for your insights I will fix the comments as soon as possible. Also, I will facilitate the discussions around some patterns.

@nLight and @DanielMSchmidt I worked on the comments from @nLight maybe.

Merge at will

:ship:


add missing buffer functions to Python interface

The buffer functions are missing in the Python DLL interface files.

Thanks! We will take a look at your PR.

@miguelgrinberg would you mind to add a "by buffer" example like the one in the .cpp for the .py example?

Thanks.
Merging without example, we'll add the example later.


Add getters for raw private/public keys

When we added support for setting raw private/public keys we should have added getters at the same time. This fixes that omission.
Fixes #6259.

All of the above issues fixed, as well as fixing a couple of errors identified by travis. Fixup commits pushed.

Pushed a couple more commits to address travis/appveyor failures.

Grrr..still had some travis/appveyor failures. Now fixed (hopefully). @richsalz - please can you reconfirm?

and also reconfirm.

Pushed. Thanks.


Refactor docs

Now that the interactive docs have been merged, we can remove the static docs and update the links to point to the new docs. This way we don't need to maintain two version of the docs.

LGTM, maybe keeping the doc/README.md with only the link to the live doc (user) and the link on where to contribute (dev).

@jourdain Done. I also added a link to the api reference (i'll update the link later when I add it to the dash-docs)


Fix handleLCL consensus bug

Consensus::checkLCL can change the internal consensus state_ but it was being called inside timerEntry after switch(state_).  In rare cases, this could end up calling stateEstablish even when the state_ was open.

Codecov Report

Merging #2065 into develop will decrease coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop    #2065      +/-   ##
===========================================
- Coverage    68.54%   68.53%   -0.01%
===========================================
  Files          679      679
  Lines        49554    49554
===========================================
- Hits         33965    33964       -1
- Misses       15589    15590       +1

Impacted Files
Coverage Δ

src/ripple/consensus/Consensus.h
81.08% <100%> (+0.18%)
:arrow_up:

src/ripple/beast/asio/io_latency_probe.h
97.01% <0%> (-1.5%)
:arrow_down:

src/ripple/app/main/Application.cpp
60.57% <0%> (-0.13%)
:arrow_down:

Continue to review full report at Codecov.

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

Force pushed with the updated comment.  Thanks for stumbling on the issue and reviewing @scottschurr!


Core: Sidebar can be shown in no distraction mode.

Steps:

Open index.html
Click View -> No Distractions
Click View -> Show Sidebar

Expect result:
Sidebar can not be shown.
Actual result:
Sidebar was shown as well as No Distractions was checked.

Pull open

@bhldev  We essentially are collapsing the project tree. You can still pull open the project tree in no distraction mode. Could you double check on this and update this thread?

This functions as follows currently: if the sidebar is on and no distraction mode is turned on, the sidebar gets turned off. If No distraction mode is on and the sidebar gets turned on, the sidebar appears still. Please update with intended functionality so I can either retract pull or modify pull to match UX experience. Thanks.

Allowing the user to open the sidebar was conscious decision we made . This is to help with users to see the project contents even in no distraction mode for some time if he choose. Closing .


flagfile reloading doesn't work

for whatever reason, only a small fraction of the running tasks pick up the new flag values.

fixed by setting an environment variable which disables the tf.gfile cache semantics.  (ugh).  PR to come.


Unable to clear log using Activity::cleanLog();

Thank you for such a nice module. It works great. Its simple and working.
I just need to know why Activity::cleanLog(); is not working? Its showing this error
BadMethodCallException in Builder.php line 2161:
Call to undefined method Illuminate\Database\Query\Builder::cleanLog()

I am just using it normally, like below i have a controller with name LogsController and i have used use Spatie\Activitylog\Models\Activity; in which i have just 2 functions as below:
public function index()
    {
        $latestActivities = Activity::with('user')->latest()->get();
        return view('logs.logs')->with('logs', $latestActivities);
    }

    public function clearlog(){
        Activity::cleanLog();
        Activity::log('Logs cleared By : '.Auth::user()->name);
        \Session::flash('success', 'Logs deleted successfully');
    }

I have defined a route for this:
Route::get('logs/clearlog', 'LogsController@clearlog');
Route::resource('logs', 'LogsController');

I dont know where i am making mistake. Please help me get this thing straight. I am new to laravel.
Thank you!

Did you import the Activity-facade at the top of your file?
use Activity;

No its not allowing me to import two files. I have already import
use Spatie\Activitylog\Models\Activity;
What should i do?

Here is how my full controller looks like...
<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;

use Auth;
use App\Http\Requests;
use App\Http\Controllers\Controller;
use Spatie\Activitylog\Models\Activity;
//use Activity;

class LogsController extends Controller
{
    /**
     * Display a listing of the resource.
     *
     * @return \Illuminate\Http\Response
     */
    public function index()
    {
        $latestActivities = Activity::with('user')->latest()->get();
        return view('logs.logs')->with('logs', $latestActivities);
    }

    public function headerLogs()
    {
        $latestActivities = Activity::with('user')->latest()->limit(10)->get();
        return view('layouts.header')->with('logs', $latestActivities);
    }

    public function clearlog(){
        Activity::cleanLog();
        Activity::log('Logs cleared By : '.Auth::user()->name);
        \Session::flash('success', 'Logs deleted successfully');
        return redirect('logs');
    }
}

You'll need to rename one of your imports with the as keyword.
use Activity;
use Spatie\Activitylog\Models\Activity as ActivityModel;


FirebaseAuth.signInWithEmailAndPassword internal NullPointerException

Step 1: Are you in the right place?

For issues or feature requests related to the code in this repository file a Github issue.

If you are filing an issue about FCM in the background make sure to read #4 and #89 first!

For general technical questions, post a question on StackOverflow tagged appropriately.
For general Firebase discussion, use the firebase-talk google group
For help troubleshooting your application that does not fall under one of the above categories, reach out to the personalized Firebase support channel

Step 2: Describe your environment

Android device: hudl 2
Android OS version: 4.4.2
Google Play Services version: unknown
Firebase Auth SDK version: 11.8.0

Step 3: Describe the problem:
I'm getting a NullPointerException from a crash report when the user logs in. So far the only affected OS version is 4.4.2. API versions > 20 log in correctly.
com.google.android.gms.internal.zzdvv.zzb (Unknown Source)
com.google.android.gms.internal.zzdwc.zzb (Unknown Source)
com.google.firebase.auth.FirebaseAuth.signInWithEmailAndPassword (Unknown Source)
>>> com.mypackage.UserManager.login

com.google.android.gms.internal.zzdvv.zzb (Unknown Source)
com.google.android.gms.internal.zzdwc.zza (Unknown Source)
com.google.firebase.auth.FirebaseAuth.createUserWithEmailAndPassword (Unknown Source)
>>> com.mypackage.UserManager.register

@crazyhatfish can you show some of the code you are running that produces this exception?

Sure, I forgot to mention that it occurs when registering and logging in.
this.auth.signInWithEmailAndPassword(items.getString(0), items.getString(1))
                        .addOnCompleteListener(new OnCompleteListener<AuthResult>() {
                            @Override
                            public void onComplete(@NonNull Task<AuthResult> task) {
                                resultCallback.invoke(task.isSuccessful());
                            }
                        });

The code that calls createUserWithEmailAndPassword is identical to this.

@crazyhatfish could you:

Show some more of the crash logs (more is better)
Provide more information about items and resultCallback.

Right now my only guess is that resultCallback could be null when the task completes.  If you us the forma .addOnCompleteListener(this, listener) when adding the listener it will be automatically removed at the end of the Activity lifecycle, which could help.

@crazyhatfish since there hasn't been a response in 19 days I assume you figured this out?  If not let me know and we can re-open this.

Apologies, I've been waylaid on another project at work.
That's as high as the callstack as it goes, the rest of the error log (its a crashlytic error log) is pretty uninformative, although I may be wrong.
The full stack is as follows:
com.google.android.gms.internal.zzdvv.zzb (Unknown Source)
com.google.android.gms.internal.zzdwc.zzb (Unknown Source)
com.google.firebase.auth.FirebaseAuth.signInWithEmailAndPassword (Unknown Source)
com.mypackage.UserManager.login (UserManager.java:118)
java.lang.reflect.Method.invokeNative (Method.java)
java.lang.reflect.Method.invoke (Method.java:515)
com.facebook.react.bridge.JavaMethodWrapper.invoke (JavaMethodWrapper.java:374)
com.facebook.react.bridge.JavaModuleWrapper.invoke (JavaModuleWrapper.java:162)
com.facebook.react.bridge.queue.NativeRunnable.run (NativeRunnable.java)
android.os.Handler.handleCallback (Handler.java:733)
android.os.Handler.dispatchMessage (Handler.java:95)
com.facebook.react.bridge.queue.MessageQueueThreadHandler.dispatchMessage (MessageQueueThreadHandler.java:31)
android.os.Looper.loop (Looper.java:149)
com.facebook.react.bridge.queue.MessageQueueThreadImpl$3.run (MessageQueueThreadImpl.java:194)
java.lang.Thread.run (Thread.java:841)

items is a ReadableArray passed in from React Native JS
resultCallback is a React Native Callback

I've verified that items, the getString return values and resultCallback are not null and it doesn't even get to the callback on an emulated Nexus 10 API 16.

@crazyhatfish thanks for getting back to me.  So if I understand correctly this is an issue in a React Native app only affecting users on KitKat and below.
I will leave this open for now, but as you noted there's not much information here and it doesn't seem that either of us can reproduce this too easily so I'm not quite sure how to proceed with debugging ... I'll see if I can do anything with those logs.

Fwiw I looked into the proguard mapping and all I can see about zzdvv and zzdwc are that they are inside the operations you specified, so it's very hard to figure out where the NPE is coming from.

Thanks for looking at this for me.
Here is a minimal app reproducing the crash: tmpapp.tar.gz
Ran on an emulated Nexus 10 API 16, stock image.
It's a React Native app, wasn't sure if that was a contributing factor. Should be just a npm i, npm start to get it running.

@crazyhatfish ok I was able to reproduce this on an API 16 emulator.

@crazyhatfish ok I installed a newer version of Google Play services on the emulator and this did not crash anymore (see below).  The API 16 emulator came with a version old version (9.x) and there must be some binary incompatibility between the new SDK and that one.
Because the emulator does not have the play store, there's no way for these devices to upgrade Google Play services which causes the crash.
So I would guess that your users have a very old version of Google Play services, or are running on a device that was never meant to have Google Play services.
While Firebase Auth normally does these checks for you, there must be something strange going on on these devices.  The best I can offer you is the GoogleApiAvailability class which would allow you to make some of these checks in your app code and possibly warn the user that their device is not compatible:
https://developers.google.com/android/reference/com/google/android/gms/common/GoogleApiAvailability.html#isGooglePlayServicesAvailable(android.content.Context)

Gonna report this as a bug, but closing it here since there's nothing more I can do :-/

Awesome, that's perfect :100:. Cheers


Circular JSON? Changing Direction

When using the toolbar to change direction, I'm getting this error in the console:
Uncaught SyntaxError: "undefined" is not valid JSON
at JSON.parse ()
at $.Ot [as refresh] (MindElixir.js:953:1)
at $.$t [as initRight] (MindElixir.js:944:1)
at i.onclick (MindElixir.js:835:1)
Ot @ MindElixir.js:953
$t @ MindElixir.js:944
i.onclick @ MindElixir.js:835
Working with MindElixir.js I see that when it gets to the "refresh" function "Ot" that "e" is undefined
I've taken it back to the bare bones example and am still having this issue. Ideas?

I went back to version 2.1.0 to get this to work.
Any call to .refresh() in version 3 and above have the "undefined" error above.  As info.


feat: poke user when supplying faulty configurations

bors r+

This does not seem to work yet hmm


b2: Final fix for link sharing

What is the purpose of this change?

Check if path provided is a valid file or directory.
used a simple list with a response limit of 1 to check if the path is a directory.
update docs for B2 bucket style link behavior.
it passes the integration tests on my end, please test it before merging as i want to get it fully right this time.
Was the change discussed in an issue or in the forum before?
#3327

Checklist

[X] I have read the contribution guidelines.
[X] I have added tests for all changes in this PR if appropriate.
[X] I have added documentation for the changes if appropriate.

[X] I'm done, this Pull Request is ready for review :-)

This doesn't appear to have the fix in we discussed in https://github.com/ncw/rclone/pull/3327#issuecomment-514771727 :confused:
Did you push it, or has github gone bonkers...

Sorry didn't commit everything. Comes from working late at night.
should be fixed.

That looks perfect now - thank you very much :-)


Automatically remove the fi and ff ligatures in Meslo font

Menlo and Meslo both have fi and ff ligatures which cause annoyances
when trying to render something fixed width.  This removes them from the
resulting patched powerline font.

This has been annoying me for weeks! I thought it was just my system messing up the font

This merge really should be accepted. The patched powerline fonts repo still has this issue.

Correct me if I am wrong: U+FB01 is a separate character which should be rendered when U+FB01 codepoint is found in a string, not when there is U+0066 (f) followed by U+0069 (i). For fi there is separate technique named “substitution” which may make fi render as U+FB01 or may make fi render as something different. So to fix the font one should not remove U+FB01 glyph, making it impossible to  render U+FB01 should it appear in text, but remove the substitution.
Also:

Unicode contains more ligatures then just fi and ff.
Fontpatcher adds powerline glyphs, it is none of fontpatcher business to correct mistakes made by font authors. If I was maintaining this repository I would not accept this change unless you say that unpatched font does not have the problem and patching font added it.

@ZyX-I

Unicode contains more ligatures then just fi and ff.
It appears that only "fi" and "ff" are render like a ligature.

Fontpatcher adds powerline glyphs, it is none of fontpatcher business to correct mistakes made by font authors. If I was maintaining this repository I would not accept this change unless you say that unpatched font does not have the problem and patching font added it.
The rendering issue only happens after the fonts are patched.

@ZyX-I

Unicode contains more ligatures then just fi and ff.

It appears that only "fi" and "ff" are render like a ligature.

Fontpatcher adds powerline glyphs, it is none of fontpatcher business to correct mistakes made by font authors. If I was maintaining this repository I would not accept this change unless you say that unpatched font does not have the problem and patching font added it.

The rendering issue only happens after the fonts are patched.

The rendering issue only happens after the fonts are patched.

Hm. If original fonts has this ligatures as well, but does not have this issue, then something else needs to be fixed. I do not know what though.


helidon version fails without Java

helidon version
Looking up latest Helidon version
Looking up latest Helidon version
Java version at /usr/lib/jvm/jdk-11.0.5 is not supported: 11 or later is required.

It should not fail but print a warning and report at least the cli version.
Also note in the output above that it fails to detect jdk11 properly

Fixed in https://github.com/oracle/helidon-build-tools/pull/215


{afd} docs update to fix issue 27309

Resolve https://github.com/Azure/azure-cli/issues/27309

afd


Move /_schemas directory in playground examples into /policies

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Playground directory structure of the example templates look like
.
├── README.md
├── _schemas
│   ├── some_schema.json
│   └── user_attr_schema.json
├── policies
│   ├── derived_roles
│   │   ├── common_roles.yaml
│   │   └── tenant_assignments.yaml
│   └── resource
│       └── some_resource.yaml
└── tests
    └── testdata
        ├── principals.yaml
        └── resources.yaml

while they should look like:
.
├── README.md
├── policies
│   ├── _schemas
│   │   ├── some_schema.json
│   │   └── user_attr_schema.json
│   ├── derived_roles
│   │   ├── common_roles.yaml
│   │   └── tenant_assignments.yaml
│   └── resource
│       └── some_resource.yaml
└── tests
    └── testdata
        ├── principals.yaml
        └── resources.yaml

This is quite misleading. I literally tried an hour to figure out why nothing is working until I read in some PR in the repo that the _schemas dir must be in /policies.
Makes absolute sense, not that I know it, as the store dir is defined as /policies but for someone who literally heard about cerbos 2h ago this wasn't as obvious as it might should have been.
Also, the example in the playground download modal doesn't work.
Expected Behavior
No response
Steps To Reproduce
Use any playground template, download and try to start the server with the provided example command.
Environment
- OS:
- Cerbos version:
- Docker version:
- Go version:

Anything else?
No response

(Recreating as moved by accident - original issue by @jsbrain)

Thanks for reporting @jsbrain - that certainly isn't expected and will look into this now. Which example were you working with?

Thanks we've identified the issue - the configuration file it suggests isn't pointing to the root. A change will be rolling out shortly to correct this.

This change has rolled out. Thanks for reporting it.


add no_std support

I really needed JSON serde support for a no_std project. I managed to add two new features "std" and "no_std" to support both std and no_std/alloc environments. I know that there is a serde_json_core crate, but this crate doesn't offer full json support such as escapes values.
Usually, one only adds one feature for "std" and has that be enabled by default, but in this specific case, I decided to add a no_std feature because I needed to depend on core_io for no_std support (AFAIK there is no way to conditionally disable a dependency with a feature hence the no_std feature).
Due to the way core_io works, this will only compile one some specific compiler versions (the latest one being nightly-2019-07-01)
I should also note that the tests can only be run in std mode. Some of the dev dependencies depend on serde with std enabled and that causes std to be enabled globally. This only affects the tests though, if this crate is used in another crate as a dependency the dev dependencies don't kick in.

almost everything because ser::Serializer, de::Deserializer, to_vec and from_str all depend on io::Write or io::Read
If that's a huge issue we could try rewriting/copying small parts of std::io for no_std mode instead of depending on core_io.

But to_vec and from_str don't require io. It seems easily possible to implement those signatures without std::io, only alloc.
https://docs.serde.rs/serde_json/fn.to_vec.html
https://docs.serde.rs/serde_json/fn.from_str.html
Could you try that approach? We would have only a "std" feature and that feature would be purely additive, adding only those public APIs that can't exist without std::io.

Even Deserializer can be provided without io, only the Deserializer::from_reader method would need to be std-only.

but to_vec and from_str depend on Serializer and Deserializer which require io. Deserializer also strongly depends on std::io. even methods that don't mention io like from_slice internally convert the slice to a reader and use that to create a deserializer
the only way i can see to get rid of core_io is to rewrite/copy parts from std::io

What I mean is that it is possible to implement the exact public API of to_vec and from_str and nearly all of the public API of Deserializer using some alternative implementation when not using std. That's the direction that I would prefer for this to go, rather than using core_io or vendoring parts of core_io.
As far as I can tell none of the implementation of Deserializer requires std::io outside of the from_reader constructor which can be std-only.

I see. I was confusing read::Read with io::Read. I'll give it a try

okay, so that was way easier than I thought it would be. I created an abstraction over Write just like you already did with Read in serde_json::read and used that in all methods. I completely removed the no_std feature

hey, so I fixed all build errors, let me know if there's anything else you'd want me to change

Hi @Freax13 -- are you still interested in continuing to work on this PR?


Add server API

#12901  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


Loading parts DB fails

Describe the bug
Loading the parts DB fails with an exception “… library.py, line 368, … no such column: parts”.
I think the problem is introduced by commit ed495c0. The SQL statement is changed from
query = '''SELECT "LCSC Part" AS lcsc, "Stock" AS stock, "Library Type" AS type FROM parts WHERE parts MATCH ?'''
for number in lcsc:
    match_query = f'"LCSC Part:{number}"'
    cur.execute(query, (match_query,))

to
query = """SELECT "LCSC Part" AS lcsc, "Stock" AS stock, "Library Type" AS type FROM parts WHERE parts MATCH :number"""
cur.execute(query, {"number": number})

This drops the LCSC Part column name.
I think, this should be
query = """SELECT "LCSC Part" AS lcsc, "Stock" AS stock, "Library Type" AS type FROM parts WHERE "LCSC Part" MATCH :number"""

To Reproduce
Steps to reproduce the behavior:

Open JLCPCB-Tools

Expected behavior
The main window should open.
KiCad Version
KiCad 8.0.5
JLCPBC tools 2024.10.01

Can confirm this problem exists. Ran into it on Kicad 8.0 plugin version 2024.10.01, made the change as described by OP, and the window would succesfully open again.

Problem is solved (mostly).
What I did:

Uninstalled plugin
Applied pending uninstall
Checked: No longer there
Installed version 2024-11-01
Applied pending
Open PCB
Open plugin (unrelated error, might be a bug in KiCad)

Error log
--- Logging error ---
Traceback (most recent call last):
  File "C:\Program Files\KiCad\8.0\bin\Lib\logging\__init__.py", line 1113, in emit
    stream.write(msg + self.terminator)
    ^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'write'
Call stack:
  File "C:\Users\xxx\Documents\KiCad\8.0\3rdparty\plugins\com_github_bouni_kicad-jlcpcb-tools\plugin.py", line 28, in Run
    dialog = JLCPCBTools(None)
  File "C:\Users\xxx\Documents\KiCad\8.0\3rdparty\plugins\com_github_bouni_kicad-jlcpcb-tools\mainwindow.py", line 478, in __init__
    self.init_library()
  File "C:\Users\xxx\Documents\KiCad\8.0\3rdparty\plugins\com_github_bouni_kicad-jlcpcb-tools\mainwindow.py", line 497, in init_library
    self.library = Library(self)
  File "C:\Users\xxx\Documents\KiCad\8.0\3rdparty\plugins\com_github_bouni_kicad-jlcpcb-tools\library.py", line 48, in __init__
    self.setup()
  File "C:\Users\xxx\Documents\KiCad\8.0\3rdparty\plugins\com_github_bouni_kicad-jlcpcb-tools\library.py", line 54, in setup
    self.logger.info(
Message: "Data directory 'jlcpcb' does not exist and will be created."
Arguments: ()
2024.11.05 11:24:14 - INFO - setup -  Data directory 'jlcpcb' does not exist and will be created.
2024.11.05 11:24:14 - INFO - setup -  Data directory 'jlcpcb' does not exist and will be created.
2024.11.05 11:24:14 - INFO - setup -  Data directory 'jlcpcb' does not exist and will be created.
--- Logging error ---
Traceback (most recent call last):
  File "C:\Program Files\KiCad\8.0\bin\Lib\logging\__init__.py", line 1113, in emit
    stream.write(msg + self.terminator)
    ^^^^^^^^^^^^
AttributeError: 'NoneType' object has no attribute 'write'
Call stack:
  File "C:\Users\xxx\Documents\KiCad\8.0\3rdparty\plugins\com_github_bouni_kicad-jlcpcb-tools\plugin.py", line 28, in Run
    dialog = JLCPCBTools(None)
  File "C:\Users\xxx\Documents\KiCad\8.0\3rdparty\plugins\com_github_bouni_kicad-jlcpcb-tools\mainwindow.py", line 478, in __init__
    self.init_library()
  File "C:\Users\xxx\Documents\KiCad\8.0\3rdparty\plugins\com_github_bouni_kicad-jlcpcb-tools\mainwindow.py", line 497, in init_library
    self.library = Library(self)
  File "C:\Users\xxx\Documents\KiCad\8.0\3rdparty\plugins\com_github_bouni_kicad-jlcpcb-tools\library.py", line 49, in __init__
    self.check_library()
  File "C:\Users\xxx\Documents\KiCad\8.0\3rdparty\plugins\com_github_bouni_kicad-jlcpcb-tools\library.py", line 72, in check_library
    self.create_rotation_table()
  File "C:\Users\xxx\Documents\KiCad\8.0\3rdparty\plugins\com_github_bouni_kicad-jlcpcb-tools\library.py", line 235, in create_rotation_table
    self.logger.debug("Create SQLite table for rotations")
Message: 'Create SQLite table for rotations'
Arguments: ()
2024.11.05 11:24:14 - DEBUG - create_rotation_table -  Create SQLite table for rotations
2024.11.05 11:24:14 - DEBUG - create_rotation_table -  Create SQLite table for rotations
2024.11.05 11:24:14 - DEBUG - create_rotation_table -  Create SQLite table for rotations

List is empty
Parts DB is downloaded
List is (still) empty
Close plugin
Reopen plugin
List is populated

So the plugin is usable now but there should be a refresh after downloading the parts DB (and the UI should be disabled while downloading the DB).

I noticed that the list is not populated after a DB download but I had no time to look into it so far ...
Good that it works for you otherwise


feat: add find_best_run_in_dir function to model eval

Minor utility function for finding best model.

Hey guys!
Excellent that you're contributing shared code; highly encourage getting this reviewed by me or Lasse before merging into common 👍


Close this useless function

In this directory -----
seq2annotation.trainer.cli_keras---
def classification_report(y_true, y_pred, labels):
......

fixed in the latest master branch, please check.


Error in signif(nevergrad_hp_val[[co]][index], 6) :    non-numeric argument to mathematical function

Project Robyn
Demo.R
Describe issue
Error in signif(nevergrad_hp_val[[co]][index], 6) :
non-numeric argument to mathematical function
Provide reproducible example
Libraries
library(tidyverse)
library(Robyn)
library(reticulate)
set.seed(123)
Force multi-core use when running RStudio
Sys.setenv(R_FUTURE_FORK_ENABLE = "true")
options(future.fork.enable = TRUE)
Create virtual environment
conda_create("r-reticulate")
Use the environment created
use_condaenv("r-reticulate", required = T)
Check simulated dataset or load your own dataset
data("dt_simulated_weekly")
head(dt_simulated_weekly)
Check holidays from Prophet
data("dt_prophet_holidays")
head(dt_prophet_holidays)
Directory where you want to export results to (will create new folders)
robyn_directory <- "MyDemo.RDS"
All sign control are now automatically provided: "positive" for media & organic
InputCollect <- robyn_inputs(
dt_input = dt_simulated_weekly,
dt_holidays = dt_prophet_holidays,
date_var = "DATE", # date format must be "2020-01-01"
dep_var = "revenue", # there should be only one dependent variable
dep_var_type = "revenue", # "revenue" (ROI) or "conversion" (CPA)
prophet_vars = c("trend", "season", "holiday"), # "trend","season", "weekday" & "holiday"
prophet_country = "IT", # input country code. Check: dt_prophet_holidays
context_vars = c("competitor_sales_B", "events"), # e.g. competitors, discount, unemployment etc
paid_media_spends = c("tv_S", "ooh_S", "print_S", "facebook_S", "search_S"), # mandatory input
paid_media_vars = c("tv_S", "ooh_S", "print_S", "facebook_I", "search_clicks_P"), # mandatory.
organic_vars = "newsletter", # marketing activity without media spend
factor_vars = c("events"), # force variables in context_vars or organic_vars to be categorical
window_start = "2016-11-23",
window_end = "2018-08-22",
adstock = "geometric" # geometric, weibull_cdf or weibull_pdf.
)
print(InputCollect)
hyperparameter names are based on paid_media_spends names too. See right hyperparameter names:
hyper_names(adstock = InputCollect$adstock, all_media = InputCollect$all_media)
Guide to setup & understand hyperparameters
Example hyperparameters ranges for Geometric adstock
hyperparameters <- list(
facebook_S_alphas = c(0.5, 3),
facebook_S_gammas = c(0.3, 1),
facebook_S_thetas = c(0, 0.3),
print_S_alphas = c(0.5, 3),
print_S_gammas = c(0.3, 1),
print_S_thetas = c(0.1, 0.4),
tv_S_alphas = c(0.5, 3),
tv_S_gammas = c(0.3, 1),
tv_S_thetas = c(0.3, 0.8),
search_S_alphas = c(0.5, 3),
search_S_gammas = c(0.3, 1),
search_S_thetas = c(0, 0.3),
ooh_S_alphas = c(0.5, 3),
ooh_S_gammas = c(0.3, 1),
ooh_S_thetas = c(0.1, 0.4),
newsletter_alphas = c(0.5, 3),
newsletter_gammas = c(0.3, 1),
newsletter_thetas = c(0.1, 0.4),
train_size = c(0.5, 0.8)
)
InputCollect <- robyn_inputs(InputCollect = InputCollect, hyperparameters = hyperparameters)
OutputModels <- robyn_run(
InputCollect = InputCollect, # feed in all model specification
cores = 11, # NULL defaults to (max available - 1)
iterations = 2000, # 2000 recommended for the dummy dataset with no calibration
trials = 5, # 5 recommended for the dummy dataset
ts_validation = TRUE, # 3-way-split time series for NRMSE validation.
add_penalty_factor = FALSE # Experimental feature. Use with caution.
)
Environment & Robyn version
Make sure you're using the latest Robyn version before you post an issue.

Check and share Robyn version: packageVersion("Robyn") ? 3.10.3
R version (Please, check and share: sessionInfo() or R.version$version.string)
R version 4.3.1 (2023-06-16 ucrt)
Platform: x86_64-w64-mingw32/x64 (64-bit)
Running under: Windows 10 x64 (build 19042)

Matrix products: default
locale:
[1] LC_COLLATE=Spanish_Bolivia.utf8  LC_CTYPE=Spanish_Bolivia.utf8    LC_MONETARY=Spanish_Bolivia.utf8
[4] LC_NUMERIC=C                     LC_TIME=Spanish_Bolivia.utf8
time zone: America/La_Paz
tzcode source: internal
attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base
other attached packages:
[1] reticulate_1.31 Robyn_3.10.3    lubridate_1.9.2 forcats_1.0.0   stringr_1.5.0   dplyr_1.1.3
[7] purrr_1.0.2     readr_2.1.4     tidyr_1.3.0     tibble_3.2.1    ggplot2_3.4.3   tidyverse_2.0.0
loaded via a namespace (and not attached):
[1] gtable_0.3.4        shape_1.4.6         QuickJSR_1.0.5      processx_3.8.2      inline_0.3.19
[6] lattice_0.21-8      callr_3.7.3         tzdb_0.4.0          ps_1.7.5            vctrs_0.6.3
[11] tools_4.3.1         bitops_1.0-7        generics_0.1.3      stats4_4.3.1        parallel_4.3.1
[16] fansi_1.0.4         pkgconfig_2.0.3     prophet_1.0         Matrix_1.6-1        ggridges_0.5.4
[21] rngtools_1.5.2      RcppParallel_5.1.7  lifecycle_1.0.3     compiler_4.3.1      munsell_0.5.0
[26] minpack.lm_1.2-3    codetools_0.2-19    h2o_<IP_ADDRESS>        RCurl_1.98-1.12     yaml_2.3.7
[31] glmnet_4.1-8        crayon_1.5.2        pillar_1.9.0        nloptr_2.0.3        StanHeaders_2.26.28
[36] doRNG_1.8.6         iterators_1.0.14    rpart_4.1.19        foreach_1.5.2       rstan_2.26.23
[41] tidyselect_1.2.0    rvest_1.0.3         zip_2.3.0           digest_0.6.33       stringi_1.7.12
[46] splines_4.3.1       grid_4.3.1          colorspace_2.1-0    cli_3.6.1           magrittr_2.0.3
[51] loo_2.6.0           patchwork_1.1.3     pkgbuild_1.4.2      survival_3.5-5      utf8_1.2.3
[56] withr_2.5.0         prettyunits_1.1.1   rappdirs_0.3.3      lares_5.2.2         scales_1.2.1
[61] timechange_0.2.0    extraDistr_1.9.1    httr_1.4.7          matrixStats_1.0.0   rpart.plot_3.1.1
[66] gridExtra_2.3       png_0.1-8           hms_1.1.3           openxlsx_<IP_ADDRESS>    doParallel_1.0.17
[71] rlang_1.1.1         Rcpp_1.0.11         glue_1.6.2          xml2_1.3.5          pROC_1.18.4
[76] rstudioapi_0.15.0   jsonlite_1.8.7      R6_2.5.1            plyr_1.8.8

Ok. I've installed again all the packages and it worked.


Wrong ID for earch_search with S2_MSI_L1C/S2_MS1_L1A

As noted in https://github.com/CS-SI/eodag/issues/136#issuecomment-808082569, the ID attributed to an S2_MSI_L1C product obtained from earth_search is inconsistent with other providers:
{
   "astraea_eod":"S2A_OPER_MSI_L1C_TL_EPAE_20200815T043159_A026884_T56XMG_N02.09",
   "aws_eos":"S2B_MSIL1C_20200816T023549_N0209_R089_T56XMG_20200816T041557",
   "creodias":"S2A_MSIL1C_20200815T021611_N0209_R003_T56XMH_20200815T043159",
   "earth_search":"S2A_56XMG_20200815_0_L1C",
   "mundi":"S2A_MSIL1C_20200815T021611_N0209_R003_T56XMH_20200815T043159",
   "peps":"S2A_MSIL1C_20200815T021611_N0209_R003_T56XMG_20200815T043159",
   "sobloo":"S2A_MSIL1C_20200815T021611_N0209_R003_T56XMH_20200815T043159"
}

earth_search actually provides the right ID, it"s stored in the attribute features.properties.sentinel:product_id:, see for instance this response.

This is also true for S2_MSI_L1A products (see this response)

As mentioned here https://github.com/CS-SI/eodag/pull/218#issuecomment-812596493, earth_search and astraea_eod have their own id format to keep the ability to search by id on these providers.
The title attribute format  has been updated on these providers with #218, to make the destination path consistent with others providers.


Setting for neighborhood size

For our larger recordings we need the ability to set a smaller neighborhood size for snippet extraction, as otherwise we are running out of RAM.

That makes sense, but I worry about the impact on manual curation time. I can imagine ways that we could propagate labels from one chunk to the next (assuming the sort was done on a longer chunk) but it certainly would be nice to have a single solution that could make it easy to track units through.

On Oct 13, 2021, at 3:35 PM, Jeremy Magland @.***> wrote:
This is actually more difficult than I thought because it involves coordinating the change in a bunch of places in both the front end and the back end. I'd prefer to solve this in a different way by structuring the system to operate on time segments. In my opinion, for larger recordings, it's best not to have a massive file of all the snippets.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/magland/sortingview/issues/155#issuecomment-942770217, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABV4PSPJYS77AOYNK4KAA6DUGYCTVANCNFSM5F3Q75CQ.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I believe this is resolved. Closing for now.


fix(TDI-38700): Azure storage wizard can't save schema which was checked

What is the current behavior? (You can also link to an open issue here)
https://jira.talendforge.org/browse/TDI-38700
the user can finish the wizard at any point and this cause some schema lo be lost
What is the new behavior?
We force the user to go through all wizard pages to preserve the already selected schema
Please check if the PR fulfills these requirements

[x] The commit message follows Talend standard
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[ ] Other... Please describe:

Does this PR introduce a breaking change?

[ ] Yes
[x] No

If this PR contains a breaking change, please describe the impact and migration path for existing applications: ...
Other information:

Pull request test result: success :star:


Revert "WIP: Implement Cholesky-based OPs"

Rely on a github shortcut one time and it merges too soon, sigh..
Reverts JuliaApproximation/ClassicalOrthogonalPolynomials.jl#63

@dlfivefifty The code I merged works as intended but I was intending to run it by you first (note to self, do not trust the Github GUI sync button). We can revert the merge if you prefer.
The next step is hooking SemiClassicalOPs.jl into this functionality, which should be simple enough. I will wait for you to confirm how you want to handle this before starting on SemiClassicalOPs.jl.


Deployments fail due to timing issues with kill/scale of tasks

On the first start of the DCOS package the deployment will fail with a 409 Conflict response because the kill task executes and is still in progress prior to the scaling up to X number of nodes.
This is not a huge deal as the app will restart and go green, but this should be handled more gracefully.

Addressed in b6a390b590d49ab205a6ab27bd0eb20abe7984ca.


Not able to deposit ATOM intp savers

Prerequisites
Asgardex with ATOM
Expected Behavior
Expect deposits to savers to work
Current Behavior
Fails to deposit.
Steps to Reproduce

Have ATOM in Asgardex
Go to wallets page and locate ATOM
Press the "Earn" button for ATOM
Enter an amount to deposit (leaving a balance of 6 ATOM)
Enter wallet password and then press confirm

I thought that would be sufficient, but nothing happens.
I tried to do the same on THORSwap and that also failed :shrug: .
Possible Solution
Your Environment - Release Version (skip if not using a release version)

ASGARDEX desktop version: 1.22.6 from debian package on GH
Operating system and version: Debian Testing

Asgardex is having an issue with adding savers - this is a thornode issue which should be fixed in the next release.
Info:
Issue https://discord.com/channels/838986635756044328/1290719412672004146/1290722424756768899
THORNode Fix https://gitlab.com/thorchain/thornode/-/merge_requests/3764 in v2.137.0
We don't know when thornode v2.137.0 will be released. It should return to normal once the next version is released.

Duplicate of #416


Can't load flutter

After upgrade flutter sdk to 1.12, start my flutter activity, it's black.
#0      _AssertionError._doThrowNew (dart:core-patch/errors_patch.dart:42:39)
#1      _AssertionError._throwNew (dart:core-patch/errors_patch.dart:38:5)
#2      SchedulerBinding.addTimingsCallback (package:flutter/src/scheduler/binding.dart:232:12)
#3      WidgetsBinding.drawFrame (package:flutter/src/widgets/binding.dart:768:33)
#4      RendererBinding._handlePersistentFrameCallback (package:flutter/src/rendering/binding.dart:283:5)
#5      SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:1102:15)
#6      SchedulerBinding.handleDrawFrame (package:flutter/src/scheduler/binding.dart:1041:9)
#7      SchedulerBinding.scheduleWarmUpFrame.<anonymous closure> (package:flutter/src/scheduler/binding.dart:850:7)
#8      _rootRun (dart:async/zone.dart:1122:38)
#9      _CustomZone.run (dart:async/zone.dart:1023:19)
#10     _CustomZone.runGuarded (dart:async/zone.dart:925:7)
#11     _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:965:23)
#12     _rootRun (dart:async/zone.dart:1126:13)
#13     _CustomZone.run (dart:async/zone.dart:1023:19)
#14     _CustomZone.bindCallback.<anonymous closure> (dart:async/zone.dart:949:23)
#15     Timer._createTimer.<anonymous closure> (dart:async-patch/timer_patch.dart:23:15)
#16     _Timer._runTimers (dart:isolate-patch/timer_impl.dart:384:19)
#17     _Timer._handleMessage (dart:isolate-patch/timer_impl.dart:418:5)
#18     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:174:12)

Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, v1.12.13+hotfix.5, on Mac OS X 10.15.2 19C57, locale zh-Hans-CN)

[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 11.3)
[✓] Android Studio (version 3.5)
[✓] VS Code (version 1.40.2)
[✓] Connected device (1 available)

• No issues found!

Hi @elrondliu
could you please run
flutter clean
flutter pub cache repair
If the issue persist please provide your updated
flutter doctor -v and flutter run --verbose
together with a a minimal code sample to reproduce the problem.
Are you using add-to-app?
thank you

I'm sorry I have found our bug, thank you.