text:
Test failure: LoggingConfigurationSectionPassedToLoggerByDefault

This issue was made automatically. If there is a problem contact ryanbrandenburg.
This test failed with the following error:
System.IO.IOException : Failed to bind to address http://127.0.0.1:5000: address already in use.
---- Microsoft.AspNetCore.Connections.AddressInUseException : Only one usage of each socket address (protocol/network address/port) is normally permitted
-------- System.Net.Sockets.SocketException : Only one usage of each socket address (protocol/network address/port) is normally permitted
   at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.AddressBinder.BindEndpointAsync(ListenOptions endpoint, AddressBindContext context)
   at Microsoft.AspNetCore.Server.Kestrel.Core.LocalhostListenOptions.BindAsync(AddressBindContext context)
   at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.AddressBinder.DefaultAddressStrategy.BindAsync(AddressBindContext context)
   at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.AddressBinder.BindAsync(IServerAddressesFeature addresses, KestrelServerOptions serverOptions, ILogger logger, Func`2 createBinding)
   at Microsoft.AspNetCore.Server.Kestrel.Core.KestrelServer.StartAsync[TContext](IHttpApplication`1 application, CancellationToken cancellationToken)
   at Microsoft.AspNetCore.Hosting.Internal.WebHost.StartAsync(CancellationToken cancellationToken)
   at Microsoft.AspNetCore.Hosting.Internal.WebHost.Start()
   at Microsoft.AspNetCore.WebHost.StartWith(String url, Action`1 configureServices, Action`1 app, String applicationName) in /_/src/Microsoft.AspNetCore/WebHost.cs:line 112
   at Microsoft.AspNetCore.WebHost.Start(String url, RequestDelegate app) in /_/src/Microsoft.AspNetCore/WebHost.cs:line 43
   at Microsoft.AspNetCore.WebHost.Start(RequestDelegate app) in /_/src/Microsoft.AspNetCore/WebHost.cs:line 31
   at Microsoft.AspNetCore.Tests.WebHostFunctionalTests.LoggingConfigurationSectionPassedToLoggerByDefault() in /_/test/Microsoft.AspNetCore.FunctionalTests/WebHostFunctionalTests.cs:line 142
