discovery: don't merge a peer when no message is sent to it

This is an attempt to fix #6531 and can be viewed as an alternative to #7239. When we broadcast messages, we'd skip peers who have syncer as the syncing logic will take care of sending the messages. However, it is suspected that some of the syncers haven't finished setting up yet, which ends up having  g.remoteUpdateHorizon == nil that causes some messages to never be sent to the peers. In general, even if the remoteUpdateHorizon is nil, the syncer will still send the messages later once it's active, so a nil remoteUpdateHorizon alone is not the cause, which has been manifested in a local itest. However, it's possible that by the time the remoteUpdateHorizon is set, the timestamp used here has already passed those used in the above messages, causing them to be filtered out, hence nothing gets sent.
This PR fixes it by not skipping the peers if no message is sent from their syncers. Meanwhile, the pinned syncers seem to be missing from GossipSyncers, and they are added back.
Another mitigation might be adding a delta value here instead of using time.Now(). Something like time.Now() - 1 min so we can add some redundancy when requesting channel updates, at the cost of receiving potentially duplicate messages.

In general, even if the remoteUpdateHorizon is nil, the syncer will still send the messages later once it's active

It might be the case that the node turns off historical graph dumps and doesn't send it

@yyforyongyu, remember to re-request review from reviewers when ready

@yyforyongyu, remember to re-request review from reviewers when ready

@yyforyongyu, remember to re-request review from reviewers when ready

@yyforyongyu, remember to re-request review from reviewers when ready

@yyforyongyu, remember to re-request review from reviewers when ready

@yyforyongyu, remember to re-request review from reviewers when ready

@yyforyongyu, remember to re-request review from reviewers when ready

replaced by #7373


Problems with Addback events with AngularCorrelationHelper

HOSTNAME     = gammatron
SYSTEM       = Linux
OS           = Ubuntu
VER          = 18.04
GRSISYS      = /home/pietro/Software/Ge80-grsisort/GRSISort
ROOTSYS      = /home/pietro/Software/Grsisort_S1916/root
ROOT Version = 6.20/08
Computer and Path to File that failed:
GRSISort Branch = main
commit 937b9f1c9d3786e8a232770ff69a6b3366a3d366 (HEAD -> main, tag: v<IP_ADDRESS>, origin/main, origin/HEAD)
Merge: 8578a126 77378af5
Author: Vinzenz Bildstein<EMAIL_ADDRESS>Date:   Wed Oct 25 15:12:17 2023 -0400
Merge pull request #1369 from VinzenzBildstein/main

Improvement for TPeakFitter, TNucleus, and TEfficiencyCalibrator

I am having problems with the AngularCorrelationHelper.
The helper seems to work fine for singles i.e. grif.GetGriffinHit(g1) but will crash if it tries to use addback.
I have attached a copy of the helper being used.
The line that causes the crash is
fH1[slot].at("aMult")->Fill( grif.GetAddbackMultiplicity() );
I have also attached a copy of the output that was printed to the terminal when running grsiframe.
I am very unsure why using addback would cause a crash as I have other helpers that utilise addback without issue and can not figure out the what the problem is.
MyAngularCorrelationHelper.hh.txt
MyAngularCorrelationHelper.cxx.txt
grsiframe-log.txt

I just ran this over an old analysis file I had lying around, and it crashed for me too with the same backtrace. Looking at the file specified, it crashes when we delete the old addback hits. This was introduced about 7 months ago, and the PR says "maybe fixed a memory leak by deleting contents of vector instead of just clearing".
But we clear the vectors of this object, not the rhs object. And the Copy function is using the same weird logic as root, where a->Copy(b) doesn't mean that a copies b, but that a is copied to b. So if at all we would want to delete the hits the rhs objects vectors. I tried that, and the helper doesn't crash anymore. I do however get a ton of messages "Warning in TVector3::SetMag: zero vector can't be stretched". I don't know if that is an issue of the helper itself or of the underlying code in GRSISort that is called by the helper.
I will add a corresponding PR for the fix of the crash.

@Spagnole, can you confirm whether this fixes the issue you were having?

Sorry I thought this issue was closed.
Yes the changes you made have fixed the issue on my end. I can now sort addback events with the helper.


Konata Theme

Describe the bug

Konata's theme is very bright, of course, since it is a light theme.
The colors for the toolbar_field & toolbar_field_text blend too well together. It is difficult to see the text in the address bar very well while both in focus and out of focus.
A solution I recommend is changing the toolbar to  these settings instead
To Reproduce

1.Load the extension
1.Select Konata
1.Stress your eyes
Expected Behavior

I am expected to have a nice browsing experience, with a great theme featuring an anime character.
Environment

Screenshots

Before

After

:thinking:
So as part of my work with the new themes, I was also planning on re-imagining Konata's theme.
It should make the theme a bit easier to maintain and more usable. The platforms aren't really built for this type of vivid theme :sweat_smile:

Alright this is what the new Konata looks like

I think it's usable now :+1:


BUG- Standalone mode failed to skip 2 subsequent tasks

Description
On Standalone execution mode, assume the pipeline has 2 or more subsequent tasks with Global Datanode with cacheable property.
After the first submit, both tasks is completed.
On the second submit, both tasks should be skipped (true on Development execution mode).
However, on Standalone execution mode, only the first task is skipped and the second task is blocked.
How to reproduce

Configuration files or code:

from taipy.config.common.frequency import Frequency
from taipy.config.common.scope import Scope
from taipy.config.config import Config

from .algorithms import *

historical_data_set = Config.configure_csv_data_node(
    id="historical_data_set",
    path="./daily-min-temperatures.csv",
    scope=Scope.GLOBAL,
    cacheable=True,
)

dates_to_forecast = Config.configure_data_node(
    id="dates_to_forecast",
    scope=Scope.GLOBAL,
    default_data=[datetime(1991, 2, 2).isoformat()],
    cacheable=True,
)

arima_model = Config.configure_data_node(
    id="arima_model",
    cacheable=True,
    scope=Scope.GLOBAL,
)

forecast_values = Config.configure_excel_data_node(
    id="forecast_values",
    has_header=False,
    path="./res.xlsx",
    cacheable=True,
    scope=Scope.GLOBAL,
)

arima_training_algo = Config.configure_task(
    id="arima_training",
    input=historical_data_set,
    function=train,
    output=arima_model,
)

arima_scoring_algo = Config.configure_task(
    id="arima_scoring",
    input=[arima_model, dates_to_forecast],
    function=predict,
    output=forecast_values,
)

arima_pipeline = Config.configure_pipeline(
    id="arima_pipelines",
    task_configs=[arima_training_algo, arima_scoring_algo],
)

arima_scenario_config = Config.configure_scenario(
    id='Arima_scenario',
    pipeline_configs=[arima_pipeline],
    frequency=Frequency.DAILY,
)

A code fragment

import taipy as tp
from taipy.config import Config

from shared.config import arima_scenario_config

Config.configure_job_executions(mode="standalone", max_nb_of_workers=2)

core = tp.Core()
core.run()
scenario = tp.create_scenario(arima_scenario_config)
tp.submit(scenario, wait=True)

Expected behavior
On Standalone execution mode, both tasks should be skipped.
Runtime environment

Taipy version: 2.0 and develop

@toan-quach found out that "edit_in_progress" property of data node "arima_model" was not assign correctly on Standalone mode.
Specifically, after task ""arima_training" finished, "edit_in_progress" property of "arima_model" should be set to False, which is not the case.

Oh ?!?

#406


ARROW-14171: [C++][Python][Packaging] Upgrade VCPKG version and add google-cloud-cpp dependency

Ping? Is there something else I need to do here?

It seems that this is a problem.
https://github.com/ursacomputing/crossbow/runs/4039103923?check_suite_focus=true#step:6:1189
#12 14.69 Package google-cloud-cpp does not have a storage feature

I see.  The code checks out version 2021.04.30:
https://github.com/apache/arrow/blob/faf8b827493afee60ea44893b64d36a7d5f3fadc/.env#L75
That version does not include the storage feature, neither does the latest tag at 2021.05.12.
How do you feel about pinning vcpkg to a specific SHA, but not to a tag?

How do you feel about pinning vcpkg to a specific SHA, but not to a tag?

@ianmcook @kszucs What do you think about this?

We used to pin specific SHAs beforehand, so it's fine.

Thanks, I have updated this PR accordingly.  I also sent ursacomputing/crossbow#6 for review, I am not sure which one should be merged first, please let me know if I should help with that.

Thanks! Since it affects the build cache we should wait for the 6.0.1 patch release with both of the PRs.

Merged the crossbow PR. I'm going to rebuild and push the windows docker images for the windows wheels then we can submit the crossbow jobs here.

After fixing the vcpkg/ports.patch I'm getting a vcpkg dependency problem on windows:
✖ REPO=ghcr.io/ursacomputing/arrow PYTHON=3.10 archery docker build --using-docker-cli python-wheel-windows-vs2017
Error response from daemon: manifest unknown
docker pull ghcr.io/ursacomputing/arrow:python-3.10-wheel-windows-vs2017-vcpkg-9675ab15def74a0b96d954bddc7ddabb42c92452 exited with non-zero exit code 1
Sending build context to Docker daemon  453.1kB
Step 1/19 : FROM abrarov/msvc-2017:2.10.0
 ---> 48bb8164f545
Step 2/19 : RUN choco install --no-progress -r -y cmake --installargs 'ADD_CMAKE_TO_PATH=System' &&     choco install --no-progress -r -y gzip wget ninja
 ---> Using cache
 ---> 84da6558371c
Step 3/19 : RUN setx path "%path%;C:\Program Files\Git\usr\bin"
 ---> Using cache
 ---> 744f355e6df4
Step 4/19 : ARG vcpkg
 ---> Using cache
 ---> 6fd7cae10c08
Step 5/19 : RUN git clone https://github.com/Microsoft/vcpkg &&     vcpkg\bootstrap-vcpkg.bat -disableMetrics &&     setx PATH "%PATH%;C:\vcpkg" &&     git -C vcpkg checkout %vcpkg%
 ---> Using cache
 ---> a85f4b3090f6
