xlib11-dev, libopenblas-dev, and liblapack-dev installed but not found

Expected Behavior
Build via Android Studio cmake of a shared library .so file in a Android Studio project directory of my own jni_face_detection.cpp file which is a rewrite of webcam_face_pose_ex.cpp. Right now I use zero dlib features. I just initialize fields of dlib components but I don't use them. What should happen is a clear build without errors.
Current Behavior
I hit the build button of Android Studio, it executes the following build.gradle file of my module:
apply plugin: 'com.android.feature'

task deleteSOs(dependsOn: build) { // Remove LIB_DIR content to avoid gradle build errors having "More than one file was found with OS independent path 'lib/...'"
    delete "src/main/jniLibs/"
}

android {
    compileSdkVersion minimalSDK.toInteger()
    defaultConfig {
        minSdkVersion minimalSDK.toInteger()
        targetSdkVersion maximalSDK.toInteger()
        versionCode 1
        versionName appVersion.toString()
        externalNativeBuild { // https://developer.android.com/studio/projects/gradle-external-native-builds
            cmake { // https://developer.android.com/ndk/guides/cmake
                arguments "-DANDROID_STL=c++_shared",
                          "-DANDROID_CPP_FEATURES=rtti exceptions",
                          "-DANDROID_TOOLCHAIN=clang"
                abiFilters abi2Build.toString() // Denotes what ABI is build
            }
        }
        ndk { // https://developer.android.com/studio/projects/gradle-external-native-builds
            abiFilters abi2Build.toString() // Denotes what ABI is packed
        }
    }
    splits.abi { // Make sure this is in every module!
        enable true
        reset() // Use reset() and include for specific ABIs
        include abi2Build.toString()  //select ABIs to build APKs for
    }
    buildTypes {
        release {
            minifyEnabled shrinkAndMinify.toBoolean()
        }
    }
    compileOptions {
        sourceCompatibility JavaVersion.VERSION_1_8
        targetCompatibility JavaVersion.VERSION_1_8
    }
    buildToolsVersion buildToolVersion.toString()
    sourceSets {
        main {
            jniLibs.srcDirs "src/main/jniLibs" // https://stackoverflow.com/questions/44954122/more-than-one-file-was-found-with-os-independent-path-lib-x86-libusb-so/44962630#44962630
            jni.srcDirs = [] //disable automatic ndk-build call
        }
    }
    externalNativeBuild {
        cmake {
            path '../../dlib-19.16/examples/CMakeLists.txt'
        }
    }
}

dependencies {
    implementation 'com.android.support:appcompat-v7:' + androidSupportVersion.toString()
    implementation 'com.android.support:design:' + androidSupportVersion.toString()
    implementation 'com.android.support:recyclerview-v7:' + androidSupportVersion.toString()
    /*implementation'com.android.support:preference-v7:' + androidSupportVersion.toString()*/
    implementation 'com.android.support.constraint:constraint-layout:1.1.3'
    implementation 'com.google.ar:core:' + arCoreVersion.toString() // = 1.6.0
    implementation 'com.google.ar.sceneform:core:' + arCoreVersion.toString()
    implementation 'com.google.ar.sceneform.ux:sceneform-ux:' + arCoreVersion.toString()
    implementation project(':base')
    implementation files('libs/xz.jar')
}

which calls with the installed Android Studio cmake 3.6.0 my CMakeLists.txt file:
cmake_minimum_required(VERSION 3.6.0)                              # RTFM of cmake.org, https://cmake.org/cmake/help/v3.6/ , Andriod Studio had no higher version
project(face_detection)

set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=c++11 -Wall -Werror") # https://github.com/tzutalin/dlib-android/blob/master/CMakeLists.txt
set(USE_AVX_INSTRUCTIONS ON CACHE BOOL "Use AVX instructions")     # -https://stackoverflow.com/a/51628855
set(DLIB_WRAPPER_NAME   "jni_face_detection")                      # Wrapper class used by Java and JNI
set(DLIB_NATIVE_NAME    "dlib_native")                             # Final library name
set(OpenCV_INCLUDE_DIRS ${CMAKE_CURRENT_SOURCE_DIR}/../../opencv-4.0.1-android-sdk/OpenCV-android-sdk/sdk/native/jni)
set(LIB_DIR             ${CMAKE_CURRENT_SOURCE_DIR}/../../arnatomy/controller/src/main/jniLibs/)

if(IS_DIRECTORY ${LIB_DIR})                                        # Only build if the directory is missing, much faster Android Studio build but if you change any .cpp file you also have to delete the dir
    message("Omitted native build since ${LIB_DIR} exists.")
else()
   option(DLIB_NO_GUI_SUPPORT     "" OFF)
   option(DLIB_ENABLE_STACK_TRACE "" OFF)
   option(DLIB_USE_BLAS           "" ON)
   option(DLIB_USE_LAPACK         "" ON)
   option(DLIB_USE_CUDA           "" OFF)
   option(DLIB_USE_MKL_SEQUENTIAL "" OFF)
   option(DLIB_PNG_SUPPORT        "" OFF)
   option(DLIB_GIF_SUPPORT        "" OFF)
   option(DLIB_JPEG_SUPPORT       "" OFF)
   option(DLIB_LINK_WITH_SQLITE3  "" OFF)
   option(DLIB_USE_FFTW           "" ON)
   option(DLIB_USE_MKL_FFT        "" ON)
   option(DLIB_ENABLE_ASSERTS_STR "" OFF) # Decleared before add_subdirectory the options overwrite others https://stackoverflow.com/a/14067342
   add_subdirectory(../dlib dlib_build) # check this!!!

    add_library(${DLIB_NATIVE_NAME} SHARED "${DLIB_WRAPPER_NAME}.cpp") # Decleare what library is build
    set_target_properties(${DLIB_NATIVE_NAME}
                          PROPERTIES
                          LIBRARY_OUTPUT_DIRECTORY
                          "${LIB_DIR}/${ANDROID_ABI}")             # Output directory of final, generated library

    find_library(ANDROID_LOG_LIB log)                              # Include Android libs: https://stackoverflow.com/a/47803975/10030695
    if(NOT ANDROID_LOG_LIB)
      message(FATAL_ERROR "android log library not found")
    endif()

    include(${OpenCV_INCLUDE_DIRS}/OpenCVConfig.cmake)
    target_include_directories(${DLIB_NATIVE_NAME} ${OpenCV_INCLUDE_DIRS} include) # https://cmake.org/cmake/help/v3.6/command/target_include_directories.html
    target_link_libraries(${DLIB_NATIVE_NAME} dlib::dlib ${ANDROID_LOG_LIB})

    #add_executable(${DLIB_WRAPPER_NAME} ${DLIB_WRAPPER_NAME}.cpp)
    #add_executable(webcam_face_pose_ex webcam_face_pose_ex.cpp)
    #add_executable(face_landmark_detection_ex face_landmark_detection_ex.cpp)
    #add_executable(train_shape_predictor_ex train_shape_predictor_ex.cpp)
endif()

which produces the following output in Android Studio:
Error configuring CMake server (./Android/Sdk/cmake/<IP_ADDRESS>88404/bin).
Check for working C compiler: ./Android/Sdk/ndk-bundle/toolchains/llvm/prebuilt/linux-x86_64/bin/clang
Check for working C compiler: ./Android/Sdk/ndk-bundle/toolchains/llvm/prebuilt/linux-x86_64/bin/clang -- works
Detecting C compiler ABI info
Detecting C compiler ABI info - done
Detecting C compile features
Detecting C compile features - done
Check for working CXX compiler: ./Android/Sdk/ndk-bundle/toolchains/llvm/prebuilt/linux-x86_64/bin/clang++
Check for working CXX compiler: ./Android/Sdk/ndk-bundle/toolchains/llvm/prebuilt/linux-x86_64/bin/clang++ -- works
Detecting CXX compiler ABI info
Detecting CXX compiler ABI info - done
Detecting CXX compile features
Detecting CXX compile features - done
Using CMake version: 3.10.2
Compiling dlib version: 19.16.0
Looking for pthread.h
Looking for pthread.h - found
Looking for pthread_create
Looking for pthread_create - found
Found Threads: TRUE
 *****************************************************************************
 *** DLIB GUI SUPPORT DISABLED BECAUSE X11 DEVELOPMENT LIBRARIES NOT FOUND ***
 *** Make sure libx11-dev is installed if you want GUI support.            ***
 *** On Ubuntu run: sudo apt-get install libx11-dev                        ***
 *****************************************************************************
Searching for BLAS and LAPACK
Searching for BLAS and LAPACK
Found PkgConfig: /usr/bin/pkg-config (found version "0.29.1")
Checking for module 'cblas'
  No package 'cblas' found
Checking for module 'lapack'
  Found lapack, version 0.3.3+ds
Looking for sys/types.h
Looking for sys/types.h - found
Looking for stdint.h
Looking for stdint.h - found
Looking for stddef.h
Looking for stddef.h - found
Check size of void*
Check size of void* - done
 *****************************************************************************
 *** No BLAS library found so using dlib's built in BLAS.  However, if you ***
 *** install an optimized BLAS such as OpenBLAS or the Intel MKL your code ***
 *** will run faster.  On Ubuntu you can install OpenBLAS by executing:    ***
 ***    sudo apt-get install libopenblas-dev liblapack-dev                 ***
 *** Or you can easily install OpenBLAS from source by downloading the     ***
 *** source tar file from http://www.openblas.net, extracting it, and      ***
 *** running:                                                              ***
 ***    make; sudo make install                                            ***
 *****************************************************************************
Building a C++11 test project to see if your compiler supports C++11
C++11 activated.
Found OpenCV: ./Application/opencv-4.0.1-android-sdk/OpenCV-android-sdk (found version "4.0.1")
CMake Error at CMakeLists.txt:42 (target_include_directories):
  target_include_directories called with invalid arguments

Configuring incomplete, errors occurred!
See also "./Application/arnatomy/controller/.externalNativeBuild/cmake/debug/arm64-v8a/CMakeFiles/CMakeOutput.log".

I know it says xlib11-dev is missing, but I installed it with
sudo apt-get install libx11-dev

and a rerun delivers:
Reading package lists... Done
Building dependency tree
Reading state information... Done
libx11-dev is already the newest version (2:1.6.7-1).
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.

However, cmake from Android Studio does not find it. Similar to blas:
sudo apt-get install libopenblas-dev liblapack-dev

and rerun gives:
Reading package lists... Done
Building dependency tree
Reading state information... Done
liblapack-dev is already the newest version (3.8.0-1build1).
libopenblas-dev is already the newest version (0.3.3+ds-1).
0 upgraded, 0 newly installed, 0 to remove and 0 not upgraded.

However, cmake from Android Studio does not find only one and strangely the Android Studio output says:
Checking for module 'cblas'
  No package 'cblas' found
Checking for module 'lapack'
  Found lapack, version 0.3.3+ds

and you can see that Android Studio says0.3.3+ds should be the version of lapack but the console output says this is the version of libopenblas-dev.
Steps to Reproduce
I guess this problem is very project specific, but it is Android Studio 3.2.1, dlib 19.6, and OpenCV for android 4.0.1, cmake 3.6.0, on Ubuntu 18.10 x64. I got dlib from this very github repository.
So how can I let cmake find xlib11-dev, libopenblas-dev, and liblapack-dev?

No response after one month, is it really that hard? I still have no solution.

Maybe no one knows the answer to this question, I certainly don't.  I'm not familiar with android or its build tooling.

However, thanks for your reply.