----- Inner Stack Trace -----
   at Microsoft.AspNetCore.Server.Kestrel.Transport.Sockets.SocketTransport.BindAsync()
   at Microsoft.AspNetCore.Server.Kestrel.Core.KestrelServer.<>c__DisplayClass21_0`1.<<StartAsync>g__OnBind|0>d.MoveNext()
--- End of stack trace from previous location where exception was thrown ---
   at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.AddressBinder.BindEndpointAsync(ListenOptions endpoint, AddressBindContext context)
----- Inner Stack Trace -----
   at System.Net.Sockets.Socket.UpdateStatusAfterSocketErrorAndThrowException(SocketError error, String callerName)
   at System.Net.Sockets.Socket.DoBind(EndPoint endPointSnapshot, SocketAddress socketAddress)
   at System.Net.Sockets.Socket.Bind(EndPoint localEP)
   at Microsoft.AspNetCore.Server.Kestrel.Transport.Sockets.SocketTransport.BindAsync()
------- Stdout: -------
| [0.001s] TestLifetime Information: Starting test LoggingConfigurationSectionPassedToLoggerByDefault at 2018-10-19T12:36:17
| [0.182s] TestLifetime Information: Finished test LoggingConfigurationSectionPassedToLoggerByDefault in 0.1812432s

Other tests within that build may have failed with a similar message, but they are not listed here. Check the link above for more info.
This test failed on 2.2.
CC Eilon

This comment was made automatically. If there is a problem contact ryanbrandenburg.
Please use this workflow to address this flaky test issue, including checking applicable checkboxes and filling in the applicable "TODO" entries:

Is this actually a flaky test?

No, this is a regular test failure, fix the test/product (TODO: Link to commit/PR)
Yes, proceed below...

Is this test failure caused by product code flakiness? (Either this product, or another product this test depends on.)

[ ] File a bug against the product (TODO: Link to other bug)
Is it possible to change the test to avoid the flakiness?

Yes? Go to the "Change the test!" section.
No?

[ ] Disable the test (TODO: Link to PR/commit)
[ ] Wait for other bug to be resolved
[ ] Wait for us to get build that has the fix
[ ] Re-enable our test (TODO: Link to PR/commit)
[ ] Close this bug

Is it that the test itself is flaky? This includes external transient problems (e.g. remote server problems, file system race condition, etc.)

Is there is a way to change our test to avoid this flakiness?

Yes? Change the test!

[ ] Change the test to avoid the flakiness, for example by using a different test strategy, or by adding retries w/ timeouts (TODO: Link to PR/commit)
[ ] Run the test 100 times locally as a sanity check.
[ ] Close this bug

No?

Is there any logging or extra information that we could add to make this more diagnosable when it happens again?

Yes?

[ ] Add the logging (TODO: Link to PR/commit)

No?

[ ] Delete the test because flaky tests are not useful (TODO: Link to PR/commit)

This comment was made automatically. If there is a problem contact ryanbrandenburg.
There were 1 failures with about the same error on 2.2 at 10:13:55 PM:

LoggingConfigurationSectionPassedToLoggerByDefault

@muratg please assign

@JunTaoLuo could you take a look?

This test shouldn't be listening on the default port 5000, reconfigure it.

authors: Eilon Tratcher aspnet-hello muratg

repo: aspnet/MetaPackages


text:
Inconsistency with Url.Content("~")

In previous versions of ASP.NET MVC, Url.Content("~") returns the application's root directory (ie. /foo). On ASP.NET MVC 6, this just returns a verbatim "~", and you need to actually use Url.Content("~/"). It's a subtle difference, but it appears that older versions just check that the first character is "~", whereas newer versions check that the string starts with "~/" (ie. it contains a tilde and a slash)
Old code:
bool isAppRelative = contentPath[0] == '~';

New code:
if (path.StartsWith("~/", StringComparison.Ordinal))

Thanks! :+1:

source: gharchive/issue

repo: aspnet/Mvc

license_type: permissive

license_source: github-api


text:
Add back StatusDescription to HttpStatusCodeResult

I think everything is in the title.
Previously in MVC HttpStatusCodeResult allowed users to customize the http status message returned to client.

@Tratcher I think you've had thoughts on this before, and concluded that it was not a good idea. Do you recall the reason?

Won't fix. https://github.com/aspnet/HttpAbstractions/issues/395

Done, thanks.

But... David Fowler said:

Closing this out until somebody asks for it.

Just joking. I did not know it was removed from HTTP/2, which is a good reason to obsolete the feature. Also, there is a workaround available in the linked issue so it's not like it's undoable.
BTW my use case was that I was used to signal "expected" errors in a web api (e.g. failed validation) by returning an error http status with friendly explanation in the reason phrase (which was picked up by app and shown to user). But I can do pretty much the same thing using the response body anyway, especially now that I know HTTP/2 removed this.

@jods4 thanks for the extra details! Glad you got it working.

authors: Eilon Tratcher jods4


text:
Include excluded files from project programatically

I am trying to build an application that creates an image by Drawing and saves to Temp folder in server. But that file is excluded from project automatically. How to add this file again as I am unable to use the excluded file.
Error:  File is being used by another process.
My Code:
var p = new Microsoft.Build.Evaluation.Project(@"C:\projects\BabDb\test\test.csproj");                     p.AddItem("Folder", @"C:\projects\BabDb\test\test2");                     p.AddItem("Compile", @"C:\projects\BabDb\test\test2\Class1.cs");                     p.Save();
I suppose this function is not valid when the file is in remote server.
How will I able to resolve this.
Please comment for any update.
Thanks in advance.

In .NET Core projects all files within the project folder are included, unless they are explicitly excluded. Because this is not an MVC issue, I'm closing it. This question might be better suited to https://github.com/microsoft/msbuild/issues.


text:
[Fixes #5212] Added a cookie based ITempDataProvider

@rynowak This is initial implementation with tests working. We still have open questions about the cookie size and how it should be handled.

@davidfowl - we haven't done any perf work on TempData. We should look at it but it's separate from this change

ping

:up: :date:

:up: :date:

source: gharchive/pull-request

authors: kichalla rynowak


text:
Non-english characters are encoded

Every non-english character is encoded. For example:
@{
	string s = "Добрый день"; // this is Russian text
}
@s

The result is something like:

@anfomin are any characters displayed incorrectly in the page? Encoding should be visible only when viewing the HTML source.

@dougbu characters displayed correct, but page size increased significantly :-1: Also MVC 5 rendered unicode symbol correctly in HTML source.

@anfomin one of the benefits of MVC 6 is your ability to change most of the defaults. In this case you prefer a less-strict HTML encoder. To enable that, implement IHtmlEncoder and place it in DI using services.AddInstance<IHtmlEncoder>(new CustomHtmlEncoder()) in your Startup class. (AddInstance is an example; pick the lifetime appropriate for your replacement service.)

@dougbu I know I can replace with custom IHtmlEncoder, this is not a problem. But I thought this is not a feature but a bug because MVC 6 is the only framework that encodes normal Russian symbols (I think problem is not only in Russian language).

FYI @danroth27 moved this to https://github.com/aspnet/HttpAbstractions/issues/315

authors: anfomin dougbu

repo: aspnet/Razor


text:
Consider an overload for TagHelperContent.GetContent() which returns encoded content.

Currently the content which is stored in TagHelperContent.GetContent() is unencoded. The only way to get encoded content is using WriteTo(TextWriter, IHtmlEncoder). Consider having an overload like GetContent(IHtmlEncoder) which returns the encoded content.
Reference PR comment: https://github.com/aspnet/Razor/pull/473/files#r36673441

See discussion here: https://github.com/aspnet/Razor/pull/541/files#r40223087

Close?

@rynowak do we still feel that this is important? It's relatively simple to do.

It's not really that important, especially since we have the overload that lets you pass in the encoder. If you get blocked on this it's easy to work around.

kk, going to close as wontfix since it's an easy mechanism to use/workaround if it causes you trouble.

authors: Eilon NTaylorMullen rynowak sornaks


text:
Getting project information fails if transitive project references have different relative paths

SoutionFile
     |_ src
          |_ RootProject
     |_Libraries
          |_ Library1
          |_ Library2

Library1 has
<ProjectReference Include="..\Library2\Library2.csproj" />

RootProject has
<ProjectReference Include="..\..\Libraries\Library1\Library1.csproj" />
<ProjectReference Include="..\..\Libraries\Library2\Library2.csproj" FromP2P="true"/>

Running dotnet-aspnet-codegenerator -p RootProject.csproj -controller -name EmptyController
Fails to get ProjectContext

Fixed as part of #348

repo: aspnet/Scaffolding

license: MIT


text:
Remove Duplication in Clients while Sending Message

I am having an issue with SignalR here, a client in my application can belong to more than 1 groups. When the client connects I add him in specific groups. Within Group there is no duplication. Now the issue is when the same message is broadcasted among different groups having same client and same connection ID.
This is my code. notificationService is HubContext
await this.notificationService.SendNotificationToGroups(entity.Roles, NotificationConstants.Announcement, NotificationConstants.AnnouncementMessage);
I am using core 2.1. Thanks alot

but If I dont want to add my own uuid is there any specific uuid already present in message? or how I can get list of clients from SignalR groups?

@muzzammilhussain you shouldn't assume that you can. That's implementation specific and there's no available list. You can keep track when you call AddToGroup/RemoveFromGroup.

authors: davidfowl muzzammilhussain

repo: aspnet/SignalR


text:
React to HttpAbstractions

Reacts to https://github.com/aspnet/HttpAbstractions/pull/873
@Tratcher

@HaoK,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

authors: HaoK dnfclas


text:
[Minor] Change mutation mode on Edit pages to pessimistic

Hello,
Actuellement, lorsqu'on édite une ressource, quand on appuie sur le bouton Sauvegarder, il se passe les étapes suivantes :

La page rebascule sur la vue Show de la ressource
Les données de la vue Show sont mises à jour en local seulement
[5 secondes s'écoulent]
La modification de la ressource est enregistrée en base de données
La vue se réactualise (avec potentiellement les anciennes données si la sauvegarde a échoué)

Ce mécanisme (par défaut dans react-admin) dit de "sauvegarde optimiste" est notamment prévu pour annuler aussitôt ses changements (mode undoable) et pour être plus fluide, mais pose également des problèmes UX et génère de la confusion auprès des utilisateurs, par exemple quand ceux-ci quittent la page ou la rechargent de manière trop précipité. Plusieurs cas de données mal sauvegardées ont été remontées sur des instances Archipelago, qui pourraient potentiellement venir de ce workflow complexe.
Je propose de passer en "sauvegarde pessimiste", permettant d'attendre que les modifications de la ressource soient bien enregistrées en base de données avant de basculer de page (comme la plupart des sites actuels). De plus, les modifications de ressources n'entraînant normalement pas une complexité de calcul ni un délai important, ça ne se justifie pas trop ici de faire du rendu optimiste à part faire de la sur-optimisation non-nécessaire.
PR liée (mais pas dépendante) sur Semapps
https://github.com/assemblee-virtuelle/semapps/pull/1209
Le fonctionnement du DeleteButton change avec la sauvegarde pessimiste. Désormais, il affiche une popup de confirmation lors de la suppression d'une ressource (ça permettra d'éviter de supprimer une ressource avec un missclick...)

Effectivement ça peut être plus intuitif comme ça !

repo: assemblee-virtuelle/archipelago


text:
Capability URLs

Capability URLs are another way to handle autorisations. Instead of writing somewhere that "Alice is allow to use Car X", we simply hand over a key to Alice, and this key is the "proof" she can use the car.
With Capability URLs, we could share links with people, giving them the right to edit a given ressource.
The authorisation could be shared through ActivityPub or any other mean of contact.
Implementation
Capabilities could be ressources, and use the same ontology as WebACL.
{
  "@id": "https://myserver.com/capabilities/k3kleict5ks3r4",
  "@type": "acl:Authorization",
  "acl:accessTo": "https://myserver.com/resource/x",
  "acl:mode": "Write"
}

Capability URLs may also indicate who is allowed to use the "key". We could use the acl:agentClass for that.
They can also have an expiration date.
These ressources should not be listed in the LDP container, nor queriable through SPARQL. Maybe they shouldn't be readable by anyone but Moleculer, but in this case, the frontend wouldn't be able to know if a capability URL is valid or not until the POST/PUT request is made (which is not ideal).
The URL of the capability (https://myserver.com/capabilities/k3kleict5ks3r4) could be passed in the headers when we want to edit a ressource. This would be passed in addition to the JWT token.
The validation could be handled by a Moleculer middleware, coming before the WebAclMiddleware.
In the frontend, we could add a dialog to generate capabilities URLs, like it is done in Google docs.

@nikoPLP Qu'est-ce que tu penses de cette proposition ?
Je me tâte pour l'utiliser pour inviter des utilisateurs sur ActivityPods. Grosso modo, lorsqu'on est invité, on pourrait générer une resource de type "Capacity" (une clé) qui donne le droit de temporairement voir le profil de notre "inviteur". Une fois qu'on a accepté l'invitation et qu'on est en contact avec la personne, la ressource est supprimée. On pourrait aussi imaginer qu'elle expire après un temps T.
J'ai l'impression que c'est simple à implémenter (dans notre architecture actuelle) et qu'au niveau ontologique on peut rester proche de l'ontologie WAC existante.

Done

repo: assemblee-virtuelle/semapps


text:
instead of setting location

It would be nice if it could set a flash message or something of that effect.
Much as I would like to, I dont want to cripple users on those older browsers. I just want to display a banner/flash on my app's pages.
What do you think of that option being added to browsernizer?

I've updated the gem to include this feature. Please check updated readme for details on how you can accomplish this.
thanks for tip!

authors: asanghi assembler

repo: assembler/browsernizer

license_source: bigquery


text:
Delete particular conversations from knowledge base

From @ruKurz on June 20, 2017 16:21
There should be an option to delete conversations of bad quality manually. Most likely this process might be support by our KI part.
Sometimes it occurs that somebody has a problem, but at the end he finds the solution himself.
Sometimes somebody just had a small problem, but dig into a small talk via chat.
Sometimes somebody just asked the wrong person or the problem solves itself.
In all of this szenarios we have a conversation of not very helpful messages.
If all of those message will be part of the knowledge base. These questions will overfill the knowledge base with wrong or not needful information.
There must be a way to simply delete a single conversation from the knowledge base.
Or the Knowledge Base itself is able to decide whether a message is valuable or not.
Copied from original issue: mrsimpson/Rocket.Chat#25

From @ruKurz on September 20, 2017 12:14
On Smarti side this Issue is implemented.
Maybe we want to offer an feature within the Smarti Widget for deleting a message by users.

repo: assistify/smarti


text:
Software does not continue to run at proxy check

Software does not continue to run at proxy check
I can still press STOP
version: 1.7.0

@tungtrinh1402
Thank you.
I don't understand why this happens, when developing everything is fine, the framework behaves differently.
I deleted the release, download the previous version, will look for what the problem.

@tungtrinh1402
Thank you. I don't understand why this happens, when developing everything is fine, the framework behaves differently. I deleted the release, download the previous version, will look for what the problem.

The 1.7.1 update works fine.
Using your own proxy is an effective combination https://openproxy.space/dashboard/proxy-list . I've been waiting for your update since 2019

@tungtrinh1402
Thank you. I don't understand why this happens, when developing everything is fine, the framework behaves differently. I deleted the release, download the previous version, will look for what the problem.

You can add share proxy live feature with personal account api + share directly to openproxy's public ip source for me to personally use or share for everyone to buy your Purchase key. it will create a big ip community for you.

@tungtrinh1402
I've been waiting for your update since 2019

appreciate it =))))))))

You can share the API key, then the user can select options in the builder (https://openproxy.space/api - Builder Page)

repo: assnctr/unfx-proxy-checker


text:
[bug] Parentheses around yield statements

🙁 Actual behavior
Original code:
function()
list(function())
return function()

Rewrite function() with yield function():
yield function()
list(yield function())  # invalid syntax
return yield function()  # invalid syntax

🙂 Expected behavior
To generate the correct AST, yield ... should be wrapped in extra parentheses.
yield function()
list((yield function()))
return (yield function())

If the rewrite string is wrapped in (), ast-grep inserts () even in the plain yield statements.
These are not stripped by black.
(yield function())
list((yield function()))
return (yield function())

AST grep is not responsible for generating valid AST after rewrite. That means, you need to take care of if you need to add parentheses or semicolon.
At the moment, there is no plan to support AST aware fix.

authors: HerringtonDarkholme elbaro

repo: ast-grep/ast-grep


text:
What is the minimum python version this codebase requires?

Hi, I am working on bringing in meta-RL into this codebase and re-implementing PEARL here. Since many people are using dennisl88's rand_param_envs, I found that it seems not supporting Python 3.7, but Python 3.5 or Python 3.6.
Could you give me some advice about it?
Is Python 3.6 OK for this codebase as I found out that f-string is only available after Python 3.6?

I'm using Python 3.6.9 and it's totally fine with rlpyt.

authors: ZiwenZhuang codelast

repo: astooke/rlpyt


text:
Suddenly extremely slow?

I've been using google-drive-ocamlfuse for more than a year now, and if I have e.g. a file open in a text editor, it usually saves the file within seconds. But since yesterday, everything takes much longer - I now have to wait 2-3 minutes for a save action to complete. Is there any way of troubleshooting this?

Google has some outage today: https://www.google.com/appsstatus#hl=en&v=issue&sid=4&iid=a45de3b26d6c5872f4cfe8e3424d7a82

That was it! It's fast again - thanks! :-)

repo: astrada/google-drive-ocamlfuse

license: mit


text:
ruff format not producing same results as vs code extension

in my local repo I have these pyproject.toml settings
[tool.ruff]
line-length = 100
indent-width = 4
include = ["*.py", "*.pyi", "**/pyproject.toml", "*.ipynb"]

[tool.ruff.lint]
extend-select = ["D403"]

[tool.ruff.lint.per-file-ignores]
"*.ipynb" = ["E402", "F401"]

[tool.ruff.lint.isort]
force-single-line = true
lines-between-types = 1
order-by-type = false

and produces this when I run ruff format --config=pyproject.toml
https://github.com/astral-sh/ruff/assets/11903445/f05099dd-28fc-4ab9-a1e0-98a5abb8bcee
and also ruff check --fix says "All tests passed!"
whereas my vscode extension points to a ruff.toml which looks like this
line-length = 100
extend-select = ["D403"]
extend-include = ["*.ipynb"]

[isort]
force-single-line = true
lines-between-types = 1
order-by-type = false

[per-file-ignores]
"*.ipynb" = ["E402", "F401"]

and produces this result when I save
https://github.com/astral-sh/ruff/assets/11903445/079eca51-7343-4ae7-afe5-61f43c5edb45
The vs code extension results are the ones I want and I don't know how to achieve them

It looks like in the latter case, Ruff is attempting to sort the imports, whereas in the former it's not. Is that correct? You might have organizeImports enabled in VS Code, which will attempt to organize imports even if the rule isn't selected.

@charliermarsh yes correct, but how do I turn on organizeImports using the CLI?

Change:
[tool.ruff.lint]
extend-select = ["D403"]

To:
[tool.ruff.lint]
extend-select = ["D403", "I"]

I had to also do this
ruff check --fix --config=./pyproject.toml
ruff format --config=./pyproject.toml

but that worked! Thanks @charliermarsh !

authors: alexdauenhauer charliermarsh

repo: astral-sh/ruff


text:
[red-knot] implement re-export conventions for imports

See https://typing.readthedocs.io/en/latest/spec/distributing.html#import-conventions
A normal from foo import bar in module a should not allow another module to do from a import bar; an explicit from foo import bar as bar should be required to specify a re-export.
In the current typing spec, these rules are worded as applicable to all modules. Mypy does apply them in all modules, pyright applies them only in type stubs.
I have a vague memory that these originated as stub-only conventions, but I haven't been able to dig up confirmation.
We need to decide if we will apply these conventions in all modules, or only in stubs, and then implement them accordingly.
In particular this is problematic at the moment because we don't apply these conventions in builtins.pyi, meaning we understand all symbols imported into builtins.pyi (e.g. various typing module symbols) as builtin names.

As you say, it's very important that we apply these semantics in stub files.
For runtime files, my instinct is to say that we should copy the runtime semantics exactly, but have an additional disabled-by-default error code (let's call it no-implicit-reexport) where we warn if you try to import something that's only implicitly re-exported from a module. (I actually believe that's what mypy does?) I think that should work fine for most cases — the only problem I can see is imports inside if TYPE_CHECKING blocks, which won't exist at runtime but which we will assume to exist if we're running without the no-implicit-reexport error code enabled. I'm not sure the unsoundness there is significant, though — I can't remember it ever coming up in the mypy issue tracker.

I could look into this.

Mypy's behaviour is:

Always have --no-implicit-reexport-like behaviour enabled for stub files (this comes from PEP 484)
---no-implicit-reexport applies to source, first party and third party, and is part of --strict
Mypy has per-module configuration options that let you silence this if you use some third party package that doesn't use __all__ or X as X reexports

Random thoughts:

Agreed on never paying double jeopardy, so always infer a type when you issue a warning. mypy and pyright didn't use to do this and both had to change
The check is a little pedantic in that a) when badness actually strikes later you'll usually eventually get a type check error, b) it doesn't match runtime semantics. But it's useful and I have seen it correctly flag things that would later prove to be issues or simplify weird imports
Pyright's first vs third party distinction is interesting. It solves the problem where a third party module doesn't want to do __all__ or re-export (for a while pallets projects held out). With mypy, you need to add a per-module config to solve that problem.
But I think this check's "preventing later badness" is much more useful with third party code than with first party code, since it's much more possible to have version skew with third party code between type check time and use time. So it's useful to have a way to get the strict behaviour everywhere, as with mypy
One thing not discussed is an __init__.py heuristic, see https://github.com/python/mypy/issues/10198. If red-knot wants to encourage this check, e.g. have it on by default, I'd definitely recommend treating __init__.py specially. I suspect this would be enough to obviate the need for first vs third party distinction
One other related behaviour to think through is whether you should fail open or closed on __all__ patterns that red-knot doesn't recognise

Thanks, that's useful!
One note on pyright's first vs third party distinction; I think it works in the opposite way from what you're suggesting. Pyright applies strict re-export rules to imports from third party libraries, but not to imports from your own first-party code. The idea is that this allow library authors more control over their exported api surface.

@charliermarsh are you still planning to work on this?

authors: AlexWaygood MichaReiser carljm charliermarsh hauntsaninja


text:
[ruff] Do not remove parens for tuples with starred expressions in Python <=3.8 RUF031

Summary
In Python versions <=3.8, removing the parentheses in expressions like d[(*foo,bar)] leads to a syntax error. This PR skips the check for incorrectly-parenthesized-tuple-in-subscript (RUF031) in the situation where:

lint.target-version <= 3.8
lint.ruff.parenthesize-tuple-in-subscript = false (which is the default setting)
A tuple is encountered with one of the elements a starred expression

Closes #12776

As far as I understand, it's not possible to have something like **d inside of a subscript call, so I did not try to treat the doubly-starred case.

Converting to draft until we sort out which versions this should apply to (https://github.com/astral-sh/ruff/issues/12776#issuecomment-2277945553)


text:
uv Created Virtual Environment is not Usable by PyCharm

PyCharm will not recognize a virtual environment created by uv. I'm not sure what it's looking for in a virtual environment, but one created by python -m venv .venv is somehow different from one created by uv venv --seed.
I can select the uv created environment, but PyCharm won't use it, save it or load it.

Answering my own issue - uv does not create an Include directory like the Python venv module does. That directory is empty with the python venv, so just creating an empty Include directory in the uv venv allows PyCharm to use the uv virtual environment.

Just confirming, this is on Windows right?

Oh, or not. The thing I'm surprised by is that virtualenv doesn't create an include either, but python -m venv does. Do you see the same problem if you use virtualenv?

Can you give me steps to repro? Specifically, what do you do in the PyCharm UI that isn't working?

Yes, it's on Windows. Interestingly, virtualenv does not fail even without the Include directory.

Repro steps:

In PyCharm, open Settings => Python Interpreter
Click Add Interpreter => Add Local Interpreter
Click Existing then browse to the uv venv (e.g. xxxx.venv\Scripts\python.exe)
Click OK, then click OK again
If PyCharm likes the interpreter, it will read the installed packages and display them. With the uv one, it won't show anything. If you close settings and go back in, there will be no interpreter selected for the project

Thanks, I can reproduce on Windows! (But not macOS.)

Hmm, I actually can't reproduce this though when I use --seed.

Can you try removing the interpreter from PyCharm, re-running with --seed, re-opening PyCharm, and trying again...?

I did that originally and it wasn't working, but now after doing it with a new project it is working. I think I was running into some kind of internal caching in PyCharm so this is probably not a uv issue after all.

Yeah, I see similar flakiness in the caching. I had to remove the interpreter and restart. Anyway, thank you for filing.

authors: bwanshoom charliermarsh

repo: astral-sh/uv


text:
uv-resolver: filter out sibling dependencies in a fork

When a fork is created from a list of dependencies, we were previously
adding all other sibling dependencies to every fork created. But this
isn't actually quite right, since the fork created is always created by
some marker expression. And while it is definitively disjoint from any
directly conflicting dependency specification, it is also possibly
disjoint with other dependencies. For example, as reported in #4414:
dependencies = [
  "anyio==4.4.0 ; python_version >= '3.12'",
  "anyio==4.3.0 ; python_version < '3.12'",
  "b1 ; python_version >= '3.12'",
  "b2 ; python_version < '3.12'",
]

The first two anyio requirements are conflicting with non-overlapping
marker expressions, and so a fork is created. Prior to this commit,
both b1 and b2 would be added to each fork. But of course, b2 is
impossible in the anyio==4.4.0 fork because of disjoint marker
expressions.
So in this commit, we specifically filter out any sibling dependencies
that could find their way into a fork that have disjoint markers with
that fork. We are careful to do this both when a new fork is created
from an existing set of dependencies, and when adding new dependencies
to a fork.
Fixes #4414

I added a packse test here: https://github.com/astral-sh/packse/pull/197


text:
Fix selection of free-threaded interpreters during default Python discovery

Closes https://github.com/astral-sh/uv/issues/8228
e.g., on this branch
❯ uv python install 3.13t 3.13
❯ cargo build
❯ cargo run -q --bin uvx -- --from build python -c "import sys; print(sys.base_prefix)"
/Users/zb/.local/share/uv/python/cpython-3.13.0-macos-aarch64-none
❯ cargo run -q --bin uvx -- -p 3.13 --from build python -c "import sys; print(sys.base_prefix)"
/Users/zb/.local/share/uv/python/cpython-3.13.0-macos-aarch64-none
❯ cargo run -q --bin uvx -- -p 3.13t --from build python -c "import sys; print(sys.base_prefix)"
/Users/zb/.local/share/uv/python/cpython-3.13.0+freethreaded-macos-aarch64-none

and on main
❯ cargo build
❯ cargo run -q --bin uvx -- --from build python -c "import sys; print(sys.base_prefix)"
Installed 3 packages in 12ms
/Users/zb/.local/share/uv/python/cpython-3.13.0+freethreaded-macos-aarch64-none

I want to add more test coverage around this, but I've noticed the free-threaded discovery tests are a bit off as-is and it'll be a bigger task. I think the recent bugs around discovery indicate we should invest more into that test framework.

The failing integration test looks like a real problem with this approach. Will investigate.


text:
Add Go example

Hi, since Go can compile to WASM I thought it might be nice to have it as an example.

Hey and thanks :innocent:
Silly question while my real computer is being repaired, do the tests pass? I take it you tried it?
Code looks good, will merge as soon as I test it works, don't hesitate to ping me if I forget

Also, maybe let's not add a submodule and just add the example directly in the repo

Hi, I just rebased and had to update some paths, now the tests pass again. I'm not sure what you mean by submodule exactly?

Hi, I just rebased and had to update some paths, now the tests pass again. I'm not sure what you mean by submodule exactly?

authors: astrale-sharp lukasMeKo lukasngl

repo: astrale-sharp/wasm-minimal-protocol

license: Unlicense


text:
feat: configurable polling and ws

Pull Request type

Please add the labels corresponding to the type of changes your PR introduces:

Bugfix
Feature
Code style update (formatting, renaming)
Refactoring (no functional changes, no API changes)
Build-related changes
Documentation content changes
Testing
Other (please describe):

What is the current behavior?

Resolves: #NA
What is the new behavior?

Does this introduce a breaking change?

Other information

also can you update the .env.example with the new values please ?

Hey! Spoke to 0xevolve that I dont have much time nowadays, but wanted to open this PR to show what might be nice to have to improve the bot. Feel free take as much inspiration as you want, but unfortunately I wont have time to update my branch for now ..
Re your semantics comments, I'm an absolute rust noob and most code is by Claude, which explains the "wrong" imports etc
Let me know if I can provide more context, but I dont think that I'll find time in the next weeks to update my fork with your comments, sorry about that ❤

repo: astraly-labs/vesu-liquidator


text:
chore: merge main branch to oracle feature branch

re-opening https://github.com/astriaorg/astria/pull/1861 and not squash merging this time.

closing in favour of #1866

repo: astriaorg/astria


text:
Fix ci

closes: https://github.com/astronomer/astronomer-cosmos/issues/1335

Disabled "Run-Integration-Tests-Sqlite" Job and created issue https://github.com/astronomer/astronomer-cosmos/issues/1341 to enable it
Installed eval_type_backport in tests and created an issue https://github.com/astronomer/astronomer-cosmos/issues/1342 to remove it
pin dbt version dbt-core<1.8.9 created issue to unpin it https://github.com/astronomer/astronomer-cosmos/issues/1343
pin dbt version for Kubernetes tests and create issue to unpin it https://github.com/astronomer/astronomer-cosmos/issues/1344

CI success: https://github.com/astronomer/astronomer-cosmos/actions/runs/12026097151/job/33524412817?pr=1337

Run-Integration-Tests-Sqlite is failing because the GitHub action file is from the main branch

repo: astronomer/astronomer-cosmos


text:
Look into submitting adc-streaming for inclusion to conda-forge

We should look into submitting adc-streaming to conda-forge, to make it easy for downstream dependences to install.
This would require creating a feedstock from the current recipe, and understand how we'd want to manage the in-package recipe vs. having it fully in the feedstock.

@mjuric, @spenczar, the current method for determining the version is incompatible with a conda-forge feedstock, since it requires the package to be installed from a git repository and we need to be able to install the package from an archive.
Running command python setup.py egg_info
    fatal: not a git repository (or any of the parent directories): .git
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/private/tmp/pip-req-build-iqs0q977/setup.py", line 93, in <module>
        version=git_version(),
      File "/private/tmp/pip-req-build-iqs0q977/setup.py", line 32, in git_version
        version = check_output(command.split()).decode('utf-8').strip()
      File "/Users/runner/miniforge3/conda-bld/adc-streaming_1616459830588/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_place/lib/python3.6/subprocess.py", line 356, in check_output
        **kwargs).stdout
      File "/Users/runner/miniforge3/conda-bld/adc-streaming_1616459830588/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_place/lib/python3.6/subprocess.py", line 438, in run
        output=stdout, stderr=stderr)
    subprocess.CalledProcessError: Command '['git', 'describe', '--tags', '--long', '--dirty', '--always']' returned non-zero exit status 128.

Hi Patrick, where is the draft feedstock code (to understand the problem better...)?

Here is the meta.yaml recipe: https://github.com/conda-forge/staged-recipes/pull/14349/files
Here are the logs for a failed job in the PR (now closed): https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=294010&view=logs&jobId=6f142865-96c3-535c-b7ea-873d86b887bd&j=e35d4f76-8ff2-5536-d795-df91e63eb9f7&t=fa7b4b17-b6ff-5c9c-8cfc-15f888c92310

Re-submitted to staged-recipes: https://github.com/conda-forge/staged-recipes/pull/16369

Now on conda-forge: https://anaconda.org/conda-forge/adc-streaming

authors: mjuric myNameIsPatrick

repo: astronomy-commons/adc-streaming


text:
Travel Funding for ADASS

Full name
Kelle Cruz
Description
Astronomical Data Analysis Software and Systems (ADASS) conference
Total request
TBD
Itemized budget
TBD
Advance needed
No response

@kelle You can add your travel support amounts to the issue.

repo: astropy/astropy-project


text:
Fix (?) to_pixel in SkyCircle

@keflavich : Can you have a look at this if you have a minute, please?

@astrofrog
any thoughts? :)

Since this is a show stopper, I'll merge now and leave a comment in https://github.com/astropy/regions/issues/13

I think it should be fine - of course, the best test is that if it works, it should be good to go! :)

Define "works" :) I will make a PR to see that a round trip works, and fix the details later, ok?

authors: astrofrog joleroi

repo: astropy/regions


text:
MNT: Drop Python 3.6/3.7 support, update min dependencies, switch to GA for CI

Python 3.6 is now at end of life and is no longer supported.
This PR also adds Python 3.10 to the CI test matrix.

Yes, I can switch to Actions (next PR)!

:+1: from me on the newer changes, but it might be a good idea to ask for trained eyes on this

I've updated this PR to require the following minimum dependencies

Python 3.8+
numpy 1.18+
matplotlib 3.1+
astropy 5.0+ (LTS)

These are identical to the latest astropy LTS release, which will be useful for merging regions into astropy core.
This PR also switches the CI from Azure to GitHub Actions.

It's almost identical to photutils so I know it works! :smile:

authors: keflavich larrybradley


text:
-agentpath mode consumes more code cache memory

I'm using async-profiler 3.0 against Corretto OpenJDK 21.0.2.14.1.
When starting JVMs with -agentpath:libasyncProfiler.so=start,event=itimer,event=alloc,interval=...,alloc=...,jfr,file=..., I'm able to reliably observe a meaningful increase in code cache memory utilization.
JVMs are started with the aforementioned argument and then N seconds after startup I invoke asprof stop to stop the profiler and dump the output file.
JVMs have a segmented code cache and an increase is seen in both profiled and non-profiled nmethods code heaps.
Before async-profiler, the problematic JVM in question consumes ~220 MB of code cache after running for ~30 minutes, putting it very close to the default limit of 240 MB.
With this JVM, startup registration increases profiled and non-profiled nmethods memory from ~86 MB to ~116 MB each.
With and without -agentpath, we see non-profiled nmethods go to 100% eventually. But it is much faster with -agentpath.
With -agentpath, profiled nmethods goes to 100% after a few minutes. Without -agentpath is only hits ~80% utilization.
Non nmethods memory is unaffected.
It certainly appears that attaching/recording on startup induces higher memory usage.
Is this behavior expected?
Do you have any guidance for the impact that async-profiler may have on code cache memory utilization? e.g. should we be setting aside a fixed amount of code cache memory when enabling async-profiler?

Oh, I left out the key detail that invoking asprof collect without -agentpath doesn't appear to incur a noticeable memory increase. The memory increase appears to either be isolated to use of async-profiler with -agentpath or the act of attaching async-profiler early in the JVM's lifetime. I'm still attempting to isolate conditions to reproduce.

Yes, this is somewhat expected. When libasyncProfiler is loaded, either via -agentpath or on the first asprof invocation, it enables -XX:+DebugNonSafepoints flag which forces generation of additional debuginfo alongside the compiled code. Without this flag, profiler may incorrectly map instruction addresses to bytecode indices in the topmost frame thus making stack traces less inaccurate.
The earlier profiler is attached - the more methods are compiled with extra debuginfo, and the larger is the total size of compiled methods. You will likely observe a similar increase in CodeCache utilization, if you start JVM without profiler but with -XX:+DebugNonSafepoints enabled.
BTW, there is an intention to do the same for JFR, see JDK-8307057.
Currently, there is no workaround other than increasing CodeCache capacity. What I can probably offer is to skip setting DebugNonSafepoints automatically when user explicitly disables it in the command line. However, as mentioned above, this will have a downside of producing inaccurate stack traces (at least, in their topmost inlined frames).

Thank you for the quick and detailed reply!
I added -XX:+DebugNonSafepoints and the shape/behavior of code cache memory looks nearly identical to the async-profiler -agentpath scenario. So your intuition that this extra debuginfo is eating the memory appears spot on.
I'll leave this issue open in case you want it to track docs improvements or something. But from my perspective the issue is resolved.
Thanks again.

I pushed the fix to avoid modifying DebugNonSafepoints when it is set in the command line.
With this fix, adding -XX:-DebugNonSafepoints JVM argument will prevent async-profiler from consuming more CodeCache memory (for the price of less accurate stack traces).

authors: apangin indygreg

repo: async-profiler/async-profiler


text:
Why does the version number jump from '0.1.0' to '0.99.x'? Is it misleading? Where does this confidence come from? This does not conform to the tradition of Rust！

This project (and Rust) follow semantic versioning. This often causes confusion, since many people believe that pre-1.0 indicates unstable software. However, in semantic versioning, pre-1.0 only indicates that the public API may change. (See points 4 and 5 here: https://semver.org/ -- particularly, "Version 1.0.0 defines the public API.")
@stjepang carefully notes that the library is very close to a stable API in the release post. "[async-std] will soon provide a stable interface to base your async libraries and applications on. While we don't promise API stability before our 1.0 release, we also don't expect to make any breaking changes." [1]
Aside from this, note that the versioning speed could also be justified (relative to other Rust libraries) because async-std is "mimicking [the] standard library's well-understood APIs as closely as possible." [1] This alone indicates that a stable public API might be achievable earlier than in other projects, since it must conform to the standard library's pre-existing public API.
[1] https://async.rs/blog/announcing-async-std/#

authors: danclive danwilhelm

repo: async-rs/async-std


text:
added travis-ci status badge

... and removed obsolete reference to app from the readme

@phillbarber - this is a minor / low-risk change. Only the readme file has been changed. Thus, I am going ahead and merging.


text:
🛑 Wedding HTTPS is down

In 10a1c2c, Wedding HTTPS ($WEDDINGHTTPS) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Wedding HTTPS is back up in dcc3814.

repo: atatkin/milos-uptime


text:
🛑 Wedding HTTPS is down

In 22f99b2, Wedding HTTPS ($WEDDINGHTTPS) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Wedding HTTPS is back up in 2f2f8e9.


text:
🛑 Wedding HTTPS is down

In 69ebf05, Wedding HTTPS ($WEDDINGHTTPS) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Wedding HTTPS is back up in 3f77078.


text:
🛑 Wedding HTTPS is down

In 59a7217, Wedding HTTPS ($WEDDINGHTTPS) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Wedding HTTPS is back up in 8d5f09b.


text:
Consider extending TerminateUserSessionResult type to include associated IP Address with Session ID

Running into a specific case today where having that bundled in would prove useful. Thus far the workflow is to gather the user sessions (which contains Username, IP Address and Session ID values) and then calling the Terminate() method. The results from that method leave out of the IP Address value previously associated with the Session ID.
The client code can build this support in, but including this information would be helpful.

Going to embed the UserSession value instead of picking out individual fields. I think this will offer the greatest flexibility and hopefully reduce bugs later down the line.


text:
MessageSent events not being sent via webhook

We're noticing that a good portion of delivery (MessageSent) events are not being sent via webhook, with these events being left in the webhook_requests table.  The problem has been getting worse every day.  I've added more detail in my comments in #265.

This continues to happen... only about 25% of MessageSent events now are being sent to the webhook.

Now it's about 15% of these webhook events being delivered.  Is anyone else experiencing this problem?

@adamcooke This problem continues to spiral.  Now, Postal is only delivering about 10% of the MessageSent events to our webhook... piling up the unsent ones in the webhook_requests table.  Any idea on how to fix this?  Thanks.

Now we're noticing a problem with 1000's of emails backing up in the Queue, hours old, in a Pending state.  Perhaps related to the above problem.  Is anyone at atech available to look at this?  @catphish ?

Here is my Postal's status (below).  Probably unrelated, but should I be bumping up the number of workers?  I'm pushing about 60K emails/day right now.
Procodile Version   1.0.17
Application Root    /opt/postal/app
Supervisor PID      2347
Started             2017-08-19 23:18:48 -0700

|| web
|| Quantity            1
|| Command             bundle exec puma -C config/puma.rb
|| Respawning          5 every 3600 seconds
|| Restart mode        usr1
|| Log path            none specified
|| Address/Port        none
|| => web.1            Running      19/08/2017   pid:2353     respawns:0      port:-        tag:-

|| worker
|| Quantity            1
|| Command             bundle exec ruby script/worker.rb
|| Respawning          5 every 3600 seconds
|| Restart mode        start-term
|| Log path            none specified
|| Address/Port        none
|| => worker.1         Running      19/08/2017   pid:2355     respawns:0      port:-        tag:-

|| cron
|| Quantity            1
|| Command             bundle exec rake postal:cron
|| Respawning          5 every 3600 seconds
|| Restart mode        term-start
|| Log path            none specified
|| Address/Port        none
|| => cron.1           Running      19/08/2017   pid:2357     respawns:0      port:-        tag:-

|| smtp
|| Quantity            1
|| Command             bundle exec rake postal:smtp_server
|| Respawning          5 every 3600 seconds
|| Restart mode        usr1
|| Log path            none specified
|| Address/Port        none
|| => smtp.1           Running      19/08/2017   pid:2359     respawns:0      port:-        tag:-

|| requeuer
|| Quantity            1
|| Command             bundle exec rake postal:requeuer
|| Respawning          5 every 3600 seconds
|| Restart mode        term-start
|| Log path            none specified
|| Address/Port        none
|| => requeuer.1       Running      19/08/2017   pid:2362     respawns:0      port:-        tag:-```

