3.2.x - Replacing other 3.2.x merge - please review

A PR had been created to merge 3.2.x line back to master.  The PR appeared to have lots of duplicate items.  I took the time to perform two separate passes through a rebase to see if there were truely any changes.  The original PR couldn't be merged to start with so I'm guessing over time 3.2.x was being cherry-picked from the master since it was really targetting 3.3.x rather than 3.2.x.  At any rate, based on my two passes, there were only a couple of minor changes.  I'm not sure these are even needed.  I only skipped one real change with language specific modifications for zh doc that I personally cannot undertand therefore not sure if it was actually valid.  Other than that only two items were valid.  So I have created this PR in replacement of the other but want either @harawata or @kazuki43zoo to verify the two changes they made on 3.2.x to determine validity for going to the master.  If this ends up being all junk, we should instead delete the 3.2.x branch.
For reference, the commit for zh doc that I skipped was this and it only applied to what I think is Chinese but I'm certainly no expert at all there.  Most of that commit is already on the master.
https://github.com/mybatis/mybatis-3/commit/0ae18b88fb9bf85dda0cfe605224677beafe4bf4

OK - of 3.2.x branch, commit listed as #376: Adjust the assertion of test case for 3.2.x line appaers to be invalid.  If I recall correctly, internally mybatis is now using javassist not cglib and looking at that change it certainly appears that it is mixed up.  So now just to address the ADD_MONTHS change from @harawata.  To @harawata if your change here appears to not be necessary go ahead and close this off.  I do think overall we probably could just delete 3.2.x branch and be ok.

I think this is about #399
Let me explain the background first.
We created 3.2.x branch so that we can release bug fix versions until 3.3.0 (which contains some backward incompatible changes) is released.
So, basically, we make all changes in the master and port only bug fixes to 3.2.x branch.
As we never port changes from 3.2.x to master, I think we can safely ignore #399 if that's what it tries to do.
Regarding the ADD_MONTHS change, I had to correct it in 3.2.x branch because its HSQLDB version was older than master.

@harawata , thank you for explain the background of branch management policy.
In #376, i was need changing the assertion of test case because default byte code operation library(JAVASSIST or CGLIB) is different between master(3.3 line) and 3.2.x(3.2 line).
In other words, above 3.2.x changes cannot merge to master.

So there we have it.  I'm closing this PR out.  Thanks everyone!

I updated the readme to markdown on 3.2.x branch and referenced this PR and #399

Thank you for your work @hazendaz !
I think #399 was a result of some mis-operation (we've received PRs like that from time to time).
I'm glad to have you in the team :-)

Thank you as well!  Glad to be in the fold.


