CMake compiling FreeRDP on Windows

I am following the tutorial on how to compile FreeRDP on windows for VS! I have installed all the extra layouts I need such as: meson
cmake
nasm
perl
openh264
openssl
freerdp......
And I got to the point of building the library through Cmake, and this is what I get when building:
C:\Windows\System32>cmake` -GNinja -BC:\Users\zhilk\Desktop\testRDP\freerdpBuild -HC:\Users\zhilk\Desktop\testRDP\freerdp -DCMAKE_INSTALL_PREFIX=C:\Users\zhilk\Desktop\testRDP\freerdp -DCMAKE_BUILD_TYPE=Release
CMake Deprecation Warning at CMakeLists.txt:21 (cmake_minimum_required):
Compatibility with CMake < 2.8.12 will be removed from a future version of
CMake.
Update the VERSION argument  value or use a ... suffix to tell
CMake that the project does not need compatibility with older versions.
-- The C compiler identification is MSVC 19.28.29913.0
-- The CXX compiler identification is MSVC 19.28.29913.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/Community/VC/Tools/MSVC/14.28.29910/bin/Hostx86/x86/cl.exe - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: C:/Program Files (x86)/Microsoft Visual Studio/2019/Community/VC/Tools/MSVC/14.28.29910/bin/Hostx86/x86/cl.exe - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Could NOT find PkgConfig (missing: PKG_CONFIG_EXECUTABLE)
FREERDP_VERSION=3.0.0-dev
-- Git Revision aeba30a50
-- MSVC: using dynamically-linked runtime (/MD and /MDd).
-- Looking for pthread.h
-- Looking for pthread.h - not found
-- Found Threads: TRUE
-- Looking for strndup
-- Looking for strndup - not found
-- Looking for include file fcntl.h
-- Looking for include file fcntl.h - found
-- Looking for include file unistd.h
-- Looking for include file unistd.h - not found
-- Looking for include file execinfo.h
-- Looking for include file execinfo.h - not found
-- Looking for include file inttypes.h
-- Looking for include file inttypes.h - found
-- Looking for include file sys/modem.h
-- Looking for include file sys/modem.h - not found
-- Looking for include file sys/filio.h
-- Looking for include file sys/filio.h - not found
-- Looking for include file sys/sockio.h
-- Looking for include file sys/sockio.h - not found
-- Looking for include file sys/strtio.h
-- Looking for include file sys/strtio.h - not found
-- Looking for include file sys/select.h
-- Looking for include file sys/select.h - not found
-- Looking for include file syslog.h
-- Looking for include file syslog.h - not found
-- Performing Test HAVE_TM_GMTOFF
-- Performing Test HAVE_TM_GMTOFF - Failed
-- Skipping disabled feature X11 for X11 (X11 client and server)
-- Skipping disabled feature Wayland for Wayland (Wayland client)
-- Skipping disabled feature ZLIB for compression (data compression)
-- Finding required feature OpenSSL for cryptography (encryption, certificate validation, hashing functions)
-- Found OpenSSL: C:/Program Files/OpenSSL-Win64/lib/libssl.lib;C:/Program Files/OpenSSL-Win64/lib/libcrypto.lib (found version "1.1.1j")
-- Skipping optional feature MbedTLS for cryptography (encryption, certificate validation, hashing functions)
--     Enable feature MbedTLS using "-DWITH_MBEDTLS=ON"
-- Skipping disabled feature OpenSLES for multimedia (OpenSLES audio / video)
-- Skipping disabled feature OSS for sound (audio input, audio output and multimedia redirection)
-- Skipping disabled feature ALSA for sound (audio input, audio output and multimedia redirection)
-- Skipping disabled feature Pulse for sound (audio input, audio output and multimedia redirection)
-- Skipping disabled feature Cups for printing (printer device redirection)
-- Skipping disabled feature PCSC for smart card (smart card device redirection)
-- Skipping disabled feature FFmpeg for multimedia (multimedia redirection, audio and video playback)
-- Skipping optional feature JPEG for codec (use JPEG library)
--     Enable feature JPEG using "-DWITH_JPEG=ON"
-- Skipping optional feature x264 for codec (use x264 library)
--     Enable feature x264 using "-DWITH_X264=ON"
-- Skipping optional feature OpenH264 for codec (use OpenH264 library)
--     Enable feature OpenH264 using "-DWITH_OPENH264=ON"
-- Skipping optional feature OpenCL for codec (use OpenCL library)
--     Enable feature OpenCL using "-DWITH_OPENCL=ON"
-- Skipping optional feature GSM for codec (GSM audio codec library)
--     Enable feature GSM using "-DWITH_GSM=ON"
-- Skipping optional feature LAME for codec (lame MP3 audio codec library)
--     Enable feature LAME using "-DWITH_LAME=ON"
-- Skipping optional feature FAAD2 for codec (FAAD2 AAC audio codec library)
--     Enable feature FAAD2 using "-DWITH_FAAD2=ON"
-- Skipping optional feature FAAC for codec (FAAC AAC audio codec library)
--     Enable feature FAAC using "-DWITH_FAAC=ON"
-- Skipping optional feature soxr for codec (SOX audio resample library)
--     Enable feature soxr using "-DWITH_SOXR=ON"
-- Skipping optional feature GSSAPI for auth (add kerberos support)
--     Enable feature GSSAPI using "-DWITH_GSSAPI=ON"
-- Skipping optional feature IPP for performance (Intel Integrated Performance Primitives library)
--     Enable feature IPP using "-DWITH_IPP=ON"
-- Using OpenSSL Version: 1.1.1j
using default plugins location
CMake Deprecation Warning at winpr/CMakeLists.txt:18 (cmake_minimum_required):
Compatibility with CMake < 2.8.12 will be removed from a future version of
CMake.
Update the VERSION argument  value or use a ... suffix to tell
CMake that the project does not need compatibility with older versions.
-- Could NOT find PkgConfig (missing: PKG_CONFIG_EXECUTABLE)
-- Looking for include file stdbool.h
-- Looking for include file stdbool.h - found
-- Looking for include file stdint.h
-- Looking for include file stdint.h - found
-- Looking for include file inttypes.h
-- Looking for include file inttypes.h - found
-- Looking for timer_create
-- Looking for timer_create - not found
-- Looking for timer_delete
-- Looking for timer_delete - not found
-- Looking for timer_settime
-- Looking for timer_settime - not found
-- Looking for timer_gettime
-- Looking for timer_gettime - not found
-- Could NOT find PkgConfig (missing: PKG_CONFIG_EXECUTABLE)
-- Could NOT find Cairo (missing: CAIRO_LIBRARIES CAIRO_INCLUDE_DIR)
CMake Warning at libfreerdp/CMakeLists.txt:102 (message):
-DWITH_SWSCALE=OFF and -DWITH_CAIRO=OFF, compiling without image scaling
support!
CMake Deprecation Warning at rdtk/CMakeLists.txt:18 (cmake_minimum_required):
Compatibility with CMake < 2.8.12 will be removed from a future version of
CMake.
Update the VERSION argument  value or use a ... suffix to tell
CMake that the project does not need compatibility with older versions.
-- Could NOT find PkgConfig (missing: PKG_CONFIG_EXECUTABLE)
-- Adding STATIC channel client server "drdynvc": Dynamic Virtual Channel Extension
-- Adding DYNAMIC channel client "video": Video optimized remoting Virtual Channel Extension
-- Adding DYNAMIC channel client "urbdrc": USB Devices Virtual Channel Extension
-- Adding DEVICE channel client "smartcard": Smart Card Virtual Channel Extension
-- Adding STATIC channel client server "remdesk": Remote Assistance Virtual Channel Extension
-- Adding STATIC channel client server "rdpsnd": Audio Output Virtual Channel Extension
-- Adding DYNAMIC channel client "rdpgfx": Graphics Pipeline Extension
-- Adding DYNAMIC channel client "rdpei": Input Virtual Channel Extension
-- Adding STATIC channel client server "rdpdr": Device Redirection Virtual Channel Extension
-- Adding STATIC channel client "rdp2tcp": Tunneling TCP over RDP
-- Adding STATIC channel client "rail": Remote Programs Virtual Channel Extension
-- Adding DEVICE channel client "printer": Print Virtual Channel Extension
-- Adding DYNAMIC channel client "geometry": Geometry tracking Virtual Channel Extension
-- Adding STATIC channel client server "encomsp": Multiparty Virtual Channel Extension
-- Adding DYNAMIC channel client server "echo": Echo Virtual Channel Extension
-- Adding DEVICE channel client "drive": Drive Redirection Virtual Channel Extension
-- Adding DYNAMIC channel client "disp": Display Update Virtual Channel Extension
-- Adding STATIC channel client server "cliprdr": Clipboard Virtual Channel Extension
-- Adding DYNAMIC channel client server "audin": Audio Input Redirection Virtual Channel Extension
CMake Warning (dev) at C:/Program Files/CMake/share/cmake-3.20/Modules/CPack.cmake:690 (option):
Policy CMP0077 is not set: option() honors normal variables.  Run "cmake
--help-policy CMP0077" for policy details.  Use the cmake_policy command to
set the policy and suppress this warning.
For compatibility with older versions of CMake, option is clearing the
normal variable 'CPACK_BINARY_ZIP'.
Call Stack (most recent call first):
CMakeCPack.cmake:102 (include)
CMakeLists.txt:1106 (include)
This warning is for project developers.  Use -Wno-dev to suppress it.
-- Configuring done
CMake Error: The following variables are used in this project, but they are set to NOTFOUND.
Please set them or make sure they are set and tested correctly in the CMake files:
LIBUSB_1_INCLUDE_DIR
used as include directory in directory C:/Users/zhilk/Desktop/testRDP/freerdp/channels/urbdrc
used as include directory in directory C:/Users/zhilk/Desktop/testRDP/freerdp/channels/urbdrc
used as include directory in directory C:/Users/zhilk/Desktop/testRDP/freerdp/channels/urbdrc
used as include directory in directory C:/Users/zhilk/Desktop/testRDP/freerdp/channels/urbdrc
used as include directory in directory C:/Users/zhilk/Desktop/testRDP/freerdp/channels/urbdrc
used as include directory in directory C:/Users/zhilk/Desktop/testRDP/freerdp/channels/urbdrc
used as include directory in directory C:/Users/zhilk/Desktop/testRDP/freerdp/channels/urbdrc
used as include directory in directory C:/Users/zhilk/Desktop/testRDP/freerdp/channels/urbdrc/client
used as include directory in directory C:/Users/zhilk/Desktop/testRDP/freerdp/channels/urbdrc/client
used as include directory in directory C:/Users/zhilk/Desktop/testRDP/freerdp/channels/urbdrc/client
used as include directory in directory C:/Users/zhilk/Desktop/testRDP/freerdp/channels/urbdrc/client
used as include directory in directory C:/Users/zhilk/Desktop/testRDP/freerdp/channels/urbdrc/client
used as include directory in directory C:/Users/zhilk/Desktop/testRDP/freerdp/channels/urbdrc/client
used as include directory in directory C:/Users/zhilk/Desktop/testRDP/freerdp/channels/urbdrc/client
used as include directory in directory C:/Users/zhilk/Desktop/testRDP/freerdp/channels/urbdrc/client
used as include directory in directory C:/Users/zhilk/Desktop/testRDP/freerdp/channels/urbdrc/client/libusb
used as include directory in directory C:/Users/zhilk/Desktop/testRDP/freerdp/channels/urbdrc/client/libusb
used as include directory in directory C:/Users/zhilk/Desktop/testRDP/freerdp/channels/urbdrc/client/libusb
used as include directory in directory C:/Users/zhilk/Desktop/testRDP/freerdp/channels/urbdrc/client/libusb
used as include directory in directory C:/Users/zhilk/Desktop/testRDP/freerdp/channels/urbdrc/client/libusb
used as include directory in directory C:/Users/zhilk/Desktop/testRDP/freerdp/channels/urbdrc/client/libusb
used as include directory in directory C:/Users/zhilk/Desktop/testRDP/freerdp/channels/urbdrc/client/libusb
used as include directory in directory C:/Users/zhilk/Desktop/testRDP/freerdp/channels/urbdrc/client/libusb
LIBUSB_1_LIBRARY
linked by target "urbdrc-client-libusb" in directory C:/Users/zhilk/Desktop/testRDP/freerdp/channels/urbdrc/client/libusb
CMake Warning (dev) in channels/urbdrc/client/CMakeLists.txt:
Policy CMP0021 is not set: Fatal error on relative paths in
INCLUDE_DIRECTORIES target property.  Run "cmake --help-policy CMP0021" for
policy details.  Use the cmake_policy command to set the policy and
suppress this warning.
Found relative path while evaluating include directories of
"urbdrc-client":
"LIBUSB_1_INCLUDE_DIR-NOTFOUND"

This warning is for project developers.  Use -Wno-dev to suppress it.
CMake Warning (dev) in channels/urbdrc/client/libusb/CMakeLists.txt:
Policy CMP0021 is not set: Fatal error on relative paths in
INCLUDE_DIRECTORIES target property.  Run "cmake --help-policy CMP0021" for
policy details.  Use the cmake_policy command to set the policy and
suppress this warning.
Found relative path while evaluating include directories of
"urbdrc-client-libusb":
"LIBUSB_1_INCLUDE_DIR-NOTFOUND"

This warning is for project developers.  Use -Wno-dev to suppress it.
-- Generating done
CMake Generate step failed.  Build files cannot be regenerated correctly.
I searched the Internet for this error CMake Generate step failed. Build files cannot be regenerated correctly. but did not understand anything at all. I need help, I don't understand cmake at all ...

https://github.com/FreeRDP/FreeRDP/wiki/BugReporting
https://github.com/FreeRDP/FreeRDP/wiki/Compilation

It seems like libusb is required to build even when cmake is passed -DCHANNEL_URBDRC_CLIENT=off; example for VS2k17:
cmake . -G"Visual Studio 15 Win64" -DLIBUSB_1_INCLUDE_DIR="c:\libusb-1.0.24\include\libusb-1.0" -DLIBUSB_1_LIBRARY="c:\libusb-1.0.24\VS2017\MS64\static\libusb-1.0.lib" -DCHANNEL_URBDRC_CLIENT=off

@jspraul no it is not. If you use -DCHANNEL_URBDRC=OFF it works.


build-thirdparty.sh fail on ubuntu

Describe the bug
In ubuntu (16.04 14.04）and many other Linux distribution, "sh" links to /bin/sh not /bin/bash. build-thirdparty.sh use source command which only exists in /bin/bash not /bin/sh.
Run build-thirdparty.sh by sh fails and downlaod-thirdparty.sh:65 use sh to run download-thirdparty.sh.
Because both first line of download-thirdpaty.sh and  build-thirdparty.sh is "#!/bin/bash", the bash scripts can run directly without “sh".
To Reproduce
Steps to reproduce the behavior:

sh ./thirdparty/build-thirdparty.sh
See error:
thirdparty/build-thirdparty.sh: 41: thirdparty/build-thirdparty.sh: source: not found

Expected behavior
build third-party source
Screenshots

Desktop (please complete the following information):

OS: Ubuntu
Version 16.04

Additional context
Add any other context about the problem here.

commit： ae19fbfc8b741a9cd70c780e6f70b9e9a08bb493 has fixed all the issues.


Object of type bytes is not JSON serializable

Issue Type: Bug

Behaviour
ERROR: test_xyz (xyz.xyz_tests.xyz_test.Test_XYZ)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "[REDACTED]_test.py", line 57, in test_[REDACTED]
    sys.stderr.buffer.write(struct.pack("L", 20))
  File "[REDACTED]/.vscode/extensions/ms-python.python-2021.12.1559732655/pythonFiles/visualstudio_py_testlauncher.py", line 88, in write
    _channel.send_event("stdout" if self.is_stdout else "stderr", content=data)
  File "[REDACTED]/.vscode/extensions/ms-python.python-2021.12.1559732655/pythonFiles/visualstudio_py_testlauncher.py", line 132, in send_event
    content = json.dumps(body).encode("utf8")
  File "/usr/lib64/python3.7/json/__init__.py", line 231, in dumps
    return _default_encoder.encode(obj)
  File "/usr/lib64/python3.7/json/encoder.py", line 199, in encode
    chunks = self.iterencode(o, _one_shot=True)
  File "/usr/lib64/python3.7/json/encoder.py", line 257, in iterencode
    return _iterencode(o, 0)
  File "/usr/lib64/python3.7/json/encoder.py", line 179, in default
    raise TypeError(f'Object of type {o.__class__.__name__} '
TypeError: Object of type bytes is not JSON serializable

Expected vs. Actual Behaviour
>>> 8

Steps to reproduce:
[NOTE: Self-contained, minimal reproducing code samples are extremely helpful and will expedite addressing your issue]

Write a test class
Write a test function
Call sys.stdout.buffer.write()

import unittest
import struct
import sys

class Test_XYZ(unittest.TestCase):
    def setUp(self):
        pass
    def test_xyz(self):
        sys.stderr.buffer.write(struct.pack("I", 65))

Run test (from anywhere)

Diagnostic data

Python version (& distribution if applicable, e.g. Anaconda): 3.7.12
Type of virtual environment used (e.g. conda, venv, virtualenv, etc.): VirtualEnvWrapper
Value of the python.languageServer setting: Default

User Settings

defaultLS: {"defaultLSType":"Pylance"}

downloadLanguageServer: true

envFile: "<placeholder>"

venvPath: "<placeholder>"

venvFolders: "<placeholder>"

condaPath: "<placeholder>"

pipenvPath: "<placeholder>"

poetryPath: "<placeholder>"

languageServer: "Pylance"

linting
• enabled: true
• cwd: "<placeholder>"
• Flake8Args: "<placeholder>"
• flake8Enabled: false
• flake8Path: "<placeholder>"
• lintOnSave: true
• banditArgs: "<placeholder>"
• banditEnabled: false
• banditPath: "<placeholder>"
• mypyArgs: "<placeholder>"
• mypyEnabled: false
• mypyPath: "<placeholder>"
• pycodestyleArgs: "<placeholder>"
• pycodestyleEnabled: false
• pycodestylePath: "<placeholder>"
• prospectorArgs: "<placeholder>"
• prospectorEnabled: false
• prospectorPath: "<placeholder>"
• pydocstyleArgs: "<placeholder>"
• pydocstyleEnabled: false
• pydocstylePath: "<placeholder>"
• pylamaArgs: "<placeholder>"
• pylamaEnabled: false
• pylamaPath: "<placeholder>"
• pylintArgs: "<placeholder>"
• pylintPath: "<placeholder>"

sortImports
• args: "<placeholder>"
• path: "<placeholder>"

formatting
• autopep8Args: "<placeholder>"
• autopep8Path: "<placeholder>"
• provider: "autopep8"
• blackArgs: "<placeholder>"
• blackPath: "<placeholder>"
• yapfArgs: "<placeholder>"
• yapfPath: "<placeholder>"

testing
• cwd: "<placeholder>"
• debugPort: 3000
• nosetestArgs: "<placeholder>"
• nosetestsEnabled: undefined
• nosetestPath: "<placeholder>"
• promptToConfigure: true
• pytestArgs: "<placeholder>"
• pytestEnabled: false
• pytestPath: "<placeholder>"
• unittestArgs: "<placeholder>"
• unittestEnabled: true
• autoTestDiscoverOnSaveEnabled: true

terminal
• activateEnvironment: true
• executeInFileDir: "<placeholder>"
• launchArgs: "<placeholder>"

experiments
• enabled: false
• optInto: []
• optOutFrom: []

insidersChannel: "off"

tensorBoard
• logDirectory: "<placeholder>"

Extension version: 2021.12.1559732655
VS Code version: Code 1.63.2 (899d46d82c4c95423fb7e10e68eba52050e30ba3, 2021-12-15T09:39:46.686Z)
OS version: Linux x64 5.15.16-200.fc35.x86_64
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i3-6006U CPU @ 2.00GHz (4 x 2000)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offopengl: enabled_onrasterization: disabled_softwareskia_renderer: enabled_onvideo_decode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
1, 1, 1

Memory (System)
7.67GB (0.20GB free)

Process Argv
--unity-launch --crash-reporter-id a5081874-60a8-4967-8df5-8df8194cc4a1

Screen Reader
no

VM
0%

DESKTOP_SESSION
gnome-xorg

XDG_CURRENT_DESKTOP
GNOME

XDG_SESSION_DESKTOP
gnome-xorg

XDG_SESSION_TYPE
x11

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
pythonvspyl392:30425749
pythontb:30283811
pythonvspyt551:30345470
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30384385
pythondataviewer:30285071
vscod805:30301674
pythonvspyt200:30340761
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
vsc1dst:30424896
pythonvs932:30410667
vscop804:30404766
vs360cf:30404996
vsrem710:30416614
vsbas813:30426126
vsic-wlc-tsjs:30426436

Hi @rubi-rastko, thank you for reaching out. We currently rely on stdout and stderr to pass test results around, so if your tests use them you will need to find a different way to pass your data around or you will encounter this bug.
We are currently working on changing this behaviour with #17242.

Hi @rubi-rastko, thank you for reaching out. We currently rely on stdout and stderr to pass test results around, so if your tests use them you will need to find a different way to pass your data around or you will encounter this bug.
We are currently working on changing this behaviour with #17242.

Thank you for replying.
Given what you've said there, I wouldn't consider this a bug, but a user error on my part.
Using stdout, stdin in tests is probably an anti-pattern.
I retract my issue report. This bug report is a bug.
Feel free to close it as such.

I don't think it's an anti-pattern in general, but rather that when we built the testing part of the extension we didn't consider the fact that stdout and stderr might be used by tests 🙂
I'm going to keep this issue open so we can verify that stdout and stderr are not monopolized by the extension once we're done with #17242.
Thank you!

See all the following issues as related. With the old test architecture, all these problems came from how we processed the info. I will need to look through all of these and see which ones are still outstanding and look for a fix.
https://github.com/microsoft/vscode-python/issues/18547
https://github.com/microsoft/vscode-python/issues/18431
https://github.com/microsoft/vscode-python/issues/18346
https://github.com/microsoft/vscode-python/issues/16705
https://github.com/microsoft/vscode-python/issues/11729

This is fixed on insiders when you have the testing experiment turned on. Let me know if it is not working for you!


Dataloader not working when num_workers > 0

When I train a ssd neural network using gluoncv 1.3.0 or 1.4.0 and mxnet. I use mxnet's dataloader:
from mxnet import gluon

def get_dataloader(net, train_dataset, data_shape, batch_size, num_workers):
    from gluoncv.data.batchify import Tuple, Stack, Pad
    from gluoncv.data.transforms.presets.ssd import SSDDefaultTrainTransform
    width, height = data_shape, data_shape
    # use fake data to generate fixed anchors for target generation
    with autograd.train_mode():
        _, _, anchors = net(mx.nd.zeros((1, 3, height, width)))
    batchify_fn = Tuple(Stack(), Stack(), Stack())  # stack image, cls_targets, box_targets
    train_loader = gluon.data.DataLoader(
        train_dataset.transform(SSDDefaultTrainTransform(width, height, anchors)),
        batch_size, True, batchify_fn=batchify_fn, last_batch='rollover', num_workers=num_workers)
    return train_loader

train_data = get_dataloader(net, train_dataset, 512, BATCH_SIZE, NUM_WORKERS)

To turn my dataset into batches ready for processing in a gluoncv training loop. I can train when NUM_WORKERS = 0, it works fine, not using 100% of the GPU but at least some, as I saturate a single core and only 10% of my GPU.
If I increase the num_workers > 0 the system uses 100% of CPU and refuses to continue, when ctrl-c'ding the program it shows it is stuck (must kill -9 to stop it) in:
> Traceback (most recent call last):
>   File "/usr/lib/python2.7/multiprocessing/process.py", line 258, in _bootstrap
>     self.run()
>   File "/usr/lib/python2.7/multiprocessing/process.py", line 114, in run
>     self._target(*self._args, **self._kwargs)
>   File "/usr/lib/python2.7/multiprocessing/pool.py", line 113, in worker
>     result = (True, func(*args, **kwds))
>   File "/usr/local/lib/python2.7/dist-packages/mxnet-1.5.0-py2.7.egg/mxnet/gluon/data/dataloader.py", line 394, in _worker_fn
>     batch = batchify_fn([_worker_dataset[i] for i in samples])
>   File "/usr/local/lib/python2.7/dist-packages/mxnet-1.5.0-py2.7.egg/mxnet/gluon/data/dataset.py", line 124, in __getitem__
>     return self._fn(*item)
>   File "/usr/local/lib/python2.7/dist-packages/gluoncv/data/transforms/presets/ssd.py", line 176, in __call__
>     gt_bboxes = mx.nd.array(bbox[np.newaxis, :, :4])
>   File "/usr/local/lib/python2.7/dist-packages/mxnet-1.5.0-py2.7.egg/mxnet/ndarray/utils.py", line 146, in array
>     return _array(source_array, ctx=ctx, dtype=dtype)
>   File "/usr/local/lib/python2.7/dist-packages/mxnet-1.5.0-py2.7.egg/mxnet/ndarray/ndarray.py", line 2496, in array
>     arr[:] = source_array
>   File "/usr/local/lib/python2.7/dist-packages/mxnet-1.5.0-py2.7.egg/mxnet/ndarray/ndarray.py", line 451, in __setitem__
>     self._set_nd_basic_indexing(key, value)
>   File "/usr/local/lib/python2.7/dist-packages/mxnet-1.5.0-py2.7.egg/mxnet/ndarray/ndarray.py", line 717, in _set_nd_basic_indexing
>     self._sync_copyfrom(value)
>   File "/usr/local/lib/python2.7/dist-packages/mxnet-1.5.0-py2.7.egg/mxnet/ndarray/ndarray.py", line 883, in _sync_copyfrom
>     ctypes.c_size_t(source_array.size)))
> KeyboardInterrupt
> python: malloc.c:3722: _int_malloc: Assertion `(unsigned long) (size) >= (unsigned long) (nb)' failed.
>
>
> ^C^CKilled

