T2S model inference optimization

I rewrite the inference code for t2smodel.infer_panel, which can achieves about 1.2~1.5 times faster inference speed.
I tested the the token generation speed on a Windows PC and an Ubuntu server with a Geforce 3090 GPU. The token generation speed on Windows is up to 110~120 tokens/s, and that on Ubuntu is up to170~180 tokens/s. I think the main reason of generation speed difference is that SPDA in windows does not compiled with flash attention. By the way, the inference speed with flash attention on linux is about 220~230 tokens/s, but I failed to install flash-attn on Windows and didn't include it in this commit.

Hi, great work! I've verified it on my local RTX4090 card and it boosts the throughput from 55it/s to 85it/s.
Regarding your reported gap between windows and Linux servers, I have a relevant observation. On another RTX4090 card server, with identical docker image (ubuntu) and gpu cards, it achieves 140it/s. I realized that the inference speed might also be subject to other hardware factors, e.g., cpu and motherboard. Anyway, the improvement on the same server is significant. Thanks again for the great work!

Hi, great work! I've verified it on my local RTX4090 card and it boosts the throughput from 55it/s to 85it/s.
Regarding your reported gap between windows and Linux servers, I have a relevant observation. On another RTX4090 card server, with identical docker image (ubuntu-cuda12.1-cudnn8) and gpu cards, it achieves 140it/s. I realized that the inference speed might also be subject to other hardware factors, e.g., cpu and motherboard. Anyway, the improvement on the same server is significant. Thanks again for the great work!
======Update 1=====
Although the throughput diplayed by tqdm has a large gap, the overall inference speed is consistent across servers. Probably the gap across platforms (and servers) is due to the timing error for CUDA asynchronous operations.
======Update 2=====
With the same docker image (ubuntu-cuda12.1-cudnn8), another RTX3090-TI server is x2 faster than RTX4090. Looks like the motherboard plays an important role here...

Which version of PyTorch and Flash Attention are you used in your docker image?

Hi, great work! I've verified it on my local RTX4090 card and it boosts the throughput from 55it/s to 85it/s.
Regarding your reported gap between windows and Linux servers, I have a relevant observation. On another RTX4090 card server, with identical docker image (ubuntu-cuda12.1-cudnn8) and gpu cards, it achieves 140it/s. I realized that the inference speed might also be subject to other hardware factors, e.g., cpu and motherboard. Anyway, the improvement on the same server is significant. Thanks again for the great work!
======Update 1=====
Although the throughput diplayed by tqdm has a large gap, the overall inference speed is consistent across servers. Probably the gap across platforms (and servers) is due to the timing error for CUDA asynchronous operations.
======Update 2=====
With the same docker image (ubuntu-cuda12.1-cudnn8), another RTX3090-TI server is x2 faster than RTX4090. Looks like the motherboard plays an important role here...

Which version of PyTorch and Flash Attention are you used in your docker image? In addition, as ts2 model samples the results, it usually costs different time with the same input.

Pytorch 2.2.1 with latest flash attention. The throughput is not highly dependent on the output length.
I will continue on the benchmarking but so far RTX4090 cannot achieve higher throughput than RTX3090-TI in any cases (please refer to my reply above).

已实测，有提高1.2倍的推理速度~

Hi, great work! I've verified it on my local RTX4090 card and it boosts the throughput from 55it/s to 85it/s.
Regarding your reported gap between windows and Linux servers, I have a relevant observation. On another RTX4090 card server, with identical docker image (ubuntu-cuda12.1-cudnn8) and gpu cards, it achieves 140it/s. I realized that the inference speed might also be subject to other hardware factors, e.g., cpu and motherboard. Anyway, the improvement on the same server is significant. Thanks again for the great work!
======Update 1=====
Although the throughput diplayed by tqdm has a large gap, the overall inference speed is consistent across servers. Probably the gap across platforms (and servers) is due to the timing error for CUDA asynchronous operations.
======Update 2=====
With the same docker image (ubuntu-cuda12.1-cudnn8), another RTX3090-TI server is x2 faster than RTX4090. Looks like the motherboard plays an important role here...

Which version of PyTorch and Flash Attention are you used in your docker image? In addition, as ts2 model samples the results, it usually costs different time with the same input.

Just achieved 400it/s with AMD Ryzen 9 7900X. It's interesting to see that the major bottleneck of online inference is actually CPU instead of GPU.

The GPT model used is so small that inference on high-performance Gpus is difficult to create performance bottlenecks. More of the performance bottleneck should be in the performance of the iteration itself. In this process, high-performance Gpus are difficult to fully load.

嗨，干得好！我已经在本地 RTX4090 卡上验证了它，它在同一服务器上将吞吐量从 55it/s 提高到 85it/s。
关于您报告的 Windows 和 Linux 服务器之间的差距，我有一个相关的观察。在另一台RTX4090卡服务器上，具有相同的docker镜像（ubuntu-cuda12.1-cudnn8）和gpu卡，它达到了140it/s。我意识到推理速度还可能受到其他硬件因素的影响，例如CPU和主板。无论如何，在同一个服务器上的改进是显着的。再次感谢您所做的出色工作！
======更新1=====
虽然显示的吞吐量差距tqdm较大，但各服务器的整体推理速度是一致的。跨平台（和服务器）之间的差距可能是由于 CUDA 异步操作的计时错误造成的。
======更新2=====
使用相同的 docker 映像 (ubuntu-cuda12.1-cudnn8)，另一台 RTX3090-TI 服务器比 RTX4090 快 2 倍。看来CPU在这里扮演着重要的角色......
不同硬件上相同系统和软件环境的基准测试结果（20 次运行）：
RTX4090 + X99 + Xeon® E5-2686：80it/s RTX4090 + ROG CROSSHAIR + AMD Ryzen 9 5900x：195it/s RTX4090 + ROME2d32gm + AMD EPYC 7662：100it/s RTX4090 + B650 LiveMixer + AMD Ryzen 9 7900X： RTX3090Ti + ROG Strix + AMD Ryzen 9 5900x：400it/s230it/s
看来GPU并不是当前在线推理的主要瓶颈。~如果有人可以实现比230it/sRTX4090 更高的吞吐量，我有兴趣了解系统设置。~

CPU的频率会比较大的影响推理速度

Hi, great work! I've verified it on my local RTX4090 card and it boosts the throughput from 55it/s to 85it/s on the same server.
Regarding your reported gap between windows and Linux servers, I have a relevant observation. On another RTX4090 card server, with identical docker image (ubuntu-cuda12.1-cudnn8) and gpu cards, it achieves 140it/s. I realized that the inference speed might also be subject to other hardware factors, e.g., cpu and motherboard. Anyway, the improvement on the same server is significant. Thanks again for the great work!
======Update 1=====
Although the throughput diplayed by tqdm has a large gap, the overall inference speed is consistent across servers. Probably the gap across platforms (and servers) is due to the timing error for CUDA asynchronous operations.
======Update 2=====
With the same docker image (ubuntu-cuda12.1-cudnn8), another RTX3090-TI server is x2 faster than RTX4090. Looks like the CPU plays an important role here...
Benchmark results (20 runs) with the same system and software environment on different hardwares:
RTX4090 + X99 + Xeon® E5-2686: 80it/s RTX4090 + ROG CROSSHAIR + AMD Ryzen 9 5900x: 195it/s RTX4090 + ROME2d32gm + AMD EPYC 7662: 100it/s RTX4090 + B650 LiveMixer + AMD Ryzen 9 7900X: 400it/s RTX3090Ti + ROG Strix + AMD Ryzen 9 5900x: 230it/s
It seems GPU is not the major bottleneck for current online inference. ~If anyone can achieve higher throughput than 230it/s with RTX4090, I would be interested to know the system setting.~

Thanks for you comprehensive benchmark on different hardwares. I implemented part of the inference codes with TorchScript w/o flash-attention and pushed the them to a new branch torchscript. Although it does not leverage the flash-attetntion library, it achieves 150~160 tokens/s on Windows 11 + i5-12600K + RTX 3090, and 220~230 tokens/s on Linux + AMD Ryzen 9 5950x + RTX 3090.
I think this improvement can signifiantly benefit Windows users. If possible, could you help to test this implementation on more platfroms, especially on RTX4090 + Xeon® E5-2686?
By the way, to my knowledge, flash-attention 2.2+ implemented flash decoding to improve the token generation speed, but SDPA does not.

Hi, great work! I've verified it on my local RTX4090 card and it boosts the throughput from 55it/s to 85it/s on the same server.
Regarding your reported gap between windows and Linux servers, I have a relevant observation. On another RTX4090 card server, with identical docker image (ubuntu-cuda12.1-cudnn8) and gpu cards, it achieves 140it/s. I realized that the inference speed might also be subject to other hardware factors, e.g., cpu and motherboard. Anyway, the improvement on the same server is significant. Thanks again for the great work!
======Update 1=====
Although the throughput diplayed by tqdm has a large gap, the overall inference speed is consistent across servers. Probably the gap across platforms (and servers) is due to the timing error for CUDA asynchronous operations.
======Update 2=====
With the same docker image (ubuntu-cuda12.1-cudnn8), another RTX3090-TI server is x2 faster than RTX4090. Looks like the CPU plays an important role here...
Benchmark results (20 runs) with the same system and software environment on different hardwares:
RTX4090 + X99 + Xeon® E5-2686: 80it/s RTX4090 + ROG CROSSHAIR + AMD Ryzen 9 5900x: 195it/s RTX4090 + ROME2d32gm + AMD EPYC 7662: 100it/s RTX4090 + B650 LiveMixer + AMD Ryzen 9 7900X: 400it/s RTX3090Ti + ROG Strix + AMD Ryzen 9 5900x: 230it/s
It seems GPU is not the major bottleneck for current online inference. ~If anyone can achieve higher throughput than 230it/s with RTX4090, I would be interested to know the system setting.~

Thanks for you comprehensive benchmark on different hardwares. I implemented part of the inference codes with TorchScript w/o flash-attention and pushed the them to a new branch torchscript. Although it does not leverage the flash-attetntion library, it achieves 150~160 tokens/s on Windows 11 + i5-12600K + RTX 3090, and 220~230 tokens/s on Linux + AMD Ryzen 9 5950x + RTX 3090.
I think this improvement can signifiantly benefit Windows users. If possible, could you help to test this implementation on more platfroms, especially on RTX4090 + Xeon® E5-2686?
By the way, to my knowledge, flash-attention 2.2+ implemented flash decoding to improve the token generation speed, but SDPA does not.

Notes: Both implementations have significant improvements over the original version. Torchscript version consistently outperforms the current PR.

Original
FlashAttn
Torchscript w/o flash

RTX4090 + Xeon® E5-2686
54it/s
77it/s
83it/s

RTX4090 + AMD Ryzen 9 7900X
248it/s
387it/s
405it/s

RTX4090 + Core™ i9 11900
178it/s
254it/s
283it/s

RTX4090 + Core™ i9 13900
378it/s
545it/s
585it/s

