Event page initial adds

Related Ticket(s)
Description
This is to set up unlinked pages for the design summit.
Changelog
New
New pages
Changed

{{changed thing}}

Removed

{{removed thing}}

@jeffchew @ljcarot Here is the preview link: https://ibmdotcom-library.s3.us-south.cloud-object-storage.appdomain.cloud/deploy-previews/684/events/summit-2020/introduction/index.html


#41 Add Layers support by makefile build method

@buggy merge this PR please. And next I will can going to nested application support

any updates on this going to master branch ?

Won't add. I would strongly recommend using the official AWS TypeScript support or switching to the CDK.


Tags with spaces in them in search results don't link to page

Type "plug" into the search box. Now click on "plugin library". It takes you to a 404 page.

Thanks @Marxsal I can confirm the problem

This was fixed by the switch to a TW5 UI.


[run-spec] Allow examples of the D spec to be runnable: spec_version

@CyberShadow I hope it's easier to step-by-step review this as separate PRs.
My tool is far from perfect, but it looks like it has done its job.
In general I was a quite liberate with adding $(RUNNABLE_EXAMPLE) as even playing with the compile error should be helpful to a reader.
In some cases

I changed test -> main to allow execution
As it's available as book, I try to add import std.stdio where it was missing (though it doesn't matter for dlang.org as std.stdio is always imported)
Sometimes e.g. for unittests I added $(RUNNABLE_EXAMPLE_ARGS ...) - that's "new" way to clarify  arguments (there's also $(RUNNABLE_EXAMPLE_STDIN)`
This isn't intended to be complete run, but I hope a good start with ~90% of the ones that are interesting to be runnable

@CyberShadow I hope it's easier to step-by-step review this as separate PRs.

Doesn't matter as long as the changes are split up into commits and all of them are uncontroversial.

My tool is far from perfect, but it looks like it has done its job.

That looks redundant to me, but whatever fits your workflow :)

https://github.com/dlang/dlang.org/pulls
OK, this is definitely not an improvement.
Let's please never do that again.

@wilzbach Is it OK if I batch-close them and you submit one PR with one commit per file? I think I'm going to get RSI otherwise reviewing/merging these :)

OK, this is definitely not an improvement.
Let's please never do that again.

OK. Sorry. I can close everything and put into one PR?

@wilzbach Is it OK if I batch-close them and you submit one PR with one commit per file? I think I'm going to get RSI otherwise reviewing/merging these :)

Yes of course. I can close myself. I wanted to make your life easier, not harder ...

I think this approach would be better suited for things with lots of changes that require careful review and scrutiny, such as bulk automatic changes to source code that we're not sure are correct, but I understand that this isn't such a case. Even then, it's probably better to submit them incrementally, as reviewers may discover a problem with the overall technique or idea that would make the other PRs obsolete or needing an update.

Yes of course. I can close myself. I wanted to make your life easier, not harder ...

It was an experiment - I should have asked before. Sorry.
-> https://github.com/dlang/dlang.org/pull/1754


Performance: memory

There are several issues involving memory consumption.
This task involves defining the bottlenecks and understanding tradeoffs between available options.

Most egregious memory issues have been fixed. Next step will involve implementing a gstreamer plugin.


Add tests for type reference

Purpose

$title.

Fixes #34632
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

@SandaruJayawardana, can you please fix the build failure also?


[6.x] Strict parameter added to assertJsonPath

Implements the suggestion of @BrandonSurowiec and @ttrig  at https://github.com/laravel/framework/pull/30142
This function is an alias of assertJsonPath. The tests are just the same

It is probably a better idea to make assertJsonPath strict by default. @ttrig gave a good example of the results you can get when you use this assertion in a non-strict way:
{
    "foo": false
}

$response->assertJsonPath('foo', false); // pass
$response->assertJsonPath('foo', 0); // pass
$response->assertJsonPath('foo', null); // pass

You should never want behavior like this in a test.

You have a point. I think the best place to talk about it is in the original PR, https://github.com/laravel/framework/pull/30142

Doesn't really add any benefit. assertJson, which has an optional strict argument, doesn't have its own strict method.

@ahinkle yes, just like the singleton method at Container. You could just use the bind method with one "true" additional parameter

Well, this comportament is wierd to me. I implemented the suggestion by thinking that laravel has some methods like this.
If this is really not relevant I'll close the pr.

Doesn't really add any benefit. assertJson, which has an optional strict argument, doesn't have its own strict method.

One benefit could be the readability of assertJsonPath('foo', false, true) compared to assertJsonPathStrict('foo', false).
Would be nice until assertJsonPath is strict by default.

No need.


Force inline script to always log contents?

When using an inline script the actual commands are only shown sometimes?
For example the logs for this task:
- script: |
    echo "hello world"

Would show:
2019-01-30T20:53:37.4128279Z Generating script.
2019-01-30T20:53:37.4144502Z Script contents:
2019-01-30T20:53:37.4145166Z echo "hello world"

Whereas this task:
- script: |
    echo "hello"
    echo "world"

Would be logged as:
2019-01-30T20:52:29.8966200Z Generating script.
2019-01-30T20:52:29.9008810Z [command]/bin/bash --noprofile --norc /home/vsts/work/_temp/5f705bd0-5f9f-4946-b2cd-6cd611c1f1a1.sh

The second log format makes it difficult to understand when reading the logs.
Is there any way to force it to always output the script contents?

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

ID: 450ac349-5c33-6cdf-1ecd-6eb989a00334
Version Independent ID: a7119c1a-59b2-3081-a241-e93aa0d6b4b9
Content: Bash task - Azure Pipelines
Content Source: docs/pipelines/tasks/utility/bash.md
Product: devops
GitHub Login: @steved0x
Microsoft Alias: dastahel

Moved to the tasks repo: https://github.com/Microsoft/azure-pipelines-tasks/issues/9505


Uncaught Error: Assertion Failed: A helper named 'x-toggle' could not be found

I not sure why I am getting this error in a my application. Is it because I haven't added the line below to a config file. I am new to Ember and am not sure where I'd add that information. Do I add it to environment.js or create a new file in the config folder? I am currently using Ember 2.5.1
Add a configuration for ember-cli-toggle to include only the themes that you will use.
ENV['ember-cli-toggle'] = {
includedThemes: ['light', 'default', 'flip'],
excludedThemes: ['flip'],
defaultShowLabels: true, // defaults to false
defaultTheme: 'light',   // defaults to 'default'
defaultSize: 'small',    // defaults to 'medium'
defaultOffLabel: 'False',     // defaults to 'Off'
defaultOnLabel: 'True'        // defaults to 'On'
};

the config should be in config/environment.js. Did you restart your ember serve after installing addon?

I did. I ran this: npm install ember-cli-toggle and then added this {{x-toggle value=myValue onToggle=(mut myValue)}} to my template

ember install ember-cli-toggle

Yeah! Thanks so much, that worked 👍

Glad I could help!


[Bug]: Running llama2-7b on H20, Floating point exception (core dumped) appears on float16

Your current environment
PyTorch version: 2.2.1+cu121
Is debug build: False
CUDA used to build PyTorch: 12.1
ROCM used to build PyTorch: N/A
OS: Ubuntu 22.04.4 LTS (x86_64)
GCC version: (Ubuntu 11.4.0-1ubuntu1~22.04) 11.4.0
Clang version: Could not collect
CMake version: version 3.29.2
Libc version: glibc-2.35
Python version: 3.10.12 (main, Nov 20 2023, 15:14:05) [GCC 11.4.0] (64-bit runtime)
Python platform: Linux-5.4.0-106-generic-x86_64-with-glibc2.35
Is CUDA available: True
CUDA runtime version: 12.4.131
CUDA_MODULE_LOADING set to: LAZY
GPU models and configuration:
GPU 0: NVIDIA H20
GPU 1: NVIDIA H20
GPU 2: NVIDIA H20
GPU 3: NVIDIA H20
GPU 4: NVIDIA H20
GPU 5: NVIDIA H20
GPU 6: NVIDIA H20
GPU 7: NVIDIA H20
Nvidia driver version: 550.54.15
cuDNN version: Probably one of the following:
/usr/lib/x86_64-linux-gnu/libcudnn.so.9.1.0
/usr/lib/x86_64-linux-gnu/libcudnn_adv.so.9.1.0
/usr/lib/x86_64-linux-gnu/libcudnn_cnn.so.9.1.0
/usr/lib/x86_64-linux-gnu/libcudnn_engines_precompiled.so.9.1.0
/usr/lib/x86_64-linux-gnu/libcudnn_engines_runtime_compiled.so.9.1.0
/usr/lib/x86_64-linux-gnu/libcudnn_graph.so.9.1.0
/usr/lib/x86_64-linux-gnu/libcudnn_heuristic.so.9.1.0
/usr/lib/x86_64-linux-gnu/libcudnn_ops.so.9.1.0
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True
CPU:
Architecture:                    x86_64
CPU op-mode(s):                  32-bit, 64-bit
Address sizes:                   52 bits physical, 57 bits virtual
Byte Order:                      Little Endian
CPU(s):                          192
On-line CPU(s) list:             0-191
Vendor ID:                       GenuineIntel
Model name:                      Intel(R) Xeon(R) Platinum 8469C
CPU family:                      6
Model:                           143
Thread(s) per core:              2
Core(s) per socket:              48
Socket(s):                       2
Stepping:                        8
CPU max MHz:                     3800.0000
CPU min MHz:                     800.0000
BogoMIPS:                        5200.00
Flags:                           fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush dts acpi mmx fxsr sse sse2 ss ht tm pbe syscall nx pdpe1gb rdtscp lm constant_tsc art arch_perfmon pebs bts rep_good nopl xtopology nonstop_tsc cpuid aperfmperf tsc_known_freq pni pclmulqdq dtes64 monitor ds_cpl vmx smx est tm2 ssse3 sdbg fma cx16 xtpr pdcm pcid dca sse4_1 sse4_2 x2apic movbe popcnt tsc_deadline_timer aes xsave avx f16c rdrand lahf_lm abm 3dnowprefetch cpuid_fault epb cat_l3 cat_l2 cdp_l3 invpcid_single cdp_l2 ssbd mba ibrs ibpb stibp ibrs_enhanced tpr_shadow vnmi flexpriority ept vpid ept_ad fsgsbase tsc_adjust bmi1 avx2 smep bmi2 erms invpcid cqm rdt_a avx512f avx512dq rdseed adx smap avx512ifma clflushopt clwb intel_pt avx512cd sha_ni avx512bw avx512vl xsaveopt xsavec xgetbv1 xsaves cqm_llc cqm_occup_llc cqm_mbm_total cqm_mbm_local avx512_bf16 wbnoinvd dtherm ida arat pln pts hwp hwp_notify hwp_act_window hwp_epp hwp_pkg_req avx512vbmi umip pku ospke waitpkg avx512_vbmi2 gfni vaes vpclmulqdq avx512_vnni avx512_bitalg tme avx512_vpopcntdq rdpid cldemote movdiri movdir64b md_clear pconfig flush_l1d arch_capabilities
Virtualization:                  VT-x
L1d cache:                       4.5 MiB (96 instances)
L1i cache:                       3 MiB (96 instances)
L2 cache:                        192 MiB (96 instances)
L3 cache:                        195 MiB (2 instances)
NUMA node(s):                    2
NUMA node0 CPU(s):               0-47,96-143
NUMA node1 CPU(s):               48-95,144-191
Vulnerability Itlb multihit:     Not affected
Vulnerability L1tf:              Not affected
Vulnerability Mds:               Not affected
Vulnerability Meltdown:          Not affected
Vulnerability Spec store bypass: Mitigation; Speculative Store Bypass disabled via prctl and seccomp
Vulnerability Spectre v1:        Mitigation; usercopy/swapgs barriers and __user pointer sanitization
Vulnerability Spectre v2:        Mitigation; Enhanced IBRS, IBPB conditional, RSB filling
Vulnerability Srbds:             Not affected
Vulnerability Tsx async abort:   Not affected
Versions of relevant libraries:
[pip3] numpy==1.26.4
[pip3] nvidia-nccl-cu12==2.19.3
[pip3] torch==2.2.1
[pip3] triton==2.2.0
[pip3] vllm-nccl-cu12==<IP_ADDRESS>.4.0
[conda] Could not collectROCM Version: Could not collect
Neuron SDK Version: N/A
vLLM Version: 0.4.1
vLLM Build Flags:
CUDA Archs: Not Set; ROCm: Disabled; Neuron: Disabled
GPU Topology:
GPU0    GPU1    GPU2    GPU3    GPU4    GPU5    GPU6    GPU7    CPU Affinity    NUMA Affinity   GPU NUMA ID
GPU0     X      NV18    NV18    NV18    NV18    NV18    NV18    NV18    0-47,96-143     0               N/A
GPU1    NV18     X      NV18    NV18    NV18    NV18    NV18    NV18    0-47,96-143     0               N/A
GPU2    NV18    NV18     X      NV18    NV18    NV18    NV18    NV18    0-47,96-143     0               N/A
GPU3    NV18    NV18    NV18     X      NV18    NV18    NV18    NV18    0-47,96-143     0               N/A
GPU4    NV18    NV18    NV18    NV18     X      NV18    NV18    NV18    48-95,144-191   1               N/A
GPU5    NV18    NV18    NV18    NV18    NV18     X      NV18    NV18    48-95,144-191   1               N/A
GPU6    NV18    NV18    NV18    NV18    NV18    NV18     X      NV18    48-95,144-191   1               N/A
GPU7    NV18    NV18    NV18    NV18    NV18    NV18    NV18     X      48-95,144-191   1               N/A
Legend:
X    = Self
SYS  = Connection traversing PCIe as well as the SMP interconnect between NUMA nodes (e.g., QPI/UPI)
NODE = Connection traversing PCIe as well as the interconnect between PCIe Host Bridges within a NUMA node
PHB  = Connection traversing PCIe as well as a PCIe Host Bridge (typically the CPU)
PXB  = Connection traversing multiple PCIe bridges (without traversing the PCIe Host Bridge)
PIX  = Connection traversing at most a single PCIe bridge
NV#  = Connection traversing a bonded set of # NVLinks
🐛 Describe the bug
Running llama2-7b on H20, vllm0.4.1, Floating point exception (core dumped) with float16 accuracy, float32 accuracy can be executed normally
The error screenshot is as follows

llama model init start
INFO 04-26 17:03:13 llm_engine.py:98] Initializing an LLM engine (v0.4.1) with config: model='/mnt/deep_learning_test/testsuite/dataset/llms_inference_llama7b-v2_accelerate/checkpoint/7B-V2/', speculative_config=None, tokenizer='/mnt/deep_learning_test/testsuite/dataset/llms_inference_llama7b-v2_accelerate/checkpoint/7B-V2/', skip_tokenizer_init=False, tokenizer_mode=auto, revision=None, tokenizer_revision=None, trust_remote_code=True, dtype=torch.float16, max_seq_len=4096, download_dir=None, load_format=LoadFormat.AUTO, tensor_parallel_size=1, disable_custom_all_reduce=Falsequantization=None, enforce_eager=False, kv_cache_dtype=auto, quantization_param_path=None, device_config=cuda, decoding_config=DecodingConfig(guided_decoding_backend='outlines'), seed=0)
INFO 04-26 17:03:13 utils.py:613] Found nccl from library /root/.config/vllm/nccl/cu12/libnccl.so.2.18.1
INFO 04-26 17:03:16 selector.py:77] Cannot use FlashAttention-2 backend because the flash_attn package is not found. Please install it for better performance.
INFO 04-26 17:03:16 selector.py:33] Using XFormers backend.
INFO 04-26 17:03:25 model_runner.py:173] Loading model weights took 12.5523 GB
INFO 04-26 17:03:25 gpu_executor.py:119] # GPU blocks: 9217, # CPU blocks: 512
INFO 04-26 17:03:26 model_runner.py:977] Capturing the model for CUDA graphs. This may lead to unexpected consequences if the model is not static. To run the model in eager mode, set 'enforce_eager=True' or use '--enforce-eager' in the CLI.
INFO 04-26 17:03:26 model_runner.py:981] CUDA graphs can take additional 1~3 GiB memory per GPU. If you are running out of memory, consider decreasing gpu_memory_utilization or enforcing eager mode. You can also reduce the max_num_seqs as needed to decrease memory usage.
Floating point exception (core dumped)

Does it occur everytime? Or only for certain prompt?
In addition:

If you experienced crashes or hangs, it would be helpful to run vllm with export VLLM_TRACE_FUNCTION=1 . All the function calls in vllm will be recorded. Inspect these log files, and tell which function crashes or hangs.

它每次都会发生吗？还是只针对某些提示？
另外：

如果您遇到崩溃或挂起，使用 .vllm 中的所有函数调用都将被记录下来。检查这些日志文件，并判断哪个函数崩溃或挂起。export VLLM_TRACE_FUNCTION=1

Yes, this issue is inevitable. On the H20 model, all vllm versions with float16 accuracy will experience this error

Met the same bug, and it's useful to add --enforce-eager to avoid it.
Additionally, some models (for example facebook/opt-125m) with float16 won't meet this bug.

Met the same bug with Qwen1.5-14B-Chat on the H20, and I was able to solve it with float32. But using float16 and add  --enforce-eager  does not solve the problem.

Met the same bug with almost all LLM  on the H20,  but --enforce-eager does not solve the problem.

After testing, the --enforce-eager parameter does not work, but setting dtype to float32 works. However, this means that the quantized model cannot be deployed on h20.

Try to build vllm from source using the nvcr.io/nvidia/pytorch:24.04-py3 container to avoid the bug related to cuBLAS on specific shapes (on H20).

I am using nvcr.io/nvidia/pytorch:23.10-py3，it's all right when I use the float32 and float16, but floating point exception when I use bfloat16.
I tried:

build vllm from source using the nvcr.io/nvidia/pytorch:24.04-py3 container on H20 (still , float16 good, bfloat16 floating point exception)
nvcr.io/nvidia/pytorch:23.10-py3 on H800 (all good)

Met the same bug with Qwen1.5-14B-Chat on the H20, and I was able to solve it with float32. But using float16 and add --enforce-eager does not solve the problem.

How can I use float32 with vllm?

I met the same bug recently on H20. My torch version is:  2.3.1+cu121
You need to check your conda list | grep cublas first. Ensure there is only one cublas and that the version is greater than 12.3, then run pip install nvidia-cublas-cu12==<IP_ADDRESS>.
It works for me.

for me, both bfloat16 and float16 failed but float32 works

pip install nvidia-cublas-cu12
export LD_LIBRARY_PATH=/opt/conda/lib/python3.8/site-packages/nvidia/cublas/lib/

can solve it
LD_LIBRARY_PATH should be cublas path

I met the same bug recently on H20. My torch version is: 2.3.1+cu121 You need to check your first. Ensure there is only one and that the version is greater than 12.3, then run . It works for me.conda list | grep cublas``cublas``pip install nvidia-cublas-cu12==<IP_ADDRESS>

非常感谢，我解决了此问题！

I met the same bug recently on H20. My torch version is: 2.3.1+cu121 You need to check your conda list | grep cublas first. Ensure there is only one cublas and that the version is greater than 12.3, then run pip install nvidia-cublas-cu12==<IP_ADDRESS>. It works for me.

hi, I met the following problem with this method:
The conflict is caused by:
The user requested nvidia-cublas-cu12>=<IP_ADDRESS>
torch 2.4.1 depends on nvidia-cublas-cu12==<IP_ADDRESS>; platform_system == "Linux" and platform_machine == "x86_64"
The user requested nvidia-cublas-cu12>=<IP_ADDRESS>
torch 2.4.0 depends on nvidia-cublas-cu12==<IP_ADDRESS>; platform_system == "Linux" and platform_machine == "x86_64"
The user requested nvidia-cublas-cu12>=<IP_ADDRESS>
torch 2.3.1 depends on nvidia-cublas-cu12==<IP_ADDRESS>; platform_system == "Linux" and platform_machine == "x86_64"
Do you have any advice?

I met the same bug recently on H20. My torch version is: 2.3.1+cu121 You need to check your conda list | grep cublas first. Ensure there is only one cublas and that the version is greater than 12.3, then run pip install nvidia-cublas-cu12==<IP_ADDRESS>. It works for me.

hi, I met the following problem with this method: The conflict is caused by: The user requested nvidia-cublas-cu12>=<IP_ADDRESS> torch 2.4.1 depends on nvidia-cublas-cu12==<IP_ADDRESS>; platform_system == "Linux" and platform_machine == "x86_64" The user requested nvidia-cublas-cu12>=<IP_ADDRESS> torch 2.4.0 depends on nvidia-cublas-cu12==<IP_ADDRESS>; platform_system == "Linux" and platform_machine == "x86_64" The user requested nvidia-cublas-cu12>=<IP_ADDRESS> torch 2.3.1 depends on nvidia-cublas-cu12==<IP_ADDRESS>; platform_system == "Linux" and platform_machine == "x86_64" Do you have any advice?

Simply ignore that warning. The program runs just fine :)

export LD_LIBRARY_PATH=/opt/conda/lib/python3.8/site-packages/nvidia/cublas/lib/

It works for me, thx!

pip install nvidia-cublas-cu12==<IP_ADDRESS>
Thanks, this works for me


Add MP4 support

For those who want to auto-sync video playlists as well as music.

Sure, i'm gonna add a switch or something

Added a /vid switch, it's position dependent right now, but it does work.
plync.exe playlist directory [/vid]


Add example adopted from Rodinia hotspot benchmark

In relation with #8, I request the pull against the development branch.
If there is anything else, please let me know.
Thanks,
Janka

Everything is ok, PR merged. :)
Thank you,
Filip


polus-image-quality-plugin

Please review image quality plugin

Code cleaning, adding scale for metrics, removing unused functions


How to send Video file ?

Can Anyone Help How to send Video File Like MP4 Using This API ??

Sample code here:
https://github.com/mukulhase/WebWhatsapp-Wrapper/blob/master/sample/send_media.py

I tried this but Script become Hang

Make sure that your video file is less than 16MB and that it is in format MP4 (H264 + aac)

I tried @cesarandreslopez  but Not Work,  Can you please share full code for video Sending ?
I Tried This Code

https://github.com/mukulhase/WebWhatsapp-Wrapper/blob/master/sample/send_media.py

But script hang at driver.send_media(image_path, phone_whatsapp, caption)

Have you find any solution? @rahacker2006

@sagar-bhogayata not yet

Make sure that your video file is less than 16MB and that it is in format MP4 (H264 + aac)

Why do you have this limitation? I tried to use the media upload for files bigger than 50mb but it does not work. and whatsapp allows you to send files up to 100mb. would you have any way to send files in the maximum size of 100mb?

This is actually due to poor support of mp4 in Firefox.
create Dockerfile with the following
`
FROM selenium/standalone-firefox-debug:3.14.0-curium
RUN sudo apt-get update; sudo apt-get install -y software-properties-common
RUN sudo add-apt-repository "deb http://us.archive.ubuntu.com/ubuntu/ xenial universe multiverse" && sudo apt-get update
RUN sudo apt-get install -y ubuntu-restricted-extras
`
Run it
`
sudo docker network create selenium
sudo docker build -f Dockerfile -t wewhatsapp/selinium-ff:latest .
sudo docker run -d -p 4444:4444 -p 5900:5900 --network selenium --name firefox -v /dev/shm:/dev/shm webwhatsapp/selinium-ff:latest
`

This is actually due to poor support of mp4 in Firefox.
create Dockerfile with the following
FROM selenium/standalone-firefox-debug:3.14.0-curium

RUN sudo apt-get update; sudo apt-get install -y software-properties-common
RUN sudo add-apt-repository "deb http://us.archive.ubuntu.com/ubuntu/ xenial universe multiverse" && sudo apt-get update
RUN sudo apt-get install -y ubuntu-restricted-extras

Run it
sudo docker network create selenium
sudo docker build -f Dockerfile -t wewhatsapp/selinium-ff:latest .
sudo docker run -d -p 4444:4444 -p 5900:5900 --network selenium --name firefox -v /dev/shm:/dev/shm webwhatsapp/selinium-ff:latest

To add to the previous post I solved the problem in centos installing these:
yum install -y http://li.nux.ro/download/nux/dextop/el7/x86_64/nux-dextop-release-0-5.el7.nux.noarch.rpm
yum install -y gstreamer{,1}-plugins-ugly gstreamer-plugins-bad-nonfree gstreamer1-plugins-bad-freeworld ffmpeg

this worked for me:
Dockerfile for firefox Image:
`FROM selenium/standalone-firefox-debug:3.14.0-curium
RUN sudo apt-get update; sudo apt-get install -y software-properties-common
RUN sudo apt-get install apt-transport-https ca-certificates
RUN sudo add-apt-repository "deb http://us.archive.ubuntu.com/ubuntu/ xenial universe multiverse" && sudo apt-get update
RUN sudo apt-get install -y ubuntu-restricted-extras`


Dashboard functionality

I'm testing your great lib with 20x4 LCD display. Is there a way to create a dashboard or some how separate menu from the other app ? I.e. app will have few more screens and menu is one of those.

Yep take over display is the right answer, you can have as many screens as you want that way, you can even get access to the value of the encoder and buttons, they are passed in to the drawing callback.


How to rescale the size of pose skeleton?

During inference, we rescale the length of the driving pose skeleton to approximate the length of the character’s
skeleton in the reference image. Does anyone know how to do this step? I saw it in the inference code:
pose_transform = transforms.Compose(
[transforms.Resize((height, width)), transforms.ToTensor()]
)
for pose_image_pil in pose_images[: args.L]:
pose_tensor_list.append(pose_transform(pose_image_pil))
pose_list.append(pose_image_pil)
        ref_image_tensor = pose_transform(ref_image_pil)  # (c, h, w)

But this changes the whole picture and does not help the bone size of the reference picture, which is highlighted when the reference picture is of a child.

have you solved it ?

It's just a image resize not the pose skeleton rescale mentioned in the paper. Actually, the actual implementation is not introduced in their work. So we have no idea how they achieved it.


ecc takes way too much time

I'm new to eosjs , trying to developing a dapp demo , basically just receive a request, get value from request body then send value into eos by using transact demo from documentation . and on eos side, nothing complicated ,simply saved .but  I test that dapp by using webbench , it can only handle 5 request per second with cpu hits 100% . so I look into Flame Graphs , cpu basically consumed by several functions like recoverPubKey, calcPubKeyRecoveryParam , Signature.signHash , Signature.sign , which all in  eosjs-ecc module.
Am I doing something wrong ?
I'm using eosjs 20.0.0-beta2 .
Really need help , any help will do .Many thx.

Yes. javascript-based signing algorithms are slow by nature.


added online reading resources

Added blogs/docs read resources directory

I just thought the same but let's follow YAGNI principle and only sepearte it in another package when it becomes uncomproftable.


Allow setting priorityClassName

It would be nice if I could set priorityClassName for pods created by the chart so I can control when the pods are pre-empted.
See https://kubernetes.io/docs/concepts/scheduling-eviction/pod-priority-preemption/
I propose we add priorityClassName to values
minecraftServer:
    priorityClassName: ''

And include it in the deployment pod spec:
     priorityClassName: "{{ .Values.priorityClassName }}"

+1 For "extraPodSpec".


Present alert if fan isn't spinning

When a fan is spinning at 0 RPM but requested to spin at 10+% power, display a warning.

fanSpeedRpm is always zero on EON. This alert can only be implemented on comma twos.
controlsd is already subscribing to both health and thermal, I think the logic can be implemented solely in controlsd
I would choose the threshold to 50% desired fan speed. If the fan is broken it will quickly reach that since the device will heat up. This will prevent false positives due to stuff like low car battery voltage (the fan is driven by the 12V input).
The event should be a green permanent alert (like the no gps alert).

I'm guessing HwType::dos is the comma two, is it safe to ignore HwType::uno?
I don't see where to set an alert color, I've set it to ET.PERMANENT though.

Both uno, and dos have fan speed measurement, so you can trigger on both.

cereal is updated on master to include the alert. If you rebase it should solve the conflicts.
I don't think rebooting will fix the fan. The second line can say "Contact Support"


Configuring Data Protection in Blob storage with encryption fails

Using dot net core 2.2.
The initialisation of data protection and encryption of key files in Azure Blob Storage is not behaving as expected.
I have created a proof of concept project with logs and more details here:
https://github.com/span/azure-dp-conf
Setup
I expect the data protection service to be configured like this.
services
    .AddDataProtection(options =>
    {
        options.ApplicationDiscriminator =
            Configuration["DataProtectionKeys:applicationDiscriminator"];
    })
    .PersistKeysToAzureBlobStorage(blobContainer, "azuredpconf-keys.xml")
    .ProtectKeysWithAzureKeyVault(
        Configuration["AzureKeyVault:DataProtectionKey"],
        Configuration["AzureKeyVault:ClientId"],
        Configuration["AzureKeyVault:ClientSecret"]);

This fails however since no key file is generated in
PersistKeysToAzureBlobStorage and ProtectKeysWithAzureKeyVault seems
to require the key file.
A log of the failing run can be found in ./logs/First-RunFails.log.
Expected result
Host launches with no errors and key file is generated in the blob storage.
Actual result
Host throws an exception and no key file has been generated in the blob storage.

Stack trace from First-Run-Fails.txt

fail: Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingProvider[48]
      An error occurred while reading the key ring.
System.UriFormatException: Invalid URI: The format of the URI could not be determined.
   at System.Uri.CreateThis(String uri, Boolean dontEscape, UriKind uriKind)
   at System.Uri..ctor(String uriString, UriKind uriKind)
   at Microsoft.Azure.KeyVault.ObjectIdentifier..ctor(String collection, String identifier)
   at Microsoft.Azure.KeyVault.KeyVaultClientExtensions.WrapKeyAsync(IKeyVaultClient operations, String keyIdentifier, String algorithm, Byte[] key, CancellationToken cancellationToken)
   at Microsoft.AspNetCore.DataProtection.AzureKeyVault.AzureKeyVaultXmlEncryptor.EncryptAsync(XElement plaintextElement)
   at Microsoft.AspNetCore.DataProtection.AzureKeyVault.AzureKeyVaultXmlEncryptor.Encrypt(XElement plaintextElement)
   at Microsoft.AspNetCore.DataProtection.XmlEncryption.XmlEncryptionExtensions.EncryptIfNecessary(IXmlEncryptor encryptor, XElement element)
   at Microsoft.AspNetCore.DataProtection.KeyManagement.XmlKeyManager.Microsoft.AspNetCore.DataProtection.KeyManagement.Internal.IInternalXmlKeyManager.CreateNewKey(Guid keyId, DateTimeOffset creationDate, DateTimeOffset activationDate, DateTimeOffset expirationDate)
   at Microsoft.AspNetCore.DataProtection.KeyManagement.XmlKeyManager.CreateNewKey(DateTimeOffset activationDate, DateTimeOffset expirationDate)
   at Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingProvider.CreateCacheableKeyRingCore(DateTimeOffset now, IKey keyJustAdded)
   at Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingProvider.Microsoft.AspNetCore.DataProtection.KeyManagement.Internal.ICacheableKeyRingProvider.GetCacheableKeyRing(DateTimeOffset now)
   at Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingProvider.GetCurrentKeyRingCore(DateTime utcNow)
info: Microsoft.AspNetCore.DataProtection.Internal.DataProtectionStartupFilter[0]
      Key ring failed to load during application startup.
System.UriFormatException: Invalid URI: The format of the URI could not be determined.
   at System.Uri.CreateThis(String uri, Boolean dontEscape, UriKind uriKind)
   at System.Uri..ctor(String uriString, UriKind uriKind)
   at Microsoft.Azure.KeyVault.ObjectIdentifier..ctor(String collection, String identifier)
   at Microsoft.Azure.KeyVault.KeyVaultClientExtensions.WrapKeyAsync(IKeyVaultClient operations, String keyIdentifier, String algorithm, Byte[] key, CancellationToken cancellationToken)
   at Microsoft.AspNetCore.DataProtection.AzureKeyVault.AzureKeyVaultXmlEncryptor.EncryptAsync(XElement plaintextElement)
   at Microsoft.AspNetCore.DataProtection.AzureKeyVault.AzureKeyVaultXmlEncryptor.Encrypt(XElement plaintextElement)
   at Microsoft.AspNetCore.DataProtection.XmlEncryption.XmlEncryptionExtensions.EncryptIfNecessary(IXmlEncryptor encryptor, XElement element)
   at Microsoft.AspNetCore.DataProtection.KeyManagement.XmlKeyManager.Microsoft.AspNetCore.DataProtection.KeyManagement.Internal.IInternalXmlKeyManager.CreateNewKey(Guid keyId, DateTimeOffset creationDate, DateTimeOffset activationDate, DateTimeOffset expirationDate)
   at Microsoft.AspNetCore.DataProtection.KeyManagement.XmlKeyManager.CreateNewKey(DateTimeOffset activationDate, DateTimeOffset expirationDate)
   at Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingProvider.CreateCacheableKeyRingCore(DateTimeOffset now, IKey keyJustAdded)
   at Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingProvider.Microsoft.AspNetCore.DataProtection.KeyManagement.Internal.ICacheableKeyRingProvider.GetCacheableKeyRing(DateTimeOffset now)
   at Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingProvider.GetCurrentKeyRingCore(DateTime utcNow)
   at Microsoft.AspNetCore.DataProtection.KeyManagement.KeyRingProvider.GetCurrentKeyRing()
   at Microsoft.AspNetCore.DataProtection.Internal.DataProtectionStartupFilter.Configure(Action`1 next)

