Define a version scheme

Defining a versioning scheme for contract deployments
This issue keeps track of defining a versioning scheme for contract deployments.  The current scheme assumes a global contracts_version for all contracts, but this does not work because SecretRegistry has to be unique (at least) on each chain.  Also, there are additional desired properties (see below).

Status: being drafted (last edited 2019-04-08) [proposed | rejected | accepted | deprecated | … | superseded by ADR-0005]
Deciders: @pirapira @rakanalh @hackaugusto
Date: [YYYY-MM-DD when the decision was last updated]

Technical Story: this branched out of https://github.com/raiden-network/raiden-contracts/issues/524
Context and Problem Statement
There is a demand to simplify the directory structure of raiden-contracts.  At the same time, there is also a requirement that the mainnet should not get more than one SecretRegistry contract.  First defining a versioning scheme is a good start to solve both problems.
Decision Drivers

[client-usage] Raiden client needs to find which contracts to use together easily
[partial-update] Sometimes only some of the contracts are udpated.  Typical example: there shouldn't be many SecretRegistry instances.
[multi-limit-variants] on the test net, it makes sense to have multiple variants with different limit values. A TokenNetwork variant with very low SETTLE_PERIOD_MIN is useful for scenario playing, but the same test network should also host another variant with the same parameter.
[confusing-many-versions] Already PyPI package version numbers and the contracts_version are confusing people.  More kinds of versions are more confusing.
[similarilty to the current scheme] The bigger the change, the more implementation costs and disturbances in the client&service developments.
[multiple-tokens] Deployment information for multiple tokens should be storable (one token for service fees, and another token that's transferred).
…

Considered Options

[network-address] <network>/<contract_address>.json contains the contract name etc
[network-version-name] <network>/<smart_contract_version>/<smart_contract_name>.json
[network-name-version] <network>/<smart_contract_name>/<smart_contract_version>/deploy.json
[bundles] <network>/<bundle_version>/<smart_contract_name>-<smart_contract_version>.json
[small-change] data_<bundle_version>/<module_name or smart_contract_name>-<network>-deployment.json

SecretRegistry's deployment data should live in its own file.
When many bundles share one SecretRegistry, a symbolic link points to a previous secret-registry-network-deployment.json in an existing bundle.
When multiple variants of the same contract is deployed, <bundle_version> can be shorter-settlement or something.
When different tokens are deployed, <smart_contract_name> can be CustomeToken or ServiceToken (or whichever name given as a command line argument).

…

Decision Outcome
Chosen option [small-change].  It is the best option regarding [similarity to the current scheme].  It's good enough regarding [client-usage] and [partial-update].  For [confusing-many-versions], [small-change] option is similar to the current solution.  [multiple-tokens] can be solved by using multiple names for different token instances.

deployment version 0.12.0 contains

SecretRegistry of contract version 0.12.0
The other Raiden contracts of contract version 0.12.0
Service Registry of contract version 0.11.0
Service Contracts of contract version 0.12.0

deployment version 0.13.0 contains

SecretRegistry of contract version 0.12.0
The other Raiden contracts of contract version 0.13.0
Service Registry of contract version 0.11.0
The other Raiden contracts of contract version 0.13.0

deployment version 0.13.0_shorter_settlement contains

SecretRegistry of contract version 0.12.0 (the same instance as in deployment version 0.12.0)
The other Raiden contracts of contract version 0.13.0 (newly deployed with new parameters)
Service Registry of contract version 0.11.0 (the same instance as in deployment version 0.11.0)
The other Raiden contracts of contract version 0.13.0 (newly deployed with new parameters)

Positive Consequences

The decision clarified how to avoid deploying more than one instances of SecretRegistry.

Neutral Consequences

An example should be added above, how the next release should look
We have to decide still

whether to transform the older deployment data in the file system into the new scheme

We have to implement features for

reading/storing deployment data of token contracts, using the token-name in the file name
separating the deployment of secret registry out of other raiden contract deployments

I should next explain the outcome to @hackaugusto and @rakanalh .

I had a discussion with @hackaugusto and @rakanalh .  Two changes are made on the version scheme.

deployment version should be renamed into bundle version
bundle versions should be timestamps yyyymmdd instead of semvers.

Then, the filesystem should be designed.


✨ Start import in registration by feature gate.

Summary
Related issue(s)
Fixes #

LGTM

/hold
/lgtm

/unhold


Basic Authentication?

I'm hosting our YARD Documentation Portal at heroku. Due the fact, that this is a closed source project, we need to keep the documentation password protected. Basic Authentication would be the easiest solution.
Unfortunately yard doesn't support BA (despite there's a Pull Request) and it seems like there are no plugins or examples how to setup a basic authentication.
Do you have any advice how to achieve a password protected yard installation on heroku?
Thank you very much for your time!

Found a solution on myself: Via Rack.
config.ru:
require 'bundler/setup'

Bundler.require(:default, ENV['RACK_ENV'])

use Rack::Auth::Basic, "Please Sign In" do |username, password|
  [username, password] == ['foo', 'bar']
end

libraries = {
   # ...
}

run YARD::Server::RackAdapter.new(libraries)

Run bundle exec rackup config.ru to spawn the server


Update README.md

Remove space in example for exclude-rules.

Changes incorporated into #21 instead.


ctx.request.query can not get post query obj when I make an ajax 'post' request through axios

ctx.request.query can not get post query obj when I make an ajax 'post' request through axios, but in postman I can get query obj. I don't know what happens and how can i solve this problem.
More specific description in stackoverflow:
http://stackoverflow.com/questions/43604063/koa2-ctx-request-query-is-not-defined-when-i-make-an-ajax-request-but-in-postman

I'm assuming you're using body parser? A common gotcha with postman is to set correct content-type.
The link doesn't work.

i'm guessing axios improperly sends the query obj as a request body


tests/eas/acceptance: Reduce expected LITTLE busy time for offload test

