vk_mem_alloc.h should be optional

To avoid risk of conflicts with other projects/code hierarchy/..., and confusing IDEs, vk_mem_alloc.h shouldn't be in the baseline include directory. This is especially true now that Vulkan SDK ships with VMA (as of 1.3.216)
I understand that for practical purpose, it might be convenient to have vk_mem_alloc.h shipped with vma-hpp, but at the very least it ought to be in its own directory so that the application developer can choose to use the header shipped with vma-hpp or some other one (Vulkan SDK, submodule, ...)
Also worth noting is that in the case of the SDK (1.3.216 for Windows at least), the vma header is in a vma subdirectory. It would be best if vma-hpp could handle different location like that. (Alternatively the application project can add <VK_SDK>/include/vma as an include path, but it's not as clean IMHO)

Regarding the inclusion of the vk_mem_alloc.h, I think that it does make sense to include the header with vma-hpp, since the hpp headers are generated for a particular version of the C implementation.
At least on my system, the vk_mem_alloc.h that ships with the LunarG SDK is placed straight into /usr/include.

One can just download the tagged version of vma-hpp that matches the version provided by the SDK/submodules/whatever
And I'm not suggesting to not provide the vma header, just that it be in a separate directory from the vma-hpp headers, so that one can choose which vma header to use. Currently one has no choice.

At least on my system, the vk_mem_alloc.h that ships with the LunarG SDK is placed straight into /usr/include.

Then all the more reasons to have a configurable prefix :). For me, I'm using the SDK for Windows. There are no headers files directly in <SDK>/include, all headers are in one subdirectory or another.

vk_mem_alloc.h was placed here for convenience, I don't see any point to move it into another directory, as one can just pull it from the submodule if needed, so I think it can just be deleted.

Now it's included via #include <vk_mem_alloc.h>, if one needs more control, specific VMA can be included manually before VMA-Hpp.


update dependency versions

dotnet tool install --global dotnet-outdated-tool
dotnet outdated -vl Major -u

Microsoft Reviewers: Open in CodeFlow

Thanks for doing this @danmoseley, change looks good to me. However, we may want to add one thing to your script before calling outdated tool: dotnet new nugetconfig --force. This will make sure that the feed used for the updates is nuget.org, as that will be the only way of knowing if a dependency actually has a new version.

Yes, you're quite right, I did this for aspire-samples and forgot this one's different. Let me see.

Also, I don't think we want to do version lock on Major, at least not for the Aspire Component dependencies, as we said that we would be always on latest stable, regardless of it being a major version bump.

You shared your script somewhere last time and I should have just used that...

Looks like the Azure OpenAI package has breaking changes in 2.x. Reverting that.

Looks like the Azure OpenAI package has breaking changes in 2.x. Reverting that.

Ooh, first interesting one then. I guess for this particular case it is okay because our component that references that isn't stable yet, so we just have to update and address the breaks. cc: @eerhardt @sebastienros

opened https://github.com/dotnet/aspire/issues/4466

aaand Dependabot comes to life and opens PR's for many of these. @eerhardt if you can answer the question above (or we can snip that update) I suggest we merge this one as it will likely avoid merge conflicts. Any remaining ones we can merge after. @joperezr
Also I think this should be the last time we do this so manually

I'm pretty sure the CodeAnalysis update is okay to take too as that source generator we only use internally, but totally okay to update that later too. Nice to hear that dependabot decided to work again :)

