Render State Notation Parser: Add default section (close #79, close #77)

What about default resource signatures?
Added a feature


fix netrc matching when host has no port

Whew, glad I QA'd against a real server, and not just a local test one :)
The net.SplitHostPort function expects the host string to have a port (like localhost:8008). Currently, it blows up if you use a host on a standard http port.

👍


Add working link to using odata with httpclient

Add link to the official MSFT documentation

Docs Build status updates of commit 12dd90c:
:warning: Validation status: warnings

File
Status
Preview URL
Details

Odata-docs/client/using-extensions.md
:warning:Warning
View
Details

Odata-docs/client/using-extensions.md

Line 172, Column 257: [Warning-hard-coded-locale] Link 'https://docs.microsoft.com/en-us/odata/client/using-httpclient' contains locale code 'en-us'. For localizability, remove 'en-us' from links to most Microsoft sites.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit e166d24:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Odata-docs/client/using-extensions.md
:white_check_mark:Succeeded
View

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


excess quests to get the collector quest

I discovered that to get the Collector quest you need to complete the Long Line quest and Booze quest of Ragman. In the original game, these quests are not needed to obtain the Collector quest.

Originally written by K1rush4

I've just checked the SPT quest data and can confirm The Collector has no need for the completion of Booze or Long Line
Can you elaborate on what you mean or what makes you believe they're needed?

Originally written by chomp

I completed the necessary quests to obtain The Collector, but the quest never opened. Then, using the SPT-AKI Profile Editor, I began to figure out what quests were still needed. It turned out that these two quests were needed. As proof, I provide two of my profiles. They differ only in these two quests. When importing, please remove "without_quests_" and "with_quest_" from the profile names.

Attachments

profiles.zip (428 KiB)

Originally written by K1rush4

This is every quest required for Collector in an un-modified SPT 3.7.1 installation: https://gist.github.com/DrakiaXYZ/252a31b6dfd00fc9a4f44aa6df4a335b

Originally written by DrakiaXYZ

Of course, I understand that these quests should not be necessary for the Collector. But first, load the profile yourself without_quests_061fff419d18fda1e2243a5c, and then add the Long Line and Booze quests through the SPT-AKI profile editor and make sure that it does not work correctly.

Originally written by K1rush4

"Collector" is available from Fence on the "without_quests" profile on an unmodded SPT 3.7.1 and 3.7.2 install:
https://i.imgur.com/pby6bnZ.png

Originally written by DrakiaXYZ

Okay. This means my installed mods are making their own changes. Thank you!

Originally written by K1rush4

Appears to be a mod-related issue? closing

Originally written by chomp


TR Translation

Hello! I just finished the translation to Turkish.
I have to admit that there are some points where I'm not very confident with my translation. I don't think I did any major mistakes, but maybe a quick check could be useful?
Still, there goes. Thank you for the amazing content!

Hi @ogulyurdakul ! So sorry I missed the email notification for this Pull Request a year ago. Thank you for emailing me again to let me know, and thank you for taking the time & effort to make this available to more folks! 🎉


Add extra data to max spend and diamond txn construction

Current dependencies on/for this PR:

main

PR #883   👈

This stack of pull requests is managed by Graphite.


[BUG] 3.0.0-alpha10 does not test on NonStop

The FIPS structure no longer tests properly on NonStop x86.  The build was in a clean directory. This appears to be a result of the FIPS module not being installable anymore (at 3d0b6494d5a973d516e0944bc02b22385fca318a) - 3.0.0-alpha10-dev (e3197e5ab23f7d38da940999422d9ba2dc487e94) worked fine:
INSTALL FAILED
../../util/wrap.pl ../../apps/openssl fipsinstall -out fips.cnf -module ../../providers/fips.so -provider_name fips -mac_name HMAC -macopt 'digest:SHA256' -macopt 'hexkey:f4556650ac31d35461610bac4ed81b1a181b2d8a43ea2854cbae22ca74560813' -section_name fips_sect => 139
not ok 3 - fipsinstall
# ------------------------------------------------------------------------------
#   Failed test 'fipsinstall'
#   at test/recipes/03-test_fipsinstall.t line 98.
Can't open fips.cnf for reading, No such file or directory
00000000:error:80000FA2:system library:BIO_new_file:No such file or directory:/home/ituglib/randall/jenkins/.jenkins/workspace/OpenSSL-3.0_Pipeline/crypto/bio/bss_file.c:69:calling fopen(fips.cnf, r)
00000000:error:10000080:BIO routines:BIO_new_file:no such file:/home/ituglib/randall/jenkins/.jenkins/workspace/OpenSSL-3.0_Pipeline/crypto/bio/bss_file.c:77:
VERIFY FAILED
../../util/wrap.pl ../../apps/openssl fipsinstall -in fips.cnf -module ../../providers/fips.so -provider_name fips -mac_name HMAC -macopt 'digest:SHA256' -macopt 'hexkey:f4556650ac31d35461610bac4ed81b1a181b2d8a43ea2854cbae22ca74560813' -section_name fips_sect -verify => 1
not ok 4 - fipsinstall verify
# ------------------------------------------------------------------------------
#   Failed test 'fipsinstall verify'
#   at test/recipes/03-test_fipsinstall.t line 105.
not ok 5 - fipsinstall verify fail no module mac

The build works on both ia64 and x86. Tests pass on ia64 but fail on x86 seemingly because of the FIPS install.

A libc problem?  fopen() is a very standard call.

A libc problem?  fopen() is a very standard call.

Highly unlikely. fopen is very stable and this worked previously.

Highly unlikely. fopen is very stable and this worked previously.

Please then explain the error that doesn't find fopen?

Please then explain the error that doesn't find fopen?

My presumption of innocence threshold is low these days 😀

My presumption of innocence threshold is low these days 😀

../../util/wrap.pl ../../apps/openssl fipsinstall -out fips.cnf -module ../../providers/fips.so -provider_name fips -mac_name HMAC -macopt 'digest:SHA256' -macopt 'hexkey:f4556650ac31d35461610bac4ed81b1a181b2d8a43ea2854cbae22ca74560813' -section_name fips_sect => 139

Looks like this crashed. Are you able to run this command "manually" and figure out what is going on?

../../util/wrap.pl ../../apps/openssl fipsinstall -out fips.cnf -module ../../providers/fips.so -provider_name fips -mac_name HMAC -macopt 'digest:SHA256' -macopt 'hexkey:f4556650ac31d35461610bac4ed81b1a181b2d8a43ea2854cbae22ca74560813' -section_name fips_sect => 139

Looks like this crashed. Are you able to run this command "manually" and figure out what is going on?

Please then explain the error that doesn't find fopen?

The error is that the file is not found, not the symbol! The file isn't found due to previous fipsinstall failure.

Please then explain the error that doesn't find fopen?

The error is that the file is not found, not the symbol! The file isn't found due to previous fipsinstall failure.

I tried to run the initial install manually. It seems that the location where test is run, inside the test/x directory, is not where fips.cnf is, which is in the parent directory as test/fips.cnf. Once the install fails, nothing else works. I tried running this from ./test as:
../apps/openssl fipsinstall -in fips.cnf -module ../providers/fips.so -provider_name fips -mac_name HMAC -macopt 'digest:SHA256' -macopt 'hexkey:f4556650ac31d35461610bac4ed81b1a181b2d8a43ea2854cbae22ca74560813' -section_name fips_sect -verify
version not found
VERIFY FAILED
00000000:error:80000FA2:system library:process_include:No such file or directory:/home/ituglib/randall/openssl-3.0/crypto/conf/conf_def.c:756:calling stat(fipsmodule.cnf)
00000000:error:0700006C:configuration file routines:NCONF_get_string:no value:/home/ituglib/randall/openssl-3.0/crypto/conf/conf_lib.c:282:group=fips_sect name=install-version

I tried to run the initial install manually. It seems that the location where test is run, inside the test/x directory, is not where fips.cnf is, which is in the parent directory as test/fips.cnf. Once the install fails, nothing else works. I tried running this from ./test as:
../apps/openssl fipsinstall -in fips.cnf -module ../providers/fips.so -provider_name fips -mac_name HMAC -macopt 'digest:SHA256' -macopt 'hexkey:f4556650ac31d35461610bac4ed81b1a181b2d8a43ea2854cbae22ca74560813' -section_name fips_sect -verify
version not found
VERIFY FAILED
00000000:error:80000FA2:system library:process_include:No such file or directory:/home/ituglib/randall/openssl-3.0/crypto/conf/conf_def.c:756:calling stat(fipsmodule.cnf)
00000000:error:0700006C:configuration file routines:NCONF_get_string:no value:/home/ituglib/randall/openssl-3.0/crypto/conf/conf_lib.c:282:group=fips_sect name=install-version

Please then explain the error that doesn't find fopen?

The error is that the file is not found, not the symbol! The file isn't found due to previous fipsinstall failure.

I've been unable to work out why the file is not found, other than the structure of the fips test seems to be off somehow. fips.cnf is in ./test, which is not where the test is being run.

Please then explain the error that doesn't find fopen?

The error is that the file is not found, not the symbol! The file isn't found due to previous fipsinstall failure.

I've been unable to work out why the file is not found, other than the structure of the fips test seems to be off somehow. fips.cnf is in ./test, which is not where the test is being run.

Please then explain the error that doesn't find fopen?

The error is that the file is not found, not the symbol! The file isn't found due to previous fipsinstall failure.

I've been unable to work out why the file is not found, other than the structure of the fips test seems to be off somehow. fips.cnf is in ./test, which is not where the test is being run.

Here is the configdata dump, if that helps:
Command line (with current working directory = .):

    perl ./Configure --prefix=/usr/local-ssl3.0 --openssldir=/usr/local-ssl3.0/ssl --with-rand-seed=rdcpu

Perl information:

    perl
    5.26.3 for NSV-D-nonstop_kernel

Enabled features:

    acvp_tests
    aria
    async
    autoalginit
    autoerrinit
    autoload-config
    bf
    blake2
    bulk
    camellia
    capieng
    cast
    chacha
    cmac
    cmp
    cms
    comp
    ct
    deprecated
    des
    dgram
    dh
    dsa
    dso
    dtls
    dynamic-engine
    ec
    ec2m
    ecdh
    ecdsa
    egd
    engine
    err
    filenames
    fips
    fips-securitychecks
    gost
    idea
    legacy
    md4
    mdc2
    module
    multiblock
    nextprotoneg
    pinshared
    ocb
    ocsp
    padlockeng
    pic
    poly1305
    posix-io
    psk
    rc2
    rc4
    rdrand
    rfc3779
    rmd160
    scrypt
    secure-memory
    seed
    shared
    siphash
    siv
    sm2
    sm3
    sm4
    sock
    srp
    srtp
    sse2
    ssl
    static-engine
    stdio
    tests
    tls
    ts
    ui-console
    whirlpool
    tls1
    tls1-method
    tls1_1
    tls1_1-method
    tls1_2
    tls1_2-method
    tls1_3
    dtls1
    dtls1-method
    dtls1_2
    dtls1_2-method

Disabled features:

    afalgeng            [not-linux]      OPENSSL_NO_AFALGENG
    asan                [default]        OPENSSL_NO_ASAN
    asm                 [no asm_arch]    OPENSSL_NO_ASM
    buildtest-c++       [default]
    crypto-mdebug       [default]        OPENSSL_NO_CRYPTO_MDEBUG
    devcryptoeng        [default]        OPENSSL_NO_DEVCRYPTOENG
    ec_nistp_64_gcc_128 [default]        OPENSSL_NO_EC_NISTP_64_GCC_128
    external-tests      [default]        OPENSSL_NO_EXTERNAL_TESTS
    fuzz-libfuzzer      [default]        OPENSSL_NO_FUZZ_LIBFUZZER
    fuzz-afl            [default]        OPENSSL_NO_FUZZ_AFL
    ktls                [default]        OPENSSL_NO_KTLS
    makedepend          [unavailable]
    md2                 [default]        OPENSSL_NO_MD2 (skip crypto/md2)
    msan                [default]        OPENSSL_NO_MSAN
    rc5                 [default]        OPENSSL_NO_RC5 (skip crypto/rc5)
    sctp                [default]        OPENSSL_NO_SCTP
    ssl-trace           [default]        OPENSSL_NO_SSL_TRACE
    threads             [config]
    trace               [default]        OPENSSL_NO_TRACE
    ubsan               [default]        OPENSSL_NO_UBSAN
    unit-test           [default]        OPENSSL_NO_UNIT_TEST
    uplink              [no uplink_arch] OPENSSL_NO_UPLINK
    weak-ssl-ciphers    [default]        OPENSSL_NO_WEAK_SSL_CIPHERS
    zlib                [default]
    zlib-dynamic        [default]
    ssl3                [default]        OPENSSL_NO_SSL3
    ssl3-method         [default]        OPENSSL_NO_SSL3_METHOD

Config target attributes:

    AR => "ar",
    ARFLAGS => "qc",
    CC => "c99",
    HASHBANGPERL => "/usr/bin/env perl",
    RANLIB => "ranlib",
    RC => "windres",
    bn_ops => "THIRTY_TWO_BIT",
    build_file => "Makefile",
    build_scheme => [ "unified", "unix" ],
    cflags => "-g -O2 -Wextensions -Wnowarn=203,220,272,734,770,1506 -Wbuild_neutral_library -Wverbose -Wtarget=tns/x -Wsystype=oss -Wilp32 -WIEEE_float",
    cppflags => "",
    defines => [ "OPENSSL_BUILDING_OPENSSL", "OPENSSL_VPROC=\$(OPENSSL_VPROC)", "_XOPEN_SOURCE", "_XOPEN_SOURCE_EXTENDED=1", "_TANDEM_SOURCE", "B_ENDIAN", "_TANDEM_ARCH=3" ],
    disable => [ "threads" ],
    dso_scheme => "DLFCN",
    enable => [ "egd" ],
    ex_libs => "-lrld",
    includes => [  ],
    lflags => "-Wxld=\"-set systype oss\" -Wxld=\"-set floattype ieee_float\"",
    lib_cflags => "",
    lib_cppflags => "",
    lib_defines => [  ],
    module_cflags => "",
    module_cxxflags => undef,
    module_ldflags => "-Wshared",
    perl => "/usr/bin/perl",
    perl_platform => "Unix",
    shared_argfileflag => "-Wxld_obey=",
    shared_cflag => "",
    shared_defflag => "-Wxld_obey=",
    shared_defines => [  ],
    shared_extension => ".so",
    shared_ldflag => "-Wshared",
    shared_rcflag => "",
    shared_target => "nonstop-shared",
    sys_id => "TANDEM",
    thread_defines => [  ],
    thread_scheme => "(unknown)",
    unistd => "<unistd.h>",

Recorded environment:

    AR =
    ARFLAGS =
    AS =
    ASFLAGS =
    BUILDFILE =
    CC =
    CFLAGS =
    CPP =
    CPPDEFINES =
    CPPFLAGS =
    CPPINCLUDES =
    CROSS_COMPILE =
    CXX =
    CXXFLAGS =
    HASHBANGPERL =
    LD =
    LDFLAGS =
    LDLIBS =
    MT =
    MTFLAGS =
    OPENSSL_LOCAL_CONFIG_DIR =
    PERL =
    RANLIB =
    RC =
    RCFLAGS =
    RM =
    WINDRES =
    __CNF_CFLAGS =
    __CNF_CPPDEFINES =
    __CNF_CPPFLAGS =
    __CNF_CPPINCLUDES =
    __CNF_CXXFLAGS =
    __CNF_LDFLAGS =
    __CNF_LDLIBS =

Makevars:

    AR              = ar
    ARFLAGS         = qc
    CC              = c99
    CFLAGS          =
    CPPDEFINES      =
    CPPFLAGS        =
    CPPINCLUDES     =
    CXXFLAGS        =
    HASHBANGPERL    = /usr/bin/env perl
    LDFLAGS         =
    LDLIBS          =
    PERL            = perl
    RANLIB          = ranlib
    RC              = windres
    RCFLAGS         =

NOTE: These variables only represent the configuration view.  The build file
template may have processed these variables further, please have a look at the
build file for more exact data:
    Makefile

build file:

    Makefile

build file templates:

    Configurations/common0.tmpl
    Configurations/unix-Makefile.tmpl
    Configurations/common.tmpl

Please then explain the error that doesn't find fopen?

The error is that the file is not found, not the symbol! The file isn't found due to previous fipsinstall failure.

I've been unable to work out why the file is not found, other than the structure of the fips test seems to be off somehow. fips.cnf is in ./test, which is not where the test is being run.

Here is the configdata dump, if that helps:
Command line (with current working directory = .):

    perl ./Configure --prefix=/usr/local-ssl3.0 --openssldir=/usr/local-ssl3.0/ssl --with-rand-seed=rdcpu

Perl information:

    perl
    5.26.3 for NSV-D-nonstop_kernel

Enabled features:

    acvp_tests
    aria
    async
    autoalginit
    autoerrinit
    autoload-config
    bf
    blake2
    bulk
    camellia
    capieng
    cast
    chacha
    cmac
    cmp
    cms
    comp
    ct
    deprecated
    des
    dgram
    dh
    dsa
    dso
    dtls
    dynamic-engine
    ec
    ec2m
    ecdh
    ecdsa
    egd
    engine
    err
    filenames
    fips
    fips-securitychecks
    gost
    idea
    legacy
    md4
    mdc2
    module
    multiblock
    nextprotoneg
    pinshared
    ocb
    ocsp
    padlockeng
    pic
    poly1305
    posix-io
    psk
    rc2
    rc4
    rdrand
    rfc3779
    rmd160
    scrypt
    secure-memory
    seed
    shared
    siphash
    siv
    sm2
    sm3
    sm4
    sock
    srp
    srtp
    sse2
    ssl
    static-engine
    stdio
    tests
    tls
    ts
    ui-console
    whirlpool
    tls1
    tls1-method
    tls1_1
    tls1_1-method
    tls1_2
    tls1_2-method
    tls1_3
    dtls1
    dtls1-method
    dtls1_2
    dtls1_2-method

Disabled features:

    afalgeng            [not-linux]      OPENSSL_NO_AFALGENG
    asan                [default]        OPENSSL_NO_ASAN
    asm                 [no asm_arch]    OPENSSL_NO_ASM
    buildtest-c++       [default]
    crypto-mdebug       [default]        OPENSSL_NO_CRYPTO_MDEBUG
    devcryptoeng        [default]        OPENSSL_NO_DEVCRYPTOENG
    ec_nistp_64_gcc_128 [default]        OPENSSL_NO_EC_NISTP_64_GCC_128
    external-tests      [default]        OPENSSL_NO_EXTERNAL_TESTS
    fuzz-libfuzzer      [default]        OPENSSL_NO_FUZZ_LIBFUZZER
    fuzz-afl            [default]        OPENSSL_NO_FUZZ_AFL
    ktls                [default]        OPENSSL_NO_KTLS
    makedepend          [unavailable]
    md2                 [default]        OPENSSL_NO_MD2 (skip crypto/md2)
    msan                [default]        OPENSSL_NO_MSAN
    rc5                 [default]        OPENSSL_NO_RC5 (skip crypto/rc5)
    sctp                [default]        OPENSSL_NO_SCTP
    ssl-trace           [default]        OPENSSL_NO_SSL_TRACE
    threads             [config]
    trace               [default]        OPENSSL_NO_TRACE
    ubsan               [default]        OPENSSL_NO_UBSAN
    unit-test           [default]        OPENSSL_NO_UNIT_TEST
    uplink              [no uplink_arch] OPENSSL_NO_UPLINK
    weak-ssl-ciphers    [default]        OPENSSL_NO_WEAK_SSL_CIPHERS
    zlib                [default]
    zlib-dynamic        [default]
    ssl3                [default]        OPENSSL_NO_SSL3
    ssl3-method         [default]        OPENSSL_NO_SSL3_METHOD

Config target attributes:

    AR => "ar",
    ARFLAGS => "qc",
    CC => "c99",
    HASHBANGPERL => "/usr/bin/env perl",
    RANLIB => "ranlib",
    RC => "windres",
    bn_ops => "THIRTY_TWO_BIT",
    build_file => "Makefile",
    build_scheme => [ "unified", "unix" ],
    cflags => "-g -O2 -Wextensions -Wnowarn=203,220,272,734,770,1506 -Wbuild_neutral_library -Wverbose -Wtarget=tns/x -Wsystype=oss -Wilp32 -WIEEE_float",
    cppflags => "",
    defines => [ "OPENSSL_BUILDING_OPENSSL", "OPENSSL_VPROC=\$(OPENSSL_VPROC)", "_XOPEN_SOURCE", "_XOPEN_SOURCE_EXTENDED=1", "_TANDEM_SOURCE", "B_ENDIAN", "_TANDEM_ARCH=3" ],
    disable => [ "threads" ],
    dso_scheme => "DLFCN",
    enable => [ "egd" ],
    ex_libs => "-lrld",
    includes => [  ],
    lflags => "-Wxld=\"-set systype oss\" -Wxld=\"-set floattype ieee_float\"",
    lib_cflags => "",
    lib_cppflags => "",
    lib_defines => [  ],
    module_cflags => "",
    module_cxxflags => undef,
    module_ldflags => "-Wshared",
    perl => "/usr/bin/perl",
    perl_platform => "Unix",
    shared_argfileflag => "-Wxld_obey=",
    shared_cflag => "",
    shared_defflag => "-Wxld_obey=",
    shared_defines => [  ],
    shared_extension => ".so",
    shared_ldflag => "-Wshared",
    shared_rcflag => "",
    shared_target => "nonstop-shared",
    sys_id => "TANDEM",
    thread_defines => [  ],
    thread_scheme => "(unknown)",
    unistd => "<unistd.h>",

Recorded environment:

    AR =
    ARFLAGS =
    AS =
    ASFLAGS =
    BUILDFILE =
    CC =
    CFLAGS =
    CPP =
    CPPDEFINES =
    CPPFLAGS =
    CPPINCLUDES =
    CROSS_COMPILE =
    CXX =
    CXXFLAGS =
    HASHBANGPERL =
    LD =
    LDFLAGS =
    LDLIBS =
    MT =
    MTFLAGS =
    OPENSSL_LOCAL_CONFIG_DIR =
    PERL =
    RANLIB =
    RC =
    RCFLAGS =
    RM =
    WINDRES =
    __CNF_CFLAGS =
    __CNF_CPPDEFINES =
    __CNF_CPPFLAGS =
    __CNF_CPPINCLUDES =
    __CNF_CXXFLAGS =
    __CNF_LDFLAGS =
    __CNF_LDLIBS =

Makevars:

    AR              = ar
    ARFLAGS         = qc
    CC              = c99
    CFLAGS          =
    CPPDEFINES      =
    CPPFLAGS        =
    CPPINCLUDES     =
    CXXFLAGS        =
    HASHBANGPERL    = /usr/bin/env perl
    LDFLAGS         =
    LDLIBS          =
    PERL            = perl
    RANLIB          = ranlib
    RC              = windres
    RCFLAGS         =

NOTE: These variables only represent the configuration view.  The build file
template may have processed these variables further, please have a look at the
build file for more exact data:
    Makefile

build file:

    Makefile

build file templates:

    Configurations/common0.tmpl
    Configurations/unix-Makefile.tmpl
    Configurations/common.tmpl

This is all about 03-test_fipsinstall.t, and in that case, the location of fips.cnf is a red herring, as it makes local tests of the fipsinstall command.  Its working directory is test-runs/test_fipsinstall (in the build tree, if that differs from the source tree), and that's where you should find the fips.cnf it was testing with.  It is not the same as test/fips.cnf.
So it seems that the issue is that this fips.cnf (the one in test-runs/test_fipsinstall) wasn't written for some reason, specifically on x86.  I assume that's a different "box" than the ia64 one.
As for the command that @mattcaswell said you should run, that was a bit incomplete.  This is closer to being correct:
( cd test-runs/test_fipsinstall; ../../util/wrap.pl ../../apps/openssl fipsinstall -out fips.cnf -module ../../providers/fips.so -provider_name fips -mac_name HMAC -macopt 'digest:SHA256' -macopt 'hexkey:f4556650ac31d35461610bac4ed81b1a181b2d8a43ea2854cbae22ca74560813' -section_name fips_sect )

Hint: to debug a program run like this, I do it as follows:
( cd test-runs/test_fipsinstall; ../../util/wrap.pl gdb ../../apps/openssl fipsinstall --args  ../../apps/openssl fipsinstall -out fips.cnf -module ../../providers/fips.so -provider_name fips -mac_name HMAC -macopt 'digest:SHA256' -macopt 'hexkey:f4556650ac31d35461610bac4ed81b1a181b2d8a43ea2854cbae22ca74560813' -section_name fips_sect )

This is all about 03-test_fipsinstall.t, and in that case, the location of fips.cnf is a red herring, as it makes local tests of the fipsinstall command.  Its working directory is test-runs/test_fipsinstall (in the build tree, if that differs from the source tree), and that's where you should find the fips.cnf it was testing with.  It is not the same as test/fips.cnf.
So it seems that the issue is that this fips.cnf (the one in test-runs/test_fipsinstall) wasn't written for some reason, specifically on x86.  I assume that's a different "box" than the ia64 one.
As for the command that @mattcaswell said you should run, that was a bit incomplete.  This is closer to being correct:
( cd test-runs/test_fipsinstall; ../../util/wrap.pl ../../apps/openssl fipsinstall -out fips.cnf -module ../../providers/fips.so -provider_name fips -mac_name HMAC -macopt 'digest:SHA256' -macopt 'hexkey:f4556650ac31d35461610bac4ed81b1a181b2d8a43ea2854cbae22ca74560813' -section_name fips_sect )

Hint: to debug a program run like this, I do it as follows:
( cd test-runs/test_fipsinstall; ../../util/wrap.pl gdb ../../apps/openssl fipsinstall --args  ../../apps/openssl fipsinstall -out fips.cnf -module ../../providers/fips.so -provider_name fips -mac_name HMAC -macopt 'digest:SHA256' -macopt 'hexkey:f4556650ac31d35461610bac4ed81b1a181b2d8a43ea2854cbae22ca74560813' -section_name fips_sect )

Yes, this is not on the ia64 platform, which does not support FIPS.

Yes, this is not on the ia64 platform, which does not support FIPS.

Which is a lot better, but still has a failure (SSHKDF : (KAT_KDF) : Fail).

Ah, I wonder why that happens...  this is not my area, unfortunately...

Which is a lot better, but still has a failure (SSHKDF : (KAT_KDF) : Fail).

Ah, I wonder why that happens...  this is not my area, unfortunately...

My guess would be that this is related to the char* cast of the integer that has been raised recently in #13781

My guess would be that this is related to the char* cast of the integer that has been raised recently in #13781

My guess would be that this is related to the char* cast of the integer that has been raised recently in #13781

Very possible. Our x86 is big-endian also.

My guess would be that this is related to the char* cast of the integer that has been raised recently in #13781

Very possible. Our x86 is big-endian also.

My guess would be that this is related to the char* cast of the integer that has been raised recently in #13781

Very possible. Our x86 is big-endian also.

Big-endian x86? What an awkward platform!

My guess would be that this is related to the char* cast of the integer that has been raised recently in #13781

Very possible. Our x86 is big-endian also.

Big-endian x86? What an awkward platform!

My guess would be that this is related to the char* cast of the integer that has been raised recently in #13781

Very possible. Our x86 is big-endian also.

Big-endian x86? What an awkward platform!

You have no idea how awkward ;) It is done in custom millicode. The reason is because the platform has been Big-endian since 1978, through many processor changes, and data compatibility and interoperability with other processors has to be ensured.

