Error: Input file is missing or of an unsupported image format on some requests

Hello,
This code for image upload and compression works in most cases but some requests fail ((Number of fails increase in load testing) with error: 'Error: Input file is missing or of an unsupported image format'
I checked the path whenever this error appears. The file really exists and is usable. Also this error does not appear for all requests. The exception occurs when I give the temp file path (saved to disk via streams to reduce memory footprint) to sharp.
'use strict';

const fs = require('fs');
const fsx = require('fs-extra');
const path = require('path');
const sharp = require('sharp');
const Config = require('../config/image');
const Rand = require('../utils/random');
const TAG = '\services\image';
function readImageStream(buffer, owner, tempFileName){
	const tempFilepath = Config.UPLOAD_PATH+'tmp\\'+owner+'\\'+tempFileName;
	return new Promise(function(resolve, reject){
		//const readStream = fs.createReadStream(buffer);
		const readStream = buffer;
		const writeStream = fs.createWriteStream(tempFilepath);
		let output;
		readStream.on('data', function(chunk){
			writeStream.write(chunk, {flags: 'a', encoding:null, mode:0o666});
		});
		readStream.on('end', function(){
			const path = writeStream.path;
			writeStream.end();
			resolve(path);
		});
		readStream.on('error', function(err){
			console.log("Error in reading image stream " + TAG, err);
			reject(err);
		});
	});
}

function checkImageDimensions(owner, filename){
	const path = Config.UPLOAD_PATH + "tmp\\"+ owner + "\\" + filename;
	return new Promise(function(resolve, reject){

		sharp(path)
			.metadata()
			.then(function(info){
				if(info.width < 100 || info.height <100 ||
			      (info.width / info.height < 0.5) ||
				  (info.height / info.width < 0.5)){
						reject('Bad image size');
		}
		else{
			resolve();
		}
		})
		.catch(function(err){
			console.log("Error in image size check " + TAG, err);
			reject(err);
		})
	});
}

exports.Save = async function(owner, buffer){
	const tempDir = Config.UPLOAD_PATH+"tmp\\"+owner;
	const tempFileName1 = Rand.UUID() + Config.EXTENSION; //Creates UUID for filename
	const tempFileName2 = Rand.UUID() + Config.EXTENSION;
	const tempPath = tempDir + "\\"+tempFileName1;
	const ensureDirExists = await fsx.ensureDir(Config.UPLOAD_PATH+ "tmp\\"+owner);
	//await checkMaxTempSize(owner);
	await readImageStream(buffer, owner, tempFileName1);
	await checkImageDimensions(owner, tempFileName1);
	try{
		await sharp(tempPath)
					.resize(800, 494, {
										kernel: sharp.kernel.nearest,
										fit: 'contain',
										background: { r: 255, g: 255, b: 255}
									}).toFile(Config.UPLOAD_PATH+ "tmp\\"+owner+ "\\" + tempFileName2, {quality:80});
				return {
						filename: tempFileName2
				};
	}
	catch(err){
		console.log("Error in image compression " + TAG, err);
		return err;
	}
}

Hi, there's a race condition in the following section of your code, unrelated to sharp:
readStream.on('end', function(){
	const path = writeStream.path;
	writeStream.end();
	resolve(path);
});

The call to writeStream.end() is not synchronous so resolve could occur before writeStream has finished writing to and closing the underlying file descriptor.
Instead listen for the close event on writeStream before resolving.

Hi, I've experienced the same issue. In some cases it works properly, but it often happens to get the error mentioned above. I've tried to check the details of a "working" image and a "non-working" image. The only difference is the file-type. For not working images we have:
File Type: JPEG image data, JFIF standard 1.02
My current settings to resize is:
sharp(filePath) 			.withMetadata() 			.resize(200, 200, { 				withoutEnlargement: true 			}) 			.png() 			.toFile(newFilePath);
I hope we can figure this out, and thanks in advance.

@gergo0720 Please open a new issue with a standalone code sample and example input image that consistently fails.

Thanks a million @lovell I spent almost a month on solving the issue and tried many possibilities, finally I suspected about internal streams that sharp use to open files. It now works like a charm.


Feature/glide

Godeps is not the sharpest tool in the shed #125  https://github.com/ajvb/kala/pull/130
Golang 1.7.3 is out, the GO15VENDOREXPERIMENT is not a experiment anymore.

Coverage decreased (-0.2%) to 81.083% when pulling 68cb0a7bae332d96cb5ca78fcd156bf05f480cbc on cescoferraro:feature/glide into 3b3bfec18e6d322e50fb5f8bf978aff9e0d2abce on ajvb:master.

Godeps seems to work pretty well with the latest go 1.8. I converted to vendor as part of #157. Ultimately, we will likely end up using https://github.com/golang/dep


Add support for Azure MySQL

This pull requests adds support for using the MySQL PaaS provided by
Azure [1] as the backend for SQLAlchemy. Lighting up this feature
required two changes:

Enabling SSL support in Flask-SQLAlchemy since Azure MySQL requires
SSL by default.

Enabling configuration of the MySQL row format since Azure MySQL does
not support compressed rows.

The Azure MySQL backend can be enabled by setting the following
environment variables:
DATABASE_URL=mysql://{UserName}@{ServerName}:{Password}@{ServerName}.mysql.database.azure.com:3306/{DatabaseName}?
DB_ROW_FORMAT=DEFAULT

[1] https://azure.microsoft.com/en-us/services/mysql/

Rebased the settings refactor from master in 85ffddf.

Other than that LGTM. Tested and this is working properly. 👍

The builds keep failing due to intermittent issues so I added a retry in 21d0b74.


Challenge http://www.freecodecamp.com/challenges/waypoint-line-up-form-elements-responsively-with-bootstrap has an issue

For the first task the exercise says to: "Wrap your all of your checkboxes inside one div with the class "row"." However the checkboxes had already been put into a div with the class row in the previous exercise. To get the exercise to recognize as complete I had to wrap the form text box and submit button in a div with the class "row".

Thanks for the report on this. We have a fix in staging along with a bunch of other ones and will push it up soon.


Structure the code into classes

The code will quickly become a mess if we don't structure it, therefore create classes to certain objects.

Player class has been created.

Lots of structuring done... and i have added javadocs for newly created methods.
look at this commit if in doubt.. https://github.com/CandyFace/OOSE2015/commit/b30bbab655b15e47e6ea2cb402ba1b6f68bee0e9


Cannot retreive login token from logs in HASS web UI.

Log for this addon instantly get polluted with C# exception trace which pushes away any info regarding login data out of scope.

at Azure.Core.Pipeline.RetryPolicy.ProcessAsync(HttpMessage message, ReadOnlyMemory1 pipeline, Boolean async)    --- End of inner exception stack trace ---    at Azure.Core.Pipeline.RetryPolicy.ProcessAsync(HttpMessage message, ReadOnlyMemory1 pipeline, Boolean async)
at Azure.Core.Pipeline.HttpPipeline.SendRequestAsync(Request request, CancellationToken cancellationToken)
at Azure.Core.HttpPipelineMessageHandler.SendAsync(HttpRequestMessage request, CancellationToken cancellationToken)
at System.Net.Http.HttpClient.g__Core|83_0(HttpRequestMessage request, HttpCompletionOption completionOption, CancellationTokenSource cts, Boolean disposeCts, CancellationTokenSource pendingRequestsCts, CancellationToken originalCancellationToken)
at Microsoft.Identity.Client.Http.HttpManager.ExecuteAsync(Uri endpoint, IDictionary2 headers, HttpContent body, HttpMethod method, ILoggerAdapter logger, CancellationToken cancellationToken)    at Microsoft.Identity.Client.Http.HttpManager.ExecuteWithRetryAsync(Uri endpoint, IDictionary2 headers, HttpContent body, HttpMethod method, ILoggerAdapter logger, Boolean doNotThrow, Boolean retry, CancellationToken cancellationToken)
at Microsoft.Identity.Client.Http.HttpManager.SendGetAsync(Uri endpoint, IDictionary2 headers, ILoggerAdapter logger, Boolean retry, CancellationToken cancellationToken)    at Microsoft.Identity.Client.OAuth2.OAuth2Client.ExecuteRequestAsync[T](Uri endPoint, HttpMethod method, RequestContext requestContext, Boolean expectErrorsOn200OK, Boolean addCommonHeaders, Func2 onBeforePostRequestData)
at Microsoft.Identity.Client.OAuth2.OAuth2Client.DiscoverAadInstanceAsync(Uri endPoint, RequestContext requestContext)
at Microsoft.Identity.Client.Instance.Discovery.NetworkMetadataProvider.SendInstanceDiscoveryRequestAsync(Uri authority, RequestContext requestContext)
at Microsoft.Identity.Client.Instance.Discovery.NetworkMetadataProvider.FetchAllDiscoveryMetadataAsync(Uri authority, RequestContext requestContext)
at Microsoft.Identity.Client.Instance.Discovery.NetworkMetadataProvider.GetMetadataAsync(Uri authority, RequestContext requestContext)
at Microsoft.Identity.Client.Instance.Discovery.InstanceDiscoveryManager.FetchNetworkMetadataOrFallbackAsync(RequestContext requestContext, Uri authorityUri)
at Microsoft.Identity.Client.Instance.Discovery.InstanceDiscoveryManager.GetMetadataEntryAsync(AuthorityInfo authorityInfo, RequestContext requestContext, Boolean forceValidation)
at Microsoft.Identity.Client.Instance.AuthorityManager.RunInstanceDiscoveryAndValidationAsync()
at Microsoft.Identity.Client.Internal.Requests.DeviceCodeRequest.ExecuteAsync(CancellationToken cancellationToken)
at Microsoft.Identity.Client.Internal.Requests.RequestBase.RunAsync(CancellationToken cancellationToken)
at Microsoft.Identity.Client.ApiConfig.Executors.PublicClientExecutor.ExecuteAsync(AcquireTokenCommonParameters commonParameters, AcquireTokenWithDeviceCodeParameters deviceCodeParameters, CancellationToken cancellationToken)
at Azure.Identity.AbstractAcquireTokenParameterBuilderExtensions.ExecuteAsync[T](AbstractAcquireTokenParameterBuilder1 builder, Boolean async, CancellationToken cancellationToken)    at Azure.Identity.MsalPublicClient.AcquireTokenWithDeviceCodeCoreAsync(String[] scopes, String claims, Func2 deviceCodeCallback, Boolean async, CancellationToken cancellationToken)
at Azure.Identity.MsalPublicClient.AcquireTokenWithDeviceCodeAsync(String[] scopes, String claims, Func2 deviceCodeCallback, Boolean async, CancellationToken cancellationToken)    at Azure.Identity.DeviceCodeCredential.GetTokenViaDeviceCodeAsync(TokenRequestContext context, Boolean async, CancellationToken cancellationToken)    at Azure.Identity.DeviceCodeCredential.GetTokenImplAsync(Boolean async, TokenRequestContext requestContext, CancellationToken cancellationToken)  ---> (Inner Exception #1) Azure.RequestFailedException: Permission denied (login.microsoftonline.com:443)  ---> System.Net.Http.HttpRequestException: Permission denied (login.microsoftonline.com:443)  ---> System.Net.Sockets.SocketException (13): Permission denied    at System.Net.Sockets.Socket..ctor(AddressFamily addressFamily, SocketType socketType, ProtocolType protocolType)    at System.Net.Sockets.Socket..ctor(SocketType socketType, ProtocolType protocolType)    at System.Net.Http.HttpConnectionPool.ConnectToTcpHostAsync(String host, Int32 port, HttpRequestMessage initialRequest, Boolean async, CancellationToken cancellationToken)    --- End of inner exception stack trace ---    at System.Net.Http.HttpConnectionPool.ConnectToTcpHostAsync(String host, Int32 port, HttpRequestMessage initialRequest, Boolean async, CancellationToken cancellationToken)    at System.Net.Http.HttpConnectionPool.ConnectAsync(HttpRequestMessage request, Boolean async, CancellationToken cancellationToken)    at System.Net.Http.HttpConnectionPool.CreateHttp11ConnectionAsync(HttpRequestMessage request, Boolean async, CancellationToken cancellationToken)    at System.Net.Http.HttpConnectionPool.AddHttp11ConnectionAsync(HttpRequestMessage request)    at System.Threading.Tasks.TaskCompletionSourceWithCancellation1.WaitWithCancellationAsync(CancellationToken cancellationToken)
at System.Net.Http.HttpConnectionPool.GetHttp11ConnectionAsync(HttpRequestMessage request, Boolean async, CancellationToken cancellationToken)
at System.Net.Http.HttpConnectionPool.SendWithVersionDetectionAndRetryAsync(HttpRequestMessage request, Boolean async, Boolean doRequestAuth, CancellationToken cancellationToken)
at System.Net.Http.HttpClient.g__Core|83_0(HttpRequestMessage request, HttpCompletionOption completionOption, CancellationTokenSource cts, Boolean disposeCts, CancellationTokenSource pendingRequestsCts, CancellationToken originalCancellationToken)
at Azure.Core.Pipeline.HttpClientTransport.ProcessAsync(HttpMessage message, Boolean async)
--- End of inner exception stack trace ---
at Azure.Core.Pipeline.HttpClientTransport.ProcessAsync(HttpMessage message, Boolean async)
at Azure.Core.Pipeline.HttpPipelineTransportPolicy.ProcessAsync(HttpMessage message, ReadOnlyMemory1 pipeline)    at Azure.Core.Pipeline.ResponseBodyPolicy.ProcessAsync(HttpMessage message, ReadOnlyMemory1 pipeline, Boolean async)
at Azure.Core.Pipeline.RedirectPolicy.ProcessAsync(HttpMessage message, ReadOnlyMemory1 pipeline, Boolean async)    at Azure.Core.Pipeline.RetryPolicy.ProcessAsync(HttpMessage message, ReadOnlyMemory1 pipeline, Boolean async)<---
---> (Inner Exception #2) Azure.RequestFailedException: Permission denied (login.microsoftonline.com:443)
---> System.Net.Http.HttpRequestException: Permission denied (login.microsoftonline.com:443)
---> System.Net.Sockets.SocketException (13): Permission denied
at System.Net.Sockets.Socket..ctor(AddressFamily addressFamily, SocketType socketType, ProtocolType protocolType)
at System.Net.Sockets.Socket..ctor(SocketType socketType, ProtocolType protocolType)
at System.Net.Http.HttpConnectionPool.ConnectToTcpHostAsync(String host, Int32 port, HttpRequestMessage initialRequest, Boolean async, CancellationToken cancellationToken)
--- End of inner exception stack trace ---
at System.Net.Http.HttpConnectionPool.ConnectToTcpHostAsync(String host, Int32 port, HttpRequestMessage initialRequest, Boolean async, CancellationToken cancellationToken)
at System.Net.Http.HttpConnectionPool.ConnectAsync(HttpRequestMessage request, Boolean async, CancellationToken cancellationToken)
at System.Net.Http.HttpConnectionPool.CreateHttp11ConnectionAsync(HttpRequestMessage request, Boolean async, CancellationToken cancellationToken)
at System.Net.Http.HttpConnectionPool.AddHttp11ConnectionAsync(HttpRequestMessage request)
at System.Threading.Tasks.TaskCompletionSourceWithCancellation1.WaitWithCancellationAsync(CancellationToken cancellationToken)    at System.Net.Http.HttpConnectionPool.GetHttp11ConnectionAsync(HttpRequestMessage request, Boolean async, CancellationToken cancellationToken)    at System.Net.Http.HttpConnectionPool.SendWithVersionDetectionAndRetryAsync(HttpRequestMessage request, Boolean async, Boolean doRequestAuth, CancellationToken cancellationToken)    at System.Net.Http.HttpClient.<SendAsync>g__Core|83_0(HttpRequestMessage request, HttpCompletionOption completionOption, CancellationTokenSource cts, Boolean disposeCts, CancellationTokenSource pendingRequestsCts, CancellationToken originalCancellationToken)    at Azure.Core.Pipeline.HttpClientTransport.ProcessAsync(HttpMessage message, Boolean async)    --- End of inner exception stack trace ---    at Azure.Core.Pipeline.HttpClientTransport.ProcessAsync(HttpMessage message, Boolean async)    at Azure.Core.Pipeline.HttpPipelineTransportPolicy.ProcessAsync(HttpMessage message, ReadOnlyMemory1 pipeline)
at Azure.Core.Pipeline.ResponseBodyPolicy.ProcessAsync(HttpMessage message, ReadOnlyMemory1 pipeline, Boolean async)    at Azure.Core.Pipeline.RedirectPolicy.ProcessAsync(HttpMessage message, ReadOnlyMemory1 pipeline, Boolean async)
at Azure.Core.Pipeline.RetryPolicy.ProcessAsync(HttpMessage message, ReadOnlyMemory1 pipeline, Boolean async)<---  ---> (Inner Exception #3) Azure.RequestFailedException: Permission denied (login.microsoftonline.com:443)  ---> System.Net.Http.HttpRequestException: Permission denied (login.microsoftonline.com:443)  ---> System.Net.Sockets.SocketException (13): Permission denied    at System.Net.Sockets.Socket..ctor(AddressFamily addressFamily, SocketType socketType, ProtocolType protocolType)    at System.Net.Sockets.Socket..ctor(SocketType socketType, ProtocolType protocolType)    at System.Net.Http.HttpConnectionPool.ConnectToTcpHostAsync(String host, Int32 port, HttpRequestMessage initialRequest, Boolean async, CancellationToken cancellationToken)    --- End of inner exception stack trace ---    at System.Net.Http.HttpConnectionPool.ConnectToTcpHostAsync(String host, Int32 port, HttpRequestMessage initialRequest, Boolean async, CancellationToken cancellationToken)    at System.Net.Http.HttpConnectionPool.ConnectAsync(HttpRequestMessage request, Boolean async, CancellationToken cancellationToken)    at System.Net.Http.HttpConnectionPool.CreateHttp11ConnectionAsync(HttpRequestMessage request, Boolean async, CancellationToken cancellationToken)    at System.Net.Http.HttpConnectionPool.AddHttp11ConnectionAsync(HttpRequestMessage request)    at System.Threading.Tasks.TaskCompletionSourceWithCancellation1.WaitWithCancellationAsync(CancellationToken cancellationToken)
at System.Net.Http.HttpConnectionPool.GetHttp11ConnectionAsync(HttpRequestMessage request, Boolean async, CancellationToken cancellationToken)
at System.Net.Http.HttpConnectionPool.SendWithVersionDetectionAndRetryAsync(HttpRequestMessage request, Boolean async, Boolean doRequestAuth, CancellationToken cancellationToken)
at System.Net.Http.HttpClient.g__Core|83_0(HttpRequestMessage request, HttpCompletionOption completionOption, CancellationTokenSource cts, Boolean disposeCts, CancellationTokenSource pendingRequestsCts, CancellationToken originalCancellationToken)
at Azure.Core.Pipeline.HttpClientTransport.ProcessAsync(HttpMessage message, Boolean async)
--- End of inner exception stack trace ---
at Azure.Core.Pipeline.HttpClientTransport.ProcessAsync(HttpMessage message, Boolean async)
at Azure.Core.Pipeline.HttpPipelineTransportPolicy.ProcessAsync(HttpMessage message, ReadOnlyMemory1 pipeline)    at Azure.Core.Pipeline.ResponseBodyPolicy.ProcessAsync(HttpMessage message, ReadOnlyMemory1 pipeline, Boolean async)
at Azure.Core.Pipeline.RedirectPolicy.ProcessAsync(HttpMessage message, ReadOnlyMemory1 pipeline, Boolean async)    at Azure.Core.Pipeline.RetryPolicy.ProcessAsync(HttpMessage message, ReadOnlyMemory1 pipeline, Boolean async)<---
--- End of inner exception stack trace ---
at Azure.Identity.CredentialDiagnosticScope.FailWrapAndThrow(Exception ex, String additionalMessage)
at Azure.Identity.DeviceCodeCredential.GetTokenImplAsync(Boolean async, TokenRequestContext requestContext, CancellationToken cancellationToken)
at Azure.Identity.DeviceCodeCredential.GetTokenAsync(TokenRequestContext requestContext, CancellationToken cancellationToken)
at hassio_onedrive_backup.Graph.GraphHelper.GetAndCacheUserTokenAsync() in /src/Graph/GraphHelper.cs:line 50
at hassio_onedrive_backup.Program.Main(String[] args) in /src/Program.cs:line 38

As a future improvement a simple web server that opens up the Microsoft login page could be nice!

Wow , that is a long trace...
The suggestion for an internal web page would be nice I agree. I plan to have some sort of UI added at some point so it will likely fit there.
Regardless, the exception above is related to the login flow. And I can make out "Permission Denied" errors.
Do you not see the initial logs from the Addon Startup ? Did this happen after you tried to authenticate? (And after granting permissions?)

Closing this issue (same as #2 ) so we can focus relevant discussion in one place

Wow , that is a long trace... The suggestion for an internal web page would be nice I agree. I plan to have some sort of UI added at some point so it will likely fit there.
Regardless, the exception above is related to the login flow. And I can make out "Permission Denied" errors. Do you not see the initial logs from the Addon Startup ? Did this happen after you tried to authenticate? (And after granting permissions?)
Also, are you running HassOS ?

Could not see the initial log trace. There is a character limit on how much far back the log can be displayed in the Web UI.
Running supervised install on Ubuntu Server


firewall service: respect marks in rpfilter

Motivation for this change
This allows one to add rules which change a packet's routing table:
iptables -t raw -I PREROUTING 1 -m set --match-set myset src -j MARK --set-mark 2
ip rule add fwmark 2 table 1 priority 1000
ip route add default dev iface0 table 1

to the beginning of raw table PREROUTING chain, and still have rpfilter. One use case is routing packets to specific IP addresses through other network interface (I use this to route packets to hosts blocked in Russia through VPN).
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Any news on this? If this is still relevant it would be nice to get it into 18.09.


How can I access PDF "path" objects using pdfjs

How an i access pdf path objects with PDF js

I was able to get it done in following way. You can get the SVG as a string with following code
  page.getOperatorList().then(function (opList) {
            var svgGfx = new PDFJS.SVGGraphics(page.commonObjs, page.objs);
            svgGfx.embedFonts = true;
            return svgGfx.getSVG(opList, viewport).then(function (svg) {
                var svgString = svg.outerHTML;
            });
        });


dynamic_reconfigure: 1.5.47-0 in 'kinetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository dynamic_reconfigure to 1.5.47-0:

upstream repository: https://github.com/ros/dynamic_reconfigure.git
release repository: https://github.com/ros-gbp/dynamic_reconfigure-release.git
distro file: kinetic/distribution.yaml
bloom version: 0.5.25
previous version for package: 1.5.46-0

dynamic_reconfigure
* reset received_configuration_ for every request sent (#82 <https://github.com/ros/dynamic_reconfigure/issues/82>)
* Rename arguments (with a_ prefix) to avoid Wshadow warnings. (#80 <https://github.com/ros/dynamic_reconfigure/issues/80>)
  handle infinity in python generation, fixes (#77 <https://github.com/ros/dynamic_reconfigure/issues/77>)
* Add a c++ Dynamic Reconfigure Client (#78 <https://github.com/ros/dynamic_reconfigure/issues/78>)
* Enforce valid descriptions in cfg files (#74 <https://github.com/ros/dynamic_reconfigure/issues/74>)
* Fix callback returned by get_description_callback (#73 <https://github.com/ros/dynamic_reconfigure/issues/73>) from ros/description_cb
* Contributors: Jeff Eberl, Mikael Arguedas

waiting for other packages...

Good to go, feel free to merge if no sync planned


Docs: Fixed Getting Started, Missing step.

Fixed Walkthrough for setting up the first project.

Fixed Missing command react-native start to start the packager in the Linux-Android section.

The page at Getting Started for Linux Android has a step missing. i.e react-native start without which the bundler won't start.
This step is mentioned in Windows sections but not in Linux Section.

react-native run-android usually also starts the server, so it shouldn't need to be done separately. If that isn't happening I think it would be a separate issue


Create buf-push-action go binary

As I added features the bash was getting complex, so I moved it into a go binary.
User facing enhancements are:

Add optional "track" input to specify what track to push to.
Add optional "github_token" input.
Add "commit" and "commit_url" outputs for the bsr commit and a url linking to the commit. Workflows can use this to add a status.

The action will now:

Restore a cache of its bin directory.
Runs downloadbinary.bash which:

Exits if buf-push-action already exists in bin.
Downloads buf-push-action from the github release with the same version as is being run and exits if successful.
Builds buf-push-action from source.

Runs buf-push-action.

This allows workflow runs that use released versions to run quickly without having to build from source while also allowing us to easily test unreleased changes.
The release process will be similar to bufbuild/buf. You will create a tag with the new version, run make release, create a GitHub release, then upload assets to the GitHub release.

It does seem like we pull in the kitchen sink to be able to write this in go, whereas buf-{setup,lint,breaking}-action are all typescript/js. Having probably missed the reason, why do we need go here (and do the whole release boilerplate requiring setup?)

I chose to write this in go simply because I don't know enough typescript to write anything complex.
I wish this didn't require pulling in the kitchen sink. If I weren't trying to make this consistent with our other go projects then this could be as simple as as simple as main.go, main_test.go and a couple of shell scripts. Even though it's kind of painful to set up, it's probably more valuable to be consistent with other projects than it is to be concise.

I wish this didn't require pulling in the kitchen sink. If I weren't trying to make this consistent with our other go projects then this could be as simple as as simple as main.go, main_test.go and a couple of shell scripts. Even though it's kind of painful to set up, it's probably more valuable to be consistent with other projects than it is to be concise.

The current state seems somewhat like the weirdest of all worlds. We import some code from github.com/bufbuild/buf/private, for other functionality from that repo we expect users to provide a buf binary that we then os.Exec, for other parts we re-implement (reading the buf.yaml structure). All in all the ratio boiler plate to code here seems > 10:1, with the extra make and bash scripts to maintain to do releases probably being more than the implementation of the action.
The above aside, the public interface seems reasonable to me so if this has all been discussed don't let me be the source of significant re-hashing :)

Add optional "track" input to specify what track to push to.

Shouldn't this action push to a track named after the github branch name? I believe we only wanted an option to map i.e. master in git to main in buf, this option implies that the track we push to is static.

Runs downloadbinary.bash which:

Exits if buf-push-action already exists in bin.
Downloads buf-push-action from the github release with the same version as is being run and exits if successful.
Builds buf-push-action from source.

Us having what amount to fallbacks and a stateful setup seems error-prone.

Add optional "track" input to specify what track to push to.

Shouldn't this action push to a track named after the github branch name? I believe we only wanted an option to map i.e. master in git to main in buf, this option implies that the track we push to is static.

Runs downloadbinary.bash which:

Exits if buf-push-action already exists in bin.
Downloads buf-push-action from the github release with the same version as is being run and exits if successful.
Builds buf-push-action from source.

Us having what amount to fallbacks and a stateful setup seems error-prone.

This is the only way I could think of to avoid a breaking change. If we make the default behavior push to a track named after the GitHub branch then existing workflows will begin behaving differently.

Pushing to a track named after the GitHub branch is the main product we are trying to ship here

Oh I see what's going on - let's catch up in an hour, I put a meeting on your cal

After discussion we are going to make ${{ github.ref_name }} the default value for track. That will be a breaking change, so we will bump the version to v2.


[EventRequest] Codeunit 5752 Get Source Doc. Outbound - OnCreateFromSalesOrderHideDialogOnAfterFindWhseRequest

Requesting a new event publisher on Codeunit 5752 Get Source Doc. Outbound
Event
OnCreateFromSalesOrderHideDialogOnAfterFindWhseRequest(SalesHeader : Record "Sales Header";WhseRqst : Record "Warehouse Request";VAR SkipWhseShipmentHeaderCreation : Boolean)
Code
[External] CreateFromSalesOrderHideDialog(SalesHeader : Record "Sales Header") : Boolean
IF NOT SalesHeader.IsApprovedForPosting THEN
  EXIT(FALSE);

FindWarehouseRequestForSalesOrder(WhseRqst,SalesHeader);

IF WhseRqst.ISEMPTY THEN
  EXIT(FALSE);

>>> New Code
OnCreateFromSalesOrderHideDialogOnAfterFindWhseRequest(SalesHeader, WhseRqst, SkipWhseShipmentHeaderCreation)
IF SkipWhseShipmentHeaderCreation THEN
  EXIT(FALSE);

<<< New Code End
CreateWhseShipmentHeaderFromWhseRequest(WhseRqst);
EXIT(TRUE);

In Our Solution we create Warehouse Shipment Header per Ship-to Code, so that means we bundle Orders to open Warehouse Shipment Headers for processing. Therefore we need to have the possibility to determine when to create a new Warehouse Shipment Header and when not to. Please add the requested publisher.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Revert "[Enhancement] Change Sort Key Order for AirByte tables"

Description
https://github.com/StarRocks/starrocks/pull/25026 introduced a bug which I mentioned here - https://github.com/StarRocks/starrocks/pull/25026#issuecomment-1595699817
For now, I'm reverting the change.
Basically during data load to StarRocks, we end up inserting _airbyte_emitted_at value into _airbyte_ab_id and vice-versa.
What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

I'm unable to find time to work on it.
@dirtysalt @liuzhongjun89 If someone else can fix the insert logic, that would be great. Otherwise, I'll try to find some time this weekend.
In the meanwhile, we should revert this change from the main branch.

I'll open a pr and fix the bug later.

Thanks @liuzhongjun89 !


Add n4 correction script

Checklist
GitHub

[x] I've given this PR a concise, self-descriptive, and meaningful title
[x] I've linked relevant issues in the PR body
[ ] I've applied the relevant labels to this PR
[x] I've assigned a reviewer

PR contents

[ ] I've consulted ivadomed's internal developer documentation to ensure my contribution is in line with any relevant design decisions
[ ] I've added relevant tests for my contribution
[ ] I've updated the relevant documentation for my changes, including argparse descriptions, docstrings, and ReadTheDocs tutorial pages

Description

A seperate n4 correction script, since it is part of preprocessing and preprocessing is manual we put n4 correction in a seperate script
Linked issues

resolve https://github.com/ivadomed/ivadomed/issues/3

note the piece of code uses the library older than mentioned in https://github.com/ivadomed/ivadomed/issues/3, so the syntax is a little bit different

@cakester could you add relevant unit test call (i.e. low level non cli call to the basic Python function) using data from data_testing, specifically these ones, but for all subjects:

Some changes to be discussed within the wider IvadoMed dev group before proceeding.
@cakester could you add relevant unit test call (i.e. low level non cli call to the basic Python function) using data from data_testing, specifically these ones, but for all subjects:

If the group okays the CLI approach, we will then add the functional script_caller test as well.

what else do I have to check in the unit test call except to check if the output file is there?

what else do I have to check in the unit test call except to check if the output file is there?

Somehow to check whether the intensity profile has less variance along a given axis than prior to N4?

what else do I have to check in the unit test call except to check if the output file is there?

Somehow to check whether the intensity profile has less variance along a given axis than prior to N4?

yes, that or comparing the output with a "ground truth" image, which would go through N4 and be visually checked, and be located in the testing data. Just a suggestion

I brought up the possibility to maybe MD5 hash check against a properly corrected N4 output during our last meeting. That can be a viable approach to unit test as well.
N4 process should be deterministic and barring any underlying changes in the dependency/calculation level, the bit output should be identical (to be 100% validated using methods below).
Might best to run a local quick check (maybe after locking down SimpleITK version), i.e.
Platform/Environment/OS Combo 1:

Data A -> N4 Call -> N4 Corrected Data A -> MD5 Hash

Platform/NEW Reinstalled VENV/CONDA Environment/OS Combo 2:

Data A -> N4 Call -> N4 Corrected Data A -> MD5 Hash

NEW VM/Docker/Platform/VENV/CONDA Environment/OS Combo 3:

Data A -> N4 Call -> N4 Corrected Data A -> MD5 Hash

With Data A being one of the test imaging data shown from the picture (those nii.gz files, can start with just one before doing more).
In theory, MD5 should be the same... Please let us know if you are able to experimentally confirm that, ideally across more OS. Try to keep ITK version, python etc, as close as you can.

im trying to generate md5 for the n4 corrected files but the test fails and the md5 hash is different from ci environment to my own environment and I downloaded linux (and made sure I get the same simpleitk version) and got another set of hashes......md5 should be the same for all platforms in theory, which means simpleitk generates different files across different platforms? or maybe there is multiple release of the library for the same version? redirecting to @dyt811


Updating TLS secrets

Upgrading the secret to a tls one instead of generic. This does require the pem to renamed crt.
Also adding helm values to update the manager and controller.

Support for chained/combo certs was added a while ago, so need to maintain it. If there aren't any suggestions for modifying this PR to accommodate it I will close.

Support for chained/combo certs was added a while ago, so need to maintain it. If there aren't any suggestions for modifying this PR to accommodate it I will close.

Then where is the documentation for adding chains?  What order does Neuvector expect for the chain?

A note has been added to the docs. Thanks for your input. https://open-docs.neuvector.com/configuration/console/replacecert#support-chained-certificates


changed 'sha hash' to 'shouldhash' in line 233

Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[x] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[x] Ensure the tests and linter pass
[x] Code coverage does not decrease (if any source code was changed)
[x] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

Thanks for the PR!  I think this was fine as is.


Chat bar dissapears

I joined a server, and my chat bar was just missing. I could still talk though.
This has happened to me twice, both in a different game.
The picture provided is of the error from the developer console.

Are you, or is the game maker, using a custom chat / any additional chat functionality?

Nope.


[Search] Update readme and samples to use the same format of search resource data as official docs

For "Hotel" example, Readme and Sample docs use camel case for fields but official "Hotel" example in the portal uses pascal case. In this PR, fixing the fields format of "Hotel" example in the samples and readme and removing the snippet hacks.
Fixes: https://github.com/Azure/azure-sdk-for-net/issues/13502

API change check
API changes are not detected in this pull request.


crypto: add threshold module

this PR adds a new module threshold to the existing crypto crate, which implements the encryption and threshold decryption described in the flow encryption spec.
a few properties are tested here: basic encryption/decryption, the additive homomorphism, as well as full threshold decryption (using FROST's DKG) using a randomly selected (t/n) decryption shares.

For the public API (and probably the private API too), I think it makes sense to use triple-slash /// doc-comments so that rustdoc will document those functions, and to use the Rust documentation standard of capitalizing these doc comments and terminating them with a period. Also note that you can do intra-doc links with markdown links, like:
/// Verify the `EncryptionProof` for this EncryptedValue.
///
/// See the [spec](https://...) for more details.

thanks!


[REF-1489] Hash portion of URL (after the #) is not accessible in router_data

Describe the bug
Unable to access fragments of the url for e.g. https://<IP_ADDRESS>:3000/#foo=bar
Currently only pathname is being passed to the router_data. basePath should also be passed to get the fragment
https://nextjs.org/docs/pages/api-reference/functions/use-router
To Reproduce
N/A
Expected behavior
foo=bar should be accessible
Screenshots
N/A
** Specifics (please complete the following information):**

Python Version: 3.11.4
Reflex Version: 0.1.32
OS: MacOS
Browser (Optional):

Additional context
N/A
REF-1489

I just ran into this yesterday with Supabase and Google OAuth2.
Example of the callback URL:
http://localhost:3000/#access_token=[snip]&expires_in=3600&refresh_token=[snip]&token_type=bearer```

@ZaneH yes, same case as me. I ran into this issue while trying to integrate supabase auth.

Thanks for reporting, we will look into this and fix in an upcoming release.

@ZaneH @ritwickdsouza Does router.page.raw_path (see eg below) not work?

import reflex as rx

class State(rx.State):

    def display(self):
        print(self.router.page.raw_path)

@ZaneH @ritwickdsouza Does router.page.raw_path (see eg below) not work?
import reflex as rx

class State(rx.State):

    def display(self):
        print(self.router.page.raw_path)

Console output:
event_actions={} handler=EventHandler(event_actions={}, fn=<function LoginState.display at 0x10c23b560>) client_handler_name='' args=()

Code:
# index.py
...
Returns:
    The UI for the landing page.
"""
print(LoginState.display())
return rx.flex(
    rx.box(
...

# login.py
# LoginState is defined here
...
is_loading: bool = False

def display(self):
    return self.router.page.raw_path

def on_submit(self, form_data) -> rx.event.EventSpec:
...

I'm not seeing any output I'd recognize using .raw_path

@ZaneH @ritwickdsouza Does router.page.raw_path (see eg below) not work?
import reflex as rx

class State(rx.State):

    def display(self):
        print(self.router.page.raw_path)

Console output:
event_actions={} handler=EventHandler(event_actions={}, fn=<function LoginState.display at 0x10c23b560>) client_handler_name='' args=()

Code:
# index.py
...
Returns:
    The UI for the landing page.
"""
print(LoginState.display())
return rx.flex(
    rx.box(
...

# login.py
# LoginState is defined here
...
is_loading: bool = False

def display(self):
    return self.router.page.raw_path

def on_submit(self, form_data) -> rx.event.EventSpec:
...

I'm not seeing any output I'd recognize using .raw_path

No you cant call the eventhandler (.display()) out side of the state.
import reflex as rx

class State(rx.State):

    def display(self):
       print(self.router.page.raw_path)

    @rx.var
    def raw_path(self) -> str:
        return self.router.page.raw_path

def index():
     return rx.vstack(
         rx.text(State.raw_path),
     )

 app = rx.App()
 app.add_page(index, on_load=State.display)
 app.compile()

The example above will print the path in your console when the page loads as well display the path as a text on the page using a computed var

@ZaneH @ritwickdsouza Does router.page.raw_path (see eg below) not work?
import reflex as rx

class State(rx.State):

    def display(self):
        print(self.router.page.raw_path)

Console output:
event_actions={} handler=EventHandler(event_actions={}, fn=<function LoginState.display at 0x10c23b560>) client_handler_name='' args=()

Code:
# index.py
...
Returns:
    The UI for the landing page.
"""
print(LoginState.display())
return rx.flex(
    rx.box(
...

# login.py
# LoginState is defined here
...
is_loading: bool = False

def display(self):
    return self.router.page.raw_path

def on_submit(self, form_data) -> rx.event.EventSpec:
...

I'm not seeing any output I'd recognize using .raw_path

No you cant call the eventhandler (.display()) out side of the state.
import reflex as rx

class State(rx.State):

    def display(self):
       print(self.router.page.raw_path)

    @rx.var
    def raw_path(self) -> str:
        return self.router.page.raw_path

def index():
     return rx.vstack(
         rx.text(State.raw_path),
     )

 app = rx.App()
 app.add_page(index, on_load=State.display)
 app.compile()

The example above will print the path in your console when the page loads as well display the path as a text on the page using a computed var

I see, my bad. That worked for me. Unfortunately it looks like I can only use it in certain places. I needed to use it outside of the component "render"


VM deployment not working, please help

Description
VM deployment is not working on 3bot web interface or jsng.

Error message : message="no logs available: failed to spawn vm machine process '101922-1'"

I also tried deploying a k3s masternode with jsng on my node, with approximately the same error message :

message="failed to install k3s: could not run vm: no logs available: failed to spawn vm machine process '101921-1'",

Version information

OS: e.g. Manjaro
Project commit/version: JS-NG: 11.0b13 / JS-SDK: 11.5b6
Browser: Brave Version 1.30.87 Chromium: 94.0.4606.71 (Official Build)

Installation method
rm -rf ~/.cache/pypoetry/virtualenvs/js-sdk*
curl -sSL https://raw.githubusercontent.com/python-poetry/poetry/master/install-poetry.py | python -
cd ~/git
git clone https://github.com/threefoldtech/js-sdk
cd js-sdk
poetry install
poetry shell
threebot start --local
Steps to reproduce
Follow installation method and try to deploy an Ubuntu VM
Traceback/Logs/Alerts

S-NG> failed = zos.workloads.get(101922)
2021-11-03 11:31:55.590 | DEBUG    | jumpscale.clients.explorer.explorer:log_request:26 - Request https://explorer.grid.tf/api/v1/reservations/workloads/101922 method: GET body: None headers: {'user-agent': 'python-requests/2.25.1', 'accept-encoding': 'gzip, deflate', 'accept': '/', 'connection': 'keep-alive', 'threebot-id': '1675', 'date': 'Wed, 03 Nov 2021 10:31:55 GMT', 'authorization': 'Signature keyId="1675",algorithm="ed25519",signature="Godr6DvgU5S7lrooRh9Hphj7sAoP06Eh44bEURM0SndNXrNKcA1JGNhN69CcHZe77N2vMg+V6PWDX7ofmExpBQ==",created="1635935515",expires="1635935575",headers="(created) date threebot-id"'}
JS-NG> print(failed.info.result)
ReservationResult(
category=<Category.Zdb: 0>,
workload_id='101922-1',
data_json='null',
signature=b'18baa784043918b767b1fccc86b98102a4ef6d8f73b83959b643ad84185eb16e175dfdf0c57077cf5885115f192d30420723cd7cce85a92a430d3716e4391b04',
state=<State.Deleted: 2>,
message="no logs available: failed to spawn vm machine process '101922-1'",
epoch=datetime.datetime(2021, 11, 3, 10, 31, 22, tzinfo=tzutc())

I checked this node BwFs1ybT7PtmBB8kvi5KPqxBKWzoPZn25TV5MLjsiz6r it seems to be offline in explorer. Aside from that, that failure to spawn a VM issue has been fixed with the latest zos release, but not sure if the node you are testing against is updated for that release or not, it's available on all of our freefarm nodes on testnet.
I suggest you do the following

switch to testnet
create a pool on freefarm
deploy on that farm


Added a callback prop for snapToItem

Added a function prop onSnapToItem for getting callbacks for snapToItem events. This can be used as follows:
<Carousel
    ref={'carousel'}
    onSnapToItem={(item) => {
        console.log(item);
    }}
    items={this.state.entries}
    renderItem={this._renderItem}
    sliderWidth={this.state.width}
    itemWidth={pageWidth}
    firstItem={1}
    showsHorizontalScrollIndicator={false}
/>

Hi,
could you remove the anonymous function you set as the default prop ?
And replace your call on line 334 by something like this.props.onSnapToItem && this.props.onSnapToItem(index);.
And update the README accordingly :)

Done. It will be useful if there is a prop called snapToAlignment like this.

Hi @me-abhinav! Could you be a bit more specific about the behavior you'd expect from such a prop? Because it is by design that the active slide is always centered; it seems more "natural", or at least that how we wanted it to be.
But I would gladly take a look at any picture or gif of the layout you'd like to achieve.

Here you go:
https://vid.me/aoby

@me-abhinav Thank you for the screencast. I'll look into it as soon as possible :) Follow-up discussion about this will take place in the dedicated issue's thread: #3

Any views on this pull request?

@me-abhinav I will merge it later today.

@me-abhinav I went a little bit further and made sure the callback wasn't fired when the carousel is initially mounted and when you're trying to scroll to an index < 0 or > items.length, because a snap occurs then.

Thanks. I just observed that my edit in readme was not correct. I thought the last column is an example usage and not default value.


Add a logrotate config for Suricata

Suricata logs and json files kills the logger after some time as it grows so fast in a short period of time, the applied patch should fix the issue by rotating the files hourly when the size of the file reaches 500M.

This looks helpful, thank you for making the PR @lnxg33k !


UX improvement in parent user invite flow

Describe the issue:
With this improvement https://github.com/wso2/product-is/issues/16719, now can invite multiple users to the sub organisation from parent organisation at once with the set of roles. Hence the below UI wizard needs to be updated to invite multiple users with the same list of roles.

This issue is fixed from following PR

https://github.com/wso2/identity-apps/pull/5119


Update to Cadence v1.0.0-preview.39

Description
Automatically update to:

onflow/cadence v1.0.0-preview.39
onflow/flow-go-sdk v1.0.0-preview.42
onflow/flow-go 14f9fddeda2b
onflow/flow-emulator v1.0.0-preview.36

Summary by CodeRabbit

New Features

Upgraded multiple dependencies to their latest versions, potentially introducing new functionalities and performance enhancements.

Bug Fixes

Updated libraries may include critical bug fixes improving stability and compatibility.

Chores

Routine maintenance of dependency versions to ensure security and optimal performance.

Merged into #367


Docs: add sanitize.css as alternative in css reset docs

Thanks for your PR but I don't think we want to start listing all the different CSS reset options that are available.


mod_websocket chooses the first client subprotocol by default

If a plugin doesn't manually set a subprotocol in its on_connect callback, mod_websocket will just choose the first one provided by the client. This means that unsuspecting plugins may be "agreeing" to speak a subprotocol that they know nothing about. This behavior should be removed.

Fixed in 4f876ee4.


update offline database to queue requests / dodge thread error.

[ ] mobile database queues queries/requests

@hpertaia pick it up from here & take the example i developed here & apply to mobile db:  https://github.com/micheal-w-wells/db_queue_context/tree/master/src


Can't find model 'en_core_web_sm'

python scripts/main.py  --data=image_20230408/ --out_dir output --world_size=1  --save_img
Traceback (most recent call last):
File "scripts/main.py", line 4, in
from pipeline import semantic_annotation_pipeline
File "/data2/queenie_2023/Semantic-Segment-Anything/scripts/pipeline.py", line 13, in
from blip import open_vocabulary_classification_blip
File "/data2/queenie_2023/Semantic-Segment-Anything/scripts/blip.py", line 3, in
from utils import get_noun_phrases
File "/data2/queenie_2023/Semantic-Segment-Anything/scripts/utils.py", line 2, in
nlp = spacy.load('en_core_web_sm')
File "/data2/queenie/anaconda3/envs/mmTrans/lib/python3.7/site-packages/spacy/init.py", line 60, in load
config=config,
File "/data2/queenie/anaconda3/envs/mmTrans/lib/python3.7/site-packages/spacy/util.py", line 449, in load_model
raise IOError(Errors.E050.format(name=name))
OSError: [E050] Can't find model 'en_core_web_sm'. It doesn't seem to be a Python package or a valid path to a data directory.

You can try the command python -m spacy download en_core_web_sm

Please refer to the Installation section and run python -m spacy download en_core_web_sm.

en_core_web_sm的哪个版本

en_core_web_sm的哪个版本


Browserify 5+ dependency breaks JavaScriptCore starting in 2.5.6

I use JavaScriptCore (the low-level C library that powers JavaScript in WebKit) to run JSHint. I have used this for 2+ years without issue, until I tried to update to JSHint 2.5.10. After 8+ hours of debugging and testing, here is what I've found:
The 2.5.6 pre-built version works
If I download the zipped release for 2.5.6, the jshint.js file that was built on September 21st (and is part of the zipped download) WORKS.
Rebuilding 2.5.6 fails
If, however, I re-build 2.5.6 by running the following commands, the newly-built jshint.js file FAILS to run correctly in JavaScriptCore:
cd [path to unzipped 2.5.6 folder]
npm cache clean
npm install
./bin/build

I clear my npm cache just to be sure I'm pulling down the latest packages specified in package.json The newly-built jshint.js file will fail. Specifically, JSC throws an exception when it tries to evaluate the JSHINT function. It returns the stunningly-useless message: "TypeError: undefined is not an object on line (null) of (null)" This sort of exception usually occurs when JavaScript is malformed.
Since the pre-built version of JSHint (from 21 September) works just fine, I believe the problem lies with one of the dependencies specified in package.json. Many of those dependencies have released new versions since those that were available on 21 September and one of those newer versions is breaking the JSHint build.
Note: Browserify has updated their API since the version that was available on 21 September, so I had to edit bin/build line 19 in order to run the build script. The original line was:
bundle.bundle({}, function (err, src) {

And the modification was:
bundle.bundle(function (err, src) {

2.5.5 Works In All Cases
If I download the zipped 2.5.5 release and follow the same procedure to build that version, the built jshint.js file works flawlessly.
Changing 2.5.6's Dependencies Works
To test my theory, I downloaded a fresh copy of the 2.5.6 release, opened package.json and changed the browserify devDependency from 5.x to 4.x. The latter value is what was in the package.json file for JSHint 2.5.5, which built correctly.
After doing this, I cleared my npm cache, ran npm install and then the build script. The resulting jshint.js file worked flawlessly.
This fix works for 2.5.10 too
I downloaded the 2.5.10 zipped release. The pre-built file in dist failed. However, I edited package.json to change browserify back to 4.x and then edited line 19 of bin/build to use the old Browserify API:
bundle.bundle({}, function (err, src) {

When I built this, the resulting jshint.js file worked flawlessly again.
Bottom Line: Browserify 5+ is broken
After all of the above, I tried changing the browserify devDependency in package.json to 5.10.1 and then ran npm install after cleaning my cache and deleting the existing node_modules. This had no effect: the built versions of jshint.js still failed.
There is something wrong with Browserify 5 (or one of its MANY depdendencies) that WENT wrong after 21 September and is now breaking JSHint's build in a very, very subtle way.
I also put the jshint.js file built with the latest browserify and the one built with browserify 4.x into a file-diff tool and there are MASSIVE structural differences. Huge sections are in different spots, etc.
I'm not sure what's broken, but something is.
Proposal: Shrinkwrap
I propose that from now on, for each release, npm's shrinkwrap command should be used to generate a shrinkwrap json file that specifies the EXACT versions of the dependencies that were used to build each release.
This way, it's possible to always grab an older version of JSHint and build it exactly as it was built when the release took place. Right now, that is NOT possible because the package.json file will pull down the latest version of each dependency that meets compatibility specifications. There is no way to go back and build the release exactly as it was built on the date it came out.
Note: I do not propose that the shrinkwrap file replace the standard package.json file; simply that it be created at the time of each release so that it's available if needed. That way, if a dependency breaks the build, it's possible to go back and use the shrinkwrapped json file to install the exact dependencies that were used for the release and then update them one at a time to track down the one that's breaking.
I've opened a separate issue for this request.

Update: Just grabbed JSHint 2.6.0 and built it without issue. Whatever the problem was that caused this, it seems to have been fixed in Browserify 6. Closing. Self-fixing bugs are the best kind.

For the record, though, I still think adding shrinkwrap to the build process would be a good idea!

@bdkjones Thanks for the follow up, very much appreciated :)


ImportError: cannot import name 'settings' from 'azure.core.settings'

Package Name:  azure-core
Package Version:  1.32.0
Operating System: Windows 11
Python Version:  3.12.7

Describe the bug
Cannot import anything from azure-core
To Reproduce
Steps to reproduce the behavior:

create virtual environment with python 3.12.7
pip install azure-core>=1.32.0
attempt import (from azure.core.settings import settings)

Expected behavior
import succeeds or import not attempted
Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
environment created with UV. uv lock provided below:

Thank you for the feedback @vbrandon .  I was not able to repro this issue on my side, please check if your virtual environment has not been corrupted.


Get reviewers from ConanCenter

resolves #62

Codecov Report

Merging #78 (fa086ef) into main (b345d95) will increase coverage by 0.23%.
The diff coverage is 45.45%.

@@            Coverage Diff             @@
##             main      #78      +/-   ##
==========================================
+ Coverage   39.54%   39.78%   +0.23%
==========================================
  Files          14       15       +1
  Lines         531      553      +22
==========================================
+ Hits          210      220      +10
- Misses        311      319       +8
- Partials       10       14       +4

Impacted Files
Coverage Δ

pending_review/known_reviewers.go
45.45% <45.45%> (ø)

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Add missing -ManagementPointNetworkType parameter to New-Cluster documentation

PR Summary
Add missing -ManagementPointNetworkType parameter to the New-Cluster cmdlet documentation, which is available since Windows Server 2019.
PR Checklist

[x] Descriptive Title: This PR's title is a synopsis of the changes it proposes.
[x] Summary: This PR's summary describes the scope and intent of the change.
[x] Contributor's Guide: I have read the contributors guide.
[x] Style: This PR adheres to the style guide.

@microsoft-github-policy-service agree

Hi @JohanFreelancer9 - just wondering if there was an update to this PR?

Hi @JohanFreelancer9 - just wondering if there was an update to this PR?

@scanum, can you please help to get this PR reviewed and merged? Thanks.

Hi @get-itips - in fact these additions are already in the 2019 docs, but were omitted from the 2022, which this PR aims to correct.

@tseward hope all is well. Could you please help us review? Thanks!

Any update on this one?

@tiburd
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

Any update?

@tiburd
PR has been copyedited and is ready for final review, could you please check and merge? Thanks!

close > reopen


JBPM-5253 - NPE when container dispose fails

Moved FailureOnContainerDisposeIntegrationTest kie-server-tests-all to simulate the issue.
Provided fix suggestion.

+1 to merge, thanks @sutaakar
do you mind creating same for 6.5.x (if it is affected)?

I love working with you, guys! The bug fixed the same day when it was found :heart_eyes:


Update index.html

@faisalkholid  try this year for Hacktoberfest and have a look at the updated repo and contribute.
HAPPY HACKING


Add support for matching displays using regex.

Hi @alex-courtis !
Thanks so much for making way-displays. I love how (for the most part) it makes ordering displays so seamless as I heave my laptop around. One gripe I have is similar to that discussed in #60, whereby ordering gets confused by similarly named displays. By adding support for regex matching of the descriptions, I was able to get my setup working really nicely.
Is this something you'd consider upstreaming?

The regex order entries are not shown when printing config e.g.
ORDER:
  - Sharp Corporation 0x148D
  - !blehh
  - MAG274QRF-QD CA8A271A00352

way-displays -g
Active configuration:
  Arrange in a ROW aligned at the BOTTOM
  Order:
    Sharp Corporation 0x148D

    MAG274QRF-QD CA8A271A00352
  Auto scale: ON
  Mode:
    MAG274QRF-QD CA8A271A00352: 2560x1440

User receives a "too short" warning for regex order:
touch .../cfg.yaml
 [11:40:05] Reloading configuration file: /home/acourtis/.dotfiles/config/way-displays/cfg.yaml
I [11:40:05]
I [11:40:05] New configuration:
I [11:40:05]   Arrange in a ROW aligned at the BOTTOM
I [11:40:05]   Order:
I [11:40:05]     Sharp Corporation 0x148D
I [11:40:05]
I [11:40:05]     MAG274QRF-QD CA8A271A00352
I [11:40:05]   Auto scale: ON
I [11:40:05]   Mode:
I [11:40:05]     MAG274QRF-QD CA8A271A00352: 2560x1440
W [11:40:05]
W [11:40:05] WARNING: ORDER '' is less than 4 characters, which may result in some unwanted matches.

Testing with:
Microstep MAG274QRF-QD CA8A271A00352 (DP-6)
Sharp Corporation 0x148D (eDP-1)

Microstep first
ORDER:
# - '!Sharp'
# - MAG274QRF-QD CA8A271A00352
# - Sharp Corporation 0x148D

Sharp first
ORDER:
- '!Sharp'
# - MAG274QRF-QD CA8A271A00352
# - Sharp Corporation 0x148D

ORDER:
- '!Sharp'
- MAG274QRF-QD CA8A271A00352
# - Sharp Corporation 0x148D

ORDER:
- '!Sharp'
- MAG274QRF-QD CA8A271A00352
- Sharp Corporation 0x148D

Sharp last
ORDER:
  # - '!Sharp'
  - MAG274QRF-QD CA8A271A00352
  - Sharp Corporation 0x148D

FAIL - Sharp is first
ORDER:
- MAG274QRF-QD CA8A271A00352
- '!Sharp'
# - Sharp Corporation 0x148D

It looks like we have a (pre-existing) bug in order_heads. I think we will need to do one pass testing regex, exact, partial in order.

@alex-courtis Your test above should pass now :). I didn't even think to check the existing logic, so good catch! Let me know if there's anything else that needs attention.

Many thanks for the updates; I'll get to this one tomorrow.

Apologies, I did not get to this. I should get time on the weekend.
I think its time to add cmocka... I'll put something together on master.

Asking to learn: the bitwise OR to logical OR was only a matter of convention, right? Given the boolean type I intentionally used the bitwise operator.

Asking to learn: the bitwise OR to logical OR was only a matter of convention, right? Given the boolean type I intentionally used the bitwise operator.

It works as it will result in a 0 or 1. However, the bitwise operator is usually reserved only for bitwise operations, rather than tests.
gcc is happy, however clang will get upset:
src/layout.c:112:5: error: use of bitwise '|' with boolean operands [-Werror,-Wbitwise-instead-of-logical]
                                head_matches_name_desc_regex(i->val, head)

You can use ccls to look for such errors when editing.
I use it via neovim LSP: https://github.com/neovim/nvim-lspconfig/blob/master/doc/server_configurations.md#ccls


Update ODSAccordionView.m

Fix for scroll content size, when frame.top isn't 0.

Why do you think this fix is necessary? The way I see it self.frame.origin.y is the position of the accordion view relative to the parent view. How does the position affect the size of the accordion content in any way?

When you embed accordion in other view so that it's top coordinate isn't 0,
current size doesn't take it into account. Effectively you see accordion in
some frame and it's impossible to scroll to very bottom.
2015-02-16 23:06 GMT+01:00 Johannes Seitz<EMAIL_ADDRESS>
Why do you think this fix is necessary? The way I see it
self.frame.origin.y is the position of the accordion view relative to
the parent view. How does the position affect the size of the
accordion content in any way?
—
Reply to this email directly or view it on GitHub
https://github.com/Ookami86/ODSAccordionView/pull/5#issuecomment-74579697
.

--
Pozdrawiam,
Artur Rybak


Add Xbox Series X/S and PS5

Description
Would be cool if we could add Xbox Series X/S and PS5 to this project so that it retains it's usefulness as more people get the RTX 3080.
Possible solution
Just add the URLs, etc.

This is way outside of the goal of this project, but that doesn't mean we can't do it. I plan to start implementing other products when we get a few fundamental features merged in.

I was toying with this already. Either just doing it myself one by one or adding another config file with comma-separated ASINs (Amazon) and a for loop. Now, if this was in powershell I'd actually be able to accomplish the latter myself ;o)

Linking PR for PS5 support: https://github.com/jef/streetmerchant/pull/709

I was really hoping someone would be able to add the xbox models as well before they go live on the 10th.  All of the stores in my area are doing online sales only.   I'm definitely concerned about stock.  Thanks in advance for all the amazing work

There were some comments made on Discord that we can probably get this through before tomorrow!

sad that this didnt happen :(

yep
On Tue, Nov 10, 2020 at 1:17 AM Serena<EMAIL_ADDRESS>wrote:

sad that this didnt happen :(
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jef/streetmerchant/issues/506#issuecomment-724483048,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD7FNQUP4WXLFIBLFHSJQODSPDLJJANCNFSM4SMZIEOA
.


Allow extending HonorableTheme

I think it would be useful to be able to extend the HonorableTheme so you can access additional information in projects.

Will be merged!

Actually this introduces too many errors. Can you work out something like HonorableTheme & Foo?


Use PR number for bundle destination directory

'PULL_NUMBER' is part of prow job env variables and can be used for
step registry in openshift CI. It will help us to have only latest bundle
for each PR.

https://github.com/kubernetes/test-infra/blob/master/prow/jobs.md#job-environment-variables

fixes: #383

/retest

/cherry-pick 4.7


Updating for 1.1.0 release candidate

This PR does the following:

Minor updates for the 1.1.0-rc npm package release.
The package itself has already been publish to npm.
Once the RC is QA-approved, we should update the package version and remove -rc from it.

How has this been tested?
Storybook. The npm release will be tested in Turbine.
Accessibility concerns or updates

N/A

Checklist:

[ ] I have updated the Storybook documentation accordingly.
[ ] I have added relevant accessibility documentation for this pull request.
[ ] All new and existing tests passed.

Front End Review:

[ ] Review the Vercel preview deployment once it is ready.

More significant updates were made, such as adding aria-labels to icons in the header and footer. Another look would be good.


Import Next's CJS AsyncLocalStorage modules

The node evaluation always renders with type: "commonjs" and require() calls, so we'll always import the CJS files. But here we're importing the ESM files. That means we have 2 distinct instances of requestAsyncStorage in our node instance, and they cannot properly communicate with the other.
Fixes WEB-543

Added test case


feat(reviews): Add config for min number of characters in review

@wski please have a look.

:+1:

@hoganmaps please, a quick look.

:+1:


Docs: add missing bar 'top' position option

Bar has also a valid Y position option 'top', which is also offered in the playground: https://playground.cookieconsent.orestbida.com/

But this is inconsistent with the docs, where this option is missing.

Thank you for the contribution!


Add support to configure HTTPS TLS certs

BMC
Generates CSR
Supports crt/key pair upload

M1000e
:heavy_check_mark:
:heavy_check_mark:

iDRAC8
:heavy_check_mark:
:heavy_check_mark:

iDRAC9
:heavy_check_mark:
:heavy_check_mark:

ILO4
:heavy_check_mark:
:x:

ILO5
:heavy_check_mark:
:x:

C7000
:heavy_check_mark:
:x:

Supermicro X10
:x:
:heavy_check_mark:

To support these various cases, we add bmclib methods to,

List current certificate
Generate a CSR
Upload a signed certificate
Upload a signed certificate + key file

Done:

[x] iDRAC8
[x] iDRAC9
[x] ILO4
[x] ILO5
[ ] Supermicro X10

https://github.com/bmc-toolbox/bmclib/pull/71
Support for ILOs, IDracs added - the method we use is to generate a CSR, sign and upload.
Note: IDRAC9 has no option to set the Subject Alternative Name.
Supermicros cannot generate CSRs, an option here would be to add a resources name https_cert_nocsr
in this case bmcbutler would generate a CSR, get it signed and upload the crt, key pair.

For the supermicro X10s bmcbutler generates the CSR, gets it signed and uploads the cert, key pair,


Why is the return_qastle exposed? The coffea stuff should take care of that!

See the following in our notebooks:
ds.return_qastle = True

That should not be exposed to the user!

This will be fixed https://github.com/CoffeaTeam/coffea/pull/567


Mafft doesn't work with amino acid U

Mafft yields an error when encountering amino acid "U" (selenocysteine). For now clusters with such sequences will be removed

*when using mafft

Mafft --anysymbol option


Icon and OnTap is working only in costructor

Hi,
I found this issue. When I click to button which calls function (loadPoints), marker is shown but with default icon and impossible to make it clickable.
Adding markers in constructor is working fine without problem.
https://github.com/b4rtt/geoapp/blob/master/app/pages/login/login.page.ts
Is any way how to solve it?
Thank you

@EddyVerbruggen Could you help me please, or do you know anyone who can?

@bnussey Hi, i think that you are NS+Angular expert, could you help me please? I am kinda desperated :(

@EddyVerbruggen pls, I am in really problems with this :(

@b4rtt I just cloned that repo but it is very unresponsive since you're creating a location watcher every second. But no worries about that, I'll take a look.

I've found something. I need to change the plugin a bit and you'll need to change your code as well. I can fork your repo and send a PR if you like?

Yes, thank you very much! :)

Anyone encountering issues with incorrect icons or tap issues on iOS, please use 3.0.1.

@b4rtt Just pushed an update to your repo. You may have to restore some watch logic, but the update shows how to properly wire the map in an Angular app.
If there's any further issue please create an issue in your repo and ping me.


Microsoft_DefaultRuleSet-2.1-SQLI-942340 - False Positive

Hello,
I am getting a block for rule Microsoft_DefaultRuleSet-2.1-SQLI-942340 and it exceeds the anomaly score. This is flagged on a file that does not exist and returns a 403.
TenantID: 2787d8b8-1b24-4af2-b6a4-ea129d61742e
Rule Name: Microsoft_DefaultRuleSet-2.1-SQLI-942340
details_msg_s: Detects basic SQL authentication bypass attempts 3/3
PS C:\Users\cmorin> curl -vL https://webdev.24hourfitness.com:443/favicon.ico
*   Trying <IP_ADDRESS>:443...
* Connected to webdev.24hourfitness.com (<IP_ADDRESS>) port 443 (#0)
* schannel: disabled automatic use of client certificate
* ALPN: offers http/1.1
* ALPN: server accepted http/1.1
* using HTTP/1.1
> GET /favicon.ico HTTP/1.1
> Host: webdev.24hourfitness.com
> User-Agent: curl/8.0.1
> Accept: */*
>
< HTTP/1.1 200 OK
< Keep-Alive: timeout=5, max=100
< Content-Length: 1406
< Content-Type: image/vnd.microsoft.icon
< Last-Modified: Tue, 14 Aug 2018 17:50:27 GMT
< Accept-Ranges: bytes
< ETag: "57e-57368d866009f"
< X-Cache: TCP_MISS
< Content-Security-Policy: self *.24hourfitness.com
< X-Content-Security-Policy: self *.24hourfitness.com
< Strict-Transport-Security: max-age=16070400; includeSubDomains
< X-Azure-Ref-OriginShield: 07XLRZAAAAAB1H47pag9nSZPTg9eVlI5NU0pDRURHRTA1MTgAODIwNjYxNjctMTJiMi00ZDBhLTk1ZjQtZjIzZWM4OTQwNGFh
< X-Aem-Client-Country: US
< X-Azure-Ref: 07XLRZAAAAAAQwbRUwmklQqLTTU2LZk6yTEFYRURHRTIxMTIAODIwNjYxNjctMTJiMi00ZDBhLTk1ZjQtZjIzZWM4OTQwNGFh
< Date: Mon, 07 Aug 2023 22:40:45 GMT
<
Warning: Binary output can mess up your terminal. Use "--output -" to tell
Warning: curl to output it to your terminal anyway, or consider "--output
Warning: <FILE>" to save to a file.
* Failure writing output to destination
* Closing connection 0
* schannel: shutting down SSL/TLS connection with webdev.24hourfitness.com port 443

Description

How to reproduce the misbehavior (-> curl call)

Logs

Your Environment

CRS version (e.g., v3.3.4):
Paranoia level setting (e.g. PL1) :
ModSecurity version (e.g., 2.9.6):
Web Server and version or cloud provider / CDN (e.g., Apache httpd 2.4.54):
Operating System and version:

Confirmation
[ ] I have removed any personal data (email addresses, IP addresses,
passwords, domain names) from any logs posted.

Please provide some of the error messages (-> Alert Message) and a curl call to reproduce the behavior.
If you trigger 942340 across the board, chances are it's one of the cookies that upsets the rule set.

Hey @charlesmorin1, are you still interested in this? If yes, please submit more information. Otherwise, we'll close the issue.


Document api breakage in moving to pysal 3

This ticket serves as a place of record for breakage in the pysal api as we move to python 3.0 only.

@sjsrey Can this be closed?


Can't access storage account from a logic ap.

The example shows giving access for a storage account, but then the logic app goes for ARM. Could you update the docs to show how to access the storage account?
I can't get it to work, I gave "Owner" access to the logic app, and I'm trying to call the REST API, yet I get 403. I tried calling both, blob and queue endpoints.

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

ID: cdfb9148-5d81-f65e-3cb3-aa7c5eb5867b
Version Independent ID: 5051db4d-83d1-a230-6fcb-9238a384961d
Content: Authenticate with managed identities - Azure Logic Apps
Content Source: articles/logic-apps/create-managed-service-identity.md
Service: logic-apps
GitHub Login: @kevinlam1
Microsoft Alias: klam

Hi @erndob Thank you for your feedback! We will review and update as appropriate.

I got it to work. I wasn't passing the correct api version in the header, since it's done differently on the storage account api.

@erndob Thanks for the update. I am glad that it worked. Appreciate if you can share the details that how you made it worked with snapshots to help others in the community.
we will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

I messed up because the storage API is quite different from the other, json based, apis.
Some info for people, if they Bing this:

Keep in mind that at the moment, only blob storage and queue storage is supported, so the "storage" audience can be misleading.
API version is passed as a header, not a query parameter.
If you need the date header, to get the correct format in the logic apps use utcNow("r")

@erndob Thanks for posting the solution. Appreciate your effort.


JIT: Add space between operator in emitted code

Fixes #470

I am going to look at this tomorrow; I was cranking on a deadline recently. Thanks for a fix! It would be awesome if we could get test coverage somehow.

I'm writing a test for this right now.

Thanks!


fix: remove problem 39 cache

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

7 of 7 (100.0%)  changed or added relevant lines in 1 file are covered.
5 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.3%) to 90.296%

Files with Coverage Reduction
New Missed Lines
%

src/problem_39/mod.rs
5
92.79%

Totals

Change from base Build<PHONE_NUMBER>:
-0.3%

Covered Lines:
1889

Relevant Lines:
2092

💛  - Coveralls


[LexicalTextNode] Refactor: remove unnecessary type casting

Description
noticed that DOMConversionFn<HTMLElement> -> it takes in a HTMLElement as input, but quite alot of conversion fn implementations have the input type as Node, and then later cast it as a HTMLElement (or its child classes).
seems redundant as we can directly have the input type as the HTMLElement
Test plan
npm run ci-check

failing is known flaky test & dont look related
https://github.com/facebook/lexical/actions/runs/9294132717/job/25578693971?pr=6201


[BUG]

Version of Advanced Rocketry

AdvancedRocketry 1.12.2-1.5.0-139-UNSTABLE build -139 for 1.12.2

Have you verified this is an issue in the latest unstable build

[x] Yes

Version of LibVulpes

LibVulpes-1.12.2-0.4.0-57-universal

Version of Minecraft

1.12.2

Does this occur without other mods installed

[x] Y

Crash report or log (if applicable)
https://paste.dimdev.org/movarebebu.mccrash
Description of the problem
Got this error message when opening 1 of 2 input hatches on the electric arc furnace

can you try it again with the latest builds?  (libvulpes 59 and AR 153)


JPMS open modules do not export packages

We incorrectly assumed that open modules would export all packages, which is not the case.

Open modules allow for (deep) reflection to all classes, which is something we want to allow. We have no good reason right now to restrict reflection in Mutiny.
Exported packages need to be added manually, there is no wildcard. I did a selection of packages that shall be in the public API.
I fixed transitive 'requires' clauses: Reactive Streams is part of the Mutiny API, just like SmallRye Common Annotations.

AbstractUni and AbstractMulti are definitely part of the public API (advanced use case where you need to implement your own operator). We use them in Reactive Messaging. I know that the Cassandra Quarkus extension is also using them.

@cescoffier See the latest changes to align RevAPI and the module descriptor

Renamed a class in ...operators.multi: RevAPI is ok
Renamed a class in `...operators.: RevAPI complains


Links in graph missing

          graph TD;
            A-->B;
            B-->C;
            C-->D;
            A-->D;

gives

+1

With your graph, I see only the Lines and not the arrows. ;)

Yeah, I realized: Just use graphviz, duh (aka: it’s an already solved problem).

Have you included the mermaid.css? If not that would probably solve your issue. Hope it helps.
If this is rendered via the CLI the latest unreleased version includes the stylesheet.

Arrows not need the CSS file. Here is a strange problem with BASE html-tag. Please create a blank html file and drop this in it:
<!DOCTYPE html>

<base href="XXX">
<meta charset="utf-8">
<title></title>

graph LR;
    A-->B;
    B-->C;

You will not see any arrows. BUT now remove "XXX" in href of BASE tag and its will work. Strange, or?

If you look in the svg code for an arrow it will make more sense:
<g class="edgePath" style="opacity: 1;">
<path class="path" d="M44,55L44,80L44,105" marker-end="url(#arrowhead7)" style="fill:none"></path><defs>
   <marker id="arrowhead7" viewBox="0 0 10 10" refX="9" refY="5" markerUnits="strokeWidth" markerWidth="8" markerHeight="6" orient="auto">
      <path d="M 0 0 L 10 5 L 0 10 z" style="fill: #333"></path>
   </marker>
</defs>
</g>

Look at the marker end part where it is actually a url pointing to the definition. Thats why the base path change messes it up.

Yes. I found this stackoverflow question for this topic:
http://stackoverflow.com/questions/18259032/using-base-tag-on-a-page-that-contains-svg-marker-elements-fails-to-render-marke
Its possible to add location.href directly in mermaid for solve this problem?
"location.href" without the fragment part. => http://domain.tld/folder/file.html#fragment ;)

I believe it could be fixed by replacing this line https://github.com/knsv/mermaid/blob/fcb2af7/src/diagrams/sequenceDiagram/sequenceRenderer.js#L231:
        line.attr("marker-end", "url(#arrowhead)");

with:
        line.attr("marker-end", "url(" + window.location.href + "#arrowhead)");

And the same for the #crosshead line four lines below.
That, at least, is the fix I made for another project (which worked): https://github.com/vega/vega/pull/213/files

Excellent!
That works for sequenceDiagrams. I can apply the same approach for flowcharts but through adjusting the generated code from dagre-d3.
Thanks!

Small issue. The url wont work if the actual url contains a hash section as in http://a.b.c/d#e
Using
window.location.protocol+'//'+window.location.host+window.location.pathname

instead of location.href solves this though so all is well!

Fix for this issue was released in mermaid 0.5.2. Will close this now, let me know if you issues still remain.

@knsv, thanks for implementing the fix! However, I think the solution still isn't quite right -- if your URL is something like http://a.b.c/d?e#f, then it returns "http://a.b.c/d", but the ?e should be in there also.

Also, there's the port number to consider... I found a couple of relevant questions on SO:

You are right. It is easy to underestimate urls. Have you found instances where this does not work?
In the end the code that handles this is the following:
window.location.protocol+'//'+window.location.host+window.location.pathname +window.location.search;

The port is included with window.location.host and the hash should be excluded otherwise there is two hashes in the url.
My test url:
http://<IP_ADDRESS>:8080/advancedBasteTag(.html?apa=1(a)2#as
gives:
http://<IP_ADDRESS>:8080/advancedBasteTag\(.html?apa=1\(a\)2#arrowhead
which renders the arrow heads.

That looks good to me, but I'm not an expert on this topic. This issue came to mind again because I was doing some SVG stuff in a project. I ended up using window.location.href.split("#")[0], but I don't know if that's any better or worse than your solution.

Sounds good! Thanks for keeping mermaid in mind!


split Cedar.Thm.Data.List into submodules

Another PR suggesting a reorganization that folks can feel free to shoot down if it's undesired or not idiomatic or etc.
Cedar/Thm/Data/List.lean is very long (1600ish lines).  This PR breaks it into submodules, roughly along the borders of its existing section headers.  This brings a few advantages:

Easier to find what you're looking for when it's not a 1600-line file
Potential performance improvement, as some List lemmas can now be processed in parallel during lake build
Definitely a performance improvement when working in the List file itself, as restarting the file doesn't have to restart all of the List lemmas from the top

Idiomatic Lean code prefers one long file for "grab bag lemmas" over many small files.  For example, the List lemmas module in the standard library is all in a single file with 2,800+ LOC: https://github.com/leanprover-community/batteries/blob/main/Batteries/Data/List/Lemmas.lean
I found this unusual when I started but have come to appreciate it.  When you don't know / remember the name of a lemma, searching for it across multiple files is a lot harder (for me at least) than quickly scrolling through one big file.
While we did our best to partition the list lemmas according to their functionality (as did the standard library), many of these lemmas deal with more than one function / definition (e.g., mem lemmas).  And since they can be located in only one file, if I guess wrong where a lemma about a function may be located, it's going to take me a lot longer to find it than scrolling through the mega file.  (Speaking from experience: I'm currently regretting organizing these "grab bag lemmas" into several smaller modules in the internal development early on, and will merge some of these into fewer bigger files.)
So, I'd vote to keep the One Big File for List, except for the canonical and insertCanonical lemmas.  Those are special-purpose and moving them all together into a separate Canonical module would be consistent with how the standard library is organized.  For example, they put specialized permutation lemmas into the Count module: https://github.com/leanprover-community/batteries/blob/main/Batteries/Data/List/Perm.lean

That sounds fine to me

Here is the Std-like split (and a few new lemmas): https://github.com/cedar-policy/cedar-spec/pull/321

Closing this in favor of #321


[LoadingButton] Improve spacing for text variant in ButtonStartIcon/ButtonEndIcon

[x] I have followed (at least) the PR section of the contributing guide.

Fixes #29066

Looks like this effort is a duplicate of https://github.com/mui-org/material-ui/pull/29194 I am closing this one as the other seems like it is closer to a final stage. Thank you for the work!


Make Zbar optional or replacle ZXing to improve performance and quality of the barcode scanner for android.

Steps to reproduce

Create a project for android and install react-native-camera.
Create another project for android and install react-native-zbar-android
Test them both and see the difference.

Expected behaviour
react-native-camera for android should focus on barcode easily, handle glare or bright environments better and provide a better experience by not slowing down the device.
Actual behaviour
react-native-camera zxing implementation has trouble focusing, reading barcodes when there is too much glare or if it's too bright outside. In my test with react-native-zbar-android I didn't experience any of these issues and didn't slow down the device.
Environment

Node.js version:
React Native version: 0.40
React Native platform + platform version:  Android 6.0

react-native-camera
Version:  "master"
react-native-zbar-android is not complete yet, please review this repo for an improved android implementation of the Zbar and ZXing for android:
https://github.com/dm77/barcodescanner

+1  - The current scanning capabilities work great on iOS! However on Android, Zbar does seem to scan faster. I'm assuming this wouldn't be trivial, but how difficult would it be to integrate the option to initialize the scanner w/ either scanning library (zxing or zbar)?

I have worked with Android scanners for years for automotive applications.  zxing is weak.  The native iOs scanner blows it out of the water.  The only library that comes close is zbar.

+1000

Do you all know if there are any other react native alternatives for barcode scanning?

Zxing vs Zbar vs Google Barcode
Zxing is a bad idea today because there is no android specific library that manages the camera APIs, then we need to copy-paste and hack it for work (difficult to maintain in a current v0.X). Zbar is ok but is poor in quality (often misread the numbers). Google Barcode is the best in almost all but it need the Mobile Vision library downloaded in the device.
in my projects react-native-camera work fine in ios, for android I have https://github.com/ekreutz/react-native-barcode-scanner-google and it works excellently.

@samhv   In china, react-native-barcode-scanner-google  not working. will got no_play_service error

@fenglu09 are you testing on a simulator or a physical device?

@jqn  On a physical device. Meizu M5 Note， Android 7.0.

@DavidRajPaul can u review this https://github.com/react-native-community/react-native-camera/pull/862
let us if this is good, and we can merge it


uninitialized constant Devise::Models::TwitterOauth

Hello there,
I've installed:
devise (1.4.2)
devise-twitter (0.1.1)
and have followed the instructions on the devise-twitter page.
I'm calling: http://localhost:3000/user/sign_in/twitter
and get
NameError
uninitialized constant Devise::Models::TwitterOauth
app/models/user.rb:12
class User < ActiveRecord::Base
To use devise-twitter don't forget to include the :twitter_oauth module:
e.g. devise :database_authenticatable, ... , :twitter_oauth
IMPORTANT: If you want to support sign in via twitter you MUST remove the
:validatable module, otherwise the user will never be saved
since it's email and password is blank.
:validatable checks only email and password so it's safe to remove
Include default devise modules. Others available are:
:token_authenticatable, :confirmable, :lockable and :timeoutable
devise :database_authenticatable, :registerable,
:recoverable, :rememberable, :trackable, :twitter_oauth
Setup accessible (or protected) attributes for your model
attr_accessible :email, :password, :password_confirmation, :remember_me
end
What am I missing?
Thanks
Thomas

I have the same problem with devise-1.4.9 :-(


Correction for raw affiliation B.Com Hons. St. Xavier's University, Kolkata, West Bengal, India; St. Xavier's University, Kolkata, West Bengal, India

Correction needed for raw affiliation B.Com Hons. St. Xavier's University, Kolkata, West Bengal, India; St. Xavier's University, Kolkata, West Bengal, India
raw_affiliation_name: B.Com Hons. St. Xavier's University, Kolkata, West Bengal, India; St. Xavier's University, Kolkata, West Bengal, India
new_rors: 008kngs41
previous_rors: 039543c28
works_examples: W3199187046
contact: 0815dcd014196f1e7368cd5e01308d72:b03f5362 @ ourresearch.org

This issue was accepted and ingested by the OpenAlex team on 2024-10-10. The new affiliations should be visible within the next 7 days.


bootcmd usage

RancherOS Version: 1.3
baremetal
I am trying to use the cloud-init bootcmd directive without any success.
I need a way to execute a command before system-docker running.
In fact i would like to create a cgroup and use it for system-docker container thanks to --cgroup-parent docker daemon option.
I tried to use bootcmd to do that but nothing happens while my config seems to be valid (Parsing with success).
Maybe i did not fully understand the concept behind.
I was thinking to use init as a workaround but no way to find where i can put an init script to do that.
Does the root FS used at boot time is also docker image ?
/base/var/lib/system-docker/overlay2/b83175f0a8521b8f66bee0569d30ffd4eecc3946abea4c6b04b78b8300a5125a/diff/etc/rc0.d
find: File system loop detected; '/base/var/lib/system-docker/overlay2/d8dab4a0c482a18b3c08fce656dd8add4d8e23ecd06344b6aef3f4d2058352a4/merged' is part of the same file system loop as '/'.
Many thanks for any feedbacks,

The bootcmd runs during the bootstrap process.
In order to boot speed, in fact, now bootstrap does not run every boot. It just runs when you first boot via docker-machine or iPXE.
So in this case, using the bootcmd is really invalid.
In this case, you only need to configure the dockerd args and don't need to create it manually in advance.
https://docs.docker.com/engine/reference/commandline/dockerd/#miscellaneous-options
  system_docker:
    extra_args:
    - --cgroup-parent
    - /rancher

BTW, there is currently no init script that can be executed before system-docker.
If you really need it, you may need to customize the ros.

thanks for your feedback,
I will try as soon as possible but i have some doubt.
Curently, the system-docker use a "docker" cgroup.
I would like to modify it. Change cpuset cpu restrictions / exclusivity parameters. So i mounted the root FS into the console to access it. But i am unable to change it.
I can create a new cgroup.
I can also modify the docker-user cgroup (concole c-group) after stoping docker-user service.
But i am stuck with system-docker as i am unable to stop it.
Even as root, it does not work.
Do you think that it can work with a new one ? i assume that the rancher cgroup will be created but i should have the same issue to modify it after.

@niusmallnan
i made the test using extra_args.
it works as i have my cgroup created and container inside.
But i am still blocked as i can 't edit some cgroup option (ressource busy). I had this issue with user-docker and the workaround was to stop it, change cgroup parameter, then restart user-docker. But i cant do that with system-docker.

Can you provide how did you edit some cgroup's options?

I follow the syntax described below :
2.2 Adding/removing cpus
This is the syntax to use when writing in the cpus or mems files
in cpuset directories:
/bin/echo 1-4 > cpuset.cpus		-> set cpus list to cpus 1,2,3,4
/bin/echo 1,2,3,4 > cpuset.cpus	-> set cpus list to cpus 1,2,3,4
To add a CPU to a cpuset, write the new list of CPUs including the
CPU to be added. To add 6 to the above cpuset:
/bin/echo 1-4,6 > cpuset.cpus	-> set cpus list to cpus 1,2,3,4,6
Similarly to remove a CPU from a cpuset, write the new list of CPUs
without the CPU to be removed.
To remove all the CPUs:
/bin/echo "" > cpuset.cpus		-> clear cpus list
Here are all my tries. Using the cgroup accessible by default into the console container or using a "base" mount point mounting the root host fs. I end up with "Device or resource busy" anyway.
rancher@ebu-srt2-lab:~/FlameGraph$ sudo echo 0 | sudo tee /sys/fs/cgroup/cpuset/docker/cpuset.cpu_exclusive
0
rancher@ebu-srt2-lab:~/FlameGraph$ sudo echo 0 | sudo tee /sys/fs/cgroup/cpuset/docker/cpuset.cpus
0
tee: /sys/fs/cgroup/cpuset/docker/cpuset.cpus: Device or resource busy
rancher@ebu-srt2-lab:~/FlameGraph$ ^C
rancher@ebu-srt2-lab:~/FlameGraph$ sudo echo 0-1 | sudo tee /sys/fs/cgroup/cpuset/docker/cpuset.cpus
0-1
tee: /sys/fs/cgroup/cpuset/docker/cpuset.cpus: Device or resource busy
rancher@ebu-srt2-lab:~/FlameGraph$ sudo echo 0-1 | sudo tee /base/sys/fs/cgroup/cpuset/docker/cpuset.cpus
0-1
tee: /base/sys/fs/cgroup/cpuset/docker/cpuset.cpus: Device or resource busy
rancher@ebu-srt2-lab:~/FlameGraph$ sudo echo 0 | sudo tee /base/sys/fs/cgroup/cpuset/docker/cpuset.cpus
0
tee: /base/sys/fs/cgroup/cpuset/docker/cpuset.cpus: Device or resource busy
rancher@ebu-srt2-lab:~/FlameGraph$ sudo echo 1 | sudo tee /base/sys/fs/cgroup/cpuset/docker/cpuset.cpus
1
tee: /base/sys/fs/cgroup/cpuset/docker/cpuset.cpus: Device or resource busy
rancher@ebu-srt2-lab:~/FlameGraph$ sudo echo 0 | sudo tee /sys/fs/cgroup/cpuset/docker/cpuset.cpus
0
tee: /sys/fs/cgroup/cpuset/docker/cpuset.cpus: Device or resource busy
rancher@ebu-srt2-lab:~/FlameGraph$ sudo echo 1 | sudo tee /sys/fs/cgroup/cpuset/docker/cpuset.cpus
1
tee: /sys/fs/cgroup/cpuset/docker/cpuset.cpus: Device or resource busy
rancher@ebu-srt2-lab:~/FlameGraph$ sudo echo 1 > /sys/fs/cgroup/cpuset/docker/cpuset.cpus
-bash: /sys/fs/cgroup/cpuset/docker/cpuset.cpus: Permission denied
rancher@ebu-srt2-lab:~/FlameGraph$ sudo su
root@ebu-srt2-lab:/home/rancher/FlameGraph# sudo echo 1 > /sys/fs/cgroup/cpuset/docker/cpuset.cpus
echo: write error: Device or resource busy
root@ebu-srt2-lab:/home/rancher/FlameGraph# sudo echo 1 > /base/sys/fs/cgroup/cpuset/docker/cpuset.cpus
echo: write error: Device or resource busy
root@ebu-srt2-lab:/home/rancher/FlameGraph#


Attempting to import textblob from within celery task results in SIGSEGV.

I'm attempting to use TextBlob within a Celery task.  I've isolated the line
import textblob
as causing a SIGSEGV fault, causing the worker process to exit prematurely.  The exactly same code works fine when run in-process (in this case, in the Django dev server).
I would supply more of a stacktrace, but there doesn't seem to be one available since it's a system level, fatal error. The import statement is definitely where it happens however.
If it's a misconfiguration / misuse on my end, any help is appreciated.  Thanks!

This Celery issue, and that spaCy issue, seem to point toward TextBlob not being fork safe, though I can't reproduce the problem.
This is an old issue; will close it for now, but if anyone comes across it again, please try the workaround in the Celery thread, or post some code that demonstrates it, and we can re-open it.


soapyaudio: init at 0.1.1

Motivation for this change
Adding audio input plugin for SoapySDR
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

Can you please make a separate commit for the addition of soapysdr to sdrangel.

@markuskowa Done, #107293

Result of nixpkgs-review pr 107288 run on x86_64-linux 1

13 packages built:

cubicsdr
gqrx
gr-ais
gr-gsm
gr-osmosdr
python37Packages.soapysdr-with-plugins
python38Packages.soapysdr-with-plugins
python39Packages.soapysdr-with-plugins
qradiolink
rtl_433
soapyaudio
soapysdr-with-plugins
welle-io


Problem running the keyboard example

Hi, I'm having an error when i try to run the example of the keyboard:
use rsautogui::keyboard::{self, Key};

fn main() {
    keyboard::typewrite("lorem ipsum"); // Simulates typing the string provided.

    keyboard::key_down(Key::Layout('a')); // Presses specified key down.
    keyboard::key_up(Key::Layout('a')); // Releases specified key up.
    keyboard::key_tap(Key::Layout('a')); // Performs specified key up and down.
}

I get the next error when i run cargo run .

Here it's the information about my rust, cargo and rsautogui version
OS: Windows 10
rustc 1.66.0 (69f9c33d7 2022-12-12)
cargo 1.66.0 (d65d197ad 2022-11-15)
rsautogui = "0.1.5"

Hi!
Unfortunately, this is a problem of 1 of the dependencies of this crate. I'm planning on rewriting this crate from scratch and minimizing dependencies to avoid problems like this in the future. I'm just not sure when I can start doing it as I have a lot on my hand at the moment. Sorry about the inconvenience.
Cheers!

Should be fixed now


[SPARK-26271][FOLLOW-UP][SQL] remove unuse object SparkPlan

What changes were proposed in this pull request?
this code come from PR: https://github.com/apache/spark/pull/11190,
but this code has never been used, only since  PR: https://github.com/apache/spark/pull/14548,
Let's continue fix it. thanks.
How was this patch tested?
N / A

Can one of the admins verify this patch?

cc @cloud-fan, @gatorsmile, @hvanhovell ,@davies

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/testing-k8s-prb-make-spark-distribution-unified/5749/
Test PASSed.

Test build #99702 has started for PR 23227 at commit 5cb416d.

ok to test

Test build #99704 has started for PR 23227 at commit 5cb416d.

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/testing-k8s-prb-make-spark-distribution-unified/5751/
Test PASSed.

Test build #99704 has finished for PR 23227 at commit 5cb416d.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

retest this please

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99704/
Test FAILed.

Test build #99719 has started for PR 23227 at commit 5cb416d.

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/testing-k8s-prb-make-spark-distribution-unified/5764/
Test PASSed.

Test build #99702 has finished for PR 23227 at commit 5cb416d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/SparkPullRequestBuilder/99702/
Test PASSed.

Test build #99719 has finished for PR 23227 at commit 5cb416d.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99719/
Test FAILed.

retest this please

thanks, merging to master!

Test build #99731 has started for PR 23227 at commit 5cb416d.

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/testing-k8s-prb-make-spark-distribution-unified/5772/
Test PASSed.

thanks,@cloud-fan,@HyukjinKwon

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99731/
Test FAILed.