NuGet.Server - 406 (Not Acceptable) Error

When pushing to a new deployment of NuGet.Server returns:
Pushing PackageName.2.0.6501.26693.nupkg to 'http://myserver.net/nuget'...
PUT http://myserver.net/nuget/
NotAcceptable http://myserver.net/nuget/ 614ms
Response status code does not indicate success: 406 (Not Acceptable).
Not sure what else you might want to see to diagnose this.  I couldn't find anything online to indicate the cause.

Hey @BrianKrohn, could you help with the following questions?

What version of NuGet.Server are you using?
What version of the client are using to push the package?
Could you provide a Fiddler trace of the push request?

I was using the latest when I reported this, but the browser shows "v<IP_ADDRESS>".  I cloned the master again and tried to use it, but you've moved to C# 7.0.  My company won't upgrade to VS2017, so I am unable to build.  I will attempt to get a version prior to the upgrade.  I am not that familiar with Git, so any help here would be appreciated.

I was using the latest when I reported this, but the browser shows "v<IP_ADDRESS>". I cloned the master again and tried to use it, but you've moved to C# 7.0. My company won't upgrade to VS2017, so I am unable to build. I will attempt to get a version prior to the upgrade. I am not that familiar with Git, so any help here would be appreciated.

Ah, I didn't realize you were building NuGet.Server from source. Alternatively, you could use the NuGet package and install it to an empty web project (.NET Framwork 4.6).
You can use this command to get the last release prior to moving to C# 7.
git checkout a66eaf4fe4fb6dd2c1f5577b9000de10acd22b4e

OK, I created a new, empty web app, and got the NuGet.Server package.  It built and deployed, but the site just returns HTTP 500.  I've tried giving IUSR and the IIS_IUSRS group full control access.  There's nothing logged about the error and Fiddler doesn't show anything that I can tell.

Got past the 500 error.  Turns out there are 2 "" tags in the web config.  One for 4.6.1 and one for 4.6.  Removing either produces:

Server Error in '/' Application.
Compilation Error
Description: An error occurred during the compilation of a resource required to service this request. Please review the following specific error details and modify your source code appropriately.
Compiler Error Message: The compiler failed with error code -2146232576.

There is also a long message below the error mentioning "roslyn\csc.exe".

Could you provide the long error message? I will look into the duplicate <compilation targetFramework="..."  I found this exception in the Windows application logs:

Exception information:
Exception type: MissingMethodException
Exception message: Method not found: '!!0[] System.Array.Empty()'.
at NuGet.Server.Infrastructure.ServerPackageRepository.AddPackage(IPackage package)
at NuGet.Server.Publishing.PackageService.CreatePackage(HttpContextBase context)
at nuget_server.NuGetRoutes.<>c.b__1_0(RequestContext context) in C:\Users\BrianKrohn\Downloads\Projects\nuget-server\nuget-server\DataServices\Routes.cs:line 24
at System.Web.HttpApplication.CallHandlerExecutionStep.System.Web.HttpApplication.IExecutionStep.Execute()
at System.Web.HttpApplication.ExecuteStep(IExecutionStep step, Boolean& completedSynchronously)

It happens when I try to push a package to the server.  I also get this error when clicking the "Click here to view your packages" link.

Hope it's cool I chime in here:
Not sure if this really helps debug the issue - but I had this issue this afternoon. Ultimately, I was test pushing a package that was already there and got 406'd.
All I had to do was change allowOverrideExistingPackageOnPush to true and voila!
Brian, I also had multiple duplicate web.config settings in the source I downloaded. After verifying there was only 1  tag and the system.webServer/handlers/add tags were unique, the 500 error went away and I was able to publish to my local server.
Hope this helps!

Hello,
I Had the same issue and for me it was IIS apppool permission to write to package folder.
I solve the problem with changing packagesPath key in web.config
Hope this helps !

I have tried all of the above suggestions

ensured the apppool user has write permissions to the Packages directory
removed the duplicate  tag
set allowOverrideExistingPackageOnPush to true

But I am still receiving a 406 when I try to push. I am using NuGet.Server version <IP_ADDRESS> and NuGet CLI version <IP_ADDRESS>06. Any suggestions?

Hey, @haughtm, could you add the following lines to your app startup, reproduce the issue, and then take a look at the log file? Modify the path as necessary.
Trace.Listeners.Add(new TextWriterTraceListener(@"C:\some\path\to\NuGet.Server.log"));
Trace.AutoFlush = true;

Also, could you provide the console output (stdout) of NuGet CLI command you run, including the parameters to nuget.exe? (feel free to obfuscate secrets 😄).

Hi @joelverhagen, thank you for the quick reply. Here is the output..
Here is the CLI output
nuget push automation.services.logging/bin/Debug/automation.services.logging.1.0.0.nupkg myapikey -Source http://nuget.mydomain/nuget
Pushing automation.services.logging.1.0.0.nupkg to 'http://nuget.mydomain/nuget'...
PUT http://nuget.mydomain/nuget/
NotAcceptable http://nuget.mydomain/nuget/ 207ms
406 (Not Acceptable)
And the Nuget.Server.log file shows
Info: Start rebuilding package store...
Info: Start reading packages from disk...
Info: Finished reading packages from disk.
Info: Start adding packages from drop folder.
Info: Finished adding packages from drop folder.
Info: Finished rebuilding package store.
Verbose: Created FileSystemWatcher - monitoring D:\NuGet Packages.
Info: Registering background jobs...
Info: Finished registering background jobs.
Info: Start rebuilding package store...
Info: Start reading packages from disk...
Info: Finished reading packages from disk.
Info: Start adding packages from drop folder.
Info: Finished adding packages from drop folder.
Info: Finished rebuilding package store.
Verbose: File system changed. File: myservername.cache.bin - Change: Changed
Info: Cleared package cache.
Those last 2 lines are repeated 387 times.

update
The package i was trying to push was created with command 'dotnet pack' and not 'nuget pack'. I just tried to push an application packaged with 'nuget pack' and it worked fine. This may simply be an issue related to dotnet core packages. I'll continue troubleshooting on my side.
output
5cg4450h24:automation.billing.data mhaught$ nuget push automation.billing.data.1.0.6610.22791.nupkg myapikey -Source http://nuget.mydomain/nuget
Pushing automation.billing.data.1.0.6610.22791.nupkg to 'http://nuget.mydomain/nuget'...
PUT http://nuget.mydomain/nuget/
Created http://nuget.mydomain/nuget/ 161ms
Your package was pushed.

Could you provide your .nuspec and recursive directory listing of the .nupkg?

I didn't create a nuspec file, just used the csproj file. Here is the output of the pack command.
5cg4450h24:automation.services.logging mhaught$ dotnet pack
Microsoft (R) Build Engine version <IP_ADDRESS>775 for .NET Core
Copyright (C) Microsoft Corporation. All rights reserved.
Restore completed in 53.93 ms for /Users/mhaught/Projects/automation.services.logging/automation.services.logging/automation.services.logging.csproj.
automation.services.logging -> /Users/mhaught/Projects/automation.services.logging/automation.services.logging/bin/Debug/netstandard2.0/automation.services.logging.dll
Successfully created package '/Users/mhaught/Projects/automation.services.logging/automation.services.logging/bin/Debug/automation.services.logging.1.0.0.nupkg'.

You can unpack the produced .nupkg as a .zip file and find both the directory structure and the output .nuspec (these two pieces of information imply that package compatibility -- which I imagine may be the problem).

Sorry, I didn't realize that.
Here is the nuspec

automation.services.logging
1.0.0
automation.services.logging
automation.services.logging
false
Package Description

And the directory structure

_rels
[Content_Types].xml
automation.services.logging.nuspec
lib

netstandard2.0

automation.services.logging.dll

package

services

metadata

core-properties

477a54281d484cabbca2b824c74878ea.psmdcp

Interesting, I tried producing a package targeting netstandard2.0 with .NET CLI and NuGet.Server accepted it just fine. Could you add a Trace exception logger to your application? I have a branch here with an example of how to do this:
https://github.com/NuGet/NuGet.Server/commit/a7fc19fe0217b6c1662f541cf59832a788542e29
(as a side note, I plan on adding this in a 3.2.0 release, if I have time)
After you add that exception logger, reproduce the issue then check the log file you instrumented above. I was, for example, able to reproduce the issue by manually changing the name of the .nuspec file and the package ID therein (thus breaking the OPC metadata which NuGet.Server requires).
Also, could you upload the package here so I can reproduce it locally? If you are concerned about sharing your package publicly, feel free to upload it to a file hosting site like OneDrive or Google Drive and sending a link to us at<EMAIL_ADDRESS>

So, I ended up recreating this library from scratch and it's working fine now. Thank you so much for all your help.

Hi @joelverhagen  I have  tried the above to set permissions to iis_iusr & also everyone.
 verb="GET,HEAD,POST,DEBUG,PUT,DELETE,PATCH,OPTIONS" modules="IsapiModule" scriptProcessor="%windir%\Microsoft.NET\Framework\v4.0.30319\aspnet_isapi.dll" preCondition="classicMode,runtimeVersionv4.0,bitness32" responseBufferLimit="0" />       <add name="ExtensionlessUrlHandler-ISAPI-4.0_64bit" path="*." verb="GET,HEAD,POST,DEBUG,PUT,DELETE,PATCH,OPTIONS" modules="IsapiModule" scriptProcessor="%windir%\Microsoft.NET\Framework64\v4.0.30319\aspnet_isapi.dll" preCondition="classicMode,runtimeVersionv4.0,bitness64" responseBufferLimit="0" />       <add name="ExtensionlessUrlHandler-Integrated-4.0" path="*."  When I push from build server to nuget server , I get Response status code does not indicate success: 500 (Internal Server Error).
cmd:> nuget push reflib.1.0.0.nupkg -source http://desktoptopname/ -apikey 123

Hey @ram1421, could you open a new issue please? This particular issue is getting really long (and therefore notifying a lot of people). Additionally, it seems like there are multiple root causes going on. This thread seems to be related to HTTP 406. You problem is an HTTP 500.
Thanks!

I have resolved this issue giving proper Full Control rights on ~/Packages directory to a user the application pool runs under.

Just as a note to anyone else who might be having this problem.  I was getting a 500 when trying to list packages, and a 406 when trying to push.  The solution for me was to give the app pool user in IIS full permissions to the /Packages directory, as mentioned above.

I have got the same error code...
I want to push nuget package to self-hosted nuget manager and it returns 406.

Failed to process request. 'Not Acceptable'.The remote server returned an error: (406) Not Acceptable..

I got 406 (Not Acceptable) because I got the same DLL twice in the .nupkg (in the same directory). Yes, it is possible if you misconfigure your .nuspec file. It took a while until I realized this caused the issue. Some more meaningful error message would be greatly appreciated...

I didn't receive 406 error after I changed package folder location.