Probably unrelated, but should I be bumping up the number of workers?

Almost definitely related. If you are queuing more messages than one worker process can handle in a timely manner (i.e. you see lots of pending) then you should increase the worker count in postal.yml like so https://github.com/atech/postal/issues/173#issuecomment-302656698

Restarting with 10 workers did seem to resolve the Pending email issue, but it hasn't resolved the main issue I reported here - the MessageSebt events not being delivered via webhook.
A couple odd things about the new Pending email issue: we've been sending about 60k emails per day for weeks now, with just the 1 worker, and hadn't noticed any issues like this.  Also, when restarting Postal, it took considerable time for it to shutdown.  I ended up having to send kill signals to the procodile and worker processes.  I've had similar issues with restarts before.  Could these improper shutdowns have ramifications (perhaps helping to explain the MessageSent issue)?

To further illustrate how the pending email problem seemed to come out of nowhere, it appears something has been consuming unusual amounts of CPU in the past couple days.  Here is the 1-min load avg over the past month (on our dedicated 8-vcpu host):

Here's the corresponding avg CPU graph.  I believe that inflection point around Aug 19 where the pattern changes, was the last time I restarted Postal (no config changes then, I believe) before last night (when I added more workers).  This all seems like a ton of CPU to be in use, for a SMTP application.  Anyone know how I can debug this to pinpoint where the hotspots in the code are?
Prior to Aug 19 or so, we had been warming up Postal gradually.  Since then, the volume of emails has been stable (meanwhile those unprocessed MessageSent events keep piling up in webhook_requests, with 0 attempts -- now has > 1M rows).

