CPU halted usin I2C (IDFGH-4388)

This is my code:
#include "bmp280.h"
#include <stdio.h>
#include <freertos/FreeRTOS.h>
#include <freertos/task.h>
#include <esp_system.h>
#include <stdbool.h>
#include <string.h>
#include "ssd1306.h"
#include "ssd1306_draw.h"
#include "ssd1306_font.h"
#include "ssd1306_default_if.h"
void Prueba (void *pvParameter)
{
static const int I2CDisplayAddress = 0x3c;
static const int I2CDisplayWidth = 128;
static const int I2CDisplayHeight = 64;
static const int I2CResetPin = -1;
struct SSD1306_Device I2CDisplay;

bool DefaultBusInit( void ) {
	assert( SSD1306_I2CMasterInitDefault( ) == true );
	assert( SSD1306_I2CMasterAttachDisplayDefault( &I2CDisplay, I2CDisplayWidth, I2CDisplayHeight, I2CDisplayAddress, I2CResetPin ) == true );
	return true;
	}

void SetupDemo( struct SSD1306_Device* DisplayHandle, const struct SSD1306_FontDef* Font ) {
	SSD1306_Clear( DisplayHandle, SSD_COLOR_BLACK );
	SSD1306_SetFont( DisplayHandle, Font );
	}

void SayHello( struct SSD1306_Device* DisplayHandle, const char* HelloText ) {
	SSD1306_FontDrawAnchoredString( DisplayHandle, TextAnchor_Center, HelloText, SSD_COLOR_WHITE );
	SSD1306_Update( DisplayHandle );
	}

	printf( "Ready...\n" );
	if ( DefaultBusInit( ) == true ) {
		printf( "BUS Init lookin good...\n" );
		printf( "Drawing.\n" );
	SetupDemo( &I2CDisplay, &Font_droid_sans_fallback_24x28 );
    SayHello( &I2CDisplay, "SICCU!" );
    printf( "Done!\n" );
	}
vTaskDelete(NULL);

}
void bmp280_test(void *pvParamters)
{
bmp280_params_t params;
bmp280_init_default_params(&params);
bmp280_t dev;
memset(&dev, 0, sizeof(bmp280_t));

#define SDA_GPIO 4
#define SCL_GPIO 5

bool bme280p = dev.id == BME280_CHIP_ID;
printf("BMP280: found %s\n", bme280p ? "BME280" : "BMP280");

float pressure, temperature, humidity;

while (1)
{
    vTaskDelay(2000 / portTICK_PERIOD_MS);
    if (bmp280_read_float(&dev, &temperature, &pressure, &humidity) != ESP_OK)
    {
        printf("Temperature/pressure reading failed\n");
        continue;
    }

    /* float is used in printf(). you need non-default configuration in
     * sdkconfig for ESP8266, which is enabled by default for this
     * example. see sdkconfig.defaults.esp8266
     */
    printf("Pressure: %.2f Pa, Temperature: %.2f C", pressure, temperature);
    if (bme280p)
        printf(", Humidity: %.2f\n", humidity);
    else
        printf("\n");
}
vTaskDelete(NULL);

}
void app_main()
{
xTaskCreate(&Prueba,"Prueba",1024,NULL,1,NULL);
xTaskCreate(bmp280_test, "bmp280_test", configMINIMAL_STACK_SIZE * 8, NULL, 5, NULL);
}
This is log compile:
Python requirements from C:/esp-idf\requirements.txt are satisfied.
[1/4] cmd.exe /C "cd /D D:\Eclipse\Trabajo\OLED_BMP280_Ver_1.1\build\esp-idf\partition_table && "C:\Users\C. Alejandro.espressif\tools\cmake\3.13.4\bin\cmake.exe" -E echo "Partition table binary generated. Contents:" && "C:\Users\C. Alejandro.espressif\tools\cmake\3.13.4\bin\cmake.exe" -E echo ******************************************************************************* && python C:/esp-idf/components/partition_table/gen_esp32part.py -q --offset 0x8000 --flash-size 4MB D:/Eclipse/Trabajo/OLED_BMP280_Ver_1.1/build/partition_table/partition-table.bin && "C:\Users\C. Alejandro.espressif\tools\cmake\3.13.4\bin\cmake.exe" -E echo *******************************************************************************"
Partition table binary generated. Contents:

Espressif ESP32 Partition Table
Name, Type, SubType, Offset, Size, Flags
nvs,data,nvs,0x9000,24K,
phy_init,data,phy,0xf000,4K,
factory,app,factory,0x10000,1M,

[2/4] Performing build step for 'bootloader'
ninja: no work to do.
esptool.py -p COM3 -b 460800 --before default_reset --after hard_reset write_flash --flash_mode dio --flash_freq 40m --flash_size 4MB 0x8000 partition_table/partition-table.bin 0x1000 bootloader/bootloader.bin 0x10000 HelloWorld.bin
esptool.py v2.8
Serial port COM3
Connecting........__
Detecting chip type... ESP32
Chip is ESP32D0WDQ5 (revision 1)
Features: WiFi, BT, Single Core, Coding Scheme None
Crystal is 40MHz
MAC: 24:0a:c4:85:fc:58
Uploading stub...
Running stub...
Stub running...
Changing baud rate to 460800
Changed.
Configuring flash size...
Compressed 3072 bytes to 103...
Writing at 0x00008000... (100 %)
Wrote 3072 bytes (103 compressed) at 0x00008000 in 0.0 seconds (effective 2240.3 kbit/s)...
Hash of data verified.
Compressed 25296 bytes to 14936...
Writing at 0x00001000... (100 %)
Wrote 25296 bytes (14936 compressed) at 0x00001000 in 0.3 seconds (effective 589.9 kbit/s)...
Hash of data verified.
Compressed 182848 bytes to 90556...
Writing at 0x00010000... (16 %)
Writing at 0x00014000... (33 %)
Writing at 0x00018000... (50 %)
Writing at 0x0001c000... (66 %)
Writing at 0x00020000... (83 %)
Writing at 0x00024000... (100 %)
Wrote 182848 bytes (90556 compressed) at 0x00010000 in 2.2 seconds (effective 650.1 kbit/s)...
Hash of data verified.
Leaving...
Hard resetting via RTS pin...
Checking Python dependencies...
Adding flash's dependency "all" to list of actions
Executing action: all (aliases: build)
Running ninja in directory d:\eclipse\trabajo\oled_bmp280_ver_1.1\build
Executing "ninja all"...
Executing action: flash
Running esptool.py in directory d:\eclipse\trabajo\oled_bmp280_ver_1.1\build
Executing "'C:\Users\C. Alejandro.espressif\python_env\idf4.0_py3.8_env\Scripts\python.exe' C:/esp-idf\components/esptool_py/esptool/esptool.py -p COM3 -b 460800 --before default_reset --after hard_reset write_flash @flash_project_args"...
Done
This is in terminal:
I (71) boot_comm: chip revision: 1, min. bootloader chip revision: 0
I (39) boot: ESP-IDF v4.0.1-dirty 2nd stage bootloader
I (39) boot: compile time 16:19:09
I (39) boot: Enabling RNG early entropy source...
I (44) boot: SPI Speed      : 40MHz
I (48) boot: SPI Mode       : DIO
I (52) boot: SPI Flash Size : 4MB
I (56) boot: Partition Table:
I (60) boot: ## Label            Usage          Type ST Offset   Length
I (67) boot:  0 nvs              WiFi data        01 02 00009000 00006000
I (75) boot:  1 phy_init         RF data          01 01 0000f000 00001000
I (82) boot:  2 factory          factory app      00 00 00010000 00100000
I (90) boot: End of partition table
I (94) boot_comm: chip revision: 1, min. application chip revision: 0
I (101) esp_image: segment 0: paddr=0x00010020 vaddr=0x3f400020 size=0x0afcc ( 45004) map
I (126) esp_image: segment 1: paddr=0x0001aff4 vaddr=0x3ffb0000 size=0x018d4 (  6356) load
I (129) esp_image: segment 2: paddr=0x0001c8d0 vaddr=0x40080000 size=0x00400 (  1024) load
I (133) esp_image: segment 3: paddr=0x0001ccd8 vaddr=0x40080400 size=0x03338 ( 13112) load
I (147) esp_image: segment 4: paddr=0x00020018 vaddr=0x400d0018 size=0x15b74 ( 88948) map
I (183) esp_image: segment 5: paddr=0x00035b94 vaddr=0x40083738 size=0x06e88 ( 28296) load
I (201) boot: Loaded app from partition at offset 0x10000
I (201) boot: Disabling RNG early entropy source...
I (202) cpu_start: Pro cpu up.
I (205) cpu_start: Application information:
I (210) cpu_start: Project name:     HelloWorld
I (215) cpu_start: App version:      1
I (220) cpu_start: Compile time:     Dec  7 2020 23:49:17
I (226) cpu_start: ELF file SHA256:  f670c16c0a7aaac5...
I (232) cpu_start: ESP-IDF:          v4.0.1-dirty
I (237) cpu_start: Single core mode
I (242) heap_init: Initializing. RAM available for dynamic allocation:
I (249) heap_init: At 3FFAE6E0 len 00001920 (6 KiB): DRAM
I (255) heap_init: At 3FFB2840 len 0002D7C0 (181 KiB): DRAM
I (261) heap_init: At 3FFE0440 len 0001FBC0 (126 KiB): D/IRAM
I (267) heap_init: At 40078000 len 00008000 (32 KiB): IRAM
I (273) heap_init: At 4008A5C0 len 00015A40 (86 KiB): IRAM
I (280) cpu_start: Pro cpu start user code
I (297) spi_flash: detected chip: gd
I (298) spi_flash: flash io: dio
I (298) cpu_start: Starting scheduler on PRO CPU.
Ready...
BMP280: found BMP280
BUS Init lookin good...
Drawing.
Done!
C:/esp-idf/components/freertos/queue.c:1448 (xQueueGenericReceive)- assert failed!
abort() was called at PC 0x40088925 on core 0
ELF file SHA256: f670c16c0a7aaac5
Backtrace: 0x40085785:0x3ffb6330 0x40085b05:0x3ffb6350 0x40088925:0x3ffb6370 0x400d4d2f:0x3ffb63b0 0x400d4f11:0x3ffb63e0 0x400d521d:0x3ffb6420 0x400d3b40:0x3ffb6450 0x40087f6d:0x3ffb64e0
CPU halted.
I don't know where is the error. Someone can help me?
Thanks in advance

Hi @aledoors, please look at line:
C:/esp-idf/components/freertos/queue.c:1448 (xQueueGenericReceive)- assert failed!
Probably missing an init.  I don't see any "i2c_param_config" or "i2c_driver_install"

Hi @aledoors , I can't tell which function caused the abort() without a backtrace.
Could you provide the elf, or use idf_monitor (idf.py monitor) to decode the backtrace?
and BTW, the code you provided does not directly call any IDF functions, I wonder if you are using an 3rd party lib?

