CMake-based projects fail to build using Visual Studio 2019 with toolset v141

Originally this issue was the following: #6475, but because this affects many packages I thought it was better to open a new issue.
Environment
Windows 10
Visual Studio 2019 with v141 toolset installed.
Vcpkg version: 2019.06.26-nohash
The Problem
When trying to build osg with the following triplet:
set(VCPKG_TARGET_ARCHITECTURE x64)
set(VCPKG_CRT_LINKAGE dynamic)
set(VCPKG_LIBRARY_LINKAGE dynamic)
set(VCPKG_PLATFORM_TOOLSET v141)

many of its dependencies (and itself) all fail with the same error:
[1/2] cmd /c "cd .. && "C:/Program Files/CMake/bin/cmake.exe" "C:/vcpkg/buildtrees/szip/src/szip-2.1.1-6b767acfa7" "-DSZIP_INSTALL_DATA_DIR=share/szip/data" "-DSZIP_INSTALL_CMAKE_DIR=share/szip" "-DBUILD_SHARED_LIBS=ON" "-DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=C:/vcpkg/scripts/toolchains/windows.cmake" "-DVCPKG_TARGET_TRIPLET=x64-windows-v141" "-DVCPKG_SET_CHARSET_FLAG=ON" "-DVCPKG_PLATFORM_TOOLSET=v141" "-DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON" "-DCMAKE_INSTALL_SYSTEM_RUNTIME_LIBS_SKIP=TRUE" "-DCMAKE_VERBOSE_MAKEFILE=ON" "-DVCPKG_APPLOCAL_DEPS=OFF" "-DCMAKE_TOOLCHAIN_FILE=C:/vcpkg/scripts/buildsystems/vcpkg.cmake" "-DCMAKE_ERROR_ON_ABSOLUTE_INSTALL_DESTINATION=ON" "-DVCPKG_CXX_FLAGS=" "-DVCPKG_CXX_FLAGS_RELEASE=" "-DVCPKG_CXX_FLAGS_DEBUG=" "-DVCPKG_C_FLAGS=" "-DVCPKG_C_FLAGS_RELEASE=" "-DVCPKG_C_FLAGS_DEBUG=" "-DVCPKG_CRT_LINKAGE=dynamic" "-DVCPKG_LINKER_FLAGS=" "-DVCPKG_TARGET_ARCHITECTURE=x64" "-DCMAKE_INSTALL_LIBDIR:STRING=lib" "-DCMAKE_INSTALL_BINDIR:STRING=bin" "-G" "Visual Studio 15 2017 Win64" "-DCMAKE_BUILD_TYPE=Release" "-DCMAKE_INSTALL_PREFIX=C:/vcpkg/packages/szip_x64-windows-v141""
FAILED: ../CMakeCache.txt
cmd /c "cd .. && "C:/Program Files/CMake/bin/cmake.exe" "C:/vcpkg/buildtrees/szip/src/szip-2.1.1-6b767acfa7" "-DSZIP_INSTALL_DATA_DIR=share/szip/data" "-DSZIP_INSTALL_CMAKE_DIR=share/szip" "-DBUILD_SHARED_LIBS=ON" "-DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=C:/vcpkg/scripts/toolchains/windows.cmake" "-DVCPKG_TARGET_TRIPLET=x64-windows-v141" "-DVCPKG_SET_CHARSET_FLAG=ON" "-DVCPKG_PLATFORM_TOOLSET=v141" "-DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON" "-DCMAKE_INSTALL_SYSTEM_RUNTIME_LIBS_SKIP=TRUE" "-DCMAKE_VERBOSE_MAKEFILE=ON" "-DVCPKG_APPLOCAL_DEPS=OFF" "-DCMAKE_TOOLCHAIN_FILE=C:/vcpkg/scripts/buildsystems/vcpkg.cmake" "-DCMAKE_ERROR_ON_ABSOLUTE_INSTALL_DESTINATION=ON" "-DVCPKG_CXX_FLAGS=" "-DVCPKG_CXX_FLAGS_RELEASE=" "-DVCPKG_CXX_FLAGS_DEBUG=" "-DVCPKG_C_FLAGS=" "-DVCPKG_C_FLAGS_RELEASE=" "-DVCPKG_C_FLAGS_DEBUG=" "-DVCPKG_CRT_LINKAGE=dynamic" "-DVCPKG_LINKER_FLAGS=" "-DVCPKG_TARGET_ARCHITECTURE=x64" "-DCMAKE_INSTALL_LIBDIR:STRING=lib" "-DCMAKE_INSTALL_BINDIR:STRING=bin" "-G" "Visual Studio 15 2017 Win64" "-DCMAKE_BUILD_TYPE=Release" "-DCMAKE_INSTALL_PREFIX=C:/vcpkg/packages/szip_x64-windows-v141""
CMake Error at CMakeLists.txt:2 (PROJECT):
  Generator

    Visual Studio 15 2017 Win64

  could not find any instance of Visual Studio.

-- Configuring incomplete, errors occurred!
See also "C:/vcpkg/buildtrees/szip/x64-windows-v141-rel/CMakeFiles/CMakeOutput.log".
[2/2] cmd /c "cd ../../x64-windows-v141-dbg && "C:/Program Files/CMake/bin/cmake.exe" "C:/vcpkg/buildtrees/szip/src/szip-2.1.1-6b767acfa7" "-DSZIP_INSTALL_DATA_DIR=share/szip/data" "-DSZIP_INSTALL_CMAKE_DIR=share/szip" "-DBUILD_SHARED_LIBS=ON" "-DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=C:/vcpkg/scripts/toolchains/windows.cmake" "-DVCPKG_TARGET_TRIPLET=x64-windows-v141" "-DVCPKG_SET_CHARSET_FLAG=ON" "-DVCPKG_PLATFORM_TOOLSET=v141" "-DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON" "-DCMAKE_INSTALL_SYSTEM_RUNTIME_LIBS_SKIP=TRUE" "-DCMAKE_VERBOSE_MAKEFILE=ON" "-DVCPKG_APPLOCAL_DEPS=OFF" "-DCMAKE_TOOLCHAIN_FILE=C:/vcpkg/scripts/buildsystems/vcpkg.cmake" "-DCMAKE_ERROR_ON_ABSOLUTE_INSTALL_DESTINATION=ON" "-DVCPKG_CXX_FLAGS=" "-DVCPKG_CXX_FLAGS_RELEASE=" "-DVCPKG_CXX_FLAGS_DEBUG=" "-DVCPKG_C_FLAGS=" "-DVCPKG_C_FLAGS_RELEASE=" "-DVCPKG_C_FLAGS_DEBUG=" "-DVCPKG_CRT_LINKAGE=dynamic" "-DVCPKG_LINKER_FLAGS=" "-DVCPKG_TARGET_ARCHITECTURE=x64" "-DCMAKE_INSTALL_LIBDIR:STRING=lib" "-DCMAKE_INSTALL_BINDIR:STRING=bin" "-G" "Visual Studio 15 2017 Win64" "-DCMAKE_BUILD_TYPE=Debug" "-DCMAKE_INSTALL_PREFIX=C:/vcpkg/packages/szip_x64-windows-v141/debug""
FAILED: ../../x64-windows-v141-dbg/CMakeCache.txt
cmd /c "cd ../../x64-windows-v141-dbg && "C:/Program Files/CMake/bin/cmake.exe" "C:/vcpkg/buildtrees/szip/src/szip-2.1.1-6b767acfa7" "-DSZIP_INSTALL_DATA_DIR=share/szip/data" "-DSZIP_INSTALL_CMAKE_DIR=share/szip" "-DBUILD_SHARED_LIBS=ON" "-DVCPKG_CHAINLOAD_TOOLCHAIN_FILE=C:/vcpkg/scripts/toolchains/windows.cmake" "-DVCPKG_TARGET_TRIPLET=x64-windows-v141" "-DVCPKG_SET_CHARSET_FLAG=ON" "-DVCPKG_PLATFORM_TOOLSET=v141" "-DCMAKE_EXPORT_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_PACKAGE_REGISTRY=ON" "-DCMAKE_FIND_PACKAGE_NO_SYSTEM_PACKAGE_REGISTRY=ON" "-DCMAKE_INSTALL_SYSTEM_RUNTIME_LIBS_SKIP=TRUE" "-DCMAKE_VERBOSE_MAKEFILE=ON" "-DVCPKG_APPLOCAL_DEPS=OFF" "-DCMAKE_TOOLCHAIN_FILE=C:/vcpkg/scripts/buildsystems/vcpkg.cmake" "-DCMAKE_ERROR_ON_ABSOLUTE_INSTALL_DESTINATION=ON" "-DVCPKG_CXX_FLAGS=" "-DVCPKG_CXX_FLAGS_RELEASE=" "-DVCPKG_CXX_FLAGS_DEBUG=" "-DVCPKG_C_FLAGS=" "-DVCPKG_C_FLAGS_RELEASE=" "-DVCPKG_C_FLAGS_DEBUG=" "-DVCPKG_CRT_LINKAGE=dynamic" "-DVCPKG_LINKER_FLAGS=" "-DVCPKG_TARGET_ARCHITECTURE=x64" "-DCMAKE_INSTALL_LIBDIR:STRING=lib" "-DCMAKE_INSTALL_BINDIR:STRING=bin" "-G" "Visual Studio 15 2017 Win64" "-DCMAKE_BUILD_TYPE=Debug" "-DCMAKE_INSTALL_PREFIX=C:/vcpkg/packages/szip_x64-windows-v141/debug""
CMake Error at CMakeLists.txt:2 (PROJECT):
  Generator

    Visual Studio 15 2017 Win64

  could not find any instance of Visual Studio.

-- Configuring incomplete, errors occurred!
See also "C:/vcpkg/buildtrees/szip/x64-windows-v141-dbg/CMakeFiles/CMakeOutput.log".
ninja: build stopped: subcommand failed.

Basically CMake tries to use Visual Studio 2017  as its generator instead of Visual Studio 2019.
CMake-based projects that fail

proj4
freeglut
szip
collada-dom
osg
crashrpt (reported by @meastp)

CMake-based projects that succeed

bzip
curl
expat
geos
giflib
hdf5
jasper
libiconv
libjpeg-turbo
liblzma
libpng
libpq
libwebp
libxml2
minizip
netcdf-c
openexr
openjpeg
pcre
sqlite3
tiff
uriparser
zlib

Possible Reason
The difference between projects that do and don't is in the configure_cmake(...) call in their portfiles:
Projects that succeed use the PREFER_NINJA option:
vcpkg_configure_cmake(
    SOURCE_PATH ${SOURCE_PATH}
    PREFER_NINJA
    ...
)

Workaround
Manually adding the PREFER_NINJA option to the portfiles of the failing projects seems to fix the problem (or at least allows them too build).

@ceranco But your environment is not up to date:
Y:\vcpkg>vcpkg.exe version
Vcpkg package management program version 2019.06.21-nohash

The version that will work is version 2019.06.21-nohash and not Vcpkg version: 2018.11.23-nohash

Excuse me, I put the wrong version in the issue:

I'll update the issue.

@ceranco, thanks for posting this!
From https://github.com/microsoft/vcpkg/blob/4498b6403e0ef1e441a3c4df36e45fda9b181237/docs/users/triplets.md#vcpkg_platform_toolset, I think you might need to install v142 toolset.

Hi @PhoebeHui,
Visual Studio 2019 comes installed with v142 by default, but you can optionally install and build with v141 for compatibility with Visual Studio 2017.
In the triplet I specifically ask for v141, which exists in my installation (as seen projects that use NINJA work).
From the docs:

Determine the setting for VCPKG_PLATFORM_TOOLSET from the triplet or consider it unset
Gather a list of all pairs of Visual Studio Instances with all toolsets available in those instances

This is ordered first by instance type (Stable, Prerelease, Legacy) and then by toolset version (v142, v141, v140)

