Composer does not work when PHP_XDEBUG_REMOTE_HOST is set

Hi - thanks for all your work on this project! I'm having an issue where I can't run Composer commands such as docker-compose exec --user 82 php composer update. When I run Composer commands nothing happens and the command hangs. Same for drush commands, such as docker-compose exec --user 82 php drush --version.
Debugging locally, it seems this issue is caused by the Xdebug PHP_XDEBUG_REMOTE_HOST environmental variable.
To reproduce:

Place the Docker4Drupal docker-compose.yml in a clean directory
Uncomment all of the PHP Xdebug settings except PHP_XDEBUG_REMOTE_HOST: "<IP_ADDRESS>", so your environment variables should look like:

    environment:
      PHP_SENDMAIL_PATH: /usr/sbin/sendmail -t -i -S mailhog:1025
      PHP_XDEBUG: 1
      PHP_XDEBUG_DEFAULT_ENABLE: 1
      PHP_XDEBUG_REMOTE_CONNECT_BACK: 0         # This is needed to respect remote.host setting bellow
      # PHP_XDEBUG_REMOTE_HOST: "<IP_ADDRESS>"  # You will also need to 'sudo ifconfig lo0 alias <IP_ADDRESS>'

Run docker-compose up -d
Run docker-compose exec --user 82 php composer -V and you should see Composer version 1.3.3.
Run docker-compose down -v
Edit the docker-compose.yml, uncommenting PHP_XDEBUG_REMOTE_HOST: "<IP_ADDRESS>", so your environment variables should look like:

    environment:
      PHP_SENDMAIL_PATH: /usr/sbin/sendmail -t -i -S mailhog:1025
      PHP_XDEBUG: 1
      PHP_XDEBUG_DEFAULT_ENABLE: 1
      PHP_XDEBUG_REMOTE_CONNECT_BACK: 0         # This is needed to respect remote.host setting bellow
      PHP_XDEBUG_REMOTE_HOST: "<IP_ADDRESS>"  # You will also need to 'sudo ifconfig lo0 alias <IP_ADDRESS>'

Run docker-compose exec --user 82 php composer -V and nothing is outputted, the command hangs.

I'm running OS X, and I ran sudo ifconfig lo0 alias <IP_ADDRESS>

The following configuration works for me in any cases:
PHP_XDEBUG=1
PHP_XDEBUG_DEFAULT_ENABLE=1
PHP_XDEBUG_REMOTE_HOST=<Docker0 IP address>
PHP_IDE_CONFIG=serverName=<Server name in IDE>
PHP_XDEBUG_REMOTE_CONNECT_BACK=0

Where <Docker0 IP address> is IP address of the docker0 interface (<IP_ADDRESS> for me).
You can read more about <Server name in IDE> here.
My configuration is: .
Also you may increase the number of simultaneous debug connections:

Hi @PavelPrischepa - thanks for taking the time to respond, I appreciate it! As it turns out, restarting PHPStorm fixed the problem. Now all is working with the original docker-compose.yml settings:
    environment:
      PHP_SENDMAIL_PATH: /usr/sbin/sendmail -t -i -S mailhog:1025
      PHP_XDEBUG: 1
      PHP_XDEBUG_DEFAULT_ENABLE: 1
      PHP_XDEBUG_REMOTE_CONNECT_BACK: 0         # This is needed to respect remote.host setting bellow
      PHP_XDEBUG_REMOTE_HOST: "<IP_ADDRESS>"  # You will also need to 'sudo ifconfig lo0 alias <IP_ADDRESS>'

When closing PHPStorm I recieved the message
Process 'composer' is Running
Do you want to disconnect from the process 'composer'?

I suspect that was the culprit. Thanks again, I'm going to close this issue!


Update powershell kernel version to 0.1.4

This PR updates the PowerShell Kernel to the newest version.
Changelog: https://github.com/vors/jupyter-powershell/blob/master/CHANGELOG.md

Do we need to update https://github.com/microsoft/azuredatastudio/blob/main/extensions/notebook/src/test/python/jupyterInstallation.test.ts#L232 as well?

@chlafreniere. Yup will update that as well!

Pull Request Test Coverage Report for Build 338488915

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 39.551%

Totals

Change from base Build 338464452:
0.0%

Covered Lines:
20655

Relevant Lines:
47970

💛  - Coveralls


Updates samples for updates made to NDK plugin

target is now required

Verified locally.

This might break on the build server. If so I will send an update to Hugo to let him know that he needs to update the NDK there..


Redshift copy() varchar length bug

Describe the bug
A clear and concise description of what the bug is.
I'm getting a column length error when writing a dataframe to redshift via copy method.
    raise self.error
redshift_connector.error.ProgrammingError: {'S': 'ERROR', 'C': 'XX000', 'M': 'Spectrum Scan Error', 'D': '\n  -----------------------------------------------\n  error:  Spectrum Scan Error\n  code:      15001\n  context:   The length of the data column display_name is longer than the length defined in the table. Table: 256, Data: 320, File name: https://{...}20fee7fe7_0.snappy.parquet?&X-Amz\n  query:     71875904\n  location:  dory_util.cpp:945\n  process:   fetchtask_thread [pid=19465]\n  -----------------------------------------------\n', 'F': '/home/ec2-user/padb/src/sys/xen_execute.cpp', 'L': '9226', 'R': 'pg_throw'}

In my code, I'm dynamically setting the column_length with a method like this.
column_length =  df['display_name'].str.len().max() + 1

In this particular case, this column / series of the DF is returning 218. The error says that column length is 320.
Under the hood, the copy() method is encoding the str column - which is reasonable but I had to dig under the surface to figure out why I was seeing this issue.
To Reproduce
Steps to reproduce the behavior. Also add details about Python and Wrangler's version and how the library was installed.
P.S. Don't attach files. Please, prefer add code snippets directly in the message body.

I was able to pin point where this unexpected behavior pops ups.
assert selected_df['display_name'].str.len().max() == 218.0

transformed = wr._data_types._cast_pandas_column(selected_df, 'display_name', 'string', 'bytes')

assert transformed['display_name'].str.len().max() == 320.0

I was able to pin point where this unexpected behavior pops ups.
assert selected_df['display_name'].str.len().max() == 218.0

transformed = wr._data_types._cast_pandas_column(selected_df, 'display_name', 'string', 'bytes')

assert transformed['display_name'].str.len().max() == 320.0

One suggestion for the API is to have an auto_columns_length setting. For determining column lengths on my ETL jobs, I have the below ugly function that essentially does that.  Unfortunately this bug / edge isn't accounted for in this.

def get_col_len(series):
    try:
        col_len_plus_one = series.str.len().max() + 1
    except:
        return 256
    if pd.isnull(col_len_plus_one):
        return 256
    return int(max(col_len_plus_one, 256))

def get_varchar_lengths(df):

    varchar_lengths = {}

    for column in df.columns:
        try:
            series = df[column]

            col_len = get_col_len(series)
            if col_len > 256:
                varchar_lengths[column] = col_len
        except Exception as ex:
            logger.error(ex)
            traceback.print_exc()

    return varchar_lengths

One suggestion for the API is to have an auto_columns_length setting. For determining column lengths on my ETL jobs, I have the below ugly function that essentially does that.  Unfortunately this bug / edge isn't accounted for in this.

def get_col_len(series):
    try:
        col_len_plus_one = series.str.len().max() + 1
    except:
        return 256
    if pd.isnull(col_len_plus_one):
        return 256
    return int(max(col_len_plus_one, 256))

def get_varchar_lengths(df):

    varchar_lengths = {}

    for column in df.columns:
        try:
            series = df[column]

            col_len = get_col_len(series)
            if col_len > 256:
                varchar_lengths[column] = col_len
        except Exception as ex:
            logger.error(ex)
            traceback.print_exc()

    return varchar_lengths

Again super helpful project here!  Has saved me so much time in writing something of lesser quality myself! Thank you!

Again super helpful project here!  Has saved me so much time in writing something of lesser quality myself! Thank you!

Hi @njdanielsen, Thanks for reaching outtoo us  again!
I don't think we have something wrong with the current implementation. The point is that some characters (the special ones) with demand more than 1 byte to be represented. So probably the best way to measure what you want would be:
df["COLUMN_NAME"].str.encode(encoding="utf-8").apply(len)

I think your two functions are quite useful and the auto_columns_length a terrific idea. We could start just adding these two functions adapted in our code base and making it available for everybody and then in a second step we can incorporate it into the copy fucntion through the auto_columns_length argument.
What do you think? Could you open a pull request adding your two functions in our library?

Hi @njdanielsen, Thanks for reaching outtoo us  again!
I don't think we have something wrong with the current implementation. The point is that some characters (the special ones) with demand more than 1 byte to be represented. So probably the best way to measure what you want would be:
df["COLUMN_NAME"].str.encode(encoding="utf-8").apply(len)

I think your two functions are quite useful and the auto_columns_length a terrific idea. We could start just adding these two functions adapted in our code base and making it available for everybody and then in a second step we can incorporate it into the copy fucntion through the auto_columns_length argument.
What do you think? Could you open a pull request adding your two functions in our library?

Closing due the lack of interactions.

Recreating the column in redshift using varchar(max) as column type will solve the issue


Update gems

fixes #1777 Upgrade codeclimate-test-reporter to version 1.0.5
fixes #1776 Upgrade sassc to version 1.11.2

Thanks!


Dev xxxx xxx fix #1403

I2C correction (F1, F2, F4, L1):
// Wait the STOP condition has been previously correctly sent
// This timeout can be avoid in some specific cases by simply clearing the STOP bit
timeout = FLAG_TIMEOUT;
while ((i2c->CR1 & I2C_CR1_STOP) == I2C_CR1_STOP) {
    if ((timeout--) == 0) {
        return 1;
    }
}

// Generate the START condition
i2c->CR1 |= I2C_CR1_START;

I2C correction (F0, F3, F7, L0, L4):
// Wait the STOP condition has been previously correctly sent
timeout = FLAG_TIMEOUT;
while ((i2c->CR2 & I2C_CR2_STOP) == I2C_CR2_STOP){
    if ((timeout--) == 0) {
        return 1;
    }
}

// Generate the START condition
i2c->CR2 |= I2C_CR2_START;

RTC correction (all but F1):
// Check if RTC is already initialized
if ((RTC->ISR & RTC_ISR_INITS) ==  RTC_ISR_INITS) return;

clock time out :
high speed @ 200ms in hal conf
low speed @ 100ms in hal rcc
nucleo 32 on HSI by default in system config

I still would like to find out those numbers, why there are 100/200 ms


"nodeTitle is not defined" error when adding "Join columns from 2nd layer" analysis

How to reproduce it:

Create a map with one layer.
Add a Join columns from 2nd layer analysis

The dev console will show the error nodeTitle is not defined

Ouch, related to #9173 for sure, I'll take this one

@viddo: 👌  thanks!


add node failure use script AddNode.ps1