Interestingly, I've noticed that more MessageSent events are being delivered after bumping the number of workers from 1 to (now) 8... but it's still no where near 100%, as it was originally... it's now around 42%.  Better than 7% which it had gotten down to, but still a far cry from where it should be.

So I'd love to fix this myself, and contribute back... but I've been trying to decipher the source and have struggled to identify where in the source that webhook_requests records are getting inserted with an attempts value of 0.  My ruby knowledge is minimal (more of a Java/ObjC developer).  I would've thought it would be in the WebhookRequest class, but I seem to be missing it.  The deliver method appears to only insert records with an attempts value greater than 0.  Any hints on where I should be looking would be greatly appreciated @adamcooke @catphish?

I've actually 'fixed' this by changing this line:
https://github.com/atech/postal/blob/2f0155747c5c058afb83856ce5c3c8d5f8cad341/app/models/webhook_request.rb#L46
To:
where("retry_after < ? OR (retry_after IS NULL AND created_at<SUBTIME(NOW(), '0:30:0.000000'))", Time.now).limit(100).each(&:queue)
I added the limit(100) as we've ran into a backlog of roughly 900.000 webhook requests and I didn't want to overload anything. I added the created_at check so it would only requeue webhook requests that are older than 30 minutes (and thus were never actually queued).
This is not a permanent/ideal fix, but it did temporarily solve our problem with the webhook requests.