Environment info (Required)
pip list | grep gluoncv
gluoncv                       0.4.0b20190107
pip list | grep mxnet
mxnet                         1.5.0    [Build from master]
 python diagnose.py
----------Python Info----------
('Version      :', '2.7.12')
('Compiler     :', 'GCC 5.4.0 20160609')
('Build        :', ('default', 'Nov 12 2018 14:36:49'))
('Arch         :', ('64bit', 'ELF'))
------------Pip Info-----------
('Version      :', '18.1')
('Directory    :', '/usr/local/lib/python2.7/dist-packages/pip')
----------MXNet Info-----------
('Version      :', '1.5.0')
('Directory    :', '/usr/local/lib/python2.7/dist-packages/mxnet-1.5.0-py2.7.egg/mxnet')
Hashtag not found. Not installed from pre-built package.
----------System Info----------
('Platform     :', 'Linux-4.4.0-141-generic-x86_64-with-Ubuntu-16.04-xenial')
('system       :', 'Linux')
('node         :', 'elara')
('release      :', '4.4.0-141-generic')
('version      :', '#167-Ubuntu SMP Wed Dec 5 10:40:15 UTC 2018')
----------Hardware Info----------
('machine      :', 'x86_64')
('processor    :', 'x86_64')
Architecture:          x86_64
CPU op-mode(s):        32-bit, 64-bit
Byte Order:            Little Endian
CPU(s):                24
On-line CPU(s) list:   0-23
Thread(s) per core:    2
Core(s) per socket:    6
Socket(s):             2
NUMA node(s):          2
Vendor ID:             GenuineIntel
CPU family:            6
Model:                 44
Model name:            Intel(R) Xeon(R) CPU           L5640  @ 2.27GHz
Stepping:              2
CPU MHz:               1596.000
CPU max MHz:           2261.0000
CPU min MHz:           1596.0000
BogoMIPS:              4533.49
Virtualisation:        VT-x
L1d cache:             32K
L1i cache:             32K
L2 cache:              256K
L3 cache:              12288K
NUMA node0 CPU(s):     0-5,12-17
NUMA node1 CPU(s):     6-11,18-23
Flags:                 fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe syscall nx pdpe1gb rdtscp lm constant_tsc arch_perfmon pebs bts rep_good nopl xtopology nonstop_tsc aperfmperf pni pclmulqdq dtes64 monitor ds_cpl vmx smx est tm2 ssse3 cx16 xtpr pdcm pcid dca sse4_1 sse4_2 popcnt aes lahf_lm epb kaiser tpr_shadow vnmi flexpriority ept vpid dtherm arat
----------Network Test----------
Setting timeout: 10
Timing for MXNet: https://github.com/apache/incubator-mxnet, DNS: 0.0195 sec, LOAD: 1.5077 sec.
Timing for PYPI: https://pypi.python.org/pypi/pip, DNS: 0.0316 sec, LOAD: 1.4635 sec.
Timing for FashionMNIST: https://apache-mxnet.s3-accelerate.dualstack.amazonaws.com/gluon/dataset/fashion-mnist/train-labels-idx1-ubyte.gz, DNS: 0.0617 sec, LOAD: 0.3168 sec.
Timing for Conda: https://repo.continuum.io/pkgs/free/, DNS: 0.0208 sec, LOAD: 1.1180 sec.
Timing for Gluon Tutorial(en): http://gluon.mxnet.io, DNS: 0.2185 sec, LOAD: 0.7843 sec.
Timing for Gluon Tutorial(cn): https://zh.gluon.ai, DNS: 0.1908 sec, LOAD: 0.7696 sec.

