[CRASH] Random crash on decrRefCount after command completeion

Crash report
Paste the complete crash log between the quotes below. Please include a few lines from the log preceding the crash report to provide some context.
11058:11057:C 11 Apr 2023 12:05:06.088 # oO0OoO0OoO0Oo KeyDB is starting oO0OoO0OoO0Oo
11058:11057:C 11 Apr 2023 12:05:06.089 # KeyDB version=6.3.2, bits=64, commit=00000000, modified=0, pid=11058, just started
11058:11057:C 11 Apr 2023 12:05:06.089 # Configuration loaded
11058:11057:M 11 Apr 2023 12:05:06.091 * monotonic clock: POSIX clock_gettime
11058:11057:M 11 Apr 2023 12:05:06.092 * Node configuration loaded, I'm 064af590724248d2babf8e5d9bfd7fe4d7a9bc5d
11058:11057:M 11 Apr 2023 12:05:06.092 * Expanding min-clients-per-thread to 200 due to cluster
11058:11057:M 11 Apr 2023 12:05:06.092 * Running mode=cluster, port=6000.
11058:11057:M 11 Apr 2023 12:05:06.092 # Server initialized
11058:11067:M 11 Apr 2023 12:05:06.093 * Thread 0 alive.
11058:11069:M 11 Apr 2023 12:05:06.093 * Thread 1 alive.
11058:11067:S 11 Apr 2023 12:05:06.155 * Before turning into a replica, using my own master parameters to synthesize a cached master: I may be able to synchronize with the new master with just a partial transfer.
11058:11067:S 11 Apr 2023 12:05:06.155 * Connecting to MASTER <IP_ADDRESS>:6008
11058:11067:S 11 Apr 2023 12:05:06.155 * MASTER <-> REPLICA sync started
11058:11067:S 11 Apr 2023 12:05:06.155 # Cluster state changed: ok
11058:11067:S 11 Apr 2023 12:05:06.158 * Non blocking connect for SYNC fired the event.
11058:11067:S 11 Apr 2023 12:05:06.161 * Master does not support REPLPING, sending PING instead...
11058:11067:S 11 Apr 2023 12:05:06.161 * Non blocking connect for SYNC fired the event.
11058:11067:S 11 Apr 2023 12:05:06.161 * Master replied to PING, replication can continue...
11058:11067:S 11 Apr 2023 12:05:06.161 # non-fatal: Master doesn't understand REPLCONF uuid
11058:11067:S 11 Apr 2023 12:05:06.161 * Trying a partial resynchronization (request eb5c82c4e5a04371643b1123fa5fe809f83a5d81:1).
11058:11067:S 11 Apr 2023 12:05:08.633 * Full resync from master: 57a7934ae9708681637a8ae20f3bda3b853ba41c:51985667563394
11058:11067:S 11 Apr 2023 12:05:08.633 * Discarding previously cached master state.
11058:11067:S 11 Apr 2023 12:05:09.627 * MASTER <-> REPLICA sync: receiving streamed RDB from master with EOF to disk
11058:11067:S 11 Apr 2023 12:07:13.830 * MASTER <-> REPLICA sync: Flushing old data
11058:11067:S 11 Apr 2023 12:07:13.830 * MASTER <-> REPLICA sync: Loading DB in memory
11058:11067:S 11 Apr 2023 12:07:13.845 * Loading RDB produced by version 5.0.8
11058:11067:S 11 Apr 2023 12:07:13.845 * RDB age 124 seconds
11058:11067:S 11 Apr 2023 12:07:13.845 * RDB memory usage when created 27238.78 Mb
11058:11067:S 11 Apr 2023 12:20:00.944 # Done loading RDB, keys loaded: 0, keys expired: 0.
11058:11067:S 11 Apr 2023 12:20:00.944 * MASTER <-> REPLICA sync: Finished with success
11058:11069:S 11 Apr 2023 12:54:12.862 # Manual failover user request accepted.
11058:11067:S 11 Apr 2023 12:54:12.895 # Received replication offset for paused master manual failover:<PHONE_NUMBER>4901
11058:11067:S 11 Apr 2023 12:54:12.895 # All master replication stream processed, manual failover can start.
11058:11067:S 11 Apr 2023 12:54:12.895 # Start of election delayed for 0 milliseconds (rank #0, offset<PHONE_NUMBER>4901).
11058:11069:S 11 Apr 2023 12:54:12.895 # Starting a failover election for epoch 360.
11058:11067:S 11 Apr 2023 12:54:12.902 # Currently unable to failover: Waiting for votes, but majority still not reached.
11058:11067:S 11 Apr 2023 12:54:12.903 # Failover election won: I'm the new master.
11058:11067:S 11 Apr 2023 12:54:12.903 # configEpoch set to 360 after successful failover
11058:11067:S 11 Apr 2023 12:54:12.903 # Connection with master lost.
11058:11067:S 11 Apr 2023 12:54:12.903 * Caching the disconnected master state.
11058:11067:S 11 Apr 2023 12:54:12.903 * Discarding previously cached master state.
11058:11067:S 11 Apr 2023 12:54:12.903 # Setting secondary replication ID to 57a7934ae9708681637a8ae20f3bda3b853ba41c, valid up to offset:<PHONE_NUMBER>4902. New replication ID is e1896072291a4348276a70c0df81229318eb0890
11058:11069:M 11 Apr 2023 12:54:12.909 * Replica <IP_ADDRESS>:40012 asks for synchronization
11058:11069:M 11 Apr 2023 12:54:12.909 * Partial resynchronization request from <IP_ADDRESS>:40012 accepted. Sending 20346 bytes of backlog starting from offset<PHONE_NUMBER>4902.
11058:11067:M 11 Apr 2023 12:54:12.977 * Replica <IP_ADDRESS>:6001 asks for synchronization
11058:11067:M 11 Apr 2023 12:54:12.977 * Partial resynchronization request from <IP_ADDRESS>:6001 accepted. Sending 447784 bytes of backlog starting from offset<PHONE_NUMBER>4902.
11058:11067:M 11 Apr 2023 12:54:13.063 * Replica <IP_ADDRESS>:6005 asks for synchronization
11058:11067:M 11 Apr 2023 12:54:13.063 * Partial resynchronization request from <IP_ADDRESS>:6005 accepted. Sending 1065606 bytes of backlog starting from offset<PHONE_NUMBER>4902.
11058:11069:M 11 Apr 2023 12:54:13.109 * Replica <IP_ADDRESS>:40010 asks for synchronization
11058:11069:M 11 Apr 2023 12:54:13.109 * Partial resynchronization request from <IP_ADDRESS>:40010 accepted. Sending 1376863 bytes of backlog starting from offset<PHONE_NUMBER>4902.
11058:11067:M 11 Apr 2023 12:54:13.114 * Replica <IP_ADDRESS>:6006 asks for synchronization
11058:11067:M 11 Apr 2023 12:54:13.114 * Partial resynchronization request from <IP_ADDRESS>:6006 accepted. Sending 1420738 bytes of backlog starting from offset<PHONE_NUMBER>4902.
11058:11067:M 11 Apr 2023 12:54:13.152 * Replica <IP_ADDRESS>:6008 asks for synchronization
11058:11067:M 11 Apr 2023 12:54:13.152 * Partial resynchronization request from <IP_ADDRESS>:6008 accepted. Sending 1671700 bytes of backlog starting from offset<PHONE_NUMBER>4902.
11058:11069:M 11 Apr 2023 12:54:13.749 * Replica <IP_ADDRESS>:6007 asks for synchronization
11058:11069:M 11 Apr 2023 12:54:13.749 * Partial resynchronization request from <IP_ADDRESS>:6007 accepted. Sending 5411634 bytes of backlog starting from offset<PHONE_NUMBER>4902.
11058:11069:M 11 Apr 2023 12:54:13.801 * Replica <IP_ADDRESS>:6009 asks for synchronization
11058:11069:M 11 Apr 2023 12:54:13.801 * Partial resynchronization request from <IP_ADDRESS>:6009 accepted. Sending 5735198 bytes of backlog starting from offset<PHONE_NUMBER>4902.
11058:11067:M 11 Apr 2023 15:43:30.059 * Marking node a6166d1d0562457b72544bacde53d1dfe3df24eb as failing (quorum reached).

=== KEYDB BUG REPORT START: Cut & paste starting from here ===
11058:11069:M 12 Apr 2023 07:07:28.819 # ------------------------------------------------
11058:11069:M 12 Apr 2023 07:07:28.819 # !!! Software Failure. Press left mouse button to continue
11058:11069:M 12 Apr 2023 07:07:28.820 # Guru Meditation: decrRefCount against refcount <= 0 #object.cpp:427

------ STACK TRACE ------

Backtrace:
/usr/local/bin/redis-server *:6000 [cluster](decrRefCount(robj_roptr)+0x195) [0x564315450255]
/usr/local/bin/redis-server *:6000 [cluster](+0x15ab48) [0x564315361b48]
/usr/local/bin/redis-server *:6000 [cluster](resetClient(client*)+0x24) [0x564315361ba4]
/usr/local/bin/redis-server *:6000 [cluster](commandProcessed(client*, int)+0x22) [0x5643153630b2]
/usr/local/bin/redis-server *:6000 [cluster](processCommandAndResetClient(client*, int)+0x11a) [0x56431536334a]
/usr/local/bin/redis-server *:6000 [cluster](processInputBuffer(client*, bool, int)+0x174) [0x564315369414]
/usr/local/bin/redis-server *:6000 [cluster](processClients()+0xd9) [0x5643153695a9]
/usr/local/bin/redis-server *:6000 [cluster](+0x19aec2) [0x5643153a1ec2]
/usr/local/bin/redis-server *:6000 [cluster](beforeSleep(aeEventLoop*)+0x160) [0x56431534e6d0]
/usr/local/bin/redis-server *:6000 [cluster](aeProcessEvents+0x10b) [0x564315348eab]
/usr/local/bin/redis-server *:6000 [cluster](aeMain+0x47) [0x56431534f887]
/usr/local/bin/redis-server *:6000 [cluster](workerThreadMain(void*)+0x73) [0x56431540ff23]
/lib/x86_64-linux-gnu/libpthread.so.0(+0x76db) [0x7f820c9936db]
/lib/x86_64-linux-gnu/libc.so.6(clone+0x3f) [0x7f820c6bc88f]

------ INFO OUTPUT ------
# Server
redis_version:6.3.2
redis_git_sha1:00000000
redis_git_dirty:0
redis_build_id:d352759e04d75ec2
redis_mode:cluster
os:Linux 4.15.0-96-generic x86_64
arch_bits:64
multiplexing_api:epoll
atomicvar_api:atomic-builtin
gcc_version:7.3.0
process_id:11058
process_supervised:no
run_id:c51e6a438156c5d9b7c1c5a8c736789bc6d6413f
tcp_port:6000
server_time_usec:1681283248821382
uptime_in_seconds:68542
uptime_in_days:0
hz:250
configured_hz:250
lru_clock:3561648
executable:/usr/bin/keydb-server
config_file:/etc/keydb/keydb-6000.conf

# Clients
connected_clients:21432
cluster_connections:310
maxclients:40000
client_recent_max_input_buffer:262184
client_recent_max_output_buffer:0
blocked_clients:0
tracking_clients:0
clients_in_timeout_table:0
current_client_thread:1
thread_0_clients:10632
thread_1_clients:10808