Hi @ginkgm, i'm using Eclipse IDE. I'm new to programming on FreeRTOS based systems and using ESP-IDF and Eclipse IDE. My knowledge is basic but I am learning. I ask you if you can guide me on what steps I should take to find the problem.
On the other hand, I did the compilation through ESP-IDF and I got the following result:
D:\Eclipse\Trabajo\OLED_BMP280_Ver_1.1>idf.py build flash monitor
Checking Python dependencies...
Python requirements from C:\esp-idf\requirements.txt are satisfied.
Executing action: all (aliases: build)
Running ninja in directory d:\eclipse\trabajo\oled_bmp280_ver_1.1\build
Executing "ninja all"...
[1/4] cmd.exe /C "cd /D D:\Eclipse\Trabajo\OLED_BMP280_Ver...*********************************************************"
Partition table binary generated. Contents:

Espressif ESP32 Partition Table
Name, Type, SubType, Offset, Size, Flags
nvs,data,nvs,0x9000,24K,
phy_init,data,phy,0xf000,4K,
factory,app,factory,0x10000,1M,

[2/4] Performing build step for 'bootloader'
ninja: no work to do.
Executing action: flash
Choosing default port b'COM3' (use '-p PORT' option to set a specific serial port)
Running esptool.py in directory d:\eclipse\trabajo\oled_bmp280_ver_1.1\build
Executing "'C:\Users\C. Alejandro.espressif\python_env\idf4.0_py3.8_env\Scripts\python.exe' C:\esp-idf\components/esptool_py/esptool/esptool.py -p COM3 -b 460800 --before default_reset --after hard_reset write_flash @flash_project_args"...
esptool.py -p COM3 -b 460800 --before default_reset --after hard_reset write_flash --flash_mode dio --flash_freq 40m --flash_size 4MB 0x8000 partition_table/partition-table.bin 0x1000 bootloader/bootloader.bin 0x10000 HelloWorld.bin
esptool.py v2.8
Serial port COM3
Traceback (most recent call last):
File "C:\esp-idf\components/esptool_py/esptool/esptool.py", line 3201, in
_main()
File "C:\esp-idf\components/esptool_py/esptool/esptool.py", line 3194, in _main
main()
File "C:\esp-idf\components/esptool_py/esptool/esptool.py", line 2883, in main
esp = ESPLoader.detect_chip(each_port, initial_baud, args.before, args.trace)
File "C:\esp-idf\components/esptool_py/esptool/esptool.py", line 273, in detect_chip
detect_port = ESPLoader(port, baud, trace_enabled=trace_enabled)
File "C:\esp-idf\components/esptool_py/esptool/esptool.py", line 237, in init
self.port = serial.serial_for_url(port)
File "C:\Users\C. Alejandro.espressif\python_env\idf4.0_py3.8_env\lib\site-packages\serial_init.py", line 88, in serial_for_url
instance.open()
File "C:\Users\C. Alejandro.espressif\python_env\idf4.0_py3.8_env\lib\site-packages\serial\serialwin32.py", line 62, in open
raise SerialException("could not open port {!r}: {!r}".format(self.portstr, ctypes.WinError()))
serial.serialutil.SerialException: could not open port 'COM3': PermissionError(13, 'Acceso denegado.', None, 5)
esptool.py failed with exit code 1
D:\Eclipse\Trabajo\OLED_BMP280_Ver_1.1>idf.py build flash monitor
Checking Python dependencies...
Python requirements from C:\esp-idf\requirements.txt are satisfied.
Executing action: all (aliases: build)
Running ninja in directory d:\eclipse\trabajo\oled_bmp280_ver_1.1\build
Executing "ninja all"...
[1/4] cmd.exe /C "cd /D D:\Eclipse\Trabajo\OLED_BMP280_Ver...*********************************************************"
Partition table binary generated. Contents:

Espressif ESP32 Partition Table
Name, Type, SubType, Offset, Size, Flags
nvs,data,nvs,0x9000,24K,
phy_init,data,phy,0xf000,4K,
factory,app,factory,0x10000,1M,

[2/4] Performing build step for 'bootloader'
ninja: no work to do.
Executing action: flash
Choosing default port b'COM3' (use '-p PORT' option to set a specific serial port)
Running esptool.py in directory d:\eclipse\trabajo\oled_bmp280_ver_1.1\build
Executing "'C:\Users\C. Alejandro.espressif\python_env\idf4.0_py3.8_env\Scripts\python.exe' C:\esp-idf\components/esptool_py/esptool/esptool.py -p COM3 -b 460800 --before default_reset --after hard_reset write_flash @flash_project_args"...
esptool.py -p COM3 -b 460800 --before default_reset --after hard_reset write_flash --flash_mode dio --flash_freq 40m --flash_size 4MB 0x8000 partition_table/partition-table.bin 0x1000 bootloader/bootloader.bin 0x10000 HelloWorld.bin
esptool.py v2.8
Serial port COM3
Connecting......
Detecting chip type... ESP32
Chip is ESP32D0WDQ5 (revision 1)
Features: WiFi, BT, Single Core, Coding Scheme None
Crystal is 40MHz
MAC: 24:0a:c4:85:fc:58
Uploading stub...
Running stub...
Stub running...
Changing baud rate to 460800
Changed.
Configuring flash size...
Compressed 3072 bytes to 103...
Wrote 3072 bytes (103 compressed) at 0x00008000 in 0.0 seconds (effective 2240.6 kbit/s)...
Hash of data verified.
Compressed 25296 bytes to 14936...
Wrote 25296 bytes (14936 compressed) at 0x00001000 in 0.3 seconds (effective 591.6 kbit/s)...
Hash of data verified.
Compressed 182848 bytes to 90556...
Wrote 182848 bytes (90556 compressed) at 0x00010000 in 2.3 seconds (effective 647.6 kbit/s)...
Hash of data verified.
Leaving...
Hard resetting via RTS pin...
Executing action: monitor
Running idf_monitor in directory d:\eclipse\trabajo\oled_bmp280_ver_1.1
Executing "'C:\Users\C. Alejandro.espressif\python_env\idf4.0_py3.8_env\Scripts\python.exe' C:\esp-idf\tools/idf_monitor.py -p COM3 -b 115200 d:\eclipse\trabajo\oled_bmp280_ver_1.1\build\HelloWorld.elf -m 'C:\Users\C. Alejandro.espressif\python_env\idf4.0_py3.8_env\Scripts\python.exe' 'C:\esp-idf\tools\idf.py'"...
--- idf_monitor on COM3 115200 ---
--- Quit: Ctrl+] | Menu: Ctrl+T | Help: Ctrl+T followed by Ctrl+H ---
ets Jun  8 2016 00:22:57
rst:0x1 (POWERON_RESET),boot:0x17 (SPI_FAST_FLASH_BOOT)
configsip: 0, SPIWP:0xee
clk_drv:0x00,q_drv:0x00,d_drv:0x00,cs0_drv:0x00,hd_drv:0x00,wp_drv:0x00
mode:DIO, clock div:2
load:0x3fff0018,len:4
load:0x3fff001c,len:6808
load:0x40078000,len:14076
load:0x40080400,len:4304
entry 0x400806e8
I (71) boot: Chip Revision: 1
I (71) boot_comm: chip revision: 1, min. bootloader chip revision: 0
I (39) boot: ESP-IDF v4.0.1-dirty 2nd stage bootloader
I (39) boot: compile time 16:19:09
I (39) boot: Enabling RNG early entropy source...
I (44) boot: SPI Speed      : 40MHz
I (48) boot: SPI Mode       : DIO
I (52) boot: SPI Flash Size : 4MB
I (56) boot: Partition Table:
I (60) boot: ## Label            Usage          Type ST Offset   Length
I (67) boot:  0 nvs              WiFi data        01 02 00009000 00006000
I (75) boot:  1 phy_init         RF data          01 01 0000f000 00001000
I (82) boot:  2 factory          factory app      00 00 00010000 00100000
I (90) boot: End of partition table
I (94) boot_comm: chip revision: 1, min. application chip revision: 0
I (101) esp_image: segment 0: paddr=0x00010020 vaddr=0x3f400020 size=0x0afcc ( 45004) map
I (126) esp_image: segment 1: paddr=0x0001aff4 vaddr=0x3ffb0000 size=0x018d4 (  6356) load
I (129) esp_image: segment 2: paddr=0x0001c8d0 vaddr=0x40080000 size=0x00400 (  100x40080000: _WindowOverflow4 at C:/esp-idf/components/freertos/xtensa_vectors.S:1778

load
I (133) esp_image: segment 3: paddr=0x0001ccd8 vaddr=0x40080400 size=0x03338 ( 13112) load
I (147) esp_image: segment 4: paddr=0x00020018 vaddr=0x400d0018 size=0x15b74 ( 88948) map
0x400d0018: _stext at ??:?

I (182) esp_image: segment 5: paddr=0x00035b94 vaddr=0x40083738 size=0x06e88 ( 28296) load
0x40083738: spi_flash_mmap_pages at C:/esp-idf/components/spi_flash/flash_mmap.c:200 (discriminator 4)
I (201) boot: Loaded app from partition at offset 0x10000
I (201) boot: Disabling RNG early entropy source...
I (202) cpu_start: Pro cpu up.
I (205) cpu_start: Application information:
I (210) cpu_start: Project name:     HelloWorld
I (215) cpu_start: App version:      1
I (220) cpu_start: Compile time:     Dec  7 2020 23:49:17
I (226) cpu_start: ELF file SHA256:  f670c16c0a7aaac5...
I (232) cpu_start: ESP-IDF:          v4.0.1-dirty
I (237) cpu_start: Single core mode
I (242) heap_init: Initializing. RAM available for dynamic allocation:
I (249) heap_init: At 3FFAE6E0 len 00001920 (6 KiB): DRAM
I (255) heap_init: At 3FFB2840 len 0002D7C0 (181 KiB): DRAM
I (261) heap_init: At 3FFE0440 len 0001FBC0 (126 KiB): D/IRAM
I (267) heap_init: At 40078000 len 00008000 (32 KiB): IRAM
I (273) heap_init: At 4008A5C0 len 00015A40 (86 KiB): IRAM
I (280) cpu_start: Pro cpu start user code
I (297) spi_flash: detected chip: gd
I (298) spi_flash: flash io: dio
I (298) cpu_start: Starting scheduler on PRO CPU.
Ready...
BMP280: found BMP280
BUS Init lookin good...
Drawing.
Done!
C:/esp-idf/components/freertos/queue.c:1448 (xQueueGenericReceive)- assert failed!
abort() was called at PC 0x40088925 on core 0
0x40088925: xQueueGenericReceive at C:/esp-idf/components/freertos/queue.c:1448 (discriminator 1)
ELF file SHA256: f670c16c0a7aaac5
Backtrace: 0x40085785:0x3ffb6330 0x40085b05:0x3ffb6350 0x40088925:0x3ffb6370 0x400d4d2f:0x3ffb63b0 0x400d4f11:0x3ffb63e0 0x400d521d:0x3ffb6420 0x400d3b40:0x3ffb6450 0x40087f6d:0x3ffb64e0
0x40085785: invoke_abort at C:/esp-idf/components/esp32/panic.c:155
0x40085b05: abort at C:/esp-idf/components/esp32/panic.c:172
0x40088925: xQueueGenericReceive at C:/esp-idf/components/freertos/queue.c:1448 (discriminator 1)
0x400d4d2f: i2c_dev_take_mutex at D:/esp-idf-lib-master/components/i2cdev/i2cdev.c:111
0x400d4f11: bmp280_read_fixed at D:/esp-idf-lib-master/components/bmp280/bmp280.c:347
0x400d521d: bmp280_read_float at D:/esp-idf-lib-master/components/bmp280/bmp280.c:379 (discriminator 4)
0x400d3b40: bmp280_test at D:\Eclipse\Trabajo\OLED_BMP280_Ver_1.1\build/../main/main.c:77
0x40087f6d: vPortTaskWrapper at C:/esp-idf/components/freertos/port.c:143
CPU halted.

Adding information, this is my CMakeLists.txt:
**# The following lines of boilerplate have to be in your project's
CMakeLists in this exact order for cmake to work correctly
cmake_minimum_required(VERSION 3.5)
set(EXTRA_COMPONENT_DIRS D:/esp-idf-lib-master/components)
include($ENV{IDF_PATH}/tools/cmake/project.cmake)
project(OLED_BMP280_Ver_1.1)**
I needed to add an external library to my code and by editing this file I found out how to do it.

Hello @aledoors
I quite often have this issue and it's usually the same mistake.
You're accessing a NULL pointer. That's what the assert is doing.
I dont know if you're using any queues in your program, but if you do, remember to allocate them before you write/read on them.
If you're not using queues, there might be some initialization/allocation missing. I see you use your own lib for I2C, doesn't it have an initialization function before using? You may be accessing a NULL mutex causing an abort.
The backtrace shows the error in the i2c_dev file. Review if you're missing an initialization.
Thanks!

Hello @MrCurio
Thanks for your help. I'm not using queues in my program, what I did was use two programs that work perfectly and put together a single program, separated into two tasks. Even if I cancel one task or the other the program works fine with the task enabled.
I have the problem when both tasks work simultaneously, the SSD1306 display task uses two different SCL and SDA pins than the SCL and SDA pins of the BMP280 sensor task. What I do is use two I2C channels because I don't know how to do it from the same channel. I am newly programmed FreeRTOS and I am learning and making mistakes.

Hello @aledoors
As i said you are trying to access a NULL pointer. Let me explain more in detail.
As i assume, you are using a third party library called esp-idf-lib by UncleRus. This library creates a thread safe API for reading I2C devices by using mutexes (semaphores). Let's say that your header file bmp280.h  belongs to the esp-idf-lib repo. If this is it,  you need to initialize those mutexes.
For doing so, the bmp280.h  provides some initialization function you seem to not use. Review what i am talking about cause i am quite sure this is the problem you are having. Below i leave some of the functions i am talking about:
esp_err_t bmp280_init_desc(bmp280_t *dev, uint8_t addr, i2c_port_t port, gpio_num_t sda_gpio, gpio_num_t scl_gpio);

esp_err_t bmp280_init(bmp280_t *dev, bmp280_params_t *params);

esp_err_t bmp280_init_default_params(bmp280_params_t *params);

It should be noted that if you use the esp-idf-lib repo, it does not matter if you use 2 different ports or not. Mutexes must be initialize!
Thanks!

Hello @MrCurio
Thank you for all of your help, we were able to solve the problem and now the program works correctly.
I add these two lines:
ESP_ERROR_CHECK(bmp280_init_desc(&dev, BMP280_I2C_ADDRESS_0, 0, SDA_GPIO, SCL_GPIO));
ESP_ERROR_CHECK(bmp280_init(&dev, &params));

but the error still continued
Then modify this line and change the 0 to 1 and the program started working:
ESP_ERROR_CHECK(bmp280_init_desc(&dev, BMP280_I2C_ADDRESS_0, 1, SDA_GPIO, SCL_GPIO));
I'm not entirely sure what changed, but now the program works correctly.

Hello @aledoors
You're welcome !
I'll close this issue


question: Any way to bypass the 256 color check in GifCodec#encodeGif?

Hello. It's a-me Mario.
I've been, once again, messing around with GifWrap and well, again managing to reach the limit of colors. This time, I'm fully lost on how it happened, but it did, and it's sad.
So, as the title suggests, are there any ways to bypass the need of quantizing frames?
Thank chu!
EDIT: I am able to make it run on a child process,to quantize without blocking the main thread, but I asking in case there are any other ways.

I'm not quite sure what you're asking. If you're asking whether there's a way to write a GIF with more than 256 colors, the answer is no, gives have at most 256 colors.
If you're asking for another way to reduce the color count, you could invent your own algorithm, but it is a tricky thing to do. The author of the quantization algorithm that gifwrap depends on has recently revised the module, and it may be faster now or perhaps have more options. Updating gifwrap to use the new algorithm may provide improvement -- I don't know. I'm not able to work on this right now, though.

Alright, that's fine. I'll see what I can do. Although, you might also find this interesting, but just resizing the gif that seems to cause the issue works when it is standalone, but breaks when combined with other gif frames (aka throws 256 color error). So, who knows. I'll see if I can debug it further.

I should clarify that GIFs have at most 256 colors per frame. However, you can force the encoder to use 256 colors across all frames when possible, to encode at most 256 colors across all frames, or to encode 256 per frame.  These are the colorScope options passed to encodeGif().
Gif.GlobalColorsPreferred = 0;
Gif.GlobalColorsOnly = 1;
Gif.LocalColorsOnly = 2;

If you read a GIF that encoded colors globally and pass the input GIF spec to the output encoder, it will try to encode them globally again. You may want to override that with one of the above options.

Anyway to fix this or at least quantize the output gif.
I am facing this issue when trying to resize a gif image by utilizing Jimp.

Anyway to fix this or at least quantize the output gif.
I am facing this issue when trying to resize a gif image by utilizing Jimp.

Jimp adds colors when resizing. If you resize using Jimp, you'll need to reduce the number of colors with one of the quantize*() methods before writing out the gif.

I'm assuming you've got this figured out now.


Update to 1.17-pre1

Thank you so much for this! I have very little time for Minecraft right now, because of job and real life, so I'm really glad you updated everything. Especially as it helps me use my little available time to focus on getting everything else published until/when 1.17 comes out.
I don't want to merge this directly, because I have so many mods and don't want to pollute each of them with the gradle files, and also we're already at 1.17-pre5 now, so it's kind of pointless to merge the pre1 update.
Could you please send a pull request that includes just the update to the source (extends DefaultTask) ? I could do that myself, but I want to give proper credit to you by merging instead of just stealing your code.

Alright, I opened PR #5! I'm glad I can help you.

Superseded by #7


TextBox.Filled.clearButton issue

My code: <TextBox Classes="Filled clearButton"> outputs the following after I input some text into control (dark theme):

Why are there two crosses? Can you fix that? I really want to use this control now

Hello, @CIRCULARKA
I can't reproduce this on latest nightly build. Try to install latest nightly build and check if issue still appears there or provide minimal reproduceable example please.

Hi. Sorry for not giving more information but I already tried to reproduce the issue in a new project and figured out that it was my fault, I forgot to exclude some styles that was doing all this to me. Thanks for response!


Update normalization-schema-dhcp.md

Removed a typo in the Field reference for DhcpIdSession alias

Thanks, @azurekid!


Fix PVQD loss function

Summary
The loss function in PVQD is defined as 1 - fidelity, in the current implementation it was 1 - |fidelity|².

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.004%) to 84.842%