In my case switching from the deprecated licenseUrl element to the license element seems to have fixed the issue.
I get that'll teach me to not be lazy and read the warnings 🙃

i received 406 because my nuget package had a src dir.
the nuget server handles packages with src dirs as symbol packages, so in the web.config you have to set "ignoreSymbolsPackages" to false.

I am still having the same issue.
This is the command I am using
nuget.exe push My.Packahe.Name.1.0.0.nupkg -Source http://myserverURL.com/nuget
I have the following setting set
<add key="requireApiKey" value="false" />
<add key="apiKey" value="" />
<add key="packagesPath" value="Packages" />

My packages folder is listed in the root "next to the Web.confg file.
I also tried to change <add key="ignoreSymbolsPackages" value="true" /> to <add key="ignoreSymbolsPackages" value="false" />
I tried with client version 4.9.3 and they upgraded to 5.3.0 with the same issue.
I gave IUSR full control over the Packages folder but that did not fix the issue. Any idea on how to resolve this issue?

I have the same problem
This is a piece of log information.
xsp4.exe Error: 0 : System.IO.IOException: Error reading MIME multipart body part. ---> System.IO.IOException: Unable to read data from the transport connection: Operation on non-blocking socket would block. ---> System.Net.Sockets.SocketException: Operation on non-blocking socket would block   at System.Net.Sockets.Socket.Receive (System.Byte[] buffer, System.Int32 offset, System.Int32 size, System.Net.Sockets.SocketFlags socketFlags) [0x00010] in /Users/builder/jenkins/workspace/build-package-osx-mono/2019-06/external/bockbuild/builds/mono-x64/mcs/class/referencesource/System/net/System/Net/Sockets/Socket.cs:1771    at System.Net.Sockets.NetworkStream.Read (System.Byte[] buffer, System.Int32 offset, System.Int32 size) [0x0009b] in /Users/builder/jenkins/workspace/build-package-osx-mono/2019-06/external/bockbuild/builds/mono-x64/mcs/class/referencesource/System/net/System/Net/Sockets/NetworkStream.cs:513     --- End of inner exception stack trace ---   at System.Net.Sockets.NetworkStream.Read (System.Byte[] buffer, System.Int32 offset, System.Int32 size) [0x000c3] in /Users/builder/jenkins/workspace/build-package-osx-mono/2019-06/external/bockbuild/builds/mono-x64/mcs/class/referencesource/System/net/System/Net/Sockets/NetworkStream.cs:525    at Mono.WebServer.XSP.XSPWorker.Read (System.Byte[] buffer, System.Int32 position, System.Int32 size) [0x0000a] in /Users/builder/jenkins/workspace/build-package-osx-mono/2019-06/external/bockbuild/builds/mono-xsp-c98e068/src/Mono.WebServer.XSP/XSPWorker.cs:207    at Mono.WebServer.BaseRequestBroker.Read (System.Int32 requestId, System.Int32 size, System.Byte[]& buffer) [0x0007e] in /Users/builder/jenkins/workspace/build-package-osx-mono/2019-06/external/bockbuild/builds/mono-xsp-c98e068/src/Mono.WebServer/BaseRequestBroker.cs:222    at (wrapper remoting-invoke-with-check) Mono.WebServer.BaseRequestBroker.Read(int,int,byte[]&)   at Mono.WebServer.XSPWorkerRequest.ReadInput (System.Byte[] buffer, System.Int32 offset, System.Int32 size) [0x00071] in /Users/builder/jenkins/workspace/build-package-osx-mono/2019-06/external/bockbuild/builds/mono-xsp-c98e068/src/Mono.WebServer.XSP/XSPWorkerRequest.cs:609    at Mono.WebServer.XSPWorkerRequest.ReadEntityBody (System.Byte[] buffer, System.Int32 size) [0x00042] in /Users/builder/jenkins/workspace/build-package-osx-mono/2019-06/external/bockbuild/builds/mono-xsp-c98e068/src/Mono.WebServer.XSP/XSPWorkerRequest.cs:629    at System.Web.HttpRequest.MakeInputStream () [0x0028a] in /Users/builder/jenkins/workspace/build-package-osx-mono/2019-06/external/bockbuild/builds/mono-x64/mcs/class/System.Web/System.Web/HttpRequest.cs:918    at System.Web.HttpRequest.get_InputStream () [0x00008] in /Users/builder/jenkins/workspace/build-package-osx-mono/2019-06/external/bockbuild/builds/mono-x64/mcs/class/System.Web/System.Web/HttpRequest.cs:1149    at System.Web.HttpRequestWrapper.get_InputStream () [0x00000] in /Users/builder/jenkins/workspace/build-package-osx-mono/2019-06/external/bockbuild/builds/mono-x64/mcs/class/System.Web/System.Web/HttpRequestWrapper.cs:152    at System.Web.Http.WebHost.HttpControllerHandler+<>c__DisplayClass13.<GetStreamContent>b__f () [0x00027] in <b2d5f889b8d5408ca3d4aa24eb5e947e>:0    at System.Web.Http.WebHost.HttpControllerHandler+LazyStreamContent.get_StreamContent () [0x00008] in <b2d5f889b8d5408ca3d4aa24eb5e947e>:0    at System.Web.Http.WebHost.HttpControllerHandler+LazyStreamContent.CreateContentReadStreamAsync () [0x00000] in <b2d5f889b8d5408ca3d4aa24eb5e947e>:0    at System.Net.Http.HttpContent.ReadAsStreamAsync () [0x0000e] in /Users/builder/jenkins/workspace/build-package-osx-mono/2019-06/external/bockbuild/builds/mono-x64/external/corefx/src/System.Net.Http/src/System/Net/Http/HttpContent.cs:258    at System.Net.Http.HttpContentMultipartExtensions.ReadAsMultipartAsync[T] (System.Net.Http.HttpContent content, T streamProvider, System.Int32 bufferSize, System.Threading.CancellationToken cancellationToken) [0x00082] in <dd5d13035e4d4a2e812d07ec2f066876>:0     --- End of inner exception stack trace ---   at System.Net.Http.HttpContentMultipartExtensions.ReadAsMultipartAsync[T] (System.Net.Http.HttpContent content, T streamProvider, System.Int32 bufferSize, System.Threading.CancellationToken cancellationToken) [0x00107] in <dd5d13035e4d4a2e812d07ec2f066876>:0    at NuGet.Server.V2.Controllers.NuGetODataController.UploadPackage (System.Threading.CancellationToken token) [0x000ef] in <16a18da2c3bf4533b42683dcedfda582>:0    at System.Threading.Tasks.TaskHelpersExtensions.CastToObject[T] (System.Threading.Tasks.Task1[TResult] task) [0x00063] in :0
at System.Web.Http.Controllers.ApiControllerActionInvoker.InvokeActionAsyncCore (System.Web.Http.Controllers.HttpActionContext actionContext, System.Threading.CancellationToken cancellationToken) [0x000c7] in :0
at System.Web.Http.Controllers.ActionFilterResult.ExecuteAsync (System.Threading.CancellationToken cancellationToken) [0x00137] in :0
at System.Web.Http.Dispatcher.HttpControllerDispatcher.SendAsync (System.Net.Http.HttpRequestMessage request, System.Threading.CancellationToken cancellationToken) [0x0018d] in :0
Info: Cleared package cache.
xsp4.exe Error: 0 : System.IO.IOException: Error reading MIME multipart body part. ---> System.IO.IOException: Unable to read data from the transport connection: Operation on non-blocking socket would block. ---> System.Net.Sockets.SocketException: Operation on non-blocking socket would block
at System.Net.Sockets.Socket.Receive (System.Byte[] buffer, System.Int32 offset, System.Int32 size, System.Net.Sockets.SocketFlags socketFlags) [0x00010] in /Users/builder/jenkins/workspace/build-package-osx-mono/2019-06/external/bockbuild/builds/mono-x64/mcs/class/referencesource/System/net/System/Net/Sockets/Socket.cs:1771
at System.Net.Sockets.NetworkStream.Read (System.Byte[] buffer, System.Int32 offset, System.Int32 size) [0x0009b] in /Users/builder/jenkins/workspace/build-package-osx-mono/2019-06/external/bockbuild/builds/mono-x64/mcs/class/referencesource/System/net/System/Net/Sockets/NetworkStream.cs:513
--- End of inner exception stack trace ---
at System.Net.Sockets.NetworkStream.Read (System.Byte[] buffer, System.Int32 offset, System.Int32 size) [0x000c3] in /Users/builder/jenkins/workspace/build-package-osx-mono/2019-06/external/bockbuild/builds/mono-x64/mcs/class/referencesource/System/net/System/Net/Sockets/NetworkStream.cs:525
at Mono.WebServer.XSP.XSPWorker.Read (System.Byte[] buffer, System.Int32 position, System.Int32 size) [0x0000a] in /Users/builder/jenkins/workspace/build-package-osx-mono/2019-06/external/bockbuild/builds/mono-xsp-c98e068/src/Mono.WebServer.XSP/XSPWorker.cs:207
at Mono.WebServer.BaseRequestBroker.Read (System.Int32 requestId, System.Int32 size, System.Byte[]& buffer) [0x0007e] in /Users/builder/jenkins/workspace/build-package-osx-mono/2019-06/external/bockbuild/builds/mono-xsp-c98e068/src/Mono.WebServer/BaseRequestBroker.cs:222
at (wrapper remoting-invoke-with-check) Mono.WebServer.BaseRequestBroker.Read(int,int,byte[]&)
at Mono.WebServer.XSPWorkerRequest.ReadInput (System.Byte[] buffer, System.Int32 offset, System.Int32 size) [0x00071] in /Users/builder/jenkins/workspace/build-package-osx-mono/2019-06/external/bockbuild/builds/mono-xsp-c98e068/src/Mono.WebServer.XSP/XSPWorkerRequest.cs:609
at Mono.WebServer.XSPWorkerRequest.ReadEntityBody (System.Byte[] buffer, System.Int32 size) [0x00042] in /Users/builder/jenkins/workspace/build-package-osx-mono/2019-06/external/bockbuild/builds/mono-xsp-c98e068/src/Mono.WebServer.XSP/XSPWorkerRequest.cs:629
at System.Web.HttpRequest.MakeInputStream () [0x0028a] in /Users/builder/jenkins/workspace/build-package-osx-mono/2019-06/external/bockbuild/builds/mono-x64/mcs/class/System.Web/System.Web/HttpRequest.cs:918
at System.Web.HttpRequest.get_InputStream () [0x00008] in /Users/builder/jenkins/workspace/build-package-osx-mono/2019-06/external/bockbuild/builds/mono-x64/mcs/class/System.Web/System.Web/HttpRequest.cs:1149
at System.Web.HttpRequestWrapper.get_InputStream () [0x00000] in /Users/builder/jenkins/workspace/build-package-osx-mono/2019-06/external/bockbuild/builds/mono-x64/mcs/class/System.Web/System.Web/HttpRequestWrapper.cs:152
at System.Web.Http.WebHost.HttpControllerHandler+<>c__DisplayClass13.b__f () [0x00027] in :0
at System.Web.Http.WebHost.HttpControllerHandler+LazyStreamContent.get_StreamContent () [0x00008] in :0
at System.Web.Http.WebHost.HttpControllerHandler+LazyStreamContent.CreateContentReadStreamAsync () [0x00000] in :0
at System.Net.Http.HttpContent.ReadAsStreamAsync () [0x0000e] in /Users/builder/jenkins/workspace/build-package-osx-mono/2019-06/external/bockbuild/builds/mono-x64/external/corefx/src/System.Net.Http/src/System/Net/Http/HttpContent.cs:258
at System.Net.Http.HttpContentMultipartExtensions.ReadAsMultipartAsync[T] (System.Net.Http.HttpContent content, T streamProvider, System.Int32 bufferSize, System.Threading.CancellationToken cancellationToken) [0x00082] in :0
--- End of inner exception stack trace ---
at System.Net.Http.HttpContentMultipartExtensions.ReadAsMultipartAsync[T] (System.Net.Http.HttpContent content, T streamProvider, System.Int32 bufferSize, System.Threading.CancellationToken cancellationToken) [0x00107] in :0
at NuGet.Server.V2.Controllers.NuGetODataController.UploadPackage (System.Threading.CancellationToken token) [0x000ef] in <16a18da2c3bf4533b42683dcedfda582>:0
at System.Threading.Tasks.TaskHelpersExtensions.CastToObject[T] (System.Threading.Tasks.Task1[TResult] task) [0x00063] in <f99f496cb0d249c1a945c1fcabce1695>:0    at System.Web.Http.Controllers.ApiControllerActionInvoker.InvokeActionAsyncCore (System.Web.Http.Controllers.HttpActionContext actionContext, System.Threading.CancellationToken cancellationToken) [0x000c7] in <f99f496cb0d249c1a945c1fcabce1695>:0    at System.Web.Http.Controllers.ActionFilterResult.ExecuteAsync (System.Threading.CancellationToken cancellationToken) [0x00137] in <f99f496cb0d249c1a945c1fcabce1695>:0    at System.Web.Http.Dispatcher.HttpControllerDispatcher.SendAsync (System.Net.Http.HttpRequestMessage request, System.Threading.CancellationToken cancellationToken) [0x0018d] in <f99f496cb0d249c1a945c1fcabce1695>:0

