[Bug]:  multiprocessing KeyError from cache[rtype].remove(name)

Your current environment
Collecting environment information...
PyTorch version: 2.3.0+cu121
Is debug build: False
CUDA used to build PyTorch: 12.1
ROCM used to build PyTorch: N/A

OS: Ubuntu 22.04.4 LTS (x86_64)
GCC version: (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0
Clang version: Could not collect
CMake version: version 3.29.5
Libc version: glibc-2.35

Python version: 3.10.12 (main, Nov 20 2023, 15:14:05) [GCC 11.4.0] (64-bit runtime)
Python platform: Linux-5.15.0-107-generic-x86_64-with-glibc2.35
Is CUDA available: True
CUDA runtime version: Could not collect
CUDA_MODULE_LOADING set to: LAZY
GPU models and configuration:
GPU 0: Tesla T4
GPU 1: Tesla T4

Nvidia driver version: 535.183.01
cuDNN version: Could not collect
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True

CPU:
Architecture:                       x86_64
CPU op-mode(s):                     32-bit, 64-bit
Address sizes:                      46 bits physical, 48 bits virtual
Byte Order:                         Little Endian
CPU(s):                             104
On-line CPU(s) list:                0-103
Vendor ID:                          GenuineIntel
Model name:                         Intel(R) Xeon(R) Gold 6230R CPU @ 2.10GHz
CPU family:                         6
Model:                              85
Thread(s) per core:                 2
Core(s) per socket:                 26
Socket(s):                          2
Stepping:                           7
CPU max MHz:                        4000.0000
CPU min MHz:                        1000.0000
BogoMIPS:                           4200.00
Flags:                              fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe syscall nx pdpe1gb rdtscp lm constant_tsc art arch_perfmon pebs bts rep_good nopl xtopology nonstop_tsc cpuid aperfmperf pni pclmulqdq dtes64 monitor ds_cpl vmx smx est tm2 ssse3 sdbg fma cx16 xtpr pdcm pcid dca sse4_1 sse4_2 x2apic movbe popcnt tsc_deadline_timer aes xsave avx f16c rdrand lahf_lm abm 3dnowprefetch cpuid_fault epb cat_l3 cdp_l3 invpcid_single intel_ppin ssbd mba ibrs ibpb stibp ibrs_enhanced tpr_shadow vnmi flexpriority ept vpid ept_ad fsgsbase tsc_adjust bmi1 avx2 smep bmi2 erms invpcid cqm mpx rdt_a avx512f avx512dq rdseed adx smap clflushopt clwb intel_pt avx512cd avx512bw avx512vl xsaveopt xsavec xgetbv1 xsaves cqm_llc cqm_occup_llc cqm_mbm_total cqm_mbm_local dtherm ida arat pln pts pku ospke avx512_vnni md_clear flush_l1d arch_capabilities
Virtualization:                     VT-x
L1d cache:                          1.6 MiB (52 instances)
L1i cache:                          1.6 MiB (52 instances)
L2 cache:                           52 MiB (52 instances)
L3 cache:                           71.5 MiB (2 instances)
NUMA node(s):                       2
NUMA node0 CPU(s):                  0,2,4,6,8,10,12,14,16,18,20,22,24,26,28,30,32,34,36,38,40,42,44,46,48,50,52,54,56,58,60,62,64,66,68,70,72,74,76,78,80,82,84,86,88,90,92,94,96,98,100,102
NUMA node1 CPU(s):                  1,3,5,7,9,11,13,15,17,19,21,23,25,27,29,31,33,35,37,39,41,43,45,47,49,51,53,55,57,59,61,63,65,67,69,71,73,75,77,79,81,83,85,87,89,91,93,95,97,99,101,103
Vulnerability Gather data sampling: Mitigation; Microcode
Vulnerability Itlb multihit:        KVM: Mitigation: VMX disabled
Vulnerability L1tf:                 Not affected
Vulnerability Mds:                  Not affected
Vulnerability Meltdown:             Not affected
Vulnerability Mmio stale data:      Mitigation; Clear CPU buffers; SMT vulnerable
Vulnerability Retbleed:             Mitigation; Enhanced IBRS
Vulnerability Spec rstack overflow: Not affected
Vulnerability Spec store bypass:    Mitigation; Speculative Store Bypass disabled via prctl and seccomp
Vulnerability Spectre v1:           Mitigation; usercopy/swapgs barriers and __user pointer sanitization
Vulnerability Spectre v2:           Mitigation; Enhanced IBRS; IBPB conditional; RSB filling; PBRSB-eIBRS SW sequence; BHI Syscall hardening, KVM SW loop
Vulnerability Srbds:                Not affected
Vulnerability Tsx async abort:      Mitigation; TSX disabled

Versions of relevant libraries:
[pip3] numpy==1.26.4
[pip3] nvidia-nccl-cu12==2.20.5
[pip3] torch==2.3.0
[pip3] transformers==4.41.2
[pip3] triton==2.3.0
[conda] Could not collect
ROCM Version: Could not collect
Neuron SDK Version: N/A
vLLM Version: 0.5.0.post1
vLLM Build Flags:
CUDA Archs: Not Set; ROCm: Disabled; Neuron: Disabled
GPU Topology:
GPU0	GPU1	CPU Affinity	NUMA Affinity	GPU NUMA ID
GPU0	 X 	SYS	0,2,4,6,8,10	0		N/A
GPU1	SYS	 X 	1,3,5,7,9,11	1		N/A

Legend:

  X    = Self
  SYS  = Connection traversing PCIe as well as the SMP interconnect between NUMA nodes (e.g., QPI/UPI)
  NODE = Connection traversing PCIe as well as the interconnect between PCIe Host Bridges within a NUMA node
  PHB  = Connection traversing PCIe as well as a PCIe Host Bridge (typically the CPU)
  PXB  = Connection traversing multiple PCIe bridges (without traversing the PCIe Host Bridge)
  PIX  = Connection traversing at most a single PCIe bridge
  NV#  = Connection traversing a bonded set of # NVLinks

🐛 Describe the bug
+ exec python3 -m vllm.entrypoints.openai.api_server --port 8000 --model /models/TheBloke/Llama-2-13B-chat-AWQ --tensor-parallel-size 2 --gpu-memory-utilization 0.6 --quantization awq
INFO 06-18 18:49:00 api_server.py:177] vLLM API server version 0.5.0.post1
INFO 06-18 18:49:00 api_server.py:178] args: Namespace(host=None, port=8000, uvicorn_log_level='info', allow_credentials=False, allowed_origins=['*'], allowed_methods=['*'], allowed_headers=['*'], api_key=None, lora_modules=None, chat_template=None, response_role='assistant', ssl_keyfile=None, ssl_certfile=None, ssl_ca_certs=None, ssl_cert_reqs=0, root_path=None, middleware=[], model='/models/TheBloke/Llama-2-13B-chat-AWQ', tokenizer=None, skip_tokenizer_init=False, revision=None, code_revision=None, tokenizer_revision=None, tokenizer_mode='auto', trust_remote_code=False, download_dir=None, load_format='auto', dtype='auto', kv_cache_dtype='auto', quantization_param_path=None, max_model_len=None, guided_decoding_backend='outlines', distributed_executor_backend=None, worker_use_ray=False, pipeline_parallel_size=1, tensor_parallel_size=2, max_parallel_loading_workers=None, ray_workers_use_nsight=False, block_size=16, enable_prefix_caching=False, disable_sliding_window=False, use_v2_block_manager=False, num_lookahead_slots=0, seed=0, swap_space=4, gpu_memory_utilization=0.6, num_gpu_blocks_override=None, max_num_batched_tokens=None, max_num_seqs=256, max_logprobs=20, disable_log_stats=False, quantization='awq', rope_scaling=None, rope_theta=None, enforce_eager=False, max_context_len_to_capture=None, max_seq_len_to_capture=8192, disable_custom_all_reduce=False, tokenizer_pool_size=0, tokenizer_pool_type='ray', tokenizer_pool_extra_config=None, enable_lora=False, max_loras=1, max_lora_rank=16, lora_extra_vocab_size=256, lora_dtype='auto', long_lora_scaling_factors=None, max_cpu_loras=None, fully_sharded_loras=False, device='auto', image_input_type=None, image_token_id=None, image_input_shape=None, image_feature_size=None, image_processor=None, image_processor_revision=None, disable_image_processor=False, scheduler_delay_factor=0.0, enable_chunked_prefill=False, speculative_model=None, num_speculative_tokens=None, speculative_max_model_len=None, speculative_disable_by_batch_size=None, ngram_prompt_lookup_max=None, ngram_prompt_lookup_min=None, model_loader_extra_config=None, preemption_mode=None, served_model_name=None, qlora_adapter_name_or_path=None, engine_use_ray=False, disable_log_requests=False, max_log_len=None)
WARNING 06-18 18:49:00 config.py:217] awq quantization is not fully optimized yet. The speed can be slower than non-quantized models.
2024-06-18 18:49:02,601	INFO worker.py:1753 -- Started a local Ray instance.
INFO 06-18 18:49:03 config.py:623] Defaulting to use mp for distributed inference
INFO 06-18 18:49:03 llm_engine.py:161] Initializing an LLM engine (v0.5.0.post1) with config: model='/models/TheBloke/Llama-2-13B-chat-AWQ', speculative_config=None, tokenizer='/models/TheBloke/Llama-2-13B-chat-AWQ', skip_tokenizer_init=False, tokenizer_mode=auto, revision=None, rope_scaling=None, rope_theta=None, tokenizer_revision=None, trust_remote_code=False, dtype=torch.float16, max_seq_len=4096, download_dir=None, load_format=LoadFormat.AUTO, tensor_parallel_size=2, disable_custom_all_reduce=False, quantization=awq, enforce_eager=False, kv_cache_dtype=auto, quantization_param_path=None, device_config=cuda, decoding_config=DecodingConfig(guided_decoding_backend='outlines'), seed=0, served_model_name=/models/TheBloke/Llama-2-13B-chat-AWQ)
WARNING 06-18 18:49:03 logger.py:146] VLLM_TRACE_FUNCTION is enabled. It will record every function executed by Python. This will slow down the code. It is suggested to be used for debugging hang or crashes only.
INFO 06-18 18:49:03 logger.py:150] Trace frame log is saved to /tmp/vllm/vllm-instance-55c4b1c07eb04b5cb88254a49e2e53f4/VLLM_TRACE_FUNCTION_for_process_1_thread_139769720075392_at_2024-06-18_18:49:03.498228.log
INFO 06-18 18:49:04 selector.py:131] Cannot use FlashAttention-2 backend for Volta and Turing GPUs.
INFO 06-18 18:49:04 selector.py:51] Using XFormers backend.
(VllmWorkerProcess pid=768) WARNING 06-18 18:49:10 logger.py:146] VLLM_TRACE_FUNCTION is enabled. It will record every function executed by Python. This will slow down the code. It is suggested to be used for debugging hang or crashes only.
(VllmWorkerProcess pid=768) INFO 06-18 18:49:10 logger.py:150] Trace frame log is saved to /tmp/vllm/vllm-instance-55c4b1c07eb04b5cb88254a49e2e53f4/VLLM_TRACE_FUNCTION_for_process_768_thread_140264482706560_at_2024-06-18_18:49:10.092174.log
(VllmWorkerProcess pid=768) INFO 06-18 18:49:10 selector.py:131] Cannot use FlashAttention-2 backend for Volta and Turing GPUs.
(VllmWorkerProcess pid=768) INFO 06-18 18:49:10 selector.py:51] Using XFormers backend.
DEBUG 06-18 18:49:13 parallel_state.py:526] world_size=2 rank=0 local_rank=0 distributed_init_method=tcp://<IP_ADDRESS>:53801 backend=nccl
(VllmWorkerProcess pid=768) INFO 06-18 18:49:19 multiproc_worker_utils.py:215] Worker ready; awaiting tasks
(VllmWorkerProcess pid=768) DEBUG 06-18 18:49:19 parallel_state.py:526] world_size=2 rank=1 local_rank=1 distributed_init_method=tcp://<IP_ADDRESS>:53801 backend=nccl
(VllmWorkerProcess pid=768) INFO 06-18 18:49:19 utils.py:637] Found nccl from library libnccl.so.2
(VllmWorkerProcess pid=768) INFO 06-18 18:49:19 pynccl.py:63] vLLM is using nccl==2.20.5
INFO 06-18 18:49:19 utils.py:637] Found nccl from library libnccl.so.2
INFO 06-18 18:49:19 pynccl.py:63] vLLM is using nccl==2.20.5
Traceback (most recent call last):
  File "/usr/lib/python3.10/multiprocessing/resource_tracker.py", line 209, in main
    cache[rtype].remove(name)