Filter the list based on the settings for VCPKG_VISUAL_STUDIO_PATH and VCPKG_PLATFORM_TOOLSET.
Select the best remaining option

From step 2: v141 is determined by the triplet.
From step 3: the only available pairs are (Visual Studio 2019, v141) and (Visual Studio 2019, v142).
From step 4 + 5: the (Visual Studio 2019, v141) pair should be chosen, but seems that in CMake projects it gets the (Visual Studio 2017, v141) pair, even though it doesn't exist.

@ceranco, see https://github.com/microsoft/vcpkg/blob/master/scripts/cmake/vcpkg_configure_cmake.cmake#L116, it use toolsets to match cmake generator here, so it try to get visual studio 2017. not sure if pair ( Visual Studio 2019, v141) will support in future.
Doc says that VCPKG_PLATFORM_TOOLSET valid settings is:
The Visual Studio 2019 platform toolset is v142.
The Visual Studio 2017 platform toolset is v141.
The Visual Studio 2015 platform toolset is v140.

vcpkg uses vswhere to find the toolset.  The output of vswhere also includes the version of visual studio.  To fix this issue vcpkg should insert a new cmake variable similar to VCPKG_PLATFORM_TOOLSET (VCPKG_VS_VERSION maybe?) that can be used in this file to select the correct generator.

@ceranco Please add crashrpt to the "ports that fail" list.

@meastp I've added crashrpt to the list.
Since opening this issue I've stopped using C++ and vcpkg, so I'm not sure if the packages and problems I originally listed have been fixed or if anything changed.
@Rastaban If this problem is still being worked on, it might be better to close the current one and have someone that is actively using vcpkg open a new one, whatever you prefer.

@ceranco Thanks - note that crashrpt does not build with PREFER_NINJA due to build errors with MFC/ATL/UNICODE which I haven't investigated thoroughly.

This issue still occurs in latest vcpkg(2021-08-03-1a6d67b8150e94b7f473c6ab1c0cfd) with osg port.

My idea is to add a VCPKG_GENERATOR_TOOLSET to correctly generate the -G parameter of cmake.
vcpkg_configure_cmake.cmake
        if(NOT DEFINED VCPKG_GENERATOR_TOOLSET)
            set(VCPKG_GENERATOR_TOOLSET ${VCPKG_PLATFORM_TOOLSET})
        endif()

        if("${VCPKG_GENERATOR_TOOLSET}" STREQUAL "v120" AND NOT "${VCPKG_TARGET_ARCHITECTURE}" STREQUAL "arm64")
            set(generator "Visual Studio 12 2013")
        elseif("${VCPKG_GENERATOR_TOOLSET}" STREQUAL "v140" AND NOT "${VCPKG_TARGET_ARCHITECTURE}" STREQUAL "arm64")
            set(generator "Visual Studio 14 2015")
        elseif("${VCPKG_GENERATOR_TOOLSET}" STREQUAL "v141")
            set(generator "Visual Studio 15 2017")
        elseif("${VCPKG_GENERATOR_TOOLSET}" STREQUAL "v142")
            set(generator "Visual Studio 16 2019")
        elseif("${VCPKG_GENERATOR_TOOLSET}" STREQUAL "v143")
            set(generator "Visual Studio 17 2022")
        endif()

        if("${VCPKG_TARGET_ARCHITECTURE}" STREQUAL "x86")
            set(generator_arch "Win32")
        elseif("${VCPKG_TARGET_ARCHITECTURE}" STREQUAL "x64")
            set(generator_arch "x64")
        elseif("${VCPKG_TARGET_ARCHITECTURE}" STREQUAL "arm")
            set(generator_arch "ARM")
        elseif("${VCPKG_TARGET_ARCHITECTURE}" STREQUAL "arm64")
            set(generator_arch "ARM64")
        endif()

        set(generator_toolset "-DCMAKE_GENERATOR_TOOLSET=${VCPKG_PLATFORM_TOOLSET}")

Add  VCPKG_GENERATOR_TOOLSET output in vcpkg-tool
vcpkgpaths.h
    struct Toolset
    {
        Path visual_studio_root_path;
        Path dumpbin;
        Path vcvarsall;
        std::vector<std::string> vcvarsall_options;
        CStringView version;
        std::vector<ToolsetArchOption> supported_architectures;
        CStringView vs_version;  /*Record the version of the current visual studio installer*/
    };