That works
On Sep 12, 2017 6:05 AM, "alexroosenstein" notifications@github.com wrote:

I've actually 'fixed' this by changing this line:
https://github.com/atech/postal/blob/2f0155747c5c058afb83856ce5c3c8
d5f8cad341/app/models/webhook_request.rb#L46
To:
where("retry_after < ? OR (retry_after IS NULL AND
created_at<SUBTIME(NOW(), '0:30:0.000000'))", Time.now).limit(100).each(&:
queue)
I added the limit(100) as we've ran into a backlog of roughly 900.000
webhook requests and I didn't want to overload anything. I added the
created_atcheck so it would only requeue webhook requests that are older
than 30 minutes (and thus were never actually queued).
This is not a permanent/ideal fix, but it did temporarily solve our
problem with the webhook requests.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/atech/postal/issues/330#issuecomment-328806387, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AMMA50KIp7kIA2OdjMpLvM2ZJOYewsiEks5shldygaJpZM4O-9gG
.

@alexroosenstein Thanks so much for the suggestion!  It seems to be doing the trick.  I had pondered that requeue_all method a bit, but was primarily stuck on trying to understand the underlying cause of the missed events.  Perhaps someone else can enlighten us.

@alexroosenstein You should consider contributing that fix back to the repo.  Even if it doesn't address the underlying issue of why some MessageSent events aren't even being attempted, I think it makes sense for that requeue_all to be picking those records up too.

@alexroosenstein One issue I just noticed with your SQL is that it's assuming that NOW() will be in the same timezone as the one used in created_at.  As it is, my Postal server is apparently using UTC times in that column, while NOW() returns the time in my timezone (PDT)... therefore it'll take about 6.5 hours before it's picked up.

...I've changed the line to this, and it seems to do the trick.
where("retry_after < ? OR (retry_after IS NULL AND created_at<SUBTIME(?, '0:30:0.000000'))", Time.now, Time.now).limit(100).each(&:queue)

I honestly did not (could not?) test that part properly, as I'm still processing webhook requests from a week ago.. Thanks for the updated code, I'll give that a go as well.

For what it's worth, I had over a month of webhook backlog (1M+) myself... I opted to bump your limit clause to 1000.  It just finished processing the backlog after about 24 hours, with minimal impact to ongoing email flow.  After it caught up, I've reverted the limit to 100, as I suspect that'll be enough in the steady state.

If you want to do date comparisons in SQL, use UTC_TIMESTAMP() instead of NOW(), as the name suggests, it will use the UTC timestamp, which is how Rails and Postal store database timestamps.

@catphish That was my assumption, that Time.now must be defaulting to UTC.

In the case of using ? and Time.now Rails knows to convert the timestamp to UTC before putting it in the query, so this works fine too.

authors: alexroosenstein catphish mackinra shopglobal willpower232

repo: atech/postal


text:
Having issue to integrate postal with clamav

I already setup clamav.
The problem is whenever I set
clamav:
enabled: true
host: 127.0.0.1
port: 2000
In my log /var/log/mail.log
I will get this error amavis[2261]: (02261-01) SMTP: 500 5.5.2 Error: bad syntax; detail you can see on my screenshot attach.
When this happen virus will be send to email without being filter/block.
I'm stuck. Hope I can get help from you guys.
Thank you.

You are using amavis but not pure clamav. This is the cause.

Ok will change to pure clamav. Thank you very much.

authors: igerzog wackoen


text:
🛑 FEL is down