Aspire.Workload.Tests.StarterTemplateRunTests.ResourcesShowUpOnDashboad [1 s]
  Error Message:
   System.AggregateException : One or more errors occurred. (
╔══════════════════════════════════════════════════════════╗
║ Host system is missing dependencies to run browsers.     ║
║ Please install them with the following command:          ║
║                                                          ║
║     sudo pwsh bin/Debug/netX/playwright.ps1 install-deps ║
║                                                          ║
║ Alternatively, use apt:                                  ║
║     sudo apt-get install libgbm1\                        ║
║         libxkbcommon0                                    ║

@radical ?

Aspire.Workload.Tests.StarterTemplateRunTests.ResourcesShowUpOnDashboad [1 s]
  Error Message:
   System.AggregateException : One or more errors occurred. (
╔══════════════════════════════════════════════════════════╗
║ Host system is missing dependencies to run browsers.     ║
║ Please install them with the following command:          ║
║                                                          ║
║     sudo pwsh bin/Debug/netX/playwright.ps1 install-deps ║
║                                                          ║
║ Alternatively, use apt:                                  ║
║     sudo apt-get install libgbm1\                        ║
║         libxkbcommon0                                    ║

@radical ?

Ah, I see it..
+ sudo apt-get install -y libgbm1 libxkbcommon0
E: Could not get lock /var/lib/dpkg/lock-frontend. It is held by process 1669 (dpkg)
E: Unable to acquire the dpkg frontend lock (/var/lib/dpkg/lock-frontend), is another process using it?

apt-get failed to install the dependencies, but we don't stop the test run because of that. And so it goes on to fail later in the run. Re-run should "fix" it for this PR.
In a separate one I can make it fail immediately if apt-get fails. Additionally, we should have some retries around this, and I remember looking at it for dotnet/runtime, and I can try to add something here.

Tracking it in https://github.com/dotnet/aspire/issues/4487 .

Thanks

Looks like the Azure OpenAI package has breaking changes in 2.x. Reverting that.

Ooh, first interesting one then. I guess for this particular case it is okay because our component that references that isn't stable yet, so we just have to update and address the breaks. cc: @eerhardt @sebastienros

This is known. The Azure SDK team rewrote the OpenAI library. See https://devblogs.microsoft.com/dotnet/openai-dotnet-library/.
Thanks for opening https://github.com/dotnet/aspire/issues/4466 to track this. I've also opened https://github.com/dotnet/aspire/issues/4567 to track adding a plain "OpenAI" component given this new library.


Problem with webpack

Hello. Thanks very much for the library, is very useful.
I found a problem when trying to make my app work for ie11, using webpack and babel, it happened that 'react-dropdown-aria' was not transpiled properly, leaving 'const' and default values (like {selected = false}, that broke ie11.
Investigating this I found that your distribution is in ES6 and that causes these problems.
Maybe better to make the dist for ES5?
I solved the problem in my end just making a copy of the library, then babel with webpack transpiles it, but I suppose the best way for the library itself would be to make it more compatible.
Thanks for your help.

@jfangrad I am facing the same issue in chrome and firefox as well.
@nachocs Did you take a clone of the project and build it in a different way? Can you please explain what you did?

@nachocs @sachinantony without your build config files it is hard for me to help out as I can't recreate the issues myself. I would be happy to help if you can provide me with that information!

I just cloned the library and used internally in the project, and it works fine like that.
I tried a lot of possible configurations on webpack with no luck. All rest of libraries have no problem. I only found it out when testing on IE11
Here you have my webpack config:
`import path from 'path';
import webpack from 'webpack';
import merge from 'webpack-merge';
import CleanWebpackPlugin from 'clean-webpack-plugin';
import MiniCssExtractPlugin from 'mini-css-extract-plugin';
import TerserJSPlugin from 'terser-webpack-plugin';
import OptimizeCSSAssetsPlugin from 'optimize-css-assets-webpack-plugin';
const sassPaths = ['node_modules'];
// References the application's root path
const ROOT_PATH = path.resolve(__dirname);
// Sets the variable as either development or production
const ENV = process.env.NODE_ENV || 'development';
// Sets appEnv so the the header component will point to the search app on either Dev or Prod
const appEnv = process.env.APP_ENV ? process.env.APP_ENV : 'production';
// Holds the common settings for any environment
const commonSettings = {
// path.resolve - resolves to an absolute path
// This is the path and file of our top level
// React App that is to be rendered.
entry: ['@babel/polyfill', path.resolve(ROOT_PATH, 'src/client/App.jsx')],
resolve: {
extensions: ['.js', '.jsx'],
},
output: {
// Sets the output path to ROOT_PATH/dist
path: path.resolve(ROOT_PATH, 'dist'),
// Sets the name of the bundled application files
// Additionally we can isolate vendor files as well
filename: 'bundle.js',
},
plugins: [
// Cleans the Dist folder after every build.
// Alternately, we can run rm -rf dist/ as
// part of the package.json scripts.
new CleanWebpackPlugin(),
new MiniCssExtractPlugin({
filename: 'styles.css',
}),
new webpack.DefinePlugin({
loadA11y: process.env.loadA11y || false,
appEnv: JSON.stringify(appEnv),
}),
],
};
/**

DEVELOPMENT ENVIRONMENT CONFIG

Uses the webpack-merge plugin to merge
the common app configuration with the
additional development specific settings.

*/
// Need to configure webpack-dev-server and hot-reload
// module correctly.
if (ENV === 'development') {
module.exports = merge(commonSettings, {
mode: 'development',
devtool: 'inline-source-map',
entry: {
app: [
'@babel/polyfill',
'webpack-dev-server/client?http://localhost:3000',
'webpack/hot/only-dev-server',
path.resolve(ROOT_PATH, 'src/client/App.jsx'),
],
},
output: {
publicPath: 'http://localhost:3000/',
},
plugins: [
new MiniCssExtractPlugin({
filename: 'styles.css',
}),
new webpack.HotModuleReplacementPlugin(),
new webpack.DefinePlugin({
'process.env': {
APP_ENV: JSON.stringify('development'),
},
}),
],
resolve: {
modules: ['node_modules'],
extensions: ['.js', '.jsx', '.scss'],
},
module: {
rules: [
{
test: /.jsx?$/,
exclude: /(node_modules|bower_components)/,
use: 'babel-loader',
},
{
test: /.scss?$/,
use: ['style-loader', 'css-loader', sass-loader?includePaths=${sassPaths}],
include: path.resolve(ROOT_PATH, 'src'),
},
{
test: /.(gif|png|jp?g|svg)(?v=\d+.\d+.\d+)?$/,
use: [
{
loader: 'file-loader',
options: {
name: '[name].[ext]',
outputPath: 'img/',
},
},
],
},
{
test: /.(ttf|eot|woff(2)?)(?v=\d+.\d+.\d+)?$/,
use: [
{
loader: 'file-loader',
options: {
name: '[name].[ext]',
outputPath: 'fonts/',
},
},
],
},
],
},
});
}
/**

PRODUCTION ENVIRONMENT CONFIG

Uses the webpack-merge plugin to merge
the common app configuration with the
additional production specific settings.

*/
if (ENV === 'production') {
module.exports = merge(commonSettings, {
mode: 'production',
plugins: [
new MiniCssExtractPlugin({
filename: 'styles.css',
}),
new webpack.DefinePlugin({
'process.env': {
NODE_ENV: JSON.stringify('production'),
APP_ENV: JSON.stringify(appEnv),
},
}),
],
optimization: {
minimizer: [new TerserJSPlugin({}), new OptimizeCSSAssetsPlugin({})],
},
module: {
rules: [
{
test: /.jsx?$/,
exclude: /(node_modules|bower_components)/,
use: 'babel-loader',
},
{
test: /.scss?$/,
use: [MiniCssExtractPlugin.loader, 'css-loader', sass-loader?includePaths=${sassPaths}],
include: path.resolve(ROOT_PATH, 'src'),
},
{
test: /.(gif|png|jp?g|svg)(?v=\d+.\d+.\d+)?$/,
use: [
{
loader: 'file-loader',
options: {
name: '[name].[ext]',
outputPath: 'img/',
},
},
],
},
{
test: /.(ttf|eot|woff(2)?)(?v=\d+.\d+.\d+)?$/,
use: [
{
loader: 'file-loader',
options: {
name: '[name].[ext]',
outputPath: 'fonts/',
},
},
],
},
],
},
});
}
`

@nachocs @sachinantony I have released a new version 1.3.1 of the library. I got it to work in one of my own projects with IE11 so let me know if the updated version works for you guys

Thank you very much

Closing issue now that build has changed to support ie11. Reopen if any issues.


fix: remove duplicate path ids from spectrum-icons.svg

Description
Should fix Selenium test failures:

"Document has multiple elements with the same id attribute: spectrum-icon-24-StartConversion"
"Document has multiple elements with the same id attribute: _256"

Related issues: https://jira.corp.adobe.com/browse/GRANITE-33028 and https://github.com/adobe/coral-spectrum/pull/173.
Related Issue

https://jira.corp.adobe.com/browse/GRANITE-33028
https://github.com/adobe/coral-spectrum/pull/173

Motivation and Context
Automated accessibility testing will report an error message when there are one or more duplicate ids in a document. In some cases, as with "spectrum-icon-24-StartConversion" a symbol may be duplicated in the svg, while elsewhere, as with paths with id="_256", SVG editing software might generate an id for a path, and if the path is then copied to another symbol, the id might be duplicated.
How Has This Been Tested?
Tested using aXe Chrome extension against /coral-component-icon/examples/index.html
Screenshots (if appropriate):
Types of changes

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING document.

@Pareesh can you please review this?


设备上如果屏幕底部有虚拟按键栏，这个按键栏会覆盖视图，如何解决呢

Describe the bug
A clear and concise description of what the bug is.

The version of utilcode:
The device:
The version of device:

The code of bug

put your code here

The stack of crash

put the stack of crash here

Screenshots
If applicable, add screenshots to help explain your problem.
Please delete the current line and the followings.
Thank you for supporting AndroidUtilCode.

这和我库有什么关系？ 你可以设置导航栏透明度为 0 呢？


How postgresql Db can be exported from docker? Or how we can get the path for products.csv.gz to download. Any suggestions

hi @amitlals, I'm unsure what exactly your asking for here. Is it that you want to connect to the cloud pricing API database locally?

Yes! Thanks for your response. I am exploring if I could get the zip file directly before it goes Postgresdb in docker. Not sure if works? Cheers!

Hi @amitlals you can just run the dataDownload command https://github.com/infracost/cloud-pricing-api/blob/60f7632eff19f0cba93653c2495cddc5bad2dc06/package.json#L14 which will download the file locally.

Hi @amitlals you can just run the dataDownload command https://github.com/infracost/cloud-pricing-api/blob/60f7632eff19f0cba93653c2495cddc5bad2dc06/package.json#L14 which will download the file locally.

Thanks again! Pls elaborate how this command works, is it works infacost EXE? or something? sorry for unaware here. Thanks

@amitlals you can follow this guide and after step 5 you can run the command that Hugo mentioned (npm run data:download), which downloads the file you want. Notice how this is part of the Cloud Pricing API, not part of the Infracost CLI repo.


CTD-007

cleaned up console logs and fixed html hierarchy

Console logs removed and html hierarchy updated.

ok


Do a bit refactoring

Changes
Close #1136
Checklist

[ ] An issue to be fixed by this PR is listed above.
[ ] New tests are added to ensure new features are working. End-to-end tests are usually in the ./frb_example/pure_dart example, more specifically, rust/src/api.rs and dart/lib/main.dart.
[ ] The code generator is run and the code is formatted (via just precommit).
[ ] If this PR adds/changes features, documentations (in the ./book folder) are updated.
[ ] CI is passing.

Remark for PR creator

Justfile is a task runner for the command line interface that allows you to run shell commands from a file named justfile in your project directory. You can use Justfile after installing it. Note that commands written in justfile of this repository are expected to be run in bash, not cmd or powershell. Running just ... commands in cmd or powershell will produce errors as the syntax is not compatible. On Windows, you can use git bash if you have Git installed.
If fzyzcjy does not reply for a few days, maybe he just did not see it, so please ping him.

Given that #1117 has not been merged yet, I should not touch ir, parser, and generator - otherwise it will be time-consuming for that PR to resolve conflicts


Proof of concept passing data around between Python and R

To do:

[x] Make the reticulate package optional
[x] Actually run the tests
[x] Confirm the r_to_py works
[ ] Update CI to install and test appropriately
[x] Add support for RecordBatch (will do Table/ChunkedArray later)

@nealrichardson I made two changes on my branch:

rename export_to_c (resp. import_from_c) to _export_to_c (resp. _import_from_c)
add the same methods for record batches (i.e. on the pa.RecordBatch class)

@ursabot build

@pitrou would you mind rebasing this? I don't know how to resolve the cpp conflicts. For the r/R/arrow-package.R conflict, we want to keep all of the lines that conflict; order doesn't matter.

I've pushed changes. Unfortunately I couldn't check that this compiles.

Fixed.
How do you want to proceed with this? Do you want to merge this into your c-data-interface-impl branch? Or merge that first and then I can make a followup JIRA for my branch?

Thanks for the fix. I think I'll merge into my PR.

I see this is merged but I'm not seeing the changes show up in https://github.com/apache/arrow/pull/5608, is that expected?

Uh, you're right. I must have messed something. Let me try again.


use separate type for timeline.rs's representation of Timeline state

This patch introduces crate::tenant::timeline::State which has the same variants as pageserver_api::models::TimelineState.
This will enable future commits to change state handling in modules tenant and timeline without affecting the presentation in the API.

I was also thinking about this:

This will enable future commits to change state handling in modules tenant and timeline without affecting the presentation in the API.

So is the problem that we encode the variants to the openapi yaml?

So is the problem that we encode the variants to the openapi yaml? This change would be the way to go if we'd have this enum and that enum in different repos, in released crates, but we currently don't have that issue.

Nothing outside of pageserver depends on the state variable (thank god!) but a lot of python tests check it.

Thanks for the review! Since this patch is so small and is just confusing by itself, I'll let it sit until I have the WIP on top of this ready.

As discussed moving to draft


Improve makefile and readme

I'm having this error when running make run-all in a fresh install
COMPOSE_DOCKER_CLI_BUILD=1 docker compose -p issuer -f /home/xime/devel/lala/issuer-node/infrastructure/local/docker-compose.yml up -d ui api pending_publisher notifications
WARN[0000] /home/xime/devel/lala/issuer-node/infrastructure/local/docker-compose.yml: the attribute `version` is obsolete, it will be ignored, please remove it to avoid potential confusion
env file /home/xime/devel/lala/issuer-node/.env-api not found: stat /home/xime/devel/lala/issuer-node/.env-api: no such file or directory

Solved by running a touch .env-api before. Not sure if I'm doing something wrong, if we should document this touch in readme or if we should run this touch in some place

I'm having this error when running make run-all in a fresh install
COMPOSE_DOCKER_CLI_BUILD=1 docker compose -p issuer -f /home/xime/devel/lala/issuer-node/infrastructure/local/docker-compose.yml up -d ui api pending_publisher notifications
WARN[0000] /home/xime/devel/lala/issuer-node/infrastructure/local/docker-compose.yml: the attribute `version` is obsolete, it will be ignored, please remove it to avoid potential confusion
env file /home/xime/devel/lala/issuer-node/.env-api not found: stat /home/xime/devel/lala/issuer-node/.env-api: no such file or directory

Solved by running a touch .env-api before. Not sure if I'm doing something wrong, if we should document this touch in readme or if we should run this touch in some place

.env-api is no longer needed. Docker Compose dependency with that file has to be removed.


copy button invisible in light mode Refactor CodeBlock component butt…

…on styling
PR Fixes:

1 Invisible copy button in light mode on codeblocks

Resolves #645
Checklist before requesting a review

[  ] I have performed a self-review of my code
[  ] I assure there is no similar/duplicate pull request regarding same issue

Before-

After-

There's this issue #647  where yarn.lock file is empty so i did not make any changes to yarn.lock file.. i'll update this file after this issue is resolved or please tell me if there's any changes that i need to make...


284 clear proofs

Clear Proof and Proof Drag working

Looks like it might solve issue in pr #279

Because this branch was the merged version of drag in proof and proof history I just added my fix that was needed on PR #264. This is essentially the PRs #264, #279 along with the issue for clear canvas.


docs: Provide clarity around only one active metric server in operate docs

Provide clarity around only one active metric server in operate docs which were removed in #940
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO)

I love DCO

Tangentially related to this, what does the Support Replicas column refer to here?

It's not because you can that it is supported, see https://github.com/kedacore/keda-docs/pull/940#discussion_r979884549

It's not because you can that it is supported, see #940 (comment)

It's not because you can that it is supported, see #940 (comment)

I am not sure what you mean by you can that it is supported.

You can run multiple instances but KEDA only provides support on running 1 for now

Can it be changed to Supported or Recommended?

Well... We don't support it so using recommended would be a bit confusing when things are on fire IMO.
But leaving it up to @JorTurFer & @zroubalik


fix Fifo queue delivers duplicate message(s)

Coverage decreased (-0.2%) to 94.341% when pulling 4202dfb7326f98a49b4402c694e633bbdd712485 on irahulranjan:localstack_issue_3339 into effb075b62cdb8862b9fa58ff272863c67e84635 on spulec:master.

@BeyondEvil can you please help me out with the CI failure. I have run the tests in my machine using python 2.7, 3.6, 3.7, and 3.8 and all the tests are passing.

Hey @irahulranjan, the test are all failing in server mode, where Moto is run in a standalone Docker container.
The @mock.patch has no effect there, as it wouldn't change the value of DEDUPLICATION_TIME_IN_SECONDS inside the Docker container.
I don't really see how we would be able to test this inside a Docker container, so we can skip it in server mode like this:
https://github.com/spulec/moto/blob/master/tests/test_sqs/test_sqs.py#L1491

@bblommers Thanks for the help.

Hey @irahulranjan, the test are all failing in server mode, where Moto is run in a standalone Docker container.
The @mock.patch has no effect there, as it wouldn't change the value of DEDUPLICATION_TIME_IN_SECONDS inside the Docker container.
I don't really see how we would be able to test this inside a Docker container, so we can skip it in server mode like this:
https://github.com/spulec/moto/blob/master/tests/test_sqs/test_sqs.py#L1491

Not exactly sure how the docker setup works, but you could change the DEDUPLICATION_TIME_IN_SECONDS to try and get it's value from the environment instead:
DEDUPLICATION_TIME_IN_SECONDS = os.getenv('DEDUPLICATION_TIME_IN_SECONDS', 300)

Then the test can set the env var to 1, either that will then get picked up by the docker runner automagically or it can be passed to the container when started.
Hope that makes sense @irahulranjan @bblommers

@BeyondEvil True, there are ways around it, but none of them pretty.
Changing the constant to read the environment variable means changing the code to make it more testable - which is always a code smell.
Passing the env-var to the Docker container means that value is set for the duration of the container lifetime - and we really only want to change it for the duration of a single test.
The weird travis.yml-notation is not setting the variable, but is used as a list of configurations. Travis will execute the tests once per configuration option (per python-version). That's why the list of test suites is executed twice per language, one with the env var set to True, one set to False.

@BeyondEvil True, there are ways around it, but none of them pretty.
Changing the constant to read the environment variable means changing the code to make it more testable - which is always a code smell.

Fair enough.

Passing the env-var to the Docker container means that value is set for the duration of the container lifetime - and we really only want to change it for the duration of a single test.

True.
It's already merged. But does moto support some type of configuration framework? I guess something like that could've been leveraged in a scenario like this.

The weird travis.yml-notation is not setting the variable, but is used as a list of configurations. Travis will execute the tests once per configuration option (per python-version). That's why the list of test suites is executed twice per language, one with the env var set to True, one set to False.

Oh, really. Wow, that is weird. 😅
Thanks for taking the time to explain. 🙏

It's already merged. But does moto support some type of configuration framework? I guess something like that could've been leveraged in a scenario like this.

No, not yet! I agree that would be useful, and will become more useful as we're getting more config options. One day.. 🤞

This is now part of moto >= 1.3.16.dev181


Debug help?  Problem w/paddle or underlying eldap?

I don't know whom to contact, but I'm feeling like I've hit a dead-end.  I can run an ldap query using ldapsearch and it returns valid results.  But the seemingly same query from paddle returns no results.
A redacted ldapsearch (which runs successfully) looks like:
$ ldapsearch -W -H ldaps://ldap.domain\
  -D cn=username@loc,ou=users,dc=domain\
  -b ou=users,dc=domain\
  '(&(cn=username@loc)(memberOf=cn=Developers,cn=roles,dc=domain))'
... data with valid search result
# search result
search: 2
result: 0 Success
# numResponses: 2
# numEntries: 1

The code I have in elixir looks like:
result = Paddle.get(filter: [cn: username,
                             memberOf: "cn=Developers,cn=roles,dc=domain"])
Logger.error("GROUP result=#{inspect result}")

And the output from this makes it /appear/ that the search query ran in the same manner as the ldapsearch command above, but it has no results.
20:51:40.691  level=debug Getting entries with dn: ou=users,dc=domain and filter: {:and,
             [equalityMatch: {:AttributeValueAssertion, 'cn', 'username@loc'},
              equalityMatch: {:AttributeValueAssertion, 'memberOf', 'cn=Developers,cn=roles,dc=domain'}]}
20:51:40.692  level=debug search request = {'SearchRequest',"ou=users,dc=domain",
                     wholeSubtree,derefAlways,0,0,false,
                     {'and',
                         [{equalityMatch,
                              {'AttributeValueAssertion',"cn",
                                  "username@loc"}},
                          {equalityMatch,
                              {'AttributeValueAssertion',"memberOf",
                                  "cn=Developers,cn=roles,dc=domain"}}]},
                     []}
20:51:40.729  level=debug search reply = {ok,{'LDAPMessage',4,
                       {searchResDone,
                           {'LDAPResult',success,[],[],asn1_NOVALUE}},
                       asn1_NOVALUE}}
20:51:40.729  level=debug search reply = searchResDone
20:51:40.729  level=error GROUP result={:error, :noSuchObject}

I am not certain what the debug line2 wholeSubtree... is, and how it relates to the search.  My guess is something with the context of the search is different enough that it is not matching.
Any help would be greatly appreciated.  Thx.

scratch this as an error, I restarted the service and it is now working.  I suspect it is the Genserver had died on me.


Implement approximate suggest for descendants of unaudited dependencies

See (1) in the sketch at https://github.com/bholley/cargo-vet/issues/22#issuecomment-1100798406, and the ensuing discussion.
The current implementation landed in #31 makes no suggestions for such descendants (see the long comment).

i haven't written tests for this but this should naturally fall out of unaudited entries now having criteria and therefore being desugarred to a full-audit with a flag noting that it would be nice to not use it.

i haven't written tests for this but this should naturally fall out of unaudited entries now having criteria and therefore being desugarred to a full-audit with a flag noting that it would be nice to not use it.

I don't really parse this.
The current implementation doesn't do this at all because suggest doesn't traverse past unaudited dependencies. I think we actually want an option to do this (e.g. --deep), and then we'd need the approximation heuristic.

oh this might be an ambiguity -- do you mean "dependencies that lack audits" or "dependencies with unaudited entries"

I meant "dependencies that lack audits" — since cargo vet suggest already eliminates all the unaudited entries.
Per matrix discussion, we're just going implement --deep in a criteria-less fashion for now.

this is now implemented with cargo vet suggest --guess-deeper


[SYSTEMML-1895] jcuda for win & linux (x86_64, ppc64le) included in extra jar

Ping @asurve
As per our offline discussion, we are treating SYSTEMML-1895 as a hotfix. This patch has thus gone into master and branch-0.15 separately.

LGTM to get changes to branch-0.15

Thanks, I shall merge.

Refer to this link for build results (access rights to CI server needed):
https://sparktc.ibmcloud.com/jenkins/job/SystemML-PullRequestBuilder/2069/


Cancel Jobs

Utilizing VSCdode. I have some jobs that are "stuck" on and run whenever I use the "func  host start" command. Even restarting the computer doesn't remove them! Any suggestions on how to issue a cancel command to them?

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

 Or only when using VScode?

@CKeene78 any update on this?

Behavior on both VSCode and CLI.
Turns out the jobs were "stuck" in the Azure Storage webjobs storage. Once I cleared them out of there the phantom jobs stopped appearing.

@CKeene78 thanks for that update! Glad to hear you were able to figure it out. I will close this but if you have more questions just let me know :)


update release note for SSDT for VS2017 15.9.7

@fan130 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@fan130 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@fan130 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

#sign-off


feat: added url parsing to the filter

Closes #1260
Added some url parsing using urllib.parse to stop false positives. to remove false positives, I parsed the URLs and checked if the netloc was the same. If it wasn't, I passed. I also check if the netloc is prefixed with www. or any other subdomain should people try to circumvent the filter. An example false positive would be deliciouscookies.com triggering cookies.com, a blacklisted url (not linking actual example cause it's NSFW)