Workaround
I have found two workarounds.

Create the key file manually in blob storage before launching the Host.
Remove the ProtectKeysWithAzureKeyVault during first run to make sure
the key file is created and then re-enable the key encryption.

@natemcmaster could you take a look please?

ACK. Will take a look in the next milestone.

@anurse do you know who could take a look at this?

@HaoK has done some work here before, and I could find someone, but we're pretty fully booked for preview 8 so it would likely increase risk in other areas. There is a workaround posted, which doesn't at all remove the need to fix this but perhaps indicates that it could be a candidate for 3.1 (which isn't far after 3.0 anyway)

@HaoK can you take a look please?

This is a candidate for 3.1, due to the presence of a work around.
@anurse @Pilchie can we get a 3.1 milestone yet?

There is one.

Linking to a related issue https://github.com/aspnet/AspNetCore.Docs/issues/12592

@blowdart after looking at this a bit, given that there is no PersistKeysToAzureKeyValue unlike PersistKeysToDbContext or PersistKeysToAzureBlobStorage, I believe the current behavior is probably intentional, in that you have to manually persist the key to azure key value, and then it will work fine

That said I could be confused, so could you take another look @blowdart :)

That seems odd, but also kind of makes sense, as you want to limit privileges to keyvault. Is it documented anywhere though?