Package used (Python/R/Scala/Julia):
Python with GluonCV 0.4.0
MXNet commit hash:
17c8b25367859818b7aa5ec5466472f96f8c82e1
Build config:
docs/install/install_mxnet_ubuntu_python.sh
then a python setup.py install.
Error Message:
No error message in logs etc, just when you ctrl-c the program (a kill is required to stop it).
Minimum reproducible example
Just use gluoncv train-ssd.py:
gluon-cv/scripts/detection/ssd/train_ssd.py
try with num-workers 0 it'll work then num-workers > 0 it'll show the same symptoms I have shown here.
Steps to reproduce
working:

gluon-cv/scripts/detection/ssd/train_ssd.py --num-workers 0

not working:

gluon-cv/scripts/detection/ssd/train_ssd.py --num-workers 2

What have you tried to solve it?

Tried pre-built binaries for linux system, I nuked cuda, cudann, python (all libraries too) and reinstalled all form scratch.

I even tried installing from source on MacBook with a 750M GPU and the same issue, on mac os, same versions built locally. Hence I am posting as it does not look like a local environment issue.

@evorta I ran gluon-cv/scripts/detection/ssd/train_ssd.py --num-workers 2 on Mac and also on ubuntu P2.8xlarge.. It seems to be running fine. Ran just 5 epochs though for testing. Can you check it again on MXNet master? Also did you get a chance to try fix suggested by @zhreshold?

Looks like #14541 is a duplicate issue of this. Acc @zhreshold comment there the issue is fixed in 1.4.0 and master


window.getComputedStyle()?

I want to implement a higher-order Yew Component that triggers a given Msg when the user has scrolled to the bottom (or top) of a container, for loading more data of a feed for generic infinite scrolling, similar to iron-scroll-threshold.
I found an example with jQuery using .scrollTop() and .height():
https://jsfiddle.net/brynner/QDttv/
scroll_top() is available already, is it the same as in jQuery?
But .height() seems to require window.getComputedStyle() to get the height without padding:
https://stackoverflow.com/questions/28606058/what-is-the-equivalent-of-height-and-width-of-jquery-in-pure-javascript
https://stackoverflow.com/questions/25197184/get-the-height-of-an-element-minus-padding-margin-border-widths
This doesn't seem to be available in stdweb yet:
https://docs.rs/stdweb/*/stdweb/web/trait.IElement.html?search=computed
But it would be very useful for implementing this kind of behavior..

Rather than getComputedStyle, it's much better to use getBoundingClientRect, clientHeight, offsetHeight, or scrollHeight (as appropriate):
let bounding_height: f64 = node.get_bounding_client_rect().get_height();
let client_height: i32 = js!( return @{node}.clientHeight; ).try_into().unwrap();
let offset_height: i32 = node.offset_height();
let scroll_height: i32 = js!( return @{node}.scrollHeight; ).try_into().unwrap();