@GoHomeToMacDonal @wehos Excellent! In my machine, there is approximately 50% acceleration.
Welcome to explore more acceleration techniques!
3090+i5-13490F
torch2.0.0+cu118+original code:
39%GPU+100%CPU(Single threaded)
4.746   0.415   2.697   0.739
0.118   0.032   3.328   0.220
112it/s
torch2.2.1+cu118+original code:
47%GPU+100%CPU(Single threaded)
2.962   0.397   2.811   0.400
0.117   0.031   3.056   0.320
117it/s
torch2.2.1+cu118+new code:
55%GPU+100%CPU(Single threaded)
0.113   0.033   2.190   0.340
0.115   0.035   2.064   0.508
164it/s

@GoHomeToMacDonal
3090+i5-13490F+py39+win10-19045.4046-22H2
torch2.2.1+cu118+torchscript version(b65dae788df75e29e1b16cec1681918efd7ae4f7):
60%GPU+100%CPU(Single threaded)
0.144   0.033   1.627   0.487
AR:199it/s
double speed!

@RVC-Boss
SDPA calls C/C++ function torch._C._nn.scaled_dot_product_attention , so I think torch.compile cannot improve its perormance.
Compiling flash-attn on Windows packages requires git, msvc and nvcc, and it costs about 1 hour with a 12-core CPU.

RTX6000 , 三个instance worker, 每个两个thread并发，相同请求下，能从10 QPM 提升到 15 QPM

直接替换t2s_model, 效果很显著。

RTX6000 , 三个instance worker, 每个两个thread并发，相同请求下，能从10 QPM(每分钟请求量) 提升到 15 QPM

您好， 测试过RTX6000最多能开多少个实例吗？

直接替换t2s_model, 效果很显著。

RTX6000 , 三个instance worker, 每个两个thread并发，相同请求下，能从10 QPM(每分钟请求量) 提升到 15 QPM

您好， 测试过RTX6000最多能开多少个实例吗？

按照显存用量能开到6个。一般压力测试4个instance基本上SM就能用满了。

刚才验了下
torchscript版本
控制种子的话，第一遍合成的结果，2倍加速版和baseline结果完全一样的
准备合进main了


Automate benchmark result aggregation

We need some sort of script that runs all benchmarks automatically, generating the result files, which can then be used by the make_plots script to generate fancy graphic results.

I am already working on this.


Faster completion provider tests

Codecov Report

Merging #11903 (e80e2b6) into main (a6ab021) will decrease coverage by 0%.
The diff coverage is 75%.

@@           Coverage Diff           @@
##            main   #11903    +/-   ##
=======================================
- Coverage     62%      61%    -1%
=======================================
  Files        491      490     -1
  Lines      34541    34419   -122
  Branches    5600     5583    -17
=======================================
- Hits       21644    21307   -337
- Misses     10800    11081   +281
+ Partials    2097     2031    -66

Impacted Files
Coverage Δ

src/kernels/jupyter/launcher/serverUriStorage.ts
61% <0%> (-1%)
:arrow_down:

src/kernels/jupyter/types.ts
100% <ø> (ø)

...er/finder/universalRemoteKernelFinderController.ts
62% <74%> (+38%)
:arrow_up:

...nels/jupyter/finder/universalRemoteKernelFinder.ts
86% <100%> (+74%)
:arrow_up:

src/kernels/jupyter/serviceRegistry.node.ts
100% <100%> (+2%)
:arrow_up:

...otebooks/debugger/controllers/restartController.ts
10% <0%> (-75%)
:arrow_down:

...ebooks/debugger/controllers/runByLineController.ts
13% <0%> (-68%)
:arrow_down:

src/notebooks/debugger/kernelDebugAdapter.ts
20% <0%> (-65%)
:arrow_down:

src/notebooks/debugger/protocolParser.node.ts
14% <0%> (-65%)
:arrow_down:

src/notebooks/debugger/debuggingManager.ts
30% <0%> (-50%)
:arrow_down:

... and 30 more


refactor: SipiServiceLive: Sttp instead of Apache HTTP (DEV-1627)

Pull Request Checklist
Task Description/Number

Issue Number: DEV-
PR Type

[ ] build/chore: maintenance tasks (no production code change)
[ ] docs: documentation changes (no production code change)
[ ] feat: represents new features
[ ] fix: represents bug fixes
[ ] perf: performance improvements
[x] refactor: represents production code refactoring
[ ] test: adding or refactoring tests (no production code change)
[ ] deprecated: Deprecation warning (ideally referencing a migration guide)

Basic requirements for bug fixes and features

[ ] Tests for the changes have been added
[ ] Docs have been added / updated

Does this PR introduce a breaking change?

[ ] Yes

@BalduinLandolt Yeah, I had to pay some attention not to miss anything while refactoring, but other than that quite straightforward and due to the better API, very compactable code.


Support helm charts

Support deployment for templates that are using helm charts

Right now only supporting Helm charts by using Helm as a templating engine.
Using Helm as a "package manager for Kubernetes" might not be a good direction from the GitOps perspective.
GitOps maintain a "git as a single source of truth".
When deployed in K8s the control plain will be holding another "source of truth" (The current state).
Adding another state maintained by Helm will be detrimentive rather than helpful.
On top of that Helm might introduce extrenal dependenies that will be hard to control and update (dependency on charts etc.) making the "builds" non-idempotent and not self-contained.


Support for collection relationships?

Hi, how can I use relationships between collections? For example, in the Todos example, how can I define a relationship between a todo and list such that selecting a list also selects its todos?