Interesting. From the docs

Configures the data protection system to persist keys to the specified path in Azure Blob Storage.

I would expect the data protection system to persist the keys.
Looking at the remarks they say:

The container referenced by container must already exist.

So nothing about creating the file manually, only the container.

So basically it seems like ProtectWithAzureKeyVault doesn't work unless the key already exists.  So the workaround where you run it once without Protect to create the keys, and then add it back, might be the expected flow...  Sound reasonable @blowdart ?  We can certainly update the docs to call this quirk out?

Oh wait, the key part in keyvault is the wrapping key. @span data protection keys aren't stored in keyvault.
Ok this makes sense now I remember everything. Yes, a key to encrypt the other keys needs to exist first, just like an X509 certificate needs to exist if you use that to protect keys. This seems like a doc failure.

Yes.
I always thought that PersistKeysToAzureBlobStorage would return after the file had been created (if it didn't exist) and then ProtectKeysWithAzureKeyVault would encrypt the keys in blob storage with some key in keyvault.
If it is a doc failure perhaps these methods should not be allowed to be chained either?

Oh wait, it's the file creation that fails? That does feel a little weird. The file should get created when a key is first generated. OK this is odd. @HaoK we can chat about this at triage today.

I did a migration of the proof of concept project today and it seems to be fixed in 3.0. No idea why but it seems the data protection changes sorted everything out.

Great, glad its already fixed in 3.0 @span


[image_picker] deprecate retrieveLostData

Description
As method ImagePicker.retrieveLostData() returns a deprecated class (LostDataResponse), the method should also be annotated @Deprecated.
Advise using getLostData method instead (retrieveLostData -> getLostData), this is consistent with pick_video -> get_video, and pick_image -> get_image.

adding this simple change to #2900

Because image_picker >=0.6.7 depends on image_picker_platform_interface ^1.1.0 which depends on http ^0.12.1, image_picker >=0.6.7 requires http ^0.12.1.


Added functionality for isPath

 I worked on the ispath to check whether the given string is symbolic link or not. Added docs and test file

Context of the change :

https://github.com/dictu-lang/Dictu/issues/450#issue-1014437358

[ ] Bug fix

[x] New feature

[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Why is the function called isPath if it's checking whether something is a symbolic link or not? Not all paths are symbolic links.

File name has been changed. Could you check now?

The name the issue stated is fine, isLink.
Few things as well, you will need to add the test file to here.
The code seems to be slightly mis-indented unfortunately.
And finally, this will break windows builds. If we intend this to not work on windows (which may be a viable option to begin with) we either need to 1) remove this and make sure it doesn't get compiled on windows systems entirely. 2) Do something similar to Python where it just returns False on runtimes that don't support it (less hot on this as to me it feel like a silent fail).

That new name also doesn't make sense. It should be called isLink.

Could you review now?


Management of names of archived streams #26661

Fixes: #26661
This PR:

Does not rename streams during deactivating them.
Add migration to rename previously archived streams back to their original name. Also renames handful of streams pre 1b6f68bb59dc85125ff21fbd3f98a88f39c0730d.
Add archived property in RawStreamDict, SubscriptionStreamDict, APIStreamDict and APISubscriptionDict.
Add include_archived parameter for [/api/get-streams](https://chat.zulip.org/api/get-streams) and archived_streams as client capability for [/api/register-queue](https://chat.zulip.org/api/register-queue).
Displays archived stream name and messages with (archived) postfix to the user.
Prevents users from sending message in an archived stream.
Add support to rename archived streams while creating stream with the same name.

Consider following things regarding the commits while reviewing the PR.
Commit: 3b9dcb51bc843dbd2a301724c8dde264f422352a
This is a prep commit that fixes the function calls in server_events_dispatch.js.
Commit: 45ede22f9834761604e735c95afa670b687c93c9
(Archived) postfix is added to archived stream names by checking for archived property of the streams.
The web/templates/message_view_header.hbs code looks unclean and cluttered, I couldn't make it more cleaner. Also I am not aware about any other method that will eliminate nesting if condition in web/templates/navbar_icon_and_title.hbs.
Commit: adf99b9b01f47b5e80832a47c5905d957aa57b84
A significant portion of the code for the modal window in web/src/stream_create.js is similar to the existing code in web/src/stream_edit.js. I haven't identified a method to efficiently reuse that code.

UI Changes:
Since random (an active stream) already exists, the migration did not renamed the archived stream with same old name:

When user doesn't has the permission to rename the archived stream:

When user cannot view the archived stream:

When user has the permission to rename the archived stream:

After renaming the archived stream (also named as "archived stream") to "archived_stream" the name becomes available:

Compose Box banner:

In dark mode:

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

@sanchi-t thanks for working on this!

Thanks for all the effort on this. See https://chat.zulip.org/#narrow/stream/378-api-design/topic/archived.20streams/near/1720667 for a set of decisions on the API -- can you adjust the PR to implement those decisions?
git grep client_capabilities  and git grep bulk_message_deletion should provide some helpful examples.

Yes. I'll soon update the PR.

From CZO discussion:

In /api/get-subscriptions, no special action may be required -- you can't be still subscribed to an archived stream. We probably don't need an archived property on those objects, unless we want to enforce the invariant that subscription objects always extend the stream object?

So do we want to enforce the invariant that subscription objects always extend the stream object? Or archived property should not be included?
If we want not to include an archived property on get-subscriptions objects, we need to modify the type of subscriptions. Since both subscribed and unsubscribed streams share the same type (SubscriptionStreamDict), a straightforward solution is to make the property archived: bool optional in the type. This way, we can have the archived property specifically for unsubscribed streams.

So do we want to enforce the invariant that subscription objects always extend the stream object? Or archived property should not be included?

For decisions about how the API should work like this, let's keep the discussion on the chat.zulip.org threads in "api design" -- that's where you'll get the right audience to make decisions.

@sanchi-t please do a top-level comment with a changelog and any outstanding conversations when you've addressed this last batch of feedback fully; see @prakhar1144's PRs for some good examples of how to communicate really clearly about updates.
(And maybe bump any API design conversations that are still not decided).

@timabbott this pull request isn't quite ready yet. I should have marked it as a draft. I'll notify you in the stream as soon as it is ready for review.

Updates in the recent push:

Addressed the comments.
Fixed the stream archiving logic which made every archived stream invite_only: true.
Removed the use of display_recipient for sending the archived status for stream messages.
Removed events system changes regarding archived streams.

Key conversation points:

archived property should exist for subscription objects as unsubscribed object will potentially contain streams that are archived.

CZO discussion: https://chat.zulip.org/#narrow/stream/378-api-design/topic/archived.20streams/near/1720667

Updates since last review:

Addressed the comments.
Addressed the CZO comments.
Fixed the stream archiving logic which made every archived stream invite_only: true.
Removed the use of display_recipient for sending the archived status for stream messages.
Removed events system changes regarding archived streams.
Fixed handling of deactivating stream in client.
The stream name is now updated with the (Archived) suffix immediately after deactivating it.

NOTE: I have excluded the event system changes (modifications made in Tornado) since passing the 'archived' parameter to Tornado code has no effect. I may not fully comprehend the meaning behind:

So it's harmless to send those for archived streams even when those clients haven't set the capability.

We referring to the event system here right? However, I don't see the purpose of sending the 'archived' flag to the event system.
Key conversation points:

archived property should exist for subscription objects as unsubscribed object will potentially contain streams that are archived.
CZO discussion: https://chat.zulip.org/#narrow/stream/378-api-design/topic/archived.20streams/near/1720667

Thanks for all the hard work on this @sanchi-t! I posted what I think is a complete review for the remaining changes.

Thanks! A bunch of UI tweaks here:

For the stream header bar label, it would be nice to use consistent styling with guest users:

I actually like the dimming, so we can try making it "(archived)"  but keeping that part of the styling for now, and we can get more feedback on that in #design.

For the edit modal, can we change the title to something like this? We need some clear indication that you're editing the archived stream, not the stream you're trying to create or something.

Edit #Denmark (archived stream)

I managed to get into this state.

This was after a weird flow, where I was not on this PR, I archived #Denmark, made a new #Denmark, archived that one, and then got the PR running and tried to rename it.
So maybe this is just a transition edge case to watch out for?

After I renamed the archived stream, the error message ("An archived stream with this name already exists.") and renaming link did not disappear like they should.

Archived streams appear with the private stream icon , which seems wrong?

The archived stream isn't displayed correctly to a non-admin user:

Clicking on a message produces an error:

And clicking on the stream or topic looks like this:

I'll probably have more feedback, but I think this is enough to work through for now.

@alya  I have responded to your comments, however I need to details to figure out a few things.

I actually like the dimming
By dimming you mean keeping the gray color for now right? I have updated the PR.

Done.

Okay so I don't fully understand how you managed to get into that state.

Before the PR, when deactivating a stream, its name was transformed into a hashed stream ID + "!DEACTIVATED" + stream.name. This allowed creating #Denmark multiple times without errors since each deactivation appended a unique prefix.
Now, after running this PR, the migration restores stream names to their original state. When running the migration, you'll notice one archived #Denmark stream with the name #Denmark only, while the other retains the prefix. Due to the constraint of unique stream names, the migration skips one stream.
Attempting to create another stream named #Denmark will trigger an error, indicating "An archived stream with this name already exists." The error provides an option to rename the archived stream.
Regarding the "Failed: Invalid parameter" error, I couldn't reproduce it. Can you provide more details on when it occurred? Ensure that you ran the "removing_prefix_from_archived_streams" migration after switching to this PR and reloaded.
The message in the original #Denmark will have that prefix because, as mentioned in issue #26661, the migration skips streams with the same name:

Skip renaming if there is another stream in the organization that has the same name.

I'm unable to replicate this issue either. Could you clarify if you attempted to rename the #Denmark stream mentioned in the third point or a different one? The error message disappears for me, so additional details would help diagnose the problem accurately.
Here:
rename.webm

The streams appear with the private stream icon due to an issue in the stream deactivating logic, which incorrectly marks the stream as private. We have acknowledged this problem and agreed to address it in a subsequent PR. This issue was highlighted by Tim in his comment.

Yeah, I think once we integrate this PR, we'll want to do a follow-up PR to making archiving a stream not adjust its permissions.

Since archived streams are currently marked as private, they will only be visible to admins. Non-admin users will still be able to view messages from archived streams in search results, just as they could before. However, they will not have access to view the stream topics themselves.

It's worth noting that the error you encountered when clicking on a message from an archived stream is not a result of this PR. This error was present prior to the implementation of this PR.
As for the "(archived)" suffix not being visible to non-admin users, this is due to the fact that non-admin users do not have access to archived streams. Consequently, the client cannot determine whether a stream is
export function is_stream_archived(stream_id: number): boolean {
    const sub = sub_store.get(stream_id);
    return sub ? sub.archived : false;
}

@timabbott Should I address the issue of getting an error (adjusted_terms impossibly null) when clicking on a message from an archived stream within this PR itself?

Ensure that you ran the "removing_prefix_from_archived_streams" migration after switching to this PR and reloaded.

How would I do that?

I'm unable to replicate this issue either. Could you clarify if you attempted to rename the #Denmark stream mentioned in the third point or a different one? The error message disappears for me, so additional details would help diagnose the problem accurately.

Ah, sorry this was unclear. This happens for me in general, but not all the time, so I'm not sure what the conditions are.

The experience for editing, moving or deleting a message is currently confusing; messages should appear as not editable/movable/deletable in the UI. Let's file an issue if it makes sense to change this as a separate follow-up PR.

How would I do that?

You can simply run the command ./manage.py migrate. This command will apply all the unapplied migrations.
And if for some reason you can not apply the 0501_remove_prefix_from_archived_streams.py migration, you need to reverse the migration first then apply it again.
You can do this using this command:
./manage.py migrate zerver 0500
Where 0500 is the migration number before the migration you want to apply, which in this case is 0501.

Ah, sorry this was unclear. This happens for me in general, but not all the time, so I'm not sure what the conditions are.

Ah, that's alright I will keep testing it hopefully I will be able to reproduce it.

Updates since last review:

Changed archived field to is_archived.
Fixed the bug where the error message ("An archived stream with this name already exists.") and renaming link were not disappearing after renaming in create stream.

Here is what PR implements:

Changes made to the archiving logic of streams; no renaming or prefix addition when a stream is archived.
Archived streams are now sent back to the client; previously, they were excluded.
Archived streams are now included in the unsubscribed streams list.
Archived streams are filtered out and not displayed in settings menu.
APIs '/streams/{stream_id}' and '/events' now include an is_archived property for streams and subscriptions.
For '/streams', an exclude_archived parameter is added.
For '/register', a client capability parameter archived_streams is added.
Add a new test for deactivating a stream.
Show an '(archived)' label next to archived stream names in the client, similar to the '(guests)' label.
Allow users with permissions to rename an archived stream when creating a new stream with the same name.
NOTE: There's a problem with the stream archiving process that makes archived streams invite_only: true, making them invisible to non-admin users. This will be fixed in a follow up PR.

I pushed back here after doing the adjustments noted in https://chat.zulip.org/#narrow/stream/378-api-design/topic/archived.20streams/near/1946212 as well as squashing a few commits that were not coherently separated and including the commits in #30315 here so we can merge them together. I also added a basic help/ commit.
There's a test failing that I don't understand.

In the confirmation modal: "Archiving channel" -> "Archiving this channel".

Archiving a channel and pressing "back" a couple of times in the browser, I got:

Message actions need to be handled more gracefully. Can we

Hide the actions you can't take from the message menu, message header bar, and anywhere else?
Show a more useful error if someone attempts to take the action (e.g., another user who hasn't refreshed the page)?

Some examples:

Emoji reaction:

The "resolve topic" button in the message header bar simply vanishes when you click it.

Apologies if some of this was discussed above; I haven't had a chance to review this comment thread.

In the confirmation modal: "Archiving channel" -> "Archiving this channel".

Done.

Archiving a channel and pressing "back" a couple of times in the browser, I got:

Resolved.


Reduce observe count for application notifications

LTHPasscodeViewController is singleton.
_addObserver in viewWillAppear called more than once.

Cheers!

Could you update podspec? I want to use new version with this fix.

I'm currently on El Capitan beta and cocoapods' trunk acts weirdly, it doesn't let me push :(

@yonekawa Done. Pod updated to 3.6.5.

Thank you! (sorry typo...)


chore: update docs link

Issue
Docs links in readme is pointing to master
Solution
point to v2 docs

This was fixed in https://github.com/timber/timber/pull/2853, right?

You are right :)


Canonical rules for fraction numerators

Hi all! I'm super interested in MathJSON, and I'm trying to set up MathLive/Compute in a personal project.
I was working on comparing linear equations with simplify/evaluate/canonical and found that I couldn't get a consistent representation for (-7/4)x+10
const answerbox="\\frac{-7x}{4}+10"

const answer = [  "Add", [ "Multiply", [ "Divide", -7, 4 ], "x" ], 10]

  match(
    ce.evaluate(ce.parse(answerbox)),
    ce.evaluate(answer)
  )
);

Is this a case where I need to use additional rewrite rules to coerce this into a unique 'canonical' form for comparison, or is this within the scope of the canonical function?

Great question... I need to think about it a bit more for canonical. Seems reasonable, though. However, it should definitely give the same result for simplify

@arnog Just a thought, but maybe a new canonical-no-divide rule that behaves like canonical-subtract and replaces a division with a non-1 numerator with multiplication? Changing -7/4 to -7 x 1/4 or -7 x 4^(-1)

I too would be in favour of adding a canonical-no-divide rule. Preferably one that replaces every division a/b with a * b^(-1)
QuantumSheet actually implements that, since that's what Sympy expects: https://github.com/stefnotch/quantum-sheet/blob/6b445476559ab5354b8a1c68c24a4ceb24e050e9/src/cas/pyodide-cas.ts#L101-L102

It feels a bit weird to me to not have division in the canonical form (admittedly, an earlier version of the Compute Engine did just that). That also makes some simplification rules more difficult to write since instead of detecting patterns of the form a/b you now have to pick up a*c*d where c (or d) is of the form b^-1.
Can you tell me more about Sympy's expectations in that regard? Do you have a pointer in the documentation that discusses this and perhaps a rationale for it?
On a broader note, I intend in an upcoming revision, to have clearer lines between 'formatting' (i.e. putting into a canonical form) and 'simplifying'. Formatting will not account for any calculations (i.e. multiply by 1), but will focus on sorting arguments of commutative functions, and reducing associative, idempotent and involutory functions. Simplify on the other hand will apply more rules, including ones depending on the numerical values of the arguments, and other 'assumptions' about arguments (for example whether an argument is a positive real, or a complex number, etc...

Here is the relevant sympy documentation. Typing the following into the sympy live shell (bottom right of the documentation page) does demonstrate that behavior:
expr = x/y
srepr(expr)

Of course if that turns out to be more trouble than it's worth, that's an entirely valid reason for not including it.

Interesting... Thanks for the pointer.
Of course, since Sympy has a Rational type, \frac{7x}{4} would be Mul(Rational(7, 4), x) and not Mul(7, x, Pow(4, -1))

Interesting... Thanks for the pointer.
Of course, since Sympy has a Rational type, \frac{7x}{4} would be Mul(Rational(7, 4), x) and not Mul(7, x, Pow(4, -1))

Oh yes, you're spot on! Using the Rational type whenever possible is an even better choice in Sympy. I found this out the hard way, after wondering why an automatically translated equation didn't do what I thought it would.

The Compute Engine now handles rational numbers as their own type:
console.log(ce.parse('-\\frac{7}{4}x+10').canonical.json);
//-> ["Subtract", 10, ["Multiply", ["Rational", 7, 4], "x"]]

@arnog That's great! Awesome stuff!


updated with mlc models post-processing

updated DSPy internals with post-processing for MLC Llama models - .split('\n')[0]

Closing since this is a bit hacky, but we need a more general thing for MLC and HFModel. They're not in a great state yet.


dasharo-performance/401-cpu-temperature.md: add file

Signed-off-by: Kamil Pokornicki<EMAIL_ADDRESS>

@Pokisiekk I wonder if we should use package lm-sensors instead of reading the temperature form file. I think that it would be better because in that case we also have information about which core has which temperature and what are the defined values for high temperature and critical temperature.
Example output from lm-sensors:
`acpitz-acpi-0
Adapter: ACPI interface
temp1:        +27.8°C  (crit = +106.0°C)
temp2:        +29.8°C  (crit = +106.0°C)
coretemp-isa-0000
Adapter: ISA adapter
Package id 0:  +42.0°C  (high = +85.0°C, crit = +105.0°C)
Core 0:        +38.0°C  (high = +85.0°C, crit = +105.0°C)
Core 1:        +40.0°C  (high = +85.0°C, crit = +105.0°C)
Core 2:        +42.0°C  (high = +85.0°C, crit = +105.0°C)
Core 3:        +40.0°C  (high = +85.0°C, crit = +105.0°C)
`


Update xml-spac to 0.12.0

About this PR
📦 Updates io.dylemma:xml-spac from 0.8 to 0.12.0
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

🔍 Files still referring to the old version number
The following files still refer to the old version number (0.8).
You might want to review and update them manually.
core/src/test/resources/topology1.svg
core/src/test/resources/topology2.svg
core/src/test/resources/topology3.svg
core/src/test/resources/topology4.svg

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.dylemma", artifactId = "xml-spac" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "io.dylemma", artifactId = "xml-spac" }
}]

