std::signbit cannot be found when building on Apple M1

When building (make debug) on Apple M1, I run into these errors:
/Library/Developer/CommandLineTools/usr/bin/make cmake BUILD_DIR=debug BUILD_TYPE=Debug
mkdir -p _build/debug && \
	cmake -B \
		"_build/debug" \
		-DTREAT_WARNINGS_AS_ERRORS=1 -DENABLE_ALL_WARNINGS=1 -DVELOX_BUILD_MINIMAL="OFF" -DVELOX_BUILD_TESTING="ON" -DCMAKE_BUILD_TYPE=Debug \
		-GNinja -DMAX_LINK_JOBS= -DMAX_HIGH_MEM_JOBS= \
		-DCMAKE_CXX_COMPILER_LAUNCHER=ccache \
		 \

-- Build type: Debug
-- Disabling Codegen
-- Found Bison keg installed by Homebrew at /opt/homebrew/opt/bison
-- Found Flex keg installed by Homebrew at /opt/homebrew/opt/flex
-- xsimd v9.0.1
-- Configuring done
-- Generating done
-- Build files have been written to: /Users/jimmylu/src/velox/_build/debug
/Library/Developer/CommandLineTools/usr/bin/make build BUILD_DIR=debug -j 10
cmake --build _build/debug -j 10
[1/909] Building CXX object third_party/googletest/googlemock/CMakeFiles/gmock_main.dir/src/gmock_main.cc.o
[2/909] Building CXX object third_party/googletest/googlemock/CMakeFiles/gmock.dir/src/gmock-all.cc.o
[3/909] Building CXX object third_party/googletest/googletest/CMakeFiles/gtest.dir/src/gtest-all.cc.o
[4/909] Linking CXX static library lib/libgtestd.a
[5/909] Linking CXX static library lib/libgmockd.a
[6/909] Linking CXX static library lib/libgmock_maind.a
[7/909] Linking CXX static library lib/libgtest_maind.a
[8/909] Building CXX object velox/common/base/CMakeFiles/velox_common_base.dir/BitUtil.cpp.o
FAILED: velox/common/base/CMakeFiles/velox_common_base.dir/BitUtil.cpp.o
ccache /Library/Developer/CommandLineTools/usr/bin/c++ -DBOOST_ATOMIC_DYN_LINK -DBOOST_ATOMIC_NO_LIB -DBOOST_CONTEXT_DYN_LINK -DBOOST_CONTEXT_NO_LIB -DBOOST_FILESYSTEM_DYN_LINK -DBOOST_FILESYSTEM_NO_LIB -DBOOST_PROGRAM_OPTIONS_DYN_LINK -DBOOST_PROGRAM_OPTIONS_NO_LIB -DBOOST_REGEX_DYN_LINK -DBOOST_REGEX_NO_LIB -DBOOST_SYSTEM_DYN_LINK -DBOOST_SYSTEM_NO_LIB -DBOOST_THREAD_DYN_LINK -DBOOST_THREAD_NO_LIB -DFMT_LOCALE -DGFLAGS_IS_A_DLL=0 -I/opt/homebrew/opt/icu4c/include -I/Users/jimmylu/src/velox/. -I/Users/jimmylu/src/velox/velox/external/xxhash -I/Users/jimmylu/src/velox/third_party/xsimd/include -isystem /opt/homebrew/include -isystem /Users/jimmylu/src/velox/velox -isystem /Users/jimmylu/src/velox/velox/external -isystem /Users/jimmylu/src/velox/velox/external/duckdb -isystem /Users/jimmylu/src/velox/velox/external/duckdb/tpch/dbgen/include -isystem /usr/local/include -isystem /Library/Developer/CommandLineTools/SDKs/MacOSX12.3.sdk/usr/include -isystem<EMAIL_ADDRESS>-mcpu=apple-m1+crc -std=c++17 -fvisibility=hidden -fvisibility=hidden -fvisibility-inlines-hidden -D USE_VELOX_COMMON_BASE -D HAS_UNCAUGHT_EXCEPTIONS -Werror -Wall -Wextra -Wno-unused        -Wno-unused-parameter        -Wno-sign-compare        -Wno-ignored-qualifiers        -Wno-range-loop-analysis -Wno-mismatched-tags -g -arch arm64 -isysroot /Library/Developer/CommandLineTools/SDKs/MacOSX13.0.sdk -mmacosx-version-min=12.6 -fPIC -std=gnu++17 -MD -MT velox/common/base/CMakeFiles/velox_common_base.dir/BitUtil.cpp.o -MF velox/common/base/CMakeFiles/velox_common_base.dir/BitUtil.cpp.o.d -o velox/common/base/CMakeFiles/velox_common_base.dir/BitUtil.cpp.o -c /Users/jimmylu/src/velox/velox/common/base/BitUtil.cpp
In file included from /Users/jimmylu/src/velox/velox/common/base/BitUtil.cpp:18:
In file included from /Users/jimmylu/src/velox/./velox/common/base/Exceptions.h:22:
/usr/local/include/fmt/format.h:1772:12: error: expected unqualified-id
  if (std::signbit(value)) {  // value < 0 is false for NaN so use signbit.
           ^
/Library/Developer/CommandLineTools/SDKs/MacOSX12.3.sdk/usr/include/math.h:170:5: note: expanded from macro 'signbit'
    ( sizeof(x) == sizeof(float)  ? __inline_signbitf((float)(x))        \
    ^

% clang --version
Apple clang version 14.0.0 (clang-14<IP_ADDRESS>)
Target: arm64-apple-darwin21.6.0
Thread model: posix
InstalledDir: /Library/Developer/CommandLineTools/usr/bin

Looks like a configuration issue with MacOSX SDK, is there any expert on this can provide some clue how we can solve this?

CC: @kgpai

CC: @raulcd @assignUser

hi @Yuhta , thanks for the report. I don't have an M1, I will try to get one to test (or spin and AWS instance) in the meantime, do you know if this only happens for DEBUG or this is happening on make release too?

This seems to be an issue when xcode is not installed, and a macro from math.h clashes with std::signbit I found several threads talking about this and installing xcode seems to solve it?

This lays out a solution: https://github.com/nlohmann/json/issues/2087#issuecomment-624331050
https://root-forum.cern.ch/t/compiling-6-12-4-error/27423/12
(it was also a bug in ancient gcc)

@assignUser Thanks for the help, I do have -isysroot when the compiler is invoked:
ccache /Library/Developer/CommandLineTools/usr/bin/c++ -DBOOST_ATOMIC_DYN_LINK -DBOOST_ATOMIC_NO_LIB -DBOOST_CONTEXT_DYN_LINK -DBOOST_CONTEXT_NO_LIB -DBOOST_FILESYSTEM_DYN_LINK -DBOOST_FILESYSTEM_NO_LIB -DBOOST_PROGRAM_OPTIONS_DYN_LINK -DBOOST_PROGRAM_OPTIONS_NO_LIB -DBOOST_REGEX_DYN_LINK -DBOOST_REGEX_NO_LIB -DBOOST_SYSTEM_DYN_LINK -DBOOST_SYSTEM_NO_LIB -DBOOST_THREAD_DYN_LINK -DBOOST_THREAD_NO_LIB -DFMT_LOCALE -DGFLAGS_IS_A_DLL=0 -I/opt/homebrew/opt/icu4c/include -I/Users/jimmylu/src/velox/. -I/Users/jimmylu/src/velox/velox/external/xxhash -I/Users/jimmylu/src/velox/third_party/xsimd/include -isystem /opt/homebrew/include -isystem /Users/jimmylu/src/velox/velox -isystem /Users/jimmylu/src/velox/velox/external -isystem /Users/jimmylu/src/velox/velox/external/duckdb -isystem /Users/jimmylu/src/velox/velox/external/duckdb/tpch/dbgen/include -isystem /usr/local/include -isystem /Library/Developer/CommandLineTools/SDKs/MacOSX12.3.sdk/usr/include -isystem<EMAIL_ADDRESS>-mcpu=apple-m1+crc -std=c++17 -fvisibility=hidden -fvisibility=hidden -fvisibility-inlines-hidden -D USE_VELOX_COMMON_BASE -D HAS_UNCAUGHT_EXCEPTIONS -Werror -Wall -Wextra -Wno-unused        -Wno-unused-parameter        -Wno-sign-compare        -Wno-ignored-qualifiers        -Wno-range-loop-analysis -Wno-mismatched-tags -g -arch arm64 -isysroot /Library/Developer/CommandLineTools/SDKs/MacOSX13.0.sdk -mmacosx-version-min=12.6 -fPIC -std=gnu++17 -MD -MT velox/common/base/CMakeFiles/velox_common_base.dir/BitUtil.cpp.o -MF velox/common/base/CMakeFiles/velox_common_base.dir/BitUtil.cpp.o.d -o velox/common/base/CMakeFiles/velox_common_base.dir/BitUtil.cpp.o -c /Users/jimmylu/src/velox/velox/common/base/BitUtil.cpp

@raulcd It's also happening in release build

any solution to this?

CC: @kgpai @raulcd

@CodingCat  @Yuhta This is not exactly an issue related to velox but rather with your build environment, which makes it hard to reproduce (especially as I don't have a mac :D ).
This SO post has several suggested fixes from a lot of users covering a lot of different configurations.
The main ones are setting -isysroot /Library/Developer/CommandLineTools/SDKs/MacOSX13.0.sdk/usr/include (note the usr/include is missing in your invocation :warning: ) removing and reinstalling xcode, removing the sdk folder...

you are right, I deleted all SDK and developer tools locally, it get passed now

you are right, I deleted all SDK and developer tools locally, and reinstalled....it get passed now

Hi @CodingCat , how do you resinstalled? After I executed ```
sudo rm -rf /Library/Developer/CommandLineTools
sudo xcode-select --install

The main ones are setting -isysroot /Library/Developer/CommandLineTools/SDKs/MacOSX13.0.sdk/usr/include (note the usr/include is missing in your invocation ⚠ ) removing and reinstalling xcode, removing the sdk folder...

I encountered such issue before. It's because the CLT's SDK version (e.g. 12.3) is incompatible with XCode's SDK (e.g. 13.0). My workaround is to force Clang to use XCode's SDK instead of Command Line Tool (CLT)'s SDK.
Here are the detail steps:

Create a soft link to cheat clang to use XCode's SDK instead of CLT's sdk. Then CMake will always load the correct SDK version from XCode's SDK directory.

cd /Library/Developer/CommandLineTools/SDKs/
# Let's say the CLT directory path to load is MacOSX12.3.sdk/, we backup the origin CLT's SDK directory
sudo mv MacOSX12.3.sdk MacOSX12.3.sdk.bak
sudo ln -s $(xcrun --show-sdk-path) MacOSX12.3.sdk

@darrenfu thanks for your reply, you are right. Below is my detail steps:

remove old Command Line Tools by sudo rm -rf /Library/Developer/CommandLineTools
download Installation package from https://download.developer.apple.com/Developer_Tools/Command_Line_Tools_for_Xcode_13.4/Command_Line_Tools_for_Xcode_13.4.dmg and install it
After install Command_Line_Tools_for_Xcode_13.4.dmg, the content in path /Library/Developer/CommandLineTools/SDKs/ is below:

xuedongluan@XUEDONGLUAN-MB1 velox % ls -rlt /Library/Developer/CommandLineTools/SDKs/
total 0
drwxr-xr-x  7 root  wheel  224 Mar 31  2022 MacOSX12.3.sdk
drwxr-xr-x  7 root  wheel  224 Mar 31  2022 MacOSX11.3.sdk
lrwxr-xr-x  1 root  wheel   14 Jan 12 11:54 MacOSX12.sdk -> MacOSX12.3.sdk
lrwxr-xr-x  1 root  wheel   14 Jan 12 11:54 MacOSX11.sdk -> MacOSX11.3.sdk
lrwxr-xr-x  1 root  wheel   14 Jan 12 11:55 MacOSX.sdk -> MacOSX12.3.sdk

Then the above errors gone when building, but another errors happened as below:
FAILED: velox/dwio/common/tests/CMakeFiles/velox_dwio_common_bitpack_decoder_benchmark.dir/BitPackDecoderBenchmark.cpp.o
ccache /Library/Developer/CommandLineTools/usr/bin/c++ -DBOOST_ATOMIC_DYN_LINK -DBOOST_ATOMIC_NO_LIB -DBOOST_CONTEXT_DYN_LINK -DBOOST_CONTEXT_NO_LIB -DBOOST_FILESYSTEM_DYN_LINK -DBOOST_FILESYSTEM_NO_LIB -DBOOST_PROGRAM_OPTIONS_DYN_LINK -DBOOST_PROGRAM_OPTIONS_NO_LIB -DBOOST_REGEX_DYN_LINK -DBOOST_REGEX_NO_LIB -DBOOST_SYSTEM_DYN_LINK -DBOOST_SYSTEM_NO_LIB -DBOOST_THREAD_DYN_LINK -DBOOST_THREAD_NO_LIB -DFMT_LOCALE -DGFLAGS_IS_A_DLL=0 -DVELOX_ENABLE_PARQUET -DVELOX_ENABLE_S3 -I/Users/xuedongluan/github/leoluan2009/velox/. -I/Users/xuedongluan/github/leoluan2009/velox/velox/external/xxhash -I/Users/xuedongluan/github/leoluan2009/velox/third_party/xsimd/include -isystem /opt/homebrew/include -isystem /Users/xuedongluan/github/leoluan2009/velox/velox -isystem /Users/xuedongluan/github/leoluan2009/velox/velox/external -isystem /Users/xuedongluan/github/leoluan2009/velox/velox/external/duckdb -isystem /Users/xuedongluan/github/leoluan2009/velox/velox/external/duckdb/tpch/dbgen/include -isystem /usr/local/include -isystem<EMAIL_ADDRESS>-isystem /Users/xuedongluan/github/leoluan2009/velox/third_party/googletest/googletest/include -isystem /Users/xuedongluan/github/leoluan2009/velox/third_party/googletest/googletest -isystem /Users/xuedongluan/github/leoluan2009/velox/_build/debug/third_party/arrow_ep/install/include -mcpu=apple-m1+crc -std=c++17 -fvisibility=hidden -D USE_VELOX_COMMON_BASE -D HAS_UNCAUGHT_EXCEPTIONS -Werror -Wall -Wextra -Wno-unused        -Wno-unused-parameter        -Wno-sign-compare        -Wno-ignored-qualifiers        -Wno-range-loop-analysis          -Wno-mismatched-tags          -Wno-nullability-completeness -g -arch arm64 -isysroot /Library/Developer/CommandLineTools/SDKs/MacOSX12.3.sdk -fPIE -std=gnu++17 -MD -MT velox/dwio/common/tests/CMakeFiles/velox_dwio_common_bitpack_decoder_benchmark.dir/BitPackDecoderBenchmark.cpp.o -MF velox/dwio/common/tests/CMakeFiles/velox_dwio_common_bitpack_decoder_benchmark.dir/BitPackDecoderBenchmark.cpp.o.d -o velox/dwio/common/tests/CMakeFiles/velox_dwio_common_bitpack_decoder_benchmark.dir/BitPackDecoderBenchmark.cpp.o -c /Users/xuedongluan/github/leoluan2009/velox/velox/dwio/common/tests/BitPackDecoderBenchmark.cpp
In file included from /Users/xuedongluan/github/leoluan2009/velox/velox/dwio/common/tests/BitPackDecoderBenchmark.cpp:20:
In file included from /Users/xuedongluan/github/leoluan2009/velox/./velox/dwio/common/tests/Lemire/bmipacking32.h:33:
In file included from /Library/Developer/CommandLineTools/usr/lib/clang/13.1.6/include/immintrin.h:13:
In file included from /Library/Developer/CommandLineTools/usr/lib/clang/13.1.6/include/x86gprintrin.h:15:
/Library/Developer/CommandLineTools/usr/lib/clang/13.1.6/include/hresetintrin.h:42:27: error: invalid input constraint 'a' in asm
  __asm__ ("hreset $0" :: "a"(__eax));
                          ^
In file included from /Users/xuedongluan/github/leoluan2009/velox/velox/dwio/common/tests/BitPackDecoderBenchmark.cpp:20:
In file included from /Users/xuedongluan/github/leoluan2009/velox/./velox/dwio/common/tests/Lemire/bmipacking32.h:33:
In file included from /Library/Developer/CommandLineTools/usr/lib/clang/13.1.6/include/immintrin.h:17:
/Library/Developer/CommandLineTools/usr/lib/clang/13.1.6/include/mmintrin.h:33:5: error: use of undeclared identifier '__builtin_ia32_emms'; did you mean '__builtin_isless'?
    __builtin_ia32_emms();
    ^
/Library/Developer/CommandLineTools/SDKs/MacOSX12.3.sdk/usr/include/c++/v1/math.h:670:12: note: '__builtin_isless' declared here
    return isless(__lcpp_x, __lcpp_y);
           ^
/Library/Developer/CommandLineTools/SDKs/MacOSX12.3.sdk/usr/include/math.h:545:22: note: expanded from macro 'isless'
#define isless(x, y) __builtin_isless((x),(y))
                     ^
In file included from /Users/xuedongluan/github/leoluan2009/velox/velox/dwio/common/tests/BitPackDecoderBenchmark.cpp:20:
In file included from /Users/xuedongluan/github/leoluan2009/velox/./velox/dwio/common/tests/Lemire/bmipacking32.h:33:
In file included from /Library/Developer/CommandLineTools/usr/lib/clang/13.1.6/include/immintrin.h:17:
/Library/Developer/CommandLineTools/usr/lib/clang/13.1.6/include/mmintrin.h:33:25: error: too few arguments to function call, expected 2, have 0
    __builtin_ia32_emms();

@leoluan2009 To tell from the error message, an arm64 build shouldn't be including immintrin.h. Can you run clang --version to check your clang version?

@leoluan2009 To tell from the error message, an arm64 build shouldn't be including immintrin.h. Can you run clang --version to check your clang version?

Hi @darrenfu my clang version is:
clang --version
Apple clang version 13.1.6 (clang-13<IP_ADDRESS>.5)
Target: arm64-apple-darwin21.5.0
Thread model: posix
InstalledDir: /Library/Developer/CommandLineTools/usr/bin

@darrenfu it seems like immintrin.h is included unguarded here:
https://github.com/facebookincubator/velox/blob/main/velox/dwio/common/tests/Lemire/bmipacking32.h#L33
But that was last changed two month's ago and I doubt that this went undetected for so long so :shrug:

I met the same problem, did you solve it ? @leoluan2009


Copy dlls into fuzz directory

This should fix the recent AppVeyor failures.
[extended tests]

Checklist

[ ] documentation is added or updated
[ ] tests are added or updated

Pushed. Thanks!

Reopening for backport to 1.1.0.

Ping.  This one is important for successful fuzz test on Windows for 1.1.0


autohotkey1(-installer): Add version <IP_ADDRESS>

Related: https://github.com/ScoopInstaller/Extras/pull/7448
autohotkey.com says version 1.0 is deprecated. Therefore we are add v2 to the extras bucket, and move v1 here.

I don't think so. Versions 1.0.* are deprecated, not 1.1.*.
Besides, v2 is still in alpha (or beta), not stable yet.

OK. I see.


Simplify state management for shared infrastructure

Remove unnecessary management of tenant states and move logic to the
messagingtenant-controller.
Simplify the InfraState interface and rename it to InfraClient, and
reduce the scope of the state package to concern only synchronization
and querying of configuration for a particular infrastructure.
Move usage of infra client only to places where it is needed to apply
configuration to routers and brokers.

@enmasse-ci run tests profile=isolated-shared-infra

:heavy_check_mark: Test Summary :heavy_check_mark:
TEST_PROFILE: isolated-shared-infra
TEST_CASE:
TOTAL: 19
PASS: 19
FAIL: 0
SKIP: 0
BUILD_NUMBER: 472


OPS-4820 Ensure to use TF remote state

Ensure to use TF remote state
Adding Terraform remote state to allow locking in between multiple commits/PR's.
Todo

[ ] Test with multiple commits or PR's

For example https://github.com/Flaconi/ops-infra-core/blob/2dc986fdeaaff8d8201e6b280a5e629fb9c6ff7a/terraform-1.1/envs/github/flaconi/repositories/terraform/terraform.tfvars#L39-L43


Can't remove nodes form Node-RED that have a scope

Describe the bug
When removing packages from node red that have a scope, starting with @, it seems that thinger server is receiving the character as is, but when sending the request to the Node-RED plugin the server is encoding this character causing Node-RED to not be able to find the package and returning a 404.
To Reproduce
Steps to reproduce the behavior:

Go to Node-RED plugin
Click on Options->Manage Palette
Install any dependency with scope (Ex. @flowforge/node-red-function-gpt)
Try to remove/uninstall the dependency

Expected behavior
Dependency should be removed from the palette
Additional context
According to RFC 3986 section 2.2, the at sign (@) is a reserved character and its encoding makes the URI not equivalent, and I quote:

The purpose of reserved characters is to provide a set of delimiting
characters that are distinguishable from other data within a URI.
URIs that differ in the replacement of a reserved character with its
corresponding percent-encoded octet are not equivalent.  Percent-
encoding a reserved character, or decoding a percent-encoded octet
that corresponds to a reserved character, will change how the URI is
interpreted by most applications.  Thus, characters in the reserved
set are protected from normalization and are therefore safe to be
used by scheme-specific and producer-specific algorithms for
delimiting data subcomponents within a URI.

npm scopes

Fixed on the upcoming 5.4.0 version


Fix Fix potential data leak when a partial update load which has publish conflict fails

What problem does this PR solve?
when recycling tmp rowsets, if rowset's state is RowsetStatePB::BEGIN_PARTIAL_UPDATE, the num_segments field in RowsetMetaCloudPB may not reflect the actual segments num(This may happen if partial update load writes a new segment to an existing tmp rowset in publish phase due to conflict and fails before it updates segments num in RowsetMetaCloudPB in MS successfully). So we need to delete the rowsets by prefix rather than delete by path in this case.
Release note
None
Check List (For Author)

Test

[ ] Regression test
[x] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[x] No.
[ ] Yes.

Does this need documentation?

[x] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.
Please clearly describe your PR:

What problem was fixed (it's best to include specific error reporting information). How it was fixed.
Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
What features were added. Why was this function added?
Which code was refactored and why was this part of the code refactored?
Which functions were optimized and what is the difference before and after the optimization?

run buildall

run buildall

run buildall


Strelka2 doesn't work properly with Manta vcfs

I tried running Strelka2 1.0 with an output from the latest Manta module (2.2) and it failed when trying to compress the files that were already compressed during the Manta workflow. I don't know if the compression and indexing is a more recent addition to Manta but if so, the Strelka2 module needs to be updated to fix this. If someone can confirm I'm happy to make the patch. It's currently working with this modification:
rule _strelka_input_vcf:
    input:
        vcf = CFG["inputs"]["candidate_small_indels"],
        tbi = CFG["inputs"]["candidate_small_indels"] + ".tbi"
    output:
        vcf = CFG["dirs"]["inputs"] + "{seq_type}--{genome_build}/vcf/{tumour_id}--{normal_id}--{pair_status}.candidateSmallIndels.vcf.gz",
        tbi = CFG["dirs"]["inputs"] + "{seq_type}--{genome_build}/vcf/{tumour_id}--{normal_id}--{pair_status}.candidateSmallIndels.vcf.gz.tbi"
    run:
        op.relative_symlink(input.vcf, output.vcf)
        op.relative_symlink(input.tbi, output.tbi)

#    params:
#        vcf = CFG["inputs"].get("candidate_small_indels_vcf") or "",
#        tbi = CFG["inputs"].get("candidate_small_indels_tbi") or ""
#    conda:
#        CFG["conda_envs"]["tabix"]
#    shell:
#        op.as_one_line("""
#        bgzip -c {input.manta_vcf} > {output.vcf}
#            &&
#        tabix {output.vcf}
#        """)

@whelena you are probably the most familiar with this

I can take a look, I think the last time I tested was with manta 2.1.

I think I know what the issue is, are you using the candidateSmallIndels from 99-outputs or 02-manta?

02-manta
Let me get this straight... is the Manta pipeline decompressing the candidateSmallIndels only for Strelka to then re-compress it before it runs?

yes, I didn't make any changes to manta but now that you said that maybe it would be best that manta outputs all of its vcf files in its compressed form as well

I can see why users might want them in a decompressed state for other downstream applications. Let's just leave it as is for now.


allow retune events in batches, add many many diagnostics for retuning.

_sweep_pos_to_freq

Incorrect - it is used via a thread function.


Where are the labels for Semantic Scene Completion task?

Hello,
Thank you for creating a great dataset!
I would like to ask where to download the label for the Semantic Scene Completion task?
Thank you again,
Anh-Quan Cao

Hi @yiyiliao,
I am sorry for bothering you again. Is there any news on the Semantic Scene Completion labels?
Thank you,
Quan

Hi, sorry for the delay! I can send you the (uncleaned) script for generating the Semantic Scene Completion labels from the full sequence if it is urgent.

Hi @yiyiliao,
It would be great!
Could you send to my email<EMAIL_ADDRESS>Thank you a lot for your help
Anh-Quan Cao


Fix typo in background services config

Simple typo fix

@maximvelichko Can you have a look at this. Simple minor typo correction but it shows in the profiles in the MacOS Profiles section once loaded. Thank you.

Thank you @RedVortex !

@maximvelichko I thought the combined profiles auto-updated on commit but I was mistaken. Should I update the combined files as well to fix the typo there also ? Sorry about the troubles :-/

No worries, I build combined profiles with macos/mobileconfig/combined/build.sh script, and commit them manually.
Done.


Ignore trailing spaces on input fields

When loading a Safe and copy-pasting and address, sometimes a trailing space can be copied over. This results in an error and leaves the user confused as it is a valid Safe address. We should ignore trailing spaces for address input fields (and maybe also other input fields?).

I think also leading spaces should be ignored


A warning from torch about LSTM

UserWarning: ONNX export failed on LSTM because batch_first not supported
It happens when sequence labeling models are used.
I can't find any solution to this warning.

@yhcc

@yhcc
To reproduce this warning:

checkout branch add_field_support
run  fastNLP/test/model/seq_labeling.py


must_change ignores from:, to: under Ruby 3

At c870f2e
Ruby 3.1.2
Rails 7.0.4
Minitest 5.16.3
{from: ..., to: ...} becomes the message passed to assert_changes, leaving its from & to defaulted to UNTRACKED (Object.new)
For example, changing test/rails/active_support/test_expectations.rb to include
[...]
class TestActiveSupportExpectations < ActiveSupport::TestCase
  def test_must_change
    counter = 99 # CHANGED THIS TO FORCE FAILURE

    value { counter += 1 }.must_change "counter", from: 0, to: 1
[...]

does not fail
Digging into the backtrace, I find
[...]
  #3    ActiveSupport::Testing::Assertions#assert_changes(expression="counter", message={:from=>2, :to=>0}, from=#<Object:0x...>, to=#<Object:0x...>, block=#<Proc:0x... ...) at [...]gems/activesupport-7.0.4/lib/active_support/testing/assertions.rb:179
  #4    Minitest::Expectation#must_change(args=["counter", {:from=>0, :to=>1}]) at [...]gems/minitest-5.16.3/lib/minitest/spec.rb:26
[...]

~I believe it's due to minitest's Module::infect_an_assertion not knowing/accounting-for rails's ActiveSupport::Testing::Assertions.assert_changes default args~
def assert_changes(expression, message = nil, from: UNTRACKED, to: UNTRACKED, &block)

but wanted to start the discussion here in case others found it difficult to create a failing test with this (& possibly other "infected assertions"?)

Apologies if this is off-base and/or just not helpful, but as I bump around the code and am learning how this all works, it kind of reminds me of the args/kwargs cases/handling in Minitest::Mock::{expect,method_missing}
Because of infect_an_assertion's optional dont_flip parameter, maybe a new method that takes dont_flip: and another (optional)  named argument so the caller can pass some info/hints about meth's positional and/or named arguments (maybe just the # of positional ones?)?
e.g.,
infect_an_assertion2 :assert_changes, :must_change, dont_flip: :block, positional_args: 1
(Even if the direction has merit, can probably do better than ...assertion2 🤷)

In case it helps, I tried it with other version combinations
minitest-rails c870f2e, Rails 7.0.4

Ruby
Original test(counter = 0)
Modified test(counter = 99)

3.1.2
Passes with no warnings
Passes with no warnings

3.0.4
Passes with no warnings
Passes with no warnings

2.7.6
Passes with[...]/minitest-5.16.3/lib/minitest/spec.rb:26: warning: Using the last argument as keyword parameters is deprecated; maybe ** should be added to the call[...]/gems/activesupport-7.0.4/lib/active_support/testing/assertions.rb:175: warning: The called method 'assert_changes' is defined here
Fails withTestActiveSupportExpectations#test_must_change [[...]/minitest-rails/test/rails/active_support/test_expectations.rb:8]:Expected change from 0and gives the same warning

minitest-rails v6.1.0, Rails 6.1.7

Ruby
Original test(counter = 0)
Modified test(counter = 99)

3.1.2
Passes with no warnings
Passes with no warnings

3.0.4
Passes with no warnings
Passes with no warnings

2.7.6
Passes with[...]/minitest-5.16.3/lib/minitest/spec.rb:26: warning: Using the last argument as keyword parameters is deprecated; maybe ** should be added to the call[...]/gems/activesupport-7.0.4/lib/active_support/testing/assertions.rb:175: warning: The called method 'assert_changes' is defined here
Fails withTestActiveSupportExpectations#test_must_change [[...]/minitest-rails/test/rails/active_support/test_expectations.rb:8]:Expected change from 0and gives the same warning

So it looks like it's (mostly?) related to Ruby 3.x parser changes?

I'm relatively new to minitest{,-rails}, but am happy to keep learning and try helping.
That leads me to: What kind(s) of test/examples would you want to have caught this—especially since this uses a method in one gem (infect_an_assertion) to create a new method "based on" a method in another gem (assert_changes).
Said another way: which of those methods' 'implementation details' would you want to test?
I think that testing that must_change (et al?) actually raises Minitest::Assertion when appropriate—like this minitest test/example—would be the most comprehensive. That is, testing must_change's behavior, independent of how that is implemented/provided. I think this would also take

Either duplicating things like minitest's assert_triggered in this gem's tests
Or separating them from TestMinitestAssertions so they could be used here without all of that class's actual test cases.

I could also see an argument for mocking assert_changes to check that calling must_change passes expected arguments to it—in effect, testing infect_an_assertion and that mintiest-rails called it with the expected symbols, etc., but I'm not confident that would catch parser-change-caused-problems like this one.
Of course dev and/or CI changes to test against different Ruby versions is another story...
🤷♂

Hey Bob, thank you for reporting this issue. I haven't been ignoring it, but I was recently laid off and am kinda busy with dealing with that...
I did talk a bit with Ryan about this. I have successfully reproduced this, and am working on a fix. Will try to get something out soon.

Thanks for the reply (and gem!); sorry to read of your layoff
No demand/expectation of a fix or schedule, of course—it's FOSS, right?
It's been kinda fun to dig in & see/learn some new stuff. Happy to help as able, but probably nowhere the "speed" of someone already familiar with all this when you/they get to focus on it.

Fixed by minitest/minitest@87604fc
Released in minitest v5.17.0
Thanks!

ᕕ( ᐛ )ᕗ


加载异步组件报错。

Version
2.2.4
Reproduction link
https://codepen.io/
Steps to reproduce
无法提供重现环境，这个问题是随机出现的。一个月前的某个时刻开始出现。
我的系统整体有几百个组件和模块，异步组件大概有200个。使用webpack2   vue2.2.4   vue-router2.3.0   vuex2.2.1。vue-router使用hash模式。
What is expected?
异步组件加载成功
What is actually happening?
在一段时间内，某个异步组件会加载失败。
然后过几天，这个问题会随机转移到另外一个异步组件上，出错的异步组件不是特定的。
只要是从其他组件跳转到这个异步组件就会加载失败。但是加载失败后，F5刷新页面，异步组件又会加载成功。

测试过更新vue全家桶到最新和webpack3最新版，还是有这个问题。
报错信息：
[Vue warn]: Failed to resolve async component: function (){var n={};return t.commit("loading",n),e().then(function(e){return n.close(),Promise.resolve(e)}).catch(function(e){return n.close(),Promise.reject(e)})}
Reason: TypeError: Cannot read property 'call' of undefined

Even if it doesn' fail all the time, we need a proper boiled down reproduction (for example, you should be able to remove vuex and vue-router). It can be a github repository, but it's important you boil it down to the minimal. You may even find where the problem came this way 🙂
We need this to make sure you're reporting a bug and not just asking a question as we have the forum and the Discord server for that
I'm closing this for the moment but if you boil it down and still think it's a vue bug, come back and open a new issue

OK, thanks


Redesign pink banner

@richenda said here : "The giant pink banner with the blue heart and GDPR needs to be redesigned. I'll do it."
@Richenda: before you start this, check first with @Manoj-nathwani and @taymonbeal whether it makes sense for you to do that. They might have plans to design the "privacy experience" in a different way than it currently is

https://drive.google.com/open?id=1vcqIjf5Vqo29sEe4YdFQUh_qOlWafIBV

@sebgrebe --> https://drive.google.com/file/d/1vcqIjf5Vqo29sEe4YdFQUh_qOlWafIBV/view

Lol.

Thank you @Richenda. Implemented.

Yesss!!! Liberation from pink!


Extension which uses proposed API yet is not allowed to, can still contribute global commands

Have an extension which has a global command and uses proposed API and declares it in package.json using "enableProposedApi": true
Load that without allowing it to use proposed API

The global command still shows up in the command palette.
cc @aeschli

Sure - the implementation checks only against use of proposed api not against the use of any API or static contribution. E.g. you can declare the use of proposed API but not use any and that's OK. @joaomoreno I don't understand the issue here?


not building with cordova-ios 4.0.1

all detail right here
https://github.com/dpa99c/cordova-diagnostic-plugin/issues/27
this is why "cordova-plugin-googlemaps" the error occur is
developer should be follow this in<EMAIL_ADDRESS>http://cordova.apache.org/announcements/2015/12/08/cordova-ios-4.0.0.html
and when I use
cordova plugin rm cordova-plugin-googlemaps
it will appear this error
Error: Cannot read property 'buffer' of undefined

hi @Hirbod
why this become colse ?

Because it's a known issue and we are already working on it.

OK
thanks @Hirbod
sorry to bother you


An error occurred calling InitiateAuth operation: Unable to verify secret hash for client

Hi,
I'm trying to use the package using the example you provide... but it doen't work, what I'm doing wrong?
from warrant import Cognito

POOL_ID = 'eu-central-1_J1HZ86xLX'
APP_ID = '23o6vbtasvacvd7f567go7svfk'
USERNAME = 'user1'
PASSWORD = '@perw1fg'
ACCESS_KEY = 'AKIAI5WPBLQZAQFPLGGQ'
SECRET_KEY = '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'

u = Cognito(POOL_ID, APP_ID, username=USERNAME, user_pool_region='eu-central-1')#, access_key=ACCESS_KEY, secret_key=SECRET_KEY)
u.authenticate(password=PASSWORD)

The settings are fake ;-)
The error message is the following:
Traceback (most recent call last):
  File "./warrant-test.py", line 20, in <module>
    tokens = aws.authenticate_user()
  File "/usr/local/lib/python2.7/site-packages/warrant/aws_srp.py", line 187, in authenticate_user
    ClientId=self.client_id
  File "/Users/m2tz/Library/Python/2.7/lib/python/site-packages/botocore/client.py", line 253, in _api_call
    return self._make_api_call(operation_name, kwargs)
  File "/Users/m2tz/Library/Python/2.7/lib/python/site-packages/botocore/client.py", line 557, in _make_api_call
    raise error_class(parsed_response, operation_name)
botocore.errorfactory.NotAuthorizedException: An error occurred (NotAuthorizedException) when calling the InitiateAuth operation: Unable to verify secret hash for client 23o6vbtasvacvd7f567go7svfk

Maybe I missed something, but 3 months ago it was required that

When creating the App, the generate client secret box must be unchecked because the JavaScript SDK doesn't support apps that have a client secret.

p.s. aws_srp.py module was ported from Amazon Cognito Identity SDK for JavaScript, so this requirement (client secret box must be unchecked) should apply to warrant too

@armicron : is it fixed ? can you please tell me more on how to fix it ? thank you.

@appsassociates-dev "When creating the App, the generate client secret box must be unchecked"

Are there any plans to support AWS Client apps with generated secrets?

@jonasao I think that should be a separate issue.

@bjinwright Added as issue #44

@armicron Can we close this issue.

@bjinwright yes, it's duplicate of 44.

Yes,

Client secret box must be unchecked when you create a user pool

Due to this problem, I've been studying a lot, but this must be reflected in the README page.

Due to this problem, I've been studying a lot, but this must be reflected in the README page.

Totally agreed. I had to drop print statements all around django-warrant as well as warrant to figure out what was going wrong with my implementation.

Try this answer which I posted in stackoverflow. It mention how to create SECRET_HASH in python.
link - https://stackoverflow.com/questions/37438879/unable-to-verify-secret-hash-for-client-in-amazon-cognito-userpools/56581239#56581239


update BioAssessments map functionality

Zach:

BA Map Function: I’m hoping you’ll be able to investigate how the map and table in the BA repository could be better linked or at least make the map function similar to the projects tab where folks can get access to the documents by clicking on their pin.

Consolidated unique sites (so from 11 to 6 in Pacific NW):

Added Project documents as bulleted list with document links:


Update the Godot Android Editor name from Godot Editor 4.x to Godot Editor 4

We don't really use "v4" anywhere to brand Godot. It's either "Godot Engine", "Godot Engine 4.0", "Godot Engine 4" or "Godot Engine 4.x" (the "4" one is also not used much, to avoid the similarity with Unreal Engine 4, but if we want to keep it generic while different from Godot 3 here, it could work).

We don't really use "v4" anywhere to brand Godot. It's either "Godot Engine", "Godot Engine 4.0", "Godot Engine 4" or "Godot Engine 4.x" (the "4" one is also not used much, to avoid the similarity with Unreal Engine 4, but if we want to keep it generic while different from Godot 3 here, it could work).

Updated to Godot Editor 4, and updated the play store label to match as well.

Thanks!


Support vega

I mean this package https://github.com/vega/ipyvega/ which is used by altair to render plot in offline mode.
The code below won't show any thing in vscode but works well in notbook
from vega import Vega

Vega({
  "$schema": "https://vega.github.io/schema/vega/v5.json",
  "width": 400,
  "height": 200,
  "padding": 5,

  "data": [
    {
      "name": "table",
      "values": [
        {"category": "A", "amount": 28},
        {"category": "B", "amount": 55},
        {"category": "C", "amount": 43},
        {"category": "D", "amount": 91},
        {"category": "E", "amount": 81},
        {"category": "F", "amount": 53},
        {"category": "G", "amount": 19},
        {"category": "H", "amount": 87}
      ]
    }
  ],

  "signals": [
    {
      "name": "tooltip",
      "value": {},
      "on": [
        {"events": "rect:mouseover", "update": "datum"},
        {"events": "rect:mouseout",  "update": "{}"}
      ]
    }
  ],

  "scales": [
    {
      "name": "xscale",
      "type": "band",
      "domain": {"data": "table", "field": "category"},
      "range": "width",
      "padding": 0.05,
      "round": True
    },
    {
      "name": "yscale",
      "domain": {"data": "table", "field": "amount"},
      "nice": True,
      "range": "height"
    }
  ],

  "axes": [
    { "orient": "bottom", "scale": "xscale" },
    { "orient": "left", "scale": "yscale" }
  ],

  "marks": [
    {
      "type": "rect",
      "from": {"data":"table"},
      "encode": {
        "enter": {
          "x": {"scale": "xscale", "field": "category"},
          "width": {"scale": "xscale", "band": 1},
          "y": {"scale": "yscale", "field": "amount"},
          "y2": {"scale": "yscale", "value": 0}
        },
        "update": {
          "fill": {"value": "steelblue"}
        },
        "hover": {
          "fill": {"value": "red"}
        }
      }
    },
    {
      "type": "text",
      "encode": {
        "enter": {
          "align": {"value": "center"},
          "baseline": {"value": "bottom"},
          "fill": {"value": "#333"}
        },
        "update": {
          "x": {"scale": "xscale", "signal": "tooltip.category", "band": 0.5},
          "y": {"scale": "yscale", "signal": "tooltip.amount", "offset": -2},
          "text": {"signal": "tooltip.amount"},
          "fillOpacity": [
            {"test": "datum === tooltip", "value": 0},
            {"value": 1}
          ]
        }
      }
    }
  ]
})

We need to register the main entry point of widgets with requirejs.

We could register the scripts in the renderer so that they load one notebook open.

Summary of the problems:

The widget scripts are registered only when we render ipywidgets (however vega is a regular js)
Hence we need to register the scripts for all outputs
The baseUrl is setup in the webview where ipywidgets are rendered, however regular js is rendered in a different webview. Either we have a custom renderer for regular js/html mimetypes, or we update core to render js in the same webview as other outputs.
The first two will probably be sufficient for fixing the rendering of the widget after execution, however if the user opens a regular notebook then things will not work as the renderer will load first and the extension host will still be busy fetching the scripts.

We need to either wait for the widget scripts to be loaded,
or pre-load the widget scripts in some way (e.g. via some renderer extension list plotly is included in the renderer extensions, again the path will have to be accessible via <require config baseUrl>/nbextensions/jupyter-vega, meaning the renderers will have to have a way to register some scripts before loading its own scripts.

Suggestion for updating the base url

We can add custom js code into the html/js outputs that update the baseUrl of requirejs in the webview
TODO:
Check if requirejs is available in built in renderers
This feels hackish,

hence check if its possible to have these js/html loaded in the same webview as widgets
should we create a custom renderer for js/html outputs (discuss with Matt)

Even if this is done, how will this work when re-loading the same notebook
Probably best discussed with Matt as this deals with webviews and loading of custom scripts, and the like.

There haven't been enough upvotes on this issue, hece we're closing this for now.
If this is requried, please feel free to re-open or create a new issue and upvote that


Problem linking glfw

hi i'm trying to create a .bat in vscode to build a c++ project that uses glfw. I'm using the example code in documentation to test the build with msvc, but te link.exe returns me LNK2019 (i constantly think that error is like the actual year can't be a coincidence). I'm using the .dll, .lib and .dll.lib who I downloaded froom the binary files in the glfw site. I try the x86 an 64, both with the same error.
I open a post in khronos forum of opengl too.
https://community.khronos.org/t/vscode-problem-lnk-msvc/104321

@Lkledu
While I'm sorry you're having trouble building something, this issues area is for reporting problems with the MSVC documentation, not for support. If the many, many ways to cause and fix a LNK2019 error described in the error article don't help, then I suggest you follow up in the other forums suited to questions like yours.


Support for websockets?

Is websocket supported at the moment?
If it is not, is it in the roadmap?
Does the structure of this module include a way to easily integrate websocket?

There's no reason you can't implement a data source that uses a web socket today under the hood. However there is one caveat: although the data source returns an observable which allows data to stream back to the client, currently only Observables which stream a single value are supported.  We plan to supports streaming multiple responses back to the Model from the server in an upcoming version.
One good reason to use a web socket today would be to push data or cache invalidations. You can do  this today by just listening to the web socket for notifications and then modify the Model cache  using set and invalidate.

To be clear, you can multiplex concurrent Data source operations over the web socket, but each Data source operation can only stream back a single response. Given this limitation, the  main benefit of websockets over HTTP/2 is the ability to push cache invalidations and data into the Model cache. Once again this is done manually, by simply invoking model.withoutDataSource().set(...) and model.invalidate().

I'm currently performing issue triage as we get ready to perform a proper release, and closing/tagging as I go.
Considering this answered and closing. @jhusain, please feel free to comment if anything has changed around this.


Wolverine codegen - variable name used twice in codegen

In my project I have two handlers (in two different namespaces) for the same 'CompanyCreated' event.
However, the code generated by Wolverine declares/uses the 'companyCreatedHandler' variable name twice - and the code therefore does not compile.
WolverineFx version 0.9.16.
First handler:
using Ksb.CourierMgmtSystem.Application.Common.Interfaces;

namespace Ksb.CourierMgmtSystem.Application.Features.Companies.Commands;

public class CompanyCreatedHandler
{
    public Task Handle(CompanyCreated notification, CancellationToken cancellationToken, IIdentityService identityService)
    {
        //do some work
        return Task.CompletedTask;
    }

}

Second handler:
using Ksb.CourierMgmtSystem.Application.Common.Interfaces;

namespace Ksb.CourierMgmtSystem.BlazorServer.Handlers;

public class CompanyCreatedHandler
{
    public Task Handle(CompanyCreated notification, IIdentityService identityService)
    {
        //do some work
        return Task.CompletedTask;
    }
}

Generated code - 'companyCreatedHandler' variable name used twice:
// <auto-generated/>
#pragma warning disable
using Lamar;

namespace Internal.Generated.WolverineHandlers
{
    // START: CompanyCreatedHandler560193426
    public class CompanyCreatedHandler560193426 : Wolverine.Runtime.Handlers.MessageHandler
    {
        private readonly Lamar.IContainer _rootContainer;

        public CompanyCreatedHandler560193426(Lamar.IContainer rootContainer)
        {
            _rootContainer = rootContainer;
        }

        public override async System.Threading.Tasks.Task HandleAsync(Wolverine.Runtime.MessageContext context, System.Threading.CancellationToken cancellation)
        {
            await using var nestedContainer = (Lamar.IContainer)_rootContainer.GetNestedContainer();
            var companyCreatedHandler = nestedContainer.GetInstance<Ksb.CourierMgmtSystem.Application.Features.Companies.Commands.CompanyCreatedHandler>();
            var companyCreatedHandler = nestedContainer.GetInstance<Ksb.CourierMgmtSystem.BlazorServer.Handlers.CompanyCreatedHandler>();

            /*
            */
            var identityService = nestedContainer.GetInstance<Ksb.CourierMgmtSystem.Application.Common.Interfaces.IIdentityService>();
            var companyCreated = (Ksb.CourierMgmtSystem.Application.Features.Companies.Commands.CompanyCreated)context.Envelope.Message;
            await companyCreatedHandler.Handle(companyCreated, identityService).ConfigureAwait(false);
            await companyCreatedHandler.Handle(companyCreated, cancellation, identityService).ConfigureAwait(false);
        }

    }

    // END: CompanyCreatedHandler560193426

}

Got it. Quickie workaround is to make the handler methods static.

Hi Jeremy
Confirmed, it works perfectly with static methods, thanks.

So Wolverine happily disambiguates the handlers names already if it's building everything up itself, but that wasn't covering the case of needing to use a nested container to build things. I'm working on that use case too. Somewhere in the dependency tree you've got a service registration by lambda builder that's either Scoped or Transient? That's why Wolverine is having to use the nested container


Use also as load balancer according to this tutorial - It worked but...

Hello, great work and thanks.
Is it possible to use the tutorial as seen in the following video for load balancing?
https://www.youtube.com/watch?v=QE26N9cHE2M
I tried setting the specific .conf file in /data/nginx/proxy_hosts to read/write/execute for everyone and then I "sudo nano" it adding what is mentioned in this video. It worked after restarting NginxProxyManager container but I get redirected to the first only selection.  A good thing to do is integrate this if you please...
Thanks a lot, again.

Hello hatzisn,
I have created an article about how to load balance your web servers (WordPress) using Nginx Proxy Manager and Cloudflare.
You may have a look at it.
However, it is just a temporary workaround. Hopefully, the upstream module will be included in the Nginx Proxy Manager UI in v3.
You have to modify your Nginx Proxy Manager built-in configuration files in order to make the upstream module function properly.
Moreover, you will need to create some custom configuration files.
Try my method on a flash-installed Linux VM/VPS, as it may mess up your current settings.
I am not familiar with proxy headers. Feel free to correct me if I forget to include some necessary proxy headers.

@hatzisn I did a loadbalancer setup that works with npm as the follows:
Requirements:

You have two web servers (For example #  server <IP_ADDRESS>:8081 + server <IP_ADDRESS>:8082 )

To create nginx server as LB
NPM setup and is working

Setup your nginx server (can simply run it as a docker container)
Example:
version: '3.8'
services:
  lb:
    image: nginx:latest
    container_name: web_nginx_bootstrap_loadbalancer-lab-lb
    restart: unless-stopped
    volumes:
    - /volume1/docker/containers/Bootstrap/loadbalancer-lab/lb/config/nginx.conf:/etc/nginx/nginx.conf:ro
    ports:
    - "1010:80"

Make sure to edit your nginx.conf file in the mounted file # nginx.conf
http {

    upstream all {
        server <IP_ADDRESS>:8081;
        server <IP_ADDRESS>:8082;
    }

    server {
         listen 80;
         location / {
              proxy_pass http://all/;
         }
    }

}

events { }

Finally, configure your NPM with a host to point to that LB docker container nginx with the port # 1010 for example.

Would love this to be a built in feature as i believe it's all just a wrapper over the nginx upstream config anyway. Would be super useful

Would really appreciate to see this feature inbuilt in Nginx Proxy Manager.
Load balancing will help achieve high availability  of services, in case of hardware failure/security compromised etc (if services are hosted on a single backend host instance.)
This feature will make it more attractive to enterprise users.

Agreed, having some simple load balancing option would be a good to have feature for sure

I agree: a Load Balance config option (easy like a Apple GUI)  would be great.
I would be wonderful if it allow not only localaddressess but world wide addresses.

+1


kubectl exec times out on mysql dump

Environmental Info:
K3s Version:

k3s version v1.18.6+k3s1 (6f56fa1d)
Node(s) CPU architecture, OS, and Version:

Linux oracle01 5.4.0-1021-oracle #21-Ubuntu SMP Fri Jul 10 06:33:24 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Cluster Configuration:

1 master 1 worker, only specified docker at install. It's on two Oracle Cloud instances but I've worked out the IPtable rules.
oracle01   Ready    master   2d19h   v1.18.6+k3s1
oracle02   Ready    <none>   2d16h   v1.18.6+k3s1

Describe the bug:

Trying to take a database backup by execing into a mariadb container times out with no explanation (rc=0 and no stderr)
justin@justinovh:/tmp$ time /usr/local/bin/kubectl exec -n mariadb mariadb-master-0 -- sh -c 'mysqldump --all-databases -uroot -ppassword'> /tmp/backup.sql

real	0m27.890s
user	0m0.977s
sys	0m1.181s
justin@justinovh:/tmp$

The file hovers around ~70MB when the actual full DB backup is 1.5 gigs. Something is silently timing out.
Manually execing into the pod and keeping a session alive avoids this problem. I've tried with the --tty and --stdin flags with no difference.
Steps To Reproduce:

Installed K3s (w/ docker as the engine):
Installed the latest mariadb bitnami helm chart with just a single master
Tried to dump data through kubectl exec back to the calling machine.

Expected behavior:

It returns 0 when I have the full dump or non-zero if something bad happened.
Actual behavior:

Times out and I have no idea if something went wrong beyond file size / time.
Additional context / logs:

There is no intrinsic timeout on the exec commands. Issues like this are usually related to a firewall or load-balancer between your kubectl client and the server closing the connection because it looks idle due to no data being passed for an extended period of time.

Thanks. I'm not sure what is causing it, but I'm working around it by using mysqldump installed on the machine I want to directly dump to, and going through the NodePort of the service. That seems to be working. Thanks for the help!


Fix MQTT broker host (#20)

Strange I had no issues. But if it fixes it, I'll merge it.


use gphoto for calibration

gphoto is a great CLI tool for controlling DSLRs. It currently has support for around 2700 models according to their website.
This could be used for the first calibration/initialisation instead of the webcam potentially getting better results because:

higher resolution
fixed on a trypod
clearer image by setting aperture, speed and ISO manually

I wated to implement this in the holidays but there was too much stuff regarding university.
I might have a look another time but thought I might share this idea
EDIT: sadly it seems like the part calculating the coordinates and exporting them to the coordinates.txt is not in this repo :(

Hi @Leptopoda - as you mentioned, the coordinate calculation code isn't in here. I think that's probably because it falls under the list of things too specific to Matt's setup to be easily shared!
That said, if you wanted to work on something like this, I think it'd make a nice addition to the "Further work" section of the README!
Going to close this one since it isn't an "issue" per say, but excited to see any progress! Totally understand if you don't find the time, of course.

Hi @Leptopoda - as you mentioned, the coordinate calculation code isn't in here. I think that's probably because it falls under the list of things too specific to Matt's setup to be easily shared!
That said, if you wanted to work on something like this, I think it'd make a nice addition to the "Further work" section of the README!
Going to close this one since it isn't an "issue" per say, but excited to see any progress! Totally understand if you don't find the time, of course.


false values for attributes are not written

If there a reason that attributes that have a value of false are not written to XML?

Probably because of an incorrect truthy/falsey if statement. Can you provide a quick code snippet? Should be able to get it fixed quick.


Add error/warning/info regions and disable phantoms by default.

Had a look into the regions and phantoms stuff a little bit with more attention to the regions right now.

I'd suggest to disable phantoms by default at the moment as they make files with many issues unreadable at the moment.

Errors/Warnings/Infos are added with different scopes so they can have different colors. Scopes are synced with diagnostics syntax. Decided to use sublimelinter scopes in diagnostics panel for now. But we may discuss which solution is better. I am not sure.

For details, please see commit messages.

I recommend you install mypy to check the typing manually, or the mypy plugin for the python language server: https://pypi.python.org/pypi/pyls-mypy/0.1.2
Also a request, could you keep your PRs focused to solving one problem at a time? This way we can quickly merge obvious/important stuff, and have more in-depth reviews on the more explorative work.

Will try it ;-)

Hmm, pyls-mypy didn't complain locally. And honestly - If I want to initiate an object with None, I do so. There is no need to create an empty list, which is not used but for a following empty check. Is there a way to tell mypy a list initiated with None is ok? I ask, because I never used typing notation right now.

Is there a way to tell mypy a list initiated with None is ok? I ask, because I never used typing notation right now.

I didn't use mypy before, but I guess it's similar to flow which is why you should be able to write the following:
regions = None  # type: Optional[List[sublime.Region]]
Maybe the type comment is not necessary if you initialize the variable above the if-block (and remove the else branch).

Maybe the type comment is not necessary if you initialize the variable above the if-block (and remove the else branch).

CI's mypy even complains about the line 1159 where the variable is already defined.
I definitely don't find the requirement to define types in comments after a line helpful. It already brings some noise to the interface part of functions but may be useful there even though I lived well without that stuff right now. I'll definitely won't start liking that - not mentioning the tons of json files mypy creates. 🤢

Rebased my work on the current master.

CI's mypy even complains about the line 1159 where the variable is already defined.

I just gave it a try and the following code seems to work:
    phantoms = None

    buffer_id = view.buffer_id()
    if show_diagnostics_phantoms and not view.is_dirty():
        phantoms = list(
            create_phantom(view, diagnostic) for diagnostic in diagnostics)

Mypy's problem with the current code is (as far as I can tell) that it infers the type of the phantoms variable by looking at its first (potential) assignment. This assignment is in the if-body and amounts to a list of phantoms. Afterwards, it checks that all references to the phantoms variable fulfill this type requirement, which fails when setting it to None. Initializing phantoms above the if-block with None works, since mypy can infer that it's optional.
Type checkers can be a bit tedious to deal with from time to time, but in the long run they can save a lot of time and improve code readability and reliability in my opinion. That's why I would avoid disabling the checks with # type: ignore.

Rebased to resove merge conflicts

Just had another idea how to avoid the None assignment, by following code structure when I saw this PR already merged 😄
def update_diagnostics_phantoms(view: sublime.View, diagnostics: 'List[Diagnostic]'):
    global phantom_sets_by_buffer

    buffer_id = view.buffer_id()
    if show_diagnostics_phantoms and not view.is_dirty():
        phantoms = list(
            create_phantom(view, diagnostic) for diagnostic in diagnostics)
        if phantoms:
            phantom_set = phantom_sets_by_buffer.get(buffer_id)
            if not phantom_set:
                phantom_set = sublime.PhantomSet(view, "lsp_diagnostics")
                phantom_sets_by_buffer[buffer_id] = phantom_set
            phantom_set.update(phantoms)
            return
    phantom_sets_by_buffer.pop(buffer_id, None)

def update_diagnostics_regions(view: sublime.View, diagnostics: 'List[Diagnostic]', severity: int):
    region_name = "lsp_" + format_severity(severity)
    if not show_diagnostics_phantoms or view.is_dirty():
        regions = list(create_region(view, diagnostic) for diagnostic in diagnostics
                       if diagnostic.severity == severity)
        if regions:
            scope_name = diagnostic_severity_scopes[severity]
            view.add_regions(region_name, regions, scope_name, "dot",
                             sublime.DRAW_SQUIGGLY_UNDERLINE | UNDERLINE_FLAGS)
            return
    view.erase_regions(region_name)


Bug 1825330: support creating v1beta CRDs to avoid data loss

Description of the change:
Supporting v1 CRDs originally intended to create and update all CRDs at v1 - converting those provided at v1beta1. However, due to some issues around the validation, and the fact that CRDs are not fully backward-compatible, OLM will have to create v1beta1 CRDs with the v1beta1 client. This PR adds back support for creating v1beta1 CRDs.
Motivation for the change:
Reviewer Checklist

[ ] Implementation matches the proposed design, or proposal is updated to match implementation
[ ] Sufficient unit test coverage
[ ] Sufficient end-to-end test coverage
[ ] Docs updated or added to /docs
[ ] Commit messages sensible and descriptive

It would be nice to have a test that reproduces the issue so we can be certain of the cause and catch it in the future.

Putting this here for record keeping purposes: https://github.com/kubernetes/kubernetes/issues/87231

/hold
Here is so much wrong about conversion.

Closing this in favor of adding a v1beta1 codepath (something we had originally done in prior versions of the v1 support PR). There will be no client-side conversion between v1beta1 and v1 CRD types in OLM.

/retest

/test e2e-aws-olm

/test e2e-gcp

/retest

/test e2e-aws-olm

/retest

@sttts we addressed the primary concerns you had regarding our CRD conversions - no more using internal types. That being said, OLM has a lot of responsibility around handling and upgrading CRDs and it would extremely valuable to us if we could have a minute of your time to discuss these things further on a call.
/hold cancel

/test e2e-gcp

/test e2e-gcp

/test e2e-gcp

/approve


Can't install zold

Below is transcript from my putty session.  Any ideas? Thanks!
root@ubuntu-c-1vcpu-2gb-nyc3-01:~# gem install zold
Building native extensions.  This could take a while...
ERROR:  Error installing zold:
ERROR: Failed to build gem native extension.
current directory: /var/lib/gems/2.3.0/gems/gherkin-2.12.2/ext/gherkin_lexer_ar

/usr/bin/ruby2.3 -r ./siteconf20180530-6894-1nhgbz.rb extconf.rb
checking for main() in -lc... *** extconf.rb failed ***
Could not create Makefile due to some reason, probably lack of necessary
libraries and/or headers.  Check the mkmf.log file for more details.  You may
need configuration options.

@yegor256/z please, pay attention to this issue

@silverbonegg/z this project will fix the problem faster if you donate a few dollars to it; just click here and pay via Stripe, it's very fast, convenient and appreciated; thanks a lot!


Added CrossDownloadManager plugin

Closes #23

This PR includes the new reference to the plugin. #20 still contained a version of this plugin that was not build up according to the rules for this repository.

Do you have time to merge it, or is there something I left out?
This plugin meets all requirements for a Plugin (thanks to @martijn00) and is available on NuGet and GitHub, like you can see in the README.md.

Hello! I'm the build bot for the Mono project. I need approval from a Mono team member to build this pull request. A team member should reply with "approve" to approve a build of this pull request, "whitelist" to whitelist this and all future pull requests from this contributor, or "build" to explicitly request a build, even if one has already been done. Contributors can ignore this message.

@SimonSimCity awesome! I will have a bit of feedback on the GitHub page ;)


extractors loaded for unrelated tests

in fixtures.py at the top level, we call get_*_extractor for a selection of samples to compute their hashes. because this is at the top level, this is done at import time, rather then when the relevant test is invoked. this slows down the invocation of other tests.
https://github.com/mandiant/capa/blob/19495f69d747bff25ae3bfe4022a6a7c88d18a68/tests/fixtures.py#L668-L670

since these fixtures aren't used by more than one test, they can be just as easily embedded in the single test file

fixed in #1708


Score OpenSSF

Hello Boys,
I've just run the tool OpenSSF on my machine.
My objectif was to evaluate the product and share the result with you.
More information about OpenSSF on the Microsoft DevBlogs article
This is the final result :
RESULTS
Aggregate score:  6.9 / 10
Check scores :

SCORE
NAME
REASON
DETAILS
DOCUMENTATION/REMEDIATION

10 / 10
Binary-Artifacts
no binaries found in the repo

Binary Artifacts Check

8 / 10
Branch-Protection
branch protection is not maximal on development and all release branches
Info: 'allow deletion' disabled on branch 'dev'; Info: 'force pushes' disabled on branch 'dev'; Info: required approving review count is 1 on branch 'dev'
Branch Protection Check

10 / 10
CI-Tests
16 out of 16 merged PRs checked by a CI test -- score normalized to 10

CI Tests Check

0 / 10
CII-Best-Practices
no effort to earn an OpenSSF best practices badge detected

CII Best Practices Check

5 / 10
Code-Review
Found 15/30 approved changesets -- score normalized to 5

Code Review Check

10 / 10
Contributors
project has 28 contributing companies or organizations

Contributors Check

10 / 10
Dangerous-Workflow
no dangerous workflow patterns detected

Dangerous Workflow Check

10 / 10
Dependency-Update-Tool
update tool detected
Info: detected update tool: Dependabot
Dependency Update Tool Check

0 / 10
Fuzzing
project is not fuzzed
Warn: no fuzzer integrations found
Fuzzing Check

10 / 10
License
license file detected
Info: project has a license file: LICENSE:0 Info: FSF or OSI recognized license: MIT License
License Check

10 / 10
Maintained
30 commit(s) and 28 issue activity found in the last 90 days -- score normalized to 10

Maintained Check

?
Packaging
packaging workflow not detected
Warn: no GitHub/GitLab publishing workflow detected
Packaging Check

0 / 10
Pinned-Dependencies
dependency not pinned by hash
Warn: GitHub-owned GitHubAction not pinned by hash
Pinned Dependencies Check

7 / 10
SAST
SAST tool detected but not run on all commits
Info: SAST configuration detected: CodeQL Warn: 0 commits out of 16 are checked with a SAST tool
SAST Check

10 / 10
Security-Policy
security policy file detected
Info: security policy file detected: SECURITY.md:1
Security Policy Check

?
Signed-Releases
no releases found

Signed Releases Check

0 / 10
Token-Permissions
detected GitHub workflow tokens with excessive permissions
Warn: jobLevel 'checks' permission set to 'write'
Token Permissions Check

8 / 10
Vulnerabilities
2 existing vulnerabilities detected
Warn: Project is vulnerable to: GHSA-3xgq-45jj-v275, GHSA-952p-6rrq-rcjv
Vulnerabilities Check

In my point of view, this result is really interesting to be displayed, we can plan to integrate the badge and try to improve the score with best Open Source practises proposed : https://scorecard.dev/#run-the-checks
Br,
Adrien C.

Interesting indeed... but yet another thing we would need to maintain.
Not sure if we have bandwidth for that. Might be that done are low hanging fruit.


Deposit is stuck in accessioning; hidden MacOS "Icon" files are probably to blame

This H2 deposit has hit an error at validate-bag because there's a problem validating the file transfer from the /dor/workspace to the preservation mount: https://argo.stanford.edu/view/druid:sg697rz2807
The error message is:
Error: validate-bag : Bag validation failure(s) for druid:sg697rz2807:
[{:checksum_mismatch=>"Failed manifest verification.

Differences: \n{\"data/content/CoronavirusSL5-M2seq_UBR_output/Figures/Icon"=>
{:sha256=>{\"manifest\"=>\"e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855\", \"generated\"=>nil},
:sha1=>{\"manifest\"=>\"da39a3ee5e6b4b0d3255bfef95601890afd80709\", \"generated\"=>nil},
:md5=>{\"manifest\"=>\"d41d8cd98f00b204e9800998ecf8427e\", \"generated\"=>nil}},

\"data/content/CoronavirusSL5-M2seq_UBR_output/Figures/Icon\\r\"=>
{:sha256=>{\"manifest\"=>nil,
\"generated\"=>\"e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855\"},
:sha1=>{\"manifest\"=>nil, \"generated\"=>\"da39a3ee5e6b4b0d3255bfef95601890afd80709\"},
:md5=>{\"manifest\"=>nil, \"generated\"=>\"d41d8cd98f00b204e9800998ecf8427e\"}}}"}]

I think that message means:

There's a file called Icon in the manifest but it can't be found on the filesystem: \"generated\"=>nil
There's a file on the filesystem called Icon\r but it can't be found in the manifest: \"manifest\"=>nil

The problem seems to be that MacOS "Icon" files include a carriage return in the filename and that's causing a mismatch between what's in the manifest file and what's on the filesystem.
Proposal
We should sidestep this issue by excluding MacOS Icon files from accessioning. These are similar to .DS_Store files and other MacOS-specific files that become filesystem clutter when on non-Mac filesystems. We already exclude .DS_Store and other files, so I'm not sure why we don't exclude Icon files already. These files have to do with the appearance of icons in MacOS (desktop and finder displays?). If you look at the Icon files that are in this accession, they are actually empty and provide no value. So including them is only causing accessioning to stall.

Actually, looking at this more closely, I think I'm wrong and the manifest has encoded the carriage return in the filename correctly. If you look at that line in the hexadecimal display, you can see it has the 0d character, which is ASCII carriage return:
$ grep -i icon manifest-md5.txt | xxd
00000000: 6434 3164 3863 6439 3866 3030 6232 3034  d41d8cd98f00b204
00000010: 6539 3830 3039 3938 6563 6638 3432 3765  e9800998ecf8427e
00000020: 2064 6174 612f 636f 6e74 656e 742f 436f   data/content/Co
00000030: 726f 6e61 7669 7275 7353 4c35 2d4d 3273  ronavirusSL5-M2s
00000040: 6571 5f55 4252 5f6f 7574 7075 742f 4669  eq_UBR_output/Fi
00000050: 6775 7265 732f 4963 6f6e 0d0a            gures/Icon..

0d0a translates to \r\n, which is the end of the line.
So maybe what we need is for the validate-bag code to be able to handle this character?


v4: ion-content calculates double keyboard offset using ion-grid

Ionic version:
[ ] 2.x
[ ] 3.x
[x ] 4.x
I'm submitting a ...
[ x] bug report
[ ] feature request
Current behavior:
I have a form embedded in a column in <ion-grid>, which is directly placed under <ion-content>. When a user focuses on an input in this form, the keyboard height is calculated double. Example screenshot from simulator
Expected behavior:
The height of the --keyboard-offset is calculated double. The white area should not be visible
Steps to reproduce:
<ion-content>
  <ion-grid>
    <ion-row>
      <ion-col size="12" align-self-center>
        <form *ngIf="!success" [formGroup]="forgotPasswordForm" (ngSubmit)="sendPasswordRequest()" padding>
          <input class="ion-input" placeholder="Code" formControlName="hash" type="text">
        </form>
      </ion-col>
    </ion-row>
  </ion-grid>
</ion-content>

Ionic info:
Ionic:

   ionic (Ionic CLI)             : 4.2.1 (/Users/eds/.nvm/v8.9.4/lib/node_modules/ionic)
   Ionic Framework               : @ionic/angular 4.0.0-beta.13
   @angular-devkit/build-angular : 0.7.5
   @angular-devkit/schematics    : 0.7.5
   @angular/cli                  : 6.1.5
   @ionic/angular-toolkit        : not installed

Cordova:

   cordova (Cordova CLI) : 8.1.2<EMAIL_ADDRESS>   Cordova Platforms     : android 7.1.1, browser 5.0.4, ios 4.5.5
   Cordova Plugins       : cordova-plugin-ionic-keyboard 2.1.2, cordova-plugin-ionic-webview 2.1.4, (and 8 other plugins)

System:

   ios-deploy : 2.0.0
   NodeJS     : v8.9.4 (/Users/eds/.nvm/v8.9.4/bin/node)
   npm        : 6.4.1
   OS         : macOS High Sierra
   Xcode      : Xcode 10.0 Build version 10A255

Thanks for your issue! Could you please check if this is still an issue in beta.17? That would be very nice :)


Fix for failtureTolerance (-f) option not working from the command line

NSNumber options in BPConfiguration were getting set as NSStrings when set from the command line.
This was working for most options because intValue or integerValue works on both NSString and NSNumber.
Added a BP_INTEGER option type so that integers are explicitly converted to NSNumber.

Thanks for the fixes!


Upgrade Laravel version to 7

o work with Laravel 7
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Conclusion: remove laravel/framework v7.15.0
    - Conclusion: don't install laravel/framework v7.15.0
    - toolkito/larasap V1.2.2 requires illuminate/support 5.5.x|5.6.x -> satisfiable by laravel/framework[5.5.x-dev, 5.6.x-dev], illuminate/support[5.5.x-dev, 5.6.x-dev, v5.5.0, v5.5.16, v5.5.17, v5.5.2, v5.5.28, v5.5.33, v5.5.34, v5.5.35, v5.5.36, v5.5.37, v5.5.39, v5.5.40, v5.5.41, v5.5.43, v5.5.44, v5.6.0, v5.6.1, v5.6.10, v5.6.11, v5.6.12, v5.6.13, v5.6.14, v5.6.15, v5.6.16, v5.6.17, v5.6.19, v5.6.2, v5.6.20, v5.6.21, v5.6.22, v5.6.23, v5.6.24, v5.6.25, v5.6.26, v5.6.27, v5.6.28, v5.6.29, v5.6.3, v5.6.30, v5.6.31, v5.6.32, v5.6.33, v5.6.34, v5.6.35, v5.6.36, v5.6.37, v5.6.38, v5.6.39, v5.6.4, v5.6.5, v5.6.6, v5.6.7, v5.6.8, v5.6.9].
    - toolkito/larasap V1.2.3 requires illuminate/support 5.6.x|5.7.x -> satisfiable by laravel/framework[5.6.x-dev, 5.7.x-dev], illuminate/support[5.6.x-dev, 5.7.17, 5.7.18, 5.7.19, 5.7.x-dev, v5.6.0, v5.6.1, v5.6.10, v5.6.11, v5.6.12, v5.6.13, v5.6.14, v5.6.15, v5.6.16, v5.6.17, v5.6.19, v5.6.2, v5.6.20, v5.6.21, v5.6.22, v5.6.23, v5.6.24, v5.6.25, v5.6.26, v5.6.27, v5.6.28, v5.6.29, v5.6.3, v5.6.30, v5.6.31, v5.6.32, v5.6.33, v5.6.34, v5.6.35, v5.6.36, v5.6.37, v5.6.38, v5.6.39, v5.6.4, v5.6.5, v5.6.6, v5.6.7, v5.6.8, v5.6.9, v5.7.0, v5.7.1, v5.7.10, v5.7.11, v5.7.15, v5.7.2, v5.7.20, v5.7.21, v5.7.22, v5.7.23, v5.7.26, v5.7.27, v5.7.28, v5.7.3, v5.7.4, v5.7.5, v5.7.6, v5.7.7, v5.7.8, v5.7.9].
    - Can only install one of: laravel/framework[5.5.x-dev, v7.15.0].
    - Can only install one of: laravel/framework[5.6.x-dev, v7.15.0].
    - Can only install one of: laravel/framework[5.7.x-dev, v7.15.0].
    - don't install illuminate/support 5.5.x-dev|don't install laravel/framework v7.15.0
    - don't install illuminate/support 5.6.x-dev|don't install laravel/framework v7.15.0
    - don't install illuminate/support 5.7.17|don't install laravel/framework v7.15.0
    - don't install illuminate/support 5.7.18|don't install laravel/framework v7.15.0
    - don't install illuminate/support 5.7.19|don't install laravel/framework v7.15.0
    - don't install illuminate/support 5.7.x-dev|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.5.0|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.5.16|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.5.17|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.5.2|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.5.28|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.5.33|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.5.34|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.5.35|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.5.36|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.5.37|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.5.39|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.5.40|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.5.41|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.5.43|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.5.44|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.0|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.1|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.10|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.11|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.12|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.13|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.14|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.15|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.16|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.17|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.19|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.2|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.20|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.21|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.22|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.23|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.24|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.25|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.26|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.27|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.28|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.29|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.3|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.30|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.31|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.32|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.33|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.34|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.35|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.36|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.37|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.38|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.39|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.4|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.5|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.6|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.7|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.8|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.6.9|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.7.0|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.7.1|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.7.10|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.7.11|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.7.15|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.7.2|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.7.20|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.7.21|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.7.22|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.7.23|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.7.26|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.7.27|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.7.28|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.7.3|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.7.4|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.7.5|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.7.6|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.7.7|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.7.8|don't install laravel/framework v7.15.0
    - don't install illuminate/support v5.7.9|don't install laravel/framework v7.15.0
    - Installation request for laravel/framework (locked at v7.15.0, required as ^7.0) -> satisfiable by laravel/framework[v7.15.0].
    - Installation request for toolkito/larasap ^1.2 -> satisfiable by toolkito/larasap[V1.2.2, V1.2.3].

Installation failed, reverting ./composer.json to its original content.


updated specs, removed the foreign keys hack, updated README

updated README to be more user friendly
updated to latest specs
removed the foreign keys hack (not needed with latest specs)
#25 table API feedback

hide datasource and schema object creation
infer schema when schema is ommitted
Table::headers method
Table::save method
Table::schema method
Table::read method
allow Schema to be constructed with array instead of object

Coverage decreased (-0.04%) to 91.185% when pulling a4deca59fed2832738116b02c5043d365bd5e89e on OriHoch:master into cd378b9832060bd686bb9099827271802f23291a on frictionlessdata:master.

Coverage increased (+1.06%) to 92.277% when pulling 3a9e585950c57be0c1c495677ce322f3905ff4f9 on OriHoch:master into cd378b9832060bd686bb9099827271802f23291a on frictionlessdata:master.

Coverage increased (+1.06%) to 92.277% when pulling cf429f20bf1c37b6190fde79ca97182fe0f0d170 on OriHoch:master into cd378b9832060bd686bb9099827271802f23291a on frictionlessdata:master.


fix(OverflowMenu): support decimal point calculating the size of pointng arrow

Refs IBM/carbon-components#979.
Changelog
New

Support for decimal support calculating the size of pointng arrow of overflow menu.

@joshblack Good point - Seems that we need to wait for https://github.com/speedskater/babel-plugin-rewire/issues/209 to get that function tested.


Modified design burger never reach the goal

Good evening, I have modified the design of my burger to install additional  equipment, I separated the wheels, modified the height, I made it longer, etc. Additionally, the footprint parameter has also been modified according to the approximate dimensions in the costmap_common_params_burger.yaml file.
My problem is that now when  I send the robot to 2d nav gold in Rviz the burger never reach it and moves from one place to another without being able reach the goal.
How can I solved this problem ?  thanks for your help

@jcotof
Hi,
When you are modifying the hardware design, you also need to modify the parameter that describes the robot in the source code and OpenCR firmware.
https://github.com/ROBOTIS-GIT/OpenCR/blob/master/arduino/opencr_arduino/opencr/libraries/turtlebot3/examples/turtlebot3_waffle/turtlebot3_core/turtlebot3_waffle.h#L24
Thank you.


MAINT: Pin pybind11==2.13.1

pybind11 released 2.13.3 yesterday. That might be related to the failures on Windows that just started.

We have this nice system for generating requirements files in requirements/. Shouldn't the CI tools use those files?

We have this nice system for generating requirements files in requirements/. Shouldn't the CI tools use those files?

There's probably room for improvement there, yeah, although various CI jobs deliberately test certain versions of dependencies.

Do we not want to try 2.13.2 first?

2.13.2 and 2.13.3 were released on the same day, so 2.13.3 is probably a quick fix for a problem in 2.13.2. I wanted to go back to a version that we know should work.

The two Windows jobs that were failing with a compiler error pass here, so that confirms that the problem is pybind11, particularly https://github.com/pybind/pybind11/issues/5300.
Pinning isn't necessarily the right solution at the moment--pybind11 might get a fix out quickly--so I'm closing this PR.  Our CI gurus can decide if we should actually implement the pin.


[WSL2 Ubuntu 20.04] No Backend URL generated, TF memory allocation errors on mini?

Attempting the manual setup
I run the following command to start this up:
python3 app.py --port 8080 --model_version mini

I get a ton of Tensorflow errors for memory allocation, but my VRAM usage is only about 3GB/16GB at this point - with Mini, I had thought 16GB VRAM would be more than enough to run it?
When I go to the localhost page, I also just get an output of:
{"success":true}
And nothing else, so I'm not sure what I'm supposed to do here. I'm guessing I screwed it up somehow along the line, but I'm not quite sure where to figure out where the real problem is?
I do remember when I first set up WSL I was able to get some sort of backend URL generated (from an older commit), but I did not have CUDA setup or anything, so it had been defaulting to the CPU, but I couldn't tell you exactly what I had run to get to this point, or when it ended up breaking.
I know a few of the issues have a similar issue with the memory allocation as well, but I don't think I saw anything about the backend URL just... not appearing.
The Docker setup
So after not knowing what to do, I tried the setup with Docker. So far with that, I get the frontend page that asks me for a URL, but when it starts the server, I never see any other messages, and no VRAM usage like with the manual way, nor any extra links?

I'm just really stuck and would appreciate some sort of step in the right direction, if anyone has had a similar issue (I guess this is probably more than one issue though), or knows I overlooked something.

In manual mode you have to have a second shell and cd into the interfaces directory, run npm install and then mpn start.  Then you'll have a frontend - like the one you saw in Docker - at localhost:3000.  From there you can put in the URL "http://localhost:8080" where it asks for a URL.
I believe this should be the same for the Docker version, just it only shows you the frontend having loaded.

Wow.
Alright well I'm a dingus. Thanks for the help, I was able to get it working, for some reason I guess I was fixated on having like, some sort of loca.lt link (or whatever the latest version link points to), and I don't even know why.
Thanks for the help.


[Doc improvement][Combine bots with tabs][2990300]

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/resources/bot-v3/bots-with-tabs.md
92
5
link
:white_check_mark:

msteams-platform/toolkit/add-capability.md
99
5
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.


PlatformException(Unexpected security result code, Code: -25299, Message: The specified item already exists in the keychain., -25299, null)

Can anybody let me know what could be possible fix for the issue.
Platform: iOS; Flutter Secure Storage version: 9.2.2
message_codecs.dart in StandardMethodCodec.decodeEnvelope at line 648 within flutter
platform_channel.dart in MethodChannel._invokeMethod at line 334 within flutter
flutter_secure_storage.dart in FlutterSecureStorage.write at line 114 within flutter_secure_storage

I am experiencing the same issue.

I have made little progress:
        const iOptions = IOSOptions(             accessibility: KeychainAccessibility.first_unlock_this_device);         try {           await write(key: key, value: value, iOptions: iOptions);         } on PlatformException catch (e) {           if (e.message ==               "Code: -25299, Message: The specified item already exists in the keychain.") {               await delete(key: key, iOptions: iOptions);               await write(key: key, value: value, iOptions: iOptions);            }
Until now it seems working.

I tried to clear the existing value before overwriting it didn't work.

This issue happens when we have data already in the keychain with the same key using but with different accessibility mode. delete the key the key in that accessibility mode and then try writing again in this mode, that will resolve this issue.


Use "currentColor" for the SVG icons fill

Do you think it would be possible to use currentColor instead of #000 for the fill of the SVG icons (or potentially just leave it empty, so it inherits the color value)? In this way it would be easier to tint the icons (instead of using CSS filters). Also, since the current fill is black anyway, this probably would have a minimal impact on the existing usage of the icons.
If it's something you would consider, let me know that I can open a PR.

Would definitely be open to a PR @didoo! Sorry for just seeing this now!


createProvider is missing in react-redux_v5.x.x.js

Sometimes we need that function.

Something wrong probably.
Provider
This type is incompatible with
union: key set | type application of polymorphic type: type React$ComponentType: /private/tmp/flow/flowlib_292bae27/react.js:158


Deadlock on exception when opening connection

When SqliteDict is initialized, a connection to SQLite is opened (in here). This connection might fail.
If an exception happens, the connection does not get opened. However, the initialization still tries to create the table. In the commit of that operation, the execution is deadlocked, because the code keeps waiting infinitely in the req Queue.

I think I'm having the same issue.

This crashes internally (traceback visible on stdout) but never returns, looking like the mentioned deadlocked.

Hi @arnauorriols, can you add a code example that reproduces this error (so we can fix it)?
I've opened #97 proposing a fix.

@arnauorriols realized that #97 has a race condition. #107 is my fresh approach that solves the undetected failure to open the connection (on first command exection) which currently leads to a deadlock in the calling code. It also avoids the race condition when the thread's run() is slower to set self.exception than the calling code executes a command (including a then false negative exception check) and then deadlocking on the answer.


Login to Mint failed due to timeout in the Multifactor Method Loop

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

I seem to have everything installed and running correctly, however i keep running into this error.
I followed the steps to add the 2fa authenticator token and in my config i have: mfa_method="soft-token" and mfa_token=<mytokenhere>

What version of MintAPI are you using?
mintapi==2.14
What command[s] did you run / steps to reproduce?
This happened during initialization when calling get_account_data()
Stacktrace/error received:
self.login_and_get_token(
File "/<virtualenv dir>/lib/python3.9/site-packages/mintapi/browser.py", line 141, in login_and_get_token
raise Exception(msg) from e
Exception: Could not sign in to Mint. Current page: https://accounts.intuit.com/index.html<Url params redacted in case theres anything sensitive but attached screenshot below>

This is the screen from the link:

What did you expect to happen?

I expected my code to just run and return the data i wanted.

What actually happened?

no data returned because of this error

Additional context
Add any other context about the problem here.
My full config here:
mintapi.Mint(
            email=<email>,
            password=<password>,
            # Optional parameters
            mfa_token=<token>,
            mfa_method="soft-token",
            intuit_account=<account_name>,
            headless=True,
            use_chromedriver_on_path=True,
            wait_for_sync=True,
            wait_for_sync_timeout=60 * 5
)

I literally had the same issue.
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/mintapi/browser.py", line 133, in login_and_get_token
    beta,
  File "/usr/local/lib/python3.7/site-packages/mintapi/signIn.py", line 418, in sign_in
    "Login to Mint failed due to timeout in the Multifactor Method Loop"
RuntimeError: Login to Mint failed due to timeout in the Multifactor Method Loop
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/mintapi/signIn.py", line 412, in sign_in
    expected_conditions.url_contains("{}/".format(url))
  File "/usr/local/lib/python3.7/site-packages/selenium/webdriver/support/wait.py", line 95, in until
    raise TimeoutException(message, screen, stacktrace)
selenium.common.exceptions.TimeoutException: Message:

It turned out that some of the DNS blocking (Pi-Hole) I do I my network was preventing some of the API calls.
After whitelisting intuit.com, the issue was resolved.


[tool.poetry.scripts] hypermodern-python script not found

REPRO

checkout chapter07 branch

08:51 $ git log | head
commit 71dab2c92819ba95a25fa4e4127d9ea62a580c93
Author: Claudio Jolowicz<EMAIL_ADDRESS>Date:   Thu Nov 21 09:20:04 2019 +0100

    Add Docker Hub badge

08:55 $ grep scripts pyproject.toml -A 2
[tool.poetry.scripts]
hypermodern-python = "hypermodern_python.console:main"

08:56 $ poetry install
Installing dependencies from lock file

No dependencies to install or update

  - Installing hypermodern-python (0.1.0)

08:56 $ hypermodern-python
-bash: hypermodern-python: command not found

I suspect that placing the code in the src directory is causing this.

Oops. I wasn't invoking the command properly. Here's the correct way to do this:
poetry run hypermodern-python


Count the number of words and duplicates in a text by using map

This issue is about [Algo/DS Name/Question Name](link to resource for the Algo/DS/Question)

[x] I searched or browsed the repo’s other issues to ensure this is not a duplicate

[x] Issue Title is in form [Algo/DS Name] in [Language] (ex:- BubbleSort in Java)

[x] I will send my PR only after I'm assigned to this issue by any maintainer

[x] Assign this issue to me (I want to work on this)

[x] I ensure that I am not already assigned to 2 or more issues

Please add a Hacktoberfest label for me @jhudsonsg

We can use two maps to count the word. One map is used to count the total number of words. Another map is used to count the duplicates.The functions of the map are O(1) so they are quick.

This change belongs to the other section of this project.

I want to work on this

I want to work on this

Hi There!
You guys can create a new issue. As this issue is currently assigned to @JunJul.

I already finished the issue. You can close it.@sobhanbera