2019/06/05-12:26:50.367,Info,10012,Testability.FabricClientRetryHelper,Enter ExecuteFabricActionWithRetryAsync() with function
2019/06/05-12:26:50.367,Verbose,7104,SystemFabric.ClusterManager.GetFabricUpgradeProgress,AsyncCalloutAdapter-63835064: begin
2019/06/05-12:26:50.367,Verbose,7104,SystemFabric.ClusterManager.GetFabricUpgradeProgress,SharedNativeObject-11454272 is initialized. CurrentReferenceCount=1
2019/06/05-12:26:50.367,Verbose,7104,SystemFabric.ClusterManager.GetFabricUpgradeProgress,SharedNativeObject-11454272.TryAdd: SUCCESS. CurrentReferenceCount=2
2019/06/05-12:26:50.367,Verbose,5056,SystemFabric.ClusterManager.GetFabricUpgradeProgress,AsyncCalloutAdapter-63835064: callback
2019/06/05-12:26:50.367,Verbose,7104,SystemFabric.ClusterManager.GetFabricUpgradeProgress,AsyncCalloutAdapter-63835064: end
2019/06/05-12:26:50.367,Verbose,5056,SystemFabric.ClusterManager.GetFabricUpgradeProgress,SharedNativeObject-11454272 is already initialized. CurrentReferenceCount=2
2019/06/05-12:26:50.367,Verbose,5056,SystemFabric.ClusterManager.GetFabricUpgradeProgress,SharedNativeObject-11454272.TryAdd: SUCCESS. CurrentReferenceCount=3
2019/06/05-12:26:50.367,Verbose,5056,SystemFabric.ClusterManager.GetFabricUpgradeProgress,SharedNativeObject<SharedNativeObject-11454272>.Release: CurrentReferenceCount=3
2019/06/05-12:26:50.367,Verbose,7104,SystemFabric.ClusterManager.GetFabricUpgradeProgress,SharedNativeObject<SharedNativeObject-11454272>.Release: CurrentReferenceCount=2
2019/06/05-12:26:50.367,Verbose,7104,SystemFabric.ClusterManager.GetFabricUpgradeProgress,SharedNativeObject<SharedNativeObject-11454272>.Release: CurrentReferenceCount=1
2019/06/05-12:26:50.367,Info,10012,Testability.FabricClientRetryHelper,Enter ExecuteFabricActionWithRetryAsync() with function
2019/06/05-12:26:50.367,Verbose,7084,SystemFabric.ClusterManager.GetClusterManifest,AsyncCalloutAdapter-15688314: begin
2019/06/05-12:26:50.367,Verbose,7084,SystemFabric.ClusterManager.GetClusterManifest,SharedNativeObject-52307948 is initialized. CurrentReferenceCount=1
2019/06/05-12:26:50.367,Verbose,7084,SystemFabric.ClusterManager.GetClusterManifest,SharedNativeObject-52307948.TryAdd: SUCCESS. CurrentReferenceCount=2
2019/06/05-12:26:50.367,Verbose,7084,SystemFabric.ClusterManager.GetClusterManifest,SharedNativeObject<SharedNativeObject-52307948>.Release: CurrentReferenceCount=2
2019/06/05-12:26:50.367,Verbose,5056,SystemFabric.ClusterManager.GetClusterManifest,AsyncCalloutAdapter-15688314: callback
2019/06/05-12:26:50.367,Verbose,5056,SystemFabric.ClusterManager.GetClusterManifest,SharedNativeObject-52307948 is already initialized. CurrentReferenceCount=1
2019/06/05-12:26:50.367,Verbose,5056,SystemFabric.ClusterManager.GetClusterManifest,SharedNativeObject-52307948.TryAdd: SUCCESS. CurrentReferenceCount=2
2019/06/05-12:26:50.367,Verbose,5056,SystemFabric.ClusterManager.GetClusterManifest,AsyncCalloutAdapter-15688314: end
2019/06/05-12:26:50.414,Info,5056,SystemFabricDeployer.SFDeployer,Running Best Practices Analyzer...
2019/06/05-12:26:50.414,Verbose,5056,SystemFabricDeployer.SFDeployer,Validating executing user is an Administrator.
2019/06/05-12:26:50.414,Verbose,5056,SystemFabricDeployer.SFDeployer,Validating CAB file is valid at D:\ServiceFabric\DeploymentRuntimePackages\MicrosoftAzureServiceFabric.6.4.664.9590.cab.
2019/06/05-12:26:51.539,Verbose,5056,SystemFabricDeployer.SFDeployer,Validating machines are reachable and have opened required ports.
2019/06/05-12:26:51.554,Verbose,5056,SystemFabricDeployer.SFDeployer,Using IPv4 address for <IP_ADDRESS>
2019/06/05-12:26:51.554,Verbose,5056,SystemFabricDeployer.SFDeployer,Verifying remote procedure ### call access against cluster machines.
2019/06/05-12:26:51.585,Warning,5056,FabricDeployer.FabricDeployer,System.UnauthorizedAccessException: 尝试执行未经授权的操作。
在 Microsoft.Win32.RegistryKey.Win32ErrorStatic(Int32 errorCode, String str)
在 Microsoft.Win32.RegistryKey.OpenRemoteBaseKey(RegistryHive hKey, String machineName, RegistryView view)
在 System.Fabric.FabricDeployer.Utility.GetHklm(String machineName)
在 System.Fabric.FabricDeployer.Utility.GetTempPathInner(String machineName)
在 System.Fabric.FabricDeployer.Utility.GetTempPath(String machineName)
2019/06/05-12:26:51.585,Error,5056,SystemFabricDeployer.SFDeployer,Machine <IP_ADDRESS> failed CheckRPCAccess. HRESULT=<IP_ADDRESS>
2019/06/05-12:26:51.585,Error,5056,SystemFabricDeployer.SFDeployer,Best Practices Analyzer determined environment has an issue. Please see additional BPA log output in DeploymentTraces folder.
2019/06/05-12:26:51.585,Error,5056,SystemFabricDeployer.SFDeployer,Add Node cancelled due to validation error(s) found by Best Practices Analyzer. Inspect details in DeploymentTraces log folder local to executing location.

Please try the latest 7.1 and see if you still hitting the issue. A fix had been made in this area in a 7.0 CU.


Concurrency Error in org.bouncycastle.math.ec.ECAlgorithms.implShamirsTrickWNaf

When using the SECP curves under heavy concurrent load multiplication will occasionally fail.
Caused by: java.lang.ArrayIndexOutOfBoundsException: 1
at org.bouncycastle.math.ec.ECAlgorithms.implShamirsTrickWNaf(Unknown Source)
at org.bouncycastle.math.ec.ECAlgorithms.implShamirsTrickWNaf(Unknown Source)
at org.bouncycastle.math.ec.GLVMultiplier.multiplyPositive(Unknown Source)
at org.bouncycastle.math.ec.AbstractECMultiplier.multiply(Unknown Source)
The following code, and attached source files, are a fairly minimal example.  Note, that while it fails often it is randomised.
I am using a MacBook Pro (MacBookPro14,3) running macOS 10.13.4, java version "1.8.0_131" and bcprov-jdk15on-159.jar.
import static java.util.stream.Collectors.toList;

import java.math.BigInteger;
import java.util.List;
import java.util.stream.IntStream;

import org.bouncycastle.jce.ECNamedCurveTable;
import org.bouncycastle.jce.spec.ECNamedCurveParameterSpec;
import org.bouncycastle.math.ec.ECPoint;
import org.bouncycastle.math.ec.custom.sec.SecP256K1Curve;

public class ConcurrencyError
{
	public static final ECNamedCurveParameterSpec group = ECNamedCurveTable.getParameterSpec("secp256k1");
	public static final SecP256K1Curve curve = new SecP256K1Curve();
	public static final ECPoint g = group.getG();
	public static final BigInteger q = curve.getOrder();
	public static final ECPoint aPoint = g.multiply(BigInteger.valueOf(3277561));

	public static BigInteger order() {	//Order of the group
		return curve.getOrder();
	}

	public static void foo() {
		final int N = 50000;
		final int M = 100;
		List<ECPoint> T = IntStream.range(0,N).mapToObj(__ -> aPoint).collect(toList());
		for (int i=0; i<M; ++i) {
			System.out.println("i = " + i);
			T = compute(T);
		}
	}

	private static List<ECPoint> compute(List<ECPoint> T) {
		final int N = T.size();
		final List<ECPoint> S = IntStream.range(0, N)
				.mapToObj(i -> T.get(i).add(T.get((i+1)%N)))
				.collect(toList());
		return IntStream.range(0, N).parallel()
				.mapToObj(i -> S.get(0).multiply(exponent(i)))
				.collect(toList());
	}

	public static BigInteger exponent(int i) {
		if (i%2 == 0) {
			return BigInteger.valueOf(7);
		} else {
			return  BigInteger.valueOf(3277561);
		}
	}

	public static void main(String[] args) {
		foo();
	}
}

BCECError.zip

Thanks, I can reproduce the exception here and am investigating. I'm pretty sure it's a race condition in WnafUtil.precompute or a related method, triggered by multiple threads launching a multiply on the same ECPoint simultaneously.
e.g. adding S.get(0).multiply(exponent(0)) before the parallel() task appears to make things work (but of course... random).

There's now a version in https://www.bouncycastle.org/betas for testing as well 160 b14 or later.

Released in 1.60


Swift 3.0 - Error

Is there any branch where I can get this fix?

Hi @anafinotti
Not yet, I'm working on updating all my swift libraries to Swift 3. I hope to get this done soon. I'll keep you posted.

Check out version 0.2.0 and let me know if that works for you.


MacOS static binaries

Is your feature request related to a problem? Please describe.
The releases page lacks a Darwin statically linked MacOS binaries.
Describe the solution you'd like
Build for Darwin/MacOS

Sorry, no plan for it, PRs are welcomed.
If you have trouble with brew on MacOS Ventura, check https://github.com/orgs/Homebrew/discussions/3822


Always "starting sync" on mac and failing to load on Mobile

Hi @hadynz, I had the following issues, not sure if you have any fix...
On Mac - tried to sync "My Clippings.txt", but the bottom bar always showed "starting sync" with no progress (see the screenshot);
On Mobile - every time I started the mobile app, it said "failed to lead the kindle plugin".
Any help is much appreciated.  Thanks!

It worked with the new update.  Thanks @hadynz!


Clean up NumberTextStyle

Short bullet points because 💤

make protocol simpler / more specific (overly generic)
merge internal protocols into main protocols (mostly useless)
replace Search with a method on Snapshot (less code)

The search method can use String/range(of:options:), etc, until I bump the minimum version to iOS 16+ (like, whenever Apple offers better SwiftUI-UIKit interop).


Migrate Healthy Thinking's gumplay page to to sunstar sharepoint

Page URL: https://main--sunstar--hlxsites.hlx.live/healthy-thinking/gumplay
PR: https://github.com/hlxsites/sunstar/pull/510

Done.


handle Mastodon incompatibility with www subdomain

@kickscondor reported that following their site from Mastodon isn't completing. i reproduced from<EMAIL_ADDRESS>looking at the bridgy fed logs, we get the AP Follow request, and send this Accept:
{
  "@context": "https://www.w3.org/ns/activitystreams",
  "object": {
    "object": "https://fed.brid.gy/www.kickscondor.com",
    "type": "Follow",
    "actor": "https://mastodon.technology/users/snarfed"
  },
  "type": "Accept",
  "id": "tag:fed.brid.gy:accept/kickscondor.com/https://mastodon.technology/1b4f5bd8-7f3b-4501-89cc-b9b223a09e37",
  "actor": "https://fed.brid.gy/www.kickscondor.com"
}