My guess would be that this is related to the char* cast of the integer that has been raised recently in #13781

Very possible. Our x86 is big-endian also.

Big-endian x86? What an awkward platform!

You have no idea how awkward ;) It is done in custom millicode.  Thanks for everyone's help.

Builds and tests correctly at commit e947a06. Thanks for everyone's help.


Hot module replacement reloads entire hierarchy

When I update a file in the component tree, I expected it to only update that one piece and its children, but instead the root component reloads (and everything else on the page). For example, if I change only the style in src/routes/Home/components/HomeView.scss, I see this in my console:
[HMR] bundle rebuilding
client.js?e526:126 [HMR] bundle rebuilt in 1477ms
process-update.js:27 [HMR] Checking for updates on the server...
bootstrap 78a2d30…:48 XHR finished loading: GET "http://<IP_ADDRESS>:3000/a11895716c62a5071276.hot-update.json".hotDownloadManifest @ bootstrap 78a2d30…:48hotCheck @ bootstrap 78a2d30…:267check @ process-update.js:64module.exports @ process-update.js:28processMessage @ client.js?e526:139handleMessage @ client.js?e526:65
process-update.js:100[HMR] Updated modules:
process-update.js:102[HMR]  - ./~/css-loader?sourceMap&-minimize&modules&importLoaders=1&localIdentName=[name]__[local]___[hash:base64:5]!./~/postcss-loader!./~/sass-loader?sourceMap!./src/routes/Home/components/HomeView.scss
process-update.js:102[HMR]  - ./src/routes/Home/components/HomeView.scss
process-update.js:102[HMR]  - ./src/routes/Home/components/HomeView.js
process-update.js:102[HMR]  - ./src/routes/Home/index.js
process-update.js:102[HMR]  - ./src/routes/index.js
process-update.js:107[HMR] App is up to date.