labels: library-update, old-version-remains, commit-count:1

Superseded by #233.


[ISSUE #4402] Refactor the retry module with HashedWheelTimer.

Fixes #4402 .
Modifications

Add HashedWheelTimer, replacing the retry logic based on DelayQueue.
Canonical Retryer naming (EventMeshTcpRetryer - > TcpRetryer)

Please fix the build error.

Please fix the build error.

CI error has been fixed.

LGTM

@pandaapo Please review the new changes, thanks.


Update Netlify deployment configuration in CircleCI build

See  failure: https://circleci.com/gh/KarrLab/datanator_frontend/1212
#!/bin/bash -eo pipefail
export PATH=$PATH:node_modules/.bin

# deploy
netlify deploy --site $NETLIFY_SITE_ID --auth $NETLIFY_ACCESS_TOKEN --prod --dir=build
Deploy path: /home/circleci/project/build

◈ Deployments are "locked" for production context of this site

? Would you like to "unlock" deployments for production context to proceed? (y/N
) ? Would you like to "unlock" deployments for production context to proceed? (y/N
) ? Would you like to "unlock" deployments for production context to proceed? (y/N
)
Too long with no output (exceeded 10m0s): context deadline exceeded

This should be fixed now https://circleci.com/gh/KarrLab/datanator_frontend/1213.
Note:

Disabling auto publishing causes this error, even though there was no such error previously. Builds are done "locally"(on circleci) now and published directly on netlify without building on netlify.

Looks good.


Accept credentials from environment variable in deployment rules

What is the goal of this PR?
deploy_(apt|rpm|npm|pip) rules now accept credentials via environment variables: $DEPLOYMENT_REPO_TYPE, $DEPLOYMENT_USERNAME, $DEPLOYMENT_PASSWORD, $DEPLOYMENT_EMAIL [for npm]
This is done so we're able to not expose our credentials (bazel prints out command being executed and the behaviour is non-configurable)
What are the changes implemented in this PR?

Remove $# check for number of arguments
If command line argument was not passed, default to environment variable (see above); if it's also not present, default to notset, see this guide
Check that credential is not set to "notset`; exit with error if it is

Is this how you supply the environment variables to the deployment command?
DEPLOYMENT_USERNAME=... DEPLOYMENT_PASSWORD=... DEPLOYMENT_REPO_TYPE =... bazel run //:deploy-apt

@lolski right, this should be it


Upload streaming example doesn't work

The example in "uploads / alternative shell commands" where one is supposed to tar a folder, send the output through pipe to curl and upload through streaming is not working.
tar -cf - <local_folder> | curl -# -H "http://nomad-lab.eu/prod/rae/api/uploads/?token=........." -T - | xargs echo
This actually leads to curl: no URL specified!. Tested with curl 7.29.0 and 7.66.0.

It seems to work if I remove the -H switch, but I have no idea if this is the correct way to fix it?

There might be more problems with the "Alternative shell commands" help.
In line NOMAD accepts stream data (-T <local_file>) (like in the examples above) or multi-part form data (-X PUT -f file=@<local_file>):, there should probably be -F instead of -f
and in the last example for "Upload names" curl "https://oasis.mch.rwth-aachen.de/nomad-oasis/api/uploads/?token=.........&name=<name>" -X PUT -T <local_file> there are both -X and -T switches used which seems strange?

Fix merged into v0.10.4, will be on central NOMAD after the next release.

Confirming fixed. Thank you @markus1978


Azur agent behind a proxy

Hi, I have tried to install Helm in an agent behind a corporate proxy.
The proxy is correctly configured in the .proxy file, and using a curl request (with correct proxy env variables) I can download the file from https://get.helm.sh.
When the agent is running seems that the configuration is not used and the file is not downloaded.
How can I specify the correct proxy?

@ibacci -- thank you for your feedback; we appreciate it. To make sure we have all the information needed to triage your issue, please let us know which topic you’re referencing, or provide more context around your general task.

The task I used is HelmInstaller@0 and HelmInstaller@1 with the same result, it's running in Azur Pipeline agent v2.164.8.

@ibacci -- thank you for clarifying.  Are you reading a specific docs.microsoft.com article?

@ibacci -- you may find answers here:

Azure DevOps Support Bot
Azure DevOps on Stack Overflow

I am closing this issue now.  You are welcome to @ mention me for any followup.
We hope to hear from you again.


docs: flowbite-react Datepicker appears twice

[x] I have searched the Issues to see if this bug has already been reported
[x] I have tested the latest version

as the 10th item and the last item.
Steps to reproduce

cd423142d542181398ae9c31a7b79db2cbfd0759 fixed. Closing.


fix: Analytics sendEvent should be able to send events even during sign in process

Fixes #11801

/ok-to-test sha=d74f2cd


stellar sdk uses dependency with lgpl licence.

Hey @matinkaboli ,
First of all, I would like to thank you for a great crate, it helped me a lot to work with Stellar.
I have noticed that the crate depends on nacl v0.5.3, licensed as LGPL 3.0.
From my understanding, it contradicts Stellar-SDK's Apache-2.0 license.
I am not a licensed expert, please advise.
P.S. Our product uses cargo-deny for license checks.

Hi, thanks for mentioning this issue.
I'm not expert at license too, do you think changing repo license can solve this issue or not? something like MIT?

Hi, I think the best solution is to use crate with "mit"/"apache-2.0" license
that will replace "nacl" which has "LGPL" license.
so there will be no issues with the license because I don't know if the fact that we are using "LGPL" crate makes this crate "LGPL"
for now, I uploaded pr that makes "nacl" a default feature in our repo.
so users that don't want this feature will be able to use this crate with "default-features = false".
and then they will not have license issues. if they don't need this feature.
this will not affect current users that use this crate as is.
thanks!

#5 merged. thanks @lishay1989


Remove direct imports from Cardano.Ledger

Changelog
- description: |
    Removed direct imports from Cardano.Ledger
# uncomment types applicable to the change:
  type:
  - improvement    # refactoring

Context
Issue
How to trust this PR
Highlight important bits of the PR that will make the review faster. If there are commands the reviewer can run to observe the new behavior, describe them.
Checklist

[ ] Commit sequence broadly makes sense and commits have useful messages
[x] New tests are added if needed and existing tests are updated. See Running tests for more details
[x] Self-reviewed the diff

Can we switch to using one qualified import alias, either L or Ledger?
I'd pick L, seems pretty obvious to me.

I've changed it everywhere in: 47eaafa8d01b1c912bf4c891e46ca11318cfb1b2
Is that something we want to do?

In favor of L over Ledger :+1:


Can't create package or associate with Store for single-project MSIX

Describe the bug
I have default project for Windows App SDK and I try create package or associate it with Store, but Visual Studio throws error 0xe0434352 (screenshot included) and crashes.
Logs from Event viewer:
Faulting application name: devenv.exe, version: 17.3.32519.111, time stamp: 0x1a4dcbbf
Faulting module name: KERNELBASE.dll, version: 10.0.25131.1000, time stamp: 0x220a9ff1
Exception code: 0xe0434352
Fault offset: 0x000000000008fedc
Faulting process id: 0x0x3634
Faulting application start time: 0x0x1D87A8A55CDD947
Faulting application path: C:\Program Files\Microsoft Visual Studio\2022\Preview\Common7\IDE\devenv.exe
Faulting module path: C:\WINDOWS\System32\KERNELBASE.dll
Report Id: 4a406780-77cf-4b08-9151-1b4db2ec6b4c
Faulting package full name:
Faulting package-relative application ID:
Faulting application name: WerFault.exe, version: 10.0.25131.1000, time stamp: 0x387a2982
Faulting module name: ntdll.dll, version: 10.0.25131.1000, time stamp: 0xc471e3b0
Exception code: 0xc0000409
Fault offset: 0x00000000000a40c0
Faulting process id: 0x0x80C0
Faulting application start time: 0x0x1D87A8A6B550015
Faulting application path: C:\WINDOWS\system32\WerFault.exe
Faulting module path: C:\WINDOWS\SYSTEM32\ntdll.dll
Report Id: d1ed6880-d902-48fd-9799-016d1c06f3c5
Faulting package full name:
Faulting package-relative application ID:

Steps to reproduce the bug

Create default Windows App SDK
Try associate with Store or create package
Crash

Expected behavior
Should work
Screenshots
No response
NuGet package version
No response
Packaging type
Packaged (MSIX)
Windows version
Insider Build (xxxxx)
IDE
Visual Studio 2022-preview
Additional context
No response

Windows App SDK version: 1.1 (This option missing in NuGet version list)

Problem reported in Visual Studio too if this won't be problem of WASDK

I just heard back from Visual Studio that this was an issue in their product. It was fixed in servicing for 17.2, but not 17.3 Preview 1.1. 17.3 Preview 2 will have the fix.

Sometimes problems happen :) thank you for answer