+1, same bug.
My issue was that particular packages were too large (60 MB).  Glad I found it, it was copying DLL's to the /lib folder that should just have been references to other packages.
For future references, what would the default maximum allowed size be, and what web.config entries could allow for larger packages?
Thanks
J

For this also came down to the maxRequestLength and the maxAllowedContentLength (system.webserver/security/reuestFiltering/requestLimits).
To me, a 406 error for file limits makes sense.
To me, what is not acceptable is all the other reasons that are nuspec-oriented that we get a 406.  Just sayin'

In my case it was the problem that the path to the package was too long. The following link provided helpful information about how to increase the maximum number of characters alowed: https://www.intel.com/content/www/us/en/support/programmable/articles/000075424.html

For me it was also  the maxRequestLength and the maxAllowedContentLength. Seriously, why isn't there a log file produced by default?! This is how easy it is to get it done. Open the web.config and add the following lines under the configuration element:
  <system.diagnostics>
    <trace autoflush="true" indentsize="4">
      <listeners>
        <add name="myListener" type="System.Diagnostics.TextWriterTraceListener" initializeData="log.txt" />
        <remove name="Default" />
      </listeners>
    </trace>
  </system.diagnostics>

I believe the NuGet.Server package already adds the line:
            config.Services.Replace(typeof(IExceptionLogger), new TraceExceptionLogger());
inside the NuGetODataConfig.cs file. Make sure the account the app pool runs under (or by default IIS_IUSRS) has read/write access to that log location.

Just wanted to add the workaround from issue #6543 here.
On my case it happened a couple of times where Nuget packages from Nuget.org had 2 forward slashes in the [Content_Types].xml file.
as @chenriksson states:

As a workaround, you can try opening the nupkg (7-zip), and modifying the following line of [Content_Types].xml to only have a single leading '/'.

<Override ContentType="application/octet" PartName="//THIRD-PARTY-NOTICES"/>
would become
<Override ContentType="application/octet" PartName="/THIRD-PARTY-NOTICES"/>
Nuget Server then accepts it.


Possibly outdated training instructions in Quickstart

Just a minor issue, but I noticed that when executing the following instructions from section 2: Training your first model! of the Quickstart:
# Download some test data:
ns-download-data --dataset nerfstudio --capture-name=poster
# Train model
ns-train nerfacto --data data/nerfstudio/poster

I receive the following error:
ns-download-data --dataset nerfstudio --capture-name=poster
usage: ns-download-data [-h] {blender,friends,nerfstudio,record3d,dnerf,phototourism}

ns-download-data: error: unrecognized arguments: --dataset

Using ns-download-data -h was helpful and I was able to fix it with the modification:
ns-download-data nerfstudio --capture-name=poster

Are these instructions simply out of date? If so, I can create a quick fix and make a PR.

Seems you are right to me - make the PR!

A fix has been committed. Closing this. Thank you!


[Bug]: CI will not run fastcheck if ready label is applied

This is problematic as CI looks like it ran all tests but only ran the /pr/ ones

I believe this is by design? @khluu

The problem is that some tests are skipped in this case like core-test. I don't think this is covered by /pr/
Perhaps CI needs to add some logic that if fastcheck is not yet run, it should still be run even if ready label is applied?

Escalated to the vllm buildkite repo: https://github.com/vllm-project/vllm/issues/7306

This might be the source of the issue? @khluu  https://github.com/vllm-project/buildkite-ci/commit/b95784388d61fceb94d6830002757a390d45eb71

@jon-chuang  We also enabled conditional testing recently so that's why CI didn't run all tests like before but only ran some based on your changes. Core test in this case would only run if files in vllm/core, vllm/distributed, tests/core change.

fast-check was designed to run a subset of CI tests, so when ready label is applied, we should run CI and not fast-check

Makes sense, thank you.
I don't have access to buildkite to control the tests.

I just feel a little nervous when not all tests run.
Especially as it's weird to me that a test will run in fastcheck but not the full pipeline.

@jon-chuang  I can add you to the org on Buildkite so you can trigger more tests. What's a good email for that?
Ya we are still observing how conditional testing works, and if it's stable enough we probably will sunset fast-check.

sg. My github email is jonabirdd at gmail dot com

invited!


Upgrade blueprint to support Lean 4

Per our discussion on zulip, this PR upgrades blueprint to support Lean 4.
CI passed and blueprint and doc are published to https://utensil.github.io/LeanAPAP/.
Links to Blueprint or Documentation are working, and jumping from blueprint/depgragh to doc is also working (for theorems, because their names are all in lower case, unchanged from Lean 3 to Lean 4).

After careful review, this looks great. @zeramorphic has already performed similar changes to https://github.com/YaelDillies/LeanCamCombi, but without the blueprint part. Your changes seem to be functionally the same.
Could you get this PR ready to be merged?

@YaelDillies With the last three commits in, this PR is ready for merge.

Great.

CI is green, both doc and blueprint are published.
Some jumping issue examples:

almost_periodicity: should be AlmostPeriodicity.almost_periodicity
boring_energy_eq
add_salem_spencer.L2inner_mu_conv_mu_mu_two_smul_mu

These need to be adjusted to their Lean 4 counterparts in the blueprint tex.

Thanks! Fixed.


Change details survey

[x] part of the day with checkbox instead of radio
[x] altura da semana
[x] transportation with checkbox

Done! Also fix profile page ext to display new results.


[5.6] Added some basic ARIA mark-up to auth views

Hello, this is my first pull-request to Laravel. When looking at the auth forms created with artisan make:-auth, I noted that it would be neat to have some WAI-ARIA mark-up for further accessibility, so I decided to implement this here.

Does the <form> tag actually need a label? I'm only just getting started with WAI-ARIA markup but I haven't come across any resources that indicate a need to label the form.

I wouldn't say it is needed, but it is far more comfortable to know where the form actually starts and ends, especially in complicated layouts. If for some reason you find this inappropriate, it can be removed.

It's not that I find it inappropriate, more that I hadn't seen it used in this way before. I'm only just getting into accessibility on the web so my knowledge is fairly limited. However, using VoiceOver on Mac I couldn't seem to get it to identify forms by their "aria-label".


Suggestion: compact notation for an array of string literals

It's not that uncommon that I want a list of allowed keys or something like that. So not too infrequently I find myself writing
keys := 'skeleton blank house cipher car'.split ' '

which looks a bit goofy, but it's significantly pleasanter to type than
keys := ['skeleton', 'blank', 'house', 'cipher', 'car']

because of all those pinky and ring-finger keystrokes.
LiveScript had a compact notation for this, I think it was
keys = <[skeleton blank house cipher car]>

I don't love the <[ ... ]> as it looks sort of JSX-y, but I do think a compact notation for arrays of literal strings that don't contain whitespace would be pleasant/handy.
Brainstorming some options:
keys := [''skeleton blank house cipher car''] // it's kind of quote-y, riffs on triple quote
choices := ['|'option|choice|has space|freedom] // provides a place for the delimiter
answers := [[[correct dumb thoughtful wrong]]] // when in doubt, triple it; but it does shadow
// an array of one element, which is an array of one element, which is an array of one element
// produced by a chain of function calls. So that's likely unacceptable.

Anyhow, just a thought.

Neat idea! Some random thoughts:

Operator overloading is something we'd like in the future. Maybe "skeleton blank house cipher car"|" " -> "skeleton blank house cipher car".split(" "). We could have | with a string rhs to convert to split until we have full support for binary operator customization.
For static strings known at compile time we could even generate proper TS types for the split.

Another fun thing we could do is transpile 'skeleton blank house cipher car'.split ' ' to ['skeleton', 'blank', 'house', 'cipher', 'car'], as an optimization, similar to how [0..9] transpiles. This doesn't solve the beauty/notation though.
Another way to write this today would be ssv`skeleton blank house cipher car`  for an appropriate function ssv (named to mean space-separated data). I agree it'd be nice to have something built-in though.

Another way to write this today would be

True, and I guess you could type ssv so
keys := ssv`skeleton blank etc`

would get the literal type ['skeleton', 'blank', 'etc'], without too much difficulty. I should probably write that way in the meantime.
Another random syntax possibility:
keys := []skeleton blank house cipher car[]

I'm not saying any of the suggested syntaxes are perfect, but some seem plausibly acceptable. This one is the most compact I've thought of that is currently a parse error and therefore potentially up for grabs.

By the way, it's perhaps useful to reflect on other languages that have this feature:

Perl: qw(skeleton blank house cipher car) is shorthand for ("skeleton", "blank", "house", "cipher", "car"). You can also use []s or any matching character like qw!...! or qw/.../.
Ruby: %w(skeleton blank house cipher car) is shorthand for ["skeleton", "blank", "house", "cipher", "car"]. You can also use []s or any matching character like %w!...! or %w/.../.
Bash/Zsh: (skeleton blank house cipher car) is equivalent to ("skeleton" "blank" "house" "cipher" "car") (thanks to quoting rules)

qw and %w do look a lot like template syntax, but there's the nifty idea of having arbitrary delimiters (like LaTeX's \verb) which is nice if your strings have particular characters. Admittedly we're breaking on whitespace, so that's already a bit restrictive.
Inspired by your last idea, I wonder about this syntax: []`skeleton blank house cipher car`, as a generalization of template syntax... Although I'm not sure what ${} expansion would mean, especially if this is done at compile time, so maybe this is a false analog... Though we could define some reasonable meaning, e.g.:
[]`skeleton ${key} x${key} ${...more}` → ["skeleton", key, `x${key}`, ...more]

(though perhaps I'm getting carried away)

I am actually pretty comfortable with `[]``a b c``` - I think it's pretty suggestive of "make this literal into an array in the most obvious way". Covers my typical use case pretty nicely.
I suppose to avoid the specialness of breaking on whitespace, making it just a default, it could be nice to allow e.g.,
[|]`option A|choice B`

Or maybe better
['|']`option A|choice B`

So that
[foo]`stuff`

really just becomes syntactic sugar for stuff.split(foo), with a new default of ' ' if foo is not specified (which frankly I am surprised JavaScript does not already allow in split).

Nice, I like that generalization!
I think the default should be /\s+/, not ' ', matching the default for Python's split. (I agree it's annoying that it's not the default in JavaScript.) In the special case of the default, I think it also makes sense to remove an empty first or last item, so that []` a b c ` still produces ['a', 'b', 'c']. This is especially useful for multiple lines:
[]`
  option1
  option2
  option3
`

True, and I guess you could type ssv so

 keys := ssv`skeleton blank etc`

would get the literal type ['skeleton', 'blank', 'etc'], without too much difficulty

Well actually, I spoke too soon. I was going to contribute to this discussion some working correctly-typed code for ssv (that would split on /s+/) and also support
ssv('|')`My values|have spaces`
ssv(/[.]+/)`I...like ellipses... very much!`

mostly because I would use it anyway and possibly it could provide a compilation target for
[]`The  standard \n thing makes \t 6 words.`
[' ']`Only break \n on spaces to get an array of 12 elements.`
[/[{}() ]/]`You{[bracket buster,] you!}`

But then I ran into https://github.com/microsoft/TypeScript/issues/33304. (And very gratifyingly/amusingly one of the comments on that issue is someone explicitly trying to produce a tagged template replicating ruby %w[foo bar zaz] in Typescript. So we are in fact onto something...)
Oops, who knew? So I have given up on this micro-project for the time being. But in some sense it would make it even more useful/cool if Civet did implement this latest []-template literal notation as just above, which I get the sense the two of you like best from the options floated here, in a type-safe way today.

P.S. About the meaning of interpolation in
[foo]`Some ${substituted} x${pressions} to ponder${bang}`

I personally think that since the usual backticks just deal in strings so this feels "stringy" and feels mostly about literal arrays of literal strings, this syntax should just operate on the string level and so the above should precisely mean
`Some ${substituted} x${pressions} to ponder${bang}`.split(foo)

with the caveat of when foo is absent there's the special-case behavior of splitting on /\s+/ and discarding empty first and last chunks if present.
Moreover, I would suggest it is fine to type as string[] when foo or any interpolation is present. Any finer typing, e.g. if all of the interpolated variables are known constants and/or if foo is just a literal string, say, would be icing on the cake. Re this, note that in TypeScript today
const foo = `bar baz`  // types as 'bar baz'
const quux = `quince ${foo}` // types as string

Ruby: %w(skeleton blank house cipher car) is shorthand for ["skeleton", "blank", "house", "cipher", "car"]. You can also use []s or any matching character like %w!...! or %w/.../.

@edemaine @gwhitney @STRd6 +1 vote for this! I use %w[a b c] all the time in Ruby… Also, note that (uppercase) %W[a #{variable} c] allows interpolation, and also there is %d[1 2 3] for an array of integers, amongst other % markers…


Flashing to esp

I tried to use your web flasher tool to flash to a esp device. I tried a 8266 and a d1 mini and a esp32 and they all come back unsupported via the web browser. Can you fix this because even with the 2 different 8266 small and large it does the same. I do have the proper hardware just ur tool says no.

ESPHome web tools assumes that you use an ESP32, in the future I want to improve that. If you use a different board type, you have to work with the yaml file yourself.


Is there a way to input JSON datastructure to the orgchart plugin?

My data will come from MongoDB database and i need to display the parent-child relationship. I see that the examples are all using CSV data. Can i instead use JSON data?

Yes, org chart actually receives array of objects, so JSON will work without issues

It won't work with JSON objects nested in a tree structure though.  For example, data = JSON.parse('[{...}, {...},...]'), where each object is a single layer of key/value pairs with parentIds pointing to ids of other rows.

That's true, although it's fairly easy to convert nested tree structures to the flattened structure
#93 issue was about that


Cluster nodes cannot discover each other

Hi, first of all thanks for putting this chart together.
I have tried installing this chart with default values but cluster nodes can't discover each other and form a cluster. I suspect this has something to do with the fact k8s services don't support mDNS... Is there another way to make this work or need to add a config file?
Thanks!

you need to do some hack to the template
the way this chart using for communicate ipfs and ipfs-cluster is by headless-service
in my case, I uncomment this line, and fix this problem
https://github.com/monaparty/helm-ipfs-cluster/blob/37978fd2fbb56e4270c93ec5fea0e9638fd65037/templates/ipfs.service.yaml#L17

@Os-carsun does it works really well enough with that fix? For example, do repining works?


increase spacing between bars?

the barcodes aren't very readable on my labels, due to the low quality of my label printer.
because of this, i'm looking for a way to increase the spacing between bars, or, change the width ratio between bars and spaces.
if this is possible, i really wish to know how!

use the "width" property
https://github.com/lindell/JsBarcode/wiki/Options#width

if the width is too small, the barcode will not be scanned (ex: width: 0.6, printing paper length: 35mm).
Do you have any solution? @GiovanniCardamone

@thanglong134 width defines the width of a single bar, I think you should use multiples of 1. (1 means one pixel in canvas).
If you use 0.6 you will use half a pixel, and this means you will have an average value, this isn't correct for the barcode.


04153500 crashes pipeline at screen_daily_data

@jds485 and @cstillwellusgs when I reran the pipeline to make that nonstationary figure for the presentation the other day, site 04153500 crashed the pipeline after the prescreening step.  This is because the screen_daily_data function is mapping over has_data but this site gets removed in the prescreen_daily_data step, so when it tries to map over that has_data target, there's nothing there.  I think we had talked about this before and I'm not sure if it was on anyone's to-do list to fix it? If not - I can add an issue and fix it pretty quick.
Originally posted by @slevin75 in https://github.com/USGS-R/regional-hydrologic-forcings-ml/discussions/84

This site sounds familiar. If you don't mind holding off a bit I can try to remember.

ok, I'll hold off for now.


solution issue #70 in ADP cython and pure_python version

Rewriting the history for PR #71 by @diegodoimo via a new PR without the html file generated by Cython

Rebasing and merging after tests pass


Edit Page with Titan…	⌘E

What the title says. I'm sure this is already in the cards :)
Something like this?
Opens a dialog, pre-filled with the body of the current page, with a Save button instead of Upload.
Also, what about iapetus, if adding support for that (#414)?
Maybe servers should give clients hints about its capabilities through MIME parameters? text/gemini; lang=en; titan?

I've been thinking about it, although ⌘E would be pretty much just a shortcut for ⌘C, ⌘U, ⌘A, ⌘V. This would seem convenient.
It is up to the user to choose the upload protocol, perhaps via a site-specific config setting. It's extremely unlikely that this kind of MIME parameters will be included in the Gemini specification...

Yea, I don't expect the Gemini spec to embrace that. Chosing upload protocol per site would do the trick without bending Gemini's spec in uncomfortable positions :)

Implemented for v1.10.


setup.py wants to open non-existing file

setup.py has:
with open('README.rst') as f:
but there is no such file. I guess it means to use README.md.

Hey @0-wiz-0 – Sorry for the long delay getting back to you.
This is an artifact of the way I have csvtomd building. You'll need to use setup_wrap.py instead of setup.py. It builds the RST file from the MD readme, then runs the base setup.py.
Let me know if you have any other questions.

I just tried running setup_wrap.py. It didn't run for me because it wanted pypandoc which is a wrapper for pandoc which is written in Haskell. So to run it we need to install a complete different language. I think that's quite a heavy dependency.
My suggestion: please include the generated README.rst file in the distribution tarballs uploaded to pypi, because the file is static anyway. It is not necessary that every installation of csvtomd generates this file from scratch again, that can be done once when packaging a release.
Thanks.

I will definitely consider it. Right now the build script isn't properly generating the RST file anyway, so I'm also considering just moving from Markdown to RST. Thanks for the suggestion – I'll keep you posted when I resolve this.

8 years after. Extremely heavy haskel dependency is still in source code. Could you please generate .rst file and make it part of distribution?

@cheusov This Python project is discontinued. Please see the other projects listed in the readme.


Added new example

Added a new example where user can select all/unselect all options. I tried to run it locally in my machine but there is an compile error. The code should work though.

Hey, thanks a lot!
Would it perhaps be clearer to just use buttons for "Select all" and "Deselect all"?
Something like:
phones: ['apple', 'samsung', 'nokia'],

selectedPhones: [],

actions: {
  selectAllPhones: function () {
    this.set('selectedPhones', this.get('phones').copy());
  },

  deselectAllPhones: function () {
    this.set('selectedPhones', []);
  }
}

{{multiselect-checkboxes options=phones selection=selectedPhones}}

  <button {{action 'selectAllPhones'}}>Select all</button>
  <button {{action 'deselectAllPhones'}}>Deselect all</button>

Selected phones:

    {{#each phone in selectedPhones}}

{{phone}}

    {{/each}}

Hey I updated the example. BTW when I run ember -v under demo app, I got error message Cannot read property 'pkg' of null. Do you have the same problem?

:+1: Looks good to me, merging...
I think the error might be because the demo app's ember version is quite outdated. I'll look into updating it.


python interface for RecordIO

@piiswrong

Can you also write a explaination on how it can be used from python side
Current format was not specific to mxnet's io, so maybe consider move to dmlc-core?

I can write a test case.
dmlc doesn't seem to have a python interface lib.

please move the recordio related python to recordio.py and add a few docstrings. I will merge it after that


[fix] [broker] fix topic partitions was expanded even if disabled topic level replication

Motivation

The topic partitions on the remote cluster will be expanded when namespace level Geo-Replication is enabled, even if the topic level Geo-Replication is disabled. It is not correct.

The topic partitions on the remote cluster will not be expanded if the topic level Geo-Replication is enabled. It is not correct.

Modifications
Fix the two issues.
Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository: x

/pulsarbot rerun-failure-checks

This PR contains a gap reported as issue #22967 which is fixed by #22971.


Request value validation

Implement validation of request values (e.g. with FluentValidations).

Not really needed anymore because all of that is done in the models now.


How to implement Social login such as Facebook login

Hi this can be done easily using firebase
If we have to implement can you pls assign this issue to me?


Upload stops in the middle of a file and doesn't continue

For some reason, the upload is stopping in the middle of an upload. For example, at chunk 338 out of 926 and doesn't continue.
If I .pause() then .upload() again it seems to continue however chunks are then corrupted and doesn't upload correctly (confirmed by comparing hashes).
If I refresh the page and attempt a new upload of the same file, it goes through all the test requests and continues the upload just fine from that point.
I've been investigating into why this may be but cannot find what is wrong.

Seems to only occur using chrome, even with incognito, no extensions. A request can get stuck on "pending" in network tab despite being served by the backend.

@dannydjrs Hello, did you implement testMethod on the server side?


[SYSTEMML-1235] Migrate GNMFTest to new MLContext

LGTM when the tests are passed :+1:

Thank you for reviewing @niketanpansare ! All tests passed here: https://sparktc.ibmcloud.com/jenkins/job/SystemML-PullRequestBuilder/1083/


inferno: 0.11.21 -> 0.12.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/inferno/versions.
meta.description for inferno is: Port of parts of the flamegraph toolkit to Rust
meta.homepage for inferno is: https://github.com/jonhoo/inferno
meta.changelog for inferno is: https://github.com/jonhoo/inferno/blob/v0.12.0/CHANGELOG.md
Updates performed

Rust version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.12.0 with grep in /nix/store/59irxgd4ipyk07wgalm3mbklafjdq6gx-inferno-0.12.0
found 0.12.0 in filename of file in /nix/store/59irxgd4ipyk07wgalm3mbklafjdq6gx-inferno-0.12.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

inferno

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/59irxgd4ipyk07wgalm3mbklafjdq6gx-inferno-0.12.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A inferno https://github.com/r-ryantm/nixpkgs/archive/ccf1ea953ad3c966d9a1609650a9837843421368.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/ccf1ea953ad3c966d9a1609650a9837843421368#inferno

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/59irxgd4ipyk07wgalm3mbklafjdq6gx-inferno-0.12.0
ls -la /nix/store/59irxgd4ipyk07wgalm3mbklafjdq6gx-inferno-0.12.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

inferno

Maintainer pings
cc @figsoda for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 361962

aarch64-darwin

:white_check_mark: 1 package built:

inferno


Update location-condiUpdate

Added note that Named Locations objects marked as trusted cannot be deleted.

@MicrosoftGuyJFlo
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@junhasems so what should administrators do?

@MicrosoftGuyJFlo Yes, administrator can delete after unchecking and saving.

emailed MicrosoftGuyJFlo


Shared compilation script

Adding a shared compilation script since it is copied over to different
repositories.
The script is quite flexible to allow consuming repositories to adapt it
with their different requirements.

The builds fail because install.goreleaser.com’s certificate isn’t valid.


[Google Map Support] Implement mechanism to switch provider when base layer is changed

Background
CartoDB.js doesn't currently provide a specific mechanism to switch the base layer (which might have an associated layer for labels on top) of a map. This is currently done "manually" in the context of the BUILDER (i.e: base and labels layers are added/removed/replaced when a new basemap is selected.)
Existing types of base layers:
We currently have the following types of layers in our DB:
# SELECT DISTINCT(kind) FROM layers group by kind;
    kind
------------
 tiled
 gmapsbase
 torque
 carto
 background
 wms
(6 rows)

From those, tiled, gmapsbase, background, and wms can be used as base layers. And, there's a different implementation for each type of layer and provider (Leaflet and Google Maps):
Current implementation in CartoDB.js:

Leaflet:

leaflet-gmaps-tiled-layer-view.js (fallback gmaps -> leaflet (stamen, nokia) (*)
leaflet-plain-layer-view.js
leaflet-tiled-layer-view.js (also used as labels layer)
leaflet-wms-layer-view.js

(*) Not being used at the moment AFAIK.

Google Maps:

gmaps-base-layer-view.js (roadmap, gray_roadmap, dark_roadmap, hybrid, satellite, terrain)
gmaps-plain-layer-view.js
gmaps-tiled-layer-view.js

CAUTION: WMS layers doesn't seem to be currently supported when using Google Maps (this doesn't seem right).
Current basemap providers:
The BUILDER currently supports the following basemap providers:

CARTO (*) -> Tiled layer
Stamen (*) -> Tiled layer
Here (NOKIA) -> Tiled layer
Custom:

XYZ -> Tiled layer
MAPBOX -> Tiled layer
WMS/WMTS -> WMS layer
TILEJSON -> Tiled layer
NASA -> Tiled layer

Color -> Plain/Background layer

(*) Some basemaps support labels on top.
Basemaps are defined here (internal only).
viz.json examples:
Leaflet
A Leaflet map with a tiled base layer and some labels on top:
{
  ...,
  'map_provider': 'leaflet',
  ...
  'layers': [
    {
      ...
      'type': 'tiled',
      'options': {
        urlTemplate: '....',
        ...
      }
      ...
    },
    {
      ...
      'type': 'CartoDB',
      'options': { ... }
      ...
    },
    {
      ...
      'type': 'tiled',
      'options': {
        urlTemplate: '....',
        ...
      }
      ...
    },
  ],
  ...
}

Some interesting guides from the Google Maps JavaScript API docs:

Map Types
Custom Overlays

Google Map
A Google Maps map with a "roadmap" base layer and a torque layer on top:
{
  ...,
  'map_provider': 'googlemaps',
  ...
  'layers': [
    {
      ...
      'type': 'gmapsbase',
      'options': {
        base_type: 'roadmap',
        style: [ { ... }, ... ]
      }
      ...
    },
    {
      ...
      'type': 'CartoDB',
      'options': { ... }
      ...
    },
    {
      ...
      'type': 'torque',
      'options': { ... }
      ...
    }
  ],
  ...
}

The map_provider that is received in the viz.json is set as an attribute of the Map object.
Example, using the actual API:
This is how someone would switch to a Google Maps basemap, using what we have today (not working  #yet):
cartodb.createVis('map', function (vis, layers) {
  var baseLayer = vis.getLayer(0);
  var map = vis.map;

  // Remove the current baseLayer
  map.layers.remove(baseLayer);

  // Create a GMapsBase layer at position 0, silently
  map.createGMapsBaseLayer({
    base_type: 'roadmap',
    style: ''
  }, { at: 0, silent: true });

  // Change the map provider
  map.set({ provider: 'googlemaps' });
});

Tasks

[x] Review test suite and make sure map views and layer views are well tested.
[x] Re-render visView when map provider changes.

why do we want to move this to cartodb.js?

Acceptance
This is deployed to staging (ded03).

Login to https://carto-staging.com/superadmin.
Find a user in ded03.
Edit the user and enter a valid google-maps-api-key.
Login as that user.
Verify that you can switch between from and to Google Maps basemap. Test other types (eg: plain color).
Test with one and multiple layers.
Ensure infowindows and tooltips work properly after switching basemaps (specially to and from Google Maps).
Test that you can draw geometries before and after switching basemaps .
Test that you can edit geometries before and after switching basemaps.

I can set Google Basemaps but I couldn't edit any feature, there is no button set to edit, in staging:

In prod:

Alse there is a problem while setting zoom: InvalidValueError: setZoom: not a number

Also, every time I change basemaps zoom is automatically set to 18.

Moving this back to development. I'm taking a look at the problem with the zoom changing automagically. Thanks for reporting it @dgaubert!

I fixed the issues with the zoom (hopefully). I re-deployed this to staging. @dgaubert could you please verify that zoom errors are gone? Thanks a lot! cc: @rochoa

Sure thing!
I'll back here to give some feedback! ❤

I've found a little issue, when I add a pop-up to a layer for click event, the pop-up shows null values:

After a hard refreshing, it works as expected:

Not sure if it is related to basemaps, but I've checked it in prod and pop-up works without refreshing.
Apart from that, everything else works like a charm!!
Hope this helps!

Thanks @dgaubert! Opening a separate issue with this.

Not sure if it is related to basemaps

If not, then 🇨🇭


custom theme

i followed the steps to create a custom theme
rails generate pageflow:theme my_custom_theme
and put
# config.themes.register(:default)   config.themes.register(:my_custom_theme)
into
config/initializers/pageflow.rb
and changed
~/my_pageflow/app/assets/stylesheets/pageflow/themes/my_custom_theme/variables.scss
to
$theme-name: "my_custom_theme";
$main-color: #e74e25 !default;
$main-color-dark: #c72e05 !default;
$loading-spinner-background-color: #e74e25 !default;
goal:
change the color of the startting page (loading spinner?) to orange.
that didnt work.
i restartet the server
that didnt work.
how do i implement an own theme?
where do i find the meaning of the different variables (ive seen the document, where all possible variables are listed. but this is useless, because nowhere is stated, what each variable means, and the names are far from self explanatory).
the work with pageflow is an extremely frustrating endeavour.
and when i say extremely i was overly polite.

Well. Nobody forces you to use it. Let me just say that your overall tone in the conversation so far has been more then frustrating as well - and this is also being overly polite.
Feel free to come back, once you are ready to have a constructive conversation.

well... we'll see...


Additional fields to User - verified and lastLogin

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

Failing tests?


Change realm field to be not null in Attachment

As discussed in chat,  this realm field should  not be null in the class.
Testing plan:

Merged after rewriting the commit message to provide proper context, thanks @byshen!
Check out the Zulip commit message guidelines for more details: https://zulip.readthedocs.io/en/latest/contributing/version-control.html#commit-messages


(WIIP)puller(ticdc): refactor regionTsManager

What problem does this PR solve?

Issue Number: close #xxx
What is changed and how it works?
Check List
Tests

Unit test
Integration test

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
`None`.

/run-all-tests

/run-all-tests

Codecov Report

Merging #7457 (3701e82) into master (6b48846) will decrease coverage by 6.0998%.
The diff coverage is 60.6617%.

Additional details and impacted files

Flag
Coverage Δ

cdc
67.3476% <59.8167%> (-0.2902%)
:arrow_down:

dm
52.0159% <50.0000%> (?)

engine
61.2464% <74.0000%> (+0.0626%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
@@               Coverage Diff                @@
##             master      #7457        +/-   ##
================================================
- Coverage   65.9877%   59.8878%   -6.0999%
================================================
  Files           564        811       +247
  Lines         52337      92912     +40575
================================================
+ Hits          34536      55643     +21107
- Misses        15129      32420     +17291
- Partials       2672       4849      +2177


[LookupTables] Phase 1 implementation of LookupTables rebuild

to: @ryandeivert , @blakemotl , @chunyong-lin
cc: @airbnb/streamalert-maintainers
related to:
resolves:
Background
LookupTables is making a comeback, this time with twice the gusto!
LookupTables was originally a stealth feature that was released for a very specific purpose in the Rules Engine. This PR expands LookupTables to be a publicly available feature, now functional on all Lambda Functions, instead of just on the Rules Engine.
LookupTables is intended to be a simple key value store that any StreamAlert lambda function can reach into for a variety of additional data:

Metadata that is not included in the incoming telemetry
Additional configuration that controls behavior (example: Gating new features, controlling Publishers, etc)
Whitelists/Blacklists that are not hard-coded in Python Code
(Future) Threat Intel

How to use?
Step 1) Configure
There is a new file, conf/lookup_tables.json that houses the configuration of LookupTables. This configuration allows you to define LookupTables, their backends, and their behavior.
Step 2) Put Data
Throw data into AWS S3 or AWS DynamoDB!
Step 3) Use!
The syntax is super easy. Simply do:
from stream_alert.shared.lookup_tables.core import LookupTables

my_value = LookupTables.get('my-table', 'my-key')

Furthermore, you can use this code anywhere in the application, not just in the Rules Engine!
Changes
Redesigns LookupTables syntax
As above.
LookupTables now supports DynamoDB as a backing mechanism
Documentation coming, but you can take a look at (THIS FILE) for examples on how to configure it.
Breaking Changes!!!! ⚠
Beware that in the original implementation of LookupTables, because the tables were loaded in entirety and returned as a fully-populated dict, performing dictionary iterations was possible:
map = LookupTables.get('my-table')
for k, v in map.iteritems():
  # ...

When supporting other databases such as DynamoDb, this behavior is equivalent to performing table scans. This is an anti-pattern and thus this access pattern is no longer supported. The reasoning is because this necessarily causes large amount of network I/O PLUS memory utilization burden on AWS Lambda functions, which increases costs significantly.
Existing systems that perform "table-scan-like" behavior on LookupTable will have to redesign the access pattern. More coming later ... when I get around to writing the documentation...
Testing
As if 2019-08-15 12:30 PM PDT.... Testing on stage right now...

STICKY #1

STICKY COMMENT 2

Coverage decreased (-0.3%) to 96.322% when pulling 0a6714f0c401b86742996ffc38fa2fa37a7bd531 on dw--lookuptables into a078b4a5d5cc0d473bb1644eed91b4ab89f4d3b9 on release-3-0-0.


Dependencies error

Describe the bug
I got this error when I try to run my app
pub-cache/hosted/pub.dev/reown_appkit-1.0.1/lib/modal/appkit_modal_impl.dart:116:3: Error: Type 'Logger' not found.
To Reproduce
Steps to reproduce the behavior:
update Reown dependencies on pubspec.yaml
  reown_walletkit: 1.0.2
  reown_appkit: 1.0.1

the pubspec.lock contains
reown_appkit:
  dependency: "direct main"
  description:
    name: reown_appkit
    sha256: "16d5ce70bd254db2b0fd837272438f240212caf0fadff17902834cfb6644dc26"
    url: "https://pub.dev"
  source: hosted
  version: "1.0.1"
reown_core:
  dependency: transitive
  description:
    name: reown_core
    sha256: f0dc4829b674afb3ccf3f8d6b16e5e268a9bc919034c2594c68daeae7e0aec34
    url: "https://pub.dev"
  source: hosted
  version: "1.0.2"
reown_sign:
  dependency: transitive
  description:
    name: reown_sign
    sha256: "83b6e85d853e3b6371fc7e00f39914bee52fd0ed18636ea5e082184a23171f2d"
    url: "https://pub.dev"
  source: hosted
  version: "1.0.2"
reown_walletkit:
  dependency: "direct main"
  description:
    name: reown_walletkit
    sha256: "20f4490e801909e463e6a412bb64572bac091629e6641ec94213d4f83fb70186"
    url: "https://pub.dev"
  source: hosted
  version: "1.0.2"

if I rollback to reown_walletkit 1.0.1 everything works fine

Hello @plusema86! You can install reown_appkit: ^1.1.0-beta02 which is about to be promoted to stable soon.
Also, can I ask why do you need both walletkit and appkit?

Hey @quetool, thanks, that's true, I don't need it, mistake during migrating to Reown from WalletConnect

I can do it, thanks!


fatal error: Index out of range

@chanonly123 thx for you issue. This issue will be closed after merging PR https://github.com/wangjwchn/AImage/pull/33


Add total and unarmed victims to H1 tag

Description
Modified index.js, appRoutes.js, appModelCount.js, and appController.js to add values to H1 tag showing total victims and total unarmed victims in 2020.

Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[X] 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

Checklist:

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] Any dependent changes have been merged and published in downstream modules

This looks good. Love the MVP approach to this.


How to return errors?

Hi there!
Please provide code for returning errors with custom codes
For example -31319

How to return this like exceptions ?
  "error" : {         "code" : -31050,         "message" : {             "ru" : "Номер телефона не найден",             "uz" : "Raqam ro'yhatda yo'q",             "en" : "Phone number not found"         },         "data" : "phone"     },     "result" : null,     "id" : 2032

bump, stuck going through the code to try to solve my error.
InvalidJsonRpcFormatException in ResponseParser.php line 129: Invalid Request: Params must be an array
Impossible to debug with the library.

throw new \JsonRPC\Exception\ResponseException($message, $code, $previousException, $data);


Issues in shift operators

Description:
The results for the following are not inline with the spec.

Right shift for unsigned int types (where the most significant bit is 1)

import ballerina/io;
import ballerina/lang.'int as ints;

public function main() {
    ints:Unsigned8 i = 255;
    io:println(i >> 2);
}

The spec says ">> performs a signed right shift; the bits shifted in on the left are the same as the most significant bit".
So this should print 255, but prints 63. The right shift seems to be filling 0s at the beginning.

Unsigned right shift for signed int types other than int (where the most significant bit is 1 - i.e., negative values)

import ballerina/io;
import ballerina/lang.'int as ints;

public function main() {
    ints:Signed8 i = -128;
    io:println(i >>> 2);
}

The spec says ">>> performs a unsigned right shift, the bits shifted in on the left are zero".
So this should print 32 (10 0000), but prints<PHONE_NUMBER>427387872. This seems to be filling 1s at the beginning (62 - 57 1's 5 0's).
Affected Versions:
1.2.0-rc1
Issue 1 does not work for 1.1.4 too

Fixed with https://github.com/ballerina-platform/ballerina-lang/pull/21738.

Reopening this issue since we would have to do/undo some changes based on ballerina-platform/ballerina-spec#458.

Fixed with https://github.com/ballerina-platform/ballerina-lang/pull/21769.


Server crash with 1.1.0beta1

http://pastebin.com/UABEw98N

That is already fixed in the latest build


DPO3DPKRT-764/fix local proxy cors conflicts

Fixes DPO3DPKRT-764 allowing for easier running of Proxy mode locally through an Nginx change.

re-opening to change target and move forward


DNI: testing all metal compute tests on CI

This PR is intented to run all CI tests for Metal compute test. It is CI testing only and not for an actual merge.
Closes #4291
This change is generated by a following shell script
for f in $(find tests -name '*.slang')
do
    sed -i 's|DISABLE_TEST\(.*\)-mtl\(.*\)|TEST\1-mtl\2|' "$f"
done

Closing because the issue #4291 is closed


Calico for Windows Open Source availability

With 3.16 Calico for Windows is now available in open source in line with the Linux version. This is now available for all customers deploying on premises. AKS integration will come soon.

Calico for Windows is now available in open source at https://projectcalico.org


Move prisma in a Demo repository

Prisma will no longer be used, we will integrate a SQL-lite raw driver for simplicity.


Is the search code published?

Sorry, no. Since I am no longer at Google, I do not even have access to the search code anymore.

Thank you all the same. 发自我的华为手机-------- 原始邮件 --------发件人： Chenxi Liu<EMAIL_ADDRESS>2020年2月12日周三 中午11:58收件人： "chenxi116/PNASNet.pytorch"<EMAIL_ADDRESS>pingguokiller<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>  题： Re: [chenxi116/PNASNet.pytorch] Is the search code published? (#6)Sorry, no. Since I am no longer at Google, I do not even have access to the search code anymore.
—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or unsubscribe.


Is it possible to show top k validation accuracy during training?

How can I modify the layer symbols if I want to see top k validation accuracy during training

Accuracy is computed in python with mx.metric.Accuracy. You can add a TopkAcc class.

@piiswrong Thanks for your answer!


Memory leak when using concatMap with asynchronous function

RxJS version:
Version 5.1.0.
Code to reproduce:
const Rx = require('rxjs/Rx');

const buffer_size = 1024;
const dummy_string = Array(10001).join('x');

const generator = (function() {
    const subscribers = [];
    let counter = 0;

    const observable = Rx.Observable.create(observer => {
        subscribers.push(observer);
    });

    observable.request = count => {
        setTimeout(() => {
            for (let req=count; req > 0; req--) {
                counter++;
                subscribers.forEach(s => s.next(counter));
            }
        }, 0);
    };

    return observable;
})();

function mapper_async(recs) {
    return Rx.Observable.create(observer => {
        setTimeout(() => {
            observer.next(recs.length);
            observer.complete();
        }, 1);
    });
}

generator
    .map(v => dummy_string + v)
    .bufferCount(buffer_size)
    .concatMap(mapper_async)
    .subscribe(v => {
        console.log(v);
        generator.request(buffer_size);
    });

generator.request(buffer_size);

Expected behavior:
Memory usage stays bounded.
Actual behavior:
Out of memory (after some time).
Additional information:
If using
function mapper(recs) {
    return Rx.Observable.create(observer => {
        observer.next(recs.length);
        observer.complete();
    });
}

as the mapping function instead of mapper_async, the problem does not occur.
If running the equivalent code using https://github.com/Reactive-Extensions/RxJS (v4.1.0) the memory does not blow up.
Run on node.js version 6.9.4.

Honestly, I'm not entirely sure what the intended goal of the example code is, but concatMap has an unbounded buffer internally, and for every time you're calling that monkey-patched request method, you're nexting 1024 values, which in turn calls request 1024 times, nexting 1024 * 1014 values... recursively. So each pass P through your flooding concatMap's internal buffer with 1024 ^ P more items.
The equivalent code does not "blow up" in RxJS 4, because it defaults to Queue scheduling, which runs everything breadth first. (non-recursively). So the concatMap buffer gets a chance to flush at each pass. You can do the same thing with Rx 5 by using the QueueScheduler (Rx.Scheduler.queue), but we don't default to it for performance reasons. What you're doing here isn't a good pattern in terms of concatMap.

I'm going to close this as not an issue for now. But if you find it's still an issue using the queue scheduler, please reopen.
Thanks for reporting this and caring about the community and the library, @janmarthedal!!

Fixed with release 5.1.1 (probably due to the fix of #2360).


Add some tests for events

Coverage increased (+1.3%) to 86.41% when pulling 1ee74aa0a7b5d79db96334c92eceb95433c596fb on briansteffens:test-events into 71588147e78e917309b6102548f4c480d63f7d33 on linode:master.


Add preprovisioning images to must-gather scripts

Related Issue:  N/A
Description of Changes:
These are used for defining the image ironic should deploy to a host
What resource is being added: preprovisioningimages
Is this a Hub or Managed cluster change?:
Hub Cluster
Notes:

Also opened https://github.com/stolostron/backplane-must-gather/pull/34

/lgtm


Question about Reproducing Figure 4 - Inference Time vs Vocabulary Size

I am currently trying to reproduce the results shown in Figure 4 - Inference Time vs Vocabulary Size from your project. I have a couple of questions regarding the methodology used for this figure:

What inference framework was utilized to measure the inference time?

Was the embedding layer modified to special vocab size before testing the inference speed?

Thanks

in my experiments, the NSL * inference time curve is continuously decreasing and does not exhibit an inflection point as shown in your figure 4 - Time Optimal Vocabulary

Hi thanks for the questions!

Just used multiple runs on the same hardware and tracked time naively. As long as you pick a unit of time like iteration/ms or batch/ms and keep that constant for all experiments you should find - not unusurprisingly that time increases as vocab size increases. We used an internal Meta repo for the inference, but to avoid possible caching optimisations, make sure to use random tokens in each batch.

A few special tokens are negligible. What matters is you adjust the number of tokens the embedding layer to be to the size of the vocabulary.

To obtain the optimal trade-off, like in Figure 4, also make sure that you normalise both time usage and NSL first - at the same vocabulary size.

Thanks for your reply. Both time usage and NSL has been normalized In my test. However, when I multiply these two metrics together, the result remains monotonic within a vocabulary size of up to 290k.

"to avoid possible caching optimisations, make sure to use random tokens in each batch"
-- I should try this suggestion, thanks :)


429 Too Many Requests error from acme

What version of Traefik are you using (traefik version)?
1.2.2
What is your environment & configuration (arguments, toml...)?
OnHostRule = true
caServer = "https://acme-staging.api.letsencrypt.org/directory"
...
[frontends]
  [frontends.old_dev]
  backend = "old_dev"
    [frontends.old_dev.routes.route_1]
    rule = "Host:..."

The Host rule consists of 17 (sub)domains.
What did you do?
I commented caServer, to use letsencrypt prod server.
What did you expect to see?
A new production certificate created with 16 SANs.
What did you see instead?
Error "429 Too Many Requests"

Unfortunately LetsEncrypt has its own rate limits that are easy to exceed if you are not careful.
https://letsencrypt.org/docs/rate-limits/
Are you using a file for storage?

Yes we use file storage.
We know about the rate limiting and suspect it's the “new-authz” endpoint getting more than 20 requests per second, that is the problem.
Haven't had time to investigate further, though.

I have the same problem. One domain with ~15 subdomains. On the day the certs expired, I couldn't get them renewed. No obvious errors in the logs and updating to latest release didn't help.
Using Traefik version v1.2.3 built on 2017-04-13_07:32:59PM
Deleting the acme.json file caused traefik to recreate the certs. Those gave me 429 errors from acme:
legolog: 2017/04/24 16:41:10 [INFO][mydomain.com] AuthURL: https://acme-v01.api.letsencrypt.org/acme/authz/xxxx
legolog: 2017/04/24 16:41:10 [INFO][www.mydomain.com] AuthURL: https://acme-v01.api.letsencrypt.org/acme/authz/xxx
...
legolog: 2017/04/24 16:41:10 [INFO][s1.mydomain.com] AuthURL: https://acme-v01.api.letsencrypt.org/acme/authz/xxx
etc, for the first 12 of 15 subdomains.
The first 12 requests are going out simultaneously ... maybe they need to happen one at a time?
Here is a snippit of traefik.toml
[acme]
email =<EMAIL_ADDRESS>storage= "acme.json"
entryPoint = "https"
onDemand = false
OnHostRule = false
acmelogging = true
[[acme.domains]]
main = "mydomain.com"
sans = ["www.mydomain.com", "s1.mydomain.com", "s2.mydomain.com", "s3.mydomain.com", "s4.mydomain.com","s5.mydomain.com","s6.mydomain.com","s7.mydomain.com","s8.mydomain.com",...]
I've tried with OnHostRule true and false;
Waiting 10 minutes and trying again didn't help. I thought switching to onDemand=true would help but I still see 5 or so acme Error 429 on startup.
I get similar errors with testing and prod caServer .
caServer = "https://acme-staging.api.letsencrypt.org/directory"
caServer = "https://acme-v01.api.letsencrypt.org/directory"
I am available on your slack @bhlowe if someone wants more info. Seems reproducible. This is not using the docker traefik--but I do have a docker environment set up for building from source.

Regarding hitting the limit: is it expected that traefik will fall back to the self-signed certificate if no cert could be requested via acme? currently i experience only failed handshakes when hitting the limit and wouldn't mind the self-signed certificate for a week.

I'll close the issue due to inactivity but feel free to re-open if needed.


Add remaining community health file for funding

Description
This PR adds the remaining community health files as listed in #8
Closes #8

Let's hold this, for now, we might need to change the funding providers.
I will come back to this shortly


Upgrade the support library to 24.2.1

Google recommends the upgrade:

We do, however, recommend using 24.2.1 over 24.2.0 as it contains a number of bug fixes.
https://www.reddit.com/r/androiddev/comments/52m6ac/android_support_library_revision_2421_released/d7lptza

https://github.com/grandcentrix/ThirtyInch/issues/8

LGTM 👍


"Gornet" is listed in the drop-down menu

Description:
"Gornet" is listed in the drop-down menu alongside its corresponding county according to the law
Precondition:
The website is up an running.

Step 1
Type "Gornet" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Gornet, PRAHOVA".


Added feature to use this list as a To-do list

Added feature to use this list as a To-do list

Sorry about the delayed response here. I really appreciate you adding this, but I'm trying to keep the list as simple as possible. I only added location because I now that matters a lot to some people.


Please add dnsseed main.seed.ionomy.nl

https://github.com/ionomy/ion/blob/14c03a64a90ef7e8f8ae34b25c83ff6a353d5e36/src/chainparams.cpp#L82
Please add main.seed.ionomy.nl

utACK

@RdeWilde why don't you just add the changes to a pull request? I know you can do it ;)

Because I don't know if it'll end up in the sherlock's repository. I also tried directly editing in Github but it seems to be disabled.

Don't worry about Sherlock's directory. It's just a fork of this repo. This is still the main repo. Put the PR through

https://github.com/ionomy/ion/pull/102

Merged!


Removing support for github based registries

Signed-off-by: Mohammed Zeeshan Ahmed<EMAIL_ADDRESS>What type of PR is this:
/kind task
What does this PR do / why we need it:
Which issue(s) this PR fixes:

Fixes #5579
PR acceptance criteria:

[x] Unit test

[x] Integration test

[ ] Documentation

How to test changes / Special notes to the reviewer:
You should no longer be able to add, update, list GitHub based registries
odo preference registry add should not allow github registry urls
list should not show github registries
update should not allow github registries
./odo preference registry list
NAME                       URL                                           SECURE
CheRegistry                https://che-devfile-registry.openshift.io     No
DefaultDevfileRegistry     https://registry.devfile.io                   No
./odo preference registry add test https://github.com/elsony/devfile-registry
 ✗  github based registries are no longer supported, use https://github.com/devfile/registry-support instead
 ./odo preference registry update CheRegistry https://github.com/elsony/devfile-registry
 ✗  github based registries are no longer supported, use https://github.com/devfile/registry-support instead

/test unit

/test v4.10-integration-e2e
infra issue

/approve

/override Kubernetes Integration Tests/Kubernetes Integration Tests


When choosing a language for studying the tour, it gives an error

Context: https://tour.golang.org/welcome/2
When choosing a language for studying the tour, it gives an error. For example, when I select Russian it gives the following error: Go 1.9 is no longer available. Please refer to https://goo.gl/aESk5L for more information. When will this be fixed?

Thanks for the report; but we don't control the tour translations. We only link to them.

When will this be fixed?

When the owner of the russian translation re-deploys the tour. See #1039.
Closing here.


Add ChangeLog, ESLint, editorconf and others from boilerplate

Our pluin boilerplate contains many very useful thing for open source project. You should move them:

[x] ChangeLog
[x] ESLint
[x] .editorconfig
[x] LICENSE
[x] .npmignore

Done!


Add Apache 2.0 licence headers to all code files

This is needed to be compliant with the Apache 2.0 licence.

Added to script files, css, and LICENSE files in the repo.


helm latest gives back 2.6.13

If you get the latest helm from
https://api.github.com/repos/helm/helm/releases/latest
{
  "url": "https://api.github.com/repos/helm/helm/releases/23663865",
  "assets_url": "https://api.github.com/repos/helm/helm/releases/23663865/assets",
  "upload_url": "https://uploads.github.com/repos/helm/helm/releases/23663865/assets{?name,label}",
  "html_url": "https://github.com/helm/helm/releases/tag/v2.16.3",
  "id": 23663865,
  "node_id": "MDc6UmVsZWFzZTIzNjYzODY1",
  "tag_name": "v2.16.3",
  "target_commitish": "master",
  "name": "Helm v2.16.3",
  "draft": false,
  "author": {
    "login": "bacongobbler",
    "id": 1360539,
    "node_id": "MDQ6VXNlcjEzNjA1Mzk=",
    "avatar_url": "https://avatars2.githubusercontent.com/u/1360539?v=4",
    "gravatar_id": "",
    "url": "https://api.github.com/users/bacongobbler",
    "html_url": "https://github.com/bacongobbler",
    "followers_url": "https://api.github.com/users/bacongobbler/followers",
    "following_url": "https://api.github.com/users/bacongobbler/following{/other_user}",
    "gists_url": "https://api.github.com/users/bacongobbler/gists{/gist_id}",
    "starred_url": "https://api.github.com/users/bacongobbler/starred{/owner}{/repo}",
    "subscriptions_url": "https://api.github.com/users/bacongobbler/subscriptions",
    "organizations_url": "https://api.github.com/users/bacongobbler/orgs",
    "repos_url": "https://api.github.com/users/bacongobbler/repos",
    "events_url": "https://api.github.com/users/bacongobbler/events{/privacy}",
    "received_events_url": "https://api.github.com/users/bacongobbler/received_events",
    "type": "User",
    "site_admin": false
  },

you'll retrieve 2.6.13 instead of 3.0.3 or meanwhile 3.1.0
Is this by intension or just a bug?

Duplicate of #7586

Fixed in #7589

Re-opening as this has to do with GitHub assigning "latest release" to the last published release. In this case it is "2.16.3".

it seems the "latest release" is determined from the timestamps on the tags. This is why "2.16.3" is  the latest. Maybe someone know how to change this?

GitHub does not let a project control what is determined as “latest”. “latest” is equivalent to whichever release was most recently published.


[ALLUXIO-2689] Backup the current journal implementation to journalv0

https://alluxio.atlassian.net/browse/ALLUXIO-2689
Backup the current journal implementation so that we can write tool to convert journal between versions.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14130/
Test PASSed.

I think it is not up to date. I merged a PR updating UFSJournalWriter early this morning. Will it matter?

@apc999 PTAL. I've addressed your comments. thanks

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/14196/
Test PASSed.

LGTM


Update locking-conversations.md

Why:
The current documentation doesn't cover a functionality quite widely used in GitHub: reactions and votes.
Closes #21343
What's being changed (if available, include any code snippets, screenshots, or gifs):
Article body,
Check off the following:

[ ] I have reviewed my changes in staging (look for the "Automatically generated comment" and click the links in the "Preview" column to view your latest changes).
[x] For content changes, I have completed the self-review checklist.

@keunes Thank you for opening a PR and linking it to your issue! ✨
I'll get this triaged for review! ⚡

@keunes Thanks so much for taking the time to submit this PR. :sparkles: If you are participating in hacktoberfest and looking for more ways to contribute, check out our help wanted section to find open issues you can work on.
We have a few issues that could use some extra help. If you’re up for working on another, more involved help wanted issue, look for the pumpkin-spice label.
Thanks again!


nixos/filesystems: set system.fsPackages for sshfs

Description of changes
Also set  system.fsPackages when sshfs is used this makes it more consistent with other file systems also update the manual with this change. Also see #272619

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/3258


Add .vscode TypeScript settings json

Description
Adds .vscode/settings.json telling VSCode to use repo TypeScript compiler version

Found a local solution to this that does not require a .vscode/settings.json file... for anyone else who is encountering TypeScript version issues in VSCode:
Go to VSCode settings:

Search for tsdk:

Add node_modules/typescript/lib to the input field.

Closing this out in lieu of above


Feature request: Highlight code block surrounded by selected brackets with vertical line

Please implement block highlighting with vertical line like this:

You can find such functionality in Notepad++ for example. It makes it more easier to work with large code blocks. Feature should be available for all bracket types (also user-defined in language settings).
For misaligned brackets, highlight by left-most bracket, it will help to identify misaligned code (Notepad++ in this case highlights nothing, which is not useful in my opinion). Like this:

You can see the https://marketplace.visualstudio.com/items?itemName=byi8220.indented-block-highlighting extension which is similar.

That extension does different thing - it highlights whole block, which in my opinion is less useful, it does not provide info about indentation level.

We have this issue tracking it with another extension link: https://github.com/Microsoft/vscode/issues/14017

Thanks, installed Identicator, it precisely what I needed, it's a shame though that it's not core feature.

Found a lot better alternative - Bracket Pair Colorizer, it accurately finds current block (and highlights it even if indentation is incorrect), plus it colorizes brackets. Downside - it works for limited range of languages.


Add "always" tag to the flowdock role

http://docs.ansible.com/ansible/playbooks_tags.html

This would make it always run, even if one does a "-t slurm" for example.

Added.
Unfortunately the role is still not executed in case of task failures. Maybe a flowdock callback module would be the way to go.


Feature/actions

@0xLeif it is better if you do the delete action .. I merged immediately so I can implement result builders after your work.


Login failed for user ''. For no apparent reason

Azure Data Studio Version: 1.49.1 (User)
OS Version: Windows 11, 24H2

Steps to Reproduce:

I enter all config
I select Authentication type "Microsoft Entra ID" and add my account.
I log in in the opened browser
I get redirected back to datastudio
I click on Login
Error Message appears

NOTE: I have tried this with the exact same settings on a different device with the sam Azure Data Studio Version from the same Network and there it worked flawlessly

Does this issue occur when all extensions are disabled?: Yes

I have tried checking the logs and found the following traces - Not sure if it helps:

If there are more helpful logs somewhere - please let me know

After some additional troubleshooting we discovered that the Database search is broken - when i manually copy paste the Database name and overwrite the <Default> it starts to work!


Fix regular expressions Cross-Site Scripting (XSS) vulnerability

What kind of change does this PR introduce?
dependency update, this update includes a security fix for https://github.com/advisories/GHSA-h9rv-jmmf-4pgx

Did you add tests for your changes?
no

Does this PR introduce a breaking change?
no

What needs to be documented once your changes are merged?
nothing

looks like webpack.webpack (Windows node-8) tests did not work before for webpack-4 branch
example:
https://github.com/webpack/webpack/pull/10064
https://github.com/webpack/webpack/pull/10010

Thanks


Most optimal way: how to check if string is a number, if so: make it an integer

Hi guys,
How would you do this in the most Pythonic way?
e.g.
a_set = {45, "foo", "65", "bar"}
a_set_adj = set()
for v in a_set:
    try:
        v = int(v)
        a_set_adj.add(v)
    except ValueError:
        a_set_adj.add(v)

print (a_set_adj)

input:
a_set = {45, "foo", "65", "bar"}

output:
a_set_adj = {65, 'foo', 'bar', 45}

why not use the string isdigit() method?
print '5.1'.isdigit()
#-> False
print '51'.isdigit()
#-> True
print 'Foo'.isdigit()
#-> False

Exactly, if there's a mixed possibilities for inputs that's there it'll fail. But my solution seems verbose too...

Hm, well I guess my question is why aren't you in control of how the data is being passed? In my mind, the only time that should be a worry is if you're getting values from a user, at which point couldn't you validate that they're passing correct values that stage, rather than when you're trying to convert them?
But regardless, how about this solution:
my_valid_set = set([x for x in a_set if isinstace(x, (int, str))])
my_final_set = set([x for x in a_set if not x.isdigit()]).update(set([int(x) for x in my_valid_set if x.isdigit()]))

I'll modify @Djones4822 answer a bit and do:
def to_int(x):
    try:
        return int(x)
    except ValueError:
        return x

a_set_adj = {to_int(x) for x in a_set}

Please note the set comprehension (https://docs.python.org/2/tutorial/datastructures.html#sets).

+1 ^
Ooh that's nice, I like that a lot better

That is terrific, this helped a lot guys! This really helps out becoming better coders, I think.

Yes, exactly, collective knowledge. We can all learn from eachother.

@Djones4822 hey, this was about getting values from a user (exactly), when would you validate it then?
The thing is, I'm working in P3.4, so the result of input() is always a string. I wanted to do two things: check if it's a number, if so: make it a number, and check if it's within range of (..). To check if it's within a specific range of numbers, it needs to be an integer. But if it's (accidentally) not an integer, it gives back an error when you use int(x).
This is what I currently use:
def check_range(str_input):
    try:
        int_input = int(str_input)
        if int_input not in range(1,11):
            return False
        else:
            return int_input
    except (Exception, e):
        return False

As it stands right now, I would adjust your code a bit. You don't need all of that in the try clause since all you're trying to do is recasting the string as an int., and your except statement should probably reference the expected ValueError that converting a non-digit string to an int would call.
 def check_range(str_input):
    try:
        int_input = int(str_input)
    except ValueError:
        return False

    if int_input not in range(1,11):
        return False
    else:
         return int_input

It's typically more "pythonic" to avoid making your conditionals as small as possible. This can become particularly important if you find yourself repeating a lot of code for a bunch of different conditions - just pull them out at that point.
But what I was saying about validation at the start was really referencing your list of strings and integers. I couldn't imagine a situation where a user is asked to input a list on their own, so I was thinking that at each input you can check if it was entered correctly.
But maybe a simpler approach to what you're trying might be (it might not be suitable for what you're doing):
from string import digits as strdigits

while True:
    user_input = input('What number would you like to enter?\n\n-> ')
    if user_input in strdigits:
        break
    else:
        print('You suck at entering numbers')

The built in string module has constants like ascii_lowercase ascii_uppercase, digits, punctuation, etc, that can be useful. But I realize, maybe this wouldn't be perfect since you want 1-10 (digits gives you 0-9). So maybe you just want to make your own constant that is a list of valid integer strings to input. So maybe adjust to:
strdigits = [str(x) for x in range(1,11)]

def get_input():
    while True:
        user_input = input('What number would you like to enter?\n\n-> ')

        if user_input in strdigits:
            return user_input
        else:
            print('You suck at entering numbers')

I love using while loops when getting input from users. They can get cumbersome, and maybe they're not always the best, but it can be an easy way to repeat the input() process until they give you want you want. If you use a while loop like that outside of a function, instead of return user_input you would use a break statement to end the inner-most loop.

This is highly informative! I have mainly been using iterating over functions to check a user's input until it's "right", but looks like a smart alternative.
You would have strdigits as global variable? I'm thinking of even going as far as making the function more generally usable by including a list as an argument:
def get_check_input(message, not_within_limits_message, limits_list):
    while True:
        user_input = input(message)

        if user_input in limits_list:
            return user_input
        else:
            print (not_within_limits_message)

What do you think about that? Or is it usually better practice to have multiple (but more specific) functions?

Sure that might work, passing all of those arguments to the function every time seems cumbersome. Maybe use default arguments, but with messages I don't usually assign them to an object - but what do I know. From what I've seen, messages are usually only assigned to an object if they're too long to fit on a line (see PEP-8 80 char limit).
But also, python scope is nice in that you don't have to worry too much about global objects. Since you're only calling a reference against that object and not trying to alter it at all, you can just reference it easily enough.
def get_check_input(message, error_message = not_within_limits_message, check_list = limits_list):
    while True:
        user_input = input(message)

        if user_input in check_list:
            return user_input
        else:
            print (error_message)

(Pro tip: you can get git hub to format your code blocks by typing python on the line before your code and then ending it with  another 3 ticks ())

You're great. Thanks a bunch. I'm thinking of rather using the first one as you may have different input requests with each their own error message. Great!

Yea so the first way makes it more versatile. Good for use in a class or static method that would be used in multiple circumstances.

Here's my take on this:
a_list = ['65', '65.5', 'hello', 'apple', '65apple', 80]
expected = [65, '65.5', 'hello', 'apple', '65apple', 80]

def checker(sample):
    try:
        return [int(i) if str(i).isdigit() else i for i in sample]
    except Exception, e:
        return Exception('Shit happens', e)

print checker(a_list) == expected

I reeeaaaallly like that list comp you wrote. I didn't know you could write them like that - thanks!