I have used other webpack setups where only the pieces that change are updated, and not their parents, but I can't determine what is different about this configuration. Is this a bug in the setup or expected behaviour? What can I do to get only components downstream to update?

It's the expected behavior for now until we finish integration v3 of react-hot-loader. The previous recommended implementation, using react-transform with react-transform-hmr, was deemed a less-than-desirable approach by the community and project maintainers, where they opted for more "vanilla" use of webpack HMR. If you want the type of hot reloading you mentioned, you can totally use this project's v2 release since it still uses that workflow.
See: https://github.com/davezuko/react-redux-starter-kit/releases/tag/v2.0.0

Thanks for the super fast reply and detailed answer!

No problem, I just happened to check my email at the right time :). Hopefully we'll get this fixed sooner than later, I've just been pulled a bit away from the project for the past couple of weeks.


Add missing image sobel_ops.png

Add missing image convolutional-neural-networks/conv-visualization/images/sobel_ops.png

Thank you! That image has been added now :)


add possibility to insert additional env

add a possibility to add additional env vars
closes #83

I'd be pleased to see that PR merged too.
I've forked @MickaelCa solution to be able to deploy the helm chart.
You can use https://alexandre-abrioux.github.io/wg-access-server, with version 0.4.7, while waiting for the merge.


Replace C-SPIDER with py-SPIDER

The C-based version of SPIDER is being deprecated. There is a new version written in Python being developed, which uses SciPy rather than PETSc and thereby simplifies things significantly.
It will therefore be necessary to implement pySPIDER within PROTEUS. Implementation of Dan's new atmosphere/outgassing/EOS code will probably also be necessary.

Once this has been done we can probably think about re-working the installation method for the framework. At the moment it's very clunky and will make it hard for others to reproduce our results.
Also, this issue makes Issue #43 redundant.

https://github.com/ExPlanetology/aragog/issues/6
https://github.com/ExPlanetology/aragog/issues/7
These two issues are relevant for this. In particular https://github.com/ExPlanetology/aragog/issues/6 needs to be resolved to enable physically meaningful simulations with aragog.

We plan to organise a Zoom meeting with @djbower, myself, @planetmariana, and @planetmariana in ~late Aug/early Sep to start this item.

Issue #126 should probably be addressed first.

Hi @planetmariana @timlichtenberg ,
I have setup a first draft of the aragog interface in this branch, see src/proteus/interior/aragog.py. For now it runs from some default options and the output solution is not put.
Now I need your help to:

correctly fill the initial input parameter from proteus options in SetupAragogSolver
update the boundary conditions in UpdateAragogSolver
collect the output from solver solution in GetAragogOutput

Maybe you can have a look by next Monday and we go though all this together during our meeting.

I'd highly recommend setting this up as a class instead of a global:
class AragogSolver:
    def setup(...):
    def update(...):
    def run(...):
    def get_output(...):

Shouldn't take more than 5 min to implement this. This will make it much easier to refactor/update the code at a later stage, and also keep the code decoupled for parallelization.

Hi @planetmariana and @timlichtenberg ,
I have setup two inputs for testing aragog:

dummy_aragog.toml which takes the dummy example but replaces the dummy interior by aragog
default_aragog.toml which takes the default example but replaces spider by aragog

The first one runs till the end (atmospheric escape) but cooling seems fast. The second crashes after 5 iteration with a magma temperature of -8000K. Any thoughts to debug this ?
To run these examples you need to work with the aragog-constant proteus branch and with the add-output aragog branch.