Disable AVX (via cmake and #ifdef) for clang 3.6.x

Because clang 3.6.x miscompiles AVX code it should get disabled. The user should not have to reproduce the error.

See e.g. here: https://cdash.gsi.de/buildSummary.php?buildid=52213


Test for case-insensitivity and give hints

I got reminded in https://github.com/nodeschool/discussions/issues/700 that sometimes the problem occurs that people don't see the difference in their output. uppercase/lowercase etc. It might be good to check something like:
function normalise(string) {
   return string.replace(/\.|\!|\?|\s|/g, "").toLowerCase()
}

function isProbablyTypo(expectedAnswer, answer) {
   return normalise(expectedAnswer) === normalise(answer)
}

if (isProbablyTypo(expectedAnswer, answer)) {
   console.log("Looks like you did the right thing but maybe you had a typo in there. We are a little picky to make sure that you understood the question. Maybe you want to check for typos?")
}

Now that we have diff'ing in the latest javascripting, this shouldn't be much of an issue anymore. Going to close for now.


ADCM-6077 Filters implementation #1

Hints
Suggestions for available filters (not always are "pulled" by LSP, but mostly works)


Adding check for existance of rust binary

Hey, we got stuck on a small problem when we were trying to start the tenstorage locally. The Rust binary was not built, and there was no obvious message in the logs indicating that it did not exist. I believe it is a minimal enhancement that could be helpful for people who are starting the project for the first time.

LGTM. Thanks @wojtekw92


Note: Recompile with -Xlint:deprecation for details.

Launching lib/main.dart on SM G973F in debug mode...
Running Gradle task 'assembleDebug'...
Note: /Users/bholendrasingh/Library/Android/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_core-1.2.1/android/src/main/java/io/flutter/plugins/firebase/core/FlutterFirebaseCorePlugin.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: /Users/bholendrasingh/Library/Android/flutter/.pub-cache/hosted/pub.dartlang.org/cloud_firestore-2.2.1/android/src/main/java/io/flutter/plugins/firebase/firestore/FlutterFirebaseFirestorePlugin.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: /Users/bholendrasingh/Library/Android/flutter/.pub-cache/hosted/pub.dartlang.org/cloud_firestore-2.2.1/android/src/main/java/io/flutter/plugins/firebase/firestore/streamhandler/TransactionStreamHandler.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
Note: /Users/bholendrasingh/Library/Android/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_auth-1.3.0/android/src/main/java/io/flutter/plugins/firebase/auth/FlutterFirebaseAuthPlugin.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
/Users/bholendrasingh/Library/Android/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_storage-8.1.1/android/src/main/java/io/flutter/plugins/firebase/storage/FlutterFirebaseStoragePlugin.java:38: warning: [deprecation] Registrar in PluginRegistry has been deprecated
  public static void registerWith(PluginRegistry.Registrar registrar) {
                                                ^
1 warning
Note: /Users/bholendrasingh/Library/Android/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_keyboard_visibility-5.0.2/android/src/main/java/com/jrai/flutter_keyboard_visibility/FlutterKeyboardVisibilityPlugin.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: /Users/bholendrasingh/Library/Android/flutter/.pub-cache/hosted/pub.dartlang.org/google_maps_flutter-2.0.6/android/src/main/java/io/flutter/plugins/googlemaps/Convert.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: /Users/bholendrasingh/Library/Android/flutter/.pub-cache/hosted/pub.dartlang.org/google_maps_flutter-2.0.6/android/src/main/java/io/flutter/plugins/googlemaps/TileProviderController.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
Note: /Users/bholendrasingh/Library/Android/flutter/.pub-cache/hosted/pub.dartlang.org/google_sign_in-5.0.4/android/src/main/java/io/flutter/plugins/googlesignin/GoogleSignInPlugin.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: /Users/bholendrasingh/Library/Android/flutter/.pub-cache/hosted/pub.dartlang.org/libphonenumber-2.0.0/android/src/main/java/com/codeheadlabs/libphonenumber/LibphonenumberPlugin.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: /Users/bholendrasingh/Documents/Devsig Technologies/Devsig Projects/connect/android/app/src/main/java/com/devsig/connect/MainActivity.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.

@bholendraofficial
All logs are pointing to 3rd party plugins.
Can you try flutter clean and then flutter run to see if it helps ?

Already tried but same warning

@bholendraofficial
For below:

Note: /Users/bholendrasingh/Library/Android/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_core-1.2.1/android/src/main/java/io/flutter/plugins/firebase/core/FlutterFirebaseCorePlugin.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: /Users/bholendrasingh/Library/Android/flutter/.pub-cache/hosted/pub.dartlang.org/cloud_firestore-2.2.1/android/src/main/java/io/flutter/plugins/firebase/firestore/FlutterFirebaseFirestorePlugin.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: /Users/bholendrasingh/Library/Android/flutter/.pub-cache/hosted/pub.dartlang.org/cloud_firestore-2.2.1/android/src/main/java/io/flutter/plugins/firebase/firestore/streamhandler/TransactionStreamHandler.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
Note: /Users/bholendrasingh/Library/Android/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_auth-1.3.0/android/src/main/java/io/flutter/plugins/firebase/auth/FlutterFirebaseAuthPlugin.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
/Users/bholendrasingh/Library/Android/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_storage-8.1.1/android/src/main/java/io/flutter/plugins/firebase/storage/FlutterFirebaseStoragePlugin.java:38: warning: [deprecation] Registrar in PluginRegistry has been deprecated
  public static void registerWith(PluginRegistry.Registrar registrar) {

Open the issue in dedicated repo: https://github.com/FirebaseExtended/flutterfire/issues
For below:

Note: /Users/bholendrasingh/Library/Android/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_keyboard_visibility-5.0.2/android/src/main/java/com/jrai/flutter_keyboard_visibility/FlutterKeyboardVisibilityPlugin.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: /Users/bholendrasingh/Library/Android/flutter/.pub-cache/hosted/pub.dartlang.org/libphonenumber-2.0.0/android/src/main/java/com/codeheadlabs/libphonenumber/LibphonenumberPlugin.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.

Open issue in respective plugin repo.
For below:
Note: /Users/bholendrasingh/Library/Android/flutter/.pub-cache/hosted/pub.dartlang.org/google_maps_flutter-2.0.6/android/src/main/java/io/flutter/plugins/googlemaps/Convert.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: /Users/bholendrasingh/Library/Android/flutter/.pub-cache/hosted/pub.dartlang.org/google_maps_flutter-2.0.6/android/src/main/java/io/flutter/plugins/googlemaps/TileProviderController.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.

Try running with -Xlint:unchecked and see what details it provides.
Also provide your flutter doctor -v.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


Binance_US - bot takes time to start Avellaneda MM strategy: flooded with Calculating volatility… no trades tick

Using version 1.6 of HummingBot, I am not able to use binance_us as spot connector under avellaneda market making
Steps To Reproduce

Open the bot (v1.6) in Windows using WSL
Connect to Binance US using connect binance_us and add API keys.
Create a new avellaneda market making strategy. During creation binance_us is not available as spot connector

Screenshots
https://cdn.discordapp.com/attachments/563462222878212098/1006704273846960238/unknown.png
WARNING: Do NOT publish any exchange API keys or your wallet's private key. Whoever has access to them may steal your assets!

Upon checking, could replicate on master branch 1.6 (also dev 1.7.0)
binance_us showed for autocomplition, but returns you an error when you trying to input it for avellanedaMM (value is not a valid enumeration member)

Any updated on this Bug ?

development 1.10.0
connect binance_us API Keys successfully = OK
checked balance for binance_us successfully = OK
created/started AvellanedaMM strategy using binance_us = OK
bot stuck into Calculation of volatility (using AvellanedaMM)

issue seems similar to https://github.com/hummingbot/hummingbot/issues/5771
P.s. PureMM works using binance_us

@kkapuria3
upd:
Bot was able to start trading using AvellanedaMM after ~10 minutes of volatility calculation

Thank you so much. I will try to reproduce this, may please I know which branch has the fix ?

@kkapuria3 development 1.10.0 (latest development)


fixed

Title of the Pull Request

[x] Have you renamed the PR Title in a meaningful way?

Related Issue

Closes: #1253
Description
Fixed Misplaced navbar on the Contributors page and removed unnecessary code.
How Has This Been Tested? ⚙
Go to the Contributors page and check.
Screenshots 📷

Type of change
What sort of changes have you made:

[x] Bug fix (non-breaking change which fixes an issue)
[x] Code style update (formatting, local variables)

Checklist

[x] My code follows the code style of this project.
[x] I have followed the contribution guidelines
[x] I have performed a self-review of my own code.
[x] I have ensured my changes don't generate any new warnings or errors.
[x] I have updated the documentation (if necessary).
[x] I have resolved all merge conflicts.

@vishanurag please review and merge the PR . Please add a level 2 label.


Warnings unreadable if PowerShell or Command Prompt background is changed

Brief description of your issue
I prefer whitish/grayish color as background on cmd/pwsh, but setting so affects readability of warnings from winget. I have tried changing colors in PowerShell but none of that seemed to work. And both PowerShell and Command seem to have this issue giving me an impression that it might be winget itself.
PowerShell Output

if you look closely, the entire output is -
Multiple packages found matching input criteria. Please refine the input. (unreadable because yellow on gray)
Name                Id                  Source
Microsoft PowerToys XP89DCGQ3K6VLD      msstore
PowerToys (Preview) Microsoft.PowerToys winget
Another simiar case (Command Prompt Output) -

The unreadable text is another warning that says -
Failed in attempting to update the source: winget
Multiple packages found matching input criteria. Please refine the input.
Steps to reproduce
set the background color of cmd/powershell to gray/white
try to install a package that will throw warning for multiple input crteria
Expected behavior
readable warnings
Actual behavior
Unreadable warning due to colors not updating with background color
Environment
Windows Package Manager v1.2.10271
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.19042.1526
Package: Microsoft.DesktopAppInstaller v1.17.10271.0

I've tried to change the warning colors from profile file for powershell but none of them seem to affect the outcome

Related to: https://github.com/microsoft/winget-cli/issues/1920

Duplicate of #1920


Moved 'kid' field from payload to header of JWT

Per Grant, the 'kid' field should be in the header and not in the payload.  The JWTs created by 'brkt make-jwt' already put the 'kid' in the header.

Testing:

Ran unit tests.
Ran "make-user-data --jwt <data output by 'brkt make-jwt'>"

👍


/dev/pts/0 is owned by root user, which breaks emacsclient

[nixos@nixos:~]$ emacsclient -a '' foo.txt -t
/home/nixos/.nix-profile/bin/emacsclient: can't find socket; have you started the server?
/home/nixos/.nix-profile/bin/emacsclient: To start the server in Emacs, type "M-x server-start".

Warning: due to a long standing Gtk+ bug
https://gitlab.gnome.org/GNOME/gtk/issues/221
Emacs might crash when run in daemon mode and the X11 connection is unexpectedly lost.
Using an Emacs configured with --with-x-toolkit=lucid does not have this problem.
Starting Emacs daemon.
Emacs daemon should have started, trying to connect again
*ERROR*: Could not open file: /dev/pts/0

here's a stack overflow discussion of pseudoterminals https://serverfault.com/questions/116775/sudo-as-different-user-and-running-screen/116830
I notice that in an ubuntu wsl container I have that /dev/pts/0 is in fact owned by the non-root user

/dev/pts/0 is owned by root, because syschdemd.sh is started as root in order to enter the systemd namespace. It then drops its privileges and starts the user shell.
I have just tested this locally with
nix shell nixpkgs#emacs --command emacsclient -a '' foo.txt -t

and was not able to reproduce your issue. Can you confirm, that the problem still persists for you?

Closing this, because there has been no response. Feel free to reopen if your issue persists


add securityContext field at conatiner level

Right now the field allowPrivilegeEscalation is not under securityContext but under volumeMounts, so fixing it in this PR.

This change is

@jessfraz @islinwb Can one of you do a tech review for this? Thanks.

This looks good to me thanks for fixing this.
On Tue, Oct 3, 2017 at 16:32 Andrew Chen<EMAIL_ADDRESS>wrote:

@chenopis https://github.com/chenopis requested your review on:
kubernetes/kubernetes.github.io#5734
https://github.com/kubernetes/kubernetes.github.io/pull/5734 add
securityContext field at conatiner level.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/kubernetes.github.io/pull/5734#event-1276774585,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABYNbPqFMpwEpSQ2cCXwdZL8c9HioeXfks5sopnhgaJpZM4ProVb
.

--
Jessie Frazelle
4096R / D4C4 DD60 0D66 F65A 8EFC  511E 18F3 685C 0022 BFF3
pgp.mit.edu http://pgp.mit.edu/pks/lookup?op=get&search=0x18F3685C0022BFF3


Bug: ChangeStream Handling crashes with error converting after: proto:\ │ │ u00a0invalid type: primitive.DateTime

Bug description
Hi everyone,
when using this community plugin I directly encountered an issue.
When a incoming MongoDB ChangeStream event is received, the following error occurs:
UI ERROR LOG

node mongodb-s3:mongodb stopped with error: github.com/conduitio/conduit/pkg/pipeline.(*Service).runPipeline.func2 /app/pkg/pipeline/lifecycle.go:577 - source stream was stopped unexpectedly: github.com/conduitio/conduit/pkg/pipeline/stream.(*SourceNode).Run /app/pkg/pipeline/stream/source.go:136 - error reading from source: github.com/conduitio/conduit/pkg/pipeline/stream.(*SourceNode).Run.func1 /app/pkg/pipeline/stream/source.go:87 - read stream error: error converting payload: error converting after: proto: invalid type: primitive.DateTime: github.com/conduitio/conduit/pkg/plugin/standalone/v1.unwrapGRPCError /app/pkg/plugin/standalone/v1/client.go:165

CONDUIT ERROR LOG:

2023-09-13T18:35:59+00:00 ERR node stopped error="node mongodb-s3:mongodb stopped with error: source stream was stopped unexpectedly: error reading from source: read stream error: error converting payload: error converting after: proto:\u00a │
│ 0invalid type: primitive.DateTime" component=pipeline.Service node_id=mongodb-s3:mongodb stack=[{"file":"/app/pkg/pipeline/lifecycle.go","func":"github.com/conduitio/conduit/pkg/pipeline.(*Service).runPipeline.func2","line":577},{"file":"/ap │
│ p/pkg/pipeline/stream/source.go","func":"github.com/conduitio/conduit/pkg/pipeline/stream.(*SourceNode).Run","line":136},{"file":"/app/pkg/pipeline/stream/source.go","func":"github.com/conduitio/conduit/pkg/pipeline/stream.(*SourceNode).Run. │
│ func1","line":87},{"file":"/app/pkg/plugin/standalone/v1/client.go","func":"github.com/conduitio/conduit/pkg/plugin/standalone/v1.unwrapGRPCError","line":165}]                                                                                │
│ 2023-09-13T18:35:59+00:00 INF destination connector plugin successfully torn down component=connector.Destination connector_id=mongodb-s3-dlq                                                                                │
│ 2023-09-13T18:35:59+00:00 INF destination connector plugin successfully torn down component=connector.Destination connector_id=mongodb-s3:s3                                                                                │
│ 2023-09-13T18:35:59+00:00 ERR pipeline stopped error="node mongodb-s3:mongodb stopped with error: source stream was stopped unexpectedly: error reading from source: read stream error: error converting payload: error converting after: proto:\ │
│ u00a0invalid type: primitive.DateTime" component=pipeline.Service pipeline_id=mongodb-s3 stack=[{"file":"/app/pkg/pipeline/lifecycle.go","func":"github.com/conduitio/conduit/pkg/pipeline.(*Service).runPipeline.func2","line":577},{"file":"/ap │
│ p/pkg/pipeline/stream/source.go","func":"github.com/conduitio/conduit/pkg/pipeline/stream.(*SourceNode).Run","line":136},{"file":"/app/pkg/pipeline/stream/source.go","func":"github.com/conduitio/conduit/pkg/pipeline/stream.(*SourceNode).Run. │
│ func1","line":87},{"file":"/app/pkg/plugin/standalone/v1/client.go","func":"github.com/conduitio/conduit/pkg/plugin/standalone/v1.unwrapGRPCError","line":165}]                                                                                │
│ 2023-09-13T18:35:59+00:00 ERR node stopped error="node mongodb-s3:s3-acker stopped with error: context canceled" component=pipeline.Service node_id=mongodb-s3:s3-acker stack=[{"file":"/app/pkg/pipeline/lifecycle.go","func":"github.com/condui │
│ tio/conduit/pkg/pipeline.(*Service).runPipeline.func2","line":577}]

I guess this is related to the $date BSON object but cannot confirm it.
Below, you'll find a sample configuration and payload.
Thanks and best regards,
Janes
Steps to reproduce

Environment:

Kubernetes (EKS) cluster on latest version 1.27
Conduit v0.7.2 linux/amd6
MongoDB Connector (latest): v0.1.0
S3 Connector (latest): v0.5.0
MongoDB 6.0.9

With this simple pipeline configuration:

Click me
version: 1.0 # parser version, the only supported version for now is 1.0 [mandatory]

pipelines: # a map of pipelines IDs and their configurations.
  mongodb-s3: # pipeline ID, has to be unique.
    status: "stopped" # pipelines status at startup, either running or stopped. [mandatory]
    connectors: # a map of connectors IDs and their configurations.
      mongodb: # connector ID, has to be unique per pipeline.
        type: "source" # connector type, either "source" or "destination". [mandatory]
        plugin<EMAIL_ADDRESS># connector plugin. [mandatory]
        name: "mongodb" # connector name, if not specified, connector ID will be used as name. [optional]
        settings: # map of configurations keys and their values.
          uri: "mongodb+srv:/some-cluster.someid.mongodb.net/"
          db: "SAMPLE"
          collection: "sample"
          auth.username: "my-user"
          auth.password: "my-password"
          auth.db: "admin"
          auth.mechanism: "SCRAM-SHA-1"
          snapshot: false
      s3:
        type: "destination"
        plugin<EMAIL_ADDRESS>        name: "s3"
        settings:
          aws.bucket: "my-conduit-target-sample-bucket"
          aws.region: "eu-central-1"
          aws.accessKeyId: "myaccesskey"
          aws.secretAccessKey: "mysecretkey"
          format: "json"

With the following sample document

Click me
{
    "_id": "genericId",
    "metaInformation": {
        "lastUpdate": {
            "$date": {
                "$numberLong": "1694115709655"
            }
        },
        "versionNumber": {
            "$numberLong": "1"
        }
    },
    "orders": {
        "inprogress": [
            {
                "orderId": "123456",
                "saleDate": {
                    "$date": {
                        "$numberLong": "1624579200000"
                    }
                }
            }
        ]
    },
    "customerInfo": {
        "customerRegistrationDate": {
            "$date": {
                "$numberLong": "1589760000000"
            }
        },
        "customerLoginDate": {
            "$date": {
                "$numberLong": "1575849600000"
            }
        }
    },
    "dateInfo": {
        "endDate": {
            "$date": {
                "$numberLong": "1589155200000"
            }
        },
        "plannedEndDate": {
            "$date": {
                "$numberLong": "1588896000000"
            }
        }
    },
    "schemaVersion": {
        "$numberDouble": "0.0"
    }
}

Version
Conduit v0.7.2 / MongoDB Connector v0.1.0

@jabouchleih Thanks for using the connector and also reporting this bug!
It reminds me a little bit of another bug: https://github.com/ConduitIO/conduit/issues/866.

Hi @hariso,
without the data field, processing seems to work fine, so it is indeed related to timestamps/DateTime fields.
Unfortunately the current data structure requires timestamps.

@jabouchleih Thanks for the feedback! It's definitely the issue I posted above. In a nutshell, we found some solutions, but we aren't happy with those since they are either too complex or they imply checking every field of a record (to be able to find and parse a timestamp). Given your interest in the connector, I can suggest the bug to be triaged again.

Hi @hariso,
from my side, it would be OK if I could specify a custom converter and JSON path for DateTime fields that are unsupported by default (if that simplifies the process).

@jabouchleih We do have JavaScript processors and the source processors run after we get the data from a source connector and before being sent downstream. However, that's happening in the Conduit process itself, so we'd still have that problem.:/

@jabouchleih we've triaged this ticket and are going to put in a fix to the mongo connector.


Please publish the tip-of-tree, and update it in dart's DEPS

The test on MacOS is still failing on our buildbots, because the version with the fix has not been rolled into DEPS.  Please tag, upload to pub, and update in DEPS.

I just noticed the test is broken on Windows too.  Will the fix that was committed fix that too?
00:00 +52 -1: test\package_root_info_test.dart: packagePath with a file: scheme
Expected: 'C:\Users\CHROME~2\AppData\Local\Temp\package_resolver_test7fda2d5e-b709-11e6-b9ff-005056030208\foo'
Actual: 'C:\Users\chrome-bot\AppData\Local\Temp\package_resolver_test7fda2d5e-b709-11e6-b9ff-005056030208\foo'
Which: is different.
Expected: ... :\Users\CHROME~2\ ...
Actual: ... :\Users\chrome-bot ...
^
Differ at offset 11
package:test                            expect
test\package_root_info_test.dart 144:7  main...
===== asynchronous gap ===========================
dart:async                              _Completer.completeError
test\package_root_info_test.dart 146:6  main...
===== asynchronous gap ===========================
dart:async                              _asyncThenWrapperHelper
test\package_root_info_test.dart        main..

Generally we don't publish package updates that only fix tests, since they don't provide any end-user value.
Why are we running tests of GitHub packages in the SDK repo at all? It seems like there should only be one canonical place to check for the tests for a given chunk of code—possibly excluding pub, since that's actually exported by the SDK.

We need at least a tag, to import it in DEPS, since we import by tag, not
hash. So please tag it and update deps.
We test the unit tests of packages used by the sdk, in the sdk checkout, so
that we check a package in the context of the versions of its dependencies
that are checked out in the repo.  This is much harder, nearly impossible,
unless we do it in the checkout, and easy to do in the checkout.  I also
think package_resolver is in third_party/package _tested.
This is the way that we found the bug in package_resolver, so I think it
has been useful.  So we need to start using the version with the bug fix,
so we can catch new bugs.
On Dec 1, 2016 21:59, "Natalie Weizenbaum"<EMAIL_ADDRESS>wrote:

Generally we don't publish package updates that only fix tests, since they
don't provide any end-user value.
Why are we running tests of GitHub packages in the SDK repo at all? It
seems like there should only be one canonical place to check for the tests
for a given chunk of code—possibly excluding pub, since that's actually
exported by the SDK.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dart-lang/package_resolver/issues/3#issuecomment-264293196,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AErapwiaOZeINZ8cZkc-jbW43QAVsHOKks5rDzU_gaJpZM4LAKrx
.

I think this is long stale...


Add docs for transport SSL

Checklist

[x] Added an entry in CHANGES.txt for user facing changes
[x] Updated documentation & sql_features table for user facing changes
[x] Touched code is covered by tests
[x] CLA is signed
[x] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in CHANGES.txt
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

It would be nice to have an explicit hba example later on how to implement the case where there is probably a NAT node with also an ip on the same network (e.g: <IP_ADDRESS>/32) and we want to require SSL only for this address. i think this will be a common case.

Not sure if this will already be possible with the current transport/discovery mechanism. I think this might be something for later once we have the concept of a proxy node


Feature Request: A native solution to sync user settings

Rather than using an extension, I'd like a way to be able to login to vscode so that I can sync my user settings, extensions, code snippets, etc from machine to machine. Kind of like logging into Google Chrome

/duplicate of #2743 and it's already in the works.
In the meantime you can use https://marketplace.visualstudio.com/items?itemName=Shan.code-settings-sync


fix: prevent duplicated characters in typing animation.

Changed the <TypingAnimation /> component logic to avoid duplicated characters by updating it to use text.substring(..) instead of concatenating with prevState + text.charAt(i).
Context
Using small durations, e.g., <TypingAnimation duration={5} />, can duplicate characters. This does not always happen, but when switching tabs or when the application is in the background, it is more prone to happen.

It took me a bit to find a proper way to replicate it easily: recording timelines in Safari, or adding a compute-intensive component next to < TypingAnimation ... />, makes the issue more prone to appear unless you have a powerful machine.
In the second 17, it is visible that the typing animation duplicates characters. Why this happens, I'm not 100% sure.
Browser: Safari (Version 17.4)
Device: MacBook Air (M1)
https://github.com/magicuidesign/magicui/assets/11491779/92a9f8c9-c7a9-4b0a-ba1f-ba5e171bf986


[SPARK-29664][PYTHON][SQL][FOLLOW-UP] Add deprecation warnings for getItem instead

What changes were proposed in this pull request?
This PR proposes to use a different approach instead of breaking it per Micheal's rubric added at https://spark.apache.org/versioning-policy.html. It deprecates the behaviour for now. It will be gradually removed in the future releases.
After this change,
import warnings
warnings.simplefilter("always")
from pyspark.sql.functions import *
df = spark.range(2)
map_col = create_map(lit(0), lit(100), lit(1), lit(200))
df.withColumn("mapped", map_col.getItem(col('id'))).show()

/.../python/pyspark/sql/column.py:311: DeprecationWarning: A column as 'key' in getItem is
deprecated as of Spark 3.0, and will not be supported in the future release. Use `column[key]`
or `column.key` syntax instead.
  DeprecationWarning)
...

import warnings
warnings.simplefilter("always")
from pyspark.sql.functions import *
df = spark.range(2)
struct_col = struct(lit(0), lit(100), lit(1), lit(200))
df.withColumn("struct", struct_col.getField(lit("col1"))).show()

/.../spark/python/pyspark/sql/column.py:336: DeprecationWarning: A column as 'name'
in getField is deprecated as of Spark 3.0, and will not be supported in the future release. Use
`column[name]` or `column.name` syntax instead.
  DeprecationWarning)

Why are the changes needed?
To prevent the radical behaviour change after the amended versioning policy.
Does this PR introduce any user-facing change?
Yes, it will show the deprecated warning message.
How was this patch tested?
Manually tested.

cc @imback82 and @viirya can you take a look please?

Test build #121749 has started for PR 28327 at commit eb2d788.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/26427/
Test PASSed.

Merged build finished. Test PASSed.

Test build #121749 has finished for PR 28327 at commit eb2d788.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/121749/
Test PASSed.

Test build #121863 has started for PR 28327 at commit 3ec641d.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/26536/
Test PASSed.

Test build #121863 has finished for PR 28327 at commit 3ec641d.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/121863/
Test FAILed.

retest this please

Thanks @viirya and @imback82.

Test build #121868 has started for PR 28327 at commit 3ec641d.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/26540/
Test PASSed.

Test build #121868 has finished for PR 28327 at commit 3ec641d.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/121868/
Test FAILed.

retest this please

Test build #121872 has started for PR 28327 at commit 3ec641d.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/26543/
Test PASSed.

Test build #121872 has finished for PR 28327 at commit 3ec641d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/121872/
Test PASSed.

Merged to master and branch-3.0.


⚠ Bofrost.de has degraded performance

In c3cd594, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1154 ms

Resolved: Bofrost.de performance has improved in c6b92bb.


windows phone push notification while app is running

Expected Behaviour
Get only push notification, if the app runs in the background.
Actual Behaviour
It works perfectly on iOS and android, but on windows phone I get also a push notification while the app is running in front.
Steps to Reproduce
1.) Open App and let it successful register
2.) Send a event for a push notification
3.) Push notification will be shown while app is running
Platform and Version (eg. Android 5.0 or iOS 9.2.1)
windows phone 8.1 and also 10
(Android) What device vendor (e.g. Samsung, HTC, Sony...)
Microsoft Lumia 550
Cordova CLI version and cordova platform version
cordova --version  --> 6.4.0
cordova platform version android -->4.4.2