...to which mastodon returns HTTP 401 with no body. 😢
i suspect the problem is that we're mixing kickscondor.com and www.kickscondor.com.
https://kickscondor.com/ itself redirects to https://www.kickscondor.com/ , which is fine. both domains redirect bridgy fed's paths (webfinger etc) to bridgy fed directly, which should also be fine.
however, note that the actor in the AS2 activity above is www, but the id is the naked domain. we also store and return different public keys via webfinger for www vs naked, below, which is probably the root cause of the 401.
subject<EMAIL_ADDRESS>data:application/magic-public-key,RSA.oYe2XI8f3TdSI9Laagw7kPPvXr_GSN5h6F6XLA3STBlYpqqaZTXp8Xe6lp8U0BTkG8-NF9ueSO1chbDOx5rCICUaLawSKK8eI18_DqgrOy9xHS28Z_TWXJSiHZEvBpI3hmEZPpDwgdnnaNZCC5MsmdSepl7mH9WWkPxYOiC7vMk=.AQAB

subject<EMAIL_ADDRESS>data:application/magic-public-key,RSA.rCiWTbbPprgvZJMdVbvWzCRDHw6ebGtc1BxLcWJIYhkqrQjx5YFBec2Owzn16Ht6yg_dnGQmL1p5mgS8O_f3Ghin1WGC4aShUnKkU1Ia2Hkv-1wEt757GvUhpFgXwPg3xvCTrJwBs_Ypxb5MbMqrBoRQVHXU0tg5XJIkFpiFNU8=.AQAB

(#29 is somewhat related, but probably not the same thing, since its error actually included the message Public key not found.)

Okay, wow, thanks for looking at this! I have really not wanted to bother you, considering all of the projects you have going on.
I'm okay to just use<EMAIL_ADDRESS>then - however, I wonder why that also isn't working. The 'actor' and 'id' should match in that case.

looks like a handful of www sites have nudged bridgy fed, so it has registered users and keys for them, but barely any actual usage. so i may globally drop www from domains in AS2 ids and AP usernames and hosts.

I would love to drop the 'www' but there's a trouble with Webmentions: I don't want my blog to refire all the Webmentions without the 'www' and create a bunch of duplicates.
How does bridgy handle blogs at other subdomains - such as 'blog.kickscondor.com'?

@kickscondor bridgy fed hasn't (successfully) federated any posts from your site, so afaik there wouldn't be any duplicates. (technically it's federation, not POSSE, so there aren't any "copies" of posts anywhere anyway, but that's a minor point.)

How does bridgy handle blogs at other subdomains - such as 'blog.kickscondor.com'?

you mean bridgy fed? it doesn't do anything special for any subdomains, it treats all domains/hostnames equally, regardless of level. www is the only one i'm considering special casing right now. open to thoughts.

I'm sorry - by 'copies' I was referring to all of the other sites (not brid.gy) that I interact with. I don't want to send them dupes. There are ways around this - but I'd rather just live with my mistake, if I can.

You mean bridgy fed? it doesn't do anything special for any subdomains, it treats all domains/hostnames equally, regardless of level. www is the only one i'm considering special casing right now. open to thoughts.

Yes, Bridgy Fed. I guess I'm back to my original question: if other subdomains work fine, then why wouldn't<EMAIL_ADDRESS>work right now? It has no redirects.
Thank you for your replies - I am so sorry to take your time.

np!
re why www isn't working, as mentioned earlier, bridgy fed is currently mixing www and naked domain when it talks AP:

however, note that the actor in the AS2 activity above is www, but the id is the naked domain. we also store and return different public keys via webfinger for www vs naked, below, which is probably the root cause of the 401.

that's just a bug i'll fix.

Hey, thank you for your replies, Ryan - and for all of your very useful projects.

I've been looking at this more, and i think i was actually wrong before. the idof the activity we send is indeed the naked domain even for a follow of www, but that shouldn't matter. Mastodon should still look up the actor's key from the instance hostname, which afaict it does.
so, it still seems like following the www version should work. Mastodon is returning 401, which implies that it's unhappy with our signature for some reason. so maybe it is getting the wrong key somehow, or maybe it's something else altogether.

Cool - I appreciate you digging around. It's times like this that I wish ActivityPub and Mastodon weren't so complex. Thanks for all you're doing!

Fixed in #267 !


Fails to load in Safari

Storybook Version:
    "@kadira/storybook": "^2.35.3",
    "@kadira/storybook-addon-knobs": "^1.7.1",

Safari version: 10.0.3 (12602.4.8)

Safari console:

Do you have storybook-host installed? Same thing happen to me today. I just got rid of storybook-host.

Oh heck! storybook-host - I'll take a look at that.  thanks @rafales

@philcockfield Haha, didn't notice it's your package :D

Is this something we need to take a look at?

@ndelangen I don't know. I wasn't able to debug it as there was some problem with source maps for host.tsx and safari refused to display anything useful.

Sounds like maybe an issue with storybook-host rather than storybook? Should we close this?

I think maybe I'll just close it, @philcockfield (hi!) let us know if we should reopen.


Deploy wasm to github pages on push to main

Fixes: https://github.com/StaffEngineer/velo/issues/32

Not ideal, takes 15 minute, but works... we can improve it later by caching something.


Wrong return value.

/Boat/BoAT-X-Framework/vendor/platform/XinYi-XY1100/src/port_mbedtls/boatplatform_internal.c:120]: (error) Uninitialized variable: ret
The variable was returned unassigned and the result could not be determined.

Already fixed


Suggest moving 2.4.3 (variability on select. devs) and 2.4.4 (age composition likelihoods) out of data section

They're not really data, rather technical model details.
Obvious place would be as subsections in 3.3 Description of Base Model.
Not doing yet since making edits on the text, just noting it while I think of it. No big deal if they should stay.

Fine with me, but again dependent on time available. It would be easy to just note this for next year.

Yes, I'll just leave them. They are under a subsection called 'Estimated parameters and prior probability distributions' (which would probably what should be moved out of Data section, but not now as it involves re-reading text).


lib/chef/provider/package/zypper.rb regex not compatible with zypper version 1.13.*

Description
The regex used in lib/chef/provider/package/zypper.rb are not compatible with zypper version 1.13.* introduced in e.g. Suse Linux Enterprise Server 12 SP2.
Chef Version
12.12.15
Platform Version
Suse Linux Enterprise Server 12 SP2
Replication Case
Have zypper version 1.13.* installed and try to run any resouce installing a package.
Client Output
[..]
Recipe: ntp::ntp
  * zypper_package[ntp] action install
    * No candidate version available for ntp
  ================================================================================
    Error executing action `install` on resource 'zypper_package[ntp]'
    ================================================================================

    Chef::Exceptions::Package
    -------------------------
    No candidate version available for ntp

    Cookbook Trace:
    ---------------
    /var/chef/cache/cookbooks/compat_resource/files/lib/chef_compat/monkeypatches/chef/runner.rb:41:in `run_action'

    Resource Declaration:
    ---------------------
    # In /var/chef/cache/cookbooks/ntp/recipes/ntp.rb

     20:     package p do
     21:       action :install
     22:     end
     23:   end

    Compiled Resource:
    ------------------
    # Declared in /var/chef/cache/cookbooks/ntp/recipes/ntp.rb:20:in `block in from_file'

    zypper_package("ntp") do
      package_name "ntp"
      action [:install]
      retries 0
      retry_delay 2
      default_guard_interpreter :default
      declared_type :package
      cookbook_name "ntp"
      recipe_name "ntp"
    end

    Platform:
    ---------
    x86_64-linux

Running handlers:
Running handlers complete
Chef Client failed. 2 resources updated in 06 seconds

This is fixed in #5109


[NetInfo.js] - Fix response variable from getCurrentReachability

Fixes #3333 and #2802

@facebook-github-bot shipit


fix some inspection errors & warnings

(Please check. Some of these edits are just guesses. pytest is happy.)

Fix some PyCharm inspection issues, including some PEP8 style points but I turned off several inspection types. Those settings file aren't checked in since .idea/ is .gitignored but I could still add the inspection profile if you want.
Drop some Python 2 compatibility, which means for example we can use os.makedirs(p, exist_ok=True).
Make toy_control put output into out/ so we needn't .gitignore it or explain it.
class Datum could use unit tests and more docs to explain the weird things it does and how to use it. I didn't add super().__init__() yet for lack of tests, but a quick experiment went OK.
# noinspection PyMissingConstructor hides a warning about not calling super().__init__().
(There are lots of {} mutable default args here but no straightforward workaround in Python.)
make_network.py has tab indentation (also compartments.rst). This PR could change it to spaces. It fixes the mixed indentation cases.

Inspections not fixed, ignored, or marked TODO:

Duplicate code fragments.
Packages not listed in project requirements. I could change requirements.txt for that.
Some PEP 8 style deets.
A type warning in experiment.py test_multi().
Unused imports in setup.py -- some of them might still matter.
Unused locals.

I only felt uneasy about setting self.state = [] in vivarium/core/experiment.py, line 1339.

Where's the code to set self.state?

I would be okay with having inspection code via .idea/ -- but would that find anything beyond the pylint approach that @U8NWXD started in #22?

pylint, PyCharm, mypy, and other static analysis tools check for different, but overlapping, issues. E.g. pylint doesn't say it includes a type checker. So it's helpful to run more than one tool. (It is possible to run PyCharm's inspections from the command line, if it's installed, but I used mypy to check types in wcEcoli CI because it's free and easy to install.) In any case the only effect of an inspection config file in .idea/ would be to hide warnings in the PyCharm editor and from PyCharm "Inspect Code..." output, e.g. don't warn about:

non-PEP8 names (one argument uses camelCase, some others have capital letters, maybe to denote matrices),
other PEP8 style issues like too few and too many blank lines between top level definitions, or comments that start with ##, or single-quoted docstrings including '''...''',
code that's not compatible with Python before 3.6.

And instead of # noinspection PyMissingConstructor, should we just add the constructor?

Yes, super().__init__(). I didn't add it in some cases because I wasn't sure it was just an oversight. Experimenting shows that it's fine in a case like class Datum(dict).
How does Datum get used? This code is a bit weird, subclassing dict, not callint super().__init__(), and setting self.__dict__ = self. Does it set .__dict__ to enable writing, say, datum.x in place of datum.['x']? If that's the case, it'd be safer to instead define
def __getattr__(self, item):
    return self[item]

This gets called after regular attribute lookup fails, so it shouldn't interfere with other mechanisms like properties. It probably saves memory by not adding an instance dict to each of these dicts. But it might be a bit slower by running a line of Python code in that case rather than doing the attribute lookup entirely in C code.

@1fish2 -- I just went through this again, and think it's ready to merge. There are a couple TODOs left in there that look pretty simple -- come back to them later, or fix them before the merge?

Great! We can leave the TODOs for later. I just remembered about adding the .idea inspection profile. I'll do that and double check.

run_plots() in control.py refers to undefined plot_id. I'll change it to run_plots.
make_network.py uses TABs. A PyCharm command changes it to spaces.
In Python 3, class Foo: is the same as class Foo(object):. I think a style inspection warned about the latter, but it's fine that way.
In Process.__init__(), the previous code replaced the base class self.config with an empty {}. Is it OK now to keep the inherited one, including Generator.defaults? The latest change certainly makes it simpler and leaves the class less tightly coupled to its base class.
In Experiment.update(), for state_id in self.state: requires self.state to be Iterable but the type checker points out it's a Store, which does not look Iterable. Should it have an __iter__() method? Or should this code fetch a part of it?

self.state[state_id] also won't work if self.state is a Store.

Some type check warnings that we can leave for later:

In process_in_experiment(), a list doesn't have a get() method in  timeline.get('paths', {}). Should timeline be a dict, in which case timeline = settings.get('timeline', []) should default to {}?
In test_multi(), the type checker doesn't like Experiment(dict(proton, invoke=multi.invoke)).
Unexpected type(s): (Dict[str, Union[Dict[str, Union[Proton, Dict[str, Dict[str, Electron]]]], Dict[str, Union[Dict[str, Union[Tuple[str, str], ...]], Dict[str, ...]]], Dict[str, Union[..., ...]]]], (process: Any, interval: Any, states: Any) -> Any) Possible types: (Mapping[str, Union[Dict[str, Union[Proton, Dict[str, Dict[str, Electron]]]], Dict[str, Union[Dict[str, Union[Tuple[str, str], ...]], Dict[str, ...]]], Dict[str, Union[..., ...]]]], Union[Dict[str, Union[Proton, Dict[str, Dict[str, Electron]]]], Dict[str, Union[Dict[str, Union[Tuple[str, str], Dict[str, ...]]], Dict[str, Dict[str, ...]]]], Dict[str, Union[Dict[str, float], Dict[str, ...]]]]) (Iterable[Tuple[Any, Any]], _VT).

I think this is good to go. Do you want to recheck, @eagmon ? Of course we can always add more unit tests to ensure I didn't break anything.
It'd be good to distinguish private from public library definitions via _ names or all lists so we know which definitions can be changed or removed without impacting clients.


Facilitate memory usage benchmarking

We'd like to simplify and standardize memory usage benchmarks across implementations of the underlying package. The PR adds some basic memory information before the routesum structure is created, after it's created, and then after its contents are written.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

8 of 45 (17.78%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-7.3%) to 78.049%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

cmd/routesum/main.go
6
43
13.95%

Totals

Change from base Build<PHONE_NUMBER>:
-7.3%

Covered Lines:
320

Relevant Lines:
410

💛  - Coveralls


types/known/structpb: add examples

How to convert between protobuf structure and golang structure?
I know it is possible to use json or implement a reflection converter by myself, but I want to know if there is an official, simple implementation?

Fixed with https://go-review.googlesource.com/c/protobuf/+/239167


Name too close to a particular game

https://en.wikipedia.org/wiki/Big_Rigs:_Over_the_Road_Racing
Yeah, I'm bringing it up because that particular game was notoriously infamous for particularly bad programming reasons... When I first heard "big-rig" I thought it was the game, not a Google project.

The issue with the game was that, according to every review, it was diabolically bad, right? This feels on-brand for something I've built. #tongueincheek

I think for a niche of games players this might conjure up an association, but I know that game well and I didn't think about it when this name was chosen.  I don't think it is an issue really.


Suggestion setting.

 Tho i don't rally keep/use the ruins as a setup. So my question is could ther be a setting to make the button have a deconstruction planner included. Disabled by default or what u self prefere.
Hopefuly explaind better here: Insted for using both claim and decon. Go in setting enable my suggestion ^. Then all u need to use are claim & deconstruct in one button.

Note for self: Maybe do this with reverse selection instead of a mod setting. Possibly problematic because it may be hard to discover.


issue fixed search-results

🎉 Snyk hasn't found any issues so far.
✅ security/snyk check is completed. No issues were found. (View Details)
✅ license/snyk check is completed. No issues were found. (View Details)


Add PastebinScraper by default

Add PastebinScraper by default if there are no scrapers in the handler

To fix #92 .
Imported PastebinScraper.
Initialized and added the scraper if there are no scrapers in the scraping handler.

One of the imports in pastebinscraper.py are failing. I am not able to find the cause of it.
Also, I apologize for making so many commits, I can try to squash them after this is approved.

Hi there, thanks for the effort :)
Don't worry about the number of commits. I don't really care about that number.

ImportError: cannot import name 'Paste' from 'pastepwn.core' (/home/travis/build/d-Rickyy-b/pastepwn/pastepwn/core/__init__.py)
Those errors happen because when importing the core module it imports PastePwn. Then it loads all the imports at the top of the pastepwn.py file, which includes something from the core module which then imports the pastepwn.py file again.
An easy solution (other people would call it a hack) for that would be to import only in case the if statement is true.

Pull Request Test Coverage Report for Build 138

1 of 4 (25.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+3.9%) to 62.842%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pastepwn/core/pastepwn.py
1
4
25.0%

Totals

Change from base Build 96:
3.9%

Covered Lines:
920

Relevant Lines:
1464

💛  - Coveralls

I fixed the circular import by doing import pastpwn directly. Is this a good practice? If it is, you could change all imports to do this as it would prevent circular imports.


Older updater taking up disk space unexpectedly

Microsoft PowerToys version
0.71.1
Installation method
Microsoft Store
Running as admin
Yes
Area(s) with issue?
Installer
Steps to reproduce
i dont konw how to reproduce it. It seems may happen when i ignore updating the software for several versions, so that the new updaters and older updaters(relative to the latest, but still higher than the running version) are in the same folder and occupies more than 3 GB in size.
✔ Expected Behavior
It could  be automatically updated via MS Store(where I get it) or remove the older updater when downloading the lastest one, so that I can save my disk space.
❌ Actual Behavior
Found a lot of updates (from v0.72.0 to v0.80.1, while runniing PwerToys on 0.71.1) when I was decluttering my disk.

It seem that if I do not update it, all of the updater of following version are save to my disk, and would not be removed when a lastest one is publish. And occupies more than 3 GB in size.
Other Software
No response

This has already been fixed, please update to latest version.
We are on 0.81.1: https://github.com/microsoft/PowerToys/releases/tag/v0.81.1


removing "/" safeguards from OSCAddressPattern

Hi Devin,
New to Swift, but loving SwiftOSC.
I've got an OSC server inside an app that is already written to parse specific OSC Address paths that don't include the opening "/" character.
I can send messages using other OSC frameworks (oscP5, ofxOSC, MaxMSP etc) without the opening slash - and the server successfully parses these messages. Rebuilding this server from source and altering the parser isn't feasible.
I'd like to be able to use SwiftOSC, but can't seem to successfully send any messages after changing the OSCAddressPattern to remove the opening slash "/"
Having a dig through OSCAddressPattern.swift I think I can see that it's probably returning false when the opening slash is removed. Can this be overridden safely?
Commenting out this feels hacky.. but works.
              if address.first != "/" {             return false         }

Actually, commenting out these lines only worked in the OSCTest example.
In the project I'm using where SwiftOSC was installed via CocoaPods, commenting out these lines doesn't work.
Is there an easy fix?

Sorry wrong button.

It will be safer if you change to the following.
if address.first != "/" {
    self.string = "/" + address
}

Haven't tested but it should work. The will not work without the leading "/" so this way you make it optional to add and add it if it doesn't exist.

Hi Devin,
Thanks for the speedy response.
Rebuilding the framework didn't work for the Cocoapods import and threw an error re: editing privileges.
I've tested OSC frameworks (oscP5 (processing), ofxOSC (openFrameworks), MaxMSP) and none of these are prepending their OSC messages with a leading slash.
Would a rebuild for cocoapods making your leading slash be optional be possible?
Would help people make SwiftOSC clients that can actually communicate with servers using the standard in the above frameworks and where rebuilding OSC server sources isn't possible

On 23 Nov 2018, at 01:24, devinroth<EMAIL_ADDRESS>wrote:
Should still work but you need to rebuild the framework.
The leading “/“ is apart required by the OSC spec. I highly suspect the other programs are added it in if not done by the user. It makes me wonder what would happen if you start with “//“ since one and two mean different things.
That being said couldn’t of just append a “/“ at the beginning?

On Nov 21, 2018, at 10:40 PM, Jay Curtis<EMAIL_ADDRESS>wrote:
Actually, commenting out these lines only worked in the OSCTest example.
In the project I'm using where SwiftOSC was installed via CocoaPods, commenting out these lines doesn't work.
Is there an easy fix?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

The lines you provided above force the slash - which doesn't fix the issue when server parsers aren't written to expect it

On 23 Nov 2018, at 04:09, devinroth<EMAIL_ADDRESS>wrote:
It will be safer if you change to the following.
if address.first != "/" {
self.string = "/" + address
}
Haven't tested but it should work. The will not work without the leading "/" so this way you make it optional to add and add it if it doesn't exist.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

For context:
How oscP5 servers handle addresspatterns without opening slashes from clients:

How SwiftOSC servers handle addresspatterns without opening slashes from clients:

I’m following exactly what in the OSC spec.
"An OSC Address Pattern is an OSC-string beginning with the character '/' (forward slash).”
http://opensoundcontrol.org/spec-1_0 http://opensoundcontrol.org/spec-1_0
The implementation in oscP5 (processing), ofxOSC (openFrameworks), MaxMSP is invalid according to the spec which is why they show up as invalid. I wouldn’t be surprised if all of those programs use the same open source framework that allows the invalid addresses.
Feel free to change the code to fit your needs but I will have to do some research and give it more consideration before I make it a permanent change. Playing well with others is important despite the spec. Maybe the solution is to accept the values but give a warning. To properly implement it will take me time.
For a quick fix for you just download the source and compile the framework yourself.

On Nov 22, 2018, at 6:03 PM, Jay Curtis<EMAIL_ADDRESS>wrote:
For context:
How oscP5 servers handle addresspatterns without opening slashes from clients:
https://user-images.githubusercontent.com/25715542/48926201-049a1980-ef20-11e8-90ba-96631ca47e06.png
How SwiftOSC servers handle addresspatterns without opening slashes from clients:
https://user-images.githubusercontent.com/25715542/48926231-2c897d00-ef20-11e8-8860-009e99e442b1.png
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/devinroth/SwiftOSC/issues/29#issuecomment-441143331, or mute the thread https://github.com/notifications/unsubscribe-auth/AMaIESfJM1K6utKwHk3ManXPaavzkg9eks5ux1d8gaJpZM4YuuTI.

Thanks Devin, I think you are probably right here - they all could be using the same base framework which is ignoring that spec
Requesting you pivot your code around this is probably a bit too preemptive - sorry for that!  Mind if I fork it instead?
Btw really enjoyed the generative swift midi stuff!
Cheers :)
J

