MSTest.TestAdapter and MSTest.TestFramework package should be transitive (buildtransitive instead of build)

Summary
I would like to have the .props and .targets of MSTest.TestAdapter and MSTest.TestFramework transitive - similar as it was suggested and implemented for Microsoft.Net.Test.Sdk.
Background and Motivation
I have the same test hierarchy scenario as described in the above issue and would like to distribute requirements via NuGets.
I am using these 3 libraries (Microsoft.Net.Test.Sdk, MSTest.TestAdapter, MSTest.TestFramework) together and hence depend on the transitivity of all three libraries to avoid repetitive package references.
Proposed Feature
The following .nuspecs should change like depicted below (for all build folders).
https://github.com/microsoft/testfx/blob/main/src/Package/MSTest.TestFramework.nuspec
...
<!-- net6.0 -->
<file src="Build\Net6\MSTest.TestFramework.targets" target="buildtransitive\net6.0\" />
<file src="TestFramework.Extensions\net6.0\Microsoft.VisualStudio.TestPlatform.TestFramework.dll" target="lib\net6.0\" />
<file src="TestFramework.Extensions\net6.0\Microsoft.VisualStudio.TestPlatform.TestFramework.xml" target="lib\net6.0\" />
<file src="TestFramework.Extensions\net6.0\Microsoft.VisualStudio.TestPlatform.TestFramework.Extensions.dll" target="buildtransitive\net6.0\" />
<file src="TestFramework.Extensions\net6.0\Microsoft.VisualStudio.TestPlatform.TestFramework.Extensions.xml" target="buildtransitive\net6.0\" />
<file src="TestFramework.Extensions\net6.0-windows10.0.18362.0\Microsoft.VisualStudio.TestPlatform.TestFramework.Extensions.dll" target="buildtransitive\net6.0\winui\" />
<file src="TestFramework.Extensions\net6.0-windows10.0.18362.0\Microsoft.VisualStudio.TestPlatform.TestFramework.Extensions.xml" target="buildtransitive\net6.0\winui\" />
...

https://github.com/microsoft/testfx/blob/main/src/Package/MSTest.TestAdapter.nuspec
...
<!-- net6.0 -->
<file src="Build\Net6\MSTest.TestAdapter.props" target="buildtransitive\net6.0\MSTest.TestAdapter.props" />
<file src="Build\Net6\MSTest.TestAdapter.targets" target="buildtransitive\net6.0\MSTest.TestAdapter.targets" />
<file src="TestFramework\net6.0\Microsoft.VisualStudio.TestPlatform.TestFramework.dll" target="buildtransitive\net6.0\" />
<file src="MSTestAdapter.PlatformServices\net6.0\Microsoft.VisualStudio.TestPlatform.MSTestAdapter.PlatformServices.dll" target="buildtransitive\net6.0\" />
<file src="MSTestAdapter.PlatformServices\net6.0-windows10.0.18362.0\Microsoft.VisualStudio.TestPlatform.MSTestAdapter.PlatformServices.dll" target="buildtransitive\net6.0\winui\" />
<file src="Microsoft.TestPlatform.AdapterUtilities\netstandard2.0\" target="buildtransitive\net6.0\" />
<file src="MSTest.TestAdapter\net6.0\Microsoft.VisualStudio.TestPlatform.MSTest.TestAdapter.dll" target="buildtransitive\net6.0\" />
...

Alternative Designs

Hi @Ildrial, thanks for the suggestion. We will review it and make sure it doesn't have any unwanted side effect but that does sound like a good idea.

Hi @Ildrial, I will move forward by marking this issue as won't fix. We did a rollback on vstest side as there were unexpected consequences.
Besides, it seems that NUnit and xUnit are also both avoid the build transitive configuration.
Please do keep suggesting improvement here!


Feat batching

This adds the macro batching feature.

Changes in this PR cover the following issues:

#9
#11
#12


Enable aarch64_linux test parallel

Related  https://github.com/adoptium/infrastructure/issues/2893#issuecomment-1476271354
Signed-off-by: Sophia Guo<EMAIL_ADDRESS>

@sophia-guo would you like to merge and cherry pick this into the release branch, or would you like me to take care of it?


Warn if a PR changes Cargo.lock