# Memory
used_memory:29110554048
used_memory_human:27.11G
used_memory_rss:32960708608
used_memory_rss_human:30.70G
used_memory_peak:30144604896
used_memory_peak_human:28.07G
used_memory_peak_perc:96.57%
used_memory_overhead:5528078578
used_memory_startup:8060248
used_memory_dataset:23582475470
used_memory_dataset_perc:81.03%
allocator_allocated:29121224720
allocator_active:32949698560
allocator_resident:33602326528
total_system_memory:404357001216
total_system_memory_human:376.59G
used_memory_lua:105472
used_memory_lua_human:103.00K
used_memory_scripts:8040
used_memory_scripts_human:7.85K
number_of_cached_scripts:25
maxmemory:35000000000
maxmemory_human:32.60G
maxmemory_policy:noeviction
allocator_frag_ratio:1.13
allocator_frag_bytes:3828473840
allocator_rss_ratio:1.02
allocator_rss_bytes:652627968
rss_overhead_ratio:0.98
rss_overhead_bytes:-641617920
mem_fragmentation_ratio:1.13
mem_fragmentation_bytes:3840271256
mem_not_counted_for_evict:536870912
mem_replication_backlog:536870912
mem_clients_slaves:208360
mem_clients_normal:440022218
mem_aof_buffer:0
mem_allocator:jemalloc-5.2.1
active_defrag_running:13
lazyfree_pending_objects:0
lazyfreed_objects:6
storage_provider:none

# Persistence
loading:0
current_cow_size:0
current_cow_size_age:0
current_fork_perc:0.00
current_save_keys_processed:0
current_save_keys_total:0
rdb_changes_since_last_save:358831116
rdb_bgsave_in_progress:0
rdb_last_save_time:1681214706
rdb_last_bgsave_status:ok
rdb_last_bgsave_time_sec:-1
rdb_current_bgsave_time_sec:-1
rdb_last_cow_size:0
aof_enabled:0
aof_rewrite_in_progress:0
aof_rewrite_scheduled:0
aof_last_rewrite_time_sec:-1
aof_current_rewrite_time_sec:-1
aof_last_bgrewrite_status:ok
aof_last_write_status:ok
aof_last_cow_size:0
module_fork_in_progress:0
module_fork_last_cow_size:0

# Stats
total_connections_received:173488
total_commands_processed:270461139
instantaneous_ops_per_sec:4485
total_net_input_bytes:57781899796
total_net_output_bytes:310014163991
instantaneous_input_kbps:701.29
instantaneous_output_kbps:4458.64
rejected_connections:0
sync_full:0
sync_partial_ok:8
sync_partial_err:0
expired_keys:59078451
expired_stale_perc:0.00
expired_time_cap_reached_count:42436
expire_cycle_cpu_milliseconds:0
evicted_keys:0
keyspace_hits:35669403
keyspace_misses:32188779
pubsub_channels:0
pubsub_patterns:0
latest_fork_usec:0
total_forks:0
migrate_cached_sockets:0
slave_expires_tracked_keys:0
active_defrag_hits:127325563
active_defrag_misses:18115209786
active_defrag_key_hits:55526500
active_defrag_key_misses:2684676198
tracking_total_keys:0
tracking_total_items:0
tracking_total_prefixes:0
unexpected_error_replies:0
total_error_replies:51
dump_payload_sanitizations:0
total_reads_processed:210449009
total_writes_processed:1295043744
instantaneous_lock_contention:2
avg_lock_contention:1.234375
storage_provider_read_hits:0
storage_provider_read_misses:0

# Replication
role:master
connected_slaves:8
slave0:ip=<IP_ADDRESS>,port=40012,state=online,offset=52019016970889,lag=0
slave1:ip=<IP_ADDRESS>,port=6001,state=online,offset=52019016656288,lag=1
slave2:ip=<IP_ADDRESS>,port=6005,state=online,offset=52019016656288,lag=1
slave3:ip=<IP_ADDRESS>,port=40010,state=online,offset=52019016656641,lag=1
slave4:ip=<IP_ADDRESS>,port=6006,state=online,offset=52019016891114,lag=0
slave5:ip=<IP_ADDRESS>,port=6008,state=online,offset=52019016691072,lag=0
slave6:ip=<IP_ADDRESS>,port=6007,state=online,offset=52019016773689,lag=0
slave7:ip=<IP_ADDRESS>,port=6009,state=online,offset=52019016891818,lag=0
master_failover_state:no-failover
master_replid:e1896072291a4348276a70c0df81229318eb0890
master_replid2:57a7934ae9708681637a8ae20f3bda3b853ba41c
master_repl_offset:52019017044857
second_repl_offset:51986981234902
repl_backlog_active:1
repl_backlog_size:536870912
repl_backlog_first_byte_offset:52018480173946
repl_backlog_histlen:536870912

# CPU
used_cpu_sys:8178.709064
used_cpu_user:27959.011523
used_cpu_sys_children:0.000000
used_cpu_user_children:0.000000
server_threads:2
long_lock_waits:8739
used_cpu_sys_main_thread:3481.718818
used_cpu_user_main_thread:12334.450114

# Modules

# Commandstats
cmdstat_zcount:calls=2360,usec=9768,usec_per_call=4.14,rejected_calls=0,failed_calls=0
cmdstat_zcard:calls=645,usec=1464,usec_per_call=2.27,rejected_calls=0,failed_calls=0
cmdstat_readonly:calls=1462948,usec=478263,usec_per_call=0.33,rejected_calls=0,failed_calls=0
cmdstat_zscore:calls=2724,usec=16568,usec_per_call=6.08,rejected_calls=0,failed_calls=0
cmdstat_info:calls=11121,usec=858680,usec_per_call=77.21,rejected_calls=0,failed_calls=0
cmdstat_hincrby:calls=7,usec=36,usec_per_call=5.14,rejected_calls=0,failed_calls=0
cmdstat_ping:calls=278,usec=79,usec_per_call=0.28,rejected_calls=0,failed_calls=0
cmdstat_llen:calls=42302,usec=115027,usec_per_call=2.72,rejected_calls=0,failed_calls=0
cmdstat_replconf:calls=511679,usec=206925,usec_per_call=0.40,rejected_calls=0,failed_calls=0
cmdstat_ttl:calls=993757,usec=1788455,usec_per_call=1.80,rejected_calls=0,failed_calls=0
cmdstat_exec:calls=984198,usec=8699809,usec_per_call=8.84,rejected_calls=0,failed_calls=0
cmdstat_lrange:calls=1249742,usec=7255512,usec_per_call=5.81,rejected_calls=0,failed_calls=0
cmdstat_pttl:calls=305004,usec=180014,usec_per_call=0.59,rejected_calls=0,failed_calls=0
cmdstat_hget:calls=160508,usec=611540,usec_per_call=3.81,rejected_calls=0,failed_calls=0
cmdstat_restore:calls=258280,usec=8149194,usec_per_call=31.55,rejected_calls=0,failed_calls=0
cmdstat_decrby:calls=29417,usec=107312,usec_per_call=3.65,rejected_calls=0,failed_calls=0
cmdstat_zremrangebyscore:calls=372696,usec=2451875,usec_per_call=6.58,rejected_calls=0,failed_calls=0
cmdstat_psync:calls=8,usec=404,usec_per_call=50.50,rejected_calls=0,failed_calls=0
cmdstat_hset:calls=25512482,usec=74827664,usec_per_call=2.93,rejected_calls=0,failed_calls=0
cmdstat_decr:calls=930,usec=3283,usec_per_call=3.53,rejected_calls=0,failed_calls=0
cmdstat_zrange:calls=10204,usec=54196,usec_per_call=5.31,rejected_calls=0,failed_calls=0
cmdstat_zremrangebyrank:calls=4489,usec=816241,usec_per_call=181.83,rejected_calls=0,failed_calls=0
cmdstat_expire:calls=48811850,usec=72422029,usec_per_call=1.48,rejected_calls=0,failed_calls=0
cmdstat_get:calls=40120834,usec=52171580,usec_per_call=1.30,rejected_calls=0,failed_calls=0
cmdstat_zrank:calls=9,usec=43,usec_per_call=4.78,rejected_calls=0,failed_calls=0
cmdstat_zrangebyscore:calls=10632,usec=161271,usec_per_call=15.17,rejected_calls=0,failed_calls=0
cmdstat_set:calls=41226545,usec=295345291,usec_per_call=7.16,rejected_calls=1,failed_calls=0
cmdstat_replping:calls=1,usec=1,usec_per_call=1.00,rejected_calls=0,failed_calls=0
cmdstat_select:calls=1,usec=1,usec_per_call=1.00,rejected_calls=0,failed_calls=0
cmdstat_command:calls=1,usec=541,usec_per_call=541.00,rejected_calls=0,failed_calls=0
cmdstat_dump:calls=305004,usec=7600341,usec_per_call=24.92,rejected_calls=0,failed_calls=0
cmdstat_sismember:calls=166792,usec=583965,usec_per_call=3.50,rejected_calls=0,failed_calls=0
cmdstat_zadd:calls=1010139,usec=18087074,usec_per_call=17.91,rejected_calls=0,failed_calls=0
cmdstat_zrevrange:calls=81,usec=1447,usec_per_call=17.86,rejected_calls=0,failed_calls=0
cmdstat_scan:calls=512,usec=105565403,usec_per_call=206182.42,rejected_calls=0,failed_calls=0
cmdstat_rename:calls=360592,usec=7957411,usec_per_call=22.07,rejected_calls=0,failed_calls=0
cmdstat_incr:calls=12871541,usec=69295081,usec_per_call=5.38,rejected_calls=0,failed_calls=0
cmdstat_slowlog:calls=9136,usec=9074,usec_per_call=0.99,rejected_calls=0,failed_calls=0
cmdstat_hmget:calls=22873262,usec=258242585,usec_per_call=11.29,rejected_calls=0,failed_calls=0
cmdstat_exists:calls=1432550,usec=3524752,usec_per_call=2.46,rejected_calls=0,failed_calls=0
cmdstat_config:calls=4570,usec=279949,usec_per_call=61.26,rejected_calls=0,failed_calls=0
cmdstat_zrem:calls=7,usec=51,usec_per_call=7.29,rejected_calls=0,failed_calls=0
cmdstat_hdel:calls=928458,usec=2145522,usec_per_call=2.31,rejected_calls=0,failed_calls=0
cmdstat_sadd:calls=59338,usec=281033,usec_per_call=4.74,rejected_calls=0,failed_calls=0
cmdstat_hgetall:calls=181760,usec=1809973,usec_per_call=9.96,rejected_calls=0,failed_calls=0
cmdstat_latency:calls=4568,usec=10172,usec_per_call=2.23,rejected_calls=0,failed_calls=0
cmdstat_unlink:calls=12,usec=76,usec_per_call=6.33,rejected_calls=0,failed_calls=0
cmdstat_incrby:calls=15811554,usec=31235214,usec_per_call=1.98,rejected_calls=0,failed_calls=0
cmdstat_hmset:calls=10696981,usec=125073292,usec_per_call=11.69,rejected_calls=0,failed_calls=0
cmdstat_eval:calls=26059030,usec=576469871,usec_per_call=22.12,rejected_calls=0,failed_calls=0
cmdstat_srandmember:calls=12,usec=74,usec_per_call=6.17,rejected_calls=0,failed_calls=0
cmdstat_del:calls=9472829,usec=40132610,usec_per_call=4.24,rejected_calls=0,failed_calls=0
cmdstat_multi:calls=984198,usec=96909,usec_per_call=0.10,rejected_calls=0,failed_calls=0
cmdstat_rpush:calls=1242706,usec=15383440,usec_per_call=12.38,rejected_calls=0,failed_calls=0
cmdstat_setex:calls=3647316,usec=29379207,usec_per_call=8.06,rejected_calls=0,failed_calls=0
cmdstat_cluster:calls=252539,usec=35966585,usec_per_call=142.42,rejected_calls=49,failed_calls=0

# Errorstats
errorstat_ERR:count=1
errorstat_LOADING:count=49
errorstat_MOVED:count=1

# Cluster
cluster_enabled:1

# Keyspace
db0:keys=69118600,expires=68497453,avg_ttl=1993500801,cached_keys=69118600