On 23 Nov 2018, at 18:02, devinroth<EMAIL_ADDRESS>wrote:
I’m following exactly what in the OSC spec.
"An OSC Address Pattern is an OSC-string beginning with the character '/' (forward slash).”
http://opensoundcontrol.org/spec-1_0 http://opensoundcontrol.org/spec-1_0
The implementation in oscP5 (processing), ofxOSC (openFrameworks), MaxMSP is invalid according to the spec which is why they show up as invalid. I wouldn’t be surprised if all of those programs use the same open source framework that allows the invalid addresses.
Feel free to change the code to fit your needs but I will have to do some research and give it more consideration before I make it a permanent change. Playing well with others is important despite the spec. Maybe the solution is to accept the values but give a warning. To properly implement it will take me time.
For a quick fix for you just download the source and compile the framework yourself.

On Nov 22, 2018, at 6:03 PM, Jay Curtis<EMAIL_ADDRESS>wrote:
For context:
How oscP5 servers handle addresspatterns without opening slashes from clients:
https://user-images.githubusercontent.com/25715542/48926201-049a1980-ef20-11e8-90ba-96631ca47e06.png
How SwiftOSC servers handle addresspatterns without opening slashes from clients:
https://user-images.githubusercontent.com/25715542/48926231-2c897d00-ef20-11e8-8860-009e99e442b1.png
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/devinroth/SwiftOSC/issues/29#issuecomment-441143331, or mute the thread https://github.com/notifications/unsubscribe-auth/AMaIESfJM1K6utKwHk3ManXPaavzkg9eks5ux1d8gaJpZM4YuuTI.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Please fork. And like I said I will consider making a change.
Thanks for bringing this to my attention.