In rust-lang/rust we will not automatically merge diff from Cargo.lock (see rust-lang/rust#46539 for rationale). This means all PRs modifying Cargo.lock are mutually exclusive with each other, and when one such PR is merged, all other PRs will need to be rebased.
This may bother reviewers and authors when the PR needs to be rebased and r+'ed several times. It is also pretty confusing to new contributors because a rebase often completes without any conflicts locally.
Thus, I think it is the best for rust-highfive to foreshadow such potential extra work to both reviewers and authors.

⚠ Warning ⚠
This PR modified Cargo.lock. We will not automatically merge updates to Cargo.lock, since it often leaves the file in a broken state. One unfortunate result is that this often leads to merge conflicts.
You may need to rebase this PR several times after it has been approved. We are sorry for any inconvenience which may arise.
(Note to reviewers: Please also prepare to r+ several times. Consider giving it a p=1 to minimize queuing time and reduce chance of conflict.)

Does this still apply today when Cargo.lock can be automatically merged?

Right.
Fixed by #63743.


Flux delete sync event

Flux supports the deletion of objects, its just like a regular sync event but it has no serviceID's, the existing code was strictly checking for serviceID's and slack messages were dependent on serviceID's as well. This commit is to add no serviceID support (aka, flux delete sync event)

@latchmihay we've updated the go module for flux to the 1.19.0.
Could you please rebase and update any legacy reference?

You can get the docker image with:
docker pull quay.io/tfgco/fluxcloud

@latchmihay Thanks for this PR, we've been watching and just took the opportunity to test 0.4.0 on an environment with garbage collection enabled, however fluxcloud didn't receive an event. Do you have it running successfully somewhere?

Actually, an event with empty serviceID was received, but it didn't send a Slack message.
{"id":0,"serviceIDs":[],"type":"sync","startedAt":"2020-07-06T16:39:42.068985084Z","endedAt":"2020-07-06T16:39:42.068985084Z","logLevel":"info","metadata":{"commits":[{"revision":"32202e7dc85f51eae2261806059bb031162b8ada","message":"remove thing flux patch (#568)"},{"revision":"9fbe60286e2cbee2a0e26ff2f68c5331e7617a7e","message":"disable thing (#567)"}],"includes":{"other":true}}}

We're running fluxcd/flux-prerelease:master-c26e9fe8

@latchmihay Thanks for this PR, we've been watching and just took the opportunity to test 0.4.0 on an environment with garbage collection enabled, however fluxcloud didn't receive an event for sync delete. Do you have it running successfully somewhere?

No I don't, I am not sure if something has changed in Flux 1.19 in terms of the delete payload. Take a look at the tests provided and the payload on a delete event I am basing this on

@latchmihay Ignore my comments above, your changes is correct and working as expected. The change I'm waiting for is https://github.com/fluxcd/flux/pull/3127 which will then just require a small update to the default formatter to include a list of resources. Thanks again for this contribution!

Hey @dan-slinky-ckpd! We haven't checked this suggested change for the formatter yet, but I think that will fix this problem, right?

@fdpeiter I'm not sure you need to make that change as the spurious list of resources issue has been resolved by https://github.com/fluxcd/flux/pull/3039. The change I'm referring to above is once https://github.com/fluxcd/flux/pull/3127 has been merged and a new flux release is published we can update the formatter to include event.DeletedWorkloadIDs, for example.
The result of that example looks like this:

If a user does want to adapt the message to exclude the list of resources completely they should be able to override the template with BODY_TEMPLATE, but I've not tried that myself.

Got it, gonna keep an eye on it to add this whenever merged.


feat!: add package manager support

BREAKING_CHANGE: The following CLI options have been removed:

--npm: replaced by --pm functionality; provide a version, not a path
--install-args: may be re-added later, but package managers accept different args, so this became a problem
--dir, --clean, and --force: package managers each have their own temp dir and we cannot share a dir between them

COOL FEATURE: You can now use --pm <npm|yarn>@<version|dist-tag> (one or more times) to run smoke tests against specific package managers.  This includes packing, installation and script running.  smoker now defaults to using the latest version of npm, but this may need to change to just use whatever is in the PATH.  smoker will also ignore any packageManager field in your package.json, but perhaps that should be the default.  All this works because of corepack which is kind of neat.
To start, npm v7+ and yarn v1+ are supported--though if you have any weird global config, I make no guarantees.  Note that PnP is unsupported, and whether it is possible is an unknown to me.
I think I added config file support, but I might not keep it.  I was worried that too many --pm flags on the CLI would become unwieldy (so you could have {"pm": ["npm@7", "npm@8", "npm@latest", "yarn@1", "yarn@2"] in there).  Anyway, it's whatever lilconfig allows.  I didn't test it, sorry!
pnpm support is forthcoming (no estimate).

Current dependencies on/for this PR:

main

PR #273   👈

This comment was auto-generated by Graphite.


nice work

I have some problems and I need your help

Hi thank you for the issue, I have edited the file, the function name should be "run_nerf_helpers".


Linux Auto Setup

Windows has support for auto setup (no need to download adb or scrcpy, scrcpy-gui takes care of it for the user). A similar process should take place for Linux systems.
Currently I plan to have Ubuntu systems apt install android-tools-adb and snap install scrcpy., while Arch based systems will use the AUR to install the scrcpy package (I currently don't have a running Arch system, so I'll have to set up a VM and go from there).

Auto setup has been added for Ubuntu--based systems (Ubuntu, Xubuntu, Linux Mint, etc.).

What I just added should fix things for Debian-based systems! I'm going to see if I can find some way to detect if we're running Crostini, because Crostini doesn't play nice with running PySimpleGUI as root.

By check for the presence of the CrosAdapta theme, I was able to detect Crostini. Without running a certain command, Crostini can't run GUI apps as root, so if we fail to connect to a display and we're running under Crostini, we can throw an error accordingly to run the hack (since calling it from our Python script fails).

At the moment, Debian-based systems can install adb and scrcpy through apt and snap respectively. Problematically, many systems don't have snap. The current implementation is to install snap, however, there should be an optional method to install adb and scrcpy through direct downloading. This would also allow support to vastly expand to pretty much any Linux distro!

It looks to be direct downloading would require compiling scrcpy, which would just move back the problem to downloading the tools for compilation. Maybe at some point, I'll let scrcpy-gui compile scrcpy, but for now, I'm going to leave it as is. This commit adds (UNTESTED!) support for installing adb and scrcpy through pacman and yum, so hopefully that should cover enough operating systems that most of them will support automatic setup out of the box. I'm going to close this issue now, it's been long-standing, and functionality was (hopefully added correctly) in the previously mentioned commit.


apis: update cpu scheduling plugin args and apis

Signed-off-by: Joseph<EMAIL_ADDRESS>Ⅰ. Describe what this PR does

Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews
V. Checklist

[ ] I have written necessary docs and comments
[ ] I have added necessary unit tests and integration tests
[ ] All checks passed in make test

Hi @eahydra, nice work. Please complete the checklist in your PR description as well.

/lgtm
/approve


IHostedService Usage Questions

I'm playing around with IHostedService usage today and I've got just a couple questions about usage:

Are there any plans for a mode where the hosted service would "fail fast?" What I mean is, if an IHostedService.StartAsync() method throws an exception, can we have that fail the application bootstrapping? Or do you handle that somehow in the WebHost by catching the logging output?

Any thought about executing the IHostedService methods in parallel for faster cold start times?

Today exceptions are caught and logged:
https://github.com/aspnet/Hosting/blob/release/2.0/src/Microsoft.AspNetCore.Hosting/Internal/HostedServiceExecutor.cs
No, we haven't considered parallel start. If your services are taking that long to start consider offloading them to the threadpool.
We're looking to expand on the IHostedService concept in the new generic host for 2.1. This API does fail fast if any individual service fails.
https://github.com/aspnet/Hosting/blob/bb89551d0ddbaab434de37f7df242322e35baab3/src/Microsoft.Extensions.Hosting/Internal/Host.cs#L46-L52
WebHost won't be moving to this model for 2.1 as it requires breaking changes, but perhaps in a future major version.


Change template for Deploy on Railway button.

Uses their new template engine.

Hey there,
Recently, Railway has deprecated this style of templates, so we should definitely be moving to the shortcode format. For security purposes, you could generate the template yourself.


[ci] upgrade GitHub Actions third-party actions to newest versions

I've been noticing a bunch of these warnings in GitHub Actions jobs:

The add-path command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2020-10-01-github-actions-deprecating-set-env-and-add-path-commands/

The set-env command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2020-10-01-github-actions-deprecating-set-env-and-add-path-commands/

For example, you can see these annotations on https://github.com/microsoft/LightGBM/actions/runs/343069981.
To be sure that these changes from GitHub don't disrupt our CI, this PR updates the GitHub Actions workflows in this project to use the newest versions of third-party actions. You can see, specifically, that updates to deal with this deprecation were made in the most recent release of actions/checkout: https://github.com/actions/checkout/releases/tag/v2.3.4
It also fixes a mistake I made setting them up....we should be using https://github.com/conda-incubator/setup-miniconda, not this fork of it: https://github.com/goanpeca/setup-miniconda.

Please update this line as well.
https://github.com/microsoft/LightGBM/blob/12e220b79d459396ce74f60f260786ad67a8fbb9/.github/workflows/cuda.yml#L41

ah missed that one!


Aqara Honeywell isn't detected

Hi,
I have two Aqara devices which work correctly : wireless smart switch and temperature and humidity sensor. They are detected automatically when i press the button to associate.
But the Honeywell is not detected when I press the button on the hood.
How should I do it ?
Thank you.
Domoticz version: 4.97

I guess this device isn't supported yet

Duplicates #87


[SPARK-41860][SQL] Make AvroScanBuilder and JsonScanBuilder case classes

What changes were proposed in this pull request?
Make AvroScanBuilder and JsonScanBuilder case classes from normal classes.
All the other existing ScanBuilders inheriting from FileScanBuilder are case classes which is very nice and useful. However the Json and the Avro ones aren't, which makes it quite inconvenient when working with those. I don't see any particular reason why these two would not be case classes as well given they are very similar to the other ones and very simple and stateless. Especially when AvroScan and JsonScan are also already case classes like the other scans.
Why are the changes needed?
When working on using different possible FileScanBuilder is really problematic that these 2 ScanBuilders are not case classes, especially given all the others are. This is a very simple change that helps development greatly and allows for more flexibility / feature deelopment when working with ScanBuilders.
Does this PR introduce any user-facing change?
No
How was this patch tested?
No need for testing as it's a simple change from class to case class

Can one of the admins verify this patch?

Tests are passing. Only python linter tests are failing, but there is no related change and it sems like they are failing in many other PRs so it's a flake

Also, cc @sunchao


Cannot read string that is written from react native

Hello, we've been using this package to read and write nfc tags in mobile and we're trying to read nfc data in web, but to no avail we can't read the data that is written from the mobile app.
This is the output
ACS ACR1252 1S CL Reader(2)  card removed { atr:
   <Buffer 3b 8f 80 01 80 4f 0c a0 00 00 03 06 03 00 01 00 00 00 00 6a>,
  standard: 'TAG_ISO_14443_3',
  type: 'TAG_ISO_14443_3',
  uid: 'ab27f824' }

Please point me in the right direction.

https://github.com/pokusew/nfc-pcsc#can-i-use-this-library-in-my-react-native-app


fix:[CUS-144]:AssetID too Long fix

Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List
any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

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

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also
list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Checklist:

[ ] My code follows the style guidelines of this project
[ ] My commit message/PR follows the contribution guidelines of this project
[ ] I have performed a self-review of my code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Other information:
List any documentation updates that are needed for the Wiki

Hi @AnandPaladin Can you plz write update query for all modified Insert statement.

@arunpaladin
As we have discussed over call, update statements are there for those targetType which are having name attributes as part of their DTO objects.


[PR] Resolve #89 refactors the transformer decoder block

This pull request includes several contributions to the transformer decoder block. First, refactors all sublayers to ensure they are working correctly so far. Second, it adds test cases for the decoder block to ensure the correct functionality of the model. Finally, the transformer decoder block overall performance has been improved.

This PR has resolved #96


Add PyWeakref_GetRef and use it in weakref wrappers.

Refs #4265. Also ping @SuperJappie08.
The compat definition is adapted from the C implementation in pythoncapi-compat.
See also my comment in the weakref issue about whether we should add a new API based on PyWeakref_GetRef that doesn't panic or return borrowed references.

@ngoldbaum, Cool that you looked at it. (Sorry for the slow response on my side, though)
When finalizing the initial Weakref PR, I already saw the proposed API changes, but it's building for bete/pre-release 3.13 versions had just started, so I decided to leave it for another time.
Everything looks good, except I'm wondering if it would be better to drop get_object_borrowed since it cannot be supported across all versions.

When the merge fails like that, how do you see what happened? I don't see anything on the merge queue page.

I think it just timed out, CI was a bit saturated last night.


enable ip whitelisting for create and update

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR has modified HISTORY.rst describing any customer-facing, functional changes. Note that this does not include changes only to help content. (see Modifying change log).

[x] I adhere to the Command Guidelines.

@qwordy please help with this PR.

I prefer to use None in if statement explicitly. Like if var is None or if var is not None. Be careful.

@gtracer Add test cases because new parameters are added. Thanks.

@gtracer Add test cases because new parameters are added. Thanks.

@qwordy can you be more specific about your requirements for unit tests? As you can see from our CLI code there hasn't been a single test added in the last year so I want to avoid building out our entire testing infra in this PR. If there is something in particular you require to be unit tested due to the new param?
To me, the highest value add for testing would be to ensure the helpers/validators that were created (the main purpose of this PR) were tested.

@gtracer Add test cases because new parameters are added. Thanks.

@qwordy can you be more specific about your requirements for unit tests? As you can see from our CLI code there hasn't been a single test added in the last year so I want to avoid building out our entire testing infra in this PR. If there is something in particular you require to be unit tested due to the new param?
To me, the highest value add for testing would be to ensure the helpers/validators that were created (the main purpose of this PR) were tested.

It's OK if aks doesn't have tests all the time. But be careful about quality. Sometimes a small change will break existing use case unexpectedly.
In some modules, there are lots of tests. E.g.
https://github.com/Azure/azure-cli/blob/dev/src/azure-cli/azure/cli/command_modules/vm/tests/latest/test_vm_commands.py

@gtracer Add test cases because new parameters are added. Thanks.

@qwordy can you be more specific about your requirements for unit tests? As you can see from our CLI code there hasn't been a single test added in the last year so I want to avoid building out our entire testing infra in this PR. If there is something in particular you require to be unit tested due to the new param?
To me, the highest value add for testing would be to ensure the helpers/validators that were created (the main purpose of this PR) were tested.

It's OK if aks doesn't have tests all the time. But be careful about quality. Sometimes a small change will break existing use case unexpectedly.
In some modules, there are lots of tests. E.g.
https://github.com/Azure/azure-cli/blob/dev/src/azure-cli/azure/cli/command_modules/vm/tests/latest/test_vm_commands.py

That is a fair warning - certainly appreciate how valuable tests are. I have added in tests on the new helper function and validation function that I created.

One comment regarding updating error string.
Otherwise /lgtm


[TOPI][Relay] New Op: MetaScheduleLayoutRewrite

This is part of our effort to support layout rewrite on meta schedule.
cc: @junrushao1994

Retriggering CI


xfce.xfce4-notifyd: 0.8.2 -> 0.9.2

https://gitlab.xfce.org/apps/xfce4-notifyd/-/compare/xfce4-notifyd-0.8.2...xfce4-notifyd-0.9.2
Description of changes
https://repology.org/project/xfce4-notifyd/history says this is stable release, we should no longer have issue building this after x-d-t 4.18.1.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

@ofborg test xfce


 Thanks!
What version of Go are you using (go version)?
1.8
Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
Issue is OS independent
What did you do?

What did you expect to see?
Expected to see feedback being appreciated and taken seriously as is done for 99% of open source projects.
What did you see instead?
Brad Fitzpatrick immediately closes tickets with snarky messages and claims the underlying issues are not problems at all, pretending like he could possibly see every good use for a project as vast as a programming language.  Is he James Damore in disguise? After one too many run-ins with Brad and being told why we are "wrong", we have given up and are slowly replacing all our go code with rust and ruby.

Please be nice. If you can reformulate your issue with the way the project is managed in a nicer way, I suggest you do so and then post on the golang-dev mailing list.
Closing here, since this is not a Go bug.


DNS settings on VNet level gets stuck on any app service that has VNet integration

We had performance issues on app services which had VNet integration because app services had old DNS settings (pointing to decommissioned DNS server) cached from VNet integration.
"When we run SET WEBSITE_DNS_ from Kudu console on app service it shows DNS servers that was set on VNet level long time ago".
The VNet itself doesn't have the DNS settings anymore (using azure default DNS).
We had to remove VNet integration and add it again to get rid of the settings. I'm not sure if this is a bug or a known behavior.

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

ID: a7a98803-1438-b1b5-f543-7dd88bc4294e
Version Independent ID: 37ff1d0f-ed8e-5e4d-1f4c-1b9f6cffb938
Content: Integrate app with Azure Virtual Network - Azure App Service
Content Source: articles/app-service/web-sites-integrate-with-vnet.md
Service: app-service
GitHub Login: @ccompy
Microsoft Alias: ccompy

Thanks for the feedback  @mohamed-shehata-m! We are currently investigating and will update you shortly.

Hi @mohamed-shehata-m, thank you so much for your patience. This will require a deeper investigation, if you have a support plan you may file a support ticket or send us an email at<EMAIL_ADDRESS>with a link to this thread URL and your Subscription ID
so that we can create a one-time-free support ticket for you to work closely on this matter.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Query returns empty result set

Trying to run the example code from https://docs.databricks.com/en/dev-tools/nodejs-sql-driver.html#language-JavaScript
I am using the personal token method but every query returns an empty array.
I also enabled debug level logging and seems to be no error ar anything strange:
{"level":"info","message":"Created DBSQLClient"}
{"level":"info","message":"DBSQLClient: initializing thrift client"}
{"level":"debug","message":"Session created with id: 9ece70bf-xxxxxx"}
Running query
{"level":"debug","message":"Operation created with id: b2cfd491-xxxxx"}
{"level":"debug","message":"Fetched chunk of size: 100000 from operation with id: b2cfd491-xxxxx"}
{"level":"debug","message":"Fetched all data from operation with id: b2cfd491-xxxxx"}
{"level":"debug","message":"Closing operation with id: b2cfd491-xxxxx"}
[] //this is the result
{"level":"debug","message":"Session closed with id: 9ece70bf-xxxxx"}`

Running on MacOS using node v18.18.2.

Hi @iaxentoi! Can you please share your exact script? Thank you!

hi, here is the script:
const { DBSQLClient, DBSQLLogger, LogLevel } = require("@databricks/sql");

const serverHostname = "xxxx";
const httpPath = "xxxx";
const token = "secret";

if (!token || !serverHostname || !httpPath) {
  throw new Error(
    "Cannot find Server Hostname, HTTP Path, or " +
      "personal access token. " +
      "Check the environment variables DATABRICKS_SERVER_HOSTNAME, " +
      "DATABRICKS_HTTP_PATH, and DATABRICKS_TOKEN."
  );
}

const client = new DBSQLClient({
   logger: new DBSQLLogger({level: LogLevel.debug })
});

const connectOptions = {
  token: token,
  host: serverHostname,
  path: httpPath,
};

client
  .connect(connectOptions)
  .then(async (client) => {
    const session = await client.openSession();
    console.log(`Running query`);

    const queryOperation = await session.executeStatement(
      'select "Hello"',
      {
        runAsync: true,
        maxRows: 10000, // This option enables the direct results feature.
      }
    );

    const result = await queryOperation.fetchAll();

    await queryOperation.close();

    console.log(result);

    await session.close();
    await client.close();
  })
  .catch((error) => {
    console.log(error);
  });

Actually when switching to another server it works fine, so might some kind of permissions/access issue.
The connect options seem to be fine, otherwise it would give a 403 response.
So it just fails silently for other reason which I was not able to discover yet.

Hi @kravets-levko, after some investigation this seems to be related to the Databricks runtime version.

returns empty result set on 9.1 LTS
returns correct result on 10.4 and 13.3

The processing can be seen on Databricks side and the results to be returned, but that is not picked up by the node client and an empty result set is returned instead.

Hi @iaxentoi! Thank you for you replies. It's really great that you managet to localize the issue - now I know where to start with debugging. At the same time, it's weird because the library should throw an error on unsupported result formats, so very likely there's something wrong there. I'll try to reproduce this behavior and let you know.
Also, can you try other conectors if possible (Python/Go)? I'm curious if thir behavior is the same or different. Thank you!
P.S. Your script looks good, I see nothing wrong with it

Yes, tried with Python and it works fine.


ng-options "track by" failing. 1.4.1

In version 1.4.1 track by no longer works and brings back the whole object.

please provide a plnkr with a demo.

There's nothing we can do without a reproduction of the problem. Please reopen if you can provide one,


[Feature Request] collect [id] elements in a helper property of a LitElement instance.

I think this has been requested before, I can't find the original post.
Would it be possible to reconsider this request ?
In Polymer $ contains all scoped elements that have an id attribute. Why won't LitElement implement this too ?
If I remember correctly someone suggested to make a wrapper class with a method that will create this global property but when I think about wrapping a class I think something bigger than just implementing such a trivial feature.
Also when using an id attribute on a template's element most of the time it's because this element has a particular functionality and is going to be used here and there in the code, therefore this.shadowRoot.querySelector or @query('...') syntaxes because too tedious to write.
Sometimes it's the small changes that make the big difference, I am probably over 20 projects using LitElement and I feel really tired to always write this part when I'd expect to just use something like this.$.myContainer to designate div[id=myContainer]
Please reconsider this feature. Thanks!

The reasons we left this feature out of LitElement are that

this.$ only works with initially rendered elements - it doesn't work with subsequently rendered content in composed templates from conditionals and loops.
We're trying to keep LitElement as small and as non-overlapping with the platform as possible. We already have a decent way of getting elements from the shadow root with this.shadowRoot.querySelector which importantly works in any web component, not just LitElement.
this.$ doesn't type check, is prone to typos, and hard to refactor because it's so dynamic. You would need a way to declare it's type, which brings us to...
@query() is generally better because you declare (and document if you want) the elements you expect and their type if you use TypeScript. It's on-demand because it queries the live DOM. It supports any selector, and there's @queryAll()
Perf, a little. Setting up this.$ takes work which might not be used. Maybe ids are there for CSS.

I can't argue with that. Thanks to take your time to expose the details.
Also I have to ask, are there any LitElement wrappers in the npm community that the Polymer team would recommend to use when one wants to augment the capabilities of the plain class ? With features like the one I am asking here, debouncer, and other extended features I can't think of right now.

I've seen a wrapper that adds observable fields, and I think Netflix might be working on something with computed properties. I think a mixin to add this kinds of features is definitely a good approach!

Ok. Thanks for confirming. I'll make a mixin as an individual package.


Fix wizard transition after pressing purple button

After commit 8966fab81e8a679b3be8fc5976107718b52f7686 the wizard wouldn't transition after pressing the purple button because the event listener wasn't attached
Also change fix in that commit to match previous code

Ugh, thanks!


Remove failurePolicy suggestion from "bad webhook" SL

This PR removes a service log's suggestion that the customer use the 'failurePolicy' setting to mitigate a custom webhook that's misbehaving. While the original PR that added this suggestion surely had good intentions, a recent thread shows that a customer may use this setting to mask such an issue just enough to get out of limited support (as opposed to actually fixing the broken webhook).
The linked OCP docs also discourage this suggestion: "Using Ignore can result in unpredictable behavior for all clients."

/label tide/merge-method-squash

/lgtm

/approve


More Java and Kotlin examples

This patch provides further example files, coded both in Java and Kotlin. References to the newly added files were added to the manual, too.
This patch also corrects the Kotlin MyApp example which currently does not compile:
companion object {
    @JvmStatic fun main(args: Array<String>) {
        CommandLine.run(MyApp(), *args)
    }
}

Inside the companion object, CommandLine.run() is called, however, MyApp implements Callable and not Runnable.
Since both methods run() and call() are deprecated, I changed this to CommandLine(MyApp()).execute(*args).
In order to prevent these kind of errors in the future, I would propose to add a task compileKotlin to the file build.gradle of the picocli-examples module.

Codecov Report

Merging #913 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #913   +/-   ##
=========================================
  Coverage     92.86%   92.86%
  Complexity      393      393
=========================================
  Files             2        2
  Lines          5834     5834
  Branches       1527     1527
=========================================
  Hits           5418     5418
  Misses          203      203
  Partials        213      213

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 7ea01d9...cc89e68. Read the comment docs.

@deining Compiling the Kotlin examples is a good idea, not sure why we were not doing this already... :-)
Fixed in #915.
Thanks for raising this!


Unflake the system tests

The system tests are currently hella flaky.

@kir-titievsky I've been noticing lately that sometimes when I fetch a list of snapshots, I get more than I am expecting. In my testing I create a branch new topic/subscription, so I would assume that there would be no pre-existing snapshots. After creating 1, if I list the snapshots I'm often surprised to see that there might be 2 or 3. Any ideas why this might be happening?

Nevermind! I forgot Snapshots were global to the project, so now that we run system-tests multiple times per PR, I think what I'm seeing makes sense.


Make build, code coverage, test, and benchmark phases parallel

This can be done by making separate jobs and downloading build artifacts.

Closed due to complex dependency and it's not able to be made parallel.

Closed due to complex dependency and it's not able to be made parallel.


Add vrr to hyprctl monitor

Since the (re)addition of vrr in 127e80692f78b0d91450194fcb4ce0009c2a98a4, it is very useful to know, whether a given monitor has vrr enabled or not, since not all monitors support it. Now hyprctl reports, whether a monitor has vrr enabled or not.
Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)
I'm unsure, whether my approach is the right way to check for vrr. Though through my testing with both a vrr monitor and a non-vrr monitor, it seems to work properly.
Is it ready for merging, or does it need work?
Ready for merging.

why do you use "1" instead of %i with 1 and 0? that way it's (int)w->the_prop

nvm u fixed it


skip installer stage for PR testing

Currently the PR tester triggers lots of installer jobs which all back up and can add up to an hour to the pipeline.
fixes: https://github.com/AdoptOpenJDK/openjdk-build/issues/2030

run tests

build failure was unrelated to this PR change (macOS issue)

run tests

I'm not sure this is the right move now, but maybe in the future. We don't really have testing for the installers, yet. As soon as we have a PR tester there, this is actually a good idea from my point of view. Is there another option like kicking of less installers?

I'm thinking as @aahlenst as this recent installer issue https://github.com/AdoptOpenJDK/openjdk-installer/issues/240
but maybe nightly is sufficient for detecting this.
Installer change are not so often done


Hotkey  Pouvoir sélectionnez les touche comme F9, F7, F5, F8 pour lancer cerebro...

Be able to select keys as F9, F7, F5, F8 for launch cerebro ...

Sounds reasonable, I'll implement it.
Currently only shortcuts with modifier keys (cmd, ctrl, alt, shift) are allowed. You can edit config file to set F1-F24 as a global shortcut

It would be also nice to have just cmd as hotkey i.e. if you press cmd and nothing else and then release cmd, the action should be triggered.
(sorry, if this already works, I can't test it because of compilation fails on ArchLinux)


Remove most no_run on doc tests

Looks like there are many doc tests are set as no_run. Remove these can increase lines of code coverage significantly.

@stjepang Sorry, I found there are some tests on mac/linux also need to be no_run. Then after that I realized most tests in async_io.rs will be blocked on windows.
Could you cancel workflow on this PR. I believe there are still two pending.


Support provider alias as data type with tf 0.12

Installation details

[x] IDE version: IntelliJ IDEA 2019.1.3 (Ultimate Edition) Build #IU-191.7479.19, built on May 28, 2019
[x] intellij-hcl plugin version: v0.7.0
[x] Terraform version: 0.12.0

Terraform Configuration Files
provider "aws" {
  alias   = "iam"
  region  = var.region
  version = "~> 2.12.0"
}

resource "aws_iam_user" "test" {
  name          = "test"
  path          = "/"

  provider = aws.iam
}

module "test" {
  source = "./modules/test"
  providers = {
    aws.target = aws.iam
  }
}

Exception
For resource:
Unresolved reference iam

For module:
<property> expected, got '.'
Unresolved reference iam
<identifier> or <string literal> expected, got '}'

Expected Behavior
Plugin should not report a validation error for provider alias data type when not defined as a string literal.
For Modules provider target alias should not be reported as an error if not defined as a string literal.
Actual Behavior
Provider alias is reported as error, see exception section.

Fixed in 0.7.1, will be released shortly


Problem with cart translations

Hi,
i have a problem with the translations of the cart when adding, removing or updating items.
While everything else works fine with the translations the popup windows i get aren't translated.
The url of one product is here http://cretanflavors.com/sidal/?product=klidi-germaniko-cr-v-12-13
If you add it to the cart by pushing ΠΡΟΣΘΗΚΗ ΣΤΟ ΚΑΛΑΘΙ you will get an english popup message and not a greek one even though to .po & .mo files are correctly translated.
If i disable the plugin everything works fine with the translations but i have other issues.
Looking forward for your help.
Thank you very much.

Hi,
as far as I know, the woo-poly-integration plugin doesn't touch the "add to cart" notification message in the single product page.
I understand that you deactivated the plugin and it works, but I think this is more related with a conflict with your theme. What theme are you using?
I believe 'wc_print_notices' function is responsible for this notice message.
Look in your theme or child theme code that are messing around with this part of the single product template: https://github.com/woothemes/woocommerce/blob/master/templates/content-single-product.php#L26-L31
Things that may be using the action 'woocommerce_before_single_product' or modifying 'wc_print_notices' function that is hooked to this action.

Hi,
thanks for the information.
I did some tests and the problem is fixed only when i deactivate the polylang plugin and not the hyan plugin and i have only one version of the website.
if i have both english and greek versions i get only the english cart notifications.
The theme i use is genesis from studiopress.

I would then recommend reaching out to studiopress guys, since this is a premium/paid theme they should support you if you are still entitled for support. Does genesis have support for woocommerce? didn't find that piece of information in their website.

Thank you for your help,
did some tries with the plugins and everything and it works fine right now.
Don't know how it was fixed but i am happy :)
Thanks again!

Glad to hear it!


dependencies make it impossible to start

compile time dependencies it seems..
I was wondering what I need rot do to get started. I quite like the way this is all designed with the IR, as it were. Pretty cool ideas in here.
cmd % go build .
../pkg/lunchpail/version.go:9:12: pattern version.txt: no matching files found
../pkg/fe/assembler/template.go:6:12: pattern charts.tar.gz: no matching files found
../pkg/fe/transformer/api/shell/embed.go:6:12: pattern shell.tar.gz: no matching files found
../pkg/fe/transformer/api/workerpool/embed.go:6:12: pattern workerpool.tar.gz: no matching files found
../pkg/fe/transformer/api/workstealer/embed.go:6:12: pattern workstealer.tar.gz: no matching files found

If you are not ready for dog fooding just let me know.

howdy! thanks for the feedback. you did indeed catch us just slightly in advance of being ready with the documentation. to build the CLI, currently:
./hack/setup/cli.sh

which will build ./lunchpail
also, please disregard the instructions in the top-level README. we should have an update on this by tomorrow. we still need to publish the base images.
and thanks for the patience during this initial setup. we just got approval from our company to open source, and there are some chicken-and-egg issues there (get on github, get automation in place...)
i'll ping this thread when everything is in place.


Running commands with $

I'm unable to run this command. I'd like to use replay to run bash commands that use $ and it's unsupported by fish.
❯ replay "export TEST=$(pwd)"
fish: $(...) is not supported. In fish, please use '(pwd)'.
replay "export TEST=$(pwd)"

You need to use single quotes here to please Fish.
$ replay 'export TEST=$(pwd)'
$ echo $TEST
/home/jb/tank/test

Thank you!


Pin cartopy to v0.23.0

Prevents crash due to incompatible iris patching, which is fixed in the unreleased iris 3.11.
This can be removed once iris 3.11 is released.

Contribution checklist
Aim to have all relevant checks ticked off before merging. See the developer's guide for more detail.

[ ] Documentation has been updated to reflect change.
[ ] New code has tests, and affected old tests have been updated.
[ ] All tests and CI checks pass.
[ ] Ensured the pull request title is descriptive.
[ ] Conda lock files have been updated if dependencies have changed.
[ ] Attributed any Generative AI, such as GitHub Copilot, used in this PR.
[ ] Marked the PR as ready to review.

This might not be necessary, as iris 3.11 is due to be released on Monday.


Stage 2 refining does not update weights of encoder.

Hi, I tried to stage 2 refine a vit model. The loss curves of id heads seemed fine but when i evaluated the refined encoder for a different task it was no different than the base model.
When i compared the base model and the refined model , there was no change in the weight values. Is this because "is_frozen" is set to true in the below config file?:
https://github.com/ml-jku/MIM-Refiner/blob/main/src/yamls/stage2/l16_d2v2.yaml

Hi, yes this is expected behavior. As the ID heads are random at the start of training, also the ID objective would be random and it would propagate a random signal back to the encoder. Therefore, we first train only the ID heads (with a frozen encoder) in "stage2" and once the ID heads are sufficiently good (after a couple of epochs) we propagate gradients back to the encoder ("stage 3").
For more information, see Appendix D.5 in the MIM-Refiner paper or the MAE-CT paper (Section 3 or Figure 4 describe this).
You can simply reuse your stage2 run by changing the corresponding stage_id in the stage3 yaml. So if your stage2 id is "abcdefg", change the stage_id of this yaml from "xpge1tv6" to "abcdefg"

Understood, Thank you.
I evaluated my base model and the stage 3 refined model(at epoch 2) the performance of the refined model is sub-par to the base model. could it be an initial performance dip and later the refined model performs better?

by default you should have a k-NN monitor during training that evaluates the k-NN accuracy (with only 10% of the imagenet training set). This metric should increase steadily during start of training.
This is how it looks like for MAE-Refined-L/16


Pure open source IDE

This README reads rather confusing to me: Is it possible to build fsharp support without Rider?

cd <..>\fsharp-support\ReSharper.FSharp
msbuild ReSharper.FSharp.sln

I am on Linux.

And Visual Studio is open source?

Then you need mono and xbuild I guess.

@ShalokShalom Yes, it is possible to build the Rider F# support plugin without using Rider, and it's explicitly stated in the readme that Rider is an optional tool to have.
You should have dotnet SDK installed which has msbuild bundled. To build the plugin you have to use 2.0 version or newer. dotnet build ReSharper.FSharp.sln should do the trick.
Are there any particular build steps that didn't work for you on Linux?

Yes, it is explicitly stated, just not explicitly stated described how to do.
It is stated how to build Rider, see here:
https://github.com/JetBrains/fsharp-support#building-the-plugin-and-launching-rider-in-a-sandbox
And then "if using IntelliJ IDEA" which gets compared to Gradle.
Why?
I would more expect one way which describes how to build it with Rider and one how to build it with IntelliJ.
I have not tried yet to build any of both, because I wanted to hint about this imho documentation issue.

The plugin consists of two parts: Intellij part (written in Kotlin) and Rider (.net) part, written in C#/F#. To build the plugin you need to build both parts. gradlew prepare downloads everything, gradlew runIde build the Kotlin part, downloads a Rider snapshot, copies .net part, installs the plugin into the test Rider and run it. Building ReSharper.FSharp.sln is needed in order to build .net part binaries.

So, how is this completely open source, if I need both parts? @auduchinok auduchinok says its possible without

@ShalokShalom I'm not sure what are you asking about? The plugin consists of two parts both of which are open-source with sources in this repo.

Ah, I see. So I use IDEA and the fsharp implementation of Rider in order to build. Thats fine.
As said, the README has no header/title without "Rider" in its name and that one is closed source.

As said, the README has no header/title without "Rider" in its name and that one is closed source.

If you want not to use Rider to build a Rider plugin I would recommend going with MSBuild from either Microsoft Build Tools or .NET Core SDK. Please use documentation for the tool of your choice.


Separation of response from request in a custom method

This in some ways seem similar to some of the things discussed in issue #231 where requests could be passed a cancellation token,
My usage of tower-lsp is a little weird due to some !Send + !Sync variables, these variables have to live off in their own threads.
The main Backend implementing LanguageClient keeps a channel to send these threads messages.
It then creates another thread which has a clone of Client which deals with receiving all the responses from all the threads and sending them back to the client.  This works okay, until you need to deal with a request with a response, in implementing a custom_method() it can be difficult to obtain the value that needs to be returned from Backend itself.
Part of the reasoning behind the response thread is just that I don't see a nice way to receive from within Backend.
e.g. to select! on both incoming jsonrpc and subjugate thread channels, since this is hidden within the serve function,
But that would present an alternate way of responding, If backend had a good way of also receiving from channels, There wouldn't actually be any issue.

Backend (Vec of mpsc::Sender one for each subjugate thread)
N-subjugate threads (sync, each owns a !Send + !Sync value, mpsc::Receiver from backend, mpsc::Sender to Response thread)
1-Response task (async, has a clone of Client, mpsc::Receiver).

The communications topology in particular looks like:
Client -> Backend -> Subjugate thread -> Response task -> Client
In #231 it was discussed that requests could be given a separate cancellation token as a part of the request.
I was thinking perhaps you could do a similar thing, instead of returning from the custom method a value, a variation of custom_method
which gets both the params and a ResponseToken, which it could pass to various responders.
P.S. sorry if this is a bit long & quite specific to the constraints upon which my implementation finds itself in
Edit: Making some clarifications about 'Response task', previously I had referred to this as 'Response thread',
Edit 2: After thinking about it, perhaps my Backend can allocate a future, pass it through the communications channels,
then when it arrives back in the async Response Task it becomes ready, and thus no actual modifications are needed in this case, because I may be able to make a ResponseToken without it actually being passed in from call.

FWIW, I'm going to go ahead and close this, the way I got it working was basically described in Edit 2: using a tokio::sync::oneshot, One nice thing about this is that oneshot::Sender::send is sync so the subjugate threads can just respond directly to the Backend, so the Response task is kept out of the loop unless the subjugate thread needs to invoke the client directly.


[INFO] calendar

Hello, i want ask two info:

If i clear a category why it don't adjust the size the event that overap ? I expect that in this case, into a particular day that overlap more event, for exampe from 9:00 AM to 11:00 AM that contain 3 event with different category, if i clear one of this category i want that the 2 event remained occupied the available space.
when i click into montly vision, as a return to the weekly calendar view?

Hi,

The category clear was intended to hide some events to find one by a category. So, just after that, you will probably show again the other category. That is why the positions are not processed again. It could be a feature to add on the next release. Can you explain your opinion and the usage about that (it will help us to make a choice about this suggestion) ?
To return to week view from month view, you have 2 ways :

Use the keyboard shortcut : CTRL + ARROW DOWN
Click on the week number on the left side (on this month, you have weeks : 40, 41, 42, 43, 44 on left)

The keyboard shortcuts are all like this CTRL+ARROW [LEFT;UP;DOWN;RIGHT]

hi, thanks for reply. I explain my project: i am creating a timetable for the course of my university. I'm use a rule engine for establish the constraints. right. For display the result i'm decide to use this plugin. My category are based on type of course. For every course there are different disciplines. When decided to hide one category, i want hide all discipline included in the cours , for obtain the timetable of one specific course.

this is my calendar with different courses.

this is my calendar with a specific course. It could be nice if i could visualize the only category remained as if it were the only one:

Thanks, I see what you need. It is a great idea and the suggestion will improve jquery-calendar.
So, I will discuss your suggestion with the team and reply ASAP.


Opening SSH tunnel to postgres database without using ip of the postgres container

I want to open an SSH tunnel to the postgres instance, but can't find a way to do it via hostname rather than IP. For now I'm using a workaround of netstat -an | grep 5432 to find what is listening on that port and just using that IP. The problem is that the IP is subject to change, and when it does I need to go find it again and update my connection.
One possible solution recommended by @lmars in IRC:

one option is to add the flynnbr0 interface address as a nameserver entry in /etc/resolv.conf, then the host can resolve the DNS names

However per @titanous in IRC:

that could result in a loop, we read resolv.conf to figure out the upstream resolver

Any other options on how to handle this?

Leader routing could solve this, but we need to set up TLS for Postgres first: #2395

This is now documented: https://flynn.io/docs/databases/postgres#external-access

Splendid, thanks!


Query and local targets of links missing

The extractor removes the query part of links like (see attached feed and example below):
<link>
   https://jdhitsolutions.com/blog/books/9389/powershell-scripting-and-toolmaking/?utm_source=rss&utm_medium=rss&utm_campaign=powershell-scripting-and-toolmaking
</link>

is returned as
https://jdhitsolutions.com/blog/books/9389/powershell-scripting-and-toolmaking/

and also local link targets:
<link>
    https://gist.github.com/aeveltstra/94806a1230b8165f43e9b4e4dec9bacc#file-powershell-gui-aws-lambda-start-functions-ps1
</link>

is returned as
https://gist.github.com/aeveltstra/94806a1230b8165f43e9b4e4dec9bacc

While in this case the query is not essential, for other feeds it is. The missing local target may be a usability issue for large articles. Hence the link should always be returned in its complete form.
feed.zip

This library removes advertising parameters from URLs, while preserving all other parameters intact. If you are not a marketer, it should be fine :dancers:
Here's a list of parameters targeted for removal:
https://github.com/extractus/feed-extractor/blob/main/src/utils/linker.js#L21

@ndaidong Thanks for the feedback. No, I'm not a marketeer , so removing advertising parameters is fine with me :-) However, local link targets (see second example) seems a bit too much.

@WetHat I see, the local link targets you mentioned is hash property in URLs. While they can be useful for scrolling to specific sections, we're removing them with purify() method.  This is because feed entries should ideally link to entire articles, not just a specific part.

@ndaidong I completely agree with your point that feed entries should ideally link to entire articles, not just to specific sections.
I've seen a fair share of really weird interpretations of the RSS idea where feed items were pointing to sections inside a large log like article. However, these are corner cases. Sticking with purify() sounds acceptable.


[infra:goldens] Skia Gold "settling" after golden update

There was a recent code change that affected CanvasKit goldens. It didn't land cleanly, so it was reverted then relanded:

First landing: https://github.com/flutter/flutter/pull/135249
Revert: https://github.com/flutter/flutter/pull/136860
Relanding: https://github.com/flutter/flutter/pull/136918

The change affects goldens. According to Skia Gold UI the goldens changed as expected:

However, after the reland I noticed that tests were flaky for a couple of commits after the reland happened:

It's as if Skia Gold needed some time to "settle". There's a period of time during which sometimes Skia Gold would reject the new golden. For example, this build failed with "Skia Gold received an unapproved image in post-submit testing". The test generated golden was c8d9d65546ec0142469c967cd644ade0, which, according to the Skia Gold UI, is the correct SHA.

cc @kjlubick here is the info @yjbanov collected in connection to https://discord.com/channels/608014603317936148/1166114624958124103
I heard from another contributor today about this same thing occurring in https://github.com/flutter/flutter/pull/136809, which was reverted in https://github.com/flutter/flutter/pull/137242.
The PR introduced changes to golden files, but then saw failures for those same golden files when it landed so it was reverted. Here is a post submit build where it failed: https://ci.chromium.org/ui/p/flutter/builders/prod/Mac framework_tests_libraries/13988/overview

I realized I never hit send on my earlier message...
This feels like a "Gold didn't merge the expectations from the PR in time after it landed" thing again, but I'm a bit perplexed that we would see it after the reland, since wouldn't those images (and expectations) already be in there?

e.g. might be a recurrence of https://g-issues.skia.org/issues/40045533

This has now happened twice in a row on the same image change.
PR: https://github.com/flutter/flutter/pull/136809
Revert: https://github.com/flutter/flutter/pull/137242
Reland: https://github.com/flutter/flutter/pull/137247 - we waited several hours to land it after approving goldens, but it still failed after landing.
Reverted again: https://github.com/flutter/flutter/pull/137406
I've asked @TahaTesser to provide a link to the post submit failure we saw on this last one.

@TahaTesser mentioned more than one change was reverted due to this in https://discord.com/channels/608014603317936148/613398423093116959/1167437594590715984
Can you share more details?

Just noticed the golden image changes from the reverted PR above have shown up in a totally unrelated PR that is just changing some doc strings:
PR: https://github.com/flutter/flutter/pull/137178
Gold digest: https://flutter-gold.skia.org/search?issue=137178&crs=github&patchsets=9&corpus=flutter

The goldctl version was last bumped in January I think. I am pushing a cl to bump it to the latest to see if it helps. Just working on a permissions issue: https://discord.com/channels/608014603317936148/608021351567065092/1167515846999031808

Bumping goldctl in https://flutter-review.googlesource.com/c/recipes/+/51940

I've asked @TahaTesser to provide a link to the post submit failure we saw on this last one.

Here it is
https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8766100997618152769/+/u/run_test.dart_for_framework_tests_shard_and_subshard_libraries/test_stdout

@TahaTesser mentioned more than one change was reverted due to this in https://discord.com/channels/608014603317936148/613398423093116959/1167437594590715984
Can you share more details?

I also had to revert for the same failure. https://github.com/flutter/flutter/pull/137230
https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8766100997618152769/+/u/run_test.dart_for_framework_tests_shard_and_subshard_libraries/test_stdout

03:49 +5323 ~16: /Volumes/Work/s/w/ir/x/w/flutter/packages/flutter/test/material/data_table_test.dart: Material3 - DataRow renders custom colors when pressed
══╡ EXCEPTION CAUGHT BY FLUTTER TEST FRAMEWORK ╞════════════════════════════════════════════════════
The following SkiaException was thrown while running async test code:
Skia Gold received an unapproved image in post-submit
testing. Golden file images in flutter/flutter are triaged
in pre-submit during code review for the given PR.

Both PRs were merged 10 and 7 hours after approval.
When gold link stdout logs of a failed test, it also shows cupertino golden file changes while PR is related to Cupertino. It's just a M3 ink sparkle golden check.

@Piinks is this complete? Can we close this issue?

I have not heard reports of this occurring for a few weeks. Should we wait a bit longer to close to be sure?


javax replaced with jakarta

javax.ws.rs was moved to jakarta.ws.rs therefore replaced javax dependency with latest jakarta version

https://github.com/eclipse-basyx/basyx-java-sdk/pull/109 needs to be merged first.

Thanks a lot!

Unfortunately, I have to revert this change. For details, see https://github.com/eclipse-basyx/basyx-java-sdk/pull/109#issuecomment-1252219429


404s from main nav on user account page

There's something up with the About page link:

for a non-logged in user the "About Artsy" link in the footer works  (by rendering the desktop version of the About page)

for a logged-in user the "About Artsy" link in the footer, as well as the "About Artsy" link in the main nav on the user account page (/user/edit), both result in 404s

(In all three cases the "About Artsy" link points to /about)

This also appears to be true for the "Partner with Artsy" link:

for a logged-in user the "Partner with Artsy" link in the main nav on the account page (/user/edit),  results in a 404

Thanks Roop! Fix here: https://github.com/artsy/force/pull/838


[W5.8b][W10-A2] Toh Cheryl

@tohcheryl
Some comments added. Please ack & close the PR after reading comments.


DC Firewall WAC media folder rename

If this PR needs approval, or if the commits need to be pulled into the private repo, contact @ManikaDhiman or @alkohli. Thanks.

#assign: @ManikaDhiman
@alkohli @ManikaDhiman

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off

@Jak-MS: Could you please review this PR? I had signed it off last week. Please let me know if there is anything else I need to do. Thanks!

@ManikaDhiman - thanks for tagging me again. Not sure how this one slipped past.
@dansisson - merging this. with the :::image::: syntax, there's not way for us to see if there are any broken calls due to typos, so can you make sure to check the live page after the changes are published to confirm that the images are displaying as intended?
thanks


Get API Response

Hi there,
awesome plugin! Is it possible to get the API results after calling the URL? I tried var dumping the class but there is no info about the original response request.
I need that to show the customer an error that the updater API may return (like no plugin found / license not valid).

There are several ways you could do that. One option would be to use the $prefix_request_metadata_http_result-$slug filter. For example:
$updateChecker->addFilter('request_metadata_http_result', function($httpResponse) {
    //Do something with $httpResponse here. It's the value returned by wp_remote_get().
    //Remember to return the response.
    return $httpResponse;
});

The actual filter name changed depending on the plugin/theme, so I'm using the addFilter() method for convenience. If necessary, you do something like this instead:
add_filter('puc_request_metadata_http_result-myplugin', 'my_callback_function');

Alternatively, the $prefix_request_info_result-$slug filter is also an option. It is called after the plugin metadata has already been parsed.
$updateChecker->addFilter('request_info_result', function($pluginInfo, $httpResponse) {
    //$pluginInfo is an object that pretty much maps directly to the contents of the API response.
    return $pluginInfo;
}, 10, 2);

There are several ways you could do that. One option would be to use the $prefix_request_metadata_http_result-$slug filter. For example:
$updateChecker->addFilter('request_metadata_http_result', function($httpResponse) {
    //Do something with $httpResponse here. It's the value returned by wp_remote_get().
    //Remember to return the response.
    return $httpResponse;
});

The actual filter name changed depending on the plugin/theme, so I'm using the addFilter() method for convenience. If necessary, you do something like this instead:
add_filter('puc_request_metadata_http_result-myplugin', 'my_callback_function');

Alternatively, the $prefix_request_info_result-$slug filter is also an option. It is called after the plugin metadata has already been parsed.
$updateChecker->addFilter('request_info_result', function($pluginInfo, $httpResponse) {
    //$pluginInfo is an object that pretty much maps directly to the contents of the API response.
    return $pluginInfo;
}, 10, 2);


Youtube Channel

Added one more Youtube Channel . This repo is great will feel lucky to be a contributor of it.

Thanks for helping out @manas1072


#12 Add 2 players + add quarters delimitation

12 I want to switch players

[x] Add in the gameService the currentPlayer field
[x] When rotating a quarter, switching to the other player (your turn is finished)
[x] When adding a marble, now adding in the board the value of the player (1 or 2)
[x] Add some basic CSS to detect more easily quarters (borders variations)

I think you can improve the UI a bit:

when the buttons appear, the grid moves down. It's disturbing
why ask the user to click on buttons representing the quadrants rather than the quadrants themselves?


Support editorconfig INI files for rubocop config

rubocop could use .editorconfig files to configure rubocop (only for the rules that editorconfig support).
This is a feature request, but I'm also wondering if this would be something that rubocop community would be interested in.

It would add too much complexity to the already pretty hairy configuration file parsing IMO. I'd recommend starting a side project that converts .editorconfig files to .rubocop.yml files.

No problem. Thanks for the feedback!


Validating status does not change after Onboarding is created.

I looked at the "/aws/lambda/sb-isen-onboarding-validation" log and found that the processValidateOnboardingQueue is occurring.

Probably this is the reason why it does not change from Onboarding status.
/aws/lambda/sb-isen-onboarding-validation
begin 0, end -2, length 29: java.lang.StringIndexOutOfBoundsException
java.lang.StringIndexOutOfBoundsException: begin 0, end -2, length 29
	at java.base/java.lang.String.checkBoundsBeginEnd(Unknown Source)
	at java.base/java.lang.String.substring(Unknown Source)
	at com.amazon.aws.partners.saasfactory.saasboost.OnboardingService.processValidateOnboardingQueue(OnboardingService.java:1260)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
	at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
	at java.base/java.lang.reflect.Method.invoke(Unknown Source)

begin 0, end -2, length 29: java.lang.StringIndexOutOfBoundsException java.lang.StringIndexOutOfBoundsException: begin 0, end -2, length 29 at java.base/java.lang.String.checkBoundsBeginEnd(Unknown Source) at java.base/java.lang.String.substring(Unknown Source) at com.amazon.aws.partners.saasfactory.saasboost.OnboardingService.processValidateOnboardingQueue(OnboardingService.java:1260) at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(Unknown Source) at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source) at java.base/java.lang.reflect.Method.invoke(Unknown Source)

Reproduction Steps

install-using-cloud9.md
deploy-the-sample-application
build-and-deploy-the-sample-application
onboard-a-tenant

What did you expect to happen?

Completion of Onboarding creation.
What actually happened?

Validating status does not change after Onboarding is created.
Environment

AWS Region  :ap-northeast-1
AWS SaaS Boost Version :Version v2.2.3-6@452d7ca
Workload OS  (Linux or Windows) :

Other

This is :bug: Bug Report

Maybe this change will have an impact
https://github.com/awslabs/aws-saas-boost/commit/452d7ca8902e7338e999290eebad9062c342915f

@YutaOkoshi that error is coming from the section of validation checking the Route53 Hosted Zone for duplicate tenant subdomains. There must be an A Record in the HostedZone for that tenant with an unexpected structure. You can find the hosted zone by looking at the appConfig and then run aws route53 list-resource-record-sets --hosted-zone-id YOUR_HOSTED_ZONE_ID to see what the output looks like.

@brtrvn
thanks for replay.
I re-created the Application and it worked.
Probably because the Domain Name and Hosted Zone of the Application were unintentionally specified.
If Validation finds something error, the end user may wait forever without an error message and may not know where to fix it.
Therefore, it would be beneficial to end users if error messages are output.


Unable to render outside of a notebook context

The suggested way of getting standalone HTML from ipywidgets doesn't appear to work:
from ipytree import Tree
from ipywidgets.embed import embed_minimal_html

w = Tree(stripes=True)
embed_minimal_html("index.html", w)

$ python3 -m http.server

Try the previous version.. it worked for me with voila.
pip install ipytree==0.2.0

This should be fixed for 0.2.2 (similar to #57)


(1.x) STORM-2389 Avoid instantiating Event Logger when topology.eventlogger…

….executors=0
Manually tested with cluster mode: 4 workers in local machine.

+1


Uneven list - preview return to first position

When index is not equivalent to preview index + visible items, preview click will return to to first position.
To reproduce, click next twice, and once on preview. Instead of going back once, index goes back all the way..:
http://codepen.io/anon/pen/qEzopE

This is fixed with beta 3.
test case


Integrate scylladb terraform provider

@DanielHe4rt I've just added the docs part now https://github.com/scylladb/care-pet/pull/130/commits/5f627a1409924587b594cb8ee6d3febcc7697c83 it still needs some work because the content should be pulled in from the readme and we're fixing that now

Nice work!


How Interpolation and Extrapolation works?

Hi @YyzHarry,
I hope this message finds you well. I am reaching out to you regarding your paper, specifically about the discussion on interpolation and extrapolation of sample labels. While reviewing the code, I couldn't understand whether this functionality is implemented as a separate module or if the same LDS and FDS symmetric kernel is utilized for this purpose as well.
Could you kindly provide clarification or guidance on this matter? Many thanks.

Hi, thank you for your interest! Interpolation and extrapolation are key benefits offered by LDS and FDS. These methods particularly enhance learning in imbalanced regression settings, leading to improved generalization for unseen targets as a result. For detailed results on interpolation and extrapolation, please refer to Section 4.2 in our paper.


Normalize vectors.scad, quaternion.scad naming

quaternions.scad has functions almost all named with a Q_ prefix.
vectors.scad used a v prefix.
We should probably standardize on v_ and q_ prefixes.

There are also complex functions, currently with a C_ prefix.  (Currently used only for polynomial roots, I think.

Quaternions have been re-named to start with q_

vmul() / vdiv() / vabs() / vfloor() / vceil() renamed to v_ style.

Complex math functions already have c_ style names.


Feature gate test harness for x86_64

No longer required as Earthly can use emulation to always build harness on x86_64


feat: Add feed of public transport networks in Switzerland [SOURCES]

The feed aggregates all public transport networks in Switzerland
https://opentransportdata.swiss/en/dataset/timetable-2022-gtfs2020

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

Oh I see, thank you for the review. I moved my changes to the existing source.


✍ Redesign: New User-Interface

Hello everyone 👋,
it would help me if you would help me with the redesign of the calculator.
What do you need to be able to do to help here?:

Basic HTML knowledge
Basic/Advanced CSS knowledge

⚠ Important Note: Please sync your forks to prevent merge conflicts ⚠

Hi @anubhav1206 👋,
thank you for your help, but I get an error message in the style.css file: **'Also define the standard property "background-clip for compatibility reasons css(vendorPrefix).
Determines the background painting area.
(Chrome, Safari 3)"'.
The fault may also be on my side 🙁. Maybe I copied your changes wrong. If you still have time and desire, could you please try to fix the error (if the error message is also visible to you)? Otherwise, I would merge your PR and create a new bug in the Issue tab.
Thanks in advance 🙏 - NH

Heres a screenshot of the error:

If you have questions about the translation of the error message, feel free to ask me 🙂

Heres a screenshot of the error:
If you have questions about the translation of the error message, feel free to ask me 🙂

Hi @N3v1 , that seems to be a warning rather than an error. Trying to remove the warning breaks the code on my side, I would suggest just leaving it as it is.

Hi @anubhav1206, 👋
i will now merge your PR. Thanks for your help. I'm sorry that I may have caused you any inconvenience. I'm still new to web development. Thank you very much - NH


Asset downloads: enforce the usage of a commit hash or tag zip archive (and thus single repo per asset)

We currently took a loose approach for assets, allowing people to give a directly URL to a potentially self-hosted zip archive. Reviewing the currently approved assets show a big variety of usages, but it's quickly become hard to maintain and encourages bad practices (like uploading static zip archives directly in the git repo to download as raw files).
After discussing it with @reduz on tonight's IRC meeting, we thought that it would be better to go back to the initial design:

Asset submitters should have to give the following information:

GitHub repository URL, e.g. https://github.com/KOBUGE-Games/godot-logger (or other source hosting providers if we can handle their respective APIs)
Commit hash or git tag, e.g. 74c2903670b30ec36ad306006f82d1f27dd8757b
The download zip would thus get automatically generated (e.g. https://github.com/KOBUGE-Games/godot-logger/archive/74c2903670b30ec36ad306006f82d1f27dd8757b.zip), we don't ask for a download URL

This means 1 GitHub (or other supported provider) repo == 1 addon. It won't be possible to version several plugins in the same repo (or they would be distributed all at once :p).
Security wise, it's much better, as we can check the contents of the git repo visually without having to download a Dropbox-hosted zip archive, extract it and check the contents.

It should be relatively straightforward to implement as most of the logic is there already, and building the URL for the GitHub API is easy.
Question: what other providers do we want to support if any, and how?

For the discussion part -- we might want bitbucket and gitlab support too.

@bojidar-bg Sounds good. I'd propose to design it in a way that can easily be extended with PRs, e.g. have a file which describes for each provider the URL to download archives, e.g.:

GitHub: $REPO_URL/archive/$hash.zip (example)
BitBucket: $REPO_URL/get/$hash.zip (example)
GitLab: $REPO_URL/repository/archive.zip?ref=$hash (example)
Gogs: $REPO_URL/archive/$hash.zip (example)
cgit: $REPO_URL/snapshot/$hash.zip (example)

We'd have to test of course that those URLs resolve to a file that Godot is presently capable of downloading.

As we're going to proceed with this feature quite soon, some asset devs will have to adapt their workflow accordingly. Mass pinging the concerned users: @kubecz3k @HorseMatthew @LeonardMeagher2 @marcosbitetti @GlaDOSik @Xydium @SuperUserNameMan @Geequlim
Here are the changes that would affect your workflow (and potentially require an update of your assets once the new feature has been implemented):

Self-made zip archives will no longer be accepted, so: no Dropbox, no zip archive committed directly in your git repo (phew, we've gained our place in the Torvalds paradise with this decision, git gurus can sleep in peace), no custom file hosting or archive uploaded to GitHub releases.
Instead, only archives generated automatically from a given commit hash or git tag will be accepted. This implies that:

Your git repo should only contain the asset you want to publish, with a format that makes it usable out of the box when unzipped in res:// (example that works: https://github.com/Zylann/godot_terrain_plugin)
In the assetlib, you will have to specify instead of the download URL, either a git commit hash (e.g. e836ef5bdc9d6948d25a4dd0b32f77d7722b515b, from which the assetlib would generate the download URL: https://github.com/Zylann/godot_terrain_plugin/archive/e836ef5bdc9d6948d25a4dd0b32f77d7722b515b.zip) or, better, a git tag (generated with e.g. git tag v1.0 and git push --tags, or via the GitHub interface - with a git tag as v1.0, the generated download URL would be https://github.com/Zylann/godot_terrain_plugin/archive/v1.0.zip).

The master branch will not be a valid hash (though it can be downloaded the same way), as the archive it produces changes at each new commit, and thus would render the asset invalid (to be valid, it needs to have a sha256sum that matches the one added initially during moderation).

Sorry for the trouble, but it should make things much simpler for maintainers in the long run (and more secure and reliable).

Re-mass-pinging everyone again, @kubecz3k @HorseMatthew @LeonardMeagher2 @marcosbitetti @GlaDOSik @Xydium @SuperUserNameMan @Geequlim, this is now closed, and it would probably be put live soon. Thus beware.

The change has now been deployed, so all the contributors pinged above have potentially something to edit in their assets. For some it only implies giving the proper hash (commit hash or git tag instead of master), while for others it might require you to change a bit how you manage your assets in git (split them in separate repos, etc.).

Oh no. And I was so happy I can have centralized repo for all my assets. Well, my Wireframe tool 2.0 is soon to be out so I'll do the new repo for it.


Added option for key prefix in a bucket

Updated the Readme.md to have link to the godoc
Updated some of the documentation
Updated the code to accept a prefix for keys in a backwards compatible way

Coverage increased (+6.9%) to 88.889% when pulling 0cc6fedd90749e806f0f7111cabb9ce74cf27e56 on artktec:key-prefix into c8174e5dd41077ba9d0e4697ddecef3d27f32db2 on danilobuerger:master.

Coverage increased (+6.9%) to 88.889% when pulling 0cc6fedd90749e806f0f7111cabb9ce74cf27e56 on artktec:key-prefix into c8174e5dd41077ba9d0e4697ddecef3d27f32db2 on danilobuerger:master.

Thanks for the PR @njones
I cherry-picked the travis / godoc commits so we can focus on the actual topic of the PR.
Adding a prefix is a great idea. However, I don't want to introduce functional options here as it seems overkill for such a small library. So if you would add Prefix to the Cache struct, I would be happy to merge it.

Made the requested changes...

I've made an update with the changes requested, except the Prefix as I believe it's correct.

Thanks!


Adding sandbox layout to repo list page

Introduces sandbox layout and routing to Gitness.
Temporarily duplicates files and routes to ensure stability during the transition.
All pages using Sandbox layouts will be added under the sandbox route in App.tsx. The aim is to migrate all pages to sandbox layout asap.
This PR applies sandbox layout to repo-list-page. After review by @joetaylor-harness, we can roll out the layout to other pages.
The focus of this PR is to ensure proper implementation of the sandbox layout.

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


Track expense - Track expense option is not displayed in fab.

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.64-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/4501563
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:

Go to https://staging.new.expensify.com/home
Login as new expensifail user
Tap fab

Expected Result:
Track expense option must be displayed in fab.
Actual Result:
Track expense option is not displayed in fab.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/6e871e90-005a-4e8d-b21b-6d36d11257e0

View all open jobs on GitHub

@jliexpensify I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.

We think this issue might be related to the #vip-vsb.

cc @thienlnam and @hungvu193 - you worked on that PR.

This is BE issue, when self DM is not added to allReports. We already mentioned it here while working on that PR
https://github.com/Expensify/App/pull/40409#discussion_r1572659123
cc @thienlnam

Based on this comment it seems like https://github.com/Expensify/App/pull/40409#discussion_r1572659123 this is expected and it will be handled in BE, assigning Jack

Yup, this is anticipated. We'll need to make a change to always return the self-DM on OpenApp

Not overdue. @thienlnam want to keep this as Daily or change the priority?

This should be resolved with https://github.com/Expensify/Auth/pull/10668, but we're running into some performance problems so we may need to revert it


Flame gesture API does not support multi touch

Currently, Flame Gesture API (TapDetector and etc) don't work with multiple touches, I think this is an important feature and needs to be supported by Flame.
I have created a branch (gesture-detector-multitaps) on Flame with a little game to expose the problem, you can find the example here: doc/examples/gestures/lib/main_multitap.dart
@luanpotter @renancaraujo I am researching on how to make this work but still haven't much idea on how to make it happen, I would love to have some help on this issue.

Alright, I have found a possible solution, this will be a lot of work, but will make Flame gesture api way powerful, we can use RawGestureDetector to combine the detectors that supports multi handlers, something like that:
    return RawGestureDetector(
        gestures: {
          MultiTapGestureRecognizer: GestureRecognizerFactoryWithHandlers<MultiTapGestureRecognizer>(
            () => MultiTapGestureRecognizer(),
            (MultiTapGestureRecognizer instance) {
              if (game is TapDetector) {
                final tapDetectorGame = game as TapDetector;
                instance.onTapDown = (_, TapDownDetails d) => tapDetectorGame.onTapDown(d);
                instance.onTapUp = (_, TapUpDetails d) => tapDetectorGame.onTapUp(d);
              }
            }
          )
        },

I will continue to work on this.

Is there anyway to implement MultiDragGestureRecognizer at the moment, or is that a lost cause? There's nothing built-in in Flame that supports any multitouch at the moment.
I was able to attach an ImmediateMultiDragGestureRecognizer with flame\util, but its API is very bizarre, not the conventional onDragStart. I have no idea what to do with createNewPointerState(PointerDownEvent event) → _ImmediatePointerState`.
MultiTapRecognizer doesn't support drag and PanRecognizer supports only single touch.
Any ETA on when a functional multitouch support would be implemented?

Multi tap and multi drag support added on 0.21.0


The authentication chain terminates on first failure.

Steps to reproduce
In v 2.1.0 of @feathersjs/authentication , when multiple authentication methods are used in a chain
module.exports = {
    before: {
        all: [authenticate(['jwt', 'anon'])],

the chain will terminate on the first failure.  In the example above, anon will never be tested.
An (unacceptable) suggested solution is shown in feathersjs/authentication#622 .

Feathers v4 authentication requires a strategy parameter but also allows to fully customize the authentication service to any other requirement.
Please see the migration guide for more information. Closing this issue in order to archive this repository. Related issues can be opened at the new code location in the Feathers main repository.


Jenkins dataset (global biodiversity maps)

https://biodiversitymapping.org/index.php/download/
This has a bunch of maps for different taxonomic groups

This needs to be re-projected in WGS84 as it is using an Eckert projection, not sure if we want to do this here or in other packages who consume the data.

GeoData.jl has reproject


Feature/david/contrast colors change

Task/Issue URL: https://app.asana.com/0/715106103902962/874107794298116
Product Review URL: https://app.asana.com/0/1142021229838617/1176459710279645
Description:
In a handful of scenarios in our Android app we fail to meet WCAG (Web Content Accessibility Guidelines https://www.w3.org/WAI/standards-guidelines/wcag/) standards. This results in a problematic user experience for people with visual impairments and suboptimal experience for everyone else.
This PR fixes those issues, using new colors that meet the above mentioned guidelines
Steps to test this PR:

Open the app in Light Mode and Dark Mode
Navigate through the app and check the new colors as per https://www.figma.com/file/DTS9R8KSj29PANxUUCdfWJ/Optimize-Themes?node-id=0%3A1

Internal references:
Software Engineering Expectations
Technical Design Template

@marcosholgado  I've removed the changes to the Dialogs and it's now good for re-review. Addressed the comments you had too. Thanks for the review!

Thanks for the review @marcosholgado !


Component rule for JavaFX components

JavaFX published OS/Arch specific Jars with classifiers. Would be great to add the corresponding variants with attributes, like in this example:
https://docs.gradle.org/current/userguide/component_metadata_rules.html#adding_variants_for_native_jars
Then it's straight forward to consume without using a JavaFX specific plugin or anything like that.
The rule looks roughly like this (need to double check OS/Arch values):
@CacheableRule
abstract class JavaFXMetadataRules : ComponentMetadataRule {
    data class NativeVariant(val os: String, val arch: String, val classifier: String)

    private val nativeVariants = listOf(
        NativeVariant(LINUX,   "x86-64",  "linux"),
        NativeVariant(LINUX,   "aarch64",  "linux-aarch64"),
        NativeVariant(LINUX,   "aarch64-monocle",  "linux-aarch64-monocle"),
        NativeVariant(LINUX,   "arm32-monocle",  "linux-arm32-monocle"),
        NativeVariant(LINUX,   "monocle",  "linux-monocle"),
        NativeVariant(MACOS,   "x86-64", "mac"),
        NativeVariant(MACOS,   "aarch64", "mac-aarch64"),
        NativeVariant(MACOS,   "monocle", "mac-monocle"),
        NativeVariant(MACOS,   "aarch64-monocle", "mac-aarch64-monocle"),
        NativeVariant(WINDOWS, "x86-64", "win"),
        NativeVariant(WINDOWS, "monocle", "win-monocle"),
        NativeVariant(WINDOWS, "x86", "win-x86"),
        NativeVariant(WINDOWS, "x86-monocle", "win-x86-monocle"),
    )

    @get:Inject
    abstract val objects: ObjectFactory

    override fun execute(context: ComponentMetadataContext) {
        nativeVariants.forEach { variantDefinition ->
            val name = context.details.id.name
            val version = context.details.id.version

            context.details.addVariant("${variantDefinition.classifier}Compile", "compile") {
                attributes {
                    attributes.attribute(OPERATING_SYSTEM_ATTRIBUTE, objects.named(variantDefinition.os))
                    attributes.attribute(ARCHITECTURE_ATTRIBUTE, objects.named(variantDefinition.arch))
                }
                withFiles {
                    removeAllFiles()
                    addFile("${name}-${version}-${variantDefinition.classifier}.jar")
                }
            }
            context.details.addVariant("${variantDefinition.classifier}Runtime", "runtime") {
                attributes {
                    attributes.attribute(OPERATING_SYSTEM_ATTRIBUTE, objects.named(variantDefinition.os))
                    attributes.attribute(ARCHITECTURE_ATTRIBUTE, objects.named(variantDefinition.arch))
                }
                withFiles {
                    removeAllFiles()
                    addFile("${name}-${version}-${variantDefinition.classifier}.jar")
                }
            }
        }
    }
}

Is this something the maintainers of JavaFX need to do?

Is this something the maintainers of JavaFX need to do?

Eventually yes. JavaFX could publish Gradle Metadata with the information I have sketched in this issue. Then Gradle users can depend on JavaFX components (or libraries building on top of JavaFX) without additional plugins and select the "JavaFX Platform Variant" by using the OPERATING_SYSTEM_ATTRIBUTE and ARCHITECTURE_ATTRIBUTE  attributes.
I now think a good start would be to add the rule to the official JavaFX plugin and have that use the attributes as well internally. Then the metadata publishing in JavaFX itself could be proposed as a next step.
I made a proposal: https://github.com/gradle/gradle/issues/26144

@johanvos Can you have a look?

Thanks @jjohannes and @nlisker . https://github.com/openjfx/javafx-gradle-plugin/pull/151 is merged. A potential next step is to see if we should do this in the openjfx repository itself.

Closing as this rule has been directly added to org.openjfx.javafxplugin version 0.1.0 - (https://github.com/openjfx/javafx-gradle-plugin/pull/151) and Gradle metadata might potentially be published by JavaFX at some point in the future (https://github.com/openjdk/jfx/pull/1232).


Create and implement BDD features folder

Create the BDD features/steps folders and a.feature file and accompanying steps.py files to test your RESTful API from the
outside in.

Done as part of other issues


Moving storage (3), iam (2) resources to v1beta1 version

Moving storage 3 resources to v1beta1 version

[ ] google_storage_hmac_key
[ ] google_storage_notification
[ ] google_storage_transfer_job

Moving iam 2 resources to v1beta1 version

[ ] google_iam_workforce_pool
[ ] google_iam_workforce_pool_provider

google_storage_notification - blocked due to account doesn't have permissions for publish message.


Issues within OpenFL

I am able to successfully run the test files in both Neko and Apache. I am now attempting to use PushState from within OpenFL using PushState.push. The path updates in the browser but the listener only fires when I call PushState.push. When I use the browser back button the browser updates its address but the listener does not fire. Any ideas about that? Do I need to use @:expose or something?

@jasononeil Any thoughts? :)

This is fixed now in this merge request. #12


fix typo in C1.md

I think "refuned" should probably read "refunded".
Sorry, I was reading your article and I noticed this minor typo. :)
Great work! I'll be sure to read the rest.

@maxwellfoley3 not sure if this gonna impact any pending commits you may have, but seems ok

Thank you @minhlai ! Great catch!


sophus: init at 1.22.10

Description of changes
Add the header-only C++ library Sophus. This is a small library built atop Eigen that helps with working with Lie group representations of 2D and 3D transformations.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Hm, the expression as written fails to build on our Linux stdenv… clang-only?

Shoot, you're right, thank you for catching this! It's something between fmt and Sophus in a Sophus test, but I didn't see a fix after a quick look. Will revisit adding this to nixpkgs in the future.

I think I found a work around for this, maybe try to build with the cmake flags -DBUILD_SOPHUS_TESTS=OFF and -DBUILD_SOPHUS_EXAMPLES=OFF.


Add the new discord link

Related Issue

Information about the related issue

Change the Discord Link
Describe the changes you've made
This pull request updates the old Discord link to the new one
Type of change
What sort of change have you made:

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

How Has This Been Tested?

Checklist

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[ ] I have commented on my code, particularly wherever it was hard to understand.
[ ] I have made corresponding changes to the documentation.
[ ] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] Any dependent changes have been merged and published in downstream modules.

Screenshots (if applicable)

Original
Updated

original screenshot
updated screenshot

Code of Conduct

[x] I agree to follow this project's Code of Conduct

This pull request is connected to issue #24


Consistency in param naming Athena

Enhancement
[BREAKING CHANGE] Consistency in param naming Athena
Detail

Changing ctas_database_name to match ctas_database used elsewhere in the Athena module.

Relates

#1612

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: GitHubLoadTests5656BB24-s6u9F3qN9oFy
Commit ID: b7895ea7b3769e312ea8a3ae3fc3e48e1e0de2cf
Result: FAILED
Build Logs (available for 30 days)

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

AWS CodeBuild CI Report

CodeBuild project: GitHubStandardCodeBuild8C06-llutOAimTATs
Commit ID: b7895ea7b3769e312ea8a3ae3fc3e48e1e0de2cf
Result: FAILED
Build Logs (available for 30 days)

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

AWS CodeBuild CI Report

CodeBuild project: GitHubDistributedCodeBuild6-jWcl5DLmvupS
Commit ID: b7895ea7b3769e312ea8a3ae3fc3e48e1e0de2cf
Result: FAILED
Build Logs (available for 30 days)

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

AWS CodeBuild CI Report

CodeBuild project: GitHubDistributedCodeBuild6-jWcl5DLmvupS
Commit ID: fbe9e3f39dafc32be614c7edbc5564d8382ed873
Result: SUCCEEDED
Build Logs (available for 30 days)

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

AWS CodeBuild CI Report

CodeBuild project: GitHubLoadTests5656BB24-s6u9F3qN9oFy
Commit ID: fbe9e3f39dafc32be614c7edbc5564d8382ed873
Result: SUCCEEDED
Build Logs (available for 30 days)

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

AWS CodeBuild CI Report

CodeBuild project: GitHubStandardCodeBuild8C06-llutOAimTATs
Commit ID: fbe9e3f39dafc32be614c7edbc5564d8382ed873
Result: FAILED
Build Logs (available for 30 days)

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

AWS CodeBuild CI Report

CodeBuild project: GitHubDistributedCodeBuild6-jWcl5DLmvupS
Commit ID: 71b79feae85d79d2e5ec44693ab5d360c764af3f
Result: SUCCEEDED
Build Logs (available for 30 days)

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

AWS CodeBuild CI Report

CodeBuild project: GitHubLoadTests5656BB24-s6u9F3qN9oFy
Commit ID: 71b79feae85d79d2e5ec44693ab5d360c764af3f
Result: SUCCEEDED
Build Logs (available for 30 days)

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

AWS CodeBuild CI Report

CodeBuild project: GitHubStandardCodeBuild8C06-llutOAimTATs
Commit ID: 71b79feae85d79d2e5ec44693ab5d360c764af3f
Result: SUCCEEDED
Build Logs (available for 30 days)

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


[Feature] Prints fees when using snarkos developer.

This PR prints out the minimum deployment and execution cost when using snarkos developer deploy and snarkos developer execute respectively.

These commands have been removed in #2815