feat(clerk-js): Allow customization of OrganizationProfile through OrganizationSwitcher

Description
This PR

introduces organizatinonProfileProps as prop of <OrganizationSwitcher />. This allows for the customization of OrganizationProfile when it is opened through the switcher.
introduces appearance in userProfileProps of <UserButton/>.
deprecates the usage of appearance.userProfile of <UserButton/>.

After these changes a developer can customize the Profiles like this:
      <OrganizationSwitcher
        organizationProfileProps={{
          appearance: {
            elements: {
              formButtonPrimary: {
                background: 'black',
              },
            },
          },
        }}
      />

      <UserButton
        userProfileProps={{
          appearance: {
            elements: {
              formButtonPrimary: {
                background: 'orange',
              },
            },
          },
        }}
      />

Deprecated
      <UserButton
        appearance={{
          userProfile: {
            elements: {
              formButtonPrimary: {
                background: 'black',
              },
            },
          },
        }}
      />

Fixes #1534
Checklist

[x] npm test runs as expected.
[x] npm run build runs as expected.
[ ] (If applicable) JSDoc comments have been added or updated for any package exports
[x] (If applicable) Documentation has been updated
https://github.com/clerkinc/clerk-docs/pull/367

Type of change

[ ] 🐛 Bug fix
[x] 🌟 New feature
[ ] 🔨 Breaking change
[ ] 📖 Refactoring / dependency upgrade / documentation
[ ] other:

Packages affected

[x] @clerk/clerk-js
[ ] @clerk/clerk-react
[ ] @clerk/nextjs
[ ] @clerk/remix
[ ] @clerk/types
[ ] @clerk/themes
[ ] @clerk/localizations
[ ] @clerk/clerk-expo
[ ] @clerk/backend
[ ] @clerk/clerk-sdk-node
[ ] @clerk/shared
[ ] @clerk/fastify
[ ] @clerk/chrome-extension
[ ] gatsby-plugin-clerk
[ ] build/tooling/chore

This PR has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.


Fix linker error in debug builds.

@swift-ci Please test

Merging to fix the build

Build failed
Swift Test Linux Platform
Git Sha - 5d8405e0fd12cc486cb9be4a5776f0c5f9471f93

Build failed
Swift Test OS X Platform
Git Sha - 5d8405e0fd12cc486cb9be4a5776f0c5f9471f93


pthreads: prefer -pthread during linking

The documentation of GCC and Clang state that -pthread is to be used during compiling and linking for POSIX threads support. Moreover, there is no pthread library in the Android NDK (try to search for libpthread.*) meaning there will be linker errors when using -lpthread (this flag is given in the pkg-config file, at least for arm-linux-androideabi targets).
Currently, -pthread is not used when compiling C code and -lpthread is regularly used during linking, possibly leading to errors.
This pull request changes only the linker flags for most compilers including the Intel Fortran compiler (pthread documentation) and the EXTRALIBS variable; the linker flags for the PathScale and the Open64 compilers remain unchanged because their manuals (PathScale manual from 2008, Open64 4.5.2 manual) do not mention this flag.
See the linked Google Groups threads for a discussion on whether -lpthread should work during linking:
https://groups.google.com/forum/#!topic/comp.programming.threads/NCEpG0EOCCY

POSIX votes for -lpthread and pulling macros/prototypes via headers. There is also not-so-portable -mthread and -pthreads in some niche compilers.

Reiterating around your description - you are trying to work around absence of posix pthreads in Android NDK by making a global change, can you make it with ifdef-s addressing the issue only?

