Streaming Question

I'm just trying to make sure I'm not really using double memory so I wanted to make sure I got how the streaming memory settings work. I use ocamlfuse mount with rclone encrypt. I turned off the 1G buffer I had with rclone currently.
I run with
max_cache_size_mb=100000
max_memory_cache_size=8000000000
max_retries=8
memory_buffer_size=536870912
metadata_cache_time=600
read_ahead_buffers=5
read_only=false
stream_large_files=true

So my theory was for each file I'm streaming, I will cache min ~500M as it seems to grab that in the beginning from my bandwidth consumption and it seems to work as I would expect it. At that point, i have a ~8GB cap so in theory, I should be able to hit ~16 streams at 500x16=~8GB of memory.
I bumped read_ahead to 5 from the default of 3 as I saw that in a different post, but I'm not really sure if 3 to 5 really impacts much.
I've toyed with some various 'min' settings from 10M all the way up to 1G as the goal for me would be to buffer up as much as I can to ensure that no playback issues occur. I used to do that with a rclone mount with the 1G buffer so each file opened would buffer up to 1G.
Hope that make sense and if my assumptions were correct.

With that setup, you get 14 blocks of 512MB. When you access the first byte of a file, it starts downloading the first block (of 512MB), then it spawns 5 threads to download the next 2.5GB of the same file. If you access another file, it does the same, starting to download 3GB (in 6 parallel threads). There are still 2 free blocks (you used 2 blocks for streaming and 10 blocks for read-ahead), so if you access a third file, you start downloading 3GB of this new file: you download the first GB, but then you need 4 more blocks. The oldest 4 blocks (sorted by last access) will be reused and overwritten by the content of this new file.

That helps a bit. So I'm configured bigger than I wanted and I can tweak my config to do what I want. Thanks for the detailed reply.
I'm going to drop down to 50M so I get around 250M with the 5 threads in my buffer. That should be plenty.


feat: add gui for core function

feat

GUI for transcribing video
GUI for choosing subtitle
GUI for  cut video

problem

Video bytes are not used, and additional disks are read and written #23
Streamlet's own defects: once the download button is clicked, the page will be automatically refreshed and all data will be lost

