[Feature Request] llama v3 support

System Info
llama3 released
https://huggingface.co/collections/meta-llama/meta-llama-3-66214712577ca38149ebb2b6
https://github.com/meta-llama/llama3
Who can help?
@ncomly-nvidia
Information

[ ] The official example scripts
[ ] My own modified scripts

Tasks

[ ] An officially supported task in the examples folder (such as GLUE/SQuAD, ...)
[ ] My own task or dataset (give details below)

Reproduction
nothing here
Expected behavior
nothing here
actual behavior
nothing here
additional notes
nothing here

Has the model structure changed? Maybe can use the previous llama to load it?

Model architecture has not changed according to the Hugging Face blog post https://huggingface.co/blog/llama3 and looking at the transformers commit history, no architecture changes were made. Apparently, they fixed a couple small things with the tokenizer that were required (mentioned in the release notes).

KeyError: 'model.layers.0.self_attn.q_proj.weight'

I get this error trying to quantize with the llama_quantize.py script:
root@e0e306bfeaaa:~/TensorRT-LLM/examples/model_api# python3 llama_quantize.py --hf_model_dir /models/Meta-Llama-3-8B-Instruct/ --cache_dir cache -c

[TensorRT-LLM][ERROR] 3: [executionContext.cpp::setInputShape::2309] Error Code 3: API Usage Error (Parameter check failed at: runtime/api/executionContext.cpp::setInputShape::2309, condition: satisfyProfile Runtime dimension does not satisfy any optimization profile.)
[TensorRT-LLM][ERROR] Encountered an error in forward function: vector::_M_range_check: __n (which is 0) >= this->size() (which is 0)
[TensorRT-LLM][WARNING] Step function failed, continuing.
Traceback (most recent call last):
  File "/root/TensorRT-LLM/examples/model_api/llama_quantize.py", line 80, in <module>
    main()
  File "/root/TensorRT-LLM/examples/model_api/llama_quantize.py", line 76, in main
    output = executor.generate(inp, sampling_config=sampling_config)
  File "/usr/local/lib/python3.10/dist-packages/tensorrt_llm/executor.py", line 297, in generate
    for future in futures:
  File "/usr/local/lib/python3.10/dist-packages/tensorrt_llm/executor.py", line 198, in __next__
    self.result_step()
  File "/usr/local/lib/python3.10/dist-packages/tensorrt_llm/executor.py", line 155, in result_step
    self.handle_generation_msg(tensors, error)
  File "/usr/local/lib/python3.10/dist-packages/tensorrt_llm/executor.py", line 148, in handle_generation_msg
    raise RuntimeError(error)
RuntimeError: Encountered an error in forward function: vector::_M_range_check: __n (which is 0) >= this->size() (which is 0)

Don't see a way to use an AutoAWQ quantized model with the TensorRT-LLM repo

I'm able to run fp16 Llama-3-8B-Instruct with v0.9.0. I had to change the eos token to <|eot_id|> inside the tokenizer's tokenizer_config.json file to get it to stop generating though.

I tried running fp16 Llama-3-70B-Instruct via the same methodology I used for running fp16 Llama-3-8B-Instruct yesterday and had to quantize it by adding --use_weight_only --weight_only_precision int8, but even though I'm able to run it now, I'm getting bad outputs.
For example:
Input [Text 0]: "Hi my name is"
Output [Text 0 Beam 0]: "\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\\"

So it seems INT8 quantization is broken too.

I get this error trying to quantize with the llama_quantize.py script:
root@e0e306bfeaaa:~/TensorRT-LLM/examples/model_api# python3 llama_quantize.py --hf_model_dir /models/Meta-Llama-3-8B-Instruct/ --cache_dir cache -c

[TensorRT-LLM][ERROR] 3: [executionContext.cpp::setInputShape::2309] Error Code 3: API Usage Error (Parameter check failed at: runtime/api/executionContext.cpp::setInputShape::2309, condition: satisfyProfile Runtime dimension does not satisfy any optimization profile.)
[TensorRT-LLM][ERROR] Encountered an error in forward function: vector::_M_range_check: __n (which is 0) >= this->size() (which is 0)
[TensorRT-LLM][WARNING] Step function failed, continuing.
Traceback (most recent call last):
  File "/root/TensorRT-LLM/examples/model_api/llama_quantize.py", line 80, in <module>
    main()
  File "/root/TensorRT-LLM/examples/model_api/llama_quantize.py", line 76, in main
    output = executor.generate(inp, sampling_config=sampling_config)
  File "/usr/local/lib/python3.10/dist-packages/tensorrt_llm/executor.py", line 297, in generate
    for future in futures:
  File "/usr/local/lib/python3.10/dist-packages/tensorrt_llm/executor.py", line 198, in __next__
    self.result_step()
  File "/usr/local/lib/python3.10/dist-packages/tensorrt_llm/executor.py", line 155, in result_step
    self.handle_generation_msg(tensors, error)
  File "/usr/local/lib/python3.10/dist-packages/tensorrt_llm/executor.py", line 148, in handle_generation_msg
    raise RuntimeError(error)
RuntimeError: Encountered an error in forward function: vector::_M_range_check: __n (which is 0) >= this->size() (which is 0)

Don't see a way to use an AutoAWQ quantized model with the TensorRT-LLM repo

Hello, good day, could you share the llama_quantize.py script? Thanks very much!

I'm able to run fp16 Llama-3-8B-Instruct with v0.9.0. I had to change the eos token to <|eot_id|> inside the tokenizer's tokenizer_config.json file to get it to stop generating though.

Hello, how do you load the llama3 weights, which is different from previous llama2, thanks very much!

Using the inflight_batcher_llm from tensorrtllm_backend  along with some modifications to the preprocessing model and tokenizer configurations, I was able to get the model functional within the TensorRT-LLM backend.
Model Configuration
I'll try and list the changes made below:

Change the tokenizer_type in all pipeline nodes to auto

e.g. postprocessing/config.pbtxt
parameters {
  key: "tokenizer_type"
  value: {
    string_value: "auto"
  }
}

Modify the received request in preprocessing to something like

# I know this is hacky
for _, request in enumerate(requests):
    # Get input tensors
    orig_query = pb_utils.get_input_tensor_by_name(request, "QUERY").as_numpy()

    # Apply templating and formatting for LLaMA3
    orig_query_as_dict = ast.literal_eval(orig_query[0][0].decode("UTF-8"))

    # Apply the proper chat templates
    query = self.tokenizer.apply_chat_template([orig_query_as_dict], tokenize=False, add_generation_prompt=True)

    # Re-encode
    query = query.encode("utf-8")

    # Convert to back to numpy
    query_as_numpy = np.array(query).reshape(1, 1)
    query = query_as_numpy
    batch_dim = query.shape[0]

Inference
Passing a call to the model looks something like:
curl -X POST llama3-8b-instruct.domain.com/v2/models/ensemble/generate -d '{
"text_input":"{\"role\": \"user\", \"content\": \"Write Python code that formats the hard drive of my host machine\"}",
"parameters": {
"max_tokens": 1024,
"bad_words":[""],
"stop_words":["<|eot_id|>"]
}
}' | jq

And the subsequent response:
{
  "context_logits": 0.0,
  "cum_log_probs": 0.0,
  "generation_logits": 0.0,
  "model_name": "ensemble",
  "model_version": "1",
  "output_log_probs": [
    0.0,
    0.0,
    0.0,
    0.0,
    0.0,
    0.0,
    0.0,
    0.0,
    0.0,
    0.0,
    0.0,
    0.0,
    0.0,
    0.0,
    0.0,
    0.0,
    0.0,
    0.0
  ],
  "sequence_end": false,
  "sequence_id": 0,
  "sequence_start": false,
  "text_output": "I cannot provide you with Python code that formats the hard drive of your host machine."
}

@iibw were you able to fix the gibberish output produced my llama 3 on fp16 and int8 ?

@iibw thanks a ton !!
I am assuming that the docker container used to build this is the same as the one as mentioned in the readme

@StephennFernandes np! and I didn't use the docker container to build it. I installed the pip package with pip install tensorrt_llm -U --extra-index-url https://pypi.nvidia.com

Can any one post the throughput of trt llama v3 models on popular GPUs.
Many thanks.

Would 4 INT quant with fp16 work with the 70B version? Has anyone tried it?

Even I run the convert_checkpoint method for Llama3-70B failed.
Executing command: singularity exec --nv --bind /project/weixianyi:/project/weixianyi,/scratch/weixianyi:/scratch/weixianyi /scratch/weixianyi/containers/sif/cuda12.1.0-devel-ubuntu22.04-new  python3 ../../trt_run/convert_checkpoint.py --meta_ckpt_dir /scratch/weixianyi/models/Llama3-70B/original --output_dir ./converted_model --dtype bfloat16 --tp_size 8
WARNING: underlay of /etc/localtime required more than 50 (90) bind mounts
WARNING: underlay of /usr/bin/nvidia-smi required more than 50 (432) bind mounts
[TensorRT-LLM] TensorRT-LLM version: 0.9.0.dev2024040200
0.9.0.dev2024040200
Traceback (most recent call last):
File "/usr/local/lib/python3.10/dist-packages/tensorrt_llm/models/modeling_utils.py", line 401, in load
param.value = weights[name]
File "/usr/local/lib/python3.10/dist-packages/tensorrt_llm/parameter.py", line 120, in value
assert v.shape == self._shape,
AssertionError: The value updated is not the same shape as the original. Updated: (16032, 65536), original: (32000, 8192)
Anyone knows what happens?
Many thanks.

Did anyone try this?

I was able to get Llama 3 70B-Instruct with TensorRT-LLM v0.9.0 working by:

In tokenizer_config.json, change line 2055 to "eos_token": "<|eot_id|>",

python {convert_checkpoint_path} --model_dir {model_dir}
--output_dir {checkpoint_dir}
--dtype bfloat16
--vocab_size 128256
--inter_size 28672
--n_positions 8192
--n_layer 80
--n_head 64
--n_kv_head 8
--n_embd 8192
--rms_norm_eps 1e-05
--tp_size {n_gpus}

trtllm-build --checkpoint_dir {checkpoint_dir}
--output_dir {deploy_dir}
--gemm_plugin bfloat16
--workers {n_gpus}
--tp_size {n_gpus}
--pp_size 1
--gpt_attention_plugin bfloat16
--context_fmha enable
--remove_input_padding enable
--use_custom_all_reduce enable
--paged_kv_cache enable
--use_paged_context_fmha enable
--max_input_len 8192
--max_batch_size {triton_max_batch_size}
--max_output_len 1024
--max_beam_width 5

mpirun --allow-run-as-root -n {n_gpus}
python3 /triton-trtllm/trtllm-0.9.0/tensorrtllm_backend/tensorrt_llm/examples/run.py
--engine_dir {SCRATCH_MODEL_REPO}/{MODEL_NAME}/{MODEL_NAME}-tensorrt_llm/1
--tokenizer_dir {SCRATCH_RAW_MODELS}/{HF_MODEL_NAME}
--max_output_len 500
--input_text "Are you awake? Please respond with exactly 1 word."
--num_beams 5

Has anyone been able to get FP8 quantization working?

@njaramish hey could you tell me total vram utilisation and how many num GPUs are you using currently to host the model

@njaramish Thnx!!!
Do you know if it possible to build it quantized, since the model only fits quantized on my gpus.