In 01506d0, FEL (https://fel.celtech.com.gt/index.jsp) was down:

HTTP code: 0
Response time: 0 ms

Resolved: FEL is back up in 6fd18de after 11 hours, 28 minutes.

repo: atiniguar/celtech-status


text:
chore/CAMPTASKS-442  - Convert Definition Analysis to use a ProjectionVec

This refactors code for definition analysis checking for headers within objects.
Also, clippy made me do some refactoring as well 🙂

This looks good in terms of semantics. Out of curiosity, have you noticed if there were any slowdowns/increased memory with this PR? If not, I'll merge this PR, and we can deal with optimizing it later. If yes, one change you could try would be to have a two hashmaps, one for lvalues with nonempty projections, and one with just DefId,VarId pairs.

I did not benchmark the efficiency in this change. I didn't notice any degradations that were visible, but I was only running it on a small app. I can make the change for non empty projections and one with projections in this PR if you'd like.

I'll merge it for now, and we can check perf later.

Once you rebase it, I'll merge it in.

repo: atlassian-labs/FSRT


text:
Building the docker image fails

Describe the bug
I tried to build the docker image as described in the README and I get an error:
$ docker build -t connect-security-req-tester .
Sending build context to Docker daemon  736.3kB
Step 1/9 : FROM python:3.9-slim
 ---> 4af03d07a3c9
Step 2/9 : RUN pip install pipenv
 ---> Using cache
 ---> b5babb911705
Step 3/9 : RUN apt update && apt install build-essential -y
 ---> Using cache
 ---> d71077849165
Step 4/9 : RUN useradd -m app
 ---> Using cache
 ---> 06856c0b3c09
Step 5/9 : USER app
 ---> Using cache
 ---> cdfd4c6b6d08
Step 6/9 : WORKDIR /app
 ---> Using cache
 ---> a6cb74d42ab0
Step 7/9 : COPY . /app
 ---> Using cache
 ---> 923418f9f7e2
Step 8/9 : RUN pipenv install --system
 ---> Running in 26ef623827ca
Traceback (most recent call last):
  File "/usr/local/bin/pipenv", line 8, in <module>
    sys.exit(cli())
  File "/usr/local/lib/python3.9/site-packages/pipenv/vendor/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/pipenv/cli/options.py", line 58, in main
    return super().main(*args, **kwargs, windows_expand_args=False)
  File "/usr/local/lib/python3.9/site-packages/pipenv/vendor/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python3.9/site-packages/pipenv/vendor/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/local/lib/python3.9/site-packages/pipenv/vendor/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python3.9/site-packages/pipenv/vendor/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/pipenv/vendor/click/decorators.py", line 84, in new_func
    return ctx.invoke(f, obj, *args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/pipenv/vendor/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/pipenv/cli/command.py", line 233, in install
    do_install(
  File "/usr/local/lib/python3.9/site-packages/pipenv/routines/install.py", line 61, in do_install
    ensure_project(
  File "/usr/local/lib/python3.9/site-packages/pipenv/utils/project.py", line 26, in ensure_project
    if project.s.PIPENV_USE_SYSTEM or project.virtualenv_exists:
  File "/usr/local/lib/python3.9/site-packages/pipenv/project.py", line 252, in virtualenv_exists
    if os.path.exists(self.virtualenv_location):
  File "/usr/local/lib/python3.9/site-packages/pipenv/project.py", line 449, in virtualenv_location
    self._virtualenv_location = self.get_location_for_virtualenv()
  File "/usr/local/lib/python3.9/site-packages/pipenv/project.py", line 274, in get_location_for_virtualenv
    return str(get_workon_home().joinpath(self.virtualenv_name))
  File "/usr/local/lib/python3.9/site-packages/pipenv/utils/shell.py", line 195, in get_workon_home
    os.makedirs(expanded_path, exist_ok=True)
  File "/usr/local/lib/python3.9/os.py", line 215, in makedirs
    makedirs(head, exist_ok=exist_ok)
  File "/usr/local/lib/python3.9/os.py", line 215, in makedirs
    makedirs(head, exist_ok=exist_ok)
  File "/usr/local/lib/python3.9/os.py", line 225, in makedirs
    mkdir(name, mode)
PermissionError: [Errno 13] Permission denied: '/home/app/.local'
The command '/bin/sh -c pipenv install --system' returned a non-zero code: 1

Not sure what info would be relevant, but just in case:
$ uname -a
Linux dev.qotilabs.com 5.4.0 #1 SMP Mon Mar 16 15:39:59 MSK 2020 x86_64 x86_64 x86_64 GNU/Linux
$ docker --version
Docker version 20.10.25, build 20.10.25-0ubuntu1~20.04.1
$ git log -1
commit 8bd13456dd102a045c31f55957885e527c5122d4 (HEAD -> main, origin/main, origin/HEAD)
Merge: 70574c0 5faba6a
Author: sgandrathi <117855901+sgandrathi@users.noreply.github.com>
Date:   Thu Aug 10 08:14:45 2023 -0700

    Merge pull request #66 from atlassian-labs/authnchecks

    Authnchecks

Hi @bogdanb, thank you for raising this bug. I tried building the docker image on x86_64 arch Linux image and was unable to reproduce the above permissions failure with same commands. Please check your user permissions to debug this issue.
Regarding publishing images to DockerHub, we don't have any plans as of now. However, we will look into it and announce it in case we publish an image.

authors: bogdanb sgandrathi


text:
pipx install broken?

❯ pipx reinstall sup
uninstalled sup! ✨ 🌟 ✨
  installed package sup 0.1.0, installed using Python 3.12.3
  These apps are now globally available
    - sup
done! ✨ 🌟 ✨

❯ sup
Traceback (most recent call last):
  File "/Users/scoward/.local/bin/sup", line 5, in <module>
    from sup.main import app
  File "/Users/scoward/.local/pipx/venvs/sup/lib/python3.12/site-packages/sup/main.py", line 4, in <module>
    from widgets.run_list import RunList
ModuleNotFoundError: No module named 'widgets'

Fixed by https://github.com/atmandhol/sup/commit/98f32095c4fc4c16d1f9df62cc8cf6541ea64b53

authors: atmandhol idoru

repo: atmandhol/sup


text:
🛑 Dont Tread On My Site Services is down

In 5ca7fc5, Dont Tread On My Site Services ($DONT_TREAD_ON_MY_SITE) was down:

HTTP code: 403
Response time: 483 ms

Resolved: Dont Tread On My Site Services is back up in 63fb169 after 10 minutes.

repo: atmovantage/atmostatus


text:
Revert #446 changes

Requirements for Contributing Documentation

Fill out the template below. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.
The pull request must only contribute documentation (for example, markdown files or API docs). To contribute other changes, you must use a different template. You can see all templates at https://github.com/atom/.github/tree/master/.github/PULL_REQUEST_TEMPLATE.

Description of the Change
Reverts #446.

Release Notes
N/A

I prefer the current design because the more important buttons come first. The badges are not useful for most people.

What? @ElectronicsArchiver said it's ugly, so I made this. So I don't agree why.

Most people want to download Atom and use it (used to be two million people). We only have a handful of developers who build Atom.

Well maintainers have the last word..
Good thing that this is only a maintenance
project and Atoms UI has long been finished~

Most people want to download Atom and use it (used to be two million people). We only have a handful of developers who build Atom.

And thats why the building button is on the same level as the installation button?
Not a great argument if you ask me..

I prefer the current design because the more important buttons come first. The badges are not useful for most people.

@aminya The badges are useful because they guide you to helpful links.

I also prefer the current design fwiw, just because it looks nicer to me.
The badges don't seem smushed together at all, in fact having them together feels more connected?
I think maybe it's just the size of everything; the banner then the h1 then the badges seems more pleasing than putting badges between header and banner, and in general more important things like the title go before subtitle/content:

Also the badges just happen to be in a funnel shape.
Maybe the ugly thing about it is the spacing? (header->badge smaller than badge->badge whitespace)

Note that my opinion isn't very strong on this; it isn't very clear to me that this is an improvement, unlike say, regular links to badges.
I'll just let consensus build about this though, since I don't feel like giving more effort on the readme

You could also just put them before the banner and 'out of the way', thus bringing the focus on the important things.

repo: atom-community/atom


text:
Windows 10 - Bash (Ubuntu) | How to run Atom build with bash command?

Prerequisites

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

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

Description
I got a lot of problems when I try to run commands on Atom to "bash.exe". No matter what extension I use.
Steps to Reproduce

Enable "Bash Ubuntu" for Windows 10.
Setup compiler for my application on "Bash Ubuntu".
Test from "bash.exe" and "powershell.exe". Everything work ok.
Install Atom.
Create an runner on Atom.
Get msg "Erro: 0x80070057"

Expected behavior: Expected the same result as Powershell:

Actual behavior: But I  got this error:

Reproduces how often: always happen.
Versions
Atom    : 1.15.0-beta0
Electron: 1.3.13
Chrome  : 52.0.2743.82
Node    : 6.5.0
apm  1.15.3
npm  3.10.5
node 4.4.5 x64
python
git 2.9.0.windows.
visual studio 2015
Windows 10 Pro (with all lastest updates)
Additional Information
I got the same problem with any extension I use.
I wrote an question at Stack Overflow too:
http://stackoverflow.com/questions/42121927/windows-10-bash-ubuntu-how-to-run-atom-build-on-bash
I found two things can help

Disable Legacy Mode on Powershell (https://github.com/Microsoft/BashOnWindows/issues/7)
Use Powershell ISE (https://github.com/Microsoft/BashOnWindows/issues/430)

We support building Atom on Windows using our native Windows environment only.
There is no support no plans to support compiling the Linux version of Atom on Windows using WSL.

:/ ah, ok. Thanks, anyway.

authors: damieng thenets

repo: atom/atom


text:
Open file should just open file chosen and not load folders in tree view

Prerequisites

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

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

Description
Open file should just open file chosen and not load folders in tree view
Steps to Reproduce

Start atom.
Open file (File -> Open File...) that resides in a folder that has sub folders.

Expected behavior:
Open file chosen for opening and nothing else.
Actual behavior:
Opens file and loads folder and sub folders in tree view.
Reproduces how often:
Always - 100%.
Versions
1.20.0

Thanks for taking the time to contribute!
We noticed that this is a duplicate of https://github.com/atom/atom/issues/4298 which was in turn closed as a duplicate of https://github.com/atom/atom/issues/10517, a high level issue about the different ways to start Atom. You may want to subscribe there for updates if you'd like.
Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.

authors: kathenas rsese


text:
Grey layer over whole application

This is so weird and I really don't know what's the problem.

I am using Atom for some time now but this never happened before. I am on Xubuntu 21.10. Any idea what I could try to figure out the problem? The app is always greyed out after startup. So there is not much to say how to reproduce the issue. Otherwise the editor works normal which means I can edit files, select menus etc. So there is only this grey layer over the whole application. I have this issue with Atom version 1.58.0 x64.

This seems not to be a problem specifically for atom because I have the same issue with other electron apps!


text:
Tab Length Detection

Default setting for tab length is 2 spaces.

When I open a file that uses 3 spaces for indentation, and press Tab, I expect editor to insert 3 spaces (currently 2)
When I open a file that uses 4 spaces for indentation, and press Tab, I expect editor to insert 4 spaces (currently 2)

There already were discussions to allow custom settings per language or per project, although this issue is do detect indentation automatically on file open, because there can be different guidelines for same language files on same project. It is also handy when editing unrelated files.
Default 'tab width' setting is to be still used when creating new files and for \t display width.

Here is a screenshot from proprietary Sublime Text (some of these features are for atom/whitespace, but I think initial detection should be handled in core.

@muchweb I've been tampering with Atom to make the way indentation is displayed separate from the way it is saved to a file.
So if you have a indentation width set to "4 characters" - and you load a file with tab width of 2, the loaded file, you see this:

Planning to make it so it can automatically set the indentation width to match in the event you don't really have a width preference.
Just thought I'd gather some thoughts :-)

@muchweb I've been tampering with Atom to make the way indentation is displayed separate from the way it is saved to a file.
So if you have a indentation width set to "4 characters" - and you load a file with tab width of 2, the loaded file, you see this:

Planning to make it so it can automatically set the indentation width to match in the event you don't really have a width preference.
Just thought I'd gather some thoughts :-)

@driskell What you've done looks absolutely fantastic. Is it anywhere near stable? If so, I'd love to try it out and give you my thoughts.

It is stable. I'll do some rebasing and update to latest atom though. I've been testing other patches lately.

Awesome, I look forward to testing it out.

The specific code that detects the tab length when using spaces I'll be raising as a PR here soon too so at least your original request would be implemented hopefully. Just I have another PR to fix some other issue and I don't like to raise more than 1 at once - but if that doesn't go forward in next few days I'll raise it anyway.
My changes for the enforced width preference so even 2 spaces "appears" as 4-width indentation - that's a bigger change and probably contentious - likely leave it till last :-)

@driskell is it also possible to align the cursor to non-existent indentation in empty line?
Typically the Atom (using whitespace package) removes the indentation from empty lines, such as:
function something() {
    if (something) {
        // ...
    }

    aboveLineIsEmpty();
}

If I move cursor to line above the "aboveLineIsEmpty()" cursor jumps to zero column because it was emptied by whitespace plugin. It's fine to leave the line empty, but cursor should still have indentation when I go to it, and the indentation should appear magically when I start typing there.
If I understand your effort, it can add phantom indentations and could also be used to this.

I'm going to raise a PR for detecting tab length as well as tab type when opening a file. I was waiting for my current one to merge or progress but it has completely stalled so guess I'll just do this one alongside.
@Ciantic That's a good thought. I'll ponder on that soon. I like the term "phantom whitespace" btw :-) Probably not new but first time I've heard it!

Related: #11049

Any updates on this? Its not great having to change my global indentation settings when opening a project with a different tab length :(

Just as a reminder from #11049 , there is a great article on medium about code indent detection: https://medium.com/firefox-developer-tools/detecting-code-indentation-eff3ed0fb56b

@stale how about you stop closing perfectly good issues to fake statistics, and maybe work on code to actually fix them.

authors: Ciantic JordiOrlando alexandernst driskell lee-dohm marcoms muchweb


text:
JavaScript files do not open new tab

I am running Version 0.182.0 of Atom on Mac OS X 10.10, and I have encountered a bug where opening a JavaScript file does not open the file in a new tab, which I would assume is the expected behavior. To the best of my knowledge, only JavaScript files with the .js extension are affected.

@Eagerestwolf Listing the package you've installed would be very helpful.

What would be most helpful is if you could include repro steps. Every way I've tried to open a JavaScript file has succeeded in opening it in a tab. Also, could you give the following information?

Can you reproduce the issue after completely exiting Atom and restarting using atom --safe?

@steelbrain
List of Packages installed

Stylus
linter
linter-coffeelint
linter-csslint
linter-jscs
linter-jshint
linter-less
linter-htmlhint (disabled)
linter-scss-lint (disabled)
linter-tidy (disabled)

@lee-dohm Opening the application in safe mode does open a new tab, I am therefore closing this as a package related bug.

authors: Eagerestwolf lee-dohm steelbrain


text:
Add Electron, Node, Chromium, V8 Versions to Home Page

This adds a script and small section to the front page to display what the latest version of Electron is and what versions of Node, Chromium and V8 it's using :computer:
Currently it'll just be manually run when we do a release (or maybe just minor releases) but :soon: I'll put this and the docs and releases data into a single thing that's run after a hook :fishing_pole_and_fish:

Looks good :+1:

Cool Tricks Cool Tricks, here's how to get that info via electron-prebuilt:
https://github.com/electronjs/electron-rebuild/blob/master/src/main.js#L44

@paulcbetts I don't get it :confused:

@jlord If you spawn electron with the ATOM_SHELL_INTERNAL_RUN_AS_NODE env var set, it'll act like node, so you can have it just do stuff and exit, like print process.versions.modules to output

@paulcbetts ah, I see, I see. Good to know!

vâng...tuyệt vời

authors: jlord kevinsawicki paulcbetts teobtcvn


text:
Crash Reporter not working on WINDOWS 8 with NODE 0.12

Crash Reporter is unable to send the notification to the URL mentioned in the start() once the process is being crashes ( via process. crash()).
This includes : https://gist.github.com/zcbenz/9c75ccda4f925faf92ca
https://github.com/hokein/electron-sample-apps/tree/master/crash-report
This same thing is working fine on UBUNTU. Can anyone please see if this is really on issue or there is specific problem with the code mentioned in the above URL's.
Alternatively we are receiving the following error too -> ERROR:crash_reporter_win.cc(75)] Cannot initialize out-of-process

@panbhatt we did some mock test to see whether crash reports are generating. we are able to generate them. will upload a git sample so you could test see if it  works.

Check this out https://github.com/kamesh-a/electron-samples/tree/master/CrashReporter
sample log in mac, windows8 also we tested
Spec :
node --version : v2.3.3 ( iojs )
npm --version : 2.11.3
node 12.7 has same module version should be equivalent to this version.
Hope this helps !!!
crashreporter =============== START
 Headers :  { host: '127.0.0.1:3000',
  'content-type': 'multipart/form-data; boundary=---MultipartBoundary-sezuGs3Jd5YbbtEU0cg4wIbwJy12OkjM---',
  accept: '*/*',
  'transfer-encoding': 'Chunked',
  cookie: 's_pers=%20s_fid%3D27123FF0A10925B1-3B1EDD6A1790CD5C%7C1479058412411%3B%20s_vs%3D1%7C1415901812411%3B%20s_nr%3D1415900012412-New%7C1447436012412%3B',
  'user-agent': 'crashpad_handler (unknown version) CFNetwork/673.3 Darwin/13.4.0 (x86_64) (MacBookPro11%2C3)',
  'accept-language': 'en-us',
  'accept-encoding': 'gzip, deflate',
  connection: 'keep-alive' }
****** Parsing formdata *****
{ fields:
   { _companyName: 'GitHub, Inc',
     _productName: 'crash-report-demo',
     _version: '0.1.0',
     guid: '690dbc1e-33c3-419c-924a-d5dc81f5bb4d',
     platform: 'darwin',
     process_type: 'renderer',
     prod: 'Electron',
     ver: '0.30.2' },
  files:
   { upload_file_minidump:
      { domain: null,
        _events: {},
        _eventsCount: 0,
        _maxListeners: undefined,
        size: 237472,
        path: '/var/folders/wk/qvmk0tkj34d2lgn42rsdy_1m0000gn/T/upload_94a1794ed7cf2114937ebcbe0bbf71ee',
        name: 'a253b92e-d292-4047-afa7-2ed4f912fe1d.dmp',
        type: 'application/octet-stream',
        hash: null,
        lastModifiedDate: Wed Aug 05 2015 15:12:49 GMT+0530 (IST),
        _writeStream: [Object] } } }
 crashreporter =============== END

@kamesh-a Tried to clone your repo and start it with "electron ." . sorry to say it is not working. here is my env details
npm : 2.11.3
node : V0.12.7
Errror while starting the program>
[87248:0805/113829:ERROR:crash_reporter_win.cc(75)] Cannot initialize out-of-process crash handler
[96688:0805/113830:INFO:renderer_main.cc(200)] Renderer process started
[71596:0805/113830:INFO:renderer_main.cc(200)] Renderer process started
on click on Crash in the HTML window i receive the following.

I am running windows 8.1. seems like there is some problem in starting CRASH HANDLER whilethe app is start. or may be i am doing something which is wrong here.

I'm also having problems with the crash-reporter on Windows. I tried both Windows 10 and Windows 7 with node v0.12.7 + electron v0.32.2 and the simple crash-reporter-test gist: https://gist.github.com/zcbenz/9c75ccda4f925faf92ca.
The crash is not reported:
 node .\node_modules\electron-prebuilt\cli.js .
[1040:0914/154400:INFO:renderer_main.cc(200)] Renderer process started

Here's what it looks like when it does work with the same code and versions of node and electron on Mac:
node ./node_modules/electron-prebuilt/cli.js .
[6579:0914/153922:ERROR:file_io.cc(30)] read: expected 40, observed 0
[6583:0914/153922:INFO:renderer_main.cc(200)] Renderer process started
{ _companyName: 'GitHub, Inc',
  _productName: 'crash-reporter-test',
  _version: '',
  guid: '0045a572-db57-45fc-9abd-d4eff6ea4536',
  platform: 'darwin',
  process_type: 'renderer',
  prod: 'Electron',
  ver: '0.32.2' }
[0914/153923:ERROR:http_transport_mac.mm(186)] The network connection was lost. (NSURLErrorDomain -1005)

Ok, so after looking at the crash-reporter code, it looks like this is user error on my part. On Windows the crash reporter spawns a new process based on the path of the original process, so you shouldn't run electron with node cli.js that comes with electron-prebuilt.
When I use the electron.exe executable, crash reports work. @panbhatt You might want to see if you were doing the same thing I was.

Ok, some more debugging... The 32-bit version works, but not the 64-bit version.
Also, despite what the documentation says, you need to start crash-reporter in the renderer process on Windows too.

Appears that the crash reporter being broken on 64-bit windows is a known issue: https://github.com/atom/electron/blob/f1787d747afbfcc9e6c416b298d7bf32a366241d/spec/api-crash-reporter-spec.coffee#L19

Reopening this since is broken again after the upgrade to Chrome 50.

authors: kamesh-a panbhatt thughes zcbenz

repo: atom/electron


text:
Use the textured window if it's transparent

Fixes #1665 and maybe #1661 too.
If the window's transparent or frameless then it's safer for backwards compatibility if we use the textured window style.

Thanks!

:metal:

authors: joshaber zcbenz


text:
Syntax Highlighting Incorrect for HAML files that contain an underscore in the name.

The syntax highlighting system is treating HAML filenames that have an underscore in the name as HTML.  Removing the undercores causes the HAML file to be rendered properly in the editor.

Oddly, it started working again.

repo: atom/language-gfm


text:
Missing commit? What's going on

I just had a problem using the Document invite, because apparently one of the Commits is missing.
Error opening https://atomicdata.dev/invites/1?public-key=dlOABAaykKn7F5KTOYK3gOuxSiMOLzeljtNLmcnUWNs%3D
Unable to get commits for https://atomicdata.dev/document/pnf5jlv3r2f. Resource not found. Failed to retrieve locally: 'https://atomicdata.dev/commits/Ony7Xa5pm3pdQHJrXIhSu+HUrVeGuBQ7FzzbzRBJTmZclPHUah1h35lRGwqn4dAlgML+KGyGSYM8WgV1AeKQBw=='. Resource not found. Resource https://atomicdata.dev/commits/Ony7Xa5pm3pdQHJrXIhSu+HUrVeGuBQ7FzzbzRBJTmZclPHUah1h35lRGwqn4dAlgML+KGyGSYM8WgV1AeKQBw== not found

For now I've simply created a new Resource and pointed the existing invite to there, but I don't know what has caused this.

Commits cannot be deleted by users, it is not permitted
Commits are only indexed after they are stored (or.. are they?)

A possible culprit is the get_commits_for_resource function, which uses a tpf query to find all Commits. For now I can skip over missing commits, but it still begs the question how this can happen.

repo: atomicdata-dev/atomic-data-rust


text:
Tracking issue - remove deprecated code

This is a tracking issue for the removal of deprecated code.

[ ] Remove support for embedded-hal v0.2
[ ] Remove deprecated items in sercom::dma
[ ] Optionnally, when #635 lands, remove the spi_future module - Would appreciate @bradleyharden's input on this.

I wouldn't remove embedded-hal v0.2 just yet. It can still be useful. Maybe hiding it behind a feature for now would be enogh.

Fine by me - out of curiosity, what would you consider a good timeline to start removing it?

I just looked into it further and I think it's not as bad as I thought. I knew some things were missing in v1, because they couldn't settle on an API. But I think they were sufficiently useless in their current state that they probably weren't used anyway.
Some drivers might still use v0.2, but those could be easily adapted (if they are still maintained...) or you could use embedded_hal_compat.
So it's probably safe to remove v0.2 with the next minor version bump.

authors: Sympatron jbeaurivage

repo: atsamd-rs/atsamd


text:
Add missing documentation - at_backupkey_flutter (Scores Tab)

Scores tab is missing documentation for:

[ ] support platforms
[ ] support sound null safety

Hi @sarika01 we need to review this very old ticket for accuracy, potentially close it and divert to the new pub.dev publishing issue. Creating new issue cards today.
Please tag the correct person if you are the wrong package maintainer.  Both Sameeraja and Shris are gone.


text:
feat: unit test onboarding

- What I did
Fixes #694
- How I did it
- How to verify it
- Description for the changelog

@sachins-geekyants Can you review this PR please?

authors: gkc sonle-geekyants


text:
ci: Replace Syft with sbomify

Progresses https://github.com/atsign-foundation/operations/issues/139
- What I did
Replaced syft with sbomify to generate higher quality SBOMs
- How I did it
Repeated pattern from at_python SDK
- How to verify it
Will need a release of both Dart and Python
- Description for the changelog
ci: Replace Syft with sbomify

@cpswan is this ready for review?

authors: XavierChanth cpswan

repo: atsign-foundation/noports


text:
undefined reference to `xkb_utf32_to_keysym'

trying building on ubuntu linux 20.04
git cloned main repo
i enter the folder then:

medon build    ok (i had to install libxkbcommon-dev)
ninja -C build   failed

ninja: Entering directory build' [6/7] Compiling C object 'wtype@exe/main.c.o'. ../main.c: In function ‘get_key_code_by_wchar’: ../main.c:178:21: warning: implicit declaration of function ‘xkb_utf32_to_keysym’ [-Wimplicit-function-declaration]   178 |  xkb_keysym_t xkb = xkb_utf32_to_keysym(ch);       |                     ^~~~~~~~~~~~~~~~~~~ [7/7] Linking target wtype. FAILED: wtype  cc  -o wtype 'wtype@exe/main.c.o' -Wl,--as-needed -Wl,--no-undefined -Wl,-O1 -Wl,--start-group protocol/libclient_protos.a /usr/lib/x86_64-linux-gnu/libwayland-client.so /usr/lib/x86_64-linux-gnu/libwayland-cursor.so -lrt /usr/lib/x86_64-linux-gnu/libxkbcommon.so -Wl,--end-group '-Wl,-rpath,$ORIGIN/protocol' -Wl,-rpath-link,/home/v1p3r/src/wtype/build/protocol /usr/bin/ld: wtype@exe/main.c.o: in function get_key_code_by_wchar':
main.c:(.text+0x6fd): undefined reference to xkb_utf32_to_keysym' /usr/bin/ld: wtype@exe/main.c.o: in function main':
main.c:(.text.startup+0x265): undefined reference to `xkb_utf32_to_keysym'
collect2: error: ld returned 1 exit status
ninja: build stopped: subcommand failed.

I suppose your dependencies are too old, and don't define the xkb_utf32_to_keysym function.
For Ubuntu 20.04, I checked out this commit, which made the program compile successfully: 6280f6eb59a7586d6d01b8fa4e4100880c6ae8c6

authors: allegfede larslue

repo: atx/wtype


text:
Got rid of AKManager and reintroduced AKSettings

This is a big one which changes and simplifies the central AKManager class, renaming it to the more intuitive AudioKit and moving it away from a singleton model. Now all can be done with class methods and properties. Playgrounds and examples have been updated accordingly.
AKSettings is back, also providing a namespace to update global settings for AudioKit. Had to patch up a lot of classes that used fixed 44100Hz sample rate for instance. Note that there are a few Soundpipe files that should probably take advantage of this, but C doesn't have access to this Swift class currently.
Also AKSettings will see more functionality soon (such as triggering stuff when values are changed), right now this is mostly refactoring the rest of the library to use its values.
Finally, I also took the liberty to bump the version to 3.0.1, although wondering if 3.1 might be more appropriate if we break the API.

I think it will be 3.0.1.  There haven't been any functional changes, just better code.  This is quite a PR for sure!

Truisms related to why I prefer programming AudioKit in Swift (even if the inner workings need to cooperate with AVFoundation announced at WWDC14).
AudioKit makes it easier to access the power to create — music, synthesis, signal
“Programs must be written for people to read, and only incidentally for machines to execute.”
― Harold Abelson http://www.goodreads.com/author/show/5409448.Harold_Abelson, Structure and Interpretation of Computer Programs http://www.goodreads.com/work/quotes/871745
“The computing scientist’s main challenge is not to get confused by the complexities of his own making.”
― E. W. Dijkstra
~ Robert D. Thompson

On Jan 28, 2016, at 10:55 PM, Aurelius Prochazka notifications@github.com wrote:
I think it will be 3.0.1. There haven't been any functional changes, just better code. This is quite a PR for sure!
—
Reply to this email directly or view it on GitHub https://github.com/audiokit/AudioKit/pull/273#issuecomment-176560959.

authors: aure loopbum megastep

repo: audiokit/AudioKit


text:
Multi Class ?

Hi,
Very cool work! I wonder how feasible it is to generate images with multiple classes together? For instance, "monkey bird dog" or "person car cat"
Imagine the insanity!

I've actually done some experiments with generating multiple classes together, see this blogpost. I've been meaning to put the code for the examples out here, will try to do so in the next days. :)

Nice post! I long for the day when a AI assisted painting tool exists that allows toy to paint novel scenes in the style of Picasso or Van Gough; I have a feeling we are close! I look forward to checking out the ipython notebooks!

I put out the code for generating multiple classes together now. Enjoy!

repo: auduno/deepdraw


text:
Update scala-library to 2.13.7

Updates org.scala-lang:scala-library from 2.13.3 to 2.13.7.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-lang", artifactId = "scala-library" } ]

labels: library-update, early-semver-patch, semver-spec-patch

Superseded by #345.

repo: augustjune/canoe


text:
web font not found

字体路径都没有错，还是提示没找到路径呢~

请提供有价值的信息。

authors: aui fairzhu

repo: aui/font-spider


text:
[Aura-Safe][Staking] Not validate delegate number field

Environment

Browser: Chrome
Wallet: Keplr
Environment: https://safe.serenity.aura.network/

Steps to reproduce

Go to https://safe.serenity.aura.network/
Connect wallet
Select validator
See error

21/10
Verified- FAIL

Verified. FAIL

[ ] Error when allow to delegate with invalid amount

Vẫn input được .e886

Verified.PASS

repo: aura-nw/Aura-Safe


text:
feat(scrolling): Prevent body scrolling and allow to scroll only dial…

…og itself
This should fix #31

@EisenbergEffect CLA check

@PWKad Confirmed. You are good to go.

authors: EisenbergEffect Mordred PWKad

repo: aurelia/dialog


text:
bluebird (promise framework) generates warning

Hi!
We've started using bluebird. By simply adding import 'bluebird'.
But bluebird generates a console warning that is a bit "annoying". I think it's in the aurelia code: https://github.com/aurelia/templating-router/blob/master/src/route-href.js#L45 The promise doesn't return anything. It should at least return null, see: https://github.com/petkaantonov/bluebird/blob/master/docs/docs/warning-explanations.md#warning-a-promise-was-created-in-a-handler-but-was-not-returned-from-it
The warning is this:
Warning: a promise was created in a handler but was not returned from it     at RouteHref.processChange (http://localhost:9000/jspm_packages/npm/aurelia-templating-router@1.0.0-rc.1.0.0/route-href.js:63:45)     at RouteHref.bind (http://localhost:9000/jspm_packages/npm/aurelia-templating-router@1.0.0-rc.1.0.0/route-href.js:45:12)     at Controller.bind (http://localhost:9000/jspm_packages/npm/aurelia-templating@1.0.0-rc.1.0.0/aurelia-templating.js:3370:24)     at View.bind (http://localhost:9000/jspm_packages/npm/aurelia-templating@1.0.0-rc.1.0.0/aurelia-templating.js:1433:24)     at Controller.bind (http://localhost:9000/jspm_packages/npm/aurelia-templating@1.0.0-rc.1.0.0/aurelia-templating.js:3362:19)     at Controller.automate (http://localhost:9000/jspm_packages/npm/aurelia-templating@1.0.0-rc.1.0.0/aurelia-templating.js:3307:12)     at RouterView.swap (http://localhost:9000/jspm_packages/npm/aurelia-templating-router@1.0.0-rc.1.0.0/router-view.js:209:38)     at http://localhost:9000/jspm_packages/npm/aurelia-templating-router@1.0.0-rc.1.0.0/router-view.js:169:15

I've pushed the fix suggested here. However, this isn't likely to fix it fully because there are a number of ways that Aurelia uses promises correctly that simply confuse bluebird. You may want to consider configuring bluebird to ignore those warnings. You can do it like this:
Promise.config({
  warnings: {
    wForgottenReturn: false
  }
});

Great! Thanks a lot!

authors: EisenbergEffect henebb

repo: aurelia/templating-router


text:
feat(icons): add slot element

allows custom svg icon to be set as a content
resolves #118

The issue I noticed is when both icon attribute and content is set e.g:
<ux-icon icon="add">
  <svg viewBox="0 0 24 24">
    <path d="M14.97,18.95L12.41,12.92C11.39,14.91 10.27,17 9.31,18.95C9.3,18.96 8.84,18.95 8.84,18.95C7.37,15.5 5.85,12.1 4.37,8.68C4.03,7.84 2.83,6.5 2,6.5C2,6.4 2,6.18 2,6.05H7.06V6.5C6.46,6.5 5.44,6.9 5.7,7.55C6.42,9.09 8.94,15.06 9.63,16.58C10.1,15.64 11.43,13.16 12,12.11C11.55,11.23 10.13,7.93 9.71,7.11C9.39,6.57 8.58,6.5 7.96,6.5C7.96,6.35 7.97,6.25 7.96,6.06L12.42,6.07V6.47C11.81,6.5 11.24,6.71 11.5,7.29C12.1,8.53 12.45,9.42 13,10.57C13.17,10.23 14.07,8.38 14.5,7.41C14.76,6.76 14.37,6.5 13.29,6.5C13.3,6.38 13.3,6.17 13.3,6.07C14.69,6.06 16.78,6.06 17.15,6.05V6.47C16.44,6.5 15.71,6.88 15.33,7.46L13.5,11.3C13.68,11.81 15.46,15.76 15.65,16.2L19.5,7.37C19.2,6.65 18.34,6.5 18,6.5C18,6.37 18,6.2 18,6.05L22,6.08V6.1L22,6.5C21.12,6.5 20.57,7 20.25,7.75C19.45,9.54 17,15.24 15.4,18.95C15.4,18.95 14.97,18.95 14.97,18.95Z" />
  </svg>
</ux-icon>

I get ugly error
aurelia-logging-console.js:47 ERROR [app-router] TypeError: Cannot read property 'insertBefore' of null
    at ShadowSlot.addNode (aurelia-templating.js:895)
    at Function.distributeNodes (aurelia-templating.js:1142)
    at Function.distributeView (aurelia-templating.js:1105)
    at View.bind (aurelia-templating.js:1415)
    at Controller.bind (aurelia-templating.js:3394)
    at View.bind (aurelia-templating.js:1406)
    at Controller.bind (aurelia-templating.js:3394)
    at Controller.automate (aurelia-templating.js:3339)
    at ready (router-view.js:155)
    at RouterView.swap (router-view.js:184)

I was looking for how to detect such case and log meaningful error message , but couldn't find a nice way. Maybe similar approach as described below could be used.
https://ilikekillnerds.com/2017/07/checking-view-slot-defined-aurelia/

I will be adding a comment on the icon issue for my thoughts going forward, but I do like the idea of having easy custom icons like this.

authors: ZHollingshead dudluk

repo: aurelia/ux


text:
Draft custom color scheme.

Handing over to @alexbndk for branding and new logos.

@artob @paouvrard Tried to push my changes but it looks like I don't have permission. Could you double check please?

Here's how it will look like:

authors: alexbndk paouvrard


text:
getIdTokenClaims returns undefined with react-router v6.0.0-beta.4

Describe the problem
I'm using auth0-react v1.6.0 with react-router v6.0.0-beta.4 (using only Chrome)
The response from getIdTokenClaims is always returned as undefined, but with lower versions of react-router (for example v6.0.0-beta.3), it seems to be working (getting a full response with the user's details).
What was the expected behavior?
I should get a full response from getIdTokenClaims that's not undefined.
Reproduction
This is the code I'm using-
import { useAuth0 } from "@auth0/auth0-react"

const { getIdTokenClaims, logout } = useAuth0()

const authLink = setContext(async (_, { headers }) => {
    try {
        return getIdTokenClaims().then((res: any) => {
        //res is always undefined here
        }
    }
...

Closing this due to no activity. Feel free to reply and we can reopen if needed.

Hi @frederikprijck,
I'm experiencing the same issue.
These are my deps
    "@auth0/auth0-react": "^1.5.0",
    "react": "^17.0.1",
    "react-router": "^6.0.0-beta.6",
    "react-router-dom": "^6.0.0-beta.6",

Just by poking around, I noticed the issue appears only when the Auth0 API token expiration is less than 60 seconds.

I hope this helps.

The expiration should be higher than 60 as our SDK uses an internal leeway of 60. If u set it to 60 or lower, u will never be considered authenticated.
It's good to use a short lived AT, but should probably still be a couple of minutes to avoid getting too chatty.

authors: frederikprijck shani-upstream svaponi


text:
Does JDK1.6 support?

I want to use this package in JDK1.6 environment, which version supports it?
I have the following error when using it in JDK1.6:
java.lang.UnsupportedClassVersionError: JVMCFRE003 bad major version; class=com/auth0/jwt/JWT, offset=

Hello @gitaxin, this library currently supports all Java 8, 11 & 17. We currently do not support Java 1.6 as many of the APIs we have are not supported in it.

allright,Thanks!

authors: gitaxin poovamraj


text:
🛑 Andreas Iten is down

In 904e6bd, Andreas Iten (https://www.andreasiten.ch) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Andreas Iten is back up in 21072a7.

repo: authentiqdigital/upptime