feature todo(after #23 solved)
see https://github.com/mli/autocut/issues/22#issuecomment-1312373402

@yihong0618 lint和test好像有问题，看一下？

好的，稍等

@chenqianhe 你在根目录跑一下 black . 然后再 add commit push
这个是  Lint 挂了


[OTHER] *OR BUG Picamera 0.3.19-1 hangs on easy_capture script while older versions work fine

I don't want to claim this is a bug cause its not really giving much information
I updated my system today to the latest picamera2 (0.3.19-1) and libcamera v0.2.0+120-eb00c13d - suddenly my v3 cameras stopped working in my script when they worked before.
Its a script that starts a stream and it just hangs not even allowing keyboard interrupts. In an effort to see if my cameras are toast. I used the easy_capture script to see if it works.
This is the output from terminal
Python 3.11.2 (main, Mar 13 2023, 12:18:29) [GCC 12.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from picamera2 import Picamera2
>>> Picamera2.set_logging(Picamera2.DEBUG)
>>> picam2 = Picamera2()
[0:02:04.622899557] [2013]  INFO Camera camera_manager.cpp:284 libcamera v0.2.0+120-eb00c13d
[0:02:04.635558974] [2016]  INFO RPI pisp.cpp:695 libpisp version v1.0.6 b567f0455680 17-06-2024 (10:20:00)
[0:02:04.757693176] [2016]  INFO RPI pisp.cpp:1154 Registered camera /base/axi/pcie@120000/rp1/i2c@88000/imx708@1a to CFE device /dev/media1 and ISP device /dev/media0 using PiSP variant BCM2712_C0
[0:02:04.759022655] [2013]  INFO Camera camera_manager.cpp:284 libcamera v0.2.0+120-eb00c13d
[0:02:04.771072906] [2023]  INFO RPI pisp.cpp:695 libpisp version v1.0.6 b567f0455680 17-06-2024 (10:20:00)
[0:02:04.924577790] [2023]  INFO RPI pisp.cpp:1154 Registered camera /base/axi/pcie@120000/rp1/i2c@88000/imx708@1a to CFE device /dev/media1 and ISP device /dev/media0 using PiSP variant BCM2712_C0
picamera2.picamera2 INFO: Initialization successful.
picamera2.picamera2 INFO: Camera now open.
picamera2.picamera2 DEBUG: <libcamera._libcamera.CameraManager object at 0x7fff1fca93f0>
>>> picam2.start_and_capture_file("test.jpg")
[0:02:10.324684140] [2013]  WARN V4L2 v4l2_pixelformat.cpp:344 Unsupported V4L2 pixel format RPBP
picamera2.picamera2 DEBUG: Requesting configuration: {'use_case': 'preview', 'buffer_count': 4, 'transform': <libcamera.Transform 'identity'>, 'display': 'main', 'encode': 'main', 'colour_space': <libcamera.ColorSpace 'sYCC'>, 'controls': <Controls: {'NoiseReductionMode': <NoiseReductionModeEnum.Minimal: 3>, 'FrameDurationLimits': (100, 83333)}>, 'main': {'size': (640, 480), 'format': 'XBGR8888', 'stride': 2560, 'framesize': 1228800}, 'lores': None, 'raw': {'size': (1536, 864), 'format': 'BGGR_PISP_COMP1', 'stride': 1536, 'framesize': 1327104}, 'queue': True, 'sensor': {'bit_depth': 10, 'output_size': (1536, 864)}}
picamera2.picamera2 INFO: Camera configuration has been adjusted!
[0:02:10.326091635] [2013]  INFO Camera camera.cpp:1183 configuring streams: (0) 640x480-XBGR8888 (1) 1536x864-BGGR_PISP_COMP1
[0:02:10.326338868] [2023]  INFO RPI pisp.cpp:1450 Sensor: /base/axi/pcie@120000/rp1/i2c@88000/imx708@1a - Selected sensor format: 1536x864-SBGGR10_1X10 - Selected CFE format: 1536x864-PC1B
picamera2.picamera2 INFO: Configuration successful!
picamera2.picamera2 DEBUG: Final configuration: {'use_case': 'preview', 'buffer_count': 4, 'transform': <libcamera.Transform 'identity'>, 'display': 'main', 'encode': 'main', 'colour_space': <libcamera.ColorSpace 'sYCC'>, 'controls': <Controls: {'NoiseReductionMode': <NoiseReductionModeEnum.Minimal: 3>, 'FrameDurationLimits': (100, 83333)}>, 'main': {'size': (640, 480), 'format': 'XBGR8888', 'stride': 2560, 'framesize': 1228800}, 'lores': None, 'raw': {'size': (1536, 864), 'format': 'BGGR_PISP_COMP1', 'stride': 1536, 'framesize': 1327104}, 'queue': True, 'sensor': {'bit_depth': 10, 'output_size': (1536, 864)}}
picamera2.picamera2 DEBUG: Streams: {'main': <libcamera._libcamera.Stream object at 0x7fff0d9c7270>, 'lores': None, 'raw': <libcamera._libcamera.Stream object at 0x7fff0d9c6970>}
picamera2 DEBUG: Allocated 4 buffers for stream 0 with fds [35, 38, 41, 44]
picamera2 DEBUG: Allocated 4 buffers for stream 1 with fds [47, 50, 53, 56]

After this it just stopped... So I tried again with a v2 camera and got this
Python 3.11.2 (main, Mar 13 2023, 12:18:29) [GCC 12.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from picamera2 import Picamera2
>>> picam2 = Picamera2()
[0:11:06.224905392] [2043]  INFO Camera camera_manager.cpp:284 libcamera v0.2.0+120-eb00c13d
[0:11:06.236794684] [2049]  INFO RPI pisp.cpp:695 libpisp version v1.0.6 b567f0455680 17-06-2024 (10:20:00)
[0:11:06.272370505] [2049]  INFO RPI pisp.cpp:1154 Registered camera /base/axi/pcie@120000/rp1/i2c@88000/imx219@10 to CFE device /dev/media0 and ISP device /dev/media2 using PiSP variant BCM2712_C0
[0:11:06.274088491] [2043]  INFO Camera camera_manager.cpp:284 libcamera v0.2.0+120-eb00c13d
[0:11:06.285950728] [2056]  INFO RPI pisp.cpp:695 libpisp version v1.0.6 b567f0455680 17-06-2024 (10:20:00)
[0:11:06.319488755] [2056]  INFO RPI pisp.cpp:1154 Registered camera /base/axi/pcie@120000/rp1/i2c@88000/imx219@10 to CFE device /dev/media0 and ISP device /dev/media2 using PiSP variant BCM2712_C0
>>> picam2.start_and_capture_file("test.jpg")
[0:11:12.491131810] [2043]  WARN V4L2 v4l2_pixelformat.cpp:344 Unsupported V4L2 pixel format RPBP
[0:11:12.492143403] [2043]  INFO Camera camera.cpp:1183 configuring streams: (0) 640x480-XBGR8888 (1) 640x480-BGGR_PISP_COMP1
[0:11:12.492376343] [2056]  INFO RPI pisp.cpp:1450 Sensor: /base/axi/pcie@120000/rp1/i2c@88000/imx219@10 - Selected sensor format: 640x480-SBGGR10_1X10 - Selected CFE format: 640x480-PC1B
Exception in thread Thread-2 (thread_func):
Traceback (most recent call last):
  File "/usr/lib/python3.11/threading.py", line 1038, in _bootstrap_inner
    self.run()
  File "/usr/lib/python3.11/threading.py", line 975, in run
    self._target(*self._args, **self._kwargs)
  File "/usr/lib/python3/dist-packages/picamera2/previews/null_preview.py", line 29, in thread_func
    callback(picam2)
  File "/usr/lib/python3/dist-packages/picamera2/previews/drm_preview.py", line 83, in handle_request
    picam2.process_requests(self)
  File "/usr/lib/python3/dist-packages/picamera2/picamera2.py", line 1281, in process_requests
    display.render_request(display_request)
  File "/usr/lib/python3/dist-packages/picamera2/previews/drm_preview.py", line 74, in render_request
    self.render_drm(self.picam2, completed_request)
  File "/usr/lib/python3/dist-packages/picamera2/previews/drm_preview.py", line 158, in render_drm
    raise RuntimeError("Failed to reserve DRM plane")
RuntimeError: Failed to reserve DRM plane

Finally I dug out an older but working picamera2 I had on another sdcard and running the same code (without the debug) this is 0.3.17-1
Python 3.11.2 (main, Mar 13 2023, 12:18:29) [GCC 12.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from picamera2 import Picamera2
>>> picam2 = Picamera2()
[0:07:19.986709186] [3487]  INFO Camera camera_manager.cpp:284 libcamera v0.2.0+46-075b54d5
[0:07:20.021358229] [3490]  WARN RPiSdn sdn.cpp:39 Using legacy SDN tuning - please consider moving SDN inside rpi.denoise
[0:07:20.023639494] [3490]  INFO RPI vc4.cpp:447 Registered camera /base/soc/i2c0mux/i2c@1/imx708@1a to Unicam device /dev/media1 and ISP device /dev/media0
[0:07:20.023715399] [3490]  INFO RPI pipeline_base.cpp:1144 Using configuration file '/usr/share/libcamera/pipeline/rpi/vc4/rpi_apps.yaml'
[0:07:20.026278545] [3487]  INFO Camera camera_manager.cpp:284 libcamera v0.2.0+46-075b54d5
[0:07:20.061569810] [3493]  WARN RPiSdn sdn.cpp:39 Using legacy SDN tuning - please consider moving SDN inside rpi.denoise
[0:07:20.063817391] [3493]  INFO RPI vc4.cpp:447 Registered camera /base/soc/i2c0mux/i2c@1/imx708@1a to Unicam device /dev/media1 and ISP device /dev/media0
[0:07:20.063891741] [3493]  INFO RPI pipeline_base.cpp:1144 Using configuration file '/usr/share/libcamera/pipeline/rpi/vc4/rpi_apps.yaml'
>>> picam2.start_and_capture_file("test.jpg")
[0:07:25.907053470] [3487]  INFO Camera camera.cpp:1183 configuring streams: (0) 640x480-XBGR8888 (1) 1536x864-SBGGR10_CSI2P
[0:07:25.908055810] [3493]  INFO RPI vc4.cpp:611 Sensor: /base/soc/i2c0mux/i2c@1/imx708@1a - Selected sensor format: 1536x864-SBGGR10_1X10 - Selected unicam format: 1536x864-pBAA
commit failed
commit failed
[0:07:27.071959931] [3497]  INFO Camera camera.cpp:1183 configuring streams: (0) 4608x2592-BGR888 (1) 4608x2592-SBGGR10_CSI2P
[0:07:27.076532361] [3493]  INFO RPI vc4.cpp:611 Sensor: /base/soc/i2c0mux/i2c@1/imx708@1a - Selected sensor format: 4608x2592-SBGGR10_1X10 - Selected unicam format: 4608x2592-pBAA
[0:07:27.870210299] [3497]  INFO Camera camera.cpp:1183 configuring streams: (0) 640x480-XBGR8888 (1) 1536x864-SBGGR10_CSI2P
[0:07:27.875903895] [3493]  INFO RPI vc4.cpp:611 Sensor: /base/soc/i2c0mux/i2c@1/imx708@1a - Selected sensor format: 1536x864-SBGGR10_1X10 - Selected unicam format: 1536x864-pBAA

So it worked and I got my test.jpg where the other times in 0.3.19 i did not? Or is it the latest libcamera?...

Also I am not sure if its something I am missing or did not do or should update?

Hi, I've just tried the following
from picamera2 import Picamera2
picam2 = Picamera2()
picam2.start_and_capture_file("test.jpg")

and it seemed OK. Can you fill  me in on the details of your system. What kind of a Pi is it? How did you update it? Are you connected directly with a screen and keyboard or remotely, via ssh or vnc? Also, does rpicam-hello (and the other rpicam apps) work? You could also try downgrading picamera2 using pip (which will hide the apt version), though I would generally delete it after the experiment to avoid getting muddled about which one it's using.
Thanks!

System:
Pi5 - Debian GNU/Linux 12 (bookworm) x2 tested on 2 systems one was already updated when I noticed the issue the other was working and then updated then it stopped working
Picamera2 0.3.19-1 and libcamera v0.2.0+120-eb00c13d
Pi4 - Debian GNU/Linux 12 (bookworm)
Picamera2 0.3.17-1 and libcamera v0.2.0+46-075b54d5
Update method:
sudo apt update
Connection - SSH from the report above but since you mentioned it I went back to the pi5 and ran the code on desktop the QTgiPreview window opens but no content is shown its blank.
The pi4 running the same code was over ssh and I got my test.jpg

I'm getting a bit confused here, are we saying your Pi 4 is working and your Pi 5 isn't?
Also note that the recommended update procedure is
sudo apt update
sudo apt full-upgrade

I've run this all again on a Pi 5 which works for me. I have Picamera2 0.3.19. rpicam-hello --version reports
rpicam-apps build: 49344f2a8d18 17-06-2024 (12:09:08)
libcamera build: v0.3.0+65-6ddd79b5

and the firmware version (vcgencmd version) is
2024/04/20 11:53:30
Copyright (c) 2012 Broadcom
version d1744d21 (release) (embedded)

and the Linux kernel (uname -a)
Linux raspberrypi 6.6.31+rpt-rpi-2712 #1 SMP PREEMPT Debian 1:6.6.31-1+rpt1 (2024-05-29) aarch64 GNU/Linux

so those might all be worth checking.

Oh man I am so sorry... Total pebcak moment here. sudo apt full-upgrade was the issue and fixed the problem half way. But yeah I did not run that before.
After the webcam UI I am working on is working again so YAY... BUT...
Running the easy_capture in terminal on desktop the QtGIPreview window opens but still no content and is just black and hangs there. In SSH : the output is
Python 3.11.2 (main, Mar 13 2023, 12:18:29) [GCC 12.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from picamera2 import Picamera2
>>> picam2 = Picamera2()
[0:03:09.308991256] [3538]  INFO Camera camera_manager.cpp:313 libcamera v0.3.0+65-6ddd79b5
[0:03:09.317127318] [3541]  INFO RPI pisp.cpp:695 libpisp version v1.0.6 b567f0455680 17-06-2024 (10:20:00)
[0:03:09.410094418] [3541]  INFO RPI pisp.cpp:1154 Registered camera /base/axi/pcie@120000/rp1/i2c@88000/imx708@1a to CFE device /dev/media1 and ISP device /dev/media0 using PiSP variant BCM2712_C0
[0:03:09.411833601] [3538]  INFO Camera camera_manager.cpp:313 libcamera v0.3.0+65-6ddd79b5
[0:03:09.420133056] [3544]  INFO RPI pisp.cpp:695 libpisp version v1.0.6 b567f0455680 17-06-2024 (10:20:00)
[0:03:09.546696169] [3544]  INFO RPI pisp.cpp:1154 Registered camera /base/axi/pcie@120000/rp1/i2c@88000/imx708@1a to CFE device /dev/media1 and ISP device /dev/media0 using PiSP variant BCM2712_C0
>>> picam2.start_and_capture_file("test.jpg")
[0:03:13.709175917] [3538]  WARN V4L2 v4l2_pixelformat.cpp:344 Unsupported V4L2 pixel format RPBP
[0:03:13.710166610] [3538]  INFO Camera camera.cpp:1183 configuring streams: (0) 640x480-XBGR8888 (1) 1536x864-BGGR_PISP_COMP1
[0:03:13.710367504] [3544]  INFO RPI pisp.cpp:1450 Sensor: /base/axi/pcie@120000/rp1/i2c@88000/imx708@1a - Selected sensor format: 1536x864-SBGGR10_1X10 - Selected CFE format: 1536x864-PC1B
Exception in thread Thread-2 (thread_func):
Traceback (most recent call last):
  File "/usr/lib/python3.11/threading.py", line 1038, in _bootstrap_inner
    self.run()
  File "/usr/lib/python3.11/threading.py", line 975, in run
    self._target(*self._args, **self._kwargs)
  File "/usr/lib/python3/dist-packages/picamera2/previews/null_preview.py", line 29, in thread_func
    callback(picam2)
  File "/usr/lib/python3/dist-packages/picamera2/previews/drm_preview.py", line 83, in handle_request
    picam2.process_requests(self)
  File "/usr/lib/python3/dist-packages/picamera2/picamera2.py", line 1281, in process_requests
    display.render_request(display_request)
  File "/usr/lib/python3/dist-packages/picamera2/previews/drm_preview.py", line 74, in render_request
    self.render_drm(self.picam2, completed_request)
  File "/usr/lib/python3/dist-packages/picamera2/previews/drm_preview.py", line 158, in render_drm
    raise RuntimeError("Failed to reserve DRM plane")
RuntimeError: Failed to reserve DRM plane

For now I guess its working enough for me to continue what I am doing but I don't know what the issue is with the black preview window (camera works fine otherwise)

Well, there's still something a bit weird going on there. If it's complaining about DRM, that means it thinks the desktop isn't running and that it's not connected via ssh. To be fair, figuring out how you're connected is actually quite tricky, so that can go wrong. It would certainly get confused if you use ssh but without -X (X-forwarding) and then try to display a preview window. So maybe just check what the Pi's $DISPLAY variable says once you've logged in. The rules are:

ssh without -X : you can't have a preview window.
ssh with -X : preview window should work, but you'll get a software rendered preview spamming your network with uncompressed video.

Yeah  checked export but display is not set... then for giggles I retested the easy_capture and it worked fine! Might have needed a reboot and a dodgy ssh connection... new computer and its not totally setup the way I like it yet.
Thank you so much for the support and sorry it was a silly one

No problem, good luck!!


be more specific about OSGi bundle exports

The OSGi configuration should specify version numbers and should not export third-party packages

This is no longer relevant.


Initial draft of 90 CU1

@peterpogorski @sukanyamsft @craftyhouse @dragav @shsha-msft Can you please review? Waiting to get confirmation on Service Fabric Standalone Runtime and Service Fabric Standalone Installer Package direct download links.


how to know if my connection is under wss or if the keys key.pem and cert.pem are really running?

Hey guys, I'm trying to access my server example: EchoServer.cpp with wss:// but it doesn't connect, only with ws:// what am I doing wrong?
my vcpkg list :
D:\cppSolutions\vcpkg\vcpkg>vcpkg list libuv:x64-windows                                 1.46.0              libuv is a multi-platform support library with a... libuv:x86-windows                                 1.46.0              libuv is a multi-platform support library with a... openssl:x64-windows                               3.1.1#1             OpenSSL is an open source project that provides ... openssl:x86-windows                               3.1.1#1             OpenSSL is an open source project that provides ... usockets:x64-windows                              0.8.6#1             Miniscule cross-platform eventing, networking & ... usockets:x86-windows                              0.8.6#1             Miniscule cross-platform eventing, networking & ... usockets[ssl]:x64-windows                                             Build usockets with openssl support usockets[ssl]:x86-windows                                             Build usockets with openssl support uwebsockets:x64-windows                           20.45.0             Simple, secure & standards compliant web I/O for... uwebsockets:x86-windows                           20.45.0             Simple, secure & standards compliant web I/O for... vcpkg-cmake-config:x64-windows                    2022-02-06#1 vcpkg-cmake-get-vars:x64-windows                  2023-03-02 vcpkg-cmake:x64-windows                           2023-05-04 zlib:x64-windows                                  1.2.13              A compression library zlib:x86-windows                                  1.2.13              A compression library

mee too.listenSocket is null.

EchoServer.cpp does not use SSL, so it makes no sense to assume it would accept wss. If you look at the example, it even tells you in comments how to make it run over TLS.


We need to do a complet rethinking of this in case of JS as devtools exist!

Is it applicable for Node or Browser or both
both it is applicable for all Stealify supported Languages: ECMAScript / Typescript / Javascript Engines
eg: GraalJS / NodeJS / Deno / Firefox Spidermonkey / Chromium v8 / JavaScriptCore (Apple)
we should base on the devtools protocol implementation as this adds already instrumentation on the Engine Level and JSON formated we need to define a Schema and then get all data directly via the devtools protocol.
Do you expect this plugin to be commonly used
Weekly Downloads: Should be higher then existing user base
What version of plugin are you interested in using
Versions: 1.0
Additional context

Is there a reference you could point for the well-defined lifecycle methods**
devtools protocol

Could you provide some links to the devtool protocol you propose to use? Is there already some POC for e.g. a node.js http instrumentation based on this protocol?

@Flarna That is a big effort we should maybe Schedule Weekly Iterations and you ramp up a effort to collect questions of the developers that do the integration i can lead that effort and get you the information you should get familar with:

http://google.github.io/tracing-framework/index.html
https://github.com/google/tracing-framework/

then you can jump into the puppeteer cdp connectiom implementation and code
in general the devtools protocol is simple
Terminal 1:
$ mkfifo input
$ echo '{"id": 0, "method": "Target.getTargets"}' >input

Terminal 2:
$ mkfifo output
$ cat output

Terminal 3:
$ chromium --headless --remote-debugging-pipe 3<input 4>output

Main part
$ echo '{"id": 0, "method": "Target.getTargets"}' >input
this is how the devtools protocol looks under the hood there do exist complet type definitions under the npm devtools-protocol package also the puppeteer implementation is a simple wrapper around that with some extra functions.
Importent knowleg for the Main Integrational Part is:

namedPipes
stdio in/out/ fileDescriptors
devtools protocol main methods for instrumentation

Final Goal
you will create open-telemetry run <node> ..........
the open-telemetry binary will use the devtools protocol to instrument the process what ever it is. At last that is the best Api a tool with your size of settings can come up with.

You need a set of contributors that get familar to browser automation and then apply same knowleg to nodejs and GraalVM both also support the same method but only the none browser subset like profiling and heatmaps as also other instrumentations. GraalVM is also able to implement Own Universal Instrumentations for any lang linked to the devtools protocol: https://www.graalvm.org/22.0/graalvm-as-a-platform/implement-instrument/
also the existing languages got instrumentation build in already you can see that via GraalVM's visual Graph tools and so on but it also all works with devtools

You cannot access devtools protocol within a browser (for security purpose since it give a direct access to v8) so i don't see how this would work cross runtime as ou suggested

Maybe this is a missing part a extension can access chrome.debbuger

https://developer.chrome.com/docs/extensions/reference/debugger/

this is a alternativ way to interact directly with the devtools programatical inside the browser without opening the devtools console.
else even naiv scripts in the devtools console will do the trick.

Devtools input/output
in Nodejs

devtools server running via CLI flag

--remote-debugging- or

In Browser

devtools chrome.debugger used via extension with debugger enabled manifest
devtools console

The devtools referenced here seem to server a different purpose then OpenTelemetry. OpenTelemetry traces are intended to trace distributed systems not for profiling a single application. The instrumentations target user land modules like database drivers, communication frameworks,.. not the runtime itself.
I agree that devtools offer a lot possibilities regarding how to instrument because the debugger has access to areas user code will never have. But as far as I know the overhead of the devtools once debugger is uses is very high (CPU/Heap profiling in v8 is not using the debugger).

I'm going to close this as it is too broad and out of scope. Maybe an interesting developer-focused enhancement in the future, but not applicable right now.


Changed current time location to h file.

In order to change the current time of the timer (programmatically)

I'm sorry. It will not work. I corrected in last version.


[Transporter Error Output]: ERROR ITMS-90022: "Missing required icon file. The bundle does not contain an app icon for iPhone / iPod Touch of exactly '120x120' pixels, in .png format for iOS versions >= 10.0."

I have run the fastlane beta command :-
lane :beta do
gym(scheme: "FastLaneDemo”,
include_bitcode: true) # Build your app - more options available
pilot
end
Following is the log :-
iosdev02:FastLaneDemo angshukn$ fastlane beta
[10:42:11]: Your Fastfile has had smart quotes sanitised. To avoid issues in the future, you should not use TextEdit for editing it. If you are not using TextEdit, you should turn off smart quotes in your editor of choice.
[10:42:12]: -------------------------------------------------
[10:42:12]: --- Step: Verifying required fastlane version ---
[10:42:12]: -------------------------------------------------
[10:42:12]: Your fastlane version 2.14.2 matches the minimum requirement of 2.14.2  ✅
[10:42:12]: ------------------------------
[10:42:12]: --- Step: default_platform ---
[10:42:12]: ------------------------------
[10:42:12]: Driving the lane 'ios beta' 🚀
[10:42:12]: -----------------
[10:42:12]: --- Step: gym ---
[10:42:12]: -----------------
[10:42:12]: $ xcodebuild -list -project ./FastLaneDemo.xcodeproj
[10:42:13]: $ xcodebuild clean -showBuildSettings -scheme FastLaneDemo -project ./FastLaneDemo.xcodeproj
+----------------------+--------------------------+
|             Summary for gym 2.14.2              |
+----------------------+--------------------------+
| scheme               | FastLaneDemo             |
| include_bitcode      | true                     |
| project              | ./FastLaneDemo.xcodeproj |
| destination          | generic/platform=iOS     |
| output_name          | FastLaneDemoNew          |
| clean                | false                    |
| output_directory     | .                        |
| silent               | false                    |
| use_legacy_build_api | false                    |
| buildlog_path        | ~/Library/Logs/gym       |
| xcode_path           | /Applications/Xcode.app  |
+----------------------+--------------------------+
[10:42:14]: $ set -o pipefail && xcodebuild -scheme FastLaneDemo -project ./FastLaneDemo.xcodeproj -destination 'generic/platform=iOS' -archivePath /Users/angshukn/Library/Developer/Xcode/Archives/2017-02-08/FastLaneDemoNew\ 2017-02-08\ 10.42.14.xcarchive archive | tee /Users/angshukn/Library/Logs/gym/FastLaneDemoNew-FastLaneDemo.log | xcpretty
[10:42:15]: ▸ Building FastLaneDemo/FastLaneDemo [Release]
[10:42:15]: ▸ Check Dependencies
[10:42:16]: ▸ Linking FastLaneDemoNew
[10:42:16]: ▸ Linking FastLaneDemoNew
[10:42:16]: ▸ Compiling Main.storyboard
[10:42:19]: ▸ Compiling LaunchScreen.storyboard
[10:42:19]: ▸ Processing Info.plist
[10:42:19]: ▸ Generating 'FastLaneDemoNew.app.dSYM'
[10:42:19]: ▸ Touching FastLaneDemoNew.app
[10:42:33]: ▸ Signing /Users/angshukn/Library/Developer/Xcode/DerivedData/FastLaneDemo-armwumtptzjnoaahtfcvanxjpqze/Build/Intermediates/ArchiveIntermediates/FastLaneDemo/InstallationBuildProductsLocation/Applications/FastLaneDemoNew.app
[10:42:34]: ▸ Touching FastLaneDemoNew.app.dSYM
[10:42:37]: ▸ Archive Succeeded
[10:42:37]: Generated plist file with the following values:
[10:42:37]: ▸ -----------------------------------------
[10:42:37]: ▸ {
[10:42:37]: ▸   "method": "app-store",
[10:42:37]: ▸   "uploadSymbols": true,
[10:42:37]: ▸   "uploadBitcode": true
[10:42:37]: ▸ }
[10:42:37]: ▸ -----------------------------------------
[10:42:37]: $ /usr/bin/xcrun /Users/angshukn/.fastlane/bin/bundle/lib/ruby/gems/2.2.0/gems/fastlane-2.14.2/gym/lib/assets/wrap_xcodebuild/xcbuild-safe.sh -exportArchive -exportOptionsPlist '/var/folders/7f/dr4fshls3257f4vj5ykc1kr4xsq0jv/T/gym_config20170208-3468-1dcetw2.plist' -archivePath /Users/angshukn/Library/Developer/Xcode/Archives/2017-02-08/FastLaneDemoNew\ 2017-02-08\ 10.42.14.xcarchive -exportPath '/var/folders/7f/dr4fshls3257f4vj5ykc1kr4xsq0jv/T/gym_output20170208-3468-jad1rf'
[10:43:08]: Compressing 1 dSYM(s)
[10:43:08]: $ cd '/Users/angshukn/Library/Developer/Xcode/Archives/2017-02-08/FastLaneDemoNew 2017-02-08 10.42.14.xcarchive/dSYMs' && zip -r '/Users/angshukn/Documents/Projects/Demo/FastLaneDemo/FastLaneDemoNew.app.dSYM.zip' *.dSYM
[10:43:09]: ▸ updating: FastLaneDemoNew.app.dSYM/ (stored 0%)
[10:43:09]: ▸ updating: FastLaneDemoNew.app.dSYM/Contents/ (stored 0%)
[10:43:09]: ▸ updating: FastLaneDemoNew.app.dSYM/Contents/Info.plist (deflated 52%)
[10:43:09]: ▸ updating: FastLaneDemoNew.app.dSYM/Contents/Resources/ (stored 0%)
[10:43:09]: ▸ updating: FastLaneDemoNew.app.dSYM/Contents/Resources/DWARF/ (stored 0%)
[10:43:09]: ▸ updating: FastLaneDemoNew.app.dSYM/Contents/Resources/DWARF/FastLaneDemoNew (deflated 70%)
[10:43:09]: Successfully exported and compressed dSYM file
[10:43:09]: Successfully exported and signed the ipa file:
[10:43:09]: /Users/angshukn/Documents/Projects/Demo/FastLaneDemo/FastLaneDemoNew.ipa
[10:43:09]: -------------------
[10:43:09]: --- Step: pilot ---
[10:43:09]: -------------------
[10:43:09]: Login to iTunes Connect<EMAIL_ADDRESS>[10:43:13]: Login successful
[10:43:15]: Ready to upload new build to TestFlight (App:<PHONE_NUMBER>)...
[10:43:15]: Going to upload updated app to iTunes Connect
[10:43:15]: This might take a few minutes. Please don't interrupt the script.
[10:44:02]: [Transporter Error Output]: ERROR ITMS-90022: "Missing required icon file. The bundle does not contain an app icon for iPhone / iPod Touch of exactly '120x120' pixels, in .png format for iOS versions >= 10.0."
[10:44:02]: [Transporter Error Output]: ERROR ITMS-90023: "Missing required icon file. The bundle does not contain an app icon for iPad of exactly '167x167' pixels, in .png format for iOS versions supporting iPad Pro."
[10:44:02]: [Transporter Error Output]: ERROR ITMS-90023: "Missing required icon file. The bundle does not contain an app icon for iPad of exactly '152x152' pixels, in .png format for iOS versions >= 10.0."
[10:44:02]: Transporter transfer failed.
[10:44:02]:
[10:44:02]: ERROR ITMS-90022: "Missing required icon file. The bundle does not contain an app icon for iPhone / iPod Touch of exactly '120x120' pixels, in .png format for iOS versions >= 10.0."
ERROR ITMS-90023: "Missing required icon file. The bundle does not contain an app icon for iPad of exactly '167x167' pixels, in .png format for iOS versions supporting iPad Pro."
ERROR ITMS-90023: "Missing required icon file. The bundle does not contain an app icon for iPad of exactly '152x152' pixels, in .png format for iOS versions >= 10.0."
[10:44:02]: ERROR ITMS-90022: "Missing required icon file. The bundle does not contain an app icon for iPhone / iPod Touch of exactly '120x120' pixels, in .png format for iOS versions >= 10.0."
ERROR ITMS-90023: "Missing required icon file. The bundle does not contain an app icon for iPad of exactly '167x167' pixels, in .png format for iOS versions supporting iPad Pro."
ERROR ITMS-90023: "Missing required icon file. The bundle does not contain an app icon for iPad of exactly '152x152' pixels, in .png format for iOS versions >= 10.0."
\nERROR ITMS-90023: "Missing required icon file. The bundle does not contain an app icon for iPad of exactly '152x152' pixels, in .png format for iOS versions >= 10.0."g iPad Pro."s, in .png format for iOS versions >= 10.0."
The call to the iTMSTransporter completed with a non-zero exit status: 1. This indicates a failure.
+--------------------+-----------------------------------------------------------------------------+
|                                           Lane Context                                           |
+--------------------+-----------------------------------------------------------------------------+
| DEFAULT_PLATFORM   | ios                                                                         |
| PLATFORM_NAME      | ios                                                                         |
| LANE_NAME          | ios beta                                                                    |
| IPA_OUTPUT_PATH    | /Users/angshukn/Documents/Projects/Demo/FastLaneDemo/FastLaneDemoNew.ipa    |
| DSYM_OUTPUT_PATH   | /Users/angshukn/Documents/Projects/Demo/FastLaneDemo/FastLaneDemoNew.app... |
| XCODEBUILD_ARCHIVE | /Users/angshukn/Library/Developer/Xcode/Archives/2017-02-08/FastLaneDemo... |
+--------------------+-----------------------------------------------------------------------------+
[10:44:02]: Error uploading ipa file, for more information see above
+------+-------------------------------------+-------------+
|                     fastlane summary                     |
+------+-------------------------------------+-------------+
| Step | Action                              | Time (in s) |
+------+-------------------------------------+-------------+
| 1    | Verifying required fastlane version | 0           |
| 2    | default_platform                    | 0           |
| 3    | gym                                 | 57          |
| 💥   | pilot                               | 53          |
+------+-------------------------------------+-------------+
[10:44:02]: fastlane finished with errors
[!] Error uploading ipa file, for more information see above
iosdev02:FastLaneDemo angshukn$
I received the "Missing required icon file" errors. i have put all the required icons in AppIcon.appiconset still i am facing this issue. Can someone please help me on this
Thanks & Regards
Angshuk

@fastlane-bot
No Sir. That might not be the problem

I had installed fastlane while there was xcode 7.3.1 in applications folder and xcode 8.1 in documents directory. Then i deleted xcode 7.3.1 and copied xcode 8.1 in applications directory but still the issue was coming. Then i deleted the xcode 8.1 which was present in documents directory. Now transporter is uploading fine.


Snapshot visitor improvements

@bot-gradle test QFL please.

OK, I've already triggered QuickFeedbackLinux build for you.

@bot-gradle test QFL please.

OK, I've already triggered QuickFeedbackLinux build for you.

@bot-gradle test QFL please.

OK, I've already triggered QuickFeedbackLinux build for you.

@bot-gradle test QFL please.

OK, I've already triggered QuickFeedbackLinux build for you.

@bot-gradle test this please.

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test this please.

OK, I've already cancelled the old build and triggered a new ReadyForMerge build for you.

@bot-gradle test QFL please.

OK, I've already triggered QuickFeedbackLinux build for you.

@bot-gradle test QFL please.

OK, I've already triggered QuickFeedbackLinux build for you.

@bot-gradle test QFL please.

OK, I've already cancelled the old build and triggered a new QuickFeedbackLinux build for you.

Sheet with different requirements for visitors: https://docs.google.com/spreadsheets/d/1gloPSlutQPIq0SRMLG9zrbu9mum737WLj_BrrDr53Hk

@bot-gradle test QFL please.

OK, I've already triggered QuickFeedbackLinux build for you.

@bot-gradle test QFL please.

OK, I've already triggered QuickFeedbackLinux build for you.

@bot-gradle test QFL please.

OK, I've already triggered QuickFeedbackLinux build for you.

@bot-gradle test QFL please.

OK, I've already triggered QuickFeedbackLinux build for you.

@bot-gradle test this please.

OK, I've already triggered ReadyForMerge build for you.

One more comment: How hard is it to move the absolutePath from MerkleDirectorySnapshotBuilder.preVisitDirectory() to MerkleDirectorySnapshotBuilder.postVisitDirectory() as well?

One more comment: How hard is it to move the absolutePath from MerkleDirectorySnapshotBuilder.preVisitDirectory() to MerkleDirectorySnapshotBuilder.postVisitDirectory() as well?

I did that once, but decided it was too much change. I can redo it again, it's simple and makes the caller code better.

@bot-gradle test this please.

OK, I've already cancelled the old build and triggered a new ReadyForMerge build for you.

One more comment: How hard is it to move the absolutePath from MerkleDirectorySnapshotBuilder.preVisitDirectory() to MerkleDirectorySnapshotBuilder.postVisitDirectory() as well?

I did that once, but decided it was too much change. I can redo it again, it's simple and makes the caller code better.

I think we should either pass the arguments to preVisitDirectory() or to postVisitDirectory() but not have a mix. And I also think it would be better to pass the things in postVisitDirectory().
So please re-do the change you already had. I also wonder if we should change the names of MerkleDirectorySnapshotBuilder.{pre,post}VisitDirectory() to be in-line with the ones in the snapshot visitors. Or maybe they are different after all.

Passing to postVisitDirectory() now.

I wanted to also rename those methods, but wanted to do it in a separate PR. Alas, I can do it here, too.

@bot-gradle test this please.

OK, I've already cancelled the old build and triggered a new ReadyForMerge build for you.

@bot-gradle test this please.

Sorry some internal error occurs, please contact the administrator @blindpirate

@bot-gradle test this please.

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test this please.

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test this please.

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test QFL please.

OK, I've already triggered QuickFeedbackLinux build for you.

@bot-gradle test this please.

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test this please.

OK, I've already triggered ReadyForMerge build for you.

@bot-gradle test QFL please.

OK, I've already triggered QuickFeedbackLinux build for you.

@bot-gradle test QF please.

OK, I've already triggered QuickFeedback build for you.


Add workflow and check introducer do not distinguish lowercase.

Add workflow
check introducer do not distinguish the lower or upper case.

LGTM


Init frontend framework

Why are the changes needed?
This is a pr about init frontend framework using vue3.

How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Codecov Report

Merging #3250 (a1aaba0) into master (8c2e774) will increase coverage by 0.11%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3250      +/-   ##
============================================
+ Coverage     50.94%   51.06%   +0.11%
  Complexity       13       13
============================================
  Files           469      470       +1
  Lines         26123    26224     +101
  Branches       3612     3626      +14
============================================
+ Hits          13308    13390      +82
- Misses        11544    11555      +11
- Partials       1271     1279       +8

Impacted Files
Coverage Δ

...authentication/AuthenticationProviderFactory.scala
70.00% <0.00%> (-3.69%)
:arrow_down:

...e/authentication/KyuubiAuthenticationFactory.scala
76.81% <0.00%> (-1.13%)
:arrow_down:

...he/kyuubi/ha/client/etcd/EtcdDiscoveryClient.scala
67.50% <0.00%> (-0.63%)
:arrow_down:

...n/scala/org/apache/kyuubi/engine/ProcBuilder.scala
82.60% <0.00%> (-0.63%)
:arrow_down:

...g/apache/kyuubi/operation/BatchJobSubmission.scala
74.40% <0.00%> (ø)

...uthentication/JdbcAuthenticationProviderImpl.scala
79.74% <0.00%> (ø)

...in/scala/org/apache/kyuubi/config/KyuubiConf.scala
97.40% <0.00%> (+0.04%)
:arrow_up:

...mon/src/main/scala/org/apache/kyuubi/Logging.scala
44.73% <0.00%> (+1.31%)
:arrow_up:

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Thanks @tongwl, I think it's almost there, w/ a fews license issues.
The Apache Rat reports 5 files w/o license header issue.
5 Unknown Licenses

*****************************************************

Files with unapproved licenses:

  kyuubi-server/web-ui/pnpm-lock.yaml
  kyuubi-server/web-ui/src/main.ts
  kyuubi-server/web-ui/src/vite-env.d.ts
  kyuubi-server/web-ui/src/components.d.ts
  kyuubi-server/web-ui/src/auto-imports.d.ts

*****************************************************

I think

pnpm-lock.yaml could be excluded in .rat-excludes
components.d.ts and auto-imports.d.ts should be we removed?
add license header for vite-env.d.ts if possible or exclude it in .rat-excludes
add license header for main.ts

After solving above issue, I'm going to merge this PR.

Thanks @tongwl, merging to master for 1.7

Hi @tongwl @pan3793 , Any documentation/issue/discussion/Prototype drawing for this change? I want to see what the purpose/function of the UI is?


[ELWIC] Infinity Complex - Module crashes during game message

Module Information

Identifier: ELWIC
Module Name: Infinity Complex
Version: 6.04

Describe the bug
During the game Infinity Complex alerts players to parts shipments and provides information as to where they can be collected. These messages are displayed but some of the relative locations show negative numbers and MBBSEmu crashes with a System.FormatException: Input string was not in a correct format message.
To Reproduce
Steps to reproduce the behavior:

Enter the module
Wait for a few minutes in any room for the shipment announcement to be made.

Expected behavior
Location of the parts shipment displayed relative to the players location.
Screenshots
Unhandled exception. System.FormatException: Input string was not in a correct format.
   at System.Number.ThrowOverflowOrFormatException(ParsingStatus status, TypeCode type)
   at System.Int16.Parse(ReadOnlySpan`1 s, NumberStyles style, IFormatProvider provider)
   at MBBSEmu.Module.McvFile.GetNumeric(Int32 ordinal)
   at MBBSEmu.HostProcess.ExportedModules.Majorbbs.numopt()
   at MBBSEmu.HostProcess.ExportedModules.Majorbbs.Invoke(UInt16 ordinal, Boolean offsetsOnly)
   at MBBSEmu.HostProcess.ExecutionUnits.ExecutionUnit.ExternalFunctionDelegate(UInt16 ordinal, UInt16 functionOrdinal)
   at MBBSEmu.CPU.CpuCore.Tick()
   at MBBSEmu.HostProcess.ExecutionUnits.ExecutionUnit.Execute(IntPtr16 entryPoint, UInt16 channelNumber, Boolean simulateCallFar, Boolean bypassState, Queue`1 initialStackValues, UInt16 initialStackPointer)
   at MBBSEmu.Module.MbbsModule.Execute(IntPtr16 entryPoint, UInt16 channelNumber, Boolean simulateCallFar, Boolean bypassSetState, Queue`1 initialStackValues, UInt16 initialStackPointer)
   at MBBSEmu.HostProcess.MbbsHost.WorkerThread()
   at System.Threading.ThreadHelper.ThreadStart_Context(Object state)
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location where exception was thrown ---
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
   at System.Threading.ThreadHelper.ThreadStart()

Software Information:

OS: CentOS 7.7.1908
Telnet Client Netrunner
MBBSEmu Build #090220-4

It appears that the issue is cause by MBBSEmu not properly parsing numopt fields in an MCV file.
The current logic assumes that the number will be by itself the MCV field, but it appears you can have leading text before the number and the leading text is not parsed. An example here is the MCV value { ... 1 } should be parsed as 1 by numopt().
I've added a new GetNumericMessageValue() to McvFile.cs which implements this logic for the numeric retrieval options. Testing this now but I believe it'll work.


WP Social Login

Hi, I have this problem: ?action=wordpress_social_authenticated&provider=Facebook&mode=login after login with facebook. Someone know how to fix this problem?

Did you ever figure out how to resolve this issue?  Sign Up seems to work fine, but then the user is left on a 404 page in my app. I'm a beginner so this is baffling.


Old stuff. Anyone maintaining this or new approach?

What is new approach?

I think this approach is a bit too complicated imo. I would just solve it with triggers alone, insert record to some table like "change_notifications" and have a background service that checks this table for new records and sends out notifications/ using a queue and ordering items in a queue by "notification_time". At least for my use case that would work

I looked closer to this problem and realized that it seems there is no reason to use any external libraries like this for this purpose. The only reason why people come here is multiple copy-pasted blog articles that referenced it.
I did my code based on the Microsoft.Data.SqlClient and specifically on SqlDataReader. The code is fairly simple, it's just a couple of dozens lines of code. Few month in production, works well.  No triggers required. Not sure if I understand what is "a queue by "notification_time". None of them required. The basic idea how it works is registering a "snapshot" of some SELECT which is stored as notification stored procedure on the SQL server. As soon as content of the same SELECT changes, SQL emits notification that is converted to the regular C# event.

I looked closer to this problem and realized that it seems there is no reason to use any external libraries like this for this purpose. The only reason why people come here is multiple copy-pasted blog articles that referenced it.
I did my code based on the Microsoft.Data.SqlClient and specifically on SqlDataReader. The code is fairly simple, it's just a couple of dozens lines of code. Few months in production, works well. No triggers required. Not sure if I understand what is "a queue by "notification_time". None of them required. The basic idea how it works is registering a "snapshot" of some SELECT which is stored as notification stored procedure on the SQL server. As soon as content of the same SELECT changes, SQL emits notification that fires up the regular C# event.
I don't remember how I came up to the solution. ChatGPT, I think.

So you're basically just comparing two SELECT results and detecting what got deleted, added or changed. I would still choose the triggers over this

No, of course no, No polling. I don't do it manually, server does it. Internal SQL mechanism seems to be the same to triggers but triggers have no built-in notification mechanism, they are supposed to be for data manipulation while you need notifications. So, it's like conditional triggers that emit notifications through service broker. Try to re-read the text for this library. It's not as complex as it looks like. General principles are explained correctly but you don't need this library itself: it's already built-in.

@nikriaz Are you saying about change data capture feature?

yes


Update of get_eight_params

This commit adds doc strings and an optional fill_angle argument to get_eight_params. This can be used to set structurally determined angles to zero when embedding uncertainty vectors. For Hexagonal, this fixes a bug for the γ angle, which was previously not set to zero because it is not equal to π/2.
Secondary changes:

Updates dependencies.
Sets continuous integration base version to Julia 1.8.

Codecov Report

Merging #43 (e0983cb) into main (e9004fe) will increase coverage by 1.63%.
The diff coverage is 61.11%.

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

@@            Coverage Diff             @@
##             main      #43      +/-   ##
==========================================
+ Coverage   81.38%   83.02%   +1.63%
==========================================
  Files          13       13
  Lines         951      919      -32
==========================================
- Hits          774      763      -11
+ Misses        177      156      -21

Impacted Files
Coverage Δ

src/optimize.jl
72.38% <33.33%> (+1.53%)
:arrow_up:

src/crystalphase.jl
81.63% <66.66%> (+2.91%)
:arrow_up:

src/peak.jl
91.66% <0.00%> (-0.65%)
:arrow_down:

src/phasemodel.jl
89.53% <0.00%> (-0.24%)
:arrow_down:

src/optimizationsettings.jl
94.23% <0.00%> (-0.22%)
:arrow_down:

src/util.jl
97.43% <0.00%> (+2.31%)
:arrow_up:

src/wildcard.jl
84.37% <0.00%> (+4.37%)
:arrow_up:

src/peakmodCP.jl
85.71% <0.00%> (+6.54%)
:arrow_up:

... and 1 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

The failures for the nightly, non-release build of Julia seem to be caused by a compatibility issue with DelimitedFiles. The current main branch also fails on this, so we don't have to be concerned about that at this point.


parseAwait param.

I noticed this:
this.parseAwait(refDestructuringErrors)

but parseAwait doesn't accept a param. Is this a bug?

That's a good question cc @mysticatea who introduced parseAwait.

It's a leak of fixing.
In the first implement, parseAwait had a parameter, but the parameter was removed in #468.


Add technical details

Please provide additional technical information.

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

ID: d329323e-41d1-3073-1203-438fdc0de607
Version Independent ID: d329323e-41d1-3073-1203-438fdc0de607
Content: Use custom connectors with the timeline in Power Apps - Power Apps
Content Source: powerapps-docs/maker/model-driven-apps/custom-connectors-timeline-control.md
Service: powerapps
Sub-service: mda-maker
GitHub Login: @lalexms
Microsoft Alias: laalexan

Hi @mathiasbl , I am an engineer on this project, can you provide me more specifics regarding what technical details would help you implement a custom connector successfully?

Hi @mathiabl, were you able to provide details? If this issue is not resolved, please let us know. Thanks!


Windows 11 Install error !! Error: WriteWrap.onWriteComplete

Error: write EOF
    at WriteWrap.onWriteComplete [as oncomplete] (node:internal/stream_base_commons:94:16)

Kit 1.40.1
Electron 22.0.0
win32 10.0.25276
Locale: en-US

@Mic-360 Can you please try this fix and see if it works for you?
https://github.com/johnlindquist/kit/issues/984#issuecomment-1379993381

It's already renamed as bin

@Mic-360 Can you post your logs:
Main.log: %USERPROFILE%\AppData\Roaming\Kit\logs\main.log
Kit.log: %USERPROFILE%.kit\logs\kit.log
Or let me know if anything looks suspicious. I've installed the Windows version on 3 machines at home without issue, so I need to rely on the community to help me track these down.

I removed it and changed my installed nodeJs version to Node v16 and it worked.

@Mic-360 Oh, thanks, that's super helpful. I need to check if Windows is falling back to the system node if the Kit node download fails. That could explain why it behaves differently on different Windows machines.


kie-server-tests: enable custom settings.xml when building kjars for testing

the settings.xml file can be used to define additional repos
to download artifacts from (e.g. when testing staged build)

@mswiderski, @sutaakar please review

looks good to me (though haven't executed tests with it)

Looks good to me too (didn't try it yet either).
Just one small idea, any special reason to define default value for build settings parameter? You may use constructor without default value - default will be null then.

The only reason was that I don't like nulls :) But looking at the other config options now, they are not using any default value, so I will fix that to make it consistent. Thanks for pointing this out!

Fixed - using the constructor without default value.

thanks, I'll merge locally and retest all before push


Minimize_Amplitude_After_K_Removals.py

Have you read the Contributing Guidelines on Pull Requests?
YES
Description
I have uploaded the file for Minimize Amplitude After K Removals.py #4438
Checklist

[ ✔ ] I've read the contribution guidelines.
[ ✔ ] I've checked the issue list before deciding what to submit.
[ ✔ ] I've edited the README.md and link to my code.

Related Issues or Pull Requests
issues No:- #4438 (Minimize Amplitude After K Removals.py)

sir @HarshCasper , @siddharth25pandey, @bhav09
I have made the PR,
my file can be tested

sir @HarshCasper , @siddharth25pandey, @bhav09, @ArchanGhosh
I have made the PR,
my file can be tested

Yes sir will change those @ankitaggarwal23

Yes sir done those changes @ankitaggarwal23

sir @HarshCasper , @siddharth25pandey, @bhav09, @ArchanGhosh
I have made the PR after doing the required changes
my file can be tested

tq @HarshCasper @ankitaggarwal23 sir for motivating me, I will work hard to give you the best questions for the repo

Please make an issue here and raise a PR, for an article as part of our Docs repository: https://github.com/TesseractCoding/NeoAlgo-Docs
It would be much appreciated. Thanks!


test_gdal_latest.yml: Update osgeo/proj container and Python version

Update the container of the test_gdal_latest.yml workflow to from osgeo/proj 9.1.0 to 9.4.1.
See:

https://github.com/OSGeo/PROJ/releases
https://download.osgeo.org/proj/

Closing, for now, to get #1423 first in separately.


Unable to run farmbot.rb

I followed the instructions for installing the software on my Raspberry Pi 3, but when I try to run 'ruby farmbot.rb', I get the following error:

pi@farmbot:~/farmbot-raspberry-pi-controller$ ruby farmbot.rb
/home/pi/.rvm/rubies/ruby-2.3.0/lib/ruby/2.3.0/rubygems/core_ext/kernel_require.rb:120:in require': cannot load such file -- pry (LoadError)         from /home/pi/.rvm/rubies/ruby-2.3.0/lib/ruby/2.3.0/rubygems/core_ext/kernel_require.rb:120:in require'
from /home/pi/.rvm/gems/ruby-2.3.0/gems/activesupport-4.2.3/lib/active_support/dependencies.rb:274:in block in require'         from /home/pi/.rvm/gems/ruby-2.3.0/gems/activesupport-4.2.3/lib/active_support/dependencies.rb:240:in load_dependency'
from /home/pi/.rvm/gems/ruby-2.3.0/gems/activesupport-4.2.3/lib/active_support/dependencies.rb:274:in require'         from /home/pi/farmbot-raspberry-pi-controller/lib/telemetry_message.rb:1:in <top (required)>'
from /home/pi/farmbot-raspberry-pi-controller/lib/command_objects/send_message.rb:3:in require_relative'         from /home/pi/farmbot-raspberry-pi-controller/lib/command_objects/send_message.rb:3:in <top (required)>'
from /home/pi/farmbot-raspberry-pi-controller/lib/command_objects/commands.rb:17:in require_relative'         from /home/pi/farmbot-raspberry-pi-controller/lib/command_objects/commands.rb:17:in <top (required)>'
from lib/controllers/exec_sequence_controller.rb:2:in require_relative'         from lib/controllers/exec_sequence_controller.rb:2:in <top (required)>'
from /home/pi/farmbot-raspberry-pi-controller/lib/command_objects/resolve_controller.rb:1:in load'         from /home/pi/farmbot-raspberry-pi-controller/lib/command_objects/resolve_controller.rb:1:in block in <top (required)>'
from /home/pi/farmbot-raspberry-pi-controller/lib/command_objects/resolve_controller.rb:1:in each'         from /home/pi/farmbot-raspberry-pi-controller/lib/command_objects/resolve_controller.rb:1:in <top (required)>'
from /home/pi/farmbot-raspberry-pi-controller/lib/messaging/message_handler.rb:6:in require_relative'         from /home/pi/farmbot-raspberry-pi-controller/lib/messaging/message_handler.rb:6:in <top (required)>'
from /home/pi/farmbot-raspberry-pi-controller/lib/farmbot-pi.rb:3:in require_relative'         from /home/pi/farmbot-raspberry-pi-controller/lib/farmbot-pi.rb:3:in <top (required)>'
from farmbot.rb:1:in require_relative'         from farmbot.rb:1:in '
pi@farmbot:~/farmbot-raspberry-pi-controller$

Any one understand why farmbot.rb cannot run?

The Problem
You caught me, I'm guilty. This is a real bug. I use Pry for live debugging in dev mode (on an x86). Unfortunately, Pry does not install on all ARM based CPUs, so I don't put it in the gemfile. I probably forgot to take out a require "pry" statement somewhere.
Quick Fix

gem install pry
Start over with install instructions

Real Fix

Remove all references to pry in code base and be more careful in the future, Rick.

I will do that here in a moment. Thanks for pointing it out. As a side note, if you're trying to run this code on a real bot, consider using one of the stable releases under the "releases" tab. We don't guarantee that latest master is stable.
Thanks for the heads up! Feel free to drop an issue on anything else you find.

Thanks for the reply!  I was able to find the reference to pry (lib/telemetry_message.rb), and after commenting out the offending line, the software appears to run.  Actually, it fails to connect to the Arduino MEGA, but that's ok since I don't have one connected. ;-)
FYI...  I followed instructions here: https://farmbot-software.readme.io/docs/raspberry-pi-software and in the repo README.md file.  It doesn't suggest checking out a release tag, or otherwise changing to any other branch.
Thanks again,
David


Support miner fee less than 1 sat/b

Hi Aust, per our conversation in Slack, raise this issue as a reminder.

Thanks.

Thanks. Let's see what we can do for one of the next releases 😉


Private Constructor

Would it make sense to make the constructor for CardType private?
That way the users of our enumeration class can't instantiate their own card types, and it becomes closer to an enum.

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

ID: d118f070-8743-e905-2d08-dc725a4e8b5f
Version Independent ID: fae35cbd-75ae-4448-f066-7c91ccc35779
Content: Using Enumeration classes instead of enum types
Content Source: docs/architecture/microservices/microservice-ddd-cqrs-patterns/enumeration-classes-over-enum-types.md
Product: dotnet
Technology: dotnet-ebooks
GitHub Login: @nishanil
Microsoft Alias: nanil

Hi @timyhac, thanks for your feedback.
I agree with you in the general case, when you use an enumeration class to replace an enum, that is something that has a finite set of values.
I just think there are better examples for an enumeration class, say the OrderStatus.
In this specific case, the static values are just a way to have an initial default set of values to seed a database.
So, in this specific case, it must be public, because it can be used to seed the DB from a text file, and the constructor needs to be public.
Hope this helps.

Closing this issue now but feel free to comment, will reopen if needed.


SyntaxError: Named export 'LUTCubeLoader' not found. The requested module 'postprocessing' is a CommonJS module

Description of the bug
I use postprocessing like:
import { LUTCubeLoader } from "postprocessing";

when i run my nextjs app, it show me an error with this topic:
`
import pkg from 'postprocessing';
const { EffectComposer: EffectComposer$1, RenderPass, NormalPass, DepthDownsamplingPass, EffectPass, DepthOfFieldEffect, MaskFunction, DepthPickingPass, CopyPass, BlendFunction, BloomEffect, BrightnessContrastEffect, ChromaticAberrationEffect, ColorAverageEffect, ColorDepthEffect, DepthEffect, DotScreenEffect, GlitchEffect, GlitchMode, GodRaysEffect, GridEffect, HueSaturationEffect, NoiseEffect, OutlineEffect, PixelationEffect,
ScanlineEffect, SelectiveBloomEffect, SepiaEffect, SSAOEffect, SMAAEffect, TextureEffect, ToneMappingEffect, VignetteEffect, ShockWaveEffect, LUT3DEffect, TiltShiftEffect: TiltShiftEffect$1, Effect, EffectAttribute } = pkg;
at ModuleJob._instantiate (node:internal/modules/esm/module_job:124:21)
at async ModuleJob.run (node:internal/modules/esm/module_job:190:5) {

digest: undefined
}
`
Library versions used

Three: [^0.145.0]
@react-three/postprocessing: [^2.11.4]

Desktop

OS: [Windows]
Browser [Chrome 113.0.5672.93]
Graphics hardware: [AMD Firepro]

Same here using:
import { EffectComposer, Glitch } from '@react-three/postprocessing'
While the server is running and the model is loaded:
Adding the following code works as intended.
<EffectComposer >
  <Glitch />
</EffectComposer>

But on full page refresh i receive the same following error as @mhmdnzr :
import pkg from 'postprocessing';
const { EffectComposer: EffectComposer$1, RenderPass, NormalPass, DepthDownsamplingPass, EffectPass, DepthOfFieldEffect, MaskFunction, DepthPickingPass, CopyPass, BlendFunction, BloomEffect, BrightnessContrastEffect, ChromaticAberrationEffect, ColorAverageEffect, ColorDepthEffect, DepthEffect, DotScreenEffect, GlitchEffect, GlitchMode, GodRaysEffect, GridEffect, HueSaturationEffect, NoiseEffect, OutlineEffect, PixelationEffect,
ScanlineEffect, SelectiveBloomEffect, SepiaEffect, SSAOEffect, SMAAEffect, TextureEffect, ToneMappingEffect, VignetteEffect, ShockWaveEffect, LUT3DEffect, TiltShiftEffect: TiltShiftEffect$1, Effect, EffectAttribute } = pkg;
Library versions used:
"@react-three/drei": "^9.68.3",
"@react-three/fiber": "^8.12.2",
"@react-three/postprocessing": "^2.11.4",


Potential secutiry vulnerabilities in the shared libraries which jctp depends on.

Hi, @RationalityFrontline , I'd like to report a vulnerability issue in org.rationalityfrontline:jctp:6.3.19-1.0.0.
Issue Description
org.rationalityfrontline:jctp:6.3.19-1.0.0 depends on 3 C libraries (.so). However, I noticed that one C library is vulnerable, containing the following CVEs:
libthosttraderapi_se.so from C project openssl(version:1.1.0d) exposed 5 vulnerabilities:
CVE-2019-1543, CVE-2018-0735, CVE-2017-3738, CVE-2017-3733, CVE-2019-1552
Suggested Vulnerability Patch Versions
openssl( has fixed the vulnerabilities in versions >=1.1.1l
Java build tools cannot report vulnerable C libraries, which may induce potential security issues to many downstream Java projects.
Could you please upgrade the above shared libraries to their patch versions?
Thanks for your help~
Best regards,
Helen Parr

I don't have source code of libthosttraderapi_se,  the library belongs to project CTP, which is a closed source project developed by Shanghai Futures Information Technology Co.,Ltd. I can only download the library binaries from their website.
So I don't have any idea of how to fix the issue. Do you have any suggestions?


feat: introduce organic-keyword audit trigger (SITES-19317)

Related PRs:

https://github.com/adobe/spacecat-audit-worker/pull/112
https://github.com/adobe/spacecat-audit-post-processor/pull/65

no activity, closing


IE11 cannot download OVPN file

It seems IE11 cannot download the OVPN file. Not sure what is going on here...maybe it tries to download the file twice?

This was interesting. When an attachment is offered with a 201 status code, it doesn't work in IE. It needs to be 200 (or possibly another status code as well).


Native integer array implementation

Purpose
With this PR, we can now define a integer array and use it in the native backend.
fixes #20560
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

You cannot now get the executable directly from running the ballerina command. You have to link the object file seperately.

Check List

[ ] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #20561 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #20561   +/-   ##
=======================================
  Coverage   15.21%   15.21%
=======================================
  Files          50       50
  Lines        1341     1341
  Branches      208      208
=======================================
  Hits          204      204
  Misses       1121     1121
  Partials       16       16

Continue to review full report at Codecov.

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


Add Licensing to the Repository

The repository could use a proper license. This may be dependent on what type of license is used for JavaBot, since it is used here. I already opened an issue at JavaBot: https://github.com/Vincentyification/JavaBot/issues/30

I have been informed, that JavaBot will be licensed under the Apache Software License. Until further evaluation of the exact terms of use (and I understand it well enough) development is on hiatus

I completed the other ticket, so you can do this now.


[not_landed] Fix a typo

Coverage remained the same at 24.995% when pulling 8c07ebe7df1a86df05bc27b191cfcfb5feef2d78 on calixteman:typo_not_landed into 81a30e69ea7e47e7de8b8af33a2eca822fef4d35 on mozilla:master.


Updates etcd-druid crds and dependency

How to categorize this PR?
/kind enhancement
What this PR does / why we need it:

Updates etcd-druid CRDs to v0.18.1 (although I found no change)
Updates the github.com/gardener/etcd-druid dependency to v0.18.1

Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

None

/test pull-gardener-e2e-kind-ha-single-zone

/test pull-gardener-e2e-kind


Next iteration of ronin

Hi @vdemedes! How have you been?
I've been using ronin for a while and I'm really enthusiast of the features it offers, recommending it to anyone that wants to do a CLI in Node.js. However, the project has been stale for a while and the lack of new features, specially when it comes to be more descriptive about documentation of the commands, start making ronin missing some of our goals.
I'm aware that you are super busy, but if you could kick off the next iteration and lay out the vision, me, @richardlitt and other folks would be happy to help you get there :)

Hey guys, let me get back to you tomorrow ;)

Hey @diasdavid and @richardlitt, I know it's been long overdue and I'm sorry for this, it's been a lot of stuff on my plate lately.
So, the next iteration of Ronin. Currently, it's a pile of unflexible code, but with a nice list of features and ideas. I was thinking how to make the underlying code easy and straightforward, as well as simple to customize and introduce new features.
I decided to go Express/Connect route. What this means is, now we have 2 separate projects:

Sushi (like Connect) - a middleware layer for CLI apps.
Ronin (like Express) - a project that uses Sushi underneath and bundles a lot of helpful middleware.

Ronin will keep a list of its existing features, like auto-registration of commands. However, it'll remain clean, because it will basically be a bundle of individual middleware for Sushi.
As for Sushi, it's simply a middleware layer, without any fancy functionality. Think of it like Connect for CLI apps, instead of HTTP servers. Today I pushed a rewritten version of Sushi, as well as example help middleware:

sushi - https://github.com/vdemedes/sushi
sushi-help - https://github.com/vdemedes/sushi-help

Would love to hear your feedback, ideas and suggestions! What use cases did Ronin not fulfill in your projects? What was unneeded? What features you liked (so that I know what to keep in the next major iteration of Ronin)?
Thank you a lot of understanding and your patience!


All BioGRID links go to the same place

I've just noticed a problem while looking at implementing #970: the links to BioGRID from all the gene page external reference sections go to the same place, the cdc2 page.
Here's an example, lsm3/SPBC9B6.05c:
https://thebiogrid.org/276415/summary/schizosaccharomyces-pombe/SPBC9B6.05c.html
It seems that the "schizosaccharomyces-pombe/SPBC9B6.05c" in the URL is ignored only the "276415" makes a difference to the page we get.
I'll contact them to ask how we should do the linking:
https://wiki.thebiogrid.org/doku.php/contribute#link_to_us_link_to_our_result_pages

I've removed BioGRID from the external references section until this is fixed.

It seems that the "schizosaccharomyces-pombe/SPBC9B6.05c" in the URL is ignored only the
"276415" makes a difference to the page we get.

Turns out that the "276415" is the BioGRID interactor ID (gene ID) and is provided in the files we download.  So I've changed the Chado loader to store the IDs.  And I've changed the web code to use that ID in the links to BioGRID.  It should be working again on Tuesday morning after the next load.

Fixed now.


"clear" and "CTRL + L" not working on some browsers

I'm having a problem with the clear command. I get this error:
danny@TabletPC:~$ clear
'Browser Term': unknown terminal type.

And CTRL + L works as a normal Enter key.
I'm using Chromium 37.0.2062.120 on elementaryOS Luna.

@csanonymus Check if this was fixed in the latest release (npm i -g web-term@latest).

I still have the same issue
![http://s7.postimg.org/hd5jf4v0b/Screenshot_from_2015_03_27_15_25_36.png]

@csanonymus  We reproduced this issue in an internal @jillix application. The I applied is to export the XTERM environment variable.
Run the following commands:
echo "export TERM=xterm-256color" >> ~/.bashrc
source ~/.bashrc

Then the web-term experience will be smoother.
I will make sure I add this info in the web-term docs in the next release.

Actually, just run export TERM=xterm-256color in the terminal session. I will try to force setting this environment variable in web-term.
However, I opened this issue: https://github.com/chjj/pty.js/issues/120 I'd :+1: for fixing it in the core library.

This happened because of the custom name. I addressed this :bug: in https://github.com/IonicaBizau/web-term/commit/384c2a7f8cca2e8187a1bae55577c65950a92b94.
@csanonymus Would be glad if you test the improvements branch and give feedback! :sparkle:

Yes, the branch solves the bug. Great job!


Support new Pulsar options added to plumber-schemas

Follow-up to https://github.com/batchcorp/plumber-schemas/pull/87
Related to https://github.com/batchcorp/plumber/issues/335. In addition this PR adds a few other options I'd like to be able to set.
To my knowledge, Pulsar doesn't support multiple forms of authentication, so I added a validation to raise an error in the event TLS auth and token auth are both set. I figured it's easier to remove that validation than add it in the future
cc @blinktag

Awesome thank you @ckreiling
We'll get this released shortly

My pleasure! Thanks for the speedy replies. Let me know if I need to / how to fix the coverage upload step that failed


merge package template from Kronos-Tools/npm-package-template

package.json

chore(devDependencies): add<EMAIL_ADDRESS>from template
chore(devDependencies): add<EMAIL_ADDRESS>from template
chore(engines): update<EMAIL_ADDRESS>from template
docs(package): remove keyword null

.travis.yml

chore(travis): merge from template .travis.yml

Codecov Report

Merging #77 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master     #77   +/-   ##
======================================
  Coverage    92.3%   92.3%
======================================
  Files           1       1
  Lines          26      26
======================================
  Hits           24      24
  Misses          2       2

Continue to review full report at Codecov.

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


Change "enum element" to "enum case" in diagnostics

From the perspective of the compiler implementation, they're elements. But users will think of these as cases—and many diagnostics already refer to these as enum cases.

Also, the documentation clearly refers to these as cases and never uses the term elements:

Multiple cases can appear on a single line, separated by commas:
enum Planet {
    case mercury, venus, earth, mars, jupiter, saturn, uranus, neptune
}

Goodbye C++isms, hello Swift
⛵
@swift-ci please smoke test and merge

Somehow I missed a test the first time around. 🙈 Updated.

@swift-ci please smoke test and merge

@swift-ci please smoke test

@swift-ci please smoke test


XS✔ ◾ Remove form at the top of the page

Tested locally

@karl-johan-grahn Image is available for testing. docker pull stakater/workshop-exercise:SNAPSHOT-PR-75-0cef8cc5


Using this package, I have created a trait to support auto-generation for primary keys for Laravel 10 (string form only)

For anyone who wishes to make it better or use it as is.
This uses the string form of the TSID.
https://gist.github.com/dereckmartin/06eaf720c50bb5ba1c81d83106cb7bc8

Thanks for sharing.

For testing purposes, you could implement the Odan\Tsid\TsidFactoryInterface that generates a fixed set of predefined TSID's.

I would not recommend using a string as primary key in your database, because this is slow and you need too much hacking around. Just keep the defaults as int for the primary key. Then add a second field for your TSID's as a unique key.


ViewService#delegationBalancesByAddressIndex method that returns validators as ValueViews

NOTE: Depends on #601
Right now, the StakingService has a method, validatorInfo, that returns a series of ValidatorInfos.
As of #604, we're refactoring the staking page to treat validators as tokens that an address may have a balance of. To achieve that, we're fetching validator infos, then generating ValueViews from those validators that include the address's balance of those tokens.
This logic should be moved into a new RPC method on the ViewService called delegationBalancesByAddressIndex. It should:

accept a DelegationBalancesByAddressIndexRequest with an index field, indicating which address index to return delegation balances for.
return (probably via stream) ValueViews with the address's balances of each validator's delegation tokens, generating zero-balance ValueViews on the fly for validators that the user hasn't staked in.
include the ValidationInfo for the relevant validator in each ValueView's extendedMetadata.

I think this should be in the view service, if we're expecting it to return info about the user's delegation tokens -- which the chain doesn't have.
How about rpc DelegationBalances with a request parameter that has an all_active_validators field?

Thanks @hdevalence — I'll add this to the issue description. all_active_validators would determine whether to filter for just active ones, you mean?

cc @grod220 — this is the second part of moving some staking work into Rust/the extension

Thanks @hdevalence — I'll add this to the issue description. all_active_validators would determine whether to filter for just active ones, you mean?

I was thinking it would be telling the view service whether to include zero balances at all, the confusion is maybe a sign my name is bad.

default behavior: like BalancesRequest but only returns delegation tokens the user has
with flag set to true: includes one entry for each validator in active set, regardless of balance

@hdevalence Gotcha. But there wouldn't be a scenario where we'd only want the validators that the user has staked though, right? Wouldn't we always want to show all validators?
Although, maybe you're thinking of frontends other than ours who might want to arrange that view differently. Anyway, it can't hurt to include that functionality.

Yeah, exactly. My thought was that from the extension's point of view, the implementation involves querying the user's delegation tokens and then merging that with the validator list. So there's not much extra involved in having a flag to skip the merging, and maybe that's useful for a different use case.


Change-Id: Ib49d1192f51abe4329e6dc9da3c51a4a36290082
Signed-off-by: Michael Scott<EMAIL_ADDRESS>

Checkpatch Suceeded.

LGTM, but not yet tested. Probably easy to upstream this one.

Sanity Check Succeeded.

Woops rebasing.

Checkpatch Suceeded.

Sanity Check Succeeded.

Checkpatch Suceeded.

Sanity Check Succeeded.


Is there a way to have the Pivot point parameters to be absolute and not relative to the container?

Sorry, I know it's simply not made to work this way, but I would need to be able to rotate related to a fixed pivot point and not have it moved everytime I translate the object first.
Do you think of a way around that issue than I'm not thinking of?
Thank you!

if you trying to send something like this {"payload":{"x":0,"y":0,"deg":17},"topic":"T-101_level@cx_move"} and it doesn't rotate then the problem with the SVG file.
However, I suppose you are sending a message like above and it jumps somewhere else, is it correct?

It works as intended, that is if, I do a rotation, it will rotate around the pivot point defined relatively to its container.
My first message was unclear and actually my "problem" even bigger, because the way the pivot point is defined "locally" it means it can't be a point "outside" of its container somewhere else in a bigger canvas, is that right?
Let say my canvas is 400x400 , I try to rotate a rectangle that is 50x50 and centered on this canvas, but my pivot is top left corner of the canvas: I can't rotate this way, because the pivot point can only be in the 50x50 area of the rectangle?
Sorry I'm very new to SVG AND Nodered, I might be using the wrong words and confusing you even more =)

Not confusing it's me. The first time I read your comment without reading the question. Sorry for making you to explain it again.
Answering your question: Currently it doesn't support absolute coordinates for the pivot parameter. However it is a great feature for v2 of the framework.
As a workaround you can try creating a canvas size transparent rectangle with your 50x50 rectangle inside. Then rotate the parent rectangle.
Hope it helps.

@alex-controlx thank you for taking the time to answer :)


Upgrade to Ruby 3 / Rails 7

When Rails 7 comes out we should upgrade oc-id to Rails 7 and oc-id/chef-server-ctl to Ruby 3.
Acceptance criteria:

chef-server-ctl is on Ruby 3.0

Closing as a dup: https://chefio.atlassian.net/browse/INFS-56


Pin label not shown on iOS 13.1.2 (on real devices)

VERSIONS

Xamarin.Forms.GoogleMaps - 3.2.1
Xamarin.Forms - <IP_ADDRESS>8062

PLATFORMS

[x] iOS | >=13.1.2

ACTUAL BEHAVIOR
I've created a pin with Label and Address, but only the address is visible (see screenshot below)
ACTUAL SCREENSHOTS/STACKTRACE

EXPECTED BEHAVIOR
I actually set a label to that pin.. it works on a emulator thats on ios 13.1:

HOW TO REPRODUCE

On a real device, create a map with a pin that contains a label and address
Click on the pin to open the info window. it now only shows the address

the issue comes from the new theme "dark mode" in ios 13. If you switch it back to the light mode.
You'll see your label again.
Something is not compatible with the new dark mode but I wonder if it may comes from the pin in Xamarin.Forms.Maps rather than Xamarin.Forms.GoogleMaps ?

I think this is related with Google Maps SDK for iOS or Xamarin.Google.iOS.Maps.
We should wait for fix in these.

I found fixed libraries.

https://developers.google.com/maps/documentation/ios-sdk/releases#version_350_-_october_08_2019
https://www.nuget.org/packages/Xamarin.Google.iOS.Maps/3.5.0

I will depends Xamarin.Google.iOS.Maps 3.5.0.

Oh that would be awesome thanks!

Fixed in https://www.nuget.org/packages/Xamarin.Forms.GoogleMaps/3.3.0 .

Great! Aldo i thought that it all worked but i get a linker exception:

warning : directory not found for option '-FGMps-3.5.0/Maps/Frameworks'
error : linker command failed with exit code 1 (use -v to see invocation)
error : warning: directory not found for option '-FGMps-3.5.0/Maps/Frameworks'

Do you update Deployment target to 8.0 in your iOS App's info.plist ?
Or ...
Clear and rebuild your solution:

Delete /packages directory
Delete /.vs directory
Delete /bin, /obj directory each projects

i had the deployment target on 11.0.. let me check if the clean/rebuild works

I still get this linker exceptions as soon as i update Xamarin.Google.iOS.Maps to 3.5.0.

Could you try XFGoogleMapSample and check difference between it and your proj.

Also the Sample project gives the same builderror for me. (#665 as wel)

Great! Aldo i thought that it all worked but i get a linker exception:
warning : directory not found for option '-FGMps-3.5.0/Maps/Frameworks'
error : linker command failed with exit code 1 (use -v to see invocation)
error : warning: directory not found for option '-FGMps-3.5.0/Maps/Frameworks'

Framework not found GoogleMaps

I have the same problem
MTOUCH: error MT5202: Native linking failed. Please review the build log.
clang: error : linker command failed with exit code 1 (use -v to see invocation)
error MT5209:  Native linkingerror : framework not found GoogleMaps
error MT5209:  Native linkingerror : warning: directory not found for option '-FGMps-3.5.0/Base/Frameworks'
error MT5209:  Native linkingerror : warning: directory not found for option '-FGMps-3.5.0/Maps/Frameworks'

Could you try XFGoogleMapSample and check difference between it and your proj.

strange, the sample works for me. :(, but my own project does not build.

Could you try XFGoogleMapSample and check difference between it and your proj.

strange, the sample works for me. :(, but my own project does not build.

The same here. I am using Xamarin.Forms 4.3, could this be a problem?

Could be related to #666

Here's how I solved it.
With your project still using nuget packages

Xamarin.Forms.GoogleMaps v3.3.0
Xamarin.Google.IOS.Maps v3.5.0

Remove the following directory and its contents.
/Users/rdonasco/.nuget/packages/xamarin.google.ios.maps/3.5.0
Restore your nuget packages, this will recreate the v3.5.0 directory and its contents.
Downgrade your project to use nuget packages

Xamarin.Forms.GoogleMaps v3.2.1
Xamarin.Google.IOS.Maps v3.1.0

Build your project - this should go back to successful build.
Upgrade your project to use

Xamarin.Forms.GoogleMaps v3.3.0
Xamarin.Google.IOS.Maps v3.5.0

Rebuild your project.


Establishing a new relationship should not be possible before decomposition of the old one

Readiness checklist

[x] I added/updated unit tests.
[x] I added/updated integration tests.
[x] I ensured that the PR title is good enough for the changelog.
[x] I labeled the PR.

@NikolaDmitrasinovic Now the pull request is ready for review.

When attempting to create a new relationship between two identities which have already established a relationship, the following error message is returned:
{
	"error": {
		"id": "ERR4NGLGGXFH659UACK6",
		"code": "error.platform.validation.relationshipRequest.relationshipToTargetAlreadyExists",
		"message": "A relationship to 'did:e:localhost:dids:0f3e40164b6c495c28674f' already exists. If the relationship is **terminated**, you can reactivate it.",
		"docs": "",
		"time": "2024-07-11T07:50:39.657075Z",
		"data": null
	}
}

(emphasis added)
If one of the identities terminates the relationship, the message is the same. I'm not sure this is correct.

@Dannyps Sure, that's exactly what we expect. As long as no participant has decomposed the relationship, it can be reactivated.
You should check the task description to see a table with test cases. Make sure that all of them work as expected.

Manual Review
Upon testing, I have determined:

✅ The creation of a second relationship between two identities cannot be done while there is already one present, unless its status is ReadyForDeletion, which is reached when both identities call the Decompose endpoint.
The creation of a second relationship between two identities cannot be done while there is already one present in one of the following statuses:

Pending
Active
Terminated
DeletionProposed

The creation of a second relationship between two identities cannot be done while there is already one present in one of the following statuses:

[...]
DeletionProposed

But this SHOULD actually be possible. It was the main goal of this task. @HunorTotBagi Did you not implement it?

After reviewing the acceptance criteria, I have updated the review.

If the status is DeletionProposed, and the decomposer was P2, P1 cannot create a new relationship (this was what I tested initially).

@HunorTotBagi This seems to be a bug. Can you please check that?

@HunorTotBagi This seems to be a bug. Can you please check that?

Nvm. I misread Daniels report. :)

Please consider the following scenario:

P1 creates relationship template rlt1
P2 creates relationship (using rlt1): Pending
P1 accepts: Active
P1 terminates: Terminated
P1 decomposes: DeletionProposed
--
P2 creates relationship template rlt2
P1 creates relationship (using rlt2): Pending
P2 rejects: Rejected
--
❌ P2 tries to create relationship (using rlt1):  error: You haven't decomposed the old relationship.
✔ P1 creates relationship (using rlt2): Pending

Step 8 should have failed. Despite there being a Pending new relationship, the old one must be decomposed before proceeding:

P1 has decomposed old Relationship
P2 has decomposed old Relationship
Active Identity
POST Relationships
PUT (...)[Accept/Reject]
Fits the scenario

true
true
P2
true ⚠
true ⚠
up to step 5

true
true
P1
true ⚠
true ⚠
up to step 5

true
false
P2
false ✅
false ❌
up to step 5, but bad step 8 ❌

true
false
P1
true ✅
true ⚠
up to step 10

false
true
P2
true ⚠
true ⚠
-

false
true
P1
false ⚠
false ⚠
-

false
false
P2
false ⚠
false ⚠
-

false
false
P1
false ⚠
false ⚠
-

⚠ - not tested in this scenario

I can confirm that in this test case scenario that @Dannyps wrote, the Reject function isn't working as expected. I looked into it and found that it simply doesn't find all relationships for the active identity. I'm working on it.

The runtime team decided they want this feature ater all.


Export page points to OSM instead of OHM

Bug description
Every link on https://openhistoricalmap.org/export points to OSM things and not to OHM counterparts.

[ ] Overpass links to https://overpass-api.de/api/ but should link to https://overpass-api.openhistoricalmap.org/
[ ] Planet links to https://planet.openstreetmap.org/ but should link to https://planet.openhistoricalmap.org/
[ ] Geofabrik Downloads should go away since we don't have those
[ ] Other Sources should go away too since AFAIK there are none.

We should fix this in OSM upstream that we're working on, so it rolls out with the new code for that.

[ ]  fix the license statement.

These are all fixed on the Merge OSM branch:


English suggestions

Check if this PR fulfills these requirements:

[X] My changes respect the Kodi add-on development rules
[X] I have read the CONTRIBUTING document.
[X] I made sure there wasn't another Pull Request opened for the same update/change
[X] I have successfully tested my changes locally

Types of changes

[ ] New feature (non-breaking change which adds functionality)
[ ] Feature change (non-breaking change which change behaviour of an existing functionality)
[X] Improvement (non-breaking change which improve functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

While going over the Dutch translation, I've noticed some odd English strings.

Use 'My list' consistently.
Don't use shared-library
Used …, » and « UTF-8 characters.

I'm actually thinking about the lowercase issue, English isn't my native language, but from what I can find online, we don't need to capitalise so much words (like you described here: https://github.com/CastagnaIT/plugin.video.netflix/pull/830#issuecomment-686683046). I could use some input from native English speakers here.
Feel free to comment on what you like and what you don't.

I'm aware that this will require changes to all other language files, since Kodi doesn't match by ID unfortunately.

I don't know English well either...
until now to me this look well
but before i will merge the romanian pr

I would replace ampersand with "and".

@dagwieers You mean strings "(between 0 & 10)" and "Show trailers & more"? I agree, "and would look better...

I disagree with the UTF-8 chars. I prefer the current form. The other changes are fine for me.

I prefer the UTF-8 characters, but do not think it is worth discussing :-) Rather have @CastagnaIT decide.

PS This PR should also includes the updates for the existing languages. I understand we would not do that straight away until we have settled what we want to change, but we shouldn't merge until the english strings have been updated everywhere.

I don't really mind about the UTF-8 characters, except that they look a bit nicer IMO. But I would also say it's up to @CastagnaIT.

@dagwieers I was hoping there would be a command line tool to update existing language files by the id or something.

I disagree with the UTF-8 chars.

mmh yes becouse look better and no for other reasons
I would also prefer normal characters, because most of the time you do not use non-standard characters in sentences
for example Asian languages can use different punctuation and this form may not be respected
for the dots "…" perhaps is better use the normal dot char "..."
for the »« in my opinion could be maintained as UTF8, is only an "arrow"

PS This PR should also includes the updates for the existing languages.
@dagwieers yes but it is better before define what changes to do

Is there something we should do about the Capitalisation of nouns? :)

Okay, I've updated the PR with the requested changes.
Regarding the capitalisation, it seems that there are different rules for titles as for normal text.
-> https://www.grammarly.com/blog/capitalization-rules (see point 6).

I leave it to you to speak, with pure English i would only make things worse
i help myself with the translator :D
a little thing perhaps:
tv shows > TV show

next days i will publish a minor release
you can end here and complete the changes in relative translations,
or if you want you can wait other days if there are other changes to do

I have made a small script that use polib to update the other language files when translations are added/modified in the english file. It's similar to msgmerge, but it does the matching on msgctxt. The only drawback is that it puts strings with newlines on a new line, but this is also done for the header, so maybe this doesn't matter.
@CastagnaIT how do you do this when you add a translation? Do you just change all the files and add it there manually, or do you also have a script to do this?

I have made a small script that use polib to update the other language files when translations are added/modified in the english file. It's similar to msgmerge, but it does the matching on msgctxt. The only drawback is that it puts strings with newlines on a new line, but this is also done for the header, so maybe this doesn't matter.
@CastagnaIT how do you do this when you add a translation? Do you just change all the files and add it there manually, or do you also have a script to do this?

usually i use a software to compare the files and merge the change manually
i do not like these newlines make the files difficult to compare manually and in some case also to read
but i think can be resolved easly
so i was waiting for your modifications to make another one
but at this point you can make this change for me and then solve everything
my change is replace all return chars escape with Kodi labeling:
\r\n with [CR]
\n with [CR] (some id have only \n)
in this way the script should not create new lines
this change is necessary because some languages "right to left", can have strange behaviors with the return char escapes

@CastagnaIT I've replaced all newlines with [CR] (except in the header). I've also replaced the << and >> characters in the other languages.

nicework! now it is very clear


SSL it's not fully implemented

Hi,
I was trying to use your library with AWS Clustered mode and figured out it does not work with SSL.
I figured out the problem and fixed it in my fork  but it's a hack what I did.
Basically everywhere you create an address like "redis://" -- you need a way to know if it's SSL and put "rediss://" instead. Since those places have no access to config, unfortunately I changed all to force SSL everywhere.
Do you have in plan a fix for this, or I can do it if you point me to the right way. I am trying to understand your logic and how can I be least invasive to pass a config. The simplest way would be to put a static public flag somewhere -- but that's also a hack I guess, and will only allow one type at one time, or pass a jvm property ...what do you suggest?
Thanks in advance,
Florin Moisa

Hi,
Thanks for report. Actually there is only one place where scheme should be replaced to rediss - org.redisson.client.protocol.decoder.ClusterNodesDecoder. I have fixed it.

Hi,
That was fast 👍. I also figured by debugging initially that was the only place, but it's not the only place.
I would also suggest to take a look here:
     public void changeMaster(final URI address) {
                 ....
                slaveDown(URIBuilder.create("redis://" + oldMaster.getClient().getIpAddr()), FreezeReason.MANAGER);
              ......
      }

and here:
    public RedisClientConfig setAddress(String host, int port) {
        this.address = URIBuilder.create("redis://" + host + ":" + port);
        return this;
    }

and here:
       public String getRedisServerAddressAndPort() {
            return "rediss://" + getRedisServerBindAddress() + ":" + getRedisServerPort();
        }

and here:
    public static String getDefaultRedisServerBindAddressAndPort() {
        return "redis://" + defaultRedisInstance.getRedisServerBindAddress()
                + ":"
                + defaultRedisInstance.getRedisServerPort();
    }

and here:
    private String createAddress(String host, Object port) {
        if (host.contains(":")) {
            host = "[" + host + "]";
        }
        return "redis://" + host + ":" + port;
    }

Hope it helps,
Florin Moisa

Have you tried my changes? Does it work now?

yes it does 👍, I made exactly the same change, but I figured the other places also need  update since it will fail if master changes and so on.


Implement the import for frame diff (Library 2)

User story

As a user
I want / need to load two video files from the file system
So that the video loaded into the program can be used in the Kotlin program for further processing.

Acceptance criteria

The system must provide an graphical user interface or API for the user to select two video files from the file system.
The system should only support and recognize video files that have a lossless compression
The video files should be loaded into the program, otherwise a helpful error handling should be triggered
For the next processing steps, the the video files must be available/ accessible in the Kotlin program.

Definition of done (DoD)

All specifiec acceptance criterias are met
The feature branch is merged into the main branch and then closed
The code is clearly documented

DoD general criteria

Feature has been fully implemented
Feature has been merged into the mainline
All acceptance criteria were met
Product owner approved features
All tests are passing
Developers agreed to release

wrong project :D


Make this module obsolete

I would really, really like to make this module (hack) obsolete.
This requires submitting a good patch to upstream Go. see https://github.com/golang/go/issues/9661

want.

2.5 years later, it seems a go CL slated for go 1.11, which might go some ways toward avoiding this: https://gist.github.com/vyzo/62c3fedcc61b958fa401d1340537fd68#file-gistfile1-txt.
According to the release cycle go 1.11 should hit in late August, might be nice to time an ipfs / libp2p update shortly after.

@b5 My hopes were really hight for G1.10 but true, we we will have to wait for Go 1.11.

The necessary upstream changes have been made, I'll investigate simplifying or removing this package entirely.


fix(docker): support non amd64 dockerize in setup containers

Checklist

[x] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

This fixes an issue where the amd64 binary version of dockerize was being used to start the mysql-setup and elasticsearch-setup containers, regardless of the architecture of image being being, causing datahub docker quickstart to fail on arm64.  Instead, this fix builds dockerize in a staged build, and uses the resultant binary in the final image.  Tried first by downloading the gzipped Source asset from the dockerize repo, but this did not include all needed files for a build, hence going for a git clone solution.

Hello @tonycsoka
First of all, thank you for making a contribution to DataHub! We appreciate folks who actively try to improve the project.
Second, for this PR to be accepted could you please update the PR title and description to match our guidelines? See here: https://github.com/datahub-project/datahub/blob/master/docs/CONTRIBUTING.md#commit-message-format
In order to ensure your PR gets merged we would highly recommend adding a description to your PR that describes the problem you are trying to solve, how did you did it and any considerations reviewers and/or users of this feature may need to have.

Hello @tonycsoka
First of all, thank you for making a contribution to DataHub! We appreciate folks who actively try to improve the project.
Second, for this PR to be accepted could you please update the PR title and description to match our guidelines? See here: https://github.com/datahub-project/datahub/blob/master/docs/CONTRIBUTING.md#commit-message-format
In order to ensure your PR gets merged we would highly recommend adding a description to your PR that describes the problem you are trying to solve, how did you did it and any considerations reviewers and/or users of this feature may need to have.

Thanks @pedro93, how's that for you.

@tonycsoka did you try Peter's suggestion with go install github.com/jwilder/dockerize@latest rather than git cloning a repo?

@tonycsoka did you try Peter's suggestion with go install github.com/jwilder/dockerize@latest rather than git cloning a repo?

Not yet, there's a little issue in that v0.6.1 of dockerize used this system called glock for dependency management which I'm looking at

LGTM! Thank you for taking care of the review concerns. Once CI is green I think we can merge this.

Great and have a fine weekend!

LGTM! Thank you for taking care of the review concerns. Once CI is green I think we can merge this.

@pedro93 Are those smoke tests a little delicate, can't see why this change would cause an issue and we have 2 different failures on separate CI runs?

Yes our tests are a little flaky sometimes when big changes to the UI happen. CI was failing due to another issue that got fixed since. Merged your PR. Thank you for the contribution!