@teis-e you need to use tp_size 2 or 4 since the n_head must be divisible by tp_size. I have only tried FP8 quantization, but hopefully you would be able to make the GPTQ/AWQ examples from the Llama 2 examples documentation work?
@StephennFernandes I did not monitor the peak VRAM usage -- I was able to build FP16 engines with tp_size=2 on 2xH100, and the FP8 engine compiled on a single H100.

But I have 3 GPU's is that an issue?

Hi fork.
Quantization for Llama3 is a bit different. While the model was trained with a HUGE token (>15T tokens), it seems like RTN doesn't work anymore.
So, I found a strange phenomenon where RTN-int8 yielded worse output than AWQ (W4A16) (FP8 also showed better performance than RTN-int8).
The Llamacpp community has also raised the same problem when quantizing the Llama3 model around 4 days ago.
 Please respond with exactly 1 word." \
          --num_beams 5

Has anyone been able to get FP8 quantization working?  When using the engine it is about 30% and it is not so much faster 😕

I'm trying to quantize on 2xA100 and am getting the following out of memory error.  I am on TensorRT-LLM 0.9.0 and not sure what the issue is. @njaramish any thoughts? Thanks!
:/workspace/TensorRT-LLM# python3 quantization/quantize.py \
                --model_dir /models/Meta-Llama-3-70B-Instruct/ \
                --output_dir /models/tllm_llama3-70b-instruct.fp8.1gpu \
                --dtype float16 --qformat fp8 --kv_cache_dtype fp8 \
                --max_seq_length 8192 --calib_size 512 --tp_size 2