Files with Coverage Reduction
New Missed Lines
%

src/sabre_swap/mod.rs
3
98.84%

Totals

Change from base Build<PHONE_NUMBER>:
-0.004%

Covered Lines:
65694

Relevant Lines:
77431

💛  - Coveralls

It's probably tricky to find a test that covers this, since the minimum is the same whether we square or not: if the optimizer used is good enough it would work with both this and the previous version. One option would be to test get_loss directly, but since there is some freedom in how the loss function is defined I think it would even be ok without a dedicated test 🙂

That would simplify things for sure :)


Connecting another running container to Docker in Docker socket

I'm trying to implement an image registry scanner.  I've found that the scanner I'm using needs access to a docker socket, but I don't want to connect it to the cluster's docker socket as it appears to be very unsafe.  I'd like to run Docker in Docker so I can pull the images from the registry to dind and then connect the scanner container dind's socket to scan the images.  Is this possible?

Yes; but if you are concerned about safety, DinD is not the right answer (it requires containers with elevated privileges). The safest solution would be to have a Docker Engine on the side, dedicated to the use of your security scanner; and add an authorization plugin to limit the API commands that it can execute (and disable privileged containers, for instance).


Companion files

Trying to summarise what we discussed at the dev meeting about this.
I wasn't exactly sure where to fit this into the main roadmap document, and it includes more description of why we want this than really belongs in a roadmap. But I wanted to record this somewhere before it fell out of my head.
Ping @SylvainCorlay

@takluyver thanks for putting this together! I am excited to see this move forward and will have a look at the PR later.

I wasn't in the room for much of this conversation, so forgive me if the details are somewhere in the hackpad, but was there any discussion of the user experience around this? I understand the version control benefits. For more casual users, is the expectation that they will need to email, put in Dropbox, FTP, etc. two files together to tell a complete story (inputs + outputs)?

With the default options, sharing the .ipynb file alone should give you exactly what it does already - inputs and outputs, but not widgets. If you share the companion file as well, you will also get the widgets.
We may add an option to store some or even all outputs in the companion file instead of the .ipynb. If the user enables this, then they will need to share both files for the recipient to see those outputs. But that option will not be on by default.

:+1: good summary!!

We also discussed having the companion file be essentially the ipynb itself with the widget state in the metadata. On save, you would get something like
foobar.ipynb (no widget state)
foobar.full.ipynb (contains widget state etc)
To avoid the churn, we could also decide to remove anything that is output from the first one.

To make it so we don't have to implement our own format and yet have one that is well supported, it would be great for people to check out asar. Asar

Supports random access
Uses JSON to store files' information

@rgbkrk hooray for using existing solutions!

I've merged the PR to move the repo forward, but I'll make a quick new one with the comments and fixes made above by @damianavila and @rgbkrk, that weren't fully addressed.


Cancel concurrency issue

We found an issue in the way cancel() is called on the Call after onResponse() is called on the callback:
Even after onResponse was called on our callback on a different thread, cancel() is still called from the main thread due to its concurrency nature - therefore cancel() calls cancel() on the RetryAndFollowUpInterceptor which calls cancel() on the StreamAllocation which we believe gets us in a state where we have a successful response with no body - response.body().string() returns null.
We believe that a fix for this might be to do a return in the cancel() method on the RealCall class if the AsyncCall delivered response to the callback.

We definitely intend to support canceling a request even if its body has been (partially) delivered. I’d like to write a test to confirm that the response body is never null if its response is canceled; that would be a horrible bug.

Looks like CallTest.canceledAfterResponseIsDeliveredBreaksStreamButSignalsOnce() already exercises this case. If you could provide an executable test case that demonstrates a null response body that’d be something we would act on immediately.
We support canceling a response even if its body has been (partially) delivered is so that you can avoid downloading and processing a large response object if you change your mind. Calling cancel() will cause that response body to throw an IOException which you can recover from gracefully. Use Call.isCanceled() if you want to differentiate between IOExceptions caused by local cancelation and those caused by other means.


mgmt, improve autocent script

For nested service configure like this containerservice/Microsoft.ContainerService/fleet
Some RP defines multiple subfolder, and requests multiple SDK packages for them.

Changed logic highlighted with comments.
Others are mostly var -> const/let, or npm run format.

/check-enforcer override

all passed, but seems not able to sync to github
https://dev.azure.com/azure-sdk/public/_build/results?buildId=2871051&view=results


added guess word gamer