It seems that clientHeight and scrollHeight aren't implemented yet (but you can easily use them yourself, as shown above).
Since you're doing scroll-related stuff, you'd probably want scrollHeight and offsetHeight (since those are used by that Polymer component:
https://github.com/PolymerElements/iron-scroll-threshold/blob/d30df1a7b77c0419ab25ba7936a25d1308656518/iron-scroll-threshold.html#L197
https://github.com/PolymerElements/iron-scroll-target-behavior/blob/6f4090f7f64c087c3aa51e7537bd646d946afaee/iron-scroll-target-behavior.html#L235-L241


Respect keep_attrs when using Dataset.set_index (#4955)

[x] Closes #4955
[x] Tests added
[x] Passes pre-commit run --all-files
[ ] User visible changes (including notable bug fixes) are documented in whats-new.rst

The original issue was about DataArray.set_index, but since this simply calls Dataset.set_index, I have only added a test for the latter.  DataArray.set_index probably deserves a test too, please let me know what you think.

#5957 seems to have snuck in - can you merge master?

I just saw that the merge_indexes function gets removed in #5692. So I am not sure it makes sense to fix this here?
@benbovy FYI.

Thanks for pinging me @mathause. And sorry @gcaria I didn't see #4955 and this PR until now.
Indeed, there's no need for a fix considering that in #5692 variable attributes are now propagated properly through set_index and other operations on indexes (see https://github.com/pydata/xarray/pull/5692/commits/0086e326dcb03b76ef3e9db3220dce1e682bc23c and https://github.com/pydata/xarray/pull/5692/commits/a891e22d33ae6553db8b2d58cf0522a92f21dd5d). Also, multi-index level coordinates become real coordinates so we don't need keep_attrs to handle the coordinate attributes, we can just propagate the attributes of each of the coordinates unambiguously.

Shall we merge only the test then?

I think I added tests for attributes in #5692, but I need to check.

Should we close?


Release Google.Cloud.Tasks.V2 version 3.1.0-beta01

Changes in this release:
New features

Enable REST transport in selected APIs. Set GrpcAdapter=RestGrpcAdapter.Default in the client builder to use this transport (commit 5008946)

Created release for Google.Cloud.Tasks.V2-3.1.0-beta01


Fix pack name select prompt on new platform view

Set the 'Please select' prompt as the default option for the pack name
select element, preventing the prompt from being selected as a valid
option.

@lkhusid  Can you please check this?


Game will freeze after 1 minute of playing

Every time I start a world, the screen goes to a light blue color and everything stops rendering and the frame rate goes down to 1 frame every 3 seconds. Did I do something wrong?

You need to allocate more RAM. 4GB does the trick for me. I don't know how to fix it.


[Mention The Feature]:Add Colors to "Follow Us" Social Icons

Is your feature request related to a problem? Please describe.
Description:
I would like to propose adding colors to the "Follow Us" social icons on our website. Currently, the icons are monochrome, gray, , which may not align with the vibrant aesthetic of our overall design.
Current Situation:
The social icons are currently displayed in only black in color.
This could lead to a lack of engagement or visibility for users looking to connect with us on social media platforms.
Proposed Solution:
Change the color of the social icons to [insert desired colors, e.g., platform-specific colors or a specific color palette].
Ensure that the new colors are visually appealing and consistent with the website's branding.
Describe the solution you'd like.

Describe alternatives you've considered.
No response
Additional context.
No response
Show us the magic with screenshots
No response
Checklist

[X] I have checked the Existing Issues
[X] I have read the Contributing Guidelines
[X] I want to work on this issue. (optional)

this issue has already been created and assigned to someone else, so kindly create a new issue.


Error on RN 0.43.0

Loading dependency graph, done.
error: bundling: Error
    at DependencyGraph._getAbsolutePath (/Users/taipa/develop/react-native/RNApp04/node_modules/react-native/packager/src/node-haste/index.js:285:11)
    at load.then (/Users/taipa/develop/react-native/RNApp04/node_modules/react-native/packager/src/node-haste/index.js:228:28)
    at process._tickCallback (internal/process/next_tick.js:109:7)
Bundling `index.ios.js`  0.0% (0/1), failed.
error: bundling: UnableToResolveError: Unable to resolve module `react/lib/ReactComponentWithPureRenderMixin` from `/Users/taipa/develop/react-native/RNApp04/node_modules/react-navigation/src/views/Header.js`: Module does not exist in the module map or in these directories:
  /Users/taipa/develop/react-native/RNApp04/node_modules/react-navigation/node_modules/react/lib
,   /Users/taipa/develop/react-native/RNApp04/node_modules/react/lib

Fix is in master, point to github repo in package.json
On Apr 4, 2017 9:16 AM, "TaiPA"<EMAIL_ADDRESS>wrote:

Loading dependency graph, done.
error: bundling: Error
at DependencyGraph._getAbsolutePath (/Users/taipa/develop/react-native/RNApp04/node_modules/react-native/packager/src/node-haste/index.js:285:11)
at load.then (/Users/taipa/develop/react-native/RNApp04/node_modules/react-native/packager/src/node-haste/index.js:228:28)
at process._tickCallback (internal/process/next_tick.js:109:7)
Bundling index.ios.js  0.0% (0/1), failed.
error: bundling: UnableToResolveError: Unable to resolve module react/lib/ReactComponentWithPureRenderMixin from /Users/taipa/develop/react-native/RNApp04/node_modules/react-navigation/src/views/Header.js: Module does not exist in the module map or in these directories:
/Users/taipa/develop/react-native/RNApp04/node_modules/react-navigation/node_modules/react/lib
,   /Users/taipa/develop/react-native/RNApp04/node_modules/react/lib
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/react-community/react-navigation/issues/901, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACFCVuG6oJ5U8W6tUjHoRtjq-leyJgImks5rsb0QgaJpZM4MyWkD
.

Thank you !!!
It is fixed for me.

Just wondering - since I just upgraded to RN 0.43.0, I find that if I use the StackNavigator to display views with the mode set to 'modal', the modal screen on iOS can no longer be dismissed with a downward swipe.  Is this a known issue, or something else that has changed recently?

Still have a related error this morning using the github master repo in my package.json...
Unable to resolve module 'react/lib/ReactDebugCurrentFrame from [...]

The following fixed it for me:
npm install react-navigation@https://github.com/react-community/react-navigation.git --save

@ericvillemure - Your fix worked for me as well.

fixed
npm install react-navigation@https://github.com/react-community/react-navigation.git --save

Sounds like this is resolved by using master, and should be fixed in the next release

still not working for me
@ericvillemure, what version of react/react-native are you on?
"react": "16.0.0-alpha.6",
"react-native": "0.43.2",

@aarboleda1 I am stuck with this a hour, but this fix work for me: installing from github repo.
npm install react-navigation@https://github.com/react-community/react-navigation.git --save
"react": "16.0.0-alpha.6",
"react-native": "0.43.2",

@ericvillemure  Your fix worked for me. Thanks

I tried this:
npm install react-navigation@https://github.com/react-community/react-navigation.git --save
But got this error:
npm WARN<EMAIL_ADDRESS>requires a peer of react@* but none was installed.
After already installing react-native-cli...
Jonathans-MBP-2:newProjects neo$ npm install -g react-native-cli
/usr/local/bin/react-native -> /usr/local/lib/node_modules/react-native-cli/index.js
/usr/local/lib
└──<EMAIL_ADDRESS>Am I missing something?

@jryanhaber I found that if you manually install the missing peer dependency it works.
npm install --save react@*
Hope this works for you!

i am still having this issue with the following setup:
{   "name": "SplashWalls",   "version": "0.0.1",   "private": true,   "scripts": {     "start": "node node_modules/react-native/local-cli/cli.js start",     "test": "jest"   },   "dependencies": {     "": "",     "mongodb": "^2.2.31",     "react": "^15.6.1",     "react-native": "0.47.1",     "react-navigation": "git+https://github.com/react-community/react-navigation.git"   },   "devDependencies": {     "babel-jest": "20.0.3",     "babel-preset-react-native": "2.1.0",     "jest": "20.0.4",     "react-test-renderer": "16.0.0-alpha.12"   },   "jest": {     "preset": "react-native"   } }
any idea how to solve this issue?:

The following fixed it for me:
npm install react-navigation@https://github.com/react-community/react-navigation.git --save

wowwwwww thank you bro - you save me since last night


Add support for physical server profiles

This PR adds support for collecting (and persisting) server profiles from Intersight as well as the basic operations on server profiles: assignment of a server, deployment, and unassignment of a server from a profile.
See also related PRs in the MiQ core repository (21830) and the API repository (1151).
This and the related mentioned PRs contribute to issue #37, but UI implementation is still needed to resolve it.
Depends on #53

The remaining comment regarding Action, I don't know how to fix as using :action => action does not seem to work.

kicking. (hope it is the apt-get cache issue)

I’ll have a look at this tomorrow and try to update.

Looks like this just requires a VCR re-record

@agrare, @kbrock tests are now green again.


add / to the allowlist filter when normalizing the top-namespace

Description
Adding the forward slash / to the filter for top-namespace when normalizing

Motivation and Context

We would like to create a top level with something like foo/bar so that the module can be imported as import foo.bar.stuff.
How Has This Been Tested?

Tested this locally on my mac. Should work on most *nix systems, but I'm not certain how it will impact Windows.
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[X] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[X] I have read the code of conduct.
[X] I have read the contributing guide.
[X] I have updated the documentation accordingly (if applicable).

It's possible this is a breaking change for some folks that use --with-top-namespace with a slash already. before this change, the slash is just replaced with an empty string so a namespace of foo/bar would have just been normalized to foobar. This change would certainly break some folks if they were using /.

Thank you @ryaminal for this Pull Request!
It seems very reasonable and useful, I'll have a look!

Thank you for highlighting the possible breaking change! I think it could be a good idea to bump the major version of this plugin, to 2.0.0. I'll do this in a separate pull request.

The result of normalize_top_namespace is a string that is expected to be a proper Python namespace, in addition to when building up a OS directory path.
Maybe it should be two different functions:
one for normalizing a Python namespace, and another one for normalizing a directory path?

 The few files I checked when I was testing never actually imported anything in the top_namespace so i never saw the problem. Amateur hour. I'll work on this a bit today and get a better solution.
Sorry to waste your time.

Replace this PR with #34 where the problems surfaced here are handled. Made sure to test it a little more thoroughly.


Span across monitors still has issues (better though!)

ℹ Computer information

PowerToys version: 0.23.0
PowerToy Utility: Fancy Zones
Running PowerToys as Admin: not clear
Windows build number: 1809

📝 Provide detailed reproduction steps (if any)

Have 3 monitor setup, one portrait, two horizontal
Created 3 zones, split portrait in half, and built large spanning zone across two horizontal monitors.
Put VNC application across large spanning zone (holding shift modifier to see zone)

✔ Expected result
Expect the window to fill zone that spans two monitors
❌ Actual result
VNC APPLICATION CASE

Horizontal span is filled, vertical is way messed up - maybe due to the portrait monitor?
Seems the window isn't placed at top of the spanning region?  Can be adjusted up manually though?
Also see artifacts in the window placed in the

EXCEL APPLICATION CASE

Vertical span is messed up.  Acts like it is spanning the full portrait height rather than the height of the two monitors that it should be filling.  In other words, almost seems like it thinks the spanning region resolution is as wide as the typical monitor, but as tall as the monitor in
Effectively can't see the top of the window or the bottom.  Vertical spanning messed up?

📷 Screenshots
Monitor configuration:

Spanning regions - clearly thinks the new region is full height.

Strangely enough, when debugging this, tried to make a new custom region and it built it better the second time around - seems to fix the problem?  Maybe this was just an artifact from upgrading from the older version 0.22.0)?

After making the new custom regions again, seems to work better.
Maybe the bug is just that there is no check on the defined regions vs. the actual monitor configuration?

In retrospect, when I built the original layout, I might have started from one of the "square" templates and edited the edges.  Maybe this is why it had issues?
In any respect, probably a test case for you to handle.  (ie, what do you do when the template doesn't match the monitor area -- how do you handle a template that is rectangular, when the monitor space is odd shaped due to different monitor resolutions or orientations.)
Thanks again!

@MicroNateDram
a few infos:

all monitors need to have the same DPI to avoid problems
create a custom layout since the monitors placement isn't ideal for templates (they can be used but it's trickier)
some apps may have issues across monitors, nothing the FZ can fix (placement done by FZ and manual resizing are two different things and even if an app can be manually resize just fine, it may still happen that be zoned by FZ will produce unpredictable results)

@enricogior - For reference all monitors are the same resolution/dpi, just placement/rotation is different.  (exact same model)
As mentioned, custom layout seems to work fine!  More sharing my struggles incase it helps improve the UI and avoid issues.  Probably a lot of corner cases to handle.  Thanks again - love the free product enhancement at work!

I have this issue.  I did not have it in 0.25.  I do have it in 0.27.
2 monitors.  Both have the same DPI, Both are 4k set to 150% scaling.
If I create a zone that is the left half of monitor 1 and #2 is the right half then apply, the border between #1 and #2 appears on my right-handmonitor.
In my eyeball estimation, I'd say the box is oversized by my scaling factor.  I think someone dropped a DPI scaling term somewhere.
e.g. zone 1 bottom right-hand corner is 2200 x - then when i apply x is 6609
This issue goes away when I turn off span across monitors setting.
I wasnt going to add images as I just wanted to drop this and go, but I feel like I am not describing it well.
Heres it reproduced:
Create 3 zones, each one half a screen

Apply and use, #1 is all of monitor 1 and half of monitor 2, # 2 is right half.  and no number 3.

To me its not a huge issue, because I realized that "All zones to span across monitors...." was something I didn't want anyway.  I misinterpreted that as have zones on both monitors.


Shape of string-type output blocks

What shape should these be? To be consistent with inputs, they "should" be rounded rectangles, as they're currently rendered:

Do we want to force this consistency even though the "rounded rectangle" blocks look a little odd? Have an inconsistency between the block shape and the input shape, like Scratch 1.0 and 2.0? Pick a third shape, like a square with sliced off corners?

Resolved for now with rounded rectangles!


ipam: cell for IPAM and IPAMRestAPIHandler

This PR introduces an initial version of the IPAM Hive Cell

Initializes and provides the struct IPAM (lifecycle aspects are kept in the daemon setup for the time being - due to lifecycle dependencies)
Provides the IPAM Rest API handlers
Embeds the existing cell for IPAM metadata manager

The main goal is to enable modularization of IPAM dependent components and free up the daemon init logic.
Further steps could be

Modularize the inner parts (e.g. allocators) into cells
Extract the "infra" IPAM parts (allocating IPs for Ingress, Health, ...) of the daemon. I thought about this, but thought that this might be better for a separate PR. It's kind of sitting between IPAM (core) and the Daemon

/test

/test

This is a great step in the right direction! Thank you so much for tackling this!
I think the PR as is should be correct. I've left some minor feedback where I think we should document the "new" contract a bit better

@gandro Thanks for the review! I incorporated your feedback. PTAL

Rebased on top of main to resolve conflicts

/test


Use data from API for market landing

Related issue: #124.

Need correct mock data in dump.json. Who can help me on this? @JThanat


📝 Correction |

Description
Cloudflare dns privacy policy link is outdated
Why I am making the suggestion
Outdated info

I did post this in the forum but none of the devs fixed it
My connection with the software
None

[ ] I will keep the issue up-to-date if something I have said changes or I remember a connection with the software.

The forum thread has a bit more details

They will now log users “anonymized” ip address and the log will be kept 25h and much more none privacy friendly stuff that i did not mentioned https://developers.cloudflare.com/<IP_ADDRESS>/privacy/public-dns-resolver/ 11

How should this be updated?
Our link in the DNS providers links to Cloudflare's privacy policy page, which in turn has the link mentioned in the forum thread. It's under item 2 information we collect -> Public DNS Resolver Users

The privacy policy we currently link to seems to cover the <IP_ADDRESS> service.

"We will collect limited DNS query data that is sent to the resolvers. This data does not contain user IP addresses or any other personally identifiable information, and the bulk of the data is only stored for 24 hours."' href="https://developers.cloudflare.com/<IP_ADDRESS>/commitment-to-privacy/privacy-policy/privacy-policy/">Some</a>

Is what the link and tooltip currently say, that is a 404 error and the quote is outdated at least on the hours, as they currently say:

[...] We will collect limited DNS query data that is sent to our <IP_ADDRESS> resolver. Our <IP_ADDRESS> resolver service does not log personal data, and the bulk of the limited non-personally identifiable query data is only stored for 25 hours. [...]

on https://www.cloudflare.com/privacypolicy/

You're right. For my own future reference, this issue is referring to the link/tooltip under 'logging' and not the 'privacy policy' link as was suggested. This does need to be fixed.


Working with java synchronization

Formatting and structure checks

[x] Have you used the correct folder and file structure? - see Contribution Guidelines

[x] Is your article properly formatted in Markdown? - see Github Markdown guide

[x] Have you used our preferred heading structure? - H3 (###) and up

[x] Does your article follow any of the suggested structure formats? - see example formats

[x] Is your article over 750 words? - Tool: Wordcounter.net

[x] Is the article you are submitting an in-depth and unique article? Does it go beyond what is in the official docs and what is covered in other blog sites. See these articles as examples.

[x] Does your article provide enough value and detail about your topic? - Articles should be clear, accurate, and fully explained.

[x] Can your article be understood by beginners? - Assume the audience is smart but has no prior exposure to the common terminology in your article.

[x] Have you included a hero image that is 600x400 pixels, under 300KB in size, and have the copyright to use it? - Tools: Pixlr Image Editor for resizing and Unsplash for Creative Commons images

Grammar & plagiarism checks

[x] Have you spellchecked and grammar checked your article? - Tools: Write&Improve - ProWritingAid - Grammarly
[x] Please place your article through a 3rd party plagiarism checker. We suggest using Quetext, this tool is free and has a daily limit. We typically accept articles with 10% or less.
[x] Have you checked your article for readability? - Tool: Hemmingway
[ ] Have you added sources for quotes and images that aren't yours?

Technical checks

[x] Are your code snippets properly formatted for syntax highlighting - see Syntax guide
[x] Have you checked your code runs correctly and you've highlighted all necessary dependencies for installation?
[x] Are the software programs and packages you're highlighting in your article up to date, using current versions and not deprecated?

Topic suggestion this closes
This closes #5936

Hello, @john-amiscaray I have created another PR and done the requested changes. Please check thankyou.

@john-amiscaray Im seeing this is a duplicate - therefore if any more (major) edits are needed we will be closing this PR.

@john-amiscaray I'm seeing this is a duplicate - therefore if any more (major) edits are needed we will be closing this PR.

@hectorkambow Got it. Though I believe after the fixes I suggested above it should be good. I'm going through it with the author I don't believe it should be that difficult of a fix.

Hello @john-amiscaray @hectorkambow  I'm yet to finish my exams and seems like I'll take a little bit more time than I requested. Therefore I'm considering closing it as per now and reopen it once I'm done.

Hello @john-amiscaray would you please have a look.

@gracy20
FYI
Just going through your PR & topic form.
We can proceed to get it (reviewed &) published, but it will not be eligible for a payout. (due to the fact that it was closed and re-opened).
Then the fact that the article was re-opened after our transition announcement - that is also a main reason.
The topic form was closed on May 17th
& I just noticed that you had closed it May 4th.
Thank you - and we are working on getting all the articles reviewed and published asap 🚀

I'll try to have a look at this today or tomorrow to get this ready for a final review

jumping on this one

@gracy20
👏 Nicely done.
You'll be able to see your live published article on the site (in about 30 mins).
To give your article a boost in page engagement - share it across your social site (LinkedIn/Twitter) and tag the EngEd program so we can also share it and give it some traction as well!
(we will try and do the same once we feature it).
If you have not done so already - consider adding a skill section to your author page. This will go a long way for potential future employers.


datalog: odd characters in data log / page does not update when micro:bit is dis/reconnected

Using a program in MakeCode

Open MY_DATA page in browser and view logs. Initial results are seen
Run program (I did this until log full but I expect this reproduces without this step)
View log, which shows 'LOG FULL'

Disconnect and reconnect micro:bit. Page does not refresh. manual refresh does not update data
View source. '?' characters at point where the page no longer presents log '16.5' seconds

Scroll to end of source. All results are visible in the source

Windows 10
Edge 91
micro:bit RC1, DAPLink 0255

Similar issue reported where dis/reconnect does not update data, but unpairing the micro:bit from the browser and then refreshing the page resolves the issue. @microbit-aimee can you add your reproducer here too and we can see if it's related.

Similar issue reported where dis/reconnect does not update data, but unpairing the micro:bit from the browser and then refreshing the page resolves the issue. @microbit-aimee can you add your reproducer here too and we can see if it's related.

This was the data logged yesterday - first entry was an isolated log, thereafter data did not update until micro:bit was unpaired. Then a refresh of page delivered the new data.
microbit (5).csv

Thanks @microbit-mark  @microbit-aimee
I'm just trying to understand this a bit more. When you say "unpair"  do you mean unplugging the micro:bit from the USB port and then plugging it back in again?
Does closing the browser and reopening the MY_DATA.HTM always resolve the issue?
(p.s. Hi Aimee!)

Ah, right, gotcha.
There's nothing WebUSB related in CODAL (that's all MakeCode side), so yes, probably something to raise over there. Happy to host the issue here until resolved though of course!

@finneyj Raised in https://github.com/microsoft/pxt-microbit/issues/4203 too. I'm struggling to get a consistent reproducer, but there's defintely a bug around the data not updating on the page when the micro:bit is disconnected and reconnected. The second time I tried to reproduce, the buttons and the table did not load at all.

That data does look very weird @microbit-mark
Can you send me a copy of that MY_DATA.HTM file?

MY_DATA.HTM.zip
I just reloaded the page again and got this

Thanks @microbit-mark.
Here's a view in a HEX editor:

You can see from this that there is a well defined pattern here. The areas containing "FF" are unwritten FLASH areas, right in the middle of the HTML header... so something definitely afoot here.
I have seen this once before. It was caused by the KL27 (USB interface chip where the data is stored) rejecting/failing instruction from the micro:bit to write data. I think this happens primarily when the KL27 is busy doing other things (e.g. managing the USB connection).
I put in some robust detection for this and retry operations when it failed, which solved the problem in all my tests... but I've never tested with WebUSB active... it could be that the WebUSB interface is putting a high demand on the KL27, and it doesn't have time to record the data...

I added some additional diagnostics I think I've tracked this back to I2C transaction requests (particularly FLASH write operations) being lost when WebUSB is in use with MakeCode. This appears to be caused by the KL27 interface chip issuing out-of-date responses to I2C requests (presumably when under higher CPU load due to the WebUSB activity).
There's a very clear trend that shows it is possible to read back the result from a previous transaction even though a new one has been issued and received (at some level) by the KL27. I suspect what is happening here is that:
The KL27 is receiving the request and leaving it in a buffer while it does other things (e.g. USB activity)
The IRQ line is shared, so false positive interrupts are common, so the NRF52 may solicit a response before the request is fully processed. When it does so, it receives an out of date response (whatever happens to be in the buffer from the last transaction)
This is particularly problematic if the NRF52 issues consecutive commands with the same command code, as it is then impossible to distinguish a successful completion from an "in progress" condition. Such consecutive transactions are very common in our key data logging use case with the FLASH_WRITE command for example, and similarly with a FLASH_ERASE command when erasing/formatting the file system.
I tested this theory by interspersing every transaction with another (pointless but different) transaction such as GET_FLASH_SECTOR_SIZE, in order to separate out the two conditions. This appeared to resolve the issue at the expense of performance.
I've introduce that workaround into a patch included the v0.2.33 release. This seems to improve stability in my tests.
I suspect we'll see reports of logging performance issues  when connected to MakeCode using WebUSB however, as the logging speed is visibly reduced when WebUSB is in use.
Work is in train to fix this in the next release of DAPLink.

Tested the original reproducer in https://makecode.microbit.org/beta#editor which is tracking CODAL v0.2.33 and this no longer reproduces. 🎉
Attaching MY_DATA file which shows no corrupt data
MY_DATA.HTM.zip
microbit.csv
There appears to be a small glitch in the CSV at line 3491 where the micro:bit has been dis/reconnected and the time stamp is not logged and the accel. values remain on the same line

At this point, I wanted to leave it logging and look at another bug so I switched tabs in Chrome to another MakeCode instance, which had the effect of claiming the WebUSB interface and stopping the data logging. I don't expect many people will come across this, but I'll raise this for discussion with MakeCode.

Been playing a bit more with this and I think it's still buggy if you interact with the device console in MakeCode

https://makecode.microbit.org/_R4whXsaEFVqC
MY_DATA.HTM.zip
It seems if I select the device console window and change between this and blocks it leaves erroneous data in the log.

Thanks @microbit-mark
Erroneous how? The data log seems to look consistent apart from the last (incomplete) entry?

Perhaps it's fine. The html view shows data down to 18.12 seconds and does not update after that. At this point in the source view, we see ???? in the source

and then later on

I think this may be where I am switching to device console view in MakeCode

Hmm, yeah that does look weird. The journal metadata also still looks a bit off. There should just be one number in here, with the rest zeroed out as it becomes obsolete...

I can't seem to reproduce the bug in https://github.com/lancaster-university/codal-microbit-v2/issues/124#issuecomment-891771128 now. Tested a few time today in makecode beta.

As discussed on MakeCode call, created program to log a value every 50ms microbit-log-forever-50ms.hex.zip using makecode.microbit.org/beta
I've been running this for half an hour without issue

This should as discussed in this thread should have been fixed with the busy flag implemented in V2.2 release 0257 and 0258 beta test builds of V2.00.
Any problems similar to this one that are still happening should be captured in a new issue.


style: move items from side menu to header

For the change of the sidebar menu I've created new component for header links.
The component takes array of link information and create external Link in case the link goes outside of the application or internal navigation.
The component take the links' data from HEADER_LINKS array inside the file routes/index .

Deleted the Github component since it's not needed after the chnge.

Below is the new look

@NoamGaash

@NoamGaash
I had some annoying issue with the typescript. I've solved it in a way that changed the code that you've reviewed.
You might want to take a brief look to see that it's good.
I might add a test for the header in a future time, right now I don't have the time for it. Just wanted to fix the problem so this change could be submitted.

If you want me to make a commit with better typing, please tell me
Feel free to merge whenever you're ready :)

@NoamGaash
Hi,
I'm getting error because there is no "APPLITOOLS_API_KEY".
Isn't it a key that should be  located on the build server .env?

Hi @NoamGaash  ,
The change is ready, but there failures in the tests because of "APPLITOOLS_API_KEY" which I don't know and I need help with it.

Hi @YuvalMasada , sorry about it. I didn't think about the fact it won't work from fork branches (BTW, you have the permissions to push branch to the main repo)
I think I solved it, let's try
Apologies for the inconvenience

Hi @YuvalMasada , yes, when using Eyes, the visual baselines aren't stored in the SCM
For now the visual assertions are "turned off", I'll fix it on weekend or on Monday, as I don't have the capacity right now
Thank you for your contribution :pray: :clap:


Deploy an API in privateJet mode when the APIM running in the same kubernetes cluster

Description:
It should be able to deploy an API in privateJet mode in the same kubernetes cluster where the APIM is running. In this scenario should analyse the configurations required to deploy the API in the same cluster and support it.
Steps to reproduce:
Affected Product Version:

Environment details (with versions):

OS:
Client:
Env (Docker/K8s):

Optional Fields
Related Issues:

Suggested Labels:

Suggested Assignees:

Not require to latest product, hence closing


Added two new clustering algorithms: Markov Clustering and Chinese Whispers

Hey guys
I added two other clustering algorithms of the following library: https://github.com/nlpub/watset-java
Chinese Whispers is randomized, but Markov (MCL) might be interesting (a second algorithm that always produces the same result). I thought I create a PR, in case you want to have it in your repo as well...
Best regards,
Stefan

Cool, thx!


Javascript Capstone project first morning session issues

[ ] Your progress is way ahead of the schedule and that's awesome.
[ ] Kindly consider using async and await for each function you created.
[ ] Kindly consider using arrow function for all functions you created.
[ ] Kindly avoid using humberger menu for the header and make all menus visible on the header.

I think it would be more professional to use a :before and :after in the CSS instead of writing 3 span's in the html

https://github.com/everinurmind/JavaScript-group-capstone/blob/dce8d431fb3f5b7f721c3d7dc4dff8116efd1885/src/index.js#L13-L21

Please use toggle instead of remove or use add instead of toggle to make your code look more professional.


What is the current list of FINOS projects that use Legend, and how does one get involved/join?

https://landscape.finos.org/ - a breadth of areas that FINOS are involved in across the open source environment in financial markets specific projects we feel are most relevant in the standardization discussions:
• Financial Objects: CDM expansion (e.g. commodities payout, lifecycle events)
• Regulation Innovation SIG – tbc on specific challenge to be tackled
• Securities Reference Data – Currency model


change repository to stable, remove salt-latest component

yes

In my environment this change resulted in salt-minion of version 2015.8.8 installed:
salt-minion --version
salt-minion 2015.8.8 (Beryllium)

So I would refrain from merging it

salt may be preinstalled or will be installed by the salt bootstrap script (as a part of salt master setup sh) as we want to be able to specify specific version by salt bootstrap script (bootstrap.saltstack.com), that only works with saltstack upstream repos (according a doc) usage of salt-latest component is really an alternative.
make sure you provide enough parameters for saltstack bootstrap script so he really installs correct version.

@epcim according to MCP deployment guide salt MUST be installed with this script, not with bootstrap.sh or anything else. By merging this change you introduce yet another inconsistency between official documentation and actual steps required to achieve desired state which is not documented anywhere. The patch must be rolled back or documentation must be changed accordingly.

sure, the script is the valid still.i am not talking about an bootsteap.sh but the bootstrap.saltstack.com that is called at the -init.sh inside system_config ().note - the scrips are kind of evaluation to proof it could work this way. I am now starting work on a botstrap salt formula. that will do the job hopefully even better.On 5 May 2017 20:07, Roman Verchikov<EMAIL_ADDRESS>wrote:@epcim according to MCP deployment guide salt MUST be installed with this script, not with bootstrap.sh or anything else. By merging this change you introduce yet another inconsistency between official documentation and actual steps required to achieve desired state which is not documented anywhere. The patch must be rolled back or documentation must be changed accordingly.
—You are receiving this because you were mentioned.Reply to this email directly, view it on GitHub, or mute the thread.


RFC: Trailing Commas In List Syntax

https://wiki.php.net/rfc/list-syntax-trailing-commas

Already implemented (wow!)


aiohttp cors breaks with new aiohttp 3.0 release

aiohttp cors setup breaks with
 File "/home/idwaker/Code/webapp/app/api/common/utils/loaders.py", line 43, in init_cors
    cors = aiohttp_cors.setup(app, defaults=CORS_DEFAULTS)
  File "/home/idwaker/.local/share/virtualenvs/webapp-69lhAx_Z/lib/python3.6/site-packages/aiohttp_cors/__init__.py", line 65, in setup
    cors = CorsConfig(app, defaults=defaults)
  File "/home/idwaker/.local/share/virtualenvs/webapp-69lhAx_Z/lib/python3.6/site-packages/aiohttp_cors/cors_config.py", line 247, in __init__
    self._resources_router_adapter)
  File "/home/idwaker/.local/share/virtualenvs/webapp-69lhAx_Z/lib/python3.6/site-packages/aiohttp_cors/cors_config.py", line 117, in __init__
    self._app.on_response_prepare.append(self._on_response_prepare)
  File "aiohttp/_frozenlist.pyx", line 97, in aiohttp._frozenlist.FrozenList.append
  File "aiohttp/_frozenlist.pyx", line 19, in aiohttp._frozenlist.FrozenList._check_frozen