KeyError: '/psm_a1296931'
INFO 06-18 18:49:20 custom_all_reduce_utils.py:170] generating GPU P2P access cache in /root/.config/vllm/gpu_p2p_access_cache_for_0,1.json
(VllmWorkerProcess pid=768) INFO 06-18 18:49:59 custom_all_reduce_utils.py:179] reading GPU P2P access cache from /root/.config/vllm/gpu_p2p_access_cache_for_0,1.json
INFO 06-18 18:49:59 custom_all_reduce_utils.py:179] reading GPU P2P access cache from /root/.config/vllm/gpu_p2p_access_cache_for_0,1.json
(VllmWorkerProcess pid=768) INFO 06-18 18:49:59 selector.py:131] Cannot use FlashAttention-2 backend for Volta and Turing GPUs.
(VllmWorkerProcess pid=768) INFO 06-18 18:49:59 selector.py:51] Using XFormers backend.
INFO 06-18 18:49:59 selector.py:131] Cannot use FlashAttention-2 backend for Volta and Turing GPUs.
INFO 06-18 18:49:59 selector.py:51] Using XFormers backend.

...and then vLLM crashes.

I don't think your code crashes after this benign error. see https://github.com/vllm-project/vllm/issues/5468

You are in fact correct: of course, immediately after posting this bug report, I found out that vLLM is taking a while to connect and it's being killed by its supervisor.
Sorry for the annoyance.  Maybe it'll save someone else from the red herring in the future...?

Though benign, it's still a bit strange / not ideal that these errors appear in the log. So this might still be a legit issue to track.
FWIW it does not seem to happen when fork is used instead of spawn (VLLM_WORKER_MULTIPROC_METHOD=fork). Looks like we'll probably make this the default soon anyhow: https://github.com/vllm-project/vllm/pull/5669.

@njhill please check #5512 . That benign error comes from Python's incorrect resource tracking


update to 1.4.1

Fetch from https://github.com/brutaldesign/swipebox/tree/1.4.1 and some rails friendly update.

Closing in favor of 5025c724aa1b3a7573452cc4262402e5ff5fb3bf


FTP Upload task cannot delete files with french accents in filename

Troubleshooting
FTP Upload task has Clean the remote directory content selected but can't delete files with french accents in filename.
Environment

Server - VSTS or TFS on-premises?

If using TFS on-premises, provide the version: 2018

Agent - Hosted or Private:

Private

Error logs
2018-06-17T23:27:40.1328221Z connected: Microsoft FTP Service
2018-06-17T23:27:40.1800885Z removing remote directory contents: /MyPath
2018-06-17T23:27:40.4559143Z FTP upload failed: "Error cleaning remote path: Unable to remove remote folder: \MyPath error: Error: The system cannot find the file specified.". FTP log: "[connection] > 'DELE \MyPath\Formation/pdf/Skype/Aide-Mémoire Skype - Audio.pdf',[connection] < '550 The system cannot find the file specified. \r\n',[parser] < '550 The system cannot find the file specified. \r\n',[parser] Response: code=550, buffer='The system cannot find the file specified.'".
2018-06-17T23:27:40.4627424Z ##[error]Error cleaning remote path: Unable to remove remote folder: \MyPath\Formation error: Error: The system cannot find the file specified.

I couldn't repro this with latest FTPUpload v2:
============================================================================== |
-- | --
  | Task         : FTP upload |
  | Description  : Upload files using FTP |
  | Version      : 2.154.0 |
  | Author       : Microsoft Corporation |
  | Help         : https://docs.microsoft.com/azure/devops/pipelines/tasks/utility/ftp-upload |
  | ============================================================================== |
  | 53dc2219-5ab5-436f-891d-f688ae705de0 exists true |
  | connecting to: waws-prod-sn1-057.ftp.azurewebsites.windows.net:21 |
  | connected: 220 Microsoft FTP Service |
  | removing remote directory contents: /upload/492535/unicode |
  | files uploaded: 0, directories processed: 1, total: 1, remaining: 2, remote directory successfully created/verified: /upload/492535/unicode |
  | File: /upload/492535/unicode/Aide-Mémoire.txt Type: upload Transferred: 0 |
  | File: /upload/492535/unicode/Aide-Mémoire.txt Type: upload Transferred: 7 |
  | files uploaded: 1, directories processed: 1, total: 2, remaining: 1, successfully uploaded: D:\a\1\s\unicode characters\Aide-Mémoire.txt to: /upload/492535/unicode/Aide-Mémoire.txt |
  | File: /upload/492535/unicode/German umlauts-äöü.txt Type: upload Transferred: 0 |
  | File: /upload/492535/unicode/German umlauts-äöü.txt Type: upload Transferred: 44 |
  | files uploaded: 2, directories processed: 1, total: 3, remaining: 0, successfully uploaded: D:\a\1\s\unicode characters\German umlauts-äöü.txt to: /upload/492535/unicode/German umlauts-äöü.txt |
  | FTP upload successful |
  | host: waws-prod-sn1-057.ftp.azurewebsites.windows.net |
  | path: /upload/492535/unicode |
  | files uploaded: 2 |
  | directories processed: 1 |
  | disconnecting from: waws-prod-sn1-057.ftp.azurewebsites.windows.net |
  | Finishing: FTP Upload: unicode characters

This new task is released on hosted service already, and will be included in the next Azure DevOps 2019 update.

Please open new issue against FtpUpload V2 task if the problem is still there.


MakeListener Command overwrites existing class

Laravel Version: 8.78.1
PHP Version: 8.0.8

Description:
When running php artisan make:listener MyEventListener, the command overwrites my existing class. The Illuminate\Foundation\Console\ListenerMakeCommand class overwrites the  alreadyExists() method of  Illuminate\Console\GeneratorCommand and does a simple check if the classname (MyEventListener) exists with php's default class_exists(). This returns a false every time since we do not provide a fully qualified class name.
Steps To Reproduce:
Run php artisan make:listener MyEventListener multiple times

Can confirm this also happens on php 8.0.13.
Appears to be fixed for the generating of events: https://github.com/laravel/framework/blob/8.x/src/Illuminate/Foundation/Console/EventMakeCommand.php#L36

Let's see how the PR goes, thanks


Unit tests and fixes for get eye height

Description
This PR adds unit tests to cover the method getEyeHeight for some of the classes.
During the creation of this tests, i noticed that some behaviors were not properly implemented.
To simplify the review, i would recommend review each commit separated, since it makes it easier to identify what's the class under test.
Checklist
The following items should be checked before the pull request can be merged.

[X] Code follows existing style.
[X] Unit tests added (if applicable).

The build is failing due to an update in PaperMC, and this also affects the branch v1.21.
From what i could identify, the issue was introduced in build 1.21-R0.1-20240809.202931-130, because build 1.21-R0.1-20240807.125219-129 works as intended.

Validated that PR https://github.com/MockBukkit/MockBukkit/pull/1062 fixes the issue. Waiting for it to be merged

It's merged

Thank you for the update. PR has been updated.


Support Marker Comments

What would be involved in supporting marker comments in this extension? I would love to be able to use the section markers popularised by Xcode in my Typescript projects in Nova. In particular, getting a horizontal divider in the editor (similar or identical to Nova’s 'type and function separators'), and being able to browse the markers in the symbol list.
// MARK: - My Section Title
I’d be willing to propose a PR for this if anyone can suggest whether it’s actually possible within the extension API and roughly how to approach it.

I personally think this should actually live outside of this extension as a separate extension, since it's not a feature of the typescript language, and would apply to any language.


asynchronous for each

I find myself doing the following over and over again:
var items = [1,2,3,4]; // some array
var itemsP = items.map(function(item){
  // do some blocking operation with item which returns a promise
  return item;
});
return q.all(itemsP);

Maybe it makes sense do define a function Q.each or Q.map as follows
Q.each = function(arr, func) {
  return q.all(arr.map(func));
}

I think the API surface is pretty frozen at this point, but you can either use a library with these helpers like https://github.com/petkaantonov/bluebird or implement them yourself (like you did above)


How do I extract the title of a playlist when downloading a single video?

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I have updated yt-dlp to nightly or master (update instructions)
[X] I've searched known issues and the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
Is there any option to get the title of a playlist (playlist_title) and write it to the meta file (meta_album for my case), without downloading the entire playlist ?
When using the --no playlist option in combination with a youtube link that also contains the playlist,
https://www.youtube.com/watch?v=z5rRZdiu1UE&list=PL28E94F6A589337C8
then playlist_title is not available ( NA / null) :
[MetadataParser] Parsed meta_album from '%(playlist_title)s': 'NA'
Query example:
yt-dlp --extract-audio --audio-format mp3 --audio-quality 0 --parse-metadata "playlist_title:%(meta_album)s" --embed-metadata --write-thumbnail --no-playlist "https://www.youtube.com/watch?v=z5rRZdiu1UE&list=PL28E94F6A589337C8"
Query result:
[youtube:tab] Extracting URL: https://www.youtube.com/watch?v=z5rRZdiu1UE&list=PL28E94F6A589337C8
[youtube:tab] Downloading just the video z5rRZdiu1UE because of --no-playlist
[youtube] Extracting URL: https://www.youtube.com/watch?v=z5rRZdiu1UE
[youtube] z5rRZdiu1UE: Downloading webpage
[youtube] z5rRZdiu1UE: Downloading ios player API JSON
[youtube] z5rRZdiu1UE: Downloading mweb player API JSON
[youtube] z5rRZdiu1UE: Downloading m3u8 information
[MetadataParser] Parsed meta_album from '%(playlist_title)s': 'NA'
[info] z5rRZdiu1UE: Downloading 1 format(s): 251
[info] Downloading video thumbnail 42 ...
[info] Writing video thumbnail 42 to: Beastie Boys - Sabotage (Official Music Video) [z5rRZdiu1UE].webp
[download] Destination: Beastie Boys - Sabotage (Official Music Video) [z5rRZdiu1UE].webm
[download] 100% of    2.56MiB in 00:00:00 at 19.37MiB/s
[ExtractAudio] Destination: Beastie Boys - Sabotage (Official Music Video) [z5rRZdiu1UE].mp3
Deleting original file Beastie Boys - Sabotage (Official Music Video) [z5rRZdiu1UE].webm (pass -k to keep)
[Metadata] Adding metadata to "Beastie Boys - Sabotage (Official Music Video) [z5rRZdiu1UE].mp3"

Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