I downloaded X11 lib manually and replaced your include(FindX11) in the file CMakeLists.txt on line 388 with:
set(X11_FOUND 1)
set(X11_INCLUDE_DIR MY_ABSOLUTE_PATH/libX11-<IP_ADDRESS>-mingw32/include)
set(X11_LIBRARIES MY_ABSOLUTE_PATH/libX11-<IP_ADDRESS>-mingw32/lib)

Only then the build of dlib with the usage of some methods from webcam_face_pose_ex.cpp worked. Even when I added those two include and lib paths in my local /usr/share/cmake-3.12/Modules/FindX11.cmake file dlib using cmake, both, were  unable to locate the above files even when the paths were given. The usage of CMAKE_PREFIX_PATH had also no effect. I had to hardcode the linking of X11 which is definitive an error. I only write this here to give future programmers a quick and dirty local workaround.

If you don't need to write ui in the native layer, is it not necessary to compile x11?
Openblas is used to speed up the code, I think this is necessary, I used openblas when compiling mxnet, I need to first compile openblas into a static library through ndk, then link it when compiling mxnet.
Now I am also looking for ways to add openblas to dlib on android.

    #-DANDROID_ABI="armeabi-v7a with NEON" \
${cmake} --build . --config Release --target install -- -j16

I am trying to build openblas for android for dlib  but getting below error.
can you please help?
CMake Error at /media/newhd_g/ADTLATEST/sdk_linux/cmake/3.6.4111459/share/cmake-3.6/Modules/CMakeTestFortranCompiler.cmake:54 (message):
The Fortran compiler "/usr/bin/f95" is not able to compile a simple test
program.
It fails with the following output:
Change Dir: /media/newhd_f/Samples/dlib-face-recognition-android-master/OpenBLAS-0.3.7/android_build/CMakeFiles/CMakeTmp
Run Build Command:"/usr/bin/make" "cmTC_0ceac/fast"
/usr/bin/make -f CMakeFiles/cmTC_0ceac.dir/build.make
CMakeFiles/cmTC_0ceac.dir/build
make[1]: Entering directory
'/media/newhd_f/Samples/dlib-face-recognition-android-master/OpenBLAS-0.3.7/android_build/CMakeFiles/CMakeTmp'
Building Fortran object CMakeFiles/cmTC_0ceac.dir/testFortranCompiler.f.o
/usr/bin/f95
--sysroot=/media/newhd_g/ADTLATEST/sdk_linux/ndk-bundle/toolchains/llvm/prebuilt/linux-x86_64/sysroot
-fPIE -c
/media/newhd_f/Samples/dlib-face-recognition-android-master/OpenBLAS-0.3.7/android_build/CMakeFiles/CMakeTmp/testFortranCompiler.f
-o CMakeFiles/cmTC_0ceac.dir/testFortranCompiler.f.o
Linking Fortran executable cmTC_0ceac
/media/newhd_g/ADTLATEST/sdk_linux/cmake/3.6.4111459/bin/cmake -E
cmake_link_script CMakeFiles/cmTC_0ceac.dir/link.txt --verbose=1
/usr/bin/f95
--sysroot=/media/newhd_g/ADTLATEST/sdk_linux/ndk-bundle/toolchains/llvm/prebuilt/linux-x86_64/sysroot
-Wl,--exclude-libs,libgcc.a -Wl,--exclude-libs,libatomic.a
-static-libstdc++ -Wl,--build-id -Wl,--warn-shared-textrel
-Wl,--fatal-warnings -Wl,--exclude-libs,libunwind.a -Wl,--no-undefined
-Qunused-arguments -Wl,-z,noexecstack -Wl,--gc-sections
CMakeFiles/cmTC_0ceac.dir/testFortranCompiler.f.o -o cmTC_0ceac
f95: error: unrecognized command line option ‘-Qunused-arguments’; did
you mean ‘-Wunused-parameter’?
CMakeFiles/cmTC_0ceac.dir/build.make:97: recipe for target 'cmTC_0ceac'
failed
make[1]: *** [cmTC_0ceac] Error 1
make[1]: Leaving directory
'/media/newhd_f/Samples/dlib-face-recognition-android-master/OpenBLAS-0.3.7/android_build/CMakeFiles/CMakeTmp'
Makefile:126: recipe for target 'cmTC_0ceac/fast' failed
make: *** [cmTC_0ceac/fast] Error 2
CMake will not be able to correctly generate this project.
Call Stack (most recent call first):
cmake/f_check.cmake:24 (enable_language)
cmake/prebuild.cmake:87 (include)
cmake/system.cmake:157 (include)
CMakeLists.txt:67 (include)
-- Configuring incomplete, errors occurred!
See also "/media/newhd_f/Samples/dlib-face-recognition-android-master/OpenBLAS-0.3.7/android_build/CMakeFiles/CMakeOutput.log".

    #-DANDROID_ABI="armeabi-v7a with NEON" \
${cmake} --build . --config Release --target install -- -j16

disable x11 by -DDLIB_NO_GUI_SUPPORT=ON
link openblas by -DBLAS_LIBRARIES=${OPENBLAS_HOME}/lib/libopenblas.a

can you please check my error?


A question about onDone callback

Thanks for your working on this awesome lib.
I'm not that familiar with the streaming api, just tested a few cases, in the case, the onDone function randomely stopped.
. I guess controller.close() will terminate the whole process, even if the onDone is running?
https://github.com/SpellcraftAI/openai-streams/blob/07182c7313/src/lib/streaming/streams.ts#L57
So, should we invoke the onDone before the controller.close(); ？

onDone() only fires once, when the stream has finished. controller.close() will not interrupt it, it's just an event listener basically.

@daimajia you are right. This is not visible during local testing, but it will not work on Vercel.  So, if you are doing async work in onDone method, they won't get executed

if you are using Vercel, I suggest trying their AI library. onFinal works as expected:
https://sdk.vercel.ai/docs/api-reference/openai-stream
export async function POST(req: Request) {
  const { messages } = await req.json();
  // Create a chat completion using OpenAI
  const response = await openai.chat.completions.create({
    model: 'gpt-4',
    stream: true,
    messages,
  });

  // Transform the response into a readable stream
  const stream = OpenAIStream(response, {
      onFinal: async (content) => {
        // run your async logic here
      }
  });

  // Return a StreamingTextResponse, which can be consumed by the client
  return new StreamingTextResponse(stream);
}

thx @mixasite !


HDDS-7484. Snapshot ID on followers should match the one on the OM Leader

What changes were proposed in this pull request?
Currently Ozone manager leader and followers create SnapshotId (1 and 2) when they receive the request which causes different SnapshotIds among leader and followers. This change is to make sure that the leader and all followers have the same SnapshotId.
Proposed change is adds new optional attribute SnapshotId in CreateSnapshotRequest. Ozone manager, whichever receives the request, will update the request and add the SnapshotId. Updated request will be used to process the request and passed to other OM nodes. SnapshotId is optional and clients are not supposed to set it. Even if clients do, it will overridden by the OM at server side and new SnapshotId will returned in SnapshotInfo as part of the response.
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-7484
How was this patch tested?

Tested manually by added a log statement.
Added unit test.

There are still multiple CI  failure. Can  you validate that these failures are not related to the changes here ?

There are still multiple CI failure. Can you validate that these failures are not related to the changes here ?

Yes, it is related to my change. One check got switch in refactoring.

Thanks @hemantk-12 for the patch. @prashantpogde for reviewing.


Feat: Add Comprehensive Documentation for Monitoring and Logging

Describe the feature
adds comprehensive documentation for the Monitoring and Logging section of HelpOps-Hub. It includes guides on setting up and using Prometheus, Grafana, ELK Stack, and Graylog, along with best practices for effective monitoring and logging. This enhancement aims to assist DevOps professionals in maintaining application health, optimizing performance, and ensuring robust security.
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[ ] I'm a VSoC'24 contributor
[X] I want to work on this issue

@mdazfar2  Please assign me this issue

@arunimaChintu, why are you raising the same issue again and again? Sorry, I have to close this one as well. Now you have only one open issue that you must complete. :)


Investigate research for a zeebe modeler prototype

While refactoring the camunda-modeler to our new architecture (next-branch) it is also a main goal to make it expandable for new kind of modelers (e.g. Zeebe-Modeler).
According to it we have to create a core from the new modeler's architecture which other projects can use to build there all kind of modelers. This issue includes to have a detailled look at the new architecture and discover which components are likely to move them into a new core-project.

[x] Check out existing zeebe-modeler
[x] Examine requirements for a new zeebe-modeler
[x] Indentify missing configuration options in current modeler's architecture

Documentation related things can be found here.

Investigation is done in this place, further prototype implementation is going with https://github.com/camunda/camunda-modeler/issues/1031


Switch to VFile for file management

https://github.com/wooorm/vfile
Provides an interface for managing files virtually.

:+1: Ping me if I can help. An alternative is of course Vinyl, I’ve outlined the difference in VFile’s readme, but let me know if I can explain more in-depth.

VFile is different from (the excellent :+1:) vinyl in that it does not include file-system or node-only functionality. No buffers, streams, or stats.

Ah, I believe Metalsmith requires Buffers. So maybe Vinyl is the way to go? Could we extend VFile to allow for Buffers?

Yup, I would support the switch from file.content, which is currently a string, to buffer.
In essence, that would mean file.content.toString(file.encoding) (file.encoding not yet existing) should be used everywhere.

So as I understand it, aside from adding some nice convenience methods to the API, this would also address #58.
On the other hand, it also has the downsides discussed in #58 as well, mainly a significant loss of simplicity in Metalsmith's API. Additionally, it would also be a major breaking change, with all plugins having to be rewritten to support the new API (or at the very least, a compatibility system needing to be put in place to run older plugins).
I guess I'm just not sure I see the benefit of a change like this. What does it get us?

I've thought about this a bit more, and while I still don't think "switching to VFile" would be a good move for Metalsmith (both because it totally breaks backwards compatibility, and because it complicates the currently dead-simple files object), it seems totally feasible to provide an object that wraps the files object and provides a VFiles-like interface to it.
If you want to do something like that, I say go for it! It doesn't even need to be in the Metalsmith core, just a library that plugins can use. You might not be able to implement all the features of VFile, but you could probably get about 80% of them. Thoughts?

@Ajedi32 I agree, I don't see the benefits and a simple plugin could provide filename tracking information if that were a requirement for someone.