Can you provide examples of false positives? Is there a related issue?

@mbaruh yes, I'll link the issue and provide examples in the initial description

As a heads up, poetry.lock got all of the dependencies updated.
To fix this, please be sure you are using poetry 1.1.x
First revert poetry.lock locally.
Ensure that pyproject.toml is the same, with tldextract in it.
Next, use poetry lock --no-update This will relock poetry without updating all of the dependencies.
However, it may be worth an update seperately to the dependencies, as this updated redis, rapidfuzz, sentry, etc

this has been deemed not to be a problem as of https://discord.com/channels/267624335836053506/635950537262759947/916168658651324487

this has been deemed not to be a problem as of https://discord.com/channels/267624335836053506/635950537262759947/916168658651324487

After looking into this further, markdownify cannot be updated, as it will nerf the results of the doc command.

Fixed that change in GH-2014 👍


The entire wiki was deleted why is that

i am talking abt the github wiki not the origins documentation one

You're on the wrong repository. This is the repository for the example data packs, it never had a GitHub wiki. Try this link: https://github.com/apace100/origins-fabric/wiki


Fluid Yellorium could not be picked up with a bucket.

Hello,
I'm currently running the Direwolf20 Modpack and I have a problem with my Storage System due to the fact, that Fluid Yellorium can not be picked up in a bucket.

I want to store a few Ores in liquid form 'cause that would save a lot of space. I have a network of multiple Smelteries from TC and multiple Bedrockium Drums (as seen in the screenshot). Every Input of a Drum has a Filter.
Here is Gold as a example:

But for the yellorium filter i had to cheat the actual fluid in 'cause the Yellorium Bucket couldn't define the filter. And there are 2 types of yellorium and both have to be in the filter to work properly.

As you can see in the screenshot the fluid is not well localized here (in NEI it is). The name of the other fluid is fluid.aobd.aobdyellorium.
Also I can't take the Fluid Yellorium out of the drum with a bucket.
I don't know how to explain that better, and sorry for my bad english. I'm native german.
(I can provide more information when needed)

The liquid yellorium from BigReactors is not intended to be used as a fluid to store the yellorium, it is a placeholder that is used for BigReactors internal fuel handling in the reactor. Because of the fluid not being implemented as a useable fluid, it has already even caused crashes.

okay thanks for the info, good to know.
Poking @erogenousbeef - Could it be poissble to implement Fluid Yellorium as a useable fluid?