Thanks for the clarification!
Instead of --no-playlist, a parameter like --single-video(?) would be logical, but the playlist metadata is preserved in the stream, in cases where the information associated with the playlist is desired.
Or another solution would be to add another parameter like --metadata-playlist(?), which would keep the playlist metadata in the flow and could be used later. In this way, it would maintain compatibility, just need to add the logic of using a new parameter (option) to the code.
It's just a useful idea to be able to use metadata from playlists even in cases where only one video from the playlist is downloaded.
Maybe it would have been better to ask the question in a 'site-enhancement" thread and maybe someone/someday will implement this option!!?


Add initial support for hppa-unknown-linux-gnu

Add libc-files to allow rust to be used on the hppa/parisc platform.
Signed-off-by: Helge Deller<EMAIL_ADDRESS>

Please drop/close this pull request.
I'll open a new one with the build failure fixed.

Please drop/close this pull request. I'll open a new one with the build failure fixed.

You can just force-push your fixed commit to update this PR. No need to open a new one.


Abort install

Checklist

[ ] I read the README
[ ] I followed instructions to configure VPN clients
[ ] I searched existing Issues
[ ] This bug is about the VPN setup script, and not WireGuard VPN itself

Describe the issue
Installing WireGuard, please wait...

apt-get -yqq update
apt-get -yqq install wireguard qrencode
wireguard.sh: line 652: /etc/wireguard/wg0.conf: No such file or directory
chmod: cannot access '/etc/wireguard/wg0.conf': No such file or directory
systemctl enable --now wg-iptables.service
grep: /etc/wireguard/wg0.conf: No such file or directory
grep: /etc/wireguard/wg0.conf: No such file or directory
wireguard.sh: line 401: /etc/wireguard/wg0.conf: No such file or directory
grep: /etc/wireguard/wg0.conf: No such file or directory
grep: /etc/wireguard/wg0.conf: No such file or directory
wg: Key is not the correct length or format
grep: /etc/wireguard/wg0.conf: No such file or directory
grep: /etc/wireguard/wg0.conf: No such file or directory
systemctl enable --now<EMAIL_ADDRESS>
To Reproduce

Install WG
Deinstall WG
Install WG

Expected behavior
A clear and concise description of what you expected to happen.
Logs
Add error logs to help explain the problem, if applicable.
Server (please complete the following information)

OS: [e.g. Debian 11]
Hosting provider (if applicable): [e.g. GCP, AWS]

Client (please complete the following information)

Device: [e.g. iPhone 12]
OS: [e.g. iOS 15]

Additional context
Add any other context about the problem here.

@lexmaxak Hello! Thanks for reporting this issue. I could not reproduce this issue on a Ubuntu 22.04 or Debian 11 system. Most likely it's because the command apt-get -yqq install wireguard qrencode failed for some reason when you reinstalled WireGuard, and it's not an issue with the script itself.


How correctly update flow?

Hello,
updated flow-bin with yarn global add flow-bin@latest. yarn global ls shows<EMAIL_ADDRESS>flow version shows Flow, a static type checker for JavaScript, version 0.37.4. How update flow to latest version?
OS: Linux Ubuntu 16.04
Yarn: latest
P.S. I also did reboot several times...
Thanks for any help!

Fixed.

Check where is flow - whereis flow
Remove it - sudo rm -rf /usr/bin/flow
Add Flow - yarn global add flow-bin@latest

I don't remember if I installed Flow with NPM or no, even after deleting whole Node.js and NPM and reinstalling, it was always 0.37.4 version, but after that fix everything seems fine.

I think it is better to keep flow-bin as dev dependency in your project.


XLM Missing Though Confirmed Transaction

I tried to move my XLM from my ledger to coinbase. Ledger says it sent and was confirmed, yet the XLM is not in my coinbase wallet. The address was accurate. What happened?

Hi, did u find any solution?
same issue as me


gazelle_cabal fails to execute on cabal because of Paths_* files

Describe the bug
gazelle_cabal fails to execute on https://github.com/haskell/cabal with the following error:
gazelle: cabalscan: MissingModuleFile {modulePath = "Paths_Cabal", cabalFile = "/home/googleson78/git/cabal/Cabal/Cabal.cabal", componentName = "Cabal"}
gazelle: exit status 1

I believe this is because Paths_Cabal doesn't have an associated file, as it is autogenerated. Other files like this should have the same issue, but Paths_* is encountered much more often in my experience.
To Reproduce

Clone https://github.com/googleson78/cabal
Checkout 3749f6089a015bf18d661f89efa6672613210455
nix-shell
bazel run //:gazelle

Probably @facundominguez may know more about getting rid of Paths_* as he probably was running gazelle_cabal against cabal code. Essentially, I mean [this context](https://www.tweag.io/blog/2022-06-23-haskell-module.
BTW: I was encountering the same behavior in case of bazelifiyng example-servant-elm

When building Cabal, I wrote a Paths_Cabal.hs module instead of having cabal generate it:
https://github.com/facundominguez/cabal/commit/4e0ab0dbabe76824732a5746bc268b17312b65b1#diff-0028f8835685d6b965059b98ee873c6fbf4f1466097c0c82da6504d564aaa37c
Maybe we should list this workaround in the readme of gazelle_cabal. It doesn't look like generating the file with with gazelle_cabal would be easy.


[Bug][Team] The users.csv template downloaded is not encoded in UTF-8

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
A user wanted to use the team feature. He followed the guide and uploaded users.csv, and it turned out that the table.users in MySQL was empty. The data was not successfully imported.
The user then found out it was due to the CSV encoding format, the users.csv template he downloaded was not encoded in UTF-8.

What do you expect to happen

[ ] The templates should be in encoded in UTF-8 by default.
[ ] Add a line to the team doc to tell users that the CSV should be encoded in UTF-8.

How to reproduce
Follow the team doc to import team data.
Anything else
No response
Version
main
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

CSVs are plain-text files, it is up to applications (likely Excel in this case) to determine which encoding to use.

Use libre office instead: it would offer a dialog to choose the encoding
Manual choose encoding in Excel, you may google "how to" based on the version you are using

And I totally agree with the second point, we should update our document to stress the encoding is UTF-8 and what actions they should take.


#fix the resign.sh. get the wrong .app file name in the temp dir.

reason: app not the only file within the Payload directory,such as StoreKit folder.
fix: grep the first .app file in the temp dir.

@Acorld Thanks for the PR. The code seems to work as written but it would be great if you could clean this up a little bit according to the comments I left. :+1:

@asfalcone Thanks for your advises.
but i doubt that which is need to fix,the code or the comments?

@Acorld Thanks, the comments look much more helpful now! Please squash this down to 1 commit and it will be ready to merge :rocket:

@asfalcone df9d980 All done.

@Acorld Looks like there are still 2 commits on this branch. Are you able to squash the merge from master into the 1 commit?

@asfalcone Sorry for my merge commit.Done now.😊

Thanks @Acorld :tada:


Remove version: VMware.HorizonClient version 8.10.37628

Bad manifest version causing unnecessary headaches.
Microsoft Reviewers: Open in CodeFlow

Please note my PR to fix version - https://github.com/microsoft/winget-pkgs/pull/119123.
Thanks.


Adding Javascript Boilerplate

Creating a very basic JavaScript-Boilerplate.
Just do some npm install and npm run test.

Thx a lot. Really appreciated.


Chronos network bridge.

@enxebre Can you explain the motivation for this change?

Hi there!
First reason is that current approach did not work for me as chronos was trying to register continuously as it was been seen as localhost by mesos master.
Second reason is avoiding embracing network isolation by using "network": "BRIDGE"
https://github.com/docker/docker/issues/6401
https://github.com/docker/docker/issues/14767
Thanks!

You're right in that we want to try and avoid the scenario mentioned in those two docker tickets, however the current implementation of mesos requires that host networking be used when running a framework inside a docker container. This is due to the fact that libmesos opens a random socket in order to receive messages from the master.
Please double check your logs and ensure that when you have bridge networking enable for chronos you see a log line similar to the following in stderr:
sched.cpp:448] Framework registered with 20150723-182640-3540320266-5050-1210-0019

If you don't see this line, chronos didn't actually fully register as a framework with mesos and won't be able to receive resource offers.