# KeyDB
mvcc_depth:1

------ CLIENT LIST OUTPUT ------

Aditional information
Ubuntu 18.04.4, Linux 4.15.0-96-generic #97-Ubuntu SMP Wed Apr 1 03:25:46 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux.
The server has crashed and gracefully restarted.
Probably somehow related to problem depicted in https://github.com/Snapchat/KeyDB/pull/250

hi @r00t3g can you provide more details about your configuration?

Hi @msotheeswaran .
We are running KeyDB as a replacement to one of our production Redis Cluster shards. Basically we took the same config file, added some KeyDB-related settings (server-threads, etc) and basically switched all the nodes within one cluster shard to KeyDB.
The reported behavior occurred only on the primary (master) node, did not occur on secondaries (slaves).
The rest of the cluster remained on Redis 5.x.
After downgrading to 6.2 the problem stopped occurring.
Probably, exactly https://github.com/Snapchat/KeyDB/commit/27cb5dba75af168d02c66bfdce68f4e04c6e2199 for v. 6.3.2 might have caused the problem.

Oh, and please don't be confused with /usr/local/bin/redis-server binary path - it's KeyDB, we just had to rename the binary as our monitoring currently uses Prometheus Named Process Group exporter which monitors redis-server, and, since it is a test-drive, we have just kept the process name.


Fix removal of internal metadata information in 350 shim

Fixes #8844. This PR adds fix for removing internal metadata information from the schema while using CTAS. This feature was added in Spark 3.5.0.
Changes:
Created a new class SchemaMetadataShims. It includes a new method getCleanedSchema() which calls spark's removeInternalMetadata().

This function removes the metadata properties having keys that are meant for internal usage (eg, FILE_SOURCE_METADATA_COL_ATTR_KEY)

schema = SchemaMetadataShims.getCleanedSchema(result.schema)

Testing

Added unit test to use CTAS query to create a new table as select from another DF.
Added a metadata property having internal keys.
Tested if the metadata was finally removed (set as NULL).
Tested in Spark 3.4.0, metadata does not get removed.

build


Restructure for to use new base plugin

Please coordinate with @lukemelia or @achambers for merge timing

@LevelbossMike you are good to merge this anytime. This plugin with no longer work with the 0.5.0-dev branch of ember-cli until this is merged

Thx!


How can I inject dapr.io/config: "tracing" using tye?

Currently, I use tye to deploy my application to AKS. Everything is fine now. But with Dapr in-place we can do the tracing out of the box. See at https://github.com/RicardoNiepel/dapr-docs/blob/master/best-practices/troubleshooting/tracing.md
So I have a quick question that how can we inject dapr.io/config: "tracing" into the deployment to enable the tracing with Open Telemetry using tye?

Are you talking about adding annotations for deployment using tye?

Yes, it is. Just adding more like below
annotations:
	...
    dapr.io/config: "tracing"
    dapr.io/log-level: "debug"

cc @rynowak

@thangchung - thanks for blogging us, I enjoyed the read, and I'm glad you find our project interesting.

We haven't done anything to enable this yet.
Some ideas:

We will eventually add some features to customize the k8s manifests annotations and labels, but this doesn't seem like a great case for that because....
We can add per-extension settings that don't necessary map 1:1 to k8s, or have a different meaning in local development

Example:
extensions:
- name: dapr
  config: myconfig
  log-level: debug

This would map to the annotations you defined above when deployed, and could map to config myconfig -log-level=debug when running locally.
This seems like it should just work would all you to have access to all of dapr's config settings in local dev.
If you're interested in sending a PR - all the changes would go into DaprExtension. Otherwise, I can add this sometime in the next few days.

Keeping open because I need to fix something with the local dev part of this. Deployment support is merged. Thanks @thangchung 👍


[Mixed Reality] Update the autorest.md file to use README.md as an input instead of swagger json file

We should be using README file from the spec repo instead of json as an input-file in autorest.md because readme.md file has all the configuration. If you want to change the configuration, you can override it in the autorest.md.
For example check require parameter here, we pass readme.md file as an input.

Validated that the autorest.md file now takes input from the spec repo.


accepted extensions list is working only if required is set to true

if you set for example this rule:
object_picture: {required: false, accept:"jpg|png|gif"}
you get the following error:

exception occured when checking element object_picture, check the 'accept' method
Uncaught TypeError: Cannot call method 'call' of undefined
TypeError
arguments: Array[2]
0: "call"
1: undefined

No error if required is set to true!

I'm sorry for the lack of activity on this issue. Instead of leaving it open any longer, I decided to close old issues without trying to address them, to longer give the false impression that it will get addressed eventually.
To the reporter (or anyone else interested in this issue): If you're affected by the same issue, consider opening a new issue, with a testpage that shows the issue. Even better, try to fix the issue yourself, and improve the project by sending a pull request. This may seem daunting at first, but you'll likely learn some useful skills that you can apply elsewhere as well. And you can help keep this project alive. We've documented how to do these things, too. A patch is worth a thousand issues!


feat/make binary and varbinary accept buffer

test: create test allowing buffer use for binary column
test: write test enforcing retro compatibility
feat: return buffer always for binary type
test: update zod tests to work with binary buffer
test: update drizzle-valibot tests to work with binary buffer
test: update drizzle-typebox tests to work with binary buffer
chore: fix mispelling
test: create test ensuring varbinary retro compatbility + buffer type
feat: make possible to receive buffer on varbinary
test: set missing length on varbinary creation
test: set varbinary as buffer on typebox, valibot and zod

this is the continuation of the job started on #425.
/claim #1188

@AndriiSherman, can you review this? the bounty can be collected only once the PR is merged :smile:

So far looks good to be merged. Can you please sign all the commits so I can merge this PR?

@ologbonowiwi @AndriiSherman Any updates on this? Anything we can do to help move it along?

@tslater is pretty much done on my side. I'm waiting for @AndriiSherman.
For precaution, I solved the conflicts. It should be ready to be merged.

any updates?

I'd love to hear from the maintainers on this. The branch has many conflicts, and the beta branch seems to have broken types/tests (the test:types failed; I'm testing without my changes on the beta branch).
I'm considering closing this PR since I don't see any interest from the maintainers in merging it. Therefore, it looks like I'm investing my time for nothing.
@tslater, did you have any news from @AndriiSherman about whether we plan to merge this anytime soon?
I'm becoming less and less motivated to work on this without seeing the maintainers' interest in getting this done. I don't want to be rude, but I've been working on this for free for more than one year already without even knowing if this will ever become part of the actual codebase.


Usability on mobile

I thought whipping up some vertex shaders would be a fun way to pass time on a long bus ride.
However, when opening up one of my WIP shaders, I experienced some major usability issues, to my great sadness. In portrait orientation, where sufficient space would be available over the keyboard for the visualization and the code editor, the latter simply isn't present, nor is the menu, or any text. In landscape mode, everything seems to be working fine, except that after opening up the keyboard, there is so little screen space left, that makes any meaningful editing impossible.

If you have ideas for mobile PRs welcome. Instructions on how to run it on your home PC are in the README if you want to try modding it. Ideally whatever needs to happen on mobile would be CSS only or close to it.

As it turns out, simply using the "Request desktop site" function of Chrome makes it a whole lot better.
A bit ironic, but works for quite a few websites.

That is odd has anything changed that may have influenced this because currently I can use my iPad to edit, but if I do the request desktop site I can't even waste my cursor or anything

If you say "Ideally whatever needs to happen on mobile would be CSS only or close to it." so someone fix the issue. request desktop hasn't designed for interactive websites ,just old static websites. for example chrome zooms in when typing.
Best wishes.


Create user

I am writing a init script and wanted to put default users in the parse db.
   var usr = new Parse.Object("User"); // I tried _User and Parse.User but throwing exception
    usr.set("username", "xyz");
    usr.set("email", "xyz@gmail.com");
    usr.set("password", "xyz");
    usr.set("Active", true);
    usr.save().then(function (u) { console.log(u.toJSON()); }, function (err) { console.log(err); });

What is the correct way to create the user in the cloud code?
changed the above code to below
var usr = new Parse.User();
  usr.set("username", "xyz");
  usr.set("email", "xyz@gmail.com");
  usr.set("password", "xyz");
  usr.set("Active", true);
  usr.signup().then(function (u) { console.log(u.toJSON()); }, function (err) { console.log(err); });

--> Uncaught internal server error, usr,signup is not a function.
So what is the correct way to save the user object?
edit:  option 2 to save user object

I had to fix the before save code and able to set the password successfully. And also signUp instead of all small cases. Thanks


Existing ids are reused for new model instances

I have loaded some fixtures into the app but when I create a new model instance, id=1 is assigned to it, which already exists and thus causes confusion:

My routes use the shorthand method:
// mirage/config.js
(...)
  this.get('/bands');
  this.post('/bands');
  this.get('/bands/:id');

This is for version 0.2.0-beta.5.

I actually just fixed this last night.
Just published beta.6, try that and you should be all set!

It seems like now the new band gets the latest id and not the latest id + 1:

Can I see your fixtures?

Sure, they are here, but let me paste it in here, too:
// bands.js
export default [
  { id: 1, name: 'Pearl Jam', links: { songs: '/api/bands/1/songs' } },
  { id: 2, name: 'Led Zeppelin', links: { songs: '/api/bands/2/songs' } },
  { id: 3, name: 'Foo Fighters', links: { songs: '/api/bands/3/songs' } },
  { id: 4, name: 'Radiohead', links: { songs: '/api/bands/4/songs' } }
];

// songs.js
export default [
  { id: 1, title: 'Daughter', rating: 5, bandId: 1 },
  { id: 2, title: 'Yellow Ledbetter', rating: 5, bandId: 1 },
  { id: 3, title: 'Animal', rating: 4, bandId: 1 },
  { id: 4, title: 'Inside Job', rating: 4, bandId: 1 },
  { id: 5, title: 'Who We Are', rating: 2, bandId: 1 },
  { id: 6, title: 'Black Dog', rating: 4, bandId: 2 },
  { id: 7, title: 'Achilles Last Stand', rating: 5, bandId: 2 },
  { id: 8, title: 'Immigrant Song', rating: 4, bandId: 2 },
  { id: 9, title: 'Pretender', rating: 3, bandId: 3 }
];

You were right! There was another regression.
Fixed and published beta-7, that should work now (I tried it with your app locally). Let me know if you run into any other problems!

Thank you, that has fixed it!


I want my one-to-one user in chat to have a profile image field. can u provide me with any reference material to add to my users

I want my one-to-one user in chat to have a profile image field. can u provide me with any reference material to add to my users

It is easy, your user just should to have the avatar and our server will set this field to the photo field of the private dialog. There is documentation on how to set user avatar https://developers.connectycube.com/flutter/authentication-and-users?id=user-avatar. Also, this functionality is presented in our Chat sample.


Add more informative error message when pl.render_shapes color table columnhas a missmatch of length

I ran into the following:
sc.pp.filter_cells(sdata.tables["table"],min_counts=10)
sdata.pl.render_shapes("cellpose_boundaries",color="total_counts").pl.show()

File /work/rwth1209/enviroments/spatial_data/lib/python3.9/site-packages/spatialdata_plot/pl/utils.py:264, in _get_collection_shape.<locals>.assign_fill_and_outline_to_row(shapes, fill_c, outline_c, row, idx)
    262     row["outline_c"] = outline_c
    263 else:
--> 264     row["fill_c"] = fill_c[idx]
    265     row["outline_c"] = outline_c[idx]

IndexError: index 3424 is out of bounds for axis 0 with size 3424

sdata.shapes["cellpose_boundaries"]  = sdata.shapes["cellpose_boundaries"].loc[sdata.tables["table"].obs.index]
sdata.pl.render_shapes("cellpose_boundaries",color="total_counts").pl.show()
fixed!

