Common problem with all your documentation

THis is true for all your code samples...  More often than not this leaves me searching for which Library I have to use...

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

ID: baed064b-f762-0197-aea8-5597efea9eba
Version Independent ID: a15d1fb8-485f-bd7a-7771-7cf8767fe735
Content: How to: Draw an Existing Bitmap to the Screen - Windows Forms .NET Framework
Content Source: dotnet-desktop-guide/framework/winforms/advanced/how-to-draw-an-existing-bitmap-to-the-screen.md
Product: dotnet-framework
Technology: dotnet-winforms
GitHub Login: @adegeo
Microsoft Alias: adegeo

Hello. Sorry for the inconvenience. This is a design decision we made because of the following reasons:

Usings/Imports adds extra code lines that require extra introduction and explanation.

The links in the article to the types (such as Bitmap) go to the API reference which instantly tells you which namespace and library:

Modern versions of Visual Studio and other IDEs include tip bubbles that search for and import the correct namespaces.

However, newer tutorial content will generally always include the namespaces. And if the article is complex, the author may add the namespace, but at their discretion.

Hello. Sorry for the inconvenience. This is a design decision we made because of the following reasons:

Usings/Imports adds extra code lines that require extra introduction and explanation.

The links in the article to the types (such as Bitmap) go to the API reference which instantly tells you which namespace and library:

Modern versions of Visual Studio and other IDEs include tip bubbles that search for and import the correct namespaces.

However, newer tutorial content will generally always include the namespaces. And if the article is complex, the author may add the namespace, but at their discretion.

The particular page I flagged, vs did not have a suggestion because it required a nuget package which was beyond vs to suggest...
The lack of a suggestion on the page as to what I needed to do led to quite a bit of extra searching which I feel could have been avoided if you have a policy of including just a little extra information on your part.
Get Outlook for Androidhttps://aka.ms/ghei36