RuntimeError: Cannot modify frozen list.

on latest 3.0 release of aiohttp
my init_cors is something like this
async def init_cors(app: web.Application):
    logger = get_logger(app['name'])
    cors = aiohttp_cors.setup(app, defaults=CORS_DEFAULTS)

    # add resources to cors
    for resource in list(app.router.resources()):
        if isinstance(resource, web.Resource):
            cors.add(resource)
    logger.info("Initialized CORS for {}".format(app['name']))

and i am loading init_cors on app.on_startup
The issue seems to occur when i defer cors setup to on_startup signal like
app.on_startup.append(init_cors)

But it works normally if i put the cors init code on app setup like
 # register routes here
    register_routes(app)

    cors = aiohttp_cors.setup(app, defaults=CORS_DEFAULTS)

    # add resources to cors
    for resource in list(app.router.resources()):
        if isinstance(resource, web.Resource):
            cors.add(resource)

I don't think the appending cors setup to startup signal was good idea anyway, so i removed that with simple function call for setup which works as desired.


TECH-1545 : use webpack shared configuration

https://jira.jahia.org/browse/TECH-1545

Won't be done yet in current jahia parent version ( < 8.2.x )


App error link https://www.coronawarn.app/de/faq/#10 does not redirect to explicit FAQ entry

