ST_SubDivide (Snowflake) fails even on documentation example

Expected behavior
SELECT ST_SubDivide(ST_GeomFromText('POLYGON((35 10, 45 45, 15 40, 10 20, 35 10), (20 30, 35 35, 30 20, 20 30))'), 5)
==============================================================================================
MULTILINESTRING ((0 0, 5 5), (5 5, 10 10), (10 10, 21 21), (21 21, 60 60), (60 60, 85 85), (85 85, 100 100), (100 100, 120 120))
doc_example
Actual behavior

Steps to reproduce the problem
https://sedona.apache.org/1.5.1/setup/snowflake/install/
sedona-snowflake-1.5.1.jar: Sedona's Maven Central repository
geotools-wrapper-1.5.1-28.2.jar: GeoTools-wrapper's Maven Central repository
java -jar sedona-snowflake-1.5.1.jar --geotools-version 1.5.1-28.2 > sedona-snowflake.sql
Settings
Sedona version = 1.5.1
Environment = Snowflake
SEDONA-519

@andreypanchenko Thanks for reporting this. Do you have other Java-based UDF or jars used in the same warehouse?

@jiayuasu
Hi. This exactly from sedona schema as mentioned in guide. Any othe java UDfs at all.

I really keen to use this feature. What additional info I can provide to u?

@jiayuasu btw the same issue when I tried to call this function from snowflake store application snowsedona