Here's a trace plot that shows an example of what I'm talking about in the commit message (note that the blue tasks are placed on the same CPU when the 'migrator' tasks start running, which reduces the busy time. IIUC although this isn't ideal behaviour it is not unexpected.):

Where the 'migrator' tasks in the OffloadMigrationAndIdlePull are
initially placed on the same CPU, the busy time of the LITTLE CPUs that
are exercised can be below 99% during the perdiod where big CPUs are
occupied. This shouldn't be considered a failure.
In my experiments 97% appeared to be a low enough value to prevent these
failures. In fact I observed no failures with the value at 98% either,
but it seems wise to allow a little margin for error.

Merged as bbcae82 .  I fixed a typo in the commit message (s/perdiod/period/)


when http://localhost/legacymine/adminpanel/public/   then  SQLSTATE[42S02]: Base table or view not found: 1146 Table 'lgmine.settings' doesn't exist (SQL: select * from settings limit 1)

Run php artisan migrate and then php artisan db:seed. it should solve this error.


What I studied this week

• Studying redis usage
• Understanding oAuth

The Redis is the really famous in-memory database.


work on adapter cut-release and nightly-release update

resolves #https://github.com/dbt-labs/actions/issues/76

Description

Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have signed the CLA
[ ] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[ ] I have run changie new to create a changelog entry

workflow chart https://github.com/dbt-labs/dbt-redshift/actions/runs/4325439845
changelog: branch pr: https://github.com/dbt-labs/dbt-redshift/pull/354
currently existing test branch: https://github.com/dbt-labs/dbt-redshift/tree/1.5.test


HEIF/AVIF file signature (magic number)

Hi there!
Recently I started to use pyheif-pillow-opener for one project to add support for heif/avif files. It was great until I faced with some issue, that I want to share with you, because I can see you start developing your addon a month ago and you hopefully can fix it faster on your side
So, in as_opener.py you have check_heif_magic method. It checks, if 5-8 bytes of signature is ftyp or 9-12 bytes of signature is one of the listed brands, but I think it should be logical and instead of logical or in return statement and that's why:

Links in function comments https://github.com/strukturag/libheif/issues/83 https://github.com/GNOME/gimp/commit/e4bff4c8016f18195f9a6229f59cbf41740ddb8d shows, that acceptable magic numbers is ftypheic, ftypheix, ftypmif1 etc., not just ftyp or [heic, heix, mif1]
I found out that some other files can have 5-8 bytes as ftyp, for my particular situation it was .m4a, it has ftypM4A magic number, and I found some other files, that is not HEIF/AVIF photos, but have 5-8 bytes as ftyp (here you can find some them, but I bet we can find more: https://www.garykessler.net/library/file_sigs.html) and because of that .m4a files can be validated as HEIF files with this function, but this is not HEIF at all

I can't find too much info about HEIF/AVIF file signature, so maybe I'm wrong, but from my experience and from discussions above I assume, that this could be a bug
Thank you for your attention, would be gladful if you fix this, or tell me that I've missed something!

Will look at this tomorrow.

@DimonLavron Release will be ready on Tuesday or Wednesday, need add some additional stuff, like caching in GitHub Actions for faster build(arm build takes too long time currently) and include libaom library for AV1 format, write tests for these images: nokia heif repo and maybe will have time to make windows binary bistro.

@bigcat88 good to hear, thanks for quick reaction!

When they will fix this(https://github.com/pypa/cibuildwheel/issues/902) at the same day new version will be posted...
:(

@DimonLavron release published.

@bigcat88 thanks a lot!

@DimonLavron published a new release(this is first version with windows wheels), there was some changes in opener mode(it must be faster now), can you please  check it ?
Tried to cover all with tests, but it is a bit complex task, need some more time for that.


Getting and error when trying to build as a separate project

HI, hope you can help.
I am trying to build this project but get the following error after cloning and building after npm i:
$ npm run build

<EMAIL_ADDRESS>build C:_git\vue-render-plugin
rete --build rete.config.js

Error when using sourcemap for reporting an error: Can't resolve original location of error.
(node:6484) UnhandledPromiseRejectionWarning: Error: 'default' is not exported by node_modules\vue-runtime-helpers\dist\normalize-component.js
at error (C:_git\vue-render-plugin\node_modules\rollup\dist\rollup.js:3356:15)
at Module.error (C:_git\vue-render-plugin\node_modules\rollup\dist\rollup.js:13140:9)
at handleMissingExport (C:_git\vue-render-plugin\node_modules\rollup\dist\rollup.js:12612:21)
at Module.traceVariable (C:_git\vue-render-plugin\node_modules\rollup\dist\rollup.js:13246:17)
at ModuleScope.findVariable (C:_git\vue-render-plugin\node_modules\rollup\dist\rollup.js:12404:29)
at Identifier$$1.bind (C:_git\vue-render-plugin\node_modules\rollup\dist\rollup.js:9586:40)
at CallExpression.NodeBase.bind (C:_git\vue-render-plugin\node_modules\rollup\dist\rollup.js:9106:23)
at CallExpression.bind (C:_git\vue-render-plugin\node_modules\rollup\dist\rollup.js:10162:31)
at ExportDefaultDeclaration$$1.NodeBase.bind (C:_git\vue-render-plugin\node_modules\rollup\dist\rollup.js:9106:23)
at ExportDefaultDeclaration$$1.bind (C:_git\vue-render-plugin\node_modules\rollup\dist\rollup.js:9760:31)
at Program.NodeBase.bind (C:_git\vue-render-plugin\node_modules\rollup\dist\rollup.js:9102:31)
at Module.bindReferences (C:_git\vue-render-plugin\node_modules\rollup\dist\rollup.js:13094:18)
at Graph.link (C:_git\vue-render-plugin\node_modules\rollup\dist\rollup.js:20738:20)
at C:_git\vue-render-plugin\node_modules\rollup\dist\rollup.js:20833:19
at processTicksAndRejections (internal/process/next_tick.js:81:5)
at process.runNextTicks [as _tickCallback] (internal/process/next_tick.js:51:3)
(node:6484) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not
handled with .catch(). (rejection id: 1)
(node:6484) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

Added a fix for this in PR: https://github.com/retejs/vue-render-plugin/pull/7
as per: https://stackoverflow.com/questions/54379726/rollup-plugin-vue-v4-6-2-has-a-problem-with-vue-runtime-helpers


Edit product

Description
Fix: styles
Set data for edit forms
Update Product is work

The latest updates on your projects. Learn more about Vercel for Git ↗
Name	Status	Preview	Updated
did-admin-panel	❌ Failed (Inspect)		Sep 13, 2022 at 2:01AM (UTC)

@e-azocar can i see the details?

@Eghinner

Validated, when you click on the product you see the change, the product data is working correctly.
-Save and delete functionality along with validations will be added in task https://github.com/Azordev/did-admin-panel/issues/168 ?


data

Add data file support - for supported device for example - All devices specifications can be stored into a yaml file for examples and read at startup. Information on a connected device can be retrieve by parsing the yaml. We can consider to offer the possibility to add a new device configurations using json structure.

File are open after reset from system memory using a get_id command. Add description file checking using cerberus package.

link this issue with devices branch.

Need to add option bytes in device scheme


legion prof: Rendering gather/scatter copies

Currently, gather/scatter copies are not rendered correctly in the timeline, because their source/target instances are not logged by Realm. @streichler proposed to add the following fields to OpreationMemoryUsage:
struct IndirectSrcTgt { Memory memory; size_t size; };
std::vector<IndirectSrcTgt> indirect_sources, indirect_targets;

There is a bigger question of how to render gather/scatter copies with multiple indirect sources/target instances. Our current timeline associates each copy with only one memory-to-memory channel and this isn't quite compatible with gather/scatter copies whose dataflow is not one-to-one. I'd like to brainstorm with the group to decide how to visualize gather/scatter copies.

This was fixed by @eddy16112 a long time ago.


Posts: check YAML formatting

Part of #1551
Some of our posts contain an extra space in the YAML header between a key and value, e.g.:
title : 'Bulletin Hebdomadaire Bitcoin Optech #290'

Jekyll strips the extra space, so the key becomes title.  Hugo preserves the extra space, so the field doesn't work as expected.
This PR eliminates the extra space in the source file and adds a test to help avoid future extra spaces.

Post merge ACK


Coordinate system conversion utilities

Consider adding functions to convert between ENU, NED and NWU.

[ ] Implementation
[ ] Unit tests
[ ] Documentation
[ ] Example code

Decided this isn't in scope for the library, and the IMU should be configured for the desired system.


Java Indentation Fix

The Java code had no indentation inside the main method, The missing indentation is now added

Hey,  no need to make a PULL Request for such a small error.


Dataset:  Inconsistent coding format----   what's & what’s (shown as what\u2019s)

Hi, thanks to your generousity of the opensource of Finetune dataset of your 5k workers!
Howevery, I notice that there's inconsistent coding format in 15k json dataset, sometimes words with special symbols (' - prounouces etc.) are shown in unicode format sometimes not, e.g :
"instruction": "Alice's parents have three daughters: Amy, Jessy, and what\u2019s the name of the third daughter?"
in this sentence the 'what's' word must be type as 'what’s'.
Did you notice that? Will it cause any problems?

Hey thanks for raising it. I think it might actually be good for the training data to have these different variations in punctuation so that models trained on it aren’t too rigid in input expectations.


webgpu implementation?

Don't know the API, sorry, and no time to learn right now. It's not a difficult piece of code to port, but it's not something I am planning to do any time soon. The thing I would port first is the new and improved "psrdnoise", though, in an adjacent repo (github.com/stegu/psrdnoise).

Thanks!


Update ckan role to work with ansible 2.x

variables passed to with-items can no longer be bare and must be in
quotes/double-braces: "{{myVariable}}"

I confirmed with Frank that he has ansible 2.x installed, but it would still be good to check with other devs to see whether anyone uses 1.x and whether these changes break things for them.

@mattboehm We've been updating the https://github.com/cfpb/aurora code for Ansible 2.x, I think generally that's the way to go at this point, but @sleitner @JeffreyMFarley do you all have an opinion?

Yeah, I agree. devs should be upgrading if they're not already on 2.


Control diff2html font sizes

diff2html CSS sets font sizes to 10-15px, and font family to two Menlo, Consolas... In VSCode, I think we should honour editor font settings.
This PR does that. Because I'd prefer not to replicate the list of diff2html CSS classes that have font settings on them, I just use body * to win in specificity; but it means that where we set a font size ourselves, we have to do body <our selector> to get the right specificity for these settings to apply. It's a bit ugly, but I can't think of a cleaner solution right now.
Additionally, the PR sets line-height to 1.4 which is more than the default and more in line with the default styles in VSCode; I don't think there's an easy way of replicating the line height of the editor for this. The default is a bit too cramped.
Finally, while there, I also removed the file hiding workaround which is no longer necessary.

Thanks for merging!
1.3.1? 8-)

Sure! I'll do it this week.

Thank you!
If any issues come in because of my stuff, feel free to tag me.


android:textIsSelectable="true" causes weird layout issues

adding android:textIsSelectable="true" to the inner TextView causes weird layout issues when toggling the expanded mode. This is crucial for an app i'm working on right now.

I saw this problem.

I saw this problem.

Is there a solution to this problem I have encountered？


Unable to find instance for EXCHANGIS-SERVICE

Unable to find instance for Exchangis-Service after opening the WEB interface

The reason may be that my environment is using mysql-8.0+ and I cannot connect to the database

You can check if you have installed mysql command on your server.If not, try to init database manually according to the installation document.

The reason may be that my environment is using mysql-8.0+ and I cannot connect to the database

have same error, how to connet mysql 8.0+

Recompile and run with the MySQL8 driver to see the information below the error log prompt for the service
"Caused by: com.mysql.cj.exceptions.InvalidConnectionAttributeException: The server time zone value 'EDT' is unrecognized or represents more than one time zone. You must configure either the server or JDBC driver (via the 'serverTimezone' configuration property) to use a more specifc time zone value if you want to utilize time zone support."
My solution:
1.In..This problem is resolved by finding the location of the application.yml configuration and adding serverTimezone=UTC, as follows
Url: JDBC: mysql: / / ${DB_HOST: <IP_ADDRESS>} : ${3306} DB_PORT: / ${DB_DATABASE: exchangis}?UseUnicode = true&characterEncoding = utf-8 & serverTimezone = UTC
2.Install the MySQL5 database

Please check whether there is a problem with the startup procedure. The latest version is exchangis1.1.1. You can pay attention to it.


Fusion results color problems

Hi. Thank you for sharing. I want to reproduce the effect of color images in the project, and I want to achieve the effect of the three channels output. But the result image in the output folder of your project directory turns out to be an 8 bits gray image. I would like to ask how to modify the code to produce the effect I want. If you see, please reply to me as soon as possible! Thank you very much again!

Thank you for your comments. We have provided the rgb inference in each folder of the subtask. Please refer to "test_image_rgb.py" for more detail. And, yes, the original code can only deliver grayscale output. Thus, we achieve this objective based on the ycbcr transformation.


Pull upstream changes

The ingestor has been missing the latest changes. Upstream should be pulled.

this is done
https://github.com/CelestiaScan/juno/commit/b171a15b7469b50be82bf110854061742ae79467

Thanks a lot for your help.
Can you please open PRs to Celestia forks containing the changes, so we have an updated  version as well ?


gvrf-physics: stop/start the simulation on pause/resume

Doesn't quite immediately stop on pause. Few simulation steps do happen away from user's eyes. Doing something about it would require rearranging some potentially sensitive things in onPause.

GearVRf-DCO-1.0-Signed-off-by: Mihail Marinov<EMAIL_ADDRESS>

Hi @liaxim,
Looks good to me
but I think we should consider the time spend on pause and sum it to the mLastSimulTime
otherwise mTimeStep may get a wrong value.
 mTimeStep = mSimulationTime - mLastSimulTime;

@ragner On resume, mLastSimulTime gets set to the current uptime. Some time later mSimulationTime is set to the current uptime at that point. The delta should be reasonable. How exactly do you see mTimeStep being wrong?

Hi @liaxim,
Your patch is correct.
I did a mistake, sorry.
Looks good to me.

Thanks. Planning to merge on Friday.


[iOS 16] App freeze when hitting back on navigation bar, using a WrappingHastack in a view as a NavigationLink

Describe the bug
On iOS 16, app meets a severe hang when using a wrappingHastack inside of a navigation link if you hit back button on the navigation bar.
To Reproduce
Steps to reproduce the behavior:
struct ContentView: View {
    var body: some View {
        NavigationView {
            VStack {
                NavigationLink("Hit me ") {
                    WrappingView()
                }
            }
        }
    }
}

import WrappingHStack

struct WrappingView: View {
    @State private var array: [String] = []
    var body: some View {
        VStack {
            WrappingHStack(array,
                           id: \.self,
                           alignment: .center,
                           spacing: .constant(10)) { topic in
                Text(topic)
            }
        }
        .onAppear {
            array = ["test",
                     "test3",
                     "test4",
                     "test5",
                     "test6",
                     "test7"]
        }
    }
}

run the app with code above,
hit the navigation link
hit back button on the navigation bar

Expected behavior
Go back the previous screen and not freeze the app
Screenshots
If applicable, add screenshots to help explain your problem.

Context:

WrappingHStack version: [e.g. 2.2.1]
Model: iPhone 13 pro, simulator , iPhone 11
OS: iOS 16

Additional context
Freeze happens only if you use this lib inside of a Navigation link

It seems to be related to an issue when using NavigationLink with contained GeometryReaders that are not inside a NavigationView. The workaround is to set a NavigationView around the WrappingHStack.
In your example, you could use:
struct ContentView: View {
    var body: some View {
        NavigationView {
            VStack {
                NavigationLink("Hit me ") {
                    WrappingView()
                }
            }
        }
    }
}

import WrappingHStack

struct WrappingView: View {
    @State private var array: [String] = []
    var body: some View {
       NavigationView { // <<<< HERE
            VStack {
                WrappingHStack(array,
                               id: \.self,
                               alignment: .center,
                               spacing: .constant(10)) { topic in
                    Text(topic)
                }
            }
            .onAppear {
                array = ["test",
                     "test3",
                     "test4",
                     "test5",
                     "test6",
                     "test7"]
            }
        }
    }
}

For more information see this link

I have also added this use case to the example app

Thank you 🙏

@dkk I have a similar problem in two cases but a more complicated structure. The workaround doesn't help.
Case 1
TabView {
    NavigationView {
        ScrollView {
            WrappingHStack {

Case 2
TabView {
    NavigationView {
        ScrollView {
            NavigationLink {
                ScrollView {
                    WrappingHStack {

There are also intermediate views and ZStacks and VStacks that I didn't mention.


Update config.py

Fine 😊
On Tue, Jul 11, 2023, 9:08 AM ᴍʀ. x @.***> wrote:

@.**** commented on this pull request.
On config.py
https://github.com/AnonymousX1025/FallenMusic/pull/28#discussion_r1259136765
:
Why bro why ?
—
Reply to this email directly, view it on GitHub
https://github.com/AnonymousX1025/FallenMusic/pull/28#pullrequestreview-1523351876,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BBEXWIQYRHJGEC776S44VO3XPTDCPANCNFSM6AAAAAA2D27KZI
.
You are receiving this because you authored the thread.Message ID:
@.***>


Posts adapter

posts adapters and item_post to create a layout for each post

There are too many commits added to this single branch.
Did you create a new branch every time cherry picking?
But since you already have the separated commits, try Github Desktop dragging feature: https://github.blog/2021-03-30-github-desktop-now-supports-cherry-picking/


Upgraded librouteros version

Description:
New RouterOS has new authentication mode, the librouteros 2.1.0 has this feature implemented.

Hi @dejx,
It seems you haven't yet signed a CLA. Please do so here.
Once you do that we will be able to review and accept this pull request.
Thanks!

Don't remove the PR template and make sure you read it


should we rename didReceivePart to didReceiveBodyPart ?

https://github.com/swift-server/async-http-client/blob/e0eeb0481c0733f7dd6323916c9f4b453ed05be5/Sources/AsyncHTTPClient/HTTPHandler.swift#L322
this is a little unclear, a part of what? I'd say we should rename to didReceiveBodyPart

Haha I thought about that a while ago but decided "body part" was a bit gruesome 😁

"body part" was a bit gruesome

haha. RequestBodyPart / BodyChunk (not so great because it might make people think about chunked encoding which isn't necessary for streaming) / smth else?

"body part" is the correct terminology so if it helps make the API clearer then maybe we just go with it...


App indexes

Instead of hardcoding OpenSearch indices to application-*, we use the service name as the index.
The application-* index was becoming crowded and large.

Sorry, delete this. Opened a PR on the wrong repo.


Run the "no-default-features" tests in CI

I created the "no-default-features" tests as part of https://github.com/awslabs/smithy-rs/pull/2055
In order to unblock the release, we decided not to implement a new CI mechanism to run those tests in CI. However, the tests aren't very useful if we don't start doing that.
To whomever takes this on: update CI to start running integration tests that aren't associated with a specific service and that reside in their own crate.

I spoke to the examples team about moving this into their repo and they aren't interested unless we can demonstrate the usefulness of these tests as examples.


python310Packages.urllib3: 1.26.14 -> 2.0.3

https://github.com/urllib3/urllib3/releases/tag/2.0.3
Description of changes
upgrade to include 2.0.3

Things done

Built on platform(s)

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

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

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

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

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

[ ] Fits CONTRIBUTING.md.

This triggers over 5000+ rebuilds, thus can't go to master. Also, it will probably break a couple of hundreds packages as they have a constrain for urllib3<2.
https://github.com/urllib3/urllib3/releases/tag/2.0.0

This triggers over 5000+ rebuilds, thus can't go to master. Also, it will probably break a couple of hundreds packages as they have a constrain for urllib3<2.
https://github.com/urllib3/urllib3/releases/tag/2.0.0

🤦 ; @fabaff , thanks!

Looks like I have a more to learn about where to merge this.  Will read the docs.  In case it helps place where I'm at in nix journey ... still figuring out the relation of release branches [ ex: 23.05 ] - if not point in time snapshots - to master.  Will start with https://github.com/NixOS/nixpkgs/blob/master/README.md and then find other.

Breaking other packages:  Are packages that tightly coupled?  Meaning is that a problem for the other packages to eventually fix?  Or would need to be part of a PR to make sure to fix the hundreds of other things that don't haven't updated their constraints to urllib3<3 -- any pointers for reading here?

LOL -->
https://github.com/NixOS/nixpkgs/commit/b53d78210594bac731001a3c45f4f8778a8f4c2d
and
https://github.com/NixOS/nixpkgs/commit/c033f340de6b9ea78cc5fbc3c296b99b27354dd5 [ a PR of yours ]
made it seem like upgrading versions was simple :-)

This is a good resource for understanding the staging workflow: https://nixos.org/manual/nixpkgs/stable/#submitting-changes-commit-policy.

Breaking other packages: Are packages that tightly coupled? Meaning is that a problem for the other packages to eventually fix? Or would need to be part of a PR to make sure to fix the hundreds of other things that don't haven't updated their constraints to urllib3<3 -- any pointers for reading here?

From my observation, it's unfortunately very common for upper bound constraints to exist on urllib3. A saving grace is that many of the rebuilds needed probably depend on urllib3 transitively, so perhaps fewer packages need fixing.
I like to think we're all on the same team pushing changes to balance keeping packages working with other goals (such as fixing bugs, closing security issues, etc.). So we have to do due diligence to not break many packages without a seriously compelling reason.

Onto the PR itself, thank you putting it up! I've been slowly working on the side on it too, so I'd be interested in combining efforts to get this done.
To start with, here are 3 PRs that will be needed to minimize breakages:

[ ] https://github.com/NixOS/nixpkgs/pull/240785
[ ] https://github.com/NixOS/nixpkgs/pull/240796
[ ] https://github.com/NixOS/nixpkgs/pull/241075

Where I need help the most currently is testing botocore (and, relatedly, awscli2) with the new version. It is unfortunately not ready upstream (see https://github.com/boto/botocore/issues/2926), but they need to take into account cloud (e.g. AWS) infrastructure, whereas we might be able to do this faster in nixpkgs since we carry all newer versions of OpenSSL.

Breaking other packages: Are packages that tightly coupled? Meaning is that a problem for the other packages to eventually fix? Or would need to be part of a PR to make sure to fix the hundreds of other things that don't haven't updated their constraints to urllib3<3 -- any pointers for reading here?

I guess that there are a little over 250 packages present which depends directly on urllib3. urllib3does semantic versioning as far as I can tell and so many modules declare something like urllib3 <2.0, >=1.x.x in their requirements. Unlike PyPI we have only one version of a Python module (there are exceptions but only as a last resort).
Getting widely used modules updated like urllib3, httpx or paramiko updated usually means to get their consumers updated first.

made it seem like upgrading versions was simple :-)

Those urllib3 releases didn't contains breaking changes.

I discovered that botocore truly isn't ready yet. urllib3 made changes to its implementation of HTTPConnection in https://github.com/urllib3/urllib3/pull/2565 which breaks botocore's implementation to support "100 Continue" HTTP responses from servers. I will try to work on a fix upstream.

I'll use this comment to record any additional PRs that will be needed before we update this.

[ ] https://github.com/NixOS/nixpkgs/pull/244753

We can update botocore to 1.31.15 now and unpin its urllib3 dependency to get us one more step closer to bumping this. Please feel free to keep going on this if anyone has the time.

Already on master.


Evaluate a policy via pipes

Is your feature request related to a problem? Please describe.
I love the DSL of LetMe, but I feel like it is missing a "throw or continue" way of evaluating policies.
Currently, we're locked into the :ok or {:error, :reason} tuple pattern that is common across Elixir, so evaluating a policy  might look like:
def update_membership(current_user, id) do
  membership = Repo.get(Membership, id)

  Membership.Policy.authorize!(:membership_update, membership, current_user)

  Membership.update(membership, %{something: true})
rescue
  LetMe.UnauthorizedError ->
    {:error, :unauthorized}
end

There is nothing wrong with this approach – but I feel like there could be better ergonomics with assessing policies via pipe operators.
Describe the solution you'd like
I'd love to be able to do something like:
def update_membership(current_user, id) do
  Membership
  |> Repo.get(id)
  |> Membership.Policy.authorize!(:membership_update, current_user)
  |> Membership.update(%{something: true})
rescue
  LetMe.UnauthorizedError ->
    {:error, "something bad"}
end

Describe alternatives you've considered
I've looked at Bodyguard as well, and it seems like this isn't supported either. I'm a big fan of the LetMe DSL – does something like this exist?

Is your feature request related to a problem? Please describe.
I love the DSL of LetMe, but I feel like it is missing a "throw or continue" way of evaluating policies.
Currently, we're locked into the :ok or {:error, :reason} tuple pattern that is common across Elixir, so evaluating a policy might look like:
def update_membership(current_user, id) do
  membership = Repo.get(Membership, id)

  Membership.Policy.authorize!(:membership_update, membership, current_user)

  Membership.update(membership, %{something: true})
rescue
  LetMe.UnauthorizedError ->
    {:error, :unauthorized}
end

I don't know whether the example is incomplete and you're doing additional things in the rescue, but if all you want is an error tuple as a return value, I'd write it with a with:
def update_membership(current_user, id) do
  membership = Repo.get!(Membership, id)

  with :ok <- Membership.Policy.authorize(:membership_update, membership, current_user) do
    Membership.update(membership, %{something: true})
  end
end

Or if you have a fetch variant in your repo module or something similar:
def update_membership(current_user, id) do
  with {:ok, membership} <- Repo.fetch(Membership, id),
          :ok <- Membership.Policy.authorize(:membership_update, membership, current_user) do
    Membership.update(membership, %{something: true})
  end
end

There is nothing wrong with this approach – but I feel like there could be better ergonomics with assessing policies via pipe operators.
Describe the solution you'd like
I'd love to be able to do something like:
def update_membership(current_user, id) do
  Membership
  |> Repo.get(id)
  |> Membership.Policy.authorize!(:membership_update, current_user)
  |> Membership.update(%{something: true})
rescue
  LetMe.UnauthorizedError ->
    {:error, "something bad"}
end

Describe alternatives you've considered
I've looked at Bodyguard as well, and it seems like this isn't supported either. I'm a big fan of the LetMe DSL – does something like this exist?

The reason why the object is the last argument in the authorize functions is that the object is optional. For example, policies for retrieving a list of items or create policies usually don't require the object.
I don't see a lot of value in being able to pipe the object into authorize!/3. Maybe I'm missing context, but in the example, it looks like piping for the sake of piping to me.


Package license file?

This repo is missing a license - can this be packaged? Thanks!

I reached out internally a few days ago and I'm waiting on a response. I'll let you know if something comes back.

https://github.com/sspinc/terraform-provider-credstash/commit/b2efd0826001685fec1a1a6a2ab73c5fbd26fc10 adds a license file. We are using MIT so go ahead and package this.
Sorry this took so long.


the previous dependency  is only one time step during inference?

https://github.com/wilson1yan/VideoGPT/blob/d157da51b3b9766648eb1e54a1008ff965e26b65/videogpt/gpt.py#L97-L107
hi, @wilson1yan!  In these lines, it seems that the iterative generation of the next code only depends on the one previous time step? The shape of embeddings_slice is always [bs, 1, 1, 1, embed_dim].

The generation of the next token does depend on all previous tokens. The sampling code uses caching to make sampling more efficient / faster. Specifically, as each token is generated, the intermediate hidden units are stored and reused at later timesteps so they don't have to be re-calculated. The relevant caching code can be found here

thank you @wilson1yan! you are right.


Obfuscated code fails to work in IE8

Hello! I know IE8 is a little bit outdated, but still =)
Running the obfuscated script on IE8 outputs the following error:
'filter' is null or not an object
I assume this is caused by this JSFuck line, as Array.filter support started from IE9:
Window: '[]["filter"]["constructor"]("return this")()',
If this is the case, it should probably be safe to change this for a more supported method, such as push:
Window: '[]["push"]["constructor"]("return this")()',

what version you use?

I've used 0.8.0. Should it help to update to 0.8.1?

Only one place where []["filter"]["constructor"]("return this")() still using is following line:
https://github.com/javascript-obfuscator/javascript-obfuscator/blob/master/src/templates/custom-nodes/debug-protection-nodes/debug-protection-function-node/DebugProtectionFunctionTemplate.ts#L13
I will replace this line today on {}.constructor('debugger')()

Thanks! I see that this line is intended to be used with debug protection mode. However, my script has been built without debugProtection flag set to true (actually, without this flag at all, but readme says its default value is false). Is this a desired behavior?

maybe it is line here https://github.com/javascript-obfuscator/javascript-obfuscator/blob/0514793257b306d38b0936ec16031818b5e783c7/src/enums/JSFuck.ts
but i tested code before 0.8.0 release on ie9 in ie8 compatibility mode and it doesnt't throw any errors.
Anyway i change this lines today and i hope it helps for you.

Thanks again! One more issue: disableConsoleOutput option works not as expected:
With value set to true, console.log function is replaced with the following one:

This obviously hangs the page with an infinite loop, and surely this is not an expected behavior (this option is turned on by default!)

One more issue: disableConsoleOutput option works not as expected
already fixed in 0.8.1

That's correct, I was using an outdated dependency, my fault.

reopen until fix

Ready in<EMAIL_ADDRESS>Please try and check.

Thanks, I will perform heavier autotesting on it next week. For now, I've performed several manual builds and they seem to work in IE8.

Nice!

any test results?

hi,I'm using the webpage to do the obfuscate, the obfuscated code doesn't work in IE8.

hi, show me your obfuscated code please

@sanex3339
hi, Timofey
pls see below,
obfuscated (1).js.zip

i will look into it

Haven't any runtime errors.
If you doesn't have any output to the console - set disableConsoleOutput to false

thanks, after disable the consoleoutput, it works.

Nice!

So,  i will close this issue.  If you will any problems with ie8 just create new issue.


Assignment count not shown for experiments with just one bucket

http://localhost:8080/#/experiments/{experimentID}/false/ does not show assignment counts in the right column when the experiment only has one bucket.

Resolved by #99 .


HOW DO I OPEN .ZGE FILES IN ZGAMEEDITOR VISUALIZER??

hi im trying to open the .zge file I just exported but I'm growing gray hairs trying to find out how to import the dang thing into fl studio can you help?

I should have added instructions for that. You just need to move that .zge file into the ZGameEditor folder within your FL Studio installation folder.
The usual installation folder is:
C:\Program Files\Image-Line\FL Studio 21\Plugins\Fruity\Effects\ZGameEditor Visualizer\Effects\Postprocess
But if that doesn't exist, you can just find out where FL Studio is installed, go to that folder and then follow this folder path:
\Plugins\Fruity\Effects\ZGameEditor Visualizer\Effects\Postprocess
I'll try and get these instructions added and maybe include a video for reference.


Add files via upload

mexc key renewal error fix
Before submitting this PR, please make sure:

[ ] Your code builds clean without any errors or warnings
[ ] You are using approved title ("feat/", "fix/", "docs/", "refactor/")

A description of the changes proposed in the pull request:
Tests performed by the developer:
Tips for QA testing:

@bossdebitbotz what happened to this PR?


Can not override jsx option in tsconfig

Environment
Nitro 2.7.2
Node 20.9.0
Reproduction
https://stackblitz.com/edit/github-nwbg1j?file=nitro.config.ts
Run npm run typecheck after installing dependence.
Describe the bug
In https://github.com/unjs/nitro/pull/1303, in order to support jsx/tsx, the default parameters of jsxFactory and jsxFragment are added. But this affects me setting jsx option of tsconfig to react-jsx even though I'm not using nano-jsx.
Additional context
No response
Logs
No response

Hi!
I tried include in nitro.config.ts
esbuild: {
  options: {
    jsxFactory: 'React.createElement',
    jsxFragment: 'React.Fragment',
  }
}

And in tsconfig.json:
{
  "extends": "./.nitro/types/tsconfig.json",
  "compilerOptions": {
    "jsxFactory": "react",
    "jsxFragmentFactory": "React.Fragment"
  }
}

And it worked for use react and react-dom to render in server side with renderToStaticMarkup from react-dom/server


Inherit contact model

Hi, I have such problem:
I`d like to inherit contact model and when i do this, migrations appear in my venv, but not at my app migrations folder. Is it bag or how can i fix it?

There shouldn't be migrations appearing in your venv, do you have an example of the migrations being created?


More social networks, and existing ones adjusted

Thanks for this pull request - However the strings in the source files should be supplied in English, then they can be translated into other languages on Transifex:  https://app.transifex.com/openstreetmap/id-editor/community/

I updated the strings to English.


Name issues with other popular packages

I just wanted to create a list with name conflicts with other popular Julia packages,
maybe one could help users with a hint or warning messages ("use PlutoUI.### instead of ### to prevent name conflict"), if two packages with conflicting names were used.
One of such name conflicts is Slider with the GLMakie.jl package:
https://makie.juliaplots.org/stable/examples/layoutables/slider/index.html

Also #166 !

Now that https://github.com/fonsp/Pluto.jl/issues/890 is fixed, this is not an issue anymore :)


gradlew build Failed Godot 3.1 beta 8 'could not find unity-ads'

gradle builf failed after compiling for android :
FAILURE: Build failed with an exception.

* What went wrong:
Could not resolve all files for configuration ':debugCompileClasspath'.
> Could not find :unity-ads:.
  Searched in the following locations:
      file:/D:/Applications/godot_v3.1_beta/source_code/godot-3.1-beta8/modules/GodotAds/libs/unity-ads.aar
      file:/D:/Applications/godot_v3.1_beta/source_code/godot-3.1-beta8/modules/GodotAds/res/unity-ads.aar
  Required by:
      project :

i downloaded unity-sdk to a wrong path


Build script and readme fixes

Fixes for build.sh failing with tags -debug not defined issue and minor updates to readme.

@cmonr - Those are good suggestions, changes are applied.


Support DNS Service (SRV) records to get the port numbers of upstream servers via DNS

This would allow users to add servers with only a dns record instead of specifying an ip addrses and port. This may be required in order to do https://github.com/hjr3/alacrity/issues/49

See https://www.nginx.com/blog/dns-service-discovery-nginx-plus/ for some details.

for info, support in hyper: https://github.com/hyperium/hyper/blob/master/src/client/dns.rs


psdk_ros2: 1.1.0-1 in 'humble/distribution.yaml' [bloom]

Increasing version of package(s) in repository psdk_ros2 to 1.1.0-1:

upstream repository: https://github.com/umdlife/psdk_ros2.git
release repository: https://github.com/ros2-gbp/psdk_ros2-release.git
distro file: humble/distribution.yaml
bloom version: 0.10.7
previous version for package: 1.0.0-1

psdk_interfaces
* Merge pull request #51 <https://github.com/umdlife/psdk_ros2/issues/51> from umdlife/feat/hms-support
  Integration of Health Monitoring System (HMS) as a module
* Merge pull request #59 <https://github.com/umdlife/psdk_ros2/issues/59> from umdlife/extend_single_battery_info
  Extend single battery info
* Merge pull request #57 <https://github.com/umdlife/psdk_ros2/issues/57> from umdlife/feat/single_battery_info
  Add new message types HmsInfoMsg, HmsInfoTable
* Contributors: Stevedan, Victor Massagué Respall, amoramar, biancabnd, marta

psdk_wrapper
* Merge pull request #51 <https://github.com/umdlife/psdk_ros2/issues/51> from umdlife/feat/hms-support
  Integration of Health Monitoring System (HMS) as a module
* Merge pull request #59 <https://github.com/umdlife/psdk_ros2/issues/59> from umdlife/extend_single_battery_info
  Extend single battery info
* Merge pull request #57 <https://github.com/umdlife/psdk_ros2/issues/57> from umdlife/feat/single_battery_info
  Adding topic for reading battery info
* Contributors: Stevedan, Victor Massagué Respall, amoramar, biancabnd, marta

Holding for Humble sync


buggy type inference for props with array type

Version
2.5.16
Reproduction link
https://raw.githubusercontent.com/indus/stuff/master/vue_issue.jpg
(sorry - haven't found an online tool that shows type annotations)
Steps to reproduce
type the following in an TS enabled Editor like VS Code:
new Vue({
    el: '#app',
    props: {
        object: {
            type: Array,
            required: true,
            default: () => []
        }
    },
    data: function () {
        return {
            message: 'Hello Vue.js!'
        }
    },
    created: function () {
        this.message;
    }
})

What is expected?
correct type inference (at least) for 'message'
What is actually happening?
getting Error: "Property 'message' doesn't exist"

This doesn't happen when the type of the prop is Object:
new Vue({
    el: '#app',
    props: {
        object: {
            type: Object,
            required: true,
            default: () => {}
        }
    },
    data: function () {
        return {
            message: 'Hello Vue.js!'
        }
    },
    created: function () {
        this.message;
    }
})

or 'data' is just a simple object and not a function
new Vue({
    el: '#app',
    props: {
        object: {
            type: Array,
            required: true,
            default: () => []
        }
    },
    data: {
        message: 'Hello Vue.js!'
    },
    created: function () {
        this.message;
    }
})

This looks like duplicate of #7640 if you are using TypeScript 2.7.
If it's not the same issue, please provide a example github repo that reproduce it.

This is exactly the duplicate of #7640.
type: Array breaks the overloading function Vue.extend. The last alternative of overloading is picked so this-type isn't included.
The workaround is manual annotation.
import Vue, {PropOptions} from "vue"

Vue.extend({
    props: {
        myProp: {
            type: Array,
        } as PropOptions<any[]>,
    },
})

@HerringtonDarkholme Thanks for that workaround. Kind of hard to believe this is still an issue.


[BUG] 403 on Any Videos

Bug Description
Getting 403 error on any youtube video
Reproduction Steps
Steps to reproduce the behavior:

Just try to use a youtube video

Attachments

Versioning:

OS: Ubuntu 22.04
Node Version v22
Module Version az29

Hi @BennyDaBee ,
Thank you for the bug report!
Please provide more details on reproducing this issue.
Every release goes through a simple E2E flow, and no issues were seen.

When providing a youtube link to queue.play() it will throw a 403 error regardless of what the video is.

Thank you for the response.
Your statement regarding " 403 error regardless of what the video is" is clearly not the case. If it was, it would have been caught before release.
As usual, please provide more details (i.e, the url you are using, and the stack trace when it errors out), so I can better assist you.

https://www.youtube.com/watch?v=6Zbi0XmGtMw&pp=ygUPV2UgbGlrZSB0byBwYXJ5 - Example URL, again it happens with any video. Here is the error I see in the logs.
https://www.youtube.com/watch?v=6Zbi0XmGtMw&pp=ygUPV2UgbGlrZSB0byBwYXJ5 WARNING: @distube/ytdl-core is out of date! Update with "npm install @distube/ytdl-core@latest". WARNING: Could not parse n transform function. Please report this issue with the "1722106383910-base.js" file on https://github.com/distubejs/ytdl-core/issues. Stream URL may be slow but still works. node:events:511     throw err; // Unhandled 'error' event     ^ Error [ERR_UNHANDLED_ERROR]: Unhandled error. ('Status code: 403')     at Player.emit (node:events:509:17)     at StreamConnection.<anonymous> (/Bot/node_modules/discord-music-player/dist/managers/Queue.js:185:47)     at StreamConnection.emit (node:events:520:28)     at AudioPlayer.<anonymous> (/Bot/node_modules/discord-music-player/dist/voice/StreamConnection.js:91:18)     at AudioPlayer.emit (node:events:520:28)     at Encoder.onStreamError (/Bot/node_modules/discord-music-player/node_modules/@discordjs/voice/dist/index.js:1122:14)     at Object.onceWrapper (node:events:635:26)     at Encoder.emit (node:events:532:35)     at emitErrorNT (node:internal/streams/destroy:170:8)     at emitErrorCloseNT (node:internal/streams/destroy:129:3) {   code: 'ERR_UNHANDLED_ERROR',   context: 'Status code: 403' }

My apologise, was due to my outdated ytdl-core. I will close this.


The camera setting of gobjaverse

Hi, could you provide the camera intrinsics setting of gobjaverse, like the focal_length / fov? And what formula did you use to determine the distance between the object and the camera?

see in  camera.py

Thanks for replying, but I'd like to know how are xyz and origin defined here?

x,y,z mean camera position in Unity coordinate while origin it is homo coefficient. General speaking, origin is set to 1.


How can i handle multiple connection, with different ip and different ports ?

I need to handel the connection on different servers, is possible to open different connection, with like 5 different servers, like:
websockifyMaker({
source: '<IP_ADDRESS>:8081',
target: 'remoteServerIp:5901'
}, {
source: '<IP_ADDRESS>:8082',
target: 'remoteServerIp:5902'
}, {
source: '<IP_ADDRESS>:8083',
target: 'remoteServerIp:5903'
}
);
Thank you

You can just make multiple instances
let websockify = []
websockify[0] = require('@maximegris/node-websockify');
websockify[1] = require('@maximegris/node-websockify');
websockify[2] = require('@maximegris/node-websockify');

websockify[0]({
    source: '<IP_ADDRESS>:8081',
    target: 'remoteServerIp:5901'
});

websockify[1]({
    source: '<IP_ADDRESS>:8082',
    target: 'remoteServerIp:5902'
});

websockify[2]({
    source: '<IP_ADDRESS>:8083',
    target: 'remoteServerIp:5903'
});

@emulk https://github.com/sukkamehu/node-multi-websockify Just moment ago I created probably something which you could utilize?

You can just make multiple instances
let websockify = []
websockify[0] = require('@maximegris/node-websockify');
websockify[1] = require('@maximegris/node-websockify');
websockify[2] = require('@maximegris/node-websockify');

websockify[0]({
    source: '<IP_ADDRESS>:8081',
    target: 'remoteServerIp:5901'
});

websockify[1]({
    source: '<IP_ADDRESS>:8082',
    target: 'remoteServerIp:5902'
});

websockify[2]({
    source: '<IP_ADDRESS>:8083',
    target: 'remoteServerIp:5903'
});

Hi, it doens't work.
`WebSocket settings:
- proxying from <IP_ADDRESS>:8001 to <IP_ADDRESS>:5900
- Running in unencrypted HTTP (ws://) mode
no callbacks
WebSocket settings:
- proxying from <IP_ADDRESS>:8002 to <IP_ADDRESS>:5900
- Running in unencrypted HTTP (ws://) mode
C:\Websockify\node_modules\ws\lib\websocket-server.js:83
throw new Error(
^
Error: The HTTP/S server is already being used by another WebSocket server
at new WebSocketServer (C:\Websockify\node_modules\ws\lib\websocket-server.js:83:15)
at Server. (C:\Websockify\node_modules@maximegris\node-websockify\websockify.js:199:14)
at Object.onceWrapper (node:events:628:28)
at Server.emit (node:events:526:35)
at emitListeningNT (node:net:1785:10)
at process.processTicksAndRejections (node:internal/process/task_queues:81:21)
Node.js v18.17.1`

I need to handel the connection on different servers, is possible to open different connection, with like 5 different servers, like:
websockifyMaker({ source: '<IP_ADDRESS>:8081', target: 'remoteServerIp:5901' }, { source: '<IP_ADDRESS>:8082', target: 'remoteServerIp:5902' }, { source: '<IP_ADDRESS>:8083', target: 'remoteServerIp:5903' }
);
Thank you

did you solve?


Don't Merge! PR real time test for webhook case

Docs Build status updates of commit 172696d:
:clock10: Pending: waiting for processors (26 builds ahead of you)
:warning: Docs Build is busy, currently there are 26 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 172696d:
:clock10: Full build: the average full build time is 7 min(s) 16 sec(s), this is based on the last 30 build(s) for this repository.

Docs Build status updates of commit 172696d:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-conceptual/testing.md
:warning:Warning
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)
Details

docs-ref-conceptual/testing.md

[Warning] Missing attribute: either ms.prod or ms.service is required. Use ms.prod for on-premise products, or ms.service for cloud services.
[Warning] Missing attribute: title.
[Warning] Missing attribute: either ms.prod or ms.service is required. Use ms.prod for on-premise products, or ms.service for cloud services.
[Warning] Missing attribute: title.
[Warning] Missing attribute: either ms.prod or ms.service is required. Use ms.prod for on-premise products, or ms.service for cloud services.
[Warning] Missing attribute: title.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Event data doesn't seem to be working with numeric data

I'm trying to use the Event Data in a test environment, and in most cases besides "count" I'm not getting what I'm expecting.
I'm only using one event data so this should be pretty clear what to expect. Here is the data in my event_data table:
event_data_id  | event_id |               event_data
---------------+----------+---------------------------------------------------------------
             1 |        1 | {"distance": 12800, "preferPaved": 1, avoidMainRoads": 0}

Getting the average of a number field always shows 0, even though count shows there is 1 event with that field as expected

Getting the sum of a number field also doesn't seem to work for me, always showing 0:

Not sure if I'm missing something here, but would appreciate any help. Thanks!

I haven't been able to get anything but count = 0 out of that screen and there are no instructions on how it's supposed to work.

We are currently rewriting the event data functionality for the app. Storing all the data in a single json column with no restrictions and trying to query and aggregate that data has not been optimal.

@franciscao633 That makes sense, thanks for the response. I'll keep my eyes open for when those updates are in.


[QUESTION] Export in Excel my collection with the relationship

Versions

PHP version: 7.1.3
Laravel version: 5.7
Package version: 3.1

Description
Hi,
I'm trying to export in Excel my Order collection with the relationship (relative Course), but is exported only the collection without the relationship.
Here my export class:
namespace App\Exports;

use Maatwebsite\Excel\Concerns\FromCollection;

class OrderExport implements FromCollection {
    public function collection() {
        return Order::where('status', 'approved')->with('course')->get();
    }
}

How can I export the orders with relative course purchased?
Additional Information
The relationship is: Order belongsTo Course
Best regards,
Luca

See our example in the docs for $invoice->user: https://docs.laravel-excel.com/3.1/exports/mapping.html

Thank you @patrickbrouwers , now it works!
Best regards,
Luca


Expand plus and minus sign not pixel perfekt

Because of the different font-size of the chars +- (26 resp 40) with possible combination of char placement in font, it looks like they are little off center.
Not 100% certain if it is a easy fix or just a fix for given font-family, but you could off the - sign little to make the feel more aligned.
Have tried with replacing:
.attr('y', this.isEdge() ? 10: 0)
with
.attr('y', ({ children }) => this.isEdge() ? 10 : children ? -3 : 0)
and it feels a little better.

Will take a look at this once will find time to keep working on new features

In the new version, we will have new icons for expand collapse, and that button will be user-configurable/overridable as well

https://github.com/bumbeishvili/d3-organization-chart/blob/b64f9ef60f9eeff48969df2ecead9a5504c501f3/src/d3-org-chart.js#L626

https://github.com/bumbeishvili/d3-organization-chart/blob/b64f9ef60f9eeff48969df2ecead9a5504c501f3/src/d3-org-chart.js#L65

The new version is published, closing this.
You can get more info about the new org-chart version in this Medium article


Log Analytics Just Does Not Work

[Enter feedback here]

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

ID: 00ceee8d-f5ae-6d30-8a07-9f5def035fe7
Version Independent ID: ea531828-e26a-3c60-3b66-429ec06f895e
Content: API access and authentication - Azure Monitor
Content Source: articles/azure-monitor/logs/api/access-api.md
Service: azure-monitor
Sub-service: logs
GitHub Login: @guywi-ms
Microsoft Alias: guywild

@adobayua-rpay
Thanks for your feedback! We will investigate and update as appropriate.

Hi @adobayua-rpay,
This azure-docs repository deals about feedback related to particular Azure document page (like correcting doc bugs, doc enhancements, product issues related to wrong doc instructions, etc.).

If the feedback is related to any of it, please elaborate and share it here.
If the issue is with your product usage or with the product itself, please ask the question in public forums like Microsoft Q&A.


feat: added fluid step breadcrumb component

Why
This is a prototype for the fluid-step-breadcrumb component that will be used for from wizard navigation.

It has been roughed out with design input needed.
Co-authored-by: @cwervo

Prototype closing


Undocumented behavior: syncSupabase subscriptions require soft-delete

See here: https://github.com/LegendApp/legend-state/blob/5555c0d515c49d580cde29b8624798f6fbe1e0c5/src/sync-plugins/supabase.ts#L222-L226
Docs only indicate that use of diff-sync requires soft delete, but it looks like subscriptions also have this requirement.
That said, as far as I can tell, we should be able to do a hard-delete with subscriptions, probably just need to add an extra case here. Let me know if there are any gotchas you can think of, otherwise I can put up a PR for it.

Added a draft PR (implementation not complete) just to get the idea across. @jmeistrich just want a quick sanity check... is there a fundamental reason why synced-subscribes require soft-deletion?

I think there is one more undocumented behavior here.
https://github.com/LegendApp/legend-state/blob/7e7cfeaf29fa6604b1854d94c6b559b48d05cd85/src/sync-plugins/supabase.ts#L202-L221
If I don't use diff-sync as the docs says (or just dont have fieldCreatedAt and fieldUpdatedAt), when INSERT and UPDATE events happens it will prevent them from being updated, which deviates from common expectations.

See here:
https://github.com/LegendApp/legend-state/blob/5555c0d515c49d580cde29b8624798f6fbe1e0c5/src/sync-plugins/supabase.ts#L222-L226
Docs only indicate that use of diff-sync requires soft delete, but it looks like subscriptions also have this requirement.
That said, as far as I can tell, we should be able to do a hard-delete with subscriptions, probably just need to add an extra case here. Let me know if there are any gotchas you can think of, otherwise I can put up a PR for it.

This is not actually a soft delete. symbolDelete is used internally to mark nodes as needing to be deleted (as opposed to set to undefined/null) for the merge functions to perform the delete.  So I believe subscriptions should work fine with hard deletes. Or are you seeing them not working?

@jmeistrich yeah, I was hitting this error until I added a deleted column to my table.

I think there is one more undocumented behavior here.
https://github.com/LegendApp/legend-state/blob/7e7cfeaf29fa6604b1854d94c6b559b48d05cd85/src/sync-plugins/supabase.ts#L202-L221
If I don't use diff-sync as the docs says (or just dont have fieldCreatedAt and fieldUpdatedAt), when INSERT and UPDATE events happen it will prevent observables from being updated, which deviates from common expectations.
I agree soft-delete is only required for persist sync, since it's the only way to keep them in db before we query again. For realtime we can receive the delete event with record id and just delete it right away.

@NekodRider Ah you're right, that was an oversight! But that is important to protect against receiving self-updates. It seems like the realtime subscription also includes rows that were just upserted by this client, and they seem to , and it uses the updated_at time to check that the update is newer than the value returned by upsert. Otherwise it might overwrite local state on a slow connection:

Set text to A
That triggers upsert with A
Set text to B
Upsert returns with A - the save logic merges results that weren't changed locally since the save, so it leaves text as B
Realtime subscription triggered with text changed to A - ignores it because it's the same timestamp as #4

So if we don't have the updated_at field, I'm not sure how to ignore realtime events from local changes. Do you know of techniques to do that in Supabase?

@jmeistrich yeah, I was hitting this error until I added a deleted column to my table.

@warrenbhw Ah ok, that is a problem. I reopened that issue and will look into it. Can you share what your syncedSupabase options look like? It may be some combination of options I didn't test enough.

here are my current options:

configureSyncedSupabase({
  generateId: () => nanoid(),
  changesSince: "last-sync",
  fieldCreatedAt: "created_at",
  fieldUpdatedAt: "updated_at",
  // Optionally enable soft deletes
  fieldDeleted: "deleted",
});

  const workflows$ = observable(
    syncedSupabase({
      supabase,
      collection: "workflows",
      filter: (select) => select.eq("workspace_id", workspaceId),
      actions: ["read", "create", "update", "delete"],
      realtime: { schema: "public" },
      persist: { name: "workflows" }, // Persist data and pending changes locally
    }),
  );

but I hit this issue before I enabled  changesSince: "last-sync"

@jmeistrich oh I see the problem. Using updated_at to compare version is likely the most practical way to solve this. Firebase has timestamp by default, but supabase has nothing helpful afaik.
I think you are right we need updated_at for this. Maybe make it a required param instead of setting undefined, or perhaps give an option for "unsafe" mode?

@jmeistrich are soft-deleted rows ever automatically garbage collected (that is to say, hard-deleted) either in local persistence or in the remote data store?

@jmeistrich possibly related:
wen I turn off local persistence, but turn on realtime subscription from supabase, deleting a row in supabase (from a different client) causes my local state to update so that I actually create a new row with no attributes, but don't delete the row that should be deleted.
Creates from a different client behave as expected.
Updates from a different client don't show up at all.

I put a similar comment on this issue, since I wondered if the bugs are somehow related.

I believe alpha.26 should fix the deleting problems. And I think it should fix https://github.com/LegendApp/legend-state/issues/327 too. Does it work better for you after the update?

Let me give it a try right now!


How to include multiple recipient's email?

Thank you all for this amazing library!
Is there any way to include multiple emails in recipient_email?

@VictorSanh I'd love to work on this! Is it possible to assign it to me?
Cheers!

Hi @VictorSanh ,
@rish-16  has already submitted a PR and it looks fine.
If you also find it suitable, please accept his PR.

It has been merged in PR #23.
I'm closing this.


Fix CS

This PR fixes common standards (PSR-2) thanks to the php-cs-fixer.
This includes using spaces instead of tabs, removing useless tabs/useless statements, etc...

Nah... spaces suck :)

@gossi of course this can be changed to your own standards but imo it's always better to configure this tool to have a consistent codebase :-)

I updated this PR to only remove inconsistencies in the codebase.


DROOLS-1305 Move DROOLS-1278 from droolsjbpm-integration/droolsjbpm-i…

…ntegration-tests to jbpm/jbpm-test-coverage
As per requested at droolsjbpm/droolsjbpm-integration#649 .

@mswiderski could you kindly revise this PR, with reference to what requested at droolsjbpm/droolsjbpm-integration#649 , please?

+1 to merge


No provider for Storage!

Hey,
Great work with Ionic 2 guys. I've imported and injected the ionic-storage export ({ Storage } from '@ionic/storage') as a provider in NgModule. I've then imported it and DI it into a service. The problem is, as well as injecting this service into any component which uses it, we are also having to import Storage as well.
Does anyone know why this would be the case?
Thanks

That sounds right.
In my file LocalStorageProvider which accesses storage, I have:
import { Storage } from '@ionic/storage';
And then in app.module.ts:
import { Storage } from '@ionic/storage';
import { LocalStorageProvider } from '../providers/local-storage';

@NgModule({
  providers: [
    Storage,
    LocalStorageProvider,
    ...

In our app.module.ts we have the following.
` import {ProfileService} from '../providers/profile.service';
import {Storage} from '@ionic/storage';
@NgModule({
  providers: [
    Storage,
    ProfileService,
    ...

However for any page that utilises the profileService, where we run the following
import {Storage} from '@ionic/Storage'
constructor(private af: AngularFire, public storage:Storage) {
   af.auth.subscribe(state => {
     if (!state) {
       return;
     }
     this.storage.set('userUID', state.uid);...

We are getting an error main.js:51716 ORIGINAL EXCEPTION: No provider for Storage!
Struggling to understand why ?

The issue was on our end. We were declaring the services in each page and not module wide. Fixed now, thanks @shawnlauzon. Closing issue.


Question

Does it crawl in memory DOM or I have to create my bot with Selenium (https://github.com/SeleniumHQ/selenium)?

AbotX has javascript rendering which would give you the final rendering of the page. Abot will download page source only. If you want to interact with the page and click on things (that are using javascript) then you would need an actual browser simulation like selenium.


Fix: fail to install Kruise rollout in customized namespace

Signed-off-by: StevenLeiZhang<EMAIL_ADDRESS>Fix: #36
Investigation:
The namespace could be specified by user, when he run  helm install with
--set installation.namespace=myroll
but serviceaccount and webhook's names are binded with NS. That makes pod run into error state
How to fix it?
Just set there names with "rollout.name", which is defined in helm template file.
How to test it:
run
 helm install  krollout ./kruise-rollout --set installation.namespace=myroll --set image.pullPolicy="IfNotPresent" --debug

to install this Chart, and all Pod can be up successfully.
Note on DCO:
If the DCO action in the integration test fails, one or more of your commits are not signed off. Please click on the Details link next to the DCO action for instructions on how to resolve this.
Checklist:

[ ] I have bumped the chart version according to versioning
[ ] I have updated the chart changelog with all the changes that come with this pull request according to changelog.
[ ] Any new values are backwards compatible and/or have sensible default.
[ ] I have signed off all my commits as required by DCO.

Changes are automatically published when merged to master. They are not published on branches.

@StevenLeiZhang tks, I think you can remove the values.yaml rollout.fullname parameter, and use explicit string for name.
for example:
apiVersion: v1
kind: ServiceAccount
metadata:
  name: kruise-rollout-controller-manager
  namespace: {{ .Values.installation.namespace }}

@StevenLeiZhang tks, I think you can remove the values.yaml rollout.fullname parameter, and use explicit string for name. for example:
apiVersion: v1
kind: ServiceAccount
metadata:
  name: kruise-rollout-controller-manager
  namespace: {{ .Values.installation.namespace }}

No，It does not work。
Please refer to  https://github.com/openkruise/rollouts/blob/08dd7878ff4786f9f500f6ba3d216fc8ff6cdbc7/pkg/webhook/util/controller/webhook_controller.go#L50
	mutatingWebhookConfigurationName   = "kruise-rollout-mutating-webhook-configuration"
	validatingWebhookConfigurationName = "kruise-rollout-validating-webhook-configuration"

but in. https://github.com/openkruise/charts/blob/201a65e191d2b50e2d281d0aadc46e73a22d87da/versions/kruise-rollout/0.3.0-rc/templates/webhookconfiguration.yaml#L6
It is named by NamaSpace.  That cause mismatch and Controller pod can not be up.

@StevenLeiZhang tks, I think you can remove the values.yaml rollout.fullname parameter, and use explicit string for name. for example:
apiVersion: v1
kind: ServiceAccount
metadata:
  name: kruise-rollout-controller-manager
  namespace: {{ .Values.installation.namespace }}

No，It does not work。 Please refer to
https://github.com/openkruise/rollouts/blob/08dd7878ff4786f9f500f6ba3d216fc8ff6cdbc7/pkg/webhook/util/controller/webhook_controller.go#L50
	mutatingWebhookConfigurationName   = "kruise-rollout-mutating-webhook-configuration"
	validatingWebhookConfigurationName = "kruise-rollout-validating-webhook-configuration"

but in.
https://github.com/openkruise/charts/blob/201a65e191d2b50e2d281d0aadc46e73a22d87da/versions/kruise-rollout/0.3.0-rc/templates/webhookconfiguration.yaml#L6
It is named by NamaSpace. That cause mismatch and Controller pod can not be up.

You are right, so all name with format {{ .Values.installation.namespace }}-xxxx can be convert to kruise-rollout-xxxx.

@StevenLeiZhang tks, I think you can remove the values.yaml rollout.fullname parameter, and use explicit string for name. for example:
apiVersion: v1
kind: ServiceAccount
metadata:
  name: kruise-rollout-controller-manager
  namespace: {{ .Values.installation.namespace }}

No，It does not work。 Please refer to
https://github.com/openkruise/rollouts/blob/08dd7878ff4786f9f500f6ba3d216fc8ff6cdbc7/pkg/webhook/util/controller/webhook_controller.go#L50
	mutatingWebhookConfigurationName   = "kruise-rollout-mutating-webhook-configuration"
	validatingWebhookConfigurationName = "kruise-rollout-validating-webhook-configuration"

but in.
https://github.com/openkruise/charts/blob/201a65e191d2b50e2d281d0aadc46e73a22d87da/versions/kruise-rollout/0.3.0-rc/templates/webhookconfiguration.yaml#L6
It is named by NamaSpace. That cause mismatch and Controller pod can not be up.

@StevenLeiZhang You are right, so all name with format {{ .Values.installation.namespace }}-xxxx must be convert to kruise-rollout-xxxx.
for exmaple: change {{ .Values.installation.namespace }}-mutating-webhook-configuration to kruise-rollout-mutating-webhook-configuration.

Yes, so my fix is
{{ template "rollout.name" . }}-mutating-webhook-configuration

because {{ template "rollout.name" . }} is exactly "kruise-rollout", which is defined in Chart template.

In case user wants to customize it, he just need change name in Chart template.
These places feel that they should not be modified, as if the user should not have a modified scene. In addition, I have verified your pr locally, but it cannot work, for example:

apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRoleBinding
metadata:
  name: {{ .Values.installation.namespace }}-manager-rolebinding
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: ClusterRole
  name: {{ .Values.installation.namespace }}-manager-role
subjects:
  - kind: ServiceAccount
    name: {{ .Values.installation.namespace }}-controller-manager
    namespace: {{ .Values.installation.namespace }}

In case user wants to customize it, he just need change name in Chart template.

These places feel that they should not be modified, as if the user should not have a modified scene. In addition, I have verified your pr locally, but it cannot work, for example:
apiVersion: rbac.authorization.k8s.io/v1
kind: ClusterRoleBinding
metadata:
  name: {{ .Values.installation.namespace }}-manager-rolebinding
roleRef:
  apiGroup: rbac.authorization.k8s.io
  kind: ClusterRole
  name: {{ .Values.installation.namespace }}-manager-role
subjects:
  - kind: ServiceAccount
    name: {{ .Values.installation.namespace }}-controller-manager
    namespace: {{ .Values.installation.namespace }}

updated my PR, and followed https://github.com/openkruise/rollouts/blob/master/docs/tutorials/basic_usage.md to test it.

/lgtm

/approve


NullPointerException when using system.dataset.fromExcel() on dataset with an empty cell

I'm getting a NullPointerException when trying to import excel data via system.dataset.fromExcel() that has empty cells like the following:

Excel sheet attached.
data1.xlsx
Script I'm using:
excelFilePath = "C:\Users\jayen\Documents\data1.xlsx"
dataset = system.dataset.fromExcel(input=excelFilePath, headerRow=0, sheetNumber=0)
system.util.getLogger("ExcelImportScript").info("Starting Excel import..." + str(dataset))
system.dataset.print(dataset)

Thanks for the work put into this. We found your module looking for efficiencies in our current work flow working with a customer with a lot of excel sheets.

This should be fixed as of #48, but isn't yet available in a public release. If you'd like, you can check out the repo, build the unsigned module locally (or self-sign it) and test out the fix. I'll probably release a new public version in the next week or two.

Thanks!

The fix worked thank you! I'll check back for the public release soon.

I went ahead and released 0.8.0:
https://github.com/IgnitionModuleDevelopmentCommunity/ignition-extensions/releases/tag/0.8.0


Add a batch script that will build all the projects and run all the t…

…ests.
Will be invoked from Jenkins.

:+1:


Fixes a bug with DDPG

It's the change I made regarding issue https://github.com/reinforceio/tensorforce/issues/513.

Thanks!


Disable Proof of burn chooser

What's wrong

Whales (in Burn auction) or operator (in POB) can lock deposits.
We do not recommend using POB chooser in production. Also, in the current implementation operator can buy out 100 blocks and go offline. We recommend adding force bids with higher amounts to replace malicious operator or implement a force withdrawal procedure.

How can we fix it
Remove it completely.

Fixed in #499


Number of concurrent streams are not increasing even after setting GRPC_ARG_MAX_CONCURRENT_STREAMS

What version of gRPC and what language are you using?
1.51.1
What operating system (Linux, Windows,...) and version?
Linux, centos-8
What runtime / compiler are you using (e.g. python version or version of gcc)
9.4.0
What did you do?
Please provide either 1) A unit test for reproducing the bug or 2) Specific steps for us to follow to reproduce the bug. If there’s not enough information to debug the problem, gRPC team may close the issue at their discretion. You’re welcome to re-open the issue once you have a reproduction.
Either with single client or multiple clients, the maximum number of streams that can server accept is 15.
Even when I tried to set the below arguments for client even then the max number of streams were 15.
grpc::ChannelArguments channel_args;
  channel_args.SetInt(GRPC_ARG_MAX_CONCURRENT_STREAMS, 30);
  std::shared_ptr<::grpc::Channel> channel = grpc::CreateCustomChannel(server_address,grpc::InsecureChannelCredentials(), channel_args);

I have even tried to set on server as well but of no use.
builder.AddChannelArgument(GRPC_ARG_MAX_CONCURRENT_STREAMS,200);

What did you expect to see?
to enable more streams between server and either single or multiple clients
What did you see instead?
its always 15 streams either with one client or multiple clients and when i enable more streams the processes get stuck
Make sure you include information that can help us debug (full error message, exception listing, stack trace, logs).
See TROUBLESHOOTING.md for how to diagnose problems better.
Anything else we should know about your project / environment?

MAX_CONCURRENT_STREAMS defaults to uint32 max<PHONE_NUMBER>), so you should not need to set it.
Can you show your client code and server code?  If your server is using the C++ sync API, maybe you are running into some other limit, like a thread pool size.

@markdroth i am using bi-directional reactor apis on both side.
I have attached the proto file, server transport header and one of the client's header file.
Please let me know if you need any other information.
transport_server.h.txt
inference_server.proto.txt
client_server_configuration.h.txt

@markdroth  when i tried to build a simple example with single rpc to reproduce the error, it worked fine. so i will try out with multiple rpcs and multiple streams and will reach out if i see any issue.
thanks for help.

Note that each RPC is a single stream, so if the problem you're trying to reproduce is a limit on the number of simultaneous streams, I don't think you'll be able to reproduce that with a single RPC.

@markdroth yeah each rpc is a single stream ans we have a use case where each rpc is invoked multiple times in parallel. so earlier when i mentioned i was trying with single rpc, i was invoking the same rpc multiple times in parallel

Ah, okay -- I think I understand what you mean, but I think you're misusing terminology here in a way that confused me.  The term "RPC" is generally used to indicate an actual call that you are making, not for a specific RPC method definition.  So I think what you meant to say was that you are making multiple RPCs in parallel to the same method.
Anyway, please let us know if you're able to come up with a reproduction case that we can try out.


Updated connectivity package, added toast docs

Fixes #27 #28

Please remove FlatButton and replace it with alternative

@all-contributors please add @pranathireddyk  for code

Thank you @all-contributors bot


fixsort node should materialized required slots for itself

Proposed changes
pick from master https://github.com/apache/doris/pull/27605

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Array parameters and extending the library

Morning!
I've been working towards moving all of my DB access in F# to using your libraries, especially in the case of replacing EF Core. Kudos on the great work; the API is really clean and the docs are great!
I don't know if I'm looking in the wrong place or if it's not available, but I'm trying to add a string array parameter which seems to be supported by the underlying Npgsql library, but I can't seem to find the DU representation in the docs or source.
First of all, If I've totally overlooked something, please let me know and I'll be happy to contribute docs or whatever is needed. But if this isn't available, is it worth extending the DU to allow this scenario, but also consider a way for extensibility outside of source changes?
For example, a DU case that is simply NpgsqlOther of NpgsqlParameter (or unit -> NpgsqlParameter) , or possibly a function that is called with the overall Npgsql connection (Or whatever other mutable object it uses for state tracking) prior to execution, to perform any additional transformations outside of the scope of the API?

Hello James
Very glad to see that the library will also be used in another prod code :)
Do you mean Arrays type?
Actually all the core wrapper logic is in Vp.FSharp.Sql and Vp.FSharp.Sql.PostgreSql contains only the Command, Transaction and Types. So yes I think it will be easy to add this in Types.fs but I am not sure if it's possible to extend without forking.
Wdyt @natalie-perret-1986?
I'll try make PR this week.

Thanks for feedback.
I totally understand - As soon as you implement it, it gets relied upon and becomes a lot harder to change!
There's no rush, I can totally understand being swamped.
I have forked it locally for now and I've added the following temporary workarounds for my usage...
Extended PostgreSqlDbValue DU:
    | IntegerArray of int32 array
    | UuidArray of Guid array
    | NpgsqlOther of NpgsqlParameter

Extended DbValueToParameter function (I also had to make the local let parameter binding mutable for the NpgsqlOther to work):
    | IntegerArray value ->
        parameter.Value <- value
        parameter.NpgsqlDbType <- NpgsqlDbType.Array ||| NpgsqlDbType.Integer
    | UuidArray value ->
        parameter.Value <- value
        parameter.NpgsqlDbType <- NpgsqlDbType.Array ||| NpgsqlDbType.Uuid
    | NpgsqlOther p ->
        parameter <- p

Obviously I've only tackled a couple of array types that I needed as well as adding a back-door. I'm not sure what greater ramifications this could have, but this seems to be working just fine for me at the moment.
Happy to chat about this here or on Slack, and I'm happy to contribute changes &  tests where required.

WIP - sticky notes
Passively tinkering: https://www.npgsql.org/efcore/mapping/array.html

For example, a DU case that is simply NpgsqlOther of NpgsqlParameter (or unit -> NpgsqlParameter)

@absolutejam
So I gave it some thought, and this does look tempting.
We might need to agree upon a name but other than that, should be fine.
It would be a great addition in terms of extensibility for the library, when it comes to using parameters that aren't yet fully baked-in. This also could be done for other Vp.FSharp.Sql DB providers
I like NpgsqlParameter over NpgsqlOther, maybe Raw could just do as well.
I'm still giving it a few more days to see if there is any sort of caveats that we may have missed out.
Thanks again for opening this issue, really appreciated!

and there she is, back 👩🚀
It's nice to take some time to think about a design, and I might have found something that in terms of consistency isn't all grand-like.
Assuming new DU cases like below, in the case of a broad generic system to support whatever NpgsqlParameter:
    | IntegerArray of int32 array
    | UuidArray of Guid array
    | NpgsqlOther of NpgsqlParameter

My issue is that if you pass a NpgsqlParameter, this class actually carries a parameter name of its own which may or may not take precedence over the "other" parameter name already carried by the tuple:
https://github.com/veepee-oss/Vp.FSharp.Sql.PostgreSql/blob/e6f795723e5fa70a218815a7cfd15d0800b0c6d6/Vp.FSharp.Sql.PostgreSql/Types.fs#L104-L109
Which makes things not only inconsistent but also counter-intuitive and to some extent might mislead the developers using the library.
So in order to make things slightly more consistent, I'd the suggest instead to have the DU case as follows:

//... whatever goes here
    | NpgsqlOther of (NpgsqlDbType * obj)

//... some other things go here
    | NpgsqlOther (dbType, value) ->
        parameter.Value <- value
        parameter.NpgsqlDbType <- dbType

@absolutejam @akhansari wdyt?
Btw, I'm still not too convinced about the name, i.e. NpgsqlOther (despite the fact that I cannot come up with anything substantially better).

Makes sense to me!
I left it fully open initially in case there was some super-advanced stuff needed to be done with the raw NpgsqlParameter (I have no idea if there is; I never delve this deep :joy:). But your proposal makes it nice and apparent and more searchable too.
And from my super in-depth testing, I think the boxing is automatic in this case:
type Foo =
  | CaseA of string
  | CaseB of string * obj

> let x [| CaseB ("x", 10); CaseB ("y", "apple") |];;
val x : Foo [] = [|B ("x", 10); B ("y", "apple")|]

> match x.[0] with
- | B (first, second) -> second
- | _ -> failwith "lol";;
val it : obj = 10

I'll leave the naming to you... that's the hardest part :smiling_imp:
Did you want to bother with explicit array parameters, or just leave these to the open-ended case?

@absolutejam 🙋♀ long time no see, you might be interested in https://github.com/veepee-oss/Vp.FSharp.Sql.PostgreSql/pull/16/files
(I know, it's probably wayyyy long overdue)
With the impl available in the PR, this is what it would look like:
// Valid stuff in regard to PG / .NET type consistency, but no safeguard when it comes to the initial intent
PostgreSqlDbValue.Array         (NpgsqlDbType.Integer, [| 123;   456   |]) |> ignore // Valid
PostgreSqlDbValue.Array         (NpgsqlDbType.Text,    [| "abc"; "def" |]) |> ignore // Valid

// Valid stuff in regard to PG / .NET type consistency, safeguard when it comes to the initial intent via generics
PostgreSqlDbValue.Array<int32>  (NpgsqlDbType.Integer, [| 123;   456   |]) |> ignore // Valid
PostgreSqlDbValue.Array<string> (NpgsqlDbType.Text,    [| "abc"; "def" |]) |> ignore // Valid

// Invalid stuff in regard to PG / .NET type consistency
PostgreSqlDbValue.Array         (NpgsqlDbType.Text,    [| 123;   456   |]) |> ignore // Invalid (runtime)
PostgreSqlDbValue.Array         (NpgsqlDbType.Integer, [| "abc"; "def" |]) |> ignore // Invalid (runtime)
PostgreSqlDbValue.Array<int32>  (NpgsqlDbType.Text,    [| 123;   456   |]) |> ignore // Invalid (runtime)
PostgreSqlDbValue.Array<string> (NpgsqlDbType.Integer, [| "abc"; "def" |]) |> ignore // Invalid (runtime)

// Invalid stuff in regard to PG / .NET type consistency, safeguard when it comes to the initial intent via generics
PostgreSqlDbValue.Array<string> (NpgsqlDbType.Text,    [| 123;   456   |]) |> ignore // Invalid (compilation)
PostgreSqlDbValue.Array<int32>  (NpgsqlDbType.Integer, [| "abc"; "def" |]) |> ignore // Invalid (compilation)

Solved in https://github.com/veepee-oss/Vp.FSharp.Sql.PostgreSql/releases/tag/v1.0.31


dumb http transport does not support --depth

Hi,
I'm not sure where to search on, but I did find this: https://github.com/FiloSottile/gvt/blob/b309889f398bc664c99850d18a2f19448bc86a2f/gbvendor/repo_test.go#L75 so I'm curious about the behaviour of this particular package.
 Thanks, you found a bug. To make fetching faster we do shallow clones. They are not supported by all git servers, but we fail to retry without.

Any progress on this?

The same issue, how to resolve or is there is a workaround ?

As a workaround, you can comment out this line and recompile gvt.

Thank you @dcelasun


Correct way to add GPG key

using /etc/apt/keyrings is deprecated, see https://manpages.debian.org/testing/apt/apt-key.8.en.html
Description of change

Updated the documentation on how to add gpg key and install on debian/ubuntu distros.
Using /etc/apt/keyrings is deprecated, see https://manpages.debian.org/testing/apt/apt-key.8.en.html
Checklist

[X] documentation is changed or added (if applicable)
[X] contribution guidelines followed here

@ecspresso Are you able to sign the Eclipse CLA?

@ecspresso Are you able to sign the Eclipse CLA?

How do I check this?
I was able to add the key, update and install without any error messages.

Eclipse Foundation Contributor's Agreement (ECA), details here: https://adoptium.net/docs/eca-sign-off/

@ecspresso :wave:  Any news from you ? Do you have created an account for the Eclipse Foundation Contributor ?

Hello again, I have created an account and signed the Eclipse Contributor Agreement.
Most the email used for my Eclipse account as my public email on Github?


Support data analysis workflow

What this PR does / why we need it:
Support data analysis workflow
Special notes for your reviewer:
Release note:

@sfzeng Can you discuss about this in next Friday morning's OpenSDS Technical Meeting?  Thanks.

@xing-yang Sure.

Add release note at the top of the PR.

Please add a release note.


Update dependencies

Update dependencies for Scala 2.10.  Everything compiled cleanly, just changes to build.  Not sure what to do with the sbt-package-dist plugin, so I commented it out.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Brian Topping seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


prevent the main process from exceptional exit with catching the ption in listeners and keep going

it's a complementary PR for #1221 , the idea is, any exception in reporter (or, listener) should not make the main process exceptional exit.
@cbeust , @juherr , @krmahadevan

LGTM :+1: Ping @cbeust


fix: add missing prod Terragrunt modules

Summary
Add the missing backup_plan and conformance_pack modules to the production Terragrunt configuration.
Also fixes the alarms module Terraform source reference.
Related

#298

Really not a problem - this is exactly the type of thing I do to myself all the time 😄


Update cedar dependency to 4.1.0 and add new utility methods

Update cedar dependency to 4.1.0 this causes breaking changes in input schema so also upgrading cedarpy version to 1.0.0 as per semantic versioning guidelines
Add new methods to convert cedar policies to/from cedar natiive and json representation

Thank you David! We really appreciate the PR and will take a look this week.

David - thank you for putting all this together.  I hadn't anticipated a change to Cedar's schema.  So I need to learn more about that change and think about how to handle it.  Thank you for providing references.
Just an fyi... I will try to take a look at this soon and provide some concrete feedback and get the change on its way to merging.
I'm headed to re:Invent.  I'd love to connect if you'll be there.

Yeah unfortunately not going to re:Invent, no worries not urgent

Overall, these changes look great to me. I am going to try and mirror Cedar's conventions around branch naming.
So will merge this into a dev/4.1.x branch for some tweaking, then main, then eventually a release/4.1.x for maintenance.
Already branched main into release/2.2.x.

Thank you David!


App crash when click on the Update button on dialog

android.content.ActivityNotFoundException: No Activity found to handle Intent { act=android.intent.action.VIEW dat=market://details?id=xxxx
and also how to hide info button on the dialog.
here is my code:-
in the Main Activity onCreate:-
MAHUpdaterController.init(this, GlobalDataACT.SERV_UPDATE_URL);

Ok. @hpng6 . Thanks for this report.

I will investigate that bug and inform you
There is not any function to hide that button now. I will update lib and add new function for it soon.

Bug has fixed in source files.
Binaries will be updated in jCenter soon.
closes #7


Fix switch state event conditions required params + make default para…

…m not required
Signed-off-by: Tihomir Surdilovic<EMAIL_ADDRESS>Many thanks for submitting your Pull Request :heart:!
Please specify parts this PR updates:

[ x] Specification
[ x] Schema
[ ] Examples
[ ] Extensions
[ ] Roadmap
[ ] Use Cases
[ ] Community
[ ] TCK
[ ] Other

What this PR does / why we need it:

Fixes the switch state event conditions schema (bad cut/paste)
Makes the "default" property of conditions not required
Special notes for reviewers:

Additional information (if needed):

i think there were no objections to move this forward. will go ahead and merge. please feel free to sent any update prs if there are any needed. ty


cannot assign "onreadystatechange" for XMLHttpRequest in Opera 12 in strict mode

<!DOCTYPE html>
<html>
<head>
  <script src="https://unpkg.com/zone.js/dist/zone.js"></script>
  <script>
  (function () {
    "use strict";
    var url = "data:text/plain,test";
    var xhr = new XMLHttpRequest();
    xhr.open("GET", url, true);
    xhr.onreadystatechange = function () {};
    xhr.send();
    alert("OK");
  })();
  </script>
</head>
<body>
</body>
</html>

It seems, canPatchViaPropertyDescriptor defines unconfigurable onreadystatechange property on XMLHttpRequest.prototype.

@Yaffle, can you provide a reproduce repo or plunker? I can't reproduce your issue in zone.js 0.7.4

@Yaffle, can you provide a reproduce repo or plunker? I can't reproduce your issue in zone.js 0.7.4

@JiaLiPassion , I have updated my initialcomment to include the code of full HTML file.

@JiaLiPassion , I have updated my initialcomment to include the code of full HTML file.

@Yaffle, thank you, the reason is in Opera 12, XMLHttpRequest.prototype.onreadystatechange is undefined by default, I'll make a PR to let it work.

@Yaffle, thank you, the reason is in Opera 12, XMLHttpRequest.prototype.onreadystatechange is undefined by default, I'll make a PR to let it work.

@mhevery Thank you!


solana deployed program

How can I call deployed contract with program and storage address ?

dup https://github.com/solana-labs/solana-solidity.js/issues/16


(1-c) Result page #4-BaharBazargan

Description
I have made the result page which includes table that received all information from the Json file and prepare table for filtering data which came from pre page 's data
Related to
(1-c) Result page #4
Checklist:

[x] My code follows the style guidelines of this project
[ ] I have carefully reviewed my own code
[ ] I have commented my code
[ ] I have updated any documentation

Could you double check that the linked in the Related to points to the correct issue?

Could you double check that the linked in the Related to points to the correct issue?

YES,I sort it out.


Can we run this Project at local?

How can I run this project at localhost to develop and debug etc...
I attempt to run "npm run server" but not able to see any widget UI

It is not possible to run this project locally. The main purpose of this extension is to act as a bridge between collection view and Thingworx mashups so it doesn't make sense to run it outside of a mashup.
If you are interested in working with the underlying BMCollectionView class, you can include bm-core-ui as a dependency in your project and import it in either the extension or local project, or both.


Migrate to Pulsar Proxy Extension

related to https://github.com/datastax/pulsar/pull/12

@cbornet PTAL


Already done for https://github.com/microsoft/ALAppExtensions/issues/6866.

Already done for https://github.com/microsoft/ALAppExtensions/issues/6866.
Originally posted by @StanPesotskiy in https://github.com/microsoft/ALAppExtensions/issues/7046#issuecomment-630023343

Issue #7046 is about Sell-To Contact No. and not about Bill-To Contact No. So could you please review issue 7046 again because we really need the requested publisher...
Thanx.

I have posted the answer in https://github.com/microsoft/ALAppExtensions/issues/7046


MJML API

Hey first of all incredible work on both the email signature and the email builder. I am wondering on a cross-client testing factor if you are considering using MJML framework allowing for a most browsers to be compatible with your code?
https://mjml.io/

Hi @mariovass
Thanks for your feedback. I used Litmus for testing, I think this is enough to render correctly in all major email clients


free registration image is blurry

We need a better quality image

I want to work on this one.

this should be closed, seems fixed.

was closed