text:
How to allocate more memory to my Ryzen APU's GPU?

I am running AMD 6800U on my Ubuntu 22.04 and I installed the AMD driver. I checked that the default system would allocate 512MB RAM to VRAM to the GPU.
I followed some instruction from other github issue to create a rocm/pytorch docker image and it has no problem detecting my GPU but it has problem running sample program, due to OutOfMemoryError.
torch.cuda.OutOfMemoryError: HIP out of memory. Tried to allocate 2.00 MiB (GPU 0; 512.00 MiB total capacity; 150.39 MiB already allocated; 312.00 MiB free; 168.00 MiB reserved in total by PyTorch) If reserved memory is >> allocated memory try setting max_split_size_mb to avoid fragmentation.  See documentation for Memory Management and PYTORCH_HIP_ALLOC_CONF

So my guess is ROCm support APU but I just need to allocate more system memory to my GPU before going into the docker environment. Are there any people who know how to modify the memory allocation of AMD APU? Thanks in advance

An alternative route is to change the DMC buffer to 2GB in your BIOS.

Thanks
I tried to find that but it doesn't exist. Also it seems to me that AMD APU memory setting are now on driver level

It appears ROCm does not take into account dynamic VRAM GTT allocation on APUs (handled by amdkfd?).
For example on my system:
[    3.524465] [drm] amdgpu: 64M of VRAM memory ready
[    3.524466] [drm] amdgpu: 15916M of GTT memory ready.

This means the iGPU could allocate up to 16GB of VRAM, however due to the small dedicated VRAM (64MB) that is reserved for the iGPU, ROCm processes fail to allocate memory.
This also seems to propagate to OpenCL:
$ clinfo |grep -i memory
  Global memory size                              67108864 (64MiB)
  Global free memory (AMD)                        65536 (64MiB) 65536 (64MiB)
  Global memory channels (AMD)                    4
  Global memory banks per channel (AMD)           4
  Global memory bank width (AMD)                  256 bytes
  Max memory allocation                           57042528 (54.4MiB)
  Unified memory for Host and Device              No
  Shared Virtual Memory (SVM) capabilities        (core)
  Global Memory cache type                        Read/Write
  Global Memory cache size                        16384 (16KiB)
  Global Memory cache line size                   64 bytes
  Local memory type                               Local
  Local memory size                               65536 (64KiB)
  Local memory size per CU (AMD)                  65536 (64KiB)
  Local memory banks (AMD)                        32

This is NOT the case for OpenGL tho (snippet of glxinfo -B):
Memory info (GL_ATI_meminfo):
    VBO free memory - total: 206 MB, largest block: 206 MB
    VBO free aux. memory - total: 15138 MB, largest block: 15138 MB
    Texture free memory - total: 206 MB, largest block: 206 MB
    Texture free aux. memory - total: 15138 MB, largest block: 15138 MB
    Renderbuffer free memory - total: 206 MB, largest block: 206 MB
    Renderbuffer free aux. memory - total: 15138 MB, largest block: 15138 MB
Memory info (GL_NVX_gpu_memory_info):
    Dedicated video memory: 256 MB
    Total available memory: 16172 MB