yup, is definitely working for me when using bridge networking, you can see it in action at the minute here http://<IP_ADDRESS>:8080/ and http://<IP_ADDRESS>:5050  and http://<IP_ADDRESS>:31787/#jobs/test
But thanks to your comment just realised that might be due to the fact that in our scenario all containers are bind to a weave network (http://docs.weave.works/weave/latest_release/how-it-works.html). Does it make sense for you?

Ah yes, weave definitely changes the behavior.  Thanks for the additional details.  When running on DCOS without any extra networking layer the bridge networking option will not work, so I suspect this PR won't be merged since we won't be able to guarantee that weave will be running for all DCOS clusters.
I'm going to defer to @jsancio for the final decision though. Thanks for helping me understand.

Sure, that make sense. Sorry haven't reallised about weave was coming the game before.
Thank you!!

How about making network parameter overridable from the config.json file? Have you considered that posibility?


Fix the max ShardsNum

The max shard number of a collection in Milvus should be 64.

/assign @AnthonyTsu1984


Deploy RoadKill on Google Cloud. livenessProbe enabled?

Hello!!,
My company has been using this app for a long time, now I was instructed to migrate to the cloud (Google Cloud).
I was able to dockerize it and deploy it on GKE (Google Kubernetes Engine) however when I set up the Ingress, it requires me to properly declare the livenessProbe and readinessProbe.
Since the application is running fresh. when I do 'http:localhost:80/, automatically redirects me to /install endpoint.
What URL can I use to pass the liveness and readiness proof?
Thanks

I'm impressed you got a Windows container working on GKE!
Could you use /favicon.ico for the liveness? There isn't any health check feature as it's a pre-container or health check application.

Sure, thank you for your advice, I'll use that then. Thank you so much ;)


Feature Request - world save notifications

Feature Request - world save notifications
So that players know when the last time the world was saved (ie the server crashed, how much time did they lose... peace of mind... etc)

In 1.1.0


Chessmint does not work against bots

I have engine depth, threads, move and hints analysis and nothing shows up on the chessboard. maybe chess.com has patched chessmint? could you maybe release a new update or find a fix? i have tried reinstalling the extension and still no luck. thanks

Repository was updated 6 months ago last time. I think it's outdated 4 now


Health and note ratings, like - you know - the actual game for once

Work on this once menu scenes are somewhat good-looking and most other issues have been squashed.
Hey, maybe it'll be out before christmas :trollface:

It won't be out before christmas :trollface: :trollface: :trollface:

Okay, christmas 2023, calling it now! Also closing the issue, note ratings are pretty crappy, but in! ( 52558e6954a7c0c09109c4dab5825df582b13696 )


client update

fixes #37, #38, #39
This is just an fyi PR to close the 3 issues with context. The issues were resolved via https://github.com/hoodiehq/hoodie-account-client/releases/tag/v4.0.2 and https://github.com/hoodiehq/hoodie-account-client/releases/tag/v4.0.3

tests fail due to saucelabs hickups, tested locally with saucelabs a few times, all good


MoVQ implementation question

I have some questions regarding the implementation of MoVQ and would appreciate your clarification.
From the original MoVQ paper, it is mentioned that a multi-channle VQ is adopted.

However, the implementation of kandinsky3 does not involve any vector quantization operation:
class MoVQ(nn.Module):

    def __init__(self, generator_params):
        super().__init__()
        z_channels = generator_params["z_channels"]
        self.encoder = Encoder(**generator_params)
        self.quant_conv = torch.nn.Conv2d(z_channels, z_channels, 1)
        self.post_quant_conv = torch.nn.Conv2d(z_channels, z_channels, 1)
        self.decoder = Decoder(zq_ch=z_channels, **generator_params)

    @torch.no_grad()
    def encode(self, x):
        h = self.encoder(x)
        h = self.quant_conv(h)
        return h

    @torch.no_grad()
    def decode(self, quant):
        decoder_input = self.post_quant_conv(quant)
        decoded = self.decoder(decoder_input, quant)
        return decoded

May I ask if it is a misunderstanding on my part regarding MoVQ, or if Kandinsky has made some modifications to the implementation of MoVQ?

I've noticed the same issue. It appears that the author has just utilized the encoder and decoder network architecture from movqgan and has retrained a VAE model. I'm not sure if my understanding is correct, so I would appreciate clarification from the author.

After training, each token of the encoder's output is very close to a certain vector in the codebook. I've tried adding the VQ to refine the encoder's output and find the MSE between the features before VQ and after VQ is small, thus w/ VQ or w/o VQ produces nearly the same decoding results.


Cannot use variables as keys in Twig hashes

My only Twig environment is within a Symfony2 app, so I'm not sure how to reproduce this otherwise, but I'm unable to use a variable as a hash key.
{% set str = 'foo' %}
{% set hash = { str: 'bar' } %}

The above yields an exception: "A hash key must be a quoted string or a number (unexpected token "name" of value "str" in..."
Should this be supported?  I believe I have a legitimate use-case for it - I'm generating Symfony2 routes and am defining a common route parameter key as a Twig global.  Without this, I have to resort to hard-coding the parameter key.
Note: tested the above with a1c33ec0d3dd3927d6e5c87c533d8d882cb4f658 from today.

I'm experiencing same issue, I think this should be considired as an enhancement and variables should be allowed as keys
I have a search form and I want to pass it's values to a link like {{ path('_route', {form.name: form.vars.value}) }}
The app should be decoupled and it's correct to use form.name neither than tie it with a string.

@getme: An alternative solution would be to create a Twig helper function that creates the path() argument.


How to call 'new' method of GDScript class

I haven't idea, because typeof(someScript._new) is GDScript, not Variant. Also __traits(getMember, scr, "new") doesn't work.

Oh, I didn't notice that name conflict before. It's trying to use the static _new() internally called by memnew. This will be fixed in the next compatibility-breaking update, which removes the static _new(), since it's just a leftover from how the C++ bindings implemented Godot class construction (memnew!GDScript in Godot-D).
For now, here are two workarounds that call GDScript's new function properly:
Ref!GDScript someScript = ResourceLoader.load(gs!"res://some_gdscript.gd").as!GDScript;

// Option 1: call it through script API
Variant instance = someScript.call(gs!"new");
// Option 2: force it to use the correct `_new`, a template function
Variant instance = someScript._new!()();

print(instance);
instance.as!GodotObject.call(gs!"some_function", 1, 2, 3);

Thank you!


Revert "Temporarily remove compiling against non roslyn project referenc...

...es"
This reverts commit 088bcac02854aad665880b011d97a508976efda2.
/cc @davidfowl

:shipit:


torrential: 2.0.1 -> 3.0.0

https://github.com/davidmhewitt/torrential/compare/2.0.1...3.0.0
/cc https://github.com/davidmhewitt/torrential/pull/201
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

@ofborg build torrential


Retain server connection even when certain processes fail

The server should shutdown if ocamlmerlin fails but not if something less import like refmt fails. In the latter case, we should try to recover, notify the user, and keep running.

The server should shutdown if ocamlmerlin fails

@freebroccolo Why is this the case? Shouldn't the server try to restart ocamlmerlin?
A very common case I often run into is that I type the ñ key (Spanish keyboard, happens very often due to where that key is located 😁 ) which totally kills merlin, and thus the server. The outcome is that every time I accidentally press ñ, I have to restart vscode.


Support workspace dependencies

Rust 1.64 supports declaring cargo dependencies at the workspace level. We should take those into account when vendoring crates
https://blog.rust-lang.org/2022/09/22/Rust-1.64.0.html

Just to start a discussion on this, since we are running into this at Prisma and we want to start using crane. I tried using crane with a repo with workspace dependencies.
In the repo itself, no problem. It looks like cargo transparently does the job.
In another repo that depends on the repo with workspace dependencies via git, here's what we get:
@nix { "action": "setPhase", "phase": "unpackPhase" }
unpacking sources
unpacking source archive /nix/store/2yiwx5262v6hj2z17s0rg2fdd7jrdhqr-dummy-src
source root is dummy-src
@nix { "action": "setPhase", "phase": "patchPhase" }
patching sources
Executing configureCargoCommonVars
@nix { "action": "setPhase", "phase": "configurePhase" }
configuring
will append /build/dummy-src/.cargo-home/config.toml with contents of /nix/store/b3p3fckc4jx9x20xd8i2y9qydmky7bp7-vendor-cargo-deps/config.toml
default configurePhase, nothing to do
@nix { "action": "setPhase", "phase": "buildPhase" }
building
++ command cargo --version
cargo 1.64.0 (387270bc7 2022-09-16)
++ command cargo check --target=wasm32-unknown-unknown
error: failed to get `prisma-fmt` as a dependency of package `prisma-fmt-build v0.1.0 (/build/dummy-src)`

Caused by:
  failed to load source for dependency `prisma-fmt`

Caused by:
  Unable to update https://github.com/prisma/prisma-engines#18aec693

Caused by:
  failed to update replaced source https://github.com/prisma/prisma-engines#18aec693

Caused by:
  failed to parse manifest at `/nix/store/b3p3fckc4jx9x20xd8i2y9qydmky7bp7-vendor-cargo-deps/a0c466a33f02e3dbd1af7484529b17f40ec298f145519133ca66603a48fab109/native-types-0.1.0/Cargo.toml`

Caused by:
  failed to find a workspace root

I interpret this as cargo looking for the workspace Cargo.toml to resolve the dependencies of vendored crates, but since the workspace crates are vendored individually, the workspace root Cargo.toml is not available.
I'm not super familiar with crane internals, so it may not be a good idea, but could the fix be as simple as detecting which crates use workspace dependencies in vendorGitDeps.nix, then copying the workspace dependencies from the root Cargo.toml in there so the vendored deps have all they need?

@tomhoule could you provide a flake.nix with which I can reproduce the issue? (bonus points if it's a minimal setup)

Wonder if this is an issue with cargo itself. Looks like cargo vendor does not (yet?) work with dependencies which use workspace inheritance: https://github.com/rust-lang/cargo/issues/11192
We don't use cargo vendor directly, but its behavior is a reference implementation for sure. If there is a way to massage the crate sources to make cargo work I'm all for it, but I don't know off the top of my head what that format would be

Here's the minimal setup: https://github.com/tomhoule/crane-workspace-deps-repro — it depends on a large workspace so it takes a while to vendor the dependencies but it's minimal otherwise. I expect the result would be the same replacing the dependency on prisma-fmt with any git dependency with workspace dependencies.

We use latest crane git commit.

There's progress on the cargo vendor issue: https://github.com/rust-lang/cargo/pull/11414


Request: Update to 1.20

I would like for the mod to be updated to 1.20

https://modrinth.com/mod/customhud/version/3.1.0+1.20

Forgot to comment here, thanks!


Negative Binary Literals are Positive

When implementing amzn/ion-rust#214, I noticed in this commit that equiv testing was failing for binary notation of integers.
Specifically these two skip-list files:
    // ion-c seems to have a problem with negative binary literals amzn/ion-c#235
    "ion-tests/iontestdata/good/equivs/binaryInts.ion",
    "ion-tests/iontestdata/good/equivs/intsWithUnderscores.ion",

When running our Element tests in ion-rs, the values parse out as positive for the binary literals.  I will add a direct reproduction in ion-c when I get a chance, but I wanted to make sure I cut the issue so we don't lose track of it.

Verified in a unit test in IonC:
// reproduction for amzn/ion-c#235
TEST(IonTextInt, BinaryLiterals) {
    const char *ion_text = "-0b100";
    hREADER  reader;
    ION_TYPE type;
    int64_t value;

    ION_ASSERT_OK(ion_test_new_text_reader(ion_text, &reader));
    ION_ASSERT_OK(ion_reader_next(reader, &type));
    ASSERT_EQ(tid_INT, type);
    ION_ASSERT_OK(ion_reader_read_int64(reader, &value));
    ASSERT_EQ(-4, value);
}

Produces the following failure:
Failure
      Expected: -4
To be equal to: value
      Which is: 4


Windows setup/start script for headless

I don't know if anyone would be interested in this but I made a setup/start script for my friends and I so we can easily farm multiple accounts (works with one). Makes it fairly simple to get the headless version up and running.
https://github.com/MainFighter/salien-bot/tree/winstartscript
Sorry if I shouldn't be posting it here, you can remove it if you like.

`@echo off
title Salien-bot starter
echo "Steam Summer Sale (Salien)"
echo "------------------------------------"
set botpath="c:\path_to_bot_folder"
set botfoldername="salien-bot-master"
cd %botpath%/%botfoldername%
node headless
pause`
why not just use this instead? :o main page of bot has the instructions, the above code saved into a file like bot.bat can start up your bot, you only have to change two things (or 1).. the bot folders path.
easy..

yes, but not all of us having git and extra commands. like me, i just installed nodejs just cuz of this.
but yeah, dont get me wrong, thx for the commitment, thx for writing such, but in my opinion its too much thing to just install. :)
and btw i have checked your code :)
auto update is a nice feature though, so thats ++.

i can't take your code myself, but if you'd like to make a pr others and i can decide if it would be a good addition.


add my pixel

Checklist

[x ] I ran npm test locally and it passed without errors.
[x ] I only edited the _data/pixels.json file.
[x ] I entered the username in the pixels.json that I'm also using to create this pull request.
[x ] I acknowledge that all my contributions will be made under the project's license.

Hey @AnTheMaker! Thank you so much for your contribution. Unfortunately the PR couldn't be merged yet because your coordinates have to be less than 40. Could you please update your pixel to another coordinate? Thank you!

Done! I changed the coordinates of the pixel.
Greetings, An


Include the current git version in comipiled jars

This is an ant script which takes the output of git rev-parse head and shoves it in any compiled jars.
For the less git-inclined, rev-parse head returns the current git commit hash.
The process is all automated with ant, and shouldn't require any special code by whomever chooses to deploy.

Addresses issue #23.

Latest commits refactored to work with tags in addition to just the SHA hash. (describe --tags vs. rev-parse head)


Compact resources

Solves https://github.com/androguard/androguard/issues/1066
Offers support for FLAG_OFFSET16 and FLAG_COMPACT.
It is a continuation of a quick and dirty approach that was created in this branch

I didn't think this was complete yet?


fix typos

fix typos

Hey, @omahs!
We really appreciate your contribution, but this is not a substantive enough PR to be merged, per our Contributing Guidelines.
We would welcome your participation in broadly improving the quality of our documentation. We are also always eager for new developers to contribute, so please have a peek at open issues and see if you feel comfortable tackling any of them. New opportunities to contribute also emerge almost daily in our Discord server as users report issues or feature requests.
We hope to see you contributing to gpt-engineer soon!