Plugin version
phonegap-plugin-push 1.9.0 "PushPlugin"

Sample Code that illustrates the problem
var push = PushNotification.init({
         "android": {
             "senderID": "XXXXXXX"
          },
         "ios": {
            senderID: "XXXXXXXX",
             alert: "true",
             badge: "true",
             sound: "true",
             gcmSandbox: "false"
         },
              "windows": {}
    });

No one?

firstly what type of push notification are you sending eg. toast/raw

@nates-dennis toast/raw

if it is a toast notification you need to implment the onActivated event and use launchArgs this should help
#1033 (comment)

I am facing same issue, when app is in foreground in IOS, event handler automatically calls.,

I have the same problem, on windows phone I get also a push notification while the app is running in front.
I saw here something interessting :
https://docs.microsoft.com/fr-fr/windows/uwp/controls-and-patterns/tiles-and-notifications-adaptive-interactive-toasts#a-namehandling-activation-foreground-and-backgroundagestion-de-lactivation-au-premier-plan-et-en-arrière-plan
Here a piece of code
<toast launch? duration? activationType? scenario? >
  <visual lang? baseUri? addImageQuery? >
    <binding template? lang? baseUri? addImageQuery? >
      <text lang? hint-maxLines? >content</text>
      <image src placement? alt? addImageQuery? hint-crop? />
      <group>
        <subgroup hint-weight? hint-textStacking? >
          <text />
          <image />
        </subgroup>
      </group>
    </binding>
  </visual>
  <audio src? loop? silent? />
</toast>

There is a field 'ActivationType' who take this values
activationType? = "foreground | background | protocol | system"
So I tried to put  backgroung but it doesn't work. I think this functionality is not implemented yet to this plugin.
If someone has other idea, it would be great.

Yes, i did many test.
But if i understand well the doc , the parameter launch let to launch the notification in background, but my problem is different, i have the toast in backgroung I just doesn't  want to get  the toast in foreground.

i understand what you mean now. i could not find a way of controlling that either so when the app is in the foreground unlike Android and IOS it seems the toast will always come i have not seen anything that says that can be changed.

Any solution for this problem available?


feat(config): always remove config

This allows that a stopped container can be started again. Until now, you need to remove and recreate it because you can't configure an already configured runner.

@tfenster could you dispatch the workflow as well? I don't have access rights


Update prefix defense

Add restrictions to the character length of address at 6-100 chars. The rationale for 6 is to ensure that users can put purely postal code if they have no time to key the full address.
Added restrictions to the name as well. It cannot go over 50 chars and below 2 chars.
Emails are restricted to 6-50 chars.
Testing of char limits should be done with ParserUtil. It is where all the exceptions are thrown and would be a better way to consolidate all exceptions.
Removed phone char limit validation in Phone to provide better error messages. Moved this to ParserUtil instead.
I will work on adding tests for these restrictions in the following milestone.