fix(chat): add optimistic update for new entity creation (Issue #725)

Issue: https://github.com/epam/ai-dial-chat/issues/725

/deploy-review


feature: Try out Landmine (PR Mutation Testing)

Description:
Try out mutation testing.. will it work on this PR?  YEAH!
The idea of these "landmines" (mutation testing) is that you are checking if things that make code invalid will trigger the tests and catch these errors.

If the invalid code is caught by tests and they error, then we pass the "landmine" and the action auto-resolves the comment and passes the pr-landmine action.
If the invalid code is not caught be the tests, then the "landmine" is not resolved and the action fails

Q: Should we make this action required?
Related:
closes #15
Visual:
TBD
TODO:

[ ] Updated README documents
[ ] Complete PR Body

closing and re-opening after fixing # nothing bomb to resolve

closing and re-opening after fixing # nothing bomb to resolve

landmine on comment?

landmine on comment?

looks like you can comment on the suggestion and that will re-run the landmine action with any edited changes to the original comment, so that will work!

looks like you can comment on the suggestion and that will re-run the landmine action with any edited changes to the original comment, so that will work!

!merge

!merge


NoClassDefFoundError: javax/xml/bind/JAXB

I just downloaded the app and when i start it with java -jar "CG Local.jar" i get the following exception

Exception in thread "FXLauncher-Thread" java.lang.NoClassDefFoundError: javax/xml/bind/JAXB
at fxlauncher.AbstractLauncher.syncManifest(AbstractLauncher.java:210)
at fxlauncher.AbstractLauncher.updateManifest(AbstractLauncher.java:92)
at fxlauncher.Launcher.lambda$start$0(Launcher.java:137)
at java.base/java.lang.Thread.run(Unknown Source)
Caused by: java.lang.ClassNotFoundException: javax.xml.bind.JAXB
at java.base/jdk.internal.loader.BuiltinClassLoader.loadClass(Unknown Source)
at java.base/jdk.internal.loader.ClassLoaders$AppClassLoader.loadClass(Unknown Source)
at java.base/java.lang.ClassLoader.loadClass(Unknown Source)
... 4 more

And the application freezes with spinning dots

Hmm, that's weird. What operating system are you using?

Another bug was recently issues via the forums, which led me to the conclusion that CG Local doesn't work well with OpenJDK at the moment. Are you by any chance using OpenJDK?

Im using the default oracle jdk SE 1.8.0_73
For me this sounds like  a simple missing dependency
I just redownloaded the app from https://jaspervanmerle.com/cg-local/files/CG Local.jar and now i get a 'Error during Create Application phase' Dialog

java.lang.ClassNotFoundException: com.jaspervanmerle.cglocal.CGLocal
at java.net.URLClassLoader.findClass(URLClassLoader.java:381)
at java.lang.ClassLoader.loadClass(ClassLoader.java:424)
at java.lang.ClassLoader.loadClass(ClassLoader.java:357)
at fxlauncher.AbstractLauncher.createApplicationEnvironment(AbstractLauncher.java:170)
at fxlauncher.Launcher.lambda$start$0(Launcher.java:149)
at java.lang.Thread.run(Thread.java:745)

What operating system are you using? There is an open issue in the FXLauncher repository (FXLauncher is used for auto-updating and initially downloading all the required files) about the same type of error, which is apparently Mac specific.

I was running it on Win10

Hmm, I don't know exactly why it fails yet, but it's a Java version issue. It indeed doesn't seem to work with 1.8.0_73, but works fine with 1.8.0_151.
I will attempt to fix the issue tomorrow.

works much better with a version that is not 2 years old ;)

also just appeared with
Java(TM) SE Runtime Environment (build 9.0.1+11)
and yes i know it is also not the newest version

Regarding the JAXB error, I opened an issue here: https://github.com/edvin/fxlauncher/issues/100
The second error seemed like an SSL error, so I might be able to fix that one myself. I don't know for sure though, and am pretty busy for the next few days, but it's still on my todo list.

No rush, i currently got a working version, just wanted to inform you

I just removed the dependency on fxlauncher in the development branch, so this issue won't appear anymore.


Change the way user guide is build to have improved search

Related to: https://github.com/ministryofjustice/cloud-platform/issues/1470
To improve the layout of user guide to have better search feed. Split long pages to shorter one and placed inside folders
Kept one index page for landing and included all sections as part of the landing page
Added erb files to documentation/sub folders which opens as a seperate page but still included in the search items
Moved the documentation/Markdown files to .erb.
Have to discuss with the team to get feedback and to apply for the rest of the user guide.

I like this a lot. The search function is a lot more usable.
The left-hand side panel is a bit sparse. Is there any way to have more items in there (i.e. another level of indexing)?

Duplicate PR. Work completed on another PR.


Generated package size

The generated packages seem to be very large
5:04
Uploading robotframework_browser-1.1.1-py3-none-any.whl
100%|██████████| 5.13M/5.13M [00:51<00:00, 104kB/s]
Uploading robotframework-browser-1.1.1.tar.gz
100%|██████████| 4.65M/4.65M [00:24<00:00, 195kB/s]
5:06
0.15.1 versions have been around 500kB
5:07
1.0.0 is about 4.5MB
5:07
.. so might be that there is some garbage in the build

Fixed


chore: update dependencies

Signed-off-by: Anish Ramasekar<EMAIL_ADDRESS>
Reason for Change:

Requirements

[ ] squashed commits
[ ] included documentation
[ ] added unit tests and e2e tests (if applicable).

Issue Fixed:

Does this change contain code from or inspired by another project?

[ ] Yes
[ ] No

If "Yes," did you notify that project's maintainers and provide attribution?
Special Notes for Reviewers:

Codecov Report

Merging #956 (7b7fead) into master (7772b90) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #956   +/-   ##
=======================================
  Coverage   59.13%   59.13%
=======================================
  Files          10       10
  Lines        1062     1062
=======================================
  Hits          628      628
  Misses        399      399
  Partials       35       35


feat: allow passing client options

What kind of change does this PR introduce?
Allow passing of client options for nextjs and remix https://github.com/supabase/auth-helpers/issues/420
What is the current behavior?
Client options can´t be passed.
What is the new behavior?
client options can be passed:
import { createBrowserSupabaseClient } from '@supabase/auth-helpers-nextjs';

const supabaseClient = createBrowserSupabaseClient({
	options: {
		...
	}
});

The auth key is omitted as it is set internally by the helper

@david-plugge if this is ready, can you run changeset for it? https://github.com/supabase/auth-helpers/blob/main/development.md#preparing-a-release


⚠ Bofrost.gr has degraded performance

