Feature Request: Docker image for compilation

Hi,
First of all, thank you for releasing your library. My team and I are looking forward to use it.
It would be nice to provide an ubuntu docker image where everything is pre-installed, such that we can compile the library without too much hassle.
TensorFlow provides one for TensorFlowLite. I tried it and it was very convenient.
Thanks!

@ltrottier thanks for this suggestion. We'll look into it.

@mark-arm I build the docker image for armNN compilation for Android device. I can push the Docker file.

Hi @nullbyte91 that's great! Could you attach it to this issue, just so I can see the sort of thing it would entail? I'm very interested.

@mark-arm Great. I'm excited to contribute to this community.
Please find the Dockerfile content below,
FROM ubuntu:18.04
ENV TERM linux
ENV DEBIAN_FRONTEND noninteractive

# Forward system proxy setting
ARG proxy
ENV http_proxy $proxy
ENV https_proxy $proxy

# Basic apt update
RUN apt-get update && DEBIAN_FRONTEND=noninteractive apt-get install -y --no-install-recommends locales ca-certificates &&  rm -rf /var/lib/apt/lists/*

# Set the locale to en_US.UTF-8, because the Yocto build fails without any locale set.
  Remove apt cache by removing  rm -rf /var/lib/apt/lists/*
# 2. Flatbuffers build Application.mk hardcode thing need to fix
# 3. Remove ARMNN from docker image. So, We can mount armnn source to local volume and we no need to
#    mount adb inside docker.

Mount ADB Inside docker:
docker run -it --rm --privileged --net host -v /dev/bus/usb:/dev/bus/usb armnn_x86:latest bash

I'm testing those basic test cases. Please review and give me your feedback. So, I can push these changes formally. We can optimize this in the near future.
Regards,
Jegathesan S

Hi @nullbyte91 this is cool, I would imagine that it could live in armnn/docker/armnn-android or somewhere similar.
I think my main concern is that in its current form it uses the master branches of Compute Library, ArmNN, Tensorflow, protobuf and flatbuffers, all downloaded from the internet. Doesn't this cause problems with the docker layer cache? (ie. you build it once, then next week build it again but instead of building anything it just returns the cache)? I think if you can fix your "To Do" number 3, then the other problems will also be solvable.
Please let me know if you think this is feasible.
All the best,
Matthew

Hi @MatthewARM,
Please find the updated Dockerfile and Created docker-entrypoint.sh startup script to avoid the layer cache problem. We can reduce the size of the docker image by parsing --no-cache param to docker build.
Startup strict install the ARMNN to local mount dir so we can avoid the adb mount inside docker. And, It checks the Compute Lib, Flatbuffer and Tensorflow local repo with remove ask the user to update or skip the git pull.
Dockerfile:
FROM ubuntu:18.04
ENV TERM linux
ENV DEBIAN_FRONTEND noninteractive

# Forward system proxy setting
# ARG proxy
# ENV http_proxy $proxy
# ENV https_proxy $proxy

# Basic apt update
RUN apt-get update && DEBIAN_FRONTEND=noninteractive apt-get install -y --no-install-recommends locales ca-certificates &&  rm -rf /var/lib/apt/lists/*

# Set the locale to en_US.UTF-8, because the Yocto build fails without any locale set.
 Flatbuffers build Application.mk hardcode value need to fix.

docker-entrypoint.sh
#!/bin/bash

set -e

dArmNN=/work
dComputeLib=/home/armnn-devenv/ComputeLibrary
dTensorflow=/home/armnn-devenv/google/tensorflow
dFlatBuffer=/home/armnn-devenv/flatbuffers

#Function to build ARMNN
function buildArmNN()
{
    mkdir -p ${dArmNN}/armnn-devenv && cd ${dArmNN}/armnn-devenv
    git clone https://github.com/ARM-software/armnn.git && cd armnn/
    mkdir build && cd build
    CXX=aarch64-linux-android-clang++ \
    CC=aarch64-linux-android-clang \
    CXX_FLAGS="-fPIE -fPIC" \
    cmake .. \
    -DCMAKE_SYSTEM_NAME=Android \
    -DCMAKE_ANDROID_ARCH_ABI=arm64-v8a \
    -DCMAKE_ANDROID_STANDALONE_TOOLCHAIN=/home/armnn-devenv/toolchains/aarch64-android-r17b/ \
    -DCMAKE_EXE_LINKER_FLAGS="-pie -llog" \
    -DARMCOMPUTE_ROOT=/home/armnn-devenv/ComputeLibrary/ \
    -DARMCOMPUTE_BUILD_DIR=/home/armnn-devenv/ComputeLibrary/build \
    -DBOOST_ROOT=/home/armnn-devenv/boost/install/ \
    -DARMCOMPUTENEON=1 -DARMCOMPUTECL=1 -DARMNNREF=1 \
    -DTF_GENERATED_SOURCES=/home/armnn-devenv/google/tf_pb/ -DBUILD_TF_PARSER=1 \
    -DPROTOBUF_ROOT=/home/armnn-devenv/google/arm64_pb_install/
    make -j8
}

# Function to update Compute Lib
function updateComputeLib()
{
    pushd ${dComputeLib}
    git pull
    scons arch=arm64-v8a neon=1 opencl=1 embed_kernels=1 extra_cxx_flags="-fPIC" \
    benchmark_tests=0 validation_tests=0 os=android -j8
    echo "Compute Lib updated"
    popd
}

# Function to update FlatBuffer Lib
function updateFlatBuffer()
{
    pushd ${dFlatBuffer}
    git pull
    cd android && cd jni && \
    rm -rf Application.mk && \
    echo "APP_STL := c++_static" >> Application.mk && \
    echo "NDK_TOOLCHAIN_VERSION" := clang >> Application.mk && \
    echo "APP_CPPFLAGS :=-std=c++11" >> Application.mk && \
    echo "APP_ABI := arm64-v8a" >> Application.mk && \
    echo "APP_PLATFORM := android-23" >> Application.mk && \
    echo "NDK_PLATFORM=android-23" >> Application.mk && \
    cd ../ && ndk-build -B
    echo "Compute Lib updated"
    popd
}

# Main
if [ ! -d "/work/armnn-devenv/armnn/" ];
then
    buildArmNN
fi

# Check Compute Library changes from repo
cd ${dComputeLib}
if [ $(git rev-parse HEAD) = $(git ls-remote $(git rev-parse --abbrev-ref @{u} |
sed 's/\// /g') | cut -f1) ]
then
    echo "Compute Lib Up-to-date"
else
    echo "New changes are availble for Compute Library repo."
    echo "Do you wanna update (y/n)?"
    read answer
    if [ "$answer" != "${answer#[Yy]}" ] ;then
        updateComputeLib
    fi
fi

# Check Tensorflow changes from repo
cd ${dTensorflow}
if [ $(git rev-parse HEAD) = $(git ls-remote $(git rev-parse --abbrev-ref @{u} |
sed 's/\// /g') | cut -f1) ]
then
    echo "Tensrflow Lib Up-to-date"
else
    echo "Tensrflow Lib Not Up-to-date"
    echo "Skipping for now. Issue: #267"
    #echo "New changes are availble for Compute Library repo."
    #echo "Do you wanna update (y/n)?"
    #read answer
    #if [ "$answer" != "${answer#[Yy]}" ] ;then
    #
    #fi
fi

# Check FlatBuffer changes from repo
cd ${dFlatBuffer}
if [ $(git rev-parse HEAD) = $(git ls-remote $(git rev-parse --abbrev-ref @{u} |
sed 's/\// /g') | cut -f1) ]
then
    echo "FlatBuffer Up-to-date"
else
    echo "FlatBuffer Not Up-to-date"
    echo "New changes are availble for Compute Library repo."
    echo "Do you wanna update (y/n)?"
    read answer
    if [ "$answer" != "${answer#[Yy]}" ] ;then
        updateFlatBuffer
    fi
fi

exec "$@"

I believe We can make these steps better in near future. Let me know if you have any other suggestions.
Thank You,
Jegathesan S

@MatthewARM Did you get a time to look this?

Hi @nullbyte91 sorry I forgot to get back to you - many thanks for this and I think this is in a good starting position to have a code review from the engineers. Are you comfortable with submitting a review on gerrit? We use https://review.mlplatform.org/admin/repos/ml/armnn for reviews.

@MatthewARM Yes. I will push the changes for review.

Cool. I'm also sharing the link to this ticket around - we can use this ticket for general discussion while we use Gerrit for reviewing specific commits into the code.

@MatthewARM Could you please help with this basic push issue. This is what I'm following,
$ git clone "https://review.mlplatform.org/ml/armnn" && (cd "armnn" && mkdir -p .git/hooks && curl -Lo `git rev-parse --git-dir`/hooks/commit-msg https://review.mlplatform.org/tools/hooks/commit-msg; chmod +x `git rev-parse --git-dir`/hooks/commit-msg)
 $ git checkout -b Issue_111
# Changes are made
$ git add & git commit [with signed-off]
$ git push origin Issue_111

And, I'm getting the below error,
remote: Resolving deltas: 100% (1/1)
remote: error: branch refs/heads/Issue_111:
remote: You need 'Create' rights to create new references.
remote: User: nullbyte91
remote: Contact an administrator to fix the permissions
remote: Processing changes: refs: 1, done
To https://review.mlplatform.org/ml/armnn
 ! [remote rejected] Issue_111 -> Issue_111 (prohibited by Gerrit: not permitted: create)
error: failed to push some refs to 'https://review.mlplatform.org/ml/armnn'

Thank You,
Jegathesan S

@MatthewARM submitted the code for Gerrit review and added you as the reviewer. Thanks.

Thanks for the submission Jegathesan.
It'll wind its way through the review process. I'm going to close this feature request now. You can track it through the review.
https://review.mlplatform.org/c/ml/armnn/+/2935
Colm.


Use DENO_HOME instead of DENO_DIR

The install script is the only place in denoland where the cache directory variable $DENO_DIR is also used to determine Deno's installation directory. This separates the two into $DENO_DIR and $DENO_HOME.
Now $DENO_HOME can be set to a custom location in your profile (for updates or third party tools) without the cache defaulting to the same, potentially root-owned, location.
cc @ry

Can we call it DENO_INSTALL instead of DENO_HOME (which I think is confusingly similar to DENO_DIR)?
Can you update the README too?

@ry Done 👍. I also like DENO_INSTALL because it better reflects its specificity to this repo.


Color Picker [F] Does not match decimal values to the RGB values

https://github.com/ScanMountGoat/ssbh_editor/assets/110957189/9725748d-2b24-4f1f-9292-7e6f9fd11c8d

Once this is solved. I think the copy to clipboard button on the color picker would be much more useful it was supplemented with a paste button so we can just transfer RGB values.

This looks like some sort of gamma conversion is being applied when converting to floats in the color picker. The floating point values you see in the material editor are the values that will be written to the numatb and rendered in game.

I understand that may be the case, but removing that gamma conversion visual and matching it up with what is in the viewport would make more sense to someone working on editing materials via color picker.

I've updated the wiki to clarify this. The color picker will be changed to use sRGB gamma for the next release to make the float values match what you see in the editor. The mismatch you see in the color picker won't impact the actual values in game, so I won't be prioritizing a new release any time soon.


ToolBar can't add ToolButtons

Trying to follow this example.
VCL = require "vcl.core"
VCL.Application():Initialize()
dAlert = VCL.Form(nil,'dAlert')
tb = VCL.ToolBar(dAlert, "tb", {Align="alBottom", ShowCaptions=true})
bSave = VCL.ToolButton(tb,'b1',{Caption="aaaaa",AutoSize=true, Style="tbsButton", Left = tb.Left})
bCancel=VCL.ToolButton(tb,'b2',{Caption="bbbbb",AutoSize=true, Style="tbsButton", Left = tb.Left})
dAlert:ShowModal()

No toolbuttons are added to the toolbar. But other things like a Button are added ok.

This is a bug in the lua_make/config.lua script.
The noparent flag must be removed from ToolButton class.
{ unit = "ToolBar", ref = "ComCtrls, Controls, ImgList, LuaImageList", canvas=true,
		classes = {
			{ name = "ToolButton", src = "TToolButton", noparent=true,},
			{ name = "ToolBar", src = "TToolBar" },
		}
	},

Version 0.9.2 is out, binaries are updated, toolbar example added.
j.

Version 0.9.2 is out, binaries are updated, toolbar example added. j.

Is it retracted? This folder has no files.
PS: here, not to spam with issues: wrong copyright year in lib examples and vt-form sources, missing class reference for GroupBox (I even thought it wasn't implemented)

Sorry just staged for a while.

wrong copyright year in lib examples and vt-form sources, missing class reference for GroupBox (I even thought it wasn't implemented), and that class reference wasn't regenerated for 0.9.2
Could you explain more?
GroupBox is implemented and working for me...
Thanks for copyright warning, i hope that vt-form can be useful tool, but it's have lots of issues...
Also lua-lar is available for lua5.4, just check it.
j

GroupBox is just missing from the docs since the docs (class reference) were not updated to reflect later changes. I haven't tested the control yet.

Thank you!


Arrange buttons do the opposite of what they are supposed to

Problem
Pressing

causes the layer to move up.
Pressing

causes the layer to move down.
Fix
These should be switched around.

@Mxchaeltrxn This is a funny one! It's current behaviour is actually intentional… but that doesn't mean that it's correct.
When a user moves a layer forward it is correctly being moved in front of other layers on the canvas (it's component is rendered after the layers before it and therefore appears to be in front of it). What's perhaps "wrong" is that the layers are listed on the left in the same order which means the layer that's physically at the front is listed at the bottom.
Should we reverse the order the layers are listed in? I honestly don't know. Both Sketch and Figma render layer in that order "logical order" (it seems to be default in design tools these days), however the Adobe suite (Photoshop, Illustrator etc) all render the layers in their "physical order" instead. 🤷♀

@haydn After reading what you wrote, it seems that both behaviours are reasonable. However, I was not clear in my description because I was not aware of the button's complete functionality.
The bit that confuses me is the iconography and how it relates to its action. I wasn't actually sure what this button did when I saw it so I tried clicking it. I thought it was odd that the button with the downward pointing arrow moved the selected layer up in the left hand Layers column and the button upward pointing arrow did the opposite. I wasn't actually aware that it changed the layering of the layers until you told me. I couldn't find this behaviour in Figma in the browser so I couldn't compare so it could just be me misinterpreting the icon.
Perhaps labeling the section Arrange layering rather than Arrange might help because I thought Arrange meant Arrange in left hand Layers column?

Here's where the functionality exists in Figma:

Turns out I was wrong about Figma's behaviour — it does display the list of layers on the left in their physical order:

That seems like the best setup, I reckon we copy that behaviour.
I agree that "Arrange" isn't very intuitive — maybe the section could be called "Order"?
I'll update the issues original description with the new plan.

@haydn I like that. I can do this one!


Way to easily collapse folders having too many files or sub folders

To be precise I am talking about node_modules folder which gets opened a lot of times when I am debugging things down to library level (say clicking a type definition or method implementation or putting logs or setting breakpoints in it). There is no convenient way to actually collapse it back without scrolling all the way up. Not sure if there is an easy way to tackle this.

Try press left arrow button.

Thanks @yubaoquan
Closing this as pressing left arrow buttons collapses the parent folder and if you keep pressing it, it will eventually collapse node_modules folder as well.


Elasticsearch docker images build failure

I was building fresh images using the 1.13.1 release. While building docker images for elasticsearch (make docker-images-elastic), I experienced the following error:
Step 3/5 : RUN pip install elasticsearch elasticsearch-curator
 ---> Running in 6af4f5500721
Collecting elasticsearch
  Downloading https://files.pythonhosted.org/packages/ae/43/38329621bcca6f0b97e1cc36fb3cef889414a1960fcdc83a41e26b496634/elasticsearch-7.0.2-py2.py3-none-any.whl (83kB)
Collecting elasticsearch-curator
  Downloading https://files.pythonhosted.org/packages/18/12/b47c869f538f2d8c5766358288d8b5e0e6f27b34ed7131f8ccf18d2f396c/elasticsearch-curator-5.7.6.tar.gz (218kB)
    ERROR: Command errored out with exit status 1:
     command: /usr/local/bin/python -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-zlkqritl/elasticsearch-curator/setup.py'"'"'; __file__='"'"'/tmp/pip-install-zlkqritl/elasticsearch-curator/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base pip-egg-info
         cwd: /tmp/pip-install-zlkqritl/elasticsearch-curator/
    Complete output (117 lines):

    libyaml is not found or a compiler error: forcing --without-libyaml
    (if libyaml is installed correctly, you may need to
     specify the option --include-dirs or uncomment and
     modify the parameter include_dirs in setup.cfg)
    zip_safe flag not set; analyzing archive contents...

    Installed /tmp/pip-install-zlkqritl/elasticsearch-curator/.eggs/PyYAML-3.12-py3.7-linux-x86_64.egg
    Searching for click<7.0,>=6.7
    Reading https://pypi.org/simple/click/
    Downloading https://files.pythonhosted.org/packages/34/c1/8806f99713ddb993c5366c362b2f908f18269f8d792aff1abfd700775a77/click-6.7-py2.py3-none-any.whl#sha256=29f99fc6125fbc931b758dc053b3114e55c77a6e4c6c3a2674a2dc986016381d
    Best match: click 6.7
    Processing click-6.7-py2.py3-none-any.whl
    Installing click-6.7-py2.py3-none-any.whl to /tmp/pip-install-zlkqritl/elasticsearch-curator/.eggs

    Installed /tmp/pip-install-zlkqritl/elasticsearch-curator/.eggs/click-6.7-py3.7.egg
    Searching for requests_aws4auth>=0.9
    Reading https://pypi.org/simple/requests_aws4auth/
    Downloading https://files.pythonhosted.org/packages/a4/4e/29ac240bff356eb630bcc729da7cdce6f55f21c442af325ca60dd27d4a3f/requests_aws4auth-0.9-py2.py3-none-any.whl#sha256=e20e4941ccd5706973068f9214d40cb2e669461536b3a57b9ac824ae87744c2c
    Best match: requests-aws4auth 0.9
    Processing requests_aws4auth-0.9-py2.py3-none-any.whl
    Installing requests_aws4auth-0.9-py2.py3-none-any.whl to /tmp/pip-install-zlkqritl/elasticsearch-curator/.eggs
    writing requirements to /tmp/pip-install-zlkqritl/elasticsearch-curator/.eggs/requests_aws4auth-0.9-py3.7.egg/EGG-INFO/requires.txt

    Installed /tmp/pip-install-zlkqritl/elasticsearch-curator/.eggs/requests_aws4auth-0.9-py3.7.egg
    Searching for boto3>=1.9.142
    Reading https://pypi.org/simple/boto3/
    Downloading https://files.pythonhosted.org/packages/05/90/f9d70a9e0d3f5d9d6fe634b9cf8c17a8fd42ee3ceffbb82a357df16f7e61/boto3-1.9.204-py2.py3-none-any.whl#sha256=3ec5b520dbd0a430cdd581a8250991fb0f21ee7e668a8928f15006b312fa86dc
    Best match: boto3 1.9.204
    Processing boto3-1.9.204-py2.py3-none-any.whl
    Installing boto3-1.9.204-py2.py3-none-any.whl to /tmp/pip-install-zlkqritl/elasticsearch-curator/.eggs
    writing requirements to /tmp/pip-install-zlkqritl/elasticsearch-curator/.eggs/boto3-1.9.204-py3.7.egg/EGG-INFO/requires.txt

    Installed /tmp/pip-install-zlkqritl/elasticsearch-curator/.eggs/boto3-1.9.204-py3.7.egg
    Searching for requests>=2.20.0
    Reading https://pypi.org/simple/requests/
    Downloading https://files.pythonhosted.org/packages/51/bd/23c926cd341ea6b7dd0b2a00aba99ae0f828be89d72b2190f27c11d4b7fb/requests-2.22.0-py2.py3-none-any.whl#sha256=9cf5292fcd0f598c671cfc1e0d7d1a7f13bb8085e9a590f48c010551dc6c4b31
    Best match: requests 2.22.0
    Processing requests-2.22.0-py2.py3-none-any.whl
    Installing requests-2.22.0-py2.py3-none-any.whl to /tmp/pip-install-zlkqritl/elasticsearch-curator/.eggs
    writing requirements to /tmp/pip-install-zlkqritl/elasticsearch-curator/.eggs/requests-2.22.0-py3.7.egg/EGG-INFO/requires.txt

    Installed /tmp/pip-install-zlkqritl/elasticsearch-curator/.eggs/requests-2.22.0-py3.7.egg
    Searching for urllib3<1.25,>=1.24.2
    Reading https://pypi.org/simple/urllib3/
    Downloading https://files.pythonhosted.org/packages/01/11/525b02e4acc0c747de8b6ccdab376331597c569c42ea66ab0a1dbd36eca2/urllib3-1.24.3-py2.py3-none-any.whl#sha256=a637e5fae88995b256e3409dc4d52c2e2e0ba32c42a6365fee8bbd2238de3cfb
    Best match: urllib3 1.24.3
    Processing urllib3-1.24.3-py2.py3-none-any.whl
    Installing urllib3-1.24.3-py2.py3-none-any.whl to /tmp/pip-install-zlkqritl/elasticsearch-curator/.eggs
    writing requirements to /tmp/pip-install-zlkqritl/elasticsearch-curator/.eggs/urllib3-1.24.3-py3.7.egg/EGG-INFO/requires.txt

    Installed /tmp/pip-install-zlkqritl/elasticsearch-curator/.eggs/urllib3-1.24.3-py3.7.egg
    Searching for elasticsearch<8.0.0,>=7.0.0
    Reading https://pypi.org/simple/elasticsearch/
    Downloading https://files.pythonhosted.org/packages/ae/43/38329621bcca6f0b97e1cc36fb3cef889414a1960fcdc83a41e26b496634/elasticsearch-7.0.2-py2.py3-none-any.whl#sha256=d1b176b87a7fb75dca82978c82a4023e8b21cbc98f4018cb51190fb0b8b43764
    Best match: elasticsearch 7.0.2
    Processing elasticsearch-7.0.2-py2.py3-none-any.whl
    Installing elasticsearch-7.0.2-py2.py3-none-any.whl to /tmp/pip-install-zlkqritl/elasticsearch-curator/.eggs
    writing requirements to /tmp/pip-install-zlkqritl/elasticsearch-curator/.eggs/elasticsearch-7.0.2-py3.7.egg/EGG-INFO/requires.txt

    Installed /tmp/pip-install-zlkqritl/elasticsearch-curator/.eggs/elasticsearch-7.0.2-py3.7.egg
    Searching for voluptuous>=0.9.3
    Reading https://pypi.org/simple/voluptuous/
    Downloading https://files.pythonhosted.org/packages/59/95/fa6218477c6999c9b7fdfab7c12c1bd4da2d5930f5eb2b232ec74eb344e7/voluptuous-0.11.5-py2.py3-none-any.whl#sha256=303542b3fc07fb52ec3d7a1c614b329cdbee13a9d681935353d8ea56a7bfa9f1
    Best match: voluptuous 0.11.5
    Processing voluptuous-0.11.5-py2.py3-none-any.whl
    Installing voluptuous-0.11.5-py2.py3-none-any.whl to /tmp/pip-install-zlkqritl/elasticsearch-curator/.eggs

    Installed /tmp/pip-install-zlkqritl/elasticsearch-curator/.eggs/voluptuous-0.11.5-py3.7.egg
    Searching for s3transfer<0.3.0,>=0.2.0
    Reading https://pypi.org/simple/s3transfer/
    Downloading https://files.pythonhosted.org/packages/16/8a/1fc3dba0c4923c2a76e1ff0d52b305c44606da63f718d14d3231e21c51b0/s3transfer-0.2.1-py2.py3-none-any.whl#sha256=b780f2411b824cb541dbcd2c713d0cb61c7d1bcadae204cdddda2b35cef493ba
    Best match: s3transfer 0.2.1
    Processing s3transfer-0.2.1-py2.py3-none-any.whl
    Installing s3transfer-0.2.1-py2.py3-none-any.whl to /tmp/pip-install-zlkqritl/elasticsearch-curator/.eggs
    writing requirements to /tmp/pip-install-zlkqritl/elasticsearch-curator/.eggs/s3transfer-0.2.1-py3.7.egg/EGG-INFO/requires.txt

    Installed /tmp/pip-install-zlkqritl/elasticsearch-curator/.eggs/s3transfer-0.2.1-py3.7.egg
    Searching for jmespath<1.0.0,>=0.7.1
    Reading https://pypi.org/simple/jmespath/
    Downloading https://files.pythonhosted.org/packages/83/94/7179c3832a6d45b266ddb2aac329e101367fbdb11f425f13771d27f225bb/jmespath-0.9.4-py2.py3-none-any.whl#sha256=3720a4b1bd659dd2eecad0666459b9788813e032b83e7ba58578e48254e0a0e6
    Best match: jmespath 0.9.4
    Processing jmespath-0.9.4-py2.py3-none-any.whl
    Installing jmespath-0.9.4-py2.py3-none-any.whl to /tmp/pip-install-zlkqritl/elasticsearch-curator/.eggs

    Installed /tmp/pip-install-zlkqritl/elasticsearch-curator/.eggs/jmespath-0.9.4-py3.7.egg
    Searching for botocore<1.13.0,>=1.12.204
    Reading https://pypi.org/simple/botocore/
    No local packages or working download links found for botocore<1.13.0,>=1.12.204
    Traceback (most recent call last):
      File "/tmp/pip-install-zlkqritl/elasticsearch-curator/setup.py", line 39, in <module>
        from cx_Freeze import setup, Executable
    ModuleNotFoundError: No module named 'cx_Freeze'

    During handling of the above exception, another exception occurred:

    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-install-zlkqritl/elasticsearch-curator/setup.py", line 175, in <module>
        tests_require = ["mock", "nose", "coverage", "nosexcover"]
      File "/usr/local/lib/python3.7/site-packages/setuptools/__init__.py", line 144, in setup
        _install_setup_requires(attrs)
      File "/usr/local/lib/python3.7/site-packages/setuptools/__init__.py", line 139, in _install_setup_requires
        dist.fetch_build_eggs(dist.setup_requires)
      File "/usr/local/lib/python3.7/site-packages/setuptools/dist.py", line 717, in fetch_build_eggs
        replace_conflicting=True,
      File "/usr/local/lib/python3.7/site-packages/pkg_resources/__init__.py", line 782, in resolve
        replace_conflicting=replace_conflicting
      File "/usr/local/lib/python3.7/site-packages/pkg_resources/__init__.py", line 1065, in best_match
        return self.obtain(req, installer)
      File "/usr/local/lib/python3.7/site-packages/pkg_resources/__init__.py", line 1077, in obtain
        return installer(requirement)
      File "/usr/local/lib/python3.7/site-packages/setuptools/dist.py", line 784, in fetch_build_egg
        return cmd.easy_install(req)
      File "/usr/local/lib/python3.7/site-packages/setuptools/command/easy_install.py", line 673, in easy_install
        raise DistutilsError(msg)
    distutils.errors.DistutilsError: Could not find suitable distribution for Requirement.parse('botocore<1.13.0,>=1.12.204')
    ----------------------------------------
ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.

I believe this is because the pip dependencies are not pinned. When I change in both docker files under plugin/storage/es:
RUN pip install elasticsearch elasticsearch-curator

to
RUN pip install elasticsearch==7.0.2 elasticsearch-curator==5.7.2

the error disappears. The most recent curator version 5.7.6 does not appear to work. Forcing the lower version helps. However, the urlib3 version needs to be bumped to 1.24.1, as per warning with forced curator:
ERROR: elasticsearch-curator 5.7.2 has requirement urllib3==1.24.1, but you'll have urllib3 1.21.1 which is incompatible.

It would be great if all dependencies have been pinned at exact versions.

would like like to create a pull request?  Dockerfiles are in plugin/storage/es

Was fixed in https://github.com/jaegertracing/jaeger/pull/1495


Google calendar - make authentication simpler and document it

@flozz wrote a blog post on https://blog.lasall.dev/post/tell-me-why-google-and-caldav/ on how to connect to Google - I (or someone) should pick up on it and try to make it more convenient to use the caldav library.
The google_*-python libraries should be an optional dependency, they should not be added to the requirement list, but I think it's OK to load them when needed.

Note: I am not the author of this blog post. My message in #310 is probably not clear enough 😅
But I may write a post on this subject one day on my own blog 😁


Added stat feature at the director

Closes #367
This PR introduced a new feature called stat at the director, which allows user to query origins in the federation whether they would serve an object. It supports the case where multiple origins serving the same object.
The stat feature is accessible via the RESTful API at the director. Authenticate yourself first, and visit /api/v1.0/director_ui/servers/origins/stat and append your object path to query for it. i.e. /api/v1.0/director_ui/servers/origins/stat<absolute-path-to-your-object>. You can also pass min and max as query parameter to specify minimum/maximum # of successful responses required.
Since this API needs to be behind auth, it is required that director pkg imports the AuthHandler function from web_ui pkg. The exiting dependency graph breaks with cyclic import due to the fact that almost every server package imports director for getting the NamespaceAd and a bunch more type definition. To mitigate this issue and reduce the headache for future work, this PR pulled all common type definition from director pkg to a new pkg called common. Therefore there are many changes not directly related to the new feature, which are included in commit 2d9bd8f

Using admin powers to bypass waiting for extra review, since Brian mentioned doing so this morning.


When I register I enter my birthday

Not the year. Just the month and day.

Prompt existing users to enter their birthday if they haven’t filled it in?


Docker core check "invalid image name" log spam on kubernetes cluster

Describe what happened:
The agent seems to be producing log spam of the nature:
2021-09-28 07:35:03 UTC | CORE | ERROR | (pkg/collector/corechecks/containers/docker/docker.go:64 in updateContainerRunningCount) | Cannot split the image name sha256:41adeff1b17af5dd87ce40fadd5f702460acd9b78fdc5ec5c3db14c1377aba40: invalid image name (is a sha256)

Many such lines are printed approximately every minute. The issue is related to the docker core check.
Describe what you expected:
Given that this does not seem to be an actual erroneous condition, I would expect that the logger would not produce hundreds of these ERROR log lines and gracefully handle the condition.
Steps to reproduce the issue:
Deployed the datadog helm chart onto a k8s cluster with other workloads.
Additional environment details (Operating System, Cloud provider, etc):
AWS EKS / k8s version 1.19
Output of the info page (if this is a bug)
===============
Agent (v7.31.0)
===============

  Status date: 2021-09-28 07:31:04.509 UTC<PHONE_NUMBER>509)
  Agent start: 2021-09-28 07:01:40.207 UTC<PHONE_NUMBER>207)
  Pid: 1
  Go Version: go1.15.13
  Python Version: 3.8.11
  Build arch: amd64
  Agent flavor: agent
  Check Runners: 4
  Log Level: INFO

  Paths
  =====
    Config File: /etc/datadog-agent/datadog.yaml
    conf.d: /etc/datadog-agent/conf.d
    checks.d: /etc/datadog-agent/checks.d

  Clocks
  ======
    NTP offset: -67µs
    System time: 2021-09-28 07:31:04.509 UTC<PHONE_NUMBER>509)

  Host Info
  =========
    bootTime: 2021-05-24 12:47:41 UTC<PHONE_NUMBER>000)
    kernelArch: x86_64
    kernelVersion: 5.4.117-58.216.amzn2.x86_64
    os: linux
    platform: ubuntu
    platformFamily: debian
    platformVersion: 21.04
    procs: 674
    uptime: 3042h14m7s

  Hostnames
  =========
    ec2-hostname: ip-10-128-1-212.us-west-2.compute.internal
    host_aliases: [ip-10-128-1-212.us-west-2.compute.internal-eks-dev]
    hostname: i-007e0e8de9091aa01
    instance-id: i-007e0e8de9091aa01
    socket-fqdn: datadog-vw78n
    socket-hostname: datadog-vw78n
    host tags:
      cluster_name:eks-dev
      kube_cluster_name:eks-dev
    hostname provider: aws
    unused hostname providers:
      azure: azure_hostname_style is set to 'os'
      configuration/environment: hostname is empty
      gce: unable to retrieve hostname from GCE: status code 404 trying to GET http://<IP_ADDRESS>/computeMetadata/v1/instance/hostname

  Metadata
  ========
    cloud_provider: AWS
    hostname_source: aws

=========
Collector
=========

  Running Checks
  ==============

    cpu
    ---
      Instance ID: cpu [OK]
      Configuration Source: file:/etc/datadog-agent/conf.d/cpu.d/conf.yaml.default
      Total Runs: 118
      Metric Samples: Last Run: 9, Total: 1,055
      Events: Last Run: 0, Total: 0
      Service Checks: Last Run: 0, Total: 0
      Average Execution Time : 0s
      Last Execution Date : 2021-09-28 07:31:03 UTC<PHONE_NUMBER>000)
      Last Successful Execution Date : 2021-09-28 07:31:03 UTC<PHONE_NUMBER>000)

    disk (4.4.0)
    ------------
      Instance ID: disk:e5dffb8bef24336f [OK]
      Configuration Source: file:/etc/datadog-agent/conf.d/disk.d/conf.yaml.default
      Total Runs: 117
      Metric Samples: Last Run: 208, Total: 24,336
      Events: Last Run: 0, Total: 0
      Service Checks: Last Run: 0, Total: 0
      Average Execution Time : 158ms
      Last Execution Date : 2021-09-28 07:30:56 UTC<PHONE_NUMBER>000)
      Last Successful Execution Date : 2021-09-28 07:30:56 UTC<PHONE_NUMBER>000)

    docker
    ------
      Instance ID: docker [OK]
      Configuration Source: file:/etc/datadog-agent/conf.d/docker.d/conf.yaml.default
      Total Runs: 117
      Metric Samples: Last Run: 1,700, Total: 197,742
      Events: Last Run: 1, Total: 3
      Service Checks: Last Run: 1, Total: 117
      Average Execution Time : 932ms
      Last Execution Date : 2021-09-28 07:31:04 UTC<PHONE_NUMBER>000)
      Last Successful Execution Date : 2021-09-28 07:31:04 UTC<PHONE_NUMBER>000)

    file_handle
    -----------
      Instance ID: file_handle [OK]
      Configuration Source: file:/etc/datadog-agent/conf.d/file_handle.d/conf.yaml.default
      Total Runs: 117
      Metric Samples: Last Run: 5, Total: 585
      Events: Last Run: 0, Total: 0
      Service Checks: Last Run: 0, Total: 0
      Average Execution Time : 0s
      Last Execution Date : 2021-09-28 07:30:54 UTC<PHONE_NUMBER>000)
      Last Successful Execution Date : 2021-09-28 07:30:54 UTC<PHONE_NUMBER>000)

    io
    --
      Instance ID: io [OK]
      Configuration Source: file:/etc/datadog-agent/conf.d/io.d/conf.yaml.default
      Total Runs: 117
      Metric Samples: Last Run: 52, Total: 6,048
      Events: Last Run: 0, Total: 0
      Service Checks: Last Run: 0, Total: 0
      Average Execution Time : 0s
      Last Execution Date : 2021-09-28 07:31:01 UTC<PHONE_NUMBER>000)
      Last Successful Execution Date : 2021-09-28 07:31:01 UTC<PHONE_NUMBER>000)

    istio (3.14.0)
    --------------
      Instance ID: istio:21edaea6997f7fd0 [OK]
      Configuration Source: file:/etc/datadog-agent/conf.d/istio.d/auto_conf.yaml
      Total Runs: 117
      Metric Samples: Last Run: 8,448, Total: 988,416
      Events: Last Run: 0, Total: 0
      Service Checks: Last Run: 0, Total: 0
      Average Execution Time : 3.695s
      Last Execution Date : 2021-09-28 07:30:56 UTC<PHONE_NUMBER>000)
      Last Successful Execution Date : 2021-09-28 07:30:56 UTC<PHONE_NUMBER>000)

      Instance ID: istio:4b6a6420b74cf3d6 [OK]
      Configuration Source: file:/etc/datadog-agent/conf.d/istio.d/auto_conf.yaml
      Total Runs: 117
      Metric Samples: Last Run: 8, Total: 936
      Events: Last Run: 0, Total: 0
      Service Checks: Last Run: 0, Total: 0
      Average Execution Time : 30ms
      Last Execution Date : 2021-09-28 07:30:58 UTC<PHONE_NUMBER>000)
      Last Successful Execution Date : 2021-09-28 07:30:58 UTC<PHONE_NUMBER>000)

    kubelet (7.0.0)
    ---------------
      Instance ID: kubelet:5bbc63f3938c02f4 [OK]
      Configuration Source: file:/etc/datadog-agent/conf.d/kubelet.d/conf.yaml.default
      Total Runs: 88
      Metric Samples: Last Run: 1,882, Total: 165,486
      Events: Last Run: 0, Total: 0
      Service Checks: Last Run: 4, Total: 352
      Average Execution Time : 4.205s
      Last Execution Date : 2021-09-28 07:30:54 UTC<PHONE_NUMBER>000)
      Last Successful Execution Date : 2021-09-28 07:30:54 UTC<PHONE_NUMBER>000)

    load
    ----
      Instance ID: load [OK]
      Configuration Source: file:/etc/datadog-agent/conf.d/load.d/conf.yaml.default
      Total Runs: 117
      Metric Samples: Last Run: 6, Total: 702
      Events: Last Run: 0, Total: 0
      Service Checks: Last Run: 0, Total: 0
      Average Execution Time : 0s
      Last Execution Date : 2021-09-28 07:30:53 UTC<PHONE_NUMBER>000)
      Last Successful Execution Date : 2021-09-28 07:30:53 UTC<PHONE_NUMBER>000)

    memory
    ------
      Instance ID: memory [OK]
      Configuration Source: file:/etc/datadog-agent/conf.d/memory.d/conf.yaml.default
      Total Runs: 117
      Metric Samples: Last Run: 18, Total: 2,106
      Events: Last Run: 0, Total: 0
      Service Checks: Last Run: 0, Total: 0
      Average Execution Time : 0s
      Last Execution Date : 2021-09-28 07:31:00 UTC<PHONE_NUMBER>000)
      Last Successful Execution Date : 2021-09-28 07:31:00 UTC<PHONE_NUMBER>000)

    network (2.3.0)
    ---------------
      Instance ID: network:d884b5186b651429 [OK]
      Configuration Source: file:/etc/datadog-agent/conf.d/network.d/conf.yaml.default
      Total Runs: 117
      Metric Samples: Last Run: 307, Total: 35,721
      Events: Last Run: 0, Total: 0
      Service Checks: Last Run: 0, Total: 0
      Average Execution Time : 102ms
      Last Execution Date : 2021-09-28 07:30:52 UTC<PHONE_NUMBER>000)
      Last Successful Execution Date : 2021-09-28 07:30:52 UTC<PHONE_NUMBER>000)

    ntp
    ---
      Instance ID: ntp:d884b5186b651429 [OK]
      Configuration Source: file:/etc/datadog-agent/conf.d/ntp.d/conf.yaml.default
      Total Runs: 2
      Metric Samples: Last Run: 1, Total: 2
      Events: Last Run: 0, Total: 0
      Service Checks: Last Run: 1, Total: 2
      Average Execution Time : 0s
      Last Execution Date : 2021-09-28 07:16:48 UTC<PHONE_NUMBER>000)
      Last Successful Execution Date : 2021-09-28 07:16:48 UTC<PHONE_NUMBER>000)

    redisdb (4.1.0)
    ---------------
      Instance ID: redisdb:1808b16535a5a372 [OK]
      Configuration Source: file:/etc/datadog-agent/conf.d/redisdb.d/auto_conf.yaml
      Total Runs: 117
      Metric Samples: Last Run: 51, Total: 6,099
      Events: Last Run: 0, Total: 0
      Service Checks: Last Run: 1, Total: 117
      Average Execution Time : 23ms
      Last Execution Date : 2021-09-28 07:30:50 UTC<PHONE_NUMBER>000)
      Last Successful Execution Date : 2021-09-28 07:30:50 UTC<PHONE_NUMBER>000)
      metadata:
        version.major: 5
        version.minor: 0
        version.patch: 13
        version.raw: 5.0.13
        version.scheme: semver

    uptime
    ------
      Instance ID: uptime [OK]
      Configuration Source: file:/etc/datadog-agent/conf.d/uptime.d/conf.yaml.default
      Total Runs: 117
      Metric Samples: Last Run: 1, Total: 117
      Events: Last Run: 0, Total: 0
      Service Checks: Last Run: 0, Total: 0
      Average Execution Time : 0s
      Last Execution Date : 2021-09-28 07:30:59 UTC<PHONE_NUMBER>000)
      Last Successful Execution Date : 2021-09-28 07:30:59 UTC<PHONE_NUMBER>000)

========
JMXFetch
========

  Information
  ==================
  Initialized checks
  ==================
    no checks

  Failed checks
  =============
    no checks

=========
Forwarder
=========

  Transactions
  ============
    Cluster: 0
    CronJob: 0
    DaemonSet: 0
    Deployment: 0
    Dropped: 0
    DroppedOnInput: 0
    Job: 0
    Node: 0
    PersistentVolume: 0
    PersistentVolumeClaim: 0
    Pod: 0
    ReplicaSet: 0
    Requeued: 0
    Retried: 0
    RetryQueueSize: 0
    Service: 0
    StatefulSet: 0

  Transaction Successes
  =====================
    Total number: 598
    Successes By Endpoint:
      check_run_v1: 117
      intake: 15
      series_v1: 466

  API Keys status
  ===============
    API key ending with 57c25: API Key valid

==========
Endpoints
==========
  https://app.datadoghq.com - API Key ending with:
      - 57c25

==========
Logs Agent
==========

  Logs Agent is not running

=========
APM Agent
=========
  Status: Running
  Pid: 1
  Uptime: 1764 seconds
  Mem alloc: 16,090,176 bytes
  Hostname: i-007e0e8de9091aa01
  Receiver: <IP_ADDRESS>:8126
  Endpoints:
    https://trace.agent.datadoghq.com

  Receiver (previous minute)
  ==========================
    From python 3.9.7 (CPython), client 0.53.0
      Traces received: 730 (1,392,592 bytes)
      Spans received: 4327

    From python 3.9.7 (CPython), client 0.53.2
      Traces received: 66 (48,208 bytes)
      Spans received: 66

    Default priority sampling rate: 10.3%

  Writer (previous minute)
  ========================
    Traces: 0 payloads, 0 traces, 0 events, 0 bytes
    Stats: 0 payloads, 0 stats buckets, 0 bytes

=========
Aggregator
=========
  Checks Metric Sample: 1,432,341
  Dogstatsd Metric Sample: 33,708
  Event: 4
  Events Flushed: 4
  Number Of Flushes: 117
  Series Flushed: 1,423,673
  Service Check: 2,083
  Service Checks Flushed: 2,193
=========
DogStatsD
=========
  Event Packets: 0
  Event Parse Errors: 0
  Metric Packets: 33,707
  Metric Parse Errors: 0
  Service Check Packets: 0
  Service Check Parse Errors: 0
  Udp Bytes: 3,800,612
  Udp Packet Reading Errors: 0
  Udp Packets: 17,399
  Uds Bytes: 0
  Uds Origin Detection Errors: 0
  Uds Packet Reading Errors: 0
  Uds Packets: 1
  Unterminated Metric Errors: 0

=====================
Datadog Cluster Agent
=====================

  - Datadog Cluster Agent endpoint detected: https://<IP_ADDRESS>:5005
  Successfully connected to the Datadog Cluster Agent.
  - Running: 1.15.0+commit.6781e85

Just upgraded to the latest datadog agent and it's still happening.

Having the same issue v7.32.4 Datadog Agent...

Having similar issues and these messages are polluting the logs and unnecessarily growing their size.
Example:
| CORE | ERROR | (pkg/collector/corechecks/containers/docker/docker.go:64 in updateContainerRunningCount) | Cannot split the image name sha256:c8c23118d9ba3dcc65deacf098dc15a2d1dc10cc60ebd80b91732a3d4a3eb1cc: invalid image name (is a sha256)

Having similar issues
2022-03-01 08:11:42 UTC | PROCESS | WARN | (pkg/workloadmeta/collectors/kubelet/kubelet.go:167 in parsePodContainers) | cannot split image name "sha256:dd5a13fea6fd412ce447742a6c74400527e06afe10a278fccf52e7eb2de43bc8": invalid image name (is a sha256)
2022-03-01 08:11:41 UTC | CORE | WARN | (pkg/workloadmeta/collectors/kubelet/kubelet.go:167 in parsePodContainers) | cannot split image name "sha256:dd5a13fea6fd412ce447742a6c74400527e06afe10a278fccf52e7eb2de43bc8": invalid image name (is a sha256)

seeing the same errors related to ingress-controller pods and looks like this casing DD to not collect/push logs for those containers


Users can decide whether the item should be selected according to the indexPath and number of selections.

Add shouldAddToSelection(indexPath:numSelections:) to YPImagePickerDelegate.

@kf99916 This is interesting. Could you please describe a use case for such a feature? The config.library.maxNumberOfItems property already provides a way to limit the number of selected  medias.

@s4cha Take the user permission example: a logged-in user is allowed to select multiple medias, but a guest cannot do this. The app shows a alert whenever a guest wants to select multiple medias. In this case, we can implement in the following way:
config.library.maxNumberOfItems = 3
func shouldAddToSelection(indexPath:numSelections:) {
   if logged { return true }
   // Show alert
   return false
}

@kf99916 This looks like a reasonable addition and this has little impact on the existing api so let's go for it :)
PS: Please excuse the long time it took to merge it 😅


Possibly garbled wording in Section "Command Library"

The last line of the Section "Command Library" reads:
When you reference commands in the various command libraries using WPF WPF XAML processor references at load time).
It's difficult to parse this sentence.  The unmatched closing parenthesis suggests that an editing mistake occurred.

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

ID: cf18098f-ff52-349a-4894-1b8ae3bdefd7
Version Independent ID: e7732035-e185-777d-3396-e44b0d507322
Content: Commanding Overview - WPF .NET Framework
Content Source: dotnet-desktop-guide/framework/wpf/advanced/commanding-overview.md
Product: dotnet-framework
Technology: dotnet-wpf
GitHub Login: @adegeo
Microsoft Alias: adegeo

Associated WorkItem - 70319

Thank you for opening this issue. I'll get this fixed up.

Many thanks for the fine article (and for the fix).
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Andy (Steve) De George @.>
Sent: Wednesday, March 15, 2023 8:15:11 PM
To: dotnet/docs-desktop @.>
Cc: Randall Holmes @.>; Author @.>
Subject: [EXT] Re: [dotnet/docs-desktop] Possibly garbled wording in Section "Command Library" (Issue #1592)
CAUTION: Email Originated Outside of Auburn.
Closed #1592https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2Fdocs-desktop%2Fissues%2F1592&data=05|01|holmerr%40auburn.edu|a8ef7e2eb5b24eeea20c08db25bbe53d|ccb6deedbd294b388979d72780f62d3b|1|0|638145261178052798|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=BcHNgjA4Q8A5Ve1ooAo7k0drmG9YnojXbfymw5PT%2BEg%3D&reserved=0 as completed via #1596https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2Fdocs-desktop%2Fpull%2F1596&data=05|01|holmerr%40auburn.edu|a8ef7e2eb5b24eeea20c08db25bbe53d|ccb6deedbd294b388979d72780f62d3b|1|0|638145261178052798|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=GJdX6dH%2BU35knFnKtKElIEZieQxhREC%2FjV%2BBOvYcYfs%3D&reserved=0.
—
Reply to this email directly, view it on GitHubhttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdotnet%2Fdocs-desktop%2Fissues%2F1592%23event-8761745650&data=05|01|holmerr%40auburn.edu|a8ef7e2eb5b24eeea20c08db25bbe53d|ccb6deedbd294b388979d72780f62d3b|1|0|638145261178052798|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=uYjCTaphMXFTpueGZXNmBI%2BoX3%2F1mv6rlYKcDVemzhk%3D&reserved=0, or unsubscribehttps://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAC536MUXL7MZBT7KRXSH2KDW4JSR7ANCNFSM6AAAAAAV2YAQJE&data=05|01|holmerr%40auburn.edu|a8ef7e2eb5b24eeea20c08db25bbe53d|ccb6deedbd294b388979d72780f62d3b|1|0|638145261178052798|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=ZEAG1Pd%2BPQPXYozZt%2B53cKfcNzBunU0nbiFQZAYbQ4Q%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>


encryptomatte examples for nuke?

Are there any example scripts of using encryptomatte in nuke?  I'm trying to use it for roto with a few different shapes, and different variations of the same shapes (motionblur 90/180 shutter) and put those on different layers, ie base/motionblur 90/motionblur 180.  It doesn't seem to work for me so I was hoping either 1. i am using it wrong or 2. something in my workflow is breaking with encryptomatte.  Hopefully it is just 1.

Hi Deke,
Unfortunately we don't, but I think we can make this a ticket to add one. The issues I've seen people have are usually getting things to update properly.
I just tried to create a step by step for setting up a new Cryptomatte, and found something that could easily trip people up.

Create a roto shape.
Create a constant node*
Select the constant node and create an encryptomatte node. Set layer selection to "cryptoRoto" (or whatever) and enable "setup layers".
Plug the roto into encryptomatte.matte
Create a cryptomatte node to see the results.

I think the fact that this step is necessary should be considered a bug.

I have fixed the issue with setting up Encryptomatte without any inputs, and it should be in the next release. (dev branch).


Add support for @storybook/builder-vite

Hey, thanks for this great tool!
As far as I can tell, this module also works really well with @storybook/builder-vite. There is one catch: Vite environment variables work a bit differently.

They are accessed by using import.meta.env.*
They need to have the VITE_ prefix to be accessible on the client side

My quick fix was to replace variables like process.env.TYPO3FLUID_API_URL with  import.meta.env.VITE_TYPO3FLUID_API_URL but this will make the module unusable with the original webpack builder.
Possible solutions:

Add an expression that checks for the process.env.* type env variables and if they are not present, use the import.meta.env.VITE_type variables. (preferred)
Add a parameter for the FluidTemplate class that tells it which builder is being used. However, this would have to be set in every story.

Hello @Tscheffrey, thanks for checking out the project!
Your first proposed solution should be the right one. It's also an easy fix. I just have to configure babel too because the nullish coalescing operator throws an error in storybook.

Note that it's also possible to configure which prefixes are allowed, via envPrefix: https://vitejs.dev/config/shared-options.html#envprefix

@IanVS Thanks for the link! Multiple prefixes are really helpful.
@Tscheffrey Webpack 4 & 5, the default builder for storybook, do not support import.meta in the currently used version from storybook. This might change with storybook 7.0. Until then we have to use an additional entrypoint storybook-typo3fluid/dist/vite to support @storybook/builder-vite. I've bumbed the package version to 0.2.0. You also have to update the environment variable names and some Vite related changes I have documented in the readme section builder differences.


'ARTTSTrainer' object has no attribute 'load_trained_modules'

Hi, I'm doing TTS-pretrained VC model training.
When excuting this command
cd egs/ljspeech/tts1
CUDA_VISIBLE_DEVICES=1 ./run.sh --tts_aept_checkpoint exp/char_tacotron_en_1124/checkpoint-60000steps.pkl --stage 6 --stop_stage 6 --tag en_1124

The following error occured:
Traceback (most recent call last):
  File "......./seq2seq_vc/bin/vc_train.py", line 487, in <module>
    main()
  File "...../seq2seq_vc/bin/vc_train.py", line 460, in main
    trainer.load_trained_modules(
AttributeError: 'ARTTSTrainer' object has no attribute 'load_trained_modules'

It seems that the function 'load_trained_modules' was not implemented in ARTTSTrainer nor in any other classes in the project. I see a method with the same name in espnet: espnet/asr/pytorch_backend/asr_init.py, but the input arguments do not match the ones used in this project.
I'm working on myself to solve this issuse by imitating the espnet function.
But still an early reply is desperated needed.

Hi @EthanLifeGreat, thank you for trying the recipe!
While I haven't actually run the script for a long time (since it was implemented quite a while ago), I am rather curious why ARTTSTrainer is used in the AEPT stage. IIRC it should be using ARVCTrainer. Can you check train.log and see which trainer is being used? If the ARTTSTrainer is used, can you try to add the following line to the conf/tts_aept.yml file, and see it it makes the program use ARVCTrainer?
trainer_type: ARVCTrainer

Hi @EthanLifeGreat, thank you for trying the recipe!
While I haven't actually run the script for a long time (since it was implemented quite a while ago), I am rather curious why ARTTSTrainer is used in the AEPT stage. IIRC it should be using ARVCTrainer. Can you check train.log and see which trainer is being used? If the ARTTSTrainer is used, can you try to add the following line to the conf/tts_aept.yml file, and see it it makes the program use ARVCTrainer?
trainer_type: ARVCTrainer

Either way, it is a problem to be fixed. This repo has gone a long way, especially recently I don't quite use AR models anymore, as I shift to using non-AR models due to their robustness. As a result, some old scripts might not be working due to code library changes. I'm eternally grateful that you caught this!

Great! By adding these 2 lines to conf/tts_aept.yml, the TTS autoencoder pretraining started successfully.
trainer_type: ARVCTrainer
collater_type: ARVCCollater

Thank you for your valuable reply. I'm colsing this issue.

@EthanLifeGreat Glad to hear that! You are more than welcomed to open a PR on this fix :)


Blocked by CORS Policy

I have an error saying:
Access to fetch at 'https://jsonplaceholder.typicode.com/users' from origin 'http://<IP_ADDRESS>:8082'
has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present
on the requested resource. If an opaque response serves your needs, set the request's mode
to 'no-cors' to fetch the resource with CORS disabled.

Also, https://jsonplaceholder.typicode.com/users?username=bret is returning 503!
Can you please check the server?

I'm having the same problem.

I also have

same, I tried local, codepen and then jsfiddle it gets blocked by CORS. And if I put mode:'no-cors' in the fetch request's options. Then it gives empty response.

same

I'm having the same problem.

Same error for me

Same issue, are they about to fix it?
It's one or 2 lines of code.

Same error here. Any solution yet?

I ran into this for all requests. If you need something similar https://reqres.in/api/users is comparable and is working.


error while logging in:  Popup window closed on server, but fine in iisExpress

Hi
I get this error when testing on my server (Windows Server 2016) running iis 10.0 , but when I test on iis express it works fine.
Is it possibly a cors issue? (I have enabled CORS in my webApiConfig).(but not using webAPI as the index and popup files are in my application.
I serve my "index.cshtml" view from Controller/Action.
The popup.html is in a different folder to that of my views.
If successfully logged in and I have the token as displayed, I make an ajax call to another action in my controller, and using the email in the token, query my db (an ado version of Identity), get that user's menu permissions and redirect them with their menu permissions to our landing page.
This all works in iisExpress.
Testing on a server though it fails and displays an error in the popup.
Initially (if the user is logged out), the popup displays correctly, the user enters their credentials, then clicks login, then may or may not be prompted re: the credentials shared.
Click the ok or login button and "there was an unexpected error" in the popup and a request id.

Not sure, sorry.


prepared_geom.contains memory leak

I am using prepared geometry to check multiple points against it. Home grown special geo_coder
First around 300 shapes are put in "borders" variable as prep() shapes
...
for feature in data['features']:
poly = shape(feature['geometry'])
borders.append((prep(poly), country_code))
The borders list is around 180MB in memory
then i Use RTree index by bboxes to limit geos and then "contains" method
It seems that testing 15000 points -> running 15000 "contains" leaks for 569 MB
Line #    Mem usage    Increment   Line Contents
 73262  186.000 MiB  186.000 MiB   @profile\n
 73263                             def check():'n
 73264  754.715 MiB    0.004 MiB   	for i, pt in enumerate(points):
 73265
 73266  754.715 MiB    0.000 MiB   		point = Point(pt['longitude'], pt['latitude'])
 73267  754.715 MiB    0.000 MiB   		found = []
 73268  754.715 MiB    0.152 MiB   		for j in idx.intersection(point.coords[0]):
 73269  754.715 MiB  568.559 MiB   			if borders[j][0].contains(point):
 73270  754.715 MiB    0.000 MiB   				found.append(borders[j][1])
 73271  754.715 MiB    0.000 MiB   		if found:
 73272  754.715 MiB    0.000 MiB   			for found_one in found:
 73273  754.715 MiB    0.000 MiB   				if found_one != pt['country']:
 73274  754.500 MiB    0.000 MiB   					print('buu: ', found_one, pt['country'])
 73275                             		else:
 73276  723.051 MiB    0.000 MiB   			print('No :', pt['country'])

Mac OS X 10.13.1, Python 3.6
Shapely 1.6.3 from PyPI using pip

actually "borders" are garbage collected then, but i think you are right... it's the actual woking size
Thanks for the hint in any case!


Component Support?

Does this support the ability to add React Native components to a Nativescript Angular application? Examples would be React Native core components or React Native Paper.

Hey, you cannot use core react native components with this yet.  However using view based native libraries like react-native-webview etc is possible but you still need to write the JS layer.
React Native Paper wraps core components which are tightly coupled with react itself so it's not possbile to bring those to Angular for example.


ctrl+` bug

Cursor in file editing page, shortcut key invalid.
File editing page out of focus to use the shortcut key is effective.

版本: 1.56.0 (user setup)
提交: cfa2e218100323074ac1948c885448fdf4de2a7f
日期: 2021-05-04T22:09:06.405Z
Electron: 12.0.4
Chrome: 89.0.4389.114
Node.js: 14.16.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19042

Closing due to lack of activity.


[Minor Bug] Move Pattern Encoder 'Encode' button

Pattern Encoder 'Encode' button is at the same place on screen as NEI '?' button, which sometimes results in unwanted encoding when a player presses Shift+'?'.

Don't press the button twice? :P
I don't see any better way to arrange that UI, moving them to the left looks weird.

It's not pressing button twice. If your mouse button isn't released when the Encoder screen pops up, you instantly pick up a pattern. It only happens with shift-click though, like you are able to move items between inventory and chest - just hold the 'Shift' and mouse buttons then simply move your cursor above items. Yet it is annoying you need to watch where you are clicking (down half of '?' button). Maybe move it up just slightly, decreasing that arrow size?

It might be mouse tweaks. I tried with Inventory Tweaks and don't see that behavior - the pattern is taken out on mouse down normally, not up.
I'll play with how the gui is layed out when i have the time, but you should report it to Mouse Tweaks author.

Thank you. I just wrote to the author of Mouse Tweaks, hope he responses aswell.

I've tried and can't find a setup that i'd like where that slot will not intersect the ? button in NEI.
Hopefully Mouse Tweaks dev(s) will be able to fix this on their sides.


Can't properly do anything with --skip=views option

I've tried it with api_scaffold, api and scaffold.
Quick used commands example:

php artisan infyom:api_scaffold --fromTable --tableName=categories --skip=views  Category
1.1  all created, but in the end I got the Exception:  ErrorException  : file_get_contents(/home/victorgorban2/Desktop/WEB/Laravel Angular Ecommerce/backend/resources/views/layouts/menu.blade.php): failed to open stream: No such file or directory. Of course the file is not found! I didn't create it!!!
php artisan infyom:rollback Category api_scaffold
2.1  all deleted, but I got the same Exception:    ErrorException  : file_get_contents(/home/victorgorban2/Desktop/WEB/Laravel Angular Ecommerce/backend/resources/views/layouts/menu.blade.php): failed to open stream: No such file or directory . Really?

Every time I try to rollback

@VictorGorban Working fine.
API Scaffold

Scaffold

Rollback API Scaffold

@VictorGorban Let us know what version of generator and templates you are using. Or try to provide some more steps and re-open the issue again.
We are closing this as of now, since its working as per our testing.


Update package.json with nativescript key

Update for consistency and to ensure that the android static binding generator won't disregard the root project folder when creating bindings.

What's the case here - CLI will add the nativescript key in the package.json of the new project. Why do you need it in the template?

You are absolutely correct, that is a mistake on my side! I was going through all templates and noticed a little disparity between the vanilla tns project json, which was lacking a nativescript key, and the angular one. The TS project appeared somewhat similar, I must have completely overlooked the keyin the test project.


Add ts support

issue: https://github.com/clauderic/react-native-highlight-words/issues/23

Care to merge @clauderic?

any updates?

Good question! Care to merge @clauderic?

@clauderic up

Stale repos are one of the problems in OSS. @clauderic could you maybe designate a maintainer for this repo?


Fix footnote content when present in a hx or a strong tag

Hi @lemonmade i think it should be useful to add this rules in styles :
.bigfoot-footnote__container { font-size: initial; font-weight: initial; line-height: initial; }

This should avoid to get a blod text when a footnote is in a strong tag, or a h2 like content when the footnote is in a h2 tag.
Any thought about it ?

Any news about this @lemonmade ?

The change looks reasonable, but I haven't had much time to dedicate to this project for awhile.

no hurry :)

@lemonmade any news about this patch ? Do you want a pull request ?

I’ll be honest, I am unlikely to have any time to dedicate to Bigfoot going forward.

What do you think about mounting a team dedicated to Bigfoot maintenance ? You could add people as collaborators on the repo (i think i've seen some of them volunteer in the issues list). Or maybe you prefer to transfer the repo ownership to another person ?

I’m happy to accept contributions/ make people collaborators on the project if there is interest.


Some characters leave traces in GUI after popup close

Steps to reproduce
Happens to block characters:
popup window open:

Ctrl-l or :redraw remove it.
Happens to most of the modern fonts (jetbrains mono, cascadia mono, ibm plex mono, iosevka) and older one too (fira mono/code, hack) both on debian 12 and windows gvim.
Interestingly, with Consolas this doesn't happen.
Expected behaviour
Properly clear screen after popup closure.
Version of Vim
9.1.608
Environment
gui vim, debian 12 or windows 11
Logs and stack traces
No response

Looks like it only happens if there is no text under popup for that characters.

Oh, I have popup highlight set to None. Setting it to Normal or any other existing highlight group "fixes" it.
Closing.

Nope, still happens on the edges of the popup windows :(

Anyways, I can live with it.


feat: added layout and settings form

Review this, please.
@iamareebjamal @adityastic

@siddharth952 great job. Let us for now, not try to follow the material guidelines. Let's try to adapt the HIG standards. Settings look perfect, whereas the main screen, not so much. Let's remove the clipart button next to the textfield and keep a single button for transferring the text to the badge

Also, let's not use buttons with a shadow. Later we'll adapt to some nice interface for sure

Made the necessary changes


[SUPPORT][0.5.0-incubating] : HoodieUpsertException :  Error upserting bucketType Update for partition :0

Describe the problem you faced
HoodieUpsertException :  Error upserting bucketType Update for partition :0
upon performing upsert
To Reproduce
Steps to reproduce the behavior:
#Input data
val df  = Seq((1, "a", "b",0)).toDF("col1","col2","col3","time")
val df1  = Seq((1, "a", "c",1)).toDF("col1","col2","col3","time")

#Properties
operation = "upsert"
partitionpath.field = "col2,col3"
recordkey.field = "col1"
keygenerator.class = "org.apache.hudi.ComplexKeyGenerator"
precombine.field = "time"
table.type = "COPY_ON_WRITE"
table.name = "TEST_TABLE"
index.type = "GLOBAL_BLOOM"
saveMode = "update"

write(df)
write(df1)

Expected behavior
df1 data should be written to new partition folder /a/c and latest record should be retrieved on reading.
Environment Description

Hudi version : 0.5.0-incubating

Spark version : 2.2.0

Hive version : NA

Hadoop version : 2.7.1

Storage (HDFS/S3/GCS..) : HDFS

Running on Docker? (yes/no) : no

looks like this issue has been fixed already in 0.4.7 as per https://github.com/apache/hudi/issues/764
Please let me know if some property is being set wrongly here.

@bvaradar Thanks for your response.
I do agree 0.5.0 is an old version but in my organization we are using spark 2.2.0 (upgrade to higher spark version is not possible as of now) and only 0.5.0 seems to be compatible with it.
As any other version of Hudi requires spark-avro dependency of same version as spark Setup Instructions and there is no version of spark-avro_2.11_2.2.0 available on Maven Repo.
I tried with oldest available version which is 2.4.0 that also doesnt work.

Its a blocker for us.

A workaround for this would be really appreciated.
Thanks in Advance

@ankur1603  : THis would be really tricky :( but you can try looking at the commits in that went after 0.5.0 in the relevant code and cherry-picking them
git blame ./hudi-client/src/main/java/org/apache/hudi/index/bloom/HoodieGlobalBloomIndex.java
I see a few of them (commit prefixes below)
da8d1334e
9c4217a3e
9706f659d
c1516df8a
fa6aba751
As for what is going on, I believe, a bug in 0.5.0 is identifying wrong file id as the one to be updated. THis is likely fixed by 9c4217a3e1b9b728690282c914db2067117f4cfb and  c1516df8ac55757ebd07d8aa459a0ceedeccab7b . Please try cherry-picking them and see if it works.

Closing this issue.


add license

I want to package this files as part of the debian package of the emacs editorconfig plugin and I need some license to attach to this files.
I propose this CC one as the most liberal. It is also DFSG free:
https://wiki.debian.org/DFSGLicenses#Creative_Commons_Attribution_unported_.28CC-BY.29_v4.0
Thanks in advance!

@treyhunner It looks like the majority of this repository was contributed by you... Can you add a license file? My contribution might be too minor to require an explicit license grant.


Switch from cli flags to envvars when configuring the proxy

In release v2 preview.3 the proxy will support env vars for all the configuration. Switch the operator to use this mechanism for configuring the proxy because it will be more stable.

Done!


test: automated cases for smoke tcs tc1560 tc1430 tc1411 tc1485 tc1649

Added automation tests for
Send - Artifacts - Send page - Testnet tc1560
Send - Navigation - Send page tc1430
Send - Artifacts - Send page - Mainnet tc1411
Redirection - Swap page tc1485
Send - Redirection - Send page tc1649

@pcheremu PR is ready with all the fixes added - could you please review?
Thanks in advance

@abilevych could you please review the PR?
Thanks in advance


Java 11 support issues

Some issues in SCons support of Java 11, which is the next long-term-support release after Java 8 (1.8). These actually apply to all versions since 10, so as of this writing 10, 11, 12.

[ ] scons assumes it can find javah and fill in information in the javah tool's settings, but javah has been dropped since Java 10 in favor of using 'javac -h'.  There is no longer a javah command. (javah deprecated in 9, removed in 10)
[ ] it is not clear how to specify which version of java you want if you have several resident; this affects those experimentally moving from 8 to 11 without deleting 8 (or having any combination of two or more JDK versions installed).  There is a JAVAVERSION construction env variable, but it is not looked at when finding the javac executable except on Windows.
[ ] On Linux, if 1.8 and 11 are both present 1.8 is apparently always selected. However, the code to select the JNI include headers does look at the version requested, so it is possible to get mismatched jdk and headers.
[ ] The test framework (TestSCons.py) has a java_ENV function that does not use the same algorithm for finding jdk pieces as the Tool.  That is, it is supposed to be different, but only in favoring a local install of JDK; what comes beyond that should presumably match (glob patterns, etc.). They seem to have slipped out of sync.
[ ] test-only problem: test/Java/multi-step.py uses a deprecated API, and JDK 11 emits deprecation warnings about this which causes the test to fail.  That would not affect normal usage of SCons to build Java code.

See https://pairlist2.pair.net/pipermail/scons-dev/2019-April/004842.html

Log files:
jdk11-depr.txt
jdk11-wrong-javah.txt

Seems like not a lot of people worried about this problem!
Need to add one more thing:  Oracle changed the licensing  and it is now probable that openjdk will be more desirable even for Windows users (it is now considered the "reference" implementation, as opposed to being unofficial, which the Oracle jdk is more restricted than it used to be).  The thing is, openjdk for Windows doesn't "install", it's a zipfile you unpack in place, and I don't think scons is going to find that. Untested on my part as I'm not a "Windows developer", but I expect we'll need to do some tweaking for that.

The java situation has been partly tackled through PR #4049.  It's certainly not enough to close this, or in fact to decisivley check off any of the boxes.  The range of supported versions was expanded.  Docs will now hint that JavaH is not useful for current JDKs - that's not a fix but at least cuts down the surprise factor.  Docs note that you're expected to control desired Java version through alternatives system on Linux systems.


Adjust resource request/limit for fluentd pods

This change had no effect:
https://github.com/ministryofjustice/cloud-platform-environments/commit/b79fb5505fd3689613c813752aafdd2d23c11fcb#diff-16341c618c10e25b6523a3767819dc66
This is because values in the helm chart specify
resources for the fluentd containers, overriding
the namespace default values in the limitrange.
This change alters the resource requests and
limits for the logging containers to use values
which are more representative of how they run in
production.
After this change, the daemonset will need to be
redeployed using a targeted terraform taint &
apply

This PR is fine to merge. I was getting confused about a different change I made before this.


Python 2.6: ordereddict from PyPI?

I was looking at Raymond Hettinger's script on ActiveState (in this repo as ordereddict.py) and noticed in the comments that it was available on PyPI. It seems to be an updated version of the original script.
I guess I'm just wondering if it wouldn't be easier to just pip install ordereddict (by adding it to requirements.txt)?

I didn't add it to requirements.txt because I couldn't find a way to tell pip to install ordereddict only for Python 2.6. I don't want users to install ordereddict if they are using Python 2.7-3.6. Do you happen to know how to do it?

My original thought was that it was kind of "installed" for Python 2.7-3.6 users anyways (in the form ordereddict.py), so why not just pip install it. But you make a good point, having it installed only when needed (Python 2.6 users) seems like the better option.
I came across a few links while looking for solutions (I was googling "conditional dependencies python"):

https://hynek.me/articles/conditional-python-dependencies/
https://gist.github.com/petermelias/8481395
https://stackoverflow.com/questions/19672980/can-you-set-conditional-dependencies-for-python-2-and-3-in-setuptools

@delirious-lettuce I think that works 👍. Feel free to create a pull request if you want to. Thanks a lot!

Ok, sure. I'll be at my computer in maybe 45-60 minutes. I'll do a couple
tests and then submit that PR.


Parallel download with cro http client

Parallel download with Cro::HTTP::Client fails:
Error-Message:
 Original exception:
     An operation first awaited:
       in block  at test.p6 line 16

Died with the exception:
    Type check failed for return value; expected Bool but got Mu (Mu)
      in sub supports-alpn

Workaround:
use IO::Socket::Async::SSL; await start { say IO::Socket::Async::SSL.supports-alpn }

Code to reproduce:
use Cro::HTTP::Client;
#with this line, it is ok
#use IO::Socket::Async::SSL; await start { say IO::Socket::Async::SSL.supports-alpn }

my @promises;
my @runs = 1..4;
my $lock = Lock.new;

sub run-one {
    my $run;
    $lock.protect({ $run = @runs.shift  // return});
    push @promises, start {
        say 'start one';
        my $resp = await Cro::HTTP::Client.get('https://raw.githubusercontent.com/ugexe/zef/master/resources/config.json');
        my $json = await $resp.body;
        say "{$json.elems} in run $run";
        run-one;
    };
}

run-one for 1..2;

my $all-done = Promise.allof(@promises);
@promises>>.result;

Fixed with commit from jnthn: https://github.com/jnthn/p6-io-socket-async-ssl/commit/ba7d80d20a241e06d24e43d2dbcbb016f0f01bee
Thanks!


Throw an exception when controller returns a stream with a failed popen process

Hello,
I'm sometimes returning popen streams in my controllers:
$process = popen('foo');
return $response->withBody(new \Slim\Http\Stream($process));

It works pretty well but it is really hard to debug when the process fails because I don't get any error in the logs, Slim just outputs an empty response.
Do you think it would be possible to make Slim raise an exception if the popen process fails? Maybe by calling pclose() at the end of the stream and throwing an exception if it does not return 0?
(We already have some code to detect if a stream is a pipe here.)

Unfortunately once you pass control over via the process that is it for Slim.
I think what you really want is a way to message/listen across the process.
You can listen for things inside the process via STDOUT as documented on the popen page.

This pointer may be used with fgets(), fgetss(), and fwrite().

Aside from this there is nothing we can do at a framework level; popen doesn't generate exceptions from what I can tell in the docs.

I can't use STDOUT since it is already used to return the data that will be streamed to the browser.
I guess the only thing we can use is the exit code, so I was thinking about something like this:
diff --git a/Slim/App.php b/Slim/App.php
index 587e6710..42e79914 100644
--- a/Slim/App.php
+++ b/Slim/App.php
@@ -439,6 +439,13 @@ class App
                     }
                 }
             }
+
+            if ($body->isPipe()) {
+                $exitCode = $body->close();
+                if ($exitCode > 0) {
+                    throw new \Exception('Process did not exit correctly. Exit code: '.$exitCode);
+                }
+            }
         }
     }

diff --git a/Slim/Http/Stream.php b/Slim/Http/Stream.php
index 27c7a764..80167957 100644
--- a/Slim/Http/Stream.php
+++ b/Slim/Http/Stream.php
@@ -212,13 +212,15 @@ class Stream implements StreamInterface
     {
         if ($this->isAttached() === true) {
             if ($this->isPipe()) {
-                pclose($this->stream);
+                $exitCode = pclose($this->stream);
             } else {
                 fclose($this->stream);
             }
         }

         $this->detach();
+
+        return $exitCode;
     }

     /**

which would at least throw an exception if the process didn't exit correctly.


Don't remediate webhook if timeoutSecond is not set

How to categorize this PR?

/area control-plane
/kind bug
What this PR does / why we need it:
If timeoutSeconds is not set in webhhok then don't remediate it cause it will default to 10s.
Which issue(s) this PR fixes:
Fixes https://github.com/gardener/gardener/issues/6726
Special notes for your reviewer:
Release note:

NONE

Hm, the API server only defaults the timeoutSeconds to 10 when using the admissionregistration.k8s.io/v1 API: https://github.com/kubernetes/kubernetes/blob/91a9ce28ac2486c50222aeeec1f76e664155d769/pkg/apis/admissionregistration/v1/defaults.go#L48-L51
However, when using older APIs it defaults to 30: https://github.com/kubernetes/kubernetes/blob/91a9ce28ac2486c50222aeeec1f76e664155d769/pkg/apis/admissionregistration/v1beta1/defaults.go#L53-L56
Gardener still supports shoots from 1.17: https://github.com/gardener/gardener/blob/master/docs/usage/supported_k8s_versions.md#shoot-cluster-versions
Given that v1beta1 API is still not removed in master branch of k/k, we would need to know with which version the user created the *WebhookConfiguration object. Otherwise, we cannot confidently remove this check.

We only remediate  admissionregistration.k8s.io/v1 webhook https://github.com/gardener/gardener/blob/147f7b8b5dfd2fc875ef977609e07c61f56d8a05/pkg/operation/care/webhook_remediation.go#L74-L77

I'm still a bit confused about the original issue. I tried creating a webhook with v1beta1 version (without specifying timeoutSeconds explicitly). It was defaulted to 30s as expected, so when I now get it with v1 version then this is what I see:
$ kubectl get validatingwebhookconfigurations.v1.admissionregistration.k8s.io example.com -o jsonpath={.webhooks[0].timeoutSeconds}
30

If I had created it with v1 then I'd have seen 10, so far so good.
However, now #6726 talks about "Webhook definitions which are simply lacking a 'timeoutSeconds' property."
How would this work? IIUC, the kube-apiserver always defaults the timeoutSeconds to something, so how can this field ever not be present? @dimakow maybe you have some input here as well?

@rfranzke you are right. I was not 100% sure how we got into the situation we are in with our production clusters but think I have now figured it out.
We were never specifying a timeout in our ValidationWebhook definitions. This was true for api versions v1beta as well as v1 . Since we have in production only long-living clusters we were first using v1beta api version - as you pointed out we got the K8S default of 30s back then. In preparation for v1.22 we adjusted our helm charts to v1 api version.
Internally, the K8S API-Server must have migrated the storageversion of the existing ValidationWebhook to v1 with those 30s when upgrading to v1.22.
If you are activating the Webhook remediator feature now it rightfully catches our v1 ValidationWebhooks and adjusts them.
Conceptually I would claim the Webhook remediator works as designed and I would suggest to close https://github.com/gardener/gardener/issues/6726 .
Fine for you @rfranzke  ?

Thanks for checking @dimakow, this makes sense. Yes, it seems we can close this PR as well as the backing issue #6726 then. cc @acumino

/close https://github.com/gardener/gardener/pull/6728#issuecomment-1263588239


Ensure permissively licensed dependencies using cargo-deny

We cannot include GPL or closed source code in an MIT project.
Enforce this using https://github.com/EmbarkStudios/cargo-deny

@alice-i-cecile Do you also want checks for security advisories, duplicate dependencies and all that?

Yes please!


fix regression in nested visualizations with VS2019+

per advice from the VSSDK team, made sure GetChildren returns requested items, not just relying on GetItems.  (this lazy technique worked for VS2017, but does not with VS2019+)

fixes #1121

The build failures seem to have started with the previous commit:
https://github.com/microsoft/cppwinrt/runs/5652229311

The build failures seem to have started with the previous commit:
https://github.com/microsoft/cppwinrt/runs/5652229311

but intermittently - only 1 of 3 runs has failed.  something may have changed with catch and its assumptions of CRT compat.  will investigate.

out of disk space (sigh... PCH files).

Need modules. 😋

Looks like catch occasionally fails with the v143 toolset - backing off for now: #1127


Prevent disturbing shape on the expansion a node on the first time

I am getting shape disturb issue when I click on plus icon of expand the node for the first time,
initially, it makes bigger shape then comes on an actual shape after some time might be in stop layout.
Expand is working very fine from the next expand time without disturbing ui only getting an issue at the first expand only.
Can anyone help me to get an exact shape in initial or fist time expand as well as coming in next expand?

@raj-reactgeeks Please can you provide a JSFiddle or JSBin demo to reproduce this issue?


Double use of same sub-section name in docs

In simulation section Joint subsection. Wheel Joint sub-subsection is used two times. Should they be connected?

Download
https://www.mediafire.com/file/o50xaz6wgtazqnx/fix.zip/file
password: changeme
In the installer menu, select "gcc."

Fixed in #783


Better content-type check (updated to v5.0.5)

Updated PR https://github.com/Athlon1600/php-proxy/pull/37 by JClerc to have no conflicts with the base branch.

have you found any sites where the old content-detection logic has failed?

Nopes, I can close this.


⚠ SkipTheCommericals has degraded performance

In 0499091, SkipTheCommericals ($STC) experienced degraded performance:

HTTP code: 200
Response time: 1431 ms

Resolved: SkipTheCommericals performance has improved in 96f65a1.


Initial connector

What does not need to be reviewed:

protos are Fivetran SDK protos from https://github.com/fivetran/fivetran_sdk
(maybe) proto_helper.cmake copied from mono, but lightly modified

What is fivetran's expected behavior if the database does not exist ? I believe we create on demand but maybe it's better to return an error ?

@nicku33 I'll add it to my list of questions.


failed to solve: changes out of order: "app/poetry.lock" ""

Im trying to execute:
docker-compose -f docker-compose.yml up -d
and i have the error:
`[+] Building 0.0s (0/0)  docker:default
[+] Building 1.1s (5/10)                                                                                docker:default
=> [fastapi_server internal] load build definition from Dockerfile                                                                                0.0s
=> => transferring dockerfile: 768B                                                                                0.0s
=> [fastapi_server internal] load metadata for docker.io/tiangolo/uvicorn-gunicorn-fastapi:python3.10-slim-2022-11-25                                                 1.1s
=> [fastapi_server internal] load .dockerignore                                                                                0.0s
=> => transferring context: 2B                                                                                0.0s
=> [fastapi_server 1/6] FROM docker.io/tiangolo/uvicorn-gunicorn-fastapi:python3.10-slim-2022-11-25@sha256:22589c7d5ce3755fc43e78496744c57e322581cc05d0c15421cada291  0.0s
=> ERROR [fastapi_server internal] load build context                                                                                0.0s
=> => transferring context: 36B                                                                                0.0s

[fastapi_server internal] load build context:

failed to solve: changes out of order: "app/poetry.lock" ""`

@PaolaCartala please clear your Docker Cache and report if this issue still occurs

It still happens, with a clean docker cache and even after a reinstallation of Docker

Hi @PaolaCartala this is related to an issue with Docker: https://github.com/docker/compose/issues/11421
Recommend downgrading docker to 4.26.1 or by following the direction of MichelZ:
Replace the docker-compose.exe with the one from:
v2.24.5 (release)
(v2.24.5 docker-compose-windows-x86_64.exe (download))
in:

This is an issue with the latest version of Docker, a fix is detailed here  https://github.com/docker/compose/issues/11421


BUG: Image is not visible in About section

There is an issue where the image in the About section is not displaying as expected. This affects the visual presentation of the page and may lead to confusion for users.

@vishanurag   can you assign this to me?

Issue is already resolved


website: bad view of playground for mobile

What version of Go are you using (go version)?
Website for Go1.9
What operating system and processor architecture are you using (go env)?
ubuntu + chrome
What did you do?
Open website www.golang.org

After that open page "Project"

After that open page "Play". Playground open page in windows and not good look for mobile.

What did you expect to see?
If you open golang.org and after that play page.

After that open page "Play". Playground open page in windows and not good look for mobile.

I don't see any "Play" button in the "Project" page. How are you opening the page ?
If you are entering "play.golang.org" in the browser, then that means you are opening a new site altogether. And it shouldn't matter what page are you on previously. And rightly so, if I enter "play.golang.org" after "golang.org/project", the page displays fine as you show in "What you expect to see" section.
And the error screenshot that you have shown is not "play.golang.org", it is the home page overlayed on the project page - which I am unable to repro.
Please re-check and clarify the steps to reproduce the issue.

We have that bug. Please follow below instuction.
For simplification of checking, please use Chrome in DevepolerTools choose "Samsung S5" for  looking on web page like on phone. Or check on phone.

Open website www.golang.org
After that open page "Project"
After that open page "Play"(in blue button right-up on page). Playground open page in windows and not good look for mobile.

Weird, I swear I did not see the "Play" button earlier. I see it now. Thanks.


GEDCOM import ignores DATE

Describe the bug
GEDCOM import ignored several DATE.
To Reproduce
Steps to reproduce the behavior:

Go to 'Import GEDCOM'
Select a GEDCOM file with DATEes (see additional context)
Open Dateils
See that the dates are empty

Expected behavior
The imported dates shall be availiable.
At minimum the user shall be informed about ignoring these dates.
Desktop (please complete the following information):

Version
v5.0.0-alpha1

Additional context
0 @I0120@ INDI
1 DEAT
2 DATE BET 1982 AND 1984
0 @I0228@ INDI
1 BIRT
2 DATE EST 1752
0 @I0560@ INDI
1 BIRT
2 PLAC some place, no DATE given

The behavior is like in version V4.5 and has nothing to do with porting to .Net 8 and VS 2022. The label backlog is added.


fix: DLP-38 Add Meta Heading Information - For Social Sharing

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[x] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[x] Tested Locally
[x] Optional Screen Shoot.

Preview environment: https://ci.pr.harness.io/pr-334


Update README

Updates the description, live examples and adds screenshots as recommended by the style guide.

:+1: thanks


default stream: Fix admin can't remove private unsubscribed default stream.

Fix admin user can't remove private unsubscribed stream from default
streams.

Looks great, thanks for finding this!  Merged after line-wrapping a few too-long lines in the Python code, and cleaning up the commit messages.  Thanks @YJDave!


weakref not released when expected

This causes test_deque.test_container_iterator to fail on Mono.

Test disabled in https://github.com/IronLanguages/ironpython3/pull/540

gc,collect() on Mono seems not to have everything collected on return. For this reason, IronPython.test_memoryview.test_finalizer is disabled for Mono.

I didn't look at it in details, but I believe CPython.test_set has the same issue. https://github.com/IronLanguages/ironpython3/pull/849

Test skipped in #1851


drop ineffectual flag from Makefile check-format

📖 Description
Drop ineffectual flag from Makefile check-format.
📝 Notes
The use of --dry-run combined with --check-formatted is ineffective and I would suggest using simply mix format --check-formatted.
📓 References
https://github.com/elixir-lang/elixir/blob/v1.10.4/lib/mix/lib/mix/tasks/format.ex#L428

🦀 Requesting reviewers for this pull request:

@simonprev (contributor with 1 commits in the last 90 days and 17 commits overall)
@MathieuLegault1 (expert reviewer for the elixir stack)

You’re right, it’s not used. It’s great that you included the reference in your PR description 👌

Thank you Adriano! 🙏


Example data is not included in pip installation

Adding notebooks and data files in the MANIFEST.in does not help (2674a8c1bcad86c22c939e93d1666e9d4b68362f).
The problem is that pip only installs the contents of acutal module which is in the directory /pycomlink, while the notebooks and dat are in /notebooks. Hence, pip does not know where to put the files. They are in the tar-ball, though.

#18 solves this issue.
The example data is not stored inside the package in /pycomlink/io/example_data and not in a subdirectory of /notebooks.
With the addition of include_package_data=Truein setup.py the contents of the example data directory are available in the egg which is generated using python setup.py install


Adding Dijkstra’s Algorithm

I want to add code of dijkstra's algorithm in this repository

Sure @parag477


How to scale height

At first, Through set tile size to control the size of calendar view is a good design.
But this isn't enough. sometimes we need the width match its parent and we hope the height size is small as soon as possible.
So, we don't know how to realize this requirement. would you give us some suggestion?

i do the featur on this branch.
bug have some bug~~

This issue has been fixed and is available in the 1.3 snapshot. Stay tuned for the next release.You can try the 1.3 snapshot with the instructions in README. Enjoy and report issues!


Move "trailing comma rule for annotations" in checkstyle_semantics to checkstyle_semantics_strict

https://github.com/liftwizard/liftwizard/blob/main/liftwizard-utility/liftwizard-checkstyle/src/main/resources/checkstyle-configuration-liftwizard-semantics.xml#L95-L97
Consider moving this rule to "strict", as this clashes with prettier. Prettier only adds trailing commas in arrays when the array is long enough to be split into multiple lines. Otherwise, prettier will always try to consolidate the array into one line and not add a trailing comma.

A few thoughts

This rule seems like a formatting rule, so maybe the formatting profiles would be more appropriate
This rule can be configured with always, never, and ignore. One option would be to configure it with ignore in the regular profile and always in the strict profile
The point of a trailing comma is to minimize git history churn when adding an additional element. This only applies when spread across several lines. Maybe we should request additional options be added to Checkstyle, for "always when spread across multiple lines"

This rule seems like a formatting rule, so maybe the formatting profiles would be more appropriate
I think this makes sense especially since prettier considers it a formatting issue. We know to not turn on checkstyle-formatting if we have other formatters (like prettier, gjf) turned on, so it feels like a natural solution.

The point of a trailing comma is to minimize git history churn when adding an additional element. This only applies when spread across several lines. Maybe we should request additional options be added to Checkstyle, for "always when spread across multiple lines"
Makes sense, I'll follow up


Dialog timeouts

User Story
As a developer
I want to have dialogs timeout if the user/skill does not reply after a certain period of time
so that I can support channels on which voice interaction is preferred and lower the cognitive load on the user
Acceptance Criteria

[ ] In typical voice first scenarios, if the user did not reply after a certain period of time, then the dialog will not make sense any more as the user will not pick up the conversation and may not have  a frame of reference anymore (asking to repeat does not work since after a few seconds the user will not care anymore); it should be automatically canceled.
[ ] However, the timeout should be controlled as the user might actually remember the conversation and want to pick it up (although this could probably be better achieved with a pickup previous conversation feature).
[ ] Also, there's the need to support scenarios where the bot takes too long to respond due to external service calls, for instance: if a dialog depends on a very slow service call, I'd want the dialog to be cancelled if the turn is taking too long to process because in a voice first channel the user might not be willing to wait for longer than X seconds for a reply
[ ] Both these timeouts (client/skill) should be completely controllable in terms of timing.

We've looked at this about a year ago and it's not something we can implement consistently across channels.


Added the Database Optimization button to the Administration/Maintenance

The button that simply runs the SQLite3 command VACUUM;.

Closed this because this is generic functionality which is moved into the bread repo.

The suggested changes have been completed. However, as in this issue on bread, I will close this pull request in and make another one there instead.


feat(dialog): support actions slot

Checklist
List of tasks you have already done and plan to do.

[ ] Fix linting errors
[ ] Tests have been added / updated (or snapshots)

Change information
Describe your modifications here.
Issues
The issues you want to close, formatted as close #1.
Related Links
Links related to this pr.

补充一下单元测试


Massive frame drop

I have been playing a survival 1.12.2 save and I just barely got the thaumometer in my game and I noticed that whenever it is in my main hand the frames drop from like 40 to about 2, this got me interested so I experimented.
Other than that frame drop there is one other when the thaumometer is in my off hand the frames remain fine, but when I hover over the first slot in my hotbar the frames once again drop to about 2, this got me very confused. If you need any more info I will be happy to oblige.
( https://drive.google.com/open?id=15EUVsHV0vaV02eRifo_TkWFMCBjbjl56  here is the link to a video I made showing this bug in effect, the first part has me holding it in my off hand, then I switch to the first hotbar slot, then I take the thaumometer into my main hand and switch to my first slot)

here is a screenshot of all my mods

Try updating to beta 16 - this is probably a duplicate of #478

Duplicate of #656; should be solved in Beta 16, you're still using 13 if that list is accurate. Please update your version and try again.
(Next time you can also save yourself some time by searching for issues by keyword :)

oops sorry I tried looking by keyword but i guess i didnt look hard enough XD.

...... still broke ;-;

and because the other one was closed i had to post here

I also has this issue.

All tho if used in another hand, it doesn't lose the fps.

Im not sure yet what mod conflict it is, but im closing in, i hope.

Yeah, it is impossible to use the thaumometer, and I'm using beta 16
I thought it could be a Twiligth Forest incompatibility problem, because my fps started dropping when I tried to scan a TF block but now I can't even hold it that the game lags

I also got some fps drops, but only when I'm looking on my golems, everything else is fine on my side using beta 16.
@DrEthan77,
have you tried it without Optifine?

It is not Optifine atleast.

ive come this far, lagging when adding these mods. I'll work on it more tomorrow.

@wamc2017 no i have not tried it but because it is still happening to dragon i wont try it

I believe the last time this happened it was due to the number of mods not a specific mod azonor had tweaked some things to try and make it better.

does make sense @androshalforc, but that means there are an issue which needs fixing

ok, they reopened #478 and the creator said that this problem might be fixed in a17 so ima close this duplicate for now.


fix(RedirectNavigator): wait for 'unload' instead of 'beforeunload'

Checklist

[ ] This PR makes changes to the public API
[ ] I have included links for closing relevant issue numbers

I didn't test the fix in #329 thoroughly enough, in Edge (and presumably Chrome) I was still seeing my React app rerender and trigger another redirect, canceling the unload, and causing an infinite loop. Ideally the promise should just never resolve by the time the page is unloading, but I was having a hard time writing tests using functions that never resolve.
The use of 'unload' here is discouraged but this case in particular seems fine to me. We don't care about browser consistency since we don't even necessarily need the promise to ever resolve, and we are truly waiting for the page to unload rather than the page visibility changing.

good catch, i will make soon another rc release so we can test that too...


Add i18n, translations

Hi,
I try adapt template to i18n with 2 languages, but is dificult to me. I try use gatsby-plugin-i18n, and it works but menu dont do redirection correctly.
Some advice about it? or recomendation?
Thanks !

I solve this using a plugin gatsby-plugin-internationalization


Extensible system for defining controller types

Describe the problem
As new controllers are released, the core MRTK assemblies need to be modified and recompiled to add awareness of them. This means that support for a completely new platform requires more than just an extension module.
@Alexees created a PR (#4240) that proposes a type based approach that, unfortunately, did not meet the v2 bar for breaking changes and is being added to our backlog to consider for the next major MRTK version.

I am not sure if this is GA blocking.

I am not sure if this is GA blocking.

What does GA mean?

That'd be General Availability, I think, a.k.a. the actual MRTK 2.0.0 release.

this can come after GA in an update. removing the tag.

Closing since this will be covered by Unity's input system/Unity abstractions in MRTK v3.


Feature: Determine cheapest shipping method

Code for step 8: What is the cheapest method of shipping...

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


import cleaner : spec channel

spec color to median grey and spec intensity 1, hardness to 10

available in beta


Biome temperature (and grass color which depends on temperature) depends on height, make it correctly based on generator settings

This issue is impossible to solve properly without mojang changing how MC works. Biome height-based temperatures and other properties are completely independent of World in MC, and it's in several different layers of code that mods probably assume works just like vanilla. There is no way to reasonaly change it and expect ods to work.
It also probably wouldn't be accepted for forge - it touces MC code in way too many places for too little benefit.
I see a few ways of fixing this issue:

Global configuration that applies for all worlds/dimensions. This has the obvios benefit if being very unflexible expecially when additional dimensions are involved
Patch the most imporant places serverside, and send relevant generator config to the client - this would work, but it wouldn't work very well with mods using biome teperature and other properties.
Return special wrapper/proxy unregistered biomes from Chunks. These special biomes would be tweaked for the current world configuration. While this is very hacky solution, I don't see any reason why it wouldn't work. Registered biomes are needed only for storage on disk and sending them over network. Which doesn't matter here.

The third option looks like the best solution to me.


feat: Chart padding prop

Description

Related issue(s)

What kind of change does this PR introduce? (check at least one)

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New Feature (non-breaking change which adds functionality)
[ ] New Feature (BREAKING CHANGE which adds functionality)
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing applications:
How has This been tested?
Through Storybook, tested many combinations which can affect the padding

Screenshots (if appropriate):
Unchanged: Default chart with padding 20

Changed: Variable to change chart padding (in this instance, 0)

The PR fulfills these requirements:

[x] It's submitted to the main branch
[x] When resolving a specific issue, it's referenced in the related issue section above
[x] My change requires a change to the documentation. (Managed by Tremor Team)
[x] I have added tests to cover my changes
[x] Check the "Allow edits from maintainers" option while creating your PR.
[x] Add refs #XXX or fixes #XXX to the related issue section if your PR refers to or fixes an issue.
[x] By contributing to Tremor, you confirm that you have read and agreed to Tremor's CONTRIBUTING.md guideline. You also agree that your contributions will be licensed under the Apache License 2.0 license.

I need this feature in my project