Codecov Report

Merging #179 (512667a) into master (6d61487) will decrease coverage by 0.35%.
The diff coverage is 33.33%.

@@             Coverage Diff              @@
##             master     #179      +/-   ##
============================================
- Coverage     67.37%   67.02%   -0.36%
  Complexity      688      688
============================================
  Files           122      122
  Lines          2293     2311      +18
  Branches        276      280       +4
============================================
+ Hits           1545     1549       +4
- Misses          626      636      +10
- Partials        122      126       +4

Impacted Files
Coverage Δ

.../main/java/seedu/address/model/person/Address.java
80.00% <ø> (ø)

...rc/main/java/seedu/address/model/person/Email.java
80.00% <ø> (ø)

src/main/java/seedu/address/model/person/Name.java
80.00% <ø> (ø)

...rc/main/java/seedu/address/model/person/Phone.java
80.00% <ø> (ø)

...c/main/java/seedu/address/model/person/Remark.java
66.66% <ø> (ø)

...in/java/seedu/address/logic/parser/ParserUtil.java
79.79% <33.33%> (-6.41%)
:arrow_down:

...u/address/logic/parser/FindOrderCommandParser.java
15.15% <0.00%> (-2.71%)
:arrow_down:

...c/main/java/seedu/address/model/order/Details.java
72.72% <0.00%> (-2.28%)
:arrow_down:

...in/java/seedu/address/model/order/DateChecker.java
75.00% <0.00%> (ø)

...seedu/address/logic/commands/EditOrderCommand.java
93.65% <0.00%> (ø)

Continue to review full report at Codecov.

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


Q: How to clear field input

Hi @kensoh
I'm ashamed to ask but how can I clear a (textarea) input?
For example a basic text field input or textarea?
Currently when using textarea:
type //textarea as Lorem ipsum
It adds to the existing text in the textarea.

Hi @thim81 this is a good question. At the moment I haven't implement that directly into TagUI. However, as TagUI uses CasperJS, you can use normal CasperJS code to do that.
js this.sendKeys(tx('//textarea'), '', {reset: true});

The js step in front is to explicitly treat as JavaScript code, otherwise for rapid prototyping the { and } by default will be auto-parsed into nested code blocks in CasperJS syntax. Can you tell more is this a regular use case for you and usually in what situations you clear the text fields? If that is commonly used maybe I can add something into the type step, for eg type //textarea as [clear].

Use case is typically, to modify existing forms or data
Examples:

modify an account
modify a project name
...

Thanks @thim81 it sounds like this will be better to be part of the type / enter step. Probably adding a [clear] keyword to signify emptying the field. Firefox/PhantomJS should be straightforward, Chrome need to add something more. Classifying as enhancement.

Hi @thim81 I've just committed to master with following comment.
For users wanting to use this before the packaged releases are out for the next release, you can download from master and overwrite your local copy (be careful if you already made local changes!)
Introduce [clear] keyword for step type / enter

Examples -
type element as [clear]
type email_element as<EMAIL_ADDRESS>type password_element as [clear]123456[enter]

Assumption is [clear] is always used at the start of the string to
type. Not a normal use case to type something then clear it. So TagUI
checks whether the string to type begins with [clear].

this doesn't seem to be in the docs? @kensoh . I ran into this today.

Yes @siowyisheng it's in the API section under type step. Can include into samples.
On this topic, there is a discrepancy between [clear] for web automation (clearing entire textbox) and [clear] when used in SikuliX eg keyboard [clear] or type abc.png as [clear]. In SikuliX, that clear only do a backspace.
You can review to see if there's value to convert that to [ctrl]a and then [del] for Windows and [cmd]a and [del] for mac to mean clearing the entire field. Not sure the combination for Linux.

It wasn't straightforward and maybe to assume that the keystroke combination will clear all fields in different apps is too simplistic and not practical, so I didn't include that when the API for SikuliX was first created.


Added support to get all available queues and all dead letter queues

Coverage decreased (-1.73%) to 77.04% when pulling 1d71697c0c0c14373a279525e9e9feb8b736542f on esanchezm:get_all_queues_support into 2bf6ea03436ebbb06c0ded4e0d8d2de599a77500 on igalarzab:master.

Coverage increased (+0.01%) to 78.78% when pulling 6d501df5b2fad2814be64dcf3b690b6f75cfe3dd on esanchezm:get_all_queues_support into 2bf6ea03436ebbb06c0ded4e0d8d2de599a77500 on igalarzab:master.

Coverage increased (+0.01%) to 78.78% when pulling b59c7f6b4694dab8405f15db7db5c28db79ac8bd on esanchezm:get_all_queues_support into 2bf6ea03436ebbb06c0ded4e0d8d2de599a77500 on igalarzab:master.

Coverage increased (+0.05%) to 78.82% when pulling 50f130dd1806a68dcd3409f7bcb3676f4092a8dc on esanchezm:get_all_queues_support into 2bf6ea03436ebbb06c0ded4e0d8d2de599a77500 on igalarzab:master.

Coverage increased (+0.05%) to 78.82% when pulling b9f5b1b0481b751e264eeb8af6b7c3eaffafeb16 on esanchezm:get_all_queues_support into 2bf6ea03436ebbb06c0ded4e0d8d2de599a77500 on igalarzab:master.

thanks!


[microtvm][Zephyr] Add project overlay to overwrite device tree configs

This PR adds a mechanism to overwrite device tree configs in zephyr project. It also adds an overlay for nucleo_l4r5zi board to change processor frequency from 80MHz which is the default to 120MHz.
cc @alanmacd @areusch @gromero @guberti

@mehrdadh Nice findings, I would if you would like to upstream it to Zephyr also so other projects could pick this up?

@erwango thanks for catching this and @gromero for the follow up PR!


Bug? "More than one build script specified" when calling build.ps1 with a custom cake script and extra parameters

What You Are Seeing?
When I specify a custom cake script with extra parameters using the build.ps1 bootstrap file, I get the following error:
More than one build script specified.
Note: If I remove the extra parameter that I would like to pass to my script everything works
What is Expected?
Cake executes using the custom cake script file
What version of Cake are you using?
0.13.0
Are you running on a 32 or 64 bit system?
64
What environment are you running on?  Windows? Linux? Mac?
Windows
Are you running on a CI Server?  If so, which one?
None
How Did You Get This To Happen? (Steps to Reproduce)
.\build.ps1 -Script "3rdPartyPackages.cake" -Settings=3rdPartyPackages.settings.json
Output Log
.\build.ps1 -Script "3rdPartyPackages.cake" -Settings="3rdPartyPackages.settings.json"
Preparing to run build script (3rdPartyPackages.cake)...
Running build script...
& "E:\Projects\tools\Cake\Cake.exe" "3rdPartyPackages.cake" -target="Default" -configuration="Release" -verbosity="Verbose"    -Settings=3rdPartyPackages.settin
gs\.json
More than one build script specified.

Usage: Cake.exe [build-script] [-verbosity=value]
                [-showdescription] [-dryrun] [..]

Note sure if this is the most elegant way - however this seems to fix the issue.
Change the end of build.ps1 to look like this
[string] $ScriptArgsFormatted = "";
$ScriptArgs | % {
    $temp = $_ -Split '='
    "{0}=`"{1}`"" -f $temp[0],$temp[1]
} | % { $ScriptArgsFormatted += $_ }

# Start Cake
Write-Host "Running build script..."
Invoke-Expression "& `"$CAKE_EXE`" `"$Script`" -target=`"$Target`" -configuration=`"$Configuration`" -verbosity=`"$Verbosity`" $UseMono $UseDryRun $UseExperimental $ScriptArgsFormatted"
exit $LASTEXITCODE

I have seen this before, it seems to come from when the extra param value contains a '.'
> .\build.ps1 -Target "my-target" -Version="1.2.3"
this fails with the same output.  from the notes i have on my fix, i looked at the cake source and see that the extra arguments will support '-' or '--' syntax.
i was able to change my use-case to the following to get it to work without modifying the build script or the underlying binary.
> .\build.ps1 -Target "my-target" --Version="1.2.3"
this is with 0.15.2

Cool, I will try that - with luck it will fix both of my issues (I had the version issue too).

I have a PR to the cake.resources project that I think might help solve this, but it needs feedback - https://github.com/cake-build/resources/pull/12
More or less it would let you be consistent with the format of the parameters to be 100% powershell rather than mixing single dashes and equal signs depending on the parameter you are passing. It is all PS style.

@ravensorb does the suggestion in this similar stackoverflow post answer work for you?
http://stackoverflow.com/questions/40046752/how-do-i-pass-my-own-custom-arguments-to-build-ps1

I'm hitting this issue too, using the -ScriptArgs= technique is not ideal but works when inside PowerShell. However, when executing PowerShell from cmd (Sadly I'm using a Go.CD CI server) I get the following error:

build.ps1 : A positional parameter cannot be found that accepts argument '--Param=Value'

@patriksvensson can you comment on this one?  I think this is similar to an issue that you solved a while back.  Thanks!

Having the same issue.  Trying to run Cake from a GO CD task, but getting the error "Cake.exe : More than one build script specified.".  It doesn't seem to let me specify the Script location
.\nuget.PackageName.1.2.3\Tools\build-cake.ps1 -Script '.\nuget.PackageName.1.2.3\Tools\build.cake' -Target 'DeployRelease' -octopusReleaseVersion '1.0.456' -OctopusApiKey='API-???????' -GoCDUsername='user_name' -GoCDPassword='123456'

If GO CI does something with the command line, perhaps a simplified bootrapper could be one option where you just pass along arguments as they're entered.
Example from Cake.Kudu/build.ps1
& tools\Cake\Cake.exe "$CakePath" $args
exit $LASTEXITCODE

But things like API keys/usernames etc. could also be passed as environment, you can add variables to an environment by editing the configuration of the environment in GO CD. And access them using the built in EnvironmentVariable aliases.