In 40086cf, Bofrost.gr (https://www.bofrost.gr) experienced degraded performance:

HTTP code: 200
Response time: 2460 ms

Resolved: Bofrost.gr performance has improved in 1938020.


New functions for WebAssembly package

New functions:

Getter for cookies

Document.Cookie()

Getter and setter for title

Document.Title()
Document.SetTitle("New title")

document.writeln function (document.write + '\n')

Document.Writeln("String with a newline")

Setter and getter for text content

Node.TextContent()
Node.SetTextContent("Some text")

Setter and getter for inner text

Node.InnerText()
Node.SetInnerText("Very cool text")

I removed some important stuff from fork so I'll better close it and send a new branch


Double HP bars if too many players

I honestly think this was a mix of the bug seen in https://github.com/apetavern/grubs/issues/404 and a player (it was Borris IIRC) leaving mid-game. Both of these should be fixed by now, but this scenario is hard to test by myself thoroughly, so I'll leave this open until after the next playtest.


Type SparkFunSuite in package org.bdgenomics.utils.misc is not available

Not sure what this is supposed to mean
$ mvn clean install
...
[INFO] --- scala-maven-plugin:3.2.2:testCompile (scala-test-compile-first) @ adam-snpeff_2.10 ---
[WARNING]  Expected all dependencies to require Scala version: 2.10.5
[WARNING]  com.twitter:chill_2.10:0.5.0 requires scala version: 2.10.4
[WARNING] Multiple versions of scala libraries detected!
[INFO] Compiling 1 Scala source to /Users/heuermh/working/adam-snpeff/target/scala-2.10.5/test-classes...
[ERROR] bad symbolic reference. A signature in ADAMFunSuite.class refers to type SparkFunSuite
in package org.bdgenomics.utils.misc which is not available.
It may be completely missing from the current classpath, or the version on
the classpath might be incompatible with the version used when compiling ADAMFunSuite.class.
[ERROR] /Users/heuermh/working/adam-snpeff/src/test/scala/com/github/heuermh/adam/snpeff/AdamSnpEffSuite.scala:30: not found: value sparkTest
[ERROR]   sparkTest("load vcf and pipe through SnpEff") {
[ERROR]   ^
[ERROR] two errors found
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------

I also tried with version Scala 2.10.4
$ mvn clean install
...
[INFO] --- scala-maven-plugin:3.2.2:testCompile (scala-test-compile-first) @ adam-snpeff_2.10 ---
[WARNING]  Expected all dependencies to require Scala version: 2.10.4
[WARNING]  com.twitter:chill_2.10:0.5.0 requires scala version: 2.10.4
[WARNING]  com.typesafe.akka:akka-remote_2.10:2.3.11 requires scala version: 2.10.4
[WARNING]  com.typesafe.akka:akka-actor_2.10:2.3.11 requires scala version: 2.10.4
[WARNING]  com.typesafe.akka:akka-slf4j_2.10:2.3.11 requires scala version: 2.10.4
[WARNING]  org.apache.spark:spark-core_2.10:1.6.1 requires scala version: 2.10.5
[WARNING] Multiple versions of scala libraries detected!
[INFO] Compiling 1 Scala source to /Users/heuermh/working/adam-snpeff/target/scala-2.10.4/test-classes...
[ERROR] bad symbolic reference. A signature in ADAMFunSuite.class refers to type SparkFunSuite
in package org.bdgenomics.utils.misc which is not available.
It may be completely missing from the current classpath, or the version on
the classpath might be incompatible with the version used when compiling ADAMFunSuite.class.
[ERROR] /Users/heuermh/working/adam-snpeff/src/test/scala/com/github/heuermh/adam/snpeff/AdamSnpEffSuite.scala:30: not found: value sparkTest
[ERROR]   sparkTest("load vcf and pipe through SnpEff") {
[ERROR]   ^
[ERROR] two errors found
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------

I'm depending on fnothaft issues/1112-pipes-api branch per #1114.

Can you push your branch? I tried building from ToT --> https://github.com/heuermh/adam-snpeff/commit/7336bb4b9dbe80e067d28ee468f3c368b4cc3607 but the pom.xml is missing. I have a thought as to what is missing.

If I add an explicit test scoped dependency on utils test-jar, then it works.  I don't understand why that isn't pulled in as a transitive dependency.  Will push shortly.

Yeah, that's what I was going to say to add. I don't think you get transitive dependencies in the test scope, no?

I don't think you get transitive dependencies in the test scope, no?

I don't know.  Why not?  :)

I don't think you get transitive dependencies in the test scope, no?

I don't know. Why not? :)

Agreed, I'm not sure why.

WontFix

I've run in to this before; test dependencies are included in the "test-jar" but not the "jar"; if you want to depend on a library + its test deps, it has to have published a "test-jar" (usually *-tests.jar) that you depend on, otomh.

Yeah that is what we have, a <type>test-jar</type> for adam-core
https://github.com/heuermh/adam-snpeff/blob/master/pom.xml#L336
It has a transitive dependency on the test-jar from bdg-utils, but for some reason that isn't brought in.  I had to add a separate dependency on the utils test-jar
https://github.com/heuermh/adam-snpeff/blob/master/pom.xml#L342


CableInfo extensions

When/What do we need the pullForce attribute for?
Info:
Name: pullForce
DataType: Force
Description: Maximum applied pulling force.
When/What do we need the bendRadius attribute for?
Info:
Name: bendRadius
DataType: Length
Description: Minimum bending radius

Name: pullForce
DataType: Force
Description: Maximum allowed pulling force.
Need new DataType Force that is Newton
Name: bendRadius
DataType: Length
Description: Minimum allowed bending radius that will not exceed the cables bending capability.


Improve throughput

Using sharded concurrent map could improve the throughput.

This feature makes code a lot more complicated, and gains are really low (10-20% performance improvement).

I am closing the PR without merging. This feature makes code a lot more complicated and gains are really low (10-20% performance improvement).


remove unnecessary warning from virtualenvwrapper.plugin.zsh

I think this warning message is meaningless since 90a5bd06ca76aff04f76a5d43a432ed9340dd78d (from https://github.com/robbyrussell/oh-my-zsh/pull/6842), the reason being:

if virtualenvwrapper.sh is being used, $WORKON_HOME will be
immediately (to the default value), no need for this warning
if virtualenvwrapper_lazy.sh is being used, $WORKON_HOME will be set
(to the default value) later, automatically:

if none of the initialization scripts are available, for example
if virtualenvwrapper was uninstalled, the installation command will be
printed during zsh start:

cc @lbolla, RFC @mcornella

Hi @timfeirg, see #6882 for discussion.


[MINOR] Increase CI timeout for UT FT other modules to 4 hours

Change Logs
UT FT other modules consistenly taking more than 3 hours. HUDI-6682 to track better redistribution of tests.
Impact
none
Risk level (write none, low medium or high below)
none
Documentation Update
Describe any necessary documentation update if there is any new feature, config, or user-facing change

The config description must be updated if new configs are added or the default value of the configs are changed
Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the
ticket number here and follow the instruction to make
changes to the website.

Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

4 hours is quite long, not sure we should do this.


The "fileName" or "name" properties of emitted chunks and assets must be strings that are neither absolute nor relative paths

I'm getting the following error when running azd up:
ERROR: error executing step command 'package --all': failed building service 'web': failed invoking event handlers for 'prebuild', 'prebuild' hook failed with exit code: '1', Path: 'C:\Users\charris\AppData\Local\Temp\azd-prebuild-1235103698.ps1'. : exit code: 1, stdout:
up to date, audited 265 packages in 801ms

found 0 vulnerabilities

><EMAIL_ADDRESS>build
> tsc && vite build

vite v4.5.5 building for production...
transforming...
✓ 4146 modules transformed.
rendering chunks...
✓ built in 9.12s
, stderr: [vite:build-html] The "fileName" or "name" properties of emitted chunks and assets must be strings that are neither absolute nor relative paths, received "d:/git/azure_agent/evals/GH_MVPS/src/frontend/index.html".
error during build:
    at async Bundle.generate (file:///d:/git/azure_agent/evals/GH_MVPS/src/frontend/node_modules/rollup/dist/es/shared/node-entry.js:17930:9)
    at async file:///d:/git/azure_agent/evals/GH_MVPS/src/frontend/node_modules/rollup/dist/es/shared/node-entry.js:26785:27
    at async catchUnfinishedHookActions (file:///d:/git/azure_agent/evals/GH_MVPS/src/frontend/no    at async Bundle.generate (file:///d:/git/azure_agent/evals/GH_MVPS/src/frontend/node_modules/rollup/dist/es/shared/node-entry.js:17930:9)
    at async file:///d:/git/azure_agent/evals/GH_MVPS/src/frontend/node_modules/rollup/dist/es/shared/node-entry.js:26785:27
    at async catchUnfinishedHookActions (file:///d:/git/azure_agent/evals/GH_MVPS/src/frontend/node_modules/rollup/dist/es/shared/node-entry.js:25910:16)
    at async build (file:///d:/git/azure_agent/evals/GH_MVPS/src/frontend/node_modules/vite/dist/node/chunks/dep-b2890f90.js:48085:22)
    at async CAC.<anonymous> (file:///d:/git/azure_agent/evals/GH_MVPS/src/frontend/node_modules/vite/dist/node/cli.js:842:9)

This SO thread has a few ideas:
https://stackoverflow.com/questions/77694425/react-vite-typescript-fails-in-jenkins-but-succeeds-locally
Is there something funky about the D drive? Is it using symlinks or something?
Also is your node up to date?

Good call! I do use symlinks so it appears that all my files are on my C: drive.
The following fixed the issue:

Nice! Okay, then I think I can safely make that change to the repo.


Update Eliatra Partner Information

Organization name
Eliatra
Eliatra changed its logo and products/services.
We provide a PR with the changes.

Completed when #2649 merged. Thanks Eliatra!


using scipy.optimize.least_squares when already evaluated many values

I'm using scipy.optimize.least_squares to optimize my function func
The challenge I have is that I already evaluated this function in many input X values and have their corresponding Y values.
I want to somehow "input" that into the the least_squares function and allow it to use this information, so it can achieve a better result.
P.S I'm using method="dogbox"

You can try to add a memoization functionality to your function If the values you are giving to your function is exactly what you have in the lookup table. Have a look at this post to get a rough idea; https://stackoverflow.com/questions/1988804/
This is not related to optimize code itself so I will close it but please feel free to continue the discussion as you wish.

Memoization will only help if these are the EXACT values. Since I use floating point, it's never the case

So how do you plan to use the readily made values in any case ? Interpolation?

I intend the optimizer to user that as extra information about the way my function looks and improve the way it tries to optimize.

Yes but I don't know how it can do that. Interpolation or curve fitting or something else?

The  basic (over-simplified)logic of any optimization function is:
def smart_optimize(func):
  current_values = {}
  while True:
    next_position = find_next_position_to_evaluate(current_values)
    value = func(value)
    current_values[next_position] = value
    if some_stop_condition(current_values):
      return best_value

My goal is to start with current_values already populated with some information that I have

The short answer is that there is no ready-made algorithm implemented in scipy.optimize that can use this information. You'll need to cook up your own routine for this.


Improve require.js integration

Right now Divolte has some basic support for JavaScript modules, but it doesn't work easily with Require.js, which seems to be the most popular module loader/injector currently in use.
Some problems right now are that:

Loading divolte.js explicitly with require.js fails because Divolte incorrectly registers itself as a module.
If require.js has been loaded prior to divolte.js, Divolte will throw an exception. (Once again, because it tries to incorrectly register itself as a module.)

In both cases, the error thrown is of the form:
Mismatched anonymous define() module.

There is currently a workaround possible: loading divolte.js as an anonymous module. Instead of the documented <script>…</script> tag, loading via require.js is possible using something like:
require('/path/to/divolte.js', function(divolte) {
   // Module has been loaded here. It can be used within this scope, or assigned globally.
   window.divolte = divolte;
});

See #147

Any news about this?

Interestingly I've been looking at this during the week. Contrary to the original description, it seems that loading divolte via require.js seems to work okay in most cases. (There are some edge cases with IE11 but they're not normally triggered.)
As things currently stand I think the following works…
require(['https://divolte.your.domain/divolte']);

Note the lack of .js on the end. This should be enough to load it and get the automatic pageView events, but the API obviously won't be in the global scope.
To get the API, it's better to configure the module path and go that way…
// Declare the module and path.
requirejs.config({
  paths: {
    divolte: 'https://divolte.your.domain/divolte'
  }
});
// Trigger the initial load.
require(['divolte']);

// To signal an event or use the API.
require(['divolte'], function(divolte) {
  divolte.signal('someEvent', {…});
});

… and so on.
Does this match your experience?


Fix dispatch saving code after 749a2c2d

In 749a2c2d iree_device_map and iree_target_map have been made functions but not all of the uses were updated.

@PhaneeshB


Request of newest nginx version seems to be broken

Getting this, while executing nginxpwner:
Traceback (most recent call last):   File "/opt/nginxpwner/nginxpwner.py", line 47, in <module>     last_version =(soup.findAll('a', attrs={'href': re.compile("^/nginx/nginx/releases/tag")})[0].get('href')).split("-")[1] IndexError: list index out of range
Fixed it temporarily with hardcoding the version number, but it seems that the request to /nginx/nginx/releases/tag isn't successful somehow, any insights?

Having same problem

same problem,broken

Hey, sorry for the delay guys, didn't notice this, already merged a commit from @cornerpirate which solves this issue


Expected Behavior
Cloning the repository should not cause a users antivirus to detect a threat.
Actual Behavior
AntiVirus Action Log
Filename: nsis_with_bzip.exe
Threat name: Heur.AdvML.BFull Path: C:\Users\astre\Documents\ghidra_plugins\ghidra-nsis-extension\nsis\src\testData\nsis_with_bzip.exe

____________________________

____________________________

On computers as of
7/7/2020 at 9:55:13 AM

Last Used
7/7/2020 at 9:57:15 AM

Startup Item
No

Launched
No

Threat type: Heuristic Virus. Detection of a threat based on malware heuristics.

____________________________

nsis_with_bzip.exe Threat name: Heur.AdvML.B
Locate

Very Few Users
Fewer than 5 users in the Norton Community have used this file.

Very New
This file was released less than 1 week  ago.

High
This file risk is high.

____________________________

Source: External Media

____________________________

File Actions

Infected file: C:\Users\astre\Documents\ghidra_plugins\ghidra-nsis-extension\nsis\src\testData\ nsis_with_bzip.exe Repaired
____________________________

File Thumbprint - SHA:
Not available
File Thumbprint - MD5:
Not available

Steps to Reproduce the Problem

git clone https://github.com/googleinterns/ghidra-nsis-extension.git and wait

It may be a good idea to serialize only the portions of the files required for testing instead of having the full files in the repo.

Damn, good catch. I'll look into this! Thanks for submitting an issue


[Tizen] Do not notify the resolve error of dnssd

Problem
There is a notification about resolve error of dnssd to caller.
It can cause a collision in the caller's user callback.
To explain in detail,
If the user callback is called in case of failure in the process of resolve of dnssd, It can cause unintended situations.
For example, user calls the resolve api of dnssd and cleans up the user data if it fails.
In addition, the user callback can also clean up the user data after processing the resolve operation of dnssd.
In this case, the user callback is used to complete the resolve operation of dnssd,
But if the user callback is called even if it fails, it may cause an unintended operation such as double free.
Change overview
Remove a notification about resolve error of dnssd.
Testing
chip-tool and chip-lighting-app examples

@hyunuktak - please explain "It can cause a collision in the caller's user callback."

@hyunuktak - please explain "It can cause a collision in the caller's user callback."

It seems that the explanation was insufficient.
I updated the detailed explanation. please refer to it.

@hyunuktak - please explain "It can cause a collision in the caller's user callback."

It seems that the explanation was insufficient. I updated the detailed explanation. please refer to it.

The explanation is still vague. I do not understand it. The way I read it is:
"An error is reported when resoltion fails, but this may cause an error to the user like double free".
Why don't you fix the double free then? Fix on the other side. Reporting an error when an error occurs is ok.
Unless you prove (with code links or some clear explanation) why user code never expects error reports and this is documented.

For example, user calls the resolve api of dnssd and cleans up the user data if it fails. Why the vagueness of 'for example'?
If you fix it, make the user properly handle the cleanup of the data. This seems to be a bug in the user usage of resolution not in the implementation of error handling.

I guess explanation could be done by example:
Linux avahi: https://github.com/project-chip/connectedhomeip/blob/master/src/platform/Linux/DnssdImpl.cpp#L693 does NOT callback error on failure
Darwin: https://github.com/project-chip/connectedhomeip/blob/master/src/platform/Darwin/DnssdImpl.cpp#L344 does NOT callback on failure.
I will update the summary


Ability to disable multi touch gesture

Hi , unless I've missed something within the API - I couldnt see anyway to disallow multi touch gestures ?
Currently its easy to scratch out the entire widget in just a few gestures by using two fingers, your thumb and an index finger - ensuring that your index finger is at the top of the widget to be scratched out and your thumb is at the bottom ... then you can scratch out the entire widget with one swipe gesture...

Is there any update to this or any workaround ? This scratcher plugin is ideal but as @alexda12 mentioned, two fingers placed above a widget will reveal everything in one go ....


Investigate: Second Prometheus for resource usage service

In order to decouple the metrics we collect for rough developer / app-team insights from proper resource metrics, we want to have a second Prometheus instance that only scrapes the cAdvisor.
Ideally we could have a AWS AMP managed prometheus for this, but connection to the docker swarm might not be trivial and even impossible. If the AMP is unfeasible (or too costly), we can have a second short-lived prometheus that is part of the oSparc-Simcore stack, potentially.

Done in
https://github.com/ITISFoundation/osparc-ops-environments/pull/179
https://github.com/ITISFoundation/osparc-ops-environments/pull/184
https://github.com/ITISFoundation/osparc-ops-environments/issues/174


Make apostrophes fit tightly

Description
The way LaTeX deals with apostrophes is way prettier in my opinion. I would love to see typst adapt this too!
Apostrophes should be positioned like superscripts.
Typst: $ a_1', a'_1 $

LaTeX: $$ a_1', a'_1$$

Typst: $ a_1^'' $

As you can see using ^' isn't really a solution. The spacing is off and the apostrophe is positioned to high.
Use Case
It's just about beauty.

Duplicate of https://github.com/typst/typst/issues/439


Remove version: taers232c.GAMADV-XTD3 version 6.69.00

This version is older than what has been set in max-versions-to-keep by the publisher.
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/taers232c/GAMADV-XTD3/releases/download/v6.69.00/gamadv-xtd3-6.69.00-windows-x86_64.msi
OK
(Automated message - build 910)


Support C++ :: syntax for scoping

Slang currently follows the modern language convention of using the "dot operator" on types to name static members (e.g., MyEnum.SomeCase), but C++ programmers are more used to the :: operator being used to reference static members. When developers try to share headers between Slang and C++, they would benefit from being able to use :: to name enum cases in both languages.
We should support this by:

Making sure :: lexes as its own token (I believe it might already)

Having the parser allow :: in the same cases where it handles ., but construct a StaticMemberExpr instead of a MemberExpr.

Making sure that the semantic checking code handles the member lookup appropriately for StaticMemberExpr (which had previously only appeared in already-checked syntax). This should include validating that the left-hand side really is a type, and that the right-hand side is a member that we can reference statically.

I think @jsmall-nvidia implemented the majority of this in #882.


fix(es/parser): Use unicode id instead of xid

Description:
Related issue (if exists):

Closes #3341
Closes #3862

I published a ~2x faster and 2kb smaller https://crates.io/crates/unicode-id-start, a fork from https://github.com/dtolnay/unicode-ident, would you like to give it a try and bench? Tests are passing in the Rome repo https://github.com/rome/tools/pull/2600

@Boshen It seems to be actually slower.
master:
     Running benches/parser.rs (/Users/kdy1/projects/master-swc/target/release/deps/parser-3fa8a5c343758a1f)
WARNING: HTML report generation will become a non-default optional feature in Criterion.rs 0.4.0.
This feature is being moved to cargo-criterion (https://github.com/bheisler/cargo-criterion) and will be optional in a future version of Criterion.rs. To silence this warning, either switch to cargo-criterion or enable the 'html_reports' feature in your Cargo.toml.

Gnuplot not found, using plotters backend
es/parser/colors        time:   [14.627 us 14.653 us 14.695 us]
Found 15 outliers among 100 measurements (15.00%)
  7 (7.00%) high mild
  8 (8.00%) high severe

es/parser/angular       time:   [7.6256 ms 7.6393 ms 7.6583 ms]
Found 11 outliers among 100 measurements (11.00%)
  11 (11.00%) high severe

Benchmarking es/parser/backbone: Warming up for 3.0000 s
Warning: Unable to complete 100 samples in 5.0s. You may wish to increase target time to 5.3s, enable flat sampling, or reduce sample count to 60.
es/parser/backbone      time:   [1.0485 ms 1.0501 ms 1.0520 ms]
Found 12 outliers among 100 measurements (12.00%)
  5 (5.00%) high mild
  7 (7.00%) high severe

es/parser/jquery        time:   [5.8608 ms 5.8670 ms 5.8741 ms]
Found 16 outliers among 100 measurements (16.00%)
  9 (9.00%) high mild
  7 (7.00%) high severe

es/parser/jquery mobile time:   [9.2385 ms 9.2490 ms 9.2612 ms]
Found 11 outliers among 100 measurements (11.00%)
  4 (4.00%) high mild
  7 (7.00%) high severe

es/parser/mootools      time:   [4.5426 ms 4.5555 ms 4.5703 ms]
Found 16 outliers among 100 measurements (16.00%)
  16 (16.00%) high severe

es/parser/underscore    time:   [912.10 us 913.79 us 916.06 us]
Found 10 outliers among 100 measurements (10.00%)
  2 (2.00%) high mild
  8 (8.00%) high severe

es/parser/three         time:   [26.538 ms 26.596 ms 26.674 ms]
Found 17 outliers among 100 measurements (17.00%)
  6 (6.00%) high mild
  11 (11.00%) high severe

es/parser/yui           time:   [4.5326 ms 4.5425 ms 4.5558 ms]
Found 16 outliers among 100 measurements (16.00%)
  5 (5.00%) high mild
  11 (11.00%) high severe

With unocide-id-start
    Finished bench [optimized + debuginfo] target(s) in 47.80s
     Running benches/parser.rs (/Users/kdy1/projects/s/perf/target/release/deps/parser-fc34b825e591d37b)
WARNING: HTML report generation will become a non-default optional feature in Criterion.rs 0.4.0.
This feature is being moved to cargo-criterion (https://github.com/bheisler/cargo-criterion) and will be optional in a future version of Criterion.rs. To silence this warning, either switch to cargo-criterion or enable the 'html_reports' feature in your Cargo.toml.

Gnuplot not found, using plotters backend
es/parser/colors        time:   [14.964 us 14.988 us 15.013 us]
                        change: [-0.1938% +0.0311% +0.2661%] (p = 0.79 > 0.05)
                        No change in performance detected.
Found 23 outliers among 100 measurements (23.00%)
  7 (7.00%) low severe
  10 (10.00%) low mild
  1 (1.00%) high mild
  5 (5.00%) high severe

es/parser/angular       time:   [7.6198 ms 7.6294 ms 7.6413 ms]
                        change: [-1.5204% -1.2412% -0.9750%] (p = 0.00 < 0.05)
                        Change within noise threshold.
Found 8 outliers among 100 measurements (8.00%)
  2 (2.00%) high mild
  6 (6.00%) high severe

Benchmarking es/parser/backbone: Warming up for 3.0000 s
Warning: Unable to complete 100 samples in 5.0s. You may wish to increase target time to 5.3s, enable flat sampling, or reduce sample count to 60.
es/parser/backbone      time:   [1.0530 ms 1.0573 ms 1.0619 ms]
                        change: [-1.9814% -1.2038% -0.4412%] (p = 0.00 < 0.05)
                        Change within noise threshold.
Found 13 outliers among 100 measurements (13.00%)
  6 (6.00%) high mild
  7 (7.00%) high severe

es/parser/jquery        time:   [5.8711 ms 5.8894 ms 5.9114 ms]
                        change: [-1.2975% -0.9091% -0.4743%] (p = 0.00 < 0.05)
                        Change within noise threshold.
Found 14 outliers among 100 measurements (14.00%)
  5 (5.00%) high mild
  9 (9.00%) high severe

es/parser/jquery mobile time:   [9.2244 ms 9.2401 ms 9.2619 ms]
                        change: [-1.6064% -1.3953% -1.1664%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 13 outliers among 100 measurements (13.00%)
  6 (6.00%) high mild
  7 (7.00%) high severe

es/parser/mootools      time:   [4.5215 ms 4.5279 ms 4.5372 ms]
                        change: [-1.6521% -1.4016% -1.1575%] (p = 0.00 < 0.05)
                        Performance has improved.
Found 15 outliers among 100 measurements (15.00%)
  4 (4.00%) high mild
  11 (11.00%) high severe

es/parser/underscore    time:   [915.10 us 916.52 us 918.25 us]
                        change: [-1.3710% -1.0421% -0.6754%] (p = 0.00 < 0.05)
                        Change within noise threshold.
Found 15 outliers among 100 measurements (15.00%)
  3 (3.00%) high mild
  12 (12.00%) high severe

es/parser/three         time:   [26.553 ms 26.596 ms 26.647 ms]
                        change: [-1.0552% -0.7822% -0.5174%] (p = 0.00 < 0.05)
                        Change within noise threshold.
Found 18 outliers among 100 measurements (18.00%)
  6 (6.00%) high mild
  12 (12.00%) high severe

es/parser/yui           time:   [4.5356 ms 4.5415 ms 4.5485 ms]
                        change: [-0.8592% -0.6833% -0.5203%] (p = 0.00 < 0.05)
                        Change within noise threshold.
Found 15 outliers among 100 measurements (15.00%)
  9 (9.00%) high mild
  6 (6.00%) high severe

Try removing the ascii chars check? The crate handles it with a lookup table https://github.com/Boshen/unicode-id-start/blob/master/src/tables.rs#L9-L21

@Boshen It's even worse.
WARNING: HTML report generation will become a non-default optional feature in Criterion.rs 0.4.0.
This feature is being moved to cargo-criterion (https://github.com/bheisler/cargo-criterion) and will be optional in a future version of Criterion.rs. To silence this warning, either switch to cargo-criterion or enable the 'html_reports' feature in your Cargo.toml.

Gnuplot not found, using plotters backend
es/parser/colors        time:   [15.170 us 15.183 us 15.200 us]
                        change: [+1.1468% +1.3429% +1.5396%] (p = 0.00 < 0.05)
                        Performance has regressed.
Found 8 outliers among 100 measurements (8.00%)
  1 (1.00%) low mild
  2 (2.00%) high mild
  5 (5.00%) high severe

es/parser/angular       time:   [7.8974 ms 7.9021 ms 7.9073 ms]
                        change: [+3.4026% +3.5746% +3.7247%] (p = 0.00 < 0.05)
                        Performance has regressed.
Found 8 outliers among 100 measurements (8.00%)
  1 (1.00%) low mild
  5 (5.00%) high mild
  2 (2.00%) high severe

Benchmarking es/parser/backbone: Warming up for 3.0000 s
Warning: Unable to complete 100 samples in 5.0s. You may wish to increase target time to 5.5s, enable flat sampling, or reduce sample count to 60.
es/parser/backbone      time:   [1.0864 ms 1.0875 ms 1.0883 ms]
                        change: [+2.4183% +3.1326% +3.8099%] (p = 0.00 < 0.05)
                        Performance has regressed.
Found 17 outliers among 100 measurements (17.00%)
  6 (6.00%) low severe
  3 (3.00%) low mild
  8 (8.00%) high severe

es/parser/jquery        time:   [6.0440 ms 6.0535 ms 6.0627 ms]
                        change: [+2.3816% +2.7870% +3.1473%] (p = 0.00 < 0.05)
                        Performance has regressed.
Found 17 outliers among 100 measurements (17.00%)
  14 (14.00%) low mild
  2 (2.00%) high mild
  1 (1.00%) high severe

es/parser/jquery mobile time:   [9.4372 ms 9.4553 ms 9.4739 ms]
                        change: [+2.0196% +2.3283% +2.6087%] (p = 0.00 < 0.05)
                        Performance has regressed.

es/parser/mootools      time:   [4.6195 ms 4.6252 ms 4.6314 ms]
                        change: [+1.9055% +2.1470% +2.3492%] (p = 0.00 < 0.05)
                        Performance has regressed.
Found 12 outliers among 100 measurements (12.00%)
  7 (7.00%) high mild
  5 (5.00%) high severe

es/parser/underscore    time:   [931.71 us 932.77 us 934.04 us]
                        change: [+1.3094% +1.6378% +1.9671%] (p = 0.00 < 0.05)
                        Performance has regressed.
Found 14 outliers among 100 measurements (14.00%)
  9 (9.00%) high mild
  5 (5.00%) high severe

es/parser/three         time:   [27.137 ms 27.174 ms 27.216 ms]
                        change: [+1.9320% +2.1715% +2.4062%] (p = 0.00 < 0.05)
                        Performance has regressed.
Found 13 outliers among 100 measurements (13.00%)
  5 (5.00%) high mild
  8 (8.00%) high severe

es/parser/yui           time:   [4.5894 ms 4.5933 ms 4.5978 ms]
                        change: [+0.9607% +1.1396% +1.3039%] (p = 0.00 < 0.05)
                        Change within noise threshold.
Found 14 outliers among 100 measurements (14.00%)
  7 (7.00%) high mild
  7 (7.00%) high severe


feat(extension-chrome): Add signing data implement

This PR implement the sign message UI logic
when call NotificationService#requestSignData, it will open a notification dialog.

then user input the correspond  password, the method will resolved and return the wallet password.

There is an error while compiling, why we should change the requestSignData's payload from BytesLike to ArrayBufferLike


Fix vesting ante

Closes: #XXX
Description
Fixes the issue with the delegation of vesting coins.

For contributor use:

[+] Targeted PR against correct branch (see CONTRIBUTING.md)
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[+] Code follows the module structure standards.
[ ] Wrote unit and integration tests
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/)
[ ] Added  relevant godoc comments.
[ ] Added a relevant changelog entry to the Unreleased section in CHANGELOG.md
[+] Re-reviewed Files changed in the Github PR explorer

For admin use:

[+] Added appropriate labels to PR (ex. WIP, R4R, docs, etc)
[+] Reviewers assigned
[ ] Squashed all commits, uses message "Merge pull request #XYZ: [title]" (coding standards)

Codecov Report
Attention: 9 lines in your changes are missing coverage. Please review.

Comparison is base (1cdd044) 67.81% compared to head (a5c5549) 67.78%.

:exclamation: Current head a5c5549 differs from pull request most recent head 730f350. Consider uploading reports for the commit 730f350 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #277      +/-   ##
==========================================
- Coverage   67.81%   67.78%   -0.03%
==========================================
  Files         200      200
  Lines       16149    16156       +7
==========================================
+ Hits        10951    10952       +1
- Misses       4662     4668       +6
  Partials      536      536

Files
Coverage Δ

app/ante/cosmos/fees.go
92.96% <100.00%> (ø)

app/ante/evm/fee_checker.go
100.00% <100.00%> (ø)

app/ante/evm/fees.go
74.72% <100.00%> (ø)

app/ante/sigverify.go
78.94% <ø> (ø)

app/ante/cosmos/eip712.go
5.20% <50.00%> (ø)

app/ante/handler_options.go
54.28% <50.00%> (ø)

app/ante/cosmos/vesting.go
9.85% <22.22%> (+0.48%)
:arrow_up:


0.2.0 upgrade

Needed to update dependencies in addition to work from #1

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


System.InvalidOperationException: Shutdown has already been called

What version of gRPC and what language are you using?
Tested:

2.28.1 c# 7.3, Protobuf 3.11.4
1.20.0 c# 7.3, Protobuf 3.7.0

What operating system (Linux, Windows,...) and version?
Windows
What runtime / compiler are you using (e.g. python version or version of gcc)
Visual Studio 2019
What did you do?
I will update the following information, whenever I have new information or better test cases.

Open 4 channels. 2 channels connect to the same port. (A simpler setup might be possible)
Start a server side streaming call, passing a cancellationtoken
Receive an answer from the server side by accessing call.ResponseStream.Current.
Add a delay inside the answer processing, to avoid calling MoveNext() to fast
On a parallel Task cancel the Token first, then call channel1.ShutdownAsync().ConfigureAwait(false) on all channels
The Delay task should now end and MoveNext() should be called again
System.InvalidOperationException: Shutdown has already been called Is thrown.

What did you expect to see?
An RPC Exception with StatusCode IsCancelled.
What did you see instead?
The Invalidoperationexception should not happen. It is important to note here, that this issue can only be reproduced, if all channels are shutdown. I am using multiple channels, with multiple ports. Some clients connect to the same port, some to a different.
It seems that the MoveNext() method does not check the cancellationtoken first, but rather throws an error, if all channels are shut down. If still one channel is alive the error does not occur! Thatswhy I assume that this is a possible grpc but and not my wrong usage.
Anything else we should know about your project / environment?
I currently have no simple reproducible testcase, nor am I using the latest version. I want to use this bugreport to collect information, maybe someone else also stumbles across this issue. The following code snippets demonstrates how to reproduce the bug:
      protected async Task Test()
        {
            var channel1 = new Channel(_host, _port, ChannelCredentials.Insecure);
            var channel2 = new Channel(_host, _port, ChannelCredentials.Insecure);
            var userPresenceClient = new UserPresenceService.UserPresenceServiceClient(channel1);
            var otherClient = new ContactListService.ContactListServiceClient(channel2);

            var cts = new CancellationTokenSource();
            var token = cts.Token;

            var grpcCallTask = Task.Run(async () =>
            {
                try
                {
                    using (var call = userPresenceClient.SubscribeUserPresences(new SubscribeUserPresencesRequest(),
                        cancellationToken: token))
                    {
                        while (await call.ResponseStream.MoveNext(token))
                        {
                            var response = call.ResponseStream.Current;
                            await Task.Delay(8000);
                        }
                    }
                }
                catch (Exception e)
                {
                    // Add a breakpoint here to verify the thrown exception
                    throw;
                }
            });

            await Task.Delay(1000);
            cts.Cancel();
            cts.Dispose();

            channel1.ShutdownAsync().ConfigureAwait(false);
            channel2.ShutdownAsync().ConfigureAwait(false);
            try
            {
                await grpcCallTask;
            }
            catch (Exception e)
            {
                // Add a breakpoint here to verify the thrown exception
                throw;
            }
        }

A workaround for this issue:
catch when (token.IsCancellationRequested)
{
    throw new OperationCanceledException();
}

We are discouraging questions posted as issues here. Please feel free to post to stackoverflow.

Well, this is no question, it is a bug report. At least I think so, if I am wrong, please explain me why.

I think we can classify this as "works as intended".

Looks like you have an active call and you explicitly shutdown all the channels in your application (including the channel on which you started the call) and then expect the in-flight call to continue to work normally (Btw doing so actually prints a warning).
The correct usage pattern is to only shutdown the channel once all the calls on that channel have finished.
Based on the documentation, the channel.ShutdownAsync() doesn't have to explicitly cancel all that call on a given chanel (since doing that would require extra tracking and have performance costs). If for whatever reason you want to get your calls automatically cancelled upon channel shutdown is requested, you can use channel.ShutdownToken and register it explicitly with the calls you start (var call = client.InvokeFooBar(cancellationToken: channel.ShutdownToken)). See relevant docs here:
https://github.com/grpc/grpc/blob/730599bd5b68f12436907cd96ab3fb20579734e7/src/csharp/Grpc.Core.Api/ChannelBase.cs#L60

I think what's happening under the hood is that shutting down all the channels in your app removes the all the references to the "grpc environment" (what makes the entire grpc work) held internally and shuts it down. You then get the "Shutdown has already been called" error from one of the internal components of grpc (I think here: https://github.com/grpc/grpc/blob/730599bd5b68f12436907cd96ab3fb20579734e7/src/csharp/Grpc.Core/Internal/CompletionQueueSafeHandle.cs#L110)
As a side note, Grpc.Core is now in maintenance mode and we're only going to be doing critical fixes going forward and since you issue doesn't seem like an actual bug, but more as an inconvenience that can be easily avoided by correct usage, I'm going to close this.


Update repositories.txt

@ArduinoBot

@ArduinoBot

A problem was found with your submission https://github.com/goodisplayshare/esp32-epd
❌ ERROR: Library is missing a library.properties metadata file.
How should I handle this matter? There is this file in the library？

@ArduinoBot

❌ ERROR: Library is missing a library.properties metadata file.
I obviously have this file in my library, what should I do? ?

❌ ERROR: Library is missing a library.properties metadata file.
I obviously have this file in my library, what should I do? ?  @per1234

Hello, I have installed and added the library.properties file to the github library, but I still get an error.
Can you help me?
My English is not very good and I didn't check my previous comments in time. I'm very sorry.
Thank you ~
At 2024-10-25 14:40:07, "per1234" @.***> wrote:
@per1234 commented on this pull request.
Hello @goodisplayshare. I see you have fixed the error reported by the bot. However, Library Manager works exclusively from Git tags, so the bot won't be able to see the fix you made until you create a new release or tag in the library repository.
If you look at the repository contents at the latest tag, you can see that the version the bot is checking still has the problem:
https://github.com/goodisplayshare/esp32-epd/tree/v1.0.1
(note that the repository doesn't contain a library.properties file at that point in the revision history)
❗ Remember to update the version field in the library's library.properties file as needed before making a release/tag.
Once you have done that, simply mention @ArduinoBot in a comment here to trigger it to run the automated checks again. After those are passing, your library will automatically be added to Library Manager.
If you have any questions or concerns, please let me know.
Regards, Per
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

@per1234, can you tell me what to do next?

@goodisplayshare I already explained what you need to do. Please read it again very very carefully:
https://github.com/arduino/library-registry/pull/5289#pullrequestreview-2394335332
Note that the explanation includes links you can follow to get even more information.
If there is something you don't understand, comment here with a detailed explanation of what exactly you don't understand and I will provide further assistance. DO NOT CREATE MORE PULL REQUESTS!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!

My English is not very good

I use Google Translate every day and find it does an excellent job.

Please use Google Translate to translate the text of my comment at https://github.com/arduino/library-registry/pull/5289#pullrequestreview-2394335332 to your native language, then read it very carefully. I have explained the cause of the problem and how you can fix it there.
If there is something specific you don't understand about what I wrote, you can provide a detailed description of exactly which part of what I wrote you don't understand, and what you don't understand about it. Or if you understood what I wrote, but encountered a problem while trying to follow the instructions I provided, you can provide a detailed description of what you tried and what problem you encountered.

”I see you have fixed the error reported by the bot. However, Library Manager works exclusively from Git tags, so the bot won't be able to see the fix you made until you create a new release or tag in the library repository."  At present, it is still in an error state and the error issue has not been resolved. It still prompts for missing files  @per1234

It is because you haven't done the thing I told you to do to resolve the problem:

create a new release or tag in the library repository.

Why haven't you done that?
If you don't know how to do it, click the links in my comment to learn how. Since you don't have a high level of technical competence, I recommend using GitHub's "release" feature (when you create a GitHub release, it automatically creates a Git tag) as doing so is more simple:
https://docs.github.com/github/administering-a-repository/releasing-projects-on-github/managing-releases-in-a-repository

@ArduinoBot

@ArduinoBot

@ArduinoBot

@ArduinoBot

@ArduinoBot


Breaking change: non-public parameterless ctors will not be used for deserialization with JsonSerializer

Non-public parameterless ctors will not be used for deserialization with JsonSerializer
Version introduced
.NET 5.0 preview 1
Old behavior

On .NET Core 3.0+, we allow internal/private ctors of POCOs and call them while deserializing.
On netstandard2.0/2.1, we don’t and throw a MissingMethodException: No parameterless constructor defined for this object. (from Activator.CreateInstance).

New behavior
Non-public constructors are ignored by the serializer by default. This includes parameterless constructors. The serializer will use one of the following constructors for deserialization.

Public constructor annotated with JsonConstructorAttribute.
Public parameterless constructor.
Public parameterized constructor (if it is the only public constructor present)

If none of the above are present, a NotSupportedException will be thrown if deserialization of the type is attempted.
Reason for change

Enforce consistent behavior between all TFMs that System.Text.Json builds for (.NET Core 3.0+, .netstandard 2.0+)
JsonSerializer shouldn't be calling non-public surface area of types by default (whether that's ctor, properties, or fields).

Recommended action

If the user controls the own type and it is feasible, make the parameterless constructor public
Implement a JsonConverter<T> for the type and control the deserialization behavior

In .NET 6, the JsonConstructorAttribute (new in .NET 5) will be honored when placed on a non-public ctor which will serve as an explicit opt-in for non-public ctors to be used - https://github.com/dotnet/runtime/issues/38327.
Category

[ ] ASP.NET Core
[ ] C#
[ ] Code analysis
[x] Core .NET libraries
[ ] Cryptography
[ ] Data
[ ] Debugger
[ ] Deployment
[ ] Globalization
[ ] Interop
[ ] JIT
[ ] LINQ
[ ] Managed Extensibility Framework (MEF)
[ ] MSBuild
[ ] Networking
[ ] Printing
[ ] Security
[ ] Serialization
[ ] Visual Basic
[ ] Windows Forms
[ ] Windows Presentation Foundation (WPF)
[ ] XML, XSLT

Affected APIs
JsonSerializer.Deserialize
JsonSerializer.DeserializeAsync

Issue metadata

Issue type: breaking-change

FYI @tdykstra

Non-public parameterless ctors will not be used for deserialization with JsonSerializer

Consider adding these qualifiers to be explicit about the extent of the change:
By default, non-public parameterless ctors will not be used for deserialization with JsonSerializer**, to be consistent across all supported TFMs**


8653.3 Update Admiral to show link for Docker Installation and add other docker functions.

https://github.com/Shippable/pm/issues/8653

This is in RC.


Displaying the select closed at loading

Hi,
thanks for your great work.
Is there a way that the select will display just the parent option on loading the page ?

Like startCollapsed?

how, sorry !
Thanks a lot


don't update packages when running os-imager on ci builds

We decided against this for now, also this pr is stale.


Missing CHANGELOG.md

Please maintain a CHANGELOG.md. Bolt was released today in version 1.28.0 and the website knows only about version 1.27.1

We had a bit of a process hitch getting the doc updates published for this release, sorry about that!
We've also been discussing improvements to how we track and where we publish release notes, so I definitely appreciate your feedback about a CHANGELOG.md.

Are changelog and release notes different?  No - we currently have this concept spread across 3 sources. We should add New Features, Bug fixes, and other user-facing changes to the CHANGELOG.md in the PR where the change happened. We should not add maint or docs changes to the changelog - the git history should act as a more thorough changelog.
Part of this work is removing release notes docs from pre-docs
The existing (New Features)[https://puppet.com/docs/bolt/latest/bolt_new_features.html], (Deprecations)[https://puppet.com/docs/bolt/latest/bolt_deprecations_and_removals.html] and (Resolved Issues)[https://puppet.com/docs/bolt/latest/bolt_resolved_issues.html] content should be added to the new Changelog. The format of the changelog can be as such, but whoever picks this up should play with formatting to see what feels right (and research how other projects do it):
## Version 1.2.0
- (GH-1111)[link-to-github-issue] Title of Change **Type of change (ie. Feature, Bug, Breaking Change)**
Description of change

### (GH-1112)[link-to-gh-issue] Title of Change **Type of change**
Description of change


fix: s3 upload index.html at last

Description of changes

This PR is to mitigate https://github.com/aws-amplify/amplify-cli/issues/2087 as proposed by @timoteialbu .
Issue: since index.html is not always uploaded to S3 at last, it sometimes causes file not found issue if a dependent file is uploaded after index.html
Fix: await Promise.all all other files except index.html, then upload index.html at the end.
Note:

This PR is just a mitigation. To fundamentally resolve the issue,

Issue #, if available

https://github.com/aws-amplify/amplify-cli/issues/2087
Description of how you validated changes
Checklist

[ ] PR description included
[ ] yarn test passes
[ ] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies
[ ] Pull request labels are added

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #12852 (1136ce0) into dev (ec9a2ba) will increase coverage by 48.53%.
The diff coverage is 61.81%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@             Coverage Diff             @@
##              dev   #12852       +/-   ##
===========================================
+ Coverage    0.00%   48.53%   +48.53%
===========================================
  Files        1296      841      -455
  Lines      149743    38089   -111654
  Branches     1296     7755     +6459
===========================================
+ Hits            0    18487    +18487
+ Misses     148447    18013   -130434
- Partials     1296     1589      +293

Impacted Files
Coverage Δ

...ify-category-function/src/events/prePushHandler.ts
33.33% <0.00%> (+33.33%)
:arrow_up:

...ider-utils/awscloudformation/utils/layerHelpers.ts
21.80% <0.00%> (+21.80%)
:arrow_up:

...ib/S3AndCloudFront/helpers/configure-CloudFront.js
87.06% <ø> (+87.06%)
:arrow_up:

...lify-category-hosting/lib/S3AndCloudFront/index.js
89.65% <ø> (+89.65%)
:arrow_up:

...ifications/src/commands/notifications/configure.ts
0.00% <0.00%> (ø)

...e/src/amplify-node-pkg-detector/lock-file-types.ts
100.00% <ø> (+100.00%)
:arrow_up:

.../src/amplify-node-pkg-detector/yarn-lock-parser.ts
100.00% <ø> (+100.00%)
:arrow_up:

packages/amplify-cli-core/src/constants.ts
100.00% <ø> (+100.00%)
:arrow_up:

...es/amplify-cli-core/src/context/plugin-platform.ts
33.33% <0.00%> (+33.33%)
:arrow_up:

...s/amplify-cli-core/src/errors/amplify-exception.ts
82.35% <ø> (+82.35%)
:arrow_up:

... and 36 more

... and 1258 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


chore: use exit break plugin instead

Using one of their solutions to replace setTimeout. We decided to use their own plugin. The whole conversation can be read here: https://slate-js.slack.com/archives/C013QHXSCG1/p1631208644108100?thread_ts=1631022429.081900&cid=C013QHXSCG1
The second solution would be just adding a select: true property to Transforms.insertNodes such as:
Transforms.insertNodes(editor, paragraph, { at: currentPath as Path, select: true }

Note: the commit type should be refactor instead of chore :)


Show a table of manifest IDs

A table which shows manifest IDs along with dates would be useful.

Done.


bio_asn1 infinite loop

When using an asn1 BIO and the sink BIO fails to write returning -1 (error), the state machine keeps retrying in an infinite loop.
The problem is caused by this line.
I assume the intention of the break was to exit the for loop, but it only makes the switch case end and thus returning to the same state over and over.
It could be solved by replacing the break in the indicated line by a goto done;.

This was fixed with f125430 in master and 723f616 in 1.1.0.


feat: enable choice of OCP registry storage

Description
Added new variable enable_registry_storage (bool) to allow consumer to enable or disable an IBM Cloud Object Storage bucket to be used for OpenShift internal container image storage. Previously this was not optional and was always provisioned with COS storage, but recent changes to underlying API now makes it possible to turn storage off to take care of certain compliancy restrictions.
Release required?

[ ] No release
[ ] Patch release (x.x.X)
[x] Minor release (x.X.x)
[ ] Major release (X.x.x)

Release notes content
Added new variable enable_registry_storage (bool) to allow consumer to enable or disable an IBM Cloud Object Storage bucket to be used for OpenShift internal container image storage. Previously this was not optional and was always provisioned with COS storage, but recent changes to underlying API now makes it possible to turn storage off to take care of certain compliancy restrictions.
Run the pipeline
If the CI pipeline doesn't run when you create the PR, the PR requires a user with GitHub collaborators access to run the pipeline.
Run the CI pipeline when the PR is ready for review and you expect tests to pass. Add a comment to the PR with the following text:
/run pipeline

Checklist for reviewers

[ ] If relevant, a test for the change is included or updated with this PR.
[ ] If relevant, documentation for the change is included or updated with this PR.

For mergers

Use a conventional commit message to set the release level. Follow the guidelines.
Include information that users need to know about the PR in the commit message. The commit message becomes part of the GitHub release notes.
Use the Squash and merge option.

/run pipeline

/run pipeline

/run pipeline

:tada: This PR is included in version 3.13.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Experimental DRM solution

No idea if this works :P don't have DRM content at moment.

The 3rd party F1 AppleTV also had to adjust the UserAgent to something "official" to work, so this also might need to be adjusted here eventually.

This still has not been tested right? I guess we will find out this weekend?

Nop, not tested, but at least it still works with non DRM content 🗡


office-addin-dev-settings: fix sideload command

Need better logic for the case where the is an app specified and the manifest contains multiple hosts.

@TCourtneyOwen Please review.


Build and publish a release against current Scala 2.12 milestone

It would be great if Breeze could build and publish artifacts against the current Scala 2.12 milestone (currently 2.12.0-M4). This is a blocker for building Apache Spark against Scala 2.12 (see SPARK-14438).

I'm swamped with work for the next 2 weeks so I won't be able to do this
quickly. I'd be happy to help people with issues if they wanted to give it
a shot
On Wed, Apr 6, 2016 at 12:38 PM, Josh Rosen<EMAIL_ADDRESS>wrote:

It would be great if Breeze could build and publish artifacts against the
current Scala 2.12 milestone (currently 2.12.0-M4). This is a blocker for
building Apache Spark against Scala 2.12 (see SPARK-14438
https://issues.apache.org/jira/browse/SPARK-14438).
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/scalanlp/breeze/issues/520

It looks like we're blocked by a few other libraries which still need to cross-publish, so I'll use this list to track them:

[ ] macro paradise, which will likely be published soon: https://gitter.im/scalamacros/paradise?at=5704cc58dc7a73775697af7f
[ ] shapeless
[ ] spire-math

superseded by #604


Can we generate Json Stats Data using output option ?

  $ vite-bundle-visualizer <command> [options]

Can we generate Stats data in json file format?

Support in v.0.5.0
$ vite-bundle-visualizer -t json -o ./output.json

Json file is created. Thanks for your support.


fix(cxx_extractor): the main source file should be marked always_process

There is some argument to be made that this logic should (also?) go in the indexer as the indexer should always process the primary source file, regardless of claiming, but we already have a facility for this in the extractor and just need to change one place so that the primary source file is always processed.
Background: when using dynamic claiming, prospective claims are issued with a timeout, but if a given compilation unit abandoned midway through indexing it can sometimes be retried before the previous prospective claims have expired, thus skipping files which it should index.  While this is problematic for header files as well, it is most troublesome for source files which are unlikely to be indexed at all as a result. Header files may potentially be included by another compilation unit and therefore indexed separately.
Pros of this approach: very small change, uses pre-existing facility.
Cons:

This arguably belongs in the indexer itself
can't be flag-controlled in extractor
need to update all of the tests due to the change in the resulting proto files

This seems like a nice workaround. Our original idea for static claiming was too expensive to be practical on google3, but this is a decent tradeoff.


robot_controllers: 0.5.3-0 in 'kinetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository robot_controllers to 0.5.3-0:

upstream repository: https://github.com/fetchrobotics/robot_controllers.git
release repository: https://github.com/fetchrobotics-gbp/robot_controllers-release.git
distro file: kinetic/distribution.yaml
bloom version: 0.5.26
previous version for package: 0.5.2-0

robot_controllers
* remove limiter
* Improve cartesian twist controller (#26 <https://github.com/fetchrobotics/robot_controllers/issues/26>)
* Contributors: Hanjun Song, Michael Ferguson

robot_controllers_interface
* add error message when pluginlib fails
* fix cmake warnings on kinetic (#28 <https://github.com/fetchrobotics/robot_controllers/issues/28>)
* Contributors: Michael Ferguson

robot_controllers_msgs

No changes

This release was improprely bloomed so that it has indigo dependecies?
https://github.com/fetchrobotics-gbp/robot_controllers-release/blob/debian/ros-kinetic-robot-controllers-interface_0.5.3-0_xenial/debian/control
It's failing the msgs package and blocking downstream: http://build.ros.org/view/Kbin_uX32/job/Kbin_uX32__robot_controllers_msgs__ubuntu_xenial_i386__binary/22/console


[Bug]: Seed Job failing with AttributeError for 'ldap' module

What happened?
e.g. https://ci-beam.apache.org/job/beam_SeedJob/9949/console
INFO:root:Generating Committers.groovy
Traceback (most recent call last):
  File "main.py", line 143, in <module>
    generate_groovy(args.output_dir, args.ldap_uris, args.cert_path)
  File "main.py", line 53, in generate_groovy
    github_usernames=get_committers_github_usernames(
  File "main.py", line 66, in get_committers_github_usernames
    ldap.set_option(ldap.OPT_X_TLS, ldap.OPT_X_TLS_DEMAND)
AttributeError: module 'ldap' has no attribute 'OPT_X_TLS'
+ echo 'ERROR: Failed to fetch committers'
ERROR: Failed to fetch committers

Also affects standalone seed job (https://ci-beam.apache.org/job/beam_SeedJob_Standalone/8489/console)
Issue Priority
Priority: 2
Issue Component
Component: testing

.add-labels tests

.add-labels test-failures

Sounds related to the recent change on pyhton-ldap package: https://pypi.org/project/python-ldap/#history
Release time coincidence with the time the job begins to fail.

Sounds right, release notes mentions that OPT_X_TLS is deprecated: https://github.com/python-ldap/python-ldap/releases


Accessibility service and package installer?

In January my scanner got a few additional checks. So on today's release it reported:
! repo/com.kieronquinn.app.classicpowermenu_172.apk declares intent-filter(s): android.accessibilityservice.AccessibilityService
! repo/com.kieronquinn.app.classicpowermenu_172.apk declares sensitive permission(s): android.permission.REQUEST_INSTALL_PACKAGES android.permission.QUERY_ALL_PACKAGES android.permission.CAMERA

Could you please clarify what for Accessibility Services are used – and what packages your app wants to install? Further: Concerning CAMERA, a screenshot shows what could be a QR code scanner. Is it? And what for does the app need QUERY_ALL_PACKAGES?
Thanks in advance!

Accessibility is used to know when the power menu is open. Install packages is used for updates.
QUERY_ALL_PACKAGES is for device controls, camera does indeed come from the QR library.

Thanks again! As with Ambient, same questions concerning the updater here (and same hints for the DEPENDENCY_INFO_BLOCK).
What I did not understand: QUERY_ALL_PACKAGES is for listing all installed apps. I must admit I'm not using ClassicPowerMenu myself, so my question might sound a little weird: but what does that have to do with "device controls"? I probably miss what they are :zany_face:

And am I assuming correctly that android.permission.REQUEST_INSTALL_PACKAGES is again an enforced self-updater here, @KieronQuinn? :cry:

And am I assuming correctly that android.permission.REQUEST_INSTALL_PACKAGES is again an enforced self-updater here, @KieronQuinn? 😢

Yes, as has always been the case. All of my apps have this, it's not new.


Not able to resolve includes and copybooks

Development environment used

Z Open Editor version:
Editor Platform

[x] Visual Studio Code
[ ] Red Hat CodeReady Workspaces
[ ] Eclipse Che
[ ] Standalone Theia

Editor Platform Version:
Operating System (on which VS Code runs such as Windows 10 2004, otherwise name and version of platform such as OpenShift v4.3): Windows 10 version 21H2
Java Version (when using VS Code or Theia, execute java -version and paste the details here):OpenJDK Runtime Environment Corretto-<IP_ADDRESS>.1 (build <IP_ADDRESS>+10-LTS)
[ ] Related to RSE API?

RSE API Plugin version:
Zowe CLI version:
Node.js version:

Logs attached (see here how to get them): no

Problem Description
Detailed steps for reproducing the problem:

First step

Observed behavior
I downloaded a cobol source file to my PC. I created a folder called C:\Cobol and put this file in there with the extension .cbl, so MYPGM.cbl. This source has a few INCLUDEs and COPY members. I created a subfolder in C:\Cobol and downloaded all those members in there with the extension .cpy, so CPYBOOK1.cpy, INCLUDE1.cpy and so on.
In the C:\Cobol folder I have a zapp.yaml file.  These are the contents of the file
name: sample-zapp     description: Wazi's code samples with COBOL application     version: 1.1.0     author:       name: IBM CORPORATION     ###############################################################     # This property group tells our extension to do the following:     #   First, look for copybooks inside local COPYBOOK folder     #   Second, look for copybooks on remote MVS syslib     ###############################################################     propertyGroups:       - name: cobol-copybooks         language: cobol         compilerOptions: ~(OR)         libraries:           - name: syslib             type: local             locations:               - "**/COPYBOOK"           - name: syslib             type: local             locations:               - "/COPYBOOK"                         - name: syslib             type: local             locations:               - "COPYBOOK"
I have the Z Open Editor extension installed.
I open the folder c:\Cobol in VSCode and click on the MYPGM.cbl file in the LHS pane. It opens the program and I can see the formatting and syntax highlighting etc.
But I get a series of messages
You opened a file with include files, but do not have a Property Group defined that specifies where to find them. Review the documentation and your settings or Zapp files.
And a tonne of undeclared variables
Expected behavior
-I shouldn't get that message and should not have an issue with undeclared variables.

@mahadea  You can check Z Open Editor log to see exactly why copybooks are not found.
Here are direction how to view the log: https://ibm.github.io/zopeneditor-about/Docs/locating_local_client_logs.html#troubleshooting-using-log-files
You can change the log level to "DEBUG" for most detailed oputput if "INFO" is not enough.
It's possible your zapp.yaml file is not formatted correctly, I cannot tell from your pasted text. Also, based on your example the subfolder should be called COPYBOOK.

That was helpful. There was indeed an issue with zapp.yaml. Sorry that I didn't pay attention to how it was copy pasted into this window and thank you for not giving me a hard time about it.  Dang those yaml files are finicky, almost as bad as JCL.

FWIW, I am attaching an image of the zapp.yaml file that finally worked for me


refactor(All): move build to rollup, update core deps

Follow up the release of<EMAIL_ADDRESS>&<EMAIL_ADDRESS>we can register global resources with direct view model references. Now ux elements as a single bundle file, like following pic

The entry configure will look like this:
import {
  FrameworkConfiguration,
  bindingMode,
} from 'aurelia-framework';

import { ValueAttributeObserver, EventSubscriber } from 'aurelia-binding';

import { AureliaUX } from '@aurelia-ux/core';
import { UxSelect } from './ux-select';
import { UxOptGroup } from './ux-optgroup';
import { UxOption } from './ux-option';

export { UxOption, UxOptionElement } from './ux-option';
export { UxOptGroup, UxOptGroupElement } from './ux-optgroup';
export { UxSelect, UxSelectElement } from './ux-select';
export { UxSelectTheme } from './ux-select-theme';

export function configure(config: FrameworkConfiguration) {
  config.container.get(AureliaUX).registerUxElementConfig(uxSelectConfig);
  config.globalResources([
    UxSelect,
    UxOptGroup,
    UxOption,
  ]);
}

const uxSelectConfig = {
  tagName: 'ux-select',
  properties: {
    value: {
      defaultBindingMode: bindingMode.twoWay,
      getObserver(element: Element, _: string) {
        return new ValueAttributeObserver(element, 'value', new EventSubscriber(['change']));
      }
    }
  }
};

This can be applied for all elements, which should help reduce build time too.
@EisenbergEffect @ZHollingshead I not sure how to update jspm dependencies properly.
resolves #162
TODO list:

[x] Button https://github.com/aurelia/ux/pull/175/commits/c4a82970593bf55b2d1a8d65e9a7bd913fec6e37
[x] Card https://github.com/aurelia/ux/pull/175/commits/9c20ebf3a0b9b4682e01b539898c087d91afdb5c
[x] Checkbox https://github.com/aurelia/ux/pull/175/commits/3f3ae9a9ce2e980f6fe8dc5824636bea6e561f3f
[x] ChipInput https://github.com/aurelia/ux/pull/175/commits/ff825f80222fa8d71c840a65fc878f286f6545b1
[x] Components
[x] Core https://github.com/aurelia/ux/pull/175/commits/c114d781d3f8306aae6c18cdd22e0343c58cabe8
[x] Datepicker https://github.com/aurelia/ux/pull/175/commits/e2273a4453dbd1057d9caf2cbaa0e306c0a8676f
[x] Form https://github.com/aurelia/ux/pull/175/commits/6adda495dae06f8feebaa24bf8268876bda46f79
[x] Grid https://github.com/aurelia/ux/pull/175/commits/b4c8bb46ad2001b11b726e85068d9d524bdc0b75
[x] Icons Renamed to Icon https://github.com/aurelia/ux/pull/175/commits/935254544c6eff034972fd3f9d9bc55da976903a
[x] Input https://github.com/aurelia/ux/pull/175/commits/12b61a6a86e40e039ae3610d6b95306a2224d4b6
[x] Input-info https://github.com/aurelia/ux/pull/175/commits/ce0d803866bcd5cf1da4ee658d1e80c50b2e503e
[x] List https://github.com/aurelia/ux/pull/175/commits/31ebac018335ed164096fac6f0595a972145d966
[x] Radio https://github.com/aurelia/ux/pull/175/commits/62f921540f7e0c27163d70d204d00305186f3557
[x] Select https://github.com/aurelia/ux/pull/175/commits/b8ccff4f5e93b2537e85d93767e8c17d04b4a4e8
[x] Switch https://github.com/aurelia/ux/pull/175/commits/6025ed0961aff95ecc9b58d12b9ffa82ce7c2df9
[x] Textarea https://github.com/aurelia/ux/pull/175/commits/a41573f6fa10fb14c778c7ed0f9a7c166559007a

@ZHollingshead Sorry Didn't state clearly WIP, I'm not sure if I should move wholesale to rollup, or one or few first and the rest later.

@ZHollingshead @EisenbergEffect  It would be also nice if we don't have to duplicate css files for each dist format, instead copy all css into a folder name css

@bigopon Oh, whoops. Yeah I will get in and start testing.
I would be okay with doing it all at once if it works.
On the folder structure, originally you proposed an idea where we had the dist folder look like the following:
css/
types/
amd-index.js
commonjs-index.js
es2015-index.js
native-modules.js

If we do that, then we can just put the folders in the root one time yeah? What is required for that to work though, is that a situation where we need to configure loaders specially?
To be honest I am not sure how to update JSPM dependencies either. I am most experienced with Webpack at this point.

Maybe @jods4 can comment on how to have it work with webpack, without having to nested inside a folder.

I'd love to see this converted all at once, if possible :)

@EisenbergEffect @ZHollingshead That's pretty much it for the base work. Needs to update Webpack for html import in test, beside css test for all loaders to see if any issues.

close as no longer relevant, resolved by #229