So far this package provides no support for relationships or joins. There are other packages that implement reactive joins (for example method Meteor.publishComposite() [https://github.com/englue/meteor-publish-composite]) but they may need to be modified to work with Oracle.Collection

Ok, thanks. I found a decent solution using the models pattern suggested by David Weldon [http://dweldon.silvrback.com/models]. It's essentially the same thing as collection-helpers, but it doesn't extend Mongo.Collection, so it's database-independent.
I noticed that you have reactive joins in your roadmap. Have you started work on that yet?

We didn't come up with an efficient design for reactive joins, I guess we should wait to see how it is addressed in Meteor/Apollo.

Ok, I'll watch Apollo. Thanks!


[C++][Parquet] Account for Arrow API changes in ARROW-1199

Reporter: Wes McKinney / @wesm
Note: This issue was originally created as PARQUET-1054. Please see the migration documentation for further details.

Wes McKinney / @wesm:
Issue resolved by pull request 370
https://github.com/apache/parquet-cpp/pull/370


invalid pointer to UINT casts in src/Cedar/Command.c

Prerequisites

[x] Can you reproduce?
[X] Are you running the latest version of SoftEtherVPN?

SoftEther version: 5.02.5180, but bug exists in master too
Component: [all]
Operating system: [Linux/musl libc] - you can test it on alpine linux
Architecture: [64 bit (x86_64)]
Description
[Description of the bug]
vpncmd - 3 - check crashes on 64bit systems due to invalid pointer to int casts:
              Pass
Checking 'Network system'...

Thread 1 "vpncmd" received signal SIGSEGV, Segmentation fault.
[Switching to LWP 17438]
0x00007ffff7db7027 in a_cas (s=-2147483647, t=0, p=0xfffffffff6451be8)
    at ./arch/x86_64/atomic_arch.h:4
4       ./arch/x86_64/atomic_arch.h: No such file or directory.
(gdb) bt
#0  0x00007ffff7db7027 in a_cas (s=-2147483647, t=0, p=0xfffffffff6451be8)
    at ./arch/x86_64/atomic_arch.h:4
#1  __lock (l=l@entry=0xfffffffff6451be8) at src/thread/__lock.c:24
#2  0x00007ffff7db9642 in pthread_kill (t=0xfffffffff6451b38, sig=sig@entry=10)
    at src/thread/pthread_kill.c:12
#3  0x00000000005228aa in Disconnect (sock=sock@entry=0x7ffff6719820)
    at /src/build/softethervpn/SoftEtherVPN-5.02.5180/src/Mayaqua/Network.c:13066
#4  0x000000000040a6df in CheckNetwork ()

as you can see in backtrace, t=0xfffffffff6451b38 is an occurence of a 64bit pointer which was cast to 32 bit UINT and then cast back to void, where it got sign-extended and lost the upper 32 bits.
and the build log is full of warnings of this type:
/src/build/softethervpn/SoftEtherVPN-5.02.5180/src/Cedar/Command.c: In function 'CheckThread1':
/src/build/softethervpn/SoftEtherVPN-5.02.5180/src/Cedar/Command.c:316:15: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
   InputToNull((void *)check_thread_global_1);
               ^
/src/build/softethervpn/SoftEtherVPN-5.02.5180/src/Cedar/Command.c: In function 'CheckMemory':
/src/build/softethervpn/SoftEtherVPN-5.02.5180/src/Cedar/Command.c:606:16: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
    InputToNull((void *)(UINT)(((UCHAR *)pp[i])[j] = j % 256));
                ^
/src/build/softethervpn/SoftEtherVPN-5.02.5180/src/Cedar/Command.c: In function 'StrToPortList':
/src/build/softethervpn/SoftEtherVPN-5.02.5180/src/Cedar/Command.c:7886:19: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
   if (IsInList(o, (void *)n))
                   ^
/src/build/softethervpn/SoftEtherVPN-5.02.5180/src/Cedar/Command.c:7892:10: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]
   Add(o, (void *)n);
          ^
/src/build/softethervpn/SoftEtherVPN-5.02.5180/src/Cedar/Command.c: In function 'PsClusterSettingMember':
/src/build/softethervpn/SoftEtherVPN-5.02.5180/src/Cedar/Command.c:7969:17: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
    t.Ports[i] = (UINT)LIST_DATA(ports, i);
...

Expected behavior:
working
Actual behavior:
crashing
Steps to reproduce

build from source and see warnings
run vpncmd , hit 3, then enter check, see segfault

solution
whenever  a pointer needs to be cast to integer and vice versa (which is hackish to begin with), an integer type of sufficient size needs to be used. POSIX provides the uintptr_t type for this purpose, which is defined in stdint.h.

actually more problematic, and likely direct source of the crash, are the casts done in src/Mayaqua/Network.c
/src/build/softethervpn/SoftEtherVPN-5.02.5180/src/Mayaqua/Network.c: In function 'NewCancelSpecial':
/src/build/softethervpn/SoftEtherVPN-5.02.5180/src/Mayaqua/Network.c:10169:17: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
  c->pipe_read = (int)hEvent;
                 ^
/src/build/softethervpn/SoftEtherVPN-5.02.5180/src/Mayaqua/Network.c: In function 'SecureRecv':
/src/build/softethervpn/SoftEtherVPN-5.02.5180/src/Mayaqua/Network.c:12125:26: warning: implicit declaration of function 'pthread_self' [-Wimplicit-function-decl$
    sock->CallingThread = pthread_self();
                          ^~~~~~~~~~~~
/src/build/softethervpn/SoftEtherVPN-5.02.5180/src/Mayaqua/Network.c:12125:24: warning: assignment makes pointer from integer without a cast [-Wint-conversion]
    sock->CallingThread = pthread_self();
                        ^
/src/build/softethervpn/SoftEtherVPN-5.02.5180/src/Mayaqua/Network.c: In function 'Recv':
/src/build/softethervpn/SoftEtherVPN-5.02.5180/src/Mayaqua/Network.c:12359:23: warning: assignment makes pointer from integer without a cast [-Wint-conversion]

the latter ones are caused by not including pthread.h, therefore lacking the prototype for pthread_self(), so the pointer type retruned gets truncated.

OK. So the cause of the crash is irrelevant to your first allegation.
And your second allegation has been outdated since #1534 .

yes, including pthread.h fixes the crash. i still couldnt get a connection to vpngate working though, as all tutorials are written for gui users, so there's no info for what values to fill into "accountcreate" "accountpassword", but i suppose what's the real blocker is that "niccreate" doesn't create a tun/tap device, so on "accountconnect" the status stays "connecting" forever.
either way, the warnings mentioned should be fixed as this is a real codesmell and undefined behaviour. anyone who's ever ported a 32bit app to 64bit and experiencing a crash will immediately assume it's caused by these illegal conversions.

NicCreate creates a tap device if given an option.
VPN gate is supposed to be used from the proprietary vpn gate client. If using the open source softether client, enter vpn as username and password, vpngate as hub.
Compiler warnings had, of course, better be fixed. However they are part of the historical debt of this decade-old project. It needs to be done very carefully.

CentOS7版视频安装教程：https://youtu.be/fWGw6Qdh1pY
Windows版视频安装教程：https://youtu.be/lXb7FxxEcE0


Added translations for pl_PL

I could not find it at Transifex (the Poll plugin is already there), so here is a PR :gift:

Thanks!


add guide to build and deploy contract with Celatone

What is the purpose of the change
This pull requests adds instructions on using Beaker and Celatone to build and deploy contracts on the Osmosis network, providing developers with an initial guide on using Celatone in their development workflow
Brief change log

Add page on building contract with Beaker and uploading/instantiating contracts with Celatone

Verifying this change
This change has been tested locally by rebuilding the doc website and verified content and links are expected
To Dos

[ ] Add guide on using Celatone to upload code once feature is in production
[ ] Add guide on using Celatone to add addresses to uploadAccess list once feature is in production

Closing this issue due to long inactivity on the PR


Player needs a way to be opened after it's been closed

right now if you close the player, it keeps playing, but there isn't a way to get it back open unless you click play on the audio again.
Would be nice for it to minimize into just the show title / 1 line high

Can we get a hotkey? 👀

Ooo yah, what should the key be?

'p' for us screen reader users jumps to the next paragraph and reads it. Single key hotkeys are mostly taken up by screen reader manufacturers. Kind of a bummer and a pain, I know.
I can "pass a key through to the application/browser, but that can provide UX issues for those of us who need to use assistive technologies to equitably access browser functionality.
Best,
David

I think this issue needs to be in three parts:

We need to stop the audio when users click the x
We need a way to minimize the player / make a slim player
we need keyboard shortcuts

I'm really keen to help out with this 🙋🏽♂
I converted my hotkey PR to a draft for now - I think I could maybe tackle part 1 & 3 perhaps with some guidance on what you would want the shortcuts to be/do.
Also, perhaps a part 4 would be having some UI that shows you what shortcuts are available?

Awesome, yes would love the help!
Some guidance:

you'll have to expand the close() function in player.ts. probably do two things:

state.audio?.pause(); // pause it
state.current_show = null; // remove current show

youll need to make a new $player.status state called "MINIMIZED", which corresponds to a slim UI of the player. Maybe just the scrubber bar and an icon to maximize it again?

you did it

Site wide ? keyboard shortcut that opens up a popover (see Popover API stuff scott did) to display all the possible shortcuts. Ideally this is generated from a list of the shortcuts available and not just HTML

This has been fixed with a minimized player and closing player resetting the player.


When official review is deleted, previous review should become official

Description
As stated here, only reviewers latest review of type approve and reject shall count as "official". So when submitting a new "official" review, the "official" status of existing reviews of that user are cleared.
However, when deleting a review (currently only possible through the API), the "official" status of the previous approval/rejection of that user is never restored.
Gitea Version
1.18.0
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots
No response
Git Version
No response
Operating System
No response
How are you running Gitea?
docker
Database
None

Maybe we should disallow to delete an approval or reject review?

I hope not. I use it in my workflow. We have a bot account that automatically submits approvals in some scenario's.
If the bot was triggered by mistake we use this functionality to delete the approval.


About Wait-Debugger

When I deploy the function code to Azure, will it be broken if I keep "Wait-Debugger" in the code?
[Enter feedback here]

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

ID: c91c3121-aea2-9a38-c44c-45cc5bbc9248
Version Independent ID: c3c5199d-cff8-f313-dfc6-41499941e505
Content: Debug PowerShell Azure Functions locally
Content Source: articles/azure-functions/functions-debug-powershell-local.md
Service: azure-functions
GitHub Login: @TylerLeonhardt
Microsoft Alias: tyleonha

@RonnieQuan Thank you for the question. We will review and get back to you.

@RonnieQuan Yes, it will break the code with " Wait-Debugger" in azure. However, We are closing this issue as this is not related to documentation enhancement. If you have any related question please post that in Microsoft QnA forum.


Task - New Taxon Name - Make sidebar vertical sticky

   when I scroll the page, and the top bar is sticky,
     then the ride side name preview and soft validation panels should also stick

Like this:

Issue: We would need a double scroll if validation list gets too large.

Beautiful!


Modify one of the examples to showcase resolveConflicts()

Currently none of the examples (canvas, chat, grid) demonstrates the use of resolveConflicts(). One important purpose of examples is to showcase the programmability of CROs, where conflict resolution strategy plays a key role in enabling richer CRO behavior.
We already have three examples, whose CROs are designed in similar ways. Suggestion here is to modify one of the examples to showcase meaningful use of conflict resolution.

Can I take this?


Request: Include Embedding Models

It would be great to have some embedding models as part of the Ollama model collection (such as "all-MiniLM-L6-v2"). Mistral and the others (I have tried) are unfortunately slow to handle embedding tasks for QA retrieval applications.

Yes, this is a major pain when working with open source models / tooling today.

Do you maybe know if this slowness is universal across all (or most) embedding models that currently run locally? I have limited experience in this field :D.

Thanks so much for adding this issue. Good small embedding models are definitely an important thing for us to support but there is a duplicate issue #327 that also covers this. So I will go ahead and close this one to make it a bit easier to track. Thanks for being part of this great community.


Improve HMR performance

Coverage remained the same at 60.0% when pulling f9e086d65e7a342c1fe08d9d00a8e5a2fb1fc4e0 on feature/hmr into 7c5438e6b5c21860ef65d7210a34f462e7b9d0fa on master.


Roll Flutter Engine from ddfd3dce13bc to ddf8d7c44ae4 (1 revision)

https://github.com/flutter/engine/compare/ddfd3dce13bc...ddf8d7c44ae4
2023-02-25<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from yH66bYEkvZAwnmz1h... to wtSn1iCpHYqcNOWFY... (flutter/engine#39879)
Also rolling transitive DEPS:
fuchsia/sdk/core/mac-amd64 from yH66bYEkvZAw to wtSn1iCpHYqc
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC chinmaygarde@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


discussion: unix piping and error resource

Naturally these tools are great for piping things around. Perhaps it may be a good idea to define a generic Error object. Then tools upstream in the pipeline can emit these errors when they cannot proceed and they can be handled gracefully.
---
apiVersion: v1
kind: Namespace
metadata:
  name: example
---
apiVersion: generatingToolName/v1
kind: Error
metadata:
  name: Warning
  message: Missing required parameter to generate Kubernetes Manifest

Essentially if kapp comes across an Error object, It will not allow deployment and will gracefully show the error received (rather than some generic unable to parse YAML). The kind would always be "Error" and any tool can define its own apiVersion.

is there a particular problem you are solving with this solution? ytt for example doesnt output things to stdout if there was an error as far as i remember. and kapp will refuse to deploy "no resources" stream (unless particular flag is given).

No. No problems in particular.

i see. ive thought previously about adding something like that, however, only use case i could find is when output is streamed. most tools out there dont stream "configuration" output (including ytt, kbld), so i did not end up implementing this.  this is why i was wondering if you hit another use case that needed this feature.


Improve ERP tutorial

I tried to streamline the ERP tutorial a bit, after which it hopefully reads a bit better.
Two questions:

Screenshots of Raw.plot() or Epochs.plot() should use the matplotlib backend I think because it looks better in non-interactive documents.
Do we not append () to methods and functions? If so, should I prepend :meth: (or :func:) in this tutorial?

Before vs. after

@cbrnr can you share links to rendered doc before and after? 🙏

EEG processing - Event Related Potentials (ERPs)
or
EEG analysis - Event Related Potentials (ERPs)
fits?
and
Averaging across channels with regions of interest (ROIs)
is short enough?
Message ID: @.***>

But why do you think that "EEG" is necessary here? The tutorial is about ERP processing, it shows all steps that are involved. Just because some steps can also be applied in different analyses doesn't mean we have to mention it in the title. The first sentence of the tutorial is: "This tutorial shows how to perform standard ERP analyses in MNE-Python." So I'd think that "Event-related potentials (ERPs)" is a good title? It is also in the category "Estimating evoked responses", which fits perfectly.

is there another tutorial that is focus on basid EEG processing steps? that we could people towards before this tutorial?

We are already providing links to other tutorials that contain more details. This tutorial aims to briefly summarize all steps required for ERP processing, and we mention this in the first paragraph:

This tutorial shows how to perform standard ERP analyses in MNE-Python. Most of the material here is covered in other tutorials too, but for convenience the functions and methods most useful for ERP analyses are collected here, with links to other tutorials where more detailed information is given.

I will not argue too much about this. But I want to make sure the decision is not too shortsighted. If @drammock did it this way I am sure he had a reason. Maybe accessibility/find-ability etc... @drammock spent months to improve our doc so let's make sure we pursue this fantastic effort 🙏 @cbrnr

Of course! This is also my goal here, and I think using the shorter title would further clarify it. However, if there are reasons unknown to me it is best that we wait for @drammock to comment (which means that we have to wait at least until mid-May though). We can also merge the rest for now and I'll open an issue to deal with the title.
The other heading "Averaging across channels with regions of interest (ROIs)" is still too long. It would be one line if we scrape the "(ROIs)" at the end. OK?

Only one issue remaining: do we use :meth:, :func:, or similar tags to format function/method names with parentheses? Or no?

if you put parentheses you cannot use :meth: or :func: AFAIK. Why do you need the () ?

No, the problem is that currently most of the tutorial uses neither :meth: nor parentheses, so a method (function) looks like this: mne.set_eeg_reference. I can't remember what we ended up deciding how to handle method/function formatting in our docs.

I would just follow how it's done in other tutorials eg
https://github.com/mne-tools/mne-python/blob/main/tutorials/preprocessing/10_preprocessing_overview.py

Message ID: @.***>

This one's finally ready: https://output.circle-artifacts.com/output/job/59d97ec1-522c-4398-b6fb-cb6bc7bab6e6/artifacts/0/dev/auto_tutorials/evoked/30_eeg_erp.html

The tight layout change works, docs look good:
https://output.circle-artifacts.com/output/job/fd3a8c4e-d847-43af-a170-1d05356ac190/artifacts/0/dev/auto_tutorials/evoked/30_eeg_erp.html

Thanks @cbrnr !


Please add support for disabling autoDeploy in server.xml

Currently, the value is hard coded to true in server.xml.erb.
Please consider making this an attribute to allow configuration.

First off I'm very sorry that it's taken so long to get back to you on this issue.  For a significant period of time there was no active Chef Software Inc. maintainer for this repository and things languished.  We're now actively maintaining this cookbook again, and as part of that I've rewritten the cookbook with new custom resources for installing Tomcat from Apache Org sources.  This will ship in an upcoming 2.0 release and then we'll remove the existing recipes and providers in a future 3.X release.  With those new providers I believe the need for this change goes away so I'm going to close this out.  I encourage you to give the new providers a try and let me know if we can tackle more Tomcat management without the attributes bloat the current code suffers from.  They're on the master branch and once the documentation and testing improves a bit they'll be released as a 2.0 cookbook.


Trying to access any select2 method on the select throws an error.

Trying to access any select2 method on the select throws an error.
It seems that DAL doesn't pass the select2 class to it.
$('#my_select2').select2('open')

The select2('open') method was called on an element that is not using Select2.

I've discovered that the error occurs because autocomplete.js isn't loaded yet on "$(document).ready()".
So, how can I know that it has been initialized?


updated deployment to get kubearmor hostname fix

Ref: kubearmor/kubearmor#736
Signed-off-by: Rahul Jadhav<EMAIL_ADDRESS>

We do need an automatic CI trigger on changes merged to individual package, to trigger dependency update PR 😅

Auto-updating the dependency is also a risky affair, hence I am ok with this. WDYT?

We just raise a PR, would need a review from maintainers before we merge and create release.
But with the frequency we have right now, I guess it should be fine for now.

That will be great, if a bot could raise a PR for maintainers to review!


objects not always discovered by scala_test rule

a test like:
object Foo extends FunSuite {
  test("foo") { assert(false) }
}

Seems to silently look like a passed test as no suites are discovered.
Changing to class Foo extends FunSuite seems to fix.
I don't know if this is a bug with scala-test or rules scala or a combination.
I think we should probably error when someone makes a test that has no discovered suites. It might help people catch issues like this.

We error in junit discovery in such a case so I definitely agree
On Fri, 2 Nov 2018 at 2:56 P. Oscar Boykin<EMAIL_ADDRESS>wrote:

a test like:
object Foo extends FunSuite {
test("foo") { assert(false) }
}
Seems to silently look like a passed test as no suites are discovered.
Changing to class Foo extends FunSuite seems to fix.
I don't know if this is a bug with scala-test or rules scala or a
combination.
I think we should probably error when someone makes a test that has no
discovered suites. It might help people catch issues like this.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/rules_scala/issues/637, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABUIF3GNmbYxrUpOOBCop6Li6U2jEDvtks5uq5g5gaJpZM4YKa4T
.

yeah, and also munit has this behavior if I'm not mistaken. I just never use this pattern anymore.
I think we can close this.


chore: release main

:robot: I have created a release beep boop
baggie: 2.3.0
2.3.0 (2023-11-16)
Features

Add defaultValue as option property for useQueryState (d6a658f)
Add useQueryMapper (1663d31)

Bug Fixes

Make react package type module (98838f1)
Storybook exports typescript errors (8a9eaf5)

react: 1.2.0
1.2.0 (2023-11-16)
Features

Add defaultValue as option property for useQueryState (d6a658f)
Add useQueryMapper (1663d31)

Bug Fixes

Make react package type module (98838f1)
Storybook exports typescript errors (8a9eaf5)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/saxofonsolo/baggie/releases/tag/react-v1.2.0 :sunflower:


Implement D3 in React

For now we will proceed with BubbleChart.js and potentially look into changing the implementation of visualization to D3 at a later date.


Datasource not loading in experience builder preview.

I am working on a custom widget and my code works while in the builder mode of experience builder. However, whenever I preview my widget, the datasource is undefined. Is this by design? What causes this?
I appreiciate any help.

`import {
DataSource,
DataSourceManager,
React,
type AllWidgetProps,
} from "jimu-core";
import { type IMConfig } from "../config";
import { JimuLayerView, JimuMapView, JimuMapViewComponent } from "jimu-arcgis";
import { MouseEvent, useState } from "react";
import { Dropdown, DropdownButton, DropdownItem, DropdownMenu } from "jimu-ui";
import ConditionController from "./conditionController";
import * as reactiveUtils from<EMAIL_ADDRESS>const Widget = (props: AllWidgetProps) => {
const [dataSources, setDataSources] = useState<DataSource[]>([]);
const [selectedDataSource, setSelectedDataSource] = useState({
dataSource: null,
schema: null,
});
function onActiveViewChange(activeView: JimuMapView) {
if (activeView) {
activeView.whenAllJimuLayerViewLoaded().then((jimuLayerViews) => {
const dataSources = Object.values(jimuLayerViews).map((jimuLayerView) =>
jimuLayerView.getLayerDataSource()
);
    setDataSources(dataSources);
  });
} else {
  setDataSources([]);
}

}
function handleDataSourceSelection(mouseEvent: MouseEvent<any, MouseEvent>) {
const ds = dataSources.find(
(ds) => ds.id === (mouseEvent.target as HTMLInputElement).value
);
setSelectedDataSource({ dataSource: ds, schema: ds.getSchema() });

}
return (

QueryBuilder Widget
  {dataSources.length > 0 && (
    <Dropdown activeIcon menuItemCheckMode="singleCheck" menuRole="listbox">
      <DropdownButton>
        {selectedDataSource.schema?.label || "Select a Datasource"}
      </DropdownButton>
      <DropdownMenu>
        {dataSources.length > 0 &&
          dataSources.map((dataSource) => {
            return (
              <DropdownItem
                active={dataSource.id === selectedDataSource.dataSource?.id}
                value={dataSource.id}
                key={dataSource.id}
                onClick={handleDataSourceSelection}
              >
                {dataSource.getSchema().label}
              </DropdownItem>
            );
          })}
      </DropdownMenu>
    </Dropdown>
  )}
  {selectedDataSource.schema && selectedDataSource.schema.fields && (
    <ConditionController
      fields={Object.keys(selectedDataSource.schema.fields).map(
        (k) => selectedDataSource.schema.fields[k]
      )}
      key={selectedDataSource.dataSource?.id}
    />
  )}
  <JimuMapViewComponent
    useMapWidgetId={props.useMapWidgetIds?.[0]}
    onActiveViewChange={onActiveViewChange}
  />
</div>

);
};`

For performance reasons, the data source is created on demand by the widget in runtime. Could you use the  to create the data source?

I tried using the createDataSource method from the DataSourceManager instance but I get an error saying that the datasource couldn't be created.

  function onActiveViewChange(activeView: JimuMapView) {     if (activeView) {       activeView.whenAllJimuLayerViewLoaded().then((jimuLayerViews) => {         const layerViewDataSourceIds = Object.values(jimuLayerViews).map(           (jimuLayerView) => jimuLayerView.layerDataSourceId         );         layerViewDataSourceIds.forEach((id, i) => {           DataSourceManager.getInstance()             .createDataSource(id)             .then((ds) => {               console.log(ds);             });         });       });     } else {       setDataSources([]);     }   }

The child data sources can't be created directly in the DatasourceManager, can you try the ?

The child data sources can't be created directly in the DatasourceManager, can you try the ?

Can I try what?

I dont want to use the DataSourceComponent because I want to pull all the datasources from the map widget selected. I did manage to get my code working by using jimuLayerView.createLayerDataSource() but I don't see this method documented by esri.
I would like to use the DataSourceManager.getInstance().createDataSource(jimuLayerView.layerDataSourceId) since it's documented but I can't seem to get it to work. Do you know why using this method doesn't work?

jimuLayerView.createLayerDataSource(), you can use this method. We'll add more docs step by step in releases.
The reason why DataSourceManager.getInstance().createDataSource(jimuLayerView.layerDataSourceId) does not work is DataSourceManager can't create child data sources directly, child data sources are created by the root data sources. In this case, by the map data source.
Creating all data sources from a map is not a good practice; creating too many data sources may cause performance issues.

What would be the proper way of creating child data sources? Should I continue to use the jimuLayerView.createLayerDataSource() method? Will the DataSourceManager be capable of creating child data sources in the future?


When installed with cordova-plugin-camera we get error code 65 for command: xcodebuild.

$ cordova plugin add cordova-hot-code-push-local-dev-addon
$ cordova build ios

gives me on macOS Sierra:
CHCP Local Development Add-on:
    Config-file is not set, local-development mode is enabled by default.
    Setting config-file to local server: https://6d322d08.ngrok.io/chcp.json
    iOS bundle version set to 2
Swift support activation hook:
    iOS project now has deployment target set to: 8.0
    iOS project option EMBEDDED_CONTENT_CONTAINS_SWIFT set as: YES
    iOS project Runpath Search Paths set to: @executable_path/Frameworks
    iOS project "Use Legacy Swift Language Version" set to: NO
    Injected swift header My_Project-Swift.h into plugin's main header.
CHCP plugin after prepare hook:
    config-file preference is not set.
Reading build config file:
Building project: /Users/me/Development/my-project/platforms/ios/My Project.xcodeproj
       	Configuration: Debug
       	Platform: emulator
Build settings from command line:
    CONFIGURATION_BUILD_DIR = /Users/me/Development/my-project/platforms/ios/build/emulator
    SDKROOT = iphonesimulator10.0
    SHARED_PRECOMPS_DIR = /Users/me/Development/my-project/platforms/ios/build/sharedpch

Build settings from configuration file '/Users/me/Development/my-project/platforms/ios/cordova/build-debug.xcconfig':
    CLANG_ALLOW_NON_MODULAR_INCLUDES_IN_FRAMEWORK_MODULES = YES
    CODE_SIGN_IDENTITY = iPhone Developer
    ENABLE_BITCODE = NO
    GCC_PREPROCESSOR_DEFINITIONS = DEBUG=1
    HEADER_SEARCH_PATHS = "$(TARGET_BUILD_DIR)/usr/local/lib/include" "$(OBJROOT)/UninstalledProducts/include" "$(OBJROOT)/UninstalledProducts/$(PLATFORM_NAME)/include" "$(BUILT_PRODUCTS_DIR)"
    IPHONEOS_DEPLOYMENT_TARGET = 8.0
    OTHER_LDFLAGS = -ObjC
    PROVISIONING_PROFILE = e1e4c804-c279-4aaa-a069-1cbdf2d51bd4
    SWIFT_OBJC_BRIDGING_HEADER = $(PROJECT_DIR)/$(PROJECT_NAME)/Bridging-Header.h
    TARGETED_DEVICE_FAMILY = 1,2

=== BUILD TARGET CordovaLib OF PROJECT CordovaLib WITH CONFIGURATION Debug ===

Check dependencies

=== BUILD TARGET My Project OF PROJECT My Project WITH CONFIGURATION Debug ===

Check dependencies

Write auxiliary files
write-file /Users/me/Development/my-project/platforms/ios/build/My\ Project.build/Debug-iphonesimulator/My\ Project.build/My\ Project.hmap
write-file /Users/me/Development/my-project/platforms/ios/build/My\ Project.build/Debug-iphonesimulator/My\ Project.build/My\ Project-project-headers.hmap
write-file /Users/me/Development/my-project/platforms/ios/build/My\ Project.build/Debug-iphonesimulator/My\ Project.build/Objects-normal/i386/My\ Project.LinkFileList

PhaseScriptExecution Copy\ www\ directory build/My\ Project.build/Debug-iphonesimulator/My\ Project.build/Script-304B58A110DAC018002A0835.sh
    cd /Users/me/Development/my-project/platforms/ios
    /bin/sh -c \"/Users/me/Development/my-project/platforms/ios/build/My\ Project.build/Debug-iphonesimulator/My\ Project.build/Script-304B58A110DAC018002A0835.sh\"

ProcessInfoPlistFile build/emulator/My\ Project.app/Info.plist My\ Project/My\ Project-Info.plist
    cd /Users/me/Development/my-project/platforms/ios
    export PATH="/Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/usr/bin:/Applications/Xcode.app/Contents/Developer/usr/bin:/Users/me/.nvm/versions/node/v6.6.0/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/MacGPG2/bin:/Users/me/Library/Android/sdk/tools:/Users/me/Library/Android/sdk/platform-tools"
    builtin-infoPlistUtility /Users/me/Development/my-project/platforms/ios/My\ Project/My\ Project-Info.plist -genpkginfo /Users/me/Development/my-project/platforms/ios/build/emulator/My\ Project.app/PkgInfo -expandbuildsettings -format binary -platform iphonesimulator -additionalcontentfile /Users/me/Development/my-project/platforms/ios/build/My\ Project.build/Debug-iphonesimulator/My\ Project.build/MainViewController-PartialInfo.plist -additionalcontentfile /Users/me/Development/my-project/platforms/ios/build/My\ Project.build/Debug-iphonesimulator/My\ Project.build/assetcatalog_generated_info.plist -o /Users/me/Development/my-project/platforms/ios/build/emulator/My\ Project.app/Info.plist
error: couldn't parse contents of '/Users/me/Development/my-project/platforms/ios/My Project/My Project-Info.plist': The data couldn’t be read because it isn’t in the correct format.

** BUILD FAILED **

The following build commands failed:
       	ProcessInfoPlistFile build/emulator/My\ Project.app/Info.plist My\ Project/My\ Project-Info.plist
(1 failure)
Error: Error code 65 for command: xcodebuild with args: -xcconfig,/Users/me/Development/my-project/platforms/ios/cordova/build-debug.xcconfig,-project,My Project.xcodeproj,-target,My Project,-configuration,Debug,-sdk,iphonesimulator,-destination,platform=iOS Simulator,build,CONFIGURATION_BUILD_DIR=/Users/me/Development/my-project/platforms/ios/build/emulator,SHARED_PRECOMPS_DIR=/Users/me/Development/my-project/platforms/ios/build/sharedpch

What version of Xcode you are using?
Try to open platforms/ios/My Project.xcodeproj file in Xcode. It should not ask you to "Convert Swift code to new format", but if it will - don't do that. Run the "build" from it (cmd + B) and see what errors it will display.  Post the screen here, please, if you can.

Xcode Version 8.0 (8A218a
I created a completely new project and it works 😳
I will clean my existing project and try to set it up new. It seems that in my project is something that is cached or something. Thank you for the quick response 👍

Update: I found out that when you add cordova-plugin-camera your cordova-hot-code-push-local-dev-addon will not build anymore. I can use either cordova-plugin-camera or cordova-hot-code-push-local-dev-addon but not both at the same time.

Cool, at least you found what causing the problem. Will check it out. Also, will rename the issue to be more clear.

Any news about that? It seems that not only cordova-plugin-camera fails here. I've got other plugins and the build is failing as well.

Was busy with the main plugin: preparing new release with bug fixes and improvements. So, didn't have time for that one.
What other plugins?

I can't figure it out. I have +10 plugins and need all of them. Without them my App doesn't work respectively doesn't start.

Okay, the problem is that this plugin does some changes in application's .plist, and in some situations it breaks it's format. And for some reason that happens only if other plugins add stuff to .plist as well.

Released v0.4.1 with the fix. Should work now. If not - please, reopen the issue.

Now I get a Error: Cannot find module 'plist-native'

I

Maybe some caching issue, I don't know. Try to reinstall the plugin, cleanup the project.
If it doesn't help - just go to YOUR_PROJECT_NAME/plugins/cordova-hot-code-push-local-dev-addon/ and run npm i, or even npm i plist-native.

npm i:
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/Users/me/.nvm/versions/node/v6.7.0/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:276:23)
gyp ERR! stack     at emitTwo (events.js:106:13)
gyp ERR! stack     at ChildProcess.emit (events.js:191:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:215:12)
gyp ERR! System Darwin 16.0.0
gyp ERR! command "/Users/me/.nvm/versions/node/v6.7.0/bin/node" "/Users/me/.nvm/versions/node/v6.7.0/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /Users/me/Development/frame/plugins/cordova-hot-code-push-local-dev-addon/node_modules/libxmljs
gyp ERR! node -v v6.7.0
gyp ERR! node-gyp -v v3.4.0
gyp ERR! not ok
npm ERR! Darwin 16.0.0
npm ERR! argv "/Users/me/.nvm/versions/node/v6.7.0/bin/node" "/Users/me/.nvm/versions/node/v6.7.0/bin/npm" "i"
npm ERR! node v6.7.0
npm ERR! npm  v3.10.8
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>install: `node-gyp rebuild`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the libxmljs package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs libxmljs
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls libxmljs
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/me/Development/frame/plugins/cordova-hot-code-push-local-dev-addon/npm-debug.log

npm i plist-native: the same

It seems like it is not compatible with your OS for some reason. Okay, will try to use another package...

Please, try now by installing plugin like this:
cordova plugin add https://github.com/nordnet/cordova-hot-code-push-local-dev-addon.git

Thank you. Now it seems to work 👍

Cool, closing then. Uploaded this version on npm.


Microsoft SQL Driver issue: Driver 11

My request is timing out when trying to connect a data loader to my MS SQL server. It seam that I am using " ODBC Driver 11 for my SQL Server ", but mage io does not support this driver.
Steps to reproduce the behavior:
If I run the loader with " ODBC Driver 18 for SQL Server " in the config file I get the error message

and if I run the loader with  " ODBC Driver 11 for SQL Server " in the config file I get the error message

Is there anyone that can assist me with this as I am not sure how to change the ODBC Driver to 11.

Hi @Orionku14 ODBC Driver is installed in the docker image by default. https://github.com/mage-ai/mage-ai/blob/master/Dockerfile#L17
If you want to use ODBC Driver 11, you'll need to build a custom docker image and install this driver. I think the driver should be msodbcsql11

Hi @wangxiaoyou1993 thank you. I will try this fix. Much appreciated.


[website] Add a Connect plug-in page

[x] I have followed (at least) the PR section of the contributing guide.

Pending: the hero section's video.

https://deploy-preview-40515--material-ui.netlify.app/connect
https://deploy-preview-40515--material-ui.netlify.app/material-ui
https://deploy-preview-40515--material-ui.netlify.app/design-kits

Is this PR ready to be merged?

@siriwatknp not yet — we're missing the video we'll insert in the hero section. cc @DavidCnoops for updates on that!


How prepare data

Hi, Could you share how to put file in train/val/test directory? I don't know how to put file in it. Could you provide a example?
Thanks in advance!

Hi, this will be how you can prep your data file.
      data/           train/               class1/                   image1.jpg                   image2.jpg                   ...               class2/                   image1.jpg                   image2.jpg                   ...               ...           val/               class1/                   image1.jpg                   image2.jpg                   ...               class2/                   image1.jpg                   image2.jpg                   ...               ...           test/               class1/                   image1.jpg                   image2.jpg                   ...               class2/                   image1.jpg                   image2.jpg                   ...               ...
If you still need an example, please let me know


Question about the number of runs used

Hi,
Thank you for the very nice repo and congrats for your paper being accepted at ICLR 2023!
I have a question regarding the number of runs you use to generate your results as it is not very clear even in the CoOp paper which you base your experiments on. How many few-shot evaluation runs are used per datapoint?
Thanks in advance!

Hi,
Thanks for your attention to our method.  In the few-shot setting, we follow the protocol in CoOp which runs the experiments three times with seed=1,2,3 for 1,2,4,8,16-shots. Please refer to the scripts (running file) for the details.
Hope it can help.  Please don't hesitate to let me know if you still have questions.

Thank you! That's helpful!
The same seed might generate different results on different machines. I will run your code and check if I get the same result with the same seed.
Best regards,


Fix wrong position for color dialog and sticky popups when styling analysis (+ metrics fix)

Related to https://github.com/CartoDB/support/issues/1649, https://github.com/CartoDB/support/issues/1673 and https://github.com/CartoDB/cartodb/issues/14183
This issue deals with 2 blocks:
A. style issues
B. metric issues
The most important part is the first one (metrics is a little bonus).
A. Styles
Context
There are issues regarding to color picker being placed in a wrong position (#1649 and #1673) and also information popups getting stuck at the same time (#1673).
The situation has been clearly reproduced for #1649, which is a map with an analysis (Add Columns from Second Dataset). For the second case #1673, we didn't have confirmation but it seems a very similar case with another analysis.
Cause
When dealing with analyses layers (not simple layers) an event can be raised, in the interaction with carto.js, and that was causing an unexpected re-render in the style tab, breaking its color dialog (and sometimes its popups). The offending handler was this Line 118 in style-content-view.js:
this.listenTo(this._queryGeometryModel, 'change:status', this.render);

I have checked it and it was added here (before the size & color project): Little refactor improvements,
I have removed it, tests pass, and it seems that it solves our issues, but I honestly can't see the point why it was there in the first place. I made some tests on non-geocoded and empty layers, and it seems they are not affected... @ivanmalagon @elenatorro In what situation could it be useful?
Acceptance A.
Try at staging with https://displaced-dialog.carto-staging.com/

[ ] Test one map with an analysis (and simple) layers, altering its color fill properties several times (fixed mode).
[ ] Repeat testing with by value mode
[ ] Expected: emerging dialog to define color has a correct position and popups disappear.

B. Metrics
Acceptance B.

[ ] Open any map
[ ] Edit the style
[ ] Open the console / network in devtools to see there is no error regarding to metrics

retest this please

retest this please

@ramiroaznar This is ready for acceptance! Steps 🔝

retest this please

LGTM!
I see a google error in the console. But I do not think it is related to the metrics:

Our metrics point to our backend, with urls like: .../api/v3/metrics. This googletagmanager 404 seems unrelated, and it looks like something not available at staging.
So we're gonna merge and deploy this...


create a custom service with UUID

is possible to create a custom service setting the UUID ?
M.

Yes, you can do that by using a custom characteristic/service type. If you just want to associated a uuid with an accessory, you can just use a custom characteristic like this.

func NewUUID(value string) *characteristic.C {
	uuid := characteristic.NewString(CharacteristicTypeUUID)
	uuid.Permissions = []string{characteristic.PermissionRead, characteristic.PermissionHidden}
	uuid.Description = "ID"
	uuid.SetValue(value)
	return uuid.C
}

uuid := NewUUID("1234")
a := accessory.New(accessory.Info{...}, accessory.TypeOutlet)
a.Info.AddC(uuid)

thanks,
great work!
M.


Bug: corrections erase by cache ?

Issue Summary
Hello, we are three journalists using Ghost for our blog. We correct each other papers, but some times our corrections are erase by cache, if another writer don't reload the Content page before clicking the edit button. (sorry, bad english)
Steps to Reproduce

Joe writes an article and goes to Content.

On another computer, Bob opens that article and makes some corrections (save and quit).

Joe doesn't reload Content and just click on Edit. => Bob's corrections are gone

This is a bug because… well, Ajax is great, but if many people edit a post, we gonna lost many corrections.
Technical details

Ghost Version: 0.6.4
Client OS: Mac OS X, Debian Jessie
Server OS: not sure, some Linux I think. it's a simple host on Gandi
Node Version: 0.10
Database: mongoDB

Hi @toblerone1963, thanks for the detailed bug report.
I think that we need to be reloading the post in the edit route? Ember Data has a reload method, I tried it very quickly but it didn't seem to have the desired impact. Still I think it does make sense to implement some sort of force refresh on the edit route.

Clever stuff on how to fix this from @jaswilli: https://ghost.slack.com/archives/ghost/p1434644483003699


3 iterations run only once

Hi Adrian,
Maybe I found an Issue with the iterations I tried to use the 3 iterations with 1 API test, and it run only once.
Version:
Build: 1.0.5 Oct 21, 5:39
Commit: b1fb48aefb8f
Steps to reproduce:

Select one test to run with 3 iterations
Run it
You will see the test will run only once

The attached Logs, db and print screen:
Logs.zip
db.zip

Please investigate this issue.
Thank you.
Best Regards
Gyula Barna

A test will only be executed multiple times if it fails. Fore example, if you run a test session with 2 tests and 3 iterations, if the second test fails at the first iteration but succeeds on the second iteration, you'll end up with:

Test 1 executed only once and passed
Test 2 executed twice and passed

It should be noted that:

Test 2 is still considered as passed, although it failed on the first try.
Iteration 3 is never executed, since all tests passed during the first two iterations.

The "iterations" feature was introduced to alleviate the issue of "flaky" tests. Obviously, you should try to avoid depending on this feature and create stable tests instead, but that's oftentimes easier said than done.

Hi Adrian,
Thanks for your answer, and I understand now why the test run only once.
But sometimes we need to run the test case multiple times to check one test to working properly.
Can you add for example check box which will apply if the test case successfuly ended the session will not end and it will run the test case as many times as it setted up in the iteration numbers?
Thank you
Best regards,
Gyula Barna


Add CLI installation instructions in fish on Linux & macOS.

It creates an autoloading function in fish shell. Sourcing it is not required as fish shell autoloads functions.
Notes:

Using printf for better new-line (\n) support instead of echo. New-lines make the function more human-readable.
Using > as gi.fish should only contain this one function. Hence, appending to the file is not required.

Testing:

✅ Manually tested on macOS Sierra v10.12.3 (16D32) with iTerm2 v3.0.13 with fish shell v2.5.0.
❌ Was not able to test on Linux.

Suggestions for improvement are welcome! Thanks!

Codecov Report

Merging #307 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #307   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          18     18
  Lines         381    381
=====================================
  Hits          381    381

Continue to review full report at Codecov.

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


Scrolling does not work on Firefox (

Please help solve the scrolling problem. I’m on GitHub for the first time, so I think you’ll understand me. I know I need to set layout.css.overflow-overlay.enabled to true in about:config, but I don’t know how to do it. Where do I put it? Please explain to me the points if possible

Go to about:config, search for layout.css.overflow-overlay.enabled, double click "False" to set it to "True".

Go to about:config, search for layout.css.overflow-overlay.enabled, double click "False" to set it to "True".

I did this and updated my Firefox to the latest version. Alas, scrolling still doesn't work.

If you have uBlock Origin you can use: bing.com##.scroller-enabled:style(overflow:auto !important;)
in your filter list :)

Thank you tom-321! My scrolling is fixed now! Also I love the reduced visual clutter :D

Thanks man now working smoothly, just a script change!

Can we add this an extended README?


Empirical workaround for numpy SVD NaN problem from issue 3318

more Voodoo than fix, actual problem probably just papered over by the -mfma option
fixes #3318 (for now)

Please forgive my ignorance about the testing framework, but it is practical add a test to exercise this, perhaps with the original array from #3318 ?

Only if I manage to convert it to C - right now I am still wondering what this is trying to tell me (or if is just a persistent gcc bug)

@martin-frbg, using GCC one would very much prefer to use -march= options instead of -msse2, -mavx and so forth. -mavx for example does not switch on -mfma, you have to use -march=sandybridge. Instead of -mavx2 one should use -march=haswell. The -march= flags controls much more instructions.
This behaviour is different to clang and MSVC. On the other hand with GCC  you have fine control over the instructions set.
I'm able to prepare a PR for that if desired.

I am not convinced as the original issue appeared to have been caused by applying -mfma to "too many" files so I do not yet think an even less fine-grained application of options would help.

Stupid question that probably is already present in testing: is there a way to check that the xmm/ymm registers are being properly saved/restored and wrap the kernel calls with it for testing?

Unfortunately not, so far it has always been retroactive fixing as things blew up with the next smarter release of gcc and/or somebody shouting "oi, you need to save that". (I do believe most such bugs have been fixed in my time here, but I fear I still know just enough assembly to be dangerous. And actually the last change to the Haswell DGEMV microkernel has been to proactively save all xmm/ymm registers instead of just those directly touched by the code - but reverting that had no bearing on the issue)

A wild theory (I have lots of these): maybe it is outside the kernel, and somewhere else someone is not restoring a (different?) register. By adding the flag the registers are used differently and so the problem does not appear. Not sure what would be the easiest way to verify this and how much effort it would be.


DEVPROD-13209: template authenticode key name

This commit templates our authenticode key name in preparation for the authenticode 2021 deprecation. The variable will be added to our Evergreen project as mongo-authenticode-2021, and later updated to 2024 when that certificate is ready.

@fmenezes I think the issue yesterday was that we hadn't yet added the variable to this Evergreen project -- @gssbzn did so this morning and now everything except the code coverage check is passing!


Encountered : Google API error - [429] Quota exceeded

Hi @theoephraim ,
Iam getting the below error. Please help me out with this.
Error: Google API error - [429] Quota exceeded for quota group 'ReadGroup' and limit 'Read requests per user per 100 seconds' of service 'sheets.googleapis.com' for consumer 'project_number:210163281712'.
Thanks.

This issue has nothing to do with this library.
Read the error - youre exceeding your api limit on google. Either reduce the number of calls youre making, or put them into a queue or something so you can rate limit.


Correct link to change log

The link pointed to a non-existent file in a different location,
I presume that it was simply outdated.

[ ] All compiled assets are included (atom packages are git tags and hence the built files need to be a part of the source control)  <-- Can I assume that this isn't relevant for a documentation-only fix?

If this commit doesn't fit the workflow somehow and you want to make your own commit, don't worry about attributing me for this four-byte documentation fix 😄

Thanks.


modify controlplane query

Description
This PR fixes #
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

@vineethvanga18 thanks for iterating on this. There are a couple of build checks failures to look into.

@leecalcote  https://github.com/layer5io/meshery/pull/2492 should be merged first.

@vineethvanga18  can you PR to #2492 instead of master?

I think I don't have permission


Authentication Index Page Revamp

Re: https://docs.blockstack.org/authentication/overview

Replace graphic that doesn't work in Dark Mode
Rename as Authentication Architecture Reference; position as how to roll your authentication, how the auth protocol works under the hood when using ConnectJS
Overview page should be new: Talk about decentralized identity and what it means to us

Note related conversation on Discord: https://discord.com/channels/621759717756370964/623217767356694547/786978554734182440


Can't set the position of the tooltip

I am trying to change the location of the tooltip, but no matter what I put into the boxTopY, and boxLeftX options, the tooltip stays in the same place. Also, want to center the tooltip (regardless of screen resolution) which doesn't seem to be possible with the current config as far as I can tell.
Thanks! :)

Never mind, setting boxLeftX and boxRightX both to -1 causes the tooltip to be centered. And the issue was that you have to restart the MC client for the changes to take effect. Sorry for the ping ^_^


fix: Optimise item grid component to avoid unnecessary re-renders

Fixes DHIS2-6597.
Problem:
Chart items re-render in dashboards app even if props stay the same (triggers by editing text item).
Previous proposal:
Make props deep comparison in Chart plugin.
New proposal:
Use shouldComponentUpdate function in ItemGrid component to determine if it should re-render the grid.
Apply comparison checks to following props:

dashboardItems
edit
isLoading

Another test case: Expand the interpretations section of the item. Without the fix, the chart is also re-rendering there.


Feature request: Validation custom error messages

The AjvValidator package does not accept custom error messages and uses a default of not user friendly messages.
Integrating the Ajv-Errors package should solve this, allowing us to use:
 email: {
          type: "string",
          format: "email",
          maxLength: 254,
          errorMessage: "Invalid email"
},

I've tried it myself to integrate it - but failed miserably.
It should tho not be so hard, based on Ajv-errors documentation:
var Ajv = require('ajv');
var ajv = new Ajv({allErrors: true, jsonPointers: true});
// Ajv options allErrors and jsonPointers are required
require('ajv-errors')(ajv /*, {singleError: true} */);

Edit
My bad, it is possible - had to dig through AjvValidator issues to find a solution since I couldn't find it in the documentation.
This code below works
        email: {
          type: "string",
          format: "email",
          maxLength: 254,
          errorMessage: { format: "Invalid email" }
        },

But it would still be nice to use one error message for all failed validation keywords, since this message only applies to the format keyword.

You can create one BaseModel and use extends on own Models:

then you can use errorMessage inside the models:


"x" button displaying incorrectly

When no file has been uploaded, there is an "invisible" button that is still clickable and displays in the wrong position which can interfere with other styling on the page. The button can be seen by mousing over it or clicking it.
Because of the positioning of this button, the upload button moves down when no file is selected, and up when a file is selected.

Fixed in ad8a9da68aff5d4ccc77b05df0a988c44a0d489d


Added module for sign in with Apple

I can merge this PR if you reopen it :slightly_smiling_face:

Superseded by #135.


Enable scrolling in collections sidebar #118

Loving bruno so far! I just converted our postman files containing 50+ collections and ran into #118 so I thought I'd try my hand at fixing it.

Make sidebar height 100vh
absolute position for collections scrolling container
absolute position for sidebar footer

Thanks for the awesome work! Very much looking forward to diving deeper with bruno! So much nicer than the alternatives.

I also figured out how to get around the rollup error, here's the diff of my changes to packages/bruno-graphql-docs/rollup.config.js
module.exports = [
  {
    input: "src/index.ts",
    output: [
      ...
    ],
    plugins: [
      postcss({
        minimize: true,
+       extract: 'dist/esm/index.css',
        extensions: ['.css']
      }),
      peerDepsExternal(),
      nodeResolve({
        extensions: ['.css']
      }),
      commonjs(),
      typescript({ tsconfig: "./tsconfig.json" }),
      terser()
    ],
    external: ["react", "react-dom", "index.css"]
  },
  {
    input: "dist/esm/index.d.ts",
    output: [{ file: "dist/index.d.ts", format: "esm" }],
-   plugins: [dts.default()],
+   plugins: [dts.default(), postcss()],
  }
];

@jeffprinty Welcome!
Glad that you like Bruno!
Many Thanks for your work on the two issues.

Scrolling in collections sidebar - I had taken a stab at it initially thinking it would be easy, but it was more complex than I thought. Appreciate your work on this.

Rollup error - This has been a long standing error and we kind of have lived with it (lazily) and you've finally resolved it. Thanks!

Looking towards more code contributions as well as issue reports from you :)


Fix error when skipping string with unicode

Codecov Report

Merging #91 into master will decrease coverage by 0.03%.
The diff coverage is 60%.

@@            Coverage Diff            @@
##           master     #91      +/-   ##
=========================================
- Coverage   98.74%   98.7%   -0.04%
=========================================
  Files          36      36
  Lines        5161    5166       +5
=========================================
+ Hits         5096    5099       +3
- Misses         51      52       +1
- Partials       14      15       +1

Impacted Files
Coverage Δ

decode_string.go
98.8% <60%> (-1.2%)
:arrow_down:

Continue to review full report at Codecov.

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


HiResScreenCapture improvements

HiResScreenCapture module (available only for preview builds), works but UI is very ugly (see below), and works differently from every other Slicer module. It also restricts saving only as PNG (not sure why).

Saving the output directly to the scene, and then let the user save whichever format they want from the ExportAs File interface.
alternatively, revise the UI to choose the file path and allow other formats.
It is saving the topbar (blue bar) from the 3D viewer. This is unnecessary.
There is no communication of task being complete beyond the file is generated.

Hi,
I believe adding save type options is possible, I’ll also add an image saved dialog box that displays the file path so the user knows it’s done.
Sent from my iPhone
On Sep 3, 2024, at 4:01 PM, muratmaga @.***> wrote:

HiResScreenCapture module (available only for preview builds), works but UI is very ugly, and works differently from every other Slicer module. It also restricts saving only as PNG (not sure why).

Saving the output directly to the scene, and then let the user save whichever format they want from the ExportAs File interface.
alternatively, revise the UI to choose the file path and allow other formats.
It is saving the topbar (blue bar) from the 3D viewer. This is unnecessary.
There is no communication of task being complete beyond the file is generated.

—
Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https://github.com/SlicerMorph/SlicerMorph/issues/342__;!!DZ3fjg!6879P3nJAgt1jy_nRuV7DDiIb0SQw1oPjr4EZVFkMPJRaMathmGgCD6XMrow_WPbtbTr2eERFVvK-Gf3PlQv3qQsBNMA$, or unsubscribehttps://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AEN5VLIEXA3PJEZTOQ754L3ZUY5R3AVCNFSM6AAAAABNTC6K3OVHI2DSMVQWIX3LMV43ASLTON2WKOZSGUYDGOJWGMYTMMA__;!!DZ3fjg!6879P3nJAgt1jy_nRuV7DDiIb0SQw1oPjr4EZVFkMPJRaMathmGgCD6XMrow_WPbtbTr2eERFVvK-Gf3PlQv3hYQwLZX$.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

Updates to output file types, intial directory, and invalid path error checking here: 78ce89e608a9ee04629ade9d1daf42269613b385

Added TIFF extension to output types:<PHONE_NUMBER>b0f463acb97c63388231bc20568f10


Export the lua functions underlying the commands

Language server
all
Requested feature
The pyright config exports the PyrightOrganizeImports command, which I previously had bound to a key mapping like this:
vim.api.nvim_set_keymap('n', '<leader>i', '<cmd>PyrightOrganizeImports<cr>', {noremap = true})

However, I would like to replace that with vim.keymap.set for which it would be most efficient to directly reference the corresponding lua function:
https://github.com/neovim/nvim-lspconfig/blob/0f94c5fded29c0024254259f3d8a0284bfb507ea/lua/lspconfig/server_configurations/pyright.lua#L19-L25
So that I can write something like
vim.keymap.set('n', '<leader>i', organize_imports)

However, this organize_imports() function is not exported, so I can't do that. My current work-around is this:
vim.keymap.set('n', '<leader>i', function()
  vim.lsp.buf.execute_command({ command = 'pyright.organizeimports', arguments = { vim.uri_from_bufnr(0) } })
end, { buffer=true })

Basically, I just copied the definition of the function into my config file.
Other clients which have this feature
No response

use command and function is same.


Improve how muted streams are displayed on Streams screen

In the subscribed streams UI, we should make the following changes to how muted streams are displayed:

Show the stream's privacy icon, not a special "muted" icon. This will make it possible for the user to tell what the stream's privacy is, and we already convey the fact that the stream is muted with fading.
Sort muted streams at the bottom of their section.

These changes will also make the mobile app UI more consistent with the left sidebar in the webapp.
CZO discussion

Hey @alya @chrisbobbe I have proposed the solution in the CZO discussion. Please review once so that I can raise PR for it.

See also #5425 which overlaps with this.

I want to work on this issue. Can you please assign me this issue?

@TusharMinche The next step on this issue is to address the feedback on #5522. Do you understand how to do that? If so, you could take over finishing that pull request.

Hey @alya @chrisbobbe, I want to start contributing in open source can you assign me some first issues, or provide me some tips so that I can start my journey in open-source.

@MohdSaad Hello! Please take a look at our contributing guide. A good place for questions after that is our development community on Zulip. Let's keep this thread focused on the specific issue that it's about.

@zulipbot claim

@alya
Let me work on this
Please assign

@chrisbobbe
Should I start contributing to the flutter codebase?
https://github.com/zulip/zulip-flutter.

what is the status of this issue??

@samr874 See above. If there were a change in the status since then, there'd have been an update here on the issue thread.


Use graphql-java builtin scalars for CQL scalars

Use GraphQLShort, GraphQLLong, GraphQLByte, GraphQLBigDecimal and GraphQLBigInteger to represent their CQL counterparts.
See also #261 .

Non-standard builtin scalars in graphql-java library are now deprecated: https://github.com/graphql-java/graphql-java/pull/2016
Let's go over them and make sure that the value from/to the user is coerced in the correct data type expected by the db.


Where could I find os_images history like 'OS_IMAGES.md' file

Hi,
I noticed this file 'OS_IMAGES.md' was deleted. But I wanted to know if update Linux kernel.
Where could I find this information?

@edwardstudy we have moved to a more automated approach of bumping stemcells. you can still find history in git on each branch or via tracker stories marked 'label: stemcell'.

@cppforlife OK. How about adding a tag to filter updated CVE base images? I remember OS_IMAGES.md contained this information.

@edwardstudy release notes [1] contain same exact info that we used to put into os_images.md, though per stemcell version. i hope that's good enough.
[1] https://github.com/cloudfoundry/bosh-linux-stemcell-builder/releases


[QUERY] Guide on customization, when polymorphic class be customized to be "abstract"

Query/Question
Case https://github.com/Azure/azure-sdk-for-java/pull/39109#discussion_r1515501782
Service set the superclass to abstract. There the code of "fromJson" will no longer compile.
We need a guide on how to customize such case in azure-core serialization.
Why is this not a Bug or a feature Request?
A clear explanation of why is this not a bug or a feature request?
Setup (please complete the following information if applicable):

OS: [e.g. iOS]
IDE: [e.g. IntelliJ]
Library/Libraries: [e.g. com.azure:azure-core:1.16.0 (groupId:artifactId:version)]

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[x] Query Added
[ ] Setup information Added

@alzimmermsft


react-router-dom -> react-router/dom

I'm seeing a lot of issues come up relating to mismatched versions/instances of the Context pair. This is made more likely because we have two packages (react-router and react-router-dom), one of which is an implicit dependency (i.e., you should be installing only react-router-dom and react-router comes along for the ride).
The most common issue seems to be when someone is using both react-router-dom and a library that depends on just react-router. A version mismatch is handled by npm by giving each package their own copy of react-router, which will result in two Context instances. Right now we only have one version with the new Context API, but expect this to flare up more and more with new versions, especially if we bump the minor or major.
I would like to see about including the react-router-dom contents within the react-router package as a separate importable module. To ease the transition, I would suggest making it available at react-router/dom, so that the change is only one character and can be easily find-and-replace'ed.
This is just a preliminary thought, so I haven't figured out all the edge cases yet. But I wanted to discuss it further before proceeding to write any code. So, is it a great idea or the greatest idea?

I'm in favor.
Libraries aimed at both -dom and -native will be worse off after the transition is complete, but I think that can be justified by the orders of magnitude difference in popularity[1].
[1] 5000 dependents vs. 45 dependents  and 2 million weekly downloads vs. 13k weekly downloads.

We can always keep releasing react-router-dom indefinitely. It just wouldn't be needed anymore.

A version mismatch is handled by npm by giving each package their own copy of react-router, which will result in two Context instances. Right now we only have one version with the new Context API, but expect this to flare up more and more with new versions, especially if we bump the minor or major.

I don't understand how a react-router/dom bundle would help this problem (though I really like the idea of ditching a monorepo and getting back to publishing a single package). Wouldn't you still have 2 context instances if one of your deps is using the router as well?

I don't think that you could fully ditch the monorepo; react-router-native would add react-native as a dependency to every application (unless the package was abandoned at which point there wouldn't be a need for a -dom qualifier).

I don't think we should do react-router/dom, the main reason being that I do not want people using react-router-native to get react-dom as a peer dep. We don't peer-depend on react-dom in react-router-dom currently, but we could in the future.
The root of the problem is people depending on react-router and not react-router-dom or react-router-native. People should never depend on react-router or install it directly. It's just a dependency of react-router-dom and react-router-native.
This is mostly a problem that we (probably mostly me?) have created by telling people to install react-router and import stuff directly from react-router in our docs and examples. There is a helpful note in the react-router README about this, but we still do have quite a few docs telling people to import stuff from react-router directly. We need to get rid of those and make it a lot more clear that our expectation is for people to never depend directly on react-router.
I'll comb through the docs today and sniff out all the faulty references. Also, I'll probably try to move all the docs to the root of the repo to make them a little more prominent and let people discover them more easily.

This is something we deal with on react-redux, where we support both dom and native, proxying the batchedUpdates API from each as a batch function publicly.
The good news is there is a fix coming, the bad news is only recent versions of npm support it: https://github.com/reduxjs/react-redux/pull/1390

You could just make both the native and dom packages vendor whats currently react-router, instead of depending on it. (I don't know if you'd actually need the react-router` package at that point or not. Does react have popular other targets?)
That would get rid of the issue entirely, but it has some mean downsides. I'm mostly suggesting it for completeness sake.

You'd definitely end up with mismatched context instances in that case.

Not a maintainer and I don't have huge expertise at all here, so take with a pinch of salt.
If my understanding (after a painful 7 hours understanding this and filing this issue) is correct, this is much more of a communications issue than a technical one.
The entire web is scattered with examples where react-router is installed directly, ignoring react-router-dom, which must seriously obfuscate this issue. In a fairly aggressive google, here is the only reference I can get to a member of the RR team actually saying we should be using RRD instead of RR almost all the time (I assume the same is true of RRN for native uses?).
Even major libraries like connected-react-router encourage importing directly from RR in their examples.
So just thinking outside the box, would

an afternoon of finding and correcting / filing issues with the most readily available tutorials / related libraries on the web, and
A docs section for library developers suggesting use of only RRD or RRN, and that they be installed as external/peer dependencies

go most of the way to fixing this?

An alternative solution would be to put the Context initialisation in a separate module react-router-context. react-router could then depend on react-router-context and import the Context from there.
Regardless of whether other modules depend on react-router or react-router-dom, or what versions, there'll only be one react-router-context version, and so it'll get hoisted to the top of the dependency tree => no mismatches.
A bit hacky, but would solve the problem with no breaking changes.

NB I know this issue has been closed, but it's still listed on the Roadmap issue #6885 so I thought it worth raising.

I need to update the public roadmap. Thanks for the reminder, @overlookmotel.


Failing e2e tests - scorecard-head on ossf-tests/scorecard-action

Matrix: null
Repo: https://github.com/ossf-tests/scorecard-action/tree/main
Run: https://github.com/ossf-tests/scorecard-action/actions/runs/4933717305
Workflow name: scorecard-head
Workflow file: https://github.com/ossf-tests/scorecard-action/tree/main/.github/workflows/scorecards-head.yml
Trigger: push
Branch: main

This is a consistent failure whenpublish_results is true. Introduced by #1140.


APPSRE-8634 vanilla tf image

For our dynatrace-config CI we need a simple vanilla tf container. Will be enhanced with provider plugin caching in a later iteration.

Why not directly use the official image? https://hub.docker.com/r/hashicorp/terraform

@hemslo My understanding is that we should use ubi images as base for internal workloads.

Then what you need is just to copy terraform cli to ubi images, right? Can directly copy from their official image:
ARG TF_VERSION=1.4.6
FROM hashicorp/terraform:${TF_VERSION} as terraform

FROM registry.access.redhat.com/ubi9/ubi-minimal:9.2
COPY --from=terraform /bin/terraform /usr/local/bin/terraform

I could - but I also intend to extend this in follow-up MRs, like caching providers in that downloader part similar to how we do it in our qr image. I feel like staying within our common base image is ok in that case.

However, thinking about it one thing that is definitely missing here is checksum/sig check. Will add soon :)

Use multi stage build and copy from docker images is faster than curl, it can use docker cache, concurrent download, requires less tools and safer.

It also requires more to download, which kind of cancels the concurrent download aspect.
Safer Im also not so sure. Why is it safer than checksum/sig check?
less tools - true, but thats only curl and gpg, which is neglectable I feel.

The docker cache is a good point though - didnt think about it. Though I dont think that its a big factor in this case, as this image wont be built often enough on same agent to profit from this. But in general its a good aspect.

I guess essentially my mind is at reducing external docker dependencies. I feel like its easier long-term to rely on internal base images as much as possible.

Copy from image makes dependencies explicit, it's a container native way of distribution. Use curl, unzip, gpg just put bash scripts into Dockerfile, not really in a container way.
Images are downloaded from registry, usually from CDN or local mirror, that makes it usually faster than other sources, especially in network restricted env, such as only allow traffic to registry/mirror.
It's safer because less traffic destinations, less binaries used, why use more when all needed features are built in already? If plan to include things not available from other container images, then it makes sense to curl them. Otherwise stick to images to simplify Dockerfile like this, produce the equivalent image with less code.
ARG TF_VERSION=1.4.6
FROM hashicorp/terraform:${TF_VERSION} as terraform

FROM registry.access.redhat.com/ubi9/ubi-minimal:9.2
COPY --chown=0:0 --from=terraform /bin/terraform /usr/local/bin

WORKDIR /terraform

If registry mirror is needed, then it's like this
ARG TF_REPO=hashicorp/terraform
ARG TF_VERSION=1.4.6
FROM ${TF_REPO}:${TF_VERSION} as terraform

FROM registry.access.redhat.com/ubi9/ubi-minimal:9.2
COPY --chown=0:0 --from=terraform /bin/terraform /usr/local/bin

WORKDIR /terraform

TF_REPO can point to different registries.

Thing is I will use curl in the downloader anyways to cache the plugins in follow-up MR.
Concerning safety because of less traffic destinations/less binaries: I need curl, unzip and gpg (I will need then anyways in follow-up step, regardless of which image Im using now). Curl and gpg are already part of the base image - the only thing that is extra is unzip. I.e., even if I use the hashicorp image, I will need to install them in some stage anyways. So Im not sure the security aspect applies in this case.
Also I dont mind having some extra lines in Dockerfile or extra sig file. If it removes an external dep, then Im good with that.
That being said, you raise a good point with the mirror: My concern was purely removing external docker images - but what if we do not have a network path to the external binary either? The mirror is a more robust approach here. Will set it up for tf (we probably already have it somwhere :) ) :+1:

Just found this https://developer.hashicorp.com/terraform/cli/commands/providers/mirror
Can avoid curl etc all along

Do you have an example of later usage of curl in the follow up steps? I prefer docker image dep than download from url in Dockerfile for explicit dependency control. And from your new findings of terraform providers mirror, I think you have a new way too?

The problem with terraform mirror is that it can only use a single version per provider. I we want to prebake multiple provider versions, it would be easier to go with curl https://github.com/app-sre/container-images/blob/master/qontract-reconcile-base/Dockerfile#L42
For now I dont need multiple provider versions, so it is ok.
Added the changes to use the mirror for now.


Feature/353 pre run callback in setup project

#353 -> Adds a pre run callback feature to setup_project, with some basic args exposed for users who want to use it without storing local config options.

On our project, we have a script that needs to run with the wanted flavor passed as an argument. This needs to happen before running the app.

As an aside, do you want me to rename those commits to be conventional commit standard?

As an aside, do you want me to rename those commits to be conventional commit standard?

Yes, then we would have a nice changelog.


change regex format

Added 3 lines of code for package installation.
I changed also the rule of datetime_pat and sender_pat to adapt to the current Whatsapp import chat format.

Thanks - i just recently updated the notebook - should be fixed now - thanks!


Time Series Forecasting with LSTM and different/mixed frequencies

I could not find anything in the docs about how to handle different frequencies of time series. I have a Dataset A with monthly data that i want to use to predict the values from Dataset B that contains quarterly based data. So the target value e.g. quarter 1 is based on the values from month 1-3.
Dataset A (Features):

Month
Value1
Value2
Value3

1
91
72
23

2
93
68
33

3
92
57
27

4
91
72
23

5
93
68
33

6
92
57
27

Dataset B (Target):

Quarter
Target_Value

1
51

2
57

@janwendt ,
Hi,This question is better asked on StackOverflow since it is not a bug or feature request. There is also a larger community that reads questions there.
You can also refer this link related to LSTM time series with mixed frequency data and also article.Thanks!

@janwendt ,
Any update on the issue ?Thanks!

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


[TASK] Tweak the "manual transfer" section of the Readme

Fixes the numbering and removes the need to actually download patches to a file.

Thanks a lot Karsten for the improvements and examples!


Stable version needed for this repo instead of beta version

Can we get a Stable version for angular-google-maps ?
I see there wasn't a Stable version since 0.17 more than an year ago. If we wan't to use this library professionally, we need a Stable version instead of beta version.
So, wondering if we can get a Stable version.

Wondering if we can get a Stable version of angular google maps. I appreciate your help.


chore: pull request template

Can we add a line in there for release note description?
Something along the lines of

If this is a user-facing feature or may the overall Dendron experience, please summarize the feature or impact in 1-2 lines for the changelog release notes. If this builds upon a previous PR covering the same area, please link to that PR instead.

made some updates


Guidance for unit testing when FromSql<> is used.

We have unit tests that use the InMemory provider.
However, a few of our objects under test are calling the FromSql() method to materialise objects from stroed procedures.
When using the InMemoryProvider this obviously doesn't work.
One solution i can think of is for the object under test to use the Repository pattern - so we can swap EF7 out entirely at test time, for an InMemory repository. However I think it would be good if the InMemory provider had a way to handle FromSql at test time, so you could map an abitrary set of results when needed?

The InMemory Provider is not a relational provider, use SQLite InMemory or localdb

SqlLite sounds like a good shout - thanks 👍

Although worth mentioning, the reason we use InMemoryProvider during testing, is because we want to eliminate the need for having an external database asset to manage. This means we can run tests on any machine quite easily, with minimal setup.
Using sqllite provider isn't an option if you call an SP, or use a different variant of SQL language. (i.e postgres)
In the situation where you are executing raw sql, you are basically bypassing the ORM's language translation capabilities, and just using it as a mapper that can execute via ADO and

@dazinator yep using InMemory or SQLite in-memory mode are all an approximation for running against a real database. In general, SQLite in-memory is a decent approximation for another relational database, but it's never going to be identical. Even if it supports the same features, there is no guarantee it will behave exactly the same as the database provider your application ultimately runs against.
Where you are using something like raw sql to stored procedures, you'll need to either run against the database that supports them, or introduce a layer of abstraction so that you can create a test double for testing.

or introduce a layer of abstraction so that you can create a test double for testing.
Yup thats what we did. Have made a note: don't call FromSql<> directly!

Yup thats what we did. Have made a note: don't call FromSql<> directly!

@dazinator would you mind elaborating a bit on how you did that? I'm in the same boat, and I'd rather steal from you than reinvent the wheel :-)

Thanks for responding.  I did just what you suggested as soon as I deleted the comment.
Thanks again!