This is caused by dependency changes of geotools-wrapper-1.5.1-28.2.jar, Apache commons-lang3 was removed from the geotools-wrapper jar, and it is required by ST_SubDivide function, thus caused this problem. I am already working on a fix for this (see https://github.com/apache/sedona/pull/1290 and https://github.com/wherobots/sedona-snowflake-tester/pull/20).
You can also try replacing geotools-wrapper-1.5.1-28.2.jar with geotools-wrapper-1.5.0-28.2.jar to workaround this issue.

@andreypanchenko You can download the geotools-wrapper-1.5.0-28.2.jar here and replace the current geotools-wrapper jar in your Snowflake stage.

@andreypanchenko The SedonaSnow app on Snowflake marketplace is now pending approval. Once that is done, we will release it to your account automatically. Will keep you posted.

@andreypanchenko You can download the geotools-wrapper-1.5.0-28.2.jar here and replace the current geotools-wrapper jar in your Snowflake stage.

@jiayuasu
It's working, thank you!


To investigate: Is DDL working correctly?

I have the following DDL in my .datafusionrc but the tables are not being created in the correct schema (the table name is "staging.min_aggs" instead of creating "min_aggs" in "staging" schema.  I dont yet know whether my DDL is wrong or if this is a dft issue - i am just creating this to make sure i dont forget about it.
CREATE SCHEMA staging;
CREATE SCHEMA production;

CREATE EXTERNAL TABLE staging.min_aggs STORED AS PARQUET LOCATION 'ny2://sip/aggregates/minute_by_ticker_monthly_v2/year_month=2015-08/data.01.parquet';
CREATE EXTERNAL TABLE staging.trades_v2 STORED AS PARQUET LOCATION 'ny2://sip/trades_v2/date=2024-08-30/2024-08-30.01.parquet';

CREATE EXTERNAL TABLE "production".min_aggs STORED AS PARQUET LOCATION 'ny5://sip/aggregates/minute_by_ticker_monthly/year_month=2015-08/data.01.parquet';

CREATE TABLE wide AS VALUES (1,2,3,4,5,6,7,8,9,10,11,12,13,14,15,16,17,18,19,20);

Note to self, add tests for this.


check-files: Don't check same-named files

The check-files script contains the strings "TODO" and "todo" in order to
search for files that contain TODO items. So, any check-files script would
need to be excluded from the list of files that gets checked for "TODO".
Normally, the script excludes itself from checks, but with the addition of
the crypto submodule, there is another copy of the script present from the
project root. We must avoid checking check-files scripts for TODO items.
This also helps if you run check-files from another working tree in your
working tree.

This has been merged into development-psa and can now be closed.


Remove blacklight iiif AutoSuggest from iiif_print generator

We need to disable blacklight's autosuggest. It is buggy anyways and causes issues with the solr build.
ref change Rob made for hyku:
Ideally, when we run iiif_print's generator it will no longer apply these changes to enable auto suggest.
https://github.com/samvera/hyku/pull/1926/commits/86c791fd1884f2fc1721a1d15edcdf52ab7d8b8b
Notes
https://github.com/boston-library/blacklight_iiif_search/blob/b2fc5abdb3027762a249dcba9afb8998c6da8ffe/lib/generators/blacklight_iiif_search/controller_generator.rb
https://github.com/scientist-softserv/iiif_print/blob/main/lib/generators/iiif_print/install_generator.rb#L18

consider making a fork for black light iiif_search and a PR


Surround modes are flipped

Surround modes are flipped
Description
The surround modes are flipped. I deduced this based on observed behavior using the API, as well as cross-referencing other libraries (such as SOCO)
Your checklist for this pull request
🚨 Please review the guidelines for contributing to this repository.

[x] Make sure you are requesting to pull a topic/feature/bugfix branch (right side). Don't request your main!
[x] Make sure you are making a pull request against the main branch (left side). Also you should start your branch off svrooij/sonos-api-docs/main.
[x] Check your code additions will fail neither code linting checks nor unit test. (mine sometimes also fail, will probably ignore the lint failures)
[x] If you changed the documentation.json be sure to also regenerate the services files.
[x] The docs/services/index.md and the docs/services/*.md files should not be manually changed, only with the generator.

💔 Thank you!

I never used the surround parts, so I should just believe you or the docs in soco
Looks fine by me!

You forgot one thing 😉
@all-contributors please add @jkossis for docs


What's the state of this project?

Is this the recommended version of coffescript? will it replace the current coffeescript? will there be more releases? Is there a maintainer for this project?

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/32995185-what-s-the-state-of-this-project?utm_campaign=plugin&utm_content=tracker%2F33145&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F33145&utm_medium=issues&utm_source=github).

See #336.

Anyone is free to work on this project. I stopped working on it after a disagreement between myself and the other CoffeeScript maintainers [...]. I don't see a future for the CoffeeScript language without adoption of ES6 features. Unfortunately, I was the only one that believed we should adopt the semantics of any equivalent features in ES6 [...]. So because I no longer use the language for my own projects, I am not motivated to work on it. But there's nothing stopping anyone from opening issues and sending PRs.

TL;DR: It's dead.

w.r.t adopting ES6 features, I'll be announcing something in a month or few.

(I hope CoffeeScript doesn’t die—it’s by far my favorite language right now.)
Peter Alvin

On Apr 19, 2016, at 04:15, Nikita Litvin<EMAIL_ADDRESS>wrote:
See #336 https://github.com/michaelficarra/CoffeeScriptRedux/issues/336.
Anyone is free to work on this project. I stopped working on it after a disagreement between myself and the other CoffeeScript maintainers [...]. I don't see a future for the CoffeeScript language without adoption of ES6 features. Unfortunately, I was the only one that believed we should adopt the semantics of any equivalent features in ES6 [...]. So because I no longer use the language for my own projects, I am not motivated to work on it. But there's nothing stopping anyone from opening issues and sending PRs.
TL;DR: It's dead.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub https://github.com/michaelficarra/CoffeeScriptRedux/issues/361#issuecomment-211787145


Update removing-user-from-restricted-users-portal-after-spam.md

I have seen this take more than an hour on multiple occasions before the user can send out emails. The user is also complaining the same. I have made changes to reflect it as 24 hours. Let me know if you think this can be reduced to say 8 or 4 hours.
Fixes #https://github.com/MicrosoftDocs/microsoft-365-docs/issues/10323

@JohanFreelancer9 Please copy review

Learn Build status updates of commit 6601a4d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/office-365-security/removing-user-from-restricted-users-portal-after-spam.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 99fc866:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/office-365-security/removing-user-from-restricted-users-portal-after-spam.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit 032bd73:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/office-365-security/removing-user-from-restricted-users-portal-after-spam.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit a31631c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-365/security/office-365-security/removing-user-from-restricted-users-portal-after-spam.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


fix(runtime-dom): insertStaticContent correctly if parent has only one child

fix #4104

Thanks for the PR, but I refactored static insertion to simpler and safer logic in c0db807844194a227f6d0dad6cc3e1e8f148770f


Add folder lock to collector's buffer folder

Please answer these questions before submitting pull request

Why submit this pull request?

[ ] Bug fix

[X] New feature provided

[ ] Improve performance

Related issues
#1204

Bug fix

Bug description.
More than one collector thread use same buffer folder to write segments into data file.

How to fix?
Try to lock file in buffer folder when collector starting.

Coverage increased (+0.004%) to 23.499% when pulling c286f852a8d960997bccdc17b7ff74d3a02dd0de on peng-yongsheng:feature/1204 into 9c4b95ab9f6ae3e49ad485aa63df392bb79d4dfc on apache:master.


Add and edit docstrings of recon_surf

This pull request adds docstrings  to the methods of the recon surf files.
The conventions from the Googe style guide were mostly used for the docstrings.
The changes are not final, as some descriptions are missing or incorrect. The critical parts are marked with a "[help]" to find them more easily.
TODOS:

Fill/check marked descriptions

also please re-base this, as it is difficult to find what you actually changed and what is from dev.


Build wheel

Name "smv2" exists already in PyPI. Will use "pysmv" as package name, although still import "smv".


Compile-time warnings: declared inside parameter list, incompatible-pointer-types, unused-but-set-variable

Description
The compiler emits 3 distinct types of warnings when compiling the stack:

declared inside parameter list:In file included from ../dependencies/canopen-stack/canopen/include/co_nmt.h:28,
                 from ../dependencies/canopen-stack/canopen/source/co_nmt.c:21:
../dependencies/canopen-stack/canopen/include/co_obj.h:276:56: warning: 'struct CO_NODE_T' declared inside parameter list will not be visible outside of this definition or declaration
  276 | typedef uint32_t (*CO_OBJ_SIZE_FUNC) (CO_OBJ *, struct CO_NODE_T *, uint32_t);
      |                                                        ^~~~~~~~~

-Wincompatible-pointer-types:../dependencies/canopen-stack/canopen/source/co_nmt.c:72:42: warning: initialization of 'int16_t (*)(CO_OBJ *, struct CO_NODE_T *, void *, uint32_t)' {aka 'short int (*)(struct CO_OBJ_T *, struct CO_NODE_T *, void *, long unsigned int)'} from incompatible pointer type 'int16_t (*)(CO_OBJ *, struct CO_NODE_T *, void *, uint32_t)' {aka 'short int (*)(struct CO_OBJ_T *, struct CO_NODE_T *, void *, long unsigned int)'} [-Wincompatible-pointer-types]
   72 | const CO_OBJ_TYPE COTNmtHbCons = { 0, 0, COTypeNmtHbConsRead, COTypeNmtHbConsWrite };
      |                                          ^~~~~~~~~~~~~~~~~~~

-Wunused-but-set-variable:../dependencies/canopen-stack/canopen/source/co_pdo.c:908:16: warning: variable 'cod' set but not used [-Wunused-but-set-variable]
  908 |     CO_DICT   *cod;
      |                ^~~

As I understand it, issues 1 and 2 are caused by the order of definitions: the function prototypes that take CO structs as arguments are declared before the definition of those structs. To solve these issues, the header files will need to be restructured.
Issue 3 is much simpler to solve: variable cod should be removed from COTypePdoComIdWrite and inside COTypeNmtHbConsWrite.
Steps to Reproduce
Below is a verbose build log. The log file includes the GCC invocations used to compile the CANopen stack:
build.log
Expected behavior
The stack should compile without warnings. Compiler warnings usually point to underlying issues that can unexpectedly become real problems.
Environment

Target: STM32F103
Cross-Compiler: GNU Arm embedded toolchain 9-2020-q2-update
CANopen Stack: 4.0.2

I'm seeing similar warnings/errors using Keil uVision 5 targeted for an STM32F405, but I'm just getting started setting things up, so it could be a PBKAC on my part. Relevant messages are:
Build started: Project: AFBR_S50_TOF_rangefinder
*** Using Compiler 'V5.06 update 6 (build 750)', folder: 'C:\Keil_v5\ARM\ARMCC\Bin'
...
compiling co_nmt.c...
Drivers/canopen/include/co_obj.h(276): warning:  #231-D: declaration is not visible outside of function
  typedef uint32_t (*CO_OBJ_SIZE_FUNC) (CO_OBJ *, struct CO_NODE_T *, uint32_t);
...
Drivers\canopen\source\co_nmt.c(72): warning:  #144-D: a value of type "int16_t (*)(CO_OBJ *, struct CO_NODE_T *, void *, uint32_t)" cannot be used to initialize an entity of type "CO_OBJ_READ_FUNC"
...
Drivers\canopen\source\co_nmt.c(299): error:  #167: argument of type "CO_NODE *" is incompatible with parameter of type "struct CO_NODE_T *"
      err = COObjRdValue(obj, node, &num, CO_BYTE, 0);
Drivers\canopen\source\co_nmt.c(470): warning:  #550-D: variable "cod"  was set but never used
      CO_DICT     *cod;

Adding a forward struct declaration "struct CO_NODE_T;" to line 251 of "co_obj.h" resolved the above-mentioned warnings, and brought me to a lot of undefined-symbol linker errors, which is what I would expect as I have not yet coded the system-specific interface functions.

Ok, I currently use compiler Microsoft Visual C++ V19.24 for my development and test environment. Unfortunately, I'm new in using "Modern Cmake" and I didn't realize that I need to enable the warnings explicitly. In the branch "develop" I fix all warnings, which rises up after enabling all warnings in MSVC.
Could you both check if this covers all warnings in your environments as well, please?

The develop branch fixes most warnings in GCC. Here's the build output:
[23/90] Building C object dependencies/CMakeFiles/canopen.dir/canopen-stack/canopen/source/co_nmt.c.o
../dependencies/canopen-stack/canopen/source/co_nmt.c: In function 'COTypeNmtHbConsWrite':
../dependencies/canopen-stack/canopen/source/co_nmt.c:470:18: warning: variable 'cod' set but not used [-Wunused-but-set-variable]
  470 |     CO_DICT     *cod;
      |                  ^~~
[37/90] Building C object dependencies/CMakeFiles/canopen.dir/canopen-stack/canopen/source/co_pdo.c.o
../dependencies/canopen-stack/canopen/source/co_pdo.c: In function 'COTypePdoComIdWrite':
../dependencies/canopen-stack/canopen/source/co_pdo.c:908:16: warning: variable 'cod' set but not used [-Wunused-but-set-variable]
  908 |     CO_DICT   *cod;
      |                ^~~

These last two warnings are easy to fix: the object dictionary pointer is never used, and can be deleted without consequence.
diff --git a/canopen/source/co_nmt.c b/canopen/source/co_nmt.c
index b9787e2..669b6e8 100644
--- a/canopen/source/co_nmt.c
+++ b/canopen/source/co_nmt.c
@@ -467,14 +467,12 @@ void CONmtHbConsMonitor(void *parg)
 */
 int16_t COTypeNmtHbConsWrite(CO_OBJ* obj, struct CO_NODE_T *node, void *buf, uint32_t size)
 {
-    CO_DICT     *cod;
     CO_HBCONS  *hbc;
     int16_t     result = CO_ERR_TYPE_WR;
     uint32_t    value  = 0;
     uint16_t    time;
     uint8_t     nodeid;

-    cod  = &node->Dict;
     hbc  = (CO_HBCONS *)obj->Data;
     if (hbc == 0) {
         node->Error = CO_ERR_CFG_1016;
diff --git a/canopen/source/co_pdo.c b/canopen/source/co_pdo.c
index 5d4b325..a148728 100644
--- a/canopen/source/co_pdo.c
+++ b/canopen/source/co_pdo.c
@@ -905,7 +905,6 @@ int16_t COTypePdoMapWrite(CO_OBJ *obj, struct CO_NODE_T *node, void *buf, uint32
 */
 int16_t COTypePdoComIdWrite(CO_OBJ* obj, struct CO_NODE_T *node, void *buf, uint32_t size)
 {
-    CO_DICT   *cod;
     CO_NMT   *nmt;
     CO_TPDO  *tpdo = 0;
     CO_RPDO  *rpdo = 0;
@@ -928,7 +927,6 @@ int16_t COTypePdoComIdWrite(CO_OBJ* obj, struct CO_NODE_T *node, void *buf, uint
         return (CO_ERR_OBJ_RANGE);
     }

-    cod     = &node->Dict;
     nmt     = &node->Nmt;
     pcomidx = CO_GET_IDX(obj->Key);
     if ((pcomidx >= 0x1400) && (pcomidx <= 0x15FF)) {

Hmm, with all warnings enabled (/W4 /WX) in MSVC v19.23.28106.4 for x64, the warning C4189 is not detected; Strange!?
I think it is best to set up a parallel GCC environment for checking the warning-free compilation with future changes.

I also get only those two warnings for the unused object dictionary pointer "cod" under Keil uVision with the updated code. Looks like you're on the right track. Thanks for maintaining this package!

Ok, I setup my local environment to IAR EWARM and see the same remaining warnings. Now the develop branch is warning-free.

Great, everything compiles without warnings on GCC as well!
A CI job could be used to compile/check GCC output/test automatically.
I've set this up in the past with Gitlab CI, though I'm not really with the way it works in Gitlab Actions...

fixed in v4.0.3


Native build error on windows and macOS suddenly

I  can't reopen the issue, removed the version still not works.
https://github.com/optimism-java/hildr/actions/runs/7231369944/workflow

It seems you are building with Gradle and caching is enabled, which you may want to clear. I'm not sure Gradle actually runs on JDK 21 already. Could you try running it on JDK 17 and then set GRAALVM_HOME to a GraalVM for JDK 21? Here's an example how that can be done: add set-java-home: 'false' to ensure the action does not overwrite a previously set JAVA_HOME.

@fniephaus I can't get you, I use this template only adding cache https://github.com/graalvm/setup-graalvm?tab=readme-ov-file#quickstart-template.

@fniephaus I can't get you, I use this template only adding cache https://github.com/graalvm/setup-graalvm?tab=readme-ov-file#quickstart-template.

Also after I change ./gradlew nativeCompile to native-image, it works fine. It should be an issue about graalvm gradle plugin.

Ok good! Feel free to file a ticket against the Native Build Tools at https://github.com/graalvm/native-build-tools/issues


[tool_crash] DartDevelopmentServiceException

Command
flutter run

Steps to Reproduce

...
...
...

Logs
DartDevelopmentServiceException
#0      new IOWebSocketChannel._withoutSocket.<anonymous closure> (package:web_socket_channel/io.dart:87:24)
#1      _invokeErrorHandler (dart:async/async_error.dart:45:24)
#2      _HandleErrorStream._handleError (dart:async/stream_pipe.dart:272:9)
#3      _ForwardingStreamSubscription._handleError (dart:async/stream_pipe.dart:157:13)
#4      _rootRunBinary (dart:async/zone.dart:1378:47)
#5      _CustomZone.runBinary (dart:async/zone.dart:1272:19)
#6      _CustomZone.runBinaryGuarded (dart:async/zone.dart:1178:7)
#7      _BufferingStreamSubscription._sendError.sendError (dart:async/stream_impl.dart:360:15)
#8      _BufferingStreamSubscription._sendError (dart:async/stream_impl.dart:378:16)
#9      _BufferingStreamSubscription._addError (dart:async/stream_impl.dart:280:7)
#10     _SyncStreamControllerDispatch._sendError (dart:async/stream_controller.dart:737:19)
#11     _StreamController._addError (dart:async/stream_controller.dart:615:7)
#12     _rootRunBinary (dart:async/zone.dart:1378:47)
#13     _CustomZone.runBinary (dart:async/zone.dart:1272:19)
#14     _CustomZone.runBinaryGuarded (dart:async/zone.dart:1178:7)
#15     _BufferingStreamSubscription._sendError.sendError (dart:async/stream_impl.dart:360:15)
#16     _BufferingStreamSubscription._sendError (dart:async/stream_impl.dart:378:16)
#17     _BufferingStreamSubscription._addError (dart:async/stream_impl.dart:280:7)
#18     _SyncStreamControllerDispatch._sendError (dart:async/stream_controller.dart:737:19)
#19     _StreamController._addError (dart:async/stream_controller.dart:615:7)
#20     new Stream.fromFuture.<anonymous closure> (dart:async/stream.dart:166:18)
#21     _rootRunBinary (dart:async/zone.dart:1378:47)
#22     _CustomZone.runBinary (dart:async/zone.dart:1272:19)
#23     _FutureListener.handleError (dart:async/future_impl.dart:169:20)
#24     Future._propagateToListeners.handleError (dart:async/future_impl.dart:719:47)

[✓] Flutter (Channel master, 2.1.0-11.0.pre.145, on macOS 11.0.1 20B29 darwin-x64, locale en-CN)
    • Flutter version 2.1.0-11.0.pre.145 at /Users/jiaozebo/gitee/flutter
    • Framework revision a0ba646408 (7 days ago), 2021-03-05 22:09:03 -0500
    • Engine revision f751d04fa0
    • Dart version 2.13.0 (build 2.13.0-107.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at /Users/jiaozebo/Library/Android/sdk
    • Platform android-30, build-tools 30.0.3
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✗] Xcode - develop for iOS and macOS
    ✗ Xcode installation is incomplete; a full installation is necessary for iOS development.
      Download at: https://developer.apple.com/xcode/download/
      Or install Xcode via the App Store.
      Once installed, run:
        sudo xcode-select --switch /Applications/Xcode.app/Contents/Developer
        sudo xcodebuild -runFirstLaunch
    ✗ CocoaPods not installed.
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To install see https://guides.cocoapods.org/using/getting-started.html#installation for instructions.

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio
    • Android Studio at /Applications/Android Studio 4.2 Preview.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6842174)

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] VS Code (version 1.54.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.20.0

[!] Proxy Configuration
    • HTTP_PROXY is set
    • NO_PROXY is localhost
    ! NO_PROXY does not contain <IP_ADDRESS>
    • NO_PROXY contains localhost

[✓] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 89.0.4389.82

! Doctor found issues in 2 categories.

Flutter Application Metadata
Type: app
Version: 1.0.0+1
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: master
Creation framework version: a0ba6464089f906fefbf610732b2c9a7b66a1260

@TahaTesser
possible duplicate of https://github.com/flutter/flutter/issues/77077

@jiaozebo
according to this message the issue seems to be solved  in the latest stable, can you confirm?
cc @TahaTesser

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.


shard map implemenation of pmap only works for the first signature

Description
from jax.experimental.shard_map import pmap
from jax import numpy as jnp

def f(x):
  return x * x

f = pmap(f, axis_name='batch')
f(jnp.ones((8,)))
f(jnp.ones((8, 2))) # raises StoreException: Store occupied

Giving multiple input signatures fails on the shard map implemenation of pmap
System info (python version, jaxlib version, accelerator, etc.)
latest version of jax

This is not polished or fully tested yet. It's purely experimental right now.


Update Navbar, Footer and forum pages

Moved Navbar and Footer to App.jsx
Removed duplicates in forum pages

Other pages should implement this change now.


To add calculator for calculating Biggest Reuleaux Triangle inscribed within a square which is inscribed within a hexagon

Is your feature request related to a problem? Please describe.
To add calculator for calculating Biggest Reuleaux Triangle inscribed within a square which is inscribed within a hexagon

Hi, I want to work on this issue I will start working on it as soon as I get assigned!! I am a part of GSSoC'21.
Kindly assign this issue to me!
Discord Username : Abhijeet Sinha(P)
Discord Tag: #4018


Add method to retrieve currently configured strategies

Q
A

Bug fix?
no

New feature?
yes

BC breaks?
no

Deprecations?
no

Related tickets
none

Documentation
Will supply if appropriate/acceptable feature

License
MIT

What's in this PR?
Adds a method to retrieve the currently configured discovery strategies.
Why?
When testing a consumer, it is useful to clear strategies to simulate a Not Found Exception, i.e. Psr18ClientDiscovery::setStrategies([])' but in order to reinstate the default behaviour of the lib after doing this, strategies would need to be duplicated in consumer tests.
Example Usage

$currentStrategies = Psr18ClientDiscovery::getStrategies();
Psr18ClientDiscovery::setStrategies([]);
try {
  // Test my thing…
  $this->fail('An exception was not thrown');
} catch (ExpectedError $error) {
  $this->makeMoreAssertions();
} finally {
  Psr18ClientDiscovery::setStrategies($currentStrategies);
}

Checklist

[x] Updated CHANGELOG.md to describe new feature
[ ] Documentation pull request created (if not simply a bugfix)

To Do

[x] Tests… Sorry, I'm not familiar with phpspec but would be happy to write a unit tests??

Thanks for the phpspec pointers @dbu - pull duly rebased. Sorry for the noisy commits!


Uncaught Error: tunneling socket could not be established, statusCode=500

[Enter steps to reproduce:]

...
...

Atom: 1.37.0 x64
Electron: 2.0.18
OS: Mac OS X 10.14.5
Thrown From: Atom Core
Stack Trace
Uncaught Error: tunneling socket could not be established, statusCode=500
At internal/streams/legacy.js:59

Error: tunneling socket could not be established, statusCode=500
    at ClientRequest.onConnect (/app.asar/node_modules/tunnel-agent/index.js:166:19)
    at Object.onceWrapper (events.js:319:30)
    at emitThree (events.js:136:13)
    at ClientRequest.emit (events.js:217:7)
    at Socket.socketOnData (_http_client.js:472:11)
    at emitOne (events.js:116:13)
    at Socket.emit (events.js:211:7)
    at addChunk (_stream_readable.js:263:12)
    at readableAddChunk (_stream_readable.js:250:11)
    at Socket.Readable.push (_stream_readable.js:208:10)
    at TCP.onread (net.js:594:20)

Commands
     -3:53.5.0 application:open (atom-workspace.workspace.scrollbars-visible-when-scrolling.theme-one-dark-syntax.theme-one-dark-ui)
  3x -3:37.6.0 core:close (ul.list-inline.tab-bar.inset-panel)
     -3:23.4.0 core:confirm (input.hidden-input)
     -0:30.6.0 core:select-all (input.hidden-input)
  6x -0:29.3.0 core:backspace (input.hidden-input)
  2x -0:24.8.0 core:confirm (input.hidden-input)
     -0:11.1.0 core:select-all (input.hidden-input)
  2x -0:09.5.0 core:backspace (input.hidden-input)
     -0:07.3.0 core:confirm (input.hidden-input)

Non-Core Packages

Thanks for the report - this looks like an error related to the proxy you're using?  If the error is intermittent or only happened once, then there might be a temporary issue with your network or proxy and retrying might help.  If the error is consistently reproducible and you've configured Atom to work with your proxy, then a good place to get help is our official message board.  People in the community that have configured Atom to work with their proxies can chime to share what worked for them and provide suggestions.


Error excluding fields when a class is annotated with @JsonTypeInfo

I have a spring boot service wich returns a complex json with one class annotated with @JsonTypeInfo(use = JsonTypeInfo.Id.MINIMAL_CLASS, include = JsonTypeInfo.As.PROPERTY, property = "@class") because is extended by other two classes.
For example in this json, the attachment attribute:
{
"id": "ISSUE-1",
"issueSummary": "Dragons Need Fed",
"issueDetails": "I need my dragons fed pronto.",
"reporter": {
"firstName": "Daenerys",
"lastName": "Targaryen"
},
"assignee": {
"firstName": "Jorah",
"lastName": "Mormont"
},
"attachment": {
"@class": ".MailAttachment",
"name": null,
"from"<EMAIL_ADDRESS>"to"<EMAIL_ADDRESS>"text": "content"
},
"actions": [
{
"id": null,
"type": "COMMENT",
"text": "I'm going to let Daario get this one..",
"user": {
"firstName": "Jorah",
"lastName": "Mormont"
}
},
{
"id": null,
"type": "CLOSE",
"text": "All set.",
"user": {
"firstName": "Daario",
"lastName": "Naharis"
}
}
],
"properties": {
"priority": "1",
"email"<EMAIL_ADDRESS>}
}
when I send fields with exclusion of one parameter the return response is not the expected. For example with fields "-properties" the response is:
{
"id": "ISSUE-1",
"issueSummary": "Dragons Need Fed",
"issueDetails": "I need my dragons fed pronto.",
"reporter": {
"firstName": "Daenerys",
"lastName": "Targaryen"
},
"assignee": {
"firstName": "Jorah",
"lastName": "Mormont"
},
"attachment": {
"@class": ".MailAttachment"
}
}
after "@class" attribute all attributes are ignored.
Curiously if this attribute is declared as array, we haven`t got this problem. With this json example:
{
"id": "ISSUE-1",
"issueSummary": "Dragons Need Fed",
"issueDetails": "I need my dragons fed pronto.",
"reporter": {
"firstName": "Daenerys",
"lastName": "Targaryen"
},
"assignee": {
"firstName": "Jorah",
"lastName": "Mormont"
},
"attachment": [
{
"@class": ".MailAttachment",
"name": "Subject",
"from"<EMAIL_ADDRESS>"to"<EMAIL_ADDRESS>"text": "content"
}
],
"actions": [
{
"id": null,
"type": "COMMENT",
"text": "I'm going to let Daario get this one..",
"user": {
"firstName": "Jorah",
"lastName": "Mormont"
}
},
{
"id": null,
"type": "CLOSE",
"text": "All set.",
"user": {
"firstName": "Daario",
"lastName": "Naharis"
}
}
],
"properties": {
"priority": "1",
"email"<EMAIL_ADDRESS>}
}
the response with fields "-properties" returns the correct answer:
{
"id": "ISSUE-1",
"issueSummary": "Dragons Need Fed",
"issueDetails": "I need my dragons fed pronto.",
"reporter": {
"firstName": "Daenerys",
"lastName": "Targaryen"
},
"assignee": {
"firstName": "Jorah",
"lastName": "Mormont"
},
"attachment": [
{
"@class": ".MailAttachment",
"name": "Subject",
"from"<EMAIL_ADDRESS>"to"<EMAIL_ADDRESS>"text": "content"
}
],
"actions": [
{
"id": null,
"type": "COMMENT",
"text": "I'm going to let Daario get this one.."
},
{
"id": null,
"type": "CLOSE",
"text": "All set."
}
]
}
spring-boot-JsonTypeInfo.zip

@jgacp
First, I want to thank you for providing a working example right away.  It saves me a ton of time!
Now to your issue.  When you have the filter "-properties", Squiggly actually expands the filter to "base,-properties".  There is code that determines which fields are considered "base" fields.  It looks like there is an issue with this logic, that I'll have to address.
In the meantime, you can work around this issue if you don't care about Squiggly Property Views.   Instead of using the filter "-properties", you can use the filter "**,-properties".

Thank you @bohnman
I tested the workaround and in the working example is right! but in my real project I have the same problem sending fields with "**,", I dont know why yet. I will test in the next days.

@jgacp Are you using nested filters like **,someProperty[-someValue]?

@bohnman in this case I am using fields=**,-customer.contactMedia

@jgacp You have me stumped!  I can't see to reproduce the issue.  I welcome your insight!

Hello @bohnman,
I tested with my real interface and I send you one example with filter ",-customer.conctactMedia" problem.
You can invoke using the next url "http://localhost:8080/test?fields=,-customer.contactMedia"
Thanks!!
spring-boot.zip

Hi @bohnman,
The problem is not in your code, the problem is when JsonGenerator is instanciate. In case of attributes in "attachment", the sentence "getOutputContext().getParent().getCurrentValue()" return "MailAttachment" instead of "Issue".
Do you know if I could set objectMapper to instanciate the correct JsonGenerator?

@jgacp Not that I know of.  However, I do have a fix in mind.  I was hoping to give you a workaround in the mean time.

Thank you @bohnman I wait for news

Hi, we've got a simillar issue. Any news on this??


Change over library playground to be an ansible built machine

What maintenance needs to be done?
Currently the library playground is an Research computing built machine
We should change over to utilize an ansible built server for this.  We will need help from Research Computing to see what we need to change with our build including how to setup the datastores, and anything else/
This work cannot be undertaken until we get license files (https://github.com/pulibrary/tigerdata-app/issues/966).
Level of urgency

[ ] High
[x] Moderate
[ ] Low

Why is this maintenance needed?
We need to converge
Acceptance criteria
Implementation notes, if any

Hey team! Please add your planning poker estimate with Zenhub @bess @carolyncole @JaymeeH @jrgriffiniii @leefaisonr


[More Options] NOT changing to Yellow when Hover!

Description
The [More Options] NOT changing to Yellow when Hover, that is when the user puts their cursor on that portion.
I have worked on this problem and would like to solve this.
Kindly assign this issue.
Screenshots
Attaching a Screenshot for reference.

After Results.

Go ahead !

Can you assign this issue to me? #643

Hi, i'd like to work on this issue
Please assign it to me

Can you assign this issue to me?

can you assign me this issue


Apply spell/style check to top level markdown files

Spell/style check examples.md
Spell/style check overview.md
Spell/style check quickstart.mdx
Spell/style check tour.mdx
This PR fixes https://github.com/restatedev/documentation/issues/142.

Thanks a lot for your review @gvdongen. I've addressed your comments. Merging this PR now.


Found an issues with Functions.T4 when genericArg was null

When genericArg was null there were errors.
Also when return type is int then we need the FirstorDefault() when query is SqlQueryRawAsync

i'll do some testing on this on monday and merge provided the tests pass.
Thanks!

your change is in build 1.3.6

MarkI’ve found that the return grows generally for table functions in sql aren’t being created properly.  I’d like to debug the code but don’t know how to debug the “dot net “ command line.  I’ve setup a test project under the “tests” folder and can run tests on code tests but how do I debug the command lines?Sent from my iPhoneOn 20 Dec 2023, at 5:12 am, Mark Randell @.***> wrote:
I merged your pull request BUT had to roll back your FirstOrDefault addition because it broke the code.  if you have a special function that only  works with FirstOrDefault, provide a sample to me and I will add support for it and ensure it does not break other scenarios
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

I have tests with table functions.  Provide a sample of one that is not working for you and I'll get it fixed up.

Table return function like this
I have removed the inner code but you get the drift
CREATE FUNCTION [dbo].[fnt_RaceGroupEntryCategoryAncestorHierarchy]
(
   @EntityId uniqueidentifier

)
RETURNS @T table(id uniqueidentifier, Name nvarchar(80), ParentEntityId
uniqueidentifier, IsEntrantCategory bit, IsRaceDivisionCategory bit,
IsResultsCategory bit, Level int )
AS
BEGIN
END
GO
From: Mark Randell @.>
Sent: Saturday, December 23, 2023 12:50 AM
To: R4ND3LL/EntityFrameworkRuler @.>
Cc: John @.>; Author <
@.>
Subject: Re: [R4ND3LL/EntityFrameworkRuler] Found an issues with
Functions.T4 when genericArg was null (PR #31)
I have tests with table functions. Provide a sample of one that is not
working for you and I'll get it fixed up.
—
Reply to this email directly, view it on GitHub
https://github.com/R4ND3LL/EntityFrameworkRuler/pull/31#issuecomment-1867710686,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AESHI4KIC4JKWXG477E2TYDYKWFYZAVCNFSM6AAAAABAVWWFROVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNRXG4YTANRYGY
.
You are receiving this because you authored the thread.[image: Image
removed by sender.]Message ID: <R4ND3LL/EntityFrameworkRuler/pull/31/
@.***>


Do not synthesise DISCONNECT on session change.

v12 clients can handle session change based on identity.

@paulwe @davidzhao this is ready for a second look.

LGTM!
I think fullReconnect e2e should pass now.

Yes it does running locally.


[PyTorch] Fix for type checking failure on custom callables

Fix for TE checkpoint incorrectly passing custom classes/callables to the native PyTorch checkpoint.

/te-ci pytorch


Unhandled Promise Rejection when creating HostedFields with authorization

General information

SDK version: 3.90.0
Environment: development
Browser and OS: Safari 186<IP_ADDRESS>.5, MacOS 13.2 (22D49)

Issue description
The following code won't catch the authentication error due to bad client token:
hostedFields.create({ authorization, fields, styles })
  .catch(console.error)

It will instead resolve the promise with the following error in the browser console:
Unhandled Promise Rejection: BraintreeError: Authorization is invalid. Make sure your client token or tokenization key is valid.

Instantiating the client will catch a BraintreeError as expected so I guess this rejection is lost somewhere.
Is this an already known issue? I may be able to send a PR with a fix, provided this is somehow a feature not a bug 😃.

Hi @josemigallas,
If something causes the client creation to error out (such as a malformed authorization string), Hosted Fields will appear to set up just fine, but you will see an unhandled promise rejection in the console and any method calls that rely on the client will fail with that same error.
If we adjusted the code to bubble up the error from the client creation into the creation of Hosted Fields, it would take away all the speed gains of allowing the Hosted Fields inputs to render and be ready for a customer to enter their card details before the client is actually available.
The unhandled error in the console is a known side effect to pushing the underlying client creation to an asynchronous process. We're aware of it, but the times it actually happens in production are fairly rare. You have to do something like 'intentionally mess up the authorization string' to trigger it on demand.
Let me know if you have more questions on this.

If there is more that is required on this ticket, then we can either reopen this back or create a new ticket.
Marking this as closed. Thank you.


SW-4387 Add Sub-locations to Add inventory flow

Looks clean straightforward. Can you give the mobile view of sublocations dropdown full width?

Looks clean straightforward. Can you give the mobile view of sublocations dropdown full width?

Oh good catch! Thanks


Is there a way of having only the group shown in the multiselect box rather than all the options within that group?

I'm using the multi-select for some groups. However some of these groups contain loads of options so when I add them, the text box is filled with options and fills the entire screen. Is there a way I get it to show only the group name and allow you to hit the x button to remove that entire group.

That’s probably not possible unless you extend the component and provide your own template that supports that.


Add negative tests for cast decimal with zero precision

@Agirish could you please review this negative tests?


Bug fix: callback two times in bizkeeper handler.

Coverage decreased (-0.07%) to 62.538% when pulling 5acb6ae8c2f01aae9faa2fcdcf5e13e4254e2b5b on little-cui:master into 02bb75d9d727f2596cc2bc26202f66ccf4c40023 on ServiceComb:master.


multiple element addition on multiple clicks

Fixes #
Describe the changes you have made in this PR -
Screenshots of the changes (If any) -
Note: Please check Allow edits from maintainers. if you would like us to assist in the PR.

Coverage remained the same at 84.379% when pulling 1ac69c361e83a9a118c804951ee190f93e7a7eec on abstrekt:multiadd into 4fca4d3b5bf5d906850c1ebe07145afe99c37bd5 on CircuitVerse:new_UI.


Investigate int Welcome experience

Ideas for exploration:

[ ] Overlay pointing out the main parts of the workbench
[ ] Welcome editor with "Getting Started" actions
[ ] Welcome editor with short introduction and interactive samples

I would also like to suggest that there should be a way to disable the welcome experience for a fresh install before first launch.
I'm helping a team who wants to use VS Code to debug code on machines that get re-built from machine images daily and made a PowerShell script for them that automates the process of installing VS Code.  One of their complaints is that the first launch always launches the browser with the VS Code homepage.
@joaomoreno  gave me a workaround for that, but it'd be nice to have a command line flag like -skip-welcome which would allow the caller to skip the first welcome and make it not appear in subsequent launches, especially if the welcome experience becomes more elaborate.
All that said, I think having a more interactive welcome experience is a great idea!

@daviwil What is the workaround you use to avoid the webpage opening?

Closing the plan item, no more work is planned on this for January.


play_motion: 0.4.8-1 in 'melodic/distribution.yaml' [bloom]

Increasing version of package(s) in repository play_motion to 0.4.8-1:

upstream repository: https://github.com/pal-robotics/play_motion.git
release repository: https://github.com/pal-gbp/play_motion-release2.git
distro file: melodic/distribution.yaml
bloom version: 0.9.0
previous version for package: null

play_motion
* Fixes for shadowed variables
* Contributors: Jordan Palacios

play_motion_msgs

No changes

I'm going to merge this but we strongly recommend also having doc and source entries for better indexing.


WORMHOLE token in MetaMask - Bridging Back to Phantom? Please Help.

Good Morning,
I was hoping someone can help me with an issue I ran into last night. I was attempting to bridge my SOL in my Phantom wallet over to my MetaMask on the ETH network.
It went through, but it is now WORMHOLE. I have WORMHOLE in my MetaMask, but when I try to bridge that back, it prompts me to create an associated token account. I see my wallet connected, but the tokens are being sent to an address I don't recognize.
I'm a bit lost now. I would just like to get the SOL I sent to my MetaMask back to my Phantom wallet.
Any help would be much appreciated.

You will have re-access both phantom wallet and metamask via the wormhole node on https://www.wormholebridge.org/
It will reapprove the stacked transaction

You will have re-access both phantom wallet and metamask via the wormhole node on https://www.wormholebridge.org/
It will reapprove the stacked transaction

Good Morning,
I was hoping someone can help me with an issue I ran into last night. I was attempting to bridge my SOL in my Phantom wallet over to my MetaMask on the ETH network.
It went through, but it is now WORMHOLE. I have WORMHOLE in my MetaMask, but when I try to bridge that back, it prompts me to create an associated token account. I see my wallet connected, but the tokens are being sent to an address I don't recognize.
I'm a bit lost now. I would just like to get the SOL I sent to my MetaMask back to my Phantom wallet.
Any help would be much appreciated.

You will have re-access both phantom wallet and metamask via the wormhole node on https://www.wormholebridge.org/ It will reapprove the stacked transaction

Good Morning,
I was hoping someone can help me with an issue I ran into last night. I was attempting to bridge my SOL in my Phantom wallet over to my MetaMask on the ETH network.
It went through, but it is now WORMHOLE. I have WORMHOLE in my MetaMask, but when I try to bridge that back, it prompts me to create an associated token account. I see my wallet connected, but the tokens are being sent to an address I don't recognize.
I'm a bit lost now. I would just like to get the SOL I sent to my MetaMask back to my Phantom wallet.
Any help would be much appreciated.


weex doctor error

My mac installed xcode 10 and it runs well every day.
But when I run
weex doctor
I get this error:
[!] iOS toolchain - develop for iOS devices
•  Xcode at /Library/Developer/CommandLineTools
✗  Weex requires a minimum Xcode version of 9.0.0.
      Download the latest version or update via the Mac App Store.
✗  Xcode requires additional components to be installed in order to run.
  '
      Launch Xcode and install additional required components when prompted.
•  CocoaPods version 1.6.1

✔ Weex Cli Environment (v2.0.0-beta.2, on darwin 18.2.0):
weex -v
2.0.0-beta.2

@weex-cli/generator : v2.0.0-beta.3
@weex-cli/compile : v2.0.0-beta.2
@weex-cli/preview : v2.0.0-beta.2
@weex-cli/run : v2.0.0-beta.2
@weex-cli/device : v2.0.0-beta.2
@weex-cli/doctor : v2.0.0-beta.2

@myeveryheart Is the weex run ios command works fine?

@myeveryheart Is the weex run ios command works fine?

not ok
weex run ios
xcrun: error: unable to find utility "instruments", not a developer tool or in PATH
[✔] Compile JSBundle done
xcrun: error: unable to find utility "instruments", not a developer tool or in PATH
? Select one of the device (Use arrow keys)

It can not find my simulaters.

This is my xcode problem, I have fixed. Thank you for your time.


Portainer goes down when using Docker API

Bug description
I'm using a web proxy with nginx from evertramos.
So, i connected to my docker env using TLS config Domain:Port (not IP, i used domain because IP is dynamic)
As example i will use "example.com:1234"
After i add a container ("nginx:alpine in my case) which points to my domain ("example.com"), the Portainer endpoint is no more accesible, is down.
Expected behavior
Portained endpoints will be accesible in order to show all my running images, volumesc, etc
Steps to reproduce the issue:

Connect to a docker machine using Docker API using Domain:Port.
Add nginx container which points to used domain and deploy it.
Refresh portainer

Technical details:

Portainer version: 1.22.1
Docker version (managed by Portainer): 19.03.4, build 9013bf583a
Platform (windows/linux): linux (ubuntu)
Command used to start Portainer (docker run -p 9000:9000 portainer/portainer): none from my side, it's starting automatically from script
Browser: Chrome 78.0.3904.97

Additional context

Hi there, we need more info to debug this. Can you please provide the logs of the Portainer container?

Hi!
I will provide the logs, but i think i can't do it soon. When i will have some spare time, i will do it. Until then, you can leave it this.
Also, about this bug, if i use real ip adress (like 123.456.789.098), Portainer is working. As i remember, in Chrome console, "ping" method is throwing 502 when i use domain ("example.com" which is pointing to 123.456.789.098 as example).

As a debugging step I would check from your Portainer host machine that the new domain name is externally reachable, if it is not, Portainer will not be able to manage the docker environment and the endpoint will show as down

You could try resolving the address from a busybox docker container to make sure that docker networking is ok also


Problem with valuePrimitive

Description
Bug is using formly/kendo newest version

"@ngx-formly/core": "^6.0.4",
"@ngx-formly/kendo": "^6.0.4",

The error was

grid-context-menu.component.ts:133 ERROR Error: Expected value of primitive type. See http://www.telerik.com/kendo-angular-ui/components/dropdowns/dropdownlist/#toc-value-selection
at DropDownListComponent.verifySettings (kendo-angular-dropdowns.js:6383:19)
at DropDownListComponent.ngAfterContentChecked (kendo-angular-dropdowns.js:5968:14)

Minimal Reproduction
https://stackblitz.com/edit/ngx-formly-ui-kendo-nkzbgm?file=src/app/app.component.ts
Your Environment

Angular version: 14.0.0
Formly version:
"@ngx-formly/core": "^6.0.4",
"@ngx-formly/kendo": "^6.0.4",

Additional context
additionaly, in my project situation is worse because for me even dropdown is not shown. but in my project dwopdown is inside dialog opened by dialogService,.
i cannot make the same example with dialog in stackblitz because of kendo license
whole package.json and angular.json is in stackblitz
difference between stackblitz is 3 errors on it

on my private its continuously throwing the errors about valuePrimitive and crash browser ( few thousands in a minute )

I don't understand why valuePrimitive is fixed in true and not a prop do configure.
For my forms the ideal is to use valuePrimitive = false and use the object as the value
How can I configure the whole object as the value?


Rename stuff

Renames stuff. Merge conflicts guaranteed.

Uh oh, something went wrong with the build. Need to check on that

Conflicts do not look too bad. Current failure in Jenkins is due to unit test config and could be considered a non-issue.
We'll probably want to prioritize this PR and get it merged ASAP to avoid a brewing conflict-storm - it may not be too bad right now but merge half a dozen other PRs before this one? Fun times a-comin

it may not be too bad right now
O good sir, try and fix the merge conflicts before you say that :P

Closing, on account of #257


[Request] Flamethrower output become adjustable

I often use flamethrower to combat too many enemies and feel hard to handle this a little.
Charcoals, metal ingots, or others from flame hit blocks frequently waste my inventry.
Therefore I want that flamethrower launching power be adjustable, notably both of High heat mode(can smelt, high damage and high gas consume) and Low heat mode(cannot smelt, low damage and low gas consume).

I actually love this idea- I'll try and get this done for the next minor release.

So... Mode names:
Mildly Warm, Scalding Hot, Burnt to Cinders
XD

Mode names is okay like as high heat and low heat. Simple name is easy to use.

i would say there needs to be a option to do smelt or not smelt..
it should only damage mobs on not smelt..  on smelt it damages and does damage to area IE lights trees on fire and smelts

I'm thinking "Combat," which only does damage to mobs, "Heat", which deals mob damage and smelts items, and "Inferno", which does deals damage, smelts, and lights fires.

What if heat can light fires on flammable materials, but inferno can ignite anything?

your idea of comabt heat and inferno is good! it would make me want to use it as a normal weapon!!!
One question could it be enchanted ? with fortune or beheading?

Happy release 8.1.8! thankyou

How to change the modes? I tried to use Armor mode switch and Item mode switch, alone and combined with shift-key, but it still stuck in Combat mode.


New Feature asynchronous output window -> Would you accept such PR?

Suggestions
Feature to have an asynchronous output for logs / long running tasks
Split window in two or three parts
|-----------------------------------------------------------|

------ output part -> configurable in height --------------

------ last user input (after return) ---------------------

-----------------------------------------------------------

------------------------ prompt ---------------------------

-----------------------------------------------------------

Questions
I'd really need this feature and wanted to ask if you're going to accept a patch for this?
Information

I would go to implement this as Option / selectable feature to have no impact to the actual functionality
This would be done mostly inside the render

If you would include this feature, is there anything that you would like me to include or guidelines where / how the usage should be implemnted?
BR
Matthias

Thank you for your suggestion and sorry for late reply. I've never found the solution about asynchronous output, but it sounds your suggestion makes sense.

If you would include this feature, is there anything that you would like me to include or guidelines where / how the usage should be implemented?

It seems the implementation is not easy. I'll answer this question after tidy up my thoughts.

I've created a PR for this: https://github.com/c-bata/go-prompt/pull/148


 support

We should support images.
It'd be a lot easier to do after the toImageSync stuff can land - see https://github.com/flutter/flutter/issues/77289 and linked issues

some questions: what format do we support for inline images? We could just do the bytes, no need for base64 since we're not text.
How do we decode these images? Do we make sure they end up in the flutter image cache? if so, do we need to give them some sort of stable identifier?
When rendering a vg, do we delay rendering until image decoding has finished?

I think inline images should just be the raw bytes of the original (encoded) image, which will typically be zlib compressed png. It should not be base 64 encoded.
This is the harder part. flutter_svg did not put the min the Flutter Image cache and this caused some confusion for customers who were memory sensitive. Instead it ended up in the picture cache with the SVG itself, which VG doesn't have. I think using the image cache would make sense, and we could come up with an ID based on the asset name or network URL of the buffer itself, plus the index of the image in the buffer.
I think we delay until image is available. We're not doing progressive rendering anywhere else for this, and right now we just atomically draw the product as a Picture which can't be modified after the fact.

We could also make a cache specifically for vector_graphics, but instead of a global cache more of a Weakreference-based best effort cache. i.e. if you have N vgs with the same image on the page we decode once, but if you leave the screen it still gets cleared. WDUT?

I guess that saves an ImageCache slot for some image the user realizes they're rendering?
I'm a little worried that if we start implementing caching here we'll just get it wrong for some use case and instead should just have a generic way of plugging in a cache for images/pictures - and for images, it should probably just use the Flutter ImageCache interface for people who want to reuse that.

I just don't know how we'd key the image cache entires - I'd be worried about collision at the least

`final String key = '$vectorGraphicKey_$indexOfImage`` right?
That has the potential to add duplicate entries if the same image is used across different vector graphics but oh well

ship it

Now that we're doing eager rasterization, i don't think we need to populate the image cache for inline images (network/asset is another story).


Use a GitHub Action Service for MySQL

I think this would solve the auth issue with Trilogy. If not we can alter the user to no longer use caching_sha2.

It;s passing now: https://github.com/byroot/scenic-mysql_adapter/actions/runs/10516179187/job/29137828348

Thanks!


Cool Feature

Description

Impact

Additional information

[ ] I'm willing to contribute to this feature

closed per demo purpose


How to create a component with customizable animation?

Some context: I have a component which is animated when it mounts. The component's animation can be changed by giving it props animation and animationStyle. The component also has default animation. I tried defaultProps but it's not possible use state there.
Example component which illustrates what I'm trying to do:
var React = require('react-native');

var {
  Animated,
  StyleSheet,
  Text,
  View,
} = React;

var styles = StyleSheet.create({
  container: {
    flex: 1,
    justifyContent: 'center',
    alignItems: 'center'
  }
});

class Circle extends React.Component {

  componentDidMount() {
    // Run animation immediately after mounting
    this.props.animation.bind(this)();
  }

  render() {
    return (
      <Animated.View style={[styles.container, this.props.animationStyle]}>

      </Animated.View>
    );
  }
};

// This is in quite strange place, outside of the component
var circleAnimation = function() {
  this.state.bounceValue.setValue(.5);
  Animated.spring(this.state.bounceValue, {
    toValue: 1,
    friction: 3,
  }).start();
}

Circle.propTypes = {
  animation: React.PropTypes.func,
  animationStyle: React.PropTypes.object
};

Circle.defaultProps = {
  animation: circleAnimation, // Animation function
  animationStyle: {transform: [{scale: this.state.bounceValue}]} // Fail...
};

@jozan this still an issue for you?
You can't reference this.state in default props since defaultProps object is used for every instance of something.
If you need specific help, well let me know on slack :)
@brentvatne

Thanks for the answer! I wanted to know how to customize animation (f.e. easings) but I gave up on this since it was only an experiment. I'll close this for now.
:baby_chick:


RuboCop extension suggestion not easily applicable

In a repo I work on I got the following suggestion of extensions
Tip: Based on detected gems, the following RuboCop extension libraries might be helpful:
  * rubocop-factory_bot (https://rubygems.org/gems/rubocop-factory_bot)
  * rubocop-rspec_rails (https://rubygems.org/gems/rubocop-rspec_rails)

If like for other rubocop extensions (we have rubocop-performance and rubocop-rails), I just add rubocop-rspec_rails to Gemfile and require it in .rubocop.yml, running rubocop fails with
cannot load such file -- rubocop/rspec/language/node_pattern

After reading the README, that was caused by not having rubocop-rspec. Adding this gem made it work, but I saw, that rubocop-rspec  already has rubocop-rspec_rails as dependency. Wouldn't it make more sense to suggest rubocop-rspec as an extension then? Or should rubocop-rspec_rails depend on rubocop-rspec? I find it also a bit confusing that https://github.com/rubocop/rubocop-rspec does not mention the extracted cops in the README at all.
> bundle exec rubocop -V
1.63.5 (using Parser <IP_ADDRESS>, rubocop-ast 1.31.3, running on ruby 3.3.1) [arm64-darwin23]
  - rubocop-performance 1.21.0
  - rubocop-rails 2.24.1
  - rubocop-rspec_rails 2.28.3

Thanks for reporting
Can you please provide a full backtrace for “cannot load such file”?
rubocop-rspec should be suggested if you have rspec in your gemfile. I think it’s a deficiency if it doesn’t when only rspec-rails is in the gemfile. Would you like to contribute this (to the main rubocop repo)?

Fixed in https://github.com/rubocop/rubocop/pull/12908#pullrequestreview-2062407567
So far you ought to require rubocop-rspec from your .rubocop.yml for everything to function properly.
We can’t make an explicit dependency on rubocop-rspec, as rubygems complains about a circular dependency.
We plan to untangle the two gems completely in the next major versio s, really soon, hold on!
And please accept my apologies for the inconvenience in the meantime.


Method strtolower break case sensitive aliases

@weierophinney you was right with breakage. Simple case from Magento 2.3. Host is not always domain or IP address. When we using docker we can call host via container names.
Example:
host = MAGENTO2_varnish
When we call strtolower then we getting error:
[2019-03-12 11:10:23] main.CRITICAL: Unable to connect to magento2_varnish:80 . Error #0: stream_socket_client(): unable to connect to magento2_varnish:80 (php_network_getaddresses: getaddrinfo failed: Name or service not known) {"method":"GET","url":"http:/","invalidateInfo":{"server":"[object] (Zend\\Uri\\Uri: http://magento2_varnish:80/)","tagsPattern":".*"}} []
So MAGENTO2_varnish != magento2_varnish.
If anyone ( like me ) using aliases on magento then this patch break eg. Varnish Purge request.
Originally posted by @bmxmale in https://github.com/zendframework/zend-uri/pull/29#issuecomment-471958647

@bmxmale What is the context for your usage of zend-uri? Is it with the HTTP client? or an incoming request? Or something else?

@weierophinney  I'm just community contributor of magento but will try to explain it in simple way.
This case come from PurgeCache event when we are using Varnish Cache as FPC.
On magento we need to set http_cache_host config to tell Magento where it should send purge request.
Magento config with Varnish host ( used on docker env )
return [
    'http_cache_hosts' => [
        [
            'host' => 'MAGENTO2_varnish'
        ]
    ]
];

Here you can check this option on magento devdocs.
Docker compose used to:
version: '2'
services:
  magento2_nginx:
    container_name: MAGENTO2_nginx
    image: nginx:stable
    volumes:
      - ../magento/magento-2.3:/srv/magento2.3
      - ./mount/nginx/magento.conf:/etc/nginx/conf.d/default.conf
    network_mode: "DOCKER_network"
  magento2_php:
    container_name: MAGENTO2_php
    image: bmxmale/magento2-php:2.3
    volumes_from:
      - magento2_nginx
    network_mode: "DOCKER_network"
  magento2_varnish:
      container_name: MAGENTO2_varnish
      image: bmxmale/docker-magento-varnish:5.2.1
      volumes:
        - ./mount/varnish/varnish.vcl:/etc/varnish/default.vcl
      environment:
        - DEFAULT_VCL=0
      network_mode: "DOCKER_network"
      labels:
        - "traefik.backend=MAGENTO2_varnish"
        - "traefik.port=80"
        - "traefik.frontend.rule=Host:magento.udviklet.dk"
        - "traefik.docker.network=DOCKER_network"

Step by step

When we Refresh/Flush cache on magento, it will send purge requst to varnish server.
Magento\CacheInvalidate\Model\PurgeCache::sendPurgeRequest

Now sendPurgeRequest method call getUris method from cacheServer class. Magento\PageCache\Model\Cache\Server::getUris

Here getUris method create UriFactory to create array of servers to send purge request. Zend\Uri\UriFactory::factory

Method UriFactory create new Uri object. Zend\Uri\Uri::setHost

Method setHost with param $host = 'MAGENTO2_varnish' execute strtolower.
As result $host variable will contain magento2_varnish

I'm open to call with you to answer any other questions.
With best regards, Mateusz

@weierophinney any update on this?
In my opinion we have 2 options:

revert changes from https://github.com/zendframework/zend-uri/pull/29
add statement to check if host is valid domain or ip

Please select one or suggest better solution and I can prepare PR for this.

This repository has been closed and moved to laminas/laminas-uri; a new issue has been opened at https://github.com/laminas/laminas-uri/issues/1.


MINIFICPP-791: Change to v2.8.3

Thank you for submitting a contribution to Apache NiFi - MiNiFi C++.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with MINIFICPP-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically master)?

[ ] Is your initial contribution a single, squashed commit?

For code changes:

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file?
[ ] If applicable, have you updated the NOTICE file?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Travis failure is apache rat related

@apiri  the impetus for this is https://github.com/libressl-portable/portable/issues/512
Though I submitted a fix in their related project we can't rely on the 2.9.0 tag any longer.

Verified builds.  Change looks good.  Will merge.


required python and CUDA version

Please confirm the

python version
CUDA version

We have confirmed that the code runs under the following conditions.
Python 3.7.16
CUDA 11.7
GPU 3090
Thank you for your asking!


Giving less accuracy while running predict.py

Hi davidsandberg,
while running predict.py i got less accuracy like this --

please resolve my issue why its given less accuracy as early as possible.
Thanks in Advance

Hi,using predict.py needs classifier.pkl.But I don’t konw how to get this file. Could you told me how to get it? Thank you.

Hi@Tom Hu,
First you need to create .pkl file for your dataset, for that run classifer.py in train mode with your datase thent you will get pkl file.
python src/classifier.py TRAIN /home/david/datasets/lfw/lfw_mtcnnalign_160 /home/david/models/model-20170216-091149.pb ~/models/lfw_classifier.pkl --batch_size 1000 --min_nrof_images_per_class 40 --nrof_train_images_per_class 35 --use_split_dataset
Thanks@Regards
Bindu

@himabinduyeddala  Hi, How do you get the file 20170216-091149.pb and lfw_classifier.pkl??

Hi @moonyoung430 ,
For creating .pb file you need to run freezegraph.py &
for creating .pkl file you need to run classifier.py with mode= TRAIN.
Best
Himabindu

@himabinduyeddala Hi,I run classifer.py and created .pkl, downloaded the pretrain model 20170512--110547,then run predict.py .
I met the same problem with you, and I guess the problem is classifer,I will check classifier.py


client(feat): add an accessor for the request body

Allow users to access the body of the request.
Useful when one wants to modify the request based
on the data in the body, e.g. to add checksum headers.

[x] The commit messages match the guidelines in https://github.com/hyperium/hyper/blob/master/CONTRIBUTING.md#git-commit-guidelines

Thanks for contributing! Unfortunately, I'm here to tell you there were the following style issues with your Pull Request:

Commit: 0ef8760ca393b19622fd3b8a80bdadcb21d8efe9
Invalid type. Valid types are feat, fix, docs, style, refactor, perf, test, chore, revert

Guidelines are available at https://github.com/hyperium/hyper/blob/master/CONTRIBUTING.md

This message was auto-generated by https://gitcop.com

Thanks for the PR! I'm curious, what sort of code do you imagine using this?

I'm using it to, amongst other things, write a function on Request that computes and injects a Content-Length header if one doesn't already exist.  I anticipate it being useful for other such code that wants to derive some information from the body.

Well, it mirrors the other setters and getters. Depending on the body type, it may be possible to inspect or not. Thanks!


Option 1: Install with Helm via helm template wrong option

I was just trying to install istio with the info provided here
I tried to run the help template command but I get the follow error:
helm template istio.io/istio --name istio-init --namespace istio-system Error: unknown flag: --name
It seems that the --name option is not anymore available. Therefore what would be an equivalent option, if available?
Output of helm version:
version.BuildInfo{Version:"v3.2.4", GitCommit:"0ad800ef43d3b826f31a5ad8dfbb4fe05d143688", GitTreeState:"clean", GoVersion:"go1.13.12"}
Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.6", GitCommit:"dff82dc0de47299ab66c83c626e08b245ab19037", GitTreeState:"clean", BuildDate:"2020-07-15T16:58:53Z", GoVersion:"go1.13.9", Compiler:"gc", Platform:"linux/amd64"} Server Version: version.Info{Major:"1", Minor:"14+", GitVersion:"v1.14.9-eks-49202c", GitCommit:"49202cb0a6d5e481319ba21f0b068f112a29a4e5", GitTreeState:"clean", BuildDate:"2020-05-23T20:16:40Z", GoVersion:"go1.12.12", Compiler:"gc", Platform:"linux/amd64"}
Cloud Provider/Platform (AKS, GKE, Minikube etc.):
AWS

The Istio docs you linked are for Helm v2 (without and with Tiller). Helm v2 uses this syntax: helm template [flags] CHART
In Helm v3, the syntax looks like this: helm template [NAME] [CHART] [flags] - see https://helm.sh/docs/helm/helm_template/#helm-template for more detail.

Hi, @zioalex - I hope those docs I linked helped you solve this. Please let us know if the problem persists.


[SPARK-14700] [Python] adding SQL Row equality and inequality overrides

What changes were proposed in this pull request?
This PR adds equality and inequality overrides to pyspark.sql.Row. Without this override, Row simply uses the equality operator of tuple, which doesn't consider the private member __fields__ of Row to determine the implicit ordering needed to compare the tuples correctly.
Without this PR, we get seemingly illogical equality checks like:
Row(a=1) == Row(b=1) # True
r1 = Row('b', 'a')(2, 1) # Row(b=2, a=1)
r1 == Row(b=2, a=1) # False
r1 == Row(a=2, b=1) # True

How was this patch tested?
Unit tests were added to pyspark/sql/types.py in the Row docstring. A handful of newly failing tests in pyspark/sql/tests.py were fixed.
This PR is my original work and I license the work to the project under the project's open source license.
@davies @rxin Mind having a look?

Can one of the admins verify this patch?

@JasonMWhite you said in the jira ticket that there were a few bugs exposed. Can you submit those too?
I talked more with @davies offline. I think the main problem here is that column names can have duplicates, and in those cases it is difficult to define what the correct behavior is if the equality check is column-order-agnostic.

The "few bugs" are included in this PR, all column naming issues. They're in python/pyspark/sql/tests.py
Could you provide an example about what could cause these column name duplications? Is it something like df.select('a', 'a'), post-join behaviour, or something else?

Hmm, I see what you mean.
r1 = df.select('a', 'b', df['b'].alias('a')).collect()[0]
r1 # Row(a=1, b=2, a=2)

What does that even mean? And what should r1['a'] return? Throw an exception for the ambiguity, perhaps? If so, how would the user disambiguate the columns again?

Can they do it by position?

After some experimentation on the Scala side, I see that this PR would be a significant departure from Scala's interface, where the schema and field names are not factors in equality checks at all. Column ordering is all that matters. That seems odd to me, but perhaps that changes in 2.0.
There's a couple of other departures from Scala-style Rows:

Accessing an ambiguous column doesn't throw an error
You can't check row equality like Row(1, 2) == Row(b=1, a=2)

This second issue makes unit tests a little counterintuitive. The use of kwargs at all, and making Row look dict-ish, might be an issue. I'll create separate issues for these. Closing PR.


CUDA error: invalid configuration argument on large

Hi,
I'm trying to reproduce the pretraining results with multi-GPU training, and encountered this error: RuntimeError: CUDA error: invalid configuration argument during validation, but only when the validation batch size is large, e.g. 2048, not when the validation batch size is, say, 512. Have you encountered this before, or know how to fix it?
Thank you.

Sorry... It's my first time to see this issue. Could you try using a PyTorch version of 1.2.0 (or higher)? If it still happens, I might not know the answer TAT...

Thanks, will try.


Adding more alternatives for "value" prop in the TextField component

Is your feature request related to a problem? Please describe.
In our project, we use the TextField EDS component to select the number of iterations for an algorithm. This should only be a positive number >= 1. Right now we have to do validation on the backend that the number is >= 1. It would be nice to have this validation directly in the TextField component
Describe the solution you'd like
Adding more options for the value prop in TextField:
For example

positiveNumber
strictlyPositiveNumber

Describe alternatives you've considered
Alternative:  do number validation on the backend
Additional context

How about you get access to the input directly? 😁
We currently have a big change internally on streamlining how we handle our inputs coming up! #2336 The InputWrapper (as its called now) was made for internal use, but we might expose it publicly or have an easier way of implementing some similar feature like this.
We'll have a look on how validation and such as this might be solved when we land our feet on the InputWrapper

@oddvernes will check up what the status of this

Front end validation should be handled outside our components, there are a couple examples in the storybook
Closing this issue


innerHTML error

i run example and it was ok
than i changed options to data got from database and i get this error
treeselectjs.mjs.js:1 Uncaught TypeError: Cannot set properties of null (setting 'innerHTML')     at l (treeselectjs.mjs.js:1:2410)     at Tt (treeselectjs.mjs.js:1:13692)     at treeselectjs.mjs.js:1:13096     at Array.forEach (<anonymous>)     at St (treeselectjs.mjs.js:1:12989)     at Mt.updateValue (treeselectjs.mjs.js:1:14884)     at new Mt (treeselectjs.mjs.js:1:16902)     at me.Rt (treeselectjs.mjs.js:1:29632)     at me.mount (treeselectjs.mjs.js:1:25214)     at new me (treeselectjs.mjs.js:1:29418)
what is this for?

Hi @mbashirim, can you provide a part of the data (options) you are trying to set from db, I will try to use this data and see an error in the not minimized file.


Interpolate environment variables in .cfg config files

Description
Added possibility to refer to environment variables in the config files using ${variable_name} syntax.
In Python 3 the feature is implemented by changing the default interpolation in LuigiConfigParser. In Python 2 ConfigParser does not support custom interpolations, so instead the LuigiConfigParser overrides the _interpolate method from the parent class.
Motivation and Context
Addresses issue #2354. A typical use case is to include secrets in the config file, for example:
[database]
host=someserver.com
uid=someuser
pwd=${DATABASE_PASSWORD}

An alternative implementation of this feature exists in PR #2038 (not merged), which uses the %(variable_name)s syntax. However, that implementation is not backward compatible because it changes the default behavior of ConfigParser (and therefore LuigiConfigParser) where the same syntax is used to refer to other options in the same config section.
Have you tested this? If so, how?
I have included unit tests.

@dlstadther In case someone uses ${something} then this PR indeed introduces a breaking change. However, should this corner case occur, the user may easily fix by using config.get(section, option, raw=True) to get the literal value.

Thanks for following back up! I'm still good with this after another look-through
I'll merge


Guard dummy user, mute warnings in development

Add guard around use of arbitrary host for dummy user. Fixes #16
Suppress warnings for missing social signon values when in development mode. Fixes #13

This change is

Reviewed 30 of 42 files at r1, 11 of 12 files at r2.
Review status: 24 of 25 files reviewed at latest revision, 2 unresolved discussions.

auth-app/src/main/java/org/gameontext/auth/Health.java, line 155 at r2 (raw file):

	            + "googleOAuthConsumerKey=%b, googleOAuthConsumerSecret=%b, "
	            + "twitterOAuthConsumerKey=%b, twitterOAuthConsumerSecret=%b, "
	            + "kafka=%b",

Nice.. I learnt something new about %b =)

auth-app/src/main/java/org/gameontext/auth/dummy/DummyAuth.java, line 47 at r2 (raw file):

    private String callbackSuccess;

    /** Something like https://<IP_ADDRESS>/#/login/callback provided by the environment */

cut & paste comment..

Comments from Reviewable

Review status: 24 of 25 files reviewed at latest revision, 1 unresolved discussion.

auth-app/src/main/java/org/gameontext/auth/dummy/DummyAuth.java, line 47 at r2 (raw file):
Previously, BarDweller (Ozzy Osborne) wrote…
> cut & paste comment..

Done.

Comments from Reviewable

Reviewed 1 of 42 files at r1, 1 of 1 files at r3.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


python

Hello, thank you very much for providing this program. However, during the execution process, I encountered some issues. May I ask which version of Python you are using?

I tried to run the code and I also got an error because of some mutable arguments in the dataclass TransformationKinetics. I made a small change to bainitemodel.py and now it's working for me. Could you please download the updated file, try again, and let me know if it's working?
Thanks!

Thank you very much! I'll try running it and see how it goes. I'm currently using Python 3.12, which might not be fully compatible, so I'll give it a shot with Python 3.7 instead. Thanks again for your help!

= RESTART: E:\B\bainite-model-main (1)\bainite-model-main\Example2.py
Traceback (most recent call last):
File "E:\B\bainite-model-main (1)\bainite-model-main\Example2.py", line 23, in
fit = fit_single_curve(bainite_params=bainite_params, minimizer_options={'method': 'differential_evolution'})
File "E:\B\bainite-model-main (1)\bainite-model-main\bainitemodel.py", line 638, in fit_single_curve
if (bainite_params.kinetics.experiment_time_datapoints == np.array([]) or
ValueError: operands could not be broadcast together with shapes (2401,) (0,)
I used version 3.12, and example one runs successfully, but example two encountered the following error

Okay, I see! I made a few changes and tried again using Python 3.11. Now all three examples worked for me, and I guess they should also work for Python 3.12. Could you please try again and let me know if it works?

Hello, thank you very much for providing the revised version. I tried rerunning it, but the following issues still occurred:

First Example:
The first example runs normally.
Second Example:

The second example throws the following error:
File "E:\B\bainite-model-main new\bainite-model-main\bainitemodel.py", line 663, in residuals
return experiment - model
~~~~~^
ValueError: operands could not be broadcast together with shapes (2401,) (4,)

Third Example:

The third example runs, but the following warnings still appear during execution:
E:\B\bainite-model-main new\bainite-model-main\bainitemodel.py:477: RuntimeWarning: overflow encountered in scalar multiply
n_t_a = n_s_a * (f_sum / sub_unit_length) * (1 - f_a / f_a_max) * enrichment_factor if f_a_max > 0 else 0
E:\B\bainite-model-main new\bainite-model-main\bainitemodel.py:480: RuntimeWarning: overflow encountered in exp
dn_gb_dt = vibration_factor * n_t_gb * np.exp(- q_gb_t / (r * temp_kelvin))
E:\B\bainite-model-main new\bainite-model-main\bainitemodel.py:481: RuntimeWarning: overflow encountered in exp
dn_a_dt = vibration_factor * n_t_a * np.exp(- q_a_t / (r * temp_kelvin))
E:\B\bainite-model-main new\bainite-model-main\bainitemodel.py:462: RuntimeWarning: invalid value encountered in scalar add
f_sum = f_gb + f_a
This issue also appeared in the second error.
This is the result after the error occurred. There doesn’t seem to be a numerical difference between the two.

Good. I changed the second example, and it should be running now. There was an issue with Scipy,  it wasn't solving correctly the ODEs.
The warnings you got running Example 3 are normal, and they don't interfere with the result. Example 3 considers the same activation energy for the two prior austenite grain sizes, so the activation energy you find should indeed be the same. The example is running correctly.
This code here on GitHub sometimes cannot find a good fit. I'm using a different optimization approach right now that I plan on publishing here in October/November. If you get poor fitting or some weird results, email me at<EMAIL_ADDRESS>and I can share the new code I'm using with you.


Fetch all the content again

Remove all inspections stuff
Reimport all the key stage 1 & 2 stuff (except for 67 pages that have been redirected - we'll come back to these)

Best reviewed commit by commit.
Paired/mobbed with @carolinegreen @rubenarakelyan and @jackscotti

👍


feat: Lake shared library

Fixes #2436
Next step: when libLake_shared is in stage 0, --load-dynlib it when building stage 1 Lake

@tydeu Moving Lake.Main out of Lake/ seemed like the easiest option to exclude it from the library outputs

@Kha Sounds good to me! 😄 Style-wise. I would probably just name it Main.lean unless that is an issues. As the file is already under src/lake and Main.lean is currently the standard naming convention used by executable templates.  Functionally-wise, the TOML configurations need updating to point the lean_exe to the new module (i.e., in src/lake/lakefile.toml and src/lakefile.toml.in).

The reason being the file is already under src/lake

I was hoping it would not stay there :)

@tydeu I don't actively use the TOML files atm and don't want to touch them without testing, could you update them in a separate PR?


(BKR 24) Updated docs with explanation of terminal output coloring

Refer to this link for build results (access rights to CI server needed):
http://jenkins-beaker.delivery.puppetlabs.net//job/qe_beaker_btc-intn/2619/


Notes cannot be opened from graph view

In Dendron version 0.24.0, clicking a given node in the graph view doesn't open/navigate to its associated note in the editor. Version 0.23.0 works as expected.
Steps to reproduce the behavior:

Open a Dendron workspace
Run dendron.showNoteGraph
Left-click on a node in the graph view that has an associated markdown file (i.e. isn't virtual)

Expected behavior
The editor pane opens/switches to the associated markdown file.

Thanks for catching this. This was a regression since we updated our core package (dendron) and not the markdown links package.
This has now been fixed with dendron-markdown-links v0.6.19.
You can also see the updated docs here
Will leave this open for confirmation by user. If this has fixed the issue, feel free to close this issue :)

Thanks for catching this. This was a regression since we updated our core package (dendron) and not the markdown links package.
This has now been fixed with dendron-markdown-links v0.6.19.
You can also see the updated docs here
Will leave this open for confirmation by user. If this has fixed the issue, feel free to close this issue :)


Compute New Seek Position From Current Offset

Expected Behavior
I have a use-case where I only want to seek to a specified offset if it is before the consumer's current offset. Ideally I would like to have a method on ConsumerSeekCallback where I would provide a Function<Long, Long> that computes the new offset given the current one.
Current Behavior
Currently the framework only provides a way to seek to a specific offset unconditionally, or provide a relative offset from the current one. There is no way to access the current offset value when performing a seek.
Context
For my use-case, there are certain events that require re-processing records that were already processed before. However, I should not seek forward in the Kafka partition, because the partition may contain other records that are unrelated to the event and shouldn't be skipped (sorry if this is a bit vague).
So, I need to know what the current partition offset is before I decide to actually seek from the event.
In order to try to accomplish this, I have the Kafka listener extend AbstractConsumerSeekAware, then use the methods from that to perform the seek when the event occurs.
I have considered using the onIdleContainer method to store the current offsets, but reading the source code it seems that this is only called after a certain time has elapsed, so these offsets are not guaranteed to be up-to-date by the time I decide to seek.
I have also considered using the KafkaListenerEndpointRegistry, however this only allows pausing/resuming/stopping containers, and does not provide access to the underlying Consumer.

@eblocha, I want to understand this use case a bit more. The seekRelative method allows you to seek relatively from the current offset by setting toCurrent to be true. See the javadocs.
/**
 * Perform a seek relative to the start, end, or current position. When called
 * from {@link ConsumerSeekAware#onPartitionsAssigned(Map, ConsumerSeekCallback)}
 * or from {@link ConsumerSeekAware#onIdleContainer(Map, ConsumerSeekCallback)}
 * perform the seek immediately on the consumer. When called from elsewhere, queue
 * the seek operation. The queued seek will occur after any pending offset
 * commits. The consumer must be currently assigned the specified partition.
 * @param topic the topic.
 * @param partition the partition.
 * @param offset the offset; positive values are relative to the start, negative
 * values are relative to the end, unless toCurrent is true.
 * @param toCurrent true for the offset to be relative to the current position
 * rather than the beginning or end.
 * @since 2.3
 */
void seekRelative(String topic, int partition, long offset, boolean toCurrent);

When you are setting toCurrent to true, you are, in effect, getting access to the current offset (not the application directly, but the framework internally goes to the current offset and then does a relative seek). Since the framework does that already, what benefit will it gain by the application having access to the current offset? Please elaborate more if possible. We are open to making any API changes if there is room for improvement or a need. Thanks!

@sobychacko Thanks for the response! With seekRelative, it requires the application to know the relative offset, but I only know the absolute offset. In order to get a relative one, I'd need to access the current offset. I also only want to seek if the current offset is ahead of the target offset.
For a bit more context, the topic contains change-data-capture records from a database, and the events mark when a snapshot was completed for a table (the snapshot data is not sent to the topic). The producer is paused during the snapshot, and saves metadata containing the last offset/partition for the table data before restarting the producer.
I can initialize data or recover from failures by consuming the last taken snapshot, then seeking the consumer back to when it was taken. If I am doing this for multiple tables whose snapshots may have been taken at different times/offsets, I don't want to jump forward in the partition after consuming another snapshot for a different table.
topic1, partition1 ---------|-------------------------|------
                            ^                         ^
                      snapshot, table_a         snapshot, table_b

Rough diagram above. If a consumer reads snapshot for table_a, then snapshot for table_b, I don't want to seek forward to the position table_b was taken on, because I may skip changes that happened to table_a after the snapshot.

Correct, I only have access to the absolute offset I should seek to, with the caveat that I should not seek if I'm currently behind that offset.
I think that API would work. If the function is computing the relative offset, it would be implemented as follows:
public class ComputeRelativeOffset implements Function<Long, Long> {
    private final Long absoluteTargetOffset;

    public SeekBackwardsOnly(Long absoluteTargetOffset) {
        this.absoluteTargetOffset = absoluteTargetOffset;
    }

    public Long apply(Long currentOffset) {
        if (currentOffset > absoluteTargetOffset) {
            return absoluteTargetOffset - currentOffset;
        } else {
            return 0L; // do not change position
        }
    }
}

If the function is computing the absolute offset, it would be implemented like this:
public class ComputeNewOffset implements Function<Long, Long> {
    private final Long absoluteTargetOffset;

    public SeekBackwardsOnly(Long absoluteTargetOffset) {
        this.absoluteTargetOffset = absoluteTargetOffset;
    }

    public Long apply(Long currentOffset) {
        if (currentOffset > absoluteTargetOffset) {
            return absoluteTargetOffset;
        } else {
            return currentOffset;
        }
    }
}

@eblocha We added this feature on the main branch. Please take a look at the commits above. Please try it on the latest snapshot (3.2.0-SNAPSHOT) and give us any feedback.

@eblocha We added this feature on the main branch. Please take a look at the commits above. Please try it on the latest snapshot (3.2.0-SNAPSHOT) and give us any feedback.

Hi @sobychacko, thanks for the quick turnaround! I tried out the feature, but I am getting the following error:
java.util.ConcurrentModificationException: KafkaConsumer is not safe for multi-threaded access
	at org.apache.kafka.clients.consumer.KafkaConsumer.acquire(KafkaConsumer.java:2491)
	at org.apache.kafka.clients.consumer.KafkaConsumer.acquireAndEnsureOpen(KafkaConsumer.java:2475)
	at org.apache.kafka.clients.consumer.KafkaConsumer.position(KafkaConsumer.java:1747)
	at org.apache.kafka.clients.consumer.KafkaConsumer.position(KafkaConsumer.java:1717)
	at org.springframework.kafka.listener.KafkaMessageListenerContainer$ListenerConsumer.seek(KafkaMessageListenerContainer.java:3330)

I think this is happening because the implementation reads the current offset when the seek function is called, instead of when it is actually applied. The existing seek methods seem to defer accessing the current offset until the seeks are performed.

@eblocha The concurrency issue is addressed via https://github.com/spring-projects/spring-kafka/issues/3108. Could you try it on the latest snapshot and let us know about any further issues?

@sobychacko It's working beautifully, thank you!


feat(logger): add logger plugin

What this PR does / why we need it:
Add logger plugin. Now log options can be modified by flags.
Which issue(s) this PR fixes (optional, close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
/cc @zoumo
/cc @ddysher
Release note:

NONE

/approve

/lgtm


Using time as random seeds

Refer to https://csrc.nist.gov/publications/detail/sp/800-90a/archive/2012-01-23

I believe this has been resolved.  Thank you!


Fix header UI button alignment

Super basic issue but the buttons in the header are not aligned. Simple fix is adding "items-center" utility to
 in header.tsx
pre:

post:

Fixed, thank you!


POC for FormBuilder replacing Update Axis.

I didn't look at every single line, but overall, very elegant looking code. 👍

Thanks, @gaigstroliac . The goal of the form builder ultimately is to allow plugins to easily collect form data from the user (inputs for scripts, adding objects to the product model, etc.). As a side effect, we can leverage the same code to dynamically populate our basic existing modals and clean up our static js/html.


script/sync:add support to sync nodes

Adds support to sync nodes to S3
The PR https://github.com/boxen/puppet-nodejs/pull/48 make use of this.

The node binaries are only synced when no arguments are passed to the sync command.
I thought that may be this could be something like this.
script/sync #sync everything
script/sync --brew emacs/24.3-boxen2
script/sync --ruby 1.9.3-p448
script/sync --node 0.12.2

Any other idea?

Great, thanks!


Render suggestions on first focus

I'd like to show suggestions when the input is focused for the first time.  If I define
shouldRenderSuggestions: function(value) {
    return true;
}

, suggestions are only displayed after the input value is first changed.  If this is intended, then is there a solution to force a paint bound to the onFocus event?

@jbasrai Make sure that the function that calculates the suggestions returns a non-empty list of suggestions when the input value is blank.
If it doesn't help, could you create please a Codepen that demonstrates the issue?

My mistake, the suggestions weren't being populated until the later.  Thanks for the insight!


[BUG] Failed to load block 0 with --no-keep-data

commands:
bash deploy_blockchain_genesis_gcp.sh dev <Username> 0 --no-keep-code --no-keep-data

logs:
  1 {"name":"testnet-dev","hostname":"dev-node-0-taiwan","pid":15086,"level":50,"msg":"[NODE] [loadAndExecuteChainO
    nDb] Failed to load block 0.\nError\n    at Function.logErrorWithStackTrace (/home/ain-blockchain-2022-01-08T18
    -25/common/common-util.js:799:33)\n    at Function.finishWithStackTrace (/home/ain-blockchain-2022-01-08T18-25/
    common/common-util.js:809:16)\n    at BlockchainNode.loadAndExecuteChainOnDb (/home/ain-blockchain-2022-01-08T1
    8-25/node/index.js:687:20)\n    at BlockchainNode.initNode (/home/ain-blockchain-2022-01-08T18-25/node/index.js
    :267:17)\n    at P2pClient.startBlockchainNode (/home/ain-blockchain-2022-01-08T18-25/p2p/index.js:384:36)\n
     at P2pClient.run (/home/ain-blockchain-2022-01-08T18-25/p2p/index.js:50:20)\n    at processTicksAndRejections
    (node:internal/process/task_queues:96:5).","time":"2022-01-08T18:25:41.833Z","v":0}

Fixed, thanks!