How to create multiple lifecycles for existing bucket

How to create multiple lifecycles for existing bucket

Hi @surendarkaniops
I see that this issue is neither a feature request or a bug report but rather a support request. If you have questions on how to use our modules, we urge you to join our public Slack (https://sweetops.com/slack/) and post in the terraform channel. If there is insufficient documentation, we urge you to make a pull request or open an issue specifically pointing at where the documentation is missing.
Regardless, I'm happy to help:
The following fixtures in examples/complete show how multiple lifecycle rules can be created for an s3 bucket using this module:
https://github.com/cloudposse/terraform-aws-s3-bucket/blob/0c0155fc0cf736c7e1116cd5c2ac08fdecca4fec/examples/complete/lifecycle.us-east-2.tfvars#L11-L54
In an actual module instantiation, it would look like this:
module "s3_bucket" {
  source = "cloudposse/s3-bucket/aws"
  # Cloud Posse recommends pinning every module to a specific version
  # version = "x.x.x"
  acl                      = "private"
  enabled                  = true
  user_enabled             = true
  versioning_enabled       = false
  allowed_bucket_actions   = ["s3:GetObject", "s3:ListBucket", "s3:GetBucketLocation"]
  name                     = "app"
  stage                    = "test"
  namespace                = "eg"
  lifecycle_rules = [
    {
      prefix  = null
      enabled = true
      tags    = { "temp" : "true" }

      enable_glacier_transition            = false
      enable_deeparchive_transition        = false
      enable_standard_ia_transition        = false
      enable_current_object_expiration     = true
      enable_noncurrent_version_expiration = true

      abort_incomplete_multipart_upload_days         = null
      noncurrent_version_glacier_transition_days     = 0
      noncurrent_version_deeparchive_transition_days = 0
      noncurrent_version_expiration_days             = 30

      standard_transition_days    = 0
      glacier_transition_days     = 0
      deeparchive_transition_days = 0
      expiration_days             = 1
    },
    {
      prefix  = null
      enabled = true
      tags    = {}

      enable_glacier_transition            = false
      enable_deeparchive_transition        = false
      enable_standard_ia_transition        = false
      enable_current_object_expiration     = true
      enable_noncurrent_version_expiration = true

      abort_incomplete_multipart_upload_days         = 1
      noncurrent_version_glacier_transition_days     = 0
      noncurrent_version_deeparchive_transition_days = 0
      noncurrent_version_expiration_days             = 30

      standard_transition_days    = 0
      glacier_transition_days     = 0
      deeparchive_transition_days = 0
      expiration_days             = 30
    }
  ]
}

Because of what I explained earlier regarding the nature of this issue, I'm going to close it. If I've misunderstood the purpose of this issue, feel free to reopen it.


Update Composer dependencies (2023-12-09-00-11)

Visual regression test passed!
View the visual regression test report


Update nwmatcher dependency

Resolves: https://github.com/tmpvar/jsdom/issues/1740
Due to a bug in the nwmatcher (https://github.com/dperini/nwmatcher/issues/103), query selectors with single characters would return an error. Update nwmatcher minimum version to include bug fix.

Thanks! I rolled this into https://github.com/tmpvar/jsdom/pull/1907.


add a detrainment closure that produces constant area fraction

Purpose

Adds a closure where detrainment = entrainment - vertical divergence, so that the area stays roughly constant. We would like to use this to test prognostic edmf bomex. I don't plan to use this for diagnostic edmf for now, and calculating divergence in diagnostic edmf is a bit annoying, so I didn't implement it. I add an assert to make sure we don't run diagnostic edmf with constant area detrainment.
To-do

Content

[ ] I have read and checked the items on the review checklist.

bors r+

bors r+

bors r+


[WIP] Feat: show project tributes

https://github.com/SporosDAO/sweat-token/issues/93

Nice @cptnanon  ! Very clean code ! On standby to review and test thoroughly when all checks are green.


Add conversion method call 'method()' to '/method'

There will be a command system in the future which will act like this somehow, for now the conversion doesn't really matter.


[Features] FullTypedThrows should only be enable-able in asserts builds.

The implementation of this experimental feature is not complete and it's not ready to be enabled in production.

@swift-ci please smoke test


Explicitly convert params to integers where needed in previews

Authors: Please fill out this form carefully and completely.
Reviewers: By approving this Pull Request you are approving the code change, as well as its deployment and mitigation plans.
Please read this description carefully. If you feel there is anything unclear or missing, please ask for updates.
What are you trying to accomplish?
Fixes https://github.com/primer/view_components/issues/2734.
When you change a param in lookbook, that value gets sent as a string, even if it is a number. We have different components where we allow changing the number of things rendered, and if you try to change the number, it errors currently because we are passing in a string instead of an integer.
This PR just updates those previews to call .to_i, if the param is present. I've confirmed that this fixes changing the params for these previews in lookbook.
Risk Assessment

[x] Low risk the change is small, highly observable, and easily rolled back.

Take a look at the What we look for in reviews section of the contributing guidelines for more information on how we review PRs.

Huh, I'm surprised Lookbook doesn't automatically coerce these to integers. Might be worth an upstream PR?


Added client.buildjobs.artifacts

Hi,
I added the API for getting the artifacts info for a job id.
The usage is

client.buildjobs.artifacts(job_id)`:

client.buildjobs.artifact_url(job_id, file_name)

for job in build['jobs']:
    job_id = job['jobId']
    print("Artifacts for job {}: ".format(job_id))

    artifacts = client.buildjobs.artifacts(job_id)

    for artifact in artifacts:
        file_name = artifact['fileName']
        file_url = client.buildjobs.artifact_url(job_id, file_name)

        print("{file_name}: {file_url}".format(file_name=file_name, file_url=file_url))

fix #14

closing as nobody seems to maintain

@JanC I am no longer using this. Would you like to help maintain it?

My usage was a one off during AppVeyor evaluation and I'm also not actively using this client


Allowing variables in traffic-server's profile derived parameters

Supporting CACHE_IPV4, CACHE_IPV6.
Adding additional variables should be even more simple and safe.
Tested with Kabletown example:
Setting LOCAL proxy.local.incoming_ip_to_bind CAHCE_IPv4 [CAHCE_IPv6]
and finding
LOCAL proxy.local.incoming_ip_to_bind <IP_ADDRESS> [2033:D0D0:3300::2:4/64]
In http://<IP_ADDRESS>:3000/genfiles/view/atsec-chi-02/records.config
Br.
Nir

@nir-sopher - if you rename this PR "[TC-121] Allowing variables in traffic-server's profile derived parameters" I think it will link it to the jira issue...

Cool!
I was started to look for a way to integrate the 2.
Thank you!
Nir
On Wed, Jan 25, 2017 at 7:16 PM, Jeremy Mitchell<EMAIL_ADDRESS>wrote:

@nir-sopher https://github.com/nir-sopher - if you rename this PR
"[TC-121] Allowing variables in traffic-server's profile derived
parameters" I think it will link it to the jira issue...
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/incubator-trafficcontrol/pull/230#issuecomment-275170701,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXGkYEvpBI5yqvff1ZGEVslicZDL1v8Uks5rV4NZgaJpZM4LtvhK
.

PR was rebased so it cam be merged. However, it still needs to be verified again by adding a record to  records.config and checking the value

Closing from now, need to look further into the changes in the files generation code in order to fix


Multiple action buttons show/hide

Hi
I'm using two action buttons - one on the left side and the other on the right side.
The problem is when I open one action button and then tap on the other, both say open.
Is there a way to make action button close when taping on another?
Thanks

yes there is. there are two ways to achieve that:

you can either pass a prop state to handle the active state of RNAB
or you give both RNABs a ref and call the reset method on each other, e.g. when the right button is pressed, you call: this.refs.buttonLeft.reset()

Great, thanks!


Feat: use ruff as python linter!

Description
Replace Flake8 and Black by Ruff!
Ruff is a linter and formatter coded in Rust, allowing real-time warnings (no need to save the file to see the errors!), proposes fixes for most common errors, showing a link with a full description of the problem,
and a far better integration into vscode!
It also now catches more useful errors, and removed previous errors that were not useful.

@rravelli Need your help: I don't understand why the test for resetting the password do not pass

@rravelli Need your help: I don't understand why the test for resetting the password do not pass

Je check ça...


Boolean subquery support

Currently to generate the condition,
 AND (parent_id = 1 OR (SELECT 1 FROM `table` WHERE id = `table`.parent_id AND parent_id = 1))

I have to hack QueryBuilder a bit,
->andWhere(['', 'parent_id = 1 OR', (new Query)->select([new Expression(1)])->from('table')->where('`parent_id` = 1 AND `id` = `table`.`parent_id`')])

Would feel a bit cleaner if you could do the below.
->andWhere(['or', 'parent_id = 1', (new Query)->select([new Expression(1)])->from('table')->where('`parent_id` = 1 AND `id` = `table`.`parent_id`')])

Not sure if there's a better place but to test I put the Query check in the buildAndCondition method.
    public function buildAndCondition($operator, $operands, &$params)
    {
        //...
        if (is_array($operand)) {
            $operand = $this->buildCondition($operand, $params);
        } elseif ($operand instanceof Query) {
            list($operand, $params) = $this->build($operand, $params);
        }
        //...
    }

I can do a PR if needed?

As far as I see, it is already implemented for some conditions:

buildInCondition()
buildSimpleCondition()
buildHashCondition()

Therefore I'm suggesting to expand the practice to the neighbour methods to make them consistent.


Django >= 1.10 support

Hi, thanks for the contribution.
I think it's worth merging however, since the project hasn't been updated in a while, test suite keeps failing.
I'm afraid I don't have much time to work on the project. Would you be willing to try your luck in updating the travis.yml file so that the tests are green again?
I would assume testing against the currently supported Django versions that are 2.2, 2.1 and 1.11 would be enough. I'm also happy with ditching Python 2 from the tests and only focusing on Python 3.


Escaping characters

zrobić merge czy jeszcze czekać na review Dominika?

IMHO merge. Also - may I remind you that this is a public repository :P


Changes to run node

Run node via subprocess.run so that we have access to returncode an…d standard streams
Add language as javascript to JavascriptCodeHandler
Remove option for users to use shorthand for languages for now
Do not throw error for node when status code >= 0 since we get access to message on stderr stream which is not read when in base.py for RuntimeError

This has been deployed as version 0.9.5


WIP universe/uffizzi : cleanup/refactor actions, add test

[x] rename preview actions to environment actions
[x] Create #Run, a resuable construct to help write other actions which
run in the uffizzi container
[x] Add test for uffizzi create environment action which use secrets from sercrets_sops.yaml
[ ] Add test for uffizzi delete environment action which takes in the output of the above create env action

Hi @waveywaves,
You seem to be facing some disjunction issues. Do you need a hand on that ?

@grouville Thanks for letting me know ! I have updated the PR. This PR searches for a UFFIZZI_PASSWORD in the secrets_sops.yml. Is there any way I can add it ?

UFFIZZI_PASSWORD

I don't see any impact on our CI. @gerhard, could you please add the UFFIZZI_PASSWORD password to the Github env vars. I don't have the rights.

This PR searches for a UFFIZZI_PASSWORD in the secrets_sops.yml. Is there any way I can add it ?

Just to make sure that we all understand the purpose of secrets_sops.yml, this is meant for our team to store encrypted secrets in a public repository without worrying that someone outside of our team can access them. When our pipelines run in a forked repository - as is the case here - the forks do not have access to the secret which is required to decrypt secrets_sops.yml. This means that even if we add a UFFIZI_PASSWORD to secrets_sops.yml, it would not be accessible in repository forks.
Would it be possible @waveywaves to rewrite the integration tests so that they don't require a UFFIZZI_PASSWORD value?

I was looking at the Test Universe failure:

they fail due to sops 100 error exit: https://github.com/dagger/dagger/runs/7661497292?check_suite_focus=true#step:8:1091
this error corresponds to NoFileSpecified: https://github.com/mozilla/sops/blob/66043e71a81787d6513bc2e5505a29aac67dc6f1/cmd/sops/codes/codes.go#L24
this can be triggered due to a few reasons: https://github.com/mozilla/sops/blob/e1edc059487ddd14236dfe47267b05052f6c20b4/cmd/sops/main.go

I ran just the uffizzi tests locally, and they pass:

I then ran the entire test suite as the CI runs it, and they fail 🤔

And yet, if I run the same command locally, it works:

I am running out of time today & will be unable to get back to this in the next week. Leaving it with @grouville. Expect best effort progress with this @waveywaves. Thank you!

@gerhard

I am running out of time today & will be unable to get back to this in the next week. Leaving it with @grouville. Expect best effort progress with this @waveywaves. Thank you!

Thanks for helping me out. Looking forward to see this through all the way !


Make v1beta1 cvss proto message compatible with both v3 and v2.

The current v1beta1 CVSS message only supports v3. We cannot correctly fill in the cvss_v2 field in Vulnerability. It cannot represent a CVSS v2 message that, for example, includes IMPACT_PARTIAL.
@aysylu can you check this PR when you are free? This is a fix to https://github.com/grafeas/grafeas/pull/513, which is a PR you have reviewed several months ago. Thank you!
/cc @wkozlik @nms2140

Thank you for this fix.


Need for new release

Installed the latest version and trying to use "CREATE_NEW_USERS" as instructed in the documentation, but was hit with "not a valid configuration directive" error. Turns out it was included in the latest commit that comes after 1.3.1
I understand we can install the latest commit as such:
pip install git+git://github.com/jobec/django-auth-adfs@41fa2b8eca8f80be9db2026a6310673f9cbde1bb
However I reckon a new minor release would be more suitable. So any chance we can release a 1.3.2 to bring the latest changes up to date?

Hi. 1.4.0 is released.

Hi. 1.4.0 is released.


Removed actants handling

When actant is deleted, remove it from other positions, e.g., bookmarks
Also, when server search for actants, do not return those that are not in db


UX Design - Customizations for MVP Features in PDF Tron

Title of ticket: UX Design - Customizations for MVP Features in PDF Tron
Description
This ticket includes the remaining UX Designs needed for the MVP version of PDF Tron
Dependencies
Are there any dependencies?
DOD

[x] How and where will users go to export whole package
[x] How and where will users go to export for consults
[x] Showing Total Pages of all Documents  in Document Tree
[x] Showing individual page totals for each records in the document Tree
[x] Tabs and Options to be removed that are not useful or can damage records in Default version of PDF Tron

@lmullane @m-prodan designs updates listed above have been added to the prototype:
https://jacklyn808742.invisionapp.com/console/share/8QVPHCNXSGA/975452421

Updated spacing and look of left most panel (document tree) to include page flag symbols, total page count, and document page count. And overall improvement of spacing based of participants in the IAO Sessions saying they found it crowded
Updated top bar to remove the options that users do not need/should not have access to.
And added "Left Off Here" button is same style as PDF Tron for IAO Analyst to easily get pulled to page they used that flag on. We should say in the AC that this flag can only be assigned to one page at a time, and user can remove a flag by right clicking and deselecting
Export modal is app is currently designed to be all encompassing, so user can export to desktop or attachment (pending what happens with Abin figuring out encrypted files). And user selecting consults for specific Ministries or specific types of exporting based off of needs

@lmullane @m-prodan @liseandtea  - please share any feedback on anything listed above or in designed


add image to webpage lacks valid path (extension)

add image to webpage lacks valid path (extension)
Challenge Name
Add Images to your Website
Issue Description
"Try it with this image:
https://bit.ly/fcc-relaxing-cat" needs an extension.
https://www.freecodecamp.com/challenges/add-images-to-your-website

Browser Information

Browser Name, Version:
Operating System:
Mobile, Desktop, or Tablet:

Your Code

Screenshot

@smkatz
https://bit.ly/fcc-relaxing-cat is a shortened URL for https://s3.amazonaws.com/freecodecamp/relaxing-cat.jpg
That said, I can understand your confusion when you see this in the instructions, well the shortened URL should work in any case, but can be confusing for some.
./cc @freeCodeCamp/moderators thoughts?

I guess we could maintain a set of redirection links, so campers see https://freecodecamp.com/public/images/relaxing-cat.png instead of the bit.ly link. But I doubt it would be worth the effort of maintaining the redirects.
Another idea is to clarify what the link stands for - Note: the exact contents of the link is not important - all you need to know is that the src attribute allow us to tell the browser from where it should get images

I love FCC. However, as someone who has some familiarity with HTML and CSS,
and some difficulty with certain elements of syntax, I notice that
sometimes things are not explained as completely as a book would do. I
understand all of the above and what I would suggest is at some point
talking about paths -- relative paths, absolute paths, and supported file
extensions. Uploading a .psd is not going to render inline, and they need
to know that. I once had someone make a Dreamweaver site completely of
absolute paths. Needless to say, that didn't go well. Thanks for this great
tool.
On Sat, Feb 18, 2017 at 2:36 PM Samuel Plumppu<EMAIL_ADDRESS>wrote:

I guess we could maintain a set of redirection links, so campers see
https://freecodecamp.com/public/images/relaxing-cat.png instead of the
bit.ly link. But I doubt it would be worth the effort of maintaining the
redirects.
Another idea is to clarify what the link stands for - Note: the exact
contents of the link is not important - all you need to know is that the
src attribute allow us to tell the browser from where it should get images
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/freeCodeCamp/issues/13419#issuecomment-280872969,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AV8PwltEpT1SQ6EGaWMsl0PLEpzVpscAks5rd1ZQgaJpZM4ME_5y
.

@smkatz Thanks for your thoughtful feedback! :+1:
I really like the idea of adding a challenge (to the Basic HTML section?) explaining how links work by giving a couple of examples.
Here's a few types of links I can think of from the top of my head:

Relative - /some/nice/image.png
Absolute - https://example.com/some/nice/image.png
Using the protocol that was used to get the current website - //example.com/some/nice/image.png Explanation of how they work

@smkatz Since you came up with the idea, would you like to help create a challenge to teach this highly valuable concept? Me and other contributors would be happy to guide you through the process :smile:

Like your example with the .psd file, we could also add a challenge to explain different image (or file-) formats. Then we could explain why some images don't work on the web, even though they work in other environments outside of the browser.

I have always been a believer in open source... and would be happy to
contribute in this way. I was trying to master some shell stuff today which
is what delayed me. I'll forgive you for having the same name as me.
On Sun, Feb 19, 2017 at 1:45 PM Samuel Plumppu<EMAIL_ADDRESS>wrote:

Like your example with the .psd file, we could also add a challenge to
explain different image (or file-) formats. Then we could explain why some
images don't work on the web, even though they work in other environments
outside of the browser.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/freeCodeCamp/issues/13419#issuecomment-280943072,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AV8PwrQFVhdZvSj-qie35NGYeKaN-v9Iks5reJvdgaJpZM4ME_5y
.

@smkatz Glad to hear that! :smile:
I think a good work flow could be to create an outline for the challenge as a comment in this issue. Then me and other contributors can 1. give feedback and 2. help you get the challenge added to the website.
So don't worry about having a local environment for the moment - let's focus on how to structure the challenge. Here are some guidelines to help with the structure the challenge. Feel free to ask if you need anything! :blush:

@smkatz Thanks for your feedback. Extensions aren't required for image paths. And since it would significantly increase the length of the URL - and the likelihood of a camper mistyping someone, I'm in favor of leaving this as is.
I like @Greenheart's suggestion of covering relative VS absolute path. If that's something you're interested in pursuing, please open a new issue specific to that, or create an issue and create a PR. I think late in the HTML section would be a good place for it, if we can make it reliably work.

Image paths do not require extensions? I recognize short URLs work because
they redirect but they do require extensions in the standard.
I came up with the rel vs. Absolute path idea.
Bootstrap is awesome.
On Feb 20, 2017 11:55 PM, "Quincy Larson"<EMAIL_ADDRESS>wrote:

@smkatz https://github.com/smkatz Thanks for your feedback. Extensions
aren't required for image paths. And since it would significantly increase
the length of the URL - and the likelihood of a camper mistyping someone,
I'm in favor of leaving this as is.
I like @Greenheart https://github.com/Greenheart's suggestion of
covering relative VS absolute path. If that's something you're interested
in pursuing, please open a new issue specific to that, or create an issue
and create a PR. I think late in the HTML section would be a good place for
it, if we can make it reliably work.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/freeCodeCamp/issues/13419#issuecomment-281252073,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AV8PwttssI555IeKDlm7OAwWEfqNvSnHks5renxIgaJpZM4ME_5y
.

@QuincyLarson Yeah, that was @smkatz's idea! :blush:
Image paths doesn't seem to require extensions


Enable automatic tests on develop branch

Currently build on CI fails.
Package lzo-decompress fails to build.
Some usefu info may be found in https://www.npmjs.com/package/lzo-decompress#instructions-for-tidepool-engineers

Need to continue with this at another time


Console sometimes shows WAITING as the build status just prior to starting task execution

This is a regression introduced by the console performance improvements in 4.9-rc-1.

Expected Behavior
The console build status should transition from CONFIGURING directly to EXECUTING.
Current Behavior
The console build status sometimes transitions from CONFIGURING to WAITING and then to EXECUTING.

Fixed via https://github.com/gradle/gradle/pull/5877


Web version doesn't work with ad blocker

When trying to acces the app from a browser with an ad blocker, an empty screen is shown and the following error is in the console:

This behaviour was observed on Firefox with the uBlock Origin plugin installed.
@GeorgeMD can you please share your experience?

Confirmed
Chrome 86.0.4240.75
uBlock Origin v1.30.2

Related plugin issue: https://github.com/FirebaseExtended/firebase-dart/issues/335

Update:
The linked issue has a workaround that we can try: https://github.com/FirebaseExtended/firebase-dart/issues/335#issuecomment-797003830


Feature/extend history

History table creation query updated.
Updated retrieval of last migration version.
Write migration comment to DB.
Fix tests to use updated methods.

Closes #8


Update the rules during the package installation

It would be nice to have a fresh installation of library with the updated rules.

It is a good idea but if for some reason the remote server does not respond you end up with a failed install. Sometimes new updates break the tests because of new edge cases so they have to be run again in order to make sure the download is solid and effective — if it happens, better the test to rely only on the native Node assert module to avoid relying on devDependencies.
I'd rather expose a CLI command that people can use in their workflow to generate an updated list of rules:
{
  "name": "@my/project",
  "dependencies": {
    "tldjs": "^..."
  },
  "scripts": {
    "postinstall": "npm run update-tld",
    "update-tld": "tldjs update"
  }
}


Don't close dialogs when doing a mouse-up outside of them

I find I close dialogs all the time because it's triggering the close on mouse-up, rather than a full click.
Maybe changing it to a full click might be fine?

Yeah I found I was doing that as well. Couldnt get something inbetween to work, so just removed. It snuck in that last PR so should be good now


Ping Candidate if remote.LastReceived is over than keepaliveInterval

Description
I suffered issue from ICE Connection Status. Issue link
I changed checkKeepalive function, and it successfully works.
local.LastSent time is below than 1 second, but remote.LastReceived value is bigger than before.
So I need to PingCandidate when remote.LastReceived value is over than keepaliveInterval value.
when remote.LastReceived value is over than connectionTimeout, ICEConnectionState goes to disconnected.
Please review my code and fix my issue! Thanks.

435c65f61d7fd35cdbb3e1c0e68f52e92eb11ff7


Allow Builds on all branches

@HahnNYU I think we should set CI to run on more than only main branch. Could we set (revert) it to allow any branch?
https://github.com/gcivil-nyu-org/spring2021-cs-gy-9223-class/blob/0f3a74118050eccd45ca82780525f5ad5013cd9f/.travis.yml#L6-L8

It would be helpful to set up branch-specific travis builds.


[tranche-staking-slice] Tranche Staked SLICE strategy

Fixes # .
Changes proposed in this pull request:

@ayush-jibrel Hey can you just revert the change on package-lock.json?