fluid.aobd.aobdyellorium is automatically created by the mod Another One Bites the Dust to add Tinkers' Construct and Mariculture melting recipes for Yellorium ingots (ore multiplication via smeltery/crucible), any issues related to that one of the fluids is not a Big Reactor issue (and it really should not affect the filter for the liquid fuel, though it should solve your storage problem as it allows molten yellorium)


Make grs work with deleted files

I changed the alias for 'grs' to 'git reset --'. This way, it also works when you add a deleted file to the index and want to unstage that deletion.
To reproduce:

Create empty repository
Add a single file and commit
grm
grs

Without the patch, the last command will fail.

This is really helpful, thanks! That did annoy me from time to time, I wasn't aware that -- would fix this.


Appstudio update test-component-pac-wzds

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-wzds-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
8 seconds
init

✅ Succeeded
20 seconds
clone-repository

✅ Succeeded
43 seconds
build-container

✅ Succeeded
1 minute
sast-snyk-check

✅ Succeeded
27 seconds
inspect-image

✅ Succeeded
25 seconds
deprecated-base-image-check

✅ Succeeded
10 seconds
clair-scan

✅ Succeeded
40 seconds
clamav-scan

✅ Succeeded
10 seconds
sbom-json-check

✅ Succeeded
25 seconds
label-check

✅ Succeeded
5 seconds
show-sbom

✅ Succeeded
5 seconds
show-summary

Pipelines as Code CI/test-component-pac-wzds-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
8 seconds
init

✅ Succeeded
20 seconds
clone-repository

✅ Succeeded
1 minute
sast-snyk-check

✅ Succeeded
43 seconds
build-container

✅ Succeeded
27 seconds
inspect-image

✅ Succeeded
25 seconds
deprecated-base-image-check

✅ Succeeded
10 seconds
clair-scan

✅ Succeeded
40 seconds
clamav-scan

✅ Succeeded
10 seconds
sbom-json-check

✅ Succeeded
25 seconds
label-check

✅ Succeeded
5 seconds
show-summary

✅ Succeeded
5 seconds
show-sbom


Verify if target de-registration works with telemetry on.

Yes, this works, but be aware that if the leader is deregistered, all targets are unhealthy, so traffic is sent to the remaining (unhealthy) targets. This means the /v1/sys/health will return a non-200 response for a while.
This may have an effect on the route53 or load balancer that route traffic to clusters.


move first output after sanity check

It's not possible to change response headers after you write to response stream.
This will fix redirects listed in the routes/index.js

Makes perfect sense 👍


Make it possible to run the mock version of Vuforia as a server

From @adamtheturtle on December 21, 2016 15:10
Like Mimic
The idea I have for this:

[X] Create a Flask mock
[X] Make the verify-fake fixture also use the Flask mock, like in https://github.com/adamtheturtle/todo/blob/master/todoer/tests/test_todoer.py#L36
[X] WONTDO: ~Maybe replace the current implementation of the mock with that - if it is as fast~
[X] Allow the wrapper to take a host argument
[X] Add docker containers to run that mock

We need some way of configuring databases on the image