From: Andy De George<EMAIL_ADDRESS>Sent: Monday, January 4, 2021 8:02:26 PM
To: dotnet/docs-desktop<EMAIL_ADDRESS>Cc: Mick Lord<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [dotnet/docs-desktop] Common problem with all your documentation (#164)
Closed #164https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2Fdocs-desktop%2Fissues%2F164&data=04|01||6af7da87812a4dd996ce08d8b0eba964|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637453873484526739|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=NQqmi5eyeZGrgTqPqZKXIiLpXjsmVRPlWdlj1ms%2Fkwc%3D&reserved=0.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2Fdocs-desktop%2Fissues%2F164%23event-4166507705&data=04|01||6af7da87812a4dd996ce08d8b0eba964|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637453873484536694|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=KTV1poGc6x2vCS6ewLfC4%2FgIsie9rxiVrrXzKTTFOOk%3D&reserved=0, or unsubscribehttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAOHN4G4AH2TDWSO6QHBXD7DSYINFFANCNFSM4VSUWQ5Q&data=04|01||6af7da87812a4dd996ce08d8b0eba964|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637453873484536694|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=NYa0b%2FgQesa5n%2BCyNgKVK1gEkxFNytTQXsfn6b4T79E%3D&reserved=0.

The particular page I flagged, vs did not have a suggestion because it required a nuget package which was beyond vs to suggest...
The lack of a suggestion on the page as to what I needed to do led to quite a bit of extra searching which I feel could have been avoided if you have a policy of including just a little extra information on your part.
Get Outlook for Androidhttps://aka.ms/ghei36

From: Andy De George<EMAIL_ADDRESS>Sent: Monday, January 4, 2021 8:02:26 PM
To: dotnet/docs-desktop<EMAIL_ADDRESS>Cc: Mick Lord<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [dotnet/docs-desktop] Common problem with all your documentation (#164)
Closed #164https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2Fdocs-desktop%2Fissues%2F164&data=04|01||6af7da87812a4dd996ce08d8b0eba964|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637453873484526739|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=NQqmi5eyeZGrgTqPqZKXIiLpXjsmVRPlWdlj1ms%2Fkwc%3D&reserved=0.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2Fdocs-desktop%2Fissues%2F164%23event-4166507705&data=04|01||6af7da87812a4dd996ce08d8b0eba964|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637453873484536694|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=KTV1poGc6x2vCS6ewLfC4%2FgIsie9rxiVrrXzKTTFOOk%3D&reserved=0, or unsubscribehttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAOHN4G4AH2TDWSO6QHBXD7DSYINFFANCNFSM4VSUWQ5Q&data=04|01||6af7da87812a4dd996ce08d8b0eba964|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637453873484536694|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=NYa0b%2FgQesa5n%2BCyNgKVK1gEkxFNytTQXsfn6b4T79E%3D&reserved=0.

@micklord49 Interesting, it shouldn't require any NuGet package. What package did you add?

@micklord49 Interesting, it shouldn't require any NuGet package. What package did you add?

I was adding the imaging package for .net core which is standard in .net
From: Andy De George<EMAIL_ADDRESS>Sent: 12 January 2021 18:08
To: dotnet/docs-desktop<EMAIL_ADDRESS>Cc: Mick Lord<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [dotnet/docs-desktop] Common problem with all your documentation (#164)
@micklord49https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicklord49&data=04|01||2a71e647f9d64eb1c58a08d8b724f330|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637460716609201871|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=%2F8Xn8CefusQjt7y%2BXKfL56sm%2FDZmxhki6OBBhWuF2sc%3D&reserved=0 Interesting, it shouldn't require any NuGet package. What package did you add?

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2Fdocs-desktop%2Fissues%2F164%23issuecomment-758839293&data=04|01||2a71e647f9d64eb1c58a08d8b724f330|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637460716609201871|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=rvzycKEfXsnsmNxY1x1HnF3F%2B7IoMq5TN8%2B%2BfXIcBzc%3D&reserved=0, or unsubscribehttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAOHN4G4OEAPO6HYRYBCNMU3SZSFWXANCNFSM4VSUWQ5Q&data=04|01||2a71e647f9d64eb1c58a08d8b724f330|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637460716609201871|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=EDa79v3O6p71l6PfG6nSErsG097enkWq2B1oDXogr7M%3D&reserved=0.

I was adding the imaging package for .net core which is standard in .net
From: Andy De George<EMAIL_ADDRESS>Sent: 12 January 2021 18:08
To: dotnet/docs-desktop<EMAIL_ADDRESS>Cc: Mick Lord<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [dotnet/docs-desktop] Common problem with all your documentation (#164)
@micklord49https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicklord49&data=04|01||2a71e647f9d64eb1c58a08d8b724f330|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637460716609201871|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=%2F8Xn8CefusQjt7y%2BXKfL56sm%2FDZmxhki6OBBhWuF2sc%3D&reserved=0 Interesting, it shouldn't require any NuGet package. What package did you add?

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2Fdocs-desktop%2Fissues%2F164%23issuecomment-758839293&data=04|01||2a71e647f9d64eb1c58a08d8b724f330|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637460716609201871|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=rvzycKEfXsnsmNxY1x1HnF3F%2B7IoMq5TN8%2B%2BfXIcBzc%3D&reserved=0, or unsubscribehttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAOHN4G4OEAPO6HYRYBCNMU3SZSFWXANCNFSM4VSUWQ5Q&data=04|01||2a71e647f9d64eb1c58a08d8b724f330|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637460716609201871|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=EDa79v3O6p71l6PfG6nSErsG097enkWq2B1oDXogr7M%3D&reserved=0.

Did you first create a new Windows Forms app for .NET Core/5? That should add everything you need automatically, you shouldn't need to add any NuGet references.

Did you first create a new Windows Forms app for .NET Core/5? That should add everything you need automatically, you shouldn't need to add any NuGet references.

Nope, I was using it in an ASP project
From: Andy De George<EMAIL_ADDRESS>Sent: 12 January 2021 18:12
To: dotnet/docs-desktop<EMAIL_ADDRESS>Cc: Mick Lord<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [dotnet/docs-desktop] Common problem with all your documentation (#164)
Did you first create a new Windows Forms app for .NET Core/5? That should add everything you need automatically, you shouldn't need to add any NuGet references.

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2Fdocs-desktop%2Fissues%2F164%23issuecomment-758842110&data=04|01||468f47bf55394a2afe2d08d8b7259afb|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637460719422283893|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=KmXJGk7jK9sbcBupaPFUo8ayyTlbiysnwzupqCmf%2FVY%3D&reserved=0, or unsubscribehttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAOHN4G2DQ6UMPFKNH56GEATSZSGIJANCNFSM4VSUWQ5Q&data=04|01||468f47bf55394a2afe2d08d8b7259afb|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637460719422283893|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=YoMAhmQFyboLyXd3Tl4doqwJ0NBrG8%2Fh4pCooVOM4oQ%3D&reserved=0.

Nope, I was using it in an ASP project
From: Andy De George<EMAIL_ADDRESS>Sent: 12 January 2021 18:12
To: dotnet/docs-desktop<EMAIL_ADDRESS>Cc: Mick Lord<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [dotnet/docs-desktop] Common problem with all your documentation (#164)
Did you first create a new Windows Forms app for .NET Core/5? That should add everything you need automatically, you shouldn't need to add any NuGet references.

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2Fdocs-desktop%2Fissues%2F164%23issuecomment-758842110&data=04|01||468f47bf55394a2afe2d08d8b7259afb|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637460719422283893|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=KmXJGk7jK9sbcBupaPFUo8ayyTlbiysnwzupqCmf%2FVY%3D&reserved=0, or unsubscribehttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAOHN4G2DQ6UMPFKNH56GEATSZSGIJANCNFSM4VSUWQ5Q&data=04|01||468f47bf55394a2afe2d08d8b7259afb|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637460719422283893|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=YoMAhmQFyboLyXd3Tl4doqwJ0NBrG8%2Fh4pCooVOM4oQ%3D&reserved=0.

(I was using it to extract Exif information from an uploaded image - which had it's own problems because BitConverter.ToInt32 insisted on being Littlendian even though the asp.net application was running on a linux docker container and the image loader stored everything as Bigendian.
From: Andy De George<EMAIL_ADDRESS>Sent: 12 January 2021 18:12
To: dotnet/docs-desktop<EMAIL_ADDRESS>Cc: Mick Lord<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [dotnet/docs-desktop] Common problem with all your documentation (#164)
Did you first create a new Windows Forms app for .NET Core/5? That should add everything you need automatically, you shouldn't need to add any NuGet references.

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2Fdocs-desktop%2Fissues%2F164%23issuecomment-758842110&data=04|01||468f47bf55394a2afe2d08d8b7259afb|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637460719422283893|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=KmXJGk7jK9sbcBupaPFUo8ayyTlbiysnwzupqCmf%2FVY%3D&reserved=0, or unsubscribehttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAOHN4G2DQ6UMPFKNH56GEATSZSGIJANCNFSM4VSUWQ5Q&data=04|01||468f47bf55394a2afe2d08d8b7259afb|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637460719422283893|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=YoMAhmQFyboLyXd3Tl4doqwJ0NBrG8%2Fh4pCooVOM4oQ%3D&reserved=0.

(I was using it to extract Exif information from an uploaded image - which had it's own problems because BitConverter.ToInt32 insisted on being Littlendian even though the asp.net application was running on a linux docker container and the image loader stored everything as Bigendian.
From: Andy De George<EMAIL_ADDRESS>Sent: 12 January 2021 18:12
To: dotnet/docs-desktop<EMAIL_ADDRESS>Cc: Mick Lord<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [dotnet/docs-desktop] Common problem with all your documentation (#164)
Did you first create a new Windows Forms app for .NET Core/5? That should add everything you need automatically, you shouldn't need to add any NuGet references.

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2Fdocs-desktop%2Fissues%2F164%23issuecomment-758842110&data=04|01||468f47bf55394a2afe2d08d8b7259afb|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637460719422283893|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=KmXJGk7jK9sbcBupaPFUo8ayyTlbiysnwzupqCmf%2FVY%3D&reserved=0, or unsubscribehttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAOHN4G2DQ6UMPFKNH56GEATSZSGIJANCNFSM4VSUWQ5Q&data=04|01||468f47bf55394a2afe2d08d8b7259afb|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637460719422283893|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=YoMAhmQFyboLyXd3Tl4doqwJ0NBrG8%2Fh4pCooVOM4oQ%3D&reserved=0.

Ahh, I see. The documentation is written for Windows Forms, which is why it doesn't need to really give you much setup.

Ahh, I see. The documentation is written for Windows Forms, which is why it doesn't need to really give you much setup.


Add footnotes to verses

Title of change
Add footnotes to verses
Checklist

[x] Unit tests written
[x] Manually tested
[x] Prettier & ESLint were run
[x] New dependencies are included in package-lock.json

Screenshot
insert screenshot if needed

Deployed to: http://staging.quran.com:32887


Install and uninstalls query the full repository content to search for containers

The install and uninstall routines use State::find_installed_npk heavily which is a very expensive operation. Instead of going down to the FS the maps in State should hold all the information we need to check if a container/resource is present or not.
@marcmo Can you check on this? Think you're active in this area anyway these days..

will do 👍

this probably does not make sense anymore now since we will do the mount at startup differently


Bump versions for 1.30

What this PR does / why we need it:
Which issue this PR fixes(if applicable):
fixes #
Special notes for reviewers:

Release note:

NONE

cc @dulek @jichenjc

/lgtm
You can approve this yourself if needed.

/approve

/test pull-cloud-provider-openstack-check

/test pull-cloud-provider-openstack-check

cc @jichenjc @dulek at least one e2e test passed now with 1.30 rc2 k3s

/lgtm


mipmapFilter stopped working since 3.60 beta 11

http://labs.phaser.io/view.html?src=src/textures\set mipmap filter.js

You sure about this? Looks fine to me (only tested on Chrome on macOS though)

Yeah, not sure what's going on. Chrome on windows 7 and windows 10 both show the issue.

Thank you for submitting this issue. We have fixed this and the fix has been pushed to the master branch. It will be part of the next release. If you get time to build and test it for yourself we would appreciate that.


Treat .js as JSX

We should add a unit test with --allowJs and some JSX syntax in there, and make sure no errors are reported.

:+1:

Why exactly are we doing this?

@DanielRosenwasser Because ReactNative uses JSX directly in .js files, so this is becoming popular. VSCode has asked for this. As JSX is a superset, there is no ambiguity with plain JavaScript.

Need to update the tests also. As expected, some errors changed now that parsing default has changed (though only 2, and the errors look like an improvement - good job! :-) ).


Create the concept of RunnableModule and ReactiveModule

Runnable Module is a reactive module with "virtual void run()".

All modules are reactive. Run is also intended to be implemented as an interface. The loop mechanism which generates the "Run" message is TBD.


Keeping the application alert in lua

This for making the user to understand what is wrong with the script

The game still says what's wrong with the script, and this only appears on Mac/Linux due to a crash caused by lime


Don't modify ~/.profile if .poetry/bin already in $PATH

I'd like to make it easier to install poetry on all my dev machines; using the curl method always modifies ~/.profile which is annoying since I carefully manage all my dotfiles, so I have to go around undoing that mod after installing poetry. (I add it to the path myself.)
It would be nice if get-poetry.py would check if ~/.poetry/bin is already in $PATH before modifying .profile.
I know I can get the installer and then run it locally and answer 'n', but there's also no cmd line option for --no-set-path for when I want to do it silently. (I can pipe yes n to it on Mac & Linux, that's a workaround.)

@garyo something like this? ^

Sweet!


Enhancement: extend shortcuts to include alphabet for token classification

Description
This is a stop-gap solution to issue https://github.com/argilla-io/argilla/issues/1852
A more sophisticated solution can be found but in the mean time this solves the inconvenience when there are tags more than 10.
Type of change
(Please delete options that are not relevant. Remember to title the PR according to the type of change)

[x] Improvement (change adding some improvement to an existing functionality)

How Has This Been Tested
(Please describe the tests that you ran to verify your changes. And ideally, reference tests)

[ ] Test A
[ ] Test B

Checklist

[ ] I have merged the original branch into my forked branch
[ ] I added relevant documentation
[ ] follows the style guidelines of this project
[x] I did a self-review of my code
[ ] I added comments to my code
[ ] I made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works

I understand that this is designed as a simple temporary solution, but I want to point out that this will only work for the "default" QWERTY keyboard layout. Users on QWERTZ, AZERTY or others would likely find this unintuitive.

okay realized shortcuts are uppercased in code so changed that.
Also if anyone is looking for a quick custom shortcutkeys fix:
(chrome) using developer tools search for "1234567890" in js files, there should be 2.
then override, reload should work!

@leiyre could this be merged already?

Hello everyone.
Just for information, as mention by @dvsrepo, this correction has also impact in dataset settings feature. See #2444. So I moved the solution to the the front ORM.
Second things, I was looking for a way to get the keyboard layout dynamically (which could solve the problem pointed by @tomaarsen), and there is a keyboard API in javascript for that but it's experimental (see this link).
So when this API will be stable, it would be a nice solution to have dynamical shorcuts depending on the users keyboard.


Feat/precompile call

Description
[PR description]
Issue Link
[link issue here]
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Contents

[item]

Rationale
[design decisions and extended information]
How Has This Been Tested?
[explanation]

How to fill a PR description
Please give a concise description of your PR.
The target readers could be future developers, reviewers, and auditors. By reading your description, they should easily understand the changes proposed in this pull request.
MUST: Reference the issue to resolve
Single responsability
Is RECOMMENDED to create single responsibility commits, but not mandatory.
Anyway, you MUST enumerate the changes in a unitary way, e.g.
This PR contains:
- Cleanup of xxxx, yyyy
- Changed xxxx to yyyy in order to bla bla
- Added xxxx function to ...
- Refactored ....

Design choices
RECOMMENDED to:

What types of design choices did you face?
What decisions you have made?
Any valuable information that could help reviewers to think critically

@silathdiir tesed this branch with testool and found some "precompile revert" cases failed.   I asked steven to have a look and debug these failures.
https://circuit-release.s3.us-west-2.amazonaws.com/testool/default.1684987723.f2c623c.html

during reviewing https://github.com/scroll-tech/zkevm-circuits/pull/517, i realized something.. we should some constraints to check the calldata input for identity is same as the output of identity. Currently, we try to implement this by doing so:
// input
cb.copy_table_lookup(
                        cb.curr.state.call_id.expr(),
                        CopyDataType::Memory.expr(),
                        callee_call_id.expr(),
                        CopyDataType::Precompile.expr(), ...

// ouput
cb.copy_table_lookup(
                            callee_call_id.expr(),
                            CopyDataType::Precompile.expr(),
                            callee_call_id.expr(),
                            CopyDataType::Memory.expr(), ...

But i think it is still not enough?   Did yo constrain in copy circuit that with same call_id, CopyDataType::Precompile for src and dst be equal?   (we should not constraint this, since input/output for other precompiles are not equal)
I am afraid the idea that "BasePrecompile can be IdentityPrecompile" should not be possible.

i think this new idea will work well

@lispc checking input_rlc and output_rlc fixed in https://github.com/scroll-tech/zkevm-circuits/pull/514/commits/33497f8e90f467b1d84371c92a087e0718a6318b and https://github.com/scroll-tech/zkevm-circuits/pull/514/commits/92d98ebee38fd2f19a8a0949e412a8b0d929085f

the new constrains (input_rlc/len == output_rlc/len) looks correct, while i am considering is it possible to move
                    "input and output bytes are the same",
                    input_bytes_rlc,
                    output_bytes_rlc,
                );
                cb.require_equal(
                    "input length and precompile return length are the same",
                    cd_length,
                    precompile_return_length,
                );

into identity gadgets?
Later we will implement other precompile, i think writing these input/output related constraints in separate files would be better than putting them all in the precompile_gadget.rs, there will be a long long cb.condition(address.value_equals(PrecompileCalls::XXX/YYY/ZZZ)..

i think this pr is ready and @silathdiir can you run testool again for this PR to make sure it will not break too many old tests?  If everything goes ok, i will merge this

@lispc @roynalnaruto may need to fix the new failed testool cases (95 failed in total +64):
https://circuit-release.s3.us-west-2.amazonaws.com/testool/default.1686102269.9bb5523.html
I am debugging on these.

Re-run testool on this branch again, remains 32 failed cases (not all related with precompile).
https://circuit-release.s3.us-west-2.amazonaws.com/testool/default.1686137818.fecc428.html


DNS challenge DNS record name

I follow the steps for the DNS challenge.
https://github.com/porunov/acme_client/wiki/Get-a-wildcard-certificate
In Step 7, we need to create 2 txt record in the DNS with the same name.  Is it possible?  Or is it a typo?
Quoted text:
First record. Name: _acme-challenge value: ;
Second record. Name: _acme-challenge value: .
Wait till your DNS records are updated and check that your two digests are available to be checked for the domain: _acme-challenge.example.com.

thank you.


map changes and unpinning

Lots of changes to make the mapping plot more flexible. Also unpinned versions in setup.cfg to be more flexible going forward.

Codecov Report

Merging #56 (6d12af8) into main (5f178d2) will decrease coverage by 3.92%.
The diff coverage is 8.88%.

@@            Coverage Diff             @@
##             main      #56      +/-   ##
==========================================
- Coverage   60.11%   56.19%   -3.92%
==========================================
  Files          16       16
  Lines         880      952      +72
==========================================
+ Hits          529      535       +6
- Misses        351      417      +66

Flag
Coverage Δ

unittests
56.19% <8.88%> (-3.92%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

ocean_model_skill_assessor/main.py
38.88% <ø> (ø)

ocean_model_skill_assessor/plot/map.py
15.07% <8.88%> (-9.00%)
:arrow_down:


Fix/#523 consul http https connection

related issue: https://github.com/networknt/light-4j/issues/523
the previous fix didn't cover all the scenarios,
the previous fix of this issue will cause a problem:

when a service itself is not registered in Consul, but trying to discover other services, it will create too many connections

Codecov Report

Merging #570 into 1.6.x will increase coverage by 0.15%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##              1.6.x     #570      +/-   ##
============================================
+ Coverage     50.37%   50.53%   +0.15%
  Complexity     1819     1819
============================================
  Files           254      254
  Lines         10993    10959      -34
  Branches       1567     1542      -25
============================================
  Hits           5538     5538
+ Misses         4804     4770      -34
  Partials        651      651

Impacted Files
Coverage Δ
Complexity Δ

.../com/networknt/consul/client/ConsulClientImpl.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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

@BalloonWen As we have discussed, I think we should cross-check the consulUrl to make the decision if HTTP/1.1 or HTTP/2 is used. If the consulUrl is http:// then HTTP1.1 should be used even the http2 is enabled in the consul.yml file. There are some users who do not know that Consul only supports HTTP/2 in the case HTTP/S is used. Thanks.

 Fix/#523 consul http https connection

 Fix/#523 consul http https connection


About the code snippet

I want to implement a vscode code snippets plugin, general code snippets plugin code snippets are maintained in the plugin's snippets directory
I want to implement a vscode code snippets plugin, I hope to be able to read the vscode-snippets directory under the vscode workspace code snippets file to realize dynamic code snippets.
The final result is as follows:
a project workspace, vscode-snippets has a,b,c code snippets, vscode opens a project workspace, then can use a,b,c code snippets.
Under project b workspace, vscode-snippets have code snippets e,f,g. If vscode opens project b workspace, it can use code snippets e,f,g.
What should I do?

We closed this issue because it is a question about using VS Code rather than an issue or feature request. Please search for help on StackOverflow, where the community has already answered thousands of similar questions. You may find their guide on asking a new question helpful if your question has not already been asked. See also our issue reporting guidelines.
Happy Coding!


Possible that subchart values never get "finalized"/built

StreamOrientedChartLoader seems in general to be OK, but when it finds a subchart's values.yaml file, it does not appear to actually finalize the ConfigBuilder such that the resulting Config object's raw property contains anything.

From debugging the generated gRPC code, it looks like Chart.Builder#build() does not…there is something about the way that its dependenciesBuilder is not initialized.
For reference, when loading the chart, we do this:
protected void installConfig(final Chart.Builder chartBuilder, final InputStream stream) throws IOException {
  Objects.requireNonNull(chartBuilder);
  Objects.requireNonNull(stream);
  Config returnValue = null;
  // The line below is I think the problem; it doesn't seem to set the Chart.Builder's dependenciesBuilder
  final Config.Builder builder = chartBuilder.getValuesBuilder();
  assert builder != null;
  final ByteString rawBytes = ByteString.readFrom(stream);
  assert rawBytes != null;
  builder.setRawBytes(rawBytes);
}

Need to do more gRPC research.

Actually, that snippet above is another problem.  I think fundamentally I am not understanding the way that sub-builders are initialized.

Yes, here is a portion of the Chart.Builder.java partialBuild() method:
public hapi.chart.ChartOuterClass.Chart buildPartial() {
  hapi.chart.ChartOuterClass.Chart result = new hapi.chart.ChartOuterClass.Chart(this);
  int from_bitField0_ = bitField0_;
  int to_bitField0_ = 0;
  if (metadataBuilder_ == null) {
    result.metadata_ = metadata_;
  } else {
    result.metadata_ = metadataBuilder_.build();
  }
  if (templatesBuilder_ == null) {
    if (((bitField0_ & 0x00000002) == 0x00000002)) {
      templates_ = java.util.Collections.unmodifiableList(templates_);
      bitField0_ = (bitField0_ & ~0x00000002);
    }
    result.templates_ = templates_;
  } else {
    result.templates_ = templatesBuilder_.build();
  }
  if (dependenciesBuilder_ == null) {
    if (((bitField0_ & 0x00000004) == 0x00000004)) {
      dependencies_ = java.util.Collections.unmodifiableList(dependencies_);
      bitField0_ = (bitField0_ & ~0x00000004);
    }
    result.dependencies_ = dependencies_;
  } else {
    result.dependencies_ = dependenciesBuilder_.build();
  }
  // and so on

While debugging, I've noticed that dependenciesBuilder_ is null.  This suggests that even though I'm loading subcharts I'm not doing so properly somehow.  That is, I think that if I were doing it properly, then a build() call on a "top level" Chart.Builder should "cascade" and all of these subbuilders referenced above should not be null.

The problem might very well be in the createSubchartBuilder(String) method:
private static final Chart.Builder createSubchartBuilder(final String chartPath) {
  Chart.Builder returnValue = null;
  final String chartName = getChartName(chartPath);
  if (chartName != null) {
    returnValue = Chart.newBuilder();
    assert returnValue != null;
    final Metadata.Builder builder = returnValue.getMetadataBuilder();
    assert builder != null;
    builder.setName(chartName);
  }
  return returnValue;
}

That subchart builder is never going to be associated with its parent.

That's it.  OK, working on a fix.


[libx11] Optimize Build

[x] Changes comply with the maintainer guide.
[x] SHA512s are updated for each updated download.
[x] The "supports" clause reflects platforms that may be fixed by this new version.
[x] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

To beat:

x64-windows: 28 min
x86-windows: 28 min
x64-windows-static-md: 26 min
x64-windows-static: 26 min
arm64-windows: 26 min

To beat:

x64-windows: 28 min
x86-windows: 28 min
x64-windows-static-md: 26 min
x64-windows-static: 26 min
arm64-windows: 26 min

I can confirm that the build of libx11 is indeed significantly faster compared to the CI workflow. However, during PR workflows, the builds here were slower. I think the reason for that is that here all ports that depend on libx11 are also included which causes additional x11 ports to be present. Maybe there is an installation order issue hiding somewhere?

To beat:

x64-windows: 28 min
x86-windows: 28 min
x64-windows-static-md: 26 min
x64-windows-static: 26 min
arm64-windows: 26 min

I can confirm that the build of libx11 is indeed significantly faster compared to the CI workflow. However, during PR workflows, the builds in this PR are slower. I think the reason for that is that here all ports that depend on libx11 are also built which causes additional x11 ports to be present. Maybe there is an installation order issue hiding somewhere?

Installation order: Install indirect and direct dependencies of libx11 first, then install ports that have libx11 as a dependency.

Record: Differences in the share folder after modification.


Automated Package Upgrade

:tada: This PR is included in version 5.0.132 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Use Module#prepend only when it exists

JRuby 1.7.x does not have a Module#prepend method. #457 made the assumption that if Ruby version is 2.0 or higher, then the prepend method exists. This affected me because I'm using JRuby 1.7 in "2.0 mode".
It's always better to use duck typing instead of assuming that a version of Ruby has specific methods.
I believe it's correct to treat a missing prepend method by using the pre-Ruby 2.0 method (inserting OldDebugExceptionsCatcher), and it seems to work correctly. But it may not be the correct behavior.

Can you rebase this against the latest travis.yml?

LoadError: no such file to load -- raven
Does that failure mean anything to you? I get the feeling that it's not related to a change that I made.

Heh, no, I was hoping you would know :laughing:

Doh! It makes sense...the gemfile doesn't exist, so raven doesn't exist in the bundle.

Wow, thanks for the fix, didn't know that something like JRuby 1.7 in 2.0 mode exists..
(I was the one who made the fix for the Module#prepend stuff)
Good work!

Thanks @Elektron1c97. I hate to admit that the reason I know about it is that I'm using it in production!

Merged in a4624495bf99c8da04f6


Fixed broken map.php

Map.php was completely broken.
I copied the correct file from upstream (zofe/rapyd-laravel)
Please review and merge.

@dannoso thank you


premisson denied

when I use this command to start my ghtorrent:ruby -Ilib bin/ght-retrieve-repo -c config.yaml gousiosg github-mirror;
It will generate this warning:
WARN, 2020-10-15T18:29:16+08:00, ghtorrent -- api_client.rb: Failed request. URL: https://api.github.com/users/gousiosg?per_page=100, Status code: 401, Status: Unauthorized, Access: place your , IP: <IP_ADDRESS>, Remaining: 59
WARN, 2020-10-15T18:29:16+08:00, ghtorrent -- api_client.rb: Unauthorised request with token: place your github token here
WARN, 2020-10-15T18:29:16+08:00, ghtorrent -- ghtorrent.rb: Transaction failed (989 ms)
401 Unauthorized
seems like Permissions issue

It looks like you haven't added your own GitHub API token to your config.
That error message comes from here: https://github.com/gousiosg/github-mirror/blob/5f001e57264ffb25b6b8add50732758574b52480/lib/ghtorrent/api_client.rb#L205
You can either update the settings directly here: https://github.com/gousiosg/github-mirror/blob/8f2fdc3f2ae2eb45156b3dcee6af9e94bd2cdf04/lib/ghtorrent/settings.rb#L62
Or a better idea would be to enter the token key inside your config.yaml like this: https://github.com/gousiosg/github-mirror/blob/master/config.yaml.tmpl#L43
Hope that helps.


memory leak in beta 6? (speculating and investigating)

Hey there,
Sorry for the lack of detail on this one, as I'm still trying to investigate further detail myself. I have a private project that has been in production status for a while now, and also has been on the 2.x.x branch of development long before there were beta releases for the milestone. I've been on this train for a minute.
My solution is no small project, it receives well over 15 HTTP requests per second (sustained), and does encounter malicious queries from time to time. These malicious queries helped me identify issues like PR 404. I learn about these issues in my production environment, as they happen. Its a good time :D
As I check my notes and possibly A/B test over the next days, I'll give you the punchline of a suspected issue that has seemingly emerged in my work:
I believe hummingbird 2 beta 6 introduced some amount of memory leakage that was not present in the earlier versions, possibly even nonexistent in the prior beta 5. This is a tentative but educated speculation based on an unexpected crash of my environment, which is otherwise very stable. I continue to investigate.
For the sake of discussion: My production server as it runs today is WAY OVER PROVISIONED with 24GB of ram (headless with no other applications to host), but unfortunately, this headroom is currently not enough to keep my process running more than a week or so.
frankly I was very caught off guard by this and as such, have very minimal info to go off of. but we can start by looking at the syslogs logs and confirming why exactly my process was killed:
Jun 15 18:41:08 localhost kernel: [6328716.711816] oom-kill:constraint=CONSTRAINT_NONE,nodemask=(null),cpuset=/,mems_allowed=0,global_oom,task_memcg=/user.slice/user-0.slice/session-11329.scope,task=pricedb,pid=1242483,uid=1000
Jun 15 18:41:08 localhost kernel: [6328716.711850] Out of memory: Killed process 1242483 (pricedb) total-vm:322911892kB, anon-rss:24076600kB, file-rss:0kB, shmem-rss:0kB, UID:1000 pgtables:67016kB oom_score_adj:0

To be clear: the last time I've seen this process outright die like this was when I first discovered PR 404. At this point, I've given beta6 and beta5 roughly equivalent time in prod. I can say with certainty that beta5 never crashed but of course, this does not mean the issue does not exist in beta5.
DISCLAIMER: I have to acknowledge that this could easily be a bug that sits in my stack, not hummingbird. This is what I'm trying to sort out. For the sake of kicking this off, I'm inclined to believe this is NOT the most likely scenario, simply because my latest update to my process and prod environment was a single change.......hummingbird 2.x.x beta5 to beta6 in my Package.swift.
The prior release was seemingly very stable.
Thank you for the continued development of this awesome framework.

There are very few differences between beta 5 and 6. I'll investigate further though. Which Hummingbird packages are you using?

I couldn't find anything obvious. I did a test where I had a route that responded with the body that was in the request. And using wrk hit that route 52107270 times, transferring about 153GB and memory usage never went over 9MB according to the activity monitor.

after a thorough investigation and a few A/B tests I can confidently rule out hummingbird project from my issue here.
thank you, as always, for the excellent work.
cheers!


Cant start my App.  Unable to copy the app.exe to win-x64 Access to the path is denied

Here is the output after i have given command  "dotnet electronize start"
C:\Program Files\dotnet\sdk\2.1.201\Microsoft.Common.CurrentVersion.targets(4529,5):
error MSB3021: Unable to copy file "C:\Users\Siva\Dropbox\Electron APP\APPS\myapp\myapp\obj\netcoreapp2.0\win-x64\host\myapp.exe" to "bin\Debug\netcoreapp2.0\win-x64\myapp.exe". Access to the path 'C:\Users\Siva\Dropbox\Electron APP\APPS\myapp\myapp\bin\Debug\netcoreapp2.0\win-x64\myapp.exe' is denied. [C:\Users\Siva\Dropbox\Electron APP\APPS\myapp\myapp\myapp.csproj]
My CLI version :
ElectronNET.CLI Version: <IP_ADDRESS>

Problem resolved after i restarted my machine .


Auto-Detect PR error on Workflow dispatch, resource not accessible

Since there's no PR information on a workflow_dispatch the action will try to fetch it
https://github.com/cypress-io/github-action/blob/f6f2cc1665fa2dcd75170785c5dcaad1fe01fb24/index.js#L474-L497
However, even though the call is in a try/catch it still errors my whole run.
Unable to fetch related PR data for commit: '46a72ff9cd29aeaa59f536eee6c008e3ffbcf56d':  RequestError [HttpError]: Resource not accessible by integration
    at /home/runner/work/_actions/cypress-io/github-action/v6/dist/index.js:41343:21
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async detectPrNumber (/home/runner/work/_actions/cypress-io/github-action/v6/dist/index.js:7[48](https://github.com/MaintainX/maintainx/actions/runs/9684418723/job/26722269427#step:17:49)81:22)
    at async runTests (/home/runner/work/_actions/cypress-io/github-action/v6/dist/index.js:75152:3) {

Right now I suspect it fails because we are using the following permissions on the job
permissions:
      contents: read # Required to checkout source code
      id-token: write # Required to setup AWS credentials

I'm not sure yet which permission I need to give yet, but I would have expected that it wouldn't fail the run at least

I found that adding the following permission solves this error.
      pull-requests: read # Needed for workflow_dispatch execution

@Cellule
Thank you for reporting your issue! I understand that you have resolved your error condition.
The documentation does not cover permissions settings, so there is some scope for adding information to help users avoid the issue which you ran into.
Could you please post your failing workflow? That would help us to reproduce your issue. We could also check if the error condition needs to be fatal or whether it could be a warning only.

Hi @MikeMcC399 , @Cellule ,
I've ran into the same problem. For us the issue occured right after we've done a forced upgrade to Enterprise account in Github.
Before that cypress-io actions were running for us for years. And the issue happened right after I've upgraded our organization to Enterprise level structure which Github will be enforcing for all users which use Enterprise plans.
I suspect that for the Enterprise there might be some different default rules for GITHUB_TOKEN.
In our case I've added the following line on the top level in the workflow where we use cypress-io, and that fixed it:
permissions: read-all

Closing as there was no further response and I was unable to reproduce the issue.

Related to this issue I have submitted two enhancement requests:

#1272
#1273


Configurable time between update checks, or store last update time outside .gitconfig

Hi, I have my dotfiles including .gitconfig stored in a repository.
The githooks update mechanism stores the last time an autoupdate check was performed in .gitconfig, and checks every 24 hours.
This means that basically every time I go into my dotfiles repo, I'll see a spurious diff in the timestamp caused by githooks having executed an autoupdate check since my last commit.
Minor annoyance but it'd be nice to get it fixed.
I think .gitconfig isn't the best place for this, I think one better place for autoupdate time might be XDG_CACHE_DIR or something.
Or alternatively, another thing that would make the slight annoyance less annoying would be ability to set githooks to autoupdate on a longer interval than 24 hours - i.e. having an option for this interval rather than it being hardcoded in https://github.com/gabyx/Githooks/blob/f4cc437323cf685d8580a8a1d4c39fb0deebdc84/githooks/apps/runner/runner.go#L386
Thanks for your work on this, good piece of software!

Hi, I also have that issue, that the gitconfig is changed due to this...
You are totally right. I guess its only the autoupdate time which is a problem, right.
So I could easily move that part out to the ~/.githooks/.last-update-time. Would make also my dotfiles easier.
I might do that in while.
Thanks for the report.

thanks for the quick fix :)


Right-to-left and left-to-right text

When an OpenType text layout engine applies the Unicode bidi algorithm

This is great! But please note that in the section above "Text processing with OpenType Layout fonts", no mention was made of needing to apply the Unicode bidi algorithm.
You need a complete description of how to shape and layout text, in one place, not scattered throughout various sections.

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

ID: 5b15ab9e-86c5-f6e0-0145-f9de77c60141
Version Independent ID: 0ebe3dec-cc24-b6a4-01b8-73bfdb79a71a
Content: Advanced typographic tables - OpenType Layout - Typography
Content Source: typographydocs/opentype/spec/ttochap1.md
Product: typography
GitHub Login: @PeterCon
Microsoft Alias: PeterCon

Valid suggestion. But probably a large work item, so seems like it wouldn't need to be a high priority for OT 1.8.4.

Pondering: Does that belong in the OT (file format) spec, or in a companion spec on shaping? The OT spec touches on shaping at points, but it's not yet clear to me how the topics ultimately would best be organized and inter-related.


Solarized interferes with bash functionality

Hello,
I'm using Tmux and Vim on a MacBook Air, OS X 10.11.3. I recently used Solarized and love the color scheme but every once in a while, Tmux becomes very buggy. Some errors I've noticed are that bash functions like clear are not working. Opening files in Vim in other cases sometimes changes or wipes out what's displayed on other windows in a Tmux session.
I'm not exactly sure what the problem is and any help would be appreciated. In the example attached, the "clear" command is not working and switching to the leftmost window shows that for some reason, it is also lower down than it should be.

Any help would be really appreciated. Thank you!

You could try using the xterm-256color version (or changing Terminal.app's Preferences > Profiles > Advanced > Declare terminal as to xterm-256color). This solved a problem with Mutt for me.


Fix cuda error when multi process is used

don't be too smart to detect # of GPU which cause a GPU context created in very early stage. Due to CUDA limitation, fork() process cannot run well if we create GPU context before fork.

build fail

Could you rebase?


Default language is always en

Is there an existing issue for this?

[X] I have searched the existing issues

Version of Corteza
2023.9.7
Current Behavior
Even if there is only one configured language (that is not en, e. g. LOCALE_LANGUAGES: "de") the default language for a user is always en. This is the case for all UI parts (one, compose, admin, ...). Only auth/profile picks de correctly.
On startup in the profile "German" is correctly selected as preferred language, but i18next-cookie is en on startup. If the profile is saved again with "German" selected (even it was selected before) then i18next-cookie gets correct de value and language works as expected in the other UI parts.
Expected Behavior
Language is set correctly from the beginning (without saving the profile again).
Steps To Reproduce
No response
Environment and versions
Edge 126

Anything else?
No response

I experience the same issue on Chrome Version 127.0.6533.88.


Can't set elapsed-time response header.

I can't set response header.
content_types_provided(Req, State) ->
{[
{{<<"application">>, <<"json">>, '*'}, to_json}
], Req, State}.
to_json(Req0, State) ->
Delta = binary:encode_unsigned(1560238796411),
Req1 = cowboy_req:set_resp_header(<<"elapsed-time">>, Delta, Req),
{jsx:encode([{list, [something, something]}]), Req1, []}
Ranch listener middleware_http_listener, connection process <0.711.0>, stream 1 had its request process <0.712.0> exit with reason badarith and stacktrace [{list_handler,to_json,2,[{file,"src/list_handler.erl"},{line,78}]},{cowboy_rest,call,3,[{file,"src/cowboy_rest.erl"},{line,1128}]},{cowboy_rest,set_resp_body,2,[{file,"src/cowboy_rest.erl"},{line,1019}]},{cowboy_rest,upgrade,4,[{file,"src/cowboy_rest.erl"},{line,238}]},{cowboy_stream_h,execute,3,[{file,"src/cowboy_stream_h.erl"},{line,205}]},{cowboy_stream_h,request_process,3,[{file,"src/cowboy_stream_h.erl"},{line,184}]},{proc_lib,init_p_do_apply,3,[{file,"proc_lib.erl"},{line,247}]}]

badarith is a math operation exception, for example divide by 0 or trying to add a number and a string, so you must be doing something that fails that's not in this snippet. The error occurs in your to_json function.

Right. My mistake.


Applicability to Azure SQL Database Managed Instance

This issue relates to https://github.com/MicrosoftDocs/sql-docs/issues/4076
Some file group features apply to Azure SQL MI in addition to SQL Server. For example, see https://learn.microsoft.com/en-us/sql/t-sql/statements/alter-database-transact-sql-file-and-filegroup-options?view=azuresqldb-mi-current&preserve-view=true.
I think this page should be updated to:

List Azure SQL MI as one of the applicable platforms
Enumerate any differences in support between SQL Server and Azure SQL MI

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

ID: 8b1faba5-e9b8-a343-308b-509a70d08db0
Version Independent ID: c5156c9e-8e82-0c25-a8f5-4d1c37ee6720
Content: Database Files and Filegroups - SQL Server
Content Source: docs/relational-databases/databases/database-files-and-filegroups.md
Service: sql
Sub-service: configuration
GitHub Login: @WilliamDAssafMSFT
Microsoft Alias: wiassaf

@VitalyMCT -- Vitaly, thank you for your clear feedback.
@WilliamDAssafMSFT -- William, please look into this issue.

Thanks @VitalyMCT, it appears the program group made this exact change to the Database Files and Filegroups article in mid-December.


SDI-477: 🔥 Remove unit tests that are tested as functional tests

SDI-477: 🔥 Remove as tested in functional.pluginmanagers.SpringBootPluginManagerJarTest.Spring Boot info endpoint contains git info
SDI-477: 🔥 Remove as tested in functional.configmanagers.BaseConfigManagerTest.Junit 5 tests can be executed
SDI-477: 🔥 Remove as tested in functional.configmanagers.DependencyCheckPluginManagerTest
SDI-477: 🔥 Remove as tested in functional.pluginmanagers.JavaSpringBootPluginManagerJarTest
SDI-477: 🔥 Remove as tested in functional.pluginmanagers.KotlinPluginManagerTest

Also wondering if we should bump the version just in case we end up publishing by accident - would it override the existing version or not? (I don't actually know 😄 )


[TF/PT] Add FAST detection model

This PR:  (post 0.8.0 release)

Add FAST model

Any feedback is welcome :hugs:
Todo:

[x] Check with min pooling in target building -> (same results as with a tuned shrink_ratio)
[x] Check training
[x] Check inference

Some notes:

We skip complex reparameterize (brings only ~5% speed up)
Instead of only crop on the binary mask from logits we use the current post proc (better controlable with bin_thresh and more save filtering for text rich documents)
The extensive use of interpolation from the reference implementation is reduced to the required ones

used PolyLR

Inference example (model trained only on FUNSD)

Congrats @felixdittrich92  !!

pooling size tuned boxes are now mostly perfect

Any estimate on when can we use FAST detection model. DB_resnet50 is very slow for my usecase (i need to run e2e ocr in like 1 sec per image) but it takes longer.

Any estimate on when can we use FAST detection model. DB_resnet50 is very slow for my usecase (i need to run e2e ocr in like 1 sec per image) but it takes longer.

The plan is to do the release next week and afterwards start with the pytorch fast_tiny
With cuda and half precision (cuda().half()) it should also be possible to run db_resnet50 1sec/image additional you can set a higher batch size for text recognition rec_bs=1024 for example (depends on your hardware) which would be another speed up.
Maybe torch.compile could also be an option but we haven't tested our models with it yet. :)
CC @odulcy-mindee


docs(data-classes): Correct import for DynamoDBRecordEventName

Issue #, if available:
#298
Description of changes:
Correct import for DynamoDBRecordEventName, should be:
from aws_lambda_powertools.utilities.data_classes.dynamo_db_stream_event import DynamoDBStreamEvent, DynamoDBRecordEventName

Checklist

[X] Meet tenets criteria
[X] Update tests
[X] Update docs
[X] PR title follows conventional commit semantics

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@gianrubio i patched the documentation to be correct.


Rework the "trusted extensions" system

Steps to reproduce
Right now the current trusted extensions system makes the app nearly unusable. After manually adding a repo, you have to tap each source and trust each and every one. Then after they update you have to trust them again. This is after having nothing to go off of to ensure the extension repo is trusted, it doesn't link you to the source code or anything so the entire system is just a cumbersome set of dialog boxes.
This is fine for a couple repos but if you have 20-50 repos it becomes a nightmare. A better solution to this would be to trust the repository rather than each and every extension repo.
Expected behavior
.
Actual behavior
.
Crash logs
No response
Mihon version
0.16.4
Android version
Android 13
Device
Pixel 7 Pro
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version 0.16.4.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

#320


ZNC sends actual NICK too late to IRC client

When connecting to IRC with irssi via ZNC, the NICK is sent too late. irssi isn't aware of the actual nick and uses the wrong one to set mode +i.
See raw log from irssi, NICK nope is what is configured in irssi, NICK cxl is the actual nick on that IRC server, and irssi sends MODE <nick> +i:
<< CAP LS
<< PASS ***/libera:irssi
<< NICK nope
<< USER nope nope bnc.example.org :nope
>> :irc.znc.in CAP unknown-nick LS :batch cap-notify echo-message multi-prefix server-time userhost-in-names znc.in/batch znc.in/self-message znc.in/server-time-iso
<< CAP REQ :multi-prefix
>> :irc.znc.in CAP nope ACK :multi-prefix
<< CAP END
>> :tantalum.libera.chat 001 nope :Welcome to the Libera.Chat Internet Relay Chat Network cxl
<< MODE nope +i
>> :tantalum.libera.chat 002 nope :Your host is tantalum.libera.chat[<IP_ADDRESS>/6697], running version solanum-1.0-dev
>> :tantalum.libera.chat 003 nope :This server was created Mon Nov 22 2021 at 18:12:28 UTC
>> :tantalum.libera.chat 004 nope tantalum.libera.chat solanum-1.0-dev DGQRSZaghilopsuwz CFILMPQSTbcefgijklmnopqrstuvz bkloveqjfI
>> :tantalum.libera.chat 005 nope WHOX MONITOR=100 SAFELIST ELIST=CMNTU ETRACE FNC CALLERID=g KNOCK CHANTYPES=# EXCEPTS INVEX CHANMODES=eIbq,k,flj,CFLMPQSTcgimnprstuz :are supported by this server
>> :tantalum.libera.chat 005 nope CHANLIMIT=#:250 PREFIX=(ov)@+ MAXLIST=bqeI:100 MODES=4 NETWORK=Libera.Chat STATUSMSG=@+ CASEMAPPING=rfc1459 NICKLEN=16 MAXNICKLEN=16 CHANNELLEN=50 TOPICLEN=390 DEAF=D :are supported by this server
>> :tantalum.libera.chat 005 nope TARGMAX=NAMES:1,LIST:1,KICK:1,WHOIS:1,PRIVMSG:4,NOTICE:4,ACCEPT:,MONITOR: EXTBAN=$,ajrxz :are supported by this server
>> :tantalum.libera.chat 251 nope :There are 67 users and 42759 invisible on 28 servers
>> :tantalum.libera.chat 252 nope 41 :IRC Operators online
>> :tantalum.libera.chat 254 nope 21929 :channels formed
>> :tantalum.libera.chat 255 nope :I have 655 clients and 1 servers
>> :tantalum.libera.chat 265 nope 655 4545 :Current local users 655, max 4545
>> :tantalum.libera.chat 266 nope 42826 53550 :Current global users 42826, max 53550
>> :tantalum.libera.chat 250 nope :Highest connection count: 4546 (4545 clients) (368870 connections received)
>> :nope!~hey_its_m@user/cxl NICK :cxl
>> :tantalum.libera.chat 375 cxl :- tantalum.libera.chat Message of the Day -
>> :tantalum.libera.chat 372 cxl :- This server provided by Hyperfilter (https://hyperfilter.com)
>> :tantalum.libera.chat 372 cxl :-
>> :tantalum.libera.chat 372 cxl :-
>> :tantalum.libera.chat 372 cxl :- Welcome to Libera Chat, the IRC network for
>> :tantalum.libera.chat 372 cxl :- free & open-source software and peer directed projects.
>> :tantalum.libera.chat 372 cxl :-
>> :tantalum.libera.chat 372 cxl :- Use of Libera Chat is governed by our network policies.
>> :tantalum.libera.chat 372 cxl :-
>> :tantalum.libera.chat 372 cxl :- To reduce network abuses we perform open proxy checks
>> :tantalum.libera.chat 372 cxl :- on hosts at connection time.
>> :tantalum.libera.chat 372 cxl :-
>> :tantalum.libera.chat 372 cxl :- Please visit us in #libera for questions and support.
>> :tantalum.libera.chat 372 cxl :-
>> :tantalum.libera.chat 372 cxl :- Website and documentation:  https://libera.chat
>> :tantalum.libera.chat 372 cxl :- Webchat:                    https://web.libera.chat
>> :tantalum.libera.chat 372 cxl :- Network policies:           https://libera.chat/policies
>> :tantalum.libera.chat 372 cxl :- Email<EMAIL_ADDRESS>>> :tantalum.libera.chat 376 cxl :End of /MOTD command.
>> :cxl!~hey_its_m@user/cxl MODE cxl :+Ziw

is sent too late.

Uhm... when would be "not too late"?
This is only a guess, but I bet that from ZNC's point of view, it sends the following all at once, even before irssi sends the MODE:
>> :tantalum.libera.chat 001 nope :Welcome to the Libera.Chat Internet Relay Chat Network cxl
>> :tantalum.libera.chat 002 nope :Your host is tantalum.libera.chat[<IP_ADDRESS>/6697], running version solanum-1.0-dev
>> :tantalum.libera.chat 003 nope :This server was created Mon Nov 22 2021 at 18:12:28 UTC
>> :tantalum.libera.chat 004 nope tantalum.libera.chat solanum-1.0-dev DGQRSZaghilopsuwz CFILMPQSTbcefgijklmnopqrstuvz bkloveqjfI
>> :tantalum.libera.chat 005 nope WHOX MONITOR=100 SAFELIST ELIST=CMNTU ETRACE FNC CALLERID=g KNOCK CHANTYPES=# EXCEPTS INVEX CHANMODES=eIbq,k,flj,CFLMPQSTcgimnprstuz :are supported by this server
>> :tantalum.libera.chat 005 nope CHANLIMIT=#:250 PREFIX=(ov)@+ MAXLIST=bqeI:100 MODES=4 NETWORK=Libera.Chat STATUSMSG=@+ CASEMAPPING=rfc1459 NICKLEN=16 MAXNICKLEN=16 CHANNELLEN=50 TOPICLEN=390 DEAF=D :are supported by this server
>> :tantalum.libera.chat 005 nope TARGMAX=NAMES:1,LIST:1,KICK:1,WHOIS:1,PRIVMSG:4,NOTICE:4,ACCEPT:,MONITOR: EXTBAN=$,ajrxz :are supported by this server
>> :tantalum.libera.chat 251 nope :There are 67 users and 42759 invisible on 28 servers
>> :tantalum.libera.chat 252 nope 41 :IRC Operators online
>> :tantalum.libera.chat 254 nope 21929 :channels formed
>> :tantalum.libera.chat 255 nope :I have 655 clients and 1 servers
>> :tantalum.libera.chat 265 nope 655 4545 :Current local users 655, max 4545
>> :tantalum.libera.chat 266 nope 42826 53550 :Current global users 42826, max 53550
>> :tantalum.libera.chat 250 nope :Highest connection count: 4546 (4545 clients) (368870 connections received)
>> :nope!~hey_its_m@user/cxl NICK :cxl
>> :tantalum.libera.chat 375 cxl :- tantalum.libera.chat Message of the Day -
>> :tantalum.libera.chat 372 cxl :- This server provided by Hyperfilter (https://hyperfilter.com)
>> :tantalum.libera.chat 372 cxl :-
>> :tantalum.libera.chat 372 cxl :-
>> :tantalum.libera.chat 372 cxl :- Welcome to Libera Chat, the IRC network for
>> :tantalum.libera.chat 372 cxl :- free & open-source software and peer directed projects.
>> :tantalum.libera.chat 372 cxl :-
>> :tantalum.libera.chat 372 cxl :- Use of Libera Chat is governed by our network policies.
>> :tantalum.libera.chat 372 cxl :-
>> :tantalum.libera.chat 372 cxl :- To reduce network abuses we perform open proxy checks
>> :tantalum.libera.chat 372 cxl :- on hosts at connection time.
>> :tantalum.libera.chat 372 cxl :-
>> :tantalum.libera.chat 372 cxl :- Please visit us in #libera for questions and support.
>> :tantalum.libera.chat 372 cxl :-
>> :tantalum.libera.chat 372 cxl :- Website and documentation:  https://libera.chat
>> :tantalum.libera.chat 372 cxl :- Webchat:                    https://web.libera.chat
>> :tantalum.libera.chat 372 cxl :- Network policies:           https://libera.chat/policies
>> :tantalum.libera.chat 372 cxl :- Email<EMAIL_ADDRESS>>> :tantalum.libera.chat 376 cxl :End of /MOTD command.
>> :cxl!~hey_its_m@user/cxl MODE cxl :+Ziw

And only after it sends all of the above does it receive the MODE nope +i from irssi.
You could check this by running ZNC in debug mode (znc --debug). However, that doesn't really help, so... no idea what could be done about this.

Couldn't >> :nope!~hey_its_m@user/cxl NICK :cxl be the first thing ZNC sends, even before > :tantalum.libera.chat 001 nope :Welcome to the Libera.Chat Internet Relay Chat Network cxl? Just spitballing here, I don't know if that would make any sense.

I'm going to quote from the modern IRC docs' section about connection registration (without taking the time to trace each piece of information back to the RFCs, because the site is a reference for how things are implemented):

The first parameter of the RPL_WELCOME (001) message is the nickname assigned by the network to the client. Since it may differ from the nickname the client requested with the NICK command (due to, e.g. length limits or policy restrictions on nicknames), the client SHOULD use this parameter to determine its actual nickname at the time of connection. Subsequent nickname changes, client-initiated or not, will be communicated by the server sending a NICK message.

To me, it then seems incorrect that ZNC sends nope as the nick in 001 if the network side is connected. One could say ZNC has an implicit "limit or policy restriction on nicknames", and should inform the client as soon as possible that the NICK it requested was altered. The explicit NICK command sent later can be removed, then. Does that sound reasonable?

I switched to SASL EXTERNAL and seems like this is still hapenning. My version is slightly outdated since it is the package from debian: 1.8.2+deb3.1+deb12u1.


Re-import AttrsInstance in attrs ns

cf. https://github.com/python-attrs/attrs/issues/987#issuecomment-1206077089

I need to fix that Mypy caching bug for the magic attribute, don't I?

If you want it fixed… 😇

If I don't fix it, mypy won't recognize the magic attribute and the protocol will be useless. 😭


schedules: Warn the user if the schedule cannot be updated

For exact schedules imported from GTFS, there is no oubound/inbound, frequencies or number of units. It is thus not possible to re-generate schedules for those paths, when the path is updated for example. Just taking the current trips and updating the stop times is not a good idea, as the user cannot know which trips were using the updated paths, so cannot really understand the result.
Instead, we should just warn the user that trips using this path for service X cannot be updated and they should manually change it.

In the meantime, having old schedules on updated paths, if the number of nodes is different will prevent trRouting from running for ever and ever.
The following SQL queries in the DB can help find out which lines/paths are problematic:
select a.acronym, shortname, p.updated_at, t.updated_at, cardinality(nodes), cardinality(node_arrival_time_seconds), cardinality(node_departure_time_seconds)
from demo_transition.tr_transit_lines l
inner join demo_transition.tr_transit_agencies a on a.id = l.agency_id
inner join demo_transition.tr_transit_paths p on p.line_id = l.id
inner join demo_transition.tr_transit_schedule_trips t on t.path_id = p.id
where cardinality(nodes) != cardinality(node_arrival_time_seconds)


[react-form-state] validate fields that aren't dirty

Overview
This is somewhat related to #299 as well... But essentially we need way to force validation on fields that haven't been dirtied yet. This can be an empty as well as a field that has data populated from the server.
Motivation

What inspired this enhancement? What makes you think this should be in quilt?

Same thing as #299, we need to inform the merchant what fields are invalid according to our app. The merchant hasn't necessarily interacted with the page yet.
cc @michelleyschen

Checklist

[x] Please delete the labels section before submitting your issue
[x] I have described this issue in a way that is actionable (if possible)

When we add validation on submit and possibly validation on initialize, those will definitely not bail out when the field is not dirty. Given that being the case, is that sufficient? Or do you want to also validate fields that are not dirty when they are blurred?

I would always want to validate not dirty. Except for submit.

Yeah it's almost like we need an additionally option where we don't just validate dirty fields. Like force validation or something like that. @michelleyschen I think this option could be at the form level right? I don't think we'd change between the two during the same form interaction.

That's correct. Form level is sufficient

I'm not necessarily against a form level prop for this (eg. validatePristine) , but I just want to check in with you to make sure this would not be solved by the validateForm method added in 0.3.x. Additionally, if this would be solved by validating fields that are touched (have ever been interacted with at all) instead of dirty.

My use case is actually covered by using a combination of validateForm and validateOnSubmit, since now the logic is consistent in both to validate all the fields even if its not touched.


New version: GitKraken.cli version 2.1.2

Pull request has been created with WinGet Releaser :rocket:
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Installation failed with exit code -1978335187
2024-11-04 03:28:16.528 [FAIL] Installer failed security check. Url: https://github.com/gitkraken/gk-cli/releases/download/v2.1.2/gk_2.1.2_Windows_x86_64.zip Result: 0x80004005

Automated error analysis suggests -1978335187 may mean Installer failed security check. Another script should retry this every 18 hours.
Automated error analysis suggests 80004005 may mean Resource missing
(Automated response - build 897.)

@wingetbot run

@stephengillie
I cannot reproduce the error. Everything seems to work fine on my machine

@wingetbot run


PENG-2281 upgrade pydantic settings to ^2.7.4 in the agent project

What
This PR upgrades the pydantic version in the agent project, as well as adds pydantic-settings as a dependency. Notice that some unit tests have been slightly changed: this was needed because the way comparison works between pydantic models and dictionaries in the most recent version.
Why
Keep the project up to date with the most recent technologies
Task: https://sharing.clickup.com/t/h/c/18022949/PENG-2281/S5C5TX46EBVEU8Z

Peer Review
Please follow the upstream omnivector documentation concerning
peer-review guidelines.

I have no idea what's going on with the CI. Locally, the CI passes with no error


[WFLY-8051] Merge Latest WildFly Elytron Integration Changes

:bangbang: Commits from this branch are used in numerous other topic branches so please do not cherry-pick or rebase. :bangbang:
This PR requires the following Core PR https://github.com/wildfly/wildfly-core/pull/2159
The following issues can be resolved with this PR: -
https://issues.jboss.org/browse/WFLY-8051
https://issues.jboss.org/browse/WFLY-8052
https://issues.jboss.org/browse/WFLY-7945
https://issues.jboss.org/browse/WFLY-7705
https://issues.jboss.org/browse/WFLY-7977
https://issues.jboss.org/browse/WFLY-7653
https://issues.jboss.org/browse/WFLY-7996

Superseded by #9618


What's the string for Max 2 Pro?

Really appreciate your help to confirm the string for Max 2 Pro.
By the pattern of the strings for the other models, it seems that it could be
Max2Pro
?

Indeed it is Max2Pro
http://data.onyx-international.cn/api/firmware/update?where={"buildNumber":0,"buildType":"user","deviceMAC":"","lang":"en_US","model":"Max2Pro","submodel":"","fingerprint":""}

@Hagb Could you close this?


Send AttrOff when clearing the screen

In tcell v2.4, the following calls would reset the screen
to its default color:

SetContent() to change the bg color to maroon.
Clear()
Show()

In tcell v2.5, the behavior changed to clear the screen
to the last-used style passed to SetContent.
This happens because commit 62f5502f set the dirty flag
to false for all cells, on the assumption that sending
ti.Clear would clear the cells anyway. Unlike the drawCell
calls triggered by the dirty flag, however, clearScreen
does not send AttrOff. This causes the screen to clear
to the last-used background color.
Issue: https://github.com/gdamore/tcell/issues/522

Thanks for resolving this.


added formal2casual

This transformation transfers style from formal to casual.

#75

Formality Change PR #152 is merged.


Handle empty case in reduce

close #222

Codecov Report

Merging #224 into master will decrease coverage by 0.45%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #224      +/-   ##
==========================================
- Coverage   87.88%   87.43%   -0.46%
==========================================
  Files          19       19
  Lines        1222     1202      -20
==========================================
- Hits         1074     1051      -23
- Misses        148      151       +3

Continue to review full report at Codecov.

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

Codecov Report

Merging #224 into master will increase coverage by 4.54%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #224      +/-   ##
==========================================
+ Coverage   87.88%   92.43%   +4.54%
==========================================
  Files          19       19
  Lines        1222     1282      +60
==========================================
+ Hits         1074     1185     +111
+ Misses        148       97      -51

Impacted Files
Coverage Δ

src/reduce.jl
93.02% <100%> (+1.66%)
:arrow_up:

Continue to review full report at Codecov.

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


[wip] vimPlugins.vim-markdown-composer: init at 2018-11-05

Motivation for this change
adds a markdown viewer, for some reason displays a blank page for now :/ .
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

I managed to complete it but the preview would remain blank. Will try with another one


Fix error capturing and formatting in DB operation failures

Created a helper function in db errors to capture, log and transform sql error to a readable format.
As part of that added 2 new labels to the prometheus error counters (query, select)
Identified db errors will now result in http status 400 from server (instead of internal server error)

@nopcoder @arielshaqed  Hi guys, please note that I plan on adding test cases to verify changes didn't break anything but I prefer to first get your initial feedback to understand if I'm on the right path.


Questions about the code semantic

Hi. I've read your paper and now I'm trying to understand your code. Most of the part is intuitive, thanks for the readable and runnable code :). Unfortunately I'm having trouble understanding the reason to use labels filled with only zeros. Please understand I'm quite new to pytorch so the question might be silly.
labels = torch.zeros(args.bsize, dtype=torch.long, device=DEVICE)
...
loss = criterion(out, labels[:out.size(0)])

Hi Kyoungrok! Thanks for your question.
For optimization, we treat the task as binary classification between <query, +ve passage> and <query, -ve passage>. The positive passages are placed in position zero (and the negatives in position one), hence the labels are all zeros.

Thanks for the answer. As I observe out = (bsize, 2) and labels = (bsize) with zeros. As your explanation If I do F.cross_entropy(out, labels) then the labels evaluated only on the first column (positive samples) so we have binary classification effect? I think I'm lack of knowledge how this cross_entropy function works.

It's just standard cross-entropy with softmax, over two classes, with the labels corresponding to the positive passages.
So, yes, this is binary classification between the positive and negative examples. Notice this is different from pointwise binary classification (namely, classifying each passage as positive or negative in isolation).


[3.2] C#: Re-work solution build output panel

The bottom panel button was renamed from Mono to .NET and now display an error/warning icon if the last build had issues.
Removed item list that displayed multiple build configurations launched. Now we only display the last build that was launched.
Display build output next to the issues list. Its visibility can be toggled off/on. This build output is obtained from the MSBuild process rather than the MSBuild logger. As such it displays some MSBuild fatal errors that previously couldn't be displayed.
Added a context menu to the issues list with the option to copy the issue text.

Replaced the 'Build Project' button in the panel with a popup menu with the options:

Build Solution
Rebuild Solution
Clean Solution

This PR is for the 3.2 branch. I'll be opening one for master/4.0 soon.

The bottom panel button was renamed from Mono to .NET

Why make this change now? I thought we were still using Mono?

The bottom panel button was renamed from Mono to .NET

Why make this change now? I thought we were still using Mono?

We're still using the Mono runtime. But it doesn't make much sense to use the name of the runtime (Mono or CoreCLR) here. .NET is the term most developers are familiar with.

I would argue that Godot developers are extremely familiar with "Mono", since we have the "Mono module" and on the download page it's called "Mono version (C# support)". I think labeling things as .NET right now would cause confusion with people thinking that Godot is either using .NET Framework or .NET Core when it's not.

The new "Build" popup menu doesn't really stand out as being an interactive menu, maybe it can be styled differently? (@Calinou @YeldhamDev)
Same for the other one on the same line, which also seem to have a toggled state (blue font) but it's not super obvious IMO that they're toggleable. UX advice welcome (also ping @groud).

Setting flat to off would likely help indicate that those are buttons. Either that or icons.

UX advice welcome (also ping @groud).

This is what I would do personally:

Move the actions outside of the menu. We can have three buttons there, there is no need to hide 3 actions when we have plenty of space. Eventually move the buttons to the bottom, so that it looks less like a title.
Use icons for each action, or, as @YeldhamDev suggested, make the buttons not flat.
I am not sure what the Warning/Errors buttons are for. If those are used to toggle the visibility, I am not sure it is very useful. We don't have it in the debugger, so I would keep things consistent there.
If there is only one tab, using tabs is not very useful. So I'd remove the tab.

This is what I would do, but he current interface is already quite good. It can be merged in the current state too.

Also, if there is only one tab, maybe instead of "Mono" or ".NET" this menu should be "MSBuild"?

The new "Build" popup menu doesn't really stand out as being an interactive menu, maybe it can be styled differently?

I thought so too, but this seems to be the look of menu buttons. It's the same for the text editor for example, but it's specially confusing in this case with only one button.

Move the actions outside of the menu. We can have three buttons there, there is no need to hide 3 actions when we have plenty of space.

I think that would look bad unless they were icon-only buttons (and I can't find any icons for them). It also is an overload of information as it's not very common to rebuild or clean the solution.

I am not sure what the Warning/Errors buttons are for. If those are used to toggle the visibility, I am not sure it is very useful. We don't have it in the debugger, so I would keep things consistent there.

Pretty much every editor/IDE has this option, and I myself found it useful many times when there are lots of warnings. That said I will make them use icons as suggested by @YeldhamDev.
I'm not sure about them being flat. Are toggle buttons in the Godot editor generally flat or non-flat?

If there is only one tab, using tabs is not very useful. So I'd remove the tab.

Initially the plan was to add other tabs later on, but the only ones I have in mind now are not for sure and may even be better somewhere else. So yeah, I suppose it's better to remove them and go with the "MSBuild" name for the bottom panel as @aaronfranke suggested.

Updated. Did everything I mentioned in my last comment. Additionally:

Added an icon to the Build menu button to make it more clear it's not a label.
Made the Show Warnings, Show Errors and Show Output buttons non-flat to make it more clear they're not labels.
Removed spacers. All those buttons are aligned to the left now.

Let me know what you think. (You can compare before and after in the top comment edition history).

One more recommendation: Move the "Show Output" button to the right, together with the side that the output actually appears.

Let's merge to give it some testing in the upcoming 3.2.4 beta 1, and we can improve from there.

Thanks!


'stopOldScripts' event is sent multiple times.

Describe the bug
I have multiple wxt extension .
Then the "stopOldScripts" event is sent multiple times.
Then "ctx.onInvalidated" will be called.
Reproduction
Not required
Steps to reproduce
Not required
System Info
 System:
    OS: macOS 15.1
    CPU: (8) arm64 Apple M1
    Memory: 103.83 MB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 23.1.0 - /opt/homebrew/bin/node
    npm: 10.9.0 - /opt/homebrew/bin/npm
    pnpm: 9.12.3 - /opt/homebrew/bin/pnpm
  Browsers:
    Chrome: 130.0.6723.117
    Edge: 130.0.2849.80
    Safari: 18.1
  npmPackages:
    wxt: ^0.19.13 => 0.19.13

Used Package Manager
pnpm
Validations

[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Please follow the template and provide a reproduction, in this case two WXT extensions with some logs showing that one is effecting the other? If I understand your report correctly.
Will reopen once I have a reproduction.

Oh interesting, I thought I used the browser.runtime.id in the event name... Woops.


chore(release): 2.64.0

See CHANGELOG

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 4b87d15651d760409a64d63b252de0e967622539
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 3fff2c2b21e3ef6a804bb41a5075000f31c40ec4
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Update http example integration test

This patch will remove the check for deploymentconfig and deployment
which is causing issue for windows.

/retest

/retest

@jsliacan @adrianriobo WDYT?

This PR fix related problem described at https://github.com/code-ready/crc/issues/1785. This problem was produced due to inconsistent resources expectations, with this PR this is solved as we ensure only deployment is created and it covers the same functionality..
The only comment is what @guillaumerose suggested using an image based on nginx, it seems it would not change the expected http response code 403 but the image size is 1/3.
IMAGE                                                   TAG                 IMAGE ID            SIZE
quay.io/bitnami/nginx                                   1.18.0              974152eb6fa49       93.3MB
registry.redhat.io/rhscl/httpd-24-rhel7                 latest              f6c37c3a78961       328MB

The only comment is what @guillaumerose suggested using an image based on nginx, it seems it would not change the expected http response code 403 but the image size is 1/3.

In short; approved?

quay.io/bitnami/nginx has a default webpage, so it will be 200.

If so, I would prefer go with quay.io/bitnami/nginx

do we have something that exists in an OpenShift namespace?

1m and 60s

I would prefer to see 1m... and everything above should be possible in 1m 30 instead of 90s... but enforcing this?
This is merely preference and more about maintainability... and should be for the whole test

/lgtm

/lgtm


Enable the JavaFX Maven plugin to support JDK 8 and above

The goal of this change is to make sure the JavaFX Maven plugin runs on JDK8, as well as later releases. On JDK8 it doesn't have to add any of the JavaFX libraries, as they are already in there.

This is a necessary update if NB is to support old JDK8 and new JDK releases at the same time.  Two very different requirements are necessary.  New project menu should indicate older JDK and newer JDK as required.

While I can build the plugin, using it like mvn clean javafx:run with JDK 8 still fails:
org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal org.openjfx:javafx-maven-plugin:0.0.4-SNAPSHOT:run (default-cli) on project hellofx: Error
 ...
Caused by: org.apache.maven.plugin.MojoExecutionException: Error
        at org.openjfx.JavaFXRunMojo.execute(JavaFXRunMojo.java:117)
 ...
Caused by: java.lang.IllegalArgumentException: invalid flag: --release
        at com.sun.tools.javac.api.JavacTool.processOptions(JavacTool.java:206)
        at com.sun.tools.javac.api.JavacTool.getTask(JavacTool.java:156)
        at com.sun.tools.javac.api.JavacTool.getTask(JavacTool.java:107)
        at com.sun.tools.javac.api.JavacTool.getTask(JavacTool.java:64)
        at org.codehaus.plexus.compiler.javac.JavaxToolsCompiler.compileInProcess(JavaxToolsCompiler.java:125)

The reason being the release flag used in Compile, which is not supported in Java 1.8.
This argument should be removed when usingOldJDK is true.

Even if we manage to fix the release flag, I wonder what possible use cases do we have to force the plugin being always compatible with JDK 1.8, and removing any possibility of using Java 9 and beyond features.
Wouldn't it be better to use profiles? If you want to keep your project compatible with JDK 8 and 11+, it doesn't really make sense to include in its pom: all the JavaFX 11+ dependencies and the JavaFX plugin for JavaFX 11, and then just remove its use, because you might want to run with 8.

With JDK 11+: mvn -PJava11 clean javafx:run...

Does this make sense?

While I can build the plugin, using it like mvn clean javafx:run with JDK 8 still fails:
org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal org.openjfx:javafx-maven-plugin:0.0.4-SNAPSHOT:run (default-cli) on project hellofx: Error
 ...
Caused by: org.apache.maven.plugin.MojoExecutionException: Error
        at org.openjfx.JavaFXRunMojo.execute(JavaFXRunMojo.java:117)
 ...
Caused by: java.lang.IllegalArgumentException: invalid flag: --release
        at com.sun.tools.javac.api.JavacTool.processOptions(JavacTool.java:206)
        at com.sun.tools.javac.api.JavacTool.getTask(JavacTool.java:156)
        at com.sun.tools.javac.api.JavacTool.getTask(JavacTool.java:107)
        at com.sun.tools.javac.api.JavacTool.getTask(JavacTool.java:64)
        at org.codehaus.plexus.compiler.javac.JavaxToolsCompiler.compileInProcess(JavaxToolsCompiler.java:125)

The reason being the release flag used in Compile, which is not supported in Java 1.8.
This argument should be removed when usingOldJDK is true.

https://github.com/openjfx/javafx-maven-plugin/pull/44/commits/1a4062226de31dfc197ab4bec7fd71ab975cd849
That should solve that.

It looks better now. I've left a few comments.
Also: We should consider the case where the project is modular and has a module-info descriptor.
Compiling for 8 works only if we exclude this descriptor. So far, the maven-compile-plugin supports this:
    ```

module-info.java

```
However, the JavaFX maven plugin doesn't support it yet, and the only way to make it work would be running mvn clean compile javafx:run. But we could introduce the excludes parameter, and add by default the module-info exclusion if Java.version is 1.8:
In JavaFXBaseMojo:
    /**
     * A list of excluded classes passed to the compiler.
     */
    @Parameter
    private List<String> excludes;

    void compile() throws MojoExecutionException {
        if (compilerArgs == null) {
            compilerArgs = new ArrayList<>();
        }
        if (excludes == null) {
            excludes = new ArrayList<>();
        }
        String specifyRelease = isOldJDK() ? null : release;
        Compile.compile(project, session, pluginManager, source, target, specifyRelease, compilerArgs, excludes);
    }

and in Compile:
public static void compile(MavenProject project, MavenSession session, BuildPluginManager pluginManager,
                               String source, String target, String release,
                               List<String> compilerArgs, List<String> excludes) throws MojoExecutionException {
...
        Element[] config = new Element[release == null ? 4 : 5];
        config[0] = element(name("source"), source);
        config[1] = element(name("target"), target);
        config[2] = element(name("compilerArgs"), compilerArgs.stream()
                .filter(Objects::nonNull)
                .map(s -> new Element("arg", s))
                .toArray(Element[]::new));
        if (release == null && ! excludes.contains("module-info.java")) {
            excludes.add("module-info.java");
        }
        config[3] = element(name("excludes"), excludes.stream()
                .filter(Objects::nonNull)
                .map(s -> new Element("exclude", s))
                .toArray(Element[]::new));
        if (release != null) {
            config[4] = element(name("release"), release);
        }
...
}

I like it José! Having this would certainly help NetBeans to support JFX across all important platforms.
Can you apply your changes and merge it all? Or will we tease @gwielenga with that? Geertjan isn't that good when it comes to applying patches & co.

OK, I believe all the comments from @jperedadnr have been integrated via my today's commit to the pr.

@geertjanw Can you please enable "Allow edits from maintainers" in the PR?

That's already been done.


nevermind

nevermind

you know what?  forget it.

Describe the bug
Hi. There's an extremely clear and well described bug with reproductions in #5470.
This bug also exists in many other bugs by other people, including many other reproduction cases, and it has been declared fixed twice.
Several core authors ran my reproduction case, agreed with it, diagnosed it, and located the problem code.
Unfortunately, you run a bot that closes issues after a month, so @kdy1 asked me to open a new ticket with a reproduction case, instead of just retaining the one there already is.
This is that ticket.
The reproduction case is in incorrectly closed ticket #5470.
I'm sorry that I sound so angry, but this is a significant bug that has already been scuttled half a dozen times, and I don't like that it's being scuttled again when several people already honored it, because each time I reopen this, I have to re-justify it.
Input code
The input code is in incorrectly closed ticket #5470.
Config
The config is in incorrectly closed ticket #5470.  It is the Deno config.
Playground link
No response
Expected behavior
The expected behavior is to keep valid tickets about medium severity parser bugs with good reproductions in real world high profile open source projects open, until they have been addressed.
Actual behavior
The ticket gets closed and you're told to start over from scratch. See #5470 .
Version
This appears to go back more than three years, but I cannot say how far. It is in current.
Additional context
I'm really sorry that I sound so angry
I hope that you guys reconsider swc-bot. This is the sixth time it has incorrectly buried this bug. I think bots closing issue tickets is very bad practice, personally.

I'll block you from the org


Does default sample rate cause resampling and setting buffer size for minimum latency?

Hello, Firstly HUGE thanks for the library. Working with AudioUnit RemoteIO is a real pain!
I am wondering two things:
1 - I see that you set the sample rate here with the following code:
    // TODO: check this works on iOS!
    _inputFormat.mSampleRate = 44100.0;
    _outputFormat.mSampleRate = 44100.0;

AFAIK newest iphones have a default sample rate of 48kHz, is there a resampling done to ensure 44100kHz gets resampled to 48kHz? Is this done by Apple automatically?
2 - Can I set the buffer size with AudioSessionSetProperty? Normally with AudioUnits you can ask for a preferred buffer size with the following code:
float aBufferLength = 0.005; // In seconds
AudioSessionSetProperty(kAudioSessionProperty_PreferredHardwareIOBufferDuration,
                        sizeof(aBufferLength), &aBufferLength);

Does this also affect novocaine's internal buffer size?

1 is Mentioned here, so I am closing this issue :)
https://github.com/alexbw/novocaine/pull/134


[WORKSPACE; 19.2 fabric] World Crash on lighting issue

"Bandaid" applied in commit linked below
Information:
Version used: Workspace Development
Workspace Instance: 1.19.2
Launcher Instance/Version: Farbic
Workspace Itteration used: as of Commit 588a554efa436138ba067f9f608b1a10e2ca7c63
Issue: Upon world Load after a minute or so the workspace crashes wtih an error regarding a lighting issue
Additional Info: issue ceased when Phosphorus was disabled from the workspace
Crash Report [GameLog if former is not available]:
crash-2023-02-15_17.18.43-client.txt
Report POI: ```Description: Unexpected error
java.lang.NoSuchMethodError: 'void net.minecraft.world.level.lighting.LayerLightEngine.method_15483(long)'
at net.minecraft.world.level.lighting.LayerLightEngine.cancelUpdatesForChunk(LayerLightEngine.java:698)
at net.minecraft.world.level.lighting.LayerLightSectionStorage.handler$beg000$preRemoveSection(LayerLightSectionStorage.java:816)
at net.minecraft.world.level.lighting.LayerLightSectionStorage.clearQueuedSectionBlocks(LayerLightSectionStorage.java)
at net.minecraft.world.level.lighting.LayerLightSectionStorage.addQueuedLightmaps(LayerLightSectionStorage.java:1423)
at net.minecraft.world.level.lighting.LayerLightSectionStorage.markNewInconsistencies(LayerLightSectionStorage.java:852)
at net.minecraft.world.level.lighting.LayerLightEngine.runUpdates(LayerLightEngine.java:186)
at net.minecraft.world.level.lighting.LevelLightEngine.runUpdates(LevelLightEngine.java:63)
at net.minecraft.client.renderer.LevelRenderer.renderLevel(LevelRenderer.java:1198)
at net.minecraft.client.renderer.GameRenderer.renderLevel(GameRenderer.java:1039)
at net.minecraft.client.renderer.GameRenderer.render(GameRenderer.java:816)
at net.minecraft.client.Minecraft.runTick(Minecraft.java:1177)
at net.minecraft.client.Minecraft.run(Minecraft.java:768)
at net.minecraft.client.main.Main.run(Main.java:244)
at net.minecraft.client.main.Main.main(Main.java:51)
at net.fabricmc.loader.impl.game.minecraft.MinecraftGameProvider.launch(MinecraftGameProvider.java:461)
at net.fabricmc.loader.impl.launch.knot.Knot.launch(Knot.java:74)
at net.fabricmc.loader.impl.launch.knot.KnotClient.main(KnotClient.java:23)
at net.fabricmc.devlaunchinjector.Main.main(Main.java:86)

Head --
Thread: Render thread
Stacktrace:
at net.minecraft.world.level.lighting.LayerLightEngine.cancelUpdatesForChunk(LayerLightEngine.java:698)
at net.minecraft.world.level.lighting.LayerLightSectionStorage.handler$beg000$preRemoveSection(LayerLightSectionStorage.java:816)
at net.minecraft.world.level.lighting.LayerLightSectionStorage.clearQueuedSectionBlocks(LayerLightSectionStorage.java)
at net.minecraft.world.level.lighting.LayerLightSectionStorage.addQueuedLightmaps(LayerLightSectionStorage.java:1423)
at net.minecraft.world.level.lighting.LayerLightSectionStorage.markNewInconsistencies(LayerLightSectionStorage.java:852)
at net.minecraft.world.level.lighting.LayerLightEngine.runUpdates(LayerLightEngine.java:186)
at net.minecraft.world.level.lighting.LevelLightEngine.runUpdates(LevelLightEngine.java:63)
at net.minecraft.client.renderer.LevelRenderer.renderLevel(LevelRenderer.java:1198)
at net.minecraft.client.renderer.GameRenderer.renderLevel(GameRenderer.java:1039)

Have you made sure it doesn't happen without hex casting? I don't see what in hex could be causing this

How would I do that? It's literally what the workspace is built off of.
But if it's any correlation, I am using the computer with an AMD radon graphics card

To be honest, that would be more plausible… Considering there's some drinking shit that goes on between Nvidia and AMD

Recommended course of action: make an addendum or some thing to notify the user to disable phosphorus if they're running into any issues and/or if they are using AMD graphics, as it may be just a case of phosphor being more designed to run on the Nvidia graphics than AMD graphics, which is plausible

I don't really see how this is our problem?


add sideEffects false to modules/package.json

Thanks for all the work done on this project.
I've been trying to use tree-shacking with the module. Looks that the current convention among bundler is to assume all code has side effects (cannot be removed, even if not used directly), unless there is sideEffect: false set on package.json (https://github.com/evanw/esbuild/issues/50#issuecomment-634107700)
The solution I'm proposing here can be see in action here:

https://github.com/marcin-wosinek/underscore-tree-shaking/commit/ec894eb66d0d20b4f32048e128f800fd21cbed31 build has 19.7 KiB:

$ npm ci; npm run webpack

...

asset main.js 19.7 KiB [emitted] [minimized] (name: main)
orphan modules 83.1 KiB [orphan] 161 modules
runtime modules 670 bytes 3 modules
./src/index.js + 161 modules 83.2 KiB [built] [code generated]
webpack 5.50.0 compiled successfully in 1252 ms

while https://github.com/marcin-wosinek/underscore-tree-shaking/commit/9a52666b8cdbe2dc9f8939ef9361fffdc11d05a3 build to 2.9 KiB:
$ npm ci; npm run webpack

...

asset main.js 2.9 KiB [emitted] [minimized] (name: main)
orphan modules 83.1 KiB [orphan] 161 modules
./src/index.js + 26 modules 12.4 KiB [built] [code generated]
webpack 5.50.0 compiled successfully in 754 ms

Sorry but no. This will actually break things because of https://underscorejs.org/docs/modules/index-default.html (note "Add all of the Underscore functions to the wrapper object"). In other words, the fully composed Underscore cannot be tree-shaken because all functions are mixed into _.
You can have perfect "treeshaking" by composing your own Underscore. I extensively documented how to do that in https://juliangonggrijp.com/article/introducing-modular-underscore.html (which is also linked from the Underscore website). If you prefer a video introduction, refer to https://www.telerik.com/react-wednesdays/what-i-learned-modularizing-underscore-with-julian-gonggrijp or https://www.youtube.com/watch?v=d4ORLdr3vWE.


Increase test coverage

PR #117 adds display of test coverage as a badge in the README, on every Python 3.6 travis run and on local test runs using tox.
Let's work on increasing coverage on these files (in priority order):

[ ] archive.py
[ ] publish.py
[ ] handlers.py
[ ] clone.py
[x] utils.py

This will get us closer to our release benchmark in #96.

#127 adds tests to reach 100% file coverage of s3_paths.py

#128 adds a few tests for archive and a test stub file for publish.
@mpacer @MSeal @rgbkrk After these land we should be at approx 49% coverage.

Now @mpacer's latest PR pushes us to high 60s. Woo! 👍

We landed on 79% by the time of release — it feels pretty darn good.
Closing!