On Nov 22, 2018, at 11:36 PM, Jay Curtis<EMAIL_ADDRESS>wrote:
Thanks Devin, I think you are probably right here - they all could be using the same base framework which is ignoring that spec
Requesting you pivot your code around this is probably a bit too preemptive - sorry for that! Mind if I fork it instead?
Btw really enjoyed the generative swift midi stuff!
Cheers :)
J

On 23 Nov 2018, at 18:02, devinroth<EMAIL_ADDRESS>wrote:
I’m following exactly what in the OSC spec.
"An OSC Address Pattern is an OSC-string beginning with the character '/' (forward slash).”
http://opensoundcontrol.org/spec-1_0 http://opensoundcontrol.org/spec-1_0
The implementation in oscP5 (processing), ofxOSC (openFrameworks), MaxMSP is invalid according to the spec which is why they show up as invalid. I wouldn’t be surprised if all of those programs use the same open source framework that allows the invalid addresses.
Feel free to change the code to fit your needs but I will have to do some research and give it more consideration before I make it a permanent change. Playing well with others is important despite the spec. Maybe the solution is to accept the values but give a warning. To properly implement it will take me time.
For a quick fix for you just download the source and compile the framework yourself.

On Nov 22, 2018, at 6:03 PM, Jay Curtis<EMAIL_ADDRESS>wrote:
For context:
How oscP5 servers handle addresspatterns without opening slashes from clients:
https://user-images.githubusercontent.com/25715542/48926201-049a1980-ef20-11e8-90ba-96631ca47e06.png
How SwiftOSC servers handle addresspatterns without opening slashes from clients:
https://user-images.githubusercontent.com/25715542/48926231-2c897d00-ef20-11e8-8860-009e99e442b1.png
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/devinroth/SwiftOSC/issues/29#issuecomment-441143331, or mute the thread https://github.com/notifications/unsubscribe-auth/AMaIESfJM1K6utKwHk3ManXPaavzkg9eks5ux1d8gaJpZM4YuuTI.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/devinroth/SwiftOSC/issues/29#issuecomment-441171172, or mute the thread https://github.com/notifications/unsubscribe-auth/AMaIEZ9z7T1rjVr0YZBUMmFPjRAWwaOMks5ux6WNgaJpZM4YuuTI.

Thanks Devin - after doing some digging around here I found this. Lots of great thoughts about this same issue. I think you are spot on about being built on the same framework (oscpack perhaps)
https://github.com/colinbdclark/osc.js/issues/72
It feels like somewhere a decision was made to support using plain strings and ints as addresspatterns for use with supercollider.
Adhering to the protocol seems like a good idea though!
Mind if I fork SwiftOSC?
Btw, really enjoyed your generative midi pieces - cool stuff :D

On 23 Nov 2018, at 18:02, devinroth<EMAIL_ADDRESS>wrote:
I’m following exactly what in the OSC spec.
"An OSC Address Pattern is an OSC-string beginning with the character '/' (forward slash).”
http://opensoundcontrol.org/spec-1_0 http://opensoundcontrol.org/spec-1_0
The implementation in oscP5 (processing), ofxOSC (openFrameworks), MaxMSP is invalid according to the spec which is why they show up as invalid. I wouldn’t be surprised if all of those programs use the same open source framework that allows the invalid addresses.
Feel free to change the code to fit your needs but I will have to do some research and give it more consideration before I make it a permanent change. Playing well with others is important despite the spec. Maybe the solution is to accept the values but give a warning. To properly implement it will take me time.
For a quick fix for you just download the source and compile the framework yourself.

On Nov 22, 2018, at 6:03 PM, Jay Curtis<EMAIL_ADDRESS>wrote:
For context:
How oscP5 servers handle addresspatterns without opening slashes from clients:
https://user-images.githubusercontent.com/25715542/48926201-049a1980-ef20-11e8-90ba-96631ca47e06.png
How SwiftOSC servers handle addresspatterns without opening slashes from clients:
https://user-images.githubusercontent.com/25715542/48926231-2c897d00-ef20-11e8-8860-009e99e442b1.png
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub https://github.com/devinroth/SwiftOSC/issues/29#issuecomment-441143331, or mute the thread https://github.com/notifications/unsubscribe-auth/AMaIESfJM1K6utKwHk3ManXPaavzkg9eks5ux1d8gaJpZM4YuuTI.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks. That was a good read. I think I agree with @colinbdclark on this. Best of luck with your app.


Update README.md

There's no we....
I'm just helping you with this project, so I think it should be I instead of We

The way you put "There's no we...". Merging...

The way you put "There's no we...". Merging...

Lol


Intellisense is slow and takes 20 seconds before loading suggestions

win7 x64
vscode installed using zip version
6gb ram
ssd hard drive
I type: Array. and 15-20 seconds after i get the list.
no extensions installed
v 1.3.3

Could you append the source code you are doing this in? Which language are you editing?

jsx react, js
I could even open a new js file and type Array.
same thing.

Does this reproduce in the latest VS Code insiders build with all extensions disabled?

Closing as more information is needed


Add features section (needs polish on images)

Background

Description
Added the features section. Need to upload proper cropped images so every section has one, but the layout for desktop is done. I imagine the mobile layout would be the same as with the old design, but let's circle back with designers.

I'm marking this one as ready for review. It will get changes to the screenshots and a dropdown instead of tabs on mobile, but feel free to look at the code, @saihaj.


Call for Submissions: Use Cases and Scenarios Built with ZTM #ZTMUseCases #Web3Innovation

Zero Trust Networking (ZTM) is a revolutionary, HTTP2 tunneling-based, decentralized network infrastructure software designed for the Web3 era. It's open-source and can operate on any existing IP network, such as LANs, the Internet, container networks, and more. Let's dive into the endless possibilities of ZTM in the Web3 landscape!

usecase: use ztm to support Kubernetes-native inner loop development
background: Kubernetes-native inner loop development with Quarkus
as a developer, how do you develop and test an individual microservice that is part of a larger system?

Requests : Develop Locally with Your Kubernetes Environment
integrated with mirrord service.

https://mirrord.dev/#installation


Right to left TabPageIndicator

Hi,
How to right to left TabPageIndicator?
Thanks.

Hi every body
i have problem this library when a phone suported RTL.
the pagaer direction work wrong and tab page indicator also wprk wrong.
pls help us

hi all , i have same problem , i need change gravity to right ,
pls help

salam az actionbarRTLizer estefade kon

mahdi jan dastet dard nakone

fadat. telegram @mahdiazadbar Zendegi Rasm Khoshayanist...MahdiAzadbar
 On Tuesday, September 8, 2015 6:01 PM, sajadmartiny<EMAIL_ADDRESS>wrote:

mahdi jan dastet dard nakone —
Reply to this email directly or view it on GitHub.


How to support 3D models for both native Android and iOS SDK's ?

Hi master,
I want to implement model 3d into map, Please help me  documentation for both platforms that showcases an example of how this is actually supposed to be done.
Port "antenna building" example from gl-js https://docs.mapbox.com/mapbox-gl-js/example/add-3d-model/
Thanks!!!

3D is not supported for MapLibre GL Native yet.
Please see #252


all: merge internal/glfw, internal/cglfw, and internal/goglfw

Operating System

[X] Windows
[X] macOS
[X] Linux
[X] FreeBSD
[X] OpenBSD
[ ] Android
[ ] iOS
[ ] Nintendo Switch
[ ] Xbox
[ ] Web Browsers

What feature would you like to be added?
Now Ebitengine has these internal packages:

internal/glfw: GLFW binding for cglfw and goglfw
internal/cglfw: GLFW in C for UNIX and Linux
internal/goglfw: GLFW in Go for Windows (this has files for macOS, but the implementation is empty)