I have never actually tried SPIDER+dummy atmosphere, which roughly corresponds to the first case you describe. However, with the dummy atmosphere we should expect quite fast cooling since it's effectively just a bare rock. Do the plots show the same behaviour as with the other models?
I'll give this a go if I have time today.

And on the second one: I would suggest investigating where the -8000 K comes from. Is it from the atmosphere or the interior? Does it happen with SPIDER/dummy interior too?

I have been able to run the dummy_aragog.toml example (although it actually had interior.module="dummy", but I changed this locally) and can see that yes it does cool very fast. Strange.

Also I guess this is something that needs to be looked at later: but Aragog does not like it when you try to resume the PROTEUS simulation from a previous point. I think it's getting semi-reset to the beginning?

Also I guess this is something that needs to be looked at later: but Aragog does not like it when you try to resume the PROTEUS simulation from the disk. I think it's getting semi-reset to the beginning?

The resume feature is not supported yet.

Yes, that makes sense to me.


[SR-15916] Not implemented: @MainActor-isolated default arguments for @MainActor-isolated functions

Previous ID
SR-15916

Radar
None

Original Reporter
@groue

Type
Bug

Environment
Xcode 13.3 beta 3 (13E5104i)

Additional Detail from JIRA

Votes
2

Component/s
Compiler

Labels
Bug

Assignee
None

Priority
Medium

md5: 618b69843e92ff0edf9587c87894767b

Issue Description:
Hello,
The following snippet does not compile:
@MainActor class Foo { }

// error: call to main actor-isolated initializer 'init()' in a synchronous nonisolated context
// @MainActor func f(foo: Foo = Foo()) { }
//                               ^
// note: calls to initializer 'init()' from outside of its actor context are implicitly asynchronous
// @MainActor class Foo {
//                  ^
@MainActor func f(foo: Foo = Foo()) { }

It should compile, since everything in the declaration of the f function is MainActor-isolated.

Such an annoying bug. Are there any rough plans for when it will be fixed? @ktoso @DougGregor

@AnthonyLatsis are there plans to fix this bug in Swift 5.9?

I have a feeling fixing this might warrant a formal evolution proposal that outlines the semantics and restrictions for public approval, so I suppose not, but we should really ask the concurrency experts (@ktoso or @DougGregor ping).

Might need a mini proposal, let me track it with a radar. rdar://112171197
@hborla FYI

I have a draft implementation of this at https://github.com/apple/swift/pull/68794. I'm also working on a proposal to detail the semantics. I'm thinking we want similar rules for default value expressions of (global-)actor-isolated instance properties.

Here's the pitch thread, in case anybody would like to participate in the discussion: https://forums.swift.org/t/pitch-isolated-default-value-expressions/67714
I'm also going to reopen this - it was automatically closed when I merged my implementation PR, but the feature is behind an experimental feature flag.

The given example compiles under -strict-concurrency=complete in Swift 5.10. This is resolved by https://github.com/apple/swift/pull/70839


Matterbridge Hangs on Backlog of Requests - Telegram To Discord

Describe the bug
I have a telegram to Discord gateway setup and if this channel gets really busy (10k+ users) chatting, sometimes the gateway gets backed up and if that is the case, once the gateway gets backup, it takes multiple restarts of the gateway to get it caught back up.  Otherwise it just hangs.
To Reproduce
Steps to reproduce the behavior:
Setup a telegram to Discord gateway
I have three channels setup with lots of chatter
Expected behavior
A clear and concise description of what you expected to happen.
The gateway should handle messages properly
Environment (please complete the following information):

OS: linux
Matterbridge version:  1.22.1 b979aff2

Additional context
Setup three channels with lots of chatter going from TG to Discord
Workaround: setup a script to reboot the service every night

Discord ratelimits you, so matterbridge has to wait for the ratelimit to end before it can send new messages.
If you add debuglevel=1 to your [discord.yourdiscord] config and run matterbridge --debug you can see every message being sent to discord with the ratelimit limits + waiting time.
Warning this gives you a lot of output and also outputs your tokens.

If this were the case though, why does immediately stopping and restarting the app work?
So if I start the bridge, it starts to catch up on backlog and then stops, then I stop the program and immediately start it, then it continues on the backlog, and I keep cycling like this until backlog cleared
if I don't do the above, the process hangs and never catches up

Did you do the discord debug as I proposed above?

I did not this, resolves the issue as a workaround -- in the docker file I force a restart every 15 mins or so
&& printf "*/15 * * * * reboot now >/dev/null 2>&1" > /etc/crontabs/matterbridge
&& chmod 0644 /etc/crontabs/matterbridge
&& crontab /etc/crontabs/matterbridge
ENTRYPOINT ["/bin/sh", "-c", "crond && /bin/matterbridge -conf /etc/matterbridge/matterbridge.toml"]


Temporary fix: specify commit id to get a working config

Fixes #14 and #16
Fixes various issues before release of benchcab.
Add commit id and comment so we remember to remove it once the issue is fixed.
Move the experiment keyword.

Integration tests with latest version of benchcab (9482458) is working


Update Set-SPEnterpriseSearchCrawlLogReadPermission.md

Formatted example.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/Set-SPEnterpriseSearchCrawlLogReadPermission.md
:white_check_mark:Succeeded
View (sharepoint-ps)

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


Remove GPU readback from NoiseTexture3D.get_format()

I caught this issue while reviewing another PR. When accessing the format of the NoiseTexture3D it was requesting a GPU->CPU copy of the texture data, then throwing away the texture data and only using the format. This is not only an expensive operation, it also causes a full GPU->CPU sync making it very expensive to call at runtime.
This is a very low risk bug fix
Bugsquad edit:

Fixes #79360
Fixes #80302

Cherry-picked for 4.1.2.


HHH-9340 - Streams API for query result processing.

