CUDA libs not installed in Torch CUDA-enabled Docker

Hello dear UV maintainers,
I faced a mismatch in the installed versions of packages in the Docker environment which is affecting it to work properly with CUDA payloads (I used Vertex AI A100 accelerated jobs specifically). At the same time, PIP was able to resolve dependencies as expected and the produced image was ready to use in the CUDA environment.

Thank you!

On your first example, what the output if you use uv pip freeze instead of pip freeze?
> docker run -it pytorch/pytorch:2.2.2-cuda12.1-cudnn8-runtime bash
> pip install uv
> uv pip install "torch>=2.2.0" datasets transformers peft "bitsandbytes==0.41.0" scipy --verbose --system
-> pip freeze
+> uv pip freeze

@samypr100 , if I do uv pip freeze the mentioned packages are visible, but still not usable from python scripts doing some CUDA-enabled codes.

Sure @samypr100 , if you run the following code inside a docker configured with uv as I described above you will hit an error. Important to note that you would need a proper CUDA GPU available in that docker, other wise you will see a No GPU found. error, which is nothing to do with the initially reported behaviour.
Code to reproduce:
from transformers import AutoModelForCausalLM, BitsAndBytesConfig
import torch

bnb_config = BitsAndBytesConfig(
        load_in_8bit=True,
        bnb_4bit_use_double_quant=True,
        bnb_4bit_quant_type="nf4",
        bnb_4bit_compute_dtype=torch.bfloat16,
    )

model = AutoModelForCausalLM.from_pretrained(
    "mistralai/Mistral-7B-v0.1", quantization_config=bnb_config, device_map="auto"
)

Error you will see:
ImportError: Using `bitsandbytes` 8-bit quantization requires Accelerate: `pip install accelerate` and the latest version of bitsandbytes: `pip install -i https://pypi.org/simple/ bitsandbytes`

On a clean pytorch/pytorch:2.2.2-cuda12.1-cudnn8-runtime  image (no uv, or anything), what's your output of
#> python
>>> import torch
>>> torch.cuda.is_available()

Are you missing --gpus all on your docker run command by any chance?

@samypr100 ,
On a clean setup I get torch.cuda.is_available() -> True.
The reason, why I gave my examples as just docker run ... without the gpus flag is the fact, that the GPUs are not available to the docker build and this is how we prepared the images for Vertex train jobs. But afterward when the Vertex training is running proper GPU flags are passed, they use the Nvidia Container Toolkit for that, afaik. The part about how images are spun up is out of our control, once we pass it down to Vertex, but there is a clear difference in how the uv built image behaves vs the pip built image, thus this issue was raised.
Hope this clarifies a bit the background and the context.
Let me know, if I can give you more info - would be happy to do it!

Thanks, I think we'll likely need a more specific reproduceable example to figure out what the exact issue is. I gave your example a try and was able to run it with uv and with pip. Do note that without --gpus all it would fail on either uv or pip with the error you showed.

Thanks for checking, sure thing!
Here are 2 dockerfiles:
pip.Dockerfile
FROM pytorch/pytorch:2.2.2-cuda12.1-cudnn8-runtime
RUN pip install "torch>=2.2.0" datasets transformers peft "bitsandbytes==0.41.0" scipy

uv.Dockerfile
FROM pytorch/pytorch:2.2.2-cuda12.1-cudnn8-runtime
RUN pip install uv
RUN uv pip install "torch>=2.2.0" datasets transformers peft "bitsandbytes==0.41.0" scipy --verbose --system

Can you do:

docker build -t uv-uv-issue . -f uv.Dockerfile
docker build -t pip-uv-issue . -f pip.Dockerfile
docker run -it --rm pip-uv-issue --gpus all bash - run my example here, there should be no issues
docker run -it --rm uv-uv-issue --gpus all bash - run my example here - you should see the error

I bet that it worked for you because you did a docker run with GPUs and then installed libs it inside, but as I said no GPUs will be available on build.