hi @artimanputra
I have attached my word guesser game in which one can play it on real time.
one has to type keys on the keyboard only to answer this.
pls review it

hi @artimanputra I have attached my word guesser game in which one can play it on real time. one has to type keys on the keyboard only to answer this. pls review it

Please share a screen recording of your project

hey @artimanputra
I have shared the video
pls have a look to it
https://github.com/ssitvit/Code-Canvas/assets/97821844/c750d635-18b2-49ad-b603-e54597d1a8e1

hey @artimanputra
I have shared the video
pls have a look to it
it is above
thank you for your time

Good work !!


[WIP] Pluggable source provider

What is the context for this pull request?

More soon.
What changes were proposed in this pull request?

More soon.
Does this PR introduce any user-facing change?

Yes. This PR exposes a new way to plug in a source from which an index can be created.
More soon.
How was this patch tested?

Will add some tests.

@sezruby Did you get a chance to implement #197 using this PR?


Advanced Color-Node

We should have a special color node that includes hex, keyword and rgb(a), hsl(a)

addd HexColor node. rest ones seem useless right now as they already exist as simple Functions


Method of updating files in Doctor data dir

Files stored in the Doctor data dir need to be updated somehow, if they are out of date (e.g. Lua filters getting an update).

Ended up just storing all cached embedded files in a directory called embedded. On update, Doctor simply deletes this directory, which means it will write all files again. This might lead to long wait times when storing Tectonic and Pandoc here in the future, but that'll have to wait for now


CON-2114 Unwanted  is no longer rendered in an empty cart

Coverage decreased (-0.02%) to 33.214% when pulling 3f6a84e987bb1a54c0de41a7295083251c1a19e5 on CON-2114-Unwanted-spacing-when-cart-is-empty into 2d75828ed5e6d75b999f4d383b0027f0f117ea3a on master.


Build Failed while compiling UWP in Xamarin.Forms Project

Hi, When I add the acr.userdialogs to the UWP Project and compile it the error below shows up. I'm using Acr.Userdialogs 4.2.0
Severity    Code    Description Project File    Line
Error       Could not copy the file "C:\Users.nuget\packages\Acr.UserDialogs\4.2.0\lib\uap10.0\Acr.UserDialogs\Acr.UserDialogs.xr.xml" because it was not found.

Having the same problem here. Would be nice to see a fix for this as soon as posible :)

Does 4.2.1 on nuget resolve the issue for you?

Sorry, No Still unable to copy the file Acr.UserDialogs.xr.xml but the file was there in the specified destination.

No such luck for me I'm afraid :(
still get "Error		Payload file 'C:\Users\myuser.nuget\packages\Acr.UserDialogs\4.2.1\lib\uap10.0\Acr.UserDialogs\Acr.UserDialogs.xr.xml' does not exist.
One thing to note: My UWP project references Xamarin Forms PCL project, so should'nt the reference to Acr come from the portable-win+net45+wp8+win8+wpa81 folder instead of uap10.0?

Please confirm it says Acr.UserDialogs.xd.xml and not Acr.UserDialogs.xr.xml (it should have the XD portion, not xr)

It is Acr.UserDialogs.xr.xml Manually copying it under Acr.UserDialogs Folder works for me.

Hi, I using version 4.2.1.
The same error:
Payload file 'C:\Users\wcoder\.nuget\packages\Acr.UserDialogs\4.2.1\lib\uap10.0\Acr.UserDialogs\Acr.UserDialogs.xr.xml' does not exist.

@Femilshajin, thanks!
Your hack is working!
But, I thing it is not good.
Someone has bug, maybe in Nuget?

@aritchie this issue still exists.  I can confirm that is is the Acr.UserDialogs.xr.xml and NOT Acr.UserDialogs.xd.xml as you say it should be.
I installed the package using Visual Studio's "Package Manage Console" and the command Install-Package Acr.UserDialogs.  After running that command, I do not have a 'C:\Users\<name>\.nuget\packages\Acr.UserDialogs\4.2.1\lib\uap10.0\Acr.UserDialogs\' folder in my system; but once I create on and copy the Acr.UserDialogs.xr.xml from 'C:\Users\<name>\.nuget\packages\Acr.UserDialogs\4.2.1\lib\uap10.0' into it; the compiling works fine.

I am having a hell of a time causing this issue.  All compiles work for me on past versions.  I have released an update to remove any trace of the xr.xml inside the nuget packages which is what they recommend for this XD files as well.
Please give it a try and send feedback if good please

I lied.  I actually made sure it includes the Xr.xml in the proper folder.  Hopefully this solves things

4.3.4 seems to have fixed UWP for me.

@cleardemon Thanks for the update.  Anyone else?

It doesnt work for me. I cant initiliaze it. no instance. how to get an instance? I am able to run same code in XF on android

Works for me, too

When will be resolved bug with ShowLoading on WP 8.1 (XAML parsing failed)?

Probably never.  I'm not getting any help with wp81 and the platform is absolutely useless to me to invest any time in to.
Please don't ask about timelines in the future!  If you want support for something in the future, why not make a PR to help out.

@aritchie, sorry for the (very) late update but we we stuck in an unrelated issue updating our systems.  Now that we have, your fix has worked wonderfully.  Thank you.

I have the problem resurrected in  March 2017 - beta version <IP_ADDRESS> -beta
Payload file 'C:\Users\john.nuget\packages\Xam.Plugin.Connectivity\<IP_ADDRESS>-beta\lib\UAP10\Plugin.Connectivity\Properties\Plugin.Connectivity.rd.xml' does not exist.	nGar10k.UWP
going back to the stable version just for the UWP project gets around it
trouble is that the nuget install on the latest version does not create any deeper folders that UAP10 so the ..UAP10\Plugin.Connectivityand subfolders and files just dont exist

@johnm01 I think you have the wrong project

Ahh yes - but the symptoms arevery similar- Thought  I was commenting on Plugin.Connectivity thread
thanks anyway

I have the same issue but for 7.0.3 release...


<Box width={[1, 1/2]} />  always has 'display:block'

not quite sure if I understand correctly, but as far as I can see, the  element is already rendered as a  with  display:block, not inline-block, so wouldn't it always take the full width?
here is an example from the documentation, don't understand how it works:
// Different widths at different breakpoints
<Box
  width={[
    1/2,
    1/3,
    1/4,
    1/6
  ]}
/>

I see....
it effectively takes the entire rendering line, but leaves spaces on the right , so the effect as padding-right. Shouldn't Box has display:inline-block ??

the other options is to
<Flex>
    <Box width={1/2}/>
    <Box width={1/2}/>
</Flex>

since the div is wrapped inside a flex, it wont' take over the entire row;  but again, this is not straight-forward and I would't expect to do so

CSS width can be used for a lot of different things, but your second example is almost identical to the first example in the readme, so I think you've figured it out – see https://developer.mozilla.org/en-US/docs/Web/CSS/width for more about using width


Roaming Status API Proposal

A proposal for a Roaming Status API has been uploaded here. Please add any comments you may have.
Unfortunately, I don't have permissions to add labels to this issue, so maybe somebody could do that for me.

Reopened in the DeviceStatus Sub Project

I uploaded the original document to here and fixed the broken link in my original comment on this issue.
The idea is to discuss the use cases proposed in the document as a possible way to evolve the Device Status API, so I will keep this issue open for now.

@eric-murray please close Issue


Investigate Dependabot improvements wrt release notes

In Enable dependabot updates #1044, we enabled Dependabot updates for this repository. As part of that we changed the release notes linter to ignore Dependabot PRs because the Dependabot configuration does not allow to define the PR description and therefore cannot have a release note section.
The side effect of that is that release notes will never contain Dependabot updates. We might want to change this, especially as Dependabot updates are security-relevant.
One idea was to update the release notes generation to specifically handle Dependabot PRs. But there might be other approaches.

From grooming: For anyone picking-up this one, it might be needed to modify the draft_release_notes.sh to look for specific PR's generated by dependabot.


Collapse build.yaml and build-pr.yaml together

They're equivalent, except triggering conditions. The config can just specify both push and pull_request.

Since secrets aren't passed into runs for PRs, all the acceptance tests are skipped on PRs: https://github.com/docker-credential-magic/docker-credential-magic/pull/42/checks#step:6:7
We could either keep this how it is, or explicitly skip acceptance tests if the run is being triggered by PR.


Private symbol class properties treated as public

TypeScript Version:  2.7.0-dev.20171116
Code
const _data = Symbol('data');

export class User {
    private [_data] : any;
};

Expected behavior:
Compiles successfully.
Actual behavior:
TS4031: Public property '[_data]' of exported class has or is using private name '_data'.
The issue is that [_data] is not a public property, so it shouldn't care if it's using a private name because it's not meant to be visible outside of the User class.

some background in https://github.com/Microsoft/TypeScript/issues/19545#issuecomment-340490459.
Today there is no way to write this file a .d.ts and not expose _data. that is basically what the error is telling you.
we can possibly fix this by adding an auto-generated export {} to the bottom of the generated .d.ts and not show the error.

I think I was under the assumption that private properties don't get emitted to the .d.ts files. Is that not the case?

I think I was under the assumption that private properties don't get emitted to the .d.ts files. Is that not the case?

They need to be emitted to the .d.ts to avoid derived classes creating members with he same name.

Two things:

The symbol is technically still accessible, it's just not possible to reference it directly.
Automatically exporting the symbol seems questionable.

They need to be emitted to the .d.ts to avoid derived classes creating members with he same name.

If the symbol isn't exported, it's literally unknowable, so you can't have the same name it in a derived class (which is, of course, our entire reason for not exporting it)

If the symbol isn't exported, it's literally unknowable, so you can't have the same name it in a derived class (which is, of course, our entire reason for not exporting it)

There is some historical context why things work the way they do.. please see https://github.com/Microsoft/TypeScript/issues/19545#issuecomment-340490459 for more details.

@dpogue I've opened a PR to elide the property like @mhegazy was talking about; but your original error in the OP is not the error you'd get from the declaration emit of that property - the error you've given occurs when a global Symbol type can't be found - do you have lib: ["es6"] included?

I do have lib: [..., "es2015.symbol", ...] in my tsconfig.json


Remove deprecations for sphinx 1.8

Getting ready for sphinx 2.0

We still have access to builder.css_files and builder.script_files.
https://github.com/sphinx-doc/sphinx/blob/2b1512749a2532f5e8bc7909ca70ce6395f169fd/sphinx/builders/html.py#L258-L262

Also, I was thinking in removing this whole block
https://github.com/rtfd/readthedocs-sphinx-ext/blob/e0e3f9c85599f2afcfe8917a4f3d4436fda9e22a/readthedocs_ext/readthedocs.py#L67-L93
It's only used to support sphinx_rtd_theme < 4.0, but 4.0 was released in 2013 https://pypi.org/project/readthedocs-sphinx-ext/#history, and from the rtd side we always install kind of the latest version of the theme, so, if people have the theme pinned to <4.0, the docs will look weird, but they can fixed it with just pinind the extension too or updating the theme.

And we need to consider this https://github.com/rtfd/readthedocs-sphinx-ext/issues/35#issuecomment-396081131, but as I mentioned above, this is only needed for sphinx_rtd_theme < 4.0 (2013)


Remove six tile wide flow limitation

I'm submitting a ...  (check one with "x")
[X] bug report => search github for a similar issue or PR before submitting
[X] feature request
[] support request
[] general question

