Distance-to-object Jupyter Notebook Error

Required Info

Camera Model
{ R200 / F200 / SR300 / ZR300 / D400 }

Firmware Version
(Open RealSense Viewer --> Click info)

Operating System & Version
{Win (8.1/10) / Linux (Ubuntu 14/16/17) / MacOS

Kernel Version (Linux Only)
(e.g. 4.14.13)

Platform
PC/Raspberry Pi/ NVIDIA Jetson / etc..

SDK Version
{ legacy / 2.. }

Language
{C/C#/labview/nodejs/opencv/pcl/python/unity }

Segment
{Robot/Smartphone/VR/AR/others }

Issue Description
Just trying to follow your Jupyter notebook sample...??
Push run on first block... import error..??
ImportError                               Traceback (most recent call last)
 in
----> 1 import cv2                                # state of the art computer vision algorithms library
2 import numpy as np                        # fundamental package for scientific computing
3 import matplotlib.pyplot as plt           # 2D plotting library producing publication quality figures
4 import pyrealsense2 as rs                 # Intel RealSense cross-platform open-source API
5 print("Environment Ready")
/srv/conda/envs/notebook/lib/python3.6/site-packages/cv2/init.py in
3 import sys
4
----> 5 from .cv2 import *
6 from .data import *
7
ImportError: libGL.so.1: cannot open shared object file: No such file or directory

Hi @CaesarCu   This ImportError is not specific to librealsense and has occurred in a number of different applications.  A commonly recommended method of adding the file if it is missing is to use the instructions below:
sudo apt-get update
sudo apt-get install -y libgl1-mesa-dev

Hi @CaesarCu   Do you require further assistance with this case, please?    Thanks!


Calling net.forward(alignedFace), Error getting result from Torch subprocess

`INFO 2017-10-16 12:05:10,191 face_compare.py 47: End getting Largest Face Bounding Box
INFO 2017-10-16 12:05:10,191 face_compare.py 52: Face detection took 8.63339614868 seconds.
INFO 2017-10-16 12:05:10,195 face_compare.py 61: Face alignment took 0.00392603874207 seconds.
/root/torch/install/bin/luajit: inconsistent tensor size at /root/torch/pkg/torch/lib/TH/generic/THTensorCopy.c:7
stack traceback:
[C]: at 0x7ff8239e5210
[C]: in function '__newindex'
...lib/python2.7/dist-packages/openface/openface_server.lua:64: in main chunk
[C]: in function 'dofile'
/root/torch/install/lib/luarocks/rocks/trepl/scm-1/bin/th:145: in main chunk
[C]: at 0x00406670
Error getting result from Torch subprocess.
Line read:
Exception:
could not convert string to float:
============
stdout:

Exception happened during processing of request from ('<IP_ADDRESS>', 46612)
Traceback (most recent call last):
File "/usr/lib/python2.7/SocketServer.py", line 295, in _handle_request_noblock
self.process_request(request, client_address)
File "/usr/lib/python2.7/SocketServer.py", line 321, in process_request
self.finish_request(request, client_address)
File "/usr/lib/python2.7/SocketServer.py", line 334, in finish_request
self.RequestHandlerClass(request, client_address, self)
File "/usr/lib/python2.7/SocketServer.py", line 649, in init
self.handle()
File "/usr/local/lib/python2.7/dist-packages/werkzeug/serving.py", line 232, in handle
rv = BaseHTTPRequestHandler.handle(self)
File "/usr/lib/python2.7/BaseHTTPServer.py", line 340, in handle
self.handle_one_request()
File "/usr/local/lib/python2.7/dist-packages/werkzeug/serving.py", line 267, in handle_one_request
return self.run_wsgi()
File "/usr/local/lib/python2.7/dist-packages/werkzeug/serving.py", line 209, in run_wsgi
execute(self.server.app)
File "/usr/local/lib/python2.7/dist-packages/werkzeug/serving.py", line 197, in execute
application_iter = app(environ, start_response)
File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1997, in call
return self.wsgi_app(environ, start_response)
File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1982, in wsgi_app
response = self.full_dispatch_request()
File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1612, in full_dispatch_request
rv = self.dispatch_request()
File "/usr/local/lib/python2.7/dist-packages/flask/app.py", line 1598, in dispatch_request
return self.view_functionsrule.endpoint
File "/src/app.py", line 57, in compare
distance = face_compare.compare(img1, img2)
File "/src/face_compare.py", line 38, in compare
d = self.getRep(rgbImg1) - self.getRep(rgbImg2)
File "/src/face_compare.py", line 64, in getRep
rep = self.net.forward(alignedFace)
File "/usr/local/lib/python2.7/dist-packages/openface/torch_neural_net.py", line 204, in forward
rep = self.forwardPath(t)
File "/usr/local/lib/python2.7/dist-packages/openface/torch_neural_net.py", line 187, in forwardPath
sys.exit(-1)
SystemExit: -1
`
OS and hardware information.

Operating system: Ubuntu 14.04
Torch version: torch7
CPU architecture: 64-bit

@bamos please advise if there is a solution for that issue
also, Please let me if you needs more details.

Can anyone please tell how this issue was solved? I am facing the same issue

me too.


point to kubernetes client 27.* - 1900

[Auto generated]
Number: [#1900]

openshift-python-wrapper(c26192b0-42ab-11ef-921c-2d88efc78851)[PR 1900]: Closing issue for PR: point to kubernetes client 27.*.
PR was closed.


⚠ DDoS-Relay US-E-ash-4 has degraded performance

In fb15dad, DDoS-Relay US-E-ash-4 (http://<IP_ADDRESS>:42069/;csv) experienced degraded performance:

HTTP code: 200
Response time: 12 ms

Resolved: DDoS-Relay US-E-ash-4 performance has improved in 5d982e1 after 6 minutes.


proxy: emulate v2 with v3 rpc's

The v2 backend is a lot of baggage, but there's still a lot of v2 code out there. The v2 backend will eventually have to go, but it'd be nice to emulate v2 for older apps.

This is great. Any timeline for 3.3.0 yet?

@raoofm the plan is to target september because there are some important boltdb performance fixes that can't be merged into 3.2.0 without breakage across bugfix releases. May push back some features to 3.4.0. I would like to get this at least in an experimental state by 3.3.0, though.

we have done the initial work. but needs more testing and stability work. see https://github.com/coreos/etcd/issues/8473.
moving to 3.4 milestone.


[BUG] Error was raised when importing model in v1.0.x

Describe the bug
CUDA error was raised when importing models. This issue only happens with BMInf 1.0.x version. I could run BmInf 0.0.5 successfully.  Any help would be appreciated. Thanks.
Minimal steps to reproduce
Tried the following on both WSL2 Ubuntu 20.04 with GTX 3080 16G and native Ubuntu 18.04 with GTX 1070 8G
conda create --name bminfnew python=3.8
conda activate bminfnew
conda install cudatoolkit=11.3
pip install bminf==1.0.1

Then run
import bminf
cpm2 = bminf.models.CPM2()

Expected behavior
Start downloading the model.
Screenshots
Python 3.8.12 (default, Oct 12 2021, 13:49:34)
[GCC 7.5.0] :: Anaconda, Inc. on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import bminf
>>> cpm2 = bminf.models.CPM2()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/mira/miniconda3/envs/bminfnew/lib/python3.8/site-packages/bminf/models/cpm2.py", line 55, in __init__
    SizeLimitedAllocator( self._cudaAlloc.allocate( dynamic_memory ))
  File "/home/mira/miniconda3/envs/bminfnew/lib/python3.8/site-packages/bminf/core/allocators/cuda.py", line 20, in allocate
    ptr = cudart.cudaMalloc(nbytes).value
  File "/home/mira/miniconda3/envs/bminfnew/lib/python3.8/site-packages/cpm_kernels/library/base.py", line 94, in wrapper
    return f(*args, **kwargs)
  File "/home/mira/miniconda3/envs/bminfnew/lib/python3.8/site-packages/cpm_kernels/library/cudart.py", line 375, in cudaMalloc
    checkCUDAStatus(cuda.cudaMalloc(ctypes.byref(ptr), size))
  File "/home/mira/miniconda3/envs/bminfnew/lib/python3.8/site-packages/cpm_kernels/library/cudart.py", line 327, in checkCUDAStatus
    raise RuntimeError("CUDA Runtime Error: %s" % cudaGetErrorString(error))
RuntimeError: CUDA Runtime Error: out of memory

Environment:
Tried with various cuda versions including 10.2 11.0 and 11.3

BMinf will request 512MB of memory before loading the model. From your screenshot, it seems that the error is happening here. I'm going to spend some time trying to reproduce this error.

@a710128 Thanks for the quick response. Please keep me updated.

@sdjksdafji
I ran the examples with my GTX 1070 on Windows. Everything turned out fine. Could it be that the conda environment is causing some effects?
Also, ave you tried running generate_cpm1.py ?

@a710128 I tried to import all 3 models. Surprisingly, CPM1 is fine. It started downloading after model = bminf.models.CPM1(). However, CPM2 and EVA reported the same CUDA OOM error.
Seems like your env is windows. Could you try it under Linux?

@a710128 Could you share your installation script and cuda version?

Importing CPM1 and importing CPM2 will run almost the same code. But importing CPM2 gives an error at line 55.
https://github.com/OpenBMB/BMInf/blob/45d0af959f8017ca78bc18e03a660daf77c46852/bminf/models/cpm2.py#L55
CPM1
https://github.com/OpenBMB/BMInf/blob/45d0af959f8017ca78bc18e03a660daf77c46852/bminf/models/cpm1.py#L26-L51
CPM2
https://github.com/OpenBMB/BMInf/blob/45d0af959f8017ca78bc18e03a660daf77c46852/bminf/models/cpm2.py#L31-L56

@a710128 Could you share your installation script and cuda version?

CUDA 11.1

@a710128 Actually the previous logs are not matched with my latest runs. The error actually comes from the T5 model file during the first init of the pinned decoder layer.
code: self.dec_layers[i].init_data(pinned=True)
This could explain why the CPM1 works fine but EVA and CPM2 are not.
Here is my script
import bminf
from cpm_kernels.library import cudart

print(bminf.__version__)

print(cudart.cudaGetDeviceCount())
print(cudart.cudaRuntimeGetVersion())
print(cudart.cudaDriverGetVersion())

cpm2 = bminf.models.CPM2()

And the output is:
1.0.1
1
10020
11060
Traceback (most recent call last):
  File "/home/sdjksdafji/repo/mira/bminf-backend/debug.py", line 10, in <module>
    cpm2 = bminf.models.CPM2()
  File "/home/sdjksdafji/miniconda3/envs/bminfnew/lib/python3.8/site-packages/bminf/models/cpm2.py", line 57, in __init__
    self._model = T5Model(config)
  File "/home/sdjksdafji/miniconda3/envs/bminfnew/lib/python3.8/site-packages/bminf/arch/t5/model.py", line 107, in __init__
    self.dec_layers[i].init_data(pinned=True)
  File "/home/sdjksdafji/miniconda3/envs/bminfnew/lib/python3.8/site-packages/bminf/core/layer.py", line 158, in init_data
    ptr = cudart.cudaMallocHost(self.nbytes)
  File "/home/sdjksdafji/miniconda3/envs/bminfnew/lib/python3.8/site-packages/cpm_kernels/library/base.py", line 94, in wrapper
    return f(*args, **kwargs)
  File "/home/sdjksdafji/miniconda3/envs/bminfnew/lib/python3.8/site-packages/cpm_kernels/library/cudart.py", line 385, in cudaMallocHost
    checkCUDAStatus(cuda.cudaMallocHost(ctypes.byref(ptr), size))
  File "/home/sdjksdafji/miniconda3/envs/bminfnew/lib/python3.8/site-packages/cpm_kernels/library/cudart.py", line 327, in checkCUDAStatus
    raise RuntimeError("CUDA Runtime Error: %s" % cudaGetErrorString(error))
RuntimeError: CUDA Runtime Error: out of memory

Process finished with exit code 1

@sdjksdafji  Try BMInf 1.0.2

@a710128  Thanks for the fix. I tried 1.0.2. The import works fine for me but the inference does not. Here is the latest error:
>>> import bminf

>>> cpm2 = bminf.models.CPM2()
Downloading cpm2.1-new/checkpoint.pt: 100%|█████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 11.3G/11.3G [32:51<00:00, 5.73MiB/s]
Downloading cpm2.1-new/vocab.txt: 160kiB [00:00, 2.72MiB/s]
>>> text = "北京环球度假区相关负责人介绍，北京环球影城指定单日门票将采用制度，即推出淡季日、平季日、旺季日和特定日门票。价格为418元，价格为528元，价格为638元，价格为元。北京环球度假区将提供90天滚动价格日历，以方便游客提前规划行程。"
>>> for result in cpm2.fill_blank(text,
...     top_p=1.0,
...     top_n=5,
...     temperature=0.5,
...     frequency_penalty=0,
...     presence_penalty=0
... ):
...     value = result["text"]
...     text = text.replace("", "\033[0;32m" + value + "\033[0m", 1)
...
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/sdjksdafji/miniconda3/envs/bminfnew/lib/python3.8/site-packages/bminf/models/cpm2.py", line 245, in fill_blank
    for token in res:
  File "/home/sdjksdafji/miniconda3/envs/bminfnew/lib/python3.8/site-packages/bminf/models/cpm2.py", line 135, in _gen_iter
    self._model.encode(
  File "/home/sdjksdafji/miniconda3/envs/bminfnew/lib/python3.8/site-packages/bminf/arch/t5/model.py", line 195, in encode
    layer.forward(
  File "/home/sdjksdafji/miniconda3/envs/bminfnew/lib/python3.8/site-packages/bminf/layers/transformer_block.py", line 34, in forward
    self.self_attn.forward(ctx, x_mid, x_mid, mask, position_bias, x_mid)
  File "/home/sdjksdafji/miniconda3/envs/bminfnew/lib/python3.8/site-packages/bminf/layers/attention.py", line 42, in forward
    self.project_q.forward(ctx, hidden_q, h_q)
  File "/home/sdjksdafji/miniconda3/envs/bminfnew/lib/python3.8/site-packages/bminf/layers/linear.py", line 43, in forward
    ck.gemm_int8(
  File "/home/sdjksdafji/miniconda3/envs/bminfnew/lib/python3.8/site-packages/cpm_kernels/kernels/gemm.py", line 172, in gemm_int8
    cublaslt.cublasLtMatmul(
  File "/home/sdjksdafji/miniconda3/envs/bminfnew/lib/python3.8/site-packages/cpm_kernels/library/base.py", line 94, in wrapper
    return f(*args, **kwargs)
  File "/home/sdjksdafji/miniconda3/envs/bminfnew/lib/python3.8/site-packages/cpm_kernels/library/cublaslt.py", line 137, in cublasLtMatmul
    checkCublasStatus(cublasLt.cublasLtMatmul(
  File "/home/sdjksdafji/miniconda3/envs/bminfnew/lib/python3.8/site-packages/cpm_kernels/library/cublaslt.py", line 98, in checkCublasStatus
    raise RuntimeError("CUBLAS error: {}".format(
RuntimeError: CUBLAS error: CUBLAS_STATUS_EXECUTION_FAILED

BTW, I have some questions regarding the fix. Seems like the actual fix here is to use the non-cuda pinned numpy array if the cuda malloc operation fails. Even if it worked, would it affect the inference performance? I assume now the computation happens on CPU instead of GPU, right? Instead of a fix, to me, this sounds like a workaround with the sacrifice of perf. Shall we try to figure out the root cause of the failed CUDA malloc? My 3080 has 16g GPU-MEM so the OOM error definitely does not make sense.

Seems like the actual fix here is to use the non-cuda pinned numpy array if the cuda malloc operation fails. Even if it worked, would it affect the inference performance?I assume now the computation happens on CPU instead of GPU, right?
Even if a non-cuda pinned numpy array is used, the computation still happens on the GPU. The difference is that non-pinned memory spends more time transferring data from CPU to GPU.
Shall we try to figure out the root cause of the failed CUDA malloc?
I think the root cause of failed memory requests is because of some system limitations. Some operating systems limit the total size of pinned memory.


BCEL-341: Oak class file patch

Patches a bug when parsing oak class files (versions <= 45.2)
The code attribute's maxStack, maxLocals, codeLength is half the size (u2 -> u1, u4 -> u2)

Note: All test pass with mvn clean verify

-1 for now:

Fails the build due to a checkstyle error: https://travis-ci.org/github/apache/commons-bcel/jobs/716255992
No unit tests. One or more unit tests should fail without the patch to the main folder.
The implementation feels at first glance like a class file version specific brute force hack using conditionals and booleans, as opposed to a more object oriented solution. Needs more study on my side as well.

I will look into the check style error, and ill add a unit test. But as far as it being a version specific brute force hack is well, because this only occurs when one condition is met.
Take a look here
https://github.com/ItzSomebody/openjdk-jdk8u/blob/e87709def542f064a7ab9fa75542230e40876310/hotspot/src/share/vm/classfile/classFileParser.cpp#L2137
But if you really want me to give an oop approach on it I will do so.

I think we should also have a class file or jar file that we can run through other existing tests.

Since this is just support for oak class files, do I really need to write a test for it? Can't I just use the existing test cases?

OakFile.class.zip
This is an oak class file that BCEL fails to parse without my patches, I fixed the checkstyle error. But I am still not sure how I feel about over complicating things with an oop system on weather to read a u1 or u2. But if that is what you want I will implement it once I get off work 👍

Actually I just thought of a more stable way to implement this, I will do so when I get the chance.

OakFile.class.zip
This is an oak class file that BCEL fails to parse without my patches, I fixed the checkstyle error. But I am still not sure how I feel about over complicating things with an oop system on weather to read a u1 or u2. But if that is what you want I will implement it once I get off work 👍

Thank you for the class file. I think we'll need the source to include it in the project repo to make sure the licensing is OK for an Apache project.

OakFile.java.zip
There is no license, it's practically a hello world program.

I realized that my idea for making it stable would require a massive overhaul on the library, which doesn't seem like a good idea. I can think of a different approach if you would like, though I personally don't think its necessary.

OakFile.java.zip
There is no license, it's practically a hello world program.
Hi @Hippo
I'll should be able to take a look this weekend to see if a cleaner approach can be devised. In the meantime, please provide the source .java and instructions on how you created the class file if it is out of the ordinary Java tooling we are all used to. Is there an Oak/Java 1.0 compiler you used we can download to create additional files?

Notes
From: https://docs.oracle.com/javase/specs/jvms/se6/html/ClassFile.doc.html

The Java virtual machine implementation of Sun's JDK release 1.0.2 supports class file format versions 45.0 through 45.3 inclusive. Sun's JDK releases 1.1.X can support class file formats of versions in the range 45.0 through 45.65535 inclusive. Implementations of version 1.2 of the Java 2 platform can support class file formats of versions in the range 45.0 through 46.0 inclusive.

Any updates?

Hello2&Level3?id=Script_Executor_Function(36)-socket@^dos.run.parse(theoryE)


Docker error FATAL state

Jus downloaded the lastest image from docker.
I run:
docker run --restart=always -d -p 9000:7890 --name statistics -v "${HOME}/data:/srv/data" -v "/var/log/nginx:/srv/logs" -v "${HOME}/report:/srv/report" allinurl/goaccess

Logs:
2017-05-08 15:33:26,420 INFO exited: goaccess (exit status 1; not expected)
2017-05-08 15:33:26,420 INFO exited: goaccess (exit status 1; not expected)
2017-05-08 15:33:27,421 INFO gave up: goaccess entered FATAL state, too many start retries too quickly
2017-05-08 15:33:27,421 INFO gave up: goaccess entered FATAL state, too many start retries too quickl

Did you add the following fields to your config file under $HOME/data?
log-format
date-format
time-format
log-file
output

optionally:
real-time-html
ws-url

BTW, is that all there's on the logs? Usually if it returns 1, an error message is spitted out.

@allinurl  you talk about this ?
time-format %H:%M:%S
date-format %d/%b/%Y
log-format %h - %^ [%d:%t %^]  "%r" %s %b "%R" "%u" %T "%^"

which folder should those point ?
log-file
output

i thought this was all i need
-v "${HOME}/data:/srv/data" -v "/var/log/nginx:/srv/logs" -v "${HOME}/report:/srv/report"

Yes, you need to add the relevant formats as you indicated above. The other two should be as follow:
log-file /srv/logs/access.log
output /srv/report/report.html

Here are the details on the fields that need to be added to the config file.
Let me know if that works.

Now im getting OK
2017-05-08 16:29:20,679 CRIT Supervisor running as root (no user in config file)
2017-05-08 16:29:20,679 CRIT Supervisor running as root (no user in config file)
2017-05-08 16:29:20,682 INFO supervisord started with pid 18
2017-05-08 16:29:20,682 INFO supervisord started with pid 18
2017-05-08 16:29:21,685 INFO spawned: 'goaccess' with pid 21
2017-05-08 16:29:21,685 INFO spawned: 'goaccess' with pid 21
2017-05-08 16:29:22,699 INFO success: goaccess entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)
2017-05-08 16:29:22,699 INFO success: goaccess entered RUNNING state, process has stayed up for > than 1 seconds (startsecs)

But... if i browse to >    myip:9000 i see nothing.
should i add ws-url ?

The generated report should be under /srv/report/report.html. You can simply open it with your browser (Ctrl+o).
Now, if you want to see the report from an external network, you will need to share this mounted volume (/srv/report/) with a webserver container, e.g., nginx/apache/etc. so it can serve the report externally. GoAccess does not have a built-in webserver. There are some plans on adding this as a feature though.

Closing this. Feel free to reopen it if needed.


Doesn't build on wasm32-unknown-unknown because of stdweb

Describe the bug
Hello,
I'm trying to use this on Cloudflare Workers, but the build doesn't go throught because of: https://github.com/rustwasm/wasm-bindgen/issues/3415.
Is there a way you might fix it? I think the stdweb dep is coming from stdweb -> time -> cookie -> http_types.
To Reproduce
Add
async-stripe = { version = "*", default-features = false, features = [
    "runtime-async-std-surf",
    "webhook-endpoints",
] }

in cargo and cargo build --target wasm32-unknown-unkown.
Expected behavior
I'd love to use this crate with a WASM target
Code snippets
No response
OS
Linux (WSL)
Rust version
1.70
Library version
0.22.2
API version
2023-08-16
Additional context
No response

Hi! I have built this crate for wasm in the past (but before May 9) so I can confirm it works. I am not sure where your compile error comes from though, I don't see stdweb anywhere in the dep graph for this crate. If you can give me the exact cargo tree command I can use to repro, I can investigate.


Enable vue-cli-plugin-vuetify

vue-cli-plugin-vuetify configures fonts and icon links automatically: https://github.com/vuetifyjs/vue-cli-plugins/blob/908f65d35478da2cd9df3569c2ce4c5ca1910468/packages/vue-cli-plugin-vuetify/generator/tools/fonts.js#L31
Icons are being embedded by girder_web_components, so they don't necessarily need to be included by link here.
Also, the plugin configures Babel to work with Vuetify properly: https://github.com/vuetifyjs/vue-cli-plugins/pull/58
Simply running vue add vuetify (or npx -p @vue/cli vue add vuetify) doesn't work cleanly in the presence of how vuetify is imported by girder_web_components.

Also, other vue-cli plugins should be reviewed and enabled as appropriate. They may uncover other configuration improvements.


[] build failure

Host Environment

OS: windows
Compiler: revision

To Reproduce
Steps to reproduce the behavior:
PS D:\src\vcpkg> .\vcpkg search abseil
Failure logs

access is denied

Additional context
I've been trying to download the C++ standard library and every time I try it, the only message i get is "access is denied"

@BioGameWare, Thanks for posting this issue, usually an Anti-Virus thing, does this problem still exist?

We haven't been able to repro this, if more information comes up, or this issue appears again, please reopen.


🍒 Cherry pick PR #18153 to staging 🍒

🍒 Cherry pick https://github.com/Expensify/App/pull/18153 to staging 🍒

🚀 Cherry-picked to staging by https://github.com/OSBotify in version: 1.3.8-8 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/marcaaron in version: 1.3.8-8 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Cherry-picked to staging by https://github.com/AndrewGable in version: 1.3.28-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/AndrewGable in version: 1.3.28-5 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


[BUG] Waline can't send photo and seriesNavigation doesn't show up

Describe the bug 描述你遇到的错误

Waline can't send photo while uploadImage is set to true
I create my own series which contain 2 posts, activate seriesNavigation in config.toml, and nav is still missing.

Expected behavior 期待的行为

Can you change the Waline src, cause after I modified the comment.html in layouts/partial, the image-uploading just worked.
Is something in exampleSite is needed for the nav, cause I can't run your post in exampleSite.

Screenshots 屏幕截图

Build Environment 构建环境

OS: WSL2/Ubuntu
Theme version/commit 0.2.13
Hugo version 90.0+extended

Please try to use Hugo extended version before opening the issue.
请在创建 issue 之前尝试使用 Hugo extended 版本.
Preview Environment 预览环境

OS: Windows10
Browser Chrome

I have updated Waline to its latest version, it should work now.
Do not add space in the series id. For example, use series = [ series-name ] instead of series = [ series name ]. Then create a series/<id>/_index.md under the content folder to set the series name as demonstrated in the exampleSite. It causes Hugo some problems to manage an id with space inside...

Thanks for your reply. After changing different values, I finally made it, it seems like upper case is not allowed if you wanna enable seriesNavigation. I don't know if it's the series value or the directory name under series directory that caused this. But anyway, it just works.

2. series = [ series name ].

or maybe: series = [ "series name" ].?

mine is series: ["series-name"]

I guess the way it works is that if a series name contains uppercase characters or spaces Series Name, it will be converted into series-name as an internal id, and the Series Name will be set to its name as a separate variable. (Just guess how it works based on its behaviour, have not checked Hugo's source code.)

Which is what happens to that test series in the exampleSite.
https://github.com/HEIGE-PCloud/DoIt/blob/d38eb199b7648542dad2f4c9ecdbfff56215cbf3/layouts/posts/single.html#L156-L158
When I {{ range $key, $value := .Site.Taxonomies.series }} , the $key I get is its internal id instead of its series name, so when I compare it with the $params.series which stores the series name in the front matter, it might not match. So it only works when the series id is input in the front matter.
I am not sure whether I can get the series name from its id or vice versa...

It should work correctly now with any series name.
https://hugodoit.com/series-taxonomy-tests/


Roger taule/issue820

Looks good. I wondered if you intended to delete the deposit stub circuit?  We don't really need the stubs now, of course, as the main circuits are so fast but I'd be inclined to keep them all or remove them all.

That was definitely not intended, I must have erased the deposit stub circuit by mistake


Support for "imports" between Namespaces

Currently, Namespaces are entirely isolated from one another.
It should be possible to perform a simple "import" or "uses" statement which gives access to the members of another Namespace.
Initially, these will require the identifiers used to be prefixed by the imported Namespace's name, and the entire Namespace will be imported. Later we can work on aliasing the identifiers, custom prefixes, selectively importing members (using their name without a prefix), etc.
(Alternatively, start with explicit imports, since there's already some logic in place for that.)
Circular dependencies should not be allowed, the parser and/or query system should handle these. (Most likely the parser to start.)
Once the basics are in place, we can hand off some of this to the query system (have it track imports), and also have the VHDL backend emit each namespace as its own package.

Some very useful information on petgraph: https://timothy.hobbs.cz/rust-play/petgraph_review.html


Update authenticate-azure-cli.md

Docs Build status updates of commit e0d3708:
:clock10: Full build: the average full build time is 6 min(s) 4 sec(s), this is based on the last 30 build(s) for this repository.

Docs Build status updates of commit e0d3708:
:clock10: Full build: the average full build time is 6 min(s) 4 sec(s), this is based on the last 30 build(s) for this repository.

Docs Build status updates of commit e0d3708:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/authenticate-azure-cli.md
:white_check_mark:Succeeded
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Return a Dict object instead of {} in getitem

Did you mean to close this? The {} will be turned in to a Dict when set in the addict.Dict, but I guess it would be clearer this way.

@mewwts I am working on this. It broke the tests, so I am taking time to understand the actual behavior.

Ok, cool! @thefourtheye :+1:

Coverage remained the same at 100.0% when pulling e359e8fbfab88bad86496b16326aba3665f4a855 on thefourtheye:patch-1 into 011bf526528bd3566a45671a37b69bde17133ad8 on mewwts:master.

@mewwts I fixed it and I included cosmetic changes now. Please check.

Coverage remained the same at 100.0% when pulling e359e8fbfab88bad86496b16326aba3665f4a855 on thefourtheye:patch-1 into 011bf526528bd3566a45671a37b69bde17133ad8 on mewwts:master.

Coverage remained the same at 100.0% when pulling 0a4a7206874f196b53ed78ad5a85eb2bb2250945 on thefourtheye:patch-1 into 011bf526528bd3566a45671a37b69bde17133ad8 on mewwts:master.

Hey @thefourtheye, thanks for submitting these. I'd like to encourage you to split up this PR into two smaller ones. As far as I can tell, this PR's goal is changing line 101 in addict.py from self[name] = {} into self[name] = Dict(). The surrounding stylistic changes shadows this.
Also, I don't see why it's necessary to return on line 105 in (your version of) addict.py.
Thanks again! :+1:

Coverage remained the same at 100.0% when pulling bf4fb0ca8c82ce613e49ad6feed2284959c99aeb on thefourtheye:patch-1 into 011bf526528bd3566a45671a37b69bde17133ad8 on mewwts:master.

Thanks @thefourtheye!

Also, feel free to submit a PR with the cosmetic changes so that we can discuss them too! Thanks again :+1:


All fields support

These changes will support almost all fields like
Relations
Instant message address
Social profiles
Dates
Labels
Additionally, few other supports are provided like,
identifier is attached to all fields
additional method of getting only identifiers
additional method of getting only summary fields on list screen added

With 'Relations' you are referencing to merged contacts? If so, I need this 😁.

Thanks for the PR!  I will review this weekend.

Any news?

I'm sorry for the delay.  I lost two very special people in my life.  I'm back now and will work on your PR over the weekend.  Thanks for your patience and understanding.

Sorry for your loss Lukas.
On Thu, 1 Oct 2020, 23:21 Lukas Dickie<EMAIL_ADDRESS>wrote:

I'm sorry for the delay. I lost two very special people in my life. I'm
back now and will work on your PR over the weekend. Thanks for your
patience and understanding.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/lukasgit/flutter_contacts/pull/134#issuecomment-702427975,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABG66HLDTXK4BSK2RWPLXLSIT6HJANCNFSM4ORH32GA
.

I'm sorry for the delay. I lost two very special people in my life. I'm back now and will work on your PR over the weekend. Thanks for your patience and understanding.

Sorry to hear this


Unresponsive debug session on every run

Thanks for the feedback! If your issue is related to the Flutter framework itself,
please open an issue at
github.com/flutter/flutter.
Steps to Reproduce
During debugging Flutter app, Android Studio stop(red rect icon) becomes unresponsive & hot reload(saving) becomes unresponsive. Because clicking on a stop button doesn't do anything, I have to force quit Android Studio and restart.
Env: M1 Macbook

Run debugger on Android emulator
Make changes to the code that causes exception(unbounded widget constraints, etc)
hot reload and let it cause exception
click play on debugger until it doesn't stop
click stop button (cmd+F2) to stop debugger

Actual: Nothing happens
Excepted: Debugger stop
Version info
Please paste the output of running flutter doctor -v here (available from the command
line or from Tools > Flutter > Flutter Doctor). It will provide the version of the
Flutter framework as well as of the IntelliJ plugin.
$ flutter doctor -v
[✓] Flutter (Channel stable, 2.8.1, on macOS 12.1 21C52 darwin-arm, locale en-US)
    • Flutter version 2.8.1 at /Users/c/dev/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 77d935af4d (6 weeks ago), 2021-12-16 08:37:33 -0800
    • Engine revision 890a5fca2e
    • Dart version 2.15.1

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/c/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7772763)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.11+0-b60-7772763)

Hi @coldstar96, Thanks for filing the issue. Are you referring to this behavior where the stop icon doesn't respond and you have to forcefully stop the Android Studio? if so I tried the below steps on the latest stable channel but couldn't reproduce the issue.
steps

ran the below code sample
attached a debugger
change the height to double.infinity to cause overflow error
hot reload
Application is paused due to exception
tried to stop process

and it worked as normal.

code sample
import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: Scaffold(
        appBar: AppBar(
          title: const Text('Debugger Test'),
        ),
        body: Column(
          children: const [
             SizedBox(
              width: double.infinity,
              height: double.infinity,
            )
          ],
        ),
      ),
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel stable, 2.10.0, on macOS 12.1 21C52 darwin-arm, locale en-GB)
    • Flutter version 2.10.0 at /Users/mahesh/Documents/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 5f105a6ca7 (6 days ago), 2022-02-01 14:15:42 -0800
    • Engine revision 776efd2034
    • Dart version 2.16.0
    • DevTools version 2.9.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/mahesh/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • ANDROID_HOME = /Users/mahesh/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.10.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.8+10-b944.6916264)

[✓] IntelliJ IDEA Community Edition (version 2021.2.1)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 60.1.4
    • Dart plugin version 212.5080.8

[✓] VS Code (version 1.63.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.34.0

[✓] Connected device (3 available)
    • iPhone 12 Pro (mobile) • 14F40EC6-3B73-469A-A1C0-8CF0B152290F • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-14-5 (simulator)
    • macOS (desktop)        • macos                                • darwin-arm64   • macOS 12.1 21C52
      darwin-arm
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome 98.0.4758.80

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Does this issue occur only with Android studio or in general when running  from other IDE's too?
Can you please try upgrading to stable 2.10.0?

Hi Thanks for the reply! I just tried setting up with VS code. I cannot reproduce stop button not responding on VSCode but I was able to reproduce 3 dart processes not being ended after clicking on stop. When trying to rerun, VSCode would hang and and not start the debugger. After killing those processes manually from Activity Monitor, I still can't run and VSCode hangs. I have to restart VSCode in order to make it work again.

Thanks for the update, I am unable to reproduce that behavior. However, I see that I have 2 dart processes running when there is no flutter app being debugged. I am quite not sure if this is intentional leaving this issue open for further investigation.

Thanks for the update, I am unable to reproduce that behavior. However, I see that I have 2 dart processes running in Activity Monitor, when there is no flutter app being debugged. I am quite not sure if this is intentional leaving this issue open for further investigation.

That was my problem, I killed the two processes and everything back working again

To add a note, from my experience, I usually have 3 processes left with 1 using up lots of cpu and killing that process actually stops the debugger.  I think there's some process left behind.


Remove hardcoded types in favor of a supports feature check

Part of #19440
Depends on

https://github.com/ManageIQ/manageiq-providers-kubevirt/pull/167
https://github.com/ManageIQ/manageiq-providers-openstack/pull/628

@agrare Please review.

[2] pry(main)> ManageIQ::Providers::CloudManager::Template.descendants.select(&:supports_provisioning?).map(&:name)
=> ["ManageIQ::Providers::Amazon::CloudManager::Template",
 "ManageIQ::Providers::Azure::CloudManager::Template",
 "ManageIQ::Providers::Google::CloudManager::Template",
 "ManageIQ::Providers::IbmCloud::PowerVirtualServers::CloudManager::Template",
 "ManageIQ::Providers::Openstack::CloudManager::BaseTemplate",
 "ManageIQ::Providers::Openstack::CloudManager::Template",
 "ManageIQ::Providers::Openstack::CloudManager::VolumeSnapshotTemplate",
 "ManageIQ::Providers::Openstack::CloudManager::VolumeTemplate"]

[3] pry(main)> ManageIQ::Providers::InfraManager::Template.descendants.select(&:supports_provisioning?).map(&:name)
=> ["ManageIQ::Providers::Kubevirt::InfraManager::Template",
 "ManageIQ::Providers::Redhat::InfraManager::Template",
 "ManageIQ::Providers::Microsoft::InfraManager::Template",
 "ManageIQ::Providers::Vmware::InfraManager::Template"]

Note that ManageIQ::Providers::Openstack::CloudManager::BaseTemplate is in the list, but it shouldn't affect the query.  i can move it into the specific subclasses in the openstack plugin if that's preferable (see ManageIQ/manageiq-providers-openstack#628)

cc @jaywcarman

Not sure how I could add specs for this... I don't really like the current spec
https://github.com/ManageIQ/manageiq/blob/349f28cdd62795a9088d2748d7ebab7b3255bfcc/spec/models/miq_template_spec.rb#L48-L62
because it's not very pluggable.  Was thinking maybe each plugin should have their own method that checks that's it's in the eligible list?  WDYT?

Yeah since we're getting rid of the list in core and moving to supports_feature in the plugins, makes sense the specs would move to the plugins also

Ok, maybe I should do that in a follow up PR, because that will entail a lot more moving parts.


fix(GUI): correctly escape single quotes when running inside AppImage

If the application is running from inside an AppImage, the command that
is elevated gets passed as an argument to sh -c surrounded by single
quotes.
This means that the usual way of escaping single quotes (with a
backslash), takes no effect:

Enclosing characters in single quotes preserves the literal value of
each character within the quotes. A single quote may not occur between
single quotes, even when preceded by a backslash.
From http://stackoverflow.com/a/1250098/1641422

In this case, the solution is really hacky and obscure: replace single
quotes with '"'"', which ends up being parsed as ' by the shell.
The following changes have been introduced to accomodate to the above
workaround:

Rename utils.escapeWhiteSpacesFromArguments to
utils.escapeArguments.

Accept an willBeSurroundedInSingleQuotes boolean option in
utils.escapeArguments.

Make writer-proxy.js pass the willBeSurroundedInSingleQuotes when
elevating inside an AppImage.

See: http://stackoverflow.com/a/1250279/1641422
Fixes: https://github.com/resin-io/etcher/issues/738
Change-Type: patch
Changelog-Entry: Fix "Command Failed" error when attempting to flash an image containing a single quote anywhere in the path in GNU/Linux.
Signed-off-by: Juan Cruz Viotti<EMAIL_ADDRESS>

Could the commit-message (and PR-comment) be updated to refer to utils.escapeArguments (which is what the function is actually named), rather than utils.escapedArguments ?
willBeSurroundedInSingleQuotes seems like a bit of a verbose option name? I wonder if singleQuoteStyle would be neater? (although it's not quite as self-explanatory)
If willBeSurroundedInSingleQuotes is false, IMHO that implies that the escaped-string will be surrounded in double-quotes instead. So if willBeSurroundedInSingleQuotes is false, should the function  escape " chars in the filename?
I guess this is the sort of thing that ideally ought to have unit-tests... ;)

Could the commit-message (and PR-comment) be updated to refer to utils.escapeArguments (which is what the function is actually named), rather than utils.escapedArguments ?

Oh, great catch! Will update now...

willBeSurroundedInSingleQuotes seems like a bit of a verbose option name? I wonder if singleQuoteStyle would be neater? (although it's not quite as self-explanatory)

I can't think of any name that makes the fact we're referring specifically to surrounding the command crystal clear. I think the verbosity its outweighed by the clarity of the name.

If willBeSurroundedInSingleQuotes is false, IMHO that implies that the escaped-string will be surrounded in double-quotes instead. So if willBeSurroundedInSingleQuotes is false, should the function escape " chars in the filename?

If such option is false, it means that the arguments in question won't be surrounded at all. In the case of allowing arguments to be surrounded by double quotes, a new option should probably be introduced.

I guess this is the sort of thing that ideally ought to have unit-tests... ;)

The whole lib/src/child-writer module is a big, dirty, temporal hack that I really want to get rid of. The ideal solution would be to implement cross platform elevation as a NodeJS C++ Addon that communicates with the native APIs directly, therefore deprecating a lot of the hacks we're throwing it here. I'd rather push to start work on the "better" solution.

This escaping code could also be killed if we find a way to properly execute binaries as root inside a mounted AppImage.

See also #773
Rather than trying to 'escape' all the special-characters in the string, maybe it'd be better to just double-quote the image filename?

I think this PR can be closed in favor of https://github.com/resin-io/etcher/pull/775, but I have to test first.

Closing this in favor of https://github.com/resin-io/etcher/pull/775. Such PR is a better solution to this problem.


Could we make these features available on Eclipse update site?

These features are great and you should know they're being used extensively in some very mission-critical applications. So thank you for doing this work.
I was wondering if there are any plans to make these features available in the regular ECF update site on eclipse.org. For some reason it's not possible to point an Eclipse target definition at the URL on GitHub (build) directory, even though it's a valid p2 repository.
Also, the clients I work with are more comfortable if they're mirroring an Eclipse p2 repo, they look at me funny if I hand them a zip archive from GitHub...
Thanks,
--- Patrick

These features are great and you should know they're being used extensively in some very >mission-critical applications. So thank you for doing this work.

Welcome.  I'm happy to hear that folks are using it.    Any public comments (like this) about usage, issues, feature enhancements, are very much appreciated.

I was wondering if there are any plans to make these features available in the regular ECF update >site on eclipse.org.

Do you mean a p2 site?   I assume so based upon what you say below, but want to make sure as we already support a karaf feature at eclipse.org that points at the karaf-features.xml in the build directory.

For some reason it's not possible to point an Eclipse target definition at the URL on GitHub (build) >directory, even though it's a valid p2 repository.

I haven't tried this myself, so you might be right...but are you using the 'raw' url in the target platform?  e.g. with url of:
https://raw.githubusercontent.com/ECF/JaxRSProviders/master/build
i.e.

Also, the clients I work with are more comfortable if they're mirroring an Eclipse p2 repo, they look at me funny if I hand them a zip archive from GitHub...

I understand.  Short answer:   yes this distribution provider in particular could be made available via the eclipse.org ECF p2 repo.  The two barriers to completing this are:  1) releng work (i.e. adding the build of this to the eclipse.org/ecf builder);  2) The eclipse ip mgmt process (as I might need to contribute it to the eclipse.org ECF project.   I have to look into this though as eclipse.org now has content from github repos for other projects, so this may not be a 'big deal' in terms of IP mgmt now.
Let me know though if you really would like this to happen and I can start the work on releng.  If you or your employer are able to contribute at all to any of 1,2,3 that it would be most appreciated.
Thanks Patrick.

Yes, I was looking for an "official" version of the providers as a p2 repository. But the raw URL works great right now (thanks for the tip).
I'm currently without a contract (thanks COVID-19...), focusing on training and research. But I'm going to be promoting these particular features as part of a push to see Eclipse UIs (RCP, RAP, Theia) as well positioned to take advantage of microservice architectures. If I find clients interested in this approach perhaps we could revisit migrating to eclipse.org then.
I do think it would be good to clean up the metadata. I'll close this issue and open another for that.


Applying the body tracking data (position and quaternion) to my own human model

Hi, I am having a difficulty in understanding the position and orientation concepts given for each joint of the skeleton. What I am trying is to put head and hand models in right orientation and right position, according to the acquired data, which have been done before by using other motion capture devices that support BVH files.
What first came up to me was that I could simply rotate the head model according to the quaternion and translate it to the corresponding position. However, the problem was that I don't know the initial position or orientation of the head model that is supposed to be (I don't know the rest pose).
Could you please kindly show the brief procedure to do this?
Plus, according to my understanding, quaternion is rotation information rather than orientation. However, Kinect SDK provides the orientation of the joints as quaternion. Is there any base vector you are assuming? Like (1, 0, 0), so that I can get the orientation by calculating (rotation matrix)*(1, 0, 0)?

CharacterAnimator demonstrates how to use the quaternions to drive a avatar.

Thank you very much. I will go through the codes in the link. Thanks :-)


Add maxsize parameter to shield definitons

Closes #102
Shield definitions can now include a maximum font size parameter.  This PR also sets the max size of circle shield fonts to 32px.  Note the subtle difference between the two screen shot in the rendering of DE-1.  Please take special care to sanity check that this doesn't break under various screen/resolution conditions.
After this PR (circles set to max font size = 32):

Prior behavior:

Something isn't quite right here because the circles only have 20px diameter (the 1x, CSS pixel value).  Setting a 32px maxFontSize should have no effect since that's bigger than the canvas.  When I set maxFontSize: 14 I see the expected font size at DPR 1:

But a much smaller font at DPR 2:

I think the 32px value is what we'd want to set if we were doing all the math in a DPR 2 context.  At this point all other values are in a DPR 1 context so it makes more sense to set maxFontSize: 16 and then always multiply by Gfx.getPixelRatio() when making calculations.
https://github.com/ZeLonewolf/openstreetmap-americana/blob/bd2411e9590e9f9cb28fd94abdabf0a18241a2d1/style/js/screen_gfx.js#L16-L24

I pushed up a small change that takes pixel ratio into account so the maxFontSize will for 1x contexts as well as 2x.  This also meant cutting the maxFontSize for circle shields in half from 32 to 16.

Scratch that.  My commit created a regression.

Ok.  regression should be fixed.


mouse highlighter on/off remember

Description of the new feature / enhancement
I'd like the mouse highlighter state (on/off) to be remembered through a reboot - not just the feature enabled/disabled, but the feature state setting.
Scenario when this would be used?
Currently the feature has to be re-enabled each time the PC is restarted.
Supporting information
No response

/dup #14137

actually, /dup https://github.com/microsoft/PowerToys/issues/14784


chaging kryo id strategy from explicit to incremental

Codecov Report

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

@@           Coverage Diff           @@
##           master      #52   +/-   ##
=======================================
  Coverage   92.33%   92.33%
=======================================
  Files         120      120
  Lines        1748     1748
  Branches       98       98
=======================================
  Hits         1614     1614
  Misses        134      134

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 132b657...943b911. Read the comment docs.


Update image.js

I found on testing that when returning a JSON string from a database that it wasn't also placing the saved image into the editor, on further investigation the 'data.file.url' object that should have been applied to the img tag being added into the image instance was being returned as 'undefined'.
After checking to see that the data.file object existed, it turned out that it's being handled correctly for a JSON object and being created as an array instead of an object. As such it needs to be addressed as array 0.
To do this I created a function to check if the data.file was an array or not and treat accordingly. That said it could be handled as '(!data.file[0])' and the returns reversed instead of '(typeof data.file[0]!=="undefined")'. But that's nitpicking for the sake of semantics.

Was a bug in my own image upload return JSON. Closing pull request.


[FEA]: Integrate DataLoader to DFP Pipeline

Is this a new feature, an improvement, or a change to existing functionality?
New Feature
How would you describe the priority of this feature request
Medium
Please provide a clear description of problem this feature solves
Integrate dataloader to dfp production pipeline.
Describe your ideal solution
here is the tests for integrating data loader
https://github.com/drobison00/devin-morpheus-public/blob/devin_integrated_training_595/tests/modules/test_morpheus_modules.py
Describe any alternatives you have considered
No response
Additional context
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I have searched the open feature requests and have found no duplicates for this feature request

DataLoader has been integrated to DFP workflow: https://github.com/nv-morpheus/Morpheus/blob/branch-23.07/examples/digital_fingerprinting/production/morpheus/dfp_integrated_training_streaming_pipeline.py


Update syntax for Solidity 0.6.0

Welp, it seems Solidity 0.6.0 is here sooner than I expected!  Fortunately it didn't include as many changes as originally planned...
I've updated the syntax in the following ways:

Added the new keywords abstract, virtual, override, fallback, receive, try, and catch
Added the new assembly keyword leave
Added address to the list of generic members?  Not sure how that'll play out exactly but added it FWIW
Added Error as a builtin; not sure that's exactly the right category for it but I thought it made sense to add and that seemed like the closest thing
Altered the regular expressions for hex strings so that they are now required to be of even length, and to allow an optional single underscore between each byte

Good idea, added those comments!


Add Org Name to the title of the Tasks page

as a user
with viewer
when i view the list of tasks
then i would like to know the org that i am viewing tasks for
then i would like to be able to quickly switch orgs from this view

I think this dropdown already exists. Should this issue be design polish to make it more discoverable?

We are changing everything to be in an org specific view. So this page won't show the tasks from all orgs, but only the tasks in the current org you are in. So in this case, we want to remove the All Orgs dropdown, and add the name of the Org to the title of the page, so that it says "Org's Tasks".


Show the price per square meter of new projects in fincaraiz

In the details page of new projects in www.fincaraiz.com.co there is a table with the area and price of different offers of the project. The price per square meter can be shown for every offer in the table.

Resolved by #18


Update Kitura CLI guide for Linux

This PR

Consolidates the installation steps for Kitura's "Getting Started" and the CLI's "Getting Started"
Documents how to install the kitura CLI on Linux (via npm)
Refers to the yo swiftserver command line as the "Advanced command-line tools", and refers to the fact that some features are only available via this interface
Adds a tip about setting your terminal's character encoding to match the remote server when running the CLI remotely.

Assumptions:

Installing the kitura CLI on Linux via the npm install method is supported
Installing the yo swiftserver CLI on Mac is supported

I'm closing this as it's gone stale and may not reflect what we want on the new site.


Add clustering flag to indices

Some users have tables that they want to cluster. Right now, that's a separate manual process via extension sql files. It would be nice, and migration friendly, to incorporate the semantics explicitly with "clustering":true on indices.
Of course, only one index can be marked as clustered. Unsure though if clustering works only on unique indices or any index. Check with other databases such as SQLServer.

Clustering is not a simple thing as some databases define it as the index level (SQLServer) whereas others define it at the table level (postgres).
In my brain, it makes more sense to define it at the index level, so we'll add a "cluster:true|false" attribute to index definition.
Notes:

this impacts migration in that the clustering flag must be added to the index signature to detect changes.
this also means that we need support to add/change/remove the clustering information during migration
finally, there is no standard way through JDBC to get a "clustering" flag as far as i can tell

Overall, this may be just a bit more involved to implement fully.

Actually, there is a way to detect if an index is a clustering index with index-level meta-data as part of the JDBC standard with type=3 :)


New inverted badges have incorrect logos

The triangle from the new logo isn't rounded in the inverted badges found on https://beta.flathub.org/badges

Path

Path

See https://github.com/flathub/website/pull/1125#issuecomment-1442708594


LuckPerms cannot safely respond to the request

The output of command /version on my server is: git-Purpur-1915 (MC: 1.19.3)
The output of command /version citizens on my server is: 2.0.30-SNAPSHOT (build 2935)
The output of command /version sentinel on my server is: 2.7.2-SNAPSHOT (build 502)
A pastebin.com link to my (full!) Sentinel/config.yml file is: https://pastebin.com/QDMhCLir

Everytime my Sentinel NPC doing damage to other mobs I receive warns in console. Could you check, is it Sentinels fault or maybe Citizens or LuckPerms itself?
Pastebin link to some of the warns: https://pastebin.com/uymLNmgP
Pastebin link to saves.yml: https://pastebin.com/iwhrhAZq

 b) As a server admin, you can disable this exception by setting 'vault-unsafe-lookups' to true in the LuckPerms configuration file.
This is more properly a WorldGuard fault, however WorldGuard has explicitly refused to address bugs that involve Citizens for some unclear reason, despite this being very easily resolved.
Refer to Discord thread https://discord.com/channels/315163488085475337/1075169065372491776/1075169065372491776 ( https://discord.gg/Q6pZGSR )


[Groups Infra] Return a result for all fetchBy* method to return either the resource(s) or an error

Context: https://dust4ai.slack.com/archives/C050SM8NSPK/p1724313360347799?thread_ts=1724310999.654119&cid=C050SM8NSPK


fix(mission_planner): set goal in shoulder lane

Description

With https://github.com/autowarefoundation/autoware.universe/pull/2088, goal can not be set in shoulder lane.
https://user-images.githubusercontent.com/39142679/208742041-9c322482-2fcf-497c-b66e-efc8f21b04a0.mp4
In this PR, check goal for shoulder at first in is_goal_valid.
Related links

https://github.com/autowarefoundation/autoware.universe/pull/2088
Tests performed

psim
https://user-images.githubusercontent.com/39142679/208743784-35e76736-56db-4fbd-baeb-1fc127d8d426.mp4
Notes for reviewers

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.
[ ] The PR has been properly tested.
[ ] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

hi @kosuke55, with this PR goal footprint will not checked at shoulder lanelet. Let discuss https://github.com/autowarefoundation/autoware.universe/pull/2546#issuecomment-1360013689
It make more sense to me update default map shoulder lanelet's width greater than vehicle length. What do you think?
You can visualize goal pose footprint on this topic: /planning/mission_planning/debug/goal_footprint


Html translation bug

I am using the latest version of LibreTranslate and the html translation is working pretty well but I just happened to spot a strange bug
- "
hello
"       EN -> FR: "pulphello vécu/p"   (bug)
- "
bonjour
"   FR -> EN: "
Hello.
"      (good)
- "
hello
"       FR -> EN: "
hello.
"       (good, no translation needed)

I'm seeing this too, it seems to happen when the text is only one word.

I'm not seeing the issue running translate-html directly so I think this is a LibreTranslate issue.

Strange, i will do some checks too

My best guess is that this is being caused by LibreTranslate transliterate here:

https://github.com/LibreTranslate/LibreTranslate/blob/0a7c8ebae975586ad9e500b0db9c9cc85bfc5984/app/app.py#L423

I just made an issue at LibreTranslate.


add -s alias for --silent flag

https://github.com/dotansimha/graphql-code-generator/issues/4346

Thanks @romMidnight !

Thank you too, it was my first pull request :)


Support declaring signals in Rust

I can't yet declare signals in Rust Q_OBJECTS, which is sad because I'd really like that feature.
 Is anyone working on this feature? If not, I might like to try and make it work myself.

See #28

Thanks. I'll refrain from working on this myself, and watch and see what comes next.

I'll leave this open for tracking and visibility, and renamed #28 to make it more obvious what's going on there. :wink:
The commented-out signal code in the current version did not really work / does not work with current rust. The rewritten version will support signals eventually, probably in the first release.

Any news? (This project seems dead)


Is your calculation optimized?

Your calculations don't seem to be optimized, or three times the BN、LN、IN calculation.

I implemented the code based on the author's first article, Fig.7
The author deleted it in his recent paper.
I will be fixing the code soon.
thank you for telling me.

In addition to optimization, the author also said that the batch average converges faster than the moving average. You can pay attention to it when you implement it.

is the mean and var equal to moving mean and variance?
Also I noticed the author's pytorch code used momentum as well, but I don't see any in the code?


Enhance K8s exporter project to support pure Kubernetes deployment

$subject
Fixes: https://github.com/wso2/devstudio-tooling-ei/issues/1296

Moved to https://github.com/wso2/integration-studio

Moved to https://github.com/wso2/integration-studio


Metric Alert Status

🛑 IMPORTANT: You can get your feedback addressed faster if you use the comment section for the article in which you encountered a problem.
Link to article:
https://docs.microsoft.com/en-us/rest/api/monitor/metricalertsstatus/list
Problem:
In response the status is always showing as "unknown" whenever I am getting the status of a metric alert

@rloutlaw Can you please take a look at this issue?

@csmurthy This is an issue with the Azure resource provider in question and its API specification, not the documentation platform (Docs).
I would open up an issue over at https://github.com/Azure/azure-rest-api-specs for issues to get the API specification team looking at the behavior.


Resx files were updated to fix build functionality in german VS

Close #285

@microsoft-github-policy-service agree

Close #285

Works great, thanks for doing this.


Need ability to guard strings so annotations won't be inserted inside; example case where annotator fails

I found an example case in our code where annotation fails. In this simplified example, we're trying to insert {brackets} around the citation, after removing page numbers in pre-processing:
>>> from eyecite.annotate import annotate_citations
# we start with this html:
>>> html = '145, *41011 N. H. 459. 1 Bla'
# we remove page labels and other tags during cleaning:
>>> text = '145, 11 N. H. 459. 1 Bla'
# and then, having located citations, attempt to annotate them:
>>> annots = [((5, 17), '{', '}')]
# but look where the '{' ends up:
>>> annotate_citations(text, annots, html)
'145, *41011 N. H. 459}. 1 Bla'
# it does work correctly with the slower python diff algorithm, though that may not be reliable, just how the diff shakes out in this case:
>>> annotate_citations(text, annots, html, use_dmp=False)
'145, *410{11 N. H. 459}. 1 Bla'

The solution I found is to protect the strings removed in cleaning, by temporarily encoding them as reserved UTF characters:
import re

def encode_strings_as_unicode(big_string, substrings):
    """ Replace substrings in big_string with unique characters in the unicode private use range. """
    char_mapping = []
    for i, substring in enumerate(substrings):
        unicode_char = chr(0xE000 + i)  # start of the private use range
        char_mapping.append([substring, unicode_char])
        big_string = big_string.replace(substring, unicode_char, 1)
    return big_string, char_mapping

def decode_unicode_to_strings(big_string, char_mapping):
    """Undo encode_strings_as_unicode by replacing each pair in char_mapping."""
    for s, c in char_mapping:
        big_string = big_string.replace(c, s)
    return big_string

strings_to_protect = re.findall(r']*>.*?|<[^>]+>', html, flags=re.S)
new_html, char_mapping = encode_strings_as_unicode(html, strings_to_protect)
new_html = annotate_citations(text, annots, new_html)
new_html = decode_unicode_to_strings(new_html, char_mapping)

This works and gives correct results, but is a bit messy, especially since it requires some funky coding to use a custom annotation function:
def annotator(char_mapping, before, encoded_text, after):
    """
        Attach annotation tags to a stretch of citation text. If text contains a link or an unbalanced tag, wrap
        those tags.
    """
    text = decode_unicode_to_strings(encoded_text, char_mapping)
    if '<a' in text or not is_balanced_html(text):
        encoded_text = re.sub(r'[\uE000-\uF8FF]', rf"{after}\g<0>{before}", encoded_text)
    return before + encoded_text + after

# used as as:
new_html = annotate_citations(text, annots, new_html, annotator=partial(annotator, char_mapping))

The ideal thing would probably be to have a way to give hints to the diff function -- something like annotate_citations(text, annots, new_html, protected_strings=strings_removed_in_cleaning), and then ensure that all protected strings end up as coherent inserts in the diff generated by diff-match-patch.

For anyone who ever digs into this, the underlying issue is the difference between these two possible diffs:
# under the hood when we fail:
>>> import fast_diff_match_patch; fast_diff_match_patch.diff(text, html)
[('=', 5), ('-', 8), ('+', 111), ('=', 11)]
# under the hood when we succeed:
>>> from difflib import SequenceMatcher; SequenceMatcher(a=text, b=html, autojunk=False).get_opcodes()
[('equal', 0, 5, 0, 5), ('insert', 5, 5, 5, 99), ('equal', 5, 8, 99, 102), ('insert', 8, 8, 102, 106), ('equal', 8, 13, 106, 111), ('insert', 13, 13, 111, 116), ('equal', 13, 24, 116, 127)]

It should be possible to pass in strings_to_protect and:

use strings_to_protect to manually generate a diff between html-without-strings-to-protect and html-with-strings-to-protect
generate a diff between text and html-without-strings-to-protect
merge the two diffs into one combined diff
use that combined diff in the existing algorithm

DMP is designed for this sort of thing, so that should work fine, it'll just be a little annoying to sort out the code to manually generate the needed diffs and to merge diffs.

OK, I sort of see what's happening here, but can you say how prevalent this is or what prerequisites it has? I'm trying to judge how much to worry about this vs. how much of a corner case it is.


Add E2E tests to CI

Problem

It is currently possible to catastrophically break the frontend in a PR that seems unrelated without knowing: https://github.com/WordPress/openverse-frontend/pull/551
Description

We need to run the E2E test suite in CI to help prevent this.
Additional context

@dhruvkb recommended we look into this package to facilitate this: https://github.com/bahmutov/start-server-and-test
However, it's just occurred to me as I write this that we should run the E2E tests with the wp.org theme wrapper, to most closely replicate the production environment.
Implementation

[ ] 🙋 I would be interested in implementing this feature.

Closing this issue as the #555 and adding more tests after it have added decent E2E check to our CI. We can, however, open a new issue for the new E2E tests that we should write.


fix(pictograms): hide watsonx pictograms from carbon-website

Internal issue here

These three may or may not be replaced by new artwork, which is why we don't want to fully deprecate them just yet.

https://ibm-studios.slack.com/archives/C06110G3J9E/p1696868173709569
To Review

Verify watsonx--ai.svg, watsonx--data.svg, and watsonx--governance.svg do not show on the pictogram index/library

Mirror PR on the design-language-website:

https://github.com/carbon-design-system/design-language-website/pull/1268


Nested uncurrying + paren hugging

As @cristianoc pointed out, nested uncurrying should be possible:
add(. 2)(. 3);

now formats as
add(. 2, . 3);

Also adds test cases/support for uncurried syntax with "paren hugging":
f(. (1, 2));
f(. {
  a: "supersupersupersupersupersuperlong",
  b: "supersupersupersupersupersuperlong",
});

Hummm, not sure we should support add(. 2, . 3)? Should we?

Unfortunately, I think we should.

The behaviour in master is a bug. I'd rather have the parser be "resilient" against this case.
It's consistent with the formatting of add(2)(3) -> add(2, 3)
It's consistent with types where we allow type f = (int, . int) => unit (but here it get's reformatted to type f = int => (.int) => unit;, unfortunately we don't have enough info at print time to do the same with expressions

I don't see this as a super common use case, so I'm happy to see anything that makes the implementation cleaner and/or more consistent.

^ Sounds good then. Merging! Haven't corrected the outcome printer yet.


Mostly working Surf

I think I finally figured it out. Your choice to subtract toi from the Y velocity only at ground tick 1 was messing with later gravity calculations. From my testing, subtracting it every time doesn't seem to hamper bunny hopping acceleration, but let me know if there's a problem.
There are still some very minor details that make surfing here technically different to source engine surfing (e.g. holding forward doesn't pull you down), but I'd say it's pretty close.

Having given this some thought, this is probably still a naive solution, but it does help me narrow things down. From what I can tell, having debugged and stepped though the execution, the problem is still related to the gravity calculation:
$g = gravity \cdot g_e$
$v = v_0 - gt, s_t = s_0 + v_0t - \frac{1}{2} gt^2$
Considering we only care about the player entity, the entity gravity variable $g_e$ doesn't concern us. And you do calculate gravity in relation to deltatime, so $s_t$ is covered. This leads me to believe that our problem might rest with how player movement physics in Half-Life is calculated via the virtual frame rate:
$f_p = \tau_p^{-1}$
Where $\tau_p$ is player frame rate, and $f_p$ is the resulting virtual frame rate used for calculations.


Active Directory Certificate Services New Overview

#assign: @robinharwood, @Xelu86
@robinharwood @Xelu86
Can you review the proposed changes?
NOTE: Can you make sure that the absolute links are made into relative or site-relative links before this is merged? See https://review.learn.microsoft.com/en-us/help/contribute/contribute-best-practices?branch=main#link-responsibly . Thanks.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Some outputs should error 1 not error 0

fmt.Printf errors out with checkout code 0. However, there are some
prompts (this component doens't exist, this application doesn't exist)
that error out with 0, they should error with checkout code 1 (using
fmt.Errorf).
This creates incorrect output for PS1 / odo utils terminal code as it
creates scenarios where there is a false error code output:
if [ $? != 0 ]; then
  APP="Unable to retrieve app"
else
  APP=$(echo $APP_GET | sed "s/The current application is: //g")
fi

After discussion in #549 I'm going to close this (for now), but I'll still keep this branch just-in-case for reference.


Colors for visually impaired

The UI should be switchable to light letters on a BLACK background for the visually impaired.
I cannot even SEE, let alone read a white page with black lettering on it.   Colored pages with dark lettering aren't any better.   I need light GREY fonts (NOT white!  -- white strains what is left of my eyes in any case!).   And I need the maximum contrast possible -- except the maximum brightness of even the letters should be at most a medium-light GREY.
I need these fonts -- the lettering -- on A BLACK BACKGROUND to even function.

Hi @steph3nhayes thanks a lot for opening this issue and starting such a discussion.
I appreciate your detailed explanation as this will help us a lot when working on such a feature. I believe we could implement a "dark" mode that would also be pleasant for you and other users in the same situation.


Provider property allowedAccountIDs are interpreted as numbers in v6

What happened?
Customer reported they have upgraded the provider to 6.0.x and we are seeing this error:
Diagnostics: pulumi:providers:aws (default_6_0_4): error:
rpc error: code = Unknown desc = cannot encode provider configuration to call ValidateProviderConfig:
objectEncoder failed on property "allowed_account_ids":
set encoder failed: encoding element 0 ({5.30YYYYYYYYYe+11}): Expected a string, got: {5.30YYYYYYYYYe+11}

I wrapped this to multiple lines for readability and redacted the real account ID with Y characters.
Expected Behavior
Same behaviour as in v5.
Steps to reproduce
Start with v5 and this as working stack config:
config:
  aws:allowedAccountIds:
    - 0530YYYYYYYY

Upgrade the provider to v6 and the error appears.
Output of pulumi about
CLI
Version      3.82.1
Go Version   go1.21.0
Go Compiler  gc

Plugins
NAME    VERSION
aws     6.0.4
nodejs  unknown

Host
OS       darwin
Version  13.5.2
Arch     arm64

This project is written in nodejs: executable='/Users/ringods/.volta/bin/node' version='v16.19.1'

Current Stack: team-ce/zendesk-4011/ringo

Found no resources associated with team-ce/ringo

Found no pending operations associated with team-ce/ringo

Backend
Name           pulumi.com
URL            https://app.pulumi.com/v-ringo-pulumi-corp
User           v-ringo-pulumi-corp
Organizations  v-ringo-pulumi-corp, team-ce, pulumi

Dependencies:
NAME            VERSION
@pulumi/pulumi  3.82.1
@types/node     16.18.50
@pulumi/aws     6.0.4

Pulumi locates its logs in /var/folders/yq/10j1hf8s1ks9f23yxrdbbbb40000gn/T/ by default

Additional context
Workaround is to quote the values, so they are definitely interpreted as strings:
config:
  aws:allowedAccountIds:
    - "0530YYYYYYYY"

Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

This seems to be coming from our TFPF integration which is indeed new in v6: https://github.com/pulumi/pulumi-terraform-bridge/blob/master/pf/tfbridge/provider_checkconfig.go#L105
cc @t0yv0 as our TFPF master

I didn't check explicitely, but a similar issue might exist for forbiddenAccountIds.

The following program is sufficient to reproduce the problem:
name: repro
runtime: yaml
resources:
  aws:
    type: pulumi:providers:aws
    properties:
      allowedAccountIds:
        - 6YYYYYYYYYYY
      region: us-west-2

I got the same error message as reported.

Is it literally 6YYYYYYYYYYY or some number like<PHONE_NUMBER>90?

It's a number, yaml parses it as a number. The problem is that our SDKv2 based upstream would coerce the number to a string while the PF based upstream won't.

Thanks for that comment and just to make sure the coercion to string does not produce this:
"5.30YYYYYYYYYe+11"
right? But rather "0530YYYYYYYY"?

Good catch. I tested that we coerce without error in AWS, but it turns out that AWS doesn't validate these numbers on preview.
Fixed in https://github.com/pulumi/pulumi-terraform-bridge/pull/1371.


integrated rest docs and postman importer to karate-ui

build is failing, but this has been discussed. some refactoring will be done as well.

@rishabhbitsg thanks again for all your work on this. You may be interested in this link:
https://www.openhub.net/p/731027/contributors/3139739206100372

@ptrthomas thanks. I anyways created an issue ticket with github.

@rishabhbitsg good news, finally your name did start showing up as a contributor !!
now you can get a nice github avatar for added brag value :)


[CONTRIBUTION / PATCH] C API for ZopfliPNG library

I couldn't find any guidelines for contributing to Zopfli, so I have attached
my 3 patches as here on the issue tracker.

Patch #1 introduces build support for building ZopfliPNG as a shared library.
Patch #2 fixes a permissions bug; source files shouldn't be +x.
Patch #3 introduces a C API to libzopflipng.

To elaborate a big, I believe having a C API is beneficial in that it allows
the library to be used from a lot more places and languages (e.g. Go, other C
projects, Ruby, etc.), much easier (without SWIG madness). It also creates a
stable and define ABI for the library, something which C++ lacks.

All patches generated via git format-patch, and apply cleanly to master.

Comments welcome and thanks for your time!

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 7 Nov 2014 at 3:44
Attachments:

0001-build-Allow-building-ZopfliPNG-as-a-shared-library.patch
0002-Don-t-set-the-executable-flag-on-C-source-files.patch
0003-ZopfliPNG-Add-a-C-API.patch

This issue can be closed, since it now exists as:

https://github.com/google/zopfli/pull/48
https://github.com/google/zopfli/pull/49
https://github.com/google/zopfli/pull/50


test resync adding known host

Description

test if host connection issue can be fixed by adding to known host

will fail CI for now since CI is not updated yet.


Remove workaround in cartesian

Will not function correctly until https://github.com/scikit-hep/awkward/pull/2375 is merged and released.
I'll add a new pin to this PR once it is out.
@jpivarski @agoose77

Codecov Report

Merging #222 (8226c54) into main (ea6fc4a) will decrease coverage by 0.02%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #222      +/-   ##
==========================================
- Coverage   94.50%   94.49%   -0.02%
==========================================
  Files          19       19
  Lines        2058     2053       -5
==========================================
- Hits         1945     1940       -5
  Misses        113      113

Impacted Files
Coverage Δ

src/dask_awkward/lib/structure.py
95.38% <ø> (-0.07%)
:arrow_down:

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

@agoose77 have a look, merge when you have time - if there's other old to-do's I can drop I'm happy to add that in as well.


AMI will always upgrade

Describe the Bug
As of version 0.25.0, AMIs are dynamically sourced and will force redeployment of this EC2 instance resource whenever the AMI is found to be updated.
data "aws_ami" "default" {
  most_recent = "true"
...

Expected Behavior
Bastion host should be redeployed when I determine and not the AMI creator.
Additional Context
Perhaps use an optional ignore_changes?
  lifecycle {
    ignore_changes = [
      ami,
    ]
  }

The easy fix is to ignore ami. I think the best method is to use an asg and a launch template to define the ami. If the ami updates, then we can update the launch template which would prevent updating the existing ec2 instance.

actually you can use a filter to pin the ami
https://github.com/cloudposse/terraform-aws-ec2-bastion-server/blob/702027453501ff663d6978e65ce87f58499a3638/main.tf#L17-L23
https://registry.terraform.io/providers/hashicorp/aws/latest/docs/data-sources/ami#filter
https://docs.aws.amazon.com/cli/latest/reference/ec2/describe-images.html#options

image-id - The ID of the image.

module "ec2_bastion" {
  source  = "cloudposse/ec2-bastion-server/aws"
  version = "0.27.0"

  ami_filters = {
    "image-id" = "ami-1234567890"
  }

  instance_type               = var.instance_type
  security_groups             = compact(concat([module.vpc.vpc_default_security_group_id], var.security_groups))
  subnets                     = module.subnets.public_subnet_ids
  key_name                    = module.aws_key_pair.key_name
  user_data                   = var.user_data
  vpc_id                      = module.vpc.vpc_id
  associate_public_ip_address = var.associate_public_ip_address

  context = module.this.context
}


feat: Voice Recognition stops listening after 1 silence second, can i increase it ?

all is title

Hello,
I have the same problem. The google prompt is closing just after 1 sec of silenec. Adding this parameters doesn't seem to have an effect:
intent.putExtra(RecognizerIntent.EXTRA_SPEECH_INPUT_MINIMUM_LENGTH_MILLIS, Long.valueOf(5000));
intent.putExtra(RecognizerIntent.EXTRA_SPEECH_INPUT_POSSIBLY_COMPLETE_SILENCE_LENGTH_MILLIS, Long.valueOf(5000));

For info I am using the google prompt, not the web implementation.

I found a solution that works without the native google prompt. Will propose a PR in the coming days :)

Yes, the listeningState event is firing the the stopped event way too quickly. I need to extend it to know when to submit the data to ai. Not using prompt.

Hi @tobika, I downloaded your fork project, compiled and installed it in my Ionic project.
How can I use the allowForSilence property?
I'm trying to set 60000ms, but after about 20 seconds of silence the microphone automatically closes.
Also, could you tell me how to set the partialResults property? Should it be true or false? Where is it best to collect the results? Adding a listener for partialResults or segmentResults?
Thanks so much!!!!

Have the same problem on android. on iOS the issue doesn't seem to be present

Can confirm I observe the same issue on android. On iOS the issue doesn't seem to be present.

On IOS once clicked it ain't stopping then i'm facing this issue


Can I Override Login Method To Avoid TenantName

I'm planning to develop a Multi Tenant SaaS Application hosted in the could. I'm using ASP.NET BoilerPlate. Can I override _logInManager.LoginAsync to override allowing users to logging in without TenancyName?
I tried navigating through the code, but it seems like, the Abp.Zero Module is bundled and source code is not present in the BoilerPlate Template.

Hi,
Tenancy name is required since business rule is that: a username can be re-used accross different tenants. That means you can create an "admin" user for tenant A and an "admin" user for tenant B. If you don't provide tenancyname on login, how we can know which user you wanted to login (also, login method internally checks if given tenant is active)?
BUT... you don't have to get tenant name from user in the login form. If you are using subdomains to identify tenants (like tenantA.mydomain.com), then don't add tenancy name to login form, but get it from subdomain in the code.

@hikalkan Thanks a lot for your reply. The whole framework of ASP.NET Boilerplate is matching with that of my requirements except this one situation.
In my case, usernames will be same as emails (something which most of the application now follow to ease remembering of usernames) and thus I need to override this business logic to perform login irrespective of tenant. And upon login, the app would return me (and set) the tenant details (to session / token).
So in my case, users would be unique across tenants. Any idea how can I override this?

So, you can do it like that: In the AccountController.Login, first disable IMayHaveTenant filter and query user from IReposiory<User, long>, something like that:
int? tenantId;
CurrentUnitOfWork.DisableFilter(AbpDataFilters.MayHaveTenant)
{
    tenantId = _userRepository.FirstOrDefault(u => u.EmailAddress = input.EmailAddress)?.tenantId;
}

//get tenant from tenant repository to find it's tenancyname (or use ITenantCache)

Then you will have the tenancyName. You can pass it to the LoginManager.
BUT.. remember that this only works if all tenants are in same database. If you use database per tenant approach, you can not execute query accross multiple databases.

@hikalkan That's great!! Yes I can do that. Do you require TenantID or TenantName?
I have two more questions...

Can I remove tenantName property from Login Model? API Users of mine otherwise would get confused with that property around.

Most importatantly, how can I modify the access token? I mean I need to add more return values like TenantID and other initial values on successful login which the apps can store and send me on consecutive requests.

Login method needs tenancyname. You can see it's signature: https://github.com/aspnetboilerplate/module-zero/blob/dev/src/Abp.Zero/Authorization/AbpLoginManager.cs#L114
For the token question; you may want to check this: https://gist.github.com/hikalkan/67469e05475c2d18cb88 (it may be different in your case, but the fundamental idea is to add claims to the cookie/token)

Thanks a lot @hikalkan :)


Assertion doesn'T support dot (.) properly and make it a jsonPath

Not sure if it's a feature or a bug, but I find it not user friendly.
With the following step:
  - name: When consuming output topic, Then the topic contains a json payload.
    url: "kafka-topic:test"
    operation: "consume"
    request:
      consumerLocalConfigs:
        recordType: "JSON"
        commitSync: true
    assertions:
      records:
        - key: "$IS.NOTNULL"
          value:
            '@timestamp': "$IS.NOTNULL"
            '@version': '1'
            host: "$IS.NOTNULL"
            message: "$IS.NOTNULL"
            "SNMPv2-MIB::snmpTrapOID.0": "IF-MIB::linkDown"
            "IF-MIB::ifIndex.1073741824": "1073741824"
            "IF-MIB::ifAdminStatus.1073741824": "3"
            "IF-MIB::ifOperStatus.1073741824": "7"
      size: 1

My assertion record contains fields with dot (.) It seams to confuse the assertion processor because the resulting json path become: $.records[0].value.SNMPv2-MIB::snmpTrapOID.0
I would expect zerocode to properly escape the field name on it own so the json path would be:
$.records[0].value.['SNMPv2-MIB::snmpTrapOID.0']

Hey @ikus060  , we would need to look into it, but meanwhile I would suggest to use validators instead of matches. In a validator us specify jsonpath yourself and you could specify $.records[0].value.['SNMPv2-MIB::snmpTrapOID.0'] there.


Update tr-TR.i18n.json

Closes #ISSUE_NUMBER

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


Add custom messages like jira code

Could you explain better what you mean? Sorry, I cannot get it.

I guess it's like https://github.com/di-sukharev/opencommit/issues/59
to set prefix

I guess it's like https://github.com/di-sukharev/opencommit/issues/59
to set prefix

But on every commit the prefix can be change


Pak patch 2.8

Hi Mario @gronert-m,
I have corrected the master data issue of 2013. In addition, I thought it would be an opportunity to update the directories and comment title format for Pakistan as well. I also noticed that all original variables such as industrycat_orig and occup_orig are not coded correctly as I set some conditions (i.e. only assign values to whose lstatus is 1), which do not maintain the original values in the raw data. Therefore, you can see that for other years I also created new alt versions and, accordingly, folders.
But I do not make any essential changes that would impact the results.
Let me know if you have any questions.

@Junying-T thanks for the upate! Please delete the branch at your earliest convenience.


Add scbc token

A new token named scbc for Smart Car Block Chain.

Incorrect information submitted, please go through our guide and check again.


[SPARK-43974][CONNECT][BUILD] Upgrade buf to v1.23.1

What changes were proposed in this pull request?
The pr aims to upgrade buf from 1.20.0 to 1.23.1
Why are the changes needed?
1.Release Notes:

https://github.com/bufbuild/buf/releases/tag/v1.23.1
https://github.com/bufbuild/buf/releases/tag/v1.23.0
https://github.com/bufbuild/buf/releases/tag/v1.22.0
https://github.com/bufbuild/buf/releases/tag/v1.21.0

2.The new version brings some bug fixed and improvment, as follow:

Fix issue where buf beta graph would not print modules within a workspace that
had no dependencies or dependents.
Fix issue where buf beta graph would print warnings for missing dependencies
that were actually present.
Fix issue where locally-produced images did not have module information if the corresponding
module was stored in the new cache.
Remove buf beta registry template.
Remove buf beta registry plugin {create,deprecate,list,undeprecate,version} and replace with
buf beta registry plugin {push,delete}.
Update buf beta price with the latest pricing information.

3.Manually test: dev/connect-gen-protos.sh, this upgrade will not change the generated files.
Does this PR introduce any user-facing change?
No.
How was this patch tested?

Manually test
Pass GA

In order to address the https://github.com/apache/spark/pull/41933 issue, the above PR has been reversed and is now resubmitted.
@zhengruifeng

Recheck this upgrade will not change the generated files.

merged to master


File Upload w/o CURLFile Class in PHP Version < 5.5.0 don´t work

If the CURLFile Class doesn´t exists, the class_exists call in Curl::buildPostData (Line: 155) produce a  fatal error, cause of the missing 2nd parameter "autoload".

Can you please provide the full text of the fatal error?

Additionaly i can narrow down the issue causes an error only for using a custom autoloader.
Disable the autoload in class_exists call is required in my case.
My error log isn´t representative, i use Prado Framwork...

2016-04-06 15:30:45 Error handler:include_once(CURLFile.php): failed to open stream: No such file or directory, Zeile: 101 in Datei: D:\Zend\Apache2\htdocs\jobportal\framework\PradoBase.php,

It is important to note that include does not fatal if the directory/file is not found, it simply returns false, so include_once(CURLFile.php): failed to open stream is not what is fataling your application.
This is an issue with PradoBase.php, and not with this library.  As you can see, looking at the PradoBase.php source here, Prado is creating the fatal.  Auto-loaders should not throw exceptions or fatal errors, and should let PHP handle the resulting errors internally, otherwise, functions like class_exists (which you would not know if the class exists unless you load it) are no longer useful.  It defeats the purpose of the function, which is to conditionalize behavior on the existence of a class, and to auto load it if it can, not fatal the application.  FURTHER, auto-loaders should be able to stack, with one auto-loader attempting to load a class, and returning true or false and passing the attempt down the line.  Fataling the application breaks this behavior.  This is unacceptable.
This is not an issue with this library, and you should submit a ticket outlining the issue to the Prado repository.  That behavior breaks more things about PHP and class loading than I can list.

Made a ticket for you.

Thanks for your effort.
I still think there´s generally no need to possibly invoke a autoloader function to load PHP supported libcurl Library Class CURLFile (PHP 5 >= 5.5.0, PHP 7) here.

Unless I am missing something, that is what class_exists and auto-loaders are for.  You want to instantiate a class, but you use class_exists to see if you can so you do not fatal the application.  Using class_exists should not fatal the application on it's own.  Many frameworks do not implicitly load any class definitions, save for the auto-loaders, at all.
There are two options:

Expect it exists, is loaded, or can be loaded, and simply instantiate it $obj = new Class - This will fatal for people who do not have the class.
Check without auto-loading - This will disable it for anyone who relies on auto-loaders.

As the documented, default, and expected behavior for class_exists is to attempt to auto-load the class, I don't see why we should change the behavior.
However, ultimately, it is up to @zachborboa, I guess.

@aisn Sometimes it pays to be the squeeky wheel.  This will be resolved in Prado 3.3.1 in a few days.
https://github.com/pradosoft/prado/pull/583

@aisn Has this issue been resolved?
For php < 5.5.0 when CURLFile class doesn't exists, the library will use the old @filename file uploading.

Closing as this seems to no longer be an issue. Please comment if the issue persists. Thanks


ROX-20320: remove Clair v2 integration

Description
Clair v2 integration has been deprecated for several releases, so it is time to finally remove it.
Checklist

[x] Investigated and inspected CI test results
~[ ] Unit test and regression tests added~
[x] Evaluated and added CHANGELOG entry if required
~[ ] Determined and documented upgrade steps~
~[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)~

Testing Performed
Check the UI, and you'll see it is gone.
Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

/retest

/retest

Are we confident that removing Clair from the storage protos will not cause issues if central-db has integrations still referencing clair? Is a migration needed to cleanup any lingering integrations from DB?

good question. I'll request a review from someone on Core Workflows


ovn-k8s-cni-overlay: Update Windows CNI and remove the workarounds

This commit removes the Windows specific workarounds and makes it
look more similar to the Linux code.
The endpoint name should not depend on the container details passed
to the CNI (e.g. container ID). This is because kubelet calls
at the moment the CNI for every container within a pod when it should
be called only for the sandbox container.
Signed-off-by: Alin Balutoiu<EMAIL_ADDRESS>

/lgtm

Update: Forgot to remove the InitialPlatformCheck function from helpers_linux.go since it's no longer used.


robots/constraints docs

What type of change is this?

[x] Doc update,

Checklist

[x] I ran all tests on my computer and it's all green (i.e. invoke test).
[x] I ran lint on my computer and there are no errors (i.e. invoke lint) .

Notes
I'm hosting the sphinx docs generated from these file changes to make reviewing easier.
I took the liberty to rename type in the Constraint classes __init__ to make it clear that it's not the built-in used. Did not change the attribute name though.

Build fails on Windows Python 2.7. It fails running $ pip3 install --no-cache-dir -r requirements-dev.txt || pip install --no-cache-dir -r requirements-dev.txt, while building llvm-lite.
Last build it downloaded and installed a wheel, but this time it couldn't find one.

llvmlite seems to have dropped python 2 support. It is pulled in by numba. I think we need to pin numba in compas to fix this.
Ping @gonzalocasas and @brgcode.


File reference in image component

…the path in crx of the image.......]

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

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

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes and the overall coverage did not decrease.
[ ] All unit tests pass on CircleCi.
[ ] I ran all tests locally and they pass.

Closing this since @deepprakash345 would take this change up