Where to find the issue
https://www.coronawarn.app/de/faq/
Describe the issue
Android CWA 1.5.0 error message link
https://www.coronawarn.app/de/faq/#10
shown in https://github.com/corona-warn-app/cwa-app-android/issues/1459#issuecomment-715794387 does not redirect to
https://www.coronawarn.app/de/faq/#API10, instead it lands on the top level of
https://www.coronawarn.app/de/faq/.
Suggested change
I'm guessing, but probably:
Add redirect
"10": "API10",
to https://github.com/corona-warn-app/cwa-website/blob/master/src/data/faq_redirects.json
Notes
Currently the text on https://www.coronawarn.app/de/faq/#API10 only describes a resolved condition. It does not help with the current issue from https://github.com/corona-warn-app/cwa-app-android/issues/1459.
When the app issue leading to API error 10 in CWA Android 1.5.0 is better understood, it may be able to update the FAQ entry accordingly.

Thanks Mike,
I've added https://github.com/corona-warn-app/cwa-website/pull/513 to address this.
CH

Corona-Warn-App Open Source Team


Possible bug: fsync() required after file creation.

Similar to issue 189, this bug is about what happens during a power-failure.
Also similar to that issue, this bug is not actually triggered atop
file-systems that I'm aware of; it seems wrong from the behavior explained in
the Linux fsync() man-page, though.

The bug: Leveldb, after creating any file and issuing a fsync()-like call on
it, assumes that the file is persistent on disk. If these files actually don't
get persisted to disk, there are lots of chances of (LevelDB) corruption and
wrong-behavior. Of course, with bad system configurations consisting of lying
file-systems and lying hard drives, this can't be helped.