Step 6/19 : COPY ci/vcpkg/*.patch      ci/vcpkg/*windows*.cmake      arrow/ci/vcpkg/
 ---> Using cache
 ---> cdc9fbfb7e6e
Step 7/19 : RUN cd vcpkg && git apply --ignore-whitespace C:/arrow/ci/vcpkg/ports.patch
 ---> Using cache
 ---> c93972ad92dc
Step 8/19 : ARG build_type=release
 ---> Using cache
 ---> c130e80c70b8
Step 9/19 : ENV CMAKE_BUILD_TYPE=${build_type}     VCPKG_OVERLAY_TRIPLETS=C:\\arrow\\ci\\vcpkg     VCPKG_DEFAULT_TRIPLET=x64-windows-static-md-${build_type}     VCPKG_FEATURE_FLAGS=-manifests
 ---> Using cache
 ---> e7a18a9e6535
Step 10/19 : RUN vcpkg install --clean-after-build         abseil         aws-sdk-cpp[config,cognito-identity,core,identity-management,s3,sts,transfer]         boost-filesystem         boost-multiprecision         boost-system         brotli         bzip2         c-ares         curl         flatbuffers         gflags         glog         google-cloud-cpp[core,storage]         grpc         lz4         openssl         orc         protobuf         rapidjson         re2         snappy         thrift         utf8proc         zlib         zstd
 ---> Running in 3c89420370c9
Computing installation plan...
Error: aws-checksums[core] is only supported on '!arm & !windows'
The command 'cmd /S /C vcpkg install --clean-after-build         abseil         aws-sdk-cpp[config,cognito-identity,core,identity-management,s3,sts,transfer]         boost-filesystem         boost-multiprecision         boost-system         brotli         bzip2         c-ares         curl         flatbuffers         gflags         glog         google-cloud-cpp[core,storage]         grpc         lz4         openssl         orc         protobuf         rapidjson         re2         snappy         thrift         utf8proc         zlib         zstd' returned a non-zero code: 1
Error: docker build --build-arg BUILDKIT_INLINE_CACHE=1 --build-arg python=3.10 --build-arg vcpkg=9675ab15def74a0b96d954bddc7ddabb42c92452 -f ci/docker/python-wheel-windows-vs2017.dockerfile -t ghcr.io/ursacomputing/arrow:python-3.10-wheel-windows-vs2017-vcpkg-9675ab15def74a0b96d954bddc7ddabb42c92452 /Users/kszucs/Workspace/arrow exited with non-zero exit code 1

It has been already resolved upstream, so I pinned the HEAD commit.

@github-actions crossbow submit -g wheel

New changes:

bumped the vcpkg version to make the aws-sdk-cpp compile on windows
updated to use the install_vcpkg.sh script everywhere (including the windows docker images)
removed the duplicated vcpkg versions across the build configs, now it uses VCPKG version from .env

Still, We have several problems here:

CMake 3.22.0 has a regression causing issue with AWSSDKConfig.cmake. We need to pin 3.21.4 both on macOS and manylinux.
Manylinux2010 doesn't have realpath so we need to switch to readlink -f in the install_vcpkg.sh script.
The vcpkg installer now downloads a prebuilt binary instead of compiling the vcpkg tool itself locally but the binary links with >glibc 2.14 and the manylinux2010 image has glibc=2.12. I patched the bootstrap.sh script to force build vcpkg but getting many ambiguous calls errors during compilation. Perhaps it would be the right time to drop the EOL manylinux2010 packages.
The aws-sdk-cpp packages are restricted to non-arm architectures despite that earlier versions used to work. I assume patching the port files with a new constraint would work.

@github-actions crossbow submit -g wheel

@github-actions crossbow submit wheel-macos-big-sur-*

@github-actions crossbow submit wheel-macos-big-sur-*

@kszucs thanks for all the work in this PR.  Is there something I could do now (or could have done before) to make your life easier?

@github-actions crossbow submit wheel-macos-big-sur-*

@github-actions crossbow submit wheel-macos-mavericks-cp38-amd64

@github-actions crossbow submit wheel-macos-big-sur-cp39-arm64

@coryan seems like google-cloud-cpp fails to compile on macOS: https://github.com/ursacomputing/crossbow/runs/4326265101?check_suite_focus=true#step:9:1711

@github-actions crossbow submit -g wheel

@coryan seems like google-cloud-cpp fails to compile on macOS: https://github.com/ursacomputing/crossbow/runs/4326265101?check_suite_focus=true#step:9:1711

Maybe we can just update the vcpkg baseline in this PR, and add google-cloud-cpp separately?
As to the build failure, seems like this build is using clang++, and I think both we and the vcpkg folks test with c++ (Apple's native compiler). I wonder if that makes a difference.

It cannot find ssl/crypto:
CMake Error at /usr/local/Cellar/cmake/3.22.0/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:230 (message):
  Could NOT find OpenSSL, try to set the path to OpenSSL root folder in the
  system variable OPENSSL_ROOT_DIR (missing: OPENSSL_CRYPTO_LIBRARY
  OPENSSL_INCLUDE_DIR)
Call Stack (most recent call first):
  /usr/local/Cellar/cmake/3.22.0/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:594 (_FPHSA_FAILURE_MESSAGE)
  /usr/local/Cellar/cmake/3.22.0/share/cmake/Modules/FindOpenSSL.cmake:574 (find_package_handle_standard_args)
  /tmp/picsaba/scripts/buildsystems/vcpkg.cmake:788 (_find_package)
  google/cloud/storage/CMakeLists.txt:49 (find_package)

Installing openssl before google-cloud-sdk resolves the issue. I'm going to use that as a workaround but this should be fixed upstream.
Note, that the following triplet file is used https://github.com/apache/arrow/blob/master/ci/vcpkg/x64-osx-static-release.cmake

@github-actions crossbow submit -g wheel

@github-actions crossbow submit -g wheel

@github-actions crossbow submit wheel-macos-big-sur-*

@github-actions crossbow submit -g wheel

@github-actions crossbow submit wheel-macos-big-sur-*-universal2

@github-actions crossbow submit wheel-macos-big-sur-*-universal2

@github-actions crossbow submit wheel-macos-big-sur-*-universal2

@coryan we don't yet have python bindings for GCS, right?

@github-actions crossbow submit wheel-macos-big-sur-*-universal2

@github-actions crossbow submit wheel-macos-big-sur-*-universal2

@github-actions crossbow submit wheel-macos-big-sur-cp39-universal2

@github-actions crossbow submit -g wheel

@github-actions crossbow submit wheel-macos-big-sur-cp39-universal2

@github-actions crossbow submit wheel-macos-big-sur-cp39-universal2

@github-actions crossbow submit -g wheel

@coryan we don't yet have python bindings for GCS, right?

If you mean in Arrow, then no, they do not exist yet.  I had no plans to create them either.

If you mean in Arrow, then no, they do not exist yet. I had no plans to create them either.

That's something that other contributors can do :-)

Added a note about enabling the GCS feature in the wheels for https://issues.apache.org/jira/browse/ARROW-14892
@pitrou this should be ready for review then


Define our own trie.Key type

We are using a third-party bitset library as our key type. But it is  not properly serialized to a big endian byte array. The maintainer is not acknowledging the issue, so there is no way to fix it upstream.
Define a simple key type like;
type Key struct {
   Len byte
   Val felt.Felt
}

Is it not possible to infer the len from the felt.Felt?

Is it not possible to infer the len from the felt.Felt?

Sadly, no. But instead of a felt, I opted for a byte array in the actual implementation.


[Android] Authenticathion

Hi,
At first I want to say thank you for starting this project. I like Zulip as a project but lack of mobile clients may plays a bad role in his popularity. Nowadays mobile took a big part and any IM have to have mobile app.
I installed Zulip-Messenger from GA but can not make a login. Could you please explain in what format login/password should provided?
I use the same credentials as for Web page (copy paste) but seems Zulip-Messenger changes them or Zulip want something else.
==> ./django.log <==
2016-05-18 09:38:42,745 INFO     <IP_ADDRESS>  POST    403   1ms /api/v1/fetch_api_key (unauth via ?)
2016-05-18 09:38:42,745 INFO     status=403, data={"msg":"Your username or password is incorrect.","reason":"incorrect_creds","result":"error"}

Unfortunately I'm new with Zulip (just looking for Slack alternative)  and can not do deeper troubleshooting right now.
What data  app send for authentication and where?
Thank you for assist.

Resolved under https://github.com/zulip/zulip/issues/806


Android build fails from binutils 2.25 bug

The binutils included in the chromium tree is 2.25. It will fail with this error:
ror: /usr/lib/gcc/x86_64-linux-gnu/5.3.1/../../../x86_64-linux-gnu/crti.o: unsupported reloc 42 against global symbol gmon_start
/usr/lib/gcc/x86_64-linux-gnu/5.3.1/../../../x86_64-linux-gnu/crti.o(.init+0x7): error: unsupported reloc 42
If you build and install binutils 2.26 and update the chromium tree to use it, you will be able to build. You make all need to add libs = -latomic.

I am having exact same problem. Could you shed some light on how I should update chromium tree to use 2.26 binutils. And where I should add the flag -latomic(as you suggested in your comment)?  Thanks.


SOLR-16168: Spellcheck NPE if invalid dictionary name is provided

https://issues.apache.org/jira/browse/SOLR-16168
Description
If a user provides spellcheck.q and an invalid spellcheck.dictionary there is an NPE.
Solution
This throws an exception with a helpful error message instead of the NPE. It moves the spellcheck null check up in the processing.
Tests
Added tests for both when spellcheck.q is specified and when spellcheck.q isn't specified with an invalid spellcheck.dictionary value.
Checklist
Please review the following and check all that apply:

[x] I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
[x] I have created a Jira issue and added the issue ID to my pull request title.
[x] I have given Solr maintainers access to contribute to my PR branch. (optional but recommended)
[x] I have developed this patch against the main branch.
[x] I have run ./gradlew check.
[x] I have added tests for my changes.
[x] I have added documentation for the Reference Guide

LGTM


Partially support DetectionOutput layer

Partially support DetectionOutput layer

Thank you for your contribution, which is an excellent submission. I'm sorry for the late code review.

Would you please run tools/cpplint.sh and pycodestyle to find coding style errors reported by cpplint and pycodestyle, and fix them?
.gitlab-ci.yml shows the usage of these tools.

Would you please add some unit test cases for new added Op?
There are examples in test/ccunit/mace/ops/.

Please run

bash tools/bazel_build_standalone_lib.sh  -abi=arm64-v8a --runtimes=gpu --static

to make sure that the compiler will not report errors.

Please run you test cases by this command:

python tools/bazel_adb_run.py --target="//test/ccunit:mace_cc_test" --run_target=True --stdout_processor=unittest_stdout_processor --target_abis=arm64-v8a

Make sure that the compiler will not report errors and  your new added test cases pass.
It seems that you added test cases in test/ccunit/mace/ops/detection_output_test.cc, but running it was not done.
For example, I run
python tools/bazel_adb_run.py --target="//test/ccunit:mace_cc_test" --run_target=True --stdout_processor=unittest_stdout_processor --target_abis=arm64-v8a

But  got these errors:
test/ccunit/mace/ops/detection_output_test.cc:32:36: error: no template named 'vcetor' in namespace 'std'; did you mean 'vector'?
                        const std::vcetor<int> &offset,
                              ~~~~~^~~~~~
                                   vector
external/androidndk/ndk/sources/cxx-stl/llvm-libc++/include/iterator:1407:74: note: 'vector' declared here
    template <class _Tp, class _Alloc> friend class _LIBCPP_TEMPLATE_VIS vector;
                                                                         ^
test/ccunit/mace/ops/detection_output_test.cc:32:49: error: unused parameter 'offset' [-Werror,-Wunused-parameter]
                        const std::vcetor<int> &offset,
                                                ^
test/ccunit/mace/ops/detection_output_test.cc:71:3: error: no matching function for call to 'RunDetectionOutput'
  RunDetectionOutput<RuntimeType::RT_CPU>(
  ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
test/ccunit/mace/ops/detection_output_test.cc:26:6: note: candidate function template not viable: requires 14 arguments, but 13 were provided
void RunDetectionOutput(const std::vector<index_t> &input_shape1,
     ^
3 errors generated.

Test Env setup is too complicate. I tried MacOsX and Ubuntu. MacOsX built failed, Ubuntu can't find my Android devcie.

➜  mace git:(master) bash tools/bazel_build_standalone_lib.sh  -abi=arm64-v8a --runtimes=gpu --static

-------------build static lib for arm64-v8a +  gpu------------
WARNING: The major revision of the Android NDK referenced by android_ndk_repository rule 'androidndk' is 19. The major revisions supported by Bazel are [10, 11, 12, 13, 14, 15, 16]. Bazel will attempt to treat the NDK as if it was r16. This may cause compilation and linkage problems. Please download a supported NDK version.
ERROR: /Users/rqg/Playground/mace/mace/codegen/BUILD.bazel:26:1: no such target '@local_opencl_kernel_encrypt//:gen/encrypt_opencl_kernel.cc': target 'gen/encrypt_opencl_kernel.cc' not declared in package '' defined by /private/var/tmp/_bazel_rqg/a266a722dbdbd6fad8042544d6cf7b92/external/local_opencl_kernel_encrypt/BUILD.bazel and referenced by '//mace/codegen:encrypt_opencl_kernel_gen'
ERROR: /Users/rqg/Playground/mace/mace/codegen/BUILD.bazel:26:1: no such target '@local_opencl_kernel_encrypt//:gen/encrypt_opencl_kernel.h': target 'gen/encrypt_opencl_kernel.h' not declared in package '' defined by /private/var/tmp/_bazel_rqg/a266a722dbdbd6fad8042544d6cf7b92/external/local_opencl_kernel_encrypt/BUILD.bazel and referenced by '//mace/codegen:encrypt_opencl_kernel_gen'
ERROR: Analysis of target '//mace/libmace:libmace_static' failed; build aborted: Loading failed
INFO: Elapsed time: 0.772s
INFO: 0 processes.
FAILED: Build did NOT complete successfully (3 packages loaded)

If it's hard to make unit tests pass, you can remove your added unit tests.
If the environment is hard to prepare, you can use docker: https://mace.readthedocs.io/en/latest/installation/using_docker.html
But correct compilation without errors is required.


Update README

README shows some commands that no longer work:
$ npm start (previously started the dev server and served the test suite), maybe we can use  bundler with -watch option to achieve the same.
$ npm run doc (this no longer works) we should maybe just execute node make npm:doc for now?

Fixed.


Correctly compute tensor dimensions in deconvolution operation

This PR fixes some errors when computing the output of the deconvolution operation (i.e. convolution transpose). In particular, it correctly handles padding and halo exchanges. It should produce identical results for the existing use-case (2x2 deconv with stride 2).

Didn't verify it myself, but looks good to me.


Broken link (?)

The link in the section "Analyze" on "how to get the LCS trial Project for Customers" is not working.

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

ID: afebec12-9838-906e-1c0a-a435b9be3777
Version Independent ID: dec9bdde-ac60-4c13-72b5-b3709f6b7089
Content: Upgrade Dynamics AX 2012 to Dynamics 365 for Finance and Operations - Finance & Operations | Dynamics 365 | #MSDyn365FO
Content Source: articles/dev-itpro/migration-upgrade/upgrade-overview-2012.md
Service: dynamics-ax-platform
GitHub Login: @tariqbell
Microsoft Alias: tabell

This link is still broken, any update?

Please change
https://diag.lcs.dynamics.com/ UpgradeAnalysisReport/Report/
to
https://diag.lcs.dynamics.com/ UpgradeAnalysisReport/Report/
 is missing.

@robadawy , the two links that you mention above are exactly the same. Can you confirm the change that you are suggesting for the topic?

@Amakyusa Here is the right link: https://go.microsoft.com/fwlink/?LinkId=2017742&clcid=0x409 . The topic will be fixed within the week.


Fixing list formats and link to intro page.

Fixing list formats and link to intro page.

@tdykstra 	please explain why we can't take these changes.

@crgrieve What do you mean by "fixing" -- if you are seeing rendering issues can you send screen shots and let us know browser and OS information so we can investigate?


update machina version, deprecated machina syntax

New Machina version.
Updated syntax per:
https://github.com/ifandelse/machina.js#pulling-machina-into-your-environment

Hey @aprilb, I don't really have any intention of maintaining/using these libraries anymore. I would like to transfer them either into the treetopllc organization, or just transfer them to you directly.

ok cool @dominicbarnes how many of them besides the inline edit?  I can make a list and we can have them moved over --preferrably to the TreetopLLC Org.

Yeah, let's start with the inlineedit* ones, I'll try transferring them now and we'll see what happens.

@dominicbarnes ok sweet

If it goes through successfully, it will break your builds. (You guys aren't working on deployments right now, are you?)

Hey @aprilb I am not able to transfer to @treetopllc. (don't have admin rights) I can't transfer to you until you delete your fork.

@dominicbarnes I'll ping John to get these transferred --thanks for merging in the change.

Ok, I'll need temporary admin access to treetopllc to do so, or I can transfer them to either you or john who can then make the final transfer into treetopllc. I'm cool either way, so let me know.

ok, will do.

fwiw, I've published v0.0.8 of this repo with this PR.
Just let me know what other libs you guys want, I'll most likely transfer all of them. There may be a handful I'll want to maintain myself, but unless that's the case I don't want to be a road-block to you.


Showoff/Demo picture

It is needed an example picture, showing off the capabilities of the library.
Ideally, it will be included right after main description in the README.md

Something like that, but designed to appear in the README.md


onBackdropPress not working on IOS

Environment
info Fetching system and libraries information...
System:
    OS: macOS 10.15.4
    CPU: (12) x64 Intel(R) Core(TM) i7-8850H CPU @ 2.60GHz
    Memory: 614.26 MB / 16.00 GB
    Shell: 3.2.57 - /bin/bash
  Binaries:
    Node: 10.16.2 - /usr/local/bin/node
    Yarn: 1.21.1 - /usr/local/bin/yarn
    npm: 6.13.4 - /usr/local/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  SDKs:
    iOS SDK:
      Platforms: iOS 13.4, DriverKit 19.0, macOS 10.15, tvOS 13.4, watchOS 6.2
    Android SDK:
      API Levels: 23, 27, 28, 29
      Build Tools: 28.0.3, 29.0.2
      System Images: android-28 | Intel x86 Atom_64, android-29 | Google Play Intel x86 Atom
  IDEs:
    Android Studio: 3.5 AI-191.8<IP_ADDRESS>10548
    Xcode: 11.4.1/11E503a - /usr/bin/xcodebuild
  npmPackages:
    react: 16.9.0 => 16.9.0
    react-native: 0.61.4 => 0.61.4
  npmGlobalPackages:
    create-react-native-app: 2.0.2
    react-native-cli: 2.0.1

Platforms
IOS
Versions

iOS: 12.0
react-native-modal: 11.5.4
react-native: 0.61.4
react: 16.9.0

Description
onBackdropPress seems to have no effect on IOS, work well on Android.
Reproducible Demo
           <Modal
            isVisible={this.state.isModalMoreVisible}
            backdropOpacity={0.40}
            onBackdropPress={() => { this.setState({ isModalMoreVisible: false })} }>
            <View style={{
              flex: 1,
              flexDirection: 'column',
              alignItems: 'flex-end'}}>
              <View style={{flex:1}}>
                    <Text style={styles.modalText}> {strings.view_video.modaltextreport}</Text>
              </View>
            </View>
          </Modal>

@leoz2007 I faced the same problem, but I managed to fix it by not having flex: 1 in the View. Instead, apply the style directly to the Modal
<Modal
    style={{
      flex: 1,
      flexDirection: 'column',
      alignItems: 'flex-end'
    }}
    isVisible={this.state.isModalMoreVisible}
    backdropOpacity={0.40}
    onBackdropPress={() => { this.setState({ isModalMoreVisiblefalse })} }
>
    <View style={{flex:1}}>
      <Text style={styles.modalText}> {strings.view_videmodaltextreport}</Text>
    </View>
  </Modal>

Perfect solution, thank you @chungweileong94


ObsListItem component

This component is made up of a few smaller components. It should include:
Props:

[x] observation: the observation being rendered
[ ] onPress: function that gets fired when the user taps on the item We ended up moving press interaction out of this.

On the start side:

[x] Observation photo with default size of 62 x 62 and a border radius of 8px (this might be a shared component with ObsGridItem... but probably simpler for it not to be)

In the middle:

[x] Observation taxon name (#435)
[x] Observation location (#438)
[ ] Observation date/time (#437)

The end side should either show:

[x] Observation status if doesn't need sync (#434)
[ ] Upload status if needs sync (#400)

...and

[x] gutters between the start, middle, and end sections should be 10px

FWIW, this component may need to accept props related to uploading the observation as well, but IMO we should decide how to do that when we integrate My Observations.
Full spec is here

Looks like there's a lot of overlap w/ #436, I can work on this after that PR goes in. Should be pretty simple after

@Chrischuck I think the current approach of just the gradient is fine for now. Looks like we're just waiting on some of the icon issues resolve at #400 for this.

This should be completed via #400 and #510


⚠ Bofrost.at has degraded performance

In 719638f, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2260 ms

Resolved: Bofrost.at performance has improved in 2e24948 after 6 minutes.


Add more program context to json output

This is known to be an issue with the json output, but perhaps it applies more widely.
Currently proofs only include a line number, which is not enough to disambiguate and pinpoint when multiple proofs are present in a single source line.
It is not clear to me if we need to do something special in the presence of macros?

Thank you for suggesting this addition to the json output!
Program contexts have now been added, with commit https://github.com/static-analysis-engineering/CodeHawk-C/pull/50/commits/9c7b66642432446cad978d492089a0e46ad40ae8 .
A description and example are given here: https://github.com/static-analysis-engineering/CodeHawk-C/pull/50#issue-2634579910

Leaving this issue open to also cover the investigation of adding column numbers to the program context.


FlatList More complex, multi-select example

The example provided in the FlatList article is supposed to rerender the FlatList only when passing extraData prop. However, the FlatList rerenders on IOS and Android whether extraData prop is passed or not. The strange thing is that the example works fine on web.

I believe there is a mistake in this example. The documentation of FlatList states that:

This is a PureComponent which means that it will not re-render if props remain shallow-equal. Make sure that everything your renderItem function depends on is passed as a prop (e.g. extraData) that is not === after updates, otherwise your UI may not update on changes. This includes the data prop and parent component state.

The problem is that the renderItem prop is passed as an inline function, thus the section list will rerender in every App's rerender. Therefore, the extraData parameter is useless in this example. The documentation should show another example of how to use the extraData prop correctly.

Also, can you reword this phrase ?

Make sure that everything your renderItem function depends on is passed as a prop (e.g. extraData) that is not === after updates, otherwise your UI may not update on changes.

Seems grammatically incorrect (which is the verb, "is" or "updates" ?).

@stale This should remain open


Install folly in setup-centos8.sh

As described #3656.

Hi, @kgpai @assignUser @zhixingheyi-tian could you help review this? much thanks.


Slidebox: pager dissapears on iOS device if it is displayed over a background

I'm using a slidebox where the ion-slides have a background.
When I try to run this in the simulator, the pager disappears and reappears when scrolling ion-content up and down. Please see this video for an illustration of what is happening:
https://dl.dropboxusercontent.com/u/10543747/slider_issue.mov
Basically once a div that goes underneath the pager, that div clips over the pager once you scroll past a certain point. I have no idea why this is happening. Z-index on the pager seems to be ignored.
It could have something to do with the fact that when you scroll the pager under the status bar, it dims a little bit. However, I can't pinpoint if this dimming effect is something in the CSS, since the DOM doesn't seem to change when inspecting. Any ideas? It looks really buggy right now.

Hi,
Got the exact same issue described by @CoenWarmer with ionic v1.0.0 "uranium-unicorn".
It seems to be elements with position:absolute over the slidebox.
Those elements twinkle on scroll, only on iOS.
And yes 'It looks really buggy'.

Annoying indeed. I have just found the following solution though...
http://forum.ionicframework.com/t/slidebox-pager-issue-while-scrolling-view/30767/2
It worked for me by adding translateZ to the slider-pager as well as other position:absolute elements.
.slider-pager {
-webkit-transform: translateZ(0);
transform: translateZ(0);
}

@hasandogu Thank you ! It works for me


feat: File restructure

This PR reorganises (mostly) the sass files to follow convention a bit more, and also to make things more predictable and findable, as well as easier to import!

Thank you! I spend a lot of time hanging out in sass files, so this makes my life much better. I get lost in code trees in vscode a lot.


Problem including 3rd party css when the project is in an src directory

Hello,
I was struggling with including 3rd party css files from the node_modules directory, it would keep scoping its rules like it does with css in your src directory, obviously breaking the style. The problem was that my whole project was in an src directory, and the webpack config just tests for the string src being in the url:
https://github.com/davezuko/react-redux-starter-kit/blob/master/build/webpack.config.js#L134-L176
It's a pretty fringe case I know, but it did waste me an hour or so.

Would adding an exclude on node_modules for those loader fix this? My guess is yet, but I don't have anything to test it with right this minute.

Steps to reproduce:

Create a folder called src
Clone the repository inside it
Install a third party react component that has a css dependency, like fixed-data-table or a datepicker
Include this component in HomeView.js for example

The styles of the third party component will not appear because they will be namespaced.
You can do it with node_modules but I think just making the src path relative (/\.\/src/) is a better solution.

@borlaym, did making the include parameter of the loader relative (/\.\/src/) fix it for you?
I am on Windows so had to change the path separator (/\.\\src/) but still this doesn't work as the regex is tested against the absolute path of the file.
I think testing against the /src/ regex is a bit misleading. When I read the config, I thought intended behaviour was to enable CSS modules for all CSS/SCSS in ./src/ and disable it for all CSS/SCSS outside ./src/ (e.g. node_modules), but the broad regex can catch a lot of extra cases.
I have made my config much more explicit:
webpackConfig.module.loaders.push({
  test: /\.scss$/,
  include: pathStr => pathStr.startsWith(paths.client()),
  loaders: [
    'style',
    cssLoader,
    'postcss',
    'sass?sourceMap'
  ]
})

webpackConfig.module.loaders.push({
  test: /\.css$/,
  include: pathStr => pathStr.startsWith(paths.client()),
  loaders: [
    'style',
    cssLoader,
    'postcss'
  ]
})

// All other CSS/SCSS is global by default
webpackConfig.module.loaders.push({
  test: /\.scss$/,
  exclude: pathStr => pathStr.startsWith(paths.client()),
  loaders: [
    'style',
    'css?sourceMap&-minimize',
    'postcss',
    'sass?sourceMap'
  ]
})

webpackConfig.module.loaders.push({
  test: /\.css$/,
  exclude: pathStr => pathStr.startsWith(paths.client()),
  loaders: [
    'style',
    'css?sourceMap&-minimize',
    'postcss'
  ]
})

Should be fixed with https://github.com/davezuko/react-redux-starter-kit/commit/b0b5e7177ecb34032801557ff857e07b10d79da9. Sorry it took so long.

@davezuko How would one require a css file in node_modules?
Current Location:
/front/src/views/myView/myView.js
And I tried
require('../../node_modules/react-datepicker/dist/react-datepicker.min.css')
require('/node_modules/react-datepicker/dist/react-datepicker.min.css')


fix(vue-app): don't throw if we can't read sessionStorage

🔗 Linked issue
resolves https://github.com/nuxt/nuxt/issues/10279
📚 Description
In some situations session storage will throw an error. We can prevent and ignore that error, just as we do in Nuxt 3.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 66.23%. Comparing base (13ff5d7) to head (18694e7).

Additional details and impacted files
@@           Coverage Diff           @@
##              2.x   #27662   +/-   ##
=======================================
  Coverage   66.23%   66.23%
=======================================
  Files          93       93
  Lines        4123     4123
  Branches     1170     1170
=======================================
  Hits         2731     2731
  Misses       1126     1126
  Partials      266      266

Flag
Coverage Δ

unittests
66.23% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Sometimes doesn't work

does not work because there is no text in the console that is responsible for this. There is such a text, is it possible somehow for the program to react to it?
Started tracking Steam Net Connection to =[A:1:****]:0, handle 53d42bf8

Thanks for the issue, I'll take a look later

Fixed, thx for issue. <3

I updated and did as you said, but after that, when I started checking whether the match is located or not, it either worked or not

Do you speak in Russian?

https://discord.gg/bVFGGaHG


Redo changes lost around Februrary 7/8th

#878 got reverted by #883. https://github.com/arangodb/docs/pull/958/commits/a1fd5218def4031995b5a9376ec8fd2494c64f42 is preserved by this PR.
Also redo another change that was lost accidentally (#952 was reverted by #954).
Fix a DocuBlock rendering issue, e.g. group query parameter: before vs. after

@ansoboleva We need to re-generate the 3.10 examples:
https://deploy-preview-979--zealous-morse-14392b.netlify.app/docs/3.10/aql/functions-string.html

https://jenkins.arangodb.biz:3456/view/Documentation/job/arangodb-ANY-examples/1891/


Redirect source URLs ending in a slash should also match slashless URL

What version of Wrangler are you using?
2.0.15
What operating system are you using?
Linux
Describe the Bug
It is widespread standard practice to redirect "directory" URLs not ending in a slash to the equivalent (canonical) URL ending in a slash, provided the directory index exists, and Cloudflare Pages already does so.
For instance, https://www.example.com/foo would typically redirect to https://www.example.com/foo/.
As such, I strongly believe that a redirect like /foo/ /bar/ 301,
as well as redirecting /foo/ to /bar/,
should also redirect /foo to /bar/.
Another argument is that, let's say you have a directory index at /foo/ (and /foo), and later you decide to delete it and replace it with a redirect with source URL /foo/, then the /foo URL (redirect to canonical) will not work anymore, which doesn't seem logical. And I think /foo would still work if you added the redirect but (deliberately or accidentally) did not delete the file, which is a bit weird.
For now, the workaround is to manually create that second redirect every time, but this seems gratuitously painful.
(Note that a redirect of /foo /bar/ 301 should NOT also redirect /foo/, because /foo.txt /bar.txt 301 clearly should not redirect /foo.txt/.)

We did reproduce your issue, and it does make sense to fix. We've added to our backlog. Thanks! ☺

Now that Cloudflare Pages supports wrangler.toml, it would be easy to make the new behavior optional, retaining the old behavior by default for backwards compatibility.
How about also-redirect-slashless = true? The default would be also-redirect-slashless = false.


General scrooge performance

Right now, the code generator is not as performant as it could be... of course, this is mainly with antipatterns (a ton of thrifts which depend on a ton of thrifts) that hopefully bazel will prevent, but which exist at many companies which heavily use thrift. There is a lot of work that could be done to make this maximally performant... some ideas

Make scrooge imports jars https://github.com/bazelbuild/rules_scala/issues/44
Use the full graph of srcjars/thrift jars to minimize the number of times we extract jars
??? (there are scrooge changes that could also probably better support this... such as keeping around some intermediate data or something, but at that point maybe we just want smaller targets...)

I think most of these issues have been solved by: #143 and #137


onsets_frames_transcription_transcribe on stream

The current code of onsets_frames_transcription_transcribe.py works by
opening the WAV file a one single audio-chunk using process_records
https://github.com/tensorflow/magenta/blob/6f10a4f64b6c6e56480898aba95720394e8f8324/magenta/models/onsets_frames_transcription/onsets_frames_transcription_transcribe.py#L61-L72
Sadly both functions expect a finite amount of data:

onsets_frames_transcription_transcribe expects a regular file to open (no stdin/parec-type stream)
process_record works based on a Python list which is serialized before preprocessing is applied.

As such, no pipelining is possible.
But histogram generation implies WAV files preprocessing and splitting using framed samples anyway. This split is based on spec_hop_length which default to 512 (= sample_rate / 512 samples per second)
https://github.com/tensorflow/magenta/blob/6f10a4f64b6c6e56480898aba95720394e8f8324/magenta/models/onsets_frames_transcription/configs.py#L41
As such, there should exist a way to make the inference to work on a stream by cutting it in chunks of spec_hop_length * X milliseconds during an earlier stage?

Ouch, just discovered https://github.com/tensorflow/magenta/tree/master/magenta/models/onsets_frames_transcription/realtime


Upgrade log4j dependencies to v2.17.1

Fixes

More vulnerabilities in log4j 2.17.0 (Security Issue)

Proposed Changes

Update jars and references for log4j (download : https://logging.apache.org/log4j/2.x/download.html)

Codecov Report

Merging #200 (9650bdd) into master (7a49092) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #200   +/-   ##
=======================================
  Coverage   94.14%   94.14%
=======================================
  Files          70       70
  Lines        1827     1827
=======================================
  Hits         1720     1720
  Misses        107      107

Impacted Files
Coverage Δ

lib/codenarc-caller.js
83.45% <ø> (ø)

Continue to review full report at Codecov.

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

Thanks ! :)


Add a flag and api to disable features highlighting on hover

This will be used in flatmapvuer when the explore nearby paths tool is on

@Tehsurfer I'm tidying things up and can't recall whether you what you were wanting to do with this PR has now been done some other way -- the code in mapviewer to do with this._lastFeatureMouseEntered and this._lastFeatureModelsMouse doesn't in fact do anything apart from track state...


WFLY-15687 Add support for CDI 4 Build compatible extensions

This PR superseeds https://github.com/wildfly/wildfly/pull/15464. I have squashed Scott's commits into one and added mine on top.
JIRA link - https://issues.redhat.com/browse/WFLY-15687
There are still some remaining TCK failures, namely 12 of them.

[ERROR] Tests run: 1851, Failures: 12, Errors: 0, Skipped: 74

10 of these are JSF related (old artifact using now removed deprecated APIs) - fixed by updating JSF impl artifact.
The remaining two are what concerns us:

SyntheticBeanWithLookupTest- this one is (most likely) in WFLY's integration code but I am failing to see how to properly fix that. The last commit in this PR adds a very nasty hack which makes the test pass, but I am anything but happy with it :-/

As captured in the comments below, this will now be addressed by https://issues.redhat.com/browse/WELD-2718

StartupShutdownTest - this one needs to be fixed on Weld's side, see https://issues.redhat.com/browse/WELD-2717, there is already a pending PR

CC @starksm64

@manovotn Ok, great, I'll see if can get our Mojarra fork updated.

@manovotn Ok, great, I'll see if can get our Mojarra fork updated.

@starksm64 That fork is long updated now. I first sent https://github.com/jboss/mojarra/pull/90 which was forgotten and then later Brian did https://github.com/jboss/mojarra/pull/92 which got merged.
But I think there was no release since.

I'm working on making a release of jsf-impl now. There's some weirdness with the versions I need to figure out, but I'm actively working on this.

I've spent some "quality" tie debugging the problem with SyntheticBeanWithLookupTest and I think I've found a way to workaround this in a cleaner fashion directly in Weld.
Following JIRA captures it along with PR - https://issues.redhat.com/browse/WELD-2718
Therefore, I'll remove the hacky approach from this PR and we can probably turn the draft into actual PR.
Once Weld 5 Final is out, WFLY can update and with that should have a passing CDI TCK.

The JSF update now has a PR: https://github.com/wildfly/wildfly/pull/15487

Nice @jasondlee, thanks!
Once we merge both PRs, I could bring back WFLY testing on Weld PRs (if there is a nightly WFP build).

@manovotn The JSF upgrade is merged. We produce a WFP wildfly-preview-latest-SNAPSHOT.zip nightly from the  https://ci.wildfly.org/viewType.html?buildTypeId=WF_WildflyPreviewNightly job.

/retest

@manovotn
Please add entries for the 2 new deps to:
https://github.com/wildfly/wildfly/blob/main/ee-9/common/src/main/resources/license/preview-feature-pack-common-licenses.xml
https://github.com/wildfly/wildfly/blob/main/ee-9/ee-9-api/src/main/resources/license/preview-feature-pack-ee-9-api-licenses.xml
Note that the elements in those files are ordered by maven GA.
Other than that, LGTM.

@bstansberry updated

I have also released Weld 5 Final today, so I'll update this PR with it

Those jobs failed as well, including with SE 17.
The dependencyManagement we use when running the MP 5 TCKs with WFP is out of date though, so I'm going to get that cleaned up and then I'll see if that makes the problem go away.

@manovotn @kabir The WildFly Preview job failed with a java.lang.VerifyError in the MP Reactive Messaging TCK

I am afraid I have no idea why. I am not aware of any change in Weld between CR2 and Final that could cause it. Besides, if it were in Weld, I assume we'd see failures sooner and/or all over the place.
Did the TCK config change recently?

@manovotn I'm confused as well. I thought it might be due to WFLY-16337 (i.e. the changes here surfaced an existing problem) but a test branch combining this and a fix for that had the same issue:
https://github.com/wildfly/wildfly/pull/15516
I'll discuss with @kabir as he's our RSO expert. AIUI these failures are on the client side. The client-side classpath is tricky.
We've been running PRs against WFP+SE17 for a long time, plus nightlies. So something is odd here.

VerifyError

https://www.eclipse.org/lists/cdi-dev/msg00586.html is reporting the same VerifyError

@manovotn I've been trying simple component upgrades (i.e. not the rest of this PR) and this problem appears with this commit:
https://github.com/weld/core/commit/1f0b5c63739df5d94f1ac21a4f835a93e16141cd
To reproduce in ee-9/pom.xml I update the CDI API to 4.0.0, weld-api to 5.0.CR2, and weld-core to a SNAPSHOT built from that sha. Build the server and then
mvn install -Dts.ee9 -pl testsuite/integration/microprofile-tck/reactive-streams-operators
If I use weld 5.0.0.CR2 instead of that snapshot the problem doesn't appear.

@bstansberry the commit you linked is a POM change in Weld's SE artifacts and those aren't used in WFLY at all from what I know. Also performing a build with a revert on this commit doesn't solve anything.
If I were to guess, you meant https://github.com/weld/core/commit/18af30b7ae367ca2dac107d22b64c5c596c7bb6d?
That one seems to be the turning point; if I revert that, I get the TCK to pass.
That's weird, the issue was linked primarily to self interception and if I am not mistaken, the class being proxied is this one and that seems just fine. At least we know what's causing it, but I am still trying to grasp why.

Based on:
https://stackoverflow.com/questions/42707610/use-of-stackmap-frames-and-how-it-helps-in-byte-code-verification
And the code in this error msg:
org/glassfish/jms/injection/RequestedJMSContextManager$Proxy$_$$_WeldClientProxy.toString()Ljava/lang/String;
@4: ifne
Reason:
Expected stackmap frame at this location.
Bytecode:
0000000: 2ab4 0018 9a00 082a b700 5bb0 b800 2e4c
0000010: 2ab4 0034 b600 3ac0 001c 59b6 005c 2b59
0000020: c600 0cc0 0030 b600 3da7 0004 57a7 0013
0000030: 2b59 c600 0cc0 0030 b600 3da7 0004 57bf
0000040: 5aa5 0004 b02a c000 28b0
Exception Handler Table:
bci [16, 48] => handler: 48
Which maps to:
Begin method:
2a: __ALoad0
b4: GetField(18-66)
9a: __IfNe(82a)
2a: __ALoad0
b7: InvokeSpecial(5b-50)
b0: __AReturn
b8: InvokeStatic(2e4c)
4c: __AStore1
2a: __ALoad0
b4: GetField(34-4a)
b6: InvokeVirtual(3a-40)
c0: CheckCast(1c59)
59: Dup
b6: InvokeVirtual(5c2b)
2b: __ALoad1
59: Dup
c6: __IfNonNull(c-40)
c0: CheckCast(30-4a)
b6: InvokeVirtual(3d-59)
a7: Goto(457)
57: Pop
a7: Goto(132b)
2b: __ALoad1
59: Dup
c6: __IfNonNull(c-40)
c0: CheckCast(30-4a)
b6: InvokeVirtual(3d-59)
a7: Goto(457)
57: Pop
bf: AThrow
5a: DupX1
a5: __IfACmpEq(4-50)
b0: __AReturn
2a: __ALoad0
c0: CheckCast(28-50)
b0: __AReturn
End method:
It seems the verifier is expecting a stackmap entry for the Ifne
instruction jump that is not there. You have to see the full class file to
be able to see the stackmap table. Maybe that protected stuff is requiring
a new stackmap entry.
On May 2, 2022 at 6:04:14 PM, Matej Novotny @.***>
wrote:

@bstansberry https://github.com/bstansberry the commit you linked is a
POM change in Weld's SE artifacts and those aren't used in WFLY at all from
what I know. Also performing a build with a revert on this commit doesn't
solve anything.
If I were to guess, you meant @.***
https://github.com/weld/core/commit/18af30b7ae367ca2dac107d22b64c5c596c7bb6d
?
That one seems to be the turning point; if I revert that, I get the TCK to
pass.
That's weird, the issue was linked primarily to self interception and if I
am not mistaken, the class being proxied is this one
https://github.com/eclipse/microprofile-reactive-streams-operators/blob/master/tck/src/main/java/org/eclipse/microprofile/reactive/streams/operators/tck/arquillian/ReactiveStreamsCdiTck.java
and that seems just fine. At least we know what's causing it, but I am
still trying to grasp why.
—
Reply to this email directly, view it on GitHub
https://github.com/wildfly/wildfly/pull/15474#issuecomment-1115449467,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACRDMSHFUVPCQDB6PPSEMTVIBNO5ANCNFSM5UMHQPUA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Sorry for the bad link @manovotn :(
Yes, https://github.com/weld/core/commit/18af30b7ae367ca2dac107d22b64c5c596c7bb6d is the one I meant

So, the https://github.com/weld/core/commit/18af30b7ae367ca2dac107d22b64c5c596c7bb6d is definitely the cause.
It seems to happen when there is a hierarchy of at least two classes, where the super class is in a different package and has protected methods which are then subject to proxying by Weld.
The error, however, is completely off because those proxy files don't even have a stackmap in the first place! (you can see that if you inspect them with javap -v -p)
jboss-classfilewriter apparently creates version 50.0 files (JDK 6) which can, by JLS definition, lack the stackmap in which case JVM does type inference. So the VerifyError is actually just misleading and the cause if likely somewhere else. This is also confirmed by the fact that the error points to toString method which hasn't changed in any way and the generated bytecode for this method is identical between CR2 and Final.
That said, I was unable to determine the actual cause so far.
All I know is that from the offending commit, the change is that protected methods on proxy are now invoked via invokevirtual/invokeinterface instead of via method handler invocation (reflection). This should, in theory, be valid but instead blows up :-/

In case it's useful, ReactiveStreamsCdiTck is a javax.* namespace class, which means it's getting bytecode transformed when installed into WFP.

It might be interesting to see the difference in bytecode between the original and transformed class. @ropalka (I think you worked on this) - is there a hook for @manovotn to be able to dump the transformed class bytes?

@kabir I managed to create a reproducer yesterday and have bytecode (weld can dump all created proxies via an option).
Still didn't help though; I suspect that there is a catch somewhere in classfilewriter in combination with bytecode verification but it is more of a guess than anything else.
I will roll back the problematic commit and issue a new Weld release.

Thanks @manovotn!
FYI, the Windows - JDK 11 job is going to show some weird failures that we're investigating and that have popped up elsewhere, so not related to this PR. So don't worry about them.

Thanks, @manovotn & @starksm64 !

I did not make that comment 8 hours ago, so where did it come from? It seems like it would have been a comment made on May 5 where I then commented about the bytecode in the stack trace. It certainly is not relevant now.

I did not make that comment 8 hours ago,

Yea, that's what confused me. Probably a glitch in the Matrix :)


Invariant Violation: DatePickerAndroid has been removed from React Native. It can now be installed and imported from '@react-native-community/datetimepicker' instead of 'react-native'. See https://github.com/react-native-datetimepicker/datetimepicker

Issue
Expected Behavior
Code

Environment

react-native -v:
node -v:
npm -v:
yarn --version:
target platform: Android | iOS
operating system:

I solved the problem myself. I forked. you can use. https://github.com/mamicrose/react-native-datepicker

I solved the problem myself. I forked. you can use. https://github.com/mamicrose/react-native-datepicker

Does not work. I get :

nem i @react-native-community/datetimepicker

npm i @react-native-community/d

atetimepicker

I get 'RNTimePicker could not be found' after applying the above solution. I even copied files from your branch but still gives the same error. Please help!

and if you want date and time together, I can share my own code with you. (time screen popping up after selecting date)

Do not use this repository anymore. I am currently using it and am satisfied. https://www.npmjs.com/package/react-native-modal-datetime-picker

Thanks for the response, helped solve my problem :)

https://www.npmjs.com/package/react-native-modal-datetime-picker

@mamicrose can you share code for date and time together.
Thanks in advance

https://www.npmjs.com/package/react-native-modal-datetime-picker

@mamicrose can you share code for date and time together.
Thanks in advance

you can use it ready-made as a component.
import { View, Text, TouchableOpacity } from "react-native";
import DateTimePickerModal from "react-native-modal-datetime-picker";
import { useCallback, useState } from "react";
import moment from "moment";
import "moment/locale/tr";
moment.locale("tr");

interface Props {
  date: Date;
  onChange(date): void;
  label: string;
  mode: "date" | "datetime";
}

export default function DatePicker({ date, onChange, label, mode }: Props) {
  const [dateShow, setDateShow] = useState<boolean>(false);
  const [timeShow, setTimeShow] = useState<boolean>(false);

  const handleChange = useCallback(
    (e: Date) => {
      setDateShow(false);
      if (mode === "datetime") {
        setTimeout(() => {
          setTimeShow(true);
        }, 700);
      }
      onChange(e);
    },
    [setDateShow, onChange, mode]
  );

  const handleSetTime = useCallback(
    (e: any) => {
      setTimeShow(false);
      onChange(e);
    },
    [setTimeShow, date]
  );

  let momentType;
  if (mode === "date") {
    momentType = "DD-MM-YYYY";
  } else if (mode === "datetime") {
    momentType = "DD-MM-YYYY LT";
  } else {
    momentType = "DD-MM-YYYY LT";
  }

  return (
    <View>
      <Text className="dark:text-white">{label}</Text>
      <TouchableOpacity
        activeOpacity={0.7}
        onPress={() => setDateShow(true)}
        className="w-full bg-white border border-gray-300 rounded-lg px-3 py-2 mt-1"
      >
        <Text>{moment(date).format(momentType)}</Text>
      </TouchableOpacity>
      <DateTimePickerModal
        date={date}
        isVisible={dateShow}
        mode="date"
        onConfirm={handleChange}
        onCancel={() => setDateShow(false)}
        confirmTextIOS="Onayla"
        cancelTextIOS="İptal"
        locale="tr-TR"
        is24Hour={false}
      />

      {mode === "datetime" && timeShow && (
        <DateTimePickerModal
          date={date}
          isVisible={true}
          mode="time"
          onConfirm={handleSetTime}
          onCancel={() => setTimeShow(false)}
          confirmTextIOS="Onayla"
          cancelTextIOS="İptal"
          locale="tr-TR"
          is24Hour={true}
        />
      )}
    </View>
  );
}


CI: Migrate to Python framework

Many thanks for submitting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[ ] You have read the contributors guide
[ ] Pull Request title is properly formatted: FAI-XYZ Subject
[ ] Pull Request title contains the target branch if not targeting main: [0.3.x] FAI-XYZ Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains link to any dependent or related Pull Request
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/retest

/retest


Indexer API response with ERROR: permission denied for table app_box (SQLSTATE 42501)

Subject of the issue
When using the SDK method searchForApplicationBoxes or accessing the API at 'v2/applications/{appId}/boxes' server responds with an error "permission denied for table app_box (SQLSTATE 42501)"
The error occurs when using testnet and mainnet url. When used locally in a sandbox environment, it works correctly

It sounds like you have a readonly account to access the database. Boxes use a new table so that user might need to be updated: https://github.com/algorand/indexer/#read-only


Setup of integration failing during oauth

Describe the bug
New setup with latest HA 2023.4 failing at the end of the integration setup with a 400 bad request
I tried to setup the integration for the first time today and created a developer account, created a new application, added
https://my.home-assistant.io/redirect/oauth
as redirect URL and used the same mail address as user as I'm using in the home connect app.
My dryer is configured in the app already and has also access to the internet, checked while using 4G and the home connect app is able to talk to the dryer.
At the end of the integration setup, while getting back from the "Link account to home assistant" page (which shows my public URL and port as instance URL), the integration setup just shows "Error"
I assume it has sth to do with the oauth overall and maybe not specific with this integration, but as this is the only integration that I'm using, which uses oauth, I can't say if there is sth missing or maybe wrong with 2023.4
I've restarted HA several times already, deleted and recreated the app in the developer portal and updated the id and secret accordingly in the configuration.yaml, but it always behaves the same.
Version of the custom_component
0.6.1
Configuration
# Loads default set of integrations. Do not remove.
default_config:

http:
  ssl_certificate: /ssl/fullchain.pem
  ssl_key: /ssl/privkey.pem

# Text to speech
tts:
  - platform: google_translate

automation: !include_dir_merge_list automations
binary_sensor: !include_dir_list binary_sensors
device_tracker:  !include_dir_list devices
group: !include_dir_named groups
input_boolean: !include input/boolean.yaml
input_select: !include input/select.yaml
input_number: !include input/number.yaml
media_player: !include_dir_list media_players
notify: !include_dir_list notify
scene: !include_dir_list scenes
script: !include_dir_named scripts
sensor: !include_dir_merge_list sensors
shell_command: !include shell_commands.yaml
switch: !include_dir_list switches
template:
  - binary_sensor: !include_dir_merge_list templates/binary_sensor/
  - sensor: !include_dir_merge_list  templates/sensors/
  - trigger: !include_dir_merge_list templates/trigger/
timer: !include_dir_named timers
zone: !include_dir_list zones

home_connect_alt:
  client_id: xxxxxxxxxxxxxxxx
  client_secret: xxxxxxxxxxxxxxxxxx

logger:
  default: info
  logs:
    homeassistant.helpers.config_entry_oauth2_flow: debug
    home_connect_async: debug
    home_connect_alt: debug
    custom_components.home_connect_alt: debug

Debug log
Logger: aiohttp.server
Source: helpers/config_entry_oauth2_flow.py:209
First occurred: 7:34:11 PM (1 occurrences)
Last logged: 7:34:11 PM

Error handling request
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/site-packages/aiohttp/web_protocol.py", line 433, in _handle_request
    resp = await request_handler(request)
  File "/usr/local/lib/python3.10/site-packages/aiohttp/web_app.py", line 504, in _handle
    resp = await handler(request)
  File "/usr/local/lib/python3.10/site-packages/aiohttp/web_middlewares.py", line 117, in impl
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/security_filter.py", line 85, in security_filter_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/forwarded.py", line 100, in forwarded_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/request_context.py", line 28, in request_context_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/ban.py", line 80, in ban_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/auth.py", line 235, in auth_middleware
    return await handler(request)
  File "/usr/src/homeassistant/homeassistant/components/http/view.py", line 146, in handle
    result = await result
  File "/usr/src/homeassistant/homeassistant/components/config/config_entries.py", line 171, in get
    return await super().get(request, flow_id)
  File "/usr/src/homeassistant/homeassistant/helpers/data_entry_flow.py", line 96, in get
    result = await self._flow_mgr.async_configure(flow_id)
  File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 271, in async_configure
    result = await self._async_handle_step(
  File "/usr/src/homeassistant/homeassistant/data_entry_flow.py", line 367, in _async_handle_step
    result: FlowResult = await getattr(flow, method)(user_input)
  File "/usr/src/homeassistant/homeassistant/helpers/config_entry_oauth2_flow.py", line 315, in async_step_creation
    token = await self.flow_impl.async_resolve_external_data(
  File "/usr/src/homeassistant/homeassistant/helpers/config_entry_oauth2_flow.py", line 172, in async_resolve_external_data
    return await self._token_request(
  File "/usr/src/homeassistant/homeassistant/helpers/config_entry_oauth2_flow.py", line 209, in _token_request
    resp.raise_for_status()
  File "/usr/local/lib/python3.10/site-packages/aiohttp/client_reqrep.py", line 1005, in raise_for_status
    raise ClientResponseError(
aiohttp.client_exceptions.ClientResponseError: 400, message='Bad Request', url=URL('https://api.home-connect.com/security/oauth/token')

I tried it already, with just using the IP, same result (even if I would expect a different error message, due to a certificate error). I've googled several hours before posting and none of the found "solutions" helped.

I've recreated the application in the dev portal, waited and used the new ID/Secret
I've logged out of everything prior to setting up the integration
I've removed the HA instance from my Pi-Hole
I've removed the internal DNS resolution of my HA and resolved the name via internet DNS
I've even added my HA URL as additional redirect URL

I'm running out of ideas, on what could be misconfigured
Just to be on the safe side, I do not need to have a HA Cloud (Nabu Casa) subscription, correct?

You don not need a nabu casa subscription.
Read this issue #103
Make sure you use you HA internal address both for adding the integration and for setting the HA instance address in my.home-assistant.io
If this doesn't help then please post a screenshot of your HC app config, a screenshot of the my.home-assistant.io page you landed on after completing the authentication process and the full URL of that page (copy from the browser).

looks like it had sth to do with the mail, even if I read that before registering, seems that some autocorrect might have upped the first letter of the mail. I've deleted all accounts and re-registered with the same mail but different +aliases and also used a different developer portal account "name".
Got it registered.

I solved it by changing the registration email on home connect and on developers, in short, your email MUST be ALL lowercase

Now i have this error:
Logger: custom_components.home_connect_alt
Source: /config/custom_components/home_connect_alt/init.py:132
Integration: Home Connect Alt (documentation, issues)
First occurred: 18:43:46 (1 occurrences)
Last logged: 18:43:46
Failed to load data for the HomeConnect object
Traceback (most recent call last):
File "/usr/local/lib/python3.10/site-packages/home_connect_async/homeconnect.py", line 139, in async_load_data
response = await self._api.async_get('/api/homeappliances')
File "/usr/local/lib/python3.10/site-packages/home_connect_async/api.py", line 118, in async_get
return await self._async_request('GET', endpoint)
File "/usr/local/lib/python3.10/site-packages/home_connect_async/api.py", line 113, in _async_request
raise HomeConnectError("Failed to get a valid response from Home Connect server", 902)
home_connect_async.common.HomeConnectError: ('Failed to get a valid response from Home Connect server', 902, None, None, None)


Add new ExitHandler API as an alternative to ExitGuard

Why?
I need a better system for handling process exit to handle flushing to cache hit statistics (#8286).
What?

Supports letting another thing listen for ctrl + c. In the case of next-server, we need to let node.js own and configure these signal handlers.
Supports setting up a ctrl + c handler for you, but now explicitly panics if you would try to set up multiple global ctrl + c listeners.
Allows async work to happen during cleanup. This wasn't possible using Drop because Drop isn't async.
Allows cleanup work to be scheduled after application initialization, potentially making the API a bit more flexible.

Testing Instructions
cargo test -p turbopack-trace-utils

Add some println!() logging to the end of the on_exit future in turbopack-cli.
TURBOPACK_TRACING=overview cargo run -p turbopack-cli -- dev

Hit ctrl+c and see that my println!() runs, so I know the tracing flush finishes.

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#8286
#8547  👈
#8529
#8542
#8530
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @bgw and the rest of your teammates on  Graphite

Merge activity

Jun 24, 2:41 PM EDT: @bgw started a stack merge that includes this pull request via Graphite.


Support arguments for environment setup script

Thanks for the great plugin. I have to setup my build folder in a particular way, so I need a way to run a custom command just before the configure command. It would be great that this is run together with the configure step. How would I add such a step?
To be more concretely, how would I achieve a workflow like the following:
mkdir ${cmake.buildDirectory}

# this can be any other command
conan install ${workspaceFolder} --build=missing -s build_type=${variant:buildType}

cmake -GNinja ${workspaceFolder}

ninja all

Everything works when I do this in a terminal, it is also integrated into cmake-tools except for the custom command, which I have to manually execute each time when I configure.

You can point "environmenSetupScript" (in your kit definition) to a script that runs the command. This will make sure that the script is run every time the kit is loaded (when VSCode opens your project or when you re-select the kit). It will not be run every time before configure, but since the environment changes are persisted you might need it to be run once in a VSCode session.
Let us know if this doesn't help with your scenario.

I tried the following two things:
"environmentSetupScript": "conan install ${workspaceFolder} --build=missing -s build_type=${variant:buildType}"
and the same, but with a script file with command:
"environmentSetupScript": "setup.sh"
where setup.sh is in the same folder as cmake-kits.json (.vscode)
Am I missing something?
Bear in mind that I need different setup depending on variants and on a file in the workspace folder.

The second scenario is the one we support (path to a scrip file instead of a plain command).
It needs to be a full path and you can use variables as ${workspaceFolder} or ${variant:debug} for example to construct that full path.
Give us more details about this: Bear in mind that I need different setup depending on variants and on a file in the workspace folder. especially about on a file in the workspace folder. If this were depending only on variants, you can create a script for each variant, have them share the same name except the variant names in the same place in the string and then reference with the variant synax.
Let us know how this works for you.

It doesn't seem to work with
"environmentSetupScript": "${workspaceFolder}/setup.sh"
and setup.sh in the correct folder.
I need this setup.sh be run inside in build directory, with different parameters (depending on variants) for each build folder.
My build directories are
"cmake.buildDirectory": "${workspaceFolder}/build/${variant:buildType}-${variant:graphicsBackend}-${buildKit}",
So they take all the variants in account. Making a script for each variant is cumbersome and not ideal (e.g., a few variants with each 4 options can quickly get a lot different setups)

Yes, we are going to explain environmentSetupScript in our documentation, it is a bit out of date regarding some of the new support that's been implemented in the last year.
Currently, the environment setup script is run from its own path. We are going to fix this so that someone can specify where to run the script from.
So, in the meantime, you can place your script in the build folder, point to the build folder instead of the main root workspace folder. Since you have so many build folders, I'm thinking that you can create a special CMake "install" target that would copy setup.sh from the workspace to the current build folder. Let us know if anything changes if you do this.
But before spending the time to try writing this target, maybe copy it manually (see if it needs any command change caused by the migration to a different location) and give it a try once to verify the concept. Maybe this idea of workaround won't work in the end.
Another thing we can look at is the possibility for the user to pasts arguments to the environment setup script. We'll keep you posted with everything we will do to improve the kit environmentSetupScript user scenario.

That's nice to hear.
Running the commands just in a terminal works fine. So, I will just run the command manually for now.

Hello, I am trying to use the "environmentSetupScript " to call a simple .bat script (similar use case to above)
e.g. echo Hello
Should I see any output in the terminal to verify that this script gets called?

This would be helpful.

Hi,
I have previously opened the request #3418 regarding this topic, but never followed up on this original topic.
Please implement this feature.
Calling MSVC environment setup scripts would benefit a lot from this feature.

Providing arguments to the environmentSetupScript in Kits definition would really be helpful when initializing the environment. This option would be of great benefit.

Commenting for support. Please get a fix on this.


History setup bug

Hi!
I was configuring my mappings to jump between buffers in a historic order. And then I noticed that the object returned by require('cokeline.history') (https://github.com/willothy/nvim-cokeline/blob/main/README.md?plain=1#L877 see this example in docs) is not set up (its fields are not initialized).
However _G.cokeline.history works fine :)
This is how the error looks like:

I think it is pretty easy to fix. If I have some spare time I will do it myself and will send a PR.
Also thank you for the awesome plugin!

This is very interesting. I actually figured out that the issue came from the uses of "/" in require calls. Apparently there is some different between . and /, though I'm not quite sure what. Until now I had kept with / because that's what the creator of the plugin did initially, but I've just changed all require calls to use . which should fix it.


set autoplayTimeout to dynamic value not working

Which OS ?
iOS, Android
Version
Which versions are you using:

react-native-swiper v 1.5.14
react-native v0.59.8

Expected behaviour
link autoplayTimeout to state in order to show every view a given amount of time( ex: first image 5s, second image 10s ...).
I'm using onIndexChanged to get the index of the current view, set the index in a redux reducer variable and assign it to autoplayTimeout.
since setting state is not synchronous I'm returning a promise from my action creator
and with the help of await, I'm sure to get the actual index in the redux store
Actual behaviour
i have an array with 3 elements [{url:"", duration:5},{url:"", duration: 10},{url:"", duration: 15}]
the first and second view are shown for respectively 5 and 5 seconds
and the third view is show for 10 seconds.
How to reproduce it>
To help us, please fork this component, modify one example in examples folder to reproduce your issue and include link here.
Steps to reproduce

did you solve this problem? i've this problem too


Add message rewriting for github/gitlab webhook notifications

If possible it would be very nice to add message rewriters for messages from services like GitHub and GitLab. In Matterhorn their rendering is very noisy with useless information (user profile links and avatar images, for example). Here's an example of how they look now:
https://avatars.githubusercontent.com/u/427221?v=3&s=18 jtdaugherty
  (https://github.com/jtdaugherty) commented on
  #116 sendbuf: resource vanished (Broken pipe)
  (https://github.com/aisamanra/matterhorn/issues/116#issuecomment-260016240):
  CC @tommd

and instead I'd love to see
jtdaugherty commented on #116 sendbuf: resource vanished (Broken pipe):
  CC @tommd

Maybe rewrite to something like:
[1] jtdaugherty ([2]) pushed 1 changeset to master ([3]) at aisamanra/matterhorn ([4]): ...
and then have [1] - [4] be openable using C-o

I'm closing this because @aisamanra is onto a much more general solution that improves this across the board rather than doing it just for webhook messages.


Add Gradient-Free Method for SORM

Refer to p. 189 of "Structural and System Reliability".

It has been decided not to add the Gradient-Free method because the principal curvatures are highly dependent on the tolerances used in the FORM ($\epsilon_{1}$ and $\epsilon_{2}$). Additionally, there seems to be no good reason to add a method because its only advantage that it does not compute Hessian matrices. Given the raw computational speed of today's computers and Julia, it is not a problem.


one bug or misunderstand about enabled the profiler.logback.logging.transactioninfo=true

When i enable profiler.logback.logging.transactioninfo=true, following the document https://github.com/naver/pinpoint/blob/master/doc/per-request_feature_guide.md, however, the MDC is not added to logger message.
I check the code of com.navercorp.pinpoint.profiler.DefaultAgent, I found it using
private final Logger logger = LoggerFactory.getLogger(this.getClass());
in my option, above code will load the ch.qos.logback.classic.spi.LoggingEvent before the plugins loaded, so the transform method will never be invoked in com.navercorp.pinpoint.profiler.plugin.MatchableClassFileTransformerGuardDelegate via jvm, as jvm think it is already loaded
i also debug the agent, i found the breakpoint in the doInTransform in LogbackPlugin never be catched.
it is a bug? or i have a misunderstand about it?
I use:
spring boot 1.2.6-release
logback 1.1.3
slf4j 1.7.12

We also found problems when use spring boot with pinpoint.
We will analyze this problem.

I pushed patch code to master branch #1478.
for reference,
Pinpoint core logic uses java logger.
Spring-boot has jul-to-slf4j library as one of its dependencies, and it uses this library to change the java logger handler to logback handler at initialization time.
If the java logger handler is changed like this, all the related logback classes are loaded earlier than expected and the transformation callback is not executed.


can't install when using poetry

poetry refuses to install this package due to conflicting python version requirements
> poetry add djangorestframework-simplejwt
Using version ^4.4.0 for djangorestframework-simplejwt

Updating dependencies
Resolving dependencies... (0.1s)

[SolverProblemError]
The current project's Python requirement (^3.7) is not compatible with some of the required packages Python requirement:
  - djangorestframework-simplejwt requires Python >=3.6,<3.9

Because djangorestframework-simplejwt (4.4.0) requires Python >=3.6,<3.9
 and no versions of djangorestframework-simplejwt match >4.4.0,<5.0.0, djangorestframework-simplejwt is forbidden.

So, since poetry defines my project's python version as ^3.7 (i.e. all python patch and minor versions above 3.7, so 3.7.1, 3.8, 3.9, 3.10, etc), I guess it refuses to install djangorestframework-simplejwt since it only claims to support python >=3.6,<3.9- presumably poetry wants to ensure that if at some point I'd try to upgrade to python 3.10 then the project's dependencies would not be broken.
Looking at how other python project's (namely this package's declared requirements) handle this I see:

django requires python >= 3.6
django-rest-framework requires python >= 3.5
pyjwt seems a bit more involved, but in the end it boils down to python >= 3 except some unsupported versions

So I guess the usual solution to handling the python version requirement is to set only a lower bound for the version.
With this in mind, I'd like to ask for a change in setup.py in order to specify python_requires='>=3.6'. This shall make it easy to install this project when using poetry.

I've the same issue. How do I solve it?

You can restrict you python version, like ~3.8 (which allows 3.8.2, but not 3.9.0) instead of ^3.8 (which allows 3.9, 3.10, etc.)


Announcement Extension is not working ? I tried to install it in modern site and nothing showed up?

Category

[x] Question
[ ] Bug
[ ] Enhancement

Expected or Desired Behavior
If you are reporting a bug, please describe the expected behavior. If you are suggesting an enhancement please describe thoroughly the enhancement, how it can be achieved, and expected benefit.
Observed Behavior
If you are reporting a bug, please describe the behavior you expected to occur when performing the action. If you are making a suggestion, you can delete this section.
Steps to Reproduce
If you are reporting a bug please describe the steps to reproduce the bug in sufficient detail to allow testing. Only way to fix things properly, is to have sufficient details to reproduce it. If you are making a suggestion, you can delete this section.
Submission Guidelines
Delete this section after reading

All suggestions or bugs are welcome, please let us know what's on your mind.
If you are reporting an issue around any of the samples, please ensure that you have clear reference on the sample and possibly code file, which should be fixed.
If you have technical questions about the framework, we’ll be monitoring #spfx, #spfx-webparts, #spfx-extensions and #spfx-tooling on (SharePoint StackExchange)[http://sharepoint.stackexchange.com/]. You can also alternatively submit your question to (SharePoint Developer group)[https://network.office.com/t5/SharePoint-Developer/bd-p/SharePointDev] at Office Network.
Remember to include sufficient details and context.
If you have multiple suggestions or bugs please submit them in separate bugs so we can track resolution.

Thanks for your contribution! Sharing is caring.

Hey @sakhradam  did you ever resolve this issue?
I'll go ahead and close this as there has been no activity for a while, but feel free to open re-open the issue if you need more help.


Make test utils an sub module and fix following disabled tests

Description:

Make testUtils (http-native/src/main/java/org/ballerinalang/net/testutils) a sub module

Fix following test using interop util classes
testCallerRequestHeaderPreservation2 - http-ballerina/src/http-tests/tests/integration-tests/service/http/31-cache-validation-request-header-preservation-test.bal (Use URL client)
module-ballerina-http/http-ballerina/src/http_tests/tests/unit-tests/services/configuration/compression/compression-annotation-test.bal
module-ballerina-http/http-ballerina/src/http-tests/tests/integration-tests/service/sample/04_compression_annotation_test.bal

Currently the http-ballerina-tests module needs native dependency as test utils are located inside native. So once test utils are moved, replace http-ballerina-tests module's native dependency with test-util dependency

No longer valid


Impact page preview image update to show which funds you are supporting

Right now the generated tweet shows what the funding we generate can fund through GiveDirectly. Let's update this to show which of the causes you are supporting: i.e. extreme poverty, open source developers, climate change all 33%.

Hi can i be assigned this issue ?

Sure - just did.

Hi , getting a no Provider configured for chain error . Anywhere (TG / Discord) where we can communicate to properly discuss any challenges i face while setting up

Hi , getting a no Provider configured for chain error . Anywhere (TG / Discord) where we can communicate to properly discuss any challenges i face while setting up

See suggestion here https://github.com/Glo-Foundation/glo-wallet/issues/434#issuecomment-2396257797

Oh hm the impact image that should be updated isn't shown anymore @gabrieltemtsen
it is supposed to be

@gglucass this makes my work easier , I would get on this asap

Oh hm the impact image that should be updated isn't shown anymore @gabrieltemtsen
it is supposed to be

What do you suggest ? on this behalf @gglucass

Oh hm the impact image that should be updated isn't shown anymore @gabrieltemtsen
it is supposed to be

What do you suggest ? on this behalf @gglucass

@Otaiki1 is working on it afaik - let's let him finalize? Not sure what the changes are in your screenshot tbh but that's not what I was looking for

@Otaiki1 any updates? Otherwise there are other interested contributors.

Oh hm the impact image that should be updated isn't shown anymore @gabrieltemtsen
it is supposed to be

What do you suggest ? on this behalf @gglucass

@Otaiki1 is working on it afaik - let's let him finalize? Not sure what the changes are in your screenshot tbh but that's not what I was looking for

If I could know how to reproduce the impact page or rather understand the req better, I could chip in a PR if the assignee doesn;t give a feedback otherwise I let @Otaiki1 complete it

@gabrieltemtsen the requirement is to change the opgenraph image associated with the /impact pages. For an example first load https://app.glodollar.org/impact/glodollar.eth and then copy paste it into https://www.opengraph.xyz/
https://www.opengraph.xyz/url/https%3A%2F%2Fapp.glodollar.org%2Fimpact%2Fglodollar.eth
the bottom bit of the card that says "enough to fund" should show the wallet's chosen causes instead

@gglucass  After looking at the issue, I may need these variables to fix the issue
KV_URL=
KV_REST_API_URL=
KV_REST_API_TOKEN=
KV_REST_API_READ_ONLY_TOKEN=
VERCEL_OG_URL=

@gglucass After looking at the issue, I may need these variables to fix the issue KV_URL= KV_REST_API_URL= KV_REST_API_TOKEN= KV_REST_API_READ_ONLY_TOKEN= VERCEL_OG_URL=

Hi @gabrieltemtsen - unfortunately I cannot provide these for you for security reasons. To get these for yourself, you can sign up with https://vercel.com/ and set up a key value store.

On it now!
More complicated than I expected!

Hi @gglucass to get the wallets chosen causes would be tricky because the current api implementation to get a wallet's chosen cause has to be an authenticated route


Cartridge

This add the mode in the header of the application. Using the application store mode.

call Focus.application.changeRoute('yousuperRouteName') or Focus.application.changeMode('consult', 'edit') in order to change the header attributes: data-mode or data-route


Reselect signal after tree update

Fixes https://github.com/godotengine/godot/issues/63710#issuecomment-1399544863

Thanks!


Get the video count of a playlist

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[x] Question
[ ] Other

Description of your issue, suggested solution and other information
I just want to know the number of videos in a playlist.
For Ex. when I start downloading a playlist, it shows
Course Name: Collected 61 video ids
I only want this line for my playlist. How can I do that?

--dump-single-json and count the number of entries.

How to count the number of entries?

There is two possible solutions for future reference

add --get-title or --get-id and pipe that into wc -l
youtube-dl --dump-single-json WHATEVER | python -c 'import sys,json; print len(json.load(sys.stdin)["entries"])'


Force zone if currentRoutingBucket ~= 0 (fix "instance" voice) [TESTED]

As in my test if we don't force to updateZone one can each the other but the other can't (Sorry for my english).
Forcing updateZone while currentRoutingBucket of the player is different than 0 fix this issue and you'll be able to use routingBuckets and instance the voice to that bucket. (Tested with 5 digits numbers).
Test without external voice server (+200 players): OK (choppy voices when +250 players).
Test using another FXServer as external voice server (+200 players): OK.
Test using murmur as external voice server (+200 players): FAIL.
If you need more info, or have any suggestions, just tell me.
Edit:
Server build: 4194 (External server is same build but in Linux).
Tested with production and canary users (Mixed xd)

By forcing it, you will affect voice for players having a high ping with you're server. You can still force the update zone, so this merge won't break anything if they wanna merge it. (Tested on 250 players for me)

I don't believe this is a proper fix for this issue, though I'm going to leave this issue open in case others want a temporary fix.

Hi 👋 ,
I've done some tests after trying to fix some issues with pma-voice related to very high player counts, and i think this is the issue i've encountered, this is caused because the channel is above 1024 so that's not anymore a permanent channel, there is two ways in that case.
My idea is : NetworkSetVoiceChannel will set you to a channel but if he does not exists it will not wait until it's created to continue, so if you add a voice target channel to a channel still not existing it will simply not work, if you have time you should report it to @citizenfx for a proper fix.
Fix 1 :
Add the voice target(s) channel(s) after calling Wait(500), idk if this will work for channels around you who don't have any players in it (meaning they are not even created as this could cause issue if a player join one of them after the script called updateZone)
MumbleClearVoiceTargetChannels(voiceTarget)

NetworkSetVoiceChannel(newGrid)

if newGrid > 1024 then
	Wait(500)
end

MumbleAddVoiceTargetChannel(voiceTarget, newGrid)

Fix 2 :
Create more permanent channel(s), this is very meh as some servers probably does not use just 1 or 2 extra routing buckets and creating too many permanent channels will cause issue.

This sounds like an issue that was already reported before, but probably got lost (see: https://forum.cfx.re/t/race-condition-in-networksetvoicechannel/2096443 )
Fix 2 isn't really possible, after 1k channels the server starts taking a very long time to make more channels, likely related to a channel mutex getting too big.

Should be fixed with that push, I'll make a release when I have the time to test to make sure it works.

Thank you for the push, will do some test tommorow to see also.


Sell crypto

What version are you using?

What did you do?

What did you expect to see?
What did you see instead?
For some reason the sell button doesn't work me. I press it and it doesn't create a order. Im trying to sell RMT but it wont let me

What is the name of the app you are using?

Im using the stellar x website

Unfortunately, this isn't the place for StellarX issues. I've let the StellarX team know that their error message needs to be changed. Please submit this issue here instead: https://stellarxsupport.zendesk.com/hc/en-us/requests/new


Move Login Dropdown into a component

Move the login dropdown details into a component and render the component in the navbar instead of hard coding the dropdown in the NavBar.

Still open because it needs to be merged into master


Emails kinda stink

[ ] Linkify everything that can be linked
[ ] Standardize the presentation of information and other information.
[ ] Use some basic templating and CSS to make the emails match the app design
[ ] Markdownify reservations notes
[ ] Reservation confirmation emails don't send

I believe that reservation notes are now rendered using markdown in the notes e-mail (see 4bef51a2b3971475d8cc8ba38dce9f6e55d662fd) for admins, but not anywhere else. We are also using links in a lot of places, but it can't hurt to review all of our e-mail views and verify that we're not missing any.
I don't know that styling is necessary; we can bump that to another issue, but I'd merge that with 'standardize presentation of information'. Finally, which confirmation e-mails aren't sending?

All of these have been taken care of in various issues with the exception of overall styling, which I'm really hesitant to do given how complex it is (I've looked into it for other projects). Based on that I'm closing this issue.


[CI] Automate version bump

Support automate version bump in https://github.com/opensearch-project/opensearch-build

As the next release manager, @sejli, can you add the manual steps you take in a comment here and then after the 2.7 release, we can make a better attempt at automation?


Add impl Color for Box<Color>

Easier short-term solution than creating a big enum.

Still not great. Maybe impl <'a> Color for &Color?...

Yeah, I think that'd be better.


Getting already subscribed message

Hi, thanks for your plugin, how can I get a already subscribed message?

You are welcome @emmetje!
Do you mean the message when you try to subscribe with the same email a few times?

Yes that's exactly what I mean.

@emmetje just add onOk & onFail handlers:
$('#subscribe-form').MailChimpForm({
  url: '<some/mailchimp/url>',
  fields: '0:SOMEFIELD',
  submitSelector: '#submit-form',
  onOk: (message) => {
    console.log(message);
  },
  onFail: (message) => {
    console.log(message);
  }
});

and try to reproduce your use case.
Please let me know if will face some difficulties.

Thx, added a div with id subscribe-result to the from and this code:
onFail: (message) => {
console.log(message);
$('#subscribe-result').html('' + message + '')
}
Works fine, thanks again!

@emmetje I'm glad it helped you!
P.S. Starring the repo will be appreciated 😉


I had expected env to support env variable substituion

https://github.com/numtide/devshell/blob/8850638db70cda50210a5152b1d555b9a520c8e0/default.nix#L66
I might miss a deeper reason, but as a naive user, I would have expected:
{

  env = {
    KUBECONFIG="$HOME/.kube/config";
  };

}

to work. Instead it evaluates to $HOME/.kube/config literally. Although the workaround is straight forward, I want to ensure with this issue it is intentional.
work around
{
  bash = {
    extra = ''
    export KUBECONFIG="$HOME/.kube/config"
    '';
  };
}

It's a good point, and also something that I have been wondering about. For now I opted for a safe default but my opinion can be changed.
Here are a few considerations that I have:
a. Key ordering is not guaranteed with TOML. This can be an issue if a user sets FOO="something" and BAR="$FOO" in the same file.
b. Escaping is not super clear to me. What happens if the user wants to have "$FOO" as the value? It seems like they might need a double backtick?
c. If we add support for other shells in the future, what does it mean for the value semantic? Will bash always be involved to setup the environment?

this is now supported:
[[env]]
name = "KUBECONFIG"
eval = "$HOME/.kubeconfig"