Current behavior (how does the issue manifest):
WebUI only allows you to create a flow that contains 6 tiles wide
Expected behavior:
Either unlimited or at least a high number of consecutive steps would be allowed.  Limitation doesn't exist in json config, nor does it exist in the commercial Enterprise version.
Minimal steps to reproduce the problem (not required if feature enhancement):
What is the motivation / use case for changing the behavior?
Having 6 steps means that the UI is just a toy :-)
Please tell us about your environment (Operating system, docker version, browser & web ui version, etc):
Flogo version (CLI & contrib/lib. If unknown, leave empty or state unknown): 0.5.x
Additional information you deem important (e.g. issue happens only occasionally):

Having this limit, would developers move away to use another environment to build visually microservice applications. It is fairly easy to hit that limit if you consider that with already two Log activities would leave space for another 4 slots to build logic.
I disagree with the built-in limitation and agree with Andy.

+1
I've done some crazy stuff to get more complex logic fitted into flogo web, like a flow calling another flow using MQTT but you just hit that limit too soon.
I did hack the web UI to allow for a bigger number and that works. Just let me know if I should post that or not ;)

JFYI -- We can do a subflow invoke rather than calling another flow via MQTT 😄

the limit has been increased. if subflows do not help, please reopen this issue.


FCPX Hacks Menubar Shortcuts Sometimes Fail

@randomeizer has noticed that in some situations the FCPX Hacks menubar shortcuts (i.e. "Create Optimised Media", etc.) only seem to work when the Preferences window is open.
Chris to investigate!

Possibly fixed. Will close issue for now and reopen if problems come back.


chore: remove unneeded rpc methods from test api and add doc with rpc description

prnet cleaned-up successfully


ICECandidate IP -> Address

ICE RFC specifies this value as Address and not IP. We need to fix
this divergence to implement mDNS
Relates to pion/webrtc#699

Codecov Report

Merging #20 into master will increase coverage by 0.04%.
The diff coverage is 80%.

@@            Coverage Diff             @@
##           master      #20      +/-   ##
==========================================
+ Coverage   64.21%   64.25%   +0.04%
==========================================
  Files          11       11
  Lines        1059     1063       +4
==========================================
+ Hits          680      683       +3
- Misses        285      289       +4
+ Partials       94       91       -3

Impacted Files
Coverage Δ

jsep.go
0% <0%> (ø)
:arrow_up:

common_description.go
82.6% <100%> (ø)
:arrow_up:

unmarshal.go
61.27% <100%> (+1.06%)
:arrow_up:

ice.go
100% <100%> (ø)
:arrow_up:

util.go
62.79% <0%> (-3.88%)
:arrow_down:

Continue to review full report at Codecov.

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

Is this because the value needs to be a proper cidr for multicast? I thought that what I implemented with IP can do just that. net.IP can hold a proper CIDR in it.

@trivigy sorry I had to push it through since pion/ice and pion/webrtc were merged. I don't want to break master :/
This is for mDNS. These values are now coming across as $UUID.local

No worries at all. Don't wait for me. I am too busy with other stuff and cannot dedicate the same amount of work time.


Stack Set Deployment Option for Region Concurrency

I'm using v2.0.0 mainly for rolling out manged AWS Config rules to almost all account in all regions. So in sum more than 200 stack instances are created. The stack set is created with following deployment options:

Maximum concurrent accounts - 100
Failure tolerance - 10

See https://github.com/awslabs/aws-control-tower-customizations/blob/677d12e16ecee9ae85c7fa056ded96aac5441525/source/aws/services/cloudformation.py#L28
So in case of so much instances with the above concurrent option it takes very long to apply the change to all accounts. Therefore it would be create to make the region concurrency configurable in manifest.yaml, e.g.
resources:
  - name: pcs-config-rules
    deploy_method: stack_set
    deploy_options:
       - region_concurrency = parallel

Based on https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/stacksets-concepts.html sequential deployment is the default selection

Thanks for raising this issue. This is already on our backlog and we plan to ship the fix in the next release.

This change was released in v2.1.0.


Recover Onchain & Lightening Wallet

[ ] Need to explore that allows this and how to implement this in Tribe RGB

Completed


UG Edits

Codecov Report

Merging #164 (f0ec6a1) into master (6758270) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #164   +/-   ##
=========================================
  Coverage     75.22%   75.22%
  Complexity      661      661
=========================================
  Files           100      100
  Lines          1994     1994
  Branches        238      238
=========================================
  Hits           1500     1500
  Misses          406      406
  Partials         88       88

Impacted Files
Coverage Δ

...ain/java/seedu/address/logic/commands/Command.java
100.00% <ø> (ø)

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


ScaleManager Issues with Google Fonts