I removed some cells from the table but their polygons remained. When plotting this led to a generic error.
Could be a "me" problem but I think it might be good idea to add a more explicit error message that tells the user what is going on. An alternative might be to plot the missmatched polygons with a NaN color

Hey @pakiessling, I'd actually say that in this case it's a "you" problem 😅 You're passing the adata object, so scanpy cannot really know it's dealing with a spatialdata object and spatialdata would have to do regular consistency checks or sth to notice that you modified the contained AnnData. I added a more explicit error message though.

@flying-sheep are there any plans for scanpy to natively work with SpatialData objects? Then this could happen automagically

@LucaMarconato do we have a similar way of filtering in spatialdata ?

Yes, we have the user-friendly match_element_to_table() and match_table_to_element functions, which are subcases of the more powerful API join_spatialelement_table.

Anyway, I think that the bug reported may be related to this other bug: https://github.com/scverse/spatialdata-plot/issues/169 and not fixable by the user itself. I'll write a MRE in the linked PR.

Same bug, continuing the discussion there. Workaround: manually filtering the shapes so that there are no shapes not in the table.


Popups only appear after hot reload

I'm attempting to upgrade from 1.1.1 to 1.3.1 and hitting a rather weird issue.
Each feature should have a linked popup. The popups don't display on hover after an initial load of the page. However if I change the code of the popup and it hot reloads, then the popups display. They again do not display after a full reload of the page.
I'm not exactly sure where to start debugging this one. I tried updating to 1.2.1 instead, but then hit onClick not firing on features which appeared to be fixed in 84bfd72.
https://user-images.githubusercontent.com/1296209/156207591-b7d1aa6c-9d88-4ed6-b6cb-f09763072fbb.mov
Here is the branch of my project that I'm working on that's having this issue: https://github.com/gulfofmaine/Neracoos-1-Buoy-App/pull/1766/

Can you extract just that component as a text and post it here, I am unable to run your project

Each point comes from the PlatformLayer component which has a few options for styling. platform is a GeoJSON Feature, with a few extra properties that aren't used here.
const PlatformLayer = ({ platform, selected, old = false }: PlatformLayerProps) => {
  const navigate = useNavigate()

  let radius: number
  if (selected) {
    radius = window.innerWidth > adjustPxWidth ? 10 : 15
  } else {
    radius = window.innerWidth > adjustPxWidth ? 5 : 10
  }
  const opacity = selected ? "cc" : "7a"

  const url = generatePath(paths.platforms.platform, { id: platform.id })

  const fillColor = old ? "grey" : `#cf5c00${opacity}`
  const strokeColor = old ? "grey" : colors.whatOrange

  return (
    <RLayerVector>
      <RStyle.RStyle>
        <RStyle.RCircle radius={radius}>
          <RStyle.RFill color={fillColor} />
          <RStyle.RStroke color={strokeColor} width={1.5} />
        </RStyle.RCircle>
      </RStyle.RStyle>
      <RFeature
        feature={new GeoJSON({
          dataProjection: "EPSG:4326",
          featureProjection: "EPSG:3857",
        }).readFeature(platform)}
        onClick={React.useCallback(() => {
          navigate(url, { replace: false })
        }, [])}
      >
        <RPopup trigger={"hover"}>
          <Button
            color="dark"
            size="sm"
            onClick={React.useCallback(() => {
              navigate(url, { replace: false })
            }, [])}
          >
            {platform.id}
          </Button>
        </RPopup>
      </RFeature>
    </RLayerVector>
  )
}

Don't use the feature property for this, it is a very special property for when you need to bind the feature to an existing feature
Use geometry

Awesome, thank you. Using geometry worked.
So, to make sure I'm understanding this, feature requires a stable OpenLayers feature, so because the events can be bound?
When I changed the code and it hot reloaded within the RPopup, then it was able to connect to the (somewhat ephemeral) feature that I generated in RFeature, but on a fresh load it is not able to connect the events?
For anyone else ending up here, this is what I changed RFeature into.
<RFeature
        geometry={React.useMemo(() => {
          const feature = new GeoJSON({
            dataProjection: "EPSG:4326",
            featureProjection: "EPSG:3857",
          }).readFeature(platform)
          return feature.getGeometry()
        }, [platform])}
        onClick={React.useCallback(() => {
          navigate(url, { replace: false })
        }, [])}
      >

It used to require a stable feature, now some form of updating is supported, but as this completely bypasses React - the feature being an opaque type for React - not everything works
This property is meant mostly for interfacing with non-React OpenLayers code, I will probably underline this in the doc

Thank you, that makes sense why it wasn't working as I expected.


Typing sounds work in the editor but do not work on export

The problem
Describe the bug
When providing a style with a typing sounds folder, the typing sounds can be heard in the editor but not on export.
To Reproduce

Create a new timeline and character, and write some sample dialogue.
Choose any style and provide it with a typing SFX folder in the "sounds" tab.
Call the new timeline by any means you see fit.
Play the timeline in the editor, and see that everything is working properly.
Export the game and launch it.
Observe that dialogues have no typing SFX.

Expected behavior
Typing sounds should be heard in all platforms.
System (please complete the following information):

OS: Windows 10
Godot Version: 4.1.2
Dialogic Version: Alpha 10

Solutions
Workaround
After many, MANY attempts with console printouts, I found that the load_sounds_from_path function does not recognize exported .wav files. It does, however, recognize .import files. Therefore, this (extremely dirty) rewrite works:
static func load_sounds_from_path(path:String):
	var dir = DirAccess.open(path)
	var sounds : Array[AudioStream]
	if dir:
		dir.list_dir_begin()
		var file_name = dir.get_next()
		while file_name != "":
			print(file_name)
			if file_name.get_extension() == "import":
				file_name = file_name.split(".import")[0]
				var stream:AudioStream = load(path + "/" + file_name)
				sounds.append(stream)
			file_name = dir.get_next()
	return sounds

Possible fixes
Rewriting the function to work with Godot's internal build logic, I guess. Maybe it's something that needs to be brought up with the engine, but I dunno, I'm a noob.

@DancingEngie Thanks for the report!
We've had similar struggles before so I will try to fix it as best as possible for the next release.

I think this is fixed now on main.

I think this is fixed now on main.

Yup. Thanks :)

I think this is fixed now on main.

For some reason defining the var type in the for loop breaks that segment. Removing that makes everything actually work. Dunno why. But other than that, the issue is indeed fixed.

Oh damn. I'm so dumb. I'm already using 4.2 alpha 3 and it has static typing in for-loops.
Will remove this. We still need to decide whether we want to increase the minimal needed godot version again.


Octavia RS 5e manual benzin- unlocked Not work

Dear jilleb,
Thanks for the Good Work.
I own a octavia rs 2.0 tsi Manual gear and try out both files:
5f0 907 159 unlocked 1 and 2
But the Sound is very Lame.
What i Need to do? Must i edit the files With the 010 Editor or Are they also edited?
Must i compare the files With the rs4 firmware in the mailfolder?

Or i just Need to flash a file from rs4/5 ?

Yes, try that and see what happens 😎


Skip symlinks

I think globby/fast-glob excludes symlinks:
https://github.com/mrmlnc/fast-glob/blob/da648078ae87bce81fcd42e64d5b2b1360c47b09/src/providers/filters/deep.ts#L38
It could potentially be an option whether to skip them or treat them as a file. It'd probably be nice to skip by default though to match the behavior of globby/fast-glob
I tried to convert cspell to use tinyglobby, but its tests failed because of this

opened thecodrr/fdir#115 for upstream support


[WIP] Initial work on Emission

It's the first 1/3rd, but I really want to set the scene for what makes RN conceptually "easy" to integrate inside Eigen, as opposed to other apps with a more traditional structure.
Then the rest is on the how/what.

Proselint found issues
_posts/2016-08-15-React-Native-at-Artsy.markdown

Line
Message
Severity

32
Substitute 'damn' every time you're inclined to write 'very;' your editor will delete it and the writing will be just as it should be. Found 8 times elsewhere.
warning

57
'in the long run' is a cliché.
warning

69
Redundancy. Use 'free' instead of 'for free,'.
warning

69
'for free,' is cliché.
warning

80
Redundancy. Use 'details' instead of 'specific details,'.
warning

99
'by no means' is a cliché.
warning

126
'In the long run,' is a cliché.
warning

134
Redundancy. Use 'drop' instead of '‘drop down’'.
warning

144
'in the long run' is a cliché.
warning

186
'tip of the iceberg' is a cliché.
warning

_posts/2016-08-18-On-Emission.markdown

Line
Message
Severity

38
Substitute 'damn' every time you're inclined to write 'very;' your editor will delete it and the writing will be just as it should be.
warning

Spell Checker found issues
_posts/2016-08-15-React-Native-at-Artsy.markdown

Line
Typo