@Ristovski Thanks for the answer
I think this is a needed feature (otherwise all the APU would be ROCm-compatible but it doesn't work due to the memory allocation problem).
I am not sure if this should be fixed on the driver level or the ROCm level (or both).

This bug is indeed super annoying. Most laptop models only provide limited settings regarding UMA buffer size. So ROCm is quite limited or useless for APUs on Linux.

Thanks @asbachb I plan to just use my laptop just to do some primitive testing with a subset of data. I didn't plan to some fancy calculation with all the data. But since I am using Ryzen CPU I think I could still get acceleration using the GPU with the subset.

Just for reference some guy got some quite good results: (compared to CPU only): https://www.gabriel.urdhr.fr/2022/08/28/trying-to-run-stable-diffusion-on-amd-ryzen-5-5600g/
But he is able to set his VRAM to 16GB.

May I share what I saw in Windows envoirnment?
My laptop has 16GB of memory. This is what is being shown inside task manager

It shows that the

Dedicated GPU memory is 512MB
Shared GPU memory is 7.6GB
GPU memory is 8.1GB (It should be 7.6GB+512MB)

While I play GTA it shows that I have 8GB of GPU memory. I would wonder if it is possible that the ROCm would have the similar fashion, "thinking" that my GPU has 8GB of GPU memory.
But when I reboot back to my Ubuntu it shows that my GPU has 512MB of memory, which I guess that's the dedicated GPU memory that you can view in Windows. However in the application perspective it sees 8GB of memory, to me it should be the same as ROCm application should be able to see that my system should have 8GB of GPU memory, similar to GTA result in Windows.
It seems to me there is a missing piece in between.

@asbachb Sorry I forgot to answer the question in your post. First of all thank you very much.
I read the post and the author said that he modified the memory sharing behavior in BIOS. Which I don't have this settings in my laptop. Also in the previous answer the windows application sees the total GPU memory, not just the dedicated memory.

@winstonma, I'm the author of the "Stable diffusion on an AMD Ryzen 5 5600G" post. Sadly, I did not find any way to  increase the VRAM allocated to the iGPU without going to the BIOS/motherboard firmware.

@randomstuff Thanks. But sadly the BIOS of my laptop doesn't have the VRAM allocation option. So I wonder if there are ways to increase the dedicated memory on system level (similar to Regedit in Windows)

PyTorch  will work with replaced allocator which uses hipHostMalloc. I have a working snippet at https://github.com/pomoke/torch-apu-helper.

For all who is interested in increasing dedicated memory in system, I wrote an article Unlocking GPU Memory Allocation on AMD Ryzen™ APU and thought that method I could adjust the amount of dedicated GPU memory, even the BIOS doesn't provide that option.

PyTorch will work with replaced allocator which uses hipHostMalloc. I have a working snippet at https://github.com/pomoke/torch-apu-helper.

Thanks.
I currently using alternative ways to unlock the BIOS, modify the amount of GPU dedicated memory to 8GB. After that I could run stable-diffusion-webui using the above method without any problem.
Also I cloned your repo, created alloc.so and add your code snippet in the launch.py of stable-diffusion-webui. When I start running stable diffusion then it starts with a warning (but it can start):
Warning: caught exception 'CUDAPluggableAllocator does not yet support getDeviceStats. If you need it, please file an issue describing your use case.', memory monitor disabled

However when I start generating image I got the following error:
RuntimeError: CUDAPluggableAllocator does not yet support getDeviceStats. If you need it, please file an issue describing your use case.

And no image is being generated. I am not sure if I miss anything. @pomoke not sure if you feel free to try running stable-diffusion-webui on your APU. Thanks

In conclusion, there are several solutions for other AMD APU user

Override the BIOS settings to allocate more memory. This method like the old days that you set your dedicated video memory in BIOS. More VRAM means less system memory.
torch-apu-helper uses the the Unified Memory Architecture (UMA), the APU would be able to allocate the memory from the system dynamically. It is a good demo but this way would not get all API working (e.g. getDeviceStats). If you are using application based on PyTorch, it would be likely that it would not work. I filed an issue on PyTorch, hopefully they can add AMD APU support.

Generally, we still need something like what we get on Tegra (Such as Jetson Developer Kit). CUDA on Tegra is capable of allocating GTT properly.

Yes I think the problem now is the torch-apu-helper shows that PyTorch could grab system memory from the system as video memory. However I think it still need more PyTorch API porting so we can say this way is PyTorch ready. Not sure AMD or PyTorch would make this possible in the future.

Yes I think the problem now is the torch-apu-helper shows that PyTorch could grab system memory from the system as video memory. However I think it still need more PyTorch API porting so we can say this way is PyTorch ready. Not sure AMD or PyTorch would make this possible in the future.

Hi, I'm trying to follow your guide https://winstonhyypia.medium.com/amd-apu-how-to-modify-the-dedicated-gpu-memory-e27b75905056 but when I run this command SA_OVERRIDE_GFX_VERSION=10.3.0 python test-rocm.py
I get this message:
Command "python" not found. Maybe your meant:
"python3" command from the deb package "python3".
the "python" command from the deb package "python-is-python3".
So I changed the command to HSA_OVERRIDE_GFX_VERSION=10.3.0 python3 test-rocm.py
But I get this message now:
Traceback (most recent call last):
File "/home/myUser/test-rocm.py", line 1, in
import torch, grp, pwd, os, subprocess
ModuleNotFoundError: No module named 'torch'
I can't find any solution, and I'm not a programmer person.
Thanks for your time!

Yes I think the problem now is the torch-apu-helper shows that PyTorch could grab system memory from the system as video memory. However I think it still need more PyTorch API porting so we can say this way is PyTorch ready. Not sure AMD or PyTorch would make this possible in the future.

Hi, I'm trying to follow your guide https://winstonhyypia.medium.com/amd-apu-how-to-modify-the-dedicated-gpu-memory-e27b75905056 but when I run this command SA_OVERRIDE_GFX_VERSION=10.3.0 python test-rocm.py
I get this message: Command "python" not found. Maybe your meant: "python3" command from the deb package "python3". the "python" command from the deb package "python-is-python3".
So I changed the command to HSA_OVERRIDE_GFX_VERSION=10.3.0 python3 test-rocm.py
But I get this message now: Traceback (most recent call last): File "/home/myUser/test-rocm.py", line 1, in import torch, grp, pwd, os, subprocess ModuleNotFoundError: No module named 'torch'
I can't find any solution, and I'm not a programmer person.
Thanks for your time!

For PyTorch installation you need to check the PyTorch Official website. Here are the steps:

Go to PyTorch Official Website
Scroll down a bit. You will see the widget like the one below

Just choose the option like the above image
You will get the command you need at the end. Then just copy and paste the command on your terminal.

Yes I think the problem now is the torch-apu-helper shows that PyTorch could grab system memory from the system as video memory. However I think it still need more PyTorch API porting so we can say this way is PyTorch ready. Not sure AMD or PyTorch would make this possible in the future.

Hi, I'm trying to follow your guide https://winstonhyypia.medium.com/amd-apu-how-to-modify-the-dedicated-gpu-memory-e27b75905056 but when I run this command SA_OVERRIDE_GFX_VERSION=10.3.0 python test-rocm.py
I get this message: Command "python" not found. Maybe your meant: "python3" command from the deb package "python3". the "python" command from the deb package "python-is-python3".
So I changed the command to HSA_OVERRIDE_GFX_VERSION=10.3.0 python3 test-rocm.py
But I get this message now: Traceback (most recent call last): File "/home/myUser/test-rocm.py", line 1, in import torch, grp, pwd, os, subprocess ModuleNotFoundError: No module named 'torch'
I can't find any solution, and I'm not a programmer person.
Thanks for your time!

First of all, you need to make sure that you are
1. Running Linux (I personally prefer Ubuntu LTS but any major linux release would do)

2. You are running Ryzen APU, you can go to [AMD APU Website](https://www.amd.com/en/products/specifications/apu) and see if you can find your CPU

For PyTorch installation you need to check the PyTorch Official website. Here are the steps:
1. Go to [PyTorch Official Website](https://pytorch.org/get-started/locally/)

2. Scroll down a bit. You will see the widget like the one below
   ![image](https://private-user-images.githubusercontent.com/1215090/293612351-b34133a5-9574-49e8-aeca-c1efcb1b7dc3.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MDQxNTUyMjIsIm5iZiI6MTcwNDE1NDkyMiwicGF0aCI6Ii8xMjE1MDkwLzI5MzYxMjM1MS1iMzQxMzNhNS05NTc0LTQ5ZTgtYWVjYS1jMWVmY2IxYjdkYzMucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI0MDEwMiUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNDAxMDJUMDAyMjAyWiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9MDdjZjdmYTNlNjgzMGEwNDQyMTc3MGQ4OGMxZDFkZmE4YjFkMzU2MzYwMjU1MjI5Njk0NDJlNTEyZjVkZTkzYyZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QmYWN0b3JfaWQ9MCZrZXlfaWQ9MCZyZXBvX2lkPTAifQ.Sci9x2SI3xvOmpY82FKL4XLMMQjVcbHlUta_Lg5GmiM)
   Just choose the option like the above image

3. You will get the command you need at the end. Then just copy and paste the command on your terminal. Then wait til the installation is completed

4. After PyTorch is installed, you can rerun the test script again

Hi, thanks for your fast response.
I run the command and I get this message:
Checking ROCM support...
Cannot find rocminfo command information. Unable to determine if AMDGPU drivers with ROCM support were installed.
So that means that I'm stuck with 512mb vram? (My bios don't have the option to modify it ).

rocminfo exist only after AMD Graphic Driver is installed. You need to run the following command to install the AMD Driver
# Install AMD Driver
TEMP_FOLDER="/tmp"
TEMP_DRIVER_HTML="amd-driver.html"
DISTRO_CODENAME=$(lsb_release --codename --short)

# Find the package URL from AMD website
AMD_DRIVER_URL="https://www.amd.com/en/support/linux-drivers"
URL_RESPONSE=$(wget -U 'Mozilla/5.0' -qO- ${AMD_DRIVER_URL})
AMD_DEB_URL=$(echo $URL_RESPONSE | grep -o 'https://[^ "<]*.deb' | grep $DISTRO_CODENAME | head -1)
FILENAME=$(basename $AMD_DEB_URL)

# Download and install the driver package
wget -P $TEMP_FOLDER $AMD_DEB_URL
sudo dpkg -i $TEMP_FOLDER/$FILENAME
rm $TEMP_FOLDER/$FILENAME

amdgpu-install -y --usecase=rocm

You can also find this snippet inside the document that I wrote.
By the way I suggest you running the following command first, I would like to ensure your APU could be supported
$ lscpu | grep "Model name"

It appears ROCm does not take into account dynamic VRAM GTT allocation on APUs (handled by amdkfd?).
For example on my system:
[    3.524465] [drm] amdgpu: 64M of VRAM memory ready
[    3.524466] [drm] amdgpu: 15916M of GTT memory ready.

This means the iGPU could allocate up to 16GB of VRAM, however due to the small dedicated VRAM (64MB) that is reserved for the iGPU, ROCm processes fail to allocate memory.
This also seems to propagate to OpenCL:

If you use the mesa rusticl opencl driver you will find that it uses the GTT memory as well as the VRAM.

I guess the reason the AMD Graphics Driver doesn't officially support default Ubuntu kernel or your kernel is too old. I think you should run the following step to install the latest default Ubuntu kernel:
sudo apt update
sudo apt install linux-generic-hwe-22.04

Then reboot your system, and wait for the GRUB boot menu comes up. Once in the GRUB menu, select the Advanced options for Ubuntu using the arrow keys and press Enter. Then select the default Ubuntu kernel (in my system it is Ubuntu, with Linux 6.2.0-39-generic).
Then go back to terminal and check what kernel is running on your system (this is my output to let you know what is the expected output):
$ awk -F\' '/menuentry / {print $2}' /boot/grub/grub.cfg
Ubuntu
Ubuntu, with Linux 6.6.8-zabbly+
Ubuntu, with Linux 6.6.8-zabbly+ (recovery mode)
Ubuntu, with Linux 6.2.0-39-generic
Ubuntu, with Linux 6.2.0-39-generic (recovery mode)

Memory test (memtest86+x64.efi, serial console)
Windows Boot Manager (on /dev/nvme0n1p1)
UEFI Firmware Settings

The installed kernel is listed between Ubuntu and the extra line (so in my system it is Linux 6.6.8-zabbly+ and Linux 6.2.0-39-generic, ignore the recovery mode option). If you see not only Linux 6.2.0-39-generic is installed, remove that kernel (please check this guide for reference.
The default system for installing AMD Graphic Driver should have the output similar to this:
$ awk -F\' '/menuentry / {print $2}' /boot/grub/grub.cfg
Ubuntu
Ubuntu, with Linux 6.2.0-39-generic
Ubuntu, with Linux 6.2.0-39-generic (recovery mode)

Memory test (memtest86+x64.efi, serial console)
Windows Boot Manager (on /dev/nvme0n1p1)
UEFI Firmware Settings

Only one default Ubuntu kernel.
After you make sure everything is fine. Then reinstall the AMD Graphics Driver
amdgpu-uninstall
amdgpu-install -y --usecase=rocm

It appears ROCm does not take into account dynamic VRAM GTT allocation on APUs (handled by amdkfd?).
For example on my system:
[    3.524465] [drm] amdgpu: 64M of VRAM memory ready
[    3.524466] [drm] amdgpu: 15916M of GTT memory ready.

This means the iGPU could allocate up to 16GB of VRAM, however due to the small dedicated VRAM (64MB) that is reserved for the iGPU, ROCm processes fail to allocate memory.
This also seems to propagate to OpenCL:

If you use the mesa rusticl opencl driver you will find that it uses the GTT memory as well as the VRAM.

Just wonder if Pytorch use GTT if I use the mesa rusticl opencl driver. If mesa rusticl opencl driver doesn't work with Pytorch don't use GTT by the end then I still have to use this hacky way to fix the video memory.

Hi, working on the great initiative of https://github.com/pomoke/torch-apu-helper
I have created this other code that would let us run any PyTorch code and force to use hipHostMalloc without changing a single line of code.
The way it works, it uses LD_PRELOAD to load the functions hipMalloc and hipFree before ROCm runtime and therefore is able to intercept those function calls and then forward them to hipHostMalloc and hipHostFree. hipHostMalloc uses GTT memory and there's no need to change VRAM in the BIOS.
LD_PRELOAD=./libforcegttalloc.so  HSA_OVERRIDE_GFX_VERSION=9.0.0 python your_pytorch_python_script.py
Check the code if you think it may be useful.
https://github.com/segurac/force-host-alloction-APU

Doesn't work on ollama :-(

@segurac great job! I've done some tests, and your memory allocator worked quite well!
To make it easier for future travellers, I've created this PyPI package with your code, so anyone can overcome the memory limitation by running this:
!pip install pytorch_rocm_gtt
import pytorch_rocm_gtt
pytorch_rocm_gtt.patch()

After that, I'm able to allocate way more than the 512MB dedicated to my Radeon 760M iGPU with something like this:
>>> torch.rand(1024 * 1024 * 600, dtype=torch.float32, device="cuda") * 10000
tensor([5880.2114, 1243.6473, 3648.3640,  ..., 3309.8118, 6534.8770,
        6562.5479], device='cuda:0')

Can someone give me a command line tutorial to use more than 2GB with Stable Diffusion? I managed to run it and it almost generated the picture. But I only got 2GB of VRAM and it doesn't have enough video memory to finish the picture. I'm alreadys thinking about download windows to allocate more if possible

@DerRehberg Which CPU are you running? Could you print the result of the following command? This is the output from my system:
$glxinfo -B
...
Memory info (GL_NVX_gpu_memory_info):
    Dedicated video memory: 512 MB
    Total available memory: 8126 MB
    Currently available dedicated video memory: 57 MB
...

@winstonma
Memory info (GL_NVX_gpu_memory_info):
Dedicated video memory: 2048 MB
Total available memory: 12977 MB
Currently available dedicated video memory: 1531 MB
AMD Ryzen 7 5825U

@DerRehberg Based on your output your system I think you dedicated 2GB of memory to BIOS. Could you check one more thing? I would like to see the GTT memory allowed.
$ sudo dmesg | grep amdgpu | grep drm
[    3.929856] [drm] amdgpu kernel modesetting enabled.
[    3.954417] [drm] amdgpu: 512M of VRAM memory ready
[    3.954419] [drm] amdgpu: 7614M of GTT memory ready.
[    4.271639] [drm] Initialized amdgpu 3.57.0 20150101 for 0000:03:00.0 on minor 0
[    4.283886] fbcon: amdgpudrmfb (fb0) is primary device
[    5.698795] amdgpu 0000:03:00.0: [drm] fb0: amdgpudrmfb frame buffer device

I think you need to first disable the dedicated memory. Please use feel free to read Unlocking GPU Memory Allocation on AMD Ryzen™ APU? - Prepare the bootable USB drive then go to the next section and set it back to AUTO

Go to Device Manager→AMD CBS→NBIO Common Option→GFX Configuration
In Integrated Graphics Controller, select Auto (I am not sure the correct wording but don't use Force)
In UMA Mode, select Auto (I am not sure the correct wording but don't use UMA_SPECIFIED)

After set it back to auto then you can use force-host-alloction-APU with Pytorch to run stable diffusion on your Graphic Card.
Please leave message if you still have question.

Can you give me a detailed instruction how to use force-host-alloction-apu on arch with stable diffusion?
[    5.278774] [drm] amdgpu kernel modesetting enabled.
[    5.392349] [drm] amdgpu: 2048M of VRAM memory ready
[    5.392352] [drm] amdgpu: 10929M of GTT memory ready.
[    6.617177] [drm] Initialized amdgpu 3.57.0 20150101 for 0000:03:00.0 on minor 1
[    6.624152] fbcon: amdgpudrmfb (fb0) is primary device
[    6.667498] amdgpu 0000:03:00.0: [drm] fb0: amdgpudrmfb frame buffer device
[ 1557.803120]  amdgpu_drm_ioctl+0x4e/0x90 [amdgpu]
[ 1557.803120]  amdgpu_drm_ioctl+0x4e/0x90 [amdgpu]

@DerRehberg I think you should have 10GB of memory. But you better
# Of coz you need to install PyTorch ROCm version to replace your PyTorch CPU version
$ pip uninstall torch torchvision torchaudio
$ pip3 install torch torchvision torchaudio --index-url https://download.pytorch.org/whl/rocm5.7

# Go to your Stable Diffusion folder

# Compile the library
$ git clone https://github.com/segurac/force-host-alloction-APU.git
$ CUDA_PATH=/usr/ HIP_PLATFORM="amd" hipcc force-host-alloction-APU/forcegttalloc.c -o force-host-alloction-APU/libforcegttalloc.so  -shared -fPIC
$ sudo mv force-host-alloction-APU/libforcegttalloc.so /usr/local/lib
$ rm -rf force-host-alloction-APU

$ LD_PRELOAD=libforcegttalloc.so HSA_OVERRIDE_GFX_VERSION=9.0.0 python launch.py

@winstonma Pip uninstall says I should use the AUR xd

@winstonma Why can't it load libforcegttalloc?

@DerRehberg I also have this exact random "GPU Hang" while testing mistral-7b (including my KDE crashing). One thing that helped a little bit was increasing the dedicated memory in BIOS (I can only increase to 2GB, and mistral-7b uses way more than that, but libforcegttalloc allows the extra allocation).
I still have this crash some times, but it appears to be less frequent this way.

@DerRehberg @pappacena Do you think replacing the /lib/firmware with latest version would help? (instruction). At least my laptop get smooth VP9 decode and sleep of death problem is gone after updating the firmware folder. So I personally think it's worth to try.

@winstonma I think my firmware is actually up to date
@pappacena My Stable Diffusion crashes but Ollama works with it

The system firmware (including wifi/bluetooth/chipset) is different from the firmware of the motherboard. I would recommend you to take a look. It fixes several problems of my laptop after me not using the linux-firmware (which is 2 years old on Ubuntu Jammy) comes from the system and use the git version. I think there are some advantage using the latest linux-firmware and kernel but still it is up to you.

I use Arch @winstonma

Use ROCm 5.7 for Vega-based iGPUs. Versions from ROCm 6.0 will break things as they dropped support for GCN without explicit error message.

Well ROCM works with any Software I can imagine on my APU with Version 6. But Stable Diffusion just plainly nicely gives me an out of memory error

@DerRehberg It is just a distracting story.
I just installed Ubuntu 24.04 and find out I don't need to install AMD ROCm Driver in order to get everything working.
I also checked your system and it seems to me that you also don't need to install AMD ROCm Driver. Just need to install hip-runtime-amd, compile force-host-allocation and everything should be working without the need of ROCm Driver.

@winstonma Well I got that package installed. Still crashes on me

@winstonma Google Translate Page doesn't support Japanese. And honestly. I just generate with my fucking cpu because I don't care anymore. If I use forcegtt my whole system crashes with stable diffusion

@winstonma I found an english article, I would love to try it but at the checking script if rocm works I get an libtorch_cpu error. When Updating torchvision-rocm i also get an error: ImportError: /usr/lib/libtorch_cpu.so: undefined symbol: cblas_gemm_f16f16f32
Idk it anything if fucked up rn

@winstonma Found it out. I only need to downgrade hip-runtime-amd to 5.7.1 and Stable Diffusion works with libforcegtt even tho the rest is at 6

Great to hear. That matched my experience on Ubuntu 24.04 too. No need to install ROCm package and DKMS modules. To me using the new way doesn't provide any performance benefit, just easier installation.
Just wonder if it fix the problem you faced?

@winstonma Well I tried downgrading all packages to 5.7.1 and it crashed on me. Updating everything except runtime worked for me, idk why tho

Just saw [https://www.phoronix.com/news/Linux-6.10-AMDKFD-Small-APUs](Linux 6.10 Improves AMD ROCm Compute Support For "Small" Ryzen APUs). Although @segurac patch works flawlessly, I still think the AMD should provide native support. Hopefully using 6.10 would no longer require additional library.

@winstonma which distro are you using?

EDIT: Just ran Linux Kernel 6.10-rc1 on my 6800U laptop. As expected I can run stable diffusion without using custom mod like force-host-alloction-APU or BIOS modification. AMD eventually fix this problem!

@Snuupy I am using Ubuntu 24.04.
By the way Linux 6.9.9 backported this feature already (You can check commit 8d656c in the changelog). I use Zabbly kernel, you can follow this instruction to install kernel.
FYI Linux Kernel 6.10 has a green block while playing video bug on my device so I am still using 6.9 right now.

@winstonma Thank you for such an informative reply! I'm also running Ubuntu 24.04. I didn't know Zabbly 6.9.9 backported this, thanks! I will try upgrading my kernel 😄

@Snuupy After the kernel support I no longer need to install AMD Driver. I guess you could try uninstall the driver and reinstall the driver if anything doesn't work for you.
As it takes time for the driver team to test on distro, I think it make sense for them to support only on LTS release. But again on Ubuntu 24.04 I didn't install AMD Driver. I only install latest 6.9 kernel and everything just work.
By the way if you want to improve the performance there is a small trick, you can use RyzenAdj to adjust the vrmmax-current. I use my AMD 6800U as an example. The default value is 90mA but I changed it to 105mA. After that I could get 10-15% increase in stable diffusion. You can try that too.

@winstonma I just installed the 6.9.10 kernel on my 22.04 LTS machine. In my BIOS for my UM780 (7840HS) I set VRAM to 16GB. In amdgpu_top I see: VRAM: 16384 MiB and GTT: 23984 MiB.
I am testing this model, 27B (29 GiB) https://huggingface.co/bartowski/Big-Tiger-Gemma-27B-v1-GGUF/resolve/main/Big-Tiger-Gemma-27B-v1-Q8_0.gguf in ollama/open-webui
I originally thought it would max out the VRAM and then use GTT (~16GB in VRAM, say the remaining ~13GB in GTT allocation). Instead however, I am seeing this:
ollama       | llm_load_tensors: offloading 22 repeating layers to GPU
ollama       | llm_load_tensors: offloaded 22/47 layers to GPU
ollama       | llm_load_tensors:      ROCm0 buffer size = 12624.05 MiB
ollama       | llm_load_tensors:        CPU buffer size = 27591.06 MiB

Where the ~12GB is loaded into GTT and VRAM usage is still at 0. Is this expected and what you see as well when you load your models?

@Snuupy I think you can set your VRAM to AUTO (and by default it should be 512MB). If there is no AUTO option then you might need to follow this article to set it to AUTO.
Also the max GTT memory would be half of your system memory. But I think you can change the value (please check this comment for more information). I never try that so please contact the commenter if that doesn't work.
Lastly you can check the amount GTT memory by running the command  sudo dmesg | grep GTT.

authors: DerRehberg DocMAX Ristovski Snuupy asbachb edt-xx gwyllion92 pappacena pomoke randomstuff segurac winstonma ye-luo

repo: RadeonOpenCompute/ROCm


text:
SPIR & SPIR-V binary support

Context
Our group is a happy user of Codeplays ComputeCpp, a fully conformant SYCL 1.2.1 implementation. With Intels rock solid OpenCL infrastructure(s, they too have many to chose from but all are SPIR enabled), it's a shame how AMD seems to have put OpenCL to the side. (CPU support being discontinued on Windows, AMD APP SDK removed from developer.amd.com, fragmentation of AMDs driver stacks…) When finally AMD and Intel had something to rival CUDA in a portable manner, AMD loses interest in supporting OpenCL, the underlying infrastructure. Luckily Codeplay goes around by adding non-conform additions to ComputeCpp, which is able to emit not just SPIR and SPIR-V, but also PTX and AMDGCN ISA. I understand that they need to do this for Nvidia who sabotages OpenCL to their dying breath, but I was expecting better of AMD.
The Wigner GPU Lab is dedicated to promote SYCL and grow its ecosystem (BLAS, ML, DNN done by Codeplay, initial PRNG by us, FFT planned if someone does not beat us to it, SPARSE up for grabs).
I would like to ask for AMD to pay a little more attention to OpenCL in general. I feel it is the best thing to do before GPGPU makes it into ISO C++, and even after that via vendor extensions of OpenCL & SYCL. IMHO the effort put into HCC would have been better put into upstream SYCL support for Clang which would have enabled a far wider audience of developers to write libraries that support AMD HW (NOT JUST GPUs!).
Sad example
TF 1.8 supporting ROCm enabled GPUs via rocPRIM is nice, but given the limited resources AMD can put into domain library support, I feel the common denominator should be put first. Not putting enough effort into OpenCL/triSYCL/ComputeCpp support for TF is the reason why anyone coming to take a look at the installation instructions for both Linux & Windows are greeted by

TensorFlow with GPU support. TensorFlow programs typically run significantly faster on a GPU than on a CPU. Therefore, if your system has a NVIDIA® GPU meeting the prerequisites shown below and you need to run performance-critical applications, you should ultimately install this version.

The message is simple: GPU == CUDA Intel CPUs and IGPs being ubiquitous in both consumer and workstations and having prime-time OpenCL support, it's a miracle how AMD who initially was all in for OpenCL cannot make it to the "front-page" together with Intel. And TF is a Google project! Google is fed up with NVCC! They "single-handedly" added CUDA support for Clang to get rid of closed-source Nvidia components. If only that effort were put into SYCL...
Summa summarum
Please add support for SPIR & SPIR-V binaries in ROCm (and all other) AMD runtime stacks.

FYI: https://github.com/RadeonOpenCompute/ROCm-OpenCL-Runtime/issues/57

@MathiasMagnus
We are investing in OpenCL, we have been working on SPIR-V and OpenCL 2.1.  Honestly, OpenCL is not very cross-platform any longer MAC OS X SPIR 1.2 was never supported on it, nor was it on NVIDIA, nor ARM or Imagination.  Only Intel and AMD supported.     Mac OS X is dropping support.
Your knowledge of the investment to get Tensorflow to run fast is limited.  It is a lot more then rocPRIM.  You might want to look at:  MIOpen, rocBLAS, rocFFT, rocSOLVER, rocRAND,  EIGEN port ( which is upstream).  We have lot more libraries coming. HIP is now a standard part of CLANG and LLVM.     Also SCYL port was a very limited port on what was needed to really get Tensorflow running competitively with NVIDIA.
Right now Google is very happy with AMD and the work we are doing to truly clean up Tensorflow to get it to support more then CUDA GPU.    Everything we do we bringing to Windows.  Step one is getting a platform in place that truly competitive with CUDA.
It your opinion SYCL would have been better investment  HCC was the foundation for a lot of what when into the SYCL spec.     triSCYLE was built from the Source Tree of our C++ AMP research compiler for HSA.  I know Ronan work in the same team as me  He also rewrote much of the orginal SYCL spec
Honestly, the best if really want this is nice and not bash the team.   Thank you for opinion

@MathiasMagnus  Looks like some solving the Sycl issue https://github.com/illuhad/hipSYCL for you..  This great to see also show the power of opensource software.   This give you NVIDIA and ROCm backends, still work in progress but great start.

source: gharchive/issue

authors: MathiasMagnus gstoner zpodlovics

license: MIT

license_type: permissive

license_source: github-api


text:
[Question] Current Viewers

I noticed the viewers drops to 0 when the streamer goes offline. Is this due to the Twitch API returns 0 when the streamer is offline or the sensor just defaults to 0 when offline? Reason I ask is if I'm watching someone and they go offline, I noticed the viewer count on the site still shows people.

Hi,
It's most likely due to the way the stream data is retrieved and handled. If the stream data cannot be retrieved because the the stream is offline, then the python script will fall back and set the state to STATE_OFFLINE. The other values should default to None, which I think might be translated to 0.

authors: Krojack Radioh


text:
Information gathering

Give students access to statistics in exchange for information

Unlock matching feature in exchange for info.
Ex: https://remote.com/jobs/browse

repo: RafaelAPB/GCE-NEIIST


text:
[FR]: Bidirectional relations

Contact Details
No response
Present your request
When I add a relation to a page I would like a reciprocal to appear on the linked page.
EG: I link a video idea/project with plugin.
Video database -> Plugin database
The video has a relation inline field but the plugin doesn't.
I need to make the relation in the plugin database to the video database.
Plugin database -> Video database
Can we make this automatic?
For which platform do you request this request??
Desktop

https://github.com/RafaelGB/obsidian-db-folder/issues/581
duplicated

authors: DannyHatcher-Git RafaelGB

repo: RafaelGB/obsidian-db-folder


text:
[Bug]: csv export produces blank file

Contact Details
mylenrath@gmail.com
What happened?
I have several database folders with different fields and data.  When I try to export to csv from any database, the result is always a blank file.  Running version 3.2.4
What platform were you using?
Desktop
Version of the plugin
3.2.4
Relevant log output
No response
Are you using the latest version of the plugin in your Obsidian vault?

[X] I have verified that I am on the latest version

Are you check if there is a similar issue?

[X] I have verified that there are not similar issues

Hello @mylenrath ,
Could you share your db and some mocked file to reproduce the problem?
We are transitioning to using the paparse library due csv was not being maintained. We hope in a short term to be able to solve it.
You have available the beta version 3.3.0 where some of these changes have already been introduced.

Heres my database
On Thu, Jan 26, 2023 at 10:35 AM Rafael Gómez Bermejo <
@.***> wrote:

Hello @mylenrath https://github.com/mylenrath ,
Could you share your db and some mocked file to reproduce the problem?
We are transitioning to using the paparse library due csv was not being
maintained. We hope in a short term to be able to solve it.
You have available the beta version 3.3.0 where some of these changes have
already been introduced.
—
Reply to this email directly, view it on GitHub
https://github.com/RafaelGB/obsidian-db-folder/issues/782#issuecomment-1405358819,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFQKF5GTVF7TYH3IQU5EKRTWUKYW7ANCNFSM6AAAAAAUHX5BTY
.
You are receiving this because you were mentioned.Message ID:
@.***>

Commit 737eceae
will be fixed with a custom solution. react-css library was removed
Ready for the next version

authors: RafaelGB mylenrath


text:
Better documentation

Add better documentation and examples on the lua metatables functions. This will greatly help the AAC script creator (reading comments to get functions)

Add wiki pages

The documentation site is now live!

repo: Raggaer/castro

license: mit

license_source: bigquery


text:
In compose, tool tip to go to HTML mode says "Plain text"

The tool-tip for HTML mode doesn't seem to be correct, it doesn't change between modes and always shows "Plain text"

This seems fixed. It now shows as HTML <-> Plain Text for both modes.

Please close the issue if solved.

repo: RainLoop/rainloop-webmail


text:
🛑 RSOS服务控制面板 is down

In d4fae82, RSOS服务控制面板 (https://$US_SITE) was down:

HTTP code: 530
Response time: 285 ms

Resolved: RSOS服务控制面板 is back up in 97ab13b after 11 minutes.

repo: RainThings/uptime


text:
🛑 HK区服 is down

In 5ac0944, HK区服 (https://$HK_SERVER) was down:

HTTP code: 0
Response time: 0 ms

Resolved: HK区服 is back up in 2eb9410.


text:
Bug: DBFlow returns repeated values with simple query due to caching

DBFlow Version: 4.0.0-beta1 with Kotlin
Question
Description:
While using cache in a simple data class as shown here:
@Table(database = MyMainDatabase::class, cachingEnabled = false, cacheSize = 32)
data class MyClass(
        @PrimaryKey @Column
        var id: Long = 0,

        @Column
        var name: String = "",

        @Column
        var price: Double = 0.0,

        @Column
        var someString: String = "",

        @Column(getterName = "getSomeBoolean")
        var someBoolean: Boolean = false,

        @Column
        var someNullableLong: Long? = null
                  ) : Model
{
    companion object
    {
        @ModelCacheField
        @JvmField var modelCache: ModelCache<MyClass, *> = SparseArrayBasedCache()
    }

    override fun load() = modelAdapter<MyClass>().load(this)

    override fun save() = modelAdapter<MyClass>().save(this)

    override fun delete() = modelAdapter<MyClass>().delete(this)

    override fun update() = modelAdapter<MyClass>().update(this)

    override fun insert() = modelAdapter<MyClass>().insert(this)

    override fun exists() = modelAdapter<MyClass>().exists(this)
}

While doing the following query:
SQLite.select().from(MyClass::class.java).queryList() I am reading a repeated value, regardless of the number of items in the database (i.e. If I have A,B,C,D in the data base, I am reading A,B,C,C). That is occurring when I use SparseArrayBasedCache. But when I use any of the other two caches, it doesn't happen at all. If I disable caching it also works as expected.
Could this be a bug or a problem on my side?
Thank you.

Is this issue being reviewed?

sorry for late response, did you discover what was happening? I would debug what is in the cache and what you are attempting to retrieve upon queryList() in terms of ids.

closing due to inactivity.

authors: agrosner frnusmartobjects

repo: Raizlabs/DBFlow


text:
Special Number Calculator added

Fixes Issue🛠

Closes #1282
Description👨💻

Special Number calculator added
Type of Change📄

[x] New feature (non-breaking change which adds functionality)
[x] Style (non-breaking change which improves website style or formatting)
[x] Documentation (non-breaking change which updates or adds documentation)
[x] Responsiveness (non-breaking change which improves UI/UX on different screen sizes)

Checklist✅

[x] I am an Open Source contributor
[x] I have performed a self-review of my code
[x] My code follows the style guidelines of this project

Screenshots/GIF📷

@shreya-paul-17 Please add empty field validations.

Are you working @shreya-paul-17

@Rakesh9100 another person has included a special number calculator but with a different logic. It is continuous conflicting with my files. How to resolve these?

You can rename your calculator folder to Special-Number-2-Calculator and make the necessary changes by making the new commits in this same PR @shreya-paul-17

There were some issues that were still existing in the current branch. So I created a new branch. Thus I created a new PR #1659 . @Rakesh9100 please check that PR.

source: gharchive/pull-request


text:
Document Mixy.pick/.grab/.kxxv

Potential fix for D#2205 https://github.com/perl6/doc/issues/2205
pending resolution of https://github.com/rakudo/rakudo/issues/2109
The problem
This was created by @zoffixznet some time ago. Before deleting, I guess it's fair to give it a chance to be reviewed.
Solution provided
It's take it or leave it, we accept it as is (and then change whatever) or we reject it and the branch is deleted.

taking as is, we can review again once the rakudo issue is done.

authors: JJ coke

repo: Raku/doc


text:
Upgrade to new Mojolicious

New major release of Mojolicious is out. We need to upgrade to it and de-bitrot anything that breaks.

Presently the main problem is that Toadfarm is no longer being updated, and is incompatible with the current Mojolicious. Downgrading to Mojolicious-8.73 does work. I believe removing the Toadfarm dependency is the best way to go.
The built-in Mojolicious::Plugin::PODRenderer plugin is no longer available. It has been revived as an external fork, Mojolicious::Plugin::PODViewer.

authors: softmoth zoffixznet


text:
support for ignoring industry/repository specific fields

Dear @RalphTro,
This PR is to address #109 . Please have a look and share your feedback.
Please note that I have kept the key and schema as per below as discussed
"repository-x": "https://repository-x.example.com/"

Super, @dakbhavesh !  Many thanks.
I reserved some time for testing this in my calendar and will keep you posted.

Dear @dakbhavesh ,
Happy to report that all tests with JSON/JSON-LD files were successful. So, MANY THANKS for this!
A follow-up question: do we also want to support this feature in XML? If yes, how do you envision to list the fields to be ignored?
I drafted/tested the following XML query doc:
<soap:Envelope xmlns:soap="http://schemas.xmlsoap.org/soap/envelope/">
   <soap:Body>
      <ns4:QueryResults xmlns:epcis="urn:epcglobal:epcis:xsd:1"
         xmlns:ns3="http://www.unece.org/cefact/namespaces/StandardBusinessDocumentHeader"
         xmlns:ns4="urn:epcglobal:epcis-query:xsd:1" xmlns:example="https://ns.example.com/epcis/"
         xmlns:repository-x="https://repository-x.example.com/">
         <queryName>SimpleEventQuery</queryName>
         <resultsBody>
            <EventList>
               <ObjectEvent>
                  <eventTime>2015-03-30T10:58:56.591+02:00</eventTime>
                  <recordTime>2015-03-30T15:38:32.052+02:00</recordTime>
                  <eventTimeZoneOffset>+02:00</eventTimeZoneOffset>
                  <epcList />
                  <action>ADD</action>
                  <bizStep>urn:epcglobal:cbv:bizstep:commissioning</bizStep>
                  <readPoint>
                     <id>urn:epc:id:sgln:4054738.00005.0</id>
                  </readPoint>
                  <extension>
                     <quantityList>
                        <quantityElement>
                           <epcClass>urn:epc:class:lgtin:4054739.099961.2015-03-30a</epcClass>
                           <quantity>150</quantity>
                           <uom>KGM</uom>
                        </quantityElement>
                     </quantityList>
                  </extension>
                  <example:testField1>123</example:testField1>
                  <example:testField2>ABC</example:testField2>
               </ObjectEvent>
            </EventList>
         </resultsBody>
         <repository-x:ignoreFields>
            <example:testField1 />
         </repository-x:ignoreFields>
      </ns4:QueryResults>
   </soap:Body>
</soap:Envelope>

This doesn't work though like in JSON/JSON-LD.
Many thanks in advance for sharing your thoughts.
Kind regards,
Ralph

Dear @RalphTro, Your proposal for mentioning fields to ignore in hash calculation for XML doc seems fine to me. I will try to incorporate that in logic as soon as possible.
Thanks for your patience.

Super. Many thanks, @dakbhavesh!

Good morning @dakbhavesh,
Thanks so much for this! All the tests I conducted were successful.
For the time being, I am happy to accept/merge this PR. We may revisit the implementation (and slightly adjust the code) if @Echsecutor can present compelling use cases for the (formerly discussed) option to indicate fields to be ignored in the event hash ID.
If so, we need to discuss how this should work if the eventID field is populated with a UUID URI by the EPCIS capture application. At this moment of time, I personally still think that having a field in the EPCIS document that specifies all fields to be disregarded is the better option. But let's see.
Anyway - again MANY THANKS for this and wishing you a great week ahead!
Kind regards,
Ralph

authors: RalphTro dakbhavesh

repo: RalphTro/epcis-event-hash-generator


text:
Propagate VR condition error message to protected pvc conditions

When a VR condition is not met, we set the protected PVC condition
message using the error message returned from isVRConditionMet(). When
using csi-addons > 0.10.0, we use now the message from the condition
instead of the default message.
Since the Validated condition is not reported by older version of
csi-addons, and we must wait until the Validated condition status is
known when VRG is deleted, isVRConditionMet() returns now also the state
of the condition, which can be:

missing: condition not found
stale: observed generation does not match object generation
unknown: the special "Unknown" value
known: status is True or False

When we validate the Validate condition we have these cases:

Condition is missing: continue to next condition.

Condition is met: continue to the next condition.

Condition not met and its status is False. This VR will never
complete and it is safe to delete since replication will never start.
If VRG is deleted, we return true since the VR reached the designed
state. Otherwise we return false. In this case we updated the
protected pvc condition with the message from the VR condition.

Condition is not met and is stale or unnown: we need to check again
later. There is no point to check the completed condition since a VR
cannot complete without validation.In this case we updated the
protected pvc condition with the message generated by
isVRConditionMet() for stale or unknown conditions.

Example protected pvc DataReady condition with propagated message when
VR validation failed:
conditions:
  - lastTransitionTime: "2024-11-06T15:33:06Z"
    message: 'failed to meet prerequisite: rpc error: code = FailedPrecondition
      desc = system is not in a state required for the operation''s execution:
      failed to enable mirroring on image "replicapool/csi-vol-fe2ca7f8-713c-4c51-bf52-0d4b2c11d329":
      parent image "replicapool/csi-snap-e2114105-b451-469b-ad97-eb3cbe2af54e"
      is not enabled for mirroring'
    observedGeneration: 1
    reason: Error
    status: "False"
    type: DataReady

[!NOTE]
Using development build of csi-addons adding for testing.
We don't depend on the csi-addon release to merge this fix, but it will
be affective only when using csi-addons including this change:
https://github.com/csi-addons/kubernetes-csi-addons/pull/691

Works for VR conditions other then Validated. We propagate the messages form the VR conditions:
VR:
  status:
    conditions:
    - lastTransitionTime: "2024-11-05T14:32:43Z"
      message: failed to promote volume
      observedGeneration: 1
      reason: FailedToPromote
      status: "False"
      type: Completed
    - lastTransitionTime: "2024-11-05T14:32:43Z"
      message: failed to enable volume replication
      observedGeneration: 1
      reason: Error
      status: "True"
      type: Degraded
    - lastTransitionTime: "2024-11-05T14:32:43Z"
      message: volume is not resyncing
      observedGeneration: 1
      reason: NotResyncing
      status: "False"
      type: Resyncing
    - lastTransitionTime: "2024-11-05T14:32:43Z"
      message: 'failed to meet prerequisite: rpc error: code = FailedPrecondition
        desc = system is not in a state required for the operation''s execution: failed
        to enable mirroring on image "replicapool/csi-vol-f4737b6e-eeff-4137-8248-301cf37a3368":
        parent image "replicapool/csi-snap-e7c91292-a272-4278-9ee9-6be7a4c8bfe0" is
        not enabled for mirroring'
      observedGeneration: 1
      reason: PrerequisiteNotMet
      status: "False"
      type: Validated

VRG:
    protectedPVCs:
    - accessModes:
      - ReadWriteOnce
      conditions:
      - lastTransitionTime: "2024-11-05T14:32:43Z"
        message: failed to promote volume
        observedGeneration: 1
        reason: Error
        status: "False"
        type: DataReady
      - lastTransitionTime: "2024-11-05T14:32:44Z"
        message: PV cluster data already protected for PVC restored-pvc
        observedGeneration: 1
        reason: Uploaded
        status: "True"
        type: ClusterDataProtected
      - lastTransitionTime: "2024-11-05T14:32:44Z"
        message: failed to promote volume
        observedGeneration: 1
        reason: Error
        status: "False"
        type: DataProtected

Missing change: when Validated condition is False, we want to set the DataReady condition and DataProtected using the error message from the Validated condition. Currently we use the Validated condition only for checking if the VR is finished and can be removed.

Propgartion to protected pvcs message works now for all VR conditions:
    - accessModes:
      - ReadWriteOnce
      conditions:
      - lastTransitionTime: "2024-11-05T16:36:15Z"
        message: 'failed to meet prerequisite: rpc error: code = FailedPrecondition
          desc = system is not in a state required for the operation''s execution:
          failed to enable mirroring on image "replicapool/csi-vol-348f65fd-c658-4764-b7e7-85c45974e97e":
          parent image "replicapool/csi-snap-1ef6bed0-57e3-458f-8a99-413b823dde59"
          is not enabled for mirroring'
        observedGeneration: 1
        reason: Error
        status: "False"
        type: DataReady
      - lastTransitionTime: "2024-11-05T16:36:16Z"
        message: PV cluster data already protected for PVC restored-pvc
        observedGeneration: 1
        reason: Uploaded
        status: "True"
        type: ClusterDataProtected
      - lastTransitionTime: "2024-11-05T16:36:15Z"
        message: 'failed to meet prerequisite: rpc error: code = FailedPrecondition
          desc = system is not in a state required for the operation''s execution:
          failed to enable mirroring on image "replicapool/csi-vol-348f65fd-c658-4764-b7e7-85c45974e97e":
          parent image "replicapool/csi-snap-1ef6bed0-57e3-458f-8a99-413b823dde59"
          is not enabled for mirroring'
        observedGeneration: 1
        reason: Error
        status: "False"
        type: DataProtected

But we have 25 failed unit tests, need to understand why they fail.

We don't propagate the protected pvcs conditions to the drpc, so on the hub this does not help to debug the issue.
Maybe we can add list or errors messages from protected pvcs to make it easier to debug.
  status:
    actionDuration: 23.105201755s
    actionStartTime: "2024-11-05T17:02:08Z"
    conditions:
    - lastTransitionTime: "2024-11-05T17:02:01Z"
      message: Initial deployment completed
      observedGeneration: 1
      reason: Deployed
      status: "True"
      type: Available
    - lastTransitionTime: "2024-11-05T17:02:01Z"
      message: Ready
      observedGeneration: 1
      reason: Success
      status: "True"
      type: PeerReady
    - lastTransitionTime: "2024-11-05T17:02:02Z"
      message: VolumeReplicationGroup (ramen-ops/flatten-drpc) on cluster dr1 is reporting
        errors (All PVCs of the VolumeReplicationGroup are not ready) readying workload
        data, retrying till DataReady condition is met
      observedGeneration: 1
      reason: Error
      status: "False"
      type: Protected
    lastKubeObjectProtectionTime: "2024-11-05T17:02:04Z"
    lastUpdateTime: "2024-11-05T17:02:31Z"
    observedGeneration: 1
    phase: Deployed
    preferredDecision:
      clusterName: dr1
      clusterNamespace: dr1
    progression: Completed
    resourceConditions:
      conditions:
      - lastTransitionTime: "2024-11-05T17:02:02Z"
        message: All PVCs of the VolumeReplicationGroup are not ready
        observedGeneration: 1
        reason: Error
        status: "False"
        type: DataReady
      - lastTransitionTime: "2024-11-05T17:02:02Z"
        message: All PVCs of the VolumeReplicationGroup are not ready
        observedGeneration: 1
        reason: Error
        status: "False"
        type: DataProtected
      - lastTransitionTime: "2024-11-05T17:02:01Z"
        message: Nothing to restore
        observedGeneration: 1
        reason: Restored
        status: "True"
        type: ClusterDataReady
      - lastTransitionTime: "2024-11-05T17:02:04Z"
        message: Cluster data of all PVs are protected. Kube objects protected. Kube
          objects protected
        observedGeneration: 1
        reason: Uploaded
        status: "True"
        type: ClusterDataProtected
      resourceMeta:
        generation: 1
        kind: VolumeReplicationGroup
        name: flatten-drpc
        namespace: ramen-ops
        protectedpvcs:
        - restored-pvc
        resourceVersion: "15650"

@nirs I not very sure on how the final VRG will look like, can you please point out the comment from above?

@nirs I not very sure on how the final VRG will look like, can you please point out the comment from above?

This comment show the change in the vrg:
https://github.com/RamenDR/ramen/pull/1639#issuecomment-2457352330

LGTM, expect currently we are just setting error message to dataProtect and dataReady, but based on various conditions from VR, it should be populated and show messages accordingly.
@nirs If I am not wrong you are planning to bring this change later on as bug fix right?

LGTM, expect currently we are just setting error message to dataProtect and dataReady, but based on various conditions from VR, it should be populated and show messages accordingly. @nirs If I am not wrong you are planning to bring this change later on as bug fix right?

Setting the error message is the purpose of change. In normal condition we know the exact state so using the message from the VR is not very useful. We may simplify the code later to just use the message from the VR.
Another issue duplicating the content of DataReady and DataProtected conditions, which does not seems right, but this is not a new issue, and changing it is not in the scope of this change.

@nirs can we get one more approval on this and merge it?

@nirs can we get one more approval on this and merge it?

We don't need more approvals. We kept it to give time for more reviewers.

repo: RamenDR/ramen


text:
implementation project(':fluid-slider')

this is not working

Hello, VishStrqfes. This works only if you add fluid-slider as android studio module to your project. If you add it via dependency in your gradle dependency block you should't use implementation project(':fluid-slider').

repo: Ramotion/fluid-slider-android


text:
Klimamonitor indikator 4.1: Andel husdyrgødning anvendt til biogas

Data findes i Randers Kommunes energiregnskab fra PlanEnergi.
Opgaver

[x] Find datakilde
[x] Etabler extract
[x] Lav monitorering i Superset

Data uploadet til Open Data DK: https://admin.opendata.dk/dataset/biogas

Graf lavet i superset

authors: hkl-randers kf-randers


text:
Easier to find debug messages when debugging policies

What problem are you trying to solve?
I find myself spending a lot of time sifting through debug logs in rasa shell -vv trying to find the right line for where things went wrong.
Also, I'm testing in the shell because my failed test stories just tell me what was predicted, not why.
What's your suggested solution?
few different options:

more debug info in failed test story reports
bold / color differently the more important logs in rasa shell -vv (like what policy was eventually used)

Examples (if relevant)
No response
Is anything blocking this from being implemented? (if relevant)
No response
Definition of Done
No response

I wonder if it would be helpful to figure out what should be INFO versus DEBUG. The default could still be >=warning only, but in addition to rasa shell --debug, there could be something like rasa shell --info which could show only the basics like what message was received and how it was classified, and what action was chosen and the policy that chose that action 🤔

authors: desmarchris erohmensing

repo: RasaHQ/rasa


text:
Add new featurizer based on ConveRT model

Proposed changes:

Add a featurizer ConvertFeaturizer
Add a pipeline pretrained_embeddings_convert.

Status (please check what you already did):

[x] made PR ready for code review
[x] added some tests for the functionality
[x] updated the documentation
[x] updated the changelog
[x] reformat files using black (please check Readme for instructions)

also please add it to migration guide about constants renaming: if you have custom components ...


text:
fixed mypy operator error type

Proposed changes:

fixes #9058

Status (please check what you already did):

[x] enable mypy checks for operator
[x] all existing operator issues are fixed

Hi @twerkmeister, got back today so am just looking at this for the first time now. Could you let me know how I would check that the warnings for operator are all gone? I ran make types but this raises no issues here or on main.

I ran make types but this raises no issues here or on main.

operator issues are suppressed on main and hence raise no error there (you can edit remove it from the ignored section in setup.cfg as Thomas has done in this PR) to see all errors.
And I guess they are fixed on this branch which is why you can't see them here.

Love the cleanup 😍 Had a few small comments, otherwise looks quite good!

Thanks a lot for the comments :)

@twerkmeister is this one ready for re-review? not sure if you know, you can re-request reviews by clicking on that icon 🔁

authors: koernerfelicia m-vdb twerkmeister wochinge


text:
Build with LLVM fix

Changelog

CMakeLists1.txt updated to latest version of kernel_slicer
CMakeLists1.txt merged to CMakeLists.txt
Updated doc: README_build_with_llvm.md
Added script to integrate kernel_slicer into LLVM automatically

Note
Before merge there some additional steps.

Remove -b fix/build-with-llvm-update in integration script.
In README_build_with_llvm.md should replace github link with fix/build-with-llvm-update branch to master.

There is still some issues with llvm build.


text:
Failed to load native core functions! Couldnt find at location! (Ubuntu 20.04 + Debian 11)

Unhandled Exception:
System.IO.FileNotFoundException: Failed to load native core functions! Couldnt find at location /home/foo/Desktop/ReClass.NET-master/build/Debug/x64/NativeCore.so
I have #241 on both x64 and x86 builds on Ubuntu 22.04 and Debian 11

Built with docker and msbuild, error identical.


text:
No encode, decode in createLocation

Removes the automatic encoding of pathnames, leaving that up to the browser.
Moves the decoding to createLocation because 1) that is now used by the getDOMLocation calls to create a location and 2) we should ensure that if a user pushes a location whose pathname is encoded, we decode it for consistency.

Updated to just throw in createLocation when a pathname cannot be decoded. I believe that this makes the most sense to do, although it can be easily changed if you think otherwise @mjackson.
Internally, createPath is always passed a location object that was created by createLocation, so just having createLocation throw should be good enough. Having said that, createPath is exported by history, so maybe it should throw on its own?

The obvious issue with history throwing when it encounters a malformed URI is that developers will need to try...catch their code if they want to recover from bad percent-encoding. This would make it difficult to display an error page that looks like it is in their website.
try {
  ReactDOM.render((
    <BrowserRouter>
      <App />
    </BrowserRouter
  ), holder)
} catch (e) {
  ReactDOM.render((
    <MalformedURIMessage />
  ), holder)
}

I still think that history should throw and we can point to the RFC 3986 path parser rules to justify that, but there will still probably be people who want to use malformed URIs for some reason. I don't necessarily like the idea of automatically using the encoded pathname if the decode fails, but I think that it might be useful in the future to add an explicit "no decode" mode.

I still think that history should throw and we can point to the RFC 3986 path parser rules to justify that, but there will still probably be people who want to use malformed URIs for some reason

@pshrmn I think this is the right approach; an app using an invalid/malformed path can always be fixed to encode correctly. Thank you for fixing this.

Just bumping this since it has been a while. Whether or not this is the best approach, something definitely should be done here since the current implementation (made by myself :frowning_face:) breaks things.

@pshrmn I would love to see this added. 👍

This looks good to me. Thanks, @pshrmn!

Seems like this should probably be a patch release since we're just fixing a bug we introduced in 4.6.1. Would you agree?

I agree. More of a bug fix than "added functionality", so I don't think that it would warrant a minor bump.

As a minor addendum, the next release would also include #459, but that is also pretty much just a bug fix, so I don't think that it warrants a minor bump.

One more thing I just remembered is that #485 needs to be addressed before the next release.

OK, just got that one in for you.
@mjackson Did you want to push out a 4.6.2 release or add me to npm so I can take care of that for you? (you probably should add Ryan, BTW 😄)

Thanks, @timdorr. I'll go ahead and cut the release :)

authors: mjackson nil4 pshrmn timdorr

repo: ReactTraining/history


text:
Pinterest style modal statefulness

Hello, I posted this in the redux-router chat, but didn't get a response, so I apologize in advance for abusing the issue tracker.
We're trying to implement the Pinterest-style modal example, which we've done with success.
However, the container that launches the modal is itself stateful (i.e. contains items that were asynchronously fetched), so when the modal opens, it re-renders the whole page, and causes the container (previousChildren) to re-mount, which fires componentDidMount and causes the container to lose its state, in terms of pagination and items in memory, and triggers a fetch from the server.
In the the example the posts are hardcoded, so of course they don't change as a result of this re-render. Does anyone have an opinion on how to make this work for stateful/async fetched items?
Thanks in advance.

This is a bug tracker, not a support system. For usage questions, please use Stack Overflow or Reactiflux. Thanks!

authors: chrisbarless timdorr

repo: ReactTraining/react-router


text:
Link component is not rendered in modal using portal

Context is not passed when modal is opened using a portal, which appends the modal dom outside the context.
Version
react-router-dom 4.2.2
Test Case
https://codesandbox.io/s/yq0jy7n1nv  (without context provider)
https://codesandbox.io/s/00r23nynzw (with context provider)
When opened this issue with atlassian library, according to below link, it is coming from the latest version of react router. Please refer below link
https://bitbucket.org/atlassian/atlaskit-mk-2/issues/16/context-is-not-passed-to-modal-dialog
Steps to reproduce
Use Link component from react-router-dom to component rendered outside dom
Expected Behavior
Link should be rendered
Actual Behavior
getting error "You should not use  outside a "

It looks like they don't use React's official Portal API, so it's not surprising this doesn't work. One of the big things the official API added was context along the portal boundary. You won't be able to use "DIY portals" with React Router.

authors: dshivprasad timdorr


text:
provide a method in IX allowing batching

It would be nice if within IX we had a BatchAsync method that takes a size in and batches an IAsyncEnumerable sequence in to sized buckets.
I don't think I can do this with the current Create methods as I need to store state and the AnonymousAsyncEnumerable and the AnonymousAsyncIterator are both private.
Any help apreciated.
Cheers
Sean.

Are you looking for the Buffer method?
    IAsyncEnumerable<IList<TSource>> Buffer<TSource>(this IAsyncEnumerable<TSource> source, int count)

Hi,
Yes, it looks like that will do what I want.
Thanks, feel free to close this.
From: quinmars [mailto:notifications@github.com]
Sent: 20 October 2017 07:32
To: Reactive-Extensions/Rx.NET Rx.NET@noreply.github.com
Cc: Sean Farrow sean.farrow@seanfarrow.co.uk; Author author@noreply.github.com
Subject: Re: [Reactive-Extensions/Rx.NET] provide a method in IX allowing batching (#438)
Are you looking for the Bufferhttps://github.com/Reactive-Extensions/Rx.NET/blob/develop/Ix.NET/Source/System.Interactive.Async/Buffer.cs method?
IAsyncEnumerable<IList<TSource>> Buffer<TSource>(this IAsyncEnumerable<TSource> source, int count)

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Reactive-Extensions/Rx.NET/issues/438#issuecomment-338118172, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABY1fosOB391GN2LUXPx-HwFKzPhib90ks5suD5GgaJpZM4PxPVv.

Please close this.  @SeanFarrow even said it can be closed.

authors: SeanFarrow jzabroski quinmars


text:
package.json incorrectly defines location of Typescript types

Bug Report
Current Behavior
Installed rxjs into my TypeScript project.  Tried using rxjs but the TypeScript compiler gave me an error message saying
Could not find a declaration file for module 'rxjs'.
'D:/myWork/node_modules/rxjs/dist/cjs/index.js' implicitly has an 'any'
type.Try npm i --save-dev @types/rxjs if it exists or add a new declaration (.d.ts) file containing
declare module 'rxjs';
Expected behavior
TypeScript should be able to find the types defined inside the rxjs project.
Environment

RxJS version: 7.3.0
TypeScript v4.4.3
npm v7.18.1
node v14.15.3

Possible Solution
I had a look in rxjs's package.json file and I see
"types": "index.d.ts"
This is incorrect.   If I change this line to the following then everything starts working again.
"types": "./dist/types/index.d.ts"
I don't know why the "typesVersions" settings don't work.

It is not incorrect: https://github.com/ReactiveX/rxjs/issues/6466#issuecomment-859186944
If you see this issue on tsc > 4.2.4, please provide reproducible repo.

Closing as inactive.

authors: kwonoj mmakrzem

repo: ReactiveX/rxjs


text:
build(tsconfig): introduce extended tsconfig for target

Description:
This PR is effort to simplify our npm script, as first step instead of repeating whole same configuration for each build targes, utilizes latest tsc's config extending.
Related issue (if exists):

Coverage remained the same at 97.261% when pulling 385478b8bfe391e775081dc9cdff6052003a3771 on kwonoj:chore-ts-config into 2f395da56189090215462bb88cfd2014c842be57 on ReactiveX:master.

/cc @jasonaden for visibility.

Yes, this is targeting master only which for v6 and further. Ben branched off 5.5 stable for minor fixes.

This is SO MUCH BETTER.

authors: benlesh coveralls kwonoj


text:
How can i select hour and minute without second

I use the timePicker and I want to select hour and minute without second.
what can i do?
thank you.

You can just hide third column, i.e.:
dart   @override   List<int> layoutProportions() => [100, 100, 1];

in latest version ,you can use showSecondsColumn to achieve that

authors: Realank bkmza tjholic


text:
Add flag to allow fork-ts-checker-webpack-plugin to check for syntax errors (HappyPack / thread-loader)

Excuse the minimal post; on my phone. An issue was logged against ts-loader; details can be found here:
https://github.com/TypeStrong/ts-loader/issues/602
Essentially this comes down to ts-loader being unable to report errors to webpack when used in happyPackMode ie ts-loader is used with HappyPack or thread-loader to split work over multiple instances

copied from linked issue in ts-loader:
I think the issue is here: https://github.com/TypeStrong/ts-loader/blob/master/src/index.ts#L214
    // _module.errors is not available inside happypack - see https://github.com/TypeStrong/ts-loader/issues/336
    if (!instance.loaderOptions.happyPackMode) {
        registerWebpackErrors(
            loader._module.errors,
            formatErrors(diagnostics, instance.loaderOptions, instance.compiler, { module: loader._module })
        );
    }

In happyPackMode we cannot register transpile errors with webpack and so this info is lost. BTW the same issue would presumably apply if using thread-loader instead of happypack.  Same principle; ts-loader cannot register errors with webpack.
There's nothing that we can do from a ts-loader perspective here.

I think it should not be hard to add flag to fork-ts-checker plugin to check syntax errors.

If you could look at this that would be awesome!

Currently we are seeing an issue where a number of errors are not reported by ts-loader and are not reported by fork-ts-checker-webpack-plugin. This is fairly concerning :)
Examples include:
// Note the missing comma below:
const bar = [
    'a',
    'b'
    'c'
];

// Note the semi-colons below:
const biz = {
    a: 1,
    b: 2;
    c: 3;
    d: 4
};

// Note the missing '=>' below:
export const foo = async (bar:number):Promise<void> {
   ...
}

@johnnyreilly would this issue cause the behavior we're seeing? Thanks!

I think so yes @abirmingham. If you drop usage of thread-loader / cache-loader and stop using happypack mode for the time being the issues will be raised as you'd hope.
In the ts-loader issue @piotr-oles said:

fork-ts-checker reports only semantic errors, syntax errors should be checked by ts-loader and as far as I know it's how it works in transpileOnly mode. Maybe it's because of happy-pack mode? I think it should not be hard to add flag to fork-ts-checker plugin to check syntax errors.

And so I'm hoping he's up for adding this flag. :smile: @piotr-oles would you be able to advise if you're likely to do this anytime soon? Failing that would you be able to advise on how you had in mind to do that so someone else could have a go and submit a PR? Thanks!
Oh and thanks for all your work on this plugin; it's really useful. 💓

Having done a little digging I've an idea for a fix.  I think that the tweak needs to be made here: https://github.com/Realytics/fork-ts-checker-webpack-plugin/blob/master/lib/IncrementalChecker.js#L128
I think that this line:
diagnostics.push.apply(diagnostics, this.program.getSemanticDiagnostics(sourceFile, cancellationToken));

Should become something like this:
diagnostics.push.apply(diagnostics, this.program.getSemanticDiagnostics(sourceFile, cancellationToken), getSyntacticErrors ? this.program.getSyntacticDiagnostics(sourceFile, cancellationToken) : []);

getSyntacticErrors would need to be an option that was supplied.

Hi @abirmingham,
I've implemented a fix.  Do you want to give my fork a try?  You can find it here: https://github.com/johnnyreilly/fork-ts-checker-webpack-plugin
You'll need to supply a new option checkSyntacticErrors to ensure that syntactic errors are reported:
        new ForkTsCheckerWebpackPlugin({ checkSyntacticErrors: true })

cc @voodoorider

@johnnyreilly It worked, however after linking your fork in thread-loader example I discovered that the syntactical errors are reported even without providing the checkSyntacticErrors flag.

@voodoorider - thanks for the feedback. I think that means that this change possibly always turns on checkSyntacticErrors flag.  This would explain the strange behaviour I experienced with the tests.  I think I may need to look at how the option is passed through....
cc @piotr-oles

I think I might have fixed it @voodoorider  - want to try now?

Closed by #39

@johnnyreilly @piotr-oles works perfectly, thanks guys!

Great!

authors: abirmingham johnnyreilly voodoorider


text:
fixes button order

Describe your PR
Order of buttons in /businesses was wrong
Should be:

Business type
Zip code
Need

Current oder:

Need
Business type
Zip code

Related to #
Fixes #
Pages/Interfaces that will change
/businesses
Screenshots / video of changes
Drop some screenshots of the before and after of your work here. Better yet, take a screen recording using a tool like Loom

Steps to test

go to /businesses
check order of buttons for filtering/searching
verify buttons work the same despite order

Additional notes

 Business page filter order

repo: Rebuild-Black-Business/RBB-Website


text:
updating @redhat-cloud-services/frontend-components-config

Jira: https://issues.redhat.com/browse/RHCLOUD-36233

@Hyperkid123 Just removed the requested variables and the library, please verify if this is working.

repo: RedHatInsights/developer-sandbox-ui


text:
[tracking] [insights-operator-ldapauth] #79

https://github.com/RedHatInsights/insights-operator-ldapauth/issues/79

Fixed in https://github.com/RedHatInsights/insights-operator-ldapauth/pull/86


text:
Documentation for topic platform.notifications.ingress

Description
Documentation for topic platform.notifications.ingress
dependencies that are required for this change.
Type of change

Documentation update

Testing steps
See https://redhatinsights.github.io/insights-results-smart-proxy/overall-architecture.html after merge.
Checklist

[ ] make before_commit passes
[ ] updated documentation wherever necessary
[ ] added or modified tests if necessary
[ ] updated schemas and validators in insights-data-schemas in case of input/output change

Codecov Report
Base: 60.50% // Head: 60.43% // Decreases project coverage by -0.06% :warning:

Coverage data is based on head (dae6f10) compared to base (c6741f1).
Patch has no changes to coverable lines.

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

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #964      +/-   ##
==========================================
- Coverage   60.50%   60.43%   -0.07%
==========================================
  Files          23       23
  Lines        3228     3228
==========================================
- Hits         1953     1951       -2
- Misses       1054     1056       +2
  Partials      221      221

Impacted Files
Coverage Δ

content/content.go
76.95% <0.00%> (-0.83%)
:arrow_down:

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

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

authors: codecov-commenter tisnik


text:
OSIDB-3123: Invalid Affect Resolution

OSIDB-3123: Invalid Affect Resolution
Checklist:

[x] Linting passed
[x] Type checks passed
[x] Tests suite passed
[x] Commits consolidated
[x] Changelog updated
[x] Test cases added/updated
[x] Jira ticket updated

Summary:
When the affectedness was changed, the resolution kept the previous value, only the available options where updated but not the affect model.
Changes:

Reset the resolution value when the affectedness is changed
Simplify the logic by using the available affectedness and resolution enums
Changed the empty value ('') to an Empty string so it's easier to differentiate empty vs undefined
Added a validation to the resolution based on the possibles values. See https://github.com/RedHatProductSecurity/osidb/blob/master/osidb/constants.py#L61-L76

Considerations:
Closes OSIDB-3123
https://github.com/RedHatProductSecurity/osim/assets/4268580/c483d8ee-b05b-4c5f-a714-6616bf6d057e

Should we italicize the Empty choice in the dropdown?

Should we italicize the Empty choice in the dropdown?

I tried but seems that the option html element does not likes styles, at least firefox ignored it. We would need to use a custom dropdown element to handle styles

authors: MrMarble superbuggy

repo: RedHatProductSecurity/osim


text:
fix: updates GitHub Actions runner image and restart policy

Description

Changes

Ubuntu 24.04 has version 4 of podman which is compatible with version of Docker (Note - this public beta slated to go GA in Sept)
v4 of podman has a default restart policy of always which cause unexpected behavior during tests (follow-on issue will be needed for this).

Rationale
Why use podman and docker?
They are used in different contexts. podman is used to manage containers in E2E testing. docker GitHub Actions are used for image building in the release pipeline.

Using podman pods is required to run the mock server and trestle-bot on the same network stack. It is also what the team is more familiar with.
In terms of image building in GitHub Actions, the docker actions have the features we need for image tag management and caching.

Alternatives

Open to discuss here or on the issue

Choose a single technology
Install Podman v4 on existing ubuntu-latest. This is possible through brew or manually, but we would need to back out the changes in Sept.

Fixes #251
Type of change

[X] Bug fix (non-breaking change which fixes an issue)

How has this been tested?

[X] Verified Success -  https://github.com/RedHatProductSecurity/trestle-bot/actions/runs/9786753498/job/27022159330

Checklist

[X] My code follows the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

@gvauter PTAL

repo: RedHatProductSecurity/trestle-bot


text:
Corrected spelling and grammar

I sat down and corrected some grammar (like "i" -> "I") and some spelling of words ("trully" -> "truly") to make the project a little bit better, also I did in fact learn some things about HTTP. So thank you for that.

I sent a friend request on Chess.com and started the repo on github

repo: Reda-BENMAKDAD/pyapi


text:
Client.drop_index uses the deprecated FT.DROP command, destroying all indexed Hashes

FT.DROP is deprecated and by default deletes the indexed Hashes. Calling Client.drop_index thus deletes the indexed hashes.
This library should use FT.DROPINDEX for its drop command and expose the DD option to control whether or not to delete Hashes, with 'False' being the default. This way, dropping an index will not destroy the Hashes by default.

@emmanuelkeller What do you think about this? I'm worried that someone will try to drop a 2.0 index and accidentally destroy data.

@abrookins  You are absolutely right. This is currently done in this PR. Waiting for merge
https://github.com/RediSearch/redisearch-py/pull/108

Fixed by #108.

authors: abrookins emmanuelkeller

repo: RediSearch/redisearch-py


text:
Icon spacing gap isn't enough in the File Manager when set to the bottom of the Editor

Tested versions
v4.4.dev.custom_build [5e7e5fc60]
System information
Windows 11
Issue description
When the File Manager is set to the bottom of the screen (horizontal layout). Some icons don't have enough space between each other, this makes it messy and hard to read the file names.

Steps to reproduce
Set the File Manager to the bottom of the Redot editor and use icon display.
Minimal reproduction project (MRP)
This is not a project issue.

This is a Godot issue, is there a related Godot issue for this?

This is a Godot issue, is there a related Godot issue for this?

I couldn't find one with a quick search, lemme look again

authors: Bat-Computer SkogiB Spartan322

repo: Redot-Engine/redot-engine


text:
Prepare book for 0.5

[x] Move book to Relm4 organization
[ ] Prepare book for 0.5 release

Most links in the book go to your personal Github account

authors: AaronErhardt pentamassiv


text:
{Documentation} Unraid Community Apps/Docker Template

Here is a sample template that can be used in Unraid. Can be published with Unraid for community apps otherwise save as My-iDrive.xml under your boot usb drive in /Config/Plugins/dockerMan/templates-user/
<?xml version="1.0"?><Container version="2">
<Name>iDrive</Name>
<Repository>renofischa/idrive</Repository>
<Registry>https://hub.docker.com/r/renofischa/idrive</Registry>
<Network>host</Network>
<MyIP/>
<Shell>sh</Shell>
<Privileged>false</Privileged>
<Support/>
<Project/>
<Overview/>
<Category/>
<WebUI/>
<TemplateURL/>
<Icon>https://static.idriveonlinebackup.com/source/images/favicon.ico</Icon>
<ExtraParams/>
<PostArgs/>
<CPUset/>
<DateInstalled>1676085032</DateInstalled>
<DonateText/>  <DonateLink/>
<Requires>/work/IDriveForLinux/scripts/Idrivelib/dependencies</Requires>
<Config Name="Host Path 1" Target="/work/IDriveForLinux/idriveIt" Default="" Mode="rw" Description="" Type="Path" Display="always" Required="false" Mask="false">/mnt/user/appdata/idrive/idriveIt</Config>
<Config Name="Host Path 2" Target="/home/backup" Default="" Mode="ro" Description="" Type="Path" Display="always" Required="false" Mask="false">/mnt/user/backups/</Config>
<Config Name="Host Key 1" Target="TZ" Default="" Mode="" Description="" Type="Variable" Display="always" Required="false" Mask="false">America/New_York</Config>
<Config Name="Host Path 3" Target="/work/IDriveForLinux/scripts/Idrivelib/dependencies" Default="" Mode="rw" Description="" Type="Path" Display="always" Required="false" Mask="false">/mnt/user/appdata/idrive/dependencies</Config>
<Config Name="Host Path 4" Target="/mnt/files" Default="" Mode="rw" Description="" Type="Path" Display="always" Required="false" Mask="false">/mnt/user/appdata/idrive/files</Config>
</Container>

I've been going round in circles trying to get something to work for idrive in linux.  This template would be a fantastic help thank you.

@thecreativeone91 I've added your template to the README. Thanks!

authors: RenoFischa marshalleq thecreativeone91

repo: RenoFischa/idrive


text:
Remove js and d.ts files

Don't commit js and d.ts files

:shipit:


text:
Request for Swipe-to-Delete and Multi-Select Conversation Features

It would be beneficial to enable conversation deletion with a swipe gesture instead of requiring a right-click and delete, making the process faster. Additionally, implementing a multi-select feature for conversations would allow users to delete multiple items simultaneously.

@pmusolino thanks for the ideas. I don't often see the 'swipe to delete' gesture in native macOS apps, so I'm not sure this is the true 'macOS way'. It's probably better to implement some shortcuts instead? Or probably you have some examples of macOS apps that use this feature?
Multiselect is a great idea - I have even tested some prototypes in macai and plan to implement this in the future (most likely in 2.1.0).

@Renset one example is the spark app. Instead of deleting an email from the trash icon or using the right click, you can just swipe to delete it. A shortcut however would also be appreciated (example: command + delete).

@pmusolino ok, I'll start with multiselect and shortcuts and we'll see if we need further improvements :)

authors: Renset pmusolino

repo: Renset/macai


text:
Implement Two-Factor Authentication

This PR adds support for two-factor authentication via an authenticator app (like Google Authenticator).
Resolve https://github.com/ResonantGeoData/RGD-ScrumBoard/issues/37
To use:

To sign up for 2FA, go to /accounts/two_factor/setup/ after logging in.
To generate 2FA backup codes, go to /accounts/two_factor/backup_tokens/
To remove 2FA from an account, go to /accounts/two_factor/remove.

@mvandenburgh, this is looking great! I'm curious if there is a setting to require all users to use 2FA - sending them to the 2FA page on signup perhaps?

Also, is there a way to enforce this on the admin site too?

@mvandenburgh, this is looking great! I'm curious if there is a setting to require all users to use 2FA - sending them to the 2FA page on signup perhaps?

Yes, this is possible https://django-allauth-2fa.readthedocs.io/en/latest/advanced/#forcing-a-user-to-use-2fa.

Also, is there a way to enforce this on the admin site too?

I think so, I can look into it if you want.
Do you want me to implement these features on this branch?

Do you want me to implement these features on this branch?

Yep, if you could do those things here, that would be great!

@mvandenburgh, this is looking great! I'm curious if there is a setting to require all users to use 2FA - sending them to the 2FA page on signup perhaps?

https://github.com/ResonantGeoData/RD-OpenGeo/pull/19/commits/34a18fd94dc2093500f8e06e9de8eee9729adfd4

Also, is there a way to enforce this on the admin site too?

So, it seems the only way to enforce this is to override the admin login page and force admins to login through the default allauth view. From there, they will be required to authenticate with 2FA. Did this in https://github.com/ResonantGeoData/RD-OpenGeo/pull/19/commits/516567c15d7d897a9f2093b3d6021b85c6a79597

This is excellent work! Would you please refactor the settings in the form of a composed-configuration "mixin":
e.g. (did it myself b/c I needed to copy this over to another deployment but could you commit it here?):
from composed_configuration import ConfigMixin

class TwoFactorAuthMixin(ConfigMixin):
    @staticmethod
    def before_binding(configuration: Type[ComposedConfiguration]):
        configuration.INSTALLED_APPS += [
            # Configure the django-otp package.
            'django_otp',
            'django_otp.plugins.otp_totp',
            'django_otp.plugins.otp_static',
            # Enable two-factor auth.
            'allauth_2fa',
        ]
        configuration.MIDDLEWARE += [
            'django_otp.middleware.OTPMiddleware',
            'allauth_2fa.middleware.AllauthTwoFactorMiddleware',
            'opengeo.auth.RequireTwoFactorAuthMiddleware',
        ]

    ACCOUNT_ADAPTER = 'opengeo.auth.TwoFactorAuthAdapter'

Then could you also update all of the classes in the settings to use mutate_configuration instead of before_binding per https://github.com/girder/django-composed-configuration/pull/145

Further, after logging in for the first time and you have to set up 2FA, could you make it so that there is some sort of link to view the site? See on this page, users have no direct way to go into the app:

@mvandenburgh, is this compatible with the Python client? Can you look into that?

@mvandenburgh, is this compatible with the Python client? Can you look into that?

Not sure, I'm guessing not though. Yes I can look into this

Not merging, good to keep as a reference - needed to use LDAP for an internal project that requires 2FA. There is an internal blog on the set up

authors: banesullivan mvandenburgh

repo: ResonantGeoData/RD-OpenGeo


text:
Convert function-based views to class-based views

Transition the existing function-based views in the api module to class-based views with custom serializers.

This has been generally done. Some of the geosearch function-based endpoints will just be deprecated

repo: ResonantGeoData/ResonantGeoData


text:
Remove some unnecessary usage of _field_file_to_local_path

S3FileField behaves exactly the same as FileField once files have been uploaded.
For more information on how to use FieldFile and File objects, see: https://docs.djangoproject.com/en/3.1/topics/files/#the-file-object for more

There are a few more places where _field_file_to_local_path could still likely be removed, but were too complicated for me to confidently refactor.
For some libraries like kwcoco and gdal, a Python file-like object is not accepted as input. It is only those places where _field_file_to_local_path is actually necessary.
Otherwise, it is simpler and more efficient to interact with a FileField (i.e. an instance of a FileField field) using its native .open method to return a file-like object.


text:
Refactor serializers and add new STAC API view

Refactoring of filters to separate the SpatialEntry fields from domain model fields

removes the ModifiableEntry fields from the filters (there is a new ModifiableEntryFilterMixin but it is unused because we can't really mix filters)

Utilizes STAC Item serializer from #386 for new search endpoints

returns raster search results in STAC Item

Updates front-end templates to use new search views/filters

Resolve #352, resolve #374, resolve https://github.com/ResonantGeoData/issue-dashboard/issues/10

@AlmightyYakob, would you be able to review the changes to the Python client in rgdc/rgdc/rgdc.py with these new endpoints/changes in existing endpoints?


text:
FiO2 sensor selection

Right now we are stuck with galvanic cell O2 sensing which is more widely available but degrades quickly (12 months or faster if we use it in closed loop O2 control).
We would like to switch to oxygen fluorescence quenching but it is currently a supply chain risk.

Calling this done for now.  We have integrated this sensor into the design, and it is and will be moving forward with this.  If further testing deems a change needed a new ticket will be created.

This ticket was in the "Done" column in Hardware. Moved it to the same column after changing repositories.

This ticket was in the "Done" column in Hardware. Moved it to the same column after changing repositories.

repo: RespiraWorks/Ventilator


text:
Switch & Future?

First of all, this looks great! :)
Im currently using your LiteNetLib in a c# gameserver and implemented the rest of the logic by myself.
E.g. Once an entity moved, its written to the NetManager using .SendNetSerializable<T>(...) and similar calls.
Sending the stuff is happening once during the frame.
This is already working fine, however this add on looks very promising.
I wonder how those two compare. E.g. are there any performance benefits from using LiteEntitySystem?
Or does LiteEntitySystem allocate garbage or is even slower/faster in particular situations? ^^
When does it make sense to switch? And how long will you support this, what are the plans for the future?

@genaray this is Logic/Multiplayer Engine using LiteNetLib as transport. This isn't comparable things. LiteNetLib is like LLAPI (low level api) in Unity. This is highlevel api that has entities, players, controllers and all stuff needed for implementing games. It used similar concepts like Quake/Unreal/Source(CS:GO, HL, Dota).

Or does LiteEntitySystem allocate garbage or is even slower/faster in particular situations?

It allocates when you create new game entity. But in default loop after some little warmup it works without allocations/garbage.

When does it make sense to switch?

When you want perfect networking architecture that battle-tested for years :)
And don't want impement it by yourself

And how long will you support this, what are the plans for the future?

I use this for my game and work project and i plan to support it forever because this is just like simpliest and open implementation of classic working architecture and i don't need more. This is really enough for all my projects (and future too). Also for questions better join discord https://discord.gg/FATFPdy

authors: RevenantX genaray

repo: RevenantX/LiteEntitySystem


text:
Feature Request: Gravatar Avatars

http://gravatar.com/
Please?

Aren't gravatar's linked to emails?
As far as I know, Toxy doesn't use emails..so wouldn't there need to be a system of confirming the user's email?

Tox clients are not supposed to interact with anything except for the tox network itself and services defined in the STS. This is what every tox user takes for granted so we can't just add things like this.

I had not read the STS yet.  Thanks for sharing the link.

authors: Impyy hypereddie kevinkjt2000

repo: Reverp/Toxy


text:
Update LUA_RESTY_OIDC_VER to version 1.7.1-1

Hello,
Thanks for the helpful repository.
Because of lua-resty-oidc 1.6.1-1 has a bug and  bad performance, so we had to build the docker image ourselves, if you can update your docker hub images to the new version  1.7.1-1 , that would be a great help?
Thanks!

Thanks man

repo: Revomatico/docker-kong-oidc


text:
Add Tab() widget for Layout mode

with Layout() as layout:
    with Notebook() as nb:
        with Tab("Tab1") as tab1:
            Label("Text here")
            ...
         with Tab("Tab2") as tab2:
            ...

The Tab() widgets would add themselves to the parent Notebook's tabs.  If the parent isn't a notebook, raise an error.

Something like this

class Tab(text: str, disabled: bool = False):
    ...

    def add_widget(self, widget: Widget):
        """Add a widget to the tab's layout"""
        self.layout[0].append(widget)

    def __enter__(self):
        self.parent = get_parent()
        if not isinstance(self.parent, guitk.Notebook):
            raise ValueError(...)
        push_parent(self)
        return self

    def __exit__(self, exc_type, exc_val, exc_tb):
        self.parent._add_tab(self.layout) # adds the tab to the parent Notebook's self.tabs (this method doesn't currently exist)
        pop_parent()
        return False

repo: RhetTbull/guitk


text:
cannot send messages to connected phone but can send from phone

My phone number connected to go-whatsapp appears to me as online

but a can't send a message, it's always as not delivered

but if i send a message from connected phone the message is correct delivered to my phone, any suggestions?

Did you check the phone JID? You can do that using the Exist() method.

authors: guilhermegehres laercioamatuzzi

repo: Rhymen/go-whatsapp


text:
[Feature] 建议提供双击导航栏刷新

问题描述

右上角的刷新用鼠标定位比较麻烦
描述解决方案

双击导航栏刷新资讯比较便捷，更符合人性化
备选方案
No response
附加信息
No response
人人为我，我为人人
No response

已经有刷新按钮的情况下暂不考虑这种交互，双击标题栏在桌面应用中不是一种通用的交互行为


text:
rn-falcon-app-intro: Appears to be a git repo or submodule

First of all awesome work! it works like a charm just that the npm package contains git related file which makes any npm command fail after installing rn-falcon-app-intro.
npm ERR! path /Users/aaaa/Desktop/aaaaa/node_modules/rn-falcon-app-intro
npm ERR! code EISGIT
npm ERR! git /Users/aaaa/Desktop/aaaaa/node_modules/rn-falcon-app-intro: Appears to be a git repo or submodule.
npm ERR! git     /Users/aaaa/Desktop/aaaaa/node_modules/rn-falcon-app-intro
npm ERR! git Refusing to remove it. Update manually,
npm ERR! git or move it out of the way first

npm ERR! A complete log of this run can be found in:
npm ERR!     /Users/aaaa/.npm/_logs/2019-12-04T13_08_39_940Z-debug.log

After I manually remove .git and .github in node_modules/rn-falcon-app-intro the npm command work as expected
If you can remove it in your package for the upcoming release will be great!
Once again thanks for this awesome package!!

Hi,
thanks!
Completed the change, version 1.2.7


text:
[Terraform]create compute instance

Create OCI components using Terraform.

Compute instance

Script has no problem.
Merge
(But, It cannot create instance resource because out of capacity issue =>  #3  )