However, even with "correct" system configurations, the Linux man-page for
fsync() reports that the file need not be persistent, unless fsync() was called
on the parent directory.

Saying that, ext2 is the only file-system I know of that might actually expect
an fsync() on the parent directory. Moreover, fsync()-ing the parent directory
is not even allowed on many OSes, even Linux-family ones. On the other hand,
SQLite does fsync()s to the parent directory for this purpose, so it is
possible that there are some cared-about file-systems that actually rely on
this.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 17 Jul 2013 at 3:49

The next release includes a change that addresses this. It calls fsync on the
parent directory just before syncing the manifest near the end of a compaction.
It's not perfect but does reduce the window in which a crash could cause
corruption.

Original comment by<EMAIL_ADDRESS>on 17 Jul 2013 at 10:26

Fixed in 1.13

Original comment by<EMAIL_ADDRESS>on 10 Dec 2013 at 7:51

Changed state: Fixed


Fix logos on community page

:clipboard: Description
Heroku, NodePing and Sentry logos on https://shields.io/community are dead

Hey @chris48s can I work on this?

Sure.

I have corrected the path of the sponsor images. But it doesn't seems to work.

I fixed the error by replacing the image tag to a component tag inside the file. @chris48s is this okay?

[x] Added the image path
[x] Replaced Image tag with Component tag
[x] Made adjustments regarding the height of the SVG

If you think you have a working solution can you submit a pull request so a maintainer can review it.
I don't think it should be necessary to turn each <img> tag into a component to fix this but this conversation will be easier if we can comment on a diff rather than trying to review a screenshot.


Rework CSS to improve dark mode experience

Didn't open an issue for this, apologies. I just played around with the css and reached a point where I was happy enough to push it upstream.
The current dark mode theme always felt like a high contrast theme to me. Moreover, theme transition was afflicted by a flickage on the donut div sections. These css changes attempt to address these problems and provide a (better?) dark mode color combination based on guidelines found e.g. in this awesome article. Let me know what you think overall. Personally I think the new theme is pretty cool, ofc! 😄
@wswebcreation following linter suggestions I've also removed some directives. It doesn't seem I've broken anything...but yeah please double-check that as well since the change is not limited to the dark mode section only.

BTW in several other places the icon is shown inverted: sun on dark mode and moon on light mode. See e.g. here. I think that would look better than the current approach.


fix: add secure mysql db samples, fixes RHOAIENG-5436

Description

Added secure mysql db samples without and with istio tls
Fixes RHOAIENG-5436
Fixes RHOAIENG-6713
How Has This Been Tested?

Tested samples manually, by first creating the DB cert using make certificates and then the appropriate kubectl apply -k command.
Merge criteria:

[X] The commits and have meaningful messages; the author will squash them after approval or will ask to merge with squash.
[X] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[X] The developer has manually tested the changes and verified that the changes work

@nehachopra27 @tonyxrmdavidson we should use these secure mysql samples in QE tests as that's closer to what users will deploy in production.

Is there a procedure to update or rotate the certificates when they expire?
Is there a procedure for a user to bring in their custom certificates instead of self-generated?

Both could be just documentation details but be good to note for the users.

Is there a procedure to update or rotate the certificates when they expire?
Is there a procedure for a user to bring in their custom certificates instead of self-generated?

Good dieas, I'll add them to the readme. In general, these are just samples and users are not expected to use these exact DB configurations in production. The secure DB samples are there to test and demo model registry support for secure DB connections.


Garbage characters in a puzzle name with HTML comment

For the following comment:
<!-- @todo #493 AddCategoryDbDto: port to a new datetime API -->
@0pdd created an issue with a title AddCategoryDbDto: port to a new datetime API -->
Expected result: puzzle shouldn't have --> at the end.
This issue is similar to #104 but with another comment type.

@yegor256/z please, pay attention to this issue

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

@php-coder I don't think it's fixable... You formatted your puzzle incorrectly in the first place. It should be like this:
<!--
  @todo #493 AddCategoryDbDto: port to a new datetime API
-->

I don't think it's fixable...

@yegor256 A similar comment was fixed in #104, so it's definitely fixable. So, now we have inconsistency when one type of comments works but another doesn't.

Job gh:yegor256/pdd#111 is not assigned, can't get performer

The job is not in WBS, won't close the order

You formatted your puzzle incorrectly in the first place. It should be like this:

BTW it isn't documented anywhere and I haven't found examples of HTML comments in README. If it works not like a user expects then it should be documented at least.


Use devDependencies for node-sass in next-sass readme.md

Doesn't seem to be a reason to save node-sass as a regular dependency

I'm going to close this in favor of https://github.com/zeit/next.js/issues/8626
Thanks for your contribution 🙏


home page why-us-text

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #why-us-text
Test URLs:

Before: https://main--tlcgrp--hlxsites.hlx.page/
After: https://why-us-text--tlcgrp--hlxsites.hlx.page/

Please review

As a general comment, I would recommend considering a dedicated block (e.g. why-us) instead of using section metadata if the styles just target the default content in a given section. Section metadata are primarily meant to tweak properties of the section itself: for example background color or image, font color (e.g. if you want to invert background and text for all blocks and default content within a section).


Item count not properly displayed when switching between display options

Explain the bug
Item count not properly displayed when switching between display options.
To reproduce

Go to a directory.
Switch layout options.
See that item count is the same as the previous directory that had an adaptive layout set.
Switch layout back by pressing the same layout option.
See that item count is correct again.

Expected behavior
Item counts are properly displayed
Logfile
Nothing of use in logs
Screenshots
https://user-images.githubusercontent.com/8879244/112382875-c4e7c600-8cc2-11eb-9594-cbc1b5159731.mp4
System Information

OS Version:
Edition:	Windows 10 Pro Insider Preview
Version:	Dev
Installed on:	2021-03-24
OS build:	21343.1000
Experience:	Windows 10 Feature Experience Pack 321.7401.0.3

App version: v<IP_ADDRESS>

Additional comment
On startup, occasionally when switching display options, there won't be an item count at all.
To reproduce

Open Files for the first time.
Switch layout options.
See item count is not displayed.
Switch layout back by pressing the same layout option.
See that item count is correct again.

Screenshots
https://user-images.githubusercontent.com/8879244/112383456-856da980-8cc3-11eb-94ca-4179a8bb4010.mp4

Still an issue in <IP_ADDRESS>

Still an issue in <IP_ADDRESS>

@d2dyno1 do you want to work on this?

Is it present on latest version? @yaichenbaum

Yes


Point Package.swift to official deps

What's in this pull request?
Package.swift now points to typelift Operadics repo

Oops, missed this. Good catch.


Security token expired

Hey,
We have an issue when requesting new tokens. Every so often the IdentityServer hands out new tokens that are already expired. We have checked the clocks on the microservices, they are perfectly in sync.
We are not the only team facing this issue, since the problem is identical to these two tickets. Is there any solution/root cause known so we can fix it?
https://github.com/IdentityServer/IdentityServer4/issues/4770
https://github.com/IdentityServer/IdentityServer4/issues/1689

Is there any solution/root cause known so we can fix it?

I personally never had this problem. Please do your own investigation.

Thank you for your response. We have done extended research but do not see anything wrong with our code.
We have an AKS cluster with the IdentityServer running in a pod as well. From time to time our microservices request a new token from the IdentityServer. The token is exchanged without a problem, but the received token is expired. The expiration date is in the past.
We have asked Microsoft for support too. They checked the whole setup and verified that all the clocks are running in sync. Since we have found some tickets here on GitHub with the same issue, we thought it could be some bug in IdentityServer.


JSON-AD MIME type registration at IANA

Since the JSON-AD serialization format should have a mimetype for HTTP requests, it should officially be registered at IANA. That's why I've started an application procedure there to register it.
However, it seems like an RFC document is required (if you're not a standards organization, and if you're not willing to put vnd in the string). I'm not very excited on writing a lengthy, detailed, technical document that few will read, so this is unfortunately not something that I'm likely to do in the near future.
Some very notable standards such as TOML, phpm YAML, rust, typescript are completely absent in the official IANA list, sometimes leading to serious confusion.
I'll keep this issue posted on the process. There is a chance that application/ad+json will have to change to application/vnd.ad+json... Which is horrible.

The docs for JSON-AD mention the mime type and 'registration pending', but in the referenced issue above you say:

Ideally, we'd also register a MIME type (and the file extension) at IANA for hextuples. I've tried that [in this issue] but it was quite a bothersome process, so I never completed it.

Does that mean that it won't be registered?

Hi @aschrijver, no I'm not planning to! If someone else would like to do this I'd gladly help.


Enhancement request: Icon order in source control view