For reference, I posted this in the forums
I am using the latest phaser ce build, phaser 2.7.3. My target browser is desktop Chrome
(I don't care much for any other browser, mobile platform, etc)
Explanation of my issue: I have a text, at position (0,0) [Therefore, no possibility of it no being a whole number, etc]. It is not scaling properly.
=> Minimal fiddle testcase showing my problem: http://jsfiddle.net/1g3cj8md/
=> Expected outcome if game is not scaled: http://jsfiddle.net/encznmqq/1/
Is this a bug? Am I missing anything? Basically, the rendering of a google fonts when the game is scaled seems off. This happens whether you use a pixel-friendly font like VT323 or a more 'normal' font.
I pressume that if this is a bug, it has to do with how scaling manages text. Perhaps a workaround is using other types of fonts?

@ldd close pls


alertmanager-irc-relay: 0.4.3 -> 0.5.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/google/alertmanager-irc-relay/releases.
meta.description for alertmanager-irc-relay is: "Alertmanager IRC Relay is a bot relaying Prometheus alerts to IRC"
meta.homepage for alertmanager-irc-relay is: "https://github.com/google/alertmanager-irc-relay"
meta.changelog for alertmanager-irc-relay is: ""
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.5.0 with grep in /nix/store/nxjki6bf7drklxnbbsrahxqc4xj2v14f-alertmanager-irc-relay-0.5.0
found 0.5.0 in filename of file in /nix/store/nxjki6bf7drklxnbbsrahxqc4xj2v14f-alertmanager-irc-relay-0.5.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
alertmanager-irc-relay

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/nxjki6bf7drklxnbbsrahxqc4xj2v14f-alertmanager-irc-relay-0.5.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A alertmanager-irc-relay https://github.com/r-ryantm/nixpkgs/archive/ba9c25567c6b89398db3fcf5e9117fb0d71c9070.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/nxjki6bf7drklxnbbsrahxqc4xj2v14f-alertmanager-irc-relay-0.5.0
ls -la /nix/store/nxjki6bf7drklxnbbsrahxqc4xj2v14f-alertmanager-irc-relay-0.5.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

alertmanager-irc-relay

Maintainer pings
cc @ymatsiuk for testing.

Result of nixpkgs-review pr 196876 run on x86_64-linux 1

1 package built:

alertmanager-irc-relay


RuntimeError: DataLoader worker (pid(s) 10116, 14052) exited unexpectedly

how to solve it

Change ‘num_workers’=0 on line 240 in SCDataset.py


Added ETHStaker community videos

Added community call #3, #14 and the RocketPool Quiz. Also changed the spelling of 'eth2 clients' to 'ETH2 clients' so that ETH the currency was spelled out the same throughout the index.
Section: Community - ETHStaker
Item: Community Call #3, #14 and RocketPool Quiz.
Action: Add

By submitting this pull request, I promise I've read the contribution guidelines.

Awesome, thanks


No Way to Delete Ticket

Need to add way to delete a ticket, perhaps in the ticket view or on the dashboard?

Resolved this.


Added a composer.json file

Summary
This adds a composer.json file to allow for this repository to be included in a composer project.
Metadata

Question
Answer

Did you use a meaningful pull request title?
yes

Did you apply meaningful labels to the pull request?
yes

Did you perform a self review first?
yes

Documentation reflects changes?
no

CHANGELOG reflects changes?
no

Unit/Functional tests reflect changes?
n/a

Did you perform browser testing?
yes

Risk level
Low

Relevant links
RIG-37

Looks good!
/approved


Android - upgrade to API level 30

I updated the targetSdkVersion and compileSdkVersion to 30 in build.gradle and after rebuild I have this message on MapView.
Using on emulator Pixel_4_XL_API_30(AVD) - 11
App won't run without Google Play services, which are not supported by your device.
      ...
        compileSdkVersion = 30
        targetSdkVersion = 30

Please read the documentation and follow the issue template next time when opening an issue.
https://github.com/react-native-maps/react-native-maps/blob/2236ad9ffc18a3e273e6d302ef738c1ecb1f6322/docs/installation.md#android-emulator-issues


Lab 04 Created Bike class

Bike class has a sub class, TrainingBike.

Hey Johnny,
I am just curious, do the numbers in these commit links reflect our grade at all, because it looks like a murder scene in 'removed javadoc' and that is concerning.


header file problems

The following code is included in the two header files "logging.h" and "raw_logging.h".
namespace google{ #include "glog/log_severity.h" #include "glog/vlog_is_on.h" ... }
Why not change it to
#include "glog/log_severity.h"
#include "glog/vlog_is_on.h" namespace google{ ... }
Then add namespace in "log_severity.h" and "vlog_is_on.h".
Because when I code in eclipse,it always tips problems like "Function 'InitVLOG3__' could not be resolved".

I cannot figure out what are you asking. Please rephrase your question.


Further TemplatePart Cleanup in Code-Behind and XAML

Is your feature request related to a problem? Please describe.
After the TemplatePart cleanups adding the attribute and standard "PART_" prefix some additional ideas were brought up starting here.

Have standardized constants defining ALL control template part names in code-behind.

Use these everywhere in code behind.
Some controls already do this, but naming of these constants is very inconsistent

Use Name={x:Static MyControl.TemplatePartName} in XAML instead of magic strings

Open Discussions:

What to do with selectors: https://github.com/AvaloniaUI/Avalonia/discussions/9086
An additional idea instead of string constants is to add a TemplatePart enum to each control's code-behind. WinUI has started doing this in places. This was first mentioned here: https://github.com/AvaloniaUI/Avalonia/issues/7432#issuecomment-1066183609
Its possible to generate the TemplatePart name string constants or an enum using a source generator and the existing TemplatePart attributes: https://github.com/AvaloniaUI/Avalonia/issues/11934
If the TemplatePart attribute strings aren't used as the source, these could use a technique from UWP with the nameof() and then the control part reference. Basically, the variable would be named exactly the same as the template part itself allowing nameof() usage. This would remove the need for a string constant as well as the information is simply in the ref variable name that holds the actual template part control. Mentioned here: https://github.com/AvaloniaUI/Avalonia/issues/11934

Describe alternatives you've considered
Alternative would be do nothing. However, ideally the XAML would be locked down quite a bit after 11.0 so we don't need to keep updating styles/themes a lot in the years to come. So if this is going to be done, it should be done now.
Additional context
See the discussion here https://github.com/AvaloniaUI/Avalonia/pull/8793#issuecomment-1221491873
FYI: @rabbitism @maxkatz6 @timunie As this continues our discussion but makes it a separate issue.

Related
https://github.com/AvaloniaUI/Avalonia/discussions/9086

An additional idea instead of string constants is to add a TemplatePart enum to each control's code-behind. WinUI has started doing this in places.

Do you have a link to a one of these places? I remember it was discussed somewhere, but I don't see it now.

@maxkatz6 Here is where enums were first mentioned https://github.com/AvaloniaUI/Avalonia/issues/7432#issuecomment-1066183609

I added an open discussions section to the issue with more ideas/topics collected from the various threads.

Taking a look at the source generator idea, I think it's practical to declare template parts like below
public class MyUserControl: TemplatedControl
{
    [TemplatePart(typeof(ItemsContainer))]
    public const string PART_Container = "PART_Container";
}

Then it generates the code mentioned in https://github.com/AvaloniaUI/Avalonia.NameGenerator/issues/77, meanwhile the string itself can be utilized in control template.
For selector, I think IDE plugin/extension support is required, or it's not more beneficial to use a complex syntax than string literal.

For selector, I think IDE plugin/extension support is required, otherwise it's not more beneficial to use a complex syntax than string literal.

Compiler would still validate existence of the const in compile time.

For selector, I think IDE plugin/extension support is required, otherwise it's not more beneficial to use a complex syntax than string literal.

Compiler would still validate existence of the const in compile time.

Do you think it's possible to support static pseudoclass reference in selector as well?

Unless I am mistaken, I think all the changes discussed here are non-breaking and can be done at some point in the future once we decide how to handle everything (such as selectors).
The only potential issue I see is with bad template part naming -- that can be improved now though. However, I don't have the time myself to go through them all again. @timunie I know we talked about some further standardization there in the past.

So the real problem is we have all the information we need already in the TemplatePart attributes... however there is NO way to get a reference to the TemplatePart attribute instance to actually pull the data from it and use it in source code or XAML... Seems like a missing C# feature honestly. We should be able to define instances of an attribute as a variable somewhere then assign them as attributes of a member but also keep the attribute references around for use elsewhere.
Anyway, an Attribute can still reference other source code even if it is one-directional without reflection. This means we need an intermediate data structure to store the template part information that is usable from everywhere else: C#, XAML and the TemplatePartAttribute itself.
So rather than just generating strings, I would propose creating a new TemplatePartData struct that can be used in code directly. See example below. Unless I'm missing something this solves all issues discussed above. It is similar to defining dependency properties (avalonia properties).
public struct TemplatePartData
{
   public TemplatePartData(string name, Type type)
   {
      Name = name;
      Type = type;
   }

   public string Name { get; init; }

   public Type Type { get; init; }
}

Used as follows:
[TemplatePart(TextPresenterPart.Name, TextPresenterPart.Type, IsRequired = true)]
public class TextBox
{
    // Available for use anywhere in source code without code generation, also available from XAML x:Static
    public static readonly TemplatePartData TextPresenterPart = new TemplatePartData("PART_TextPresenter", typeof(TextPresenter));

    protected override void OnApplyTemplate(TemplateAppliedEventArgs e)
    {
        if (e.NameScope.Get<TextPresenter>(TextPresenterPart.Name) is { } textPresenter)
        {
             // Do something
        }
    }
}


subprocess.CalledProcessError: Command '('coverage', 'json', '-o', '-')' returned non-zero exit status 2.

2023-12-04T14:04:41.0481051Z ##[notice]Starting action
2023-12-04T14:04:41.2909319Z ##[notice]HTTP Request: GET https://api.github.com/repos/descope/python-sdk "HTTP/1.1 200 OK"
2023-12-04T14:04:41.2916256Z ##[notice]Generating comment for PR
2023-12-04T14:04:41.6464808Z ##[error]Critical error. This error possibly occurred because the permissions of the workflow are set incorrectly. You can see the correct setting of permissions here: https://github.com/py-cov-action/python-coverage-comment-action#basic-usage
Otherwise please look for open issues or open one in https://github.com/py-cov-action/python-coverage-comment-action/
Traceback (most recent call last):
  File "/usr/local/lib/python3.11/site-packages/coverage_comment/subprocess.py", line 18, in run
    return subprocess.run(
           ^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/subprocess.py", line 571, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '('coverage', 'json', '-o', '-')' returned non-zero exit status 2.

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/local/lib/python3.11/site-packages/coverage_comment/main.py", line 40, in main
    exit_code = action(
                ^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/coverage_comment/main.py", line 92, in action
    return process_pr(
           ^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/coverage_comment/main.py", line 123, in process_pr
    _, coverage = coverage_module.get_coverage_info(
                  ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.11/site-packages/coverage_comment/coverage.py", line 84, in get_coverage_info
    subprocess.run("coverage", "json", "-o", "-", path=coverage_path)
  File "/usr/local/lib/python3.11/site-packages/coverage_comment/subprocess.py", line 27, in run
    raise SubProcessError("\n".join([exc.stdout, exc.stderr])) from exc
coverage_comment.subprocess.SubProcessError: {"meta": {"version": "7.3.2", "timestamp": "2023-12-04T14:04:41.489052", "branch_coverage": false, "show_contexts": false}, "files": {"descope/__init__.py": {"executed_lines": [1, 10, 11, 18, 19], "summary": {"covered_lines": 5, "num_statements": 5, "percent_covered": 100.0, "percent_covered_display": "100", "missing_lines": 0, "excluded_lines": 0}, "missing_lines": [], "excluded_lines": []}, "descope/_auth_base.py": {"executed_lines": [2, 5, 6, 8, 9], "summary": {"covered_lines": 4, "num_statements": 4, "percent_covered": 100.0, "percent_covered_display": "100", "missing_lines": 0, "excluded_lines": 0}, "missing_lines": [], "excluded_lines": []}, "descope/auth.py": {"executed_lines": [1, 3, 4, 5, 6, 7, 8, 9, 10, 12, 14, 15, 19, 20, 21, 23, 35, 47, 48, 49, 54, 62, 63, 65, 74, 76, 77, 78, 86, 87, 88, 90, 91, 92, 94, 95, 96, 97, 99, 100, 102, 103, 104, 116, 123, 131, 132, 134, 137, 146, 147, 149, 152, 160, 161, 163, 164, 165, 171, 172, 173, 174, 177, 179, 180, 183, 184, 186, 187, 189, 190, 191, 192, 193, 194, 195, 196, 197, 198, 199, 200, 201, 202, 203, 204, 205, 206, 208, 210, 212, 213, 214, 220, 221, 225, 227, 228, 229, 235, 236, 240, 242, 243, 244, 250, 251, 255, 257, 258, 259, 260, 266, 267, 268, 269, 273, 274, 275, 276, 282, 283, 287, 288, 292, 295, 296, 297, 298, 302, 303, 304, 306, 307, 308, 309, 315, 316, 317, 318, 319, 325, 326, 327, 333, 334, 335, 340, 342, 343, 344, 350, 352, 353, 355, 356, 358, 364, 366, 372, 374, 375, 376, 377, 378, 379, 384, 385, 386, 387, 388, 393, 395, 396, 399, 400, 403, 404, 407, 410, 414, 415, 416, 419, 424, 428, 433, 435, 438, 440, 442, 449, 450, 451, 452, 453, 454, 455, 458, 459, 463, 465, 466, 473, 475, 481, 485, 486, 487, 489, 490, 491, 492, 493, 494, 495, 498, 501, 507, 508, 509, 510, 516, 517, 518, 522, 523, 524, 528, 529, 530, 532, 533, 541, 543, 544, 551, 552, 559, 560, 566, 567, 569, 572, 573, 579, 580, 583, 585, 588, 595, 597, 598, 600, 601, 603, 609, 610, 616, 617, 618, 620, 626, 628, 647, 648, 649, 650, 651, 652], "summary": {"covered_lines": 269, "num_statements": 288, "percent_covered": 93.40277777777777, "percent_covered_display": "93", "missing_lines": 19, "excluded_lines": 0}, "missing_lines": [16, 17, 50, 51, 389, 391, 502, 534, 545, 589, 621, 629, 630, 636, 637, 641, 642, 645, 653], "excluded_lines": []}, "descope/authmethod/__init__.py": {"executed_lines

https://github.com/descope/python-sdk/actions/runs/7086877486/job/19288591568
I do not think this is a permission error, as permissions are there and it's been working well for a while
looks like the input is truncated by this might be github logs issue and not really

Indeed, this is probably not a permission issue (this message is there because it used to be the case that most issues were permission issues, but it might not be the case anymore)
I'm not sure the truncating is by GitHub and not by someone else in the stack, but it's hard to say.
I guess one possible way to help debugging would be to output stderr before stdout when there's an issue.

I've made a PR that adds logs. You may need to activate debug mode.
Please let me know whether this gives you enough info to solve the issue on your side or if there still is a bug.
Note that I have no idea who ends up truncating the output.

https://github.com/descope/python-sdk/actions/runs/7086877486/job/19345603602
##[debug]Command failed: args=('coverage', 'json', '-o', '-') path=PosixPath('.') kwargs={} exc.stderr=''

still no idea 🤷♀

Ok, so empty stderr, the command stops just like that. (or the stderr is entirely truncated, but... How ? Why ? Who ?)
Could it be a sigkill, like an OOM or something ? I should have displayed the exit code...

So feel free to rerun the CI and check the debug message.

Hi, happy new year :)
By lack of ideas, and input, I'm going to close this if it's ok with you. Please you or anyone else experiencing the issue: reopen or recreate if relevant. I'll be delighted to try and tackle this again if we have more input for investigating.

Hi there! I believe I have ran into the same issue.
##[debug]Command failed: args=('coverage', 'json', '-o', '-') path=PosixPath('.') kwargs={} exc.stderr='' exc.returncode=2

Please let me know what other information I could provide you.
Maybe bit of a hint: my source code is actually in src/ subdirectory, perhaps that's what the problem is?
I do see the output in form of json further down the log though:
coverage_comment.subprocess.SubProcessError:
{"meta": {"format": 2, "version": "7.4.4", "timestamp": "2024-04-11T09:20:51.821149", "branch_coverage": true, "show_contexts": false}, "files":  [... continues here, but eventually gets truncated]

Can you share your configuration from the workflow files (just the relevant parts)

Have you tried with:
        id: coverage_comment
        uses: py-cov-action/python-coverage-comment-action@v3
        with:
          GITHUB_TOKEN: ${{ github.token }}
          COVERAGE_PATH: src/

?

Yes, I have. It fails with No data to report.. Please note that previous configuration (without that parameter) finds coverage's data.

can you share the coverage configuration (from .coveragerc or pyproject.tml or setup.cfg)

Sure, version is coverage==7.4.4.
[tool.coverage.run]
branch = true
omit = ['*/migrations/*', 'manage.py', 'wsgi.py']
relative_files = true

[tool.coverage.report]
fail_under = 100
show_missing = true
skip_covered = true

Ok, I'll try to reproduce on a small example and see if it fails the same way

I am running into the same issue. I have an identical workflow defined in two repositories and it fails in one and passes in the other.
Shared workflow:
https://github.com/senzing-factory/build-resources/blob/v1/.github/workflows/python-coverage-comment.yaml
Failing:
##[debug]Command failed: args=('coverage', 'json', '-o', '-') path=PosixPath('.') kwargs={} exc.stderr='' exc.returncode=1
Output:
https://github.com/senzing-garage/template-python/actions/runs/8807954189
Caller workflow:
https://github.com/senzing-garage/template-python/blob/99-skern/.github/workflows/pytest-linux.yaml
Passing:
Output:
https://github.com/senzing-garage/sz-sdk-python/actions/runs/8808786075/job/24178545086
Caller workflow:
https://github.com/senzing-garage/sz-sdk-python/blob/main/.github/workflows/pytest-linux.yaml

For posterity, I hit this same error, I was just trying to run coverage against a single file in the root of the repo.. I moved it to a src folder, added a blank init.py file and added --cov=src/ to my pytest command and this action ran without error after that.

@ewjoachim do you have any progress with this? I am having something similar:
Traceback (most recent call last):
  File "/workdir/coverage_comment/subprocess.py", line [22](---sensitive---#step:6:23), in run
    return subprocess.run(
           ^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/subprocess.py", line 571, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '('coverage', 'json', '-o', '-')' returned non-zero exit status 1.

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/workdir/coverage_comment/main.py", line 44, in main
    exit_code = action(
                ^^^^^^^
  File "/workdir/coverage_comment/main.py", line 88, in action

Wait, a __init__.py in the src ? That's definitly not something expected. It means your top level package is src (so if you were to import it, it would be import src). It's strage you had to do that.

Well, I wan't very helpful on my side so it's good that you found your issue :)

For anyone facing this, I had the same problem. My issue was the coverage package mismatch - coverage pkg I used to generate the .coverage file was different from the one used in the action (minor version was different).

You had the same error message ? I think we should improve the error message if we can detect this case.

You had the same error message ? I think we should improve the error message if we can detect this case.

 That was somewhat hard to catch, too many possibilities for why this might happen.


Update package.json

Kept the docs command as it is because it can generate the Table of Contents for our README, the same has been used in #6 to generate the Table of contents

Looks good. LGTM


d.ts with import statement exports types that are not marked exported

TypeScript Version<EMAIL_ADDRESS>(also 2.4 and 2.5)
Code
// extra.ts
export var x = 3;

// iface.d.ts
import {x} from './extra';
// note: this next line is *not* exported
declare interface Foo { bar: string; }

// user.ts
import {Foo} from './iface';

Expected behavior:
This should fail to compile, because iface.d.ts did not export the Foo type.  Error message should be
 error TS2305: Module ... has no exported member 'Foo'.

Actual behavior:
It compiles.
Extra notes:

if iface.d.ts has no import statement, but instead uses export {} to become a module, then expected error is produced
if you remove import/export statements from iface.d.ts, then as expected you get the 'is not a module' error
if you rename iface.d.ts to iface.ts, then expected error is produced

In a .d.ts file we normally assume that the things you're writing are public exports, else you would not be writing them.

if iface.d.ts has no import statement, but instead uses export {} to become a module, then expected error is produced

This is interesting though, I didn't know export {} would make other things not exported. @RyanCavanaugh is that intentional?

Sometime between 0.8.3 and 0.9.1 we saw that ample declare export did not look that nice in .d.ts files. as a result declare was banned inside a declare block, and export was deemed "optional" in an ambient context (i.e. a declare declaration, or a .d.ts file).
ES6 modules come along, and we need to support them. export {} and export default have semantics that allow for exporting/renaming a local declaration. the current rules we had for .d.ts files did not fit with that. Just keeping the rule of implicit exporting means that we export declarations on the .d.ts that were not in the .ts. We added an exception for export {} and export default.
If the .d.ts file/ambient module contains at least one export {} or an export default, only exported declarations will be visible.


Why is the BN layer not added before the softmax function?

Hello：
pytorch-auto-drive/torchvision_models/common_models.py         341 line: output = F.softmax(output, dim=1)。
Hello, why didn't you add the BN layer before the softmax function?  In addition, does this "class EDLaneExist(nn.Module) " have any optimization space when the maximum number of lanes exceeds 6?

@daigang896 Actually if you are considering optimization, the problem may be from the 5 set here. But the effects seem minor on current results (only some 6 lanes on TuSimple) so we kept it for BC.
What do you mean by BN after softmax, is it a common practice? The intuition here is to infer existence based on pixel-wise probability map.

@voldemortX Hello, I'm just adding BN layer before this softmax, I'm not sure if it's useful.  Trying to see if it works. “pytorch-auto-drive/blob/master/torchvision_ models/common_ models. py:319: self.layers_ final. append(nn.Conv2d(32, 5, (1, 1), stride=1, padding=(0, 0), bias=True))". I set the number 5 a little bigger. I understand the intention here. I'm just thinking about how to improve the lane line accuracy a little more.

@daigang896 Sounds interesting, please inform me here if it works.

@voldemortX OK.

@voldemortX  It can not improve the accuracy of lane line detection. Maybe erfnet_SCNN or erfnet_RESA can only achieve a certain accuracy. The backbone network may need to be replaced.

@daigang896  Yes I agree that the bottleneck is the algorithms themselves. We are currently exploring alternative backbones such as RepVGG, MobileNet, and Transformers. Part of RepVGG might show up along with the upcoming refactoring (it's scheduled for 12.31 but we might not make it, just stay tuned).

@voldemortX OK, look forward to new updates.

@daigang896 RepVGG, MobileNet, and Transformers are now supported (baselines and some RESA/SCNN). Furthermore, BézierLaneNet (CVPR 22) is integrated, which requires both keypoint and binary seg annotations (it should be easy to get them from keypoint annotations). However, we don't see a much higher performance on public datasets. You can try them out yourself now with the latest code.
I'll close this issue for now. Feel free to reopen or open a new one for further discussions.


Using ly as screen locker?

Thanks for this cool DM, it works well on my Debian i3 machine.
How can I use this as a screen lock though like f.i. i3lock?
If I run ly on the console it just outputs a preview but not the actual login screen.

I don't think you can, but xsimplelocker is pretty good.

Well if you are on Debian then you can use i3lock-fancy as you screen locker.

I don't know if this is possible. If so, it's probably very complex to support: I don't think we will :(

if you are on Debian

Why Debian? It's a program available on pretty much all distros

@ThatOneCalculator cause I use Debian.


support judge type and error value on submit command

Codecov Report

Merging #161 into master will decrease coverage by 0.11%.
The diff coverage is 42.85%.

@@            Coverage Diff             @@
##           master     #161      +/-   ##
==========================================
- Coverage   89.45%   89.33%   -0.12%
==========================================
  Files          54       54
  Lines        2731     2738       +7
==========================================
+ Hits         2443     2446       +3
- Misses        288      292       +4

Impacted Files
Coverage Δ

atcodertools/tools/submit.py
58.33% <42.85%> (-2.05%)
:arrow_down:

Continue to review full report at Codecov.

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


Looking for a new maintainer

Sadly I don't have the time to maintain this project anymore. Anyone wants to step up?
/CC @GantMan

I could help with grooming android PRs but seems like react-native-community is the spot. I'll wait for any move there...

Thank you @mikehardy. Dropped you an email to define the workflow we could adopt while waiting for @rebeccahughes decision.

That's fine with me, although I need permission to create repos on that org to move it

@pvinis sorry to bother you by tagging you in here, but you just moved rn-diff-purge into react-native-community. I swear I looked in the docs and googled but I didn't see where you petition to move a repo there. How do you actually do it? Apologies again, and thanks in advance :-)

Hey. :)
Most people did it through the lean core issue.
I got asked to move it to RNC so that it's more "official" or more easy for people to find it.
There are some others too that I'm guessing it was after taking with one of the core contrib people.
Maybe we can tag @cpojer and I can also raise the question with the team.
I think this is a package that should definitely be in the RNC.

Forked it over to here: https://github.com/react-native-community/react-native-device-info
Let me know if you want to maintain it and I'll give you write access.

@cpojer Thank you so much! How will the subsequent publications to npm be handled? Who should we add as maintainer in npmjs.com?

Once we find a new maintainer, we can transfer it over to them. Feel free to add me (cpojer) to the npm package for now,

@machour for continuity sake if you weren't maintainer I would be afraid - I just showed up here :-), but if you were there for an assist I'd be willing to help

@cpojer Just added you to the npm package. Could you add me as a maintainer for the new repository as suggested by @mikehardy ? I'll help with what I know until the community own it back.

@cpojer could you grant @mikehardy karma on this repo to help triage issues? (labels / close & open on issues and PRs).
Thank you!
PS: let me know if there's a better medium to bug you with this sort of things.

@machour I just gave you admin rights again on this repo so you can do these things on your own :) Added @mikehardy to this repo, too :)


Better IDE Support

I've recently started using PHPStorm and one of the main annoyances in using plates is that there isn't any easy IDE support for showing the existing variables or what their types are.
I think something similar to magento2 where you have one block variable which handles all of the interaction might be a good solution. Will need to think it through on how it would best work with 4.0.

Maybe we could change $data in the template from being an array to also accept mixed or something?
Then we could do:
/** @var SomeViewModel */
$vm = $v->data

Another aspect that could use better IDE completion is the Engine Methods, would be great if that could somehow provide better IDE help as well.

And another are render context methods. Maybe something like what php-di does would suffice:
http://php-di.org/doc/ide-integration.html

Hi @ragboyjr. I'm trying to find a solution for how to make PhpStorm understand $v->insert() method, and not flag as "not found".
Do you have any further thoughts on the subject here?
https://stackoverflow.com/questions/15634021/how-to-document-magic-call-and-callstatic-methods-for-ides?noredirect=1&lq=1 seems to be the most popular question on the subject, but I'm not sure how exactly to apply it.

@lkraav thanks for looking at this, part of me wonders if maybe we do something like: https://github.com/netz98/n98-magerun/blob/master/src/N98/Magento/Command/Developer/Ide/PhpStorm/MetaCommand.php
where we just have a tool that you can run and generate phpstorm meta output for the IDE.

@ragboyjr nice find!
Are you able to put a bare skeleton of such a generator together, maybe for just one method like insert()? Looking at MetaCommand.php, getting started might be a bit over my head, but I think I could stoke a small pre-started fire further.
AFAICT $STATIC_METHOD_TYPES has been deprecated for a while, already? Not sure what has to be adjusted. I wonder if need to find JetBrains Slack from somewhere and ask those deeper in the know.

On Discord, Daniele from JetBrains recommended looking at https://github.com/King2500/symfony-phpstorm-meta

The IDE Helper plugin for CakePHP supports automatic collection of view template variables from (PHP) files:
https://github.com/dereuromark/cakephp-ide-helper/releases/tag/0.13.19?fbclid=IwAR0KuINb6808_ZZcr_YPLD1hnr30ZC5SDW3TTWaOwhRbnA30pBsmrC6BUzw
Maybe their concept / solution could also be helpful for Plates.

I think the solution around this is moving to a new (optional) component model that is IDE friendly by design instead of mucking around with PHPStorm meta helpers.


Amplify CLI 12.10 - No queries found to generate types for

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
18.17.0
Amplify CLI Version
12.10
What operating system are you using?
Mac
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
No manual changes made
Describe the bug
after upgrading to v 12.10, amplify codegen is unable to create the API.ts file and returns the following error: No queries found to generate types for, you may need to run 'codegen statements' first
Expected behavior
API.ts file is generated
Reproduction steps
upgrade to cli v12.10. Run amplify codegen
Project Identifier
No response
Log output

# Put your logs below this line

Additional information
Downgrading to 12.9 resolves the issue
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

Hey @farshid1, thank you for reaching out. Could you run amplify diagnose --send-report and provide us the  project identifieroutput from the terminal.additionally, could you provide us the.graphqlconfig.yml` file present at the root of the project.
transferring to the codegen repository for further assistance.


feat: add logseq-git-diff-since plugin.

Submit a new Plugin to Marketplace
Plugin Github repo URL:
https://github.com/Glooory/logseq-git-diff-since
Github releases checklist

[x] a legal package.json file.
[x] a valid CI workflow build action for Github releases. (Optional for theme plugin only).
[x] a release which includes a release zip pkg from a successful build.
[x] a clear README file, ideally with an image or gif showcase. (For more friendly to users, it is recommended to have English version description).
[x] a license in the LICENSE file.

Very interesting idea!


Blog post for OI Seminar Speech (May 15th)

Title: New Open Innovation (OI) Speech by Dr. Sabine Brunswicker at Berkeley OI Seminar
Hashtags: #Research #OpenInnovation
Graphics:  OI Seminar Talk.png
Content:
[Subheading] Spring 23 Berkeley Open Innovation Seminar on May 15th, 2023.
The growing field of Open Innovation (OI) is welcoming more academic scholars today to present various research aspects in OI to a large audience of seasoned scholars, practitioners, aspiring young graduates and more at the 16th Open Innovation Seminar at Berkeley.
With the topic of “Teaching Engineers about Open Innovation” , Dr. Brunswicker from Purdue University will be talking about experiential learning, cyclical framework and theoretical foundations in teaching engineering students and how they gain experience through solving real open innovation projects. Her presentation will start at 1 PM (EST) today along with other speeches from Prof Yao Sun (New Jersey Institute of Technology), Prof. Ann Majchrzak (University of Southern California), and Prof. Arvind Malhotra (University of North Carolina at Chapel Hill).

Title: New Open Innovation (OI) Speech by Dr. Sabine Brunswicker at Berkeley OI Seminar
Hashtags: #Research #OpenInnovation
Graphics:  OI Seminar Talk 2.png
Content for Blog:
[Subheading] Spring 23 Berkeley Open Innovation Seminar on May 15th, 2023.
The growing field of Open Innovation (OI) is welcoming more academic scholars today to present various research aspects in OI to a large audience of seasoned scholars, practitioners, aspiring young graduates and more at the 16th Open Innovation Seminar at Berkeley.
With the topic of “Teaching Engineers about Open Innovation” , Dr. Brunswicker from Purdue University will be talking about experiential learning, cyclical framework and theoretical foundations in teaching engineering students and how they gain experience through solving real open innovation projects. Excitingly, the presentation will start at 1 PM (EST) today along with other speeches from Prof Yao Sun (New Jersey Institute of Technology), Prof. Ann Majchrzak (University of Southern California), and Prof. Arvind Malhotra (University of North Carolina at Chapel Hill).
Regarding Dr. Brunswicker’s speech, we compose a list of case studies that utilizes open innovation concepts to solve complex, real-life problems in different fields. A complete, informative story for each case study is attached in separate sections, feel free to view it here: https://rcodi.org/oxfordhbcases. Let us know your opinions on these open innovation concepts!

@jasminebui1225 can you add the twitter handles here that yuo want to use in twitter post (to tag). You need to reference the citation of the book chapter!

cool social media post! just saw it.


Add searching in preferences

From nitaym on 2014-04-04T05:43:17Z
There are many preferences in the preferences window and I never seem to find what I'm looking for. A search for a specific setting will ease up the process
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=1780

From ccordoba12 on 2014-04-16T09:42:39Z
Cool idea, really hard to implement.
Status: HelpNeeded

From ccordoba12 on 2014-04-16T09:42:56Z
Labels: Cat-Settings


Warn users that engine='auto' will change in future

Adds a warning to read_parquet and to_parquet that the meaning of
engine='auto' will change in the future (switching to using pyarrow
if it is installed, and falling back to fastparquet). This warning is
only raised for users that have both libraries installed and have
engine='auto'. Users without both libraries installed or that already
specify an engine will not see this warning.
First part of #8900.

Should we perhaps come to a consensus in #8900 , or at least take a vote before considering this?

Definitely. I'm just putting this up to have something concrete to talk about.

OK, got it. You may want to mark this WIP, although the comments we just added should make that clear for any other reviewer than happens to glance here.

I've updated this PR to fix the failing test - I believe (based on the conversation in #8900) that this should be good to merge after tests pass. cc @bryanwweber for a quick review to make sure I didn't bungle anything.

Hey, I want to raise a slight concern here (sorry for not doing it earlier).  For a user who doesn't care at all about this, they're going to get a confusing warning.  Are we ok with that?
My guess is that we'll cause more stress/confusion by raising a warning here than by just silently changing things.

If fastparquet and pyarrow were known to produce different results or something I would totally get wanting to emit something, but they should be drop-in replacements.  This seems like being a little bit too loud to me.
Happy to be overruled though.

My guess is that we'll cause more stress/confusion by raising a warning here than by just silently changing things.

This warning is only raised for users that have both pyarrow and fastparquet installed and no engine is explicitly specified, which is likely to be rare. There are functional differences between the two apis, so I think warning in this case is worthwhile.

that have both pyarrow and fastparquet installed and no engine is explicitly specified

I think the standard Anaconda installer has both

I think the standard Anaconda installer has both

Hmmm, ok. What do you think of the current warning logic Martin? I like it for its simplicity, but if this is likely to be noisy for naive users then we can revisit it. A few other options:

What we currently have
Warn only for writing (where the differences matter a bit more)
Warn only if the user passes in backend-specific kwargs
Don't warn at all

If no preference, then I'm likely to keep what we currently have.

I think the standard Anaconda installer has both

Yeah.  This is my concern.  The naive user probably isn't well served by this choice.

Warn only if the user passes in backend-specific kwargs

This would be great, but is it tractable? Otherwise, I could go either way between no warning and warn-on-write.

More broadly, as we change defaults here will we provide warnings?  My gut reaction is that we shouldn't.  There are probably several changes of this magnitude or greater.

This would be great, but is it tractable?

We'd just check if there are any extra kwargs passed to read_parquet/to_parquet (which are always forwarded to the backend), and if so warn then. I'll switch to this.

More broadly, as we change defaults here will we provide warnings? My gut reaction is that we shouldn't.

I think that in cases where we can warn and not have it be obnoxious we should. I thought this was a case where the warning would be unlikely, but it turns out it isn't. Generally I like to let users whose code may break know beforehand that something big is changing.

Updated. We now only warn if all of the following are true:

engine='auto'
Both pyarrow and fastparquet are installed
Backend specific options are provided

+1
On Wed, Apr 13, 2022 at 3:33 PM Jim Crist-Harif @.***>
wrote:

Updated. We now only warn if all of the following are true:

engine='auto'
Both pyarrow and fastparquet are installed
Backend specific options are provided

—
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/pull/8907#issuecomment-1098461503, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AACKZTAJC3E5AUQJJMQMGRLVE4VQ3ANCNFSM5S5VJLQA
.
You are receiving this because you commented.Message ID:
@.***>


⚠ Bofrost.lu has degraded performance

In 2de3f09, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1159 ms

Resolved: Bofrost.lu performance has improved in 14f7b7d.


Integrate with Rails.vim projections

Test.vim already has projectionist.vim integration, so if you're on an application file, and an alternate file is defined and it's a test file, running :TestFile will automatically run the test command on the alternate file.
This, however, doesn't work with projections defined by rails.vim, because it seems that rails.vim doesn't actually extend the projectionist registry with its Rails-specific projections (it doesn't call projectionist#append anywhere).

Thanks a lot. Works perfectly now !


photo_manager插件升级后偶发报错PlatformException(PHPhotosErrorDomain (-1), 未能完成操作。（PHPhotosErrorDomain错误-1。）, No failure reason provided, null)

Version
3.6.2
Platforms
dart, iOS
Device Model
iPhone 16
flutter info
[!] Flutter (Channel [user-branch], 3.24.3, on macOS 15.1 24B83 darwin-arm64, locale zh-Hans-CN)
    ! Flutter version 3.24.3 on channel [user-branch] at /Users/misszhang/development/flutter
      Currently on an unknown channel. Run `flutter channel` to switch to an official channel.
      If that doesn't fix the issue, reinstall Flutter by following instructions at https://flutter.dev/setup.
    ! Upstream repository unknown source is not a standard remote.
      Set environment variable "FLUTTER_GIT_URL" to unknown source to dismiss this error.
    • Framework revision 2663184aa7 (3 months ago), 2024-09-11 16:27:48 -0500
    • Engine revision 36335019a8
    • Dart version 3.5.3
    • DevTools version 2.37.3
    • Pub download mirror https://pub.flutter-io.cn
    • Flutter download mirror https://storage.flutter-io.cn
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
    • Android SDK at /Users/misszhang/Library/Android/sdk
    • Platform android-34, build-tools 32.0.0
    • ANDROID_HOME = /Users/misszhang/Library/Android/sdk
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 16.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 16B40
    • CocoaPods version 1.16.2

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

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

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

[✓] Connected device (7 available)
    • 00008110-000974A43A3A801E • ios            • iOS 18.1.1 22B91
    • iPhone16 (mobile)               • 00008140-001544260A8B001C • ios            • iOS 18.1.1 22B91
    • iPhone (mobile)                 • 00008130-0012498E10FA8D3A • ios            • iOS 18.2 22C150
    • iPhone15 (mobile)               • 00008130-00114C983620001C • ios            • iOS 18.1 22B82
    • macOS (desktop)                 • macos                     • darwin-arm64   • macOS 15.1 24B83 darwin-arm64
    • Mac Designed for iPad (desktop) • mac-designed-for-ipad     • darwin         • macOS 15.1 24B83 darwin-arm64
    • Chrome (web)                    • chrome                    • web-javascript • Google Chrome 131.0.6778.140

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

How to reproduce?
I found the bug when I call the Photomanager.AssetEntity.loadFile
Logs
PlatformException(PHPhotosErrorDomain (-1), 未能完成操作。（PHPhotosErrorDomain错误-1。）, No failure reason provided, null)
堆栈信息
#0      StandardMethodCodec.decodeEnvelope (package:flutter/src/services/message_codecs.dart:648)
#1      MethodChannel._invokeMethod (package:flutter/src/services/platform_channel.dart:334)
<asynchronous suspension>
#2      AssetEntity._getFile (package:photo_manager/src/types/entity.dart:765)

Example code (optional)
var data = AssetEntity();
var file = await data.loadFile(isOrigin: true);

Contact
No response

AI Summary: 用户在升级 photo_manager 插件到 3.6.2 版本后，在 iOS 平台上调用 AssetEntity.loadFile 方法时，偶发 PlatformException(PHPhotosErrorDomain (-1)) 错误。

@AlexV525
Sorry my bad. Thank you very much for your Explanation.

@zhangyanzhu88 @KlipX Can this issue be locally reproduced?

@AlexV525 The local version cannot be reproduced, and the online version appears sporadically, about once in 10 times

We must find a way to reproduce the issue or an observable pattern before we start investigations.

@AlexV525
Is there a way to get around this error ?
I tried useing a Try{}catch() but i still experience the same crashes.
  try {
        return e.asset.originFile;
      } catch (error) {
        try {
          return e.asset.file;
        } catch (e, st) {
          await FirebaseCrashlytics.instance
              .recordError(e, st, reason: "asset Loading File Error");
          return null;
        }
      }