28
application and entirely in [2 other apps](http://artsy.gith

49
everywhere in no time; 593 packages to be exactly, at th

69
a lot that you would get with UIKit for free, e.g. `UICollectionV

94
compose them to make complex UIs.

113
[our GraphQL service](http://artsy.github.

116
once, write anywhere.” is the neo-cross-platform slogan deployed by React, whi

130
[flexbox](https://en.wikipedia.org/wik

131
ound classic CSS, I find that flexbox is an abstraction that nicely

139
attern is not as ingrained in UIKit thus usually leading to more

140
replacements such as **AsyncDisplayKit.**

142
n of declarative view layout (JS, single-threaded) and technic

143
e hard constraint of having a JS/native bridge, there really i

147
than with your typical native UIKit-based

151
it out, unlike e.g. Xcode and UIKit. (Granted, you

175
ks with our application-wide [SDWebImage](https://cocoapods.org/pods/S

186
about React Native. Follow @**ArtsyOpenSource**

_posts/2016-08-18-On-Emission.markdown

Line
Typo

14
we worked in Swift with [some light-FRP][interstellar]. We were not a

14
lar]. We were not affected by 4 months of work on moving to R

24
bout the Artsy flagship app, [Eigen][eigen]. It's an app that aim

30
controllers that normally map 1:1 to routes on the Artsy webs

30
ntrollers that normally map 1:1 to routes on the Artsy websit

36
ges that it is rendering, the eigen chrome exists relatively in

44
Emission is a node module, a CocoaPod and an iOS App.

Spell Checker found issues
_posts/2016-08-15-React-Native-at-Artsy.markdown

Line
Typo

28
application and entirely in [2 other apps](http://artsy.gith

49
everywhere in no time; 593 packages to be exactly, at th

69
a lot that you would get with UIKit for free, e.g. `UICollectionV

94
compose them to make complex UIs.

113
[our GraphQL service](http://artsy.github.

116
once, write anywhere.” is the neo-cross-platform slogan deployed by React, whi

130
[flexbox](https://en.wikipedia.org/wik

131
ound classic CSS, I find that flexbox is an abstraction that nicely

139
attern is not as ingrained in UIKit thus usually leading to more

140
replacements such as **AsyncDisplayKit.**

142
n of declarative view layout (JS, single-threaded) and technic

143
e hard constraint of having a JS/native bridge, there really i

147
than with your typical native UIKit-based

151
it out, unlike e.g. Xcode and UIKit. (Granted, you

175
ks with our application-wide [SDWebImage](https://cocoapods.org/pods/S

186
about React Native. Follow @**ArtsyOpenSource**

_posts/2016-08-18-On-Emission.markdown

Line
Typo

14
we worked in Swift with [some light-FRP][interstellar]. We were not a

14
lar]. We were not affected by 4 months of work on moving to R

24
bout the Artsy flagship app, [Eigen][eigen]. It's an app that aim

30
controllers that normally map 1:1 to routes on the Artsy webs

30
ntrollers that normally map 1:1 to routes on the Artsy websit

36
ges that it is rendering, the eigen chrome exists relatively in

44
Emission is a node module, a CocoaPod and an iOS App.

  Generated by :no_entry_sign: danger

proselint has a way with words

Substitute 'damn' every time you're inclined to write 'very;' your editor will delete it and the writing will be just as it should be.

Prose lint can't claim too much credit: https://www.goodreads.com/quotes/2913-substitute-damn-every-time-you-re-inclined-to-write-very-your

@ArtsyMobile I'd love some feedback on this PR, as well as interesting things that are missing.
This post aims to answer the question "how did you add RN to Eigen", anything that fits inside that question should really be in this post.
Ash specifically, as you have no experience here, I'd love some fresh eyes on what this feels like and if you'd have more questions after reading
/cc @sarahscott @ashfurrow @mennenia @alloy

Liking it thus far 👍

I like it! It very clearly explains the different parts to making this work.
What else were you thinking of adding / how did you want to conclude the post?

Yeah, I'm struggling on this conclusion really

maybe it's the "don't rewrite" narrative, lets give that a shot

Alright, I'm happy.


Pip installable

This PR has two parts.

I added the files needed to make the wavetorch core pip installable (i.e, setup.py and requirements.txt.
I made some minimal fixes so Cell plays well with pytorch 1.2 (i.e. a type case, and removal an an inplace op in step()). I'd be very curious and grateful if you fine folks would confirm these changes are safe and ok, even if you don't want this PR. I'm using them for new work...

Note: I commented out the old/original code for the changes in point 2, really just for my reference. I'd be happy to clean that cruft up if this PR moves forward.

Right. Also, for lack of better option my email and name or in the setup.py file. I'd expect you'd change that to someone there. Not trying to claim credit. :)

Thanks for this! I've been pretty busy but I'll take a look at this in more detail as soon as possible.

So I've pushed/tagged a new version of this package. The changes you suggested in this pull request were easier to just incorporate myself given the amount of refactoring I did. Therefore, I'm going to close this pull request. I gave you a shout out in the code for the fix to the source term in pytorch 1.2. Thanks again for that!


Overlay Market Rent to Aff. Housing Units

@domaley what do you mean by this ticket? Are you making a Tableau viz or something else?


Improved structure & moved CLI to another repo

After a big think about it, the CLI is not the parser. It's 2 different projects, and they should not be in the same repository. That's the reason why I'm creating a new repository (skript-cli) in the organization and move all the CLI there.
Including this refactor:

I added some colors for logs (yellow color for warns, red for danger, blue for info and purple for debug).
I have split the init method and remove the run part. Didn't make sense to have both shuffled.
I renamed Main.class to Parser.class, which make more sense because the parser will not be an executable.
I added cache to actions and added filters to avoid to run more tests that it should

Hang on, I don't fully understand the implications of this PR yet. Does it basically end up moving the cli submodule (formerly the master branch) to a new repo, and leaving the parser submodule (formerly skript-library branch) in place?
If that is so, won't it be a problem if one wants to apply commits from the parser onto the CLI?

Does it basically end up moving the cli submodule (formerly the master branch) to a new repo, and leaving the parser submodule (formerly skript-library branch) in place?

Yeah, and added to that, I improved some parts

won't it be a problem if one wants to apply commits from the parser onto the CLI?

What do you mean? If you are talking about someone who started to work on the CLI and who commit with the CLI as submodule, he could move his work in the new repository, should not be a problem

What do you mean? If you are talking about someone who started to work on the CLI and who commit with the CLI as submodule, he could move his work in the new repository, should not be a problem

I mean that any changes that are made to the parser should be made on the CLI too, since the CLI includes the main parser code. And wouldn't it be a hassle to port all of these changes from the parser repository onto the CLI repository?

The CLI will import the parser as gradle dependency, so it should not be a problem

Right, so in its current state, the CLI only has syntaxes & registration on top of the regular parser code, so only those will be included in the new repo, right?

I worked on the CLI at my side, so what will contains the CLI repository is the:

Command registration and execution
Help command
Run command
Mini playground

All the old stuff in the CLI of the parser's repo.

I'll wait until this PR gets merged in order to apply my changes since this PR yet again changed almost every file.


TypeError: Invalid JWK EC key

Trying to upgrade my login-functions from 8.3.5 -> to v.12.x.
I got error message :
login-verify-catch-error: TypeError: Invalid JWK EC key
    at Object.ecImportKey (node:internal/crypto/ec:243:27)
    at SubtleCrypto.importKey (node:internal/crypto/webcrypto:621:10)
    at importKey (/Users/esa/devel/arkeserver/node_modules/@simplewebauthn/server/script/helpers/iso/isoCrypto/importKey.js:8:29)
    at async verifyEC2 (/Users/esa/devel/arkeserver/node_modules/@simplewebauthn/server/script/helpers/iso/isoCrypto/verifyEC2.js:62:17)
    at async Object.verifyAuthenticationResponse (/Users/esa/devel/arkeserver/node_modules/@simplewebauthn/server/script/authentication/verifyAuthenticationResponse.js:157:19)
    at async exports.loginVerify (/Users/esa/devel/arkeserver/lib/fidoSWA.js:516:29)
    at async /Users/esa/devel/arkeserver/routes/fidoRtr.js:330:16 {
  code: 'ERR_CRYPTO_INVALID_JWK'
}

NodeJS version is v22.9.0.
And I'm using plain NodeJS with require and not TypeScript at all (=old project and not need for TypeScript at a moment).
const swaServer = require('@simplewebauthn/server');
const swaHelper = require('@simplewebauthn/server/helpers');
. . . etc

Any idea what to look at my code. Can it be some isoBase64URL issue or something else.

Hello @netmiller, can you please share some sample code of how you're calling verifyAuthenticationResponse(), including the response and public key? And to confirm, responses verify fine with v8 but break once you update to v12?

Found the bug; it was just a typo in my own code.
Expression ..... = isoBase64URL.fromBuffer(_regInfo.credential.publicKey); was missing base64.
Fixed with `.... = isoBase64URL.fromBuffer(_regInfo.credential.publicKey, 'base64');
That typo was while saving registration info. Now it's working ok.

Closing this. Error happened while login-verification, and the message was something confusing.
Thanks anyway .


No CUDA runtime is found, using CUDA_HOME='/usr/local/cuda'

Thanks for your error report and we appreciate it a lot.
Checklist

I have searched related issues but cannot get the expected help.
I have read the FAQ documentation but cannot get the expected help.
The bug has not been fixed in the latest version.

Describe the bug
A clear and concise description of what the bug is.
Reproduction

What command or script did you run?

A placeholder for the command.

Did you make any modifications on the code or config? Did you understand what you have modified?
What dataset did you use?

Environment

Please run python mmdet/utils/collect_env.py to collect necessary environment information and paste it here.
You may add addition that may be helpful for locating the problem, such as

How you installed PyTorch [e.g., pip, conda, source]
Other environment variables that may be related (such as $PATH, $LD_LIBRARY_PATH, $PYTHONPATH, etc.)

Error traceback
If applicable, paste the error trackback here.
A placeholder for trackback.

Bug fix
If you have already identified the reason, you can provide the information here. If you are willing to create a PR to fix it, please also leave a comment here and that would be much appreciated!

This is a CUDA error, please make sure you have installed CUDA successfully on your computer. You can search the the install way on Google or Baidu.


CRL not validated in x509_crt_verifycrl

Note: This is just a template, so feel free to use/remove the unnecessary things
Description

Type: Bug
Priority: Minor

Bug
OS
linux
mbed TLS build:
Version: 2.7.2
Expected behavior
CRL is used for validation
Actual behavior
CRL is not used
Steps to reproduce
I went to www.renesas.com and downloaded the certificate chain and the CRLs.
When validating the chain I passed in root CA and the CRLs (From digicert)
When debugging I noticed that the CRL validation in (x509_crt_verifycrl) was bypassed.
The root CA has subject
C=US, O=DigiCert Inc, OU=www.digicert.com, CN=DigiCert Global Root CA
While the CRL has issuer
/C=US/O=DigiCert Inc/OU=www.digicert.com/CN=DigiCert Global Root CA
The name comparison fails due to this. If changing to use the x509_name_cmp function instead the validation progresses further.

Enhancement\Feature Request
Justification - why does the library need this feature?
Security bug not to use valid CRLs
Suggested enhancement
Call x509_name_cmp instead.
Question

@HenrikRosenquistAndersson Thank you for reporting this issue!
We will have a look at it

ARM Internal Ref: IOTSSL-2381

Updated internal reference: IOTSSL-1770.

Hi @HenrikRosenquistAndersson,
which tools did you use to print the subject and issuer information? I want to make sure the issue is not related to the use of / vs. , as a separator. Rather, it looks like this is due to a different character encoding used in the CRT and CRL, e.g. PrintableString vs. UTF8String. You could check this by calling dumpasn1 on the (DER-encoded) CRT and CRL.
Kind regards,
Hanno, Mbed TLS team member

Hi,
I think I printed the information from openssl and mbedtls_x509_dn_gets.
I added logs into x509_crt_verifycrl and verified that the memcmp failed.
Here are the start of the asn1dumps. Let me know if you need more of the dumps.
CRL
0 527: SEQUENCE {
4 248:   SEQUENCE {
7   1:     INTEGER 1
10  13:     SEQUENCE {
12   9:       OBJECT IDENTIFIER '1 2 840 113549 1 1 5'
23   0:       NULL
:       }
25  97:     SEQUENCE {
27  11:       SET {
29   9:         SEQUENCE {
31   3:           OBJECT IDENTIFIER '2 5 4 6'
36   2:           PrintableString 'US'
:           }
:         }
40  21:       SET {
42  19:         SEQUENCE {
44   3:           OBJECT IDENTIFIER '2 5 4 10'
49  12:           PrintableString 'DigiCert Inc'
:           }
:         }
63  25:       SET {
65  23:         SEQUENCE {
67   3:           OBJECT IDENTIFIER '2 5 4 11'
72  16:           PrintableString 'www.digicert.com'
:           }
:         }
90  32:       SET {
92  30:         SEQUENCE {
94   3:           OBJECT IDENTIFIER '2 5 4 3'
99  23:           PrintableString 'DigiCert Global Root CA'
:           }
:         }
:       }
124  13:     UTCTime 18/06/2018 17:00:00 GMT
139  13:     UTCTime 09/07/2018 17:00:00 GMT
Certificate
0 1163: SEQUENCE {
4  883:   SEQUENCE {
8    3:     [0] {
10    1:       INTEGER 2
:       }
13   16:     INTEGER 05 46 FE 18 23 F7 E1 94 1D A3 9F CE 14 C4 61 73
31   13:     SEQUENCE {
33    9:       OBJECT IDENTIFIER '1 2 840 113549 1 1 11'
44    0:       NULL
:       }
46   97:     SEQUENCE {
48   11:       SET {
50    9:         SEQUENCE {
52    3:           OBJECT IDENTIFIER '2 5 4 6'
57    2:           PrintableString 'US'
:           }
:         }
61   21:       SET {
63   19:         SEQUENCE {
65    3:           OBJECT IDENTIFIER '2 5 4 10'
70   12:           PrintableString 'DigiCert Inc'
:           }
:         }
84   25:       SET {
86   23:         SEQUENCE {
88    3:           OBJECT IDENTIFIER '2 5 4 11'
93   16:           PrintableString 'www.digicert.com'
:           }
:         }
111   32:       SET {
113   30:         SEQUENCE {
115    3:           OBJECT IDENTIFIER '2 5 4 3'
120   23:           PrintableString 'DigiCert Global Root CA'
:           }
:         }
:       }
145   30:     SEQUENCE {
147   13:       UTCTime 06/11/2017 12:23:45 GMT
162   13:       UTCTime 06/11/2027 12:23:45 GMT
:       }

Hi @HenrikRosenquistAndersson,
apologies for the long response time. I agree that the memcmp() in x509_crt_verifycrl() is too strong, but I cannot yet align this bug with the failure you're seeing, as I don't spot any binary difference in the dumps you showed. Could you point me to those if I'm missing the obvious, or attach PEM encodings of the actual CRL and CA?
Regards,
Hanno

I'm not able to reproduce this issue anymore when downloading the certificate chain today. I unfortunally don't have the old chain anymore. So I Think you can close the issue.

@HenrikRosenquistAndersson You have made a very important point, though, that the comparison of CRL issuer and CA subject name is too strict, and I have created PR #2167 to address this.
Could you let me know how you would like to be credited?

You can just mention my email address (Henrik.Andersson@se.bosch.com). Thanks!

Hi @HenrikRosenquistAndersson. We don't normally credit people by email address. Can I suggest we credit you as Henrik Andersson of Bosch GmbH?

Works fine for me


Improve Unit-tests for QPUs

Issue Description
A recent bug in the decomposition methods of some GateMaps raised concern for the quality of QPU tests. The bug was fixed with PR #91, however, we need to update the QPU test suite.

@vishal-ph has this been addressed?


Added threshold, otsu level

I spent some wasted time chasing a nonexistent bug trying to get the results to match matlab's... but it should work now. Haven't used it on actual images yet, but I will "soon".
The style may be terrible, but I am a rank amateur when it comes to rust, so I hope you'll forgive me.
I'll be swamped for a little while yet, but when I can carve out time I'm going to port some matlab code for more interesting algorithms.
I'll also add some other thresholding options eventually, too.

Looks good, thanks! I've added a selection of pretty pedantic comments - I hope you don't mind the nitpicking. Nothing major, so I'll merge this either when you make the changes or in a couple of days if you don't have the time/inclination to make them.

Any other stuff you fancy adding would be great. If the other algorithms you're considering porting are "interesting" in non-image-processing ways then maybe we should put them in a currently nonexistent computer vision sister project.

I'll happily make the changes suggested. As I said, my rust experience is essentially nil, so I appreciate any suggestions.
What I'm going to implement first are some things I'm already working with/on. One is a [phase congruency filter] (http://cvpr11.cecs.anu.edu.au/files/PhaseCongCVPR.pdf). This would be a nice addition to the library, I think.
The other is an RBG-D SLAM algorithm like kinectfusion or dvo-slam.
So SLAM would definitely need a new home. I'll start on that in earnest pretty soon, but it'll be a few weeks at the earliest before it's finished. So I'll let you know.

I'd not heard of phase congruency filters before - looks interesting. I'm also planning on implementing a few vision algorithms soon (http://www.edwardrosten.com/work/taylor_2009_robust.pdf, https://www.cs.cmu.edu/~efros/courses/LBMV07/Papers/viola-cvpr-01.pdf) but it doesn't look like there'll be much overlap in required code yet.

Great, thanks!


Connecting to peer raises warning

I need to use DBus to communicate without dbus-daemon (2 processes talk to each other directly). I've created created basic DBus server in the Vala app
static int main(string[] argv) {
  var dbus_server = new DBusServer.sync("tcp:host=localhost,port=12345", DBusServerFlags.AUTHENTICATION_ALLOW_ANONYMOUS, "82B55E5C9D9C11E5ACA2E04969A460B9");
  dbus_server.start();
  var mainloop = new GLib.MainLoop();
  mainloop.run();
  return 0;
}

Then I connect from dbus-native:
const dbus = require('dbus-native');

var dbusConn = dbus.createConnection({
  host: "<IP_ADDRESS>",
  port: 12345,
});

And the server process always raises warning:
(process:71311): GLib-GIO-WARNING **: Malformed data '4f1a540c4e6bf94f6ca79eb6d5929c4eb6b8969c909b088ca6e328cffa7e57b88c3a2b0b'

The hex string appears to be random.
The connection works anyway, but it's still unexpected behaviour.

and the equivalent in vala for your dbus-native example shows no such error? you probably need to trace what  is actually being sent.

It seems that it was because it tried to authenticate while it should treat such connection as not requiring authentication. I've disabled authentication in the client and it works fine.


debug ts/js async function and local variables all null, this variables is null

tsconfig.json
{
"$schema": "https://json.schemastore.org/tsconfig",
"compilerOptions": {
"target": "ES2015",
"module": "ES2015",
"strict": true,
"types": [
"./temp/declarations/jsb",
"./temp/declarations/cc.custom-macro",
"./temp/declarations/cc",
"./temp/declarations/cc.env"
],
"paths": {
"db://internal/": [
"D:\CocosDashboard_1.0.12\resources\.editors\Creator\3.4.1\resources\resources\3d\engine\editor\assets\"
],
"db://assets/": [
"F:\Project\Quest\Quest\assets\"
]
},
"experimentalDecorators": true,
"isolatedModules": true,
"moduleResolution": "node",
"noEmit": true,
"forceConsistentCasingInFileNames": true
}
}
{
/* Base configuration. Do not edit this field. */
"extends": "./temp/tsconfig.cocos.json",
/* Add your custom configuration here. */
"compilerOptions": {
"strict": false,
"lib": [
"es2017",
"dom",
],
}
}

Compiled js has code:
var _this2 = this;

I have same trouble.
Watch and variables (on hover) not showing in async ts function. In sync function everything is showing. In Chrome devtools everything is showing too.


Fix blurry edges

The cutout algorithm takes random 'crops' of the image from 0.5 to 1.0 the size of the image. These crops are randomly positioned however due to the size of the crop, the centre is over proportionally included in the cutouts.
We can change the algorithm so that it more often cuts to the corners, by adding a virtual 25% padding to where it chooses to take its crop. If that crop then lays outside the bounds of the image, we then snap it back into the bounds. This means roughly 1/3rd of cutouts will lie exactly on the edge, meaning the edges and corners get as much attention as the centre.
We also add a cut_pow to the cutout size to add smaller cutouts, this will also help the overrepresentation of the centre as a higher proportion of cutouts won't include it
Before Changes: (100 iterations)

After Changes: (100 Iterations)

Nice idea! I think padding by size makes more sense though, right?

ahh maybe?
My idea is essentially add a big border (or padding) around the whole image (25% the width of the sides). Then pick random coords to cutout including that border. Then if the cutout overlaps the border we snap it back to the edge.
Previously the very edge pixels had a 1 in 500 (assuming 500px wide image) to be included in the cutout, while the centre was almost always in the cutout
Now since any cutout which touches the 'border' is snapped to the edges, its about a 1/3 chance (cutouts have a ~50% chance of overlapping a 25% border)
if we did it by the cutout size then the chance of hitting the edge would vary depending on that random size


Syntax error with ESP8266.doHttpPost()

The user_agent argument has a default value when subsequent arguments have no defaults. This gives a syntax error ...
Traceback (most recent call last):
File "", line 2, in
File "esp8266.py", line 462, in ESP8266
SyntaxError: non-default argument follows default argument

I am having this issue as well. any fix?

I am having this issue as well. any fix?

def doHttpPost(self,host,path,content_type,content,user_agent='RPi-Pico',port=80):


Android: Provide a single entrypoint view that allows customers to get new tools without adding each tool individually

Menu list of the tools by name.
Click the tool and see the full-view
Easily switch between tools with a visible menu

Thinking about expanding on the details of this story a little bit more:

Create a UI that a client can place in their app in whatever screen they think is appropriate (like a developer settings screen).
The UI will be a menu that will allow a user to navigate to each of the tools.
On the Android side I'm hoping this can just be a single dependency so that the user doesn't have to pull in separate dependencies for each of the tools.
For testing this out or being able to see what it will look like, I will be adding it as a separate module and then including that module in the demo app that's part of the tools repo.
(Nice to have) In the future, as a separate story we will look at publishing to Maven locally (I was having issues) so that in can be tested in demo apps locally first. I think this presents a slightly more realistic implementation.
Existing modules can still be imported if some clients prefer to customize things more.

Let me know if I'm missing anything.

Sounds great. Whatever will get this into customers hands faster


Open Source : 6.55 Explain everything related to Spring Framework

Description
Open Source : 6.55 Explain everything related to Spring Framework

NOTE
Please put your doc 6.Popular_Projects with the name 6.55_Spring.md

Domain
Open Source

Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

Domain
Open Source
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

@Aryamanz29, can I work on this issue?


Set a spacing scale and export spacing utility classes

Add test values to a spacing scale
Set spacing properties for margin and padding
Set spacing directions - top, right, bottom, left
Set a spacing scale, using spacing scale vars. Include 0 to reset
paddings and margins.

Export classes in the format: .govuk-u-mb-2 where 2 is the spacing scale, from 0-6.
Note: we're testing this to see how these classes might work to provide spacing around components, and how these spacing vars can be combined with our typography styles. This is very likely to change.
Todo:

[x] Ensure if spacing is imported by each component, that the classes are only output once in the compiled css.

I like this a lot 🎉

In government-frontend we found that our equivalent of these classes also needed to be responsive. eg Something that is scale-3 on wide viewports and scale-2 on thin.


numpy.argmax and cupy.argmax behave differently when input is 1D

numpy.argmax returns numpy.int64 when the input array is 1-dimensional,
but cupy.argmax returns a cupy.core.core.ndarray whose (only) element is the answer.
numpy.__version__   # '1.11.3'
chainer.__version__ # '1.19.0'

numpy.argmax(numpy.array([1,2,3])) # 2
cupy.argmax(cupy.array[1,2,3]))    # array(2)

type(numpy.argmax(numpy.array([1,2,3]))) # <class 'numpy.int64'>
type(cupy.argmax(cupy.array[1,2,3])))    # <class 'cupy.core.core.ndarray'>

P.S.1
I found related functions (max, min, argmin) also behave differently in numpy and cupy.
Sorry for bothering you if these behaviors are as expected.
P.S.2
The reason why this passes the test  cases: in cupy.testing.assert_allclose(),
actual (the result of cupy.max) and desired (the result of numpy.max) are
both wrapped by cupy.asnumpy(). It turns both an integer and an 1-D array
into an 1-D array (cupy.asnumpy(1) == cupy.asnumpy(cupy.array(1)) # => True).

Sorry for late reply, but this behavior is expected. We cannot have numpy.int64 (nor any other numpy scalar types) on GPUs, while moving data between GPUs to CPU is costly. Thus, we decided to use zero-dimensional arrays in CuPy as the alternative of scalars in NumPy. It also makes the overall CuPy implementation much simpler because we do not have to deal with different types as holders of GPU data.
I will close this issue because the different behavior is expected one. Please open a new issue if you find it inconvenient and feel that CuPy should provide distinction between arrays and scalars like NumPy.


customize header, footer and homepage

Phases
MP
MM
MB
MR
JM
Total

ETA
0
0
0
0
0
0

Developing
0
0
0
0
0
0

Review
0
0
0
0
0
0

Total
-
-
-
-
-
0

ETA est.

0

ETA cust.
-
-
-
-
-
0

Problem description
https://github.com/dataquest-dev/dspace-angular/issues/188

I think :hover background color in the menu should be the same like in the https://www.tul.cz/ .
TUL:

Your:

Log in: the buttons doesn't change the color on :hover event. I think it should get a little bit darker like in the Vanilla.

dspace.tul.cz has another info in the footer than Vanilla DSpace. Please update the footer and add there Contact Us option with redirection to /contact.

On :hover the background color of TUL icon is not changed like in the tul.cz

:hover background-colors are not the same in the menu options:

@HuynhKhoa1601  I think it looks good :) Thanks.
I see one more fix and I believe for now the home page will be done.
My last suggestion is:
the dropdown menu is rolled down before the mouse cursor is changed. If you go through All of DSpace menu option slowly you should see this behaviour.


Message is not saved, when deletion of oldest message failes.

If inbucket receives many mails concurrently, there is possibility that it will try to delete one file more then one time - which causes error and the message is not saved. Error at this point should be logged, but not thrown.

Thanks, not sure when I will get a chance to look at this, but I'd like to fix it before release.


Long genres create this weird look

This is not ideal, buttons shifted.

It did not fix it sir. First image is with extension and second is without extension.

I do not know how long you should allow it to be. All I can say is long enough that it does not shift buttons 🤷♀

So you want its width to depend on the title and artists widths... I'll see what I can do, not sure if it's doable with grids though

I updated the plugin, it should be the same width of the title/artists container now.
You can re-open this issue if the issue persists.

wish i could reopen it, it's still an issue for me

(the track is like 4-5 genres iirc)
also, why'd you remove the toggle button? it was pretty neat imo.

wish i could reopen it, it's still an issue for me  (the track is like 4-5 genres iirc)

This is expected, the genres width is now fixed to the title's width or the artists' width (depending on which one is the longest)

also, why'd you remove the toggle button? it was pretty neat imo.

See https://github.com/Vexcited/better-spotify-genres/issues/8#issuecomment-2543848465

wish i could reopen it, it's still an issue for me  (the track is like 4-5 genres iirc)
also, why'd you remove the toggle button? it was pretty neat imo.

Reopen what? You can still click on the tracks to see the full list if that's what you mean.


Update system-requirements.md

@abhilashaagarwala : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@raghavendra-nataraj pls review

#sign-off


Crash on GLES3, might be related to Linux/Intel freezes.

Godot version
3.2.3(stable), 3.3(stable)
System information
Ubuntu 21.04
Issue description
A friend and I worked on a game for a gamejam the past two weeks. When playing other submissions to the jam, occasionally I would get a total freeze (flickering between two frames usually on the Godot logo) that required a hard reset.
After the jam, every new project or template I used would crash - same type of freeze. Happened 95% of the time, not every time, though. I can open the project selection window, but freezes before getting to the editor.
Similar to the issues people were having with Intel and Godot4 - I tried --single-window and running from terminal - no change.
I went back to the project my friend and I made, which happened to be in GLES2. I switched it to GLES3 and instantly got the crash again.
Since everything defaults to GLES3, new projects and templates all crash before you can change them to GLES2.
Steps to reproduce
If you're getting the Ubuntu freeze issue, try downloading a GLES2 project and opening that.
Minimal reproduction project
No response

Since everything defaults to GLES3, new projects and templates all crash before you can change them to GLES2.

As a workaround, start the editor or project with the --video-driver GLES2 command line argument.

Thanks for the info, the command line option is pretty convenient.
This issue happened with both 3D and 2D, the jam game being 2D and only freezing when switching to GLES3.
I should mention I tried a full reinstall - and cleared configs, caches, and share, too. No impact.

From your description it sounds most likely (although not definite) to be a graphics driver issue, as that is the main difference when running GLES2 / GLES3. For anything possibly graphics related it's a good idea to include info on your GPU in the issue. If it occurs in 3.2.3 as well, that's before GLES3 batching, so that suggests something fundamental your GPU is not able to deal with.
Another thing to try would be if you can force software OpenGL rendering on your system, I think there is a way to do it using llvmpipe. If that runs ok (but slow) it would also suggest a driver / GPU issue.

There is a known issue with Ubuntu freezing and flashing on Intel boards - it is apparently a driver issue, but has slightly different symptoms. This issue only occurs with Godot GLES3 for me right now, I'll keep testing and see if I can find any other program which will freeze it up, confirming the driver, Ubuntu issue rather than anything specific to Godot.

Running Godot 3.2.3 on Manjaro on a Raspberry Pi 4 has a very similar effect, except Manjaro manages to throw up an error window pointing out either outdated drivers or too old of a GPU. Godot runs fine with GLES2, so appears to be the same issue.
I am confident that this is merely an issue with Ubuntu Budgie 21.04 (and presumably older) being unable to put up the error window and freezing/flickering instead.


Use dest instead of src for relative paths of cloned repos

Suggested by @HugoPuntos
Fix #213

Question is : do we bother adding a retro-compat layer?

Codecov Report

Merging #230 into master will not change coverage.
The diff coverage is 88.67%.

@@           Coverage Diff           @@
##           master     #230   +/-   ##
=======================================
  Coverage   89.45%   89.45%
=======================================
  Files          24       24
  Lines        1148     1148
  Branches      127      127
=======================================
  Hits         1027     1027
  Misses         94       94
  Partials       27       27

Question is : do we bother adding a retro-compat layer?

Let's not. The error message is clear and points to the doc, the change is trivial, and in the worst case scenario, users can downgrade tsrc.


Include ShareSizeLimitReached error code in fileerrors

Feature Request

Can the ShareSizeLimitReached error code be included in https://github.com/Azure/azure-sdk-for-go/blob/main/sdk/storage/azfile/fileerror/error_codes.go
This is stemming from a now closed customer reported bug in AzCopy
@vibhansa-msft

@tanyasethi-msft this particular error code shall be included in azfile/fileerror/error_codes.go but I don't see this error code in auto generated code as well (azfile/internal/generated/zz_constants.go). Is this missing from swagger file itself?


Abstracting memory space within Ginkgo.

I think we need to abstract the memory handling in preparation for asynchronous operations and to prepare support for Unified virtual memory support. I have already implemented something in the memory-space-abstraction branch. This issue was already known and has an issue in Gitlab issue 85.
Idea

Separate the allocation, deallocation and the copies from the Executor class.

The executor class would be responsible for only running the operations and not for memory handling.

The current implementation in the memory-space-abstraction branch is to have a new MemorySpace class that would be responsible for the memory operations.

Concrete classes such as CudaMemorySpace, HostMemorySpace, CudaUVMSpace(NOT_YET_IMPLEMENTED) would be derived from MemorySpace and each executor would be created with a memory space.

Executors have a get_ and set_ _memory_space functions which can be used to query the memory space and change it if necessary.

CudaExecutor has CudaMemorySpace and Omp and Reference executors have HostMemorySpace as defaults.

Each concrete executor will have a function such as check validity of passed in memory_space to check if the associated memory space is possible. For example, HostMemorySpace for Cuda would not be valid but CudaUVM is okay (NOT_YET_IMPLEMENTED).

Caveats

To somewhat not break existing code, it is still possible to create an executor without a memory space but one to be created when creating the executor.
Unfortunately, for logging, we will be breaking the interface as copy , allocate and free methods no longer take executor but are associated with a memory space.

Any suggestions and improvements are welcome!

After further discussion, as suggested by others, it probably makes more sense to modify the executor copy_from to not copy if it is on the same executor. So, as the code in memory_space_abstraction will probably not be merged into develop, I push it into an expt-develop branch and if needed later, we can cherry pick the commits.


web: add backup & delete account on settings

We are adding section to back up or delete account in the Settings page.

Codecov Report
Patch and project coverage have no change.

Comparison is base (1495471) 42.45% compared to head (ac355ba) 42.45%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #294   +/-   ##
=======================================
  Coverage   42.45%   42.45%
=======================================
  Files          14       14
  Lines         954      954
=======================================
  Hits          405      405
  Misses        512      512
  Partials       37       37

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


Library Management System

The Library Management System is a comprehensive software application designed to streamline the operations of a library, enhancing the experience for both librarians and patrons. It allows users to manage book inventory efficiently, track borrowing and returning of books, and access a catalog of available resources. Key features include a user-friendly interface for browsing and borrowing books, maintaining records of borrowed items, and generating reports on library usage. The system supports a drop down menu for easy navigation between sections, ensuring that users can quickly find the information they need. By automating routine tasks, this project aims to improve resource management, reduce manual errors, and facilitate better communication within the library environment.

fix #970


fix typo and shadowed issues

the --namespaces flag was incorrect hiding that the delete wasn't working
Signed-off-by: Douglas Jacobsen<EMAIL_ADDRESS>Description
The procedure to restart strimzi related services is non-functional in the csm 1.2 upgrade.  This is at least one part of why that is true.
Checklist Before Merging

[ ] If I added any command snippets, the steps they belong to follow the prompt conventions (see example).
[ ] If I added a new directory, I also updated .github/CODEOWNERS with the corresponding team in Cray-HPE.
[ ] My commits or Pull-Request Title contain my JIRA information, or I don't have a JIRA.

Partially addressed in #2114


Name 'dll' is not defined

Hi,
Thank you for wrapping this code into a Python library! It's very much appreciated. I'm using this library on a raspberry pi (3 model B). I'm getting an error when running:
NameError: name 'dll' is not defined
Are you able to point to the proper DLL and advise on where it should be placed?
Thanks!

Coming back to this. I didn't actually solve the issue, I just moved to a different platform (Jetson Nano Developer Kit) and the problem did not follow. I'm back on a Pi (PI 5), and I'm having the same issue. Any help here would be appreciated!

My apologies, I didn't realize I needed to install the Linux ARM SDK before I installed the Linux ARM Python SDK. All settled! Thanks!

Feel free to tell me if you have another issue.


Switch to OS-specific architecture lists

Another stab at fixing #119

The failing test will be fixed in #133 but I think the matrix exclusion of '*' isn't working as expected, so don't merge this yet.

This doesn't seem quite right; if we're going to build universal2 wheels, then I don't think we also want to build separate x86_64 and arm64 wheels. Our existing universal2 wheels appear to be x86_64-only. This doesn't appear to change how they're built; i.e. I suspect this is still building a broken universal2.
You can grab the artifacts out of one of the test runs, i.e. at the bottom of https://github.com/caketop/python-starlark-go/actions/runs/3970286648 and try installing some wheels locally and see if they work.
I'm using https://github.com/asottile/setuptools-golang to build the Go part and I have a sneaking suspicion it's not universal2-aware - we might have to drop universal2 entirely and go with arch-specific wheels instead. I think that's probably fine. I'd be curious to know if the arm64 wheels this is building for macOS are any good, though, or if the Go bits are still coming out x86_64 there.

I'm out of time for this for day but I think the state after 7144cefacae8b99a59f75ba4857374998e07dd3e is the correct one but needs some work to exclude things from the matrix correctly.

setuptools-golang's author said "No" but indicated that lipo might help here.

we might have to drop universal2 entirely and go with arch-specific wheels instead

I'm leaning toward this.

Closing in favor of https://github.com/caketop/python-starlark-go/pull/134


Trying to Log In, gets sent to black webpage.

I've been attempting to log in for 30 minutes. The first time I was sent to the Alta website where it allowed me to use my log-in information. After I did nothing happened. I tried to restart Voodoo. Every time I try to log in now, I get a black webpage, occasionally asking me to open Voodoo even though it's already open. Clicking open does nothing. I have not yet made it passed the authenticating screen.

Update. I have revoked access, given access back. I can still not get passed authenticating.

Update. Voodoo has all server permissions in the server and is still not appearing as a supported server.

This issue is resolved: please make sure your system time is accurate. If you change your system clock Voodoo cannot validate tokens returned by Alta.


bonding netdev options

systemd supports options for bonding in the netdev device however when attempting to use them under the latest stable release of coreos I get a "[Bond] unknown stanza type" error or something like that in journald.  /proc/net/bonding/bond0 also shows the mode for the bond not being updated.
Here is the doc I followed:
http://www.freedesktop.org/software/systemd/man/systemd.netdev.html#[Bond] Section Options

This is still broken, CoreOS version 607.0.0
Ethernet Channel Bonding Driver: v3.7.1 (April 27, 2011)

Bonding Mode: load balancing (round-robin)
MII Status: up
MII Polling Interval (ms): 0
Up Delay (ms): 0
Down Delay (ms): 0

Slave Interface: eno4
MII Status: up
Speed: 1000 Mbps
Duplex: full
Link Failure Count: 0
Permanent HW addr: 0c:c4:7a:0b:9f:b9
Slave queue ID: 0

Slave Interface: eno3
MII Status: up
Speed: 1000 Mbps
Duplex: full
Link Failure Count: 0
Permanent HW addr: 0c:c4:7a:0b:9f:b8
Slave queue ID: 0```


zoom problem

Challenge [Change the Font Size of an Element] I do not know why I am still getting zoom error when I have my zoom at 100 as they specified. 👎

CatPhotoApp

Kitty ipsum dolor sit amet, shed everywhere shed everywhere stretching attack your ankles chase the red dot, hairball run catnip eat the grass sniff.

 Purr jump eat the grass rip the couch scratched sunbathe, shed everywhere rip the couch sleep in the sink fluffy fur catnip scratched.

@DivyaM23
Thanks for reporting this. But you got typo.  you almost get it
GitHub Issues are for reporting bugs on the website only.
Please use the Help chat room for getting challenge related help.
Happy Coding.


Update Footer to have grid with sections

We have some information that is currently not on the website that we can add.
To Fix
We should restructure the footer to a grid layout.
About Column

About BPD
Leadership
Mission
Code of Conduct

Support Column

Support

Programs and Events Column

Events
Meetup
Programs

Community Column

Community
Social Links

I will pick it up

Hello @kjaymiller, I think we should include scenarios like this to our CONTRIBUTING/MAINTAINERS file. "... contributors should not be assigned to a new issue if they have an open PR still in review, except in scenarios where the issues are exclusively related to the existing PR", something like that

I'm okay with @Selasi3 making the changes but they need to be on a separate PR.

I would like to work on this task.

CC: @kjaymiller

Hi @Selasi3, just following up to check if there are any updates on this issue. Thanks

Reopening due to lack of communication. @TimOsahenru please proceed to work on this.

Reopening due to lack of communication. @TimOsahenru please proceed to work on this.

Thanks Jay

We have some information that is currently not on the website that we can add.
To Fix
We should restructure the footer to a grid layout.
About Column

About BPD
Leadership
Mission
Code of Conduct

Support Column

Support

Programs and Events Column

Events
Meetup
Programs

Community Column

Community
Social Links

On the about column, shouldn't it be a short about on BPD and then maybe have a quick link for

About BPD
Leadership
Mission
Code of Conduct. So, something like this

About Column

Black Python Devs was created by its founder Jay Miller after seeing a trend of the same handful of Black developers speaking at major conferences, taking leadership positions, and dealing with the same challenges towards burnout.

Quick link

About BPD
Leadership
Mission
Code of Conduct


Override Feature Input Doesn't Register

On Senior Socials on staging (http://staging.beta.dosomething.org/node/9/), there is one approved report back, the total approved quantity correctly registers 15. However, inputting 1000 into the override field and running cron doesn't change the TAQ in any of the following places:
http://staging.beta.dosomething.org/campaigns/senior-socials
http://staging.beta.dosomething.org/node/9/custom-settings
http://staging.beta.dosomething.org/node/9/rb/reviewed
TAQ should say 1015.

@deadlybutter the count on the view page should be pulling from both the sum_rb_quantity and the sum_rb_quantity_override
it's doing that here
but then it's not passed into the tpl correctly


Added support to add UIKit+AFNetworking using Swift Package Manager

Issue Link :link:
https://github.com/AFNetworking/AFNetworking/issues/4616
Goals :soccer:

Added support to add UIKit+AFNetworking using Swift Package Manager

Implementation Details :construction:

Added target UIKit_AFNetworking depends on target AFNetworking

Testing Details :mag:

Create a project and try to add AFNetworking or UIKit_AFNetworking target.

Any estimates when is this going in and to be released?

Thanks for the PR! Unfortunately this makes the UIKit extensions available to all package consumers, not just those on iOS, so anyone who includes both packages and isn't on iOS will see a build failure. There doesn't seem to be a way to conditionalize the target so it's only available on iOS, which is why I didn't include it when creating the package in the first place. For that reason I'm inclined not to merge this PR and just say that anyone who wants the extensions should use other package managers, unless there's an alternate solution.

As someone who consumes platform-dependent packages from multiple platforms, I know there are many mechanisms to remove the difficulties presented here. You could have a specific target just for macOS/AppKit or watchOS/WatchKit, or you could surround imports with compiler conditionals. Merging this will not break existing code, and those requiring it will be able to easily overcome any challenges they face including it.


Pop-up chart CO2 could be a lot more interesting.

The current pop-up chart for CO2 (when you click the dashboard) is rather flat:

We already have a more in-dept analysis of the CO2-emissions per sector:

The pop-up chart would be a lot more informative when we would integrate the second chart above. I would prefer to keep the three columns rather, then we can keep the red line for the set target.

I am in favour of this 👍
The three-bar chart is a bit custom, I think @jorisberkhout or @antw know how to do this best. With some help I could also pick this up.

I like the idea of integrating those charts!

@ploh could you pick this up?

My current implementation draft looks like this (I know that the scaling is off. Will fix it):

@jorisberkhout @dennisschoenmakers I compared the gqueries of the old 3 bar chart and the new (resp. the old 2 bar chart).
The old computes the domestic co2 as V(G(final_demand_group),primary_co2_emission).sum
The new does SUM(V(INTERSECTION(G(final_demand_group),SECTOR(households)), primary_co2_emission)) and stacks that for the 7 sectors shown above.
Is that guaranteed to be equal or could it happen that some of the final_demand_group is in none of the 7 sectors?

Reopened because of open question

@ChaelKruip Could you please have a look at the (probably very easy) open question above

Is that guaranteed to be equal or could it happen that some of the final_demand_group is in none of the 7 sectors?

I think this is guaranteed. You can easily check this yourself by using:
TXT_TABLE(G(final_demand_group), key)

In the sandbox.


Add context menu entries for creating and deleting a database

This change adds a new Create Database dialog using the Object Management framework, as well as a Delete Database modal prompt using the same. Delete works now, but I'm still working out some issues in the SQL Tools Service changes for Create. If I'm not able to get Create working correctly by our code complete cutoff, then I'll disable the context menu entries for it.
Screenshots:

Anyone know how to improve the performance of the model view dropdowns with large arrays of data? The collation dropdown has 5509 items in it, so it takes a few seconds before the dropdown opens.

Anyone know how to improve the performance of the model view dropdowns with large arrays of data? The collation dropdown has 5509 items in it, so it takes a few seconds before the dropdown opens.

Please file an issue for that - it's definitely not a normal scenario but I'm kind of surprised there's that much of an impact (especially in the order of seconds...)


CAA Dimensions blocks edits

I love the idea of this extension but I had to disable it because I can't submit ANYTHING on MB website while it's active: edits, votes, edit notes, etc. I always get this message, and clicking continue doesn't do anything. I know this was implemented recently to fix some security issues, but normally only affects things like the discogs release seeder.

For what it's worth, I'm using Vivaldi but have all the ad and tracker blocking turned off for MusicBrainz.

I cannot reproduce this on Vivaldi 3.5.2115.81 with TamperMonkey v4.11. What userscript engine are you using to load it?
Frankly, it would surprise me if this userscript could cause issues like this, since it doesn't (or at least shouldn't) tamper with any forms. It appears that the origin header is being removed from the form submission, which is something that this script should not be doing.

It's definitely related to this script, I went through every script that was active, turning them off and on to test. I'm also using Tampermonkey 4.11, but I'm on Vivaldi 3.6.2165.36 which has introduced some other weird bugs that I've noticed. It's entirely possible the bug is with Vivaldi but I think you have a better chance than me of diagnosing exactly where things are going wrong.

Just to triple check, I went back and turned off ALL other scripts, even ones that wouldn't be active on that page. Interestingly, that seems to have cleared it up. Which means there's something about your script alongside another script or scripts. I'll let you know when I figure out more.

test

Looks like it's an interaction with Preferred MBS (which I thought I had turned off previously, but maybe the script was still impacting the page when I tested). If I turn either one off, it works again. Preferred MBS is hardly a necessity, so I'll turn it off for now.

I posted an issue with jesus2099, too: https://github.com/jesus2099/konami-command/issues/562

I've tried with 3.6.2165.36, this script and preferred MBS installed, and setting MBS preference to beta. The closest I've come to reproducing this issue is by opening an edit on the main site and submitting a vote, which then redirects to the beta site and shows such a warning page. Importantly though, the origin is set to musicbrainz.org, not null, and it still happens when CAA dimensions is disabled.
As far as I can tell, that warning page is generated when the MB server receives a form submission with the HTTP Origin header (or Referrer?) that differs from the domain on which the request is received. E.g., for the release seeder, the request originates from discogs.com, which is different from musicbrainz.org and therefore leads to the warning. Similarly, in my case, the origin would be musicbrainz.org while the target is beta.musicbrainz.org (whether that should lead to a warning is outside the scope of this issue). In your case, the Origin header seems to be set to null. Some extensions remove origin headers for cross-domain requests to prevent tracking etc. That still doesn't explain why it works with either of the two scripts enabled, but not when both are enabled.
Do you use any of such extensions? Does this issue still happen when submitting a form directly on beta.musicbrainz.org? Does it happen on pages that do not contain any cover art at all? Does the console show anything out of place?

@psychoadept Are you still facing this issue?


Support LLMs from vllm

🐛 Describe the bug
1

Hey @xxm1668, we don't yet have support for models from vllm but it should be easy to add. We can prioritize it for you.


Featur Request: use expanded tables for larger entrypoint mapping

https://github.com/pypa/setuptools_scm/pull/1028/files illustrates the issue in a practical manner
while for singular/small numbers of  entry-points, the inline format is fine, for large numbers its simply unreadable

Should someone put in a pull request for this, I would be happy to accept it. However, rather than be dynamic, I would prefer for the user to need to opt in into this explicitly. I was thinking something along the line of --tables flag with short or long values.


Pract.03: Add 11,12,14 | Nikolina Madzharova

Nikolina Madzharova
SI 4 0MI0600464

Good work!
Your solutions are great, with only a couple of remarks in Task12, check the already accepted solution in the repo and if you have further questions, do not hesitate to ask!
Also, do not forget that any STL functions are forbidden!!! ( Task 11 with pow(n, k) )


file upload function

Current behavior
Adds a file upload functionality, continuous loop still present for the ui after re-build
Proposed changes
Addresses: https://github.com/neokd/NeoGPT/issues/100
Checks

[ ] All commits in this Pull Request are signed and Verified by Github.

@Bchass  can you share a screenshot of the UI?

Still need to investigate the loop it causes with the Streamlit ui after the dbase builds again

@Bchass thanks for the contribution merging for now.


Effects smoothing+

I want to bring back a point which isn't properly solved. Effects and smoothing. I see the follwing issues:

Some effects require it (candle, mood blobs) and others should use it too, instead running on device latchtime border. Well, it is smooth, but the cpu usage of hyperiond is on a Pi2 at around 40%, for the swirl which is very high.
Using the global smoothing for control about this is weird. The values might be too high and what happens if you run 2 effects which wants to enable and disable it? Check against the active one again?

From a code view it might be better to implement a own smoothing for the effect engine which could be build more dynamically for better control from effects.
https://github.com/hyperion-project/hyperion.ng/issues/352
@penfold42

Personally I'd rather add smoothing settings to the effect schema and enhance the existing smoothing to be dynamic rather than just off/on

I have a nice idea. I'm on it ... stay tuned

issue solved?

I think so. Thank you
Will later on tune everything.


Attempt to fail test due to #189

see #189 -- if this passes, then #190 should fix it!

Ugh. wrong repo.

https://github.com/hoodiehq/first-timers-bot/pull/191


Updated links

Summary
Updated the deprecated links
Ticket Link

Hello @Sayanta66,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.