The source control view is awesome, thank you.
I have a UI/UX suggestion, feel free to ignore if you don't think it's a good idea.
I suggest that the order of the icons at the top be roughly in workflow order (in other words, in the order you'd typically do them as you work with Git):

Pull - Stage all - Unstage all - Commit - Push - Change Layout
(Change Layout at the end because it's not part of your Git workflow.  It could perhaps be moved to the right a little, and/or separated from the others by a small vertical bar.)
Background: I have found in teaching people how to use Git with Atom's sidebar, their top-to-bottom ordering of workflow helps people remember the order of Git operations more easily than if they were ordered another way.
Edit: see comment thread about workflow order vs. functional grouping, and a suggestion about potentially making icon order configurable.

My 2¢, I prefer having push/pull next to each other, signifying the similarities of the actions (pushing and pulling from an origin). Same with stage/unstage.
I do see the benefit of new users to git not having to remember the order, which many users of this plugin will be unfamiliar.

For reference, I asked on the Discord for some more opinions.
I agree with @Zachatoo that I like the grouping of similar actions more than the order of execution.
I will leave this issue open though.

@Vinzent03 @Zachatoo, thanks for your comments.
I still prefer workflow order, but I also see the benefit of functional grouping.  I'm fine with either choice. Thanks for leaving the issue open, Vinzent.
A way to have both would be to make the order configurable.  This would increase documentation and setup complexity somewhat, with a potential benefit of somewhat increased usability and trainability for some people.  I'm not sure if it's a good idea or not, but I'd find it useful. :-)

I decided to close this issue now, as I plan to continue with the current button order and adding an extra button is not worth it.


CUDA GPU acceleration support

Your Feature Request
The newer versions of Tesseract OCR use LSTM so implementing CUDA GPU acceleration support would be a good way to improve performance while leveraging a widely used architecture already tried for for NN tasks.
Ideally this should be available in both the command line and the API, preferably in all distributions, especially the versions available from package managers (apt).

That's correct. Do you want to implement it? Or do you know someone who can write the necessary code?

Ideally GPU acceleration should not be limited to NVIDIA GPUs. "CUDA" is specific to NVIDIA.
Note also that GPU acceleration is mainly required for the training process where it would really help to reduce the time which is required for a training.
GPU acceleration won't help much (if at all) with Tesseract text recognition. Mass OCR production with Tesseract is already very efficient on servers with many CPU threads.

We have a tiny team of volunteers working to slowly improve the existing CPU-only code.
We don't plan to implement this feature request.

GPU acceleration won't help much (if at all) with Tesseract text recognition.

@stweil Could you please clarify the above statement. Nvidia claims upto 140x faster throughput for LSTM models using cuda.
https://developer.nvidia.com/discover/lstm . Why will it not help for tesseract text recognition, since latest tesseract is based on LSTM?
Even if it provides 2x to 4x speed improvement, I will happily take it.

Tesseract does not use CUDA or other code which makes use of the NVIDIA GPU. As long as nobody adds that support, Tesseract always runs without GPU.

@stweil
I think there is a misunderstanding on my question. I am aware of the current status of tesseract.
My question regarding your general statement of "GPU acceleration won't help much (if at all) with Tesseract text recognition". Why? Because nvidia claims otherwise for LSTM inference.

I noticed, too, that my answer was incomplete, and have added more text above. We have experience with kraken OCR which supports CUDA where we use the GPU only for training because the throughput for segmentation (also with AI model) and recognition is much better on the CPU. See my related documentation.


Implement config program

https://docs.solana.com/apps/builtins#config-program

Going to close this for now unless someone requests this.


land on imported study page after import

I'm in study X's study view. I import a study Y. I get the confirmation messages that study Y was imported, but I'm still in study X's view. It would be more convenient and less confusing to land on (the newly imported) study Y.

The developper has refused.

Sí!
On Sat, Oct 10, 2015 at 3:35 PM, elisafilevich<EMAIL_ADDRESS>wrote:

The developper has refused.
—
Reply to this email directly or view it on GitHub
https://github.com/JATOS/JATOS/issues/115#issuecomment-147089913.

Reopening as 'Refresh'

Will be part of the next release.


[gitlab] Use base_branch from release.json for SMP

What does this PR do?
Use base_branch from release.json for SMP testing.
Motivation
Provides better results on stable branches by using the base from the stable branch instead of main.
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

Looks good to me! Thank you for the PR @bkabrda!
One quick question: from looking at https://github.com/DataDog/datadog-agent/blob/0f50a93c6447ad230c612206f1e7219ed1121325/.gitlab-ci.yml#L201-L202, it seems like branches 7.44.x, 7.45.x, and 7.46.x are release branches, but of these, only the 7.44.x branch has a base_branch key in release.json with a value that is not "main". Is the value of that key supposed to be "7.45.x" and "7.46.x" for the 7.45.x and 7.46.x branches, respectively?

@goxberry unfortunately the process of updating this is not great and sometimes it falls through. We will try to make sure it gets updated every time we create a new stable branch.


error: sass syntax @function highlight

@function and all @* syntax highlight detected as a string.unquoted.css

@CreativSpeed that's probably due to the @return statement not ending correctly. Fixed in 201e921ad2fb23ee06b490c58ded44d78558cc07

no, you cant do that in .sass


Fix/bridge

Coverage increased (+0.1%) to 86.482% when pulling 8a938b678f90c19b87b1ef8667248aad346d125e on fix/bridge into 5f69a7a3b97dab7b7e470892868a5cc672d4f1b1 on develop.

Coverage increased (+0.1%) to 86.482% when pulling 8a938b678f90c19b87b1ef8667248aad346d125e on fix/bridge into 5f69a7a3b97dab7b7e470892868a5cc672d4f1b1 on develop.


[Feature]: ItemLevel Search

Describe the feature
Could we have an item level search for items please? Adibags used to filter out those items which made them easier to find and sell.
Describe the implementation
Add search via iLevel value in the search bar.

This used to be great in AdiBags, hopefully comes back

this is now implemented


[bug] - Hide logs on production

Link to minimal reproducible example
labs
Summary
From this discussion: https://github.com/orgs/WalletConnect/discussions/4357
Logs shouldn't be logged on production env.

List of related npm package versions
latest

just for updates.
environment 4.1.7 still received logs in production, exposed projectId in the logs.


API timeout

Problem to solve
Some jobs are never executed, the azure api timeout.
Proposal
-> Verify the reason of the problem
-> handle a retry if the problem is the api call

Solved by https://github.com/sagacorp/yii2-queue-azure-bus-service/pull/4


height: auto on container

Hi There,
Is there anyway to overwrite the height: auto on the main container. I'm using a fullscreen plugin that requires every parent to have a height of 100%;
I've tried adding a class that sets the height but it's inconsistent and flickery as hell on all browsers.
      onStart: {
          duration: 400,
          render: function (url, $container) {
            $('#main').addClass('move-out set-height');

            content.toggleAnimationClass('is-exiting');
            // Scroll user to the top
            $body.animate({
              scrollTop: 0
            });
          }
        },

  onEnd : {
          duration: 0,
          render: function (url, $container, $content) {
           $('#main').removeClass('move-out');
            $body.css('cursor', 'auto');
            $body.find('a').css('cursor', 'auto');
            $container.html($content);

Is there anyway to overwrite this? thanks ^_^

@exgeeinteractive there's the id main so you just can temporarily fix it by the !important rule of CSS.
#main {
height: 100% !important;
}
I dont think it could be inconsistent?

height: 100% or 100vh would works.
100vh might required fallback for un-supported browser


[Event Request] Codeunit 99000770 Where-Used Management - OnBeforeBuildWhereUsedList

Hello Team,
In Codeunit 99000770 we need possibility to have our additional checks and/or avoid using standard code. Having event in procedure WhereUsedFromItem would be a perfect solution:

[IntegrationEvent(false, false)]
local procedure OnBeforeBuildWhereUsedList(var MultiLevel: Boolean)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 48086.


Axis values get too dense for more values

Hi..While plotting graphs, for a larger set of values on X-axis the values get too dense and overlap each other.

What would be the ideal behavior here?

Ideally being able to manually specify an angle for the labels so that they don't overlap anymore.

Okay, I've changed the title of #19 to specify this. Closing this one.


AZ-200/AZ-201 are still visible

Both exams are retire more than a year ago. There is not sense to show them in a list of exams
Link to article:
https://docs.microsoft.com/en-us/learn/certifications/browse/?levels=beginner&resource_type=examination&products=azure

Do Microsoft monitor their own get repo?


What would cause field:.metadata.controller does not exist?

I'm testing an operator that uses JobSet, and every once in a while I see my CRDs stop creating, and the error is in the JobSet log, specifically:
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2
        /go/pkg/mod/sigs.k8s.io/controller-runtime@v0.15.0/pkg/internal/controller/controller.go:226
2023-11-04T03:12:30Z    ERROR   Reconciler error        {"controller": "jobset", "controllerGroup": "jobset.x-k8s.io", "controllerKind": "JobSet", "JobSet": {"name":"metricset-sample","namespace":"default"}, "namespace": "default", "name": "metricset-sample", "reconcileID": "cfc44b3d-f541-422f-a589-97d6fedce45c", "error": "Index with name field:.metadata.controller does not exist"}

This isn't urgent (non blocking, because I can delete the entire cluster and start fresh) but I'd like to understand what causes the index with name field: metadata.controller to not exist, and how to either avoid or fix it. Thank you!

https://github.com/kubernetes-sigs/jobset/tree/main/docs/faq#2-jobset-is-created-but-child-jobs-andor-pods-are-not-being-created

More details here: https://github.com/kubernetes-sigs/jobset/issues/263

I was using 0.2.0 - so I'll bump up to 0.2.1. Thanks @kannon92 ! Also I didn't know you moved over to RedHat from G-Research? Congrats!

Thank you!
/close


feat(data-classes): adds support for S3 event notifications through EventBridge

Issue number:
#1656
Summary
Changes
adds dataclasses/events support for S3 event notifications through EventBridge
Note: Pay special attention to me overriding mypy checks with # type: ignore[override]
User experience
    from aws_lambda_powertools.utilities.data_classes import event_source, S3EventBridgeNotificationEvent

    @event_source(data_class=S3EventBridgeNotificationEvent)
    def lambda_handler(event: S3EventBridgeNotificationEvent, context):
        bucket_name = event.detail.bucket.name
        file_key = event.detail.object.key

Checklist
If your change doesn't seem to apply, please leave them unchecked.

[x] Meet tenets criteria
[x] I have performed a self-review of this change
[x] Changes have been tested
[x] Changes are documented
[x] PR title follows conventional commit semantics

Is this a breaking change?
RFC issue number:
Checklist:

[ ] Migration process documented
[ ] Implement warnings (if it can live side by side)

Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

Looking

Codecov Report
Patch coverage: 97.87% and project coverage change: +0.01 :tada:

Comparison is base (da87606) 97.44% compared to head (378415d) 97.46%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2024      +/-   ##
===========================================
+ Coverage    97.44%   97.46%   +0.01%
===========================================
  Files          146      146
  Lines         6660     6749      +89
  Branches       478      478
===========================================
+ Hits          6490     6578      +88
- Misses         134      135       +1
  Partials        36       36

Impacted Files
Coverage Δ

...mbda_powertools/utilities/data_classes/s3_event.py
98.81% <96.77%> (-1.19%)
:arrow_down:

...mbda_powertools/utilities/data_classes/__init__.py
100.00% <100.00%> (ø)

...wertools/utilities/data_classes/active_mq_event.py
100.00% <100.00%> (ø)

...bda_powertools/utilities/parser/models/__init__.py
100.00% <100.00%> (ø)

...powertools/utilities/parser/models/event_bridge.py
100.00% <100.00%> (ø)

...ws_lambda_powertools/utilities/parser/models/s3.py
100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Request: /live/track/get/arrangement_clip/file_path

Hi,
Really appreciate this project !
It will be nice if you can add the possibility to get the file_path property of a clip (specially in arrangement).
/live/track/get/arrangement_clip/file_path
I'm also looking for a way to be notify when a new clip is add in arrangement view but I cannot see in the LOM any properties to listen...
Thank you !

Hello
there seems to be already a "file_path" osc-request....
/live/clip/get/file_path
did you see it ?

Yes, I already see it...
it look's like it can only get the file_path of a clip in session view, I'm looking for a clip in Arrangement view.
Thank you !


Make all the non-exported functions static?

mimalloc supports having everything concatenated via static.c, which is great and convenient.
However, symbol visibility attributes don't apply to static libraries (at least on Linux and when compiling to WebAssembly).
So the static library ends up with all the internal symbols being actually visible.
Given that everything can be concatenated, would it make sense to have all the internal symbols defined as static?
I've been working on such a change, which required moving a few things around, but more importantly touches almost all functions, so merging other branches becomes virtually impossible.
Still, would you be possibly open to having the internal symbols marked static? If this is the case, that can be done when the different branches don't diverge too much.

Yes, I like the idea -- it is really a shame that is so tricky to get this right in C.  One neat way around this is to define a new mi_decl_internal that normally expands to nothing (like now), but define it as static for the static.c file.  It does clutter the mimalloc_internal.h declarations though :-(

An mi_decl_internal macro looks like the best way to do it, even though we already have quite a lot of function attributes.


[BEAM-7442][BEAM-5650] Read sequentially from bounded sources in UnboundedSourceWrapper

Bounded sources are converted to unbounded sources using the
BoundedToUnboundedSourceAdapter. This applies the streaming semantics for
unbounded sources, which is to read round-robin from all sources after
splitting. This doesn't work particularly well with a large number of files and
leads to a large number of open file handles / connections.
Instead, we can simple read the bounded sources sequentially.

@JozoVilcek Could you have a look?

Thanks for the review!

Unrelated failure in Java PreCommit: org.apache.beam.runners.dataflow.worker.fn.control.TimerReceiverTest.testSingleTimerScheduling