[X] WONTDO (hadolint isn't on apt and is a bit tricky to get in a nice way for Linux users without Docker) Lint Dockerfiles (e.g. https://github.com/hadolint/hadolint)
[x] Document how to build the images and run the docker containers
[x] Rename the "storage" backend (and associated configuration variables) to the "Target Manager"
[x] Add restrictions to the Flask storage backend on which databases can be created (e.g. no databases with duplicate names - this is like the decorator and can maybe share logic)
[ ] Add a release step to build and host the Docker images, and document using the containers from Docker Hub

Copied from original issue: adamtheturtle/vws-python#58

The idea I have for this:

Create a Flask mock
Add a docker container to run that mock
Make the verify-fake fixture also use the Flask mock, like in https://github.com/adamtheturtle/todo/blob/master/todoer/tests/test_todoer.py#L36

Maybe replace the current implementation of the mock with that - if it is as fast

Allow the wrapper to take a host argument, and allow setting a custom host on the mock so that they can connect.

One image will host both the query server and the image server, unless they somehow share a database
We need some way of configuring databases on the image

Have a look at https://github.com/timothycrosley/preconvert for serialising the dataclasses

Look into https://github.com/marketplace/actions/build-and-push-docker-images


Add docs for max passwords attempts affecting mfa attempts

We share the password attempts API for MFA attempts. We should document this.
Additionally we should probably add a test to ensure the password attempt is recorded.

Additionally we should probably add a test to ensure the password attempt is recorded.

Here's one we prepared earlier:
https://github.com/silverstripe/silverstripe-mfa/blob/0454110/tests/php/Authenticator/LoginHandlerTest.php#L485

Docs PR at #284

Here's one we prepared earlier:
/tests/php/Authenticator/LoginHandlerTest.php@0454110#L485

Damn. I went looking on my phone and I couldn't find it 😂

Merged #284


Could this extension work on embed tweets?

I don't know how hard (or time-worthy since Musk is going to be doing and undoing a lot of features) it would be to implement this extension to embed tweets. So, I'm asking if it's possible to incorporate this in embedded tweets. Thanks.

Hey, this should be working on embedded tweets already. I believe I found a major bug in my code which was causing it to occasionally fail (generating most of the bug reports here). I'm going to close this and if it's still not working after v1.2 we can revisit it and see if there are any other changes needed. Thank you for raising this!  https://github.com/wseagar/eight-dollars/commit/a1d053706e70e7c62b06ffca040d52a6499d23ba


Can't convert "Sorry, something went wrong. Please try again."

Hi, there seems to be some problem with http://www.youtubeadl.com/ I get "Sorry, something went wrong. Please try again." when submitting URLs

Hi Mattias, thanks for reporting this and sorry for the delay! The service should be back up and running now.

No problem, thanks for fixing!

Still a bit flaky, trying to convert https://www.youtube.com/watch?v=RINk9MhQ85Q to long?

Hi, seems to be broken again 😞

Hey Mattias,
I reverted back my old code and it should be working fine now and should convert faster.

👍


[BUG] Multipart/form-data request is not being generated 🐛

Hi guys!
Describe the bug
We are generating our current specification and some of the services are not properly generated. All of the multipart/form-data services are being created as common @Post calls without any @PartFile attributes, leading us on not being able to use the generator for those services. Our spec is attached for your references (using OpenAPI 3.0).
To Reproduce
 "/account/files": {
      "get": {
        "summary": "List all the user files",
        "operationId": "listFiles",
        "tags": [
          "Account"
        ],
        "responses": {
          "200": {
            "content": {
              "application/json": {
                "schema": {
                  "$ref": "#/components/schemas/UserFiles"
                }
              }
            },
            "description": "An array of File"
          },
          "default": {
            "$ref": "#/components/responses/Problem"
          }
        }
      },
      "post": {
        "summary": "Uploads File",
        "description": "Empty description",
        "operationId": "uploadFile",
        "requestBody": {
          "content": {
            "multipart/form-data": {
              "schema": {
                "type": "object",
                "required": [
                  "id",
                  "file"
                ],
                "properties": {
                  "id": {
                    "type": "string"
                  },
                  "file": {
                    "type": "string",
                    "format": "binary"
                  }
                }
              }
            }
          },
          "required": true
        },
        "responses": {
          "default": {
            "$ref": "#/components/responses/Problem"
          }
        }
      }
    },

Expected behavior
The multipart request to be created properly.
Library version used:
2.4.9
Are we missing something?

Hi @alextarrago could you please share code yo expect to be generated here?

@alextarrago  closing an issue due to no answer. Please reopen it with example of expected generated code if it will be actual.

@Vovanella95 we have also ran into this issue today
we have tried our generator with our json and the example json in this repo
It appears that the @multipart decorator isn't being added to multipart requests as per the chopper documentation
https://pub.dev/documentation/chopper/latest/chopper/Multipart-class.html
Let me know if you need anymore information about this issue, thanks in advance!

Hi @andywearearch
Can wegenerate code somehow, to fix it? If yes - please provide example of swagger request and expected generated code for it.

@Vovanella95
I have done a bit of digging and im pretty sure this is what needs to be generated from the following swagger yaml/json
I have referenced both the chopper documentation linked above and the swagger multipart file documentation here (Open API Spec 3.0):
https://swagger.io/docs/specification/describing-request-body/file-upload/
        content:
          multipart/form-data:
            schema:
              type: object
              description: Avatar File Upload
              required:
                - file
              properties:
                file:
                  type: string
                  format: binary

Should generate something like this:
@Post(path: '/avatars')
@Multipart()
 Future<chopper.Response<Avatar>> _avatarsPost(@PartFile() List<int> file);

I believe you will be able to test this using the example in your repo if you update your swagger file to use the OAS 3.0 and match what is described above
Unfortunately i cannot send any code repos but this should be enough information to generate something your side
I am happy to help test this out on a specific commit if needed as well
Again let me know if you need any more info, cheers

@andywearearch thanks for researching. I'm reopening the issue and will fix it!

Hi @andywearearch  @alextarrago
I've made initial implementation. Could you please try swagger generator version from branch and check, is it works as expected?
Your swagger example now generated with PartFile parameter and Multipart annotation.

Not Generating any parameter
version: swagger_dart_code_generator: ^2.8.1
     "/test/upload": {
            "post": {
                "operationId": "uploadFile",
                "parameters": [],
                "requestBody": {
                    "required": true,
                    "content": {
                        "multipart/form-data": {
                            "schema": {
                                "$ref": "#/components/schemas/StorageDto"
                            }
                        }
                    }
                },
                "responses": {
                    "201": {
                        "description": ""
                    }
                },
                "tags": [
                    "Test"
                ],
                "security": [
                    {
                        "FireToken": []
                    }
                ]
            }
        }

        "StorageDto": {
                "type": "object",
                "properties": {
                    "image": {
                        "type": "file",
                        "nullable": true,
                        "properties": {
                            "image": {
                                "type": "string",
                                "format": "binary"
                            }
                        }
                    },
                    "fileId": {
                        "type": "string",
                        "nullable": true,
                        "description": "Image Path",
                        "example": "fileId/{name}.png"
                    }
                }
            }

Generating like this
  @Post(path: '/test/upload')
  Future<chopper.Response> _testUploadPost();

Hi @Vovanella95, can you please help me with this issue?
i can't able to generate code is the API is form-data request

Hi @anandsubbu007, I'm on business trip till 22 Oct. I've reopened this issue, will handle it asap when return back.

Hi @anandsubbu007, I've checked your issue. I'm not really competent in multipart requests, so I need Your help.
Is it will be fine to generate this method next way? If yes - I will generate it this way. If no - please share expected generated method result.
Future<chopper.Response> testUploadPost({String? cacheControl, StorageDto? body,}) {
    return _testUploadPost(cacheControl: cacheControl, body: body);
  }

  ///
  @Post(path: '/test/upload')
  @Multipart()
  Future<chopper.Response> _testUploadPost(
      {@Header('Cache-Control') String? cacheControl,
      @Body() StorageDto? body,});

Thanks a lot and let's do this generator better together :)

Hi @Vovanella95, Thank For This Generator Package.
yes, i think so.

@anandsubbu007 cool, I will make fix in day or two. Thanks for quick answer!

Fixed

when i use multipart/form data.
Enumname is not generating

@anandsubbu007  please share part of swagger file and expected result

@Vovanella95  Sorry, for late response.
Still This warning is showing
POST test/upload
Body is null
Use @Body() annotation on your method parameter to provide a body to your request
   e.g.: Future<Response> postRequest(@Body() Map body);
Or explicitly suppress this warning by setting the optionalBody property
   e.g.: @Post(optionalBody: true)

If i use this body with enum property. it is not generating
            "StorageDto": {
                "type": "object",
                "properties": {
                    "image": {
                        "type": "file",
                        "nullable": true,
                        "properties": {
                            "image": {
                                "type": "string",
                                "format": "binary"
                            }
                        }
                    },
                    "fileId": {
                        "type": "string",
                        "nullable": true,
                        "description": "Image Path",
                        "example": "fileId/{name}.png"
                    },
                    "bucket": {
                        "description": "Storage Bucket of ",
                        "default": "Other",
                        "nullable": true,
                        "readOnly": true,
                        "$ref": "#/components/schemas/EStorageBucket"
                    }
                }
            },

throwing this error. enum value is not recognized in this.
line 90, column 81 of .: Expected to find '}'.
   ╷
90 │ Future<chopper.Response> testUploadPost({List<String>? image, String? fileId, ? bucket, }) {
   │                                                                                ^^^^^^
   ╵
line 93, column 132 of .: Expected an identifier.

may i know correct way to convet uint8list or file to List to upload file.


Updated with comments for better understanding.

Yes, I will. Thank you.
On Sat, Oct 3, 2020, 6:04 PM Bhavesh Lohana<EMAIL_ADDRESS>wrote:

@bhaveshlohana requested changes on this pull request.
Hello, can you change the file name? As it is creating a conflict.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bhaveshlohana/HacktoberFest2020-Contributions/pull/576#pullrequestreview-501543047,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANW26B7PSFXSDJ2U44TA7VLSI4K6HANCNFSM4SBXZ4SA
.

Hello,
I see the file names are changed. Should I still change the file name?
On Sat, Oct 3, 2020 at 7:21 PM Parinita Vedantam<EMAIL_ADDRESS>wrote:

Yes, I will. Thank you.
On Sat, Oct 3, 2020, 6:04 PM Bhavesh Lohana<EMAIL_ADDRESS>wrote:

@bhaveshlohana requested changes on this pull request.
Hello, can you change the file name? As it is creating a conflict.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/bhaveshlohana/HacktoberFest2020-Contributions/pull/576#pullrequestreview-501543047,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANW26B7PSFXSDJ2U44TA7VLSI4K6HANCNFSM4SBXZ4SA
.


Bug: n/no-extraneous-import doesn't recognize workspaces

Environment
Node version:
npm version:
ESLint version:
eslint-plugin-n version:
Operating System:
What rule do you want to report?
n/no-extraneous-import
Link to Minimal Reproducible Example
https://eslint-online-playground.netlify.app/#eNplUUFOwzAQ/IrlUys1DiAoqBWcuHNBXOoejLNUbhPbWjulKMrfsWNHStObtTM7OzPuqBXyJA7gSovmCNKXecCOzmi6oR3XhHCqRQOcbsIr8zjluqer2324iMbWwDw4z7wLEqqxBj3pSAVOovoG0pMfNE0QO6tI43TLNdcjvAjadEUWS/L6Rrp+uQ1nGLhaaY9y5ssKdIAf1iujXTQ4zCMCshFfgC4AyXgt0q3VyHCmRQmffzYna0zV1uEd8X5gBf26PaikvAstcLrPAAbq9UFdalPAxaPQYFpXpNhJGhANZuWr3mZxKji/g4VQhZZqpp8aSHov7OmZ3U2yJLBIdouc+H7N1uxhwhrrHkD2GBWmYX8Nnlwwlg7vxq35F6elfczR/wO7rrp/
What did you expect to happen?
When in a workspace setup and a root-level package.json has a dependency not listed in nested packages, that dependency should not be considered missing. But I'm setting it as missing.
Note that although the repro playground has a package.json "workspaces" entry, not every workspace setup has that. pnpm workspaces use a separate pnpm-workspace.yaml.
Participation

[X] I am willing to submit a pull request for this issue.

Additional comments
Related, for n/no-missing-imports: #184.

I think this is a great idea to be able to check up for a workspace package.json. I do a similar thing in yarn fairly commonly :thinking:
I do think we should only support the package.json#workspaces entry though, as I dont want to be bespoke to specific package managers where possible. (I am willing to bend a little though :sweat_smile:)

npm workspaces (package.json#workspaces)
yarn workspaces (package.json#workspaces)
pnpm workspaces (pnpm-workspace.yaml#packages) (There is an issue to support this in pnpm/pnpm #2255)


Extension will not load with PHP 7.4RC1 on IIS/Win2016

.+Please check the FAQ (frequently-asked questions) first. If you have other questions or something to report, please address the following:

PHP Driver version or file name

php_sqlsrv-5.7.0preview-7.3-nts-vc15-x64

SQL Server version

2014

Client operating system

Windows Server 2016

PHP version

php-7.4-nts-windows-vc15-x64

Microsoft ODBC Driver version

<IP_ADDRESS>

Table schema

N/A

Problem description

SQL driver extension will not load.

Expected behavior and actual behavior

Expected - > load the extension / no error message in log.
Actual: [11-Sep-2019 17:18:46 UTC] PHP Warning:  PHP Startup: Unable to load dynamic library 'sqlsrv' (tried: C:\Program Files\PHP\v7.4.0RC1\ext\sqlsrv (The specified module could not be found.), C:\Program Files\PHP\v7.4.0RC1\ext\php_sqlsrv.dll (The specified procedure could not be found.)) in Unknown on line 0

Repro code or steps to reproduce

The php.ini file hasthe following lines in it pertaining to the loading #of this extension:
extension_dir="C:\Program Files\PHP\v7.4.0RC1\ext\"
 extension=sqlsrv
I have also tried:
extension=php_sqlsrv.dll
Output of DIR command showing location of the DLL:
PS C:\Program Files\PHP\v7.4.0RC1> dir "C:\Program Files\PHP\v7.4.0RC1\ext\php_sqlsrv.dll"
Directory: C:\Program Files\PHP\v7.4.0RC1\ext

Mode                LastWriteTime         Length Name
----                -------------         ------ ----
-a----        9/11/2019   9:46 AM         253440 php_sqlsrv.dll`

Hi @GenesisCraig please download the 7.4 binaries instead

Any update, @GenesisCraig ? We will be closing this issue shortly due to inactivity.

Please feel free to reopen this if necessary.


--overwrite not working, missing argument 'path'

I get an error trying to pull strings using the cli:
> dotenv -c production tolgee pull --overwrite ./public/locales

error: missing required argument 'path'

I am using dotenv to load the API_KEY
Using version CLI version 1.2.0
Without the --overwrite everything works fine. But I want to automate this on CI so I cannot use the interactive mode.

Hey there, sorry to hear you're having troubles!
This problem seems odd, as this command is e2e tested and the --overwrite argument should be working just fine.
I recall using dotenv -- (command...) back when I was using dotenv, could you give this a try? The double dash should tell dotenv to consume the command "as is", as it might be currently consuming things it shouldn't (such as the --override) parameter leading to the problem you're encountering.

Ah yes, that seems to do the trick. Thanks!
Also it would be nice to have the .env reading built-in 🙂


Upgrades with Remote Server

Pull Request summary:
Rework of software operations for upgrades with Remote Server.
Description of changes:

rework install method to include remote server files
remove usage of old Device (coming from attrs) - instead use DeviceDetailsResponse
enhance upgrades_helper.py to include c8000V upgrades

Checklist:

[x] Make sure to run pre-commit before committing changes
[x] Make sure all checks have passed
[x] PR description is clear and comprehensive
[x] Mentioned the issue that this PR solves (if applicable)
[x] Make sure you test the changes

This issue was migrated from an old repository: https://github.com/CiscoDevNet/catalystwan/pull/497


fix(metrics): fix sink's throughput metrics

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
a code branch is missed.

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

LGTM.
Can we try merging the two branch into one? Many code are duplicated in the two branches, and it will getting harder to maintain as we implement more features on it.

I have tried but hard to implement it without new abstractions. Maybe we need a new trait that can accept StreamMessages and wrap the logWriter so that we can dispatch different logic and share some common methods on some aspects.


meta: fix CAS ('c') return values

Fixes #842 - meta will match binprot behavior.
Also fixes #844
TODO:

[x] tests
[x] look at the delete bug.
[x] decide if we want to try harder and return the underlying CAS instead of zero for some conditions.
[x] ensure docs match

I haven't semi-formally reasoned through the cas response. Matching binprot is easy (I just had to use the same flag...), which at least fixes the weird behavior of it returning a high cas value.
It could be potentially useful to return the current cas value with the EX response: then the client can retry. Except:

The client would not be making its store decision based on the value of the data without first re-fetching the value. Why would CAS be used in this case?
It should be possible to give ms a value flag that's CAS predicated; then if your CAS values it craps back a value. I won't do this right away but writing it down so I don't forget.
In all other uses the new Win token workflow should be used, which avoids cas races in the first place.

So I think this reasons through as the zero-ing behavior is correct? Correct me if wrong. :)

@petergoldstein - I don't intend on making further changes before merging. I'll hold it a little longer in hopes you can test the change.
If we're going to start returning the CAS value in conditions the binprot doesn't I'd prefer that to be a separate change. For now I've left a note in the documentation that the '0' return isn't itself an indicator, but to ignore the value and pay attention to the return code.

out in 1.6.13


GraphSizeNorm

Hi, I add a GraphSizeNorm module and its test as suggested in this issue. Please have a check.

Perfect! Thank you.


Added stylint support

This PR should add support for the stylint linter

Cool! Thanks for the contribution. I'm working on a next version that should make adding linters much easier. Right now it's not DRY enough.


Migration from Python 3.7 to 3.9 gone bad

Hi guys,
I'am in the process of upgrading my project from Python 3.7 to 3.9 but the Luma.oled library is spazzing out.. In the attached picture you see a scrolling text. With 3.7 everything works fine but with Python 3.9 the scrolling text is completely broken. Minimal static examples all work.
RPI 3B+
5.4.79-v7+
Version: luma.oled 3.8.1 (luma.core 2.2.0)
Python3.9.1 completely fresh venv

Package                          Version
-------------------------------- ---------
Adafruit-Blinka                  5.9.1
adafruit-circuitpython-bme280    2.5.1
adafruit-circuitpython-bme680    3.3.0
adafruit-circuitpython-busdevice 5.0.1
Adafruit-GPIO                    1.0.3
Adafruit-PlatformDetect          2.23.0
Adafruit-PureIO                  1.1.8
cbor2                            5.2.0
certifi                          2020.12.5
chardet                          3.0.4
Cython                           0.29.21
dbus-python                      1.2.16
Deprecated                       1.2.10
idna                             2.10
luma.core                        2.2.0
luma.oled                        3.8.1
mock                             4.0.3
pi-ina219                        1.3.0
pi-rc522                         2.2.1
Pillow                           8.0.1
pip                              20.3.2
pycairo                          1.20.0
pyftdi                           0.52.0
PyGObject                        3.38.0
pyserial                         3.5
python-lirc                      1.2.3
python-mpv                       0.5.2
pyusb                            1.1.0
requests                         2.25.0
rpi-ws281x                       4.2.5
RPi.GPIO                         0.7.0
setuptools                       49.2.1
smbus2                           0.4.0
spidev                           3.5
sysv-ipc                         1.0.1
urllib3                          1.26.2
wheel                            0.36.2
wrapt                            1.12.1
xmltodict                        0.12.0

Expected behaviour
Scrolling text without artifacts.
Code can be found here: https://github.com/majuss/wladio/blob/master/radio/display.py
Actual behaviour
Scrolling text with haevy glitches.
The runner example is also broken, if I don't stretch it across the whole screen.

I will update this issue with my findings.

can you doublecheck if the installed package versions for 3.7 vs 3.9 are the same?

luma.core                        2.2.0
luma.oled                        3.8.1
Pillow                           8.0.1

Just tested it with the same versions in Pyhton3.7. No problems there.

Hi @majuss , which oled driver is it?
I experienced some odd glitches recently with pillow > 7.1.2 & python 3.8 when using the ssd1309 driver.
Could you try python 3.9 with pillow 7.1.2 and report back please?

I use a SSD1322 and no same glitches with 7.1.2.

smbus2 recently had a big update as well, but probably unrelated to glitches..

Tried again with most recent packages. Exactly the same :(

Tried again with most recent packages. Exactly the same :(

@majuss so it doesn't work with Pillow 8.x on Python 3.9 but Pillow 7.1.2 on Python 3.9 is ok? And Python 3.7 is always ok?

Tried again with most recent packages. Exactly the same :(

Please specify which ones and version nrs.

@majuss so it doesn't work with Pillow 8.x on Python 3.9 but Pillow 7.1.2 on Python 3.9 is ok? And Python 3.7 is always ok?

Tried again with most recent packages. Exactly the same :(

Please specify which ones and version nrs.

Python3.9 never works pillow version doesn't matter. Python3.7 is always okay.

Python3.9 never works pillow version doesn't matter. Python3.7 is always okay.

Well the interesting thing here is that the tests target python 3.6, 3.7, 3.8 and 3.9, and while they generally substitute the interface (I2C / SPI) for a mock, they do check to make sure the correct sequence of bytes is being produced for all the different displays across all versions of Python (albeit on Ubuntu / intel rather than Raspian / arm)
So that leads me to the following questions:

How have you installed Python 3.9 ? From source or a specific binary?
Which OS are you using?
Have you tried Python 3.8?

Everything you've said (and not said so far) points to it being a local issue, so we really need to know more about your setup

Well the interesting thing here is that the tests target python 3.6, 3.7, 3.8 and 3.9, and while they generally substitute the interface (I2C / SPI) for a mock, they do check to make sure the correct sequence of bytes is being produced for all the different displays across all versions of Python (albeit on Ubuntu / intel rather than Raspian / arm)
So that leads me to the following questions:

How have you installed Python 3.9 ? From source or a specific binary?
Which OS are you using?
Have you tried Python 3.8?

Everything you've said (and not said so far) points to it being a local issue, so we really need to know more about your setup

I use the latest version of Raspbian on a RPI 3B+.  Didn't tried Python3.8 yet. I installed Python directly from source.

I use the latest version of Raspbian on a RPI 3B+.  Didn't tried Python3.8 yet. I installed Python directly from source.

I switched to a Pi 4 and now everything works with Python3.9

@rm-hull The exact same behaviour is back again with Python 3.11 latest raspberry pi os bookworm.

@majuss what version of Pillow are you using? Can you test with latest (10.1.0)?

@thijstriemstra how can I debug this? Even the runner example etc. is showing these artefacts again :/

The pillow version doesnt matter. The bug is there even with 9.4

Can you reproduce it on other devices?


Increase test timeouts for node state changes

These tests seem to often fail on arm and it's probably because the 2 second timeout was not enough. I'll start arm CI on this PR.
Signed-off-by: Tonis Tiigi<EMAIL_ADDRESS>

The failures in arm seem unrelated. All swarm tests passed.

Seems like arm failed on other test now :)
LGTM

LGTM 🐯


add: demo python script

Add a demo Python script that will call on a Go function, helloWorld from pykitzoid/demo/library.go. The script should be saved as follows: pykitzoid/demo/demo_app.py.
Refer here
Note that helloWorld from library.go does not take any parameters.

Hi, I'd like to be assigned to this issue please :D

Thank you for showing interest @z404, Assigned!


Can markdown preview support UML graph ？

Can markdown preview support UML graph

This is something that an extension would provide rather than the builtin markdown previewer


fix homepage link

closes #9410
thanks for the heads-up @jona-sassenhagen!

LGTM!

here's the homepage rendering: https://28643-1301584-gh.circle-artifacts.com/0/dev/index.html
the link works now.

thx @drammock
feel free to backport


I can't use RawMessageDelivery of messaging from sns to sqs

I implemented messaging from sns to sqs using RawMessageDelivery, but I could not get the expected value.
Is RawMessageDelivery supported?
Or is there a problem with my code?

my code (test.py)

# -*- coding: utf-8 -*-

import boto3
from moto import mock_sqs, mock_sns
import json
import time
from pprint import pprint

mock_sns = mock_sns()
mock_sns.start()

mock_sqs = mock_sqs()
mock_sqs.start()

sns_client = boto3.client('sns')
topic_res = sns_client.create_topic(Name="topic_name")
sns_topic_arn = topic_res['TopicArn']

sqs_client = boto3.client('sqs')
queue_url = sqs_client.create_queue(
    QueueName="queue_name"
)['QueueUrl']
sqs_queue_arn = sqs_client.get_queue_attributes(QueueUrl=queue_url,
                                                AttributeNames=['All'])['Attributes']['QueueArn']

subscribe = sns_client.subscribe(
    TopicArn=sns_topic_arn,
    Protocol='sqs',
    Endpoint=sqs_queue_arn
)
subscribe_arn = subscribe["SubscriptionArn"]
sns_client.set_subscription_attributes(
    SubscriptionArn=subscribe_arn,
    AttributeName='RawMessageDelivery',
    AttributeValue='true'
)

# send message
boto3.resource('sns').Topic(sns_topic_arn).publish(Message=json.dumps({"this": "that"}), MessageStructure='json')

time.sleep(10)

# receive message
queue = boto3.resource('sqs').get_queue_by_name(QueueName="queue_name")
msg = queue.receive_messages(
    MaxNumberOfMessages=1,
    WaitTimeSeconds=20,
    VisibilityTimeout=10
)[0]

pprint(msg.body)

mock_sqs.stop()
mock_sns.stop()

return

$ python test.py
('{\n'
 '  "Message": "{\\"this\\": \\"that\\"}",\n'
 '  "MessageId": "7753f915-78d8-4490-b5fa-5770daf0ac30",\n'
 '  "Signature": '
 '"EXAMPLElDMXvB8r9R83tGoNn0ecwd5UjllzsvSvbItzfaMpN2nk5HVSw7XnOn/49IkxDKz8YrlH2qJXj2iZB0Zo2O71c4qQk1fMUDi3LGpij7RCW7AW9vYYsSqIKRnFS94ilu7NFhUzLiieYr4BKHpdTmdD6c0esKEYBpabxDSc=",\n'
 '  "SignatureVersion": "1",\n'
 '  "SigningCertURL": '
 '"https://sns.us-east-1.amazonaws.com/SimpleNotificationService-f3ecfb7224c7233fe7bb5f59f96de52f.pem",\n'
 '  "Subject": "my subject",\n'
 '  "Timestamp": "2018-04-17T11:00:32.251Z",\n'
 '  "TopicArn": "arn:aws:sns:ap-northeast-1:123456789012:topic_name",\n'
 '  "Type": "Notification",\n'
 '  "UnsubscribeURL": '
 '"https://sns.us-east-1.amazonaws.com/?Action=Unsubscribe&SubscriptionArn=arn:aws:sns:us-east-1:123456789012:some-topic:2bcfbf39-05c3-41de-beaa-fcfcc21c8f55"\n'
 '}')

expect message

{"Message": {"this": "that"}}

environment

$ python -V
Python 3.6.3 :: Anaconda, Inc.
$ pip freeze | grep boto
boto==2.48.0
boto3==1.7.4
botocore==1.10.4
$ pip freeze | grep moto
moto==1.3.2

hi @spulec, I've just run into this issue as well and would love to get your fix for it without having to install moto off a commit -- would it be possible to release a new version? (looks like moto hasn't been released in a while...)


Reqesting support for A2DP

Since Windows 10 now supports A2DP, it might be really helpful for developers if A2DP is also supported as a service in 32feet. Thanks!

What is your use-case for this functionality? The OS has built in support for A2DP so it will appear as an audio device in the system.

Windows had this feature removed but brought it back recently. But now it needs an external client to create this connection. If 32feet has libraries to allow this, it might be a great help for people who develop Laptop - Phone interface devices.


Add installation instructions to Docs

good idea!

Done! Only took me a few months to add one line of code LOLS


feat: updated DataStore [2/n]

From #139:
A DataStore should be an abstraction over a particular "Mapeo Store", e.g. a collection of hypercores that store a particular datatype. Responsibilities:

Write a document to the write core of the store.
Read a document based on versionId.
Manage read indexing and return current indexing state.
Handle decoding/encoding of data

With this we can isolate the boundary with hypercore / core store to this module (apart from within the blob store), and largely isolate indexing: The write method can resolve when the document is indexed.
const store = new DataStore({
  coreManager,
  namespace: 'auth' | 'data',
  // DataStore will consider a block "indexed" once this function resolves.
  indexDocs: (blocks: Buffer[]) => Promise<void>
})

// Guarantee that `doc` is indexed once this resolves
store.write(doc: Exclude<MapeoDoc, 'versionId'>): Promise<MapeoDoc>

store.read(versionId: string): Promise<MapeoDoc>

store.getIndexState() // return whether is currently indexing, and estimate of indexing remaining to be done

store.on('index-state') // same as above.

Now largely complete, will be best to review once all items in #139 are done, because this might still change.

Rebased onto changes in #140

Merged in #149


Does Euclidean optimizer fail when the matrix is not skew-symmetric or skew-Hermitian?

If the last two dimensions of tensor are not square, torus_init_ cannot be applied to the tensor.
Then the tensor cannot be filled with skew-symmetric  matrix, then exponential map fail, then geometry optimization cannot be transformed into Euclidean optimization. Is that the case?
With respect to non-square matrix, how can  geometry optimization be transformed into Euclidean optimization？

Please have a look at the docs: https://geotorch.readthedocs.io/en/latest/orthogonal/stiefel.html


Make members available in subclasses

Title says it all...

Good to go, thanks.
We may want to revisit other classes whether they also have members which aren't visible in subclasses.


[CI] Add new PyPI packaging Jenkins pipeline

This patch adds a new Jenkins pipeline for packaging. This is a migration of the current Github actions workflow of uploading a Tensor learning compiler binary distribution (tlcpack) to PyPI.
This new packaging pipeline implements a parallel step that creates and uploads AArch64 packages, which is something not present in the Github actions job it is intended to replace.
I have tested the pipeline here, with some minor modifications so that the packages were uploaded to my personal test.pypi account. The code submitted in this patch will attempt to upload to the apache-tvm production package.
cc @areusch @driazati @tqchen @leandron

I still think it is valuable to maintain and use the GH action pipeline when possible(as they come for free) and use the new jenkins one for cases that are not covered by GH

I still think it is valuable to maintain and use the GH action pipeline when possible(as they come for free) and use the new jenkins one for cases that are not covered by GH

The issue is mostly about synchronisation of packages being published. If we have two separate pipelines (one in GitHub Actions and one in Jenkins), we're not guaranteed publishing the same reference package for all variants.
Apart from that, the proposed method also helps us with generating packages on demand as part of releases.

Given it is natural for different platforms to have some different config and we do have different publication pipelines for example in the case of windows, I don't think it would make too big of a difference

Has this workflow been tested / if so can you link the passing runs and example artifacts somewhere?

Yes, I tested this out here: https://ci.tlcpack.ai/blue/organizations/jenkins/pypi-packaging%2Fregular-pypi-package-upload/detail/regular-pypi-package-upload/4/pipeline
The packages were successfully produced, but these were uploaded to my personal test PyPI space.

This is approved for quite a few days, merging it now.


Add api server extra args map to cluster spec

Issue #, if available:
#2091
Description of changes:
Design doc - https://quip-amazon.com/ukGjAcVcAUYV/Allow-users-to-configure-service-account-issuer-and-service-account-jwks-uri-flags-for-kube-apiserver
Note: It took about ~30 minutes for vsphere and ~40 minutes for cloudstack to upgrade cluster when changing/removing the api-server flags. Maybe we can add this somewhere in our eksa docs later to inform the users so that they wait till the cluster is upgraded.
Validations:
Only the service-account-issuer and service-account-jwks-uri flags are supported at this time. If the user adds any other flags to the cluster spec, it will throw an invalid cluster config file error. No additional validations will be done for the api-server flags.
Testing (if applicable):

Tested locally for create and upgrade operations with the api-server flags
Unit tests are added for ApiServerExtraArgs with valid and invalid flags
E2E tests for creating and upgrading cluster with ApiServerExtraArgs are added for vSphere and CloudStack providers

Documentation added/planned (if applicable):
EKS-A docs need to be updated to add controlPlaneConfiguration.apiServerExtraArgs as an optional configuration for the cluster spec
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/hold

/unhold

/retest

/lgtm

/approve


Move Modules to Shared Library Part 1 of 3

Acceptance Criteria

[x] Institution (IER)

[x] Create a new library '@sims/integration'

Create scheduler (once per day 9AM PST)

[ ] Institution (IER)

https://optimalbits.github.io/bull/

https://github.com/felixmosh/bull-board/issues/266#issuecomment-832372178,
https://github.com/s-r-x/bull-monitor


fix(menu): wrong animation in Angular 4.2+

Fixes the new menu animation glitching out against Angular 4.2. It seems like transitioning from scale(0, 0) to scale(1, 0.5) doesn't work correctly at the moment. This is a workaround that will transition it from scale(0.01, 0.01) instead.
For reference:

Note: Screenshot tests need to be fixed


E2E for subscription-proxy test

Codecov Report

Merging #465 (f42b590) into master (16f7a20) will decrease coverage by 0.21%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #465      +/-   ##
==========================================
- Coverage   41.51%   41.30%   -0.22%
==========================================
  Files           9        9
  Lines        1869     1845      -24
==========================================
- Hits          776      762      -14
+ Misses       1040     1033       -7
+ Partials       53       50       -3

Impacted Files
Coverage Δ

controllers/registry.go
42.72% <100.00%> (+1.71%)
:arrow_up:

controllers/restic.go
54.35% <100.00%> (-2.86%)
:arrow_down:

controllers/velero.go
36.22% <100.00%> (-2.05%)
:arrow_down:

Continue to review full report at Codecov.

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

This seems to work. Manager container is unable to start due to proxy setting
2021-11-19T19:52:24.394Z	ERROR	setup	unable to start manager	{"error": "Get \"https://<IP_ADDRESS>:443/api?timeout=32s\": proxyconnect tcp: dial tcp: lookup proxy.example.com on <IP_ADDRESS>:53: no such host"}

/test operator-e2e

Seems to work detecting missing env in pods. Need to figure out what pod is missing
https://prow.ci.openshift.org/view/gs/origin-ci-test/pr-logs/pull/openshift_oadp-operator/465/pull-ci-openshift-oadp-operator-master-operator-e2e/1463204484065792000#1:build-log.txt%3A257
• Failure [43.079 seconds]
Subscription Config Suite Test
/go/src/github.com/openshift/oadp-operator/tests/e2e/subscription_suite_test.go:17
  Proxy test table
  /go/src/github.com/openshift/oadp-operator/tests/e2e/subscription_suite_test.go:37
    HTTP_PROXY set [It]
    /go/src/github.com/openshift/oadp-operator/tests/e2e/subscription_suite_test.go:103
    Expected

        <[]v1.EnvVar | len:4, cap:4>: [
            {
                Name: "NODE_NAME",
                Value: "",
                ValueFrom: {
                    FieldRef: {
                        APIVersion: "v1",
                        FieldPath: "spec.nodeName",
                    },
                    ResourceFieldRef: nil,
                    ConfigMapKeyRef: nil,
                    SecretKeyRef: nil,
                },
            },
            {
                Name: "VELERO_NAMESPACE",
                Value: "",
                ValueFrom: {
                    FieldRef: {
                        APIVersion: "v1",
                        FieldPath: "metadata.namespace",
                    },
                    ResourceFieldRef: nil,
                    ConfigMapKeyRef: nil,
                    SecretKeyRef: nil,
                },
            },
            {
                Name: "VELERO_SCRATCH_DIR",
                Value: "/scratch",
                ValueFrom: nil,
            },
            {
                Name: "AWS_SHARED_CREDENTIALS_FILE",
                Value: "/credentials/cloud",
                ValueFrom: nil,
            },
        ]
    to contain element matching
        <v1.EnvVar>: {
            Name: "HTTP_PROXY",
            Value: "http://proxy.example.com:8080",
            ValueFrom: nil,
        }
    /go/src/github.com/openshift/oadp-operator/tests/e2e/subscription_suite_test.go:91

Just added logs in https://github.com/openshift/oadp-operator/pull/465/commits/5745ca390f2e976a0460f65a3fd2edd673c9e4e7 but the prior issue got resolved somehow.


Go To definition not working for global cycle calls

Describe the bug
The go to definition functionality doesn't find global programs in the working space.
To Reproduce
With this folder structure the go to definition functionality doesn't find subPrograms.nc.

With this folder structure the go to definition functionality does find subPrograms.nc.

Fixed in https://github.com/isg-stuttgart/vscode-isg-cnc/pull/112


Simplify WebSocket handlers constructor arguments hell #9698

Motivation:
Introduction of WebSocketDecoderConfig made our server-side code more elegant and simpler for support.
However there is still some problem with maintenance and new features development for WebSocket codecs (WebSocketServerProtocolHandler, WebSocketServerProtocolHandler).
Particularly, it makes me crying with blood extremely sad to add new parameter and yet another one constructor into these handlers, when I want to contribute new feature.
Modification:
I've extracted all parameters for client and server WebSocket handlers into config/builder structures, like it was made for decoders in PR #9116.
Result:

Fixes #9698: Simplify WebSocket handlers constructor arguments hell
Unblock further development in this module (configurable close frame handling on server-side; automatic close-frame sending, when missed; memory leaks on protocol violations; etc...)

Bonuses:

All defaults are gathered in one place and could be easily found/reused.
New API greatly simplifies usage, but does NOT allow inheritance or modification.
New API would simplify long-term maintenance of WebSockets module.

Example
WebSocketClientProtocolConfig config = WebSocketClientProtocolConfig.newBuilder()
    .webSocketUri("wss://localhost:8443/fx-spot")
    .subprotocol("trading")
    .handshakeTimeoutMillis(15000L)
    .build();
ctx.pipeline().addLast(new WebSocketClientProtocolHandler(config));

I did the similar thing here - https://github.com/netty/netty/pull/9425

@netty-bot test this please

@ursaj thanks a lot!


Provide Direct Link to Enable "Enable User Email Verification" during User Creation

Describe the issue:
Currently, during the user creation process,when user try to invite user to set password via email,  users are prompted to "Enable User Email Verification," if it is not enabled,
but it would be easier to enhance the user experience by providing a direct link to enable this feature.
How to reproduce:

Expected behavior:
Users invited to set their password via email will have a direct path to enable email verification, creating a more straightforward onboarding experience.
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: wso2is-7.0.0-beta6-SNAPSHOT
OS: Mac
Database: MSSQL, H2
Userstore: JDBC

Addressed with https://github.com/wso2/product-is/issues/18232


start working on test code for zeno

@haileyschoelkopf I've started implementing tests for the Zeno upload functionality. I'm not sure where to put the test data (as we need more than just the metric results).
Furthermore, you'll need to add a ZENO_API_KEY to your GitHub secrets (probably of a Eleuther test account) similar to how we do it in our tests.


Update crate version in README

According to crates.io, mish is in version 0.2.1. According to the README, it is version 0.2.0. I figured I would let you know to avoid future confusion.

I'm not 100% sure on this one, because semver should be able to pull in 0.2.1 from the Readme's 0.2.0, also crates.io has the versioning set up there. I'm planning on doing quite a few changes to mish, so the next update may be 0.3.0, and I'd rather update the README when I do that, instead of merging this PR (bumping version number on both to 0.2.2 as I'd need to push to crates.io).
Thanks either way.

Alright, sounds good. I just was letting you know because Dependabot bamboozled me when it said 'mish' was at version '0.2.1'.


Delete all mentions of the dev environment

We don't do local dev anymore, so we don't need any of the Terraform.

I believe the kitchen tests run in dev, and if we're saying we don't have dev environments any more - should we delete them? 🤔

@issyl0 yes let's bin them

@philandstuff I removed the kitchen tests. 👩🍳


changes for the Beckn-onix-stack: phase 1: part1

Going through the govStack documentations.
Started adding the documentation(based on the govStack) in google docs: https://drive.google.com/drive/u/3/folders/1_-iSiGa7xkJ8AFdSxRbMbz-5QxGKOqY1
Needed to convert the .md format to .doc manually as, the convertors or GPT were not converting correctly.

Gov stack doc: https://govstack.gitbook.io/bb-emarketplace
Needed to convert the .md format from the govstack documents to .doc manually as, the convertors or GPT were not converting correctly.
The converted files are present here: https://drive.google.com/drive/u/3/my-drive


ocamlPackages.tezos-lmdb: fix source hash

Motivation for this change
Current hash seems wrong.
cc maintainer @ulrikstrid
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 via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.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.

What's up with the hashes? Are every changing upstream?
Sidenote: ligo updated to a later version of tezos so we can probably drop this specific package if we don't want to keep it

@vbgl will you fix another PR which drops tezos-lmdb completely, please?

I’ve amended the commit to remove the broken package.

@ulrikstrid ack?

I can double check tomorrow, but it should be fine

It seems like tezos-storage still depends on this (I didn't know), but it also depends on irmin < 2.7.0 so I'm not sure how to handle it. We can remove the module and re-introduce it if I need it.

May this be merged?

I have opened a large PR #134424 to update all of the Tezos packages to the latest release including fixing tezos-lmdb. I think we can close this PR and just concentrate our efforts there instead.

Superseded by https://github.com/NixOS/nixpkgs/pull/134424


Improve bean discovery filtering

ok to test

/cc @ppalaga

will remove in a follow up commit

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/camel-quarkus-pr/465/


[ENH] add session and fix task in dataset listing

Codecov Report
Patch and project coverage have no change.

Comparison is base (1eb3e40) 76.56% compared to head (bfca320) 76.56%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #67   +/-   ##
=======================================
  Coverage   76.56%   76.56%
=======================================
  Files           6        6
  Lines         640      640
=======================================
  Hits          490      490
  Misses        150      150

Impacted Files
Coverage Δ

cohort_creator/_parsers.py
97.67% <ø> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Adding dropdown details in a select-type input and then saving it back results in a 404 error.

Terms

[X] Before you openning this issue, i have checked if the issue has already been reported.

Bagisto Version(s) affected
Master
Issue Description
Adding dropdown details in a select-type input and then saving it back results in a 404 error.
Preconditions

Admin must be logged in.

Steps to reproduce

Go to Admin -> Catalog -> Attributes ->  Create a new attribute with select type -> Drop down.
Add dropdowm entries and save.
Again Edit and save drop-down data and we can see 404 issue.

Expected Result
There should be no issue.
Actual Result
Please check the video for the same.
https://github.com/user-attachments/assets/218268cc-6743-4121-8c58-c6efaac3a7e3
Screenshots
No response
Additional context
No response

@ashishkumar-webkul
I have carefully reviewed and verified that the issue does not appear in the 2.2 branch. For your reference, please take a look at the video provided.
Video Link

@amit-webkul
As I have checked again and there still this issue has been faced in image drop-down.
https://github.com/user-attachments/assets/b3829005-c38f-4965-a385-1933f4b943c0

Hi, are there any updates on this report? We are having the same problem and it is difficult to change the attributes. Thanks


[zh] Recurrent Neural Networks

cs-230 recurrent neural networks translation is finished.

Thank you for all your work @HsinJhao! Please feel free to invite anyone you know to review the translation.

Thank you @HsinJhao and @SpeakingTom for all your work!
@HsinJhao: would it be possible to only keep the zh/cs-230-recurrent-neural-networks.md file in this PR? That way, we could immediately move forward with the merge.

Thank you @HsinJhao and @SpeakingTom for all your work!
@HsinJhao: would it be possible to only keep the zh/cs-230-recurrent-neural-networks.md file in this PR? That way, we could immediately move forward with the merge.


Successful compilation of clearly nonsensical references before assignment?

def bogusassignment():
	testvar = testvar
	return testvar

@cython.locals(testvar=cython.int)
cpdef int bogusassignment()

Code like this compiles and runs fine, but I'm not sure if there's any legitimate use cases where it would actually be intentional.
I'm also not totally sure exactly where the values assigned to the uninitialized variable are coming from, and whether there might be adverse security implications in being able to easily accidentally read from an unknown portion of memory:
def leakyread():
	charp = charp
	l = []
	for i in range(1000):
		b = charp[i]
		l.append(b)
	return b''.join(l)

@cython.locals(charp=cython.p_char, b=bytes, l=list)
cpdef bytes leakyread()

(Like, hypothetically, if charp above were used as part of a public-facing portion of a web application, and the environment it was running in allowed it to access passwords, keys, internal program state, etc?)
Ramble: This behaviour seems weird to me based on Python semantics, but I assume there must be valid uses cases for it to be explicitly allowed with a warning. I'm not familiar enough with low-level programming to understand all of those use cases, or to have a good gauge on how difficult it is to meaningfully differentiate between valid code and presumably nonsensical assignments like above. So, apologies if this is invalid or redundant.

I don't think Cython enforces whether C variables are initialized. However there is an optional warning warn.maybe_uninitialized. A simpler example (try it with and without the line # cython: warn.maybe_unitialized = True):
def f():
    cdef int a
    return a

gcc (and probably other compilers) gives you a warning though so it's not entirely silent. Either way I don't think this is a bug.


Rancher backend can't recursively load keys ending in an index of json objects array

Hallo, I have some issue with rancher backend.
confd versions: v0.11.0 and 0.12.0.
.toml file:
[template]
src = "test.tmpl"
dest = "/test"
keys = [
  "/2016-07-29/self/service/containers",
]

.tmpl file:
{{getv "/2016-07-29/self/service/containers/0/name"}}

command (inside docker container started by rancher):
confd -onetime -backend rancher

Expected behaviour: file /test contains name of first created container of this service.
default-confd-1

Actual behaviour:
/confd/confd-0.12.0-linux-amd64[12]: FATAL template: test.tmpl:1:2: executing "config.yml.tmpl" at <getv  "/2016-07-29/se...>: error calling getv: key does not exist: /2016-07-29/self/service/containers/0/name

Note 1 If I use foolowing toml file, all works fine, but I must specify every container:
[template]
src = "test.tmpl"
dest = "/test"
keys = [
  "/2016-07-29/self/service/containers/0",
]

Note 2 If I use metadata version 2015-07-25, all keys "/2015-07-29/self/service/containers/" are loaded correct because it is array of strings (names of containers), not json objects.

Note 3 If I use following tmpl file (with toml without indexes of array), all works fine (it prints names of all containers):
{{range getvs "/2016-07-29/self/service/containers/*/name"}}
    {{.}}
{{end}}

Looks like a bug, feel free to open a PR!


Adding OpenID Connect Support for Google

OAuth2 + OpenId support using Google
A raw HTML page with Login and Logout buttons

Uses Bootstrap CSS

This change will cause the application to require 2 new startup parameters in order to run.
GOOGLE_CLIENT_ID
GOOGLE_CLIENT_SECRET
The values for these can be generated from your google developer account by going to:
https://console.developers.google.com/apis/credentials
Create Credentials > OAuth Client ID > Web Application

Provide a name
Add an Authorized Redirect URI of http://localhost:8080/login

Not sure if it would be best to commit the an ID and secret for testing purposes or disable that whole piece for local running.

It looks like you can provide a base path on the class with something like:
@RequestMapping("/api")
but there is no way to provide a default "rest controller" base path for the application.
There is a JIRA issue in the spring issue tracker requesting it.
https://jira.spring.io/browse/SPR-13882

It does look like we can make use of a property as part of the path for the root as well so we don't have to repeat /api.
https://docs.spring.io/spring/docs/current/spring-framework-reference/html/mvc.html#mvc-ann-requestmapping-placeholders


ML Extension - Validate was not called for the last page causing import for invalid model

Fixing a bug with validate was not called for the last page causing import for invalid model

This PR fixes #

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.0009%) to 42.107%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

extensions/machine-learning/src/views/models/manageModels/importModelWizard.ts
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.0009%

Covered Lines:
28151

Relevant Lines:
62378

💛  - Coveralls