This was great and it works efficiently, but trying to stream Tuples gives
java.lang.ClassCastException: [Ljava.lang.Object; cannot be cast to javax.persistence.Tuple

I opened https://hibernate.atlassian.net/browse/HHH-11743 with a test case.


Add callbacks checks to unused ignored params

Related to https://github.com/AdRoll/rebar3_hank/issues/8
Avoid ignoring files which implement behaviours but check their internal functions are not callbacks.

Some ideas…
Handling known behaviors
Instead of just checking if the file has a behaviour attribute…

collect all the behaviors that the file implements.
for each one of them evaluate Behavior:behaviour_info(callbacks) and collect all the resulting tuples in a single list.
if that call fails, ignore the whole module (because we can't be sure which functions are callbacks).
If that call works, only ignore the functions in the list.

Handling all behaviors
Just like the one above but, for each of the modules that fail in 2…

Check the list of files to analyze in search for the module.
If the module is not there, ignore the whole file (we can't be sure which functions are callbacks).
If the module is there…
3.1. collect all callback attributes.
3.2. analyze them with erl_syntax_lib.
3.3. turn them into {Name, Arity} tuples.
3.4. add them to the list obtained in the item 2 and keep moving from there.


Wire up caching client to components that utilize Releases

Towards https://github.com/giantswarm/giantswarm/issues/17506


Сontinue training

Hello,
what is the best option to continue training (after default 400k iters, for example)? And would it be useful at all? (in cases where trained model  does not provide any noticeable mouth movement).

up this issue hoping to get answer

Hi, I think continue training will not help if the model does not provide any noticeable mouth movement. It may be the problem of initialization, did you try to retrain the model, or did you check that the training video was properly processed (the video should be 25fps and should be synchronized with the audio).


Support for group attributes of registry needs to be enhanced in dubbo-ops

Support for group attributes of registry needs to be enhanced in dubbo-ops.
many users use the registry group for environment isolation, However dubbo-ops seems unable to switch interface information under different groups

Sorry, because this issue was raised very early, we have closed it temporarily. You can verify whether this problem still exists in the latest code of the develop branch. If this problem still exists, please reopen it at any time.


Various CMake cleanups and modernisations.

Thanks! I’ll give this a try with our Android and Windows toolchains (probably the second week of January after the holidays).
Looks like the Windows builds are still failing though (and we might also want to upgrade the CI to VS 2019 or 2022 according to the warning).

Talking to @weliveindetail today about extending libobjc2 tests (e.g. to cover additional EH edge cases that still have issues on Windows), we thought it would be helpful if the test suite also ran on Mac (excluding tests that cover libobjc2-specific behavior), in order to be able to compare and match behavior between the runtimes as much as possible. Do you think this would make sense @davidchisnall?
I’m adding this here, since this patch using native CMake ObjC support would probably be somewhat of a prerequisite for running the tests on Mac.

That was always one of my goals.  I used to run the tests manually on macOS.

Can you share how you ran them manually on macOS?

We’ve been looking into setting up CMake for our own projects and on Windows are running into the same CMake error mentioned above using CMake 3.24.2:
`CMAKE_OBJC_STANDARD_COMPUTED_DEFAULT` should be set for Clang

After looking at the source of the error message in CMake, I was able to work around this as follows:
foreach(lang IN ITEMS OBJC OBJCXX)
    set(CMAKE_${lang}_COMPILER_FORCED ON)
endforeach()

After this I got another error:
MSVC_RUNTIME_LIBRARY value 'MultiThreadedDebugDLL' not known for this OBJC compiler.

This can be worked around by resetting the corresponding runtime library option to an empty string (note that this must be done before the project() call, or before enable_language(OBJC)):
foreach(lang IN ITEMS OBJC OBJCXX)
    set(CMAKE_${lang}_COMPILER_FORCED ON)

    foreach(runtimeLibrary IN ITEMS MultiThreaded MultiThreadedDLL MultiThreadedDebug MultiThreadedDebugDLL)
        set(CMAKE_${lang}_COMPILE_OPTIONS_MSVC_RUNTIME_LIBRARY_${runtimeLibrary} "")
    endforeach()
endforeach()

While these workarounds seems to work fine, I was wondering if you might have any idea about what is missing for CMake on Windows to handle this correctly?

It might be worth opening bugs with CMake.  I suspect that they'd be willing to support more Objective-C platforms but without anyone on the CMake project using them they might not know that things are broken.

Makes sense, I’ll do that for the Windows error.
However I’m not sure about the second issue I wrote about, is it even possible to fix this in CMake? If there’s no runtime built yet, CMake couldn’t possible verify the ObjC compiler is working I think?

However I’m not sure about the second issue I wrote about, is it even possible to fix this in CMake? If there’s no runtime built yet, CMake couldn’t possible verify the ObjC compiler is working I think?

I'm not sure.  This was the problem that I had with depending on GNUstep Make originally: it expected that Objective-C projects would have a working Objective-C stack.  For libcxxrt, I use a CMake flag that tells it to link as C, which avoids it adding the C++ libraries, but I don't know if that approach works for Objective-C.

While these workarounds seems to work fine

I haven't been able to reproduce this, the Windows build is still failing for me.  Can you push something to this branch with your fixes?

I’ve fixed the compiler detection on Windows. The COMPILER_FORCED option must be set before CMake configures the compiler, which happens as part of the project() call if the language is listed. So to work around this we need to remove OBJC(XX) from the project() call and then enable the language after forcing the compiler like so (setting these options before project() also didn’t work for me):
project(libobjc C ASM CXX)

# fix up CMake Objective-C compiler detection on Windows before enabling languages below
if (WIN32)
	foreach(lang IN ITEMS C CXX)
		set(CMAKE_OBJ${lang}_COMPILER_FORCED ON)
		foreach(runtimeLibrary IN ITEMS MultiThreaded MultiThreadedDLL MultiThreadedDebug MultiThreadedDebugDLL)
			set(CMAKE_OBJ${lang}_COMPILE_OPTIONS_MSVC_RUNTIME_LIBRARY_${runtimeLibrary} ${CMAKE_${lang}_COMPILE_OPTIONS_MSVC_RUNTIME_LIBRARY_${runtimeLibrary}})
		endforeach()
	endforeach()
endif()

enable_language(OBJC OBJCXX)

However there’s still a build error, I’m guessing because we need to add -fobjc-exceptions to CMAKE_OBJC(XX)_FLAGS?
associate.m(171,2): error: cannot use '@try' with Objective-C exceptions disabled

Can you push the fix to this branch?

Yes already done.

However there’s still a build error, I’m guessing because we need to add -fobjc-exceptions to CMAKE_OBJC(XX)_FLAGS?

I think so, I'm a bit surprised it isn't in the default CMake flags.  We already explicitly add -fexceptions.

CMake doesn’t seem to have any reference to -fobjc-exceptions. Is that part of the compiler defaults on other platforms maybe?
In any case, I added it in CMakeLists, but now a bunch of tests are failing and there are these compiler warnings:
clang-cl: warning: unknown argument ignored in clang-cl: '-x' [-Wunknown-argument]
clang-cl: warning: unknown argument ignored in clang-cl: '-std=gnu17' [-Wunknown-argument]
clang-cl: warning: objective-c: 'linker' input unused [-Wunused-command-line-argument]

If I build with clang.exe instead of clang-cl.exe (locally) I don’t get these warnings and all tests pass. Any idea what else we might need to do to make clang-cl happy?

Just a note that you want to add -fobjc-arc-exceptions as well (if it's not added implicitly). See details here: https://reviews.llvm.org/D137939?id=475122#inline-1333144

Not all of the Objective-C things are supported by default in clang-cl, because it aims for compatibility with cl.exe (which doesn't support Objective-C).  You can pass clang arguments to the clang front end through the clang-cl driver with -Xclang -{argument}.

The reason that I was using clang-cl, not clang, originally was that CMake conflated using-the-gcc-like-front-end with using-a-unix-compat-layer in a bunch of places.  I think this was fixed around 3.19, so it's possible that we can just use clang.exe (which natively supports all of the Objective-C flags) instead now?

Yes that would be the easiest solution to just switch the CI to use clang instead of clang-cl – I’ve just pushed a change for that.
I also tried to compare clang-cl invocations between the master branch and this, and one difference is that on master the source file is passed as -c -- <source.m>, whereas on this branch the -- is missing. I’m not sure what this does but maybe this explains the 'linker' input unused warning?

If we're not using clang-cl, we may not need the horrible CCC_OVERRIDE_OPTIONS hack.  This uses a debug feature to rewrite the arguments because CMake was adding the cl.exe flags for C or C++ to the command line in a way that could not be changed.
The -- normally just indicates the end of flags, so won't treat files with flag-like names after it as flags.  It shouldn't make a difference here.

If you add -v to the ninja invocation in CI, it will show the complete command for each build step, which might help debugging.

Should we still remove CCC_OVERRIDE_OPTIONS in main.yml and CMakeLists? Additionally we could output a warning if clang-cl is used.

Yup, I think both of those are good ideas.

I’ve done these updates.
Only downside I see with not supporting clang-cl is that I think it’s no longer possible to build with Visual Studio, as that doesn’t seem to support using clang according to this. Do you see an issue with that? If not I’ll update README.windows accordingly.

Just saw there was a test failure in the most recent run, but I don’t think it’s related to my changes:
88/90 Test #29: ManyManySelectors ........................Exit code 0xc0000409
***Exception:   1.59 sec
'345067selector543eb' != '345066selector543ea'
Assertion failed: strcmp(selBuffer, sel_getName(nextSel)) == 0, file D:/a/libobjc2/libobjc2/Test/ManyManySelectors.m, line 38

That test seems to fail reliably on some of the QEMU things.  I spent some time a couple of weeks ago trying to debug it.  I strongly suspect a bug in my hash table code, so I plan on moving the selector table over to the tsl one soon to see if that fixes it.

Only downside I see with not supporting clang-cl is that I think it’s no longer possible to build with Visual Studio, as that doesn’t seem to support using clang according to this. Do you see an issue with that? If not I’ll update README.windows accordingly

That's slightly unfortunate.  I did debug the tests in Visual Studio a few times, though I eventually moved to WinDbg (time travel debugging is amazing for the test cases: I can run them until they crash and then set a watchpoint on the location of memory corruption and rewind them until I get to the point where the corruption happened).  It sounds as if VS can drive CMake  + Ninja, so that's probably fine.

Just FYI I believe even with these workarounds we put in place for Windows it still requires a newer CMake version than the current 3.16 minimum. I tried building with 3.20 and got the following error on Windows:
CMake Error at CMakeLists.txt:210 (add_library):
OBJC_STANDARD is set to invalid value ''

With CMake 3.24 it works, but I’m not sure which exact release fixes the issue. I just wanted to leave this here in case anyone else runs into this.


Problems with coords

I am having trouble with the coordinates extracted from a trinagulation.
@fverdugo can you take a look at FESpaceTests l.42
D=2
nparts1d = 2
nparts = nparts1d*ones(Int64,D)
nparts_t = tuple(nparts...)
grid = CartesianGrid(partition=nparts_t,domain=(0,1,0,1),order=1) # domain, and order are optional
trian = triangulation(grid)
coords = cellcoordinates(trian)
size(coords)
IndexStyle(coords)
coords.lid_to_gid
coords.gid_to_val
coords.cv
length(coords)
isa(coords, AbstractArray)

provide reasonable values. On the other hand, coords is wrong, it only iterates till cell 2 for a 2x2 trian. Weird behaviour, it is a sub-type of AbstractArray, I can get all indices OK, but when I iterate, it does not work... The iterate for a AbstractArray is in Julia...

It is a problem related with Cartesian indexing... The commit 777caff97 should fix it. Tell me if it works.

Working!


[Requirement] Masking tools

Executive summary
Provide masks for different physical quantities: wavelength, 2 theta, intensity and detectors
Context and background knowledge
These masks are first to be used in the Powder Diffraction workflow for DREAM.
Different physical parameters could be used in order to mask:

Faulty detectors
Contribution from SE or instrument component in the detected signal
Debye-Scherrer rings
Too-high, too low or uneven distribution of intensity

Follow-up:

adapt implementation with DREAM NeXus file
use widget to determine masks (optional and for some physical quantities only)

Inputs

For masks in 2 theta, wavelength: DataArray containing the quantity to be masked as coordinate

For masks of detectors: metadata about detectors should be stored (positions, other characteristics like sumo, segment, sector…)

Methodology

For the detectors, the masks could be specified either by the spatial coordinates of the voxels to mask or by sub-elements (i.e., sumo, sector, wire, strip). It could be a set like bank (hr, sans, endcap fwd, endcap bwd, mantle), sector (for SANS and HR) or sumo (for endcaps), module, segment, wire(s) and or strip(s), counter

Outputs
Data array with the same structure as input and with added mask.
Which interfaces are required?
Integrated into reduction workflow, Python module / function
Test cases

Mask in two theta, intensity: data from D16 @ ILL link1 or DREAM csv files below
Detector mask, wavelength: DREAM csv data  (any csv files in link2)

Examples:

mask in 2 theta

mask of pixels (2d detectors for D16 @ ILL)

Comments
Note: The file from D16 can be used in the online documentation.

So masking should be part of a workflow.
Would you expect the masks to be different for each run/sample during an experiment or can you use a mask that was saved to a file?
I'm trying to figure out how this would fit in a pipeline?
Would you have a set of parameters at the top of your notebook like a list of wavelength or two_theta ranges that would defined what to mask, or do you need to have an interactive tool where you can draw masks?
I guess maybe a mixture of the 2?
The former could be something like (API to be discussed):
params[TwothetaMasks] = [sc.array(dims=['two_theta'], values=[53, 55], units='deg),
                         sc.array(dims=['two_theta'], values=[80, 84], units='deg)]

params[WavelengthMasks] = [sc.array(dims=['wavelength'], values=[3.2, 3.7], units='angstrom)]

An interactive tool could be used to save masks to file, and then we could load them like we do with the SANS workflows (we don't have to use xml format, we can use something else).

Another question: the lines in the two_theta plots are curved. Will there be some curved lines that we also want to mask or will the unwanted signal not be curved?
If so, is it possible to convert the data into a coordinate (maybe d-spacing?) where the lines are straight? If we can delay the masking until then, it will be much easier to accurately mask features either drawing rectangles or using a range of values as in the example code snippet above.

About the 2nd question: In principle, the ranges of two theta to mask could be linear or curved. For this first implementation, we could focus on linear masking only and split this requirement into 2 parts with the mask in 2theta to be implemented later. Let me know what you think.
About the 1st question, "Would you expect the masks to be different for each run/sample during an experiment or can you use a mask that was saved to a file?"
It is a mixture of both (as you guessed):

for example with the polariser in place, masking in wavelength will have to be applied to all runs corresponding to this setting (cycle, several beamtimes...)
masking in 2theta could be different from sample to sample (for example, different textures)
for the detectors, some masking should be in the NeXus files if issues with some detectors are picked up by NICOS. But additional masking should be implemented during reduction.
masking in intensity could be detector- or sample-dependent

@nvaytet Your plan is good. Here are a few comments
The conversion to $\lambda$ and $2\theta$ should be added to a more complete version of the workflow anyway. Their presence of a specific mask also depends on the type of experiments: for example, powder diffraction with known texture, parasitic scattering by some SEs….
These masks will also be essential tools for cold and hot commissioning of the instrument.
For the workflow of POWGEN

Masking pixels could be also done at an earlier stapge in the reduction. For example, after loading if some counts are too high.
OK
OK
OK for the POWGEN workflow. But for a more general approach, can’t we have both inputs?

A mask in d-spacing would also be required. But it can be left out of this first implementation after having tested the other types of masks.

Thanks for the reply.
Another question: in one of your images, the mask in 2theta is a vertical band that spans all pixels. Could we apply masks on a 1d plot of intensity vs 2theta? (i.e. making a 1d mask)

Alternatively, is the 'spectrum' dim in those pictures a stand-in for another dim? E.g., phi (azimuthal angle) or y?

@nvaytet I think the 2D mask should be implemented since users will first look at 2D data and decide to create mask at this stage. Visualising data in 1D comes at a later stage in the pipeline after a bit of clean up.
@jl-wynen the vertical axis has been renamed as 'spectrum' when reshaping the initial NeXus file, so this axis could be associated to other physical quantities.

@celinedurniak when you find the time could you look at https://github.com/scipp/essdiffraction/pull/55 and https://github.com/scipp/scippneutron/pull/521 to see if those combined satisfy your needs?
Both are currently unreleased so you have to get essdiffraction and scippneutron from main, let me know if you need assistance with that.

@nvaytet I had a look at 2 notebooks masking_tools.ipynb and `` #55 and https://github.com/scipp/scippneutron/pull/521 are fine as first implementations.
Here are a few additional comments / questions:

An example showing how to remove one of the masks would be useful because it is likely that masks won't be set correctly at the first try.
When setting the masks, are the input values always in the same unit as the displayed parameters (i.e., x, y for the example in the notebook)?
I do not have any interactivity with the masking tools of the plot containing the masks (widgets at the top). I can click on the buttons but nothing happens
How can I change the color of the masks individually?


Adds mass edit control to nodebalancers

closes #1411

Adds mass edit control to nodebalancers
simplifies and de-dupes toggle action/reducer logic

Coverage increased (+0.0008%) to 83.448% when pulling 20f9ccf63fd15c1cd7cc1b2c543d30c98344e2ed on mparke:nodebalancer-mass-edit into 5c4bc606c43230b7f4f6d44e2cb6235512e1acc7 on linode:master.

Coverage decreased (-0.4%) to 82.836% when pulling 551bf0fb3e8ad4d6cf965034fdb7ee182d4b90d2 on mparke:nodebalancer-mass-edit into 8aceade23c6f547aceafedaca6382969f5a701a5 on linode:master.

Coverage decreased (-0.0008%) to 83.216% when pulling 551bf0fb3e8ad4d6cf965034fdb7ee182d4b90d2 on mparke:nodebalancer-mass-edit into 8aceade23c6f547aceafedaca6382969f5a701a5 on linode:master.

Coverage decreased (-0.0008%) to 83.216% when pulling cb2094409b07086ce31e59d6c68d01baffd21871 on mparke:nodebalancer-mass-edit into 8aceade23c6f547aceafedaca6382969f5a701a5 on linode:master.


Decription of two parameters need update

Hi all,
the parameter -SqlInstanceName is mentioned not to be used in case the instance is the default for SQL.
I would like to have the "Don't use..." changed into "... must not be used ..." as the the script fails in case of default instancename.
Or in addition to what is written there:  "Don't ..... or the script will fail"
But best would be that the script checks within if the given parameter is the default instance name and would ignore it in combination with the message that it is ignored due to this reason
The parameter "IISWebSite" is mentioned to be used without - ???
From my tests it seems necessary to be used like this "-IISWebSite"
In addition i would like to have the info added here that the target from the parameter -IISWebsite needs to in front of the script is being executed as the script does not create the website on its own..
Thx
Marcus

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

ID: ef6976b2-abf4-87b4-9d94-3098c2636654
Version Independent ID: 966bc208-38c1-3527-6196-7ef5fbfa817d
Content: Set up BitLocker portals - Configuration Manager
Content Source: sccm/protect/deploy-use/bitlocker/setup-websites.md
Product: configuration-manager
Technology: configmgr-protect
GitHub Login: @aczechowski
Microsoft Alias: aaroncz

Hi @MarcusR9170, thanks for the feedback.

With Microsoft Style we generally try to avoid such strong language. As reference, see the style guide for should vs. must and Verbs.
I'll check with the engineering team to confirm the usage of that parameter.

Hi Aaron,
Thx for letting m know about “style guide” Was not aware about having something like this.
So what about adding the info that the script will fail in case of using the parameter with default information (if we can not change the script to avoid this at all)?
Thx
Marcus
From: Aaron Czechowski<EMAIL_ADDRESS>Sent: Donnerstag, 6. Februar 2020 02:09
To: MicrosoftDocs/SCCMdocs<EMAIL_ADDRESS>Cc: Marcus Ribka<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/SCCMdocs] Decription of two parameters need update (#2183)
Hi @MarcusR9170https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMarcusR9170&data=02|01|Marcus.Ribka%40microsoft.com|1a1c5f5406ec4b4cd9ac08d7aaa13532|72f988bf86f141af91ab2d7cd011db47|1|0|637165481684677600&sdata=ER92je3ZJ%2B9w7w%2BGmyfVNaMqWr2DvTQx%2FUWyX2t7qKU%3D&reserved=0, thanks for the feedback.

With Microsoft Style we generally try to avoid such strong language. As reference, see the style guide for should vs. musthttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fstyle-guide%2Fa-z-word-list-term-collections%2Fs%2Fshould-vs-must&data=02|01|Marcus.Ribka%40microsoft.com|1a1c5f5406ec4b4cd9ac08d7aaa13532|72f988bf86f141af91ab2d7cd011db47|1|0|637165481684687595&sdata=XVKIo2Hecxf9PnT3lSJMJFlDkhT%2FdTjubAi5fsmka5o%3D&reserved=0 and Verbshttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fstyle-guide%2Fgrammar%2Fverbs&data=02|01|Marcus.Ribka%40microsoft.com|1a1c5f5406ec4b4cd9ac08d7aaa13532|72f988bf86f141af91ab2d7cd011db47|1|0|637165481684697590&sdata=G3EYlNTro%2B9k9jS9B2YUVLnjWsOskYxKwPiwnI3QL%2FY%3D&reserved=0.
I'll check with the engineering team to confirm the usage of that parameter.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2FSCCMdocs%2Fissues%2F2183%3Femail_source%3Dnotifications%26email_token%3DAOJPKFREIJDI7P6C3E23563RBNPMJA5CNFSM4KQG67ZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEK5R3XA%23issuecomment-582688220&data=02|01|Marcus.Ribka%40microsoft.com|1a1c5f5406ec4b4cd9ac08d7aaa13532|72f988bf86f141af91ab2d7cd011db47|1|0|637165481684697590&sdata=rQk%2BoKXemAMfo5xlKAh1vhvhKfGG%2FBxpDfnhB2m9JOk%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAOJPKFX5S7SGTQMXLCOR4C3RBNPMJANCNFSM4KQG67ZA&data=02|01|Marcus.Ribka%40microsoft.com|1a1c5f5406ec4b4cd9ac08d7aaa13532|72f988bf86f141af91ab2d7cd011db47|1|0|637165481684707588&sdata=CY4FDpDo%2F%2BlfTLi5okt6xajqNS%2FcwL62DtsUslI1F1g%3D&reserved=0.

Hi @MarcusR9170, I suggest filing a frown from the ConfigMgr console to suggest a change to the script. I'm still trying to get an answer to the other parameter question.

I confirmed, dash is required in all parameters. I'm also going to add a note that you need to pre-create a custom website.

I submitted a change for this, but as part of the version 2002 current branch content, which won't publish for another few weeks. So you'll see this issue close soon when my changes merge into our release branch. But it won't be live until version 2002 early update ring release.


How to set rtl ?

I tried this code but it didn't work
$('#using_json').jstree(
{ 'core' :
{'rtl' : true,

rtl is read from the container. So if you have a RTL page it will be on automatically - no need to turn on anything. If you want to force it only on a single element - just add a style attribute to your container DIV and set directionality with inline CSS.


Review the data-flow - use of uninitialized value at booking/logininfo.php

Variables should be initialized before use.

Alright,later I will try to fix it

It is solved!


CI checks are incomplete

For instance, we discovered in https://github.com/gtk-rs/examples/pull/320 that a removed macro was still used in the examples. It shouldn't have been possible.

This should be impossible with the new CI in the combined repository.


UF-444: Avoid concurrent Freemarker setup (uberfire-preferences-processors)

@psiroky Could you please take a look? As mentioned in my email, it would be nice to test it in a Jenkins/release build, checking if the "Home" > "Admin" page is loading correctly.

There are Arquilian tests in kie-wb-distributions/kie-wb-tests/kie-wb-tests-gui that check various things including login, goto authoring/library, create project, build and deploy etc. You could write another test for preferences.

@paulovmr could we somehow refactor the code to avoid the duplication? I've noticed this particular piece of code was copy pasted into kie-wb-common as well: https://github.com/droolsjbpm/kie-wb-common/blob/master/kie-wb-common-stunner/kie-wb-common-stunner-core/kie-wb-common-stunner-processors/src/main/java/org/kie/workbench/common/stunner/core/processors/AbstractAdapterGenerator.java#L40
The issues is not that easily reproducible I am afraid. Once merged we can run couple of builds to see what happens. With forking enabled (https://github.com/droolsjbpm/droolsjbpm-build-bootstrap/commit/60ddedd7046f8ffdc9563c642a6e99bca85c509d) the issue is for now gone, so we would also need to disable the forking again. I am actually looking at that, but I am still not able to reproduce this easily on a local machine.

jenkins retest this

@manstis Test created and merged: https://github.com/droolsjbpm/kie-wb-distributions/pull/429
@psiroky Now that we are removing the shading from freemarker dependency, the refactor will be possible. I will close this PR and soon open another one with the refactor included.


Add logging methods for migration

Currently we are providing logger to user with only Info and Error functionalities.
Now, also added methods for Debug, Notice, Warn, Fatal

Checklist:

[x] I have formatted my code using  goimport  and  golangci-lint.
[x] All new code is covered by unit tests.
[x] This PR does not decrease the overall code coverage.
[x] I have reviewed the code comments and documentation for clarity.

Why does migration need fatal logging? @srijan-27

@vikash - Didn't had any specific use-case in mind. But since, these are the methods which we are providing user to log inside migrations, so just thought of having parity with the app logger.


Make inputFiles parameter default to all files

With this change, this is the new simplest possible concat call:
var appJs = concat('js', {
    outputFile: '/all-js.js',
});

This makes the plugin easier to get started with.
Closes https://github.com/ember-cli/broccoli-concat/issues/36.

@roryokane I have now, on multiple occasions noticed this and intended to fix it myself, but apparently I always forget. Thank you :)

How’s this change?

Fantastic. Thank you

Running without elevated rights. Running Ember CLI "as Administrator" increases performance significantly.
See www.ember-cli.com/user-guide/#windows for details.
Livereload server on http://localhost:49152
Serving on http://localhost:4200/
ENOENT: no such file or directory, stat 'C:\ember\project1\godKnows\tmp\concat_with_maps-input_base_path-vgzIADAx.tmp\0\bower_components\showdown\dist\showdown.js'
Error: ENOENT: no such file or directory, stat 'C:\ember\project1\godKnows\tmp\concat_with_maps-input_base_path-vgzIADAx.tmp\0\bower_components\showdown\dist\showdown.js'
at Error (native)
at Object.fs.statSync (fs.js:844:18)
at ConcatWithMaps.keyForFile (C:\ember\project1\godKnows\node_modules\broccoli-caching-writer\index.js:90:20)
at Array.map (native)
at ConcatWithMaps.CachingWriter._conditionalBuild (C:\ember\project1\godKnows\node_modules\broccoli-caching-writer\index.js:112:65)
at C:\ember\project1\godKnows\node_modules\broccoli-caching-writer\node_modules\broccoli-plugin\read_compat.js:61:34
at lib$rsvp$$internal$$tryCatch (C:\ember\project1\godKnows\node_modules\rsvp\dist\rsvp.js:1036:16)
at lib$rsvp$$internal$$invokeCallback (C:\ember\project1\godKnows\node_modules\rsvp\dist\rsvp.js:1048:17)
at lib$rsvp$$internal$$publish (C:\ember\project1\godKnows\node_modules\rsvp\dist\rsvp.js:1019:11)
at lib$rsvp$asap$$flush (C:\ember\project1\godKnows\node_modules\rsvp\dist\rsvp.js:1198:9)
@stefanpenner  help ,i mailed you if you remember


Fix-mixcoord-activeStandby

Make the coord's activestandby behavior consistent.
What this PR does / why we need it:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] Chart Version bumped
[ ] Variables are documented in the README.md
[ ] Title of the PR starts with chart name (e.g. [mychartname])
[ ] PR only contains changes for one chart

Fix #14


[Documentation Bug]: The table contents are not aligned.

Describe the bug ✍

Provide step by step information reproduce the bug 📄
Will update the main README.md
Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

/assign

Hey @deepshikha2708 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Hey @deepshikha2708 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Request for a discussion pannel

I guess Github now propose to have a discussion panel here. Or do you have any discord where we can chat?

Just opened up discussions tab for this repo. I am also available on discord as Ryuzaki#9952


Make closest function call live

Requested by jsdom code owner.

@dreki Oh sorry, I had it done since a few days, but somewhat didn't push it corrctly, I just did it https://github.com/jsdom/jsdom/pull/1951/files


Failed: Update autorest version to prerelease 3.0.0-alpha.20240518.3

Triggered from pull request: https://github.com/Azure/autorest.csharp/pull/4432

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.NotificationHubs


Randomized dungeon entrances

This would be a (relatively) easy way to add some variety. Since many dungeons are locked until you find a key in a haystack, while others have many ways to access them, it would be nice for the more readily accessible dungeons to change from run to run.

This is definitely one of the location randomization options I'd be warmest towards. I guess seasons D2's two side entrances would need to remain the same.
But now we start getting into my dreaded "option" territory, since randomizing dungeon entrances could make things a lot more inconvenient, and make seeds take longer. I'm thinking about ages D6 entrances being far away from each other, and the D8 entrances in both games taking a long time just to reach to find out what it is.

4.x could have this feature, since 4.x versions will be about options. I wouldn't be especially keen on playing randomized dungeon seeds, but it wouldn't be very difficult to implement either.

Alternate D2 entrances in seasons should just be gone in this game mode, and the staircases to the alternate entrances should be directly linked instead.

For bookkeeping purposes – this is implemented as of 5ab374e, and will be included in 4.0


[TON] Support TON tx compiler

Description
Implement the TON transaction compiler, fix https://github.com/trustwallet/wallet-core/issues/3797

How to test
Run Rust, C++, Kotlin, Swift tests

Types of changes

Checklist

[x] Create pull request as draft initially, unless its complete.
[x] Add tests to cover changes as needed.
[x] Update documentation as needed.
[x] If there is a related Issue, mention it in the description.

If you're adding a new blockchain

[ ] I have read the guidelines for adding a new blockchain.

Hi @satoshiotomakan, I would greatly appreciate it if you could take the time to review this pull request.

Hi @10gic, thank you for the PR! I'll review today

Good job again. Please consider making changes to avoid ambiguous API and doing same work multiple times

Thank you very much for your comments. I've adjusted the code according to your advice. Could you please review it again?

Hi @satoshiotomakan. I'm wondering if it's a good time to proceed with merging the code. Please let me know if there's anything else that needs to be done beforehand.

@10gic thank you a lot.


Configmap data given in values.yaml is not applied to configmap when Neuvector chart deployed from Rancher Manager.

Rancher Server Setup

Rancher version:  v2.6-9a779926802d0c017460c1b3330225674ed711d1-head
Installation option (Docker install/Helm Chart): Helm Chart

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): K8s v1.21.0

Proxy/Cert Details:

Information about the Cluster

Kubernetes version: v1.21.0
Cluster Type (Local/Downstream): Downstream 3 nodes + imported

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider):

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom)

If custom, define the set of permissions:  admin

Describe the bug
Configmap data added to Neuvector feature charts values.yaml is not saved to configmap when Neuvector feature chart 100.0.1+up2.2.2 is deployed from Rancher Manager. Configmap data is shown as null after deployment.
To Reproduce
1, Install Neuvector feature chart version 100.0.1+up2.2.2 from Rancher Manager UI->downstream cluster-> Apps&Marketplace >feature chart->Neuvector
2, Edit YAML
3, enable configmap under controller as below
4, check created configmap data
kubectl get cm neuvector-init -n cattle-neuvector-system
sample config
controller:
  affinity:
    podAntiAffinity:
      preferredDuringSchedulingIgnoredDuringExecution:
        - podAffinityTerm:
            labelSelector:
              matchExpressions:
                - key: app
                  operator: In
                  values:
                    - neuvector-controller-pod
            topologyKey: kubernetes.io/hostname
          weight: 100
  annotations: {}
  apisvc:
    annotations: {}
    route:
      enabled: false
      host: null
      termination: passthrough
      tls: null
    type: null
  azureFileShare:
    enabled: false
    secretName: null
    shareName: null
  certificate:
    keyFile: tls.key
    pemFile: tls.pem
    secret: null
  configmap:
    data:
      userinitcfg.yaml: |
        users:
        - EMail<EMAIL_ADDRESS>          Fullname: user1
          Locale: en
          Password: password
          Role: reader
          Role_Domains:
          admin:
            - admin1
            - admin2
          reader:
            - reader1
            - reader2
          Timeout: 300
        - Fullname: admin1
          Password: password
          Role: admin
    enabled: true

Result
apiVersion: v1
kind: ConfigMap
metadata:
  annotations:
    meta.helm.sh/release-name: neuvector
    meta.helm.sh/release-namespace: cattle-neuvector-system
  creationTimestamp: "2022-08-18T22:44:10Z"
  labels:
    app.kubernetes.io/managed-by: Helm
    chart: neuvector-100.0.1_up2.2.2
    heritage: Helm
    release: neuvector
  name: neuvector-init
  namespace: cattle-neuvector-system
  resourceVersion: "123446850"
  uid: 634049d0-6e2d-4f7c-956a-01f25118b6cb

Expected Result

neuvector@NV-QA-Ubuntu16-Node1:~/helm/core$ kk get cm  neuvector-init -oyaml
apiVersion: v1
data:
  userinitcfg.yaml: |
    users:
    - EMail<EMAIL_ADDRESS>      Fullname: user1
      Locale: en
      Password: password
      Role: reader
      Role_Domains:
      admin:
        - admin1
        - admin2
      reader:
        - reader1
        - reader2
      Timeout: 300
    - Fullname: admin1
      Password: password
      Role: admin
kind: ConfigMap
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"v1","data":{"userinitcfg.yaml":"users:\n- EMail<EMAIL_ADDRESS>Fullname: user1\n  Locale: en\n  Password: password\n  Role: reader\n  Role_Domains:\n  admin:\n    - admin1\n    - admin2\n  reader:\n    - reader1\n    - reader2\n  Timeout: 300\n- Fullname: admin1\n  Password: password\n  Role: admin\n"},"kind":"ConfigMap","metadata":{"annotations":{},"labels":{"chart":"core-1.9.1","heritage":"Helm","release":"neuvector"},"name":"neuvector-init","namespace":"neuvector"}}
  creationTimestamp: "2022-08-18T21:48:26Z"
  labels:
    chart: core-1.9.1
    heritage: Helm
    release: neuvector
  name: neuvector-init
  namespace: neuvector
  resourceVersion: "123435112"
  uid: 1ecf209c-4fde-4f17-bfe9-4d04a324a00d

Screenshots

Additional context

Reproduced the issue on v2.6.7 with NeuVector neuvector:100.0.1+up2.2.2

Install neuvector chart from apps & market place >> NeuVector chart
While installing the chart, from edit as yaml >> under controller >> configmap >> data >> add the following values:

  configmap:
    enabled: true
    data:
      userinitcfg.yaml: |
        users:
        - EMail<EMAIL_ADDRESS>          Password: password
          Role_Domains:
          admin:
            - admin1
            - admin2

In the request tab, the data is not parsed and shows null as a result the config map gets created with null data.

I checked with Selvam on Slack and it doesn't look like a release blocker for 2.7.0. But we will work on triaging it soon.

@PennyScissors Do you want to look at this, or assign it to someone else on the team?

@MKlimuszka Based on Anupama's comment, I think this is a UI issue since it happens before the request is made to the backend.


read the service config files from a directory

For operators to be able to pass in references to service configurations, in the config file we would like to have a reference to a directory where these configuration files live, and the broker on start reads the service configuration files in this directory as the set of services it offers.

Fixed in this commit: a0025c7cc38af54293338e5b217e06dc2e233227


Add Orange Unified School District

Add Orange Unified School District to the existing approved list.
Unfortunately, there's no way to pinpoint which email belongs to which schools or students or teachers, as they all share @ousd.com domain.

@huy-kunaico I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host orangeusd.ousd not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


BigtableIO StreamingWrite Setup and Teardown

R: @vikkyrk

Prediction: BigtableWriteIT will break because Dataflow service does not support this feature yet. This is probably okay.

Also looks like there new methods are not supported in DoFnTester? If you have any tests using it, then you probably need to explicitly call setup/teardown?

@vikkyrk good point -- file a JIRA?

@dhalperi filed https://issues.apache.org/jira/browse/BEAM-559. Assigned to you, redirect appropriately :)

LGTM, merged


ImageView requiring qualified function call for view()

Not sure why this is happening, shouldn't dispatch choose the proper method?
julia> view(img)
WARNING: both ImageView and Base export "view"; uses of it in module Main must be qualified
ERROR: UndefVarError: view not defined

I think the issue is that ImageView.view takes any AbstractArray, and that Base does as well.

ImageView doesn't currently import Base.view (it's only in 0.5+), so the two are in conflict. That's why you need to scope it. Improvements would be accepted though.

There's still a deprecation warning needed, but once we can do that this is fixed. It's now called imshow.
Closed by #107


Use alias instead of fset

fset is really difficult to revert in the event the user wishes to use longer answers (ie "yes"/"no") for confirm/negate prompts. This patch uses advice and includes logic from #47 making use of use-short-answers when it is available.

I'll wait until we conclude the discussion in #43 before I merge this.


Can com.google.firebase.app remove Google Play Service dependencies?

https://firebase.google.com/docs/android/android-play-services says all Android packages don't have dependencies on Google Play services.
However, I only add the com.google.firebase.app Unity package, it seems that com.google.android.gms:play-services-base Android package is resolved automatically. Can you remove the Google Play services dependencies from the com.google.firebase.app Unity package, and make Unity packages which really need them have the dependencies.

@milkcocoa play-services-base contains some utility classes, your app will still be able to run on devices without Google Play services installed.

I'm sorry for replying late, but I've just confirmed you're right, com.google.firebase.app doesn't have any effect on Google Play services.


Revert refuel (neo #2444)

https://github.com/neo-project/neo/pull/2560

Codecov Report

Merging #122 (813be43) into development (48eb623) will increase coverage by 0.05%.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           development     #122      +/-   ##
===============================================
+ Coverage        83.24%   83.30%   +0.05%
===============================================
  Files               87       87
  Lines             7945     7937       -8
===============================================
- Hits              6614     6612       -2
+ Misses            1331     1325       -6

Impacted Files
Coverage Δ

neo3/contracts/native/fungible.py
61.77% <ø> (+0.49%)
:arrow_up:

Continue to review full report at Codecov.

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


[F821] False positive: anext

Summary
The lint F821 has a false positive with anext
Example
async def foo(iter: AsyncGenerator[str, None]):
    assert await anext(iter)

Environment
❯ ruff --version
ruff 0.6.4

❯ python --version
Python 3.12.5

False positive on aiter as well.

Have you specified  requires-python in your pyproject.toml or a target-version in your ruff.toml?  I'm asking because anext is only available in Python 3.10 and newer (and the default is Python 3.8)

Good catch. I have a monorepo and the root project has
[project]
# ...
requires-python = "~=3.12.0"

[tool.ruff]
target-version = "py312"

and one of the sub-packages has
[project]
# ...
requires-python = "~=3.12.0"

[tool.ruff]
target-version = "py38"

I can confirm that changing from py38 to py312 fixes this.
In cases like there where there is a new symbol introduced in a more recent version of Python, should ruff instead suggest/direct the end-user to update the settings?

In cases like there where there is a new symbol introduced in a more recent version of Python, should ruff instead suggest/direct the end-user to update the settings?

That's a neat idea. I think the error message could possibly be improved by saying that the symbol is only defined in Python XX or newer.

Thanks @AlexWaygood !


Update task-list example

Update grid container class
Remove redundant clearfix div

https://github.com/alphagov/govuk-frontend/issues/814

Thanks :(

were there any styling tweaks to compensate for grid-row's -15px margin?


[Enhancement] Decrease dependence on regex

Fix for Use explanatory variables.
It will be better use named subpatterns.

Good hint. Thx :)


Update Backend Deps According to Dependabot

Update Backend Deps According to Dependabot
Problem
Many backend deps are out of date.
Solution
Merge dependabot updates into dependatest-backend branch and test.
Result
All branches merged with all tests passing.

Haha love the branch name


Merge 1.10.x

Proposed changes:

...

Status (please check what you already did):

[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

Fixed a related issue, #7605, associated with button_type that came up in testing.

@rgstephens Have you also finished testing the behaviour of telegram channel + kafka broker?


TEMP BRANCH: use application/x-www-form-urlencoded

This ensures a smooth rollout of https://github.com/livepeer/catalyst-api/pull/764

Never mind, let's upstream this for a while, we're not shipping the new Mist version for a bit


Relaxed httpx version requirement (#73).

Dependency of httpx set to be equal or greater that current and less that 1.
Tested in project where upstream repository could not be added due to dependency conflict.

+1

+1

Sorry for the delay in response and thank you for your contribution :)


Rise ignition events only if both positions has ignition attribute

As you suggested in #2832

Looks like test is failing now. Please fix.

Oh, sorry.
Unfortunately I do not have an idea how to make lastPosition changeable in tests. I would be thankful if you point me.
Just removed real check.

Merged, thanks.


[BUG] Connect-PnPOnline "GetContextAsync() called without an ACS token generator" error in -Interactive after -UseWebLogin

After running Connect-PnPOnline with the -UseWebLogin parameter, Connect-PnPOnline with the -Interactive parameter will return an error.
$siteUrl = "https://xxx.sharepoint.com/"
Connect-PnPOnline -Url $siteUrl -UseWebLogin
Connect-PnPOnline -Url $siteUrl -Interactive

Connect-PnPOnline : GetContextAsync() called without an ACS token generator. Use GetACSAppOnlyContext() or GetAccessTokenContext()
instead or specify in AuthenticationManager constructor the authentication parameters
At line:1 char:1
+ Connect-PnPOnline -Url https://xxx.sharepoint.com/  ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Connect-PnPOnline], ArgumentException
    + FullyQualifiedErrorId : System.ArgumentException,PnP.PowerShell.Commands.Base.ConnectOnline

Message          : GetContextAsync() called without an ACS token generator. Use GetACSAppOnlyContext() or GetAccessTokenContext()
                   instead or specify in AuthenticationManager constructor the authentication parameters
Stacktrace       :    at PnP.Framework.AuthenticationManager.<GetContextAsync>d__59.MoveNext()
                   --- End of stack trace from previous location where exception was thrown ---
                      at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
                      at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
                      at PnP.Framework.AuthenticationManager.GetContext(String siteUrl, CancellationToken cancellationToken)
                      at PnP.PowerShell.Commands.Base.PnPConnection.CreateWithInteractiveLogin(Uri uri, String clientId, String
                   tenantAdminUrl, Boolean launchBrowser, AzureEnvironment azureEnvironment, CancellationTokenSource
                   cancellationTokenSource, Boolean forceAuthentication, String tenant)
                      at PnP.PowerShell.Commands.Base.ConnectOnline.ConnectInteractive()
                      at PnP.PowerShell.Commands.Base.ConnectOnline.Connect(CancellationToken& cancellationToken)
                      at PnP.PowerShell.Commands.Base.ConnectOnline.ProcessRecord()
                      at System.Management.Automation.CommandProcessor.ProcessRecord()

What is the version of the Cmdlet module you are running?
1.10.0
Which operating system/environment are you running PnP PowerShell on?

[X] Windows

@fowl2 - Although I can reproduce when using -UseWebLogin first and then -Interactive (meaning there's no issue when doing the opposite), is there a particular reason why you want to connect with both one after the other?

I mean, you use the first, get a warning asking you to use the second instead and when you try it doesn't work - it's not a great impression.

Have you tried using -ForceAuthentication?

@fowl2 - This is not for use one after the other. It's one OR the other, and the warning explains why using -Interactive is the way forward.

Have you tried using -ForceAuthentication?

Can verify this does seem to be a working an alternative to restarting the whole PowerShell session.

@fowl2 - This is not for use one after the other. It's one OR the other, and the warning explains why using -Interactive is the way forward.

Hmm, maybe I've been unclear. If both -Interactive and -UseWebLogin are specified a different error - Parameter set cannot be resolved using the specified named parameters. - is produced as expected and points to the specific problem (if slightly obtusely, but this is the standard PowerShell message).
I understand that -Interactive is the recommended method going forward (although it is not universally better - it allows switching accounts more easily, but conversely adds an extra step when keeping the same account).
Consider the scenario where an existing user used to -UseWebLogin is updating a script to use -Interactive and is unable to due to this errorr.

First time you use -Interactive you have to make sure the PowerShell Module is registered via Register-PnPManagementShell, thus the use of UseWebLogin the first time. This is how I came across the bug and I resolved by Disconnect-PnPOnline then using Connect-PnPOnline -Interactive. In MHO this is a bug because unless I had seen this discussion I would have tried so many different things but the Disconnect-PnPOnline or -ForceAuthentication. Maybe just giving a better error message would suffice


⚠ Fortnite has degraded performance

In ee21b2b, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1637 ms

Resolved: Fortnite performance has improved in ee79933.


call contract view methods

How do I make a call to a view (read only) method of a contract?
I feel I have to use Provider::call_contract, but it require signature and max_fee as the two last arguments, which doesn't make sense when calling a view.
Should I still use call_contract with and empty Vec as signature and 0 as max_fee, or is there a better way to do this ?

It's not entirely true that signature and max_fee are meaningless for call_contract requests. Imagine trying to call a contract that reads the caller address, which forces your call to go through an account. And for your call to not get aborted, you need valid signatures. A similar issue exists when doing estimate_fee.
There has been an on-going discussion on the best approach to be taken here. But in any case that's up to StarkWare and the account contract implementors to decide. Not much can be done from our side.
And yes, you'll need to supply an empty Vec and zero max_fee if you're not going through an account contract.
Closing this optimistically. Please feel free to re-open if needed.

Thanks a lot for the explainations, I will do as advised !


Custom Error Types

Curious, how hard would it be to allow for this library to support custom error types? One of the biggest advantages of using a type for error handling vs. traditionally supporting exception based promise handling is that you can actually parse and typecheck your errors.
As it stands, the custom error type is fairly opinionated: requiring an error code, error context code (which must be integers), and only being flexible on the error info as a string.
What would be the most flexible way to support custom error types beyond a string? I took a stab at looking at how hard it would be templatize the error, but it seems like this would actually be a massive change across the board.

Hi Kevin, apologies for the delay in replying to this, I had missed the notification and only spotted this when I saw the other issue you raised.
Custom error types were not a goal of this library when it was written, and would likely require some significant changes, which would be complicated by trying to keep backwards compatibility with existing usage. It's likely not impossible, but not something we would have time to look into right now.
I'll leave this open as a reminder for it being something to look into though, and thanks for the suggestion!

Yea, totally makes sense. As an alternative I've considered converting all the int32 error codes into strings to be more flexible with external systems.
As a side note, is there any reason the current ErrorInfo string is stored within the Error object as a shared pointer? Is that simply an optimization? Thanks for the followups!

As a side note, is there any reason the current ErrorInfo string is stored within the Error object as a shared pointer? Is that simply an optimization?

I believe so, though it's unclear how useful that actually is, as I suspect SD::Error isn't copied very often.

Good to know. Thanks!


redis: handle percentage in key in redis key formatter

Commit Message:  handle percentage in key in redis key formatter
Additional Description: When the redis formatter has %KEY% and the key as '%' symbol, the substitution formatter interprets it as a command and fails with the exception "Incorrect configuration: {}. Couldn't find valid command at position {}". This PR handles that case.
Risk Level: Low
Testing: Updated
Docs Changes: N/A
Release Notes: N/A

/retest

Per comment above, assigning Matt.
/assign @mattklein123

@mattklein123 added release notes. PTAL.


Use DurableFunctionApp package

Description
References
Checklist

[ ] Should the change be behind a feature flag?
[ ] Can the feature be meaningfully disabled or circumvented if there are issues (e.g., database-breaking changes)?
[ ] Has it been considered whether data is being delivered to the wrong actor?
[ ] Subsystem test executed (dev_002/dev_003)
[ ] Is there time to monitor state of the release to Production?
[ ] Reference to the task

Implemented in another PR https://github.com/Energinet-DataHub/opengeh-edi/pull/1415


vcpkg integrate install not work

i know i'ts my problem, but i have no clue how to solve this "simple problem" can you pls help me thank you

well ,i know the answer.
i want to static link but by default vcpkg will use dynamic link version.
two ways to do this:
1.open the vcpkg attribute page and setting it to static link

2.modify vs tripple file.


fix: added transpileDependencies configuration option

@rmathis there appears to be a merge conflict that prevents me from merging, could you investigate?