Looks good to me in principle (bit of googling found https://github.com/google/protobuf/issues/1373 for instance that confirms the Android case), though making it an exception for Android alone may be safer than changing the default for most platforms (and probably requires changes to fewer files)

MKL and ACML link parameters changed are dead code and would better be carefully removed.

The Groups thread linked above confirms that POSIX requires -lpthread but in practice, the very popular compilers GCC and Clang use -pthread. The goal of this patch is to adopt OpenBLAS to the functionality offered by these compilers but POSIX-conformance is definitely desirable.

It should be ifdef android in Makefile.system and os.cmake (with proper capitalisation), other changes dont change anything for any android build.
e.g. iOS/XCode has just blank libpthread.so while actual functions are in C library...

Should we merge this patch?

No, it is 'sed s' 10x where 2x if(def) should be.

@xianyi @martin-frbg Please decide which variant you want and I will modify the patch as necessary.

if(ANDROID__NDK) in cmake
ifdef ANDROID in Makefile
And dont touch other stuff that is not for android.

@brada4 lol. love your bitchy tone.

On re-reading this, it appears to me that the only actual bug to be fixed in this context might be that cmake/os.cmake is much less careful than Makefile.system in avoiding to add -lpthread on platforms that do not support this.
I suspect this is where the erroneous entry in the pkgconfig file comes from, and if that turns out to be the only actual issue it could be fixed by copying the exclusion of  CYGWIN_NT and ANDROID targets
from lines 304,305 of Makefile.system onto line 80 of cmake/os.cmake.
Or is the "-pthread" option actually mandatory for the code to function on Android (and not implied by any other option already used) ?
Changing an option that "appeared to work just fine so far" over a purely semantic issue may bring the risk of breaking compilation on some system, however exotic or outdated, that worked just fine in the past without gaining anything.

Or is the "-pthread" option actually mandatory for the code to function on Android (and not implied by any other option already used) ?

Google dropped GCC support for Android so I will only discuss building code with Clang. The Clang command line reference states the following for -pthread:

Support POSIX threads in generated code
I read this to mean that the -pthread flag must be provided at compile and at link time on all platforms.

Yet Android's libc called Bionic contains a POSIX threads implementation (see, for example, all files starting with pthread_ in the Bionic libc/bionic source code folder). Hence, you can probably get away without using -pthread at link time.
Anyway, I just found out that the root cause of the problem is somewhere else. In How to build OpenBLAS for Android the installation command is given as
make PREFIX=/path/to/install-dir install

and this is wrong because Make does not know about your cross-compilation attempt. Thus, the pkg-config file will contain the compiler and linker flags for the host system and not for the target system  (Make may find out that you compile for Android if the bin folder of the standalone toolchain comes first in the environment variable PATH). If I use the same flags on the Make command line during compilation and installation, the issue goes away. In fact, Paul Mustiere fixed the -lpthread linker problem a year ago in commit 7f7d04dcd27b76b4808c99643e84fe81697eda2e.
Another way to highlight this issue is to cross-compile for Android and set ARM_SOFTFP_ABI=1 for compilation but not for the installation. On my Linux system, the pkg-config linker flags contain -Wl,-lm_hard if I use
make PREFIX=/path/to/install-dir install

instead of
make ARM_SOFTFP_ABI=1 PREFIX=/path/to/install-dir install

The proper solution for this problem is to update the Android build instructions.


Patroni NixOS tests time out

Describe the bug
The NixOS test for patroni time out on hydra, unnecessarily delaying the channel by about 9 hours.
Steps To Reproduce
e.g.
https://hydra.nixos.org/build/199759381
https://hydra.nixos.org/build/199760600
Expected behavior
Tests should finish promptly or fail, not stall the channel.
Notify maintainers
@phfroidmont
Metadata
Happens on both nixos-22.11 and nixos-unstable.

I will take a look today

I can't reproduce the problem locally, even using the exact commit from one of your examples works:
nix build github:nixos/nixpkgs/c9538a9b7074925ac117835c775fb81d0e7f98fa#nixosTests.patroni

Now by looking at hydra logs, my understanding is that the test waits for the patroni service to be started to consider the VM booted.
To address the timeout issue, I think configuring the patroni service to fail after x attempts should do the trick.
I would like to be sure and also fix the actual root cause but I don't know how without being able to reproduce.
Any idea on how I can investigate this further?

#202567 should fix the issue


[es-build] Browser directory created in outputPath

Command
build
Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
I have an Angular application configure as below:
"outputPath": "../MyNetApplication/wwwroot",
When using webpack build, the compiled files of the Angular application went directly in wwwroot so I could publish my .NET application and it worked directly.
I tried to switch to es-build, however, after the build, the compiled files of the Angular application end up into "../MyNetApplication/wwwroot/browser" which prevents me from publishing the .NET application as this would not work.
It seems weird to me that the outputPath settings of angular.json is only partially honored.
According to me, this settings should be used and not altered during the compilation.
Minimal Reproduction
Just create a new Angular application using angular/cli 17 and run ng build.
The files will be located under dist/ng17/browser.
Exception or Error
No response
Your Environment
Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1700.0
@angular-devkit/build-angular   17.0.0
@angular-devkit/core            17.0.0
@angular-devkit/schematics      17.0.0
@angular/cli                    17.0.0
@schematics/angular             17.0.0
rxjs                            7.8.1
typescript                      5.2.2
zone.js                         0.14.2

Anything else relevant?
No response

Closing as duplicate of #26304, seems to have been submitted twice by accident.


Is this supported with the REST version of Speech-to-text ?

This page only lists examples for the SDK version and there are no mentions of this feature on the REST API page. I really need this feature with the REST version, is it supported ?

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

ID: 6c3e4d66-e6fd-3ed1-f505-875064467f00
Version Independent ID: 345501ca-4487-ff5a-b681-449508433cff
Content: Phrase Lists - Speech Services
Content Source: articles/cognitive-services/Speech-Service/how-to-phrase-lists.md
Service: cognitive-services
Sub-service: speech-service
GitHub Login: @rhurey
Microsoft Alias: rhurey

@paul-arthurthiery Thank you for your feedback, we will investigate it and get back to you soon.

@paul-arthurthiery Hi, I am sorry, this is not supported with REST. I will forward your feedback to product team, at the meantime, I would highly recommend you post your  feedback to UserVoice channel, the engineering team will consider your idea seriously. Thank you.
We will now proceed to close this thread. If there are further questions regarding this matter, please respond here and @YutongTie-MSFT and we will gladly continue the discussion.


How to setup Ackee

I really would like to know how to setup Ackee, cuz i have no idea how lol

You can find everything you need in the README.


vendor patches does not work for XML or PHTML files

To be more precise, new files are not found if they end with "L" because of the way rtrim works.
Proposed fix in src/Finder/OldToNewFilesFinder.php:
-            $newFilePath = rtrim($oldFileInfo->getRealPath(), '.old');
+            $newFilePath = substr($oldFileInfo->getRealPath(), 0, -4);

Hi, thanks for reporting.
Could you submit PR with failing fixture and the fix? Thank you

I see that it is already resolved in PR #3607 but apparently the change did not make it into a release of symplify/vendor-patches yet. Closed as duplicate

I checked again, it does fix it but when I tried this, it was in a PHP 7.4 environment so that composer did not install the latest PHP 8 only release

Ah, I see :+1:


More compatible macosx wheels

This PR sets up the macOS wheel  builds (and CI tests for verification purposes) to explicitly set the macos compatibility version to 10.9 to ensure all mac users can rely on precompiled binaries from pypi instead of users of older version of macOS having to compile the package from sdist.
Fixes #62

Coverage decreased (-0.03%) to 84.972% when pulling 43fb31ed5fa9419eebca83abd83a7481183e5202 on mtreinish:more-compatible-macosx-wheels into 9ba6884e79d0eb5a83563359007adf3253c10d24 on Qiskit:master.


Multi-GPU parallelisation support with MPI

@arnauqb this issue is in reference to JOSS review: https://github.com/openjournals/joss-reviews/issues/5776
The submitted paper claims a feature on "Multi-GPU parallelisation support with MPI".
Is there an example of this in the repository? In particular, it would be nice to have a plot (either in the paper or on the repo) which shows benchmarks for parallelisation with details of the system used.

Added this example: https://github.com/arnauqb/blackbirds/blob/joss_review/examples/gpu_parallelization.md with a basic benchmark


Is an AppService Plan required to add a Managed Identity to Azure Functions?

I'm trying to keep credentials out of my (python) Azure Function. I've stored API keys for the function to make external calls in Azure Key Vault.
In order to authenticate my function to Key Vault to get these keys, I need to supply a Client_ID, Client_Secret and Tenant_ID in my code OR use a Managed Identity.
Do I need to setup an AppService Plan to use a Managed Identity for my Azure Function? Its unclear to me from reading the docs.
Thank you

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

ID: f57d3177-ce9e-ebfa-03a7-45beb8ca3212
Version Independent ID: a59df23d-f3ac-9809-61d3-5741e31f020a
Content: Managed identities - Azure App Service
Content Source: articles/app-service/overview-managed-identity.md
Service: app-service
GitHub Login: @mattchenderson
Microsoft Alias: mahender

@SeaDude Thanks for the feedback. We are actively investigating and will get back to you soon.

@SeaDude, Apologies for the delay!
Managed identities for Azure resources is a feature of Azure Active Directory and both Azure App Service and Azure Functions are supported service. The Azure doc you're referring to outlines steps for both Azure App Service WebApp and Azure Function; so it should work with either plans.
A managed identity from Azure Active Directory (Azure AD) allows your app to easily access other Azure AD-protected resources  and that identity is managed by the Azure platform.
Just to clarify, Azure Functions has three kinds of pricing plans. You can choose the one that best fits your needs: Consumption plan, Premium plan, Dedicated (App Service) plan.  The hosting plan defines the features for your Function App, I would suggest you take a look into this Azure document 'Azure Functions scale and hosting' for more details on this topic.

Thank you. I was getting hung up in thinking that an "App Service" plan was a premium feature. What I realized after creating an Azure Function, is that the "Consumption Plan" is a form of "App Service" plan.
Makes sense now.


this.fetchUser is not a function

I am running Node 12.x.
On running, it returns the error:
(node:137) UnhandledPromiseRejectionWarning: TypeError: this.fetchUser is not a function
This is the code I've tried for testing:
/* eslint-disable no-console */
const commando = require('discord.js-commando');
const path = require('path');
const oneLine = require('common-tags').oneLine;
const sqlite = require('sqlite');
const token = token

const client = new commando.Client({
	owner: 'ownerid',
	commandPrefix: 'prefix'
});

client
	.on('error', console.error)
	.on('warn', console.warn)
	.on('debug', console.log)
	.on('ready', () => {
		console.log(`Client ready; logged in as ${client.user.username}#${client.user.discriminator} (${client.user.id})`);
	})
	.on('disconnect', () => { console.warn('Disconnected!'); })
	.on('reconnecting', () => { console.warn('Reconnecting...'); })
	.on('commandError', (cmd, err) => {
		if(err instanceof commando.FriendlyError) return;
		console.error(`Error in command ${cmd.groupID}:${cmd.memberName}`, err);
	})
	.on('commandBlocked', (msg, reason) => {
		console.log(oneLine`
			Command ${msg.command ? `${msg.command.groupID}:${msg.command.memberName}` : ''}
			blocked; ${reason}
		`);
	})
	.on('commandPrefixChange', (guild, prefix) => {
		console.log(oneLine`
			Prefix ${prefix === '' ? 'removed' : `changed to ${prefix || 'the default'}`}
			${guild ? `in guild ${guild.name} (${guild.id})` : 'globally'}.
		`);
	})
	.on('commandStatusChange', (guild, command, enabled) => {
		console.log(oneLine`
			Command ${command.groupID}:${command.memberName}
			${enabled ? 'enabled' : 'disabled'}
			${guild ? `in guild ${guild.name} (${guild.id})` : 'globally'}.
		`);
	})
	.on('groupStatusChange', (guild, group, enabled) => {
		console.log(oneLine`
			Group ${group.id}
			${enabled ? 'enabled' : 'disabled'}
			${guild ? `in guild ${guild.name} (${guild.id})` : 'globally'}.
		`);
	});

client.setProvider(
	sqlite.open(path.join(__dirname, 'database.sqlite3')).then(db => new commando.SQLiteProvider(db))
).catch(console.error);

client.registry
	.registerGroup('math', 'Math')
	.registerDefaults()
//	.registerTypesIn(path.join(__dirname, 'types'))
	//.registerCommandsIn(path.join(__dirname, 'commands'));

client.login(token);

Please read this, it is now <Client>.users.fetch()

It appears that the most recent package available for install is actually quite old. The stuff you're using is from v0.10.0. While it is the latest available for installation, it's old, released on April 29, 2018. The master branch is actually up to date, so try to manually clone by using git clone https://github.com/discordjs/Commando.git ., making sure to clean out your node_modules/discord.js-commando folder beforehand and running the git clone command in that directory.

This seemed to fix my issue but why is the npm install currently giving out this error?
(note: I ran into this error when doing the getting started guild for Commando in the discord.js guide)

This fixed my issue too. Happened when I insalled it via npm, but working now, Thank you!

This seemed to fix my issue but why is the npm install currently giving out this error?
(note: I ran into this error when doing the getting started guild for Commando in the discord.js guide)

This is most likely due to the fact that Discord.js moved to v12, which brought about changes in how certain methods are handled. Commando's repo is updated to reflect these changes, but the release has not been made yet, therefore doing npm install discord.js-commando will give you the older version which will conflict with discord.js v12

@jakehead20  but it gives me errors when I want to host my bot it say's
Error: Cannot find module 'Discord.js-commando'


Azure Platform Support

Support for the Azure platform with a few metadata attributes extracted as proof of functionality.
Checked that retrieved metadata and locality are as expected in comparison to the API curl request described on https://docs.microsoft.com/en-us/azure/virtual-machines/windows/instance-metadata-service
Metadata() is incomplete at the moment, but azure_name, azure_location, and azure_vm_id are basic attributes that have been added in. The others are parsed from the tags field. An example of the metadata:
  "PLATFORM_METADATA": {
   "azure_vm_id": "5376324d-0cd7-4e3a-b350-d967eb74d097",
   "azure_location": "centralus",
   "orchestrator": "Kubernetes",
   "creationSource": "aks-aks-nodepool1-34440236-vmss",
   "azure_name": "aks-nodepool1-34440236-vmss_0",
   "poolName": "nodepool1",
   "resourceNameSuffix": "34440236",
   "aksEngineVersion": "v0.47.0-aks-gomod-63-aks"
  },

[X ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

Hi @justinwei2. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test

@justinwei2: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
c129eda9e4ace06c214d417f4bcc7e2d5a93b8ee
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@justinwei2: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
c129eda9e4ace06c214d417f4bcc7e2d5a93b8ee
link
/test lint_istio

integ-pilot-k8s-tests_istio
c129eda9e4ace06c214d417f4bcc7e2d5a93b8ee
link
/test integ-pilot-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@justinwei2: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-k8s-tests_istio
c129eda9e4ace06c214d417f4bcc7e2d5a93b8ee
link
/test integ-pilot-k8s-tests_istio

lint_istio
3da11f157281a39ff77e913009f536d89910048f
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@justinwei2: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
f98ad606a92bde48a86d1f358bcf25a58756072f
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@justinwei2 can you add an example PLATFORM_METADATA section to your PR description that highlights what that will look like with this PR when running on AKS?

@justinwei2: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

unit-tests_istio
62e6d3e2f4558f12de663d9d937a6bb7cccf14d9
link
/test unit-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@justinwei2: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

analyze-tests_istio
0ec92aede71dabe90bc6c95186b80e30cb41097f
link
/test analyze-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@justinwei2: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

analyze-tests_istio
ae6685d7107b182e415c0a9107be99e144c3b1e9
link
/test analyze-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@justinwei2: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

analyze-tests_istio
ae6685d7107b182e415c0a9107be99e144c3b1e9
link
/test analyze-tests_istio

integ-multicluster-k8s-tests_istio
ae6685d7107b182e415c0a9107be99e144c3b1e9
link
/test integ-multicluster-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@bianpengyuan

@rshriram can we move forward here?


Add a "Check all" button for each sub-area

Some areas have a lot of individual checks that are easily doable in 1 go as you go through the area. Clicking every single checkbox therefore can be a bit tedious.
A button or checkbox next to the area name header above the checklist to check all of the checkboxes in a single click.

hi! I'd love to work on this issue if its still available!

hi! I'd love to work on this issue if its still available!

Go for it! I don't have a label for anything in progress, but it can be assumed that nothing is being actively worked on unless someone says something as far as I'm concerned.

Hey! I added this feature, but it works by right clicking a map (not a region) to mark off all of it's checks
https://github.com/Phantom5800/pmr-tracker/pull/74


Fixed an issue where the overlay’s root view controller was interce…

The window's root view controller eats the user interaction when it is shown. This fix changes that; the window's user interaction flag doesn't matter, it is only the root view controller's flag.

Thanks for the PR.

Sorry but the commit 28882ed8dcd0e9ee3c8f9229fe69b99debb14ad4 should be removed. It's a known non-bug that set the alpha will log a warning, but it's not a PKHUD issue but an Apple one.
Without the alpha the animation on hide is very ugly.
Instead regarding 718d0644e26b8da3cdf35af99824b5e1db08a9f9 I've just pushed a commit from #125 that change how the hud is shown, so you should give a try to the new  version.


python3Packages.aioboto3: init at 11.3.0

Description of changes
Wrapper to use boto3 resources with the aiobotocore async backend
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
[ ] 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.

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

4 packages built:

python310Packages.aioboto3
python310Packages.aioboto3.dist
python311Packages.aioboto3
python311Packages.aioboto3.dist


Lip sync not quite working and only doing a tiny piece of the audio.

Finally got the node working... that took a lot lol, but I am thankfuly i got it working... however there is a couple of issues.
I fed it a 4minute 49 clip (this is about the average length for the ausdio i need the avatar to speak.
It took 5 minutes to process (not too bad) but it only did 5 seconds of lipsync... and it didnt lip sync very well (that may be the node...that may be the early stage of echo mimic, that may be an error on my part, or all three lol)
This is the output I had:
https://www.youtube.com/watch?v=lux1rXjzdK8

If your audio is longer, you can try to make the "length" longer. The total length of the output video is based on the "length". I am checking the Lip sync issue. This plugin only started writing yesterday and needs to be gradually optimized

Finally got the node working... that took a lot lol, but I am thankfuly i got it working... however there is a couple of issues. I fed it a 4minute 49 clip (this is about the average length for the ausdio i need the avatar to speak. It took 5 minutes to process (not too bad) but it only did 5 seconds of lipsync... and it didnt lip sync very well (that may be the node...that may be the early stage of echo mimic, that may be an error on my part, or all three lol)
This is the output I had:
https://www.youtube.com/watch?v=lux1rXjzdK8

Cause you only process 5 seconds, 😂, how do you feel now? It's pretty slow, hah?

If your audio is longer, you can try to make the "length" longer. The total length of the output video is based on the "length". I am checking the Lip sync issue. This plugin only started writing yesterday and needs to be gradually optimized

I assumed since the resulting video (the one linked on youtube) was 4 minutes 49 seconds How do you tell it to match the length of the video to the length of the audio?

Finally got the node working... that took a lot lol, but I am thankfuly i got it working... however there is a couple of issues. I fed it a 4minute 49 clip (this is about the average length for the ausdio i need the avatar to speak. It took 5 minutes to process (not too bad) but it only did 5 seconds of lipsync... and it didnt lip sync very well (that may be the node...that may be the early stage of echo mimic, that may be an error on my part, or all three lol)
This is the output I had:
https://www.youtube.com/watch?v=lux1rXjzdK8

Cause you only process 5 seconds, 😂, how do you feel now? It's pretty slow, hah?

Nope, I fed it a 4 minute 49 second audio clip not a 5 second one. I am guessing I need to do something for the node to use the full clip, at work now, had about an hour free to try it before heading out, I will give it another look when home tomorrow if not too shattered. Thing is it took 5 minutes to do the 5 seconds it did.... so that works out at nearly 5 hours for my clip... 5 hours of 100% gpu usage on a 3090 lol there is something really not right there, I have trained ai image gen models in less time than that.
I think it may be a case that the code is not quite ready for everyday use, hopefully it will become more optimized with time.

The length (check the parameter, default value is 120) decides how long you sync ....
I would say, no, this is far from ready, the live portrait more than 30 times faster than this


Inclusion criteria and anti-feature warning?

The client is, but the server isn't. Depending on how strict we want to be it may or may not belong in the list.
As far as I'm concerned; I'd argue that it goes against the spirit of FOSS so I  would argue against including it here, but that's not to say it's not open to interpretation.
The same would go for youtube apps, now that I think of it. But somehow it feels like it's more intuitive that the server-side is not FOSS there. I guess I feel that because youtube is mostly used to download/stream videos and not so much for two way communication it's less of an issue. All this to say that it's not black and white.

Thank you for contributing.
Yes, those are definetly concerns.
The more we care about the apps being completely FOSS, smaller the list gets, e.g. Wire, Signal would end up being removed because they include Google Libraries by default.
Personally I don't use Telegram for the reason you mentioned and some other reasons but technically it is FOSS.
The solution might be to include warnings like F-Droid does it (e.g. promotes non-free networks).
But instead of writing that we should probably use icons - is there a good resource?

Maybe something like this?

I think that is a good solution to the issue.

I added the badges to the apps that implement those networks, like NewPipe and Telegram.
Not sure if I should add them to Gapps Browser and Weather as well, since they connect to non-free services by default, however it's just a WebView.

That's a bit tricky. Newpipe does the same (it's a wrapper for the youtube website).
I guess it really depends on what you (/we) want to accomplish with this list, ie what the threat model is.
As for weather, there are a couple of apps that have openweathermap as a source, which licenses their data CC BY-SA 4.0 which is also open source. Then again: I couldn't find the source code of the website that supplies the API.
All in all, I guess it's greyscale all the way down.
Maybe the best solution to this is to somehow formalize the criteria. This would mean that we'd have to figure out why this list exists, and why we want to use open source. Is it (just?) for privacy? Or are we trying to further the cause of free open source software?
Oh, and: Reddit is semi-open source as well. They've had a few parts of the site closed for a long time (parts that are built to prevent fake upvotes and stuff) but they only very recently stopped supplying the full source of the website and still supply a lot of sourcecode on github.

The formalizing of the criteria is a really good idea, however also a tricky one, as you said.
This list is about Free and Open Source apps, so I think the users could expect the apps to behave in a privacy- and FOSS-respecting way e.g. by not connecting to non-free services.
The question becomes if we should warn about everything, I mean it's rather obvious that NewPipe connects to youtube.com and that weather apps that implement a webview will display some website that may not have published it's source.
I'm definetly with you when we talk about Telegram - I thought that their server would be FOSS, but turns out it's not.
So maybe one solution would be to add small icons to the apps, a red warning sign if it uses non-free networks, icons for being completely offline, for having E2E-Encryption,...

That makes sense to me.
I'd still argue that formalizing the criteria is a good plan, because this could just become a list of apps that are available in F-droid. That list obviously already exists (ie that's what F-droid is for) so there needs to be some form of differentiation.

Thanks for editing, I was going to open up another issue and close this one.
I'm gonna add the criteria to the CONTRIBUTING.md and add some apps that got suggested to me on reddit.

I think using the :warning: :warning: emoji should work, it doesn't screw around with the layout and is pretty compact.

The emoji works, but are you sure about the inclusion criteria as they stand? This list could probably basically grow in to an F-droid index with a couple of apps that aren't in F-droid, is that the objective?

Well, you can add any app you want basically, I was thinking of quality requirements, but not all of them make sense. E.g. if we were to say that an app needs to be updated every year, a simple calculator wouldn't be on the list. Good usability or something like that might be something.
I don't really want to introduce ratings or something like that, but maybe there is a good approach to this?

Closing this for now.


Feature request: Add cisco ip sla probes support

cisco ip sla feature allows to probe between devices using icmp, tcp and udp. based on https://tools.ietf.org/html/rfc6812#section-3.1.2
I would be a nice to have that cisco can probe to a cloudprober server using udp or tcp. And also cloudprober to support probes hitting a cisco device consifured with ip sla responder as a udp echo.

@fcastello  @manugarg Could you please share a configuration of the UDP Probe ?

@fcastello  @manugarg Could you please share a configuration of the UDP Probe ?


Nested/recursive Discriminated unions don't seem to work

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
See the code below, basically the top level unions has a discriminator as expected, but Foo.x does not - it's just a simple union.
Example Code
from __future__ import annotations

from pydantic import Field, TypeAdapter, ValidationError

from typing_extensions import TypedDict, Annotated, Literal

class Foo(TypedDict):
    type: Literal['foo']
    x: Foobar

class Bar(TypedDict):
    type: Literal['bar']

Foobar = Annotated[Foo | Bar, Field(discriminator='type')]

ta = TypeAdapter(Foobar)
print(ta.validator)

try:
    ta.validate_python({'type': 'wrong'})
except ValidationError as e:
    print(e)
    assert len(e.errors()) == 1

try:
    ta.validate_python({'type': 'foo', 'x': {'type': 'wrong'}})
except ValidationError as e:
    print(e)
    # should still be one error
    assert len(e.errors()) == 3

Python, Pydantic & OS Version
pydantic version: 2.3.0
        pydantic-core version: 2.6.3
          pydantic-core build: profile=release pgo=true
                 install path: /home/samuel/code/fastui/env311/lib/python3.11/site-packages/pydantic
               python version: 3.11.2 (main, Mar 27 2023, 15:57:33) [GCC 11.3.0]
                     platform: Linux-6.2.0-32-generic-x86_64-with-glibc2.35
     optional deps. installed: ['devtools', 'typing-extensions']

As of #7473 the following works without errors:
from __future__ import annotations

from typing import Union

from typing_extensions import Annotated, Literal, TypeAliasType, TypedDict

from pydantic import Field, TypeAdapter, ValidationError

class Foo(TypedDict):
    type: Literal['foo']
    x: Foobar

class Bar(TypedDict):
    type: Literal['bar']

Foobar = TypeAliasType(
    'Foobar',
    Annotated[Union[Foo, Bar], Field(discriminator='type')],
)

ta = TypeAdapter(Foobar)

try:
    ta.validate_python({'type': 'wrong'})
except ValidationError as e:
    assert len(e.errors()) == 1

try:
    ta.validate_python({'type': 'foo', 'x': {'type': 'wrong'}})
except ValidationError as e:
    # should still be one error
    print(e)
    """
    1 validation error for tagged-union[typed-dict,typed-dict]
    foo.x
        Input tag 'wrong' found using 'type' does not match any of the expected tags: 'foo', 'bar' [type=union_tag_invalid, input_value={'type': 'wrong'}, input_type=dict]
            For further information visit https://errors.pydantic.dev/2.3/v/union_tag_invalid
    """
    assert len(e.errors()) == 1

I think this is fixed then, right @samuelcolvin ?

Hi, we are facing this issue as well. Our setup is that we have a base object that we call 'Block'. Different types of blocks are defined across multiple files. For example:
class BlockGroup(AbstractBlock):
    type: Literal[BlockType.BLOCK_GROUP]
    blocks: List[Block] = []

so BlockGroup can contain again more Blocks which causes the nested/recursive behavior.
The Block is then defined as an annotated union (of many more but I included just 2 types of Block):
Block = Annotated[
    Union[
        "Collection",
        "BlockGroup",
    ],
    Field(discriminator="type"),
]

After that all the models inside the union are imported and rebuilt.
When parsing the nested Block, only the first one in the nested structure seems to use the discriminator, all the lower levels try to be parsed as all possible options in the union.
I tried to use the TypeAliasType as suggested, but then the model_rebuild gives an error:
venv\venv\lib\site-packages\pydantic\_internal\_model_construction.py:184: in __new__
    complete_model_class(
venv\venv\lib\site-packages\pydantic\_internal\_model_construction.py:492: in complete_model_class
    schema = apply_discriminators(simplify_schema_references(schema))
venv\venv\lib\site-packages\pydantic\_internal\_discriminated_union.py:57: in apply_discriminators
    return simplify_schema_references(_core_utils.walk_core_schema(schema, inner))
venv\venv\lib\site-packages\pydantic\_internal\_core_utils.py:475: in simplify_schema_references
    schema = walk_core_schema(schema, count_refs)
venv\venv\lib\site-packages\pydantic\_internal\_core_utils.py:426: in walk_core_schema
    return f(schema.copy(), _dispatch)
venv\venv\lib\site-packages\pydantic\_internal\_core_utils.py:471: in count_refs
    recurse(definitions[ref], count_refs)
venv\venv\lib\site-packages\pydantic\_internal\_core_utils.py:202: in walk
    return f(schema, self._walk)
venv\venv\lib\site-packages\pydantic\_internal\_core_utils.py:459: in count_refs
    return recurse(s, count_refs)
venv\venv\lib\site-packages\pydantic\_internal\_core_utils.py:205: in _walk
    schema = self._schema_type_to_method[schema['type']](schema.copy(), f)
venv\venv\lib\site-packages\pydantic\_internal\_core_utils.py:214: in _handle_other_schemas
    schema['schema'] = self.walk(sub_schema, f)  # type: ignore
venv\venv\lib\site-packages\pydantic\_internal\_core_utils.py:202: in walk
    return f(schema, self._walk)
venv\venv\lib\site-packages\pydantic\_internal\_core_utils.py:459: in count_refs
    return recurse(s, count_refs)
venv\venv\lib\site-packages\pydantic\_internal\_core_utils.py:205: in _walk
    schema = self._schema_type_to_method[schema['type']](schema.copy(), f)
venv\venv\lib\site-packages\pydantic\_internal\_core_utils.py:214: in _handle_other_schemas
    schema['schema'] = self.walk(sub_schema, f)  # type: ignore
venv\venv\lib\site-packages\pydantic\_internal\_core_utils.py:202: in walk
    return f(schema, self._walk)
venv\venv\lib\site-packages\pydantic\_internal\_core_utils.py:459: in count_refs
    return recurse(s, count_refs)
venv\venv\lib\site-packages\pydantic\_internal\_core_utils.py:205: in _walk
    schema = self._schema_type_to_method[schema['type']](schema.copy(), f)
venv\venv\lib\site-packages\pydantic\_internal\_core_utils.py:349: in handle_model_fields_schema
    replaced_field['schema'] = self.walk(v['schema'], f)
venv\venv\lib\site-packages\pydantic\_internal\_core_utils.py:202: in walk
    return f(schema, self._walk)
venv\venv\lib\site-packages\pydantic\_internal\_core_utils.py:459: in count_refs
    return recurse(s, count_refs)
venv\venv\lib\site-packages\pydantic\_internal\_core_utils.py:205: in _walk
    schema = self._schema_type_to_method[schema['type']](schema.copy(), f)
venv\venv\lib\site-packages\pydantic\_internal\_core_utils.py:214: in _handle_other_schemas
    schema['schema'] = self.walk(sub_schema, f)  # type: ignore
venv\venv\lib\site-packages\pydantic\_internal\_core_utils.py:202: in walk
    return f(schema, self._walk)
venv\venv\lib\site-packages\pydantic\_internal\_core_utils.py:459: in count_refs
    return recurse(s, count_refs)
venv\venv\lib\site-packages\pydantic\_internal\_core_utils.py:205: in _walk
    schema = self._schema_type_to_method[schema['type']](schema.copy(), f)
venv\venv\lib\site-packages\pydantic\_internal\_core_utils.py:249: in handle_list_schema
    schema['items_schema'] = self.walk(items_schema, f)
venv\venv\lib\site-packages\pydantic\_internal\_core_utils.py:202: in walk
    return f(schema, self._walk)
venv\venv\lib\site-packages\pydantic\_internal\_core_utils.py:471: in count_refs
    recurse(definitions[ref], count_refs)
E   KeyError: 'tarandm_core.decision.Block'

To give a bit more story. It started with ValidationError not having the correct values in 'loc', only the location of most inner object in the nested structure was in the 'loc'. We found out that it is caused by the fact that we are using a custom __init__ for Block. But when we replaced the __init__ by validators, then this issue appered. Which means that the custom __init__ somehow fixed the issue with the not recognized discriminator in lower levels.
Please do you know how to fix this issue or some workaround?
Thank you!

I do believe this is now fixed, via https://github.com/pydantic/pydantic/pull/8904


"RCTTextView.h" file not found on react-native 0.54.0

It's not finding <React/RCTTextView.h as well.

+1

Hi,
Is there any quick workaround in order to solve this?
Thanks
</ Pranav >

1

+1

Did anyone figure out why this is happening? Just upgraded to 0.55 and it's happening


Where to find example file ?

@matlabbe hello, I'm working on a pick and place project and I need to add the object tf and send it to my robotic arm, but the link that you provide on another comment : https://github.com/introlab/find-object/blob/master/src/ros/tf_example_node.cpp is unavailable , do you know where can I see the example code? I'm new to ROS and struggling with the coding subscriber and publisher stuff, Thankyou !

New link: https://github.com/introlab/find-object/blob/master/src/ros1/tf_example_node.cpp


add Apache header on all XML

Description
@aaronmarkham @zachgk Adding pom files with ASF license.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

@lanking520 Thanks for your contribution!
@mxnet-label-bot add [ pr-awaiting-merge ]


How to support smb1.0

client min protocol = NT1
server min protocol = NT1
see https://www.samba.org/samba/docs/current/man-html/smb.conf.5.html


prims algorithm

Implement Prims algorithm.
Textbook Algorithms by Robert Sedgewick looks like a good source to learn about the theory behind both Prim's and Kruskal's Algorithms.

I fixed this issue by adding back the second insertEdge
void Graph::connectNodes(Node* a, Node* b, int edgeWeight) {

	//check if the nodes are connected yet or not
	//if not connected
		//make an edge a->b and add that edge to node a edgelist
		//make an edge b->a and add that edge to node b edgelist
	    //update the nodes

	Edge ab = Edge(a, b, edgeWeight);
	Edge ba = Edge(b, a, edgeWeight);
	this->drawEdge(&ab);
	a->insertEdge(ab);
	b->insertEdge(ba);
}

Now the incorrect MST is returned because the MST variable gets overwritten each time, so I need to refactor PrimsAlgorithm into a class of it's own.


Program creation page should include inputs for storage folder and ELN folder

There should be inputs to allow user to enter info for ELN or storage folder (IDs, urls, name, etc) or an option to have these items created automatically. Not all platforms will support program creation (eg. Benchling)

ELN input added in #32. Not sure if storage folder input are still needed, so closing this issue.


cloud-config not being applied even after reboot

I'm using terraform to provision RancherOS 0.4.3 in AWS, and trying to get the host to register with our Rancher server automatically. I'm passing various configs via cloud-config which aren't being set, even after a reboot.
The config goes into /var/lib/rancher/conf/cloud-config.d/boot.yml, exactly as provided. But very little of it is being applied - just hostname, ssh_authorized_keys and proxy settings are working.
The DNS and route settings are just ignored. If I then copy these into /var/lib/rancher/conf/cloud-config.yml (or run ros config set ...) and reboot, then it starts working.
I've seen https://github.com/rancher/os/issues/339, but at the moment I can't even get it working after a reboot without some manual tweaking.
#cloud-config
hostname: <my_hostname>
ssh_authorized_keys:
- ssh-rsa
  <pubkey>
rancher:
  network:
    dns:
      search:
        - <domain1>
        - <domain2>
      nameservers:
        - <ns>
    interfaces:
      eth0:
        dhcp: true
        gateway: <gateway>
system_docker:
    environment:
    - HTTP_PROXY=<proxy:port>
    - HTTPS_PROXY=<proxy:port>
    - NO_PROXY=<.domain>
  docker:
    environment:
    - HTTP_PROXY=<proxy:port>
    - HTTPS_PROXY=<proxy:port>
    - NO_PROXY=<.domain>
  services:
    rancher-agent1:
      image: rancher/agent:v0.11.0
      command: sudo docker run -d --privileged -v /var/run/docker.sock:/var/run/docker.sock -v /var/lib/rancher:/var/lib/rancher rancher/agent:v0.11.0 <rancher_url>
  services_include:
    ubuntu-console: true
    debian-console: false

This is due to the fact that we can't download the user-data until networking is setup, but we don't reload the networking (i.e. dns/routes) with any changes from the user-data cloud-config.

I've been trying to work around this by adding a custom /etc/dhcpcd.conf with the user-data, but although this is loaded it's ignored by the system even after a reboot.
So my custom dhcpcd.conf has static settings for routers, domain_name_servers and domain_search (with the relevant option settings from the default removed).
I didn't expect it to "just work" due to the current open bug, but surely it should be read and applied on boot?

Verified this is working in v0.4.4-rc3.
Note: You will need to set rancher.network.dns.override to true in the cloud-config to be able override the default dns settings.
rancher:
  network:
    dns:
      override: true

In order to see the changes that you passed in during boot, you can use sudo ros config export --boot.


piccolo.link sbt download link not working

[x] I've read the CONTRIBUTING guidelines

steps
brew install sbt@1 not working as the https://piccolo.link/sbt-1.2.3.tgz link isn't accessible.
problem
https://piccolo.link is down.
I downloaded the sbt packaged from github. But the package management tools are all expecting this https://piccolo.link/sbt-1.2.3.tgz link to be functional.

curl -L --head https://piccolo.link/sbt-1.2.3.tgz returns 200 for me.
Could you paste the result of curl -L --head https://piccolo.link/sbt-1.2.3.tgz for you?

haridas$ curl -L --head https://piccolo.link/sbt-1.2.3.tgz
curl: (35) LibreSSL SSL_connect: SSL_ERROR_SYSCALL in connection to
piccolo.link:443
Seems some network or SSL issue from myside then :( , The link isn't
working on my browser also.
Sorry for the trouble.
Thank you.
Hari.
On Mon, 8 Oct 2018 at 22:15, eugene yokota<EMAIL_ADDRESS>wrote:

curl -L --head https://piccolo.link/sbt-1.2.3.tgz returns 200 for me.
Could you paste the result of curl -L --head
https://piccolo.link/sbt-1.2.3.tgz for you?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sbt/sbt/issues/4413#issuecomment-427903299, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAyMEuM5vhZBvmlyIDsGEfCrXrmNwIZVks5ui4EkgaJpZM4XNP54
.

--
Have a nice day.
Haridas N.

Closing this for now then.

Hi,
It seems like we have many issues regarding this reason, I am also not able to access curl -L --head https://piccolo.link/sbt-1.2.4.tgz

Following are the tickets regrading same issue
sbt/website#729
sbt/sbt#4411
sbt/sbt#4165
sbt/sbt#4279
Above list contains closed issues also but i am referring here so that all info available at one ticket.
A better solution could be finding an alternative to https://piccolo.link.
Always reliable solution:
sbt releases can be downloaded here https://github.com/sbt/sbt/releases

I'm getting the same as @veera83372.
curl -L --head https://piccolo.link/sbt-1.2.3.tgz
results in
curl: (7) Failed to connect to piccolo.link port 443: Operation timed out

Perhaps the flaky piccolo.link problem can be mitigated by adding the github releases URL as a mirror in the homebrew formula.
See the scala formula which has two mirrors: https://github.com/Homebrew/homebrew-core/blob/master/Formula/scala.rb

Fair enough as alternative but we will loose stats. Any thoughts on that?
Based on this
https://github.com/sbt/sbt/issues/4165#issuecomment-392867287
I am thinking of writing full pledged service which can keep resource based stats , allows shorten url and can run with very minimal setup. (can be a sbt plugin after every release automatically creates a short url).

I think it's fine to lose some statistics, as a fallback to the hosting instability.

I have added mirror in sbt formula Homebrew/homebrew-core#32845. Any comments?

Yeah, you should use GitHub releases as the stable fallback, rather than some other URL shortener.

Update: brew PR merged.

lol, so now it's not using piccolo.link at all 🙄

I have tried putting multiple mirrors but brew maintainer has been ignored that.

You don't need multiple mirrors, just 1 GitHub releases URL mirror, as a fallback to piccolo.link.

The provider for piccolo.link reported that a load balancer host in Hong Kong was down, causing outages in South East Asia. They rectified the situation, and will be taking steps to prevent the issue to occur again. In addition, they will deploy an update later this week in order to address other known minor reliability issues.
Incidentally, the piccolo.link website is now also apparently whitelisted on Forcepoint/WebSense, and should therefore work fine from corporate networks that rely on those filtering tools.
Hopefully this should fix our download problems for good, so I am closing the ticket; kindly let us know if you see a reoccurrence.


TypeError for serialize model to JSON

I made serialize model to JSON using save() in predictor.py.
    def save(self, fname):

        if U.is_ner(model=self.model):
            from .text.ner import crf_loss
            self.model.compile(loss=crf_loss, optimizer=U.DEFAULT_OPT)

        # JH
        self.model.save(fname+".h5", save_format='h5')
        #self.model.save(fname, save_format='h5')

        # serialize model to JSON
        # JH
        model_json = self.model.to_json()
        print(model_json)
        with open(fname+".json", "w") as json_file:
            json_file.write(model_json)

And read it using
json_file = open('/Users/jbaek/ws/Transformers_Example/ktrain/Multi_Label_Text_Classification/model-test-2.json', 'r')
# json_file = open('./model/model_aws/'+'model.json', 'r')
loaded_model_json = json_file.read()
json_file.close()
loaded_model = model_from_json(loaded_model_json)

Return this error:
TypeError: tuple indices must be integers or slices, not list
How to solve this issue? I try to deploy this model to AWS Sagemaker Endpoint.

Reference site:

https://aws.amazon.com/blogs/machine-learning/deploy-trained-keras-or-tensorflow-models-using-amazon-sagemaker/

Hugging Face Transformer models are saved differently than other Keras models.  See ktrain.text.predictor.save method:
self.model.save_pretrained(fpath)

I don't have any experience with SageMaker.  However, the reference site you're using is over a year old and doesn't look  compatible with TensorFlow 2.  Since the underlying Transformer model  in ktrain is just a normal Hugging Face TF2 model, you might try asking questions about saving/deploying on SageMaker as an issue on the Hugging Face transformer GitHub repo.

@amaiya Thanks for your information.
What is the most common way to deploy Transformer models is created by Ktrain?
Sagemaker is just one option for me.
Can I use TensorFlow Servings?

The underlying models in ktrain are just TensorFlow Keras models (e.g., using either Sequential API, Functional API, or Subclassing API), so you should be able to deploy however you would normally deploy a TensorFlow 2 model. I believe some have deployed using TensorFlow Serving.  Others have setup a Flask app to serve their model using the ktrain Predictor instances that wrap their model and preprocessing steps.


Change location of displayed info

It would be nice to allow the info to be placed under the 'Build' category in the output window. By default VS displays that category after each build. Currently you would have to build and then select 'Build time history' from the dropdown.

But the Build output already includes the time taken for the build -- At least it does for the languages I use.
Do you really need to see the total time taken for all builds after each individual build? Or the number of total builds? What value does that provide?
Also, other people get very upset about adding things to the built-in output panes.
And, there are multiple other extensions that depend on the standardized output of the Build window (pane) to function correctly. I don't want to risk breaking them.

Gotcha. It wasn't very important.


日志排序

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
SkyWalking UI中查看收集到的日志信息没办法按照程序的顺序输出（时间戳相同，毫秒）， 貌似毫秒级不满足elasticsearch排序？希望日志的输出顺序可以跟程序控制台一致，这样更容易查看日志
Use case
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

All issues on GitHub should be in English. Closing.


[core-lro] Fix resource location check

Addresses feedback in https://github.com/Azure/azure-sdk-for-js/pull/27873/files#r1402822669

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


[5.3] Make TaggableStore implement Store contract

Both TaggedCache and TagSet constuctors are typehinted with Store contract. Which means TaggableStore must implement it in order to work properly.
https://github.com/rkgrep/framework/blob/c163e00964207092bd627c7fc6fc46eb1c206c8e/src/Illuminate/Cache/TaggableStore.php#L17

Currently seems to be working as is.


Implement an initial version of the support bundle in Alloy

PR Description
Port the Support Bundle from the Agent repository & update it for Alloy.
Which issue(s) this PR fixes

Notes to the Reviewer
PR Checklist

[x] CHANGELOG.md updated
[x] Documentation added
[ ] Tests updated

Over to @grafana/grafana-alloy-maintainers for a code review

Awesome! Thanks for adding this, the code and docs are very clear and easy to follow :) Did you get the chance to test it on windows and with fleet management?

I did not have a chance to test in windows/FM. Will connect with you tomorrow to discuss options, I don't have a windows environment set up (or know how to access a shared one yet)

Tested on Windows, it works well :) fyi if there are no logs during the recording time the log file won't be generated (which is ok imo)


[WIP] 2630 spec warnings

Resolves #2630
Description

Type of change

Bug fix (non-breaking change which fixes an issue)
Documentation update

How Has This Been Tested?

Screenshots

Nice! Any chance you want to fix some dashboard specs too? ;)

Nice! Any chance you want to fix some dashboard specs too? ;)

Happy to give it a shot, @seanmarcia; where/how do I start?

?

Not sure on the best place to start -- we've commented out several and we also have several that are flaky.


Is there a way to open a VS Code extension tab if it isn't open?

We have a treeview in our VS Code extension which can be toggled on and off by the user by calling a command. However if the user toggles the treeview on and they don't have the remote explorer open (where the treeview is displayed), it looks like the command doesn't do anything. Is there a way to say, open the remote explorer if it isn't selected, and focus on the treeview when the treeview is toggled on so the user can see the command showed a new treeview?

Closing since I found there is a way to do exactly this with the "TreeView.reveal(TreeItem)" method 😄


Mangling cache should have a size cap

And LRU semantics.
Phase 2 can be 2 LRU populations (OS Pages Cache style)

@Ivan-Shestakov please check

The size cap is soft, so new entries will always get added to cache.
At the moment no automatic invalidation happens, so /_ns-cache?invalidate or ws restart is the way to cleanup (needed only if a user abuses the system by mistake flooding the namespaces) .
Manual test for /_ns-cache json retrieval and invalidation.

Action
Expected
Actual

curl -s 'cm-well-node:9000/_ns-cache'
Check size of cache by piping to jq ".mainCache
length"

curl -s "cm-well-node:9000/_ns-cache?invalidate-all" -H "X-CM-WELL-TOKEN:xxxx-xxxx"```
{"success":true}

`curl -s 'cm-well-node:9000/_ns-cache'
Validate that the cache was cleaned
0 as expected.

The effect of namespace caching was tested on lab environments - using frontend regression and customer use cases.
Moving to Done.


Going from single mysql-queries to mysql-pool

I'm trying to switch from single mysql-queries to mysql-pool connection using the node-mysql-module, so users can share one mysql-connection, but I'm not familiar with this at all (also new to nodejs/socket.io).
The following code is what I've tried so far to send data every second to the socket in an array:
http://pastebin.com/dRFzWtEH
But this doesn't send me any data to the WebSocket. Is this approach (code-structure) even correct?
Previously I used query.on('results') to get data like this:
var selection = "SELECT * FROM auctions";
var query = mysql.query(selection), auctions = [];
query.on('result', function(auction) {
console.log ('id: '+auction.id+' ('+auction.duration+') ');
});

This worked fine showing data with auction.row but how to do this in my mysql pool connection?
Also after some seconds I'm getting an error that release() isn't even defined, but it's listed in the mysql-module documentation. I think my whole logical process is somehow incorrect?

Should I use connection.end() and .release() at all? Because the
connection should never end.
Should I still use setInterval(function () { mysql.query('SELECT 1'); }, 5000); as answered in another StackOverflow question to keep the connection alive here? (nodejs mysql Error: Connection lost The server closed the connection)

(Appreciate any tips or answers to even some of my questions! Better some answers than none, because I experienced that this topic isn't answered much at all.)

I have to look through your code, which will take a while because it's a lot and a lot of questions, I'm sorry, but I can say you have a programming error here:
mysql.getConnection(function(err,connection){
if (err) {
  connection.release(); // this line is wrong; remove it, as there is no connection if there is no err
  console.log({"code" : 100, "status" : "Error in connection database"});
  return;
}
// ...

Should I use connection.end() and .release() at all? Because the connection should never end.

You should keep calling .getConnection() and .release() for every query.

This whole topic confuses me so much and I can barerly find someone who has enough knowledge to accomplish this task. Would you be open in some hours/days to rewrite the code if I pay you?
At StackOverflow someone just suggested me that I have to rewrite it, because of my setTimeout is like unstoppable with this code and the whole logic is kinda messed up.
See this picture of our chat: http://postimg.org/image/igynwe685/

Also I know I posted so many links to get into my problem, but this is the actual situation minimized to the important things: http://stackoverflow.com/questions/30052273/cant-send-fetched-data-to-my-socket-io-stream/30057900#30057900

Just one question: Is it possible to exclude specific field items from fields when you perform connection.query(selection, function(err, fields, results){ ?

@AlexioVay fields array always match keys in every row of results array ( after all the "join tables" name mangling is applied, if enabled )


chore: create dev env on dev.hub.asap.science

gonna pass on reviewing this one, I think @johnytiago is best suited :sweat_smile:


*: add pdctl

now simply support config Get and Set
PTAL @huachaohuang @siddontang @overvenus

 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.

Rest LGTM

PTAL @huachaohuang

LGTM


GPL-842 New intergration point for lighthouse service, samples of importance.

User story
GPL-842 | As core LIMS we would like a new collection to persist if a sample is of importance or value to sequence
Who are the primary contacts for this story
Callum S (PAM)
Ryes (PAM)
Harry C
Philip VV
Eduardo MR
Acceptance criteria
To be considered successful the solution must allow:

[ ] New collection with schema
[ ] For each sample PAM are aware of add to new collection
Root_sample_id
Flag must sequence (Sequence regardless – bypasses existing business rules)
Flag of Importance (Sequence if passes existing rules)

Dependencies
Additional context
Parent feature: https://github.com/sanger/General-Backlog-Items/issues/103

More info (current thinking):

PaM make an API call to the lighthouse service which makes an upsert to the new 'samples of importance' collection
This triggers a process which:

checks if data for those samples is in Mongo already
if so, check if they have been cherrypicked already
if not yet cherrypicked:

combine information from 'filtered positive' field and 'samples of importance' to decide whether to set well state in DART to 'pickable'
set the 2 new fields in MLWH lighthouse_sample table (https://github.com/sanger/unified_warehouse/issues/172)

There will also need to be a change to the existing nightly crawler run, which:

checks if the new samples have entries in the 'samples of importance' collection
if so:

combine this info with the 'filtered positive' info to decide whether to set well state in DART to 'pickable'
set the 2 new fields in MLWH lighthouse_sample table (https://github.com/sanger/unified_warehouse/issues/172)


Integrate Mersenne31 field

Background: Reed-Solomon Codes over the Circle Group
There are a couple parts to this integration -

[ ] #58
[ ] the compression and decompression logic
[ ] #59

@dlubarov can you expand on what you mean by compression/decompression logic?

Normally an LDE over a complex domain would give complex output, but https://eprint.iacr.org/2023/824 showed that the complex output will lie on a linear subspace so it can be compressed/rectified to real values.


Auth0 users with different accounts but same username will not be able to log into the site

With the [recent commit] (https://github.com/auth0/wp-auth0/commit/40377d7a14417d3ceea450bc5822007bdee3f9d5), the error triggered by username collisions is no longer suppressed, but this means that users different accounts with the same username will receive an error message telling them that their username already exists.
The issue arises because Auth0 usernames are simply the first part of the email address, up to the @ symbol. But, several users can have the same username, since Auth0 only requires the full email to be unique.
Would it be possible to switch from using Auth0 usernames (which are not unique) to Auth0 email addresses (which are) for the WP username?

It is not as easy as change from username to email. The problem is that in cases where the connection (ie twitter or instagram) doesn't provide an email and only the username. As soon we figure out the best way to handle it we will release a new version.
The idea of the last update is to prevent the security issue that you notify us.
Thank you very much for you collaborations.

Done => https://github.com/auth0/wp-auth0/pull/55/files#diff-6e06369d1eb2c0b065f6192a23eed41cR39


unable to open the fbx files

I can't even open them.

The FBX files are directly downloaded from Mixamo.com. You should be able to open them fine

You have to follow the installation guideline.
The "Download ZIP" will cause missing texture, animation, and models, as this repository uses Git LFS, which replaces actual files with pointers.
To install properly, you have to clone the project.


Catch up python openplugincore to npm openplugin core

Description
The python openplugincore library is lagging behind the npm library, there are a couple of updates needed along with some extra features
Tasks

OpenpluginMemo: should store in ram the configurations for the plugins
Function response truncation: should truncate the API response so as to not exceed the context limit
allow fetch_plugin to take messages instead of only a prompt. It should abstract the aggregation of messages.

#17 compelte

#18 complete feature


Drive by compliment

Issue just to say, this plugin is so good. with the clickable buttons now, very awesome. Keep it up!

Thanks for the kind words :smile:


Fix MP urls

This fixes MP urls in search results by using pa_link for member.url wherever possible.

@xybrnet

Cool :+1:


Offboarding - API to share the data after tenant removal

Offboarding - API to share the data after tenant removal

Closing this issue since this is a duplicate of https://github.com/sourcefuse/arc-saas/issues/6


How to validate the array of string item with array property present in json.

Hi Team,
I want to validate the array of string item with array property present in a json. I am trying with enum keyword as mention below but not getting any solution. I want to validate the item, which is present in a enum array, if anyone enter the any value into the resultType array then error should be display. So can you guys tell me how to implement this in my project.
My schema :
"properties": {
				"resultType": {
					"$id": "#root/ruleProperties/resultType",
					"title": "Resulttype",
					"type": "array",
					"default": [],
					"enum":["a","b","c"],
					"items":{
						"$id": "#root/ruleProperties/resultType/items",
						"title": "Items",
						"type": "string",
						"default": "",
						"examples": [
							"expression"
						],
						"pattern": "^.*$"
					}
				}

My array : resultType:["a","c"]
version :  "ajv": "^7.0.0"
Thanks !

enum should be inside items, not as a sibling


Why I get different map in both Web browser and mobile browser?

I got a different map view in web browsers and mobile browsers.
I want to show Australia map in the view based on the latitude and longitude of the json data.
It works as expected in web browsers but always showing China map in mobile browsers
Do we need to specify anything to make this work as expected?

There may be some magic in AMap lib.
Sorry, I don't know neither


vm/dart_2/isolates/fast_object_copy_test turned flaky -> Timeout on dartk-reload-linux-debug-x64

There are new test failures on Bump protobuf dep, update protobuf test output...[CFE] Dedup duplicated code in source_library_builder.
The tests
vm/dart_2/isolates/fast_object_copy_test/2 Timeout (expected Pass)
vm/dart_2/isolates/fast_object_copy_test/3 Timeout (expected Pass)
vm/dart_2/isolates/fast_object_copy_test/4 Timeout (expected Pass)
vm/dart_2/isolates/fast_object_copy_test/5 Timeout (expected Pass)

are failing on configurations
dartk-reload-linux-debug-x64

These tests were flaky before.
Log:
https://dart-ci.appspot.com/log/vm-kernel-reload-linux-debug-x64/dartk-reload-linux-debug-x64/9539/vm/dart_2/isolates/fast_object_copy_test/2

Also on release, was also flaky:
The tests
vm/dart_2/isolates/fast_object_copy_test/4 Timeout (expected Pass)
vm/dart_2/isolates/fast_object_copy_test/5 Timeout (expected Pass)

are failing on configurations
dartk-reload-linux-release-x64

https://dart-ci.appspot.com/log/vm-kernel-reload-linux-release-x64/dartk-reload-linux-release-x64/10749/vm/dart_2/isolates/fast_object_copy_test/4


Fixed #22550 -- Raise error on QuerySet reverse after slicing.

https://code.djangoproject.com/ticket/22550
Ref PR #2677

Rebased after a merge conflict appeared

@timgraham Can you give it another review?

@felixxm Thanks for the review.  I've rebased and updated the PR accordingly.


There need to be restrictions on ∀I and ∃I

While implementing the deductive system of both the Calgary and Cambridge for LogicPenguin (which is nearly complete), I noticed a couple oddities.
The Cambridge version lists a restriction on $∀I$ and $∃I$. It says that the variable introduced must not already occur in the statement generalized upon.
The Calgary (and Mangus's original) list no restriction at all.
Both are wrong if a quantifier with a certain variable within the scope of another quantifier with the same variable is allowed syntactically.
The Calgary version allows the inference from $∃xRxa$ to $∃x∃xRxx$, which, at least given the normal way of treating double-binding, is semantically equivalent to $∃xRxx$, which should not be provable from $∃xRxa$. (Just as an example, there are others.)
The Cambridge rule, however, is too restrictive, since it is impossible to prove something like $∀x(Fx → ∃xFx)$, which is semantically valid.
Looking at the Cambridge recursive syntactic formation rules for formulas of FOL, it looks like it disallows both vacuous binding and double binding, so maybe this isn't really a problem for it. (Though the restriction then seems redundant since the result wouldn't even be well-formed.)
However, the Calgary version needs some kind of fix. Either the syntactic rules have to change, or there needs to be a restriction on the rules, something like "c must not occur within the scope of a quantifier $∃x$ or $∀x$ already in $A(...c...c...)$."

Yes, the original forallx did not allow $\forall x(Fx \to \exists x Fx)$ in the syntax. I made that change in forallx:YYC to align it with the OLP syntax. I think in practice it basically never comes up since you wouldn't ask students to prove $\forall x(Fx \to \exists x Fx)$ but rather $\forall x(Fx \to \exists y Fy)$. But yeah that's a mistake, thanks!

PS at least carnap.io does it right, i.e., it doesn't accept the first inference...

Because of its roots in Hardegree's system and how that book states its similar rules, the current version of Logic Penguin enforces these rules the other way around, i.e., the line you already have is required to be an instance of the line you get, replacing all free occurrences of the variable in the line you get with the same name (which they might already include). It should handle both cases correctly.
The problem is that wording the rule that way is confusing to students, since they naturally think about substitutions in the result, not in what they already have. I understand why you use notations like $A(...x...x...)$ and $A(...x...c...)$ but they're not as precise as something like $A[c/x]$ where this means what $A$ becomes when $c$ is substituted for all free occurrrences of $x$, even if there aren't any. By experience, I know intro students understand your kind of notation better than the the more precise variation, however, so I'm not against it.
LogicPenguin (optionally) allows a rule panel to pop up with all the rules and students can fill in justifications by clicking on the rule in the panel. At the moment, I'm just trying to make sure the rules are stated at least as accurately as possible.
Vacuous binding and double binding are both annoying things to have to account for, and never useful or necessary, and I understand why some people might choose to exclude them from the syntax. But if they are allowed, a proof checker needs to treat them in accordance with the semantics.

However, I do have to restrict $\forall$ E, since right now the replacement of $x$ in $A(\dots x\dots x\dots)$ is defined as  "replace all occurrences of $x$" not "all free occurrences" so we technically allow $\forall x\forall x A(x,x) \vdash \forall x A(c,x)$ which is bad.

I interpreted the change is the schemas in $∀E$ to mean all free occurrences. I guess that was just being charitable, but by all means make the wording tighter.
The example I gave for restricting ∃I is also invalid for ∀I even if "a" isn't in an undischarged premise/assumption. Let the domain be integers and interpret R as less-than. From there being something less than any arbitrary "a" does not mean that everything is such that something is less than itself.
If that doesn't maybe this one convince you?

$∀y∃x ¬x=y$   (a plausible if contingent premise)

$∃x ¬x=a$   (∀E 1)

$∀x∃x ¬x=x$   (∀I 2 ???)

$∃x ¬x=x$   (∀E 3)

Some monists might be happy, but …

Duh, thank you.

Kinda regretting I allowed vacuous quantification in the syntax now, but I made my bed so I guess I have to lie in it.

I think I've fixed it -- by defining the notation for substitution/replacement more carefully, so that the main text basically stays the same. But added a couple of paragraphs that show how you have to be careful in cases with overlapping quantifiers binding the same variables. (Since they only come into play if you use such sentences in examples, and you'd be a mean instructor if you did, I figured it's best to explain it separately for the attentive/curious students but not make the main exposition more convoluted by adding restrictions.)

Seems good enough. Probably only matters to students actively trying to break things.
Incidentally, the code for supporting most of the forallx systems, including Calgary, for LogicPenguin is mostly complete but could use some bug checking. I'll send you a link you can use to play around via email.
I did put a restriction on these rules in its pop-up rule panel.

Thanks!


fix cmdline sum command and and test

Resolves #1004

LGTM!
However, I found some issues which can be fixed very quickly.

The sum command doesn't uniqueing the reports.
Resolution: set isUnique attribute to True in both places:
https://github.com/Ericsson/codechecker/blob/a3d8f812c236bedfcc89a5cb331156e4a4d66ab4/libcodechecker/cmd/cmd_line_client.py#L409

report_filter.isUnique = True

https://github.com/Ericsson/codechecker/blob/a3d8f812c236bedfcc89a5cb331156e4a4d66ab4/libcodechecker/cmd/cmd_line_client.py#L429
all_checkers_report_filter.isUnique = True

In check_run_names function we should create a run filter which queries runs by check_names parameter:
https://github.com/Ericsson/codechecker/blob/a3d8f812c236bedfcc89a5cb331156e4a4d66ab4/libcodechecker/cmd/cmd_line_client.py#L44

run_filter = ttypes.RunFilter()
if check_names is not None:
    run_filter.names = check_names
    run_filter.exactMatch = True

run_info = {run.name: run for run in client.getRunData(run_filter)}

These bugs can be fixed in this pull request, or we can create another issue to fix it.

I would keep this pull request for the bug fix only and create separate issues for the modifications you suggested @csordasmarton .

Okay, I created separate issues for these problems: #1025, #1026.


Did you ask ANSSI about your curious way to load a system?

I decided to remove this message and close ticket. This is poor analysis and I would like to apologize.

Closing.


FR: when running locally, it would be nice if com.google.appengine.application.id System.property were set

There is apparently a way to pass app-id when you start up the GAE Java runtime locally.  At the very least, it would be good (helpful to Veneer) if:
com.google.appengine.application.id System.property() were set.

Btw, it used to be in appengine-web.xml :-)
Not sure if the devserver has an extra param now...
On Thu, Aug 3, 2017 at 4:08 PM, Les Vogel<EMAIL_ADDRESS>wrote:

There is apparently a way to pass app-id when you start up the GAE Java
runtime locally. At the very least, it would be good (helpful to Veneer) if:
com.google.appengine.application.id System.property() were set.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/appengine-plugins-core/issues/439,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAE4zRtvu_4xdfY5qIOYgCe6mNjIxiYgks5sUlLygaJpZM4OtGFY
.

For gcloud on a Mac and Linux it can be found by:

Look in ~/.config/gcloud/active_config
Look in ~/.config/gcloud/configurations/config_NAME_FROM_ACTIVE_CONFIG
grab project =

[core]
project = lesv-003
account =<EMAIL_ADDRESS>

This might work for the command-line build tools, but this doesn't make much sense to IDEs. I wonder what should be done for the IDE story?
For IDEs, this can even be dangerous if the gcloud default project ID is silently picked up and actually contacted when running the dev server locally. (If the project is not ever to be contacted/scanned, there is no need for the dev server to know it, so I assume the project is actually contacted.) IDEs don't and shouldn't depend on the gcloud config state.
Even if it were OK for the IDEs to use the gcloud default project ID, then IDEs would need to have a way to pass a login credential to the dev appserver that has an access permission to the project. (Again, IDEs don't depend on the gcloud login state and have their own login system.) I don't believe there is a way to pass a login credential currently.
Then, IDEs would need to ask users to

go out of an IDE and do gcloud init (or something to that effect) to set a project ID
log in inside the IDE and select a right login credential
to run a simple HelloWorld app locally.

At some point, the IDE's are going to need to do the equivalent of gcloud auth application-default login if you are going to work w/o the gcloud sdk to provide support for Application Default Credentials.  Project should come from a similar source.

So, this is because of the ADC for Veneer? I see. Then, I think the right UX in the context of IDE is to allow users to optionally set an IDE login credential and a GCP project ID when running applications or the dev server. If users want to set the values for whatever reasons, they could do so, but these values must certainly be optional.
For using IDE login credentials as ADC, I know this can be done through an env variable GOOGLE_APPLICATION_CREDENTIALS. However, note that if we don't enable APIs on our side (e.g., on the cloud-tools-for-eclipse GCP project for the Eclipse plugin), Veneer libraries still won't work. (AFAIK, gcloud has already enabled all necessary and feasible APIs on their project for their ADC.)

BTW, I think appengine-plugins-core should not set the system property com.google.appengine.application.id by default, to avoid conflicts with IDE clients. (We should prevent IDEs from silently picking up the gcloud project config.) Only non-IDE clients should request to set it.

I just figured this was a cross project issue so I posted it here.
I don't think requiring users to setup ADC G_A_C envVar is the best idea.
@patflynn FYI

Of course, users shouldn't set up the env var themselves. IDEs will provide a UI to select a logged in account and set up the env var under the hood when running dev servers (in addition to setting up the system property for the project).

There are discussions in the Cloud SDK Downloader design doc (written by @loosebazooka) suggesting that our build tools move away from using any of the gcloud config values (including project, user, or whatever). This will make this FR obsolete.

This is for the runtime environment locally, these could easily be sent from other sources, or we could document the requirement to set the envVar's - which I'm ok with.

After all, it looks like setting the system property com.google.appengine.application.id is ineffective. The dev appserver1 seems to always overwrite it to no_app_id even if it is defined and given.
It has also been reported that setting the env var GOOGLE_CLOUD_PROJECT doesn't work either.

It has also been reported that setting the env var GOOGLE_CLOUD_PROJECT doesn't work either.

Said too early. I was told that GOOGLE_CLOUD_PROJECT works for Veneer libraries but not for App Engine SDK services.

Updated the title, as the system property com.google.appengine.application.id doesn't work anyway for the dev appserver. (https://github.com/GoogleCloudPlatform/appengine-plugins-core/issues/439#issuecomment-328186047)

FTR, credentials can be passed with GOOGLE_APPLICATION_CREDENTIALS, which can also be a service account. Some Google Cloud APIs, such as Translation, seems to require a service account rather than a user credential.

Hi,
Any update on this topic?
I've update to AppEngine Maven Plugin (Cloud SDK based) and now I don't have a way to get the project ID in DevServer.
Thanks in advance!

@gartediego have you tried setting GOOGLE_CLOUD_PROJECT? I believe all Google Cloud libraries should pick this up. However, I think it is possible that the environment variable doesn't work for non-Cloud libraries like the old App Engine API.

Thanks for your response @chanseokoh, but it's not working.
Regards,
Diego


[Storage] Fix aio walk blobs sample

API changes have been detected in azure-storage-blob. You can review API changes here


About offline reconstruction

Hi,
I was trying to run your code to reconstruct surface offline. Input "argv[]" according to the prompt. It's:
"  -i "   -  Specify the path of the input file (.obj, .ply, .xyz)"
"  -o "  -    Specify the path of the output file (.ply)"
so, I set the parameter "-i" as the file path of  .ply(ex:"C:/Users/Desktop/tPLY"  ,but is "unknown file extension" .
I change the "-i" as ""C:/Users/Desktop/tPLY/model000.ply" ,it works .But it can read only one file .
How can I debug the program to read all .ply files offline?
Any help would be appreciated!
Thank you !

Hi,
we added the command line interface for our very specific use case of quantitative evaluation. That's why its capabilities are very limited. However, adding support for multiple scans can be done quite easily.
In BatchSession.cpp, you'll find a method that parses command line arguments and run(), which starts the pipeline. In run(), you will find:

A ScanDataSink, which implements an interface required by load_scan. It assumes that the specified file only contains a single scan and will store it in its member variable.
Calls to load_scan(), data.AddScan(), and data.IntegrateScan(). All of these need to be repeated for each new scan that you want to integrate.
Finally one call to data.extractedMesh.saveFineToPLY(), which can simply stay there (a single time).

Once you add proper command line parsing for multiple scans, you can integrate each scan in run(). Feel free to issue a pull request if you want to share these changes. I would appreciate it :)
In theory, it should be possible to use data directly in load_scan() instead of the ScanDataSink. However, data will do some additional things and I am not sure if everything is wired up correctly at this stage. But feel free to try. Then you would not need to call data.AddScan().

Thank you foe your reply. I modified the program . It works well when two files are loaded. But runtime error occurred when more than six files .

It's the code

Is data issue memory release issue maybe?
Looking forward your reply.

This piece of code looks good on first sight. Can you debug the application to see where exactly the error occurs?

I solved the runtime error , just the iterator out of the range . I can run the program successfully with my own data.
But the results are unexpected. That's my input and output.
input:

output:

I just want to fusion the overlap mesh area.But it's either over-smooth or coarse in some area.
Some key parameters  influence it in my sight. They are :
scale: the length of mesh edge;
smoothness: the smooth of surface;
R in ExtractMesh.h :resolution of the face.
Is there anything I ignore?

That looks like the patches should be smaller. Try specifying a --scale of about 0.3 to 0.5.

I encountered the similar problems about  can't dereference out of range vector iterator when test on my data .
The error is
STL_VERIFY(_Mycont->_Myfirst <= _Ptr && _Ptr < _Mycont->_Mylast, "can't dereference out of range vector iterator")
I call the stack to track error , it's mainly in  "Optimizing part" . The function call is about:
HierarchyMortonMultiPass.h : the function of  void findOrCreate(Iterator elementsBegin, Iterator elementsEnd, std::vector<std::pair<iterator, bool>>& result) in class OSR_EXPORT MortonContainer

the error is mortonIdx out of the size  dataIt , but I didn't have any idea to modify . Can you give me some advice .
Thank you for your time .

Can you provide your data and code (e.g., in a fork of the repository)?


Release a first alpha version of Zino 2

An assortment of tasks that need to be completed to make a first alpha release:

[ ] Add a small installation guide to README.md
[ ] Add a section about Zino's design principles to README.md
[ ] Merge outstanding PRs

[ ] #124
[ ] #133
[ ] #149

[ ] Make a stable branch
[ ] Tag version 2.0.0-alpha.1
[ ] Upload first package to PyPI

We haven't set up a full Sphinx docs pipeline yet, but should minimally try to explain what Zino is and how it operates in the README.md file.  The legacy Zino code is open source, but I don't think it has ever had public distribution, and there is very little written docs from before.

Release was out on Friday, closing :tada:


⚠ Bofrost.lu has degraded performance

In d642afe, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 851 ms

Resolved: Bofrost.lu performance has improved in 69cabb6 after 20 minutes.


Add Address.mailing_address method. Add tests for Faker::Address. Fix…

Fixes #841
Add tests for Faker::Address. Fix bug with zip_code(state_abbreviation), nullifying need for one test in test_faker_zip_code.rb
Previously, en.yml did not contain a list of postcodes per state, so a call to Faker::Address.zip_code "TX" would fail, unless the Faker::Config.locale had been set to :"en-US". Rather than changing config in the function call, I added postcodes per state into en.yml. This change nullified the need for the last test in test_faker_zip_code.rb

I merged this without the mailing_address method because it was too similar to the already-existing methods.


Read from ReportStream LIVD API to update devices/diseases table (be)

Background
Several of our data model changes for multiplex rely on the LIVD table as their source of truth. (What diseases does a device support? What are the potential results for a given disease?) The LIVD table is a CSV file that updates semi-regularly, and the ReportStream team has built an API to read it. We should use this API as our source of truth instead of manually updating the database.
Action requested
Add an endpoint that calls the ReportStream API and reads out all the data for the devices we support.
If any of the data is different than the device data already present in our database, we should update our database accordingly.
This should have the extra benefit of keeping us in sync with the table on things like name changes, not just supported diseases.
Additional context
Once the API endpoint has been set up, we can likely use it in other parts of the application (i.e, when adding new devices, we can rely on the API for all the data about the device rather than copy/pasting it manually as is done today).
Acceptance Criteria

[ ] We can read device data from the ReportStream LIVD API
[ ] We can update our device table based on the output of the LIVD API

@fzhao99 and I are compiling our research and tracking work in architecture decision record google doc for the time being. More details to follow here once we have a proposal.

@fzhao99 and I are compiling our research and tracking work in this architecture decision record google doc for the time being. More details to follow here once we have a proposal.

fyi @emmastephenson

Thanks @georgehager! Let me know when it's ready for review, I don't want to jump into the doc while it's still under development.

Based on the results of our research, we've created/edited the following tickets to capture the suggested implementation:
#4272 (get device data that can be used across environments)
#2011 (create a seed file/data)
#3443 (set up reset for demo and training - not behind okta)
#4270 (set up reset for environments behind okta)
#4271 (make sure set up works for local dev configurations)
#3443, #4270, and #4271 can likely be worked simultaneously but each environment group has distinct requirements, so we wanted to capture that in separate tickets.
cc: @emmastephenson @fzhao99


$appends not work in artisan

I'm using this package with Lumen 5.4. This package works nicely in my project. Just the $appends not work in 'artisan'. Anything i missed?
use Jenssegers\Mongodb\Eloquent\Model as Eloquent;

class MyData extends Eloquent{
 protected $appends = ['field_one', 'field_two'];

 public function getFieldOneAttribute()
 {
  return 'field_one';
 }
 public function getFieldTwoAttribute()
 {
  return 'field_two';
 }
}

Hi @ihsanberahim , did you manage to find the solution to this? I'm facing this issue and could not seems to find the solution anywhere. Thanks!

Actually, it works but you need to call toArray
It is not working
App\Models\Notification::first()

It is working
App\Models\Notification::first()->toArray()


Use existing parameter values, even for new templates

https://github.com/remind101/empire/pull/848 introduced a change where we would only merge in the existing parameter values from the stack, if we're updating the stack using the existing stack template.
https://github.com/remind101/empire/pull/876 introduced a change where we actually wan't to always use the previous value , even if we're updating to a new stack template
Without this change if you manually set the DNS parameter on the stack to false, then deploy a new version, it will still get changed back to true, because CloudFormation will use the default value, because we didn't provide the previous value for the parameter.
The net effect  of this change is that we always merge in the existing parameter values from the stack, but only if they're defined in the new stack template.

👍


Develop replacement for "shepherd" that calls to dockerhub far less frequently. @cconstab

Rather than replace shepherd I added a cache registry.


Img directory

Hello,
i have a problem with this directory, in your boilerplate i don't have any img directory, and when i try to create img directory the task delete it.
in DIST i have
- CSS
- JS
-  SVG
how can fix this bug?

Looking into this right now. Will let you know once it's fixed. Sorry!

This will be addressed by https://github.com/cferdinandi/gulp-boilerplate/pull/50, which I'll be merging as soon as the tests are done running...

This should now be addressed with the latest version. Please update.

And thanks for letting me know!


How do I do cookie authentication?

I am trying to implement one-database-per-user with a service on my express app using slouch, it looks like that:
const Slouch = require('couch-slouch')
const config = require('./config')

const slouch = new Slouch(config.couchDb.url)

module.exports = {
  createDb(user) {
    const dbName = `${config.couchDbBaseName}_${user.email}`
    return slouch.user.createSession({
      username: config.couchDb.username,
      password: config.couchDb.password
    })
    .then(response => slouch.user.create(user.email, user.dbpwd))
    .then(response => slouch.db.create(dbName))
    .then(response => slouch.security.set(dbName, {
        admins: { names: [], roles: [] },
        members: { names: [user.email], roles: [] }
      })
    )
  }
}

I was hoping some magic would happen and the cookie from the createSession response would be sent with the next commands. But although the user creation works (maybe my db lets anyone create users? oops), the following slouch.db.create call is failing with 'unauthorized'. If there a way to get through this?

Yeah, cookie authentication in this manner isn't something that we've needed to do yet so it is missing. I just took a few minutes to get something working and got it mostly working, but I need to make some changes downstream to finish it off. I'll post here when the changes are ready.

Here is a WIP that adds cookie authentication support in node: https://github.com/redgeoff/slouch/pull/27. I'll finish this up shortly.
Cookie authentication support for the browser cannot be added until https://github.com/redgeoff/slouch/issues/10 is completed

Awesome, thanks. I just had a quick look at it. I have something working with nano. But it looks ugly (transform callback into promise all the things), so I'll give it a shot on Monday. Slouch is really neat and so was your article on creating couchDB instance on AWS. Kind of hooked to couchDB this week...

Cool!
https://github.com/redgeoff/slouch/pull/27 was merged and is part of 0.1.0 on npm.
See Authentication Example for an example

This is working perfectly. I have a small couchDB question, it seems counter-intuitive to me to put the user in the role field using the method onlyRoleCanView, shouldn't there be a method onlyUserCanView that set the names field of the _security document?

Sure, onlyUserCanView would be a great addition to the Slouch API. Interested in submitting a PR?


Fixes issue with LDAP inserting users with blank names.

Fixes #1207.
Fails my Travis build, but so does the develop branch which succeeds on the official Travis builds, so I'm interested in why my Travis builds from my repos fail but gogs' official repos are fine...
Should pass on your Travis server.

Thanks.