class "specification"

The class is very general, and was located in incorrect place.
bfo has "action spe", "conditional spe", "data format spe", "object spe", and "plan spe" under "directive information entity". why a general class of "specification" created as sub class of "plan specification"?

Also object property of "has specification/specification of" can have broader domain and range. for example range for "has specification" can be "directive information entity" or even ICE


rspack.conf.js添加rsbuild-plugin-vue2-jsx 运行时报错

Rsbuild plugin is not supported by Rspack. You can use this plugin in Rsbuild projects.


Protect runtime RPC with authorization

On query requests to the runtime RPC, require that a user has the required roles according to the page the query is called from.
Includes a few refactors too.

I don't think we need to prioritize this until we've taken the time to change the dataprovider to not use "resource-file+function"  to call its method, but rather "page-file+datagrid". Because with the current method it's possible for every user to call arbitrary functions anyway, even if you change execQuery.
I'm not sure we should change the rpc implementation for this. I think we could put authentication information in the server context so that users can get this info with getContext(). We should be able to piggy-back on this system to implement strict access enforcement on the server functions. But again, as long as datasources provide an interface that allows calling arbitrary functions, this last thing is pretty much wasted effort.

So I'd say:

Forget about RBAC in the rpc layer for now, or refactor the dataprovider first instead.
Expand getContext with auth information. this could be a nice user facing feature.

I don't think we need to prioritize this until we've taken the time to change the dataprovider to not use "resource-file+function"  to call its method, but rather "page-file+datagrid". Because with the current method it's possible for every user to call arbitrary functions anyway, even if you change execQuery.
I'm not sure we should change the rpc implementation for this. I think we could put authentication information in the server context so that users can get this info with getContext(). We should be able to piggy-back on this system to implement strict access enforcement on the server functions. But again, as long as datasources provide an interface that allows calling arbitrary functions, this last thing is pretty much wasted effort.

So I'd say:

Forget about RBAC in the rpc layer for now, or refactor the dataprovider first instead.
Expand getContext with auth information. this could be a nice user facing feature.

Sounds good, thanks! I'll probably close this one for now then, and can add to the context in a further PR after the others are merged and authentication is released.

Closing, I'll create issues later for the features that were suggested instead of this implementation.


Azure Ai Text Analytics Readme Issue

Section link:

Reason:
The TextAnalyticsApiKeyCredential is not in the  @azure/ai-text-analytics .
Suggestion:
Update to:
const { TextAnalyticsClient, AzureKeyCredential } = require("@azure/ai-text-analytics");
const client = new TextAnalyticsClient(
  "<endpoint>",
  new AzureKeyCredential("<API key>")
);

@jongio for notification.

Thanks for catching this @v-jiaodi. Can you please send a PR to fix this?

@ramya-rao-a - Sure, I will create a PR to fix it.


【requirement】open notes with obsidian after search

when i search notes inside my vaults, can i just open it in obsidian instead of open in my default editor?

I'm not sure if I understand your question, so san you explain a little more what you're trying to accomplish? By default, if a note/file is in your Obsidian vault, it opens that file/note in Obsidian. It'll only open it in your default editor if the note/file isn't inside your Obsidian vault. Or maybe I'm misunderstanding your question?