Thanks
I was able to reproduce.
The core issue is that the original image had its pre-installed dependencies built with gpu support (gpu's were attached at built time).
When uv ends up reinstalling some of them (and given that you don't have gpu's attached at build time), the produced build and linking process ends up being inherently different. This causes the issues later when you try to run with --gpus all since some of those dependencies are not built with gpu support anymore.
This is definitely related to https://github.com/astral-sh/uv/issues/2841 as @StellarStorm pointed out and would only be resolved if uv didn't re-install these system site packages (e.g. torch, nvidia-*) when they're already pre-installed.
Some follow on investigation is still needed to determine why uv is reinstalling them.

Thanks @samypr100. Do you know where they're installed? Maybe on sys.path but not in any of the site-packages directories?

Noticed that torch torchaudio and torchvision are pre-installed as eggs which might explain the root of the problem.
Here's the PKG-INFO for torch (/opt/conda/lib/python3.10/site-packages/torch-2.2.2-py3.10.egg-info/PKG-INFO)

Metadata-Version: 2.1
Name: torch
Version: 2.2.2
Summary: Tensors and Dynamic neural networks in Python with strong GPU acceleration
Home-page: https://pytorch.org/
Download-URL: https://github.com/pytorch/pytorch/tags
Author: PyTorch Team
Author-email<EMAIL_ADDRESS>License: BSD-3
Keywords: pytorch,machine learning
Classifier: Development Status :: 5 - Production/Stable
Classifier: Intended Audience :: Developers
Classifier: Intended Audience :: Education
Classifier: Intended Audience :: Science/Research
Classifier: License :: OSI Approved :: BSD License
Classifier: Topic :: Scientific/Engineering
Classifier: Topic :: Scientific/Engineering :: Mathematics
Classifier: Topic :: Scientific/Engineering :: Artificial Intelligence
Classifier: Topic :: Software Development
Classifier: Topic :: Software Development :: Libraries
Classifier: Topic :: Software Development :: Libraries :: Python Modules
Classifier: Programming Language :: C++
Classifier: Programming Language :: Python :: 3
Classifier: Programming Language :: Python :: 3.8
Classifier: Programming Language :: Python :: 3.9
Classifier: Programming Language :: Python :: 3.10
Classifier: Programming Language :: Python :: 3.11
Classifier: Programming Language :: Python :: 3.12
Requires-Python: >=3.8.0
Description-Content-Type: text/markdown
License-File: LICENSE
License-File: NOTICE
Requires-Dist: filelock
Requires-Dist: typing-extensions>=4.8.0
Requires-Dist: sympy
Requires-Dist: networkx
Requires-Dist: jinja2
Requires-Dist: fsspec
Provides-Extra: optree
Requires-Dist: optree>=0.9.1; extra == "optree"
Provides-Extra: opt-einsum
Requires-Dist: opt-einsum>=3.3; extra == "opt-einsum"

Here's all the eggs in site-packages /opt/conda/lib/python3.10/site-packages
jsonpointer-2.1-py3.6.egg-info
mkl_service-2.4.0-py3.10.egg-info
pip-23.3.1-py3.10.egg-info
setuptools-68.2.2-py3.10.egg-info
torch-2.2.2-py3.10.egg-info
torchaudio-2.2.2-py3.10.egg-info
torchvision-0.17.2-py3.10.egg-info
triton-2.2.0-py3.10.egg-info

For example, after uv re-installs "torch>=2.2.0", you get all the proper .dist-info versions
torch-2.2.2.dist-info
nvidia_cublas_cu12-<IP_ADDRESS>.dist-info
nvidia_cuda_cupti_cu12-12.1.105.dist-info
nvidia_cuda_nvrtc_cu12-12.1.105.dist-info
nvidia_cuda_runtime_cu12-12.1.105.dist-info
nvidia_cudnn_cu12-<IP_ADDRESS>.dist-info
nvidia_cufft_cu12-<IP_ADDRESS>.dist-info
nvidia_curand_cu12-<IP_ADDRESS>.dist-info
nvidia_cusolver_cu12-<IP_ADDRESS>.dist-info
nvidia_cusparse_cu12-<IP_ADDRESS>.dist-info
nvidia_nccl_cu12-2.19.3.dist-info
nvidia_nvjitlink_cu12-12.4.127.dist-info
nvidia_nvtx_cu12-12.1.105.dist-info

and here's the contents of METADATA for torch-2.2.2 (which has the Requires-Dist for nvidia dependencies)

Metadata-Version: 2.1
Name: torch
Version: 2.2.2
Summary: Tensors and Dynamic neural networks in Python with strong GPU acceleration
Home-page: https://pytorch.org/
Author: PyTorch Team
Author-email<EMAIL_ADDRESS>License: BSD-3
Download-URL: https://github.com/pytorch/pytorch/tags
Keywords: pytorch,machine learning
Platform: UNKNOWN
Classifier: Development Status :: 5 - Production/Stable
Classifier: Intended Audience :: Developers
Classifier: Intended Audience :: Education
Classifier: Intended Audience :: Science/Research
Classifier: License :: OSI Approved :: BSD License
Classifier: Topic :: Scientific/Engineering
Classifier: Topic :: Scientific/Engineering :: Mathematics
Classifier: Topic :: Scientific/Engineering :: Artificial Intelligence
Classifier: Topic :: Software Development
Classifier: Topic :: Software Development :: Libraries
Classifier: Topic :: Software Development :: Libraries :: Python Modules
Classifier: Programming Language :: C++
Classifier: Programming Language :: Python :: 3
Classifier: Programming Language :: Python :: 3.8
Classifier: Programming Language :: Python :: 3.9
Classifier: Programming Language :: Python :: 3.10
Classifier: Programming Language :: Python :: 3.11
Classifier: Programming Language :: Python :: 3.12
Requires-Python: >=3.8.0
Description-Content-Type: text/markdown
License-File: LICENSE
License-File: NOTICE
Requires-Dist: filelock
Requires-Dist: typing-extensions (>=4.8.0)
Requires-Dist: sympy
Requires-Dist: networkx
Requires-Dist: jinja2
Requires-Dist: fsspec
Requires-Dist: nvidia-cuda-nvrtc-cu12 (==12.1.105) ; platform_system == "Linux" and platform_machine == "x86_64"
Requires-Dist: nvidia-cuda-runtime-cu12 (==12.1.105) ; platform_system == "Linux" and platform_machine == "x86_64"
Requires-Dist: nvidia-cuda-cupti-cu12 (==12.1.105) ; platform_system == "Linux" and platform_machine == "x86_64"
Requires-Dist: nvidia-cudnn-cu12 (==<IP_ADDRESS>) ; platform_system == "Linux" and platform_machine == "x86_64"
Requires-Dist: nvidia-cublas-cu12 (==<IP_ADDRESS>) ; platform_system == "Linux" and platform_machine == "x86_64"
Requires-Dist: nvidia-cufft-cu12 (==<IP_ADDRESS>) ; platform_system == "Linux" and platform_machine == "x86_64"
Requires-Dist: nvidia-curand-cu12 (==<IP_ADDRESS>) ; platform_system == "Linux" and platform_machine == "x86_64"
Requires-Dist: nvidia-cusolver-cu12 (==<IP_ADDRESS>) ; platform_system == "Linux" and platform_machine == "x86_64"
Requires-Dist: nvidia-cusparse-cu12 (==<IP_ADDRESS>) ; platform_system == "Linux" and platform_machine == "x86_64"
Requires-Dist: nvidia-nccl-cu12 (==2.19.3) ; platform_system == "Linux" and platform_machine == "x86_64"
Requires-Dist: nvidia-nvtx-cu12 (==12.1.105) ; platform_system == "Linux" and platform_machine == "x86_64"
Requires-Dist: triton (==2.2.0) ; platform_system == "Linux" and platform_machine == "x86_64" and python_version < "3.12"
Provides-Extra: opt-einsum
Requires-Dist: opt-einsum (>=3.3) ; extra == 'opt-einsum'
Provides-Extra: optree
Requires-Dist: optree (>=0.9.1) ; extra == 'optree'

Overall, seems like it still installs them in /opt/conda/lib/python3.10/site-packages as expected so no issues there, below output for sys.path seems pretty standard, except that everything is prefixed with /opt/conda instead of the traditional locations, e.g. /usr/local
>>> sys.path
['', '/opt/conda/lib/python310.zip', '/opt/conda/lib/python3.10', '/opt/conda/lib/python3.10/lib-dynload', '/opt/conda/lib/python3.10/site-packages']
>>> site.getsitepackages()
['/opt/conda/lib/python3.10/site-packages']
>>> site.getusersitepackages()
'/root/.local/lib/python3.10/site-packages'

Here's the output of get_interpreter_info on this image -everything seems prefixed correctly to /opt/conda from what I can tell.

{
    "result": "success",
    "markers": {
        "implementation_name": "cpython",
        "implementation_version": "3.10.14",
        "os_name": "posix",
        "platform_machine": "x86_64",
        "platform_python_implementation": "CPython",
        "platform_release": "<IP_ADDRESS>-microsoft-standard-WSL2",
        "platform_system": "Linux",
        "platform_version": "#1 SMP Thu Jan 11 04:09:03 UTC 2024",
        "python_full_version": "3.10.14",
        "python_version": "3.10",
        "sys_platform": "linux"
    },
    "base_prefix": "/opt/conda",
    "base_exec_prefix": "/opt/conda",
    "prefix": "/opt/conda",
    "base_executable": "/opt/conda/bin/python",
    "sys_executable": "/opt/conda/bin/python",
    "stdlib": "/opt/conda/lib/python3.10",
    "scheme": {
        "platlib": "/opt/conda/lib/python3.10/site-packages",
        "purelib": "/opt/conda/lib/python3.10/site-packages",
        "include": "/opt/conda/include/python3.10",
        "scripts": "/opt/conda/bin",
        "data": "/opt/conda"
    },
    "virtualenv": {
        "purelib": "lib/python3.10/site-packages",
        "platlib": "lib/python3.10/site-packages",
        "include": "include/site/python3.10",
        "scripts": "bin",
        "data": ""
    },
    "platform": {
        "os": {
            "name": "manylinux",
            "major": 2,
            "minor": 35
        },
        "arch": "x86_64"
    },
    "gil_disabled": false
}

Ah interesting. Yeah we don't support eggs right now. We weren't planning on supporting them at all. (I don't know any of the details on how the format works.)

torchelastic is not preinstalled an egg, which explains https://github.com/astral-sh/uv/issues/2841

@StellarStorm @avishniakov The one alternative I can think of is seeing if upstream maintainers for pytorch/pytorch:2.2.2-cuda12.1-cudnn8-runtime, and related images can be updated to avoid using egg installs. That should make the images uv compatible.

I can briefly look into what it would take to support pre-installed eggs. (I.e., recognizing that they’re present, and knowing how to uninstall them.)

I think it could make sense to at least support detecting that eggs are installed.

Makes sense to at least detect and maybe issue a warning. With the deprecation of eggs via PEP-0715, we'll soon have an egg-less future 🥚

@StellarStorm @avishniakov The one alternative I can think of is seeing if upstream maintainers for pytorch/pytorch:2.2.2-cuda12.1-cudnn8-runtime, and related images can update them to avoid using egg installs. That should make the images uv compatible.

It's possible a small update to https://github.com/pytorch/pytorch/blob/main/Dockerfile to use python -m pip install . instead of python setup.py install might do the trick.

Uninstalling them looks a little tricky though. I can't even figure out how to build and install an egg locally since easy_install is long deprecated.

I figured out how to build and install a .egg, but any idea how you get a .egg-info as above...?

The best way I know of is making a dummy python project with a setup.py and running python setup.py egg_info or python setup.py install
Given a dummy egg_test python module/project
egg_test
 ├── setup.py
 └── egg_test
      └── __init__.py # empty

and setup.py having this
from setuptools import setup, find_packages

setup(
    name='egg_test',
    version='0.1',
    description='Lorem Ipsum',
    long_description='Lorem Ipsum',
    author='Lorem Ipsum',
<EMAIL_ADDRESS>    packages=find_packages(),
    install_requires=[
        "flask"
    ],
    classifiers=[
        'License :: OSI Approved :: MIT License',
        'Programming Language :: Python :: 3'
        'Programming Language :: Python :: 3.11'
    ],
)

You can run python setup.py egg_info to see what .egg_info structure looks like

Seems like the implementation for generating .egg_info directories is here https://github.com/pypa/setuptools/blob/main/setuptools/command/egg_info.py

Put up a PR to fix this: https://github.com/astral-sh/uv/pull/3380

Thank you!

@avishniakov Would you be able to confirm 0.1.40 fixes your reported issue?

@avishniakov Would you be able to confirm 0.1.40 fixes your reported issue?

Hey @samypr100 , sorry about the delay! I checked that in the original settings as it was reported and the issue is resolved now. Thanks for your support!


Error when building from source Fedora 27 CUDA 9.1

System information

OS Platform and Distribution: Fedora 27
TensorFlow installed from (source or binary): binary
TensorFlow version: r1.4
Python version: 3.6.3
Bazel version: 0.8.1
GCC/Compiler version: 7.2.1
CUDA/cuDNN version: CUDA 9.1 cuDNN 7.0.5
GPU model and memory: NVidia Geforce GTX 960 4GB
Exact command to reproduce: bazel build -c opt --config=cuda --verbose_failures //tensorflow/tools/pip_package:build_pip_package

Describe the problem
So I'm attempting to build tensorflow from source on fedora with the version of CUDA and cuDNN I already had installed to avoid have to also install an older version. The build however errors with the following message:
ERROR: .cache/bazel/_bazel_xd009642/f9f5dea1a139b69420e1045d339dda45/external/nccl_archive/BUILD:33:1: error while parsing .d file: /home/xd009642/.cache/bazel/_bazel_xd009642/f9f5dea1a139b69420e1045d339dda45/execroot/org_tensorflow/bazel-out/k8-py3-opt/bin/external/nccl_archive/_objs/nccl/external/nccl_archive/src/all_reduce.cu.pic.d (No such file or directory)
<command-line>:0:15: warning: ISO C++11 requires whitespace after the macro name
<command-line>:0:1: error: macro names must be identifiers
Target //tensorflow/tools/pip_package:build_pip_package failed to build
INFO: Elapsed time: 0.392s, Critical Path: 0.12s
FAILED: Build did NOT complete successfully

I also tried the command bazel build --config=opt --config=cuda --incompatible_load_argument_is_label=false //tensorflow/tools/pip_package:build_pip_package from here with the same end result.
Any guidance is appreciated as this is my first time using bazel (and also trying to compile tensorflow).

I had the same issue with multiple sources. Being a minimalist, Instead of using negativo's repositories (nothing against them, I think they are great and convenient!) Here is what I did on Fedora 27 after installing all the normal development packages included in epel-release and so forth:

Compiled Video card drivers from source using NVIDIA-Linux-x86_64-390.48.run file provided by NVidia..

Compiled the Cuda using cuda_9.1.85_387.26_linux.run. as well as declined to install driver for GPU. patched using the three run files, cuda_<IP_ADDRESS>_linux.run cuda_<IP_ADDRESS>_linux.run cuda_<IP_ADDRESS>_linux.run.

Created two files within the /etc/profile.d/

touch /etc/profile.d/cuda.sh
echo "PATH=$PATH:/usr/local/cuda/bin" >> /etc/profile.d/cuda.sh
echo "export PATH" >> /etc/profile.d/cuda.sh
touch /etc/profile.d/cuda.csh
echo "set path = ( $path /usr/local/cuda/bin )" >> /etc/profile.d/cuda.csh

Installed gcc63 from package - More than likely other versions will function as well.
Created the following two symbolic links:

/usr/local/cuda/bin/gcc -> /usr/bin/gcc63
/usr/local/cuda/bin/g++ -> /usr/bin/g++63

Took the advice from "xd009642"  Patch cuda/include/host_defines.h
echo "#define _BITS_FLOATN_H" >> /usr/local/cuda/include/host_defines.h

This resolved all compiling issues with NVCC for me so far. I understand many gcc versions are not supported but it is likely that many will work as well.


Add RFC numbers & links where appropriate

Local root (and maybe local TLD?)

RFC8806

Verify root zone with ZONEMD

RFC8976

EDNS0 sizes to minimize fragmentation, especially for IPv6
Aggressive NSEC caching

RFC8189

QNAME minimization

RFC7816

Negative trust anchors

RFC7646

TTL record pre-fetch
EDNS client subnet

RFC7871

DNS cookies shared secret

RFC7873

DoT

RFC7858

DoH

RFC8484

DoQ

RFC9250

Trust anchor reporting
DNS error reporting

https://datatracker.ietf.org/doc/draft-ietf-dnsop-dns-error-reporting

I think we're good for this one. :smile:


TwigException: Unable to parse 'macro test(param = '')'

Hello
Twig.js is unable to parse a macro with default value for a parameter.
{% macro test(param = '') %} test {% endmacro %}

It’s working fine with Twig php.

Macros differ from native PHP functions in a few ways:

Default argument values are defined by using the default filter in the macro body;
Arguments of a macro are always optional.
If extra positional arguments are passed to a macro, they end up in the special varargs variable as a list of values.

https://twig.symfony.com/doc/2.x/tags/macro.html
So:
{% macro test(param) %} test {{ param|default('') }} {% endmacro %}

But ryuran is right that despite what's included in the Twig documentation, macros do accept defaults:
{% macro test(param = 2) %} {{param}} {% endmacro %}

{% import _self as macros %}

{{ macros.test() }}

returns 2
https://twigfiddle.com/dw63fe

Yes it looks like undocument feture
https://github.com/twigphp/Twig/blob/9402071768157c115ebf35195661222bd7ba85cf/lib/Twig/Node/Macro.php#L43

And it's tested by Twig integration test suite: https://github.com/twigphp/Twig/blob/2.x/test/Twig/Tests/Fixtures/macros/default_values.test

Closed by #544

Does this still work in 2022? Was this implemented correctly in #544? Because I'm still getting a parse error with default arguments but can write a macro with only positional arguments


Feature Request: enforce non-empty memo when using send-many-memo

https://github.com/blockstack/send-many-stx-cli/#stx-bulk-transfer-send-many-memo-recipients
if two tx's are added with no memo, they are combined into a single tx - adding a requirement that memo not be empty, or if 2 tx's have non-empty memos, they shouldn't be combined.
for example:
https://explorer.stacks.co/txid/0x11fdb3b129ac439adf8dd5b3f27f749ad48c4ea9339060965d7916064329e237?chain=mainnet
in the contract, there is a tx for STX tokens(5049.3925) was exactly the same as the sum of missing tokens for 2 addresses ( 331.3772 + 4718.0153 = 5049.3925 )
I think it would be more enforceable from the code to check for this type of behaviour, rather than rely on the user not doing this.
I will admit that adding this is debatable and might not be what we want, but we should have a discussion at least.

It turned out that the sender aggregated the transfers and combined two memo-less transactions before passing them to send-many-stx-cli. A non-empty memo could be enforced but it would not impact that particular issue.


Bindings for views (or xstrided_views)

I was trying to get an array view from cpp and call a python function back with it.
After some help, I managed to do it by converting the view to a xt::pytensor before proceeding with the Python call (See code below).
void cpp_f(xt::pytensor<double, 2> my_arr, py::object py_f) {
	auto v = xt::pytensor<double, 1>(xt::view(my_arr, 0, xt::all()));
	py_f(v);
}

def test_view():
    def py_f(v):
        print(v)
    cpp_f(np.array([ [1,2,3], [4,5,6], ]), py_f)

It would be nice to have some way to pass the view directly to the python function, instead of having to convert them. It seems that it is not possible to make bindings for xt::view directly, though. I'll leave the original discussion here for reference:
Tarcísio Fischer @tarcisiofischer 14:49
Do you know if there'll be bindings for the xt::view anytime soon?

Johan Mabille @JohanMabille 14:50
the thing is numpy view and xtensor views are not exactly the same so we cannot bind view directly

Sylvain Corlay @SylvainCorlay 14:50
Not as a numpy array, but there is some hope for strided views on pyarrays / pytensor.

Johan Mabille @JohanMabille 14:50
we have xstrided_views that works like numpy views, we can make bindings for them

With some of the recent additions this is definitly possible (at least for "easy" views that don't use the new islice), and for the strided view.

I am interested in working on this.  I wonder if there's a pointer (like a file or a PR) about the "easy" views that don't use the new islice?
My project needs to share large buffers among C++ and Python.  It seems that, without the requested feature, xtensor-python always copies the buffer when an array is passed across the wrapper.
c = xt.C()
self.assertEqual([0.]*10, c.array.tolist())
# use __setitem__ implemented in C++ to set the element.
c[0] = 3
self.assertEqual([3.] + [0.]*9, c.array.tolist())
# c.array returns a copy while I want a view.
arr = c.array
arr[1] = 2
# FIXME: this assertion fails
self.assertEqual([3., 1.] + [0.]*8, c.array.tolist())

The view mentioned here is the strided view. The idea would be to have a similar class in xtensor-python that embeds a xensor-python container and the python objetc for view


Defer loading textdomain after plugins

Currently the main textdomain is enforced before plugins load. This prevents user plugins from altering the global setting YOURLS_LANG as defined in the DB.
I think there would be no unwanted side effect if we loaded the language after plugins ?

This is actually very simple, if the webmaster has set the language in the configuration file, then the configuration file is the main, if not, try the language to let the plug-in load


Callout personalisation

Hi,
Thank you for a very useful plugin. Since there is the possibility of personalizing the prompt, it would be ideal to be able to customize the "callout" to change the title.

I appreciate your kinds words 🙏
Sure that shouldn't be too hard to add! ill put it on the punchlist and plan to release it in the next update

@ProfSynapse also can you add customisation for expanded by default, rather than collapsed.

sure will add it to the list


javascript in attribute

Logical AND (&&) in attributes are incorrectly minified:
>>> import htmlmin
>>> htmlmin.minify(u"

")
u'

'

But it works with logical OR:
>>> import htmlmin
>>> htmlmin.minify(u"

")
u'

'

Thanks !

There might be another issue, in that it removes the quotes, I just noticed. Taking one more look.


Does this server work with an AspNetCore app built against .NET Framework?

I am trying to get the sample working with an ASP.NET Core app that targets .NET Framework. However, the sample code does does not build because it can't find the extension UseHttpSys(). Is this a supported configuration? If so, what package must be added to get HttpSys?

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

ID: 5e347c54-6a6a-d685-4af7-39b023bf7f15
Version Independent ID: 41aa77f5-9466-6580-5ec6-b1bf97bd0f01
Content: HTTP.sys web server implementation in ASP.NET Core
Content Source: aspnetcore/fundamentals/servers/httpsys.md
Product: aspnet-core
GitHub Login: @guardrex
Microsoft Alias: tdykstra

Hello @madelson ... In theory (I haven't tried it myself) according to engineering, the answer is Yes! From Item 4 under Configure Windows Server ...

If the app is a framework-dependent deployment, install .NET Core, .NET Framework, or both (if the app is a .NET Core app targeting the .NET Framework).

.NET Core – ...
.NET Framework – If the app requires .NET Framework, see the .NET Framework installation guide. Install the required .NET Framework. The installer for the latest .NET Framework is available from from the .NET Core Downloads page.

Since the Microsoft.AspNetCore.App package is for ASP.NET Core only, you must reference individual packages in the project file when targeting the .NET Framework.
We don't provide troubleshooting advice here ... we just work on the docs. We recommend consulting with devs on a support forum, such as Stack Overflow, or a support chat, such as Slack or Gitter. Of course, MS Support will help if you have a support agreement.
Note that for 3.0+, this scenario won't be supported. See: https://github.com/aspnet/Announcements/issues/324


error: no type named 'basic_string_view' in namespace 'std'

Perhaps, it's just my system, but i got this error when compiling an example:
xx $ c++ -std=c++14 -I include example/quick_examples.cpp
In file included from example/quick_examples.cpp:13:
In file included from include/boost/pfr.hpp:12:
In file included from include/boost/pfr/precise.hpp:14:
In file included from include/boost/pfr/precise/ops.hpp:15:
In file included from include/boost/pfr/precise/io.hpp:17:
include/boost/pfr/detail/io.hpp:32:16: error: no type named 'basic_string_view' in namespace 'std'
    const std::basic_string_view<CharT, Traits>& s) noexcept {
          ~~~~~^
include/boost/pfr/detail/io.hpp:32:33: error: expected ')'
    const std::basic_string_view<CharT, Traits>& s) noexcept {
                                ^
include/boost/pfr/detail/io.hpp:31:26: note: to match this '('
inline auto quoted_helper(
                         ^
include/boost/pfr/detail/io.hpp:33:22: error: use of undeclared identifier 's'
  return std::quoted(s);
                     ^
3 errors generated.
xx $

So, i edited include/boost/pfr/detail/io.hpp to remove the basic_string_view declaration and just included the string_view header:
// Copyright (c) 2016-2017 Antony Polukhin
//
// Distributed under the Boost Software License, Version 1.0. (See accompanying
// file LICENSE_1_0.txt or copy at http://www.boost.org/LICENSE_1_0.txt)

#ifndef BOOST_PFR_DETAIL_IO_HPP
#define BOOST_PFR_DETAIL_IO_HPP
#pragma once

#include <boost/pfr/detail/config.hpp>

#include <boost/pfr/detail/sequence_tuple.hpp>
#include <iomanip>
#include <iosfwd>  // stream operators

// Forward declaration
// namespace std {
//   template <class CharT, class Traits> class basic_string_view;
// }
#include <string_view> // addded
...

The the example compiled and worked. thanks.
macOS 10.13.4 Beta (17E150g)
Apple LLVM version 9.0.0 (clang-9<IP_ADDRESS>)
Target: x86_64-apple-darwin17.5.0

Classes from the standard library should not be forward-declared anyway. Doing so is undefined behavior. See for example https://stackoverflow.com/a/307408
The right way is to include the appropriate header.

Many thanks for the report!


Export the egl* symbols via linker script

The egl symbols should be exported so applications
using it can correctly run with RenderDoc.

Sorry about that, I forgot that it's technically possible to use EGL on linux too as well as android.


Add a Collection response

The current Framework distinguishes between different informational responses and the ResultSet response.
However, ResultSet is assuming that the return are items inside a collection, hence it includes attributes like the collection type or collection id.
Obviously, that approach doesn't fit when the results returned are, actually, a set of collections (datasets or cohorts, currently).
Therefore, I suggest to add a beaconCollectionResponse that is aimed to return a list of collections (complete or that match a filtering criteria).
A PR would be added as soon as it is prepared.

Solved in commit ab2798b


Running synchronous methods on UserManager means UserStore methods have no access to HttpContext.Current

I've built my own simple UserManager and SigninManager with a custom store storing accounts to a json file.
While testing, I came across something peculiar - in the end, the demonstrator serves as a template for a bigger endeavor where the IUserStore is based on a REST service, and to make remote calls, I need access to HttpContext.Current inside the IUserStore implementation.
Yet every so often, HttpContext.Current would be null. And I finally found the culprit: whenever I would be calling UserManager.DoSomething() from one of my controllers, the corresponding calls in my IUserStore implementation would have HttpContext.Current empty. If I instead called UserManager.DoSometingAsync(), HttpContext.Current would have a value.
It almost seems as if the sync methods simply wrap the async methods but run into a different context and in that different context, HttpContext.Current is not available.
To demonstrate this, I created a sample project. It's your standard MVC5 project where I just plugged in my own ASP.NET Identity store (all files are in the UserManagement folder). To see the problem, set three breakpoints in CustomUserStore.cs on lines 39 (CreateAsync method), 63 (FindByIdAsync) and 69 (FindByNameAsync) that all hit if ctx == null.
Run the project, and create a new account. You'll see breakpoints in FindByNameAsync and CreateAsync will actually break. This because I changed AccountController.Register to create the user synchronously. Revert back to asynchronous, and that problem goes away.
Another instance to see the problem is when you change a logged in user's password. Click on the link to manage your account, and the call to HasPassword() in ManageController.Index will trigger a breakpoint hit in CustomUserStore.FindByIdAsync. Replace HasPassword() with HasPasswordAsync() and the problem goes away.
So, now I'm left wondering whether there's an error in my implementation, or if the user manager does something iffy when running the synchronous methods. Given that IUserStore only demands asynchronous methods, I'd imagine that UserManager has some kind of logic to run the async methods in the IUserStore that makes the methods being executed on a different SynchronizationContext that does not have access to the usual ASP.NET properties (just like HttpContext.Current).
My sample project can be found here

This repo is only for Identity on AspNetCore, you appear to be using the older versions of aspnet/identity in your sample, feel free to file this issue over at https://aspnetidentity.codeplex.com/workitem/list/basic
But in general yes, async can cause issues with HttpContext.Current if it runs on a different thread.
https://stackoverflow.com/questions/19111218/httpcontext-current-null-inside-async-task


Move device settings out of robot settings

To share device settings across robots, separate device settings from robot settings.

woot!


handle no_data in gdal_source

 GDALSource setzt kein NO DATA

Pull Request Test Coverage Report for Build 609863953

6 of 9 (66.67%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.002%) to 82.075%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

operators/src/source/gdal_source.rs
6
9
66.67%

Totals

Change from base Build 603333936:
-0.002%

Covered Lines:
14616

Relevant Lines:
17808

💛  - Coveralls

You guys both missed that missed:
https://github.com/geo-engine/geoengine/blob/da3aee9fda623f6396ae30db4bae111493073e64/operators/src/source/gdal_source.rs#L235-L246

You guys both missed that missed:
https://github.com/geo-engine/geoengine/blob/da3aee9fda623f6396ae30db4bae111493073e64/operators/src/source/gdal_source.rs#L235-L246

778af17


[BUG] verify not working in 2.2.10

Was using version 2.2.3, everything works fine
updated to 2.2.10 and now the most basic example is not working
const token = await jwt.sign({ chroot: "chroot" }, jwtSecret);
console.log(await jwt.verify(token, jwtSecret)) // prints false

Tested locally with Node version 20.9.0 and deployed to cloudflare pages
If you need any more infos about the setup

I just noticed this too with trying to verify a supabase token using a string hs256 key. Maybe an issue with the base64 handling in the v2.0.8 refactor? Downgrading to 2.0.7 fixes it for me.

Hey, thanks for letting me know, will do some testing myself and report back :)

I've now created a test project:
https://github.com/tsndr/jwt-test
And when depoying this to Cloudflare Workers it runs without any issues.
Here's the result:
{
  "token": "eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJzdWIiOiJ0ZXN0IiwiaWF0IjoxNzAwMTM2NjE1fQ.buu44sYllJT8Q0P-aBxSl68zIj6ysPVmXmGvGJg4pZ0",
  "verified": true
}

If you can provide more context, so that I'm able to reproduce this error, I'll try to implement a fix.

When I test your project (https://github.com/tsndr/jwt-test) it works
But if I remove the sub property from the jwt.sign payload, its broken. payload becomes null and verify return false
Here is the full code
const secret = 'super-secret'

const token = await jwt.sign({ more: '@test' }, secret)
console.log({ token }) // { token:  'eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJtb3JlIjoiQHRlc3QiLCJpYXQiOjE3MDAxNDExMTd9.vBZIxVkxvXdpNc9tfmXQUHCpcDuJSVoMBNN6MbgBl2k' }

const decoded = jwt.decode(token)
console.log({ decoded }) // { decoded: { header: null, payload: null } }

const verified = await jwt.verify(token, secret)
console.log({ verified }) // { verified: false }

So the sub property seems mandatory for some reason, and was not before.

While doing some tests I found a new weird behaviour.
for some reason if payload is { sub: "test" } the verification works
but if payload is { sub: 'chroot' } the verification fails and decoded payload is null

Got it, please try again with v2.3.0 :)

2.3.0 fixes the issue for me, I just tested a few different payloads and they don't seem to impact decode or impact in a bad way.
Thank you for the fix !


ms\applink.c: Option to suppress includes

Define APPLINK_NOiNCLUDES to suppress includes.  This is useful when the symbols in applink.c must be modified in order to use wrappers.  The latter is needed when the compiler uses a default calling convention other than cdecl for its C library functions.  (E.g. Open Watcom.)

Checklist

[x] documentation is added or updated
[x] tests are added or updated

Thank you, Mr. Caswell. Good catch.
------- Original Message -------
On Wednesday, July 26th, 2023 at 1:32 AM, Matt Caswell @.***> wrote:

@mattcaswell requested changes on this pull request.

In ms/applink.c:

@@ -34,9 +34,12 @@
#define APPLINK_MAX     22      /* always same as last macro */

#ifndef APPMACROS_ONLY
+
+#ifndef APPLINK_NOiNCLUDES
The mixed case looks very strange to me. I would just call this APPLINK_NO_INCLUDES
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

I apologize for screwing this contribution up. It is my lifetime first opensource contribution and I don't really understand the process.
I understand you to be saying that someone else can fix this and that you do not need me to do anything else. Let me know if you need me to do something.
------- Original Message -------
On Friday, July 28th, 2023 at 12:57 AM, Matt Caswell @.***> wrote:

@mattcaswell approved this pull request.
Approved.
This PR has merge commits which we do not allow, but that can be resolved by whoever commits this on push to the repo.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

Please reject PR #21556. I just deleted and reforked and redid the same changes and made a new PR: https://github.com/openssl/openssl/pull/21595.
------- Original Message -------
On Friday, July 28th, 2023 at 9:20 AM, Wo'O Ideafarm publication delayed 4 yrs (not private) @.***> wrote:

I apologize for screwing this contribution up. It is my lifetime first opensource contribution and I don't really understand the process.
I understand you to be saying that someone else can fix this and that you do not need me to do anything else. Let me know if you need me to do something.
------- Original Message -------
On Friday, July 28th, 2023 at 12:57 AM, Matt Caswell @.***> wrote:

@mattcaswell approved this pull request.
Approved.
This PR has merge commits which we do not allow, but that can be resolved by whoever commits this on push to the repo.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>


Add a million pictures of the cutest cats

We just can't have enough pictures of cute cats. That's all.

Tetiana Mazurets, you might have some pictures of cats to add.


Vickel: Watch holdonangola.com

Vickel requests the watch of the watch_keyword holdonangola\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

holdonangola\.com has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Docs being deployed on a PR to master

Whenever a PR is open that targets master, travis will run and deploy the docs.  Even if that PR isn't merged yet.
Need to change it to only deploy ON the master branch, not a target
ref

:tada: This issue has been resolved in version 3.3.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


benchmarks in bench_templates.py refactored

benchmarks refactored

template_compile moved to template_benchmarks/template_compilation
template_render and template_render_simple moved to template_benchmarks/template_render
templates moved to template_benchmarks/template_render/templates

I've rebased this one as well, if you could review that would be much appreciated. Thank you!

I checked, but I could not find any benchmarks that were using lte_13 so I had removed it.

I checked, but I could not find any benchmarks that were using lte_13 so I had removed it.

Thanks. Let's merge this. 🎉


Rosettacode: add 3 new task.

Task:

Bull and Cow
Guess the number
Rock Paper Scissor

Waiting comment to fix (it should need a review) @Morriar and @R4PaSs

test this please

There's no "Bull and Cow" module in the PR, why is it in the covering letter ?

Actually correcting the code and as the missing bulls and cow files.
However, I'm trying to figure out how to correctly handle stdin.eof.
I don't understand clearly what it does and do include it in my code.


memcpy warning

The call to memcpy in
https://github.com/CPPAlliance/url/blob/1a9b2f9faa8fa212c8088fcf15e96d0938aab5aa/include/boost/url/impl/const_string.hpp#L188
leads to this warning
warning: ‘void* __builtin_memcpy(void*, const void*, long unsigned int)’ forming offset [15, 32] is out of the bounds [0, 15] of object ‘buf’ with type ‘char [15]’ [-Warray-bounds]

The complete warning:
In function ‘void* memcpy(void*, const void*, size_t)’,
    inlined from ‘boost::urls::const_string::const_string(boost::urls::string_view, const Allocator&) [with Allocator = std::allocator<char>]’ at ./boost/url/impl/const_string.hpp:188:16,
    inlined from ‘boost::urls::const_string boost::urls::ipv4_address::to_string(const Allocator&) const [with Allocator = std::allocator<char>]’ at ./boost/url/ipv4_address.hpp:175:35,
    inlined from ‘void boost::urls::ipv4_address_test::testMembers()’ at libs/url/test/unit/ipv4_address.cpp:99:13:
/usr/include/x86_64-linux-gnu/bits/string_fortified.h:34:33: warning: ‘void* __builtin_memcpy(void*, const void*, long unsigned int)’ forming offset [15, 32] is out of the bounds [0, 15] of object ‘buf’ with type ‘char [15]’ [-Warray-bounds]
   34 |   return __builtin___memcpy_chk (__dest, __src, __len, __bos0 (__dest));
      |          ~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
In file included from libs/url/test/unit/ipv4_address.cpp:11:
./boost/url/ipv4_address.hpp: In function ‘void boost::urls::ipv4_address_test::testMembers()’:
./boost/url/ipv4_address.hpp:172:14: note: ‘buf’ declared here
  172 |         char buf[max_str_len];
      |              ^~~

obsolete


minor: navbar dropdown stops working when coming out of small responsive design

Minor issue but...
If you have a responsive page with navbar and a dropdown in the navbar, and you make your browser small, then click the responsive menu button to uncollapse the menu, then open your browser large again, the navbar dropdown no longer works.

I think the problem is here is
component-animations.less
Try commenting out  overflow:hidden (untested in browsers other than Safari)
.collapse {
  .transition(height .35s ease);
  position:relative;
  //overflow:hidden;
  height: 0;
  &.in { height: auto; }
}

This fixes the issue for me ... tested in Firefox9.01, Chrome 16 and IE9
Thanx!

When i'm using it on Samsung Galaxy Tab 10.1 when i change from Landscape to Portrait works fine, but when i get back to Landscape the nav brokes and the dropdown stop to work.
Thoughts?

I believe this has been resolved in 2.0.2-wip with #2065.


Update generated code

This is essentially a no-op, since the apiserver-builder controller framework isn't useful for created federated controllers.


Cursor trails

Description
Description
Add animated cursor trail like in osu, or like hyprtrails plugin, but for the mouse cursor, it would look nice with cursors that are circles or something like that

no
write a plugin for that yourself if you want that


Avoid user-defined dimensions being injected into our code

Fixes #37

Codecov Report

Merging #47 (08c2bda) into main (ebdfc54) will decrease coverage by 2.23%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #47      +/-   ##
==========================================
- Coverage   97.04%   94.82%   -2.23%
==========================================
  Files          10       10
  Lines         305      309       +4
==========================================
- Hits          296      293       -3
- Misses          9       16       +7

Impacted Files
Coverage Δ

src/dimensions.jl
100.00% <100.00%> (ø)

src/utils.jl
75.86% <0.00%> (-24.14%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


self autocomplete

Hey :-)
Will there ever be a "self" autocomplete on variables added in other functions ?
thanks
matze

Hey
Here is how I did it:
go.property("jump_speed", 360)
go.property("gravity", 550)

function init(self) --[[@param self PlayerData]]
	self.hello = 5
end

function update(self, dt) --[[@param self PlayerData]] --[[@param dt number]]
end

---@class PlayerData
---@field jump_speed number
---@field gravity number
---@field hello number

In the extension v0.9.8 when a script file is generated with "Create Game Object" menu option:

The generated script file will have all parameters annotated. The rest is up to you to add fields (see examples at the top) to the generated type and you will have autocompletion. This is the most I can do for you. Hope it helps.


reconnect flag

we should consider doing this for backup.target.list as well

marked as a draft until ui component is complete

Front end section is committed and ready for review

Who needs to take action next on this?


[Request] disabling animation

Is it possible to add an option to disable dropdown/popup animation ?

@johaven
Just override the css in your global css file
eg
.picker-dropdown {   animation: none !important; }
or
.picker-popup {   animation: none !important; }

.owl-moment-picker {
top: 100px !important;
&.owl-picker-popup {
animation: none !important;
-webkit-animation: none !important;
-moz-animation: none !important;
}
}
With sass overwrite it's not woking, any idea ?


CSI for Openstack Does Not Support Volumes Encrypted by Barbican / TYPE LUKS

Hello - I have confirmed that attach will not succeed on LUKS type - two issues:
provisioner : Succeeds but looks to confirm a known block type to it and fails on confirm but succeeds to create it.
attacher - fails to mount it - also cant find a known / expected type.
Request: How do we has attached dynamically pull the relevant secret from barbican and store in kubernetes secrets for use to encrypt / decrypt.
At current i had to break my default which is bad of encrypted drives in openstack to fix this, users should not have to disable defaults on the whole openstack cluster - or we should be able to state TYPE of volume in the driver we want to request as a workaround but not a solution but still permanent to give flexibility / choice.
I understand that we can encrypt ourselves in K8S but thats not the point of this change / goal etc.
Please advise

I think it's a function gap , I knew we have barbican usage in ingresss and kms
but CSI we seems don't have this at all
I am not expert on this so can you help provide API/CLI or any other things might be helpful to understand the use case above? e.g some flow might be helpful a lot

I agree as well it's a function gap, but am happy that at least for now I
can pass type. People should be able to use external decryptor - it's
likely safest way vs storing the here in k8s in case of a breach .Here is
the procedure of what to do, (API not included but that part is easiest)
https://serverfault.com/questions/1052979/luks-encryption-for-mounted-disk-how-to-decrypt-cinder-volume
Here is the barbican API:
https://docs.openstack.org/barbican/latest/api/reference/secrets.html
The cinder API would be the same really except you read the key field ,
decrypt and return as ok after.
On Mon, May 23, 2022, 8:13 AM ji chen @.***> wrote:

I think it's a function gap , I knew we have barbican usage in ingresss
and kms
but CSI we seems don't have this at all
I am not expert on this so can you help provide API/CLI or any other
things might be helpful to understand the use case above? e.g some flow
might be helpful a lot
—
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/cloud-provider-openstack/issues/1881#issuecomment-1134596048,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFSSJU7CCRBSXCE3N6INVRTVLNY6BANCNFSM5WQTAUJQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

yes, saw the history https://github.com/kubernetes/cloud-provider-openstack/issues/1864 now and
will keep read this , in case any gap or Doc change, will discuss and propose PR, thanks for all the detailed info~

Not a problem - thank you for seeing the issue through with me. For use case, in any enterprise (not where I am using this but I do work for a large one) we would never ship non encrypted drives on failure back to a vendor so as such encrypted is a default for many large orgs. For my personal use case, I have interest in cyber security products, encrypted drives are a must for me as well.
I see two ways to achieve this goal K8S wise anyways:

Allow for external KMS (barbican is most common w OpenStack) <-- This is what I use
Internally (to K8s) encrypt volumes (also a good option) using various KMS's (internal <- A native plug-in or external <- Barbican in this case)

Thanks for the detailed info ,so maybe you can assign yourself on this @nashford77 as you seems already have a solution :)

Hello - Sorry, have been busy - I will give it a try shortly. It would be a great win if it works feature wise

/remove-lifecycle stale


[CPU] Improve tile sizes selection for tensor.pack ops.

It disables special vector sizes for non-f32 cases because the logic is only for 16x16 transpose cases. The improvements of dispatch sizes are from vectorization. We are not able to vectorize named ops if they have dynamic shapes, which is fixed by https://github.com/llvm/llvm-project/commit/03529b99b36788ca836b7ce238ea9400ce89847b. The change allows backends to vectorize them because they become static shapes (by tiling with size=1). It is not a hard condition; we track it in https://github.com/openxla/iree/issues/15441
The revision takes the number of threads into account, so we have better performance on multi-threaded. It also reduces runtime overheads.
This is a step toward to https://github.com/openxla/iree/issues/15391 and https://github.com/openxla/iree/issues/15349
It improves the performance of tensor.pack op from 420 ms to 170 ms on 8-threaded x86 CPU.

Looks fine to me... I dont have much of a handle on the heuristics here... If you have any data as to why it helps, post it here for recording.

done. I include the perf number in the description.

Do you know if the Falcon7B regression is real? That's one of the main models we are working on right now.

Do you know if the Falcon7B regression is real? That's one of the main models we are working on right now.

I can take a look, but I feel that it is not real.. The regression did not happen before I rebase it to ToT. https://gist.github.com/iree-github-actions-bot/99c8f439051a13ad1a3d0ebe112056b9

I'm pretty sure that it is not real because it is on default flags, not data-tiling.


Fix or postpone remaining TODO(QUIC) items

Postpone two TODO(QUIC) items appropriately

The one in ch_rx_handle_packet() is a tuning thing -> QUIC FUTURE
The one in ossl_quic_tserver_shutdown() is a server thing -> QUIC SERVER

Add handling of SSL_get_shutdown()

Return SSL_SENT_SHUTDOWN and SSL_RECEIVED_SHUTDOWN with semantics
similar to TLS connections.

@mattcaswell please look again

@mattcaswell please look again

What about using the fake timer in the test?

@mattcaswell I've changed the test to use fake time, please reconfirm

Hmmm. CI failure is disturbing (but probably not related)

Yeah, definitely not related to this as this does not touch the tserver test and the added api is also not invoked by that test.

Added #22424 issue for the intermittent CI failure.

This pull request is ready to merge

Pushed to master.


PDA opening as a toggle.

Code's prettier this time since it's constrained to useEntity and iActivate instead of hijacking and renaming an existing Open function. Something about QoL and it feeling more natural to use.
edit: Trying to figure out git / dealing with nulls so this un-breaks.

I'm not sure if I dealt with the possible dereference of a null in a proper way, but it builds now.

I think it'd be better to add a new method to BoundUserInterface called Toggle which toggles and then replace usages of Open with Toggle.

Hm, I'll give that a go and resubmit later.

So BoundUserInterface is inside RobustServer, so would I just push something there first, then toggle the toggles for the various handhelds that would benefit from it?

Yeah, you'll need to fork RobustToolbox, but it shouldn't be much more complicated code wise than you've been doing here. Then after that you'll be able to change the stuff on space-station-14 to use the Toggle method you implemented instead of Open.


Add zsh completion to svcat

closes: #1912
This is my first PR for svcat.
Issue 1912 requires two steps to close:

[x] 1. bump spf13/cobra to 0.0.1 (or above)
[x] 2. add zsh completion - zsh completion for svcat is added

Hi @carolynvs  I wasn't able to get the GenZshCompletion() working a few iterations back so I ended up implementing the kubectl / helm way (which does indeed work).
As requested I re-implemented using cobra's GenZshCompletion.  It doesn't seem to be working for me still but you can pull and try it. We can always revert back to the previous change if you confirm using GenZshCompletion() is no good.

@carolynvs

yes!
svcat get b actually has two options: bindings and brokers. so can confirm:
svcat get b --> shows on next line:  bindings  brokers
svcat get bi --> svcat get bindings
svcat get br --> svcat get brokers

LGTM, but I'll let folks from another org give the final LGTM label


Added carousel

Created the carousel for randomly selected cards from the database (there's only one in the DB with an actual question/answer added to the card, so it "looks blank," but it's not). Added the ICardService to the HomeController for database access. Added the Card Model to the View.

For the Service methods, I'm thinking maybe they should be more descriptive? Like GetAllCards, GetCardByID, CreateCard, ....?
I know in a project that's smaller, the method names might not matter as much. But I think we could still benefit from them being more descriptive.
I'm also thinking the CardService should probably be in the Domain layer instead of the UI layer. In my professional work, all DB calls are made in the Domain layer, to not only provide separation of the UI from Domain, but also as an added security layer. It also allows more ease of Services talking to other Services. I am of course not sure what @dyohn had in mind for our Services, but those are my thoughts.
And of course please share any of yours @colewinfield, @Freytes.

My opinion is that data access should be its own layer. However, I understand that time is short, as the end of the semester is upon us. If it's possible to extract the data access into another project in the remaining time, then we should do so. If it is not, then we'll take what we can get, merge this, and I'll address this either myself or assign it as an Epic for the next cohort of contributors.

At this time we haven’t implemented service calls at the data layer.

PR #22 is a more up-to-date branch with this one merged into it.


bots: Add Youtube Bot.

sends first video from youtube search results back to the place it got a message from

Automated message from Dropbox CLA bot
@tukruic, it looks like you've already signed the Dropbox CLA. Thanks!

Looks awesome!  @derAnfaenger can you give this a review?

@timabbott will do with all pending bot commits, once I'm finished with refactoring the testing framework.

This bot breaks when entering a query that doesn't return any videos.

Merged as 089e5896a6ec7f4dce680d4b7a490342f70e4e9a, thanks @tukruic!
(And thanks for the review @derAnfaenger)


[Phcode.dev Bot] Publishing Release v1.1.1 to Extension Store

Thank you for contributing to phcode.dev extension store.
Please track extension/theme publish status by clicking this link.
You may close this issue at any time.

Extension successfully published to store. You can close this issue at any time.


【IOS】SEGV_ACCERR , crash in FEngine::loop -  MetalTexture::loadWithCopyBuffer

Describe the bug
Random crash in FEngine::loop, and the executed thread is not the main thread. There is only one code which is relavent about decoding image to texture, but i execute that code in the main thread. Please help me.
To Reproduce
Steps to reproduce the behavior:

create engine with metal backend
create texture on the main thread
decode image to texture with CGImageGetDataProvider
call Texture::setImage

Expected behavior
no crash
Logs

0
libsystem_platform.dylib
_platform_memmove + 72
1
DUApp
filament::backend::metal::MetalTexture::loadWithCopyBuffer(unsigned int, unsigned int, MTLRegion, filament::backend::PixelBufferDescriptor&, filament::backend::metal::PixelBufferShape const&) + 108

Smartphone (please complete the following information):

Additional context
Obviously Metal::loadWithCopyBuffer is called in other thread, but i create engine and texture in main thread. Is that ok?

Yes, Filament's API can be called from the main thread. The crashed thread is Filament's internal driver thread.
My guess is that you're releasing the CGImage memory too soon. The pixel buffer needs to remain valid until the PixelBufferDescriptor's callback is called:
    void* buffer = /* allocate buffer */ ;

    Texture::PixelBufferDescriptor p(
            buffer,
            size,
            Texture::Format::RGBA,
            Texture::Type::UBYTE,
            [](void* buffer, size_t, void* data) {
                /* free buffer */
            }
    );

I will try it later, thanks

I didn't release the CGImage immediately, but i don't keep a reference to it. Is This the reason make CGImage invalid ?  I use CFDataRef to generate texture and release it in the PixelBufferDescriptor callback.  This texture-parse function is invoked on the main thread,  Here is my code:
Texture* parseTexture(Engine* engine, const char* path) {

    NSBundle *bundle = [NSBundle mainBundle];
    NSString* filePath = [[NSString alloc] initWithUTF8String:path];
    UIImage * img = [[UIImage alloc]initWithContentsOfFile:filePath];
    CGImageRef image = [img CGImage];
    size_t image_width = CGImageGetWidth(image);
    size_t image_height = CGImageGetHeight(image);
    size_t size_in_bytes = CGImageGetBytesPerRow(image) * image_height;
    CFDataRef data = CGDataProviderCopyData(CGImageGetDataProvider(image));

    const unsigned char * buffer = CFDataGetBytePtr(data);

    Texture* texture = Texture::Builder()
    .width(image_width)
    .height(image_height)
    .levels(0xff)
    .format(Texture::InternalFormat::SRGB8_A8)
    .build(*engine);

    Texture::PixelBufferDescriptor desc(
            buffer,
            size_in_bytes,
            backend::PixelDataFormat::RGBA,
            backend::PixelDataType::UBYTE,
                                              [](void* buffer, size_t n, void* user) {
        CFDataRef data = (CFDataRef)user;
        CFRelease(data);

    }, (void*)data);

    texture->setImage(*engine, 0, 0, 0, image_width, image_height, std::move(desc));

    return texture;
   }

Looking forward to your reply.

Please see my response in https://github.com/google/filament/discussions/4904

@bejado I have read the answer you mentioned in #4904 , it seems right. But  there is no crash if i run that code and release CFData and CGImage in advance , the model is just rendered whole black as below:

I am afraid that this crash still happen after i use the code you mentioned in #4904


updates the prompt to display battery information only for machines that have one

Tested on iMac and MacBook Pro

Cool, good call! Thanks.


Check internal float/const lists

Adding RooMinimizerFcn locally to avoid adding extra items to floating list

This is no longer needed as the workaround already exists now in 102x, so closing


fix given bug found in nnls benchmark

Codecov Report

Merging #101 (ead9c86) into master (cd4684f) will decrease coverage by 0.40%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #101      +/-   ##
==========================================
- Coverage   73.70%   73.30%   -0.41%
==========================================
  Files          23       23
  Lines        1103     1105       +2
  Branches      166      167       +1
==========================================
- Hits          813      810       -3
- Misses        237      241       +4
- Partials       53       54       +1


Label's Clip Content always on.

Godot version:
3.0.2
OS/device including version:
Windows 10 / Ubuntu LTS 18.04
Issue description:
A Label's Clip Content seems to always be on.  Seems to be a bug.  Clip Content does work fine on Rich Text Label.
Worth noting that any text with outlining (generally ported from Godot 2.1.4 fonts) is clipped.
Steps to reproduce:
Basically any label has it on even when checked off.

@Gramps did you use bitmap font?

In 2.1.4, I think so.  I used the editor's Import Font option since there is no good way to make font outlines in 3.0.2 yet.

I'm not sure clip content is always on is true.

if clip content property is alway on for Label, then the icon should not be displayed.
(left one is on editor, right one is on running game)
I think, there is something another.
would you demonstrate your problem with image(screenshot)?

You are actually correct, to a degree.  It turns out that auto-wrap is the culprit.  If auto-wrap is off, then the outline is fine; but if it's on, then anything outside the border gets clipped for some reason.  This, however, is not the case for Rich Text Label.  The image below shows this in action.

You can see the second label has the outline clipped off with auto-wrap on, unlike the first.  Rich Text Label doesn't even have an auto-wrap toggle but works regardless.
I added the test project as requested which includes the font file I'm using from Godot 2.1.4.
Godot 3.zip

This is not a bug, if autowrap exists, stuff that is outside the label will be removed. This is the expected behavior.

It was more that the shadows or outlines should be inside the area wrapped instead of cut off like they are in the image I added; much like how Rich Text Label does so there is consistency in the two.
Though I do think there does need to be more dynamic font effects as there is no way to outline text (easily) in Godot 3 like there was in 2 so I am constantly importing font files from that.


Fix summary logic error

Fix flag to be parsed properly by verbosity
cc. @jijoongmoon
Self evaluation:

Build test: [X]Passed [ ]Failed [ ]Skipped
Run test: [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Jihoon Lee<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.4.20191203. Thank you for submitting PR #334. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/.


Error with zlib that doesn't occur in direnv

I recently started having a weird issue where emacs-direnv throws an error that doesn't occur in regular direnv.  The output from the *direnv* buffer is not very enlightening, but it looks related to zlib.  Any ideas?  Searching for this error leads to some old bug reports on Go, but I haven't been able to make any progress from there.
This happens on only one of my machines.  The offending machine runs Arch linux while the working one runs Ubuntu, so I imagine it's related to a particular version or package, perhaps?
direnv: error unmarshal() zlib opening: unexpected EOF
{
  "DIRENV_DIFF": "eJw0yl-XakAAAPDvMs_riFktztmHUawRF9MgXhzWdPOnVDSo03e_T_f99wJXoL_eH-AC9BfYYmL-ifMtJkAHgnjqz0xsi3sn_vYVE--sv47g479KEN3Y5h7ogDmLt_Ks27xB4cwtqJK5_jw3EbSHQbOU9cqbPQzjxmOK3B9ptZe1U8JqYcJ5rn1JQq0cw7gx3UeW0iXVwkGadu1tjoIrH4qSR-RA60diuM7sPgu0UX_6ifncWtEDL-fqeTsf1msm7fMOQ_pQSVSUHLd2oLR_izEV5GYclKCpM9PZyVvvOQ1kSBwrYhjWKMtghH6cS4rK3ZJKlEBVNgjE_qKgz4sd3EcYaqyvYjtwBf7l8y7wR7X7lZLQkJjmluWEEAry3KCSXKHvb_B-_wsAAP__9dxuUA==",
  "DIRENV_DIR": "-/home/karl/test",
  "DIRENV_WATCHES": "eJyM0MFqxCAQxvF38Rxi4mjU3Hss9F560MxMI3VjMe52ofTdS29hWei-wI__971-i5fQVjELuZYTyY9Qs1wKkqxUPpvsabvURXTiuWBLJxLzaAC8GoweOvF0TXvbxdzqmX66e1S_lI3Tu8RUabvIkHP5kgatY-2jts5PjiZEJK2DMzwsRsWJFUeI3iAZZ3UAGJnYAbKyY1TR382BR3IOy5A4nHPrt3Q9eodVHPL-v7OvlPOt8lc1WQB1e9LbbwAAAP__BT12dA=="
}

never seen this, sorry.
perhaps it's a data problem, not a code problem.  if you start emacs from your home dir (no direnv i presume), then open a file in a dir with a minimal envrc (export FOO=bar), do you also see it?
also it seems there is still json output in your example. does emacs-direnv pick it up?  (setting "always show summary" config may help)

I'm pretty sure this is related to trying to use direnv's support for Nix and something in my Nix configuration.  After lots of changes coming from Nix and my own code I'm not able to reproduce this anymore, so I'll close it for now.

ok, cool!


Fairly high failure rate when trying to import state

I have not tested this extensively, but quite often, when I try to resume a fuzzing state by using importState, I get an error like:
[Cli] Failed to import state: The operation could not be completed. (SwiftError error 0.)
My state file can be fairly large (a few megabytes). Not sure what the problem could be.
Suspecting it could be something to do with JSONDecoder? Will update if I find anything.

Hi! Interesting, thanks for reporting. I haven't run into this issue, but tbf I rarely use the stateImport feature myself. Maybe you can try adding some print statements around here: https://github.com/googleprojectzero/fuzzilli/blob/3459f6544ccb86e5a9f3b16576e3191318aa7e85/Sources/FuzzilliCli/main.swift#L225 to see whether the file is read successfully and the JSON decoding fails. Is this on Linux or macOS? Could it maybe be an out-of-memory issue during decoding?

We can close this issue. The problem was caused by my code.
But I still think the probability of "state.json" file getting too large or corrupted should be addressed.
Perhaps it can be split into separate corpus program JSONs for easier debugging/management.


Unit tests for drawing_classifier

Fixes #1443.

As of right now, there is one failing unit test related to save/load. Marked it as an xfail. I'm conflicted as to whether the fix should come as part of this PR or a different PR. Open to suggestions.
This PR is open for review.

One high-level question: how much total running time does this add to our Python unit tests?

@nickjong The tests take about 47 seconds to run (including the time taken for the pre-trained model tests, which are disabled right now). A lot of that time is setting up classes and checking for SFrame equality in the evaluate tests.

Do we have no way to repro the crash outside of Travis? It sucks to have to wait for each build from scratch, along with running all the prior tests. This will also make it harder to use GitHub's functionality to track which commits each reviewer has already reviewed.
If you do need to debug in Travis, could you mark this PR as a WIP and otherwise let us know when to review changes again?

What portion of your last commit, the merge from master, should be reviewed? Or what portion of it interacts with this commits that I should be reviewing?

@nickjong There is nothing in the merge from master that interacts with code here or needs to be reviewed as part of this PR. The only change that needed to be reviewed was e703ca1cb5f482a448e582eed632621dc43ad8ea


Portainer Best Practice

Hi freescout community
I'm trying to setup freescout for us and for our clients. I would like to run multiple Dockers on one server so I have decided to work with portainer. Unfortunately, I couldn't work out the installation.
Is there any tutorial on how to make it work?
Could someone look at it? If it helps we can get in touch and organize a zoom meeting.
Many thanks
Siavash

#59
See this thread. Use the Stacks option. Edit the docker-compose file and .env file to suit your environment and options, then upload the docker-compose file and import the environment variables from the .env file.


[WIP] volumes and garbage collection

Please review the commits separately @zed-rishabh @deitch
1st is a missing piece of logic to ensure that we don't redownload but wait for a VerifyImageStatus. Needs to be carried into the CAS code
2nd is a minor log reduction (some app instances to tons of DNS lookups)
3rd was discovered when the CAS code is using the CurrentSize and TotalSize; they are swapped
4-6 is a sequence of pieces to 1) pupulate the PersistImageStatus in volumemgr based on content in /persist/*/verified 2) use that internally and as a way to tell verifier to delete an unused file 3) cleanup other GC mechanisms for verifier 4) simplify the downloader GC mechanism to be immediate

The currentSize/totalSize confusion had me laughing. Every time I touched it, I needed to look up which is which. Your solution - just use a normal name - makes so much sense.

If I understood the "do not redownload" code, this is the "async" format we discussed.

On startup, volumemgr scans all existing files, and creates a PersistImageStatus for each
verifier has a handler for volumemgr/PersistImageStatus which catches those, and in turn creates a verifier/VerifyImageStatus
When volumemgr runs:

if it finds a VerifyImageStatus, things work as normal (no change)
if it finds no VerifyImageStatus but yes PersistImageStatus, it waits for the VerifyImageStatus to exist, which triggers the handler
if it finds none of either, proceed and download (as current)

If I understood it correctly, this requires volumemgr to be aware of the directory of the verified files. We worked pretty hard in the recent past to eliminate that knowledge, so that verifier can place its final verified files anywhere it wants (under its sole control), and volumemgr just accepts whatever it gets in VerifyImageStatus.
Can we reverse it a bit? I think this would work equally well.

verifier, on startup, does the scan (since it alone knows where things are) and publishes VerifyImageStatus (as usual)
verifier, on completion of its scan, publishes a single PersistImageComplete that indicates it has completed its scan
volumemgr looks for PersistImageComplete (single item). If it finds none, it knows verifier has not completed its scan, so wait. If it finds one, scan complete, continue.
volumemgr looks for verifier/VerifyImageStatus, proceed as usual
volumemgr registers a handler for verifier/PersistImageComplete, to trigger all of its doUpdateContentTree() again, thus causing downloads that were held up before.

I don't mind writing this up, if it helps.
Thoughts?

verifier has a handler for volumemgr/PersistImageStatus which catches those, and in turn creates a verifier/VerifyImageStatus

No, the verifier only tracks the PersistImnageStatus using a delete handler; the verifier deletes the verified file when the PersistImageStatus is unpublished by the volumemgr. Separately there is a VerifyImageConfig/Status exchange between the two. The only way the PersistImageStatus is used in that case is for the verifier to realize (by calling loolkupPersistImageStatus) that the file already exists hence it is done; no need to move from pending and do actual verification.

if it finds a VerifyImageStatus, things work as normal (no change)

Note that "No change" includes volumemgr publishing a PersistImageStatus when it has received a VerifyImageStatus. That is necessary so that volumemgr can later delete that to cause volumemgr to delete the file.

let me 1) split the minor things (like the size swap) in a separate PR 2) redo the GC one so we don't need any PersistImage* objects and have the verifier do the directory scan and publish the result as VerifyImageStatus with RefCount=0. Should be simpler.

Replaced by PR1161


Data download

pamaishlin[CodePlex] it is critical that 'fields to include in download' include offset, such as needed for soil moisture data distinguished by burial depth of sensor. Also, the drop down list of available variables to download for a given site presently shows all variables
for all sites even though only one site is selected. see
http://icewater.boisestate.edu/HydroServerWebsite/Data-Query, try DCEW site and variable soil moisture to see the problem. One series for one pit actually includes sensors at multiple depths, yet offset will not be provided as a download field. This is
highly problematic.

horsburgh[CodePlex] Fixed in release for ODM 1.1.1.

pamaishlin[CodePlex] Progress seems apparent on this issue in HydroDesktop
http://hydrodesktop.codeplex.com/workitem/607
Will similar progress be made for an update to HydroServer?


Bug report: VRM1 has bone problems when importing

Describe the bug
When importing a VRM1 model, many bones are placed in the wrong position or are not connected when they should be. And the eyes also have a completely different position and are always rolled (it shouldnt be rolled as it gives problem in some look animations).
When I try to import the VRM1 model direct into unity, everything is fine. But if I import a VRM1 model that has been exported with blender, then many things are broken (mostly the eyes when I use my eye animator script)
To Reproduce
Import any VRM1 model (Latest Steam VRoid Studio)
Expected behavior
Normal bone structure
Screenshots

Desktop (please complete the following information)

OS: Windows
Blender 4.0
Addon Version:  2.20.86

Because VRM 0.0 did not include “bone orientation”, it was possible for add-ons to connect bones as they saw fit.
However, this is no longer possible with VRM 1.0, because “bone orientation” is now included in the data. If the add-on specifies the bone orientation as it sees fit, it will ignore the creator's intentions.

When using Unity to correctly handle VRM1.0 eyes, please be sure to keep the following points in mind

Please use UniVRM version v0.116.0 or later.
When loading a scene in UniVRM, please make sure that the model is facing the +Z direction. Do not rotate the model.

In addition, the processing of eyes in Unity is very difficult to handle, and there are many points to be careful of. Please refer to the related issues in the UniVRM repository.

https://github.com/vrm-c/UniVRM/issues/2427
https://github.com/vrm-c/UniVRM/issues/2345
https://github.com/vrm-c/UniVRM/issues/2335
https://github.com/vrm-c/UniVRM/issues/2159
https://github.com/vrm-c/UniVRM/issues/2106


RuntimeError: Error(s) in loading state_dict for DataParallel:

Hello everyone.
I trying to run test.py. But I got an error by below:
`RuntimeError: Error(s) in loading state_dict for DataParallel:
size mismatch for module.loc.0.weight: copying a param with shape torch.Size([28, 512, 1, 5]) from checkpoint, the shape in current model is torch.Size([28, 512, 1, 1]).
size mismatch for module.loc.1.weight: copying a param with shape torch.Size([28, 1024, 1, 5]) from checkpoint, the shape in current model is torch.Size([28, 1024, 1, 1]).
size mismatch for module.loc.2.weight: copying a param with shape torch.Size([28, 512, 1, 5]) from checkpoint, the shape in current model is torch.Size([28, 512, 1, 1]).
size mismatch for module.loc.3.weight: copying a param with shape torch.Size([28, 256, 1, 5]) from checkpoint, the shape in current model is torch.Size([28, 256, 1, 1]).
size mismatch for module.loc.4.weight: copying a param with shape torch.Size([28, 256, 1, 5]) from checkpoint, the shape in current model is torch.Size([28, 256, 1, 1]).
size mismatch for module.loc.5.weight: copying a param with shape torch.Size([28, 256, 1, 5]) from checkpoint, the shape in current model is torch.Size([28, 256, 1, 1]).
size mismatch for module.loc.6.weight: copying a param with shape torch.Size([28, 256, 1, 5]) from checkpoint, the shape in current model is torch.Size([28, 256, 1, 1]).
size mismatch for module.conf.0.weight: copying a param with shape torch.Size([14, 512, 1, 5]) from checkpoint, the shape in current model is torch.Size([14, 512, 1, 1]).
size mismatch for module.conf.1.weight: copying a param with shape torch.Size([14, 1024, 1, 5]) from checkpoint, the shape in current model is torch.Size([14, 1024, 1, 1]).
size mismatch for module.conf.2.weight: copying a param with shape torch.Size([14, 512, 1, 5]) from checkpoint, the shape in current model is torch.Size([14, 512, 1, 1]).
size mismatch for module.conf.3.weight: copying a param with shape torch.Size([14, 256, 1, 5]) from checkpoint, the shape in current model is torch.Size([14, 256, 1, 1]).
size mismatch for module.conf.4.weight: copying a param with shape torch.Size([14, 256, 1, 5]) from checkpoint, the shape in current model is torch.Size([14, 256, 1, 1]).
size mismatch for module.conf.5.weight: copying a param with shape torch.Size([14, 256, 1, 5]) from checkpoint, the shape in current model is torch.Size([14, 256, 1, 1]).
size mismatch for module.conf.6.weight: copying a param with shape torch.Size([14, 256, 1, 5]) from checkpoint, the shape in current model is torch.Size([14, 256, 1, 1]).

`
I hope, everyone helps me to solve this error.
Thanks.

hello, have you fixed it? I have got same error with you.


GPU usage gets stuck at few % after status in TabItem content gets updated at least once

Tested in .net 7, looks fixed, GPU usage spikes and then goes to 0%.


feat(bridge): add ttko to dropdown

@2manslkh @jscriptcoder any updates on this?

@d1onys1us seems like there is a PR for it for bridge ui v2

@d1onys1us seems like there is a PR for it for bridge ui v2

is there any harm in adding this @jscriptcoder? or do you suggest we close it?

I'd say we can add it to the current bridge

This will add TTKO and not TTKOe, and be hardcoded for both bridges. It's better to add it as an environment variable.


Add keyboard shortcut support for navigation

Prerequisites

[X] Put an X between the brackets on this line if you have done all of the following:

Checked that your issue isn't already filed: https://github.com/GetStream/Winds/issues

Description
Add support for keyboard shortcuts to navigate through feed
Steps to Reproduce

Login to Winds
Go to RSS feed
Try pressing popular navigation keys (ex. j for next item, k for previous item, etc.)

Expected behavior:
User should be able to navigate through feed using keyboard shortcuts
Actual behavior:
Nothing happens
Reproduces how often:
100%
Versions
Version 2.0.195 (2.0.195)
Additional Information
macOS High Sierra 10.13.4

Adding into the internal backlog.

I was looking for keyboard shortcuts and found this issue.
Is the issue closed because the feature request was already addressed?


fix: sometimes crashing mutator

Description
We have the final form mutator crashing sometimes https://toptal-core.slack.com/archives/CERF5NHT3/p1692093998693089?thread_ts=1691993118.653259&cid=CERF5NHT3
This PR attempts to fix the problem by applying defencive coding without understanding the root cause because user could not reproduce.
How to test

Temploy
FIXME: Add the steps describing how to verify your changes

Screenshots

Before.
After.

Insert screenshots or screen recordings
Insert screenshots or screen recordings

Development checks

[x] Add changeset according to guidelines (if needed)
[x] Read CONTRIBUTING.md and Component API principles
[ ] Make sure that additions and changes on the design follow Toptal's BASE design, and it's been already discussed with designers at #-base-core
[x] Annotate all props in component with documentation
[x] Create examples for component
[x] Ensure that deployed demo has expected results and good examples
[x] Ensure the changed/created components have not caused accessibility issues. How to use accessibility plugin in storybook.
[x] Self reviewed
[x] Covered with tests (visual tests included)

Breaking change

[x] codemod is created and showcased in the changeset
[x] test alpha package of Picasso in StaffPortal

All development checks should be done and set checked to pass the
GitHub Bot: TODOLess action

PR commands

List of available commands:

@toptal-bot run package:alpha-release - Release alpha version
@toptal-anvil ping reviewers - Ping FX team for review

PR Review Guidelines

When to approve? ✅
You are OK with merging this PR and

You have no extra requests.
You have optional requests.

Add nit: to your comment. (ex. nit: I'd rename this variable from makeCircle to getCircle)

When to request changes? ❌
You are not OK with merging this PR because

Something is broken after the changes.
Acceptance criteria is not reached.
Code is dirty.

When to comment (neither ✅ nor ❌)
You want your comments to be addressed before merging this PR in cases like:

There are leftovers like unnecessary logs, comments, etc.
You have an opinionated comment regarding the code that requires a discussion.
You have questions.

How to handle the comments?

An owner of a comment is the only one who can resolve it.
An owner of a comment must resolve it when it's addressed.
A PR owner must reply with ✅ when a comment is addressed.

@toptal-anvil ping reviewers


Add instructions on copying and aliasing pages to the editor's guide

Resolves #7010
With this PR, adding a new page to the Editor's guide in the documentation regarding how to copy and create aliases to existing pages. I tried to follow the style of writing from other pages, let me know if there's anything I should change and/or add.

Rendered: https://wagtail--7068.org.readthedocs.build/en/7068/editor_manual/copying_aliasing_existing_pages.html

Accepted the changes and fixed a typo, should be fine now :) thank you for the suggestions!

What is the test that failed?

@vladutmd

docs/editor_manual/copying_aliasing_existing_pages.rst:58: D002 Trailing whitespace

any updates on this making it in?

Hey @vladutmd,
Will aim to get this reviewed in the next few days with a view to getting it in.
Thank you for you contribution and sorry for the delayed response!

Merged in f54e7e1e416. Thank you very much for this @vladutmd


Shuffle the tool_integration_tests within each shard

To investigate https://github.com/flutter/flutter/issues/88060, see if the flaky shard always times out on the same tests.

Hmm, it still ran them in the same order they were listed in the pub run test command....
▌18:43:25▐ RUNNING: cd packages/flutter_tools; ../../bin/cache/dart-sdk/bin/pub run test --test-randomize-ordering-seed=20210817 -rcompact -j1 --no-color test/integration.shard/ios_content_validation_test.dart test/integration.shard/break_on_framework_exceptions_test.dart test/integration.shard/deferred_components_test.dart test/integration.shard/observatory_port_test.dart test/integration.shard/analyze_size_test.dart test/integration.shard/template_manifest_test.dart test/integration.shard/overall_experience_test.dart test/integration.shard/expression_evaluation_test.dart test/integration.shard/coverage_collection_test.dart test/integration.shard/deprecated_gradle_settings_test.dart test/integration.shard/flutter_build_null_unsafe_test.dart
Shuffling test order with --test-randomize-ordering-seed=20210817

00:00 +0: loading test/integration.shard/ios_content_validation_test.dart
00:01 +0: loading test/integration.shard/ios_content_validation_test.dart
00:02 +0: loading test/integration.shard/ios_content_validation_test.dart
00:03 +0: loading test/integration.shard/ios_content_validation_test.dart

https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8838600487350714273/+/u/run_test.dart_for_tool_integration_tests_shard_and_subshard_4_4/test_stdout


PG mapper error seed #s become commit links

Input like this from gcf trac ticket 132:
*** ERROR: mapper: Reached run limit. Giving up. seed =<PHONE_NUMBER> Physical Graph: 4 Calculating shortest paths on switch fabric.
Becomes:
 *** ERROR: mapper: Reached run limit. Giving up. seed = [1335813330](https://github.com/ahelsing/tickettest2/commit/1335813330) Physical Graph: 4 ...

could make the regex require 12 chars for a commit hash?

made the regex longer so this particular case is avoided


[macCreatorType] return correct creator/type on Python 2.7

the function "getMacCreatorAndType" contains a workaround to address a bug in GetCreatorAndType function from the MacOS module.
On the Intel platform with 'little endian' byte order, the function would return strings in reverse order.
A comment in fontTools source code links to a bug report made by Just himself on the Python issue tracker (http://bugs.python.org/issue1594). They say the bug was fixed in a later python revision (this was 2009!).
I found out that the bug is still there until Python version 2.6.9, but it was finally fixed with Python 2.7.0 (I had to install both and test them on my mac).
The current patch will limit the workaround to Python versions less than 2.7 (i.e. basically Python 2.6.x, since earlier versions are no longer supported either).
The patch seem to fix the issue https://github.com/behdad/fonttools/issues/221 reported by @adrientetar.
I think there's no need to use Appkit to get the mac's file creator and type (they use that in extractor). The MacOS.getMacCreatorAndType still works fine, even on on 64 bit architecture -- provided one disables the workaround which is no longer required for Python 2.7.
I can now read LWFN Type1 fonts with t1Lib.py and save them as PFB, or even feed a TrueType resource-fork suitcase to TTX, and dump it to XML as usual.
I'm using Homebrew Python 2.7.9 on Mavericks.

cc @typemytype


Data visualisations

I don't love them. Can they be prettier. We should at least use the actual state names to avoid looking like fools.

But we are fools

I think we are going as is? So closing.


1

hello ，please reply my issue in your project of vtk_triangulate _point ? thank you a lot

ok, already did.
closing this issue, any further discussions will be there.


Update macos install instructions

I was trying to get this setup on MacOS v10.14.4 and had to do a few different/additional things to what was described in the README.  I updated the README to reflect the steps I took to get it working in the hopes they might help others.

I made one slight tweak so that the fact that you need to run make libsecp256k1 on Mac OS is more clear in the instructions. We might also consider having users use the latest jdk instead of specifically openjdk 8 now, as I don't think there's any longer any reason to use jdk 8 specifically. I haven't tested it with the main java formula for homebrew yet though so I didn't make that change myself.

@iamchrissmith you'll have to sign the CLA, then I'll get this merged. It's the "Details" button next to the red X by clahub.


HTTP request line parsing: no method implies no host:port. Fixes issue #82

Please note that this is only a tentative fix and request for discussion. I was never able to make complete sense of the HTTP and URI RFCs :)
The change does fix the problem listed in issue 82, but the requests in question are questionable.
According to RFC 1945 paragraph 5.1.2, an absolute URI is only possible if the server is a proxy. Also an absolute URI would imply a scheme part ? So the URI in the request is an abs_path which is a relative reference and can't begin with //, both from the definition in RFC 1945, and because RFC 3986 paragraph 4.2 assigns a meaning to //path, which should define an 'authority' part:
relative-part = "//" authority path-abempty
A relative reference that begins with two slash characters is termed a network-path reference;
such references are rarely used.

From a bit of testing around, it seems that some HTTP servers do refuse the double-slash, and some accept it (apache, for exemple,  seems to be ok with it).
My feeling is that libupnp should be lenient about these requests, and forget about supporting the "rarely used network-path references", which would allow to relax the syntax on the URI and treat // as /, but I have no firm opinion on the subject.
The suggested fix treats the URL as beginning with a regular path if there is no scheme present.

There were some failures in "make check", detected by Travis CI. By the way, the test_url.c source deserves some cleaning, doesn't it? I see no reason for a vararg macro here, the compiler produces lots of warnings and it is not clear what is happening to the error field.

I'm away for the week-end. Will take a look beginning of next week.

Unsurprisingly, the tests which fail have 'network path reference' in them, e.g. //localhost/somepath being translated to http://localhost/somepath
So we're stuck between being non compliant with some URLs in the RFCs (of a type which nobody ever uses as far as I can see), or breaking the "many clients"? mentionned in the problem report. I'll ask what these "many clients" are.
I just checked with older code, and libupnp never accepted the improper URLs, this is not a recent change.

Also, and by the way, URLBase is deprecated in UPnP 1.1, and the advice is quite forceful:
<URLBase>
Use of URLBase is deprecated in UPnP 1.1; UPnP 1.1 devices MUST NOT include URLBase in
their description documents. For full definition of URLBase, see the UPnP 1.0 specification.

All in all, I think that the best approach would be to change nothing and tell Gerbera to arrange for not setting URLBase (this can be done by choosing the arguments to registerRootDevicex)

I tend to agree with @medoc92 in that the best approach would be doing nothing. If we force something on URLBase we break stuff. And if Gerbera is supposed to be 1.1 compliant, it must not include URLBase in the description document.

Ok then, we agree, closing the pull request.


Invalidate normal cache on action?

I used a bunch of normal caches to not fetch data that hasn't changed between page changes, but I was expecting an action on the same page to invalidate even the normal cache (in swr manner).
In essence I was expecting it to be a strategy that behaves like swr if action is triggered, but otherwise like normal.

I somehow completely missed this. I added a new API that makes this invalidation a single liner called decacheClientLoader


TIFF read_scanlines of images with unassociated alpha didn't honor stride

This could run off the end of a buffer when reading images with
unassociated alpha, if the calling app passed an unusual ystride (or,
in the case I found, using a negative stride to flip the process
in the process of reading it).

LGTM


Separate Logs for each plot

Hello!
I'm trying to separate  logs produced by the plotter for each plot which will make it easier to analyze them later on.
The problem is that the plotter does not end his process after a plot is finished as it continues copying the plot to the destination in the background. Currently if I would loop the process with bash it will only start a new plot after the destination copy process is finished.
There is also no time output anywhere produced by the plotter.
Any ideas?
screen -d -m -S plotter bash -c 'ulimit -n 3000 && /usr/bin/plotter \
-n -1 \
-r 48 \
-u 512 \
-t /mnt/tmp/ \
-2 /mnt/plotter/ \
-d /mnt/cache/ \
-p xxxxxxxxxxxxx \
-f xxxxxxxxxxxxx \
|tee /home/chia/logs/plotter.log'

make a script with a loop that will create a single plot and add $(date) to the name of your log file

@buglantiss My problem is that I don't know when the plot is done as the process ends as the copying phase is finished.

Don't give it a final destination and run another script in parallel that keeps looking for finished plots and then moves them while your plotter is already working on the next one.

For those wanting separate logs for each run with time started:
#!/bin/bash
log=/path/to/log/folder
for i in `seq 1 300`;
do
   dt=$(date '+%Y-%m-%d_%H_%M_%S')
   echo "Currently plotting number $i and time started is $dt."  |tee -a $log/$dt.log
done

@Dartellum B y removing the "-n" parameter the plotter gives an exit code before copying is finished?

No it will wait for copy to finish

For those wanting separate logs for each run with time started:
#!/bin/bash
log=/path/to/log/folder
for i in `seq 1 300`;
do
   dt=$(date '+%Y-%m-%d_%H_%M_%S')
   echo "Currently plotting number $i and time started is $dt."  |tee -a $log/$dt.log
done

I made a complete wrapper script that will check final destination disk space and select one based on free space available. See:  https://github.com/Dartellum/chia-madmax-plot_loop.


feat: org_log_repeat

Separate logging settings for repeating TODOs
Implements #530
Unfortunately I couldn't figure out how to redraw the agenda view without setting redo in:
function Agenda:change_todo_state()
  return self:_remote_edit({
    action = 'org_mappings.todo_next_state',
    redo = true,
  })
end

I did some refactoring and added support for this in https://github.com/nvim-orgmode/orgmode/pull/738, so closing.
Why do you need to redraw the whole agenda?

It's not strictly necessary, but I figured the entry should disappear from the agenda without having to manually refresh it since it's no longer accurate.
Anyways, thanks for implementing this.


Feature request: System neutral character sheet

Could this be made into a character sheet, much like the clock module is? I do think this could be an extremely useful way to display info to the players or have players have their own project sheet outside 5e. I would love to use this as a faction sheet that players could sometimes check and see their status with factions in the world.

I'm pinning this request because people ask me about making this module system agnostic a lot, and having the answer immediately visible for anyone else who comes along to ask is probably valuable.
Short answer is no. Expanding what's here for 5e to every other system out there isn't really... realistic, unfortunately. Not every system handles skill checks, DC's, etc the same way (or even uses them at all), and I simply don't have the time (or desire) to build out a framework that could be dropped into any system and provides a similar level of automation.
Longer answer is... Maybe, but it'd be an entirely different module. If automation isn't a requirement, then it's certainly possible for me to cook something up that can be dropped into any system and used to keep track of things manually. Maybe have the option to hook activities up to macros so if people have code know-how they can do more advanced, system-specific things with it. Effectively you'd have a sheet with a bunch of items on it, and the items only have names, max values, current values, and maybe options for which players can see things. Wanna change progress? Edit the activity and put in the new number. That's all you have. Does it work? Yeah. Could you use it for faction rep? Yeah. Is it as cool as the 5e version? Not even close.
I don't know if I have enough room on my plate to develop something like that right now, but it could happen down the line. If I can eventually get to it, are there any particular features or functionality you'd like to see?

Honestly, I don't think any automation is required. Just tracking progress in some project. Like in 5e, I would love to just have a sheet displaying their party's faction status with different entities, and having it immediately there. There are no rolls to be made, just easy to update and adjust as we go on.  Just makes life easier.

Alright that seems pretty tame and manageable. I might have some time this weekend to throw together a prototype version; I'll toss a link in here when it's done.