build.cpp
static void get_generic_cmake_build_args(const VcpkgPaths& paths,
                                             Triplet triplet,
                                             const Toolset& toolset,
                                             std::vector<CMakeVariable>& out_vars)
    {
        Util::Vectors::append(&out_vars,
                              std::initializer_list<CMakeVariable>{
                                  {"CMD", "BUILD"},
                                  {"DOWNLOADS", paths.downloads},
                                  {"TARGET_TRIPLET", triplet.canonical_name()},
                                  {"TARGET_TRIPLET_FILE", paths.get_triplet_file_path(triplet)},
                                  {"VCPKG_BASE_VERSION", Commands::Version::base_version()},
                                  {"VCPKG_CONCURRENCY", std::to_string(get_concurrency())},
                                  {"VCPKG_PLATFORM_TOOLSET", toolset.version.c_str()},
                                  {"VCPKG_GENERATOR_TOOLSET", toolset.vs_version.c_str()}, /*Record the version of the current visual studio installer*/
                              });

visualstudio.cpp
std::vector<Toolset> find_toolset_instances_preferred_first(const VcpkgPaths& paths)
{
        using CPU = CPUArchitecture;
        ....
                CStringView vs_version; /*Record the version of the current visual studio installer*/
                if (major_version == "15")
                    vs_version = V_141;
                else if (major_version == "16")
                    vs_version = V_142;
                else if (major_version == "17")
                    vs_version = V_143;

                for (const Path& subdir : msvc_subdirectories)
                {
                    auto toolset_version_full = subdir.filename();
                    auto toolset_version_prefix = toolset_version_full.substr(0, 4);
                    CStringView toolset_version;
                    std::string vcvars_option;
                    ....
                    if (fs.exists(dumpbin_path, IgnoreErrors{}))
                    {
                        Toolset toolset{vs_instance.root_path,
                                        dumpbin_path,
                                        vcvarsall_bat,
                                        {vcvars_option},
                                        toolset_version,
                                        supported_architectures,
                                        vs_version};

@ceranco  You can try this PR #21342,  Just add one more line to the  x64-windows.cmake file
set(VCPKG_TARGET_ARCHITECTURE x64)
set(VCPKG_CRT_LINKAGE dynamic)
set(VCPKG_LIBRARY_LINKAGE dynamic)
set(VCPKG_PLATFORM_TOOLSET v141)
set(VCPKG_GENERATOR_TOOLSET v142)

I can confirm this issue was fixed using the following command:
.\vcpkg.exe install proj4:x64-windows-v141  --host-triplet=x64-windows-v141

@JackBoosY I created a new x64-windows-v141.cmake file
set(VCPKG_TARGET_ARCHITECTURE x64)
set(VCPKG_CRT_LINKAGE dynamic)
set(VCPKG_LIBRARY_LINKAGE dynamic)
set(VCPKG_PLATFORM_TOOLSET v141)

It is correct to implement it directly.
vcpkg install basisu:x64-windows-v141

However, when I delete the PREFER_NINJA in the portfile.cmake file of basisu port, it reports an error
CMake Error at CMakeLists.txt:2 (project):
  Generator

    Visual Studio 15 2017

  could not find any instance of Visual Studio.

set(VCPKG_PLATFORM_TOOLSET v142)

But I just want to compile with toolset v141 when only vs2019 is installed

Can you please replace vcpkg_configure_cmake / vcpkg_install_cmake with vcpkg_cmake_configure / vcpkg_cmake_install, remove PREFER_NINJA and add the following code to its vcpkg.json then try again:
{
    ...
    "dependencies": [
    ...
    {
        "name": "vcpkg-cmake",
        "host": true
    }
    ]
}


[Feature] On demand loading using 'file exists' as the criteria.

Hello @junegunn ,
Excellent plugin manager. Many thanks for your contributions to the Vim community.
I suspect this one will be closed as wontfix due to you not being a fan of on-demand loading anymore. I understand. However, it does no harm to ask.
In my case I have a weird conflict between two Tim Pope plugins: vim-rails and vim-projectionist along with FZF.vim. Those two Tim Pope plugins, useful to navigate framework dev trees, have somewhat of an overlap, so much so that there may be a conflict when both are loaded. It seems with FZF.vim Tags command having both plugins enabled actually causes a visible failure related to RailsDetect. Disable one or the other of vim-rails or vim-projectionist and all is fine.
Long story short, I want to on-demand load vim-rails only in Rails development trees and vim-projectionist only in Elixir/Phoenix or create-react-app trees.
Something like:
Plug 'tpope/vim-rails', { 'file': 'config/routes.rb' }
Plug 'tpope/vim-projectionist', { 'file': ['web/router.ex', 'src/index.js'] }

If I am not in a Rails, or Elixir/Phoenix or React app then I avoid loading up both plugins. Note, vim-projectionist takes 100 to 180ms to load.  It was not a mistake to add on-demand, it has genuine uses. Some plugins are very heavy, such as NERDTree and vim-projectionist.
Note, on-demand loading based on filetype is not fine-grained enough since Rails and React projects will both have JavaScript files; telling the difference between Rails and React projects is done by the directory structure and files contained.
As for the vim-rails and vim-projectionist and FZF.vim :Tags weirdness, I don't know whether it is a tpope issue or an FZF.vim issue (likely tpope). However, if Tim is not an FZF user he may not be interested in fixing a very unusual corner case which involves a plugin not of his making (FZF.vim).
Best regards.

Tim Pope has fixed the vim-rails and vim-projectionist issue I describe above. I don't know which commit it was (or which plugin) but the issue is now gone after I did a big plugin update.
So I don't need this feature, and I suspect it would never be implemented anyway (the author appears not to be a fan of lazy loading anymore).
Cheers.


Strange back-button behavior

My Android application uses Gluon Charm and a NavigationDrawer to handle the various views. What puzzles me is the strange and inconsistent behavior of the back-button. On my Samsung Galaxy Tab S4 with Android 10 the behavior is to immediately jump back to the users Android home screen whenever I hit the back-button. However the same apk-file installed on my Samsung Galaxy S7 phone with Android 8 shows a different behavior. The NavigationDrawer walks back the stack as expected but once it reaches the applications home screen it refuses to go back to the users Android home screen. I even added a Quit-button which calls the LifecycleService.shutdown method but that does not seem to have any effect either. So my situation is that on the tablet I always get back to the Android home screen, even if that is not intended and on the phone I never get back to that screen even if I want to. If there are any hidden Android switches, which might influence this behavior, please let me know. This has nothing to do with the recent changes because I observe this behavior already for a long time.

Using Gluon Mobile, the Back button is used to deal with the view navigation: you can go back from the current view to the previous one, as the user view navigation history is stored in some stack.
By default, the home view is not part of the stack (so once in it, the back button just quits the app), see SKIP_VIEW_STACK for instance in the Notes app: https://github.com/gluonhq/gluon-samples/blob/substrate/notes/src/main/java/com/gluonhq/samples/notes/views/AppViewManager.java#L50.
In any case, the back button is mapped to the ESC key, so there is this event handler:
scene.addEventHandler(KeyEvent.KEY_RELEASED, e -> {
                if (KeyCode.ESCAPE.equals(e.getCode())) {
                    // switchToPreviousView
                }
});

Can you add it too to your project and print out if its get called when you press Back?
Also check your view stack policy.

In order to keep things simple for the moment, I did not explicitly specify any ViewStackPolicy. So, the defaults should be used. This leaves me with the questions:

Why is the behavior different between tablet and phone? Are there any debug settings on device which might cause this?
Why does the app not go back to the Android home screen once back in the apps home screen and the back button is pressed again? I added the event handler and it is indeed called every time on the phone but never on the tablet.
Why does explicitly calling the LifecycleService.shutdown method have no effect? Calling Platform.exit instead just leads to a black screen but not to a termination of the app.

Hmm, that rings a bell... We fixed this issue a while ago in JavaFXPorts. It might explain why behavior is different between Android 8 and 10.

As is now with Substrate, on Android 10, the JavaFX view is not focused by default, and pressing back will call the onDestroy method from the MainActivity, so that explains why the app quits immediately. The ESC handler doesn't work, if the scene is not focused.

On Android 8, on the contrary, the view gets focused, and onDestroy is never called. Back button works between views (because of the ESC handler) but doesn't quit the app, because Attach doesn't actually call it.

I'll file issues in Substrate and Attach.

Two PRs have been created.
If you want to give them a try, build Substrate from https://github.com/jperedadnr/substrate/tree/713-focus and Attach from https://github.com/jperedadnr/attach-2/tree/176-shutdown.

I've built the libs but how do I convince the client-maven-plugin to use the created substrate-0.0.28-SNAPSHOT.

Use 0.1.30-SNAPSHOT, and either use the repo:
<pluginRepositories>
        <pluginRepository>
            <id>snapshots</id>
            <url>https://oss.sonatype.org/content/repositories/snapshots/</url>
        </pluginRepository>
    </pluginRepositories>

or clone the Client maven plugin and build it (mvn clean install).

I tried this already but the build of 0.1.30-SNAPSHOT seems to be broken.

Better build it and install it locally then

Ok, did that. The result is:

Tablet and phone behave identically now.
I can walk back the stack with the back-button now as you would expect it.
Once I have reached the applications home screen, one further back-button click makes the application hang or even crash.
I also cannot terminate the application via the lifecycle shutdown method. The application hangs.
Good night
Michael

Great, thanks for testing. and notifying the shutdown crash issue. It should be solved now with this: https://github.com/jperedadnr/attach-2/commit/673a618a7c67a2fa6bd598dc03351ef5a052dc27

I just tried your last patch. The result is:

Tablet and phone behave identically now.
I can walk back the stack with the back-button now as you would expect it.
Once I have reached the applications home screen, one further back-button click terminates the application as it should.
Explicitly calling the lifecycle shutdown method also terminates the application as it should.

When terminating, the application seems to go through the states pause, stop, destroy but a lifecycle event listener like this
LifecycleService.create().ifPresent(ls -> ls.addListener(LifecycleEvent.PAUSE, () -> log.info("Application paused.")));
is never called.
One further question remains: What should I do to prevent termination of the application via the last back-button click?
Michael

About Pause not called, once the activity finishes, pause, stop and destroy states are forwarded via notifyLifecycleEvent and Attach "sees" those events. The app should see them too. Your listener works for me (just replaced log.info with System.out.println):
[mar. jul. 28 11:05:36 CEST 2020][INFO] [SUB] V/GluonAttach(  544): DalvikLifecycleService::shutdown
[mar. jul. 28 11:05:36 CEST 2020][INFO] [SUB] V/GraalActivity(  544): onPause
[mar. jul. 28 11:05:36 CEST 2020][INFO] [SUB] D/GluonAttach(  544): Call Attach method from native Lifecycle: pause
[mar. jul. 28 11:05:36 CEST 2020][INFO] [SUB] D/GraalCompiled(  544): Application paused.
[mar. jul. 28 11:05:37 CEST 2020][INFO] [SUB] V/GraalActivity(  544): onStop
[mar. jul. 28 11:05:37 CEST 2020][INFO] [SUB] D/GluonAttach(  544): Call Attach method from native Lifecycle: stop
[mar. jul. 28 11:05:37 CEST 2020][INFO] [SUB] V/GraalActivity(  544): onDestroy
[mar. jul. 28 11:05:37 CEST 2020][INFO] [SUB] D/GluonAttach(  544): Call Attach method from native Lifecycle: destroy
...

For the back-button, you can simply prevent the handler with an event filter:
scene.addEventFilter(KeyEvent.KEY_RELEASED, e -> {
            if (KeyCode.ESCAPE.equals(e.getCode())) {
                e.consume();
            }
        });

Make sure you have some condition about doing it only for the home view.

Filtering the back-button works as you said but call me crazy, this code
log.info("Add listener for LifecycleEvent.PAUSE.");
LifecycleService.create().ifPresentOrElse(ls -> ls.addListener(LifecycleEvent.PAUSE,
   () -> System.out.println(">>> Application paused <<<")),
   () -> System.out.println("No LifecycleService available."));

gets executed on the postInit method of my MobileApplication class but it does not print anything when I terminate the application via an explicit call to the LifeCycle.shutdown method.
[Tue Jul 28 12:04:58 CEST 2020][INFO] [SUB] V/GluonAttach( 1529): DalvikLifecycleService::shutdown
[Tue Jul 28 12:04:58 CEST 2020][INFO] [SUB] V/GraalActivity( 1529): onPause
[Tue Jul 28 12:04:58 CEST 2020][INFO] [SUB] E/GraalGluon( 1529): Dispatching lifecycle event from native Dalvik layer: pause
[Tue Jul 28 12:04:58 CEST 2020][INFO] [SUB] D/GluonAttach( 1529): Call Attach method from native Lifecycle: pause
[Tue Jul 28 12:04:58 CEST 2020][INFO] [SUB] D/GraalGluon( 1529): ATTACH_DALVIK, tid = 1529, existed? 1, dalvikEnv at 0x7dbb0d6200
[Tue Jul 28 12:04:58 CEST 2020][INFO] [SUB] D/GraalGluon( 1529): DETACH_DALVIK, tid = 1572, existed = 0, env at 0x7d789d5200
[Tue Jul 28 12:04:58 CEST 2020][INFO] [SUB] D/GraalGluon( 1529): DETACH_DALVIK, tid = 1529, existed = 1, env at 0x7dbb0d6200
[Tue Jul 28 12:04:58 CEST 2020][INFO] [SUB] D/GluonAttach( 1529): called Attach method from native Lifecycle done
[Tue Jul 28 12:04:59 CEST 2020][INFO] [SUB] V/GraalActivity( 1529): onStop
[Tue Jul 28 12:04:59 CEST 2020][INFO] [SUB] E/GraalGluon( 1529): Dispatching lifecycle event from native Dalvik layer: stop
[Tue Jul 28 12:04:59 CEST 2020][INFO] [SUB] D/GluonAttach( 1529): Call Attach method from native Lifecycle: stop
[Tue Jul 28 12:04:59 CEST 2020][INFO] [SUB] D/GraalGluon( 1529): ATTACH_DALVIK, tid = 1529, existed? 1, dalvikEnv at 0x7dbb0d6200
[Tue Jul 28 12:04:59 CEST 2020][INFO] [SUB] D/GraalGluon( 1529): DETACH_DALVIK, tid = 1529, existed = 1, env at 0x7dbb0d6200
[Tue Jul 28 12:04:59 CEST 2020][INFO] [SUB] D/GluonAttach( 1529): called Attach method from native Lifecycle done
[Tue Jul 28 12:04:59 CEST 2020][INFO] [SUB] V/GraalActivity( 1529): onDestroy
[Tue Jul 28 12:04:59 CEST 2020][INFO] [SUB] E/GraalGluon( 1529): Dispatching lifecycle event from native Dalvik layer: destroy
[Tue Jul 28 12:04:59 CEST 2020][INFO] [SUB] D/GluonAttach( 1529): Call Attach method from native Lifecycle: destroy
[Tue Jul 28 12:04:59 CEST 2020][INFO] [SUB] D/GraalGluon( 1529): ATTACH_DALVIK, tid = 1529, existed? 1, dalvikEnv at 0x7dbb0d6200
[Tue Jul 28 12:04:59 CEST 2020][INFO] [SUB] D/GraalGluon( 1529): DETACH_DALVIK, tid = 1529, existed = 1, env at 0x7dbb0d6200
[Tue Jul 28 12:04:59 CEST 2020][INFO] [SUB] D/GluonAttach( 1529): called Attach method from native Lifecycle done

Can you try adb logcat instead of mvn client:run? Sometimes not all the logs are shown with the latter.
Anyway, this is what I have (pretty much the same):

I tried adb logcat but it doesn't make a difference. Maybe we should postpone further work on this until all the other changes are integrated into the master branches, so that we can be sure to really operate on the same code base.

Sure, that makes sense. However, now that you build Attach locally, you could add some debug info and find out...

Client plugin release 0.1.30 should fix this. For Attach, 4.0.9-SNAPSHOT.


ACLs: Extract AuthService implementation

Closes #8063
Change Description
Background
This PR extracts the AuthService implementation into a new packaged named contrib/auth-acl/
There are no code changes other than that
Testing Details
Existing test coverage to ensure nothing was broken
Breaking Change?
No

Apparently go doesn't like the package name auth-acl - renaming

Apparently go doesn't like the package name auth-acl - renaming

auth_acl, but apparently package authacl is the one of the One True Go Ways.


Add "Select All" option in the UI

Can we add a "Select All" option in the portal for policy and initiative selection? There are 100+ policy definitions and it is time consuming to add one by one. Thanks.

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

ID: 4ff59c65-dffd-a9d3-e02c-25c688322d91
Version Independent ID: 001249ab-2a55-e8ee-7d3c-e3cb7f4afd24
Content: Export Azure Policy resources - Azure Policy
Content Source: articles/governance/policy/how-to/export-resources.md
Service: azure-policy
GitHub Login: @DCtheGeek
Microsoft Alias: dacoulte

@mpchen Thanks for your comment. We will review the issue and get back to you shortly.

#reassign:DCtheGeek

@mpchen Thanks for the question.  Since this is a request for a feature/capability and not an Issue with the Docs, it belongs on UserVoice - Azure Policy.  Please post the request there and the product team will evaluate it for inclusion in future sprints.  Thanks!
@MicrosoftDocs/azure-cxp-triage #please-close


Auto clear data/tabs

Task/Issue URL: https://app.asana.com/0/856498667320406/765761023328863
Tech Design URL: https://app.asana.com/0/856498667320406/905625717624365
Description:
This PR adds option to set up automatic tabs/data clearing upon leaving the app.
Notable features:

Clearing tabs or tabs and data (e.g. cookies) when trigger conditions are met.
Can be triggered by killing the app and (optionally) after certain amount of inactivity (when app is put to the background).
Introduced new settings screen with auto clear options.
Introduced app preview blocking (using blank screen) when clear tabs/data feature is enabled.

Steps to test this PR:
There are multiple scenarios to test:
Clearing tabs

Open some tabs.
Log in to a service (email, or whatever).
Set up auto clear for 'tabs'.
Kill the app.
Expectation: upon restart, there should be no tabs open. When accessing mentioned in #2 service again, user should be logged in.

Clearing data and tabs

Open some tabs.
Log in to a service (email, or whatever).
Set up auto clear for 'tabs and data'.
Kill the app.
Expectation: upon restart, there should be no tabs open. When accessing mentioned in #2 service again, user should not be logged in.

Clearing action triggered by termination
A:

Set up clearing with 'app exit' trigger.
Validate that killing the app clears the tabs.
B:
Set up clearing for 'app exit, inactivity X mins' .
Validate that killing the app clears the tabs.

Clearing action triggered after delay

Set up clearing with 'app exit, inactivity X mins' trigger.
Put app to the background.
Expectation: coming back to the app within chosen time period should not clear the tabs/data. Coming back once time has elapsed, should result in clear action being performed.

Snapshot blocking

Turn on tabs/data clearing.
Put app to the background.
Expectation: app preview in task manager should be replaced by "blank snapshot" of the app.
Note: This snapshot has two variants, depending on enabled features (home row&centered search vs other).

Validating screen lock

Turn on tabs/data clearing.
Turn on application lock.
Put app to the background.
Expectation: app screen locking (requiring touch ID/face ID) works as expected when coming back to the app or upon restart.

Pixels
A:

Open auto clear settings.
Expectation: Pixel should be fired.
B:
Open auto clear settings.
Tweak settings.
Exit auto clear settings screen.
Expectation: Pixels for tweaked options (action, trigger, or both) should be fired after #3.
C:
Open auto clear settings.
Tweak settings, but then reset all to the initial values.
Exit auto clear settings screen.
Expectation: Pixels should not be fired after #3.

Settings

Open DDG settings screen.
Expectation: auto clear on/off status label should be properly displayed.

Internal references:
Software Engineering Expectations
Technical Design Template

@brindy Thanks for the review, I made some fixes based on your comments - could you have another look?


remove --aws-account-id flag from deployment

ACK runtime v0.15.0 removed support for the --aws-account-id
controller flag but our deployment templates were still adding this flag
to the controller entrypoint args, which causes tests to fail with:
unknown flag: --aws-account-id

Related: #213
See: https://github.com/aws-controllers-k8s/runtime/commit/60801010475969b411d192574c9a47b5acc81bb9
Signed-off-by: Jay Pipes<EMAIL_ADDRESS>By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license.

@jaypipes @brycahta need to remove from helm charts https://github.com/aws-controllers-k8s/code-generator/blob/main/templates/helm/values.yaml.tpl#L41

Functionally , this is fine today right? I agree it should be cleaned up along with its use here: test-helm.sh

@jaypipes @brycahta need to remove from helm charts https://github.com/aws-controllers-k8s/code-generator/blob/main/templates/helm/values.yaml.tpl#L41

No, the environment variables are fine. It's just the command-line flag that was at issue.

We still should remove the Helm value and corresponding environment variable from the templates - they serve no purpose anymore

I don't believe setting the environment variable has any effect in the AWS SDK Go - https://github.com/aws/aws-sdk-go/search?q=AWS_ACCOUNT_ID ? Through what mechanism would it override those results?

Sorry, was confusing with AWS_ACCESS_KEY_ID: https://github.com/aws/aws-sdk-go/blob/main/aws/credentials/env_provider.go


It doesn't make any sense or help in understanding what it is. I still have no idea after reading the page.

[Enter feedback here]

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

ID: 28550025-fde2-9a83-a826-f7e3ccb313db
Version Independent ID: 2a304432-b10c-220f-3453-e19e7a155c67
Content: Manage the Microsoft Teams Exploratory experience - Microsoft Teams
Content Source: Teams/teams-exploratory.md
Service: msteams
GitHub Login: @DaniEASmith
Microsoft Alias: danismith

Hello @LeonOrtoo
Thanks for your feedback.
Can you please specify what do you see can be fixed in this article? or explain more in detail.
Thank you

We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Panel size is not remembered correctly when panel is moved to the right

VSCode Version: 1.41.0-insider
OS Version: macOS 10.15

Steps to Reproduce:

Open a new window and the panel.
Move the panel to the right.
Make the panel bigger.
Close the window and open a new one.
Open the panel.
The panel opens at the bottom instead of on the right and it's too big.

Video of the reproduction steps: https://streamable.com/fkcox.

Does this issue occur when all extensions are disabled?: Yes

The problem is that the panel position is not being saved on window close but on shutdown. This won't reproduce on Windows or Linux for that reason (window close = shutdown).

Ignore my previous statement, the correct root cause is that when a new empty window is created on macOS and VS Code is already running(even without a window), the new window is always a new workspace id. However, from a cold start (VS Code is not running), the new empty window uses the same workspace id as before.
Panel position and visibility are stored in workspace storage and therefore in the repro case, do not exist. The size is written to global storage so it is restored.
@bpasero Does this workspace id generation sound correct? It seems strange that a new window from cold and hot starts give differing results if no windows are open from the beginning.
For the panel state, I could check if this is an empty workspace and just ignore workspace storage and revert to defualts. But, I want to check that this is how storage is expected to work.

This is correct, macOS has the concept of running apps even without windows. If you then go and click the app, it will open a new empty window with new workspace id. Starting VSCode when it is not running will restore any previous workspaces, including empty ones.

Still seems strange to treat clicking the app icon differently when no windows are open based on the app running in the background. But perhaps that's what Mac users expect

@sbatten my understanding is that clicking on the app icon when no window is open acts like "New Window" and not "Restore Window from Previous Session".

I've updated the restoration logic to ignore the global storage panel size when we aren't restoring positions but rather just using defaults.

new fix is similar but will restore size if the position matches the last saved panel size


Code coverage

Added code coverage

Current coverage is 94.25% (diff: 100%)

No coverage report found for master at d61c8ad.

Powered by Codecov. Last update d61c8ad...2914ef4


Context module training implementation plans

Thanks for creating this implementation. Do you have any plans to implement training of the context module (to allow producing full resolution segmentation maps)?

I'm really interested in training the context module too. Main reason I wish to use this model is for the high res. segmentation prediction.
Is it complex to do - could you guide us on how to do it?

Thanks for the quick reply. Yes this fits a niche - it's a better architecture than FCN & SegNet, but doesn't require 8 GPUs to train like the latest 'RefineNet's etc. It's also such nice and simple code in Keras! I can't get, for example DeepLab, in Keras....
If you implement it, I would love to use it!
Christian


Feature request: scrap by name

Hi,
This scrapper really helped me a lot with my "basic" roms, but for roms with translation patches applied, this is of no use... It would be nice to have the option to use the name of the file instead of the hash.

I have https://github.com/sselph/scraper/issues/13 where I was thinking about writing something to allow you to search for games that didn't match but never got to it. I had a prototype, but never got it cleaned up. I don't think I want to make any kind of name based auto scraping for console roms since it may be error prone.
If you need this in a timely manner, I would suggest seeing if there are some other scrapers out there that work based on name and seeing if they can be instructed to only act on entries that are missing after running this scraper.

I looked for other scrapers but didn't find any; so I thought about installing emulation station and using the internal one but it doesn't install on a mac...
I effectively need this in a timely manner, it's just that right now I have like 30 roms and I don't want to do it by hand :). I guess I'll program a quick and dirty scrapper myself.
Thanks for the answer.


Webpack 2 Compat

Is this compatible with webpack 2? Specifically, they're called "module.rules" now, instead of "module.loaders". Will this work with the new configuration?

merge.smart needs a tiny tweak, but apart from that it should be fine as the basic idea is generic. I even used it with some karma setup myself.

Included in 0.16.0. Let me know how it goes.


Make Deploys Faster and Smarter

Is your feature request related to a problem? Please describe.
Every time theme deploy is ran, all files of the the theme are uploaded. However, often only a few files actually changed. This is not only slow and inefficient but also error prone, as Shopifys API request limits are hit often, when a theme is deployed to multiple origins concurrently.
Describe the solution you'd like
It would by great if theme deploy only deploys and updates those files, that are created, updated or deleted. Also the command should be more resilient to API request limits. Instead of exiting the command when the request limit is hit, the command could pause an retry to upload a file.
Describe alternatives you've considered
For now we we are restarting theme deploy in our CICD pipeline every time the deploy failed due to hitting the API request limit.

API request limits

Themekit is throttled absolutely agains requests limits. If you are hitting the throttle then you are using the same api key and password for many instances of themekit.

If you are hitting the throttle then you are using the same api key and password for many instances of themekit.

@tanema Yes, I use a single API key to deploy to different environments of the same shop concurrently. Is the rate limit hard coded to 2 rps or does it use the Retry-After header?

@andyw8 Thanks for the awesome release v1.1.0. All my deployments are >10x faster now.


content type from volume post upload always text/plain?

I'm assuming the return content-type should be application/json?
$ curl -v -F "file=@/tmp/test.json;type=application/json" localhost:8080/7,07a27ee211
> POST /7,07a27ee211 HTTP/1.1
> User-Agent: curl/7.30.0
> Host: localhost:8080
> Accept: */*
> Content-Length: 223
> Expect: 100-continue
> Content-Type: multipart/form-data; boundary=----------------------------2f1cf300fd2b
>
< HTTP/1.1 100 Continue
< HTTP/1.1 201 Created
< Date: Mon, 29 Sep 2014 20:08:45 GMT
< Content-Length: 30
< Content-Type: text/plain; charset=utf-8
<
* Connection #0 to host localhost left intact
{"name":"test.json","size":74}

I Chris, I faced the same problem, and I have been able to fix this in a dirty way, but I think it needs some refacto...
Regarding golang documentation : http://golang.org/pkg/net/http/#Header.Write
It says : "Changing the header after a call to WriteHeader (or Write) has no effect."
In go/weed/weed_server/volume_server_handlers.go you call w.writeHeader(http.StatusCreated) first and after in common.go writeJson method you set the content-type to json but it has no-effect...
If you do the following in go/weed/weed_server/volume_server_handlers.go it will work with json content-type
	if errorStatus == "" {
	 +	w.Header().Set("Content-Type", "application/json; charset=utf-8")
		w.WriteHeader(http.StatusCreated)
	} else {
		w.WriteHeader(http.StatusInternalServerError)

Hope this helps

@kyzer028 Thanks a lot!!!!!
I took the time to fix similar problems everywhere.
commit 5b5d6341e570b2f12cd375279ee004529193b970
Author: Chris Lu<EMAIL_ADDRESS>Date:   Thu Jan 8 00:19:32 2015 -0800
Set http status after modifying response headers.

@chrislusf You're welcome, thanks for fixing it that quickly :+1:


TLS Support, Cluster Support

[ yes ] I have read the contributing guide.
[ yes ] Issue:  #42

Implementation
Converted to use net/url for config, cleaned up usage, added tls options

@chonthu can you please merge this one?
the support in tls will help m e alot!
Thanks!


depth processing seems not to work

I checked the database and saw many tracks uploaded from 2023 to 2024. They all show a status of "1" since January 8, 2023. It seems like the track upload is fine, but there might be a problem with the depth processing part.

logfile shows following entry:
!SESSION 2023-08-11 10:13:49.805 -----------------------------------------------
eclipse.buildId=unknown
java.version=1.8.0_252
java.vendor=Oracle Corporation
BootLoader constants: OS=linux, ARCH=x86_64, WS=gtk, NL=en_US
Command-line arguments:  -os linux -ws gtk -arch x86_64 -console

!ENTRY net.sf.seesea.data.io.postgis 4 0 2023-08-11 10:13:50.494
!MESSAGE [net.sf.seesea.data.io.postgis.PostInsertGISWriter(4)] The activate method has thrown an exception
!STACK 0
org.postgresql.util.PSQLException: Connection to localhost:5435 refused. Check that the hostname and port are correct and that the postmaster is accepting TCP/IP connections.
	at org.postgresql.core.v3.ConnectionFactoryImpl.openConnectionImpl(ConnectionFactoryImpl.java:262)
	at org.postgresql.core.ConnectionFactory.openConnection(ConnectionFactory.java:51)
	at org.postgresql.jdbc.PgConnection.<init>(PgConnection.java:215)
	at org.postgresql.Driver.makeConnection(Driver.java:404)

note:
I checked all config files and can confirm that "localhost" is not used, All config files include following entries for hostname and port:
server=postgis
port=5435

I made a new build of the software this week and deployed it to server. I found following error in the logfile:
org.postgresql.util.PSQLException: FATAL: no pg_hba.conf entry for host "<IP_ADDRESS>", user "osm", database "depth", SSL off
	at org.postgresql.core.v3.ConnectionFactoryImpl.doAuthentication(ConnectionFactoryImpl.java:443)
	at org.postgresql.core.v3.ConnectionFactoryImpl.openConnectionImpl(ConnectionFactoryImpl.java:217)
	at org.postgresql.core.ConnectionFactory.openConnection(ConnectionFactory.java:51)
	at org.postgresql.jdbc.PgConnection.<init>(PgConnection.java:215)
	at org.postgresql.Driver.makeConnection(Driver.java:404)
	at org.postgresql.Driver.connect(Driver.java:272)
	at java.sql.DriverManager.getConnection(DriverManager.java:664)
	at java.sql.DriverManager.getConnection(DriverManager.java:247)
	at org.postgresql.ds.common.BaseDataSource.getConnection(BaseDataSource.java:86)
	at org.postgresql.ds.common.BaseDataSource.getConnection(BaseDataSource.java:71)
	at net.sf.seesea.data.io.postgis.PostInsertGISWriter.activate(PostInsertGISWriter.java:94)

After I changed the password in the following logfiles, the error is gone.
./config/net.sf.seesea.data.io.postgis.PostgresDatasourceConfiguration-depth.cfg.
./config/net.sf.seesea.data.io.postgis.PostgresDatasourceConfiguration-userdata.cfg


Fix Empty Partition Bug

The Problem
In the k_prototypes function, we have a bug resulting from the mishandling of empty clusters in a given iteration.
If, in the previous iteration, no points are assigned to a specific cluster (new_prototypes[k] = [] for some k), then we need to intelligently handle what this cluster centre is. Currently it is being set to np.nan (due to the handling of empty arguments in np.average), which causes bugs in the next pass.
Proposed Solution
Check if any of the clusters are empty in each iteration. If they are, iteratively set the corresponding cluster centres using a method similar to the star_shaped_init function. That is, find the point in the space for which the corresponding prototype has maximal min KL-divergence from all existing cluster centres, and set that as the centre of the new cluster.

Closed as already resolved. See README where this is documented


Seperate lsb_dist detection from dist_version detection

Fixes #14924. As discussed with @jfrazelle on IRC to make the lsb_dist/dist_version detection more flexible.
Signed-off-by: Avi Miller<EMAIL_ADDRESS>

Ping @jfrazelle and @tianon for review.

Separate ${lsb_dist} and ${dist_version} is indeed a better solution!

+1

ping @erikh does this fix your issue?

[root@ceph-mon0 yum.repos.d]# curl https://raw.githubusercontent.com/Djelibeybi/docker/fix14924/hack/install.sh | sudo bash
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  9752  100  9752    0     0  25300      0 --:--:-- --:--:-- --:--:-- 25329
+ sh -c 'sleep 3; yum -y -q install docker-engine'
warning: /var/cache/yum/x86_64/7/docker-main-repo/packages/docker-engine-1.7.1-1.el7.centos.x86_64.rpm: Header V4 RSA/SHA1 Signature, key ID 2c52609d: NOKEY
Public key for docker-engine-1.7.1-1.el7.centos.x86_64.rpm is not installed
Importing GPG key 0x2C52609D:
 Userid     : "Docker Release Tool (releasedocker<EMAIL_ADDRESS> Fingerprint: 5811 8e89 f3a9 1289 7c07 0adb f762 2157 2c52 609d
 From       : https://yum.dockerproject.org/gpg

If you would like to use Docker as a non-root user, you should now consider
adding your user to the "docker" group with something like:

  sudo usermod -aG docker your-user

Remember that you will have to log out and back in for this to take effect!

I’d say yes :) Thanks so much @djelibeybi and @jessfraz!

On Jul 29, 2015, at 11:37 AM, Jessie Frazelle<EMAIL_ADDRESS>wrote:
ping @erikh https://github.com/erikh does this fix your issue?
—
Reply to this email directly or view it on GitHub https://github.com/docker/docker/pull/15103#issuecomment-126053137.

Actually no, this doesn't necessarily fix experimental. Let me hack up the script a touch to fix the hostname and validate again.

@erikh I didn't touch the hostname bits, so that's identical to upstream. This should just fix the lsb_dist and dist_version determination.

I have validated this works for both experimental and release on centos 7.

@Djelibeybi it looks like we need to rebase your branch. Can you take care of it?

Rebase committed.

ok let me run my test script on this, also ping @tianon

LGTM

This can be now removed https://github.com/docker/docker/blob/master/hack/install.sh#L54

cherry picked into #15091


[Bug]: Large images are not loaded in the preview canvas of the image upload

Currently an empty canvas is shown after selecting a big image. This empty canvas can then be uploaded. The result is, that the alt text of the image is shown inside the detail page.
There should be an image dimension and file size restriction on the image upload component.

Show an error message when those restrictions arent fulfilled by the users image upload. (Good)
Or, if possible, prevent selecting images from the file selector that do not fulfill those restrictions. (Better)


Working with restored models

Thanks for making the code publically available.
I have trained the model and now I want to use the saved model to make predictions on some hand-picked test examples. I have restored the saved model but not sure which tensor operation to evaluate in order to get predictions?
Can you please help me with that?

What version of python are you using?


don't allow fallback action to be used as other actions for phrases.

Issues

If you're having trouble with Articulate we definitely want to help you. If you can provide a little bit of information it will make our troubleshooting job easier.

A fallback action can be used right now for an action for a normal training phrase. I think the fallback actions created should be locked just to the agent settings and only created in the agent page. So for example, if you set the fallback action to a saying: I want a pizza, a window can come up that says "can not set fallback action as a standard action to a phrase!" Or something like that.

ver 21.1 release
local
win10 pro
FF browser

this is on purpose. We have places where we fall back to a more generic search, but that search my also be useful as a regular action.

ok


Separate Implementation of Feature from Usage (like Cucumber Step Definitions)

This is more a feature request to separate cleaner implementation of steps (like in cucumber step definitions) from usage of them in feature files.
I want to understand if you plane to improve that ?
If not if there is any workaround to get cleaner separation ?

@rober15 there are no plans and we believe Karate's strength is the flexibility that it offers for test-automation
my recommendation is instead of trying for cleaner separation (which benefits only developers), aim for making tests readable by non-programmers. this is explained well here: https://conference.eurostarsoftwaretesting.com/is-behaviour-driven-development-bdd-right-for-api-testing
a more detailed discussion is in this thread, and I'll link you to the last comment for convenience: https://github.com/karatelabs/karate/issues/398#issuecomment-1135475264
hope that helps !


docs(readme): Fix typographical error

Gannon -> Ganon, because Gannon is only used in the first few Legend of Zelda games, but the presence of Navi implies we're talking in more modern terms.

😹 🕹


Add/Change authorization behaviour in console

As an admin I want to be able to easily add new authorizations/roles to my end users.
Acceptance criteria

[ ] I can click the new button and add an authorization no matter if it already exists or not
[ ] When adding a new authorization that already exists, the "new roles" are added to the existing authorization

This issue is created based on the following bug report:
#4577


erro no npm i

mathe@itamachi MINGW64 ~/Documents/Nova pasta/iris (dev)
$ npm i
npm WARN deprecated<EMAIL_ADDRESS>request has been deprecated, see https://github.com/request/request/issues/3142
npm WARN deprecated<EMAIL_ADDRESS>Please upgrade to v7.0.2+ of superagent.  We have fixed numerous issues with streams, form-data, attach(), filesystem errors not bubbling up (ENOENT on attach()), and all tests are now passing.  See the releases tab for more information at https://github.com/visionmedia/superagent/releases. Thanks to @shadowgate15, @spence-s, and @niftylettuce. Superagent is sponsored by Forward Email at https://forwardemail.net.
npm WARN deprecated<EMAIL_ADDRESS>this library is no longer supported
npm WARN deprecated<EMAIL_ADDRESS>Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.
npm WARN deprecated<EMAIL_ADDRESS>request-promise has been deprecated because it extends the now deprecated request package, see https://github.com/request/request/issues/3142
npm WARN deprecated<EMAIL_ADDRESS>Please upgrade to latest, formidable@v2 or formidable@v3!  You must install peer dependencies yourself.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>preinstall: npx bin-version-check-cli python ">=2"
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>preinstall script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\mathe\AppData\Roaming\npm-cache_logs\2022-01-28T13_50_31_203Z-debug.log

eu colei o codigo tbm pq n ia caber no print

Use esse tutorial -> https://github.com/KillovSky/iris/issues/377#issuecomment-951801151


[docs] Dead link to technical docs in README.md

https://github.com/drift-labs/protocol-v2/tree/master/sdk#sdk-guide
The technical documentation for the SDK can be found here,...

Fixed!


Arithmetic methods for Swift 3 compatibility

Since Decimal does not conform to FloatingPoint protocol, this fix does not apply.

@swift-ci test

Thanks!


[pulsar-sql] Bump presto.version to run PrestoServer on JDK11+

Motivation
This is another attempt to fix #14951 without introduce workaround.
Modifications
Bump presto.version to 334.
Verifying this change

[x] Make sure that the change passes the CI checks.

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (yes)

To fix #14951. This version bump should be transparent to users.
Config option presto.version has been removed at https://github.com/trinodb/trino/pull/3708. Since PrestoServer uses strict config validations, users who configure presto.version may get a fatal error when starting PrestoServer.

The public API: (yes / no)
The schema: (yes / no / don't know)
The default values of configurations: (yes / no)
The wire protocol: (yes / no)
The rest endpoints: (yes / no)
The admin cli options: (yes / no)
Anything that affects deployment: (yes / no / don't know)

Documentation
Check the box below or label this PR directly.
Need to update docs?

[ ] doc-required
(Your PR needs to update docs and you will update later)

[x] doc-not-needed
(Please explain why)

[ ] doc
(Your PR contains doc changes)

[ ] doc-complete
(Docs have been already added)

cc @lhotari @codelipenghui
This change comes from https://github.com/apache/pulsar/issues/14951#issuecomment-1083376688. And I touch this module since java-version-trim-agent is the only module uses java.util.logging related to this thread.
Bump dependency version is clear and easy to maintain than introducing an in-place agent module IMO.
cc @gaoran10

All tests has passed at https://github.com/apache/pulsar/runs/6997463530. The latest commit is about test file changes. So I think CPP task failed on flaky tests.

LGTM, there is a similar PR, maybe we could refer to this comment https://github.com/apache/pulsar/pull/14953#pullrequestreview-926492820, if we want to upgrade the Trino version to 334, it's better to discuss in dev list, because the Trino 334 needs jdk11.

@gaoran10 Thanks for your inputs! I'm not aware of that PR before. Let me investigate a bit and decide whether to start a thread on dev@ or close this PR :)

@eolivelli thanks for your review and information! That sounds great.
ping @gaoran10 @lhotari @codelipenghui

@codelipenghui @eolivelli @lhotari could you give this patch a final review and merge if no further concern?


Angular Js

Hi Team,
This is jayaprakash ,In your angular live demo page output is not working.
NOTE:Kindly include below script  in your live demo page.

Regards,
jayaprakash K
<EMAIL_ADDRESS>

There's no team here, just me. I think you're referring to a single bin that someone has made, so you'll need to find the original author and get in touch with them.


Use ExplicitThreadLocal to avoid ThreadLocalMap hanging onto expensive objects forever via FJP

fixed #189

Closing this in favor of https://github.com/jbellis/jvector/pull/206


pgbadger stuck during csv parsing

Hello Gilles,
I try to generate a  report with log in csv format.
pgbadger seems stuck with 100% CPU :
[>                        ] Parsed   10270901 bytes of<PHONE_NUMBER> (0.65%), queries: 337, events: 1

A perf top show this perl function :
  84.84%  libperl.so                  [.] Perl_utf8_length

It do a git bisect and it is related to this commit : https://github.com/darold/pgbadger/commit/20712f8339f90a51d87b1dbc2507b664a20a2818
How can I help you to fix this bug?
Thanks

I found the line which is 51420002 characters long :( I can't send you, it contains personal data.

Yes, I run pgbadger with only this line and I can reproduce the problem. I will try to anonymize the line.
Thanks

Ok, I'm closing the issue I think it is not related to pgBadger but to the Text::CSV_XS Perl module. I could be wrong so if you have time to test it using small perl script give it a try.


Mozzila cant play it? eror media loading.

https://drive.google.com/file/d/0B7_M_N3zWT51R1RiaTRFdlVhejQ/view
cant play event i use mozilla in other its work ty for nice script.
other il make it same not work on mozilla
https://gamerse.net/watch.php?vid=aHR0cHM6Ly9kcml2ZS5nb29nbGUuY29tL2ZpbGUvZC8wQjdrQnpaY3huWE91ZVhRMmNHZFpVVTB0WTFVL3ZpZXc=&subs=&img=VHN1Z3Vtb21vLUVwaXNvZGUtMDItU3VidGl0bGUtSW5kb25lc2lhLmpwZw==

This script is still under development, the one you are using is temporary. Please wait for new updates

i have fix this probleme, you need to upload a mp4 file this probleme come with .avi and .mkv , but with mp4 its working fine so you need to upload mp4 video not avi not mkv


Add details (reason) when assigning user to archived

Ticket Description
Issue
Currently, there is a need to enhance the process of assigning users to the archived status in the website backend. The current implementation lacks the necessary details, which makes it difficult to track and understand why a developer has been placed in the archived status.
Proposed Solution
To improve this process, we need to introduce the following details while assigning users to the archived status:

Message: Include a field to capture a brief explanation or reason for putting the developer in the archived status. This will help provide context and assist in future reference or auditing purposes.

Who: Add a field to specify the name of the assignee responsible for assigning the developer to the archived status. This will allow for better accountability and facilitate communication between team members.

Superuser Name: Introduce a field to capture the name of the superuser who is performing the action. This will help identify the authority behind the decision to archive a developer and provide transparency within the system.

ETA: will add if the ticket got approved.

Ticket Description
Issue
Currently, there is a need to enhance the process of assigning users to the archived status in the website backend. The current implementation lacks the necessary details, which makes it difficult to track and understand why a developer has been placed in the archived status.
Proposed Solution
To improve this process, we need to introduce the following details while assigning users to the archived status:

Message: Include a field to capture a brief explanation or reason for putting the developer in the archived status. This will help provide context and assist in future reference or auditing purposes.
Who: Add a field to specify the name of the assignee responsible for assigning the developer to the archived status. This will allow for better accountability and facilitate communication between team members.
Superuser Name: Introduce a field to capture the name of the superuser who is performing the action. This will help identify the authority behind the decision to archive a developer and provide transparency within the system.

ETA: will add if the ticket got approved.

We should store the superuser id and the user archived id not the name.


[Feature request]  Modifier key to switch off new mouse controls

The idea is very simple. Say if I hold the modifier key and move the mouse, then the camera will rotate around Leon like it usually does in the default game. But when I release the key, then moving the mouse will just rotate Leon.
This can be applicable in both ways:
-If new mouse controls are enabled: Then the modifier key will switch the mouse controls to default vanilla controls.
-If new mouse controls are disabled: Then the modifier key will switch the mouse controls to new one.

Sounds like a good idea, similar like in Arma games where you hold ALT key.

Good idea, I think for this it'd probably be good if we could support mouse buttons as well, don't think we have anything for them right now though.

Good idea, I think for this it'd probably be good if we could support mouse buttons as well, don't think we have anything for them right now though.

Yeah, instead of modifier key, we should be able to use the side mouse buttons or the scroll wheel button as modifier if possible.

Had a try at implementing this here: https://github.com/emoose/re4_tweaks/tree/mouseturn-modifier
Seems to work, but camera acts strange after releasing modifier, somehow it stays locked at the position you move it to with modifier held, and doesn't center or anything when mouse-turning, is that something to do with the mouse-turn impl?
Here's a build if anyone wants to try it:
re4_tweaks1.7.4-MouseTurningModifier.zip
You should be able to edit the MouseTurnModifier key to mouse buttons too, MOUSE1 to MOUSE5 (or LMOUSE/MMOUSE/RMOUSE)

Had a try at implementing this here: https://github.com/emoose/re4_tweaks/tree/mouseturn-modifier
Seems to work, but camera acts strange after releasing modifier, somehow it stays locked at the position you move it to with modifier held, and doesn't center or anything when mouse-turning, is that something to do with the mouse-turn impl? I guess mouse-turning probably assumes camera is centered at all times.
Here's a build if anyone wants to try it (modifier set to ALT, you can edit it in dinput8.ini) re4_tweaks1.7.4-MouseTurningModifier.zip
You should be able to edit the MouseTurnModifier key to mouse buttons too, MOUSE1 to MOUSE5 (or LMOUSE/MMOUSE/RMOUSE), not sure if MOUSE4 & MOUSE5 actually work though since my mouse doesn't use them.
All those GetAsyncKeyState calls is probably inefficient though... would be nicer if we could hook somewhere near start of frame, and then could cache GetAsyncKeyState results for all the keys we support, maybe something to look into later on.

I'm gonna test it.
As for the camera not being centered, yeah thats how it has been even with the current mouse panning feature. You can recreate this by going in game, disabling the mouse panning [via the f1 menu] looking around Leon and then enabling the mouse panning option again.
Camera will now be stuck with whatever position it was in.

Just finished testing, yeah the camera behaves weirdly.
It has two problems.

Like you mentioned, the camera doesn't automatically centers itself after releasing the modifier key.
say if you turned Leon to right using mouse and then pressed the modifier key, the camera would instantly pan right to Leon, which makes it unusable at the moment.

Hmm, I guess because mouse position is changed while mouse-turning, when modifier is pressed game finally sees the new mouse position & then updates camera to rotate toward it, hmm...
For centering, could probably just add code that detects if mouse-turn state changed (changed in F1, or modifier pressed/released), and center camera if so, not really sure what best way to change camera is atm though.

Yeah, I struggled with camera moviment when doing the MouseTurning patches.
Maybe you should open a PR with that current branch. Then I'll push some fixes tomorrow if I managed to improve the camera stuff.

All those GetAsyncKeyState calls is probably inefficient though... would be nicer if we could hook somewhere near start of frame, and then could cache GetAsyncKeyState results for all the keys we support, maybe something to look into later on.

Can't we use the WndProcHook to get key events for this case?


ps.lookup doesn't work on AWS Lambda

You're using ps.loopup() from the ps-node package. However ps l doesn't seem to work on AWS Lambda. Is it really needed here?
I ran a few tests here:

@schickling yea; you're right. I was trying to use ps-node to detect if Chrome was already running, but like you noticed, it doesn't work on lambda. I've removed ps-node from my refactor, but will close this Issue only after that's been pushed.

Should add: I expect to finish the second and third checklist items in #4 by late next week—will include removal of ps-node

By the way: Love Graphcool! Pretty stoked I got a Star from it's founder! 😄

Thanks so much @adieuadieu 🙂
You're doing great work with this serverless package. We'll use it for our build-time pre-rendering (in next version of prep)!

@schickling I noticed something which doesn't quite make sense to me:
ps l doesn't work on lambda. However, ps lx does. 😖  Huh?
Also, since April 21st, ps-node now uses ps lx by default—so now ps.lookup() also works.
However Chrome behaves oddly in between innovations. It'll run across a few invocations, and then the process will randomly zombie:
1st invocation:
2017-05-03T08:21:23.897Z	7d66b110-2fd9-11e7-abd1-0fa82bf09592	ps lx
F UID PID PPID PRI NI VSZ RSS WCHAN STAT TTY TIME COMMAND
4 483 1 0 20 0<PHONE_NUMBER>6 ep_pol Ssl ? 0:00 /var/lang/bin/node --max-old-space-size=1229 --max-semi-space-size=76 --max-executable-size=153 --expose-gc /var/runtime/node_modules/awslambda/index.js
0 483 12 1 20 0<PHONE_NUMBER> - R ? 0:00 ps lx

2nd invocation:
2017-05-03T08:21:37.104Z	869a5830-2fd9-11e7-9483-9ff7caf0dfb8	ps lx
F UID PID PPID PRI NI VSZ RSS WCHAN STAT TTY TIME COMMAND
4 483 1 0 20 0 1137084 49408 ep_pol Ssl ? 0:00 /var/lang/bin/node --max-old-space-size=1229 --max-semi-space-size=76 --max-executable-size=153 --expose-gc /var/runtime/node_modules/awslambda/index.js
0 483 14 1 20 0 1159348 143396 ep_pol Ssl ? 0:01 /var/task/headless-chrome/headless_shell --headless --disable-gpu --window-size=1280x1696 --no-sandbox --user-data-dir=/tmp/user-data --hide-scrollbars --enable-logging --log-level=0 --v=99 --single-process --data-path=/tmp/data-path --ignore-certificate-errors --homedir=/tmp --disk-cache-dir=/tmp/cache-dir --remote-debugging-port=9222
1 483 16 14 20 0 0 0 exit Z ? 0:00 [headless_shell] <defunct>
0 483 49 1 20 0<PHONE_NUMBER> - R ? 0:00 ps lx

3rd, 4th, 5th invocation:
2017-05-03T08:21:44.045Z	8abcb102-2fd9-11e7-9955-05d484a94301	ps lx
F UID PID PPID PRI NI VSZ RSS WCHAN STAT TTY TIME COMMAND
4 483 1 0 20 0 1206364 52440 ep_pol Ssl ? 0:00 /var/lang/bin/node --max-old-space-size=1229 --max-semi-space-size=76 --max-executable-size=153 --expose-gc /var/runtime/node_modules/awslambda/index.js
0 483 14 1 20 0 1221284 202544 ep_pol Ssl ? 0:01 /var/task/headless-chrome/headless_shell --headless --disable-gpu --window-size=1280x1696 --no-sandbox --user-data-dir=/tmp/user-data --hide-scrollbars --enable-logging --log-level=0 --v=99 --single-process --data-path=/tmp/data-path --ignore-certificate-errors --homedir=/tmp --disk-cache-dir=/tmp/cache-dir --remote-debugging-port=9222
1 483 16 14 20 0 0 0 exit Z ? 0:00 [headless_shell] <defunct>
0 483 54 1 20 0<PHONE_NUMBER> - R ? 0:00 ps lx

6th invocation:
2017-05-03T08:23:07.546Z	bc762f22-2fd9-11e7-a839-cf7444a4c378	ps lx
F UID PID PPID PRI NI VSZ RSS WCHAN STAT TTY TIME COMMAND
4 483 1 0 20 0 1208000 54308 ep_pol Ssl ? 0:00 /var/lang/bin/node --max-old-space-size=1229 --max-semi-space-size=76 --max-executable-size=153 --expose-gc /var/runtime/node_modules/awslambda/index.js
0 483 14 1 20 0 0 0 exit Zsl ? 0:03 [headless_shell] <defunct>
1 483 16 14 20 0 0 0 exit Z ? 0:00 [headless_shell] <defunct>
0 483 65 1 20 0<PHONE_NUMBER> - R ? 0:00 ps lx

@adieuadieu I was also getting numerous headless_shell zombies. Using the --no-zygote flag solved the issue for me.

Thanks for the tip, @kzawadka!


Search for references to a static method returns too many results

Open the analyzer package, find the class FastUri, select the static method parse, and search for references. The resulting list will include references to the static method parse in the superclass. But, unlike an invocation of an instance method, none of the references to the static method from the superclass can ever result in the method in the subclass being invoked. They should not be in the result set.
@scheglov

https://codereview.chromium.org/2176173004

https://github.com/dart-lang/sdk/commit/dd18f931541b03768f71d0878c502f9ee5f46f5a


[Query] AzureMediaServicesClient support for Azure.Identity

Library name and version
Microsoft.Azure.Management.Media
Query/Question
Hi,
I've been trying to acquire a token using TokenCredential from Azure.Identity.. for example by using DefaultAzureCredential class.
Since it only suppports the ServiceClientCredentials class, are you excepting to support using Identity in order to streamline the developer experience?
It seems confusing.
Environment
Windows 11
.Net 4.8 and .Net 6.
IDE: Visual Studio 17.3.1

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

@MarcoEnxuto, I don't quite get your idea of Since it only suppports the ServiceClientCredentials class, are you excepting to support using Identity in order to streamline the developer experience? DefaultAzureCredential only support the below credentials

https://docs.microsoft.com/en-us/dotnet/api/azure.identity.defaultazurecredential?view=azure-dotnet
I don't see the ServiceClientCredentials  that you are referring to

Hi @xboxeer , i didn't knew you're deprecating Azure Media Services SDK. I guess you're streamiling the developer experience and this looks great. Will provide feedback later. Thanks.


Run doctests and unittest-based tests with pytest

The current hack to run doctests is starting to fail for some reason (see for example https://github.com/lab-cosmo/equistore/actions/runs/4374795802/jobs/7654672493), so let's move to pytest for this.
This PR also uses pytest to run all unittest-based tests, so we only have to change the test syntax to finish #176

:books: Documentation preview :books:: https://equistore--182.org.readthedocs.build/en/182/

The failure of the current hack is is really weird:
 ======================================================================
ERROR: test_doctests (doctests.TestDoctests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/runner/work/equistore/equistore/python/tests/doctests.py", line 16, in test_doctests
    submodule = importlib.import_module("equistore." + module.name)
  File "/opt/hostedtoolcache/Python/3.10.10/x64/lib/python3.10/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1050, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1004, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'equistore.operations.equal_metadata'

The code was trying to all sub-module by looking at all files, and then importing them. Here it is trying to import equistore/operations/equal_metadata.py, which does not exists on master or other impacted branches!!! This file is added in #147, which is not merged yet. This makes no sense to me ...

OK, culprit it 54000b8ae0d5592b1efc25adba141cada05cb278. @jwa7 you pushed that to master =)

I've reverted this commit and added protection to the master branch, this should prevent such accidental pushes

Thanks for adding this!


Add header to example

Yeah I should work on my Git CLI skills I guess

I ended up adding a linter and a script to check that future commit have these.
I fixed this one at the same time, so I'll close this PR.
Thanks for catch that though!


Adding documentation for behavior of indices.recovery.max_bytes_per_sec set to 0mb or less

After a lovely surprise when trying to limit the recovery completely and discovering the opposite was happening I see we need the documentation to include this nice extra info.
Description
missing documentation for setting indices.recovery.max_bytes_per_sec to 0mb or less.
Issues Resolved
Resolves #3648
Checklist

[x] By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license and subject to the Developers Certificate of Origin.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@hdhalter @Naarcha-AWS @dtaivpp who reviews PRs for this repo?

@nknize do you think this can be categorized as a bug in the core project?
The behavior was surprising, if you set tis param to 0mb per second then the behavior is basically "as many mb as possible per second"...
I opened this so it is at least documented somewhere.

@hdhalter @Naarcha-AWS @dtaivpp who reviews PRs for this repo?

@AmiStrn: That would be anyone on the documentation team listed here. I'm taking a look now.

@hdhalter @Naarcha-AWS @dtaivpp who reviews PRs for this repo?

@AmiStrn: That would be anyone on the documentation team listed here. I'm taking a look now.

Thanks for taking a look @Naarcha-AWS


Bump wasmer-wasi to 2.2.0.

Codecov Report

Merging #763 (191d0a6) into main (ba9215c) will decrease coverage by 0.00%.
The diff coverage is n/a.

:exclamation: Current head 191d0a6 differs from pull request most recent head 6eb85ad. Consider uploading reports for the commit 6eb85ad to get more accurate results

@@            Coverage Diff             @@
##             main     #763      +/-   ##
==========================================
- Coverage   72.24%   72.23%   -0.01%
==========================================
  Files          86       86
  Lines       11756    11756
==========================================
- Hits         8493     8492       -1
- Misses       3263     3264       +1

Would this not be done by dependabot?

Yeah this one: https://github.com/containers/youki/pull/764


New version: CloudFoundry.CLI.v8 version 8.7.10

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[x] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the directory's name containing the manifest you're submitting.

Closes #163401
Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Installation failed with exit code -1978335141
2024-07-17 07:31:14.178 [CLI ] Unable to locate nested installer at: C:\Users\VALIDA~1\AppData\Local\Temp\WinGet\CloudFoundry.CLI.v<IP_ADDRESS>\extracted\cf.exe
2024-07-17 07:30:29.915 [CLI ] Unable to locate nested installer at: C:\Users\VALIDA~1\AppData\Local\Temp\WinGet\CloudFoundry.CLI.v<IP_ADDRESS>\extracted\cf.exe

Automated error analysis suggests -1978335141 may mean Unable to locate nested installer. Does the RelativeFilePath have the previous version number in it?
(Automated response - build 886.)

https://github.com/microsoft/winget-pkgs/issues/142854


How can I organise large schemas?

I'm building an ent schema that spans 5 different Postgres DB schemas.
Each DB schema has around 15 tables, and can be inter-related, which is why I'm using the same ent schema for all 5 DB schemas.
But that means around ~50 files in the ent/schema folder.
What's the best way to organise these files?

I have the same issue, managing these files is a terrible experience. Have you found any solutions?

Nope, haven't found any solutions yet. For now, I'm keeping all files in the same schema folder.


Move generic packages from serving to pkg.

I was asked to move this to do some re-use in Eventing.
This is 1 to 1 move with:

one import line change in prober.
removing most of network.go, since it was serving/istio specific.

/cc @chizh
/assign mattmoor @tcnghia

/retest

/test pull-knative-pkg-unit-tests
/test pull-knative-pkg-go-coverage

/test pull-knative-pkg-unit-tests

/lgtm
/approve


[3.0] Patch get service export executor

What is the purpose of the change
#9168

Codecov Report

Merging #9183 (6858e04) into 3.0 (7c7ee62) will decrease coverage by 0.23%.
The diff coverage is 75.00%.

@@             Coverage Diff              @@
##                3.0    #9183      +/-   ##
============================================
- Coverage     64.94%   64.70%   -0.24%
+ Complexity      314      311       -3
============================================
  Files          1185     1185
  Lines         51017    51016       -1
  Branches       7586     7563      -23
============================================
- Hits          33132    33009     -123
- Misses        14288    14401     +113
- Partials       3597     3606       +9

Impacted Files
Coverage Δ

.../threadpool/manager/DefaultExecutorRepository.java
80.85% <75.00%> (+0.42%)
:arrow_up:

...a/org/apache/dubbo/rpc/filter/AccessLogFilter.java
24.35% <0.00%> (-44.88%)
:arrow_down:

...va/org/apache/dubbo/rpc/support/AccessLogData.java
53.16% <0.00%> (-37.98%)
:arrow_down:

...dubbo/common/status/support/LoadStatusChecker.java
46.15% <0.00%> (-15.39%)
:arrow_down:

...ache/dubbo/remoting/transport/AbstractChannel.java
75.00% <0.00%> (-12.50%)
:arrow_down:

.../remoting/transport/netty4/NettyServerHandler.java
58.13% <0.00%> (-9.31%)
:arrow_down:

...e/dubbo/remoting/transport/netty/NettyChannel.java
51.13% <0.00%> (-9.10%)
:arrow_down:

.../common/threadpool/serial/SerializingExecutor.java
70.37% <0.00%> (-7.41%)
:arrow_down:

.../apache/dubbo/remoting/transport/AbstractPeer.java
67.39% <0.00%> (-4.35%)
:arrow_down:

...c/main/java/org/apache/dubbo/rpc/RpcException.java
83.33% <0.00%> (-3.34%)
:arrow_down:

... and 28 more

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 7c7ee62...6858e04. Read the comment docs.


caddy sometimes just exists

Caddy seems to spontaneous exit... Nothing in the logs, it just stops :-( Anyway to debug this?
I do add my custom middleware, so that surely good be it, but then I would expect a panic or something :)
Feb 04 14:08:19 x64 caddy[10216]: xx.xx.xx.xx - [04/Feb/2016:14:08:19 +0000] "GET /image-test.html HTTP/2.0" 200 1188
Feb 04 14:49:33 x64 systemd[1]: Started Caddy HTTP/2 web server.

Are you using HTTPS at all?

[ Quoting<EMAIL_ADDRESS>in "Re: [caddy] caddy sometimes just ex..." ]

Are you using HTTPS at all?

Yep, with let's encrypt certs
/opt/bin/customCaddy -http2 -agree=true -email<EMAIL_ADDRESS>-conf=/opt/etc/Caddyfile
(http2 was a test thing)

See https://caddyserver.com/docs/automatic-https - Caddy will restart on occasion to renew certificates or update OCSP stapling. It gets a new process ID when this happens.
You can use the -pidfile flag to write a PID file: https://caddyserver.com/docs/cli - see that page for the flags recommended in automated environments, like with systemd. Make sure systemd follows the PID file.

[ Quoting<EMAIL_ADDRESS>in "Re: [caddy] caddy sometimes just ex..." ]

See https://caddyserver.com/docs/automatic-https - Caddy will restart on occasion to renew certificates or update OCSP stapling. It gets a new process ID when this happens.

Ah! Still there wasn't a caddy process running, but that might be systemd doing
it weird thin.

You can use the -pidfile flag to write a PID file: https://caddyserver.com/docs/cli - see that page for the flags recommended in automated environments, like with systemd. Make sure systemd follows the PID file.

Check. Had that in the unit file, but caddy couldn't write to /var/run/caddy.
Go figure.
/Miek
--
Miek Gieben

And you can put the PID file anywhere you want for systemd, I think. Anyway, hope that cleared that up.


HOCS-5677: COMP PSU add complaint outcome screen

Is Flow_1l3jiab and Flow_14rll2q useful here? Is there a further ticket to do anything more from this that I am missing?
This probably would be useful to have tests on COMP2_PSU_OUTCOME too?

The flows are used when the final response screen is added
I think COMP2 tests would be valuable, there aren't any at the moment. It may be worth a ticket to add these?

merged in https://github.com/UKHomeOffice/hocs-workflow/pull/884


math/big: printing a MaxPrec big.Float panics if it was set using a hex format string

What version of Go are you using (go version)?
play.golang.org
Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
play.golang.org
What did you do?
https://play.golang.org/p/Ou3piPGDOAF
package main

import (
	"fmt"
	"math/big"
)

func main() {
	flt := &big.Float{}
	flt.SetPrec(big.MaxPrec) // Remove this and it works
	flt.SetString("0x1.5")
	fmt.Printf("%v\n", flt)
}

What did you expect to see?
1.3125

What did you see instead?
%!v(PANIC=Format method: underflow)

Yeah, could probably be handled better? Note that:

MaxPrec = math.MaxUint32 // largest (theoretically) supported precision; likely memory-limited

In practise I'm not even sure MaxPrec is a viable option, and looking at the math/big code seems to be a little under-tested. Not that it's not just printing hex, flt.SetString("1.1") also fails with an OOM on the playground. I looks like a few math/big functions do not handle prec being to MaxPrec.

Mainly I just want to be able to set a big.Float from a hex string and not have it round the value. It's fine to have a "sane" maximum value, but like most people I'm enforcing a maximum number of digits when converting from string, which doesn't translate well to a precision value.
I don't actually need MaxPrec; I just want to be confident that my converted values are accurate to the number of digits entered.

So basically you need the Parse equivalent of what the g flag makes Text does:

A negative precision selects the smallest number of decimal digits necessary to identify the value x uniquely using x.Prec() mantissa bits.

For example you could pass -1 as the precision in big.ParseFloat and it would return a *Float with its precision set to the minimum necessary to represent the string. (Note that ParseFloat takes an uint prec so we can't overload a prec of -1 to mean that, we'll need something different).
As far as I know, this is not possible with the current math/big.Float API.
I don't think it could support any base (if you pass a decimal, for example, it could be periodic in the base math/big uses, so there wouldn't be a "smallest prec that can accurately represent the value").
In base 16 this can be done, but you could also do the computation yourself (from number of digits to precision), and then call SetPrec with that precision.


hubble: Fix misclassification of to-network reply packets

This fixes a bug where Hubble wrongly populated the is_reply field for to-network trace events. This caused the Hubble UI to populate the service map destination ports with ephemeral source ports.

This PR removes the erroneously populated values in the datapath and improves the source documentation and function signature to avoid similar bugs in the future. See individual commits for details.

/test

/test-1.23-net-next

ConformanceKind1.19 received a 403 from <IP_ADDRESS> https://github.com/cilium/cilium/runs/4469822766?check_suite_focus=true
Multicluster had a timeout in allow-all/pod-to-service https://github.com/cilium/cilium/runs/4470717557?check_suite_focus=true
ci-awscni seems stuck somehow

All of these look unrelated, as none of them use health or the improved Jenkins readiness check.

/ci-multicluster

/ci-awscni


[BUG] no meaningful message is surfaced to maven when binary-dedupe fails

Originally posted by @gerashegalov in https://github.com/NVIDIA/spark-rapids/issues/4265#issuecomment-986072583
In some situations when binary-dedupe fails Maven only receives exec exit code via ant, but not the message that is being logged to stderr. It would save time if this message was part of the exception ultimately propagated to Maven

It looks like for all instances of the exec task we might have to do something like

add errorproperty=errorMsg attribute to capture stderr
add resultproperty=exitCode attribute to capture the exit code
and then add a snippet after each exec

<fail message="exec failed  ${errorMsg}">
   <condition>
        <not> <equals arg1="${exitCode}" arg2="0"/></not>
    </condition>
</fail>

unless there is a way to make exec behave like this by default.


[UI Testing] Fail when multiple test targets are present

I have multiple test targets in my project, and collection of data from any non-ui tests fail at
https://github.com/KrauseFx/snapshot/blob/feature/uitests/lib/snapshot/collector.rb#L46, as these tests contain no ActivitySummaries entries (fails with
undefined method `each' for nil:NilClass (NoMethodError)

.)

Thanks, nice catch :+1:


fix: Project Euler MathJax

Checklist:

[x] I have read and followed the contribution guidelines.
[x] I have read and followed the how to open a pull request guide.
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Ref: #57289
I blame Oliver
https://github.com/freeCodeCamp/freeCodeCamp/pull/56749/files#diff-79658a9efce0790ead064be109693f50c41b2ff3ae94cf5ed568e957e42a86d6

@ojeytonwilliams  I am not sure about changing this to a #ref. It would close the initial issue, but not the core problem.


[FEATURE] Conditions on Actor Sheets

There are some abilities in V5 that work like buffs. For example using Host Spirit (adds 2 dice to Physical dice pools and +2 Health) or being a Werewolf in Crinos form (adds 3 dice to Physical dice pools, and probably also +3 to Health). Currently I only know of these as examples, but Horrid Form even allows specific physical traits to be raised.
The idea would be that when you add a "Buff" (or Form), your sheet goes into Buff Setup Mode (all dots vanish) and you can add new dots (preferably in a different color than usual to show they are temporary). When Buff Setup is done, the Buff becomes a toggle that can be activated to add the bonus dice/health boxes to the sheet/rolls.
Currently, I'm using duplicated character sheets for this. The problem is keeping them up to date becomes an impossible thing, over time.
I don't consider this to be high priority. It's just a nice-to have thing that probably adds some issues, but it would maybe help make Werewolves playable too, because currently, I'm having 3 sheets for them.

It would be good to track buffs like effects in DnD. Could be as simple as a list of buffs under specialties or something.

Slightly stealing this issue to encapsulate not only the original intent--an area for "buffs" on the sheets--but also extend it to have a place for conditions in general on the actor sheets.


Add migration waves initial skeleton page & wire up navigation to new menu item

Add the migration waves page

Add new migration waves sidebar menu item with RBAC

Create the migration waves table with the following columns:

Name: Name of the migration wave. This field is optional and could appear empty.
Start date: Start date for the migration wave.
End date: End date for the migration wave.
Application: Placeholder to start with. See #571
Contributors:  Placeholder to start with. See #572

@rromannissen @jortel What are the intended options for the top level bulk toolbar?


if you connect node form button, what will get？

like

it will add some code to editor

Not sure if I understood correctly, but clicking something in editor that generates code like event / signals has nothing to do with the code editor you're using.
Just not sure if the Neovim editor will trigger, I don't really use Godot atm

Not sure if I understood correctly, but clicking something in editor that generates code like event / signals has nothing to do with the code editor you're using.
Just not sure if the Neovim editor will trigger, I don't really use Godot atm

ok, thx i will get this

but this does not affect the use


Newest version in Maven central

I saw that there is a new version (5.1.6) released.
Congratulations!
Are there any plans to also include this release in the Maven central repository?
It seems that the latest version posted there is 3.8.1 which is quite old by now.
Also, minor nitpick:
The README promises that

If you want to use this project as a dependency, you do not have to compile it yourself and can include it in your pom .xml as follows. ...  <Code showing version 4.0.0>

AFAIK, there is no such version.  It would be a good idea to keep this updated to the latest version posted on the Maven repo.

Hey,
starting with version 4.0, we switched from de.rub.nds.tlsattacker to de.rub.nds.tls.attacker. Newer versions, including 5.1.6, have been published under this new group id.

Thank you for reporting this.
The readme.md has been update with release v5.2.1.


Add Chennai Mathematical Institute

http://www.cmi.ac.in/
https://en.wikipedia.org/wiki/Chennai_Mathematical_Institute

@meta-coder Regrettably, CMI web site does not answer now, please let us know if there is another URL where we can see it. Thank you.

Chennai has been hit by a cyclone. Power supply, network connectivity has been affected. The website should be up in a few days when the connectivity gets restored.

@meta-coder O, I regret that you suffered from natural disaster. I am sorry! We'll wait until the website is up, and merge the request as soon as confirm the information in it. Thank you!

Hi @philipto, the site is up now.

@meta-coder Thank you! Pull request merged.


Enrich error message in error ctor when error-type-reference is provided

Description:
Code sample
type ErrorA error;
int|string uc4 = error ErrorA("msg");

Error
ERROR [t3.bal:(2:18,2:37)] incompatible types: expected '(int|string)', found 'error

Expected error
ERROR [t3.bal:(2:18,2:37)] incompatible types: expected '(int|string)', found 'ErrorA'

This error is currently issued from
https://github.com/ballerina-platform/ballerina-lang/blob/8adfc0ac5b12d60c70d33893a15e49e7eba2920e/compiler/ballerina-lang/src/main/java/org/wso2/ballerinalang/compiler/semantics/analyzer/TypeChecker.java#L2799
Describe your problem(s)
Describe your solution(s)
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Getting the expected error on slbeta6. Keeping the issue open to add tests.

Need to also check this comment.

This works as expected in Ballerina 2201.3.0


Unresolved requirements: osgi.wiring.package=io.swagger.models.utils

In a osgi system (a RedHat fuse on the specific), i need to use the library swagger so i have installled with the following commands:
osgi:install -s mvn:io.swagger/swagger-annotations/1.5.21
osgi:install -s mvn:io.swagger/swagger-core/1.5.21
osgi:install -s mvn:io.swagger/swagger-jaxrs/1.5.21
osgi:install -s mvn:io.swagger/swagger-models/1.5.21

my bundle but  i get this error when try to install other bundle:
] osgi.wiring.package; (osgi.wiring.package=io.swagger.models.utils) Unresolved requirements: [[XXXXXXXXXXXXXXXXXXXXX [387](R 387.44)] osgi.wiring.package; (osgi.wiring.package=io.swagger.models.utils)]

it's a  error on the swagger-models MANIFEST ? like the following code?
<Export-Package>
  io.swagger.models,
  ...
  io.swagger.models.utils //is this missing?
</Export-Package>

Thanks for reporting this, fixed in #3319, available in latest snapshot and upcoming release.


Can you clearly explain how to use Cookies with the latest version of ModernHttpClient?

Hi - Would like to see know what's the most up to date way that we can use cookies in ModernHttpClient in Xamarin (iOS/Android)? Documentation on this is sparse and seems to have changed:
Our Code:
We are using ModernHttpClient to access a web service which uses Forms Authentication so we have a 2-step process below to get the client to authenticate and then further use that authenticated cookie based instance to get data from the service

In Xamarin.ios or Xamain.Android code, Create an instance of ModernHttpClient and use it to POST to the authentication page in the web service and get a cookie that contains a session token :

 //Code to Authenticate and get a cookie with session token:
			NativeCookieHandler cookieHandler = new NativeCookieHandler ();
			NativeMessageHandler messageHandler = new NativeMessageHandler (false, false, cookieHandler);
			ProjectClient = new HttpClient (messageHandler);

			string url = ServiceURL.DSISServiceEndpointUrl;
			Uri uri = new Uri (url);

			var keyValues = new List<KeyValuePair<string, string>> ();
			keyValues.Add (new KeyValuePair<string, string> ("loginForm=loginForm&j_username", username));
			keyValues.Add (new KeyValuePair<string, string> ("&j_password", username));

			HttpContent content = new FormUrlEncodedContent (keyValues);
			var response =	await ProjectClient.PostAsync (url, content);
			IEnumerable<Cookie> responseCookies = cookieHandler.Cookies;
			return responseCookies.FirstOrDefault ();

2 - The previous code retrieves the proper cookie we want - But we can't seem to get the resulting calls to actually use it to get data- So the question is - How do we use or set that particular cookie in ModernHttpClient for subsequent GET requests? Should we create a new instance of the HttpClient and pass in the cookie?, should it  not automatically use the NativeCookieHandler that we used to authenticate? -Please elaborate -
Thanks!!!

FYI.. In Xamarin Forums we can also see other users mention issues with using cookies and ModernHttpClient -e.g. this link:https://forums.xamarin.com/discussion/42410/bug-httpclient-times-out-when-redirected-on-a-post-request

FY2 - I was able to pass in the authentication cookie manually in the request header as described here https://github.com/paulcbetts/ModernHttpClient/issues/118
.. so this works for my code now -still it appears that cookie handling features are broken -either a fix or better documentation is appreciated
Btw  - thanks for this library -its really very, very helpful for those of use using Xamarin!!


how i can resolve this error..Failed to compile ./src/practicee.js Module not found: Can't resolve 'react-native' in 'D:\ofc\firstapp\src'

Bug Report or Feature Request (mark with an x)
- [ ] bug report -> please search issues before submitting
- [ ] feature request

Area
- [ ] devkit
- [ ] schematics

Versions

Repro steps

The log given by the failure

Desired functionality

Mention any other details that might be useful

Empty issue, closing.


Data and Batch should be interchangeable

A Data should have the same attributes as a Batch object.
This is because our scripts and training/evaluation functions run on Batch objects, but we frequently may want evaluate on one sample (Data) only.
For example,
    smt = to_dense_batch(batch.start_meshes_transformed, batch.start_meshes_transformed_batch)[0]
AttributeError: 'Data' object has no attribute 'start_meshes_transformed_batch'

I just got the above error, which works fine in training when "batch" is a Batch object, but not in evaluation when I want to use a Data object for "batch".

This is a design decision, similar to how PyTorch fails when the batch dimension is missing. I suggest to convert your Data to a Batch first (via Batch.from_data_list([data]), or to manually add missing attributes by yourself, e.g., data.batch = torch.zeros(data.num_nodes, dtype=torch.long).


temp fix for authentication (need to make it configurable).

Seangchan, there is no FacOps role. The correct thing is
group: "facops"
role: "web-service"
See it here to be on the safe side :-)
https://github.com/dmwm/WMCore/blob/master/src/couchapps/ReqMgr/validate_doc_update.js#L42

Actually this is current reqmgr config sets.
https://github.com/dmwm/deployment/blob/master/reqmgr/ReqMgrConfig.py#L44
https://github.com/dmwm/WMCore/blob/master/src/python/WMCore/HTTPFrontEnd/RequestManager/ReqMgrConfiguration.py#L78
I will double check and put it everywhere. :-)

Alan, you are right I clean up the roles and groups and updated the patch.
Basically only reqmgr/admin and web-service/facops can modify any field without updating status. All other roles has to update the status go modify something.

Thanks Seangchan! These permissions are still a bit confusing in my opinion, but we can improve it in the next cmsweb cycle maybe :)

Yes, I agree. This need to be redone.