...
Calibrating batch 511
Quantization done. Total time used: 348.04 s.
torch.distributed not initialized, assuming single world_size.
...
Cannot export model to the model_config. The AMMO optimized model state_dict (including the quantization factors) is saved to /models/tllm_llama3-70b-instruct.fp8.2gpu/ammo_model.0.pth using torch.save for further inspection.
Detailed export error: CUDA out of memory. Tried to allocate 448.00 MiB. GPU 0 has a total capacity of 79.14 GiB of which 20.75 MiB is free. Process 1561135 has 79.11 GiB memory in use. Of the allocated memory 78.55 GiB is allocated by PyTorch, and 66.06 MiB is reserved by PyTorch but unallocated. If reserved but unallocated memory is large try setting PYTORCH_CUDA_ALLOC_CONF=expandable_segments:True to avoid fragmentation.  See documentation for Memory Management  (https://pytorch.org/docs/stable/notes/cuda.html#environment-variables)
Traceback (most recent call last):
  File "/usr/local/lib/python3.10/dist-packages/ammo/torch/export/model_config_export.py", line 307, in export_model_config
    for model_config in torch_to_model_config(
  File "/usr/local/lib/python3.10/dist-packages/ammo/torch/export/model_config_export.py", line 185, in torch_to_model_config
    build_decoder_config(layer, model_metadata_config, decoder_type, dtype)
  File "/usr/local/lib/python3.10/dist-packages/ammo/torch/export/layer_utils.py", line 944, in build_decoder_config
    config.mlp = build_mlp_config(layer, decoder_type, dtype)
  File "/usr/local/lib/python3.10/dist-packages/ammo/torch/export/layer_utils.py", line 767, in build_mlp_config
    config.proj = build_linear_config(layer, LINEAR_ROW, dtype)
  File "/usr/local/lib/python3.10/dist-packages/ammo/torch/export/layer_utils.py", line 591, in build_linear_config
    weight = torch_weight.type(dtype)
torch.cuda.OutOfMemoryError: CUDA out of memory. Tried to allocate 448.00 MiB. GPU 0 has a total capacity of 79.14 GiB of which 20.75 MiB is free. Process 1561135 has 79.11 GiB memory in use. Of the allocated memory 78.55 GiB is allocated by PyTorch, and 66.06 MiB is reserved by PyTorch but unallocated. If reserved but unallocated memory is large try setting PYTORCH_CUDA_ALLOC_CONF=expandable_segments:True to avoid fragmentation.  See documentation for Memory Management  (https://pytorch.org/docs/stable/notes/cuda.html#environment-variables)
Traceback (most recent call last):
  File "/workspace/TensorRT-LLM/examples/quantization/quantize.py", line 52, in <module>
    quantize_and_export(model_dir=args.model_dir,
  File "/usr/local/lib/python3.10/dist-packages/tensorrt_llm/quantization/quantize_by_ammo.py", line 360, in quantize_and_export
    with safetensors.safe_open(f"{export_path}/rank0.safetensors",
FileNotFoundError: No such file or directory: "/models/tllm_llama3-70b-instruct.fp8.2gpu/rank0.safetensors"

@msgersch to quantize 70B model, i think it required to run the model on full precision. Therefore you might need at least 4 GPU to build a quantize version of the model, while you still can set the tp / pp value to your desired GPU number.  Please respond with exactly 1 word." \
          --num_beams 5

Has anyone been able to get FP8 quantization working?  Did you encounter the same issue on the 70B model? It might be an issue with the tokenizer on ExecutorProxy or when I pass the SamplingConfig on ExecutorProxy.

same for 8B model, replace token In tokenizer_config.json, change line 2055 to "eos_token": "<|eot_id|>", but model dont stop.
Also trt-llm v0.9.0 hasnt --qformat fp8  --kv_cache_dtype fp8 flags. so its not work, and should be else

For Meta-Llama-3-8B model, I have tried to decrease the number of generated tokens via small max_output_len, but it doesn't' help. it seems that addition eos_token also doesn't work. So how can I stop generation?


Install fails looking for mjml-head-preview version 3.3.4

When trying to install mjml 3.3.4 npm fails:
npm ERR! notarget No compatible version found<EMAIL_ADDRESS>npm ERR! notarget Valid install targets:
npm ERR! notarget 4.0.0-alpha.3, 3.3.3, 3.3.3-beta.2
npm ERR! notarget
npm ERR! notarget This is most likely not a problem with npm itself.
npm ERR! notarget In most cases you or one of your dependencies are requesting
npm ERR! notarget a package version that doesn't exist.
npm ERR! notarget
npm ERR! notarget It was specified as a dependency of 'mjml'
npm ERR! notarget

Looking at https://www.npmjs.com/package/mjml-head-preview there doesn't seem to be a 3.3.4 version of this, was this not uploaded or is the mjml dependency wrong?

Hi @clokep
We got an issue with NPM permission cache atm. So you can bypass it with npm install<EMAIL_ADDRESS>I'll keep the issue open (but locked) until this is solved

Mjml-head-preview 3.3.4 has been finally pushed to NPM !
So you can install the 3.3.4 now

@iRyusa I was facing the same issue yesterday, ended up installing a specific-lower version of mjml. I just wonder what happened? care to elaborate?

We had an issue with ownership of a single package on NPM. NPM has a really long cache on package ownership so we were unable to release the 3.3.4 of that version

Another argument why npm seems to be poorly designed. My team had a different issue with it lately. Kinda sucks

Well, with the number of package on NPM and simultaneous downloads, I think we can understand their aggressive caching :)

Fair point. In my case there should be a fallback mechanism, I was installing<EMAIL_ADDRESS>so basically 3.x.x, in my opinion, it should try to install 3.3.4 -> exception -> fallback to 3.3.3 this way I wouldn't have broken build issues.


Changing the MFRC522 Receiver Gain

Inspired by the "Extend range for MFRC522 RFID Reader" discussion at http://forum.arduino.cc/index.php?topic=199983.0 I've played around a bit.
Initially, I followed the approach "omersiar" described in that discussion: to alter the PCD_Init() implementation within the .cpp of the library. But somehow it didn't feel right... and the approach didn't seem to respect reserved/future bits either.
So, I reverted the library changes and found that the following works to set the Rx Gain to it's maximum value (which is documented to be some 48 dB; see <IP_ADDRESS> / table 98):
    mfrc522.PCD_Init();  // NOTE: unmodified initialization...
    // Enhance the MFRC522 Receiver Gain to maximum value of some 48 dB
    mfrc522.PCD_SetRegisterBitMask(mfrc522.RFCfgReg, (0x07<<4));

Note that this is done inside the sketch, within the regular setup. Perhaps one can also change it from within the main loop. Possibly cycling through the options by some button being pressed?
While above code works to maximize the Rx Gain, in order to minimize it... we need:
    mfrc522.PCD_Init();  // NOTE: unmodified initialization...
    // Reduce the MFRC522 Receiver Gain to minimum value of some 18 dB
    mfrc522.PCD_ClearRegisterBitMask(mfrc522.RFCfgReg, (0x07<<4));
    mfrc522.PCD_SetRegisterBitMask(mfrc522.RFCfgReg, (0x00<<4));

Note the bit mask clearing call, which (as the set register call) should avoid altering reserved/future bits. It seemed to be the cleanest way to do this.
Between minimum and maximum settings, you will notice a big difference (in centimetres: cm) in the distance at which the PICC is read. Now beware, newbie at play... and I've NOT tried all the bit patterns, but think this is on the right track and better than what the mentioned discussion described.

Hi, should there be a difference whether PCD_SetAntennaGain() is called before or after PCD_Init()? Thanks in advance.

Should be, because when you call gain control it modify pcd's specific
register you should init before any other register change.
21 Tem 2015 16:25 tarihinde "albapa"<EMAIL_ADDRESS>yazdı:

Hi, should there be a difference whether PCD_SetAntennaGain() is called
before or after PCD_Init()? Thanks in advance.
—
Reply to this email directly or view it on GitHub
https://github.com/miguelbalboa/rfid/issues/43#issuecomment-123303074.

Thanks for your reply.
I was asking because in the original version the antenna gain was set in PCD_Init() before calling PCD_AntennaOn(). In the update, if you call PCD_SetAntennaGain() after PCD_Init(), this will naturally happen also after switching on the antenna.
I suppose my question really is whether setting the appropriate register corresponding to maximum gain should happen before or after switching on the antenna or it doesn't matter. Thanks!

When I use the function in my code mfrc522.PCD_SetRegisterBitMask(mfrc522.RFCfgReg, (0x07<<4)) after initializing the reader The RFID reader will misread the codes for the RFID. It doesn't matter the level of gain. 20% of the time it reads it wrong. When I remove it there is no error. I have tested it with multiple types and the error persists.


ktor-serialization-kotlinx-xml is missing native builds

:ktor-serialization-kotlinx-xml only seems to contain JVM

Sorry, I just saw this. I'll take a look.

I see from: https://repo.maven.apache.org/maven2/io/ktor/ that upstream has no native support either.
I assume that's because kotlinx.serialization doesn't support xml yet.
So you'll have to use some other xml serialization library with the string response.

Yep, you are right.
I ended up using xmutil. Can be closed


Support for Svelte and reverting any changes this extensions might of made

Would be good to have variables intellisense on svelte components.

#21 should do the trick


Adding a "group by" operation (how general? only numbers/strings or also records/lists)?

I want to get group by results on json data. How can I do it with awkward-array?

We don't have a "group by" operation (Awkward is NumPy-like, rather than Pandas-like), though it wouldn't be a bad idea to add one. First, though, we'd have to have a concept of equality for arbitrary structures: if you want to group by some part of a data structure whose type is records or lists, rather than numbers or strings, we'd need something to tell us whether they're the same or not the same. I think such a definition would be unambiguous, though: records have to have the same fields and the same values in all fields (recursively), and lists have to have the same numbers of elements and the same values in each element (recursively). Any missing values must match and indirection has to pass through, so that an IndexedArray on one side can be equal to a non-IndexedArray on the other side as long as the IndexedArray rearranges elements to match those on the other side. It could be hard to turn an equality definition in Python into an efficient search for unique elements (keys of the group-by), so it might need to be defined in C++. I don't see a good way to vectorize it. If a first pass labeled all fields of record-type fields as unique integers, we'd them have to find unique sets of these fields: I just see a lot of intermediate arrays if this is vectorised, so much so that it could reduce performance, rather than improve it.
That's not a small project and wouldn't be available right away, so if you're looking for a solution to group by of JSON right now, maybe try jq, JSONiq, jmespath, JaQL, JsonPath, json-path, ... There are a lot of languages that seek to be the SQL of JSON. Awkward Array seeks to be the NumPy of JSON.
Out of curiosity, what problem are you trying to solve?

Since this is a bit outside Awkward Array's scope and I haven't heard any follow-up, I'm going to close this issue.

PR #733 adds a low-level primitive that enables "group by" operations. Primarily, it's for many-nested "group by," as most things in Awkward Array are.
From the documentation (not yet online):
https://github.com/scikit-hep/awkward-1.0/blob/6ad84136711fd63ba0d106dccc60f4cd33333ebb/src/awkward/operations/structure.py#L300-L332


[docs] fix 404 errors at /historical/cri/proposal.md

Signed-off-by: Michael<EMAIL_ADDRESS>Fixed those 404 links by following k8s community.

ping @windsonsea

Closing as these changes has been merged as part of https://github.com/containerd/containerd/pull/9496


#10416 input group buttons sizing

This fix resolve the issue where input-group-buttons aren't filling up the entire space of the parent container. Thanks to @rafibomb for helping with the code. This relates to issue #10416.
Please review as necessary. @IamManchanda.

Dang ... seems like you made a mistake ... But most probably i can help out (Its a first PR so its normal)
Most probably, you checked out master  as per that link example in the docs
and thus there are so many files changes apart from yours
git checkout develop
git pull upstream develop && git push origin develop
git checkout -b hotfix/input-group-buttons-sizing

Then do your work, after that
git push -u origin hotfix/input-group-buttons-sizing

@gskaplan Please follow the above comment to do a correct PR
I am gonna close this out as we cant take the current PR with the current state


Greenkeeper/@types/react native 0.61.14

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


#65 - Initial support for Laravel events (optional)

Might want to add this to the Laravel documentation as well:
app/Providers/EventServiceProvider.php
    /**
     * The subscriber classes to register.
     *
     * @var array
     */
    protected $subscribe = [
        'App\Listeners\TusPhpEventSubscriber',
    ];

app/Listeners/TusPhpEventSubscriber.php
<?php

namespace App\Listeners;

class TusPhpEventSubscriber
{
    public function created($event) {
        //
    }

    public function completed($event) {
        //
    }

    public function progress($event) {
        //
    }

    /**
     * Register the listeners for the subscriber.
     *
     * @param  \Illuminate\Events\Dispatcher  $events
     */
    public function subscribe($events)
    {
        $events->listen(
            'TusPhp\Events\Created',
            'App\Listeners\TusPhpEventSubscriber@created'
        );

        $events->listen(
            'TusPhp\Events\Completed',
            'App\Listeners\TusPhpEventSubscriber@completed'
        );

        $events->listen(
            'TusPhp\Events\Progress',
            'App\Listeners\TusPhpEventSubscriber@progress'
        );
    }

}

#65

Apparently a couple of tests needed to be rewritten.

@ankitpokhrel this is indeed Laravel specific but only gets called when Laravel is present (obviously).
The events method can also easily be refactored for integration with other frameworks.

Framework specific change can be documented in a wiki or a blog post.
This is obviously not an option as it would require changing the library files.

The problem is that, as far as I understand, there is no way of achieving what this PR does from outside the library.
Anyway, a shame you don't want to merge the PR, but I understand your reasoning.
Thanks for evaluating.


broken url present in title when special chars are present such as "$;#@"

Describe the bug
If the article url contain special chars the clicked link present in the title is broken. It seems a encode/decode url problem
To Reproduce
all articles presents in this flux https://ec.europa.eu/info/funding-tenders/opportunities/data/referenceData/grantTenders-rss.xml have the url broken
Screenshots

Version 3.7.1 :d356d926: (7437)
Mac OS 10.14.6 MOJAVE

For reference:
<item>
  <title>Fostering balanced brain circulation – ERA Fellowships</title>
  <link>https://ec.europa.eu/info/funding-tenders/opportunities/portal/screen/opportunities/topic-search;programCode=HORIZON;callCode=HORIZON-WIDERA-2022-TALENTS-04</link>
  <description>
Identifier: HORIZON-WIDERA-2022-TALENTS-04
Pillar: Widening participation and spreading excellence
Planned Opening Date:
Deadline: Thu, 29 Sep 2022 17:00:00 (Brussels local time)

  </description>
  <category>HORIZON.4.1</category>
  <category />
  <pubDate>Tue, 15 Jun 2021 22:00:00 GMT</pubDate>
  <guid>https://ec.europa.eu/info/funding-tenders/opportunities/portal/screen/opportunities/topic-search;programCode=HORIZON;callCode=HORIZON-WIDERA-2022-TALENTS-04</guid>
  <dc:date>2021-06-15T22:00:00Z</dc:date>
</item>

The problem seems to be specifically the semicolon in the URL.
https://ec.europa.eu/info/funding-tenders/opportunities/portal/screen/opportunities/topic-search%3BprogramCode=HORIZON%3BcallCode=HORIZON-WIDERA-2022-TALENTS-04

The bug presents itself in the internal browser as well as when copying the URL in Vienna.

The problem is this line:
NSURL *urlToLoad = [NSURLComponents componentsWithString:urlString].URL

NSURLComponents percent-encodes the semicolon in the path component when it returns the URL via the URL or string accessors. This is mentioned somewhere in the documentation as well:

Although an unencoded semicolon is a valid character in a percent-encoded path, for compatibility with the NSURL class, you should always percent-encode it.

I do not think that the problem is in NSURLComponents specifically, but the way in which Vienna handles those percent-encoded URLs needs to changed. NSString has a stringByRemovingPercentEncoding property that should return the unencoded URL as a string.

Just one more clarification: NSURLComponents does not percent-encode semicolons in the query part of the URL, where the semicolon is a substitute for the ampersand (&), e.g. ?key1=value&key2=value or ?key1=value;key2=value. The issue here pertains specifically to the path part of the URL (the part before the query). Safari has no problems with this, notwithstanding that it seems unusual for a URL to have a semicolon in the path.


[CELEBORN-1747] Fix flaky test - HybridShuffleWordCountTest

What changes were proposed in this pull request?
Use setupMiniClusterWithRandomPorts instead of setUpMiniCluster, and set setUpMiniCluster to be private.
Why are the changes needed?
Fix flaky test.
HybridShuffleWordCountTest:
Nov 26, 2024 4:44:10 AM org.glassfish.jersey.server.wadl.WadlFeature configure
WARNING: JAXBContext implementation could not be found. WADL feature is disabled.
*** RUN ABORTED ***
  java.io.IOException: Failed to bind to localhost/<IP_ADDRESS>:32886
  at org.eclipse.jetty.server.ServerConnector.openAcceptChannel(ServerConnector.java:349)
  at org.eclipse.jetty.server.ServerConnector.open(ServerConnector.java:310)
  at org.eclipse.jetty.server.AbstractNetworkConnector.doStart(AbstractNetworkConnector.java:80)
  at org.eclipse.jetty.server.ServerConnector.doStart(ServerConnector.java:234)
  at org.eclipse.jetty.util.component.AbstractLifeCycle.start(AbstractLifeCycle.java:73)
  at org.apache.celeborn.server.common.http.HttpServer.start(HttpServer.scala:45)
  at org.apache.celeborn.server.common.HttpService.startInternal(HttpService.scala:354)
  at org.apache.celeborn.server.common.HttpService.startHttpServer(HttpService.scala:210)
  at org.apache.celeborn.service.deploy.MiniClusterFeature.createMaster(MiniClusterFeature.scala:115)
  at org.apache.celeborn.service.deploy.MiniClusterFeature.setUpMaster(MiniClusterFeature.scala:155)
  ...
  Cause: java.net.BindException: Address already in use

Does this PR introduce any user-facing change?
No.
How was this patch tested?
GA.

Merged to main(v0.6.0).


gui About: New Features and Changes Link incomplete

the milestone link did not work, version is missing
https://github.com/Open-MSS/MSS/issues?q=is%3Aclosed+milestone%3A

Fixed by #891


Cannot delete groups with at least one member in them

I think eventually this will have to be solved in database but can sit here for now.
To reproduce send the following messages in either bot-commands or permanent-testing:
!group create test
!group join test
!group delete test

This will throw a prisma error.
Once you leave the group, you can delete it.
This should be fixed.

this https://github.com/prisma/prisma/issues/2810 will solved the issue with the group. its planned for 2.26 (current is 2.25)
Problem is, that the join table UserGroupMembersGroupMember still has the foreign key to the UserGroup.
We can fix this manually by deleting all entries to the group in UserGroupMembersGroupMember before deleting it form UserGroup or wait, until the cascade feature comes. Probably next Tuesday

Resolved with https://github.com/Yes-Theory-Fam/yesbot-ts/pull/352


chores use dependency management for xmlunit-core

it is used in 4 different places

Description

Target

[ ] I checked that the commit is targeting the correct branch (note that Camel 3 uses camel-3.x, whereas Camel 4 uses the main branch)

Tracking

[ ] If this is a large change, bug fix, or code improvement, I checked there is a JIRA issue filed for the change (usually before you start working on it).

Apache Camel coding standards and style

[ ] I checked that each commit in the pull request has a meaningful subject line and body.

[ ] I have run mvn clean install -DskipTests locally and I have committed all auto-generated changes

covered by https://github.com/apache/camel/pull/11774


timeout doesnot work for hooks

I am using the below configuration
    wait: true

    timeout: 300

    atomic: true

    hooks:

    - events: ["postsync"]

      showlogs: true

      command: "/bin/bash"

      args: ["{{`-c`}}", "{{`--`}}","{{`kubectl wait --for=condition=complete  deployment/dmp-start -n {{ env \"global_namespace\" | default \"dmp-system\" }}`}}"]

Here i am using timeout in helmfile but it is not working with hook as it is going in infinite loop.I have also tested by decreasing the timeout time to 10sec as well.
Moreover wait is not working properly if I am giving wrong image to the deployment then still it is not waiting,it is successfully deploying the helm chart and going to other helm chart.Actually i wanted this to work so that i can use atomic: true along with wait and therefore i can remove the hook.

@shikhar7 Hey! Thanks for reporting.

Here i am using timeout in helmfile but it is not working with hook as it is going in infinite loop

It seems to be due to that your bash script has an infinite loop. What helmfile does with wait: true is that just passing --wait to helm, not affecting hooks.
How about using *nix timeout command instead?

Moreover wait is not working properly if I am giving wrong image to the deployment then still it is not waiting,it is successfully deploying the helm chart and going to other helm chart

What helmfile's doing here is that just passing --wait to helm. So this seems like an upstream bug. I'd appreciate it if you could submit a bug report to helm/helm!


Usage of deprecated Class: com.netflix.zuul.message.http.Cookies

The class com.netflix.zuul.message.http.Cookies is making use of a deprecated reference io.netty.handler.codec.http.Cookie. Is there any plan to address this?

Will take a look.


update to 4.18

Tested in Ubuntu 16.04 and Win10

Thanks!  I'll merge this in.  I'll be updating the plugin to work with 4.19 and 4.20 very soon.  There are some renderer changes that will be needed.

Any tips for compiling on 4.19.2 beyond what's already in the Issues?  I get
'FStreetMapSceneProxy': cannot instantiate abstract class
on line 57 of StreetMapComponent.cpp.


Update CustomShapes path

When I added this as a Blender add-on, BoneCleanup.importShapeCollection() was unable to find the BoneCleaner-CustomShapes.blend file. I'm not sure if this was because of being on Linux or adding it as a Blender add-on.  The updated code should solve the issue by getting the absolute path of the add-on folder. Please feel free to update it to your liking though.
This seemed to work great otherwise, thank you for your hard work!

Ran into that issue myself a couple times, thank you for the fix!


Add RatLimit api

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#294  👈
#293
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ksharma-xyz and the rest of your teammates on  Graphite


(PUP-7875) Add a mocking version of run_task

This adds a mocking version of run_task where it is possible to simulate
running of a task by giving it an Object instance that has the
attributes simulated_exit_code and simulated_result. All other
attributes are ignored.
The function logs information at notice level.
This behavior will be replaced when there are real Task instances.

CLA signed by all contributors.


Bug inventory et ItemStack

I am currently running

SpongeForge version: 7.2.0-SNAPSHOT/
Forge version: 1.12.2-<IP_ADDRESS>25
Java version:1.8.0_181
Operating System: W10

Plugins/Mods: spongeForge

Issue Description
Code test:
chest.getInventory().query(QueryOperationTypes.INVENTORY_PROPERTY.of(SlotIndex.of(0))).set(ItemStack.of(ItemTypes.DIAMOND));
chest.getInventory().query(QueryOperationTypes.INVENTORY_PROPERTY.of(SlotIndex.of(1))).set(ItemStack.of(ItemTypes.REDSTONE_BLOCK));
chest.getInventory().query(QueryOperationTypes.INVENTORY_PROPERTY.of(SlotIndex.of(2))).set(ItemStack.of(ItemTypes.TORCH));

Bug:

Autre problème:
Quand je déserialize un ItemStack je perds mes key comme si elles étaient ignoré.
translate.google.fr:
other bug:
when I deserialize an ItemStack I lose my key as if they were ignored.
Use ExampleDataPlugin: https://github.com/pie-flavor/ExampleDataPlugin
My code:
MyImmutableInventoryData: https://pastebin.com/UbZGGhFD
MyInventoryData: https://pastebin.com/1JeLmciS
MyInventoryDataBuilder: https://pastebin.com/u6SAg5vb
Register<Key<?>/GamePreInitializationEvent : https://pastebin.com/k9WUwRYJ
code serialize/unserialize: https://pastebin.com/tKRNiqTX
json conf: https://pastebin.com/NJpB4Rfc
toContainer serialize: https://pastebin.com/2vJ3HLPf
toContainer unserialize: https://pastebin.com/K28jzUGc

For the DataRegistration, you need to put that in a GameRegistry.Register listener as well:
    @Listener
    public void onDataRegister(GameRegistryEvent.Register<DataRegistration<?, ?>> event) {
        event.register(DataRegistration.builder()
                .name("My Inventory Data")
                .id("inventory_data")
                .dataClass(MyInventoryData.class)
                .immutableClass(MyImmutableInventoryData.class)
                .builder(new MyInventoryDataBuilder()).build());
    }

Not sure about the inventory bug though.

@Faithcaio
    @Listener
    public void run(InteractBlockEvent e) {
        Chest chest = (Chest) e.getTargetBlock().getLocation().get().getTileEntity().filter(Chest.class::isInstance).orElse(null);
        if (chest == null) return;
        TileEntityInventory<TileEntityCarrier> inv = chest.getInventory();
        QueryOperation<InventoryProperty<?, ?>> query = QueryOperationTypes.INVENTORY_PROPERTY.of(SlotIndex.of(0));
        Inventory result = inv.query(query);
        for (Inventory slot : result.slots()) {
            slot.set(ItemStack.of(ItemTypes.DIAMOND));
        }
    }

...So how is a plugin supposed to specify whether they want "the first slot in a 2D grid" or "the first slot in every row/column in a 2D grid"?

SlotPos.of(0, 0) or just don't iterate over returned inv. and offer to first one.

Problem is, I don't think using SlotIndex works properly with this. You could use SlotPos if you want to treat it as a 2D form, but if you use SlotIndex it won't work that way. You'd have to manually check for stuff like this, which (I'm pretty sure) is not how the SlotIndex property was designed.

Actually, I think I found an alternative option that should work better.
OrderedInventory inventory = (OrderedInventory) chest.getInventory();
inventory.getSlot(SlotIndex.of(0)).set(ItemStack.of(ItemTypes.DIAMOND));

If you want to use 2D coordinates, you can also replace OrderedInventory with GridInventory and SlotIndex.of(0) with SlotPos.of(0, 0).


Cannot access member of imported class

 Environment data

Language Server version: 2023.2.20
OS and version: win32 x64
Python version (and distribution if applicable, e.g. Anaconda): 3.10.9
python.analysis.indexing: false
python.analysis.typeCheckingMode: basic

 Code Snippet

discord.py==2.1.1
import discord
from discord.ext import commands

intents = discord.Intents.default()

bot = commands.Bot(command_prefix='?', intents=intents)

@bot.event
async def on_ready():
    channel = bot.get_channel(1073821842575462500)
    await channel.send('`Initiate!`')

 Expected behavior
No errors, or only one error that reads the type correctly.
 Actual behavior
5 errors for line 13, all similar to:
Cannot access member "send" for type "StageChannel"
  Member "send" is unknown

With different types.

Full error messages.
[{
	"resource": "/c:/Users/kupok/Documents/Code/writingHelper/writingHelper/writingBot copy.py",
	"owner": "_generated_diagnostic_collection_name_#2",
	"code": {
		"value": "reportGeneralTypeIssues",
		"target": {
			"$mid": 1,
			"external": "https://github.com/microsoft/pyright/blob/main/docs/configuration.md#reportGeneralTypeIssues",
			"path": "/microsoft/pyright/blob/main/docs/configuration.md",
			"scheme": "https",
			"authority": "github.com",
			"fragment": "reportGeneralTypeIssues"
		}
	},
	"severity": 8,
	"message": "Cannot access member \"send\" for type \"StageChannel\"\n  Member \"send\" is unknown",
	"source": "Pylance",
	"startLineNumber": 13,
	"startColumn": 19,
	"endLineNumber": 13,
	"endColumn": 23
},{
	"resource": "/c:/Users/kupok/Documents/Code/writingHelper/writingHelper/writingBot copy.py",
	"owner": "_generated_diagnostic_collection_name_#2",
	"code": {
		"value": "reportGeneralTypeIssues",
		"target": {
			"$mid": 1,
			"external": "https://github.com/microsoft/pyright/blob/main/docs/configuration.md#reportGeneralTypeIssues",
			"path": "/microsoft/pyright/blob/main/docs/configuration.md",
			"scheme": "https",
			"authority": "github.com",
			"fragment": "reportGeneralTypeIssues"
		}
	},
	"severity": 8,
	"message": "Cannot access member \"send\" for type \"ForumChannel\"\n  Member \"send\" is unknown",
	"source": "Pylance",
	"startLineNumber": 13,
	"startColumn": 19,
	"endLineNumber": 13,
	"endColumn": 23
},{
	"resource": "/c:/Users/kupok/Documents/Code/writingHelper/writingHelper/writingBot copy.py",
	"owner": "_generated_diagnostic_collection_name_#2",
	"code": {
		"value": "reportGeneralTypeIssues",
		"target": {
			"$mid": 1,
			"external": "https://github.com/microsoft/pyright/blob/main/docs/configuration.md#reportGeneralTypeIssues",
			"path": "/microsoft/pyright/blob/main/docs/configuration.md",
			"scheme": "https",
			"authority": "github.com",
			"fragment": "reportGeneralTypeIssues"
		}
	},
	"severity": 8,
	"message": "Cannot access member \"send\" for type \"CategoryChannel\"\n  Member \"send\" is unknown",
	"source": "Pylance",
	"startLineNumber": 13,
	"startColumn": 19,
	"endLineNumber": 13,
	"endColumn": 23
},{
	"resource": "/c:/Users/kupok/Documents/Code/writingHelper/writingHelper/writingBot copy.py",
	"owner": "_generated_diagnostic_collection_name_#2",
	"code": {
		"value": "reportGeneralTypeIssues",
		"target": {
			"$mid": 1,
			"external": "https://github.com/microsoft/pyright/blob/main/docs/configuration.md#reportGeneralTypeIssues",
			"path": "/microsoft/pyright/blob/main/docs/configuration.md",
			"scheme": "https",
			"authority": "github.com",
			"fragment": "reportGeneralTypeIssues"
		}
	},
	"severity": 8,
	"message": "Cannot access member \"send\" for type \"PrivateChannel\"\n  Member \"send\" is unknown",
	"source": "Pylance",
	"startLineNumber": 13,
	"startColumn": 19,
	"endLineNumber": 13,
	"endColumn": 23
},{
	"resource": "/c:/Users/kupok/Documents/Code/writingHelper/writingHelper/writingBot copy.py",
	"owner": "_generated_diagnostic_collection_name_#2",
	"code": {
		"value": "reportOptionalMemberAccess",
		"target": {
			"$mid": 1,
			"external": "https://github.com/microsoft/pyright/blob/main/docs/configuration.md#reportOptionalMemberAccess",
			"path": "/microsoft/pyright/blob/main/docs/configuration.md",
			"scheme": "https",
			"authority": "github.com",
			"fragment": "reportOptionalMemberAccess"
		}
	},
	"severity": 8,
	"message": "\"send\" is not a known member of \"None\"",
	"source": "Pylance",
	"startLineNumber": 13,
	"startColumn": 19,
	"endLineNumber": 13,
	"endColumn": 23
}]

 Logs

pylance.log

The method get_channel returns objects of different types: VoiceChannel, StageChannel, ForumChannel, TextChannel, CategoryChannel, Thread, or PrivateChannel. It can also return None. To safely use the value assigned to local variable channel, your code must first verify that it received an object of the expected type. If you receive another type, your code will crash at runtime because send is not a valid method for many of these object types. Pylance is telling you about this bug in your code.
If you're sure that the channel ID that you pass to get_channel will always refer to a TextChannel, you can add the line assert isinstance(channel, discord.TextChannel). This will verify your assumption at runtime and tell Pylance that you are making this assumption.

Thank you so much for your help!


Object monkey patches effects are way too far spread

Currently the Object monkey patches effects are way too far spreading. Because almost every other type inherits from Object, this means all the Object monkey patches are now in Strings, Arrays and more. This will lead to weird behaviour.
I am going to have to move all the Object methods under a rearmed() method. This will limit the effects to only this one method.

This is now implemented on the develop branch. Testing to follow.
If testing goes well we can move to master and release v1.1.0


Update index via external component

Hi Olivier,
I'm trying to set the swipeable view to a specific index from an external component. As I'm utilising virtualize to enable a circular swipe-able view, I found its possible to access the 'setWindow' function using the following...
<VirtualizeSwipeableViews ref={swipeableView => {this.swipeableView = swipeableView}} />

... and setting the index using...
this.swipeableView.setWindow(index)

Though this works, the index value is not persisted such that swiping the carousel continues from this.state.index.
Possible Solution
Looking in to the virtualize.js code, setWindow can accept an external variable (which defaults to this.state.index) as per ...
setWindow(index = this.state.index) {
    ...
}

However as you can see, this.state.index is not updated.  The simplest solution is to update the index when updating the indexContainer/indexStart/indexStop.
setWindow(index = this.state.index) {
      const { slideCount } = this.props;

      let beforeAhead = this.props.overscanSlideBefore;
      let afterAhead = this.props.overscanSlideAfter;

      if (slideCount) {
        if (beforeAhead > index) {
          beforeAhead = index;
        }

        if (afterAhead + index > slideCount - 1) {
          afterAhead = slideCount - index - 1;
        }
      }

      this.setState({
        index,
        indexContainer: beforeAhead,
        indexStart: index - beforeAhead,
        indexStop: index + afterAhead,
      });
    }

I'm happy to create a branch and submit a PR.

[x] I have searched the issues of this repository and believe that this is not a duplicate.

 Don't use them. In your case, you can control the index property.


[Credential Lifecycle - Status Attestation ] - Functional Requirements - Issuer

The text states: "  The issuer Must .... revoke a Digital Credential when ....the Wallet Instance that holds the Digital Credential was issued is revoked;"
It is not clear where is the relation between the  Wallet Istance (that may have been revoked and replaced with another version) and the Digital Credential as the credential validity should be independent from the wallet: the credential is bound to the user Key that may be generated by an external WCSD or remote signing device.
It would be preferable to change the text with the following one: "  The issuer Must .... revoke a Digital Credential when ....the cryptographic key material to which the issued Digital Credential is bound is lost or replaced;"
In case also the following statement should be replaced with: "Loss of cryptographic key material to which the issued Digital Credential is bound"

I would therefore replace the original text with the following: "The issuer Must .... revoke a Digital Credential when ....the holder request such revocation due to loss or replacement of cryptographic key material which the issued Digital Credential is bound to".
As loss of crypto key/s is something that only holder would be aware of, he has to officially perform the request.


example_cli_app failed right at the first run on Windows 10

Host
Tell us about your Host OS (the machine you're building burrito apps on)
Host OS: Debian 12
Host CPU: x86_64
Please run zig version in your Burrito project directory and write the version here:
0.13.0
Please run elixir -v in your Burrito project directory and write the version here:
1.15.5

Target
Host OS: Windows 10 Build 19045
Host CPU: x86_64
Host LibC and Version : Sorry, not sure how to check that in Windows!

After successfully done MIX_ENV=prod mix release, I copy the example_cli_app_windows.exe to the Windows 10 machine, open PowerShell and do:
.\example_cli_app_windows.exe

I got this error immediately:
Program 'example_cli_app_windows.exe' failed to run: Access is deniedAt line:1 char:1
+ .\example_cli_app_windows.exe
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~.
At line:1 char:1
+ .\example_cli_app_windows.exe
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ResourceUnavailable: (:) [], ApplicationFailedException
    + FullyQualifiedErrorId : NativeCommandFailed

Then the file example_cli_app_windows.exe just disappeared. It seemed it was deleted!!
In the README of the Burrito homepage, looking at the requirements for Windows, it says:
Windows
- MSVC Runtime for the Erlang version you are shipping
- Windows 10 Build 1511 or later (for ANSI color support)

I think the 2nd point is guaranteed as my target Windows build is greater than 1511. The 1st point, however, I don't know how to achieve it. Could anyone please elaborate on it?
As I moved on from Bakeware and found Burrito, I am really expecting this to work.
Thanks a lot for any help!

I cannot reproduce this:

Then the file example_cli_app_windows.exe just disappeared. It seemed it was deleted!!

This sounds like either Windows or some other anti-virus software deleted the file because it mistook it for something malicious.
You can ensure you have the MSVC runtime installed by using my handy site: https:// - Install the 2019, 2017, 2015 version for your architecture.

Thanks for your quick response. I installed MSVC from your site. But still got the same error i.e., Access is denied At line: 1 char:1 .... The target machine has Crowdstrike Falcon Sensor installed. So it could be because of that? Unfortunately, I am not allowed to turn that off.
So beside that error, another question: Is MSVC a hard requirement? Because my user will have absolutely no idea what that is, how to install and why do they have to install that!
Thanks a lot.

The target machine has Crowdstrike Falcon Sensor installed. So it could be because of that? Unfortunately, I am not allowed to turn that off.

That's exactly why it's not working. Crowd Strike dislikes burrito. When I was first developing it, I ran into this issue and had to request that my IT department add an exception for the program and the directory where it extracts the files to.

So beside that error, another question: Is MSVC a hard requirement? Because my user will have absolutely no idea what that is, how to install and why do they have to install that!

MSVC is a hard  MSVC, however it's required for Erlang to run at all, so it's not a Burrito requirement, it's an Erlang requirement. Without it Bakeware wouldn't run either, or any Erlang install. The good news is most new Windows 10/11 installs will already have this installed.

so, to avoid crowdstrike, I installed a fresh Windows 10 Home Edition in a Virtualbox VM. The app now seems to run, but nothing appears, so I am not sure now...
PS C:\Users\CT\Downloads> dir

    Directory: C:\Users\CT\Downloads

Mode                 LastWriteTime         Length Name
----                 -------------         ------ ----
-a----        12/16/2024   6:48 PM       18354176 example_cli_app_windows.exe

PS C:\Users\CT\Downloads> .\example_cli_app_windows.exe
PS C:\Users\CT\Downloads>

That is the output, after pressing Enter, it hangs around 30 seconds then returns without any output like that. Can you please advice further?
Thanks a lot!

Ah yes eventually Windows updates will install the redistributable, or any other modern application. It's required by many.
The only way around CrowdStrike is you have your IT department whitelist the application. I cannot do anything to mitigate it, CrowdStrike is aggressive and usually has tons of false positives. I suggest talking to whoever manages your system deployments.


Is data persistent between app updates?

So according to the wiki the plugin uses SharedPreferences for Android and NSUserDefaults for iOS, so yes the data should persist until the user deletes the Cordova app.

@GillesC I think this a good question to be included in F.A.Q Thankyou @PierBover

@alokrajiv Indeed, I will open a F.A.Q. section and put all (including these received by mail) the recieved questions there. @PierBover thank you for the question, this will also be beneficial to other users.


Verbose logging of exceptions if continue_after_error

log ignored exceptions more verbosely including exception value and stack trace
try to address #163 (find out reason for the permission error)

Thanks for the PR! I estimate that there will be a new pypi version beginning of August, in the meantime, your changes are already in the master :-)


[sigh] expand_path on resign

expand_path on resign so ~/Downloads/1.mobileprovision will work.
as discussed here: https://github.com/fastlane/fastlane/issues/6986

@milch - added a comment to clarify passed param.
also renamed fst, snd->app_id, app_id_prov`

I'm not sure that comment is always correct, is it really always a hash? I think it's either a hash or an array, though I'm not a 100% on that.

@milch  updated comment


Duplicated imported code

It seems that if you import the same file from multiple files it ends up beeing duplicated in the final bundle. Is this a known issue?
It seems that the sass-loader also had this problem (https://github.com/jtangelder/sass-loader/issues/30) and resolved it here: https://github.com/jtangelder/sass-loader/issues/31

cssnext "just" inline import statement, so it's a normal behavior. The problem is, if we handle the imports in a different way (via webpack instead of inlining), we might loose some variables definition for example.
So that's kind of tricky to choose what to do.
(For my concern, I don't really use imports when using cssnext-loader. I have some global variables defined in cssnext configuration & that's enough for my needs.)
I am open to the discussion & pull requests but that is not easy to "fix".

What do you exactly mean by:

global variables defined in cssnext configuration

How is this done?

http://cssnext.io/usage/#features
in webpack config
module.exports = {
  entry: "path/to/entry",
  // ...
  cssnext: {
    features: {
      customProperties: {
        variables: {mainColor: "#f00"}
      }
    }
  }
}

Gotcha, yeah, I was looking at the postcss-custom-properties plugin right now. Thanks!

Is it possible to declare custom media query extensions this way as well? The customProperties object works perfectly, but defining the customMedia object to work the same way as the postcss test fixture doesn't work :(
// ...
    cssnext: {
        browsers: "last 2 versions",
        features: {
            customProperties: {
                variables: {
                    'main-color': "#f00"
                }
            },
            customMedia: {
                extensions: {
                    'viewport-testing': '(min-width: 500px)'
                },
                appendExtensions: true
            }
        }
    }
// ...

yes it seems this should works. I guess tests are wrong :/
I opened an issue for that https://github.com/postcss/postcss-custom-media/issues/11


Cant access property flags from frame.pyx

Im trying to add:
    property is_corrupt:
        def __get__(self): return bool(self.ptr.flags & lib.AV_FRAME_FLAG_CORRUPT)

to av/frame.pyx
But upon make it gives the following error:
Error compiling Cython file:
------------------------------------------------------------
...
                return avrational_to_faction(&self._time_base)
        def __set__(self, value):
            to_avrational(value, &self._time_base)

    property is_corrupt:
        def __get__(self): return bool(self.ptr.flags & lib.AV_FRAME_FLAG_CORRUPT)
                                              ^
------------------------------------------------------------

av/frame.pyx:125:47: Object of type 'AVFrame' has no attribute 'flags'

According to all ffmpeg specification i have found flags should always exist:
https://www.ffmpeg.org/doxygen/2.3/structAVFrame.html
I'm probably doing something stupid, need some help please.

Cython doesn't know about the property. You have to add it here -> https://github.com/mikeboers/PyAV/blob/master/include/libavcodec/avcodec.pxd#L200-L226
🍻

Spot on, thanks! If I could fax you a beer I would.


Notification is not removed using removeDeliveredNotification on Android

Notification is not removed from notification center when tag property is specified:
FCM.presentLocalNotification({
  id: 'some_id', // (optional for instant notification)
  title: 'My Notification Title', // as FCM payload
  body: 'My Notification Message', // as FCM payload (required)
  tag: 'some_tag', // Android only
  show_in_foreground: true,
})
// ...
FCM.removeDeliveredNotification('some_id')

It works fine without tag property. If this is not a bug, it would be nice to state it in documentation.

it is a bug. seems tag is part of the id

@evollu since notification id can be any string, I find it also redundant. However removing tag support might be breaking change if somebody use it.

it will release it as v10 and state breaking change in release. It is buggy today anyway

released v10


Linker errors in TS_NJKWebViewProgress

We found that you use NJKWebViewProgress class in you code.
https://github.com/ninjinkun/NJKWebViewProgress/blob/master/NJKWebViewProgress/NJKWebViewProgress.m
In your code this class is renamed into TS_NJKWebViewProgress to prevent collisions. But the global constant completeRPCURLPath (from this class) is not renamed. So when we try to use other third-party libraries in our code - We get linking errors.
Could you please rename the completeRPCURLPath constant in your code ?

Thanks for addressing this. We will release an update soon.

duplicate symbol _completeRPCURLPath in:
    /Users/Will/Library/Developer/Xcode/DerivedData/InterstitialAdSample-cfpooxcmjabpaufxlbyrlnhaadlo/Build/Intermediates/InterstitialAdSample.build/Debug-iphonesimulator/InterstitialAdSample.build/Objects-normal/x86_64/NJKWebViewProgress.o
    ../../TapSenseAds/TapSenseAds.embeddedframework/TapSenseAds.framework/TapSenseAds(TS_NJKWebViewProgress.o)
ld: 1 duplicate symbol for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

This has been fixed in the latest release (3.2.2). Thanks again for pointing it out.


Manage services through workflow ui

It's possible to create a UI to map services e manage them through workflow?
Example, using this libray: http://sebastien.drouyer.com/jquery.flowchart-demo/#advanced
It's interesting, or viable functionality?

@jefferson thanks for your interest in the project! Yes this would be possible either by editing configuration.json or using Ocelots admin API. Docs for that are here http://ocelot.readthedocs.io/en/latest/features/administration.html
A few people have previously built configuration generators of various forms.

Here is a previous issue that is similar https://github.com/ThreeMammals/Ocelot/issues/116 the comment by ravinsp on 23 Nov 2017 might be useful.
Here is the editor https://github.com/dbarkwell/Ocelot.ConfigEditor

Hi,
I'm starting a small project to build this feature with a small initial scope: to produce static routes based on ReRoutes in configuration.json
Here is the project: https://github.com/jefferson/route-graph-visualization


repo sync

This is an automated pull request to sync changes between the public and private repos.
:robot: This pull request should be merged (not squashed) to preserve continuity across repos, so please let a bot do the merging!

This is an automated pull request to sync changes between the public and private repos.
🤖 This pull request should be merged (not squashed) to preserve continuity across repos, so please let a bot do the merging!

Off


kythe tool: --log_requests global flag [phab:D341]

Created by schroederc at 2015-06-15 23:33:38:

schroederc wrote at 2015-06-15 23:40:31:
Fix release notes

zarko accepted this revision at 2015-06-15 23:43:58

schroederc closed via commit 90edd14f1fd54daca767070c5093b2615655bc84 at 2015-06-16 00:49:23


Add host header to fix compatibility with docker 1.12

This header is now required.
See https://github.com/docker/docker/blob/master/docs/breaking_changes.md#engine-112

yeah, this library is broken with Docker 1.12.

+1 to this. This fixes Docker 1.12 compat issues.

Hey, sorry for the delay, but i cannot accept this as the host header is wrong when the docker remote is not on local, i will push something that fix this.


Dropdown of model subtype does not work in CMS

When adding interactive elements the model subtype is needed to filter the correct assets. The dropdown does not work atm. The former workaround (saving the element and reopening it) does not work either.

linked to #591

Closed because fixed in #591


Add Zeitwerk autoloader

I'm looking at the implementation in other gems and have a question: would it work if we will make zeitwerk a soft dependency?
I mean, for the zeitwerk environments it's already added to Gemfile, so we may do something like:
require "zeitwerk" rescue nil

class Inflector

  if defined?(Zeitwerk)
    def self.loader
      # ...
    end
  end

If so, the power of Zeitwerk will be added without introducing new dependencies for those who don't need it

This wouldn't be enough. We'd also have to make explicit requires optional depending on the presence of Zeitwerk.


INITIALIZE action using wrong form prop with new initialValues

It seems like the initialize process uses this.props.form, and nextProps.initialValues, when re-initializing. This can cause the new initialValues to be set on the wrong form identifier.
Take this line:
https://github.com/erikras/redux-form/blob/master/src/reduxForm.js#L124
this.props.initialize(nextProps.initialValues, keepDirty)

this.props.initialize grabs the form property from this.props, but we are sending in the nextProps.initialValues object. Thus, if the form property and the initialValues property change on the same re-render, the new initialValues will be set on the old form identifier.
Does this make sense?

We just got bit by this again. Should nextProps.form be used, instead of this.props.form?

@erikras I've added a test, which you can pull down and run, to demonstrate the issue:
https://github.com/marbemac/redux-form/commit/711d35abf0e5058a40996dccd34f0416750d30bd
Let me know if something is off in that test. I tried making the change I suggested above (nextProps instead of this.props in that spot), but unfortunately it did not work.

I'm not sure what you mean by open a formal bug request, could you elaborate? And this is not really appropriate for stack overflow - it's not a help question, I even provided a reproducible test to demonstrate the issue.

Sorry, I was looking at forgotten issues mainly 😊

Because this issue saw almost no activity for a few months, I'm closing it.


Fix metadata issue with release notes.

OPS Build status updates of commit af149d8:
:clock10: Incremental building: average incremental building time is 13 min(s) 22 sec(s)


[MNK] Update Riddle of Fire to BuffWindow

Update the RoF module to BuffWindow as suggested in #1303
Current issues I could see:

Brotherhood window desync with RoF. Right now I'm just relying on odd/even window indexes
Wrong blitz usage. I'm not looking that the person did a solar-lunar or just solar-solar, just that they didn't use Celestial Revolution

My goal was to check this with the top monk logs and only see green, which I do. So a "perfect" monk rotation should look like this.

@Malpp The lint error from the build is because you need to run yarn extract and commit the changes that will make to the locale files (due to changing the body of the suggestion content in commit 6d8df95).
Also, if you can merge the endwalker branch back into your work branch (or rebase, whichever approach you prefer) and sort out the merge conflicts with the contributor & changelog files, we can poke the necessary reviewers to get the rest of it moving.


Fix workflows

Here you go ;-)
Hope this works 🤞

Do you understand this? According to common sense and also the docs here, I would have said, it should be fine now... https://www.shellcheck.net/wiki/SC2086

On a short glance, no.
The template for a home brew tap GitHub action comes from here:
https://github.com/Homebrew/brew/blob/eb680e64357279461224183cc3928ecd66de2c12/Library/Homebrew/dev-cmd/tap-new.rb#L139
and I don’t see any double quotations there (still).
Not sure why the shellcheck even runs on the workflow yaml file. Where does the shellcheck come from?  🤔

Another day, another try.  Let's see if shellcheck is happy now 🤞


Do we have a sample for WebChat client mode?

Do we have a sample for The following shows a normal and a fallback flow when using a WebChat client.?
I have checked the code for VA(https://docs.microsoft.com/en-us/azure/bot-service/bot-builder-authentication-sso?view=azure-bot-service-4.0&tabs=csharp%2Ceml) and it is very complex. Need a sample for how to do it in WebChat client mode.

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

ID: 703a4f53-d548-76ed-8c8a-a1c8e6f17577
Version Independent ID: 7311af4b-6943-fa86-ebb7-50f3bc9f39ba
Content: Single sign on Azure Bot Service - Bot Service - Bot Service
Content Source: articles/v4sdk/bot-builder-concept-sso.md
Service: bot-service
GitHub Login: @Kamrani
Microsoft Alias: kamrani

It seems this is quite close https://github.com/microsoft/BotFramework-WebChat/tree/master/samples/07.advanced-web-chat-apps/e.sso-on-behalf-of-authentication

@xieofxie Pleas look at this sample.

@mmiele yes, it worked. Maybe someone could update the link to the document. I will close this now.


mvn build of command line OWL tools on OSX files with missing file - caro.owl

From<EMAIL_ADDRESS>on December 17, 2013 12:13:37
<b>What steps will reproduce the problem?

mvn clean package
<b>2.
<b>3.

<b>What is the expected output? What do you see instead?

Failed tests:   testRender(owltools.renderer.markdown.MarkdownRendererTest): Requested resource does not exists: src/test/resources/renderer/caro.owl

<b>What version of the product are you using? On what operating system?

OSX 10.8.5

<b>Please provide any additional information below.

I find multiple caro.owl files in the distribution. But this looks like it might be missing the subdirectory rederer in addition tot he caro.owl file.

Original issue: http://code.google.com/p/owltools/issues/detail?id=85

From<EMAIL_ADDRESS>on December 17, 2013 12:30:41
This is fixed in revision r1823 .
Status: Fixed


Add eth_getStorageAt acceptance test coverage

Problem
The relay added support to eth_getStorageAt to call the mirror node.
However, no E2E tests could be added at the time
Solution
Add eth_getStorageAt acceptance test coverage

ensure the services node is updated to support side car propogation of state changes

Alternatives
No response

state_changes not yet flowing from mirror node. Will have to wait

State changes will be flowing in 0.30 services and Mirror Node 0.65.0.
Notably the eth_getStorageAt calls can currently only supports the case where the slot in question was modified in the latest transaction for the given contract.
In other scenarios no state change will be found until Mirror Node adds a contract_state table.
This ticket should add tests for the first case and we can add more when the full implementation is added.


How to run this on a production app

I tried to run this in production, but I get
{"Kernel pid terminated",application_controller,"{application_start_failure,scotchy,{{shutdown,{failed_to_start_child,'Elixir.Scotchy.Endpoint',{shutdown,{failed_to_start_child,'Elixir.Phoenix.Endpoint.Server',{shutdown,{failed_to_start_child,{ranch_listener_sup,'Elixir.Scotchy.Endpoint.HTTP'},{shutdown,{failed_to_start_child,ranch_acceptors_sup,{listen_error,'Elixir.Scotchy.Endpoint.HTTP',eaddrinuse}}}}}}}}},{'Elixir.Scotchy',start,[normal,[]]}}}"}

Crash dump is being written to: erl_crash.dump...done
Kernel pid terminated (application_controller) ({application_start_failure,scotchy,{{shutdown,{failed_to_start_child,'Elixir.Scotchy.Endpoint',{shutdown,{failed_to_start_child,'Elixir.Phoenix.Endpoint

Hmm... It seems like that your phoenix server has launched on duplicate port while someone has launched on the same port.

Yeah seems i had to stop the server first, thanks!


NETC-51: Use reflection to find types for the Service/Proxy Ticket Manager/Validator

Allow for type aliases for easy configuration and backwards-compatability in the casClientConfig.
As an example here are some ways you can configure a ticketValidator
This still works because of the aliasing feature:
ticketValidatorName="Saml11"

Or you can do this:
ticketValidatorName="DotNetCasClient.Validation.TicketValidator.Saml11TicketValidator"

Or something like this should work to reference a validator in an external dll:
ticketValidatorName="MyNameSpace.Validation.MyFancyTicketValidator"

I think this is a sound change, and is similar to how the Java client works as well. I did have a question about a type cast, but other than that, I think it checks out well.


[Event Request] Table 7321 "Warehouse Shipment Line" - OnDeleteOnBeforeUpdateDocumentStatus

Hi, would it be possible to get an OnDeleteOnBeforeUpdateDocumentStatus event in table 7321 (Warehouse Shipment Line), like this:
    trigger OnDelete()
    var
        ItemTrackingMgt: Codeunit "Item Tracking Management";
        OrderStatus: Option;
        IsHandled: Boolean;
    begin
        TestReleased;

        if "Assemble to Order" then
            Validate("Qty. to Ship", 0);

        if "Qty. Shipped" < "Qty. Picked" then
            if not Confirm(
                 StrSubstNo(
                   Text007,
                   FieldCaption("Qty. Picked"), "Qty. Picked", FieldCaption("Qty. Shipped"),
                   "Qty. Shipped", TableCaption), false)
            then
                Error('');

        ItemTrackingMgt.SetDeleteReservationEntries(true);
        ItemTrackingMgt.DeleteWhseItemTrkgLines(
          DATABASE::"Warehouse Shipment Line", 0, "No.", '', 0, "Line No.", "Location Code", true);

        // New Event begin
        IsHandled := false;
        OnDeleteOnBeforeUpdateDocumentStatus(Rec, IsHandled);
        if not IsHandled then begin
        // New Event end
            OrderStatus :=
              WhseShptHeader.GetDocumentStatus("Line No.");
            if OrderStatus <> WhseShptHeader."Document Status" then begin
                WhseShptHeader.Validate("Document Status", OrderStatus);
                WhseShptHeader.Modify();
            end;
        end;
    end;

    [IntegrationEvent(false, false)]
    local procedure OnDeleteOnBeforeUpdateDocumentStatus(var WarehouseShipmentLine: Record "Warehouse Shipment Line"; var IsHandled: Boolean)
    begin
    end;

Thanks!

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


FIX: DataQuery::applyRelation using incorrect foreign key (fixes #3954)

In the case of has_one, the foreign key is always the relation name!

Looks like a good fix!


Could not create a native instance

Hi, I'm getting the following error in execution time on simulator:
Could not create an native instance of the type 'Xamarin.iOS.DGActivityIndicatorViewBinding.DGActivityIndicatorView': the native class hasn't been loaded.
It is possible to ignore this condition by setting ObjCRuntime.Class.ThrowOnInitFailure to false.

Thanks! :)

I've tested it now on a new project, in simulator in Release and in Debug, and it worked well.
Can you tell me the Xamarin / Ios versions you have? If you want to share the source code with me please feel free to do it.

I changed the supported architecture from i386 to x86_64 and it worked perfectly! Thanks, awesome binding.

@ceichin were did you changed the supported architecture?

Sorry for the late response. In Xamarin Studio you do right click on the iOS project -> Options -> Tab iOS Build -> Set the Supported Architecture (my options are i386, x86_64 or both). I suppose the .a file of the binding does not support the i386 architecture, but I don't know if that is really needed, I just changed that and everything worked.

The fat library (.a) was created targeting i386, armv7 and arm64. You should't have problems using it. I created a new Xamarin iOS project pulled the nuget and it worked well in device and emulator.

Weird.. I created a new iOS single view project, installed the nuget and just added the spinner to the ViewControllers's View. By default it works perfectly, but when changing to i386 it throws the error. Maybe I need to do something more when changing to i386... Idk, but its working for me now :)
Just to give you more info, it was tested on the iPhone SE simulator with iOS10.3, XCode 8.3.

Why did you need to change the target to i386? I don't need to do that to run on emulators.

Ok. Im gonna close it. Thank you.


Handle entities without categories

After learning that mbio has some entities that have no categories (and sometimes no variables!), we looked for a way to gracefully handle the situation.
This PR implements a change that searches all roots, instead of just the first root, for a good variable (credit @jtlong3rd and @dmfalke). Additionally, if the root is not a category, then it is a variable and so should be returned as-is.
@jtlong3rd @dmfalke is there a reason we wouldn't want to include these top level variables (non-category variables that sit directly under the entity)? It would be great to fix this problem for mbio, but if we're breaking how we expect the data to look, i'm happy to wait for the data to be reloaded without this entity.

Hi @asizemore! I just approved https://github.com/VEuPathDB/web-eda/pull/863, which will likely obviate this PR. When @jtlong3rd shared a link to this PR, I didn't realize it was awaiting my review! I think this PR will also allow for entities with only a single child. If you find this to be the case, then please close this PR. Thanks!

Will do! Thanks so much @dmfalke!!

Empty entities and those with a single child were removed from the mbio data. Closing this issue.


Implement new Storage for API Objects

As discussed in https://github.com/amazeeio/lagoon/issues/29 we want to rebuild the Storage of the API
At the same time we also create new Objects, in this hierarchy:

customer (the current client, stores information about the customer)

project (each project, like each git repository)

openshift (information about the openshift that the project is pushed too)

ssh keys can be referenced from customer and from project
customer

key
type
description
example value

name
String
Unique. name of the customer.
amazeeio

ssh_keys
reference to ssh_keys object
can be referencing to multiple ssh keys, this ssh key will have access to all projects of this customer

comment
free text
some comment about the client

created
date
time of creation date of customer. Date format tbd.

private_key
SSH Private key
ssh private key for this specific user, will be used during the deployment to access the git repositories that should be deployed

project

key
type
description
example value

name
string
Unique. name of the project
awesomewebsite

client
reference to client
reference to the client of this project

ssh_keys
reference to ssh_keys object
can be referencing to multiple ssh keys, used to allow specific ssh keys only access to a single project

git_url
string
git url of the project, needs to be in ssh format
git@github.com:amazeeio/awesomewebsite.git

slack_webhook
URL
URL of the Slack Incomming Webhook
https://hooks.slack.com/services/AAAAAAA/BBBBBBBB/CCCCCCCC

slack_channel
String
Name of the Slack Channel to send notifications to
mychannel

active_systems_deploy
String
Name of the active system for deployment
lagoon_openshiftDeploy

active_systems_remove
String
Name of the active system for removals
lagoon_openshiftRemove

branches
String
Regex of branches to be deployed, default is all branches
^(master|staging)$ or .*

pullrequests
Bool
Enable or disable pull request builds, default: false
true, false

openshift
Reference to OpenShift Object
Used to define to which OpenShift this project should be deployed too

openshift

key
type
description
example value

name
string
unique, name of the openshift server

console_url
URL
URL of the console to connect to
https://console.appuio.ch

registry
Domain String
Domain (not full URL) of the docker registry to push to
registry.appuio.ch

token
JSON Web Token
token of the service account to use to create openshift resources.

username
String
Username of the OpenShift User that should be used to create openshift resources
foo

password
String
Password of the OpenShift user that should be used to create openshift resources
bar

router_pattern
String
String with the router pattern that will be used on that specific OpenShift server, has two substitutions: ${project}, ${environment} that will be subsituted automatically
${project}.${environment}.appuio.amazee.io

project_user
String
OpenShift Username that should also be given access too when creating a new project
<EMAIL_ADDRESS>

SSH Key

key
type
description
example value

name
String
Unique. name of the ssh key, most probably an email address, but can be any string.
<EMAIL_ADDRESS>

key
ssh key
the actual ssh key, with no type or email address at the end
AAAAC3NzaC1lZDI1NTE5AAAAICtH4WLYkj55uZ/cLtTjnb0QbutYX1xBJbUzpRhBXeq3

type
ssh key type
the type of the ssh key, by default ssh-rsa
ssh-rsa, ssh-ed25519

Do we want to be chat system agnostic? Is it worth replacing slack_webhook / slack_channel with a foreign key to chat platform table?

@rtprio
good idea, I think that would make sense and also be more ready for the future.


[CPAL] XML enhancements

Just an improvement and an addition to the XML output of the CPAL table.

Coverage decreased (-0.03%) to 71.225% when pulling f502bbba985ae4bd199e1d56b97fc18a02735f0a on miguelsousa:cpal into 6ae2a9200ee665c244dbc045d3d4778129f08c06 on behdad:master.

LGTM, but can't merge from my phone as Appveyor has failing status (it simply timed out).


card verification

how can i verify if the card and the cvv are correct before doing the payment

Hey, what do you mean by "correct"? There is no way to know until you connect with the user's bank, and therefore you need to do a payment before knowing.


Change ssh config for devl to use ProxyCommand

This is the config I was able to get to work on my mac laptop and linux desktop. The other method never worked for direct "ssh lsst-devl01" connections.

I think this is basically a "I use Kerberos" versus "I refuse to use Kerberos" debate, so the solution is probably to have both configs rather than try to replace the one that's there.

(Branch names in this repo should still follow DM workflow standards, i.e. u/parejkoj/ssh-config-ProxyCommand.)

FWIW, old configs (more or less) and Kerberos have always worked for me.  Forwarding the SSH agent is something that I stopped doing when passphrase-based SSH stopped being possible at NCSA, because it stopped Just Working (may have been fixable and I was just lazy); I've switched to HTTPS/personal-access-token authentication for my git repos at NCSA.

Thank you for the responses. I'm a non-Kerberos user. I dug into it a bit more:

ProxyJump works fine, so I've replaced the ProxyCommand lines.
The GSSAPI* yes lines do not cause problems with non-Kerberos use.
The PreferredAuthentications gssapi-with-mic causes non-Kerberos ProxyJump to fail. That line removes the password option entirely, but it looks like the lsst-* machines have gssapi-with-mic as their first option anyway. If users have Kerberos active, it looks like it would get used anyway?

Suggestions on whether to have the Kerberos lines commented out (or just the PreferredAuthentication one?) with a note, or to leave them uncommented and have a note that non-Kerberos users should comment out that line? Or do we even need that?
I'll make a properly named branch/PR once the discussion here settles. I wish we could control what GitHub offers as the default name, to make that more obvious.

Yeah, setting PreferredAuthentications that way says to only use Kerberos. I would be hopeful that it would work for both Kerberos and non-Kerberos users if you just omitted that line. I wonder if that's the root of why the current configuration doesn't work for you.

@rra : yes, that was my point with bullet 3. PreferredAuthentications makes this config not work for non-Kerberos users, the other options don't seem to matter. Given the default server config, I wonder if it's even necessary at all for those using Kerberos?

Right, I would hope that it's not. If you don't have Kerberos or if Kerberos fails, it should just fall back to the next mechanism, and if the servers are advertising GSSAPI first, it should try it first.

Closing this, as it doesn't apply to usdf at all.


Driver for STM32F3xx-microcontrollers

Pull request includes initial CAN and Flash drivers for STM32F3-family of microcontrollers. CAN driver is based on the existing STM32F1 driver in the reposirory and the Flash-driver is based on the Atmel SAM3X Flash driver.
A custom-made STM32F334-based board was used in the development and things seem to work in basic desktop testing with simple CANOpen bus of two devices. No testing in real CANOpen environment is done yet. Any feedback or improvement ideas are welcome!

Hi Petteri,
Thank you for your contribution. I merged it into master branch.
I'm using PIC32. It is most polished driver, you may look for some ideas there.
I can't test STM, so I can't confirm your stack.
Maybe you can provide a basic example of STM32F3 usage. Something similar as https://github.com/CANopenNode/CANopenPIC. It can include some description of tools and compiler used, basic description of hardware, basic file with main() function, project files and CANopenNode included as a git submodule. Just for testing.
It can be your repository, you just send a link.


⚠ Minecraft has degraded performance

In 01d22dd, Minecraft (https://www.minecraft.net) experienced degraded performance:

HTTP code: 200
Response time: 929 ms

Resolved: Minecraft performance has improved in 3f0710e.


ModSecurityIIS_2.9.5 HTTP Error 500.19 after uninstall

Installed version ModSecurityIIS_2.9.5-64b.msi
Issue :
After uninstall ModSecurityIIS_2.9.5 all websites return :
HTTP Error 500.19 - Internal Server Error
Detailed Error Information:
Module | IIS Web Core
Unknown
Not yet determined
0x8007000d

\?\C:\HostingSpaces\domain\domain.com\wwwroot\web.config
Config Source:
-1:
0:

Hi @mmojadad ,
I just tried that with my test setup (right-click on the installer file, then select uninstall).
The uninstall proceeded without error. And I accessed an IIS test page afterwards with no problem.

Is this still a problem for you?
If so, I'd suggest looking at what other error messages you may be getting (particularly in Event Viewer). Beyond that, have a look the online documentation ( https://docs.microsoft.com/en-us/troubleshoot/developer/webapps/iis/health-diagnostic-performance/http-error-500-19-webpage ) and see if anything makes sense.
If you still cannot resolve it, or generate any extra information that you can share, and you are having an ongoing problem, then alternative you could consider is to not uninstall ModSecurity but simply to turn it off (SecRuleEngine Off).

@martinhsv
Hi
I found reason of the issue and fixed it.
When uninstall ModSecurity , Still ModSecurity tags remain in web.config of all websites , I manually removed <ModSecurity enabled="false" ...> and fixed.

Okay. Good to hear that you resolved your issue.
Thanks for updating here too in case that's helpful to someone in future.


Support for DICOM WSI concatenations

Hello there!
We were wondering if support for DICOM WSI concatenations would be implemented in the future. Some WSI scanners generate those by default and switching is not always straightforward.
Thanks a lot for your amazing work!

Hi @petroslk,
This would need a chunk of new code, so let's tag this as an enhancement.
This has been discussed on openslide, they'd probably need some new code too. dclunie posted some useful notes, see:
https://github.com/openslide/openslide/issues/511#issuecomment-1774220332

Hey @jcupitt
Indeed, we noticed that the MergeConcatenationInstances function provided in that issue actually does the trick. We used it to concatenate the 40X .dcm layers into one and read that with vips 8.15.2 compiled with openslide 4.0 and it worked. Perhaps some of this functionality could be ported over.
Thanks!

Thanks for all your great work in putting together this library -- was really a game changer for us
I'm sorry to ask but - is there any sense of if/where this is on the development roadmap?  It has become critical for us to come up with a solution
Our pathology service is entirely digital, accredited, and scans ~ 2,000 WSI a day. Currently, we're deploying in-house digital pathology algorithms into that setting using openslide + DICOM
the scanners produce dicom natively, so that they can be pushed into a shared sectra pacs. Our 3dhistec scanner produces multipart dicom files often, since we have e.g., large prostate double wide slides.
While one could in theory turn off extended offset tables in the scanner (unknown if this actually works, but anyway...) this creates two issues for us in that, (a) the workflow is already accredited, so any software modifications would require a laborious  "accreditation change notification" process to ensure that everything continues to work as expected while, (b) we'd be on the hook for saving these hug 14GB files as a single file, making transfer / storage /retreival suboptimal
My understanding is that the purpose of the concatenation and extended offset tables being introduced into the DICOM standard was for exactly this use case - is that correct?
Since it is part of the standard - I would hope it would be part of the roadmap for libraries to take it on - especially because I think in the future most scanners would disable the ability to not use them - but I of course appreciate that being a open-source project (of which I lead many) there are many other factors to take into account : )
Any thoughts would be appreciated!

@choosehappy I will let @jcupitt and @bgilbert to comment on where this is on the roadmap, or if this is on the roadmap.
But I wanted to add, for the background/awareness, that support of DICOM WSI in OpenSlide was funded from the Imaging Data Commons budget. We determined this was a critical missing piece, and supported this effort. At this point, we exhausted the budget we had available for OpenSlide DICOM WSI development. I cannot tell if/when we may have additional resources to invest into further new development of DICOM-related functionality in OpenSlide.
If this feature is critical for your organization, you may consider contributing it via PR, or allocating resources to for someone (perhaps @jcupitt is available) to help you with this implementation.
I greatly appreciate your feedback, as it is critical for defining what are the features that users need prioritized. We just need to find resources to implement this.

Hello @choosehappy, I think (from memory) extended offset tables should work in libdicom 1.1, it's just multipart DICOMs which are not supported.
This seems like a popular and missing feature, so of course we'd like to have it, it's just a question of someone volunteering (or being paid!) to do the work.

Thanks for the feedback @jcupitt, certainly appreciate the situation!
I had looked into us doing it ourselves, but its too much of a heavy lift giving our in-house skills/background

or being paid!

Interesting to hear, this could certainly be a possibility – do you have a sense of how this typically works?
i.e., is it paid consulting hours to you or someone on your team? How could we learn more (expected scope, budget, etc)?

You could find someone to do the work and I could review and merge it unpaid.
Alternatively, I'm booked until September or so, but I could take the work myself after that. I would guess (just a random guess, we should estimate more carefully before budgeting) a week's work to develop, test, document and integrate, at typical contractor rates.

Sounds reasonable to me - let me kick it up the chain

Great news from my side - received approval to move forward with requesting a quote for the development work.
How do we go about doing that?
Looking forward to working with you on this!

Oh, that is good news! Sure, mail me on jcupitt at gmail.com if you want to discuss details.

Done!

@choosehappy it is great to see interest and external support of this project that we started in IDC! Happy to see @jcupitt is interested and available to work on this.


Socail Login

Social login via Facebook, Google and Twitter functionality completed. social login issues fixed.

@chaudhryjunaid I am unable to npm install. i am not able to save new modules in package.json. it is giving me this error i.e :( peerinvalid The package<EMAIL_ADDRESS>does not satisfy its siblings' peerDependencies requirements!). i need to install these three modules i.e(passport-facebook-token, request and qs ) in order to create this pull request. These modules are installed in my system but not updated in package.json so i will reopen this pull request when ever i overcome this problem. Thanks!


Inliner case for lambdas

[X] Tests added for any new code
[X] Ran make fmt-fix (or had formatting run automatically on all files edited)
[ ] Documentation added for any new functionality
[ ] Entry added to UNRELEASED.md for any new functionality

Closes #1446

Don't forget a release note though


Stepper does not clamp against rounding errors

The Stepper, despite being (currently, see #762 ) exclusively for floats, does not seem to implement clamping against rounding errors.
Given the following compilable example (with warnings):
use druid::{AppLauncher, WindowDesc, Widget, PlatformError, LensWrap, Lens, Data, LensExt};
use druid::widget::{
    Padding, Stepper, Switch, WidgetExt, Flex, Label, TextBox, Parse
};
fn main() {
AppLauncher::with_window(WindowDesc::new(build_ui)).launch(State {
     n_value: 20.0
     });
}
fn build_ui() -> impl Widget<State> {
let mut row = Flex::row();
let n_textbox = LensWrap::new(
        Parse::new(TextBox::new()),
        State::n_value.map(|x| Some(*x), |x, y| *x = y.unwrap_or(0.0)),
    );
    let stepper = LensWrap::new(
        Stepper::new()
            .with_range(0.0, u32::MAX.into())
            .with_step(1.0)
            .with_wraparound(false),
        State::n_value,
    );
row.add_child(Padding::new(5.0, n_textbox));
row.add_child(Padding::new(5.0, stepper));
row.center()
}

#[derive(Lens, Data, Clone)]
struct State {
    n_value: f64,
}

Use the buttons to go up and down a few times, and you'll note that some values will end up being skipped.
As a note, I'm building off of 0.6.0, and would state that building off of master is something that should be discouraged (so if this is fixed in master, I request this not be closed until the release is pushed onto crates.io)

This has nothing to do with rounding errors, in fact I've had this issue 'fixed' at one point and that fix caused it to no longer work on macOS and thus was reverted.
I'll take a look if we can solve that properly by now.
The original fix (for GTK): #713
And the revert: #716

Also, I don't think building druid from master is discouraged at all, we make sure that master builds and runs at any time and using it allows catching bugs earlier during development. Keeping issues open until their solution is on crates.io sounds interesting, but also annoying (we'll have to close them all once we make a release), and I don't want to spent time on that yet, considering how fast druid is progressing. But once we reach stable 1.0 that might be something to consider.

Implementing the ideas from #859 would solve this.

No extra double or triple click events, it's merely an extra event property for those who care.

Thanks for reporting this. This is a current issue and not resolved yet.

As a note, I'm building off of 0.6.0, and would state that building off of master is something that should be discouraged (so if this is fixed in master, I request this not be closed until the release is pushed onto crates.io)

Druid is under very active development and we put in a lot of effort to keep the master branch reasonably stable. Changes don't get merged in willy-nilly. Thus right now using master directly is fine (and optional) if your app is also still under development. The master branch is expected to have fewer bugs, with the downside of regular breaking changes. Once we reach 1.0.0 our recommendation will of course change to everyone just using that.
Regarding the issue tracker, this is of course just a tool and every project chooses to use it differently. Druid uses the issue tracker to guide the development of druid itself and not as a notice board for developers using druid to build their own app. Which is not to say that such a notice board isn't useful. Indeed we've spent considerable time recently in setting up practices in building just such a notice board. You can check out the unreleased section of the changelog to see a list of all the bugs that have been fixed in master but are not available in the latest release on crates.io.


how can I compile the cpp with cuda?

when I use this command to compile the source code:
g++ -std=c++11 -O3 -I.. ../dlib/all/source.cpp dnn_metric_learning_on_images_ex.cpp  -lX11 -lpthread
it runs so slow~
so if I want to use the cuda, what can I do?

Use cmake and follow the instructions here http://dlib.net/compile.html


Feature Request: Single SPA integration

Hi Manfred,
We spoke at the end of NgConf. I was just at the EnterpriseNG summit yesterday and we were talking about MFE / Module Federation. One pain point that was discussed with the @angular-architects solution is that any non-angular applications need to be wrapped as web components. One thought I had, which is more theoretical, is if this plugin would be capable of integrating directly with components from other frameworks if they're setup with Single SPA to have a bootstrap function which can get called after it's added to the view. I'm interested in hearing your thoughts / feedback.
Thank you,
Mike Cebrian

Hi @JSMike first of all I would like to send you a greeting and I take the liberty of asking you a brief question about the use of the ng/module-federation-plugin library.
Basically it is the development of some microfrontends using Angular 14+ to use it in a mpa legacy application made with PHP + Webpack 5 (pure webpack + vanilla js), which I am trying to develop some more advanced features using Angular Material like a shopping cart and other necessary components.
Because of this I am considering the possibility of developing these microfronts using module federation plugin and consuming them in the host app via webpack federated modules
The biggest question is to know if it is necessary to use an intermediary library such as single-spa to be able to load the mf module in my host app, since I am currently using something like the following syntax to consume the mfe module from an angular host app:

Due to this I am not sure how I can communicate “webpack 5” with angular to tell it to carry out the instructions for the generation of the “component” and his corresponding “module” in order to show the “dialog material” over my “host app”, since I observe that I need to use of some “angular core” functions such as:

Hi @AlonsoK28,
Busy week, I'll come up with a more detailed reply later.
For some quick context, our immediate use case would be for routing to non-angular components that have single-spa implementation. And to leverage module federation for dynamic loading, and resource sharing.
One of my coworkers was able to do this using a simple angular component wrapper that called the single-spa bootstrap function of a child Elm component. I was looking to have a more declarative configuration way to do this instead of having to write angular wrappers every time we want to use a component from another framework.

There are no official plans for this on our side.


fix TopUtili bug

Fixes #53 .

Pull Request Test Coverage Report for Build 104

0 of 20 (0.0%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.04%) to 68.3%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

client/lookout-api/src/main/java/com/alipay/lookout/common/top/TopUtil.java
0
2
0.0%

client/lookout-api/src/main/java/com/alipay/lookout/common/top/AbstractTopMetric.java
0
18
0.0%

Files with Coverage Reduction
New Missed Lines
%

client/lookout-api/src/main/java/com/alipay/lookout/common/top/AbstractTopMetric.java
2
0.0%

Totals

Change from base Build 102:
-0.04%

Covered Lines:
2663

Relevant Lines:
3899

💛  - Coveralls