Actually I think the fix is even simpler in my case, the default build.ps1 isn't reading the $Script variable as a parameter.  It just defaults:
[CmdletBinding()] Param(     [string]$Script = "build.cake",
Removed default:
[CmdletBinding()] Param(     [string]$Script,
It is now running the cake script that I pass in as a parameter

Having the same issue, but no dot in the values. Here is an example
.\build.ps1 -Target Information -Configuration Debug -IncludeCoverage OpenCover

This still appears to be broken...
It might also be a duplicate of https://github.com/cake-build/cake/issues/172
Here's the output of a fairly benign argument string being passed.  You can see the arguments to the powershell script, below "Running build script..." I print the values of the parsed arguments. Every single argument ends up using the default value except for the first, "environment".
  Attempting to swap the single quotes for double quotes results in "More than one build script specified."
Removing quotes for items that don't need them doesn't help.
There are no dots, there are no trailing backslashes.
[SCRIPT OUTPUT]
[Step 2/2] PowerShell arguments: -NoProfile, -NonInteractive, -ExecutionPolicy, ByPass, -File, E:\BuildAgent\work\464c071b51cb4b09\build.ps1, -ScriptArgs, "-environment='production' -octopusHost='http://octopus' -octopusApiKey='API-XXXXXX' -buildnumber='47' -branch='refs/heads/feature/build_system' -sha='bb10761154335223d2f4fb076ac2345b7c9963d7' -buildid='51944'"
[22:16:51]	[Step 2/2] Preparing to run build script...
[22:16:55]	[Step 2/2] Running build script...
[22:17:06]	[Step 2/2] target: CreateRelease
[22:17:06]	[Step 2/2] octopusHost:
[22:17:06]	[Step 2/2] octopusApiKey:
[22:17:06]	[Step 2/2] buildnumber: <IP_ADDRESS>
[22:17:06]	[Step 2/2] buildnumberFull: <IP_ADDRESS>+branch.dev_build_no_release.sha.dev_build_no_release
[22:17:06]	[Step 2/2] deployChannel: Default
[22:17:06]	[Step 2/2] deployEnvironment: Staging
[22:17:06]	[Step 2/2] buildid: 1
[22:17:06]	[Step 2/2] branch: dev_build_no_release
[22:17:06]	[Step 2/2] sha: dev_build_no_release
[22:17:06]	[Step 2/2] environment: 'production' -octopusHost='http://octopus' -octopusApiKey='API-XXXXXX' -buildnumber='47' -branch='refs/heads/feature/build_system' -sha='bb10761154335223d2f4fb076ac2345b7c9963d7' -buildid='51944'

This still appears to be broken...

Comma separate the ScriptArgs, e.g.
-ScriptArgs '--environment=Production','--buildNumber=47'

So it seems when running from CMD params need to be passed in this format: --target=Default --buildVersion=<IP_ADDRESS>7. Worked for me in TeamCity using PowerShell task. Thanks, @nukefusion !

Getting the same when trying to specify a target via build.sh:
> ./build.sh -target Clean
More than one build script specified.
For usage, use parameter --help
> ./build.sh --target Clean
More than one build script specified.
For usage, use parameter --help

And --help doesn't even mention the ability to specify a target 😕 :
> ./build.sh --help

Usage: Cake.exe [script] [--verbosity=value]
                [--showdescription] [--dryrun] [..]

Example: Cake.exe
Example: Cake.exe build.cake --verbosity=quiet
Example: Cake.exe build.cake --showdescription

Options:
    --verbosity=value    Specifies the amount of information to be displayed.
                         (Quiet, Minimal, Normal, Verbose, Diagnostic)
    --debug              Performs a debug.
    --showdescription    Shows description about tasks.
    --dryrun             Performs a dry run.
    --version            Displays version information.
    --help               Displays usage information.

Here is relevant --version output:
Version 0.28.1+Branch.main.Sha.1da1ea266471a2bb8034fc4a89017b5dcf455268
Running on .NET Core

Ugh, I just found #1690 which suggests the syntax I need is --target=Clean, which works fine. Seems like the main problem here is the confusing error message and the (lack of) docs.

I believe there is still an issue but it only happens with the task in the cake script has a dot in the name.
I have added Set-PSDebug -Trace 1 in the cake.ps1 script to check the steps:
So running
C:\Build\Tools\cake.ps1 -Script C:\Build\Cakes\App.cake -Target "App.Package it executes
DEBUG:    1+  >>>> & "C:\Build\Tools\tools\Cake\Cake.exe" C:\Build\Cakes\App.cake -target=App.Package

and then Cake.exe exit with More than one build script specified.
Removing the dot solves the issue
C:\Build\Tools\cake.ps1 -Script C:\Build\Cakes\App.cake -Target "Package" and execute as
DEBUG:    1+  >>>> & "C:\Build\Tools\tools\Cake\Cake.exe" C:\Build\Cakes\App.cake -target=Package

which works fine.
Not sure if this is related to this issue or if I need to open a new one. This used to work fine.
Now running Cake Version: 0.32.1

Ugh, I just found #1690 which suggests the syntax I need is --target=Clean, which works fine. Seems like the main problem here is the confusing error message and the (lack of) docs.

I confirm this worked for me .Net Core, Azure, powershell

This is still a problem. Both of these fail.
Cake.exe build.cake --Target Build --ScriptArgs --stagingFolderPath="C:\TempStaging" --finalFolderPath="C:\Final
Output"

Cake.exe build.cake --Target Build --ScriptArgs '--stagingFolderPath="C:\TempStaging" --finalFolderPath="C:\Final
Output"'

@dasiths and it you try...
Cake.exe build.cake --Target=Build  --stagingFolderPath="C:\TempStaging" --finalFolderPath="C:\Final
Output"

--stagingFolderPath="C:\TempStaging"

Calling Cake.exe directly works but not if you use the build.ps1 script
&.\cake-build.ps1 -Target $target " --finalFolderPath=`"$finalOutputDirectory`" --stagingFolderPath=`"$stagingDir`""

It's because if the incorrect way the powershell script passes arguments. (Ref: https://slai.github.io/posts/powershell-and-external-commands-done-right/)

@dasiths are you using the most recent version of the PowerShell bootstrapper?

@dasiths are you using the most recent version of the PowerShell bootstrapper?

I was not. I just tried the latest version which uses the following in the powershell script and it works. :)
Invoke-Expression "& $CAKE_EXE_INVOCATION $($cakeArguments -join " ")"

@dasiths glad to hear that you were able to get this working.

Tried all the variations on this page, none worked. This is what finally worked for us:
.\build.ps1 --% --ScriptArgs @('--targetTargetName','--exclusive','--buildUrl=%teamcity.serverUrl%/viewLog.html?buildId=%teamcity.build.id%&buildTypeId=%system.teamcity.buildType.id%')
The --% tells powershell to stop parsing text (ref: https://slai.github.io/posts/powershell-and-external-commands-done-right/) and the @('','',''...) notation is a command line string array (ref: https://www.varonis.com/blog/powershell-array/).


Docs: using pytest.mark.usefixtures in code snippets

Is your feature request related to a problem? Please describe.

Many of the pytest-oriented examples in https://pytest-pyfakefs.readthedocs.io/en/stable/usage.html use a fake_fs fixture in the test signature, but don't actually use it in the test body.
Here's a more specific example: https://pytest-pyfakefs.readthedocs.io/en/stable/usage.html#access-to-files-in-the-real-file-system
def test_using_fixture1(my_fs):  # Linters don't like this line
    with open(os.path.join(fixture_path, "fixture1.txt")) as f:
        contents = f.read()

Multiple linters detect this line as unused code:

pylint: invalid-name gets emitted on test functions or methods
flake8-unused-arguments: ARG001 gets emitted if test function, or ARG002 gets emitted if test method

Describe the solution you'd like

This can be ameliorated with using pytest.mark.usefixtures decorator:
@pytest.mark.usefixtures("my_fs")
def test_using_fixture1():
    with open(os.path.join(fixture_path, "fixture1.txt")) as f:
        contents = f.read()

Can we move code snippets in the pyfakefs docs to use pytest.mark.usefixtures?

Thanks, this would make sense, probably with a line explaining it.
If you want to make a PR, go ahead - otherwise this gets at the end of my personal backlog 😀


Nodejs cluster config

Implemented thanks to PM2. I didn't modify any codeline to run my app in a cluster XD


How to get opendal's benchmark?

I tried using cargo bench to get the results, but I didn't get a valid output.
Maybe I am missing something?

opendal will run benches based on env, just like we does for behavior test.

fs: OPENDAL_FS_TEST=on OPENDAL_FS_ROOT=/tmp cargo bench
s3: OPENDAL_S3_TEST=on OPENDAL_S3_ROOT="/" OPENDAL_S3_BUCKET=tests3 OPENDAL_S3_ENDPOINT=http://localhost:9000/ OPENDAL_S3_ACCESS_KEY_ID=minioadmin OPENDAL_S3_REGION=test OPENDAL_S3_SECRET_ACCESS_KEY=minioadmin cargo bench

I will fill docs for these.


Unit tests required for NovelRT::LoggingService.

As per #141 and the discussion that occured there, tests are probably needed for the logging service.

👋 I have the test codes ready. However this requires some work on the logger side to make the output readable by the test framework; currently the logger only writes to stdout and it does not seem possible to simply redirect the stdout and read from it.
Ruby suggested logging to files as well as console so the file can be read instead. However it would be nice if the logging to file can be configurable so it's disabled by default on sample apps.

Ticket is open again.

Ticket is open again.

I'm going to rework logging I think. Gonna close this for cleanup's sake.


Release 0.54

Refs #1201

Issues to reference in the commit message: #509, #1091, #1150, #1151, #1166, #1167, #1178, #1181, #1182, #1184, #1185, #1186, #1187, #1194, #1198

Issues to reference in the commit message: #509, #1091, #1150, #1151, #1166, #1167, #1178, #1181, #1182, #1184, #1185, #1186, #1187, #1194, #1198

Last thing to do: write up the annotated version of these release notes, assign it a URL on my blog and link to it from the release notes here so I can publish them simultaneously.

Last thing to do: write up the annotated version of these release notes, assign it a URL on my blog and link to it from the release notes here so I can publish them simultaneously.


Throw error for 32-bit archs

Closes #10002

WebAssembly is a 32-bit arch. Pyodide includes XGBoost and is having trouble updating to CPython 3.12 due to being unable to update XGBoost.
https://github.com/dmlc/xgboost/pull/8964 looks like it was trying to fix 32-bit errors.


Initial working app for first release

takes high level and low level configs as inputs to trigger required analysis
handles everything required (I think) for v1 other than sample sheet validation

Hello @jethror1! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file resources/home/dnanexus/run_workflows.py:

Line 619:80: E501 line too long (81 > 79 characters)
Line 631:80: E501 line too long (80 > 79 characters)
Line 679:80: E501 line too long (81 > 79 characters)


Some issues and threads

Issue link

#351
#350
#349
#298
#291
#329
Auto-close the issue?

Closes #351
Closes #350
Closes #349
Closes #298
Closes #291
Types of changes

Technical debt (a code change that doesn't fix a bug or add a feature but makes something clearer for devs)
Notes

Disabling CI node_modules caching to solve failure after truffle-hdwallet-provider upgrade. (Refs: https://circleci.com/gh/tasitlabs/TasitSDK/593)

I think it's ready to merge.


Does not build

I downloaded this repo and I can't compile it
It seems there are files missing
For example
Server.java
import io.luna.game.model.item.Shop;
But there is no such file
Same for
Bank.java
import io.luna.game.model.inter.StaticInventoryInterface;

same here

Yeah @lare96 didn't bother using a feature branch so we'll just have to wait it out.

This will be resolved shortly, sorry for the inconvenience guys!

Alright, im glad you didn't disappear :P very excited to see whats happening


Use Arrays.toString() for arrays

Current coverage is 99.71% (diff: 98.68%)

Merging #9 into master will decrease coverage by 0.28%

@@           master         #9   diff @@
========================================
  Files          10         10
  Lines         558        692   +134
  Methods         0          0
  Messages        0          0
  Branches       46         52     +6
========================================
+ Hits          558        690   +132
- Misses          0          1     +1
- Partials        0          1     +1

Powered by Codecov. Last update a4f766b...1d2bdfc


Cant get to Container Console in latest alpha5 after Updated from latest stable

Using Ubuntu Server in Oracle Cloud , Headscale in Docker [headscale/headscale:0.23.0-alpha5]
After updated to v 0.23.0-alpha5 from 0.22.3 cant get to Console from portainer also from SSH session of server :
docker exec -it headscale /bin/bash
docker exec -it headscale /bin/sh
OCI runtime exec failed: exec failed: unable to start container process: exec: "/bin/bash": stat /bin/bash: no such file or directory: unknown
OCI runtime exec failed: exec failed: unable to start container process: exec: "/bin/sh": stat /bin/sh: no such file or directory: unknown
docker exec headscale headscale nodes list Working Fine

@masterwishx This is not a bug and it is intend to work this way. We are using distroless images, this mean there is nothing inside the container that is not needed.
You can read more about distroless images here
If you have any specific question just let me know.
The debug image will container a shell: v0.23.0-alpha5-debug

This is not a bug and it is intend to work this way. We are using distroless images, this mean there is nothing inside the container that is not needed.

Thanks, got it

I was asking myself how I can answer yes/no questions of a command like:
docker exec headscale headscale nodes backfillips


added cask for vrep by coppelia robotics

I'm not really sure what went wrong here. Could someone advise me?

@ahundt Seems like you figured it out. Also, it looks like the name should be V-REP based on the homepage. When you are done making changes, please try and squash your commits. Thanks!

Thank you for the contribution. It needed some fixes, so they were made in 06b44ecfc3ba44f4e60e9de03a714b390f26cb8e. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.

cool, thanks! sorry I've been out of town the last few days. I need to figure out how to do what you did with squash commits + modify + credit.


Named export 'IStorageProvider' not found

Describe the bug
Hi,
after upgrading @unleash/proxy-client-react from 3.6.0 to 4.0.1 we have this error at build time:
SyntaxError: Named export 'IStorageProvider' not found. The requested module 'unleash-proxy-client' is a CommonJS module, which may not support all module.exports as named exports.

Could it be related to the dist having a missing piece?
Cheers,
Steps to reproduce the bug
No response
Expected behavior
To import import { useUnleashContext, FlagProvider } from '@unleash/proxy-client-react'; without error.
Logs, error output, etc.
import { IStorageProvider as Y, InMemoryStorageProvider as j, LocalStorageProvider as D, UnleashClient as G } from "unleash-proxy-client";
         ^^^^^^^^^^^^^^^^
SyntaxError: Named export 'IStorageProvider' not found. The requested module 'unleash-proxy-client' is a CommonJS module, which may not support all module.exports as named exports.
CommonJS modules can always be imported via the default export, for example using:

import pkg from 'unleash-proxy-client';
const { IStorageProvider: Y, InMemoryStorageProvider: j, LocalStorageProvider: D, UnleashClient: G } = pkg;

    at ModuleJob._instantiate (node:internal/modules/esm/module_job:128:21)
    at async ModuleJob.run (node:internal/modules/esm/module_job:194:5)
    at async Promise.all (index 0)
    at async ESMLoader.import (node:internal/modules/esm/loader:385:24)
    at async importModuleDynamicallyWrapper (node:internal/vm/module:437:15) {

Screenshots
No response
Additional context
No response
Unleash version
4.0.1
Subscription type
Pro
Hosting type
Hosted by Unleash
SDK information (language and version)
typescript, react

I also encountered the same problem.

@AdrienFromToulouse @hqwlkj can you please check<EMAIL_ADDRESS>

not sure to understand sorry, to quote myself:

after upgrading @unleash/proxy-client-react from 3.6.0 to 4.0.1 we have this error at build time:

So trying v3.0.0-beta.1 might not be specially usefull.
We are using NextJs latest version if it can help you reproduce the issue.

I'm trying to reproduce it. What's the version of Node this issue occurs on? Is it possible to reproduce it in Codesandbox or similar tool?

Thanks! I managed to reproduce it on my end

Thanks! I managed to reproduce it on my end. Strangely I have this issue with @unleash/proxy-client-react v4 but not with v4.0.1 Cleaning all dependencies, doing fresh install and maybe rebooting your IDE can help. You're not using @unleash/nextjs, right? That one shouldn't require other SDKs as explicit dependencies

We are doing all the updates via dependabot and CI/CD there is no human involved. The issue is with 4.0.1.

I'm releasing v4.0.1 now


updates rails to address CVE-2014-3483

@square/kochiku-contributors

CVE-2014-3483

:+1:

I'll go ahead and accept this patch to update Rails but I don't think it was necessary since Kochiku only officially supports MySQL.


Add customizable permission messages to fragment-based start methods

Updated to work with new fragment-based start methods as well.

@kingst @awushensky bump 😄


Exception when running FindElementsByXPath

I am running WinAppDriver on a Windows 10 machine and trying to use it with a WPF application. I keep getting an exception when running FindElementsByXPath:
### SearchMultipleElements exception: Object reference not set to an instance of an object.
The tabitem exists. I can find it with inspect and WinUIRecorder. I have to wrap FindElementsByXPath in a helper function that keeps looping until it finds it. I have posted the output below. It usually takes about 5-10 calls until it finds it. What could be causing this?
==========================================
POST /session/7977EFDF-AF37-427A-9F30-84A491E87899/elements HTTP/1.1
Accept: application/json, image/png
Connection: Keep-Alive
Content-Length: 66
Content-Type: application/json;charset=utf-8
Host: <IP_ADDRESS>:4723
User-Agent: selenium/3.141.0 (.net windows)
{"using":"xpath","value":"//TabItem[@Name ="Setup Experiment"]"}
### SearchMultipleElements exception: Object reference not set to an instance of an object.
HTTP/1.1 200 OK
Content-Length: 74
Content-Type: application/json
{"sessionId":"7977EFDF-AF37-427A-9F30-84A491E87899","status":0,"value":[]}
==========================================
POST /session/7977EFDF-AF37-427A-9F30-84A491E87899/elements HTTP/1.1
Accept: application/json, image/png
Connection: Keep-Alive
Content-Length: 66
Content-Type: application/json;charset=utf-8
Host: <IP_ADDRESS>:4723
User-Agent: selenium/3.141.0 (.net windows)
{"using":"xpath","value":"//TabItem[@Name ="Setup Experiment"]"}
HTTP/1.1 200 OK
Content-Length: 104
Content-Type: application/json
{"sessionId":"7977EFDF-AF37-427A-9F30-84A491E87899","status":0,"value":[{"ELEMENT":"7.25176.62068903"}]}
==========================================

Can you share code?

Here is my testing code. I have also attached the xml of the session window.

        protected const string WindowsApplicationDriverUrl = "http://<IP_ADDRESS>:4723";
        protected const string WinAppDriverLocation = "C:\\Program Files (x86)\\Windows Application Driver\\WinAppDriver.exe";
        private static readonly string ProgramLocation= Path.Combine(Directory.GetCurrentDirectory(), @"Program.exe");
        protected static WindowsDriver<WindowsElement> session;

        [ClassInitialize]
        public static void Setup(TestContext context)
        {
            if (session == null)
            {
                ProcessStartInfo p_info = new ProcessStartInfo();
                p_info.UseShellExecute = true;
                p_info.CreateNoWindow = false;
                p_info.WindowStyle = ProcessWindowStyle.Normal;
                p_info.FileName = WinAppDriverLocation;
                WindowsAppDriver = Process.Start(p_info);

                var appiumOptions = new AppiumOptions();
                appiumOptions.AddAdditionalCapability("app", ProgramLocation);
                appiumOptions.AddAdditionalCapability("deviceName", "WindowsPC");
                appiumOptions.AddAdditionalCapability("platformName", "Windows");

                session = new WindowsDriver<WindowsElement>(new Uri(WindowsApplicationDriverUrl), appiumOptions);

                // Set implicit timeout to 60 seconds to make element search to retry every 500 ms
                session.Manage().Timeouts().ImplicitWait = TimeSpan.FromSeconds(1.5);

            }
        }

        [TestMethod]
        public void TestProgram()
        {
            session.FindElementByXPath("//Text[@Name=\"Hardware\"]").Click();
            session.FindElementByXPath("//Button[@Name=\"Load...\"]").Click();
            session.FindElementByXPath("//TabItem[@Name =\"Setup Experiment\"]").Click();

       }

WindowXML.txt

What xpath do you get from xpath recorder?


ColorBox

hey @pradeeban
I see a ColorBox component which is used to pick the color , But I think it is not implemented fully that's why it is not in use now
please assign me I will fully implement it to enhance UX experience

Can you please elaborate?
If there are unused components, there are two reasons:

They were left unimplemented because of lack of time.
They were implemented as part of DHGWorkflow, but removed later in concore-editor as we realized it is an unneeded feature. This looks like such a feature.

Do you have a use for this feature?
For example, in DHGWorkflow, you can choose the color of edges and nodes. We intentionally removed that in concore-editor.

(because, what we need is a minimal front-end for concore. If you are in doubt, you can also check DHHWorkflow to see the colors are supported in it).
At some point, we should remove any useless code that is hanging around. But please do not attempt that refactoring at this point.

yeah
"For example, in DHGWorkflow, you can choose the color of edges and nodes. We intentionally removed that in concore-editor."
I am talking about this feature , actually there is commented code present in all files related to ColorBox which I think need some integration

ah yes, ideally, the unused code should have been removed. But we were reaching the end of GSoC 2022 and were not confident in removing altogether. So we left the code commented and hanging around.
The feature should not exist as of now. The color means nothing in concore-editor.
I am closing this issue as "Won't Fix."


Timeout waiting to lock when there is a hanging daemon due to low heap size

When there is a daemon that is hanging because of small heap size there is a possibility that it acquired a lock that won't be released. It leads to a situation when we can't start new builds because Gradle tries to acquire a lock that is held by a hanging daemon.
Expected Behavior
Gradle should be able to recover from this situation and successfully start and perform new builds.
Current Behavior
Currently, Gradle fails to perform new builds on the same machine if there is a hanging daemon that acquired a lock. The only way to resolve the issue is to manually kill hanging daemon, ./gradle --stop command doesn't work.
Here is an example of one of our builds:

We started a build with low xmx: hanging daemon client logs
When we are trying to run a completely new build it fails with random LockTimeoutExceptions: launch new build logs
Hanging daemon thread dump: thread dump
./gradlew --stop doesn't work: logs

Context
We face this issue pretty frequently as we have a shared build machine where any developer can perform a build and if a developer runs a task that requires larger heap size than current Xmx sometimes the daemon gets stuck.
We see that there are some mechanisms in Gradle that immediately expire the daemon if JVM spends too much time on GC but it seems like it's not 100% reliable. Maybe it makes sense to introduce a timeout for a lock and force kill the daemon that holds the lock for more than specified timeout?

Steps to Reproduce

There is some luck involved, it's not always reproducible but we've managed to reproduce it several times on https://github.com/gradle/perf-native-large.

Run gradle assemble --max-workers=32 -Dorg.gradle.jvmargs="-Xmx530M" --continue in perf-native-large repo
Wait until the daemon hangs

This is the most tricky part, sometimes Gradle detects "GC thrashing" and manages to kill the daemon but sometimes this doesn't happen, you may need too experiment with Xmx value. We found out that with 530M xmx it is more or less reproducible
Also, it seems like we should catch the moment when the daemon acquired any shared lock

Try to run another Gradle build, for example gradle tasks on another project, you may try to run this multiple times in case if at the moment all locks are released

The first Gradle build will be hanging:
➜  perf-native-large git:(master) ✗ ./gradlew assemble --max-workers=32 -Dorg.gradle.jvmargs="-Xmx530M" --continue

Starting a Gradle Daemon, 5 stopped Daemons could not be reused, use --status for details
Expiring Daemon because JVM heap space is exhausted
Daemon will be stopped at the end of the build after running out of JVM memory
Expiring Daemon because JVM heap space is exhausted
Expiring Daemon because JVM heap space is exhausted
Expiring Daemon because JVM heap space is exhausted
Expiring Daemon because JVM heap space is exhausted
Expiring Daemon because JVM heap space is exhausted
Expiring Daemon because JVM heap space is exhausted
<=============> 100% CONFIGURING [5m 22s]
> IDLE

And it is not possible to run any other build:
./gradlew aspoet:fatJar
Downloading https://services.gradle.org/distributions/gradle-6.8.2-bin.zip
..........10%..........20%..........30%...........40%..........50%..........60%..........70%...........80%..........90%..........100%

Welcome to Gradle 6.8.2!

Here are the highlights of this release:
 - Faster Kotlin DSL script compilation
 - Vendor selection for Java toolchains
 - Convenient execution of tasks in composite builds
 - Consistent dependency resolution

For more details see https://docs.gradle.org/6.8.2/release-notes.html

Starting a Gradle Daemon (subsequent builds will be faster)

FAILURE: Build failed with an exception.

* What went wrong:
Gradle could not start your build.
> Could not create service of type FileAccessTimeJournal using GradleUserHomeScopeServices.createFileAccessTimeJournal().
   > Timeout waiting to lock journal cache (/Users/user/.gradle/caches/journal-1). It is currently in use by another Gradle instance.
     Owner PID: 39370
     Our PID: 39689
     Owner Operation:
     Our operation:
     Lock file: /Users/user/.gradle/caches/journal-1/journal-1.lock

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 1m 13s

Your Environment
Gradle 6.8.2/6.8.3
OS: macOS/Linux

Can you reproduce it with more recent versions of Gradle?

We understand that a Daemon holding a lock, but not being healthy enough to hand it over is a generic issue, and we might not have a generic solution for it.


feat(reminder): support to delete multiple reminders at once

#167 in #suggestions

Same as #11, requires option list support.


[2009-03-20] Make sapphire and cms fully multibyte safe

created by: @chillu (ischommer)
created at: 2009-03-20
original ticket: http://open.silverstripe.org/ticket/3746

Followup on http://www.silverstripe.org/migrating-a-site-to-silverstripe/show/256159?start=0#post256720
We use UTF-8 for internal storage and output, but using non-multibyte-safe methods to process them inbetween, which can lead to problems. Best example: strlen() will give you wrong lenghts on a multibyte string.
Nice summary of the problem scope and solutions:
http://www.nicknettleton.com/zine/php/php-utf-8-cheatsheet
Some points to fix:

The mbstring extension is already a requirement to run SilverStripe (see http://doc.silverstripe.com/doku.php?id=server-requirements). Its not a default in PHP, but widely available
Set mbstring function overloading via ini_set(), if possible. This way we don't need to replace method calls like strlen() with mb_strlen(). See http://se2.php.net/manual/en/mbstring.overload.php

NOTE: This isn't possible.  We need to either move to using mb_xxx, or recommend that people enable this setting themselves.

Update our Email mailers to send UTF-8 by default (and quoted-printable or base64)
Update htmlentities()
Write some unit tests to save UTF-8 data from a form and directly through DataObject->write(), read it out from the database through the ORM, and assert that you get the same string
Ensure that we are in fact dealing solely with UTF-8 strings, and use utf8_encode() where appropriate - this will be the most tricky part.

Additional reads about UTF-8 handling in PHP5 on PHPWACT:

http://www.phpwact.org/php/i18n/utf-8
http://www.phpwact.org/php/i18n/charsets
http://www.phpwact.org/php/i18n/charsets#common_problem_areas_with_utf-8

stale enhancement.


Memory leak audit

This PR contains information around a memory leak audit of react-dart, as well as some changes made to reduce the risk of memory leaks.
Audit Steps performed
Reviewed at-risk areas noted in ticket:

Event handlers
Transactional setState and setState callbacks
Callback refs
component lifecycle - shouldComponentUpdate usage
component lifecycle - proxying handler closures

Reviewed all other areas of prod code, including:

component factory proxies and variadic children
JS bindings
SyntheticEvents

Performed tests on internal_instance_refactor branch (UIP-2492 fix, follow-up to https://github.com/cleandart/react-dart/pull/125), with no memory leaks found:

Retention of component instances by ReactElements (original UIP-2492 issue)
Retention of last props
Retention of last state
Retention of props after unmount
Retention of state after unmount

Issues found / Changes
setState and transactional setState callbacks can be retained when the component unmounts in the middle of a state change.
This isn't a big problem, since those callbacks are added within the component, but it doesn't hurt to clear them since:

they're not generally expected to outlive the component
they may contain references to globals

Testing
Verify unit tests pass: pub run test -p content-shel

+1

+1


Equity like share vote

Story
As a liquidity provider
I want to be able to take part in and see the results of equity like share votes for change market proposals (to progress the proposal even if the required majority has not been met)
So that I can take part in and see the results of equity like share votes rather than rely on using the API directly
Acceptance Criteria
https://www.figma.com/file/F995ESZ0QMpTBuQWGAgLRz/Design-System?node-id=1820%3A72391

[ ] I can see on market change proposals (proposal form and proposal details page) my equity like share and how that relates to the vote (Equity like share is a network param - governance.proposal.updateMarket.minProposerEquityLikeShare )
[ ] I can see LP votes on a proposal details page (market change only) - we'll need to work out if this is a second vote, that can only occur if the first fails or if the two run concurrently
[ ] As an LP I can vote on an equity like share vote
[ ] I can understand the difference between the regular vote and equity like share vote i.e. we've explained it
[ ] I can see which of the vote types is the one that will be applied on the proposal details page (Token or LP is set to be the decisive one).

Tasks

[ ] Explore and sketch
[ ] Team and stakeholder review
[ ] Visual Design (if needed)
[ ] Team review
[ ] Develop

Sketch
Additional details / background info
From Michael

on market change proposals: show you equity like share (liquidity provider vote, AKA LP)
Find out from @EVODelavega if you submit two votes, one for VEGA and a different for LP?
Figma file  https://www.figma.com/file/F995ESZ0QMpTBuQWGAgLRz/Design-System?node-id=1718%3A70064 (either token or LP)

@ckpringle Dex identified the following related tickets. Would be great to go through these and consolidate / refine / etc.
https://github.com/vegaprotocol/frontend-monorepo/issues/1369
https://github.com/vegaprotocol/frontend-monorepo/issues/1370
https://github.com/vegaprotocol/frontend-monorepo/issues/1110
https://github.com/vegaprotocol/frontend-monorepo/issues/1109

David Siska was keen to chat about this before we started work on it

To get your EQlike share
{
  market(id:"c97538009e19f810be9631bab5a98c4a316f0d63fb9eaa7e9910c58379a61f5a"){
    data{
      liquidityProviderFeeShare{
        party{id}
        equityLikeShare
      }
    }
  }
}

then find the connected party in the provisions. I can't find a way in the API to get this directly.
a party with 100% EQ like share returns a "equityLikeShare": "1"

and to show the current vote status:
{
  proposal(id:""){
    votes{
      yes{
        totalEquityLikeShareWeight
      }
      no{
        totalEquityLikeShareWeight
      }
    }
  }
}

Hey @ckpringle, could you please confirm that the following are now removed by design?

Details page no longer showing number of days left to vote

No longer showing proposal set to fail after it's just been submitted

days left to vote would be good to see still (could just be a key value table below) but was removed by design from the bar chart area under "votes" header.
@jtsang586 I'm not sure what you mean about showing set to fail. but technically if no-one has voted yet, it would fail (assuming no-one else votes, which is generally what the page does).

@jtsang586 This is being reintroduced in #1820 so I suggest we close this and track there


Add support to the facet setting customization at the index level

⚠ This issue is generated, which means the examples and the namings do not necessarily correspond to the language of this repository.
Also, if you are a maintainer, please add any clarification and instructions about this issue.
Related to:

parent issue: https://github.com/meilisearch/integration-guides/issues/205
engine issue: https://github.com/meilisearch/specifications/pull/157, https://github.com/meilisearch/meilisearch/issues/2368

Add a faceting index settings resource to manage customization of the maximum value per facet.

Expose faceting object on /indexes/:index_uid/settings endpoints.
Add GET/PATCH/DELETE - /indexes/:index_uid/settings/faceting endpoints.

The faceting object definition is made of the following properties:

maxValuesPerFacet type (int)

More information about what each field does could be checked in the spec.
Check the other customization methods for reference like: synonyms, searchableAttributes, filterableAttributes.
TODO:

[ ] Add the new methods to customize faceting

[ ] GET /indexes/:index_uid/settings/faceting
[ ] PATCH /indexes/:index_uid/settings/faceting
[ ] DELETE /indexes/:index_uid/settings/faceting

[ ] Add tests

From this list the following code-samples needs to be added as well:

get_faceting_settings_1: |-
update_faceting_settings_1: |-
reset_faceting_settings_1: |
settings_guide_faceting_1: |-


Using deps of java 8 under java 7

Error:scalac: missing or invalid dependency detected while loading class file 'ConfigsInstances.class'.
Could not access type Duration in value java.time,
because it (or its dependencies) are missing. Check your build definition for
missing or conflicting dependencies. (Re-run with -Ylog-classpath to see the problematic classpath.)
A full rebuild may help if 'ConfigsInstances.class' was compiled against an incompatible version of java.time.

Java 7 support was dropped in 0.4.x.


Set createValueFunc to null after the value being evaluated

Pull Request Template
Description
Set createValueFunc to null once the Func delegate is not needed. Without breaking the reference, the lambda function will keep capturing of any outer variables, in #4345, DocumentServiceRequest was captured, which prevent GC from recycling the request object, cause memory leak.
Below code, the request object will be captured by singleValueInitFunc delegate.
addresses = await this.serverPartitionAddressCache.GetAsync(
                         key: partitionKeyRangeIdentity,
                         singleValueInitFunc: (_) => this.GetAddressesForRangeIdAsync(
                             request,
                             cachedAddresses: null,
                             partitionKeyRangeIdentity.CollectionRid,
                             partitionKeyRangeIdentity.PartitionKeyRangeId,
                             forceRefresh: false),
                         forceRefresh: (_) => false);

Type of change
Please delete options that are not relevant.

[] Bug fix (non-breaking change which fixes an issue)

Closing issues
To automatically close an issue: closes #4345

Closing due to inactivity, feel free to re-open as needed.


Translate GroupBy after Join with aggregate in projection

Note: works fine in a single table
Exception: Working abnormally when using multiple tables to associate queries (EF 6.0 is normal)
example:
            var query = from a in m.BomInfo
                        join b in m.BomInfoDetail on a.Id equals b.BillId
                        group new { a,b} by new { b.BillId,a.BillDate} into g
                        select new
                        {
                           g.Key.BillDate,
                           g.Key.BillId,
                           Count=g.Count()
                        };

SQL:
SELECT [a].[Id], [a].[Batch], ......
FROM [BomInfo] AS [a]
INNER JOIN [BomInfoDetail] AS [b] ON [a].[Id] = [b].[BillId]
ORDER BY [b].[BillId], [a].[BillDate]

I can confirm the bug.
Btw, this is a bug not an "enhancement" because no warning is print and group by is not evaluate in memory.

Yes Yes.
The 2.1 documentation indicates that this method is already supported, but it does not actually take effect on multiple tables.
source:
https://docs.microsoft.com/en-us/ef/core/what-is-new/ef-core-2.1

@gongjie18675517757 Can you please post a runnable project/solution or complete code listing that demonstrates the behavior you are seeing?

@ajcvickers
EFCore.GroupTest.zip
Hello there,the file is a test solution,
you need change the db connectionstring
then run
update-database

@ajcvickers
linq :
from a in db.Order
join b in db.OrderDetail on a.Id equals b.OrderId
group new { a, b } by new { a.Id } into g
select new
 {
      Id = g.Key,
      Count = g.Count(),
      Sum = g.Sum(x => x.b.Quantity),
      Max = g.Max(x => x.b.Quantity),
      Min = g.Min(x => x.b.Quantity)
  }

should sql  is :
SELECT [a].[Id], COUNT(1) [Count], SUM([b].[Quantity]) [Sum],MAX([b].[Quantity]) [Max],MIN([b].[Quantity]) [Min]
FROM [Order] AS [a]
INNER JOIN [OrderDetail] AS [b] ON [a].[Id] = [b].[OrderId]
group by [a].[Id]

actually sql  is :
SELECT [a].[Id], [a].[OrderNumber], [b].[Id], [b].[Name], [b].[OrderId], [b].[Quantity]
FROM [Order] AS [a]
INNER JOIN [OrderDetail] AS [b] ON [a].[Id] = [b].[OrderId]
ORDER BY [a].[Id]

then group by in memory

@gongjie18675517757 From running the code, I see warnings and get the correct results. So it looks like it is being client-evaluated and returns the correct results. Translating this is considered an enhancement rather than a bug fix, unless I am missing some other way that this is a bug.
warn: Microsoft.EntityFrameworkCore.Query[20500]
      => Microsoft.EntityFrameworkCore.Query.RelationalQueryModelVisitor
      The LINQ expression 'GroupBy(new <>f__AnonymousType3`1(Id = [a].Id), new <>f__AnonymousType2`2(a = [a], b = [b]))' could not be translated and will be evaluated locally.
warn: Microsoft.EntityFrameworkCore.Query[20500]
      => Microsoft.EntityFrameworkCore.Query.RelationalQueryModelVisitor
      The LINQ expression 'Count()' could not be translated and will be evaluated locally.
warn: Microsoft.EntityFrameworkCore.Query[20500]
      => Microsoft.EntityFrameworkCore.Query.RelationalQueryModelVisitor
      The LINQ expression 'Count()' could not be translated and will be evaluated locally.
warn: Microsoft.EntityFrameworkCore.Query[20500]
      => Microsoft.EntityFrameworkCore.Query.RelationalQueryModelVisitor
      The LINQ expression 'Sum()' could not be translated and will be evaluated locally.
warn: Microsoft.EntityFrameworkCore.Query[20500]
      => Microsoft.EntityFrameworkCore.Query.RelationalQueryModelVisitor
      The LINQ expression 'Sum()' could not be translated and will be evaluated locally.
warn: Microsoft.EntityFrameworkCore.Query[20500]
      => Microsoft.EntityFrameworkCore.Query.RelationalQueryModelVisitor
      The LINQ expression 'Max()' could not be translated and will be evaluated locally.
warn: Microsoft.EntityFrameworkCore.Query[20500]
      => Microsoft.EntityFrameworkCore.Query.RelationalQueryModelVisitor
      The LINQ expression 'Max()' could not be translated and will be evaluated locally.
warn: Microsoft.EntityFrameworkCore.Query[20500]
      => Microsoft.EntityFrameworkCore.Query.RelationalQueryModelVisitor
      The LINQ expression 'Min()' could not be translated and will be evaluated locally.
warn: Microsoft.EntityFrameworkCore.Query[20500]
      => Microsoft.EntityFrameworkCore.Query.RelationalQueryModelVisitor
      The LINQ expression 'Min()' could not be translated and will be evaluated locally.

@gongjie18675517757 From running the code, I see warnings and get the correct results. So it looks like it is being client-evaluated and returns the correct results. Translating this is considered an enhancement rather than a bug fix, unless I am missing some other way that this is a bug.

Hi,
in my case the group by is not execute at all. but I use the fluent syntax with a where clause and create a DTO object in the select.
so, the query can be translate in this way:
class DtoOrder {
        public string Id { get; set; }
        public string OrderNumber { get; set; }
        public int NumberItem {get; set;}
}
//code

var query = db.Order.Join(db.OrderDetail,
o => o.Id,
d => d.OrderDetail,
(o, d) => new {o, d}).Where(a => a.o.OrderNumber == "1")
.GroupBy(a => a.o.Id)
.Select(g => new DtoOrder() {
Id = g.Key,
OrderNumber = g.Max(v => v.o.OrderNumber),
NumberItem = g.Count()
});

query.ToList();

//code

Sorry, I do not have time to create a whole project. :-(
P.S. as @gongjie18675517757 said, in the documentation the developer team wrote:
"We now support translating it to the SQL GROUP BY clause in most common cases."
https://docs.microsoft.com/en-us/ef/core/what-is-new/ef-core-2.1
A group by after a simple join is a common case IMHO. :-)

@gongjie18675517757 Agreed, but that is not implemented yet, which is why this is an open issue.
@lfongaroScp If you are seeing something different, then please file a new issue with a complete description and a runnable project/solution or complete code listing to reproduce the issue. Without that there isn't much we can do.

Possibly related to https://github.com/aspnet/EntityFrameworkCore/issues/13887

This is likely fixed in 3.0

        [ConditionalTheory]
        [MemberData(nameof(IsAsyncData))]
        public virtual Task Join_GroupBy_Aggregate(bool isAsync)
        {
            return AssertQuery<Order, Customer>(
                isAsync,
                (os, cs) =>
                    (from o in os
                     join c in cs
                         on o.CustomerID equals c.CustomerID
                     group o by c.CustomerID)
                    .Select(
                        g => new
                        {
                            g.Key,
                            Count = g.Average(o => o.OrderID)
                        }),
                e => e.Key);
        }

Generates
SELECT [c].[CustomerID] AS [Key], AVG(CAST([o].[OrderID] AS float)) AS [Count]
FROM [Orders] AS [o]
INNER JOIN [Customers] AS [c] ON [o].[CustomerID] = [c].[CustomerID]
GROUP BY [c].[CustomerID]


adjust log levels for connection related log messages

default log level is ERROR so connection msgs won't be logged by default.

Codecov Report

Merging #20 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master      #20   +/-   ##
=======================================
  Coverage   76.02%   76.02%
=======================================
  Files          21       21
  Lines        2290     2290
=======================================
  Hits         1741     1741
  Misses        499      499
  Partials       50       50

Impacted Files
Coverage Δ

v1/ao/internal/reporter/reporter_grpc.go
36.05% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

@cce whenever you get a chance to review


melee weapon required

Hi, for some reason keeps asking me this while trying to charge with kinetic blade as a kinetic knight.
Not sure why, the weapon is already equipped.

I make it so kinetic blade doesn't register as weapon for attacks of opportunity. Charge checks for such a weapon. I didn't know that.
If you activate whip infusion, it will work.

09.07.2022 19:15:01 Tequilaman @.***>:
Hi, for some reason keeps asking me this while trying to charge with kinetic blade as a kinetic knight.
Not sure why, the weapon is already equipped.
—
Reply to this email directly, view it on GitHub[https://github.com/Truinto/DarkCodex/issues/85], or unsubscribe[https://github.com/notifications/unsubscribe-auth/ALNSQ7N3WAZLT5OUMXTV5LDVTGXRJANCNFSM53DU53HQ].
You are receiving this because you are subscribed to this thread.[Verfolgungsbild][https://github.com/notifications/beacon/ALNSQ7IY7G4PZ7EYA6VVIULVTGXRJA5CNFSM53DU53H2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4TLYIENA.gif]

I make it so kinetic blade doesn't register as weapon for attacks of opportunity. Charge checks for such a weapon. I didn't know that.
If you activate whip infusion, it will work.
Will fix.

No time to test. Does it work now?


Update sites.yml

Hi! I would like to add my website to 512kb club. Greetings!
This PR is:

[X] Adding a new domain
[ ] Updating existing domain size
[ ] Changing domain name
[ ] Removing existing domain from list
[ ] Website code changes (512kb.club site)
[ ] Other not listed

[X] I used the uncompressed size of the site
[X] I have included a link to the Cloudflare report
[X] This site is not an ultra minimal site
[X] The following information is filled identical to the data file

I confirm that I have read the FAQ section, particularly the two red items around minimal pages and inappropriate content and I attest that this site is neither of these things.

[ ] Check to confirm

- domain: cri.cl
  url: https://cri.cl/
  size: 86.13kb
  last_checked: 2024-03-28

Cloudflare Report: https://radar.cloudflare.com/scan/7da9beb0-5198-443f-bfa0-860ee90e2d87/summary

Thanks!


How to enable disable connection reuse

I am looking for a way to enable tor disable connection reuse with each request. How do I set the keep alive session in dsl. I am having hard time to figure out the way.

Hello, thank you for trying and reporting this.
Currently the library does not provide any method to set this behavior, but it should be quite easy to add. Do you think you can contribute with the PR?
You can also try as workaround to set the Connection header to close when you don't want to use keep alive for some connection.

Sure, I will raise a PR for this


Add Washington county, PA

Closes #4827
Closes #4828

Hi, @iandees,
Thank you so much for implementing this in openaddresses! And thank you so much for the fast response to my issues! :-)

Thanks for pointing these out!


When loading read-more items, respect global loadImages flag.

In the app we offer a setting to globally disable loading any images, to conserve bandwidth. However, the ReadMore items in the Footer were not respecting this flag, and were continuing to load images.

This is just a mirror of the repo, this patch would need to be opened on gerrit.

damn it! ☹  in these trying times I forgot that Gerrit was a thing.