If we want to remove Cgo from macOS (#1162), we have to rewrite files in internal/cglfw into Go, and move them into internal/goglfw. This would have to be done at once since internal/glfw would have to switch it. This would be a pretty hard work.
In order to make the migration easier for #1162 and #2284, what about merging these packages and making it possible to migrate from C to Go gradually?
Why is this needed?
This would make it easier to migrate C files to Go files (#1162 and #2284)

Duplicated with #2706


Ignore transient chex deprecationwarning.

Codecov Report

Merging #3427 (47d1c51) into main (31eedf1) will decrease coverage by 0.02%.
Report is 1 commits behind head on main.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #3427      +/-   ##
==========================================
- Coverage   83.66%   83.65%   -0.02%
==========================================
  Files          56       56
  Lines        6802     6802
==========================================
- Hits         5691     5690       -1
- Misses       1111     1112       +1

Files
Coverage Δ

flax/linen/spmd.py
90.67% <100.00%> (ø)

... and 1 file with indirect coverage changes


Update SentimentAnalysis tutorial with new datasets

Update SentimentAnalysis tutorial with new datasets
Updated the tutorial with the Wikidetox data sets.
Internal Review URL

closing and reopening to restart hung build


Exits do not work when playing for the wild

SPT Version
3.10-BE
Project Type
Server
Expected Result
Get out of the raid for a wild one
Actual Result
When playing for the wild, the exits do not work
Steps To Reproduce
Choose a location, "customs, factory"
Choose a wild one
Try to get out
Server Log
Click and attach file. ↓
BepinEx Log
Click and attach file. ↓
Client Log
Click and attach file. ↓
Player Profile
No response
Screenshots
No response

Attachments

server-2024-09-19.log (122 KiB)
LogOutput.log (11 KiB)
2024.09.19_13-55-08_<IP_ADDRESS>.32291 traces.log (68 KiB)

Originally written by KRISTOFER

The scavenger's outputs correspond to the PMCs outputs, but they do not work.

Originally written by KRISTOFER

Fixed with: https://github.com/sp-tarkov/modules/commit/6ea4ac6fed711500c43759677344a929dc2422d7

Originally written by chomp

Some exfil data in base.json for Lighthouse is missing, causing a string to be zero when loading Exfils, which results in a null check failing.
Under base.json, SpawnPointParams, check if Categories contains "Player" and "Infiltration" is null, then the data is missing.

Originally written by Lacyway


Could you please add a feature to merge fmp4 fragments into an MP4 file?

This is a great project, and if you could add a function to directly combine the fmp4 fragments into an MP4 file, that would be even better.
In the documentation, I saw that this library can obtain pre-encoded media data from sources other than the WebCodecs API. Can the "pre-encoded media data" refer to the data from the mdat box within fmp4 fragments?

Sure it can, but for that you'll need to demux that other MP4 file to know where in mdat you have to look (and to determine timestamps and so on). For that, you'll need a demuxer! I haven't used a demux library myself yet, but you can check out MP4Box.js and see if that can help. This library is only a muxer, so I won't be adding any demuxing capabilities.


Add TheDude Long-term

Microsoft Reviewers: Open in CodeFlow

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: DocBox12 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


dyups: fix unlocking behavior

Fix https://github.com/alibaba/tengine/issues/1661

After patching wget https://github.com/alibaba/tengine/pull/1664.patch, cases passed
tests/nginx-tests/tengine-tests/dyups.t .. ok
All tests successful.
Files=1, Tests=78, 10 wallclock secs ( 0.04 usr  0.00 sys +  0.19 cusr  0.27 csys =  0.50 CPU)
Result: PASS

Although the code logic has bug, it will not cause real damage to nginx.
Because ngx_shmtx_unlock() will not unlock other worker's lock, it check the pid number with mutex, if the pid number(mutex lock) is not pid of current worker, it does nothing.


How to launch after issuing npm run pack (ubuntu linux):

can't figure out how to launch it after issuing npm run pack :disappointed:

sudo ./dist/upterm-0.2.158-x86_64.AppImage


Bug Fix

In IDA 8.4 - ida_typeinf.idc_print_type() may return a hex-string that ends with "LL".
When attempting to convert it to int() it throws an exception and fails.
Added replace("LL", "") to resolve it

Would be nicer to cut it off at the first non-digit character, or use a pattern like here:
https://github.com/OALabs/hashdb-ida/blob/5dfb2b737cccd74cd9cf4a66ce0aa2e75970e1d5/hashdb.py#L1089


Implement Realm.operator==

Parked until after Alpha release

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 93.35%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
379

Relevant Lines:
406

💛  - Coveralls

I would suggest making snapshot public. There are other ways to walk around the result set that also require the collection to be stable.


Consider if Tiny yolo loss function is wrong

https://github.com/qqwweee/keras-yolo3/blob/e6598d13c703029b2686bc2eb8d5c09badf42992/yolo3/model.py#L364
The anchor_mask of tiny_yolo was wrong, right? It should be [[3,4,5],[0,1,2]]

you are right. It should be [[3,4,5],[0,1,2]]

Did the author react to this? Does he share access with anyone else so they can update code and resolve issues?

no, you can folk it then pull request. However, I haven't seen the author for a long time, I think he dropped this repo and this is a simple bug that we can easy to realize and fix it by ourselves.

Yes I understand that it can be forked. A good number have. The problem is they are all different changes made to the code base. A central repo would be great. That being said it is what it is.


[bug] time boundary manager incorrect handling non-overlap realtime/offline table segments

one should use all offline table segments, instead of -1 day if realtime table doesn't exist or there's a gap between the realtime and offline table segments in terms of time overlap-ness

We'll add an API to allow user to set arbitrary time boundary if needed. See #9332

Should we atleast by default handle the first part if realtime table doesn't exist? assuming Rong means "if realtime table has no rows"

The complexity will be similar. Essentially we need to track the min time on the realtime side, and decide whether to attach the time filter


[Feature] Add statistics related show statement

What type of PR is this：

[ ] bug
[x] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Added user interface

View analyze history: show analyze status
View the analysis status of the table, record the type and time of the last successful analysis: show stats meta/ show histogram meta
View the list of automatically collected tasks: show analyze job, the meaning is the same as the original show analyze

[FE PR Coverage check]
:disappointed: fail : 237 / 421 (56.29%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/http/rest/QueryDumpAction.java
0
1
00.00%

:large_blue_circle:
com/starrocks/service/FrontendServiceImpl.java
0
5
00.00%

:large_blue_circle:
com/starrocks/load/loadv2/SparkLoadJob.java
0
3
00.00%

:large_blue_circle:
com/starrocks/statistic/StatisticAutoCollector.java
0
2
00.00%

:large_blue_circle:
com/starrocks/sql/ast/CreateAnalyzeJobStmt.java
0
1
00.00%

:large_blue_circle:
com/starrocks/sql/parser/SqlParser.java
0
1
00.00%

:large_blue_circle:
com/starrocks/load/loadv2/BrokerLoadJob.java
0
3
00.00%

:large_blue_circle:
com/starrocks/qe/StmtExecutor.java
0
7
00.00%

:large_blue_circle:
com/starrocks/persist/EditLog.java
0
15
00.00%

:large_blue_circle:
com/starrocks/journal/JournalEntity.java
0
9
00.00%

:large_blue_circle:
com/starrocks/statistic/AnalyzeManager.java
3
49
06.12%

:large_blue_circle:
com/starrocks/qe/ShowExecutor.java
4
40
10.00%

:large_blue_circle:
com/starrocks/statistic/BasicStatsMeta.java
18
30
60.00%

:large_blue_circle:
com/starrocks/statistic/HistogramStatsMeta.java
14
20
70.00%

:large_blue_circle:
com/starrocks/statistic/AnalyzeJob.java
10
14
71.43%

:large_blue_circle:
com/starrocks/sql/ast/ShowAnalyzeJobStmt.java
36
47
76.60%

:large_blue_circle:
com/starrocks/sql/ast/ShowAnalyzeStatusStmt.java
37
44
84.09%

:large_blue_circle:
com/starrocks/statistic/AnalyzeStatus.java
27
32
84.38%

:large_blue_circle:
com/starrocks/sql/ast/ShowBasicStatsMetaStmt.java
24
28
85.71%

:large_blue_circle:
com/starrocks/sql/ast/ShowHistogramStatsMetaStmt.java
24
28
85.71%

:large_blue_circle:
com/starrocks/sql/analyzer/AnalyzeStmtAnalyzer.java
24
26
92.31%

:large_blue_circle:
com/starrocks/sql/parser/AstBuilder.java
8
8
100.00%

:large_blue_circle:
com/starrocks/sql/ast/AnalyzeHistogramDesc.java
1
1
100.00%

:large_blue_circle:
com/starrocks/common/Config.java
5
5
100.00%

:large_blue_circle:
com/starrocks/statistic/Constants.java
2
2
100.00%

run starrocks_admit_test

run starrocks_admit_test


Theory Doc

Started the documentation on the theory of FEM, and how we are implementing it in our project. I've also slightly updated mkdocs.yml to change citation styles on our website to render nicely, and to add a section on theory.

If your work is done, would you like to merge this as a first draft?

Hi guys,
I've just realized that I forgot to state it, but I should have made the appropriate changes. Could you review it and let me know if all looks good?

@int-ptr-ptr Do you think it's ready to be merged?


Toolchain API checks for targetCompatibility in all/unrelated scopes, 2nd try

Because of bad behaviour in previous issue and suddenly closing an issue without verifying or talking to persons I open it again. See #20045 for details.

Yes this fails. I mentioned this on the previous (now closed issue). It looks like this has to do that some extension properties are populated top-level. You can workaround by not setting the properties on the extension but on individua tasks only. Replace forbiddenApis { by forbiddenApisMain { and duplicate in forbiddenApisTest {

Or much simpler, dont set targetCompatibility at all and append version number to signatures.

The reason for the bug is brokenness of inheritance of properties in Gradle.
I will update the documentation to tell users NOT to try to set it on the extension as this has side effects.


Update AWS Batch service API

There have been changes to AWS Batch recently that we want to use. I have updated botocore and the Batch service.
I didn't update any other services.

Thanks! I'm just going to update all the services in #1878.


update(release): merge 2 PRs into release 0.37.x

What type of PR is this?
/kind cleanup
Any specific area of the project related to this PR?
What this PR does / why we need it:
Cherry picks #3030 and #3028 from master.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/milestone 0.37.0

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [Andreagit97,FedeDP,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/unhold


[Docs] Add docstring to documentation generation with Sphinx Napoleon

Description: Add documentation generation from docstring.
Question:
Per sphinx-apidoc's documentation, building the documentation requires all required dependencies as if developing the code. Else, complete documentation will not be generated. Is there a way to test the edits in .readthedocs.yaml to see if all required dependencies are installed and that documentations are successfully generated?

These changes were copied over in a larger docs overhaul in #516


Describe MSBuild props that control web.config transform

Fixes #3607
cc/ @vijayrkn @shirhatti

Property to set is
<IsTransformWebConfigDisabled>true</IsTransformWebConfigDisabled>

https://github.com/aspnet/websdk/blob/afbcec9b0951d09088f26e98e6aca6114e0d32ee/src/Publish/Microsoft.NET.Sdk.Publish.Targets/netstandard1.0/TransformTargets/Microsoft.NET.Sdk.Publish.TransformFiles.targets#L47
I don't think this property - TransformWebConfigEnabled is used any where.

@vijayrkn Cool, thanks. I'm glad you typed out the whole thing ... it prompted me to go ahead and show it instead of only mentioning it in-text.

I don't think this property - TransformWebConfigEnabled is used any where.

AFAICT it doesn't show up anywhere in repo issue discussions. I dropped it.


Update Redirect URL and README for login flow

Changed login URL to showcase a redirect back to app. This is not part of the original behavior since Singular detects logins from other tabs. Left comments that this redirect is purely cosmetic (ID token from this flow can be ignored).
Updated readme for new redirect path back to index.html, and also noted in readme that path to singular must change. This should be revisited when looking at how to allow configurable paths to UAA singular NPM module.

I think we should merge this after the completion of https://www.pivotaltracker.com/story/show/155861030 when the PATH/TO/SINGULAR is configurable. @tnwang


Implement support for repository visibility parameter

Implements support for specifying a repositories visibility. It can be
one of public, private or internal (if the organization is associated
with an enterprise account).
The associated preview header (nebula) has also been added.
Resolves: #1470

What is the release cadence for this project - I would like to put this to use in the Github Terraform Provider ASAP!

We used to do a semver (major, minor, patch) release on each merged PR, but that got way too extreme... so now we release on demand.  I can make a new release after this PR is merged.

That would be great - thanks @gmlewis!

Thank you, @wesleimp!
Merging, then making a minor release.

@patrickmarabeas - Version v30.1.0 has now been released at your request.


Merge Math release v4.0.1 into StanHeaders_2.26

@bgoodri @rok-cesnovar

@bgoodri I'll update the StanHeaders_2.26 branches. This doesn't require a review. I'd like to prepare for Stan 2.27, but I'll wait for you as this may require extra work for backward compatibility (such as https://github.com/stan-dev/rstanarm/pull/514) and is unnecessary for the transition.

ok
On Sun, Jun 20, 2021 at 5:47 PM Hamada S. Badr @.***>
wrote:

Merged #2380 https://github.com/stan-dev/math/pull/2380 into
StanHeaders_2.26.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/stan-dev/math/pull/2380#event-4913811342, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZ2XKV5EQM3XZMCTDZH3NLTTZOWHANCNFSM4X4ZQT5A
.


C# fix hardcoded types

Current coverage is 84.96% (diff: 100%)

Merging #958 into master will not change coverage

@@             master       #958   diff @@
==========================================
  Files           338        338
  Lines         13167      13167
  Methods           0          0
  Messages          0          0
  Branches       1721       1721
==========================================
  Hits          11187      11187
  Misses         1547       1547
  Partials        433        433

Powered by Codecov. Last update 5475bf4...ed3e331


transfers/inbound: use OriginalTrace from addenda99 in return lookup

Codecov Report

Merging #637 (9c43a7e) into master (5dd64c6) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #637      +/-   ##
==========================================
+ Coverage   52.46%   52.51%   +0.04%
==========================================
  Files          95       95
  Lines        3301     3304       +3
==========================================
+ Hits         1732     1735       +3
  Misses       1210     1210
  Partials      359      359

Impacted Files
Coverage Δ

pkg/transfers/inbound/returns.go
73.21% <100.00%> (+1.51%)
:arrow_up:

Continue to review full report at Codecov.

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


Deploy fails when case of file name is changed

Summary
Source push fails when changing the case of a file name (observed with Apex classes) and its metadata file.
Steps To Reproduce:
Repository to reproduce: case-sensitive-rename

Create an org and push the contents of the case-sensitive-rename repo
Rename: force-app/main/default/classes/{badCase.cls => BadCase.cls} and the corresponding .cls-meta file
Push again - the push should fail

Expected result
The push to succeed and the class to get renamed in the scratch org.

Caveat: I would expect/tolerate this could be problematic when deploying to production. But this should probably(?) work in scratch orgs, when you may have not committed anything or deployed to an environment without source tracking.

Actual result
TYPE   PROJECT PATH                                PROBLEM
─────  ──────────────────────────────────────────  ───────────────────────────────────────────
Error  force-app/main/default/classes/BadCase.cls  File name mismatch with class name: badCase
ERROR running force:source:push:  Push failed.

The push fails.
If you change the name of the class in the file to UpperCamelCase BadCase, the error is slightly different:
TYPE   PROJECT PATH                                         PROBLEM
─────  ───────────────────────────────────────────────────  ─────────────────────────────────────────────────────
Error  force-app/main/default/classes/BadCase.cls-meta.xml  Must specify both the source file & the metadata file
Error  force-app/main/default/classes/badCase.cls-meta.xml  Must specify the metadata file
ERROR running force:source:push:  Push failed.

But the work around is the same:
Work Around
In sourcePathInfos.json, if I change the file paths to reflect the new case, the push can succeed.
System Information
{
        "cliVersion": "sfdx-cli/7.133.0",
        "architecture": "darwin-x64",
        "nodeVersion": "node-v16.13.1",
        "pluginVersions": [
                "@oclif/plugin-autocomplete 0.3.0 (core)",
                "@oclif/plugin-commands 1.3.0 (core)",
                "@oclif/plugin-help 3.3.1 (core)",
                "@oclif/plugin-not-found 1.2.6 (core)",
                "@oclif/plugin-plugins 1.10.11 (core)",
                "@oclif/plugin-update 1.5.0 (core)",
                "@oclif/plugin-warn-if-update-available 1.7.3 (core)",
                "@oclif/plugin-which 1.0.4 (core)",
                "@salesforce/lwc-dev-server 2.11.0",
                "@salesforce/sfdx-plugin-lwc-test 0.1.7 (core)",
                "alias 1.2.0 (core)",
                "apex 0.7.0 (core)",
                "auth 1.8.0 (core)",
                "config 1.3.14 (core)",
                "custom-metadata 1.0.12 (core)",
                "data 0.6.8 (core)",
                "generator 1.2.1 (core)",
                "info 1.2.0 (core)",
                "limits 1.3.0 (core)",
                "org 1.11.0 (core)",
                "salesforce-alm 53.7.2 (core)",
                "schema 1.1.0 (core)",
                "sfdx-cli 7.133.0 (core)",
                "source 1.7.0 (core)",
                "telemetry 1.4.0 (core)",
                "templates 52.7.0 (core)",
                "trust 1.1.0 (core)",
                "user 1.7.0 (core)"
        ],
        "osVersion": "Darwin 20.5.0"
}

Additional information
I would expect the behavior of this may differ on windows (case insensitive) and mac/linux (case sensitive). I am on mac.

I'm closing this in favor of a higher-level discussion about how we want this to work.  That's here.
https://github.com/forcedotcom/cli/discussions/2682


import cell2location -> AttributeError: module 'jax.random' has no attribute 'KeyArray'

[x] I have confirmed this bug exists on the latest version of cell2location. See https://github.com/BayraktarLab/cell2location#installation
[ ] I follow the instructions from the scvi-tools tutorial.

Another instance of import cell2location producing an error. However, the error message seems different from all other open and closed issues here, hence the new issue.
This GitHub action workflow reproduces the issue I have on my HPC.
I've tried suggestions describes in this issue without success.
Thanks

Note: Please read this guide detailing how to provide the necessary information for us to reproduce your bug.
Minimal code sample (that we can run without your data, using public data)
# Your code here

[Paste the error output produced by the above code here]

I got the same issue

Will it be the problem caused by the different versions of the graphics card that cause the different versions of the installed jax.
The 0.4.2 version of jax has the KeyArray function, while the 0.4.24 version of jax does not.
However, if I change jax to version 0.4.2, I will get another error: AttributeError: module 'jax' has no attribute 'Device' and several mismatch error.

I solved it temporarily by installing jax version 0.4.23.
pip install jax==0.4.23

I solved it temporarily by installing jax version 0.4.23.
pip install jax==0.4.23

Thank you, I also used your method to solve the import problem


Recoil Crosshair issues.

I see that the option to change size, and color has just been added to recoil crosshair, much like noscope crosshair and spectator list. A couple things I would like to see resolved are; 1.) the recoil crosshair bounces up and down a couple pixels after a shot has been fired, even if your movement is reset. 2.) this one is mainly for daniel, add the ability to select which guns the recoil crosshair works for.

it got fixed in the update im a retard but still selectable guns would be cool

@danielkrupinski ^^^^^ <3


/fit-in not working with 4.0.0

Using /fit-in/AxB/ is cropping images rather than fitting them within the dimensions specified.
Example: https://i1rk6c9qi5.execute-api.us-east-1.amazonaws.com/image/100x100/bart.png
A workaround to fit an image into the resize area is to use the upscale filter:
https://i1rk6c9qi5.execute-api.us-east-1.amazonaws.com/image/200x50/filters:upscale()/bart.png
The fix for this might be in the process(event) function:
https://github.com/awslabs/serverless-image-handler/blob/a1b3eb659807599eaec88fb8df21bc34a45b93ee/source/image-handler/thumbor-mapping.js#L27-L38
... the this.edits.resize.fit = 'inside' needs to be set, the same as what happens in the upscale() filter.

Hi @tjw0051 we faced a similar issue.
fit-in from thumbor isn't translated here correctly.
That's because they aren't passing anything to resize.
While 'inside' is the right way forward ( because it preserves aspect ratio ) for us this leads to another issue. For eg: if we pass 500x500 as the size, we want image to be resized based on aspect ratio but we should be able to 'fill' the remaining area with color of our choice.
This is how we did in thumbor:
https://<our_cf_url>/fit-in/500x500/filters:quality(90):fill(fff)/my_source.jpg
If we use 'inside', we get back an image with aspect ratio maintained, but we aren't able to fill it with white color so as to get back 500x500 final product.
options.background can only be used with 'contain'.
https://sharp.pixelplumbing.com/en/stable/api-resize/
So, if this was instead using
this.edits.resize.fit = 'contain',
we could pass
sharp(input)
.resize(200, 300, {
kernel: sharp.kernel.nearest,
fit: 'contain',
position: 'right top',
background: { r: 255, g: 255, b: 255, alpha: 0.5 }
})
.toFile('output.png')
.then(() => {
// output.png is a 200 pixels wide and 300 pixels high image
// containing a nearest-neighbour scaled version
// contained within the north-east corner of a semi-transparent white canvas
});

This is what we did to make /fit-in/ work: https://github.com/awslabs/serverless-image-handler/pull/130
Also includes fixes to make fetching from S3 sub directories work.
Although this pasts the unit-test, this is what works with our use cases, use at your own risk.

We have updated our solution, and I believe your issue has been fixed. If you still see the issue with the latest version (v4.2), please feel free to reopen the issue.
You can refer to the recent changes here

@beomseoklee hi! I've updated the function on a project and noticed this change. Is there a way to update the urls to use the old fit behavior as cover or should I update the code Lamba's using? Thanks in advance!

If anyone else wants the same I did, apparently it's possible to do that changing the line 59 of thumbor-mapping.js to fit cover.


Playwright reports Error: No tests found after fresh install

Describe the bug
Running the npm create svelte@latest script and then attempting to run the playwright tests gives an error.
$ playwright test
[WebServer]
[WebServer]
[WebServer]
Error: No tests found

I tried with various combinations such as the "App" template and the "Library" template, as well as Typescript v.s. Javascript.
And they all show that same error even though there appears to be a default test at:
tests/test.ts or tests/test.js
Reproduction
Run the following:
npm create svelte@latest my-app
cd my-app
npm install
npm run test

Logs
> npm create svelte@latest my-app
cd my-app
npm install
npm run test

create-svelte version 4.1.0

┌  Welcome to SvelteKit!
│
◇  Which Svelte app template?
│  SvelteKit demo app
│
◇  Add type checking with TypeScript?
│  Yes, using TypeScript syntax
│
◇  Select additional options (use arrow keys/space bar)
│  Add ESLint for code linting, Add Prettier for code formatting, Add Playwright for browser testing, Add Vitest for unit testing
│
└  Your project is ready!

✔ Typescript
  Inside Svelte components, use <script lang="ts">

✔ ESLint
  https://github.com/sveltejs/eslint-plugin-svelte

✔ Prettier
  https://prettier.io/docs/en/options.html
  https://github.com/sveltejs/prettier-plugin-svelte#options

✔ Playwright
  https://playwright.dev

✔ Vitest
  https://vitest.dev

Install community-maintained integrations:
  https://github.com/svelte-add/svelte-add

Next steps:
  1: cd my-app
  2: npm install (or pnpm install, etc)
  3: git init && git add -A && git commit -m "Initial commit" (optional)
  4: npm run dev -- --open

To close the dev server, hit Ctrl-C

Stuck? Visit us at https://svelte.dev/chat

added 231 packages, and audited 232 packages in 23s

47 packages are looking for funding
  run `npm fund` for details

found 0 vulnerabilities

><EMAIL_ADDRESS>test
> playwright test

[WebServer]
[WebServer] "confetti" is imported from external module "@neoconfetti/svelte" but never used in "src/routes/sverdle/+page.svelte".
[WebServer]
[WebServer]
Error: No tests found

System Info
System:
    OS: macOS 13.2.1
    CPU: (8) arm64 Apple M2
    Memory: 75.13 MB / 16.00 GB
    Shell: 5.8.1 - /bin/zsh
  Binaries:
    Node: 19.5.0 - ~/.nodenv/versions/19.5.0/bin/node
    Yarn: 1.22.19 - /opt/homebrew/bin/yarn
    npm: 9.3.1 - ~/.nodenv/versions/19.5.0/bin/npm
  Browsers:
    Chrome: 112.0.5615.137
    Chrome Canary: 115.0.5737.0
    Safari: 16.3
  npmPackages:
    @sveltejs/adapter-auto: ^2.0.0 => 2.0.1
    @sveltejs/kit: ^1.5.0 => 1.15.9
    svelte: ^3.54.0 => 3.58.0
    vite: ^4.3.0 => 4.3.3

Severity
annoyance
Additional Information
No response

Looks like they changed the globs in Playwright 1.33. Gah.


Skip --initialize-backup steps for standalone databases

In other words, if no replica set name file exists, and therefore no replica set exists, don't bother re-initializing the replica set. (Attempting to re-initialize the replica set could fail due to the absence of a known, privileged user in the admin database.)

Thanks @krallin for walking me through this. I've tested to confirm this works; please let me know if you think any test additions/modifications are appropriate here, and I'll write them.

Thanks @krallin ! I'll merge this then (I'll perform a quick sanity check against both replica set and standalone backups after the images are built and pushed.)


[1600] estimate gas with precompiles

This PR adds tests for estimate gas with Precompiles

Association and dissociation of accounts with tokens

Some negative scenarios

Transfer/Transfer from

Approve/allowance

There are still more tests that have to be added afterwards

Codecov Report
Attention: 13 lines in your changes are missing coverage. Please review.

Comparison is base (6aafdd7) 78.10% compared to head (1eb4832) 77.83%.
 Consider uploading reports for the commit dc69647 to get more accurate results

Files
Patch %
Lines

packages/server/src/koaJsonRpc/index.ts
81.81%
1 Missing and 3 partials :warning:

packages/relay/src/lib/eth.ts
84.21%
0 Missing and 3 partials :warning:

packages/relay/src/lib/clients/mirrorNodeClient.ts
33.33%
0 Missing and 2 partials :warning:

packages/server/src/server.ts
0.00%
1 Missing and 1 partial :warning:

packages/relay/src/lib/clients/sdkClient.ts
50.00%
1 Missing :warning:

.../lib/services/ethService/ethCommonService/index.ts
66.66%
0 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2002      +/-   ##
==========================================
- Coverage   78.10%   77.83%   -0.27%
==========================================
  Files          39       40       +1
  Lines        3060     3086      +26
  Branches      616      625       +9
==========================================
+ Hits         2390     2402      +12
  Misses        479      479
- Partials      191      205      +14

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Implement Prefixer

Prefixer is Matter subclass for autonomic identifier prefix using derivation as determined by code from key event dictionary
Requires https://github.com/WebOfTrust/cesride/issues/8.
impl - https://github.com/WebOfTrust/keripy/blob/5a19417a3c6e79e868ae96f55e82385de0c348d3/src/keri/core/coring.py#L2878
test - https://github.com/WebOfTrust/keripy/blob/8a6f67153594ffac0dd3a2ccfbc4fd9235194d79/tests/core/test_coring.py#L4282

Working on this.


Minor fixes for compilation on Windows (UnetSocket C APIs)

Few minor fixes were needed to compile on Windows after addition of more sample programs.

I guess we need to do the changes because dynamic array allocation isn't allowed in msvc?

That is correct. Having something like float buf[siglen] where siglen is a variable initialized was causing a problem.

#TIL Variable-length arrays are not a kosher C feature.. https://stackoverflow.com/questions/1887097/why-arent-variable-length-arrays-part-of-the-c-standard


EDC - Connector KIT Documentation & Support MVD (ext: CXAR-1034)

+Benefit Hypothesis+
Right now for every catena-x release the files needs to be copied from tractusx-edc repository to tractusx.github.io repository → it's done manually. I would like to have a better way here. A lot of user are asking for a manual to build/play with a data space. Its not the responsible for only one team (edc/miw/...) → it should be joint work to build such a documentation. Maybe like the old catenax@home
+Beneficiaries+
< Who will benefit for that feature>

Components: EDC
Sprints: N/A
Fix Versions: N/A
StoryPoints: N/A
Attachments: N/A

Tasks

[ ] Improve the process of KIT Documentation.
[ ] Decide about using Autodoc
[ ] Update to the current version.
[ ] Create the tutorial (Documentation) for the MXD/MVD: https://github.com/eclipse-tractusx/eclipse-tractusx.github.io/issues/313
[ ] Run the Setup at your machine locally and give feedback.
[ ] Adding Features to the existing Setup.

related: https://github.com/eclipse-tractusx/tutorial-resources/issues/34 and https://github.com/eclipse-tractusx/tutorial-resources/issues/26

@stefan-ettl

Autodoc seems to be also ok, cause the related Issue was already closed https://github.com/eclipse-tractusx/tractusx-edc/issues/700
Update the current version* means the Connector Kit. Last updates were made in August. But maybe we can use the time in PI 11 and do the restructuring => https://github.com/eclipse-tractusx/eclipse-tractusx.github.io/discussions/448
Run the setup if we are talking about the E2E Adopter Journey tutorial  => test was done by Johannes D and myself

Autodoc seems to be also ok, cause the related Issue was already closed docs: Autodoc generation/preperation #700 => but question is still... can we use it? Is the created JSON also transferred to markdown?

The autodoc json is generated and converted to markdown then put in the Devlopment View/05_EDC extensions folder

Autodoc seems to be also ok, cause the related Issue was already closed docs: Autodoc generation/preperation #700 => but question is still... can we use it? Is the created JSON also transferred to markdown?

The autodoc json is generated and converted to markdown then put in the Devlopment View/05_EDC extensions folder

Not sure what to do with this issue :( i think documentation is helpfull somewhere... initial idea was the kits... but since the process changed (also the mentioned put in the Devlopment View/05_EDC extensions folder is not needed anymore... i am not sure if the output fits to the new structure of the connector kit. what do you think? @bcronin90
For kits there is also the possibility to hvae a documentation section. We also had this initialy...

@stefan-ettl  - I think this can be closed with the merge of https://github.com/eclipse-tractusx/eclipse-tractusx.github.io/pull/558


ProcessManager - emit shutdown after kill childrens

Hey!
I suggest to put emit shutdown under the killing of childrens process
Your example is not killing childrens via It exists before killing
        $manager->onShutdown(function() use ($manager) {
            echo 'Catch sigterm.Quiting...' . PHP_EOL;
            exit(1);
        });

Because you program will exit before, without killing childrens, It's rly bad
Thanks :cake:

Dont forget to review it, before merge
For Now I think this variant will works

@misterion Thank you :beers:


Update manage-guests.md

Issue #3476 - SKU support details are updated.

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.


[BUG] Classes that extend class 'o.o.action.support.master.TransportMasterNodeAction' should not have to implement abstract method 'clusterManagerOperation()'

Describe the bug
Similar issue: https://github.com/opensearch-project/OpenSearch/issues/3663 and https://github.com/opensearch-project/OpenSearch/issues/3688
Introduced by PR https://github.com/opensearch-project/OpenSearch/pull/3617 / commit https://github.com/opensearch-project/OpenSearch/commit/60d7a098492c184e0b2a6fb0492d8c57b7626a54, which aims to resolve issue #3542
The problem:
To deprecate public methods that has "master" in the name, I created an alternative method clusterManagerOperation(...) to replace masterOperation(...), which is an abstract method.
Location: https://github.com/opensearch-project/OpenSearch/blob/157e69330891b8133d56ab993b5022f2e8c6ee04/server/src/main/java/org/opensearch/action/support/clustermanager/TransportClusterManagerNodeAction.java
It makes classes that extend the class TransportMasterNodeAction have to implement the newly added abstract method clusterManagerOperation(...), and it breaks the backwards compatibility.
The solution is to revert the changes to remove the abstract method clusterManagerOperation(...).
To Reproduce
Build k-NN plugin with OpenSearch 2.1.0 (the code at https://github.com/opensearch-project/OpenSearch/tree/304d830275fdbe5f8095a7285ca66266987c727a)
java.lang.AbstractMethodError: Missing implementation of resolved method 'abstract void clusterManagerOperation(org.opensearch.action.support.clustermanager.ClusterManagerNodeRequest, org.opensearch.cluster.ClusterState, org.opensearch.action.ActionListener)' of abstract class org.opensearch.action.support.clustermanager.TransportClusterManagerNodeAction. »  	at org.opensearch.action.support.clustermanager.TransportClusterManagerNodeAction.clusterManagerOperation(TransportClusterManagerNodeAction.java:135) »  	at org.opensearch.action.support.clustermanager.TransportClusterManagerNodeAction$AsyncSingleAction.lambda$doStart$3(TransportClusterManagerNodeAction.java:223)
Expected behavior
k-NN plugin should successfully be built with OpenSearch 2.1.0
Additional context
Add any other context about the problem here.

The temporary solution has been applied to 2.1 branch by the PR #3682

Close the issue. The masterOperation() method name have been restored in all branches.
PR https://github.com/opensearch-project/OpenSearch/pull/3681 for main branch, and PR https://github.com/opensearch-project/OpenSearch/pull/3714 for 2.x branch.

I have got a solution from @andrross to properly deprecate the abstract method 'masterOperation()'.
@Deprecated("override doSomethingWithClusterManager instead")
void doSomethingWithMaster() {
    throw new UnsupportedOperationException("Must be overridden");
}

void doSomethingWithClusterManager() {
    doSomethingWithMaster();
}

The below is wrong because it will break any child classes.
@Deprecated("override doSomethingWithClusterManager instead")
abstract void doSomethingWithMaster();

abstract void doSomethingWithClusterManager();

The final solution in PR https://github.com/opensearch-project/OpenSearch/pull/4032 has been merged, and from version 2.2.0, developers can use clusterManagerOperation() to replace masterOperation().
For more detail: https://github.com/opensearch-project/OpenSearch/issues/3542#issuecomment-1200078089

As mentioned above, the 4 classes AcknowledgedRequest, AcknowledgedRequestBuilder, AcknowledgedResponse, ShardsAcknowledgedResponse in package org.opensearch.action.support.master will not be deprecated in version 2.x, and will be moved to package org.opensearch.action.support.clustermanager directly in a future major version as a breaking change.


defaceMode zero implementation

Changes:

Accounting for --defaceMode zero flag in the defacing process.
Updating XML for --inputLandmarks flag
removing commented code (that isn't informative to how the defacing works)

Unnecessary PR. I misunderstood the not_face_region variable.
Will submit a new PR to clean up this code, but not alter the defacing method.


Price list CRUD operations

User Story
As user, I want to create a price list so I can see what my OCP environment costs.
Impacts
UI, Docs
Role
User
Assumptions

Koku API has support for Rate CRUD operations

UI Details
See mocks:
https://redhat.invisionapp.com/share/NJS0RS3Q7KE#/screens/364417349

Acceptance Criteria

[ ] A user should be able to create a new Rate, modify an existing Rate, and delete Rates.
[ ] A Rate should define a Cost that applies to a Metric and a Time. (e.g. $0.25 per CPU-hour, or $0.01 per GiB Network-IO per minute.)
[ ] A user should be able to associate a Rate with an Account or OCP Cluster

Closing in favor of #881 and #882


variants/msi_z690/building.md: Add building manual doc

Signed-off-by: Igor Bagnucki<EMAIL_ADDRESS>

Already included in this branch: https://github.com/Dasharo/docs/pull/145/commits


[material-ui][ButtonBase] Fix displaying hidden button

closes: https://github.com/mui/material-ui/issues/43374
before: https://codesandbox.io/p/sandbox/focused-kare-rfw8mc?file=%2Fsrc%2FDemo.tsx
after: https://codesandbox.io/p/sandbox/gracious-stallman-fnp7ch

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

@sai6855 Check the discussion in #20452, especially this comment explaining why it's not supported and why we should ignore it. We’ll need to update other components for consistency. Also, MDN explains when it doesn’t work natively: https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/hidden#the_hidden_state. So, I don't think we should add this.

Thanks for pointing towards discussion, I couldn't find. Closing this PR as we aren't sure if hidden will be supported across all components


Refactor Script query to use IScript property

The current ScriptQuery implementation includes separate properties at the root level for properties of a script:
https://github.com/elastic/elasticsearch-net/blob/9f3e657aaf834a7bf6c82641e3a4dcc19181ea11/src/Nest/QueryDsl/Specialized/Script/ScriptQuery.cs#L8-L28
This results in the need to have a custom JsonConverter/serialization component in order to correctly nest these properties in JSON:
{
  "script": {
    "_name": "named_query",
    "boost": 1.1,
    "script": {
      "source": "doc['numberOfCommits'].value > param1",
      "params": {
        "param1": 50
      }
    }
  }
}

These properties should be removed and replaced with an IScript Script {get;set;} property

This is now merged into 7.x in 153bbf33d8bbfa87af16c9cabe1c943a92bc7501


Updated feisty-dog-tag to use HTTPS

Thank you for the contribution. It was merged directly as e448bd2c80420a38c20e05fdeaa7eee92bf7159a to keep commit history cleaner. Your contribution is still credited to you.


Add mosaic flag on the mosics list

The flag column will show icons if the flag is true.

@AnthonyLaw do we need translations for that?

I think we can do it later.