I have the same issue.
After having set up the workflow and solving the "search vault not working" issue (https://github.com/hauselin/obsidian-alfred/issues/15#issuecomment-923818368), I tried to search for a note in my vault by doing:

opening Alfred entry window and typing "os" -> first result in list is "search within vaults (press enter to continue)"
pressing "enter" -> entry window is cleared,  one entry in list reading "search markdown files (search for ... + enter)"
typing a string -> list shows files both within the vault as well as files outside the vault
selecting a file from the list that is inside the vault + "enter" -> the file is opened in "iA Writer", not in Obsidian

Thanks,
Guy
Obsidian: 0.12.15
Alfred+Powerpack: 4.5.1
alfred-obsidian workflow: 0.3.7
macOS: 10.13.6, 11.6

Same here. Opening a note with the workflow opens it in Typora, not Obsidian, after correcting for the "os" bug.

Same issue here. I am a new user to this workflow. Is there any additional setup required to get it to open search results in Obsidian?


Request: make ::new() public

Usually tide does this for you once listening, and passes the request through middleware and to an endpoint.
However, if you wish to call an endpoint manually, then this is useful.
(We ran into this internally.)
I realize this may not be considered desirable, so... discuss, I suppose.

Could you describe the use case for calling an endpoint manually better?

I'm trying to understand why server.respond(http_types::Request) is insufficient

Hmm, I hadn't thought of that. Seems like unnecessary routing though.

does it accomplish the goal though? increasing surface area with an api that might seem like a good place to start seems like something to avoid if at all possible

Alternatively, since this is advanced usage,
let req: tide::Request<()> = tide::http::Request::new(
    tide::http::Method::Get,
    tide::http::Url::parse("http://_").unwrap(),
)
.into();

Ah. The Into is smart. I'm surprised I forgot about that. Yeah that means we don't need this.


fix: bump sc4s/splunk versions

:tada: This PR is included in version 1.8.23 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


feat: export parse function

So consumers can leverage the SDK to build DocumentBuilders from files

I also think it can be useful have that option.
In the future, I think it would be nice to export util functions like this one under @coveo/push-api-client/help


[Identity] Loading the tenant from the VSCode settings file

This approach comes from what other languages are doing. This should work, but I haven't been able to test it. I'll make sure I can test this, and I'll add some unit tests, before I get this PR out of draft.

We met offline re: test and decided it's good to go ahead and merge.

Looks like you might have deleted the spec file in your latest changes?

I deleted the test file in the wrong commit, but the test file was of doubtful value at best. I asked the team and decided to leave it out of this PR.


chore(deps): update dependency eslint-plugin-import to v2.20.1

This PR contains the following updates:

Package
Type
Update
Change

eslint-plugin-import
devDependencies
minor
2.19.1 -> 2.20.1

Release Notes

benmosher/eslint-plugin-import
v2.20.1
Compare Source
Fixed

[export]: Handle function overloading in *.d.ts ([#1619], thanks [@IvanGoncharov])
[no-absolute-path]: fix a crash with invalid import syntax ([#1616], thanks [@ljharb])
[import/external-module-folders setting] now correctly works with directories containing modules symlinked from node_modules ([#1605], thanks [@skozin])
[extensions]: for invalid code where name does not exist, do not crash ([#1613], thanks [@ljharb])
[extentions]: Fix scope regex ([#1611], thanks [@yordis])
[no-duplicates]: allow duplicate imports if one is a namespace and the other not ([#1612], thanks [@sveyret])
Add some missing rule meta schemas and types ([#1620], thanks [@bmish])
[named]: for importing from a module which re-exports named exports from a node_modules module ([#1569], [#1447], thanks [@redbugz], [@kentcdodds])
[order]: Fix alphabetize for mixed requires and imports ([#5625], thanks [@wschurman])

Changed

[import/external-module-folders setting] behavior is more strict now: it will only match complete path segments ([#1605], thanks [@skozin])
[meta] fix "files" field to include/exclude the proper files ([#1635], thanks [@ljharb])

v2.20.0
Compare Source
Added

[order]: added caseInsensitive as an additional option to alphabetize ([#1586], thanks [@dbrewer5])
[no-restricted-paths]: New except option per zone, allowing exceptions to be defined for a restricted zone ([#1238], thanks [@rsolomon])
[order]: add option pathGroupsExcludedImportTypes to allow ordering of external import types ([#1565], thanks [@Mairu])

Fixed

[no-unused-modules]: fix usage of import/extensions settings ([#1560], thanks [@stekycz])
[extensions]: ignore non-main modules ([#1563], thanks [@saschanaz])
TypeScript config: lookup for external modules in @types folder ([#1526], thanks [@joaovieira])
[no-extraneous-dependencies]: ensure node.source is truthy ([#1589], thanks [@ljharb])
[extensions]: Ignore query strings when checking for extensions ([#1572], thanks [@pcorpet])

Docs

[extensions]: improve ignorePackages docs ([#1248], thanks [@ivo-stefchev])

Renovate configuration
:date: Schedule: At any time (no schedule defined).
:vertical_traffic_light: Automerge: Disabled by config. Please merge this manually once you are satisfied.
:recycle: Rebasing: Whenever PR becomes conflicted, or if you modify the PR title to begin with "rebase!".
:no_bell: Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

:tada: This PR is included in version 2.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


non-classifier case in coremltools4.0+

❓Question
Is it possible to convert a pytorch model withtout classification layer into mlmodel? I don't know how to do it because in the example,  I am asked to specify the label.
System Information

If applicable

Just don't provide the classifier_config argument to the ct.convert call, as shown here

Hi, have you ever encountered this situation?

One day ago, I convert my pytorch model to onnx. It works well. But now I use your tool to do the conversion directly. It seems not good.

This is the complete error.
TypeError: Input strides has type <class 'coremltools.converters.mil.mil.types.type_tensor.tensor..tensor'> not compatible with expected type IntTensorInputType

it works well until the 7-th part

I run the whole thing on colab.

Hi, have you ever encountered this situation?
[image: image]
https://user-images.githubusercontent.com/30463291/86051128-d90cd880-ba4c-11ea-88aa-c3038dafa5d6.png
One day ago, I convert my pytorch model to onnx. It works well. But now I
use your tool to do the conversion directly. It seems not good.
This is the complete error.
TypeError: Input strides has type <class
'coremltools.converters.mil.mil.types.type_tensor.tensor..tensor'> not
compatible with expected type IntTensorInputType
it works well until the 7-th part
[image: image]
https://user-images.githubusercontent.com/30463291/86051706-be872f00-ba4d-11ea-9354-b908148d270e.png
On Mon, 29 Jun 2020 at 21:03, Aseem Wadhwa<EMAIL_ADDRESS>wrote:

Just don't provide the classifier_config argument to the ct.convert call,
as shown here
https://coremltools.readme.io/docs/unified-conversion-api#conversion-from-pytorch
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/onnx/onnx-coreml/issues/581#issuecomment-651329773,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHINKO6DHUZADM3JUWXVDR3RZDXRPANCNFSM4OLRJ5WA
.

@huangguanbin can you please provide the pytorch model, which can reproduce this error? Otherwise its hard to debug the error.

no problem. I will do it right now. Give me a second.
On Mon, Jun 29, 2020 at 11:23 PM Aseem Wadhwa<EMAIL_ADDRESS>wrote:

@huangguanbin https://github.com/huangguanbin can you please provide
the pytorch model, which can reproduce this error? Otherwise its hard to
debug the error.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/onnx/onnx-coreml/issues/581#issuecomment-651401396,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHINKO26V5E5WAGMSM5RQ23RZEH63ANCNFSM4OLRJ5WA
.

https://colab.research.google.com/drive/1hfv_Qbl6ZHDbXq_HB9bo3tV89KM7Skoq?usp=sharing
This is the notebook where you can reproduce the bug.
And you can download the pytorch model by clicking the shared link. It
contains a zip file.
https://drive.google.com/file/d/1etvQPPGlr4YxBEsCgOeduicErYN18BLg/view?usp=sharing
All you need to do is just go through the notebook. Make sure the zip file
is in the /content folder. The checkpoint(aka the pytorch model itself) is
in the /AlignedReID-master/util/log if you need to access it.
On Mon, Jun 29, 2020 at 11:24 PM 黄冠斌<EMAIL_ADDRESS>wrote:

no problem. I will do it right now. Give me a second.
On Mon, Jun 29, 2020 at 11:23 PM Aseem Wadhwa<EMAIL_ADDRESS>wrote:

@huangguanbin https://github.com/huangguanbin can you please provide
the pytorch model, which can reproduce this error? Otherwise its hard to
debug the error.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/onnx/onnx-coreml/issues/581#issuecomment-651401396,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHINKO26V5E5WAGMSM5RQ23RZEH63ANCNFSM4OLRJ5WA
.

If you encountered any issue to run my code, plz let me know ASAP. I will reply to you on time.

The colab is not accessible...

Is the zip accessible "?
On Mon, 29 Jun 2020 at 23:52, Aseem Wadhwa<EMAIL_ADDRESS>wrote:

The colab is not accessible...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/onnx/onnx-coreml/issues/581#issuecomment-651409968,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHINKOZJCTOK4TH6Z3O5AVLRZELKBANCNFSM4OLRJ5WA
.

https://colab.research.google.com/drive/1hfv_Qbl6ZHDbXq_HB9bo3tV89KM7Skoq?usp=sharing
try this one.
On Mon, Jun 29, 2020 at 11:52 PM 黄冠斌<EMAIL_ADDRESS>wrote:

Is the zip accessible "?
On Mon, 29 Jun 2020 at 23:52, Aseem Wadhwa<EMAIL_ADDRESS>wrote:

The colab is not accessible...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/onnx/onnx-coreml/issues/581#issuecomment-651409968,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHINKOZJCTOK4TH6Z3O5AVLRZELKBANCNFSM4OLRJ5WA
.

if it is not accessible again, copy the code below onto colab.
!unzip AlignedReID-master.zip
!pip install coremltools==4.0b1
%cd AlignedReID-master/
import torch
from util.FeatureExtractor import FeatureExtractor
from torchvision import transforms
from IPython import embed
import models
from scipy.spatial.distance import cosine, euclidean
from  util.utils import *
from sklearn.preprocessing import normalize
def pool2d(tensor, type= 'max'):
sz = tensor.size()
if type == 'max':
x = torch.nn.functional.max_pool2d(tensor, kernel_size=(int(sz[2]/8
), sz[3]))
if type == 'mean':
x = torch.nn.functional.mean_pool2d(tensor, kernel_size=(int(sz[2]/8
), sz[3]))
x = x[0].cpu().data.numpy()
x = np.transpose(x,(2,1,0))[0]
return x
if name == 'main':
os.environ['CUDA_VISIBLE_DEVICES'] = "0"
use_gpu = torch.cuda.is_available()
if torch.cuda.is_available():
map_location = lambda storage, loc: storage.cuda()
else:
map_location = 'cpu'
model = models.init_model(name='resnet50', num_classes=751, loss={
'softmax', 'metric'}, use_gpu=use_gpu,aligned=True)
checkpoint = torch.load(
"/content/AlignedReID-master/util/log/checkpoint_ep300.pth.tar"
, map_location=map_location)
model.load_state_dict(checkpoint['state_dict'])
model.eval()
Trace with random data
example_input = torch.rand(1, 3, 256,128)
traced_model = torch.jit.trace(model, example_input)
traced_model.save("reid.pt")
import coremltools as ct
Convert the saved PyTorch model to Core ML
mlmodel = ct.convert("reid.pt",
inputs=[ct.TensorType(shape=(1, 3, 256, 128))])
On Mon, Jun 29, 2020 at 11:54 PM 黄冠斌<EMAIL_ADDRESS>wrote:

https://colab.research.google.com/drive/1hfv_Qbl6ZHDbXq_HB9bo3tV89KM7Skoq?usp=sharing
try this one.
On Mon, Jun 29, 2020 at 11:52 PM 黄冠斌<EMAIL_ADDRESS>wrote:

Is the zip accessible "?
On Mon, 29 Jun 2020 at 23:52, Aseem Wadhwa<EMAIL_ADDRESS>wrote:

The colab is not accessible...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/onnx/onnx-coreml/issues/581#issuecomment-651409968,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHINKOZJCTOK4TH6Z3O5AVLRZELKBANCNFSM4OLRJ5WA
.

I changed the link of zip file by mistake. If you want to download it again. use this one.   https://drive.google.com/file/d/1ufh5kndXMdWfKPRT85dDtEwhZ1MXnPg7/view?usp=sharing

I would appreciate it if you can also check how precise the conversion is.
Because some say conversion leads to loss of precision. That means given
same input matrix but get different output matrix. THANK YOU!!
On Mon, Jun 29, 2020 at 11:55 PM 黄冠斌<EMAIL_ADDRESS>wrote:

if it is not accessible again, copy the code below onto colab.
!unzip AlignedReID-master.zip
!pip install coremltools==4.0b1
%cd AlignedReID-master/
import torch
from util.FeatureExtractor import FeatureExtractor
from torchvision import transforms
from IPython import embed
import models
from scipy.spatial.distance import cosine, euclidean
from  util.utils import *
from sklearn.preprocessing import normalize
def pool2d(tensor, type= 'max'):
sz = tensor.size()
if type == 'max':
x = torch.nn.functional.max_pool2d(tensor, kernel_size=(int(sz[2]/8
), sz[3]))
if type == 'mean':
x = torch.nn.functional.mean_pool2d(tensor, kernel_size=(int(sz[2]/8
), sz[3]))
x = x[0].cpu().data.numpy()
x = np.transpose(x,(2,1,0))[0]
return x
if name == 'main':
os.environ['CUDA_VISIBLE_DEVICES'] = "0"
use_gpu = torch.cuda.is_available()
if torch.cuda.is_available():
map_location = lambda storage, loc: storage.cuda()
else:
map_location = 'cpu'
model = models.init_model(name='resnet50', num_classes=751, loss={
'softmax', 'metric'}, use_gpu=use_gpu,aligned=True)
checkpoint = torch.load(
"/content/AlignedReID-master/util/log/checkpoint_ep300.pth.tar"
, map_location=map_location)
model.load_state_dict(checkpoint['state_dict'])
model.eval()
Trace with random data
example_input = torch.rand(1, 3, 256,128)
traced_model = torch.jit.trace(model, example_input)
traced_model.save("reid.pt")
import coremltools as ct
Convert the saved PyTorch model to Core ML
mlmodel = ct.convert("reid.pt",
inputs=[ct.TensorType(shape=(1, 3, 256, 128))])
On Mon, Jun 29, 2020 at 11:54 PM 黄冠斌<EMAIL_ADDRESS>wrote:

https://colab.research.google.com/drive/1hfv_Qbl6ZHDbXq_HB9bo3tV89KM7Skoq?usp=sharing
try this one.
On Mon, Jun 29, 2020 at 11:52 PM 黄冠斌<EMAIL_ADDRESS>wrote:

Is the zip accessible "?
On Mon, 29 Jun 2020 at 23:52, Aseem Wadhwa<EMAIL_ADDRESS>wrote:

The colab is not accessible...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/onnx/onnx-coreml/issues/581#issuecomment-651409968,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHINKOZJCTOK4TH6Z3O5AVLRZELKBANCNFSM4OLRJ5WA
.

Any progress on this?
On Tue, 30 Jun 2020 at 01:13, 黄冠斌<EMAIL_ADDRESS>wrote:

I would appreciate it if you can also check how precise the conversion is.
Because some say conversion leads to loss of precision. That means given
same input matrix but get different output matrix. THANK YOU!!
On Mon, Jun 29, 2020 at 11:55 PM 黄冠斌<EMAIL_ADDRESS>wrote:

if it is not accessible again, copy the code below onto colab.
!unzip AlignedReID-master.zip
!pip install coremltools==4.0b1
%cd AlignedReID-master/
import torch
from util.FeatureExtractor import FeatureExtractor
from torchvision import transforms
from IPython import embed
import models
from scipy.spatial.distance import cosine, euclidean
from  util.utils import *
from sklearn.preprocessing import normalize
def pool2d(tensor, type= 'max'):
sz = tensor.size()
if type == 'max':
x = torch.nn.functional.max_pool2d(tensor, kernel_size=(int(sz[2]/8
), sz[3]))
if type == 'mean':
x = torch.nn.functional.mean_pool2d(tensor, kernel_size=(int(sz[2]/
8), sz[3]))
x = x[0].cpu().data.numpy()
x = np.transpose(x,(2,1,0))[0]
return x
if name == 'main':
os.environ['CUDA_VISIBLE_DEVICES'] = "0"
use_gpu = torch.cuda.is_available()
if torch.cuda.is_available():
map_location = lambda storage, loc: storage.cuda()
else:
map_location = 'cpu'
model = models.init_model(name='resnet50', num_classes=751, loss={
'softmax', 'metric'}, use_gpu=use_gpu,aligned=True)
checkpoint = torch.load(
"/content/AlignedReID-master/util/log/checkpoint_ep300.pth.tar"
, map_location=map_location)
model.load_state_dict(checkpoint['state_dict'])
model.eval()
Trace with random data
example_input = torch.rand(1, 3, 256,128)
traced_model = torch.jit.trace(model, example_input)
traced_model.save("reid.pt")
import coremltools as ct
Convert the saved PyTorch model to Core ML
mlmodel = ct.convert("reid.pt",
inputs=[ct.TensorType(shape=(1, 3, 256, 128))])
On Mon, Jun 29, 2020 at 11:54 PM 黄冠斌<EMAIL_ADDRESS>wrote:

https://colab.research.google.com/drive/1hfv_Qbl6ZHDbXq_HB9bo3tV89KM7Skoq?usp=sharing
try this one.
On Mon, Jun 29, 2020 at 11:52 PM 黄冠斌<EMAIL_ADDRESS>wrote:

Is the zip accessible "?
On Mon, 29 Jun 2020 at 23:52, Aseem Wadhwa<EMAIL_ADDRESS>wrote:

The colab is not accessible...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/onnx/onnx-coreml/issues/581#issuecomment-651409968,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHINKOZJCTOK4TH6Z3O5AVLRZELKBANCNFSM4OLRJ5WA
.

I had the same issue (TypeError: Input strides has type <class
'coremltools.converters.mil.mil.types.type_tensor.tensor..tensor'> not
compatible with expected type IntTensorInputType).
I am wondering if it is the issue related compatibility between coreML tensor and torch Tensor?

any progress?

i had same issue:
TypeError: Input strides has type <class 'coremltools.converters.mil.mil.types.type_tensor.tensor..tensor'> not compatible with expected type IntTensorInputType

Seems like the error message for type mismatch of input tensors is not very descriptive.
@quangphap208 @JialeHu Which models are you hitting this error on? Since type mismatch can happen in any model for different reasons. Some of the them might be bugs but others might be genuine errors. This needs to investigated on a model basis.

@huangguanbin Can you please re-convert your model after incorporating this following change?
Replace this line https://github.com/apple/coremltools/blob/e87f1d0d6ca1dc9cd812f14f777ead0bba70bf43/coremltools/converters/mil/frontend/torch/ops.py#L1421
with following code snippet
    strides = inputs[2]
    if strides.op.op_type == "const"  and (not strides.val):
        strides = mb.const(val=kernel_sizes.val, name=strides.name)

Since this issue is not related to ONNX. For PyTorch related issues, please use coremltools repository https://github.com/apple/coremltools.

@huangguanbin This PR https://github.com/apple/coremltools/pull/769 fixes the reported issue. Please verify if it solves your problem.

@huangguanbin This PR apple/coremltools#769 fixes the reported issue. Please verify if it solves your problem.

This solves the issue for me. Thanks a lot.

strides = inputs[2]
if strides.op.op_type == "const"  and (not strides.val):
    strides = mb.const(val=kernel_sizes.val, name=strides.name)

I tried what you said, but it is not working yet.

I checked the latest notification. But I'm not sure what i should modify.

@JialeHu


Document how to load Images, MP3s and OGG Vorbis files at run-time

I put the descriptions everywhere I think somebody could come across, when trying to load an unimported file.

Thanks!


DependencyTagConvertible for better typed Tags

This offers to remove Tag from register and resolve APIs  in favour of DependencyTagConvertible protocol, which may represent Tag itself as well as any other type conforming to this protocol.
Specifically, this allows me to use custom enums (conforming to DependencyTagConvertible) as tags instead of String or Int literals, making it more type-safe.
I modified SampleApp to use such an enum for tags. see here
At the same time, it is still possible to use String and Int literals as tags, so this PR adds more flexibility retaining existing functionality

I think we discussed this possibility at some point with @ilyapuchka
I was in favor of using a more generic type, but in the end using protocols lead to introducing PATs (Protocol with Associated Types) which led to a bunch of issues for some use cases, like you can't create an array of PATs in Swift by design…
Have to recheck that now that the codebase evolved, maybe the issue with PATs is not that string anymore, but I know we thought about it and had a reason not to do it at that time…

Ah found it: see discussion in #3

Thanks for PR, @gavrix!
Actually I personally gave up on that feature. It is kind of convenient, but does not bring any significant value except passing some type (probably enum case) without calling tag or toTag() or whatever on the instance of it. But at the same time it complicates the API adding one more abstraction (tag convertible protocol).
I didn't run the code, but it looks for me like it's a breaking change - before we had DependencyContainer.Tag, now - DependencyTag. Is it? If we add this feature I would personally prefer not to bump the version just for the sake of it. That's my main concern about it. Even if we move Tag enum from DependencyContainer scope (but I would prefer to keep it there - I'm not a huge fan of that, just to not bump version) I would prefer to keep the name as Tag (in case of ambiguity users can use it like Dip.Tag). Having DependencyTag and DefinitionKey types can cause questions. And regarding toTag method - having tag property looks more consistent with CustomStringConvertible protocol.

Looks like I need to fix some tests to these changes.
The main idea why I think this is a better API is that from API user perspective I have no idea how tags are organized inside Dip, but I have to provide it either a string or int literal, or a custom conversion to DependencyContainer.Tag.
Literals are extremely unsafe: you can make typo, you can break something during refactoring without knowing it etc. Everyone on swift tries to replace raw literals usage with types, mostly enums.
Regarding associated types — yeah, known problem, the solution is using type erasure technique. But my implementation doesn't use associated types.
While is it still possible to have all possible tags listed in enum in my client code and have function converting that enum to DependencyContainer.Tag instance, uneducated users will continue using raw literals (btw, following examples you gave) producing low quality swift code.
Basically what I propose is to integrate that function I mentioned (converting my custom tags enum to DependencyContainer.Tag) into Dip. It is probably worth removing possibility to provide raw literal as a tag instance altogether (I intentionally preserved that possibility).

@gavrix can you please look into failing tests and fix them?
Also please add type alias like this:
class DependencyContainer {
  typealias Tag = DefinitionTag
}

This way the change will not break code that explicitly used DependencyContainer.Tag. If type aliases support @available attribute please also add it with a note that it was deprecated in favour of new type.
Regarding new type for tag I still don't feel DependencyTag is a good name. Maybe it would be better to use just Tag in global scope, or move it to DefinitionKey scope, so that the type would be DefinitionKey.Tag. Just DefinitionTag may also work, but it can cause confusion with DefinitionKey.

Sure, let me take a look at it.

Sure, let me take a look at it.

@gavrix Thanks!


[SP-4463] Backport of PDI-17436 - Scheduling a transformation more th…

…an once causes the logs to display incorrectly (7.1 Suite)
@cravobranco @ricardosilva88

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
ant -Dtestreports.dir=bin/reports/unit-test -f engine/build.xml clean-all resolve jacoco && ant -f engine/build.xml checkstyle publish-local

Cleanup Commands
rm -r ~/.ivy2/local || echo no publish local to remove

Changed files
  engine/src/org/pentaho/di/trans/Trans.java
  engine/test-src/org/pentaho/di/trans/TransTest.java

Newly Fixed Tests:
org.pentaho.di.trans.steps.textfileinput.TextFileInputMetaLoadSaveTest.testSerialization:
:large_blue_circle: java.lang.RuntimeException
java.lang.RuntimeException: Error validating dateFormatLocale
	at org.pentaho.di.trans.steps.loadsave.LoadSaveTester.validateLoadedMeta(LoadSaveTester.java:176)
	at org.pentaho.di.trans.steps.loadsave.LoadSaveTester.testMixedXmlRepoRoundTrip(LoadSaveTester.java:270)
	at org.pentaho.di.trans.steps.loadsave.LoadSaveTester.testSerialization(LoadSaveTester.java:185)
	at org.pentaho.di.trans.steps.textfileinput.TextFileInputMetaLoadSaveTest.testSerialization(TextFileInputMetaLoadSaveTest.java:167)
Ca

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes (click to expand)
org.pentaho.di.trans.step.StepMeta

Branch Change: -0.7% :small_red_triangle_down:
Complexity Change: -0.6% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:
Line Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.checksum.CheckSumMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.ldapoutput.LDAPOutputMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -2.2% :small_red_triangle_down:
Complexity Change: -2.6% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:
Line Change: -0.4% :small_red_triangle_down:

org.pentaho.di.trans.steps.processfiles.ProcessFilesMeta

Branch Change: -3.5% :small_red_triangle_down:
Complexity Change: -2.1% :small_red_triangle_down:
Instruction Change: -0.4% :small_red_triangle_down:

org.pentaho.di.trans.steps.rowgenerator.RowGeneratorMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.symmetriccrypto.symmetriccryptotrans.SymmetricCryptoTransMeta

Branch Change: -3.3% :small_red_triangle_down:
Complexity Change: -1.8% :small_red_triangle_down:
Instruction Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.uniquerows.UniqueRowsMeta

Instruction Change: -0.1% :small_red_triangle_down:


[WIP] Update Bootstrap to v4 beta

What still needs to be done before we can merge this? Is there anything I can help out on?

@gregcarlin thanks for seeing this through and sorry as always for my absence. Great work!


64 bytes private key for Account

I am initing my account object as follows
private_key = bytes([89,136,139,8,93,189,44,2, ...... , 122,168])
payer = Account(private_key)

but got the following error:
  File "/Users/yurenji/.pyenv/versions/yurenji3.8/lib/python3.8/site-packages/solana/account.py", line 29, in __init__
    self._secret = public.PrivateKey(key) if key else public.PrivateKey.generate()
  File "/Users/yurenji/.pyenv/versions/yurenji3.8/lib/python3.8/site-packages/nacl/public.py", line 89, in __init__
    raise exc.TypeError(("PrivateKey must be created from a {0} "
nacl.exceptions.TypeError: PrivateKey must be created from a 32 bytes long raw secret key

It seems the private key must be 32 bytes. My account is created with solfare.com, and the private key it generates is 64 bytes.
I wonder is there any plan to support 64 bytes private key?

@yurenji Hey. You just need to slice your 64 bytes private key in half. And pass in the second half to Account.

Isn't it the first half not second half of the 64 byte private key?


Budgeted in future stays red after covering overspend with money money tool

Defective ToolKit Feature Name: Budgeted in future negative (red) indicator
Expected Behaviour
When the budgeted in future is zero, 'Budgeted in future' in header should be white.
Actual Behaviour
After using the move money tool to correct an overspent category that had resulted in the 'Budgeted in future' header being red, although the over budgeted amount in brackets has gone, the 'Budgeted in future' header stays red until the browser page is refreshed.
Steps to Reproduce the Behaviour

Overspend in a category (in current month) when money is already budgeted in next month
Use move money tool to cover overspend

Export of Toolkit Settings (Bottom Left of Settings Page)
[{"key":"accountsClearSelection","value":false},{"key":"accountsDisplayDensity","value":"1"},{"key":"accountsEmphasizedOutflows","value":false},{"key":"accountsRowHeight","value":"1"},{"key":"accountsSelectedTotal","value":false},{"key":"accountsStripedRows","value":true},{"key":"activityTransactionLink","value":true},{"key":"betterScrollbars","value":"0"},{"key":"budgetBalanceToZero","value":true},{"key":"budgetProgressBars","value":"0"},{"key":"budgetQuickSwitch","value":true},{"key":"budgetRowsHeight","value":"1"},{"key":"calendarFirstDay","value":"0"},{"key":"categoryActivityPopupWidth","value":"1"},{"key":"changeEnterBehavior","value":true},{"key":"checkCreditBalances","value":true},{"key":"checkNumbers","value":false},{"key":"closeReconcileWindow","value":false},{"key":"collapseExpandBudgetGroups","value":true},{"key":"collapseSideMenu","value":true},{"key":"colourBlindMode","value":false},{"key":"compactIncomeVsExpense","value":true},{"key":"currentMonthIndicator","value":true},{"key":"daysOfBuffering","value":false},{"key":"daysOfBufferingHistoryLookup","value":"0"},{"key":"editButtonPosition","value":"2"},{"key":"enableRetroCalculator","value":true},{"key":"enlargeCategoriesDropdown","value":true},{"key":"enterInRegisterNow","value":true},{"key":"goalIndicator","value":true},{"key":"goalIndicatorWarningColor","value":false},{"key":"googleFontsSelector","value":"0"},{"key":"hideAOM","value":false},{"key":"hideHelp","value":true},{"key":"highlightNegativesNegative","value":false},{"key":"importNotification","value":"0"},{"key":"incomeFromLastMonth","value":"1"},{"key":"l10n","value":"0"},{"key":"largerClickableIcons","value":true},{"key":"monthlyNotesPopupWidth","value":"1"},{"key":"navDisplayDensity","value":"0"},{"key":"options.dark-mode","value":false},{"key":"pacing","value":"0"},{"key":"popupCalculator","value":true},{"key":"printingImprovements","value":true},{"key":"privacyMode","value":"2"},{"key":"reconciledTextColor","value":"1"},{"key":"removePositiveHighlight","value":true},{"key":"removeZeroCategories","value":true},{"key":"reports","value":false},{"key":"resizeInspector","value":false},{"key":"rightClickToEdit","value":true},{"key":"runningBalance","value":true},{"key":"seamlessBudgetHeader","value":false},{"key":"showIntercom","value":true},{"key":"spareChange","value":true},{"key":"splitKeyboardShortcut","value":true},{"key":"splitTransactionAutoAdjust","value":true},{"key":"squareNegativeMode","value":false},{"key":"stealingFromNextMonth","value":true},{"key":"swapClearedFlagged","value":false},{"key":"toBeBudgetedWarning","value":false},{"key":"toggleSplits","value":true},{"key":"toggleTransactionFilters","value":"0"},{"key":"warnOnQuickBudget","value":true}]

System Settings
ToolKit Version: 0.7.13
Operating System (Mac/Windows/Etc): Mac
Browser (Chrome/Firefox/Safari): Chrome
Browser Version: 57.0.2987.133
Other Extensions:

I'm having trouble reproducing this one. Are you still seeing it? Could you provide some more detailed steps on how to make this happen.
When I "overspend" on a category which is either budgeted in the future or not  I never get the warning :(

Nevermind, I was able to reproduce it was because I didn't have the incomeFromLastMonth feature on.
Fix incoming!

Hey @gsix14 the fix for this should be in our current release. If you notice the issue still exists and you're definitely running the newest version of the toolkit, please reopen! Thanks!

Thank you


docs: add react overview page

Pull Request Checklist

[ ] Did you add new tests and confirm existing tests pass? (yarn test)
[ ] Did you update relevant docs? (docs are found in the site folder, and guidelines for updating/adding docs can be found in the contribution guide)
[ ] Do your commits follow the Conventional Commits standard?
[ ] Does your PR title also follow the Conventional Commits standard?
[ ] If you have a breaking change, is it correctly reflected in your commit message? (e.g. feat!: breaking change)
[ ] Did you run lint (yarn lint:check) and fix any issues? (yarn lint:write)
[ ] Did you follow the contribution guidelines?

PR-Codex overview
This PR updates React dependencies, adds new UI components, and refactors sidebar links for better navigation.
Detailed summary

Updated React dependencies to ^18.2.0
Added TileButton, ClockForwardIcon, and PointerIcon UI components
Refactored sidebar links for React section
Updated vocs dependency to ^1.0.0-alpha.55

The following files were skipped due to too many changes: yarn.lock

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

I think we should iterate fast so not going to nit pick on text or anything unless we play with it live or if you can add screen shots to the test plan in the description.


DONT MERGE - test CI

This change is

PR is added to the queue for testing as 1. task. (9e6a9a2)

Test (Fail) 93739173 build is completed (9e6a9a2)
See https://github.com/ThaliTester/TestResults/tree/937391739e6a9a2_DONT_MERGE_-_test_CI_larryonoff/ for the logs

PR is added to the queue for testing as 3. task. (9bc1cf0)

Test (Success) 93739173 build is completed (9bc1cf0)
See https://github.com/ThaliTester/TestResults/tree/937391739bc1cf0_DONT_MERGE_-_test_CI_larryonoff/ for the logs

Test 937391739bc1cf0(9bc1cf0) has failed
See https://github.com/ThaliTester/TestResults/tree/937391739bc1cf0_DONT_MERGE_-_test_CI_larryonoff/ for the fail logs


Ajax call

Hello,
First thank you for the plugin, I've tried many others and yours is by far the best.
Just wondering how I would make a remote ajax call with a parameter? (if I type @BR to make a search on BR in my database and return a json array)
Thank you!

Just pass a url, which does the search to the source param of the plugin, like this
$('textarea').mentionsInput({source: 'http://example.com/users.json'})

But how can I pass the search query?

When you type @BR, plugin queries the url http://example.com/users.json?term=BR

Oh I get it, thank you very much!
Just another question, why does the input background needs to be transparent for the highlight to work?
I changed line 323 from transparent to white and the hightlight wouldn't work after that!

Also, whilst I have you, is there a way to get the response from getValue as json?
I get this format @username which is gonna need some playing around with preg_match to achieve what I want (storing them nicely in a database) or am I missing something again?
Sorry about all the questions, jquery isn't really my thing!

A note about background, as far as I understand you are having issues with non-white background, so that your input is also non-white. If that's the case, I've just fixed that issue, download updated code.

Can you provide an example of desired response from getValue method?
What do you mean by "storing them nicely in a database"?
The standard approach is to pass the string to the server, and then parse the string with regexp to replace mentions by links.

Regarding the highlighter, the issue is that the input background needs to be transparent for the background-color to work on the highlighter but it's not a bug.
A json array like so {name: 'alex', uid: 'user:1'}
Getting that from getValue would be a lot easier in my opinion.

This is my input on your latest commit:
http://puu.sh/eAQAV/fa8237e7b4.png

What you need is array of all mentions, contained in text, ignoring any text, right? Such a method could be added easily, since this information is already stored inside of the plugin.
I'll add an issue for your problem with focused textarea, I think it's quite a common scenario.

Would be great to see that method added to easily get the value of the tags without messing about with regex.
Ok thanks, in the meantime, I'm just changing the size of .highlighter on focus

I've added method getMentions

Awesome, thank you!


Add flutter engine html_wasm tests to monorepo CI

The flutter engine test suite chrome-dart2wasm-html-engine is important for finding new failures
in dart2wasm which will block the roll of Dart SDK into Flutter engine.
Add this to the https://dart.googlesource.com/monorepo/+/refs/heads/main/ci/builders/web_linux.json
configuration in the monorepo repository, so it is run on commits to the SDK and engine.
This was requested after the suite failed, causing issue https://github.com/flutter/flutter/issues/137120 and a rollback.

This engine test suite chrome-dart2wasm-html-engine is run as part of the web_linux build/test configuration, by a tester called "Linux run chrome-dart2wasm-html-engine suite" in the json configuration file https://github.com/flutter/engine/blob/main/ci/builders/linux_web_engine.json
The tester doesn't do much more than run felt with the right parameters, to run the test suite.
But it depends on downloading the necessary Flutter engine checkout and built files from CAS, where they have been uploaded by its build dependencies (builders in the same json configuration of the web_linux builder)
"web_tests/artifacts",
 "web_tests/test_bundles/dart2wasm-html-engine"

But these builders are different from our existing flutter-web builders. Monorepo's flutter-web builders upload the artifacts to GCS, so Flutter framework checkouts can run framework tests by downloading the artifacts using flutter doctor.
The new builders listed above do a build and upload their entire output directory to CAS, and the engine testers download it from CAS.  The builders listed above also depend on using a prebuilt Dart SDK, and cannot easily switch to building the Dart SDK.  Especially the second one, "web_tests/test_bundles/dart2wasm-html-engine", just checks out the engine source and then runs a simple dart compile command, using the dart2wasm or dart2js already downloaded in the prebuilt Dart SDK.  This builder would be much more expensive if it needed to build the Dart SDK just to run this bundling step.
So figuring out how to run engine web tests on monorepo will need a big look at the entire design of engine testing in the engine_v2 framework, and how the current split of web testing into builders and testers on Flutter engine CI can be adapted to run on monorepo CI, without a prebuilt SDK.  Adding a single web engine tester can't be done without tackling the whole framework.
@eyebrowsoffire @askeksa
An example run of this tester is at https://ci.chromium.org/ui/p/flutter/builders/prod/Linux Production Engine Drone/457572
The run of the web_linux builder that runs and collects this subbuild is at
https://ci.chromium.org/ui/p/flutter/builders/prod/Linux linux_web_engine/5162

It looks like this can't be implemented. Should we close this @mkustermann?

We may still want to have HHH setup with dart2wasm, otherwise we may land CLs that break flutter and will only know when rolls happen. There's a question around how this would work, because the flutter-web downloads a prebuilt Dart SDK rather than building it.
Though this isn't very urgent.


Sourcemaps don't play well with gulp-rev

The issue is that instead of having app-<hash>.css.map we have app.css-<hash>.map
My investigation on this topic:
Here gulp-rev tries to find reverse file name for processed source map.
But with gulp-ruby-sass the map has full file path and sourcemap has only file name.
So the pathMap is {"/usr/home/someuser/projects/app/dist/app.css": "somehash"} and reverseFilenamewhich is taken from sourcemap file field is app.css.
Then pathMap[reverseFilename] is not found and it applies the general logic to the sourcemap. And that results in wrong file name.
Keys for pathMap are taken from path property of the processed file. And this property is set to have full file path in gulp-ruby-sass here. I guess to fix that we need to set file path to have only name instead of full path as this is done in other source map generating libraries. Or maybe we should set file path in sourcemap to be full path (less possible, but I am not sure)

Thanks for the detailed summary!
A little background:
Sass reads files off disk, and then prints them back to disk en masse, instead of working file by file in memory like gulp. In order to manage sourcemaps and our Sass temp directory (that receives the Sass output, which is then read into the stream) we rewrite the sourcemap paths as relative paths, so they work with browsers no matter what directory the final files are output in. The final destination path isn't available at the time of Sass sourcemap creation because it's set farther down in the stream by gulp.dest.
Would you like to try fixing this and submitting a PR? It seems like you've done a lot of research already.

Taking a look at this today.

@robwierzbowski, sorry missed your first message on this topic. I already posted my suggestions on fixing this, but I am not sure if I am on the right way with them. So didn't create the PR.
The main idea is to set only file name instead of full path here: https://github.com/sindresorhus/gulp-ruby-sass/blob/a344f83a615ff9429db0cd07d64d47ba7d6c78fd/index.js#L129
This is how it works with other source maps generating plugins and that is how gulp-rev assumes it to be.

According to the vinyl file docs (https://github.com/wearefractal/vinyl#file), vinyl files should have an absolute value for cwd, base, and path. So I think we're creating a correct vinyl file.
Can you link to one of the projects that uses only a relative path for path?

Here, rev is looking for the sourcemap's file property:
https://github.com/sindresorhus/gulp-rev/blob/0e6fb08f46d71c4f9e12c702fb265f4fca90e10c/index.js#L81-L83
And as far as I can tell we do give the relative path in the sourcemap file. I'm going to try and reproduce your issue, and debug from there.

I'm not reproducing your original error.
gulp task:

// file sourcemaps
gulp.task('sass', function() {
  return sass('source', { sourcemap: true })
  .on('error', function (err) {
    console.error('Error!', err.message);
  })

  .pipe(rev())

  .pipe(sourcemaps.write('../maps', {
    includeContent: false,
    sourceRoot: 'source'
  }))

  .pipe(gulp.dest('result'));
});

Produces:
result/fixture-a-10790b6070.css
maps/fixture-a-10790b6070.css.map

With latest versions of packages and the code of task you provided, it doesn't even get to this line: https://github.com/sindresorhus/gulp-rev/blob/0e6fb08f46d71c4f9e12c702fb265f4fca90e10c/index.js#L81-L83
because sourcemaps array length is 0.
So, yes, I also can't reproduce the issue now. But still I wonder - if it is safe to omit sourcemaps.init() and to rely on gulp-ruby-sass?

Yep! We basically re-create sourcemaps.init here: https://github.com/sindresorhus/gulp-ruby-sass/blob/a344f83a615ff9429db0cd07d64d47ba7d6c78fd/index.js#L131-L146
It's necessary to manage ourselves because nothing can run before our source adapter, and the source adapter is necessary because we need to read the files off of disk. Sass is designed to work off the filesystem on many files at once, so doesn't work well as part of a stream.

Closing since sourcemaps seem to be working as expected. If you find any issues with them, please open a ticket. I don't do much with maps in my own work, so appreciate the experience of others that work with them more extensively.


Line plot function wrapping around

I have many lines that share an X axis to plot on top of each other. However, the plotly line function connects the end of one of these lines to the next with another line segment. How do I get it to realize that these are separate lines to plot?

You can use the line_group kwarg to use a column to identify which rows belong to the same line.

I tried this by adding an extra column containing linegroups, but then the colors now are off. I am using the same color column ("cluster") as before (I am coloring K-means cluster output by color) and now when I hover over the plot, things in the same cluster are different colors. The color was working before the addition of line_group and line group did not change the dataframe other than adding a column. What could be going on?

Sorry, looks like I had a bug in assigning the line widths, which was somehow affecting the colors. I fixed it according to the other issue I posted and the colors are looking fine. Thank you so much for your help!