Page or Component ? Missing doc ?

From @movrack on August 17, 2016 11:45
Hi,
Some times I saw page defined in doc with ´´´@Component´´´ and sometimes with ´´´@Page´´´.
What is the difference ?
Which should be used ?
Where is the doc who explain that?
Regards,
Copied from original issue: driftyco/ionic#7750

Hello @movrack ! Thanks for opening an issue with us! Are you seeing @page in our documentation right now? Or are you saying you have seen it in the past. @page was deprecated a good while ago and should definitely not be in the docs.

From @dgroh on August 17, 2016 11:52
@Page is deprecated. And why this should be an issue? Ionic forum should be the place for this kind of question and here is the answer: https://forum.ionicframework.com/t/difference-between-page-and-component/55287

fixed with https://github.com/driftyco/ionic/commit/6fb848cc4ae727cd0d9eb338efeb0ec73fe222e7


Change OnPause and OnResume code

OnPause and OnResume need to be protected override. If it's public, it is not running in the android app

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


help broker load 文档错误

Search before asking

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

Version
0.15.0
What's Wrong?
help broker load 配置带ha的HDFS集群时，18."hdfs_user"="user"后面少写一个逗号

What You Expected?
添加上这个逗号
How to Reproduce?
No response
Anything Else?
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

Hi, If you are willing to fix this problem, you can submit a PR.

What's more, Could you replace the title with English?


Admin pages + specs

Add admin pages to application
Create separate ts component + specs.
Fixes #30

Please someone could review this one, as it's blocking e2e tests work in progress, entity pagination, etc.
I've almost finished protractor tests but I'm working on this branch right now, I would feel more comfortable if this is merged.

Overall looks ok to me. I don't have time for detailed review now. If you are confident merge it as we are in WIP anyway

Same comment than Deepu.
Don't hesitate to merge directly @sahbi-ktifa

Great! Thanks guys, let's move on!


always need load model

once i begin record，it always needs loading model and there always be “An error occurred while loading the Whispermodel: .Please retry or check the application logsfor more information.”

Please try the latest version https://github.com/chidiwilliams/buzz/releases


Add optional optimizations to ignore messages from self and to have a…

… batch of messages make a single callback.

Refactored to not use the proposed (and abandoned) redis.Ping() function.


[bug][2.0.0-beta.0] PluginInitialization("http", "invalid type: map, expected unit")

Describe the bug
"plugins": {
    "http": {
      "scope": [
        "http://localhost:3000/*"
      ]
    }
  }

Error reported after adding scope configuration to http plugin
Reproduction
No response
Expected behavior
no error
Full tauri info output
[✔] Environment
    - OS: Mac OS 14.3.0 X64
    ✔ Xcode Command Line Tools: installed
    ✔ rustc: 1.75.0 (82e1608df 2023-12-21)
    ✔ cargo: 1.75.0 (1d8b05cdd 2023-11-20)
    ✔ rustup: 1.26.0 (5af9b9484 2023-04-05)
    ✔ Rust toolchain: stable-aarch64-apple-darwin (environment override by RUSTUP_TOOLCHAIN)
    - node: 20.11.0
    - pnpm: 8.14.1
    - yarn: 1.22.21
    - npm: 10.2.4

[-] Packages
    - tauri [RUST]: 2.0.0-beta.2
    - tauri-build [RUST]: 2.0.0-beta.1
    - wry [RUST]: 0.35.2
    - tao [RUST]: 0.25.0
    - tauri-cli [RUST]: 2.0.0-beta.0
    - @tauri-apps/api [NPM]: 2.0.0-beta.0
    - @tauri-apps/cli [NPM]: 2.0.0-beta.0

[-] App
    - build-type: bundle
    - CSP: unset
    - frontendDist: ../dist
    - devUrl: http://localhost:5173/
    - framework: React
    - bundler: Vite

Stack trace
No response
Additional context
No response

cross linking the plugin issue: https://github.com/tauri-apps/plugins-workspace/issues/928#issuecomment-1925719547

The old allowlist and scope configs were replaced with a permissions system: https://beta.tauri.app/guides/upgrade-migrate/from-tauri-1/#migrate-permissions. Please read through it and get back to us if you have further questions.

cross linking the plugin issue: tauri-apps/plugins-workspace#928 (comment)

The old allowlist and scope configs were replaced with a permissions system: https://beta.tauri.app/guides/upgrade-migrate/from-tauri-1/#migrate-permissions. Please read through it and get back to us if you have further questions.

The link is dead.

https://tauri.app/start/migrate/from-tauri-1/#migrate-permissions


Data not refreshing when changing

Hello , I am using nextjs latest version . when I change my data from live website no changes occur but on the other hand when I change data from local host the changes can be seen . I have also tried revalidation:1 but no effect . am I missing some thing plz guide.

I was facing the same issue. I downgraded the next sanity to version "next-sanity": "^4.1.5",
Now it's working fine. Hoping the bug in this version will be solved soon.

If you click "Purge Everything" on your Vercel Project Settings, does the changes show up afterwards?

It will then decrease the performance of my project as there will be no
backward and forward caching.
On Wed, Sep 13, 2023, 2:36 PM Cody Olsen @.***> wrote:

If you click "Purge Everything" on your Vercel Project Settings, does the
changes show up afterwards?
[image: image]
https://user-images.githubusercontent.com/81981/267607420-97a0facb-4a3e-4d9f-b99d-8b0280c12a7d.png
—
Reply to this email directly, view it on GitHub
https://github.com/sanity-io/next-sanity/issues/591#issuecomment-1717288044,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQW4T4KARR65RSJS4Z3NT5TX2F5CZANCNFSM6AAAAAA4E3JIDI
.
You are receiving this because you authored the thread.Message ID:
@.***>

@abdulhadi240 it doesn't disable caching completely, it just clears it and will fill up again. If you "Purge Everything" and see updated content, it means that things are working on our end. You'll need to open an issue at Vercel who can help you track down if this is due to a configuration in your app, or a bug in Next.js itself.


[v2] upload coverage

Codecov Report

:exclamation: No coverage uploaded for pull request base (2.x@8e8ff75). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head ec2d23a differs from pull request most recent head 9fcfb14. Consider uploading reports for the commit 9fcfb14 to get more accurate results

@@         Coverage Diff          @@
##             2.x   #657   +/-   ##
====================================
  Coverage       ?      0
====================================
  Files          ?      0
  Lines          ?      0
  Branches       ?      0
====================================
  Hits           ?      0
  Misses         ?      0
  Partials       ?      0

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 8e8ff75...9fcfb14. Read the comment docs.


UI: Pop up detail should not cover the Yes / No button

I prefer the pop up detail to be moved down, not to cover the YES / NO button.

I agree with @wanghong230, but want to add that I don't think pop-up context needs to be moved down only during promotion when it obscures the yes/no buttons. imho, it should always be shown under the freightline. Here is an example of me not actively promoting, but still having important details (namely the contents of the freight) obscured by the pop-up:


exact_balance specified in proxy metadata does not get applied to listeners

Bug Description
Possible regression for https://github.com/istio/istio/issues/18152
When using the following annotation on a deployment pod spec:
      annotations:
        proxy.istio.io/config: |
          proxyMetadata:
            INBOUND_LISTENER_EXACT_BALANCE: "true"
            OUTBOUND_LISTENER_EXACT_BALANCE: "true"

I do not see the listener config being applied
connection_balance_config:
  exact_balance: {}

Version
1.14+

Additional Information
No response

@howardjohn I was able to see the info was being sent as part of the node/xds metadata, but pilot is not propagating that to the config generation. Maybe only envs that start with ISTIO_ are respected, thoughts?
See https://github.com/istio/istio/pull/36834

yes, need ISTIO_META_ prefix


[PUBSUB_MODEL]Offset and partition meta data should expose to the user

Offset and partition meta data should expose to the user.

LGTM


AMBARI-24877 Heatmap tab, metrics tab and QuickLinks section in summary tab should be hidden if service has only client service component

How was this patch tested?
21960 passing (30s)
48 pending

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/4517/
Test PASSed.


Results.subscribe

Adds an experimental async Results.subscribe and .unsubscribe in Swift. And adds [RLMResults subscribeWithName:waitForSyncMode:onQueue:timeout:completion] and unsubscribe in Obj-c.
public func subscribe(name: String? = nil, waitForSync: WaitForSyncMode = .onCreation, timeout: TimeInterval? = nil)
 async throws -> Results<Element>

public func unsubscribe()

- (void)subscribeWithName:(NSString *_Nullable)name
          waitForSyncMode:(RLMWaitForSyncMode)waitForSyncMode
                  onQueue:(dispatch_queue_t _Nullable)queue
                  timeout:(NSTimeInterval)timeout
               completion:(RLMResultsCompletionBlock)completion

- (void)unsubscribe

Results.subscribe will create a SyncSubscription with a query which matches the local filter on the Results at the time subscribe is called. The API is an attempt to ease overhead maintaining distinct local queries made on an Results and server queries created via SubscriptionSet.update.
The subscription API opens a write transaction to commit subscription updates. Opening a write transaction and diffing the subscription against the server is not an unsubstanial workload. Therefore using the subscription API for large batch subscriptions is discouraged.
Using both the current API and this experimental API is entirely possible and encouraged. The current subscription API is well-suited for creating batch subscriptions for commonly shared data early in the application setup.
This can be combined with dynamic changes via the experimental subscription API to handle user input.
This change also adds an unnamedOnly bool flag to SyncSubscriptionSet.removeAll. And adds [RLMSyncSubscriptionSet removeAllUnnamedSubscriptions]. These can be used for periodic clean up of unnamed subscriptions that could accrue during app usage.

Needs changelog entry.
Need to figure out what's needed for beta release. Similar to something like this release.
From what I can tell there aren't any macros I need to add? Just changes in the release pipeline?

This is still missing some CHANGELOG entry so users testing this can check the added APIs without going directly to the APIs docstrings

@tgoyne I implemented a RLMAsyncSubscriptionTask which handles the timeout and the waitForSubscription asynchronous calls, and has a mutex which would prevent it from running from different threads. Let me know if this what is asked in the above comments.

@leemaguire after adding a test where the calling thread is different than the callback thread some issues with threading came out, the only way to resolve it without changing too much of the implementation is forcing the user to use the MainActor to do the results subscription. I propose releasing this preview API as it is, and check if we any user feedback regarding this part, we can then get deeper into support any actor for this call.


How to override default server name provided by Pushy server to the client?

I've created the same issue under https://github.com/chef-cookbooks/push-jobs/issues/47 but I thought it might be more appropriate here. Here is the scoop:
I set up Pushy server on my Chef 12 server and installed Pushy client on one of the Windows machines (both hosted in AWS). When started the Pushy client successfully connects to the Chef server via HTTPS but then it tries to connect to the same server (via TCP) using a completely different FQDN (see the second the third log records below). I want to make Pushy client connect to the server using the same FQDN i.e. tcp://chef.myorg.com:10002 and tcp://chef.myorg.com:10000.

[2015-09-17T18:01:03+00:00] INFO: [i-77a2f5dc] Retrieving configuration from https://chef.myorg.com/organizations/myorg/pushy/config/i-77a2f5dc ...
[2015-09-17T18:01:03+00:00] INFO: [i-77a2f5dc] Connecting to command channel at tcp://ip-10-5-5-41.ec2.internal:10002
[2015-09-17T18:01:03+00:00] INFO: [i-77a2f5dc] Listening for server heartbeat at tcp://ip-10-5-5-41.ec2.internal:10000

You can override this with zeromq_listen_address and can find information https://docs.chef.io/config_rb_push_jobs_server.html


Update devices.js

Added initial support for Danalock V3 smartlock.
Not tested with zigbee2mqtt. But is working on iobroker.zigbee.

Thanks!


[Bug]: Corrupted output with GPTQ Marlin kernel

Your current environment

The output of `python collect_env.py`
Collecting environment information...
PyTorch version: 2.5.1
Is debug build: False
CUDA used to build PyTorch: 12.4
ROCM used to build PyTorch: N/A

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

Python version: 3.12.7 | packaged by Anaconda, Inc. | (main, Oct  4 2024, 13:27:36) [GCC 11.2.0] (64-bit runtime)
Python platform: Linux-6.8.0-1020-aws-x86_64-with-glibc2.35
Is CUDA available: True
CUDA runtime version: 12.4.99
CUDA_MODULE_LOADING set to: LAZY
GPU models and configuration: GPU 0: NVIDIA A10G
Nvidia driver version: 550.54.14
cuDNN version: Could not collect
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True

CPU:
Architecture:                         x86_64
CPU op-mode(s):                       32-bit, 64-bit
Address sizes:                        48 bits physical, 48 bits virtual
Byte Order:                           Little Endian
CPU(s):                               4
On-line CPU(s) list:                  0-3
Vendor ID:                            AuthenticAMD
Model name:                           AMD EPYC 7R32
CPU family:                           23
Model:                                49
Thread(s) per core:                   2
Core(s) per socket:                   2
Socket(s):                            1
Stepping:                             0
BogoMIPS:                             5599.99
Flags:                                fpu vme de pse tsc msr pae mce cx8 apic sep mtrr pge mca cmov pat pse36 clflush mmx fxsr sse sse2 ht syscall nx mmxext fxsr_opt pdpe1gb rdtscp lm constant_tsc rep_good nopl nonstop_tsc cpuid extd_apicid aperfmperf tsc_known_freq pni pclmulqdq ssse3 fma cx16 sse4_1 sse4_2 movbe popcnt aes xsave avx f16c rdrand hypervisor lahf_lm cmp_legacy cr8_legacy abm sse4a misalignsse 3dnowprefetch topoext ssbd ibrs ibpb stibp vmmcall fsgsbase bmi1 avx2 smep bmi2 rdseed adx smap clflushopt clwb sha_ni xsaveopt xsavec xgetbv1 clzero xsaveerptr rdpru wbnoinvd arat npt nrip_save rdpid
Hypervisor vendor:                    KVM
Virtualization type:                  full
L1d cache:                            64 KiB (2 instances)
L1i cache:                            64 KiB (2 instances)
L2 cache:                             1 MiB (2 instances)
L3 cache:                             8 MiB (1 instance)
NUMA node(s):                         1
NUMA node0 CPU(s):                    0-3
Vulnerability Gather data sampling:   Not affected
Vulnerability Itlb multihit:          Not affected
Vulnerability L1tf:                   Not affected
Vulnerability Mds:                    Not affected
Vulnerability Meltdown:               Not affected
Vulnerability Mmio stale data:        Not affected
Vulnerability Reg file data sampling: Not affected
Vulnerability Retbleed:               Mitigation; untrained return thunk; SMT enabled with STIBP protection
Vulnerability Spec rstack overflow:   Vulnerable: Safe RET, no microcode
Vulnerability Spec store bypass:      Mitigation; Speculative Store Bypass disabled via prctl
Vulnerability Spectre v1:             Mitigation; usercopy/swapgs barriers and __user pointer sanitization
Vulnerability Spectre v2:             Mitigation; Retpolines; IBPB conditional; STIBP always-on; RSB filling; PBRSB-eIBRS Not affected; BHI Not affected
Vulnerability Srbds:                  Not affected
Vulnerability Tsx async abort:        Not affected

Versions of relevant libraries:
[pip3] numpy==1.26.4
[pip3] nvidia-ml-py==12.560.30
[pip3] pyzmq==26.2.0
[pip3] torch==2.5.1
[pip3] torchaudio==2.5.1
[pip3] torchvision==0.20.1
[pip3] transformers==4.47.0
[pip3] triton==3.1.0
[conda] blas                      1.0                         mkl
[conda] cuda-cudart               12.4.127                      0    nvidia
[conda] cuda-cupti                12.4.127                      0    nvidia
[conda] cuda-libraries            12.4.1                        0    nvidia
[conda] cuda-nvrtc                12.4.127                      0    nvidia
[conda] cuda-nvtx                 12.4.127                      0    nvidia
[conda] cuda-opencl               12.6.77                       0    nvidia
[conda] cuda-runtime              12.4.1                        0    nvidia
[conda] cuda-version              12.6                          3    nvidia
[conda] ffmpeg                    4.3                  hf484d3e_0    pytorch
[conda] libcublas                 <IP_ADDRESS>                      0    nvidia
[conda] libcufft                  <IP_ADDRESS>                      0    nvidia
[conda] libcufile                 <IP_ADDRESS>                      0    nvidia
[conda] libcurand                 <IP_ADDRESS>                     0    nvidia
[conda] libcusolver               <IP_ADDRESS>                      0    nvidia
[conda] libcusparse               <IP_ADDRESS>                    0    nvidia
[conda] libjpeg-turbo             2.0.0                h9bf148f_0    pytorch
[conda] libnpp                    <IP_ADDRESS>                     0    nvidia
[conda] libnvfatbin               12.6.77                       0    nvidia
[conda] libnvjitlink              12.4.127                      0    nvidia
[conda] libnvjpeg                 <IP_ADDRESS>                    0    nvidia
[conda] mkl                       2023.1.0         h213fc3f_46344
[conda] mkl-service               2.4.0           py312h5eee18b_1
[conda] mkl_fft                   1.3.11          py312h5eee18b_0
[conda] mkl_random                1.2.8           py312h526ad5a_0
[conda] numpy                     1.26.4                   pypi_0    pypi
[conda] nvidia-ml-py              12.560.30                pypi_0    pypi
[conda] pytorch                   2.5.1           py3.12_cuda12.4_cudnn9.1.0_0    pytorch
[conda] pytorch-cuda              12.4                 hc786d27_7    pytorch
[conda] pytorch-mutex             1.0                        cuda    pytorch
[conda] pyzmq                     26.2.0                   pypi_0    pypi
[conda] torchaudio                2.5.1               py312_cu124    pytorch
[conda] torchtriton               3.1.0                     py312    pytorch
[conda] torchvision               0.20.1              py312_cu124    pytorch
[conda] transformers              4.47.0                   pypi_0    pypi
ROCM Version: Could not collect
Neuron SDK Version: N/A
vLLM Version: 0.6.4.post2.dev369+g88693683
vLLM Build Flags:
CUDA Archs: Not Set; ROCm: Disabled; Neuron: Disabled
GPU Topology:
GPU0	CPU Affinity	NUMA Affinity	GPU NUMA ID
GPU0	 X 	0-3	0		N/A

Legend:

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

LD_LIBRARY_PATH=/home/ubuntu/miniconda3/envs/optimum-quanto/lib/python3.12/site-packages/cv2/../../lib64:/usr/local/cuda-12.4/lib64
CUDA_MODULE_LOADING=LAZY

Model Input Dumps
No response
🐛 Describe the bug
@dacorvo discovered a potential issue (https://github.com/huggingface/optimum-quanto/issues/332) with the GPTQ Marlin kernel where the outputs become corrupted for certain shape combinations. The corrupted values appear at different indices after each invocation.
I took some time to investigate the issue, and found two suspicious race conditions in the kernel with Compute Sanitizer:
========= Error: Race reported between Read access at void marlin::Marlin<__half, (long)1125899907892224, (int)256, (int)4, (int)16, (int)4, (int)4, (bool)0, (bool)0, (int)8, (bool)0>(const int4 *, const int4 *, int4 *, int4 *, const int4 *, const int4 *, const int *, int, int, int, int, int *, bool)::[lambda(int, int) (instance 2)]::operator ()(int, int) const+0x2df0 in [REDACTED]/vllm/csrc/quantization/gptq_marlin/gptq_marlin.cu:1044
=========     and Write access at marlin::cp_async4(void *, const void *)+0x3380 in [REDACTED]/vllm/csrc/quantization/gptq_marlin/marlin.cuh:73 [1830656 hazards]
=========
========= Error: Race reported between Read access at void marlin::Marlin<__half, (long)1125899907892224, (int)256, (int)4, (int)16, (int)4, (int)4, (bool)0, (bool)0, (int)8, (bool)0>(const int4 *, const int4 *, int4 *, int4 *, const int4 *, const int4 *, const int *, int, int, int, int, int *, bool)::[lambda(int, int) (instance 2)]::operator ()(int, int) const+0x4a10 in [REDACTED]/vllm/csrc/quantization/gptq_marlin/gptq_marlin.cu:1044
=========     and Write access at marlin::cp_async4(void *, const void *)+0x4f30 in [REDACTED]/vllm/csrc/quantization/gptq_marlin/marlin.cuh:73 [1740800 hazards]
=========
========= Error: Race reported between Read access at void marlin::ldsm4<__half>(marlin::ScalarType<T1>::FragA &, const void *)+0x5930 in [REDACTED]/vllm/csrc/quantization/gptq_marlin/gptq_marlin.cu:131
=========     and Write access at void marlin::Marlin<__half, (long)1125899907892224, (int)256, (int)4, (int)16, (int)4, (int)4, (bool)0, (bool)0, (int)8, (bool)0>(const int4 *, const int4 *, int4 *, int4 *, const int4 *, const int4 *, const int *, int, int, int, int, int *, bool)::[lambda() (instance 5)]::operator ()() const+0x5a80 in [REDACTED]/vllm/csrc/quantization/gptq_marlin/gptq_marlin.cu:1351 [59904 hazards]
=========

I believe these can be related to the issue as the other Marlin kernels work for the same shapes and don't trigger race condition errors.
Here's a small reproducer, I tested it on AWS with A10G GPUs:
https://github.com/ahadnagy/vllm/blob/d2d7def73a8e9c3843b32fdc9adc8e71605b397c/tests/kernels/test_marlin_gemm.py#L619-L686
Also, I did some parameter space exploration to see at which shapes this error starts to manifest itself:

Any suggestions are welcome as it's a really hard-to-debug issue, and can't really go further without deep-diving into the kernel.
Before submitting a new issue...

[X] Make sure you already searched for relevant issues, and asked the chatbot living at the bottom right corner of the documentation page, which can answer lots of frequently asked questions.

Thanks for reporting. Are there any specific models effected that you are aware of?

The issue was first discovered on gemma-2b because the perplexity increased.


Adding external fonts in production builds fails

Hi guys, I think the documentation on how to add external fonts to the seed is outdated. I did the following to add the font (basically the same as mentioned in the documentation):
In tools/config/project.config.ts:
export class ProjectConfig extends SeedConfig {

  PROJECT_TASKS_DIR = join(process.cwd(), this.TOOLS_DIR, 'tasks', 'project');
  FONTS_DEST = `${this.APP_DEST}/fonts`;
  FONTS_SRC = [
      'node_modules/font-awesome/fonts/**',
      'node_modules/bootstrap/dist/fonts/**'
  ];
....

Added the following task:
import * as gulp from 'gulp';
import {FONTS_SRC, FONTS_DEST} from '../../config';

export = () => {
return gulp.src(FONTS_SRC)
      .pipe(gulp.dest(FONTS_DEST));
}

And added this task at the correct locations in my gulpfile.
Current behavior
When trying to view my application, the font's aren't loaded and I see the following warnings in my terminal:

Expected/desired behavior
I would expect the fonts to load correctly.
Other information
I think that something is going wrong when all CSS files are combined together, not sure however. Do you guys have the same issue?

I have the exact same issue with prod build

I actually got this working last week. It seems like your reference to the font-awesome font files is incorrect.
First of all, can you verify that the build has copied the files to dist/prod/fonts.
Then when using font-awesome, you have to reference it correctly, depending on what method you use CSS or Sass/Less. It seems to me that this is where it's going wrong. Which method are you using?

Hi @MathijsHoogland! As @Sjiep mentioned: Depending on which stylesheet mechanism you are using you need to reference the corresponding files correctly.
In a personal project (based on the seed) i also use font-awesome in conjunction with sass.
Therefore in my src/client/css/main.scss i have the following line:
@import 'font-awesome/scss/font-awesome';

@Sjiep thanks for your help, I am using CSS. The fonts are correctly loaded, and the reference also is correct when I load the development environment. This is what it looks like when I look at my main.css file:
src: url(../../../font-awesome/fonts/fontawesome-webfont.eot?v=4.6.3);
This obviously should be src: url(../fonts/fontawesome-webfont.eot?v=4.6.3), and in the source file (font-awesome.css) this is also how it looks. Somehow this is changed during the building process of production. Do you know how I can fix this?

In my project I've used the Sass version, so I don't have any experience with the CSS version. How are you including the font-awesome.css into your project?
To me it sounds a bit strange that a source url inside a css file is changed during the build.

In my project.config.ts file I added font-awesome with the following line:
let additional_deps: InjectableDependency[] = [
{src: 'font-awesome/css/font-awesome.min.css', inject: true},
];
Inside this file, there are references to the fonts (that all work). It is after all stylesheets are converted to main.css that the reference isn't correct anymore.

@MathijsHoogland have a look here: https://github.com/mgechev/angular2-seed/issues/850#issuecomment-218736860

I also have the same issue with the new seed, has there been a solution on how to correctly use a css font in prod?  Jus tlike mentioned above, the prod build for some reason changes the font url inside the css file.  Currently the only way I can solve it is by create a seperate build.fonts task for dev and prod.

Yes - in build.html_css.ts replace this line in processExternalCss()
.pipe(isProd ? plugins.concatCss(CSS_PROD_BUNDLE) : plugins.util.noop()) with
.pipe(isProd ? plugins.concatCss(CSS_PROD_BUNDLE,{rebaseUrls:false}) : plugins.util.noop())
this worked for me, this is same as what was suggested the good folks in the comments above

Thanks @csreddy, since I am using a newever version of the seed, I used your answer and did the following:
In my project.config.ts file, I added this line at the end of the constructor function:
this.mergeObject( this.PLUGIN_CONFIGS['gulp-concat-css'], { rebaseUrls: false } );

@csreddy     Thanks for the post!! this was killing us trying to fix for a release of our beta!  thanks to all who helped out, this seed starter has been great


[IMPROVE SEO] Create a blog page

Encourage User Engagement: To increase user engagement and dwell time, create a blog, allow comments, and build a community around our content.
Must have: Database, Sign-up, and Sign-in

Create shareable content


ConnectWise PSA (Independent Publisher)

When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)
If this is your first time submitting to GitHub and you need some help, please sign up for this session.

[x] I attest that the connector doesn't exist on the Power Platform today. I've verified by checking the pull requests in GitHub and by searching for the connector on the platform or in the documentation.
[x] I attest that the connector works and I verified by deploying and testing all the operations.
[x] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[x] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[x] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[x] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

[x] I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
[x] Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.
[x] Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.
[x] If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

Hello @howellchrisj,
the files names are not as expected. Kindly follow the templates in the below link and submit the files.
https://github.com/microsoft/PowerPlatformConnectors?tab=readme-ov-file
https://github.com/microsoft/PowerPlatformConnectors/tree/dev/templates/Independent Publisher

I renamed the json file, but I can't seem to understand what other naming is not correct.  The documentation says to upload screenshots of proof the connector works, but if there is a certain name the screenshots need to be, I can't seem to find them.  Also, the README.md is what is in the template, but I also see a readme.md, which if it is lowercase vs. uppercase, let me know which is correct.
Thank you.
On Thursday, November 28, 2024 at 11:13:14 AM EST, vmanoharas @.***> wrote:
Hello @howellchrisj,
the files names are not as expected. Kindly follow the templates in the below link and submit the files.
https://github.com/microsoft/PowerPlatformConnectors?tab=readme-ov-file
https://github.com/microsoft/PowerPlatformConnectors/tree/dev/templates/Independent Publisher
image.png (view on web)
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Hello @howellchrisj,
The screenshots to be uploaded here in the comments section, please remove them from the connector artifacts.

Please add only below files except the independent-publisher-icon.png file.

Hello @howellchrisj,
The swagger validation is still failing. Kindly fix the below errors.
##[error]The 'description' property is required. ValuePath: /paths/~1v4_6_release~1apis~13.0~1service~1tickets~1{id}/get/parameters/0
##[error]The 'x-ms-summary' property is required. ValuePath: /paths/~1v4_6_release~1apis~13.0~1service~1tickets~1{id}/get/parameters/1
##[error]The 'description' property is required. ValuePath: /paths/~1v4_6_release~1apis~13.0~1service~1tickets~1{id}/get/parameters/1
##[error]The 'x-ms-summary' property is required. ValuePath: /paths/~1v4_6_release~1apis~13.0~1service~1tickets~1{id}/get/parameters/2
##[error]The 'description' property is required. ValuePath: /paths/~1v4_6_release~1apis~13.0~1service~1tickets~1{id}/get/parameters/2
##[error]The 'x-ms-summary' property is required. ValuePath: /paths/~1v4_6_release~1apis~13.0~1service~1tickets~1{id}/patch/parameters/0
##[error]The 'description' property is required. ValuePath: /paths/~1v4_6_release~1apis~13.0~1service~1tickets~1{id}/patch/parameters/0
##[error]The 'x-ms-summary' property is required. ValuePath: /paths/~1v4_6_release~1apis~13.0~1service~1tickets~1{id}/patch/parameters/1
##[error]The 'description' property is required. ValuePath: /paths/~1v4_6_release~1apis~13.0~1service~1tickets~1{id}/patch/parameters/1
##[error]The 'x-ms-summary' property is required. ValuePath: /paths/~1v4_6_release~1apis~13.0~1service~1tickets~1{id}/patch/parameters/2
##[error]The 'description' property is required. ValuePath: /paths/~1v4_6_release~1apis~13.0~1service~1tickets~1{id}/patch/parameters/2
##[error]The 'contact' property is required. ValuePath: /info

New file has been uploaded with what I believe are the changes requested.

Hello @howellchrisj,
Thank you for fixing all the issues, however there is one move error mentioned below, kindly fix this.
The 'x-ms-connector-metadata' property is required.
Additionally, rename folder without "Proposal".

Metadata section added and renamed the folder.


The extension “Tailwind CSS” has stopped responding.

Getting this error on a regular basis ...
https://p112.p2.n0.cdn.getcloudapp.com/items/6quPzmzq/Image 2020-10-22 at 2.51.41 PM.png?v=3b36853573c976d028860f7170aed005
Is there a log to show what the issue could be?
Using macOS 10.15.6 and latest Nova build.

Thanks yabdab. Not certain, but I think this might have been introduced in the 2.0 update. Looking at the problem and am in touch with Panic to see if they have any ideas.
I think I have discovered what scenario causes the issue and that is when the user types the first and only class name of the HTML element. Are you able to verify this too? Also what file type are you using when this occurs?

@jasonplatts I mainly work on *.php and ..blade.php files (PHP-HTML)

I was experiencing the same error prior to the 2.0 update.

Thanks @mattgossman for letting me know. I am doing some debugging now.

Thanks @yabdab and @mattgossman for reporting this bug. I have found the issue and will release an update shortly to fix the bug.


Error while decoding

2023-10-02 04:44:42,617 INFO [zipformer.py:178] At encoder stack 4, which has downsampling_factor=2, we will combine the outputs of layers 1 and 3, with downsampling_factors=2 and 8.
2023-10-02 04:44:42,626 INFO [decode.py:917] Calculating the averaged model over epoch range from 5 (excluded) to 30
Traceback (most recent call last):
File "./pruned_transducer_stateless7/decode.py", line 1015, in
main()
File "/mnt/efs/swetha/icefall_env_swe/lib/python3.8/site-packages/torch/autograd/grad_mode.py", line 27, in decorate_context
return func(*args, **kwargs)
File "./pruned_transducer_stateless7/decode.py", line 922, in main
model.load_state_dict(
File "/mnt/efs/swetha/icefall_env_swe/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1667, in load_state_dict
raise RuntimeError('Error(s) in loading state_dict for {}:\n\t{}'.format(
RuntimeError: Error(s) in loading state_dict for Transducer:
size mismatch for decoder.embedding.weight: copying a param with shape torch.Size([500, 512]) from checkpoint, the shape in current model is torch.Size([250, 512]).
size mismatch for joiner.output_linear.weight: copying a param with shape torch.Size([500, 512]) from checkpoint, the shape in current model is torch.Size([250, 512]).
size mismatch for joiner.output_linear.bias: copying a param with shape torch.Size([500]) from checkpoint, the shape in current model is torch.Size([250]).
size mismatch for simple_am_proj.weight: copying a param with shape torch.Size([500, 384]) from checkpoint, the shape in current model is torch.Size([250, 384]).
size mismatch for simple_am_proj.bias: copying a param with shape torch.Size([500]) from checkpoint, the shape in current model is torch.Size([250]).
size mismatch for simple_lm_proj.weight: copying a param with shape torch.Size([500, 512]) from checkpoint, the shape in current model is torch.Size([250, 512]).
size mismatch for simple_lm_proj.bias: copying a param with shape torch.Size([500]) from checkpoint, the shape in current model is torch.Size([250]).

Please show your complete command.
The error indicates you use a different set of model arguments for decode.py and train.py. Please check the commandline arguments carefully.

Yeah I resolved that error.. I'm getting very high word error rate.. what should I do to reduce

Are you using your own model or use our pre-trained model?
If you are using your own model, is your model converged?

No I'm using my own model.. I'm training the model again.. it was running before after i changed the gpu.. it is showing error like
File "/usr/lib/python3.8/shutil.py", line 675, in _rmtree_safe_fd
onerror(os.unlink, fullname, sys.exc_info())
File "/usr/lib/python3.8/shutil.py", line 673, in _rmtree_safe_fd
os.unlink(entry.name, dir_fd=topfd)
OSError: [Errno 16] Device or resource busy: '.nfsf8aa902a3d86d53e00000bf6'
Traceback (most recent call last):
File "./pruned_transducer_stateless7/train.py", line 1275, in
main()
File "./pruned_transducer_stateless7/train.py", line 1266, in main
mp.spawn(run, args=(world_size, args), nprocs=world_size, join=True)
File "/mnt/efs/swetha/icefall_env_swe/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 240, in spawn
return start_processes(fn, args, nprocs, join, daemon, start_method='spawn')
File "/mnt/efs/swetha/icefall_env_swe/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 198, in start_processes
while not context.join():
File "/mnt/efs/swetha/icefall_env_swe/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 160, in join
raise ProcessRaisedException(msg, error_index, failed_process.pid)
torch.multiprocessing.spawn.ProcessRaisedException:
-- Process 1 terminated with the following error:
Traceback (most recent call last):
File "/mnt/efs/swetha/icefall_env_swe/lib/python3.8/site-packages/torch/multiprocessing/spawn.py", line 69, in _wrap
fn(i, *args)
File "/mnt/efs/swetha/marathi/ds-icefall-scripts/pruned_transducer_stateless7/train.py", line 1145, in run
train_one_epoch(
File "/mnt/efs/swetha/marathi/ds-icefall-scripts/pruned_transducer_stateless7/train.py", line 940, in train_one_epoch
valid_info = compute_validation_loss(
File "/mnt/efs/swetha/marathi/ds-icefall-scripts/pruned_transducer_stateless7/train.py", line 753, in compute_validation_loss
loss, loss_info = compute_loss(
File "/mnt/efs/swetha/marathi/ds-icefall-scripts/pruned_transducer_stateless7/train.py", line 704, in compute_loss
raise ValueError(

Duplicate of https://github.com/k2-fsa/icefall/issues/1289


[ENHANCEMENT] Improving Windows compatibility and general readability across Chapter 2 and 3

Windows support has been missing. In #15 and #13, there are a few workarounds given but these changes aims to improve a new user's experience by providing clear instructions to run notebooks on Windows.
We have also tried to make some outputs more readable by truncating them to reduce the size of the notebook while still providing a clear picture of the output as well as marking up some headings for a quick overview.

I cannot test Ch2 Nb5 as the link there is broken. Mentioned in #43. @varunp2k any updates on it?

Similarly link also broken in Ch3 Nb6. Mentioned in #46. @varunp2k


BUG: fixed memory leaks in MINC IO

MINC IO does not free allocations before throwing an exception sometimes,
s. discussion https://github.com/InsightSoftwareConsortium/ITK/pull/3529
https://open.cdash.org/viewDynamicAnalysis.php?buildid=8097352
(reviewed writing an image).

Don't know why 'Label PR' test fails, maybe something github internal?

Have changed a little to avoid repeating code and tested exit points (to avoid double free, etc).

@vfonov

I have done trivial test, without using itkIOTestHelper.h, just read MHA and write MNC to wrong path, the leaks are there. If the path is correct -- there are no problems,

Adding a test such as the one you did would be convenient.

it is really very simple test, to try valgrind --leak-check=full ./test without ITK's testing subsystem, already merged new ITK tests with intentional wrong path (which discovered leaks) are doing the same. Maybe later more tests could be done to trigger exceptions and increase coverage, e.g. 4D image.

@vfonov

Looks ok


Introduce OpenSSF Scorecard

This allows users to quickly assess the supply-chain security, as judged by the OpenSSF scorecard, of this project.
While the score is not yet great, this scorecard will help the project to implement sensible improvements.
Unfortunately, the action will run only once the PR has been merged, as the check refuses to work on anything other the main branch, master in this case.
To check out how the reporting will look like, have a look at my fork:

Scorecard: https://scorecard.dev/viewer/?uri=github.com/rettichschnidi/goosebit
Badge:

Alternatively, the checks can be run locally in CI:
docker run -e GITHUB_AUTH_TOKEN=<your GitHub PAT> gcr.io/openssf/scorecard:stable --show-details --repo=github.com/UpstreamDataInc/goosebit

The CLI actually tests this repository, instead of my fork. As a result, the resulting score is 5.6 and not 3.2.

Looks good to me, you may just want to run pre-commit install and pre-commit run --all-files just so you dont get the pre-commit warnings here.

Looks good to me, you may just want to run pre-commit install and pre-commit run --all-files just so you dont get the pre-commit warnings here.

Just passed CI without this.
Problem is that form reading the readme, I am not clear how I would install prettier, etc. Seems to be available only via npm?

Looks good to me, you may just want to run pre-commit install and pre-commit run --all-files just so you dont get the pre-commit warnings here.

Just passed CI without this.
Problem is that form reading the readme, I am not clear how I would install prettier, etc. Seems to be available only via npm?

You don't have to, it is using a mirror that sets up an environment for you ;)

You don't have to, it is using a mirror that sets up an environment for you ;)

For whatever reasons, pre-commit was no installed, even after running poetry install --with dev,docs,tests:
$ pre-commit run --all-files
bash: pre-commit: command not found

Resolve it by resorting to pipx (pipx install pre-commit).

You don't have to, it is using a mirror that sets up an environment for you ;)

For whatever reasons, pre-commit was no installed, even after running poetry install --with dev,docs,tests:

$ pre-commit run --all-files

bash: pre-commit: command not found

Resolve it by resorting to pipx (pipx install pre-commit).

You may need to run it with poetry (prefix with poetry run

You may need to run it with poetry (prefix with poetry run

This worked, thanks!


Predicted Phenotypes included even when using --exclude-resistance-phenotypes (in development)

It looks like the command-line option --exclude-resistance-phenotypes is not quite working out. This should have the behaviour of excluding the Predicted Phenotype columns, but they are still present for the Summary and Detailed_Summary results.

This command-line option works by selecting whether or not we are using the AMRDetectionSummary.py or AMRDetectionSummaryResistance.py classes (in https://github.com/phac-nml/staramr/blob/befb44bda34e6691c0df744547146604971b3efb/staramr/detection/AMRDetectionFactory.py#L31).
To fix, you may need to shift some of the code in the AMRDetectionSummary.py class which adds the Predicted Phenotype column down to the subclass AMRDetectionSummaryResistance.py.

Fixed in #67


NoClassDefFoundError: com.api.jsonata4java.expressions.JS4JDate

Hi, I hit this error when trying to use JSONata4Java with java:
java.version = 1.8.0_231
java.runtime = Java(TM) SE Runtime Environment (<IP_ADDRESS> - pmz6480sr6-20190816_01(SR6))

This is the stack trace.
[3/8/24 12:00:10:565 GMT] 00000045 com.ibm.ws.webcontainer.util.ApplicationErrorUtils           E SRVE0777E: Exception thrown by application class 'com.api.jsonata4java.expressions.FrameEnvironment.<init>:78'
java.lang.NoClassDefFoundError: com.api.jsonata4java.expressions.JS4JDate
	at com.api.jsonata4java.expressions.FrameEnvironment.<init>(FrameEnvironment.java:78)
	at com.api.jsonata4java.expressions.Expressions.<init>(Expressions.java:132)
	at com.api.jsonata4java.expressions.Expressions.parse(Expressions.java:122)

Is this a known issue at all? I couldn't see anything for it in issues. I had run the same scenario quite a long time ago successfully and we've updated our JSONata4Java level since then - just a guess whether it is something to do with Java 11 changes that went in. Currently using 2.4.3.
I can see the class inside the jar file installed.

This seemed to go away when I restarted my server - unknown why.


NPE from _protocolAvailableSuggestionSet (Flutter SDK 1.5.5-pre.15)

Exception from analysis server (running from VSCode / Dart Code)
What I was doing
Doing some web-based stuff.
Versions

Flutter SDK 1.5.5-pre.15
Visual Studio Code 1.33.1
Dart Code 2.26.0-beta.1

Analyzer Info
The analyzer was launched using the arguments:
c:\git\flutter\bin\cache\dart-sdk\bin\snapshots\analysis_server.dart.snapshot
--client-id=Dart-Code.dart-code
--client-version=2.26.0-beta.1

Exception
Failed to handle completion domain request: {id: 3, method: completion.setSubscriptions, params: {subscriptions: [AVAILABLE_SUGGESTION_SETS]}, clientRequestTime:<PHONE_NUMBER>056}
NoSuchMethodError: The getter 'iterator' was called on null.
Receiver: null
Tried calling: iterator
#0      Object.noSuchMethod (dart:core-patch/object_patch.dart:50:5)
#1      _protocolAvailableSuggestionSet (package:analysis_server/src/domains/completion/available_suggestions.dart:141:35)
#2      createCompletionAvailableSuggestionsNotification.<anonymous closure> (package:analysis_server/src/domains/completion/available_suggestions.dart:75:14)
#3      MappedListIterable.elementAt (dart:_internal/iterable.dart:414:29)
#4      ListIterable.toList (dart:_internal/iterable.dart:219:19)
#5      createCompletionAvailableSuggestionsNotification (package:analysis_server/src/domains/completion/available_suggestions.dart:76:8)
#6      CompletionDomainHandler.setSubscriptions.<anonymous closure> (package:analysis_server/src/domain_completion.dart:482:11)
#7      _rootRunUnary (dart:async/zone.dart:1136:13)
#8      _CustomZone.runUnary (dart:async/zone.dart:1029:19)
#9      _CustomZone.runUnaryGuarded (dart:async/zone.dart:931:7)
#10     _BufferingStreamSubscription._sendData (dart:async/stream_impl.dart:336:11)
#11     _DelayedData.perform (dart:async/stream_impl.dart:591:14)
#12     _StreamImplEvents.handleNext (dart:async/stream_impl.dart:707:11)
#13     _PendingEvents.schedule.<anonymous closure> (dart:async/stream_impl.dart:667:7)
#14     _rootRun (dart:async/zone.dart:1120:38)
#15     _CustomZone.run (dart:async/zone.dart:1021:19)
#16     _CustomZone.runGuarded (dart:async/zone.dart:923:7)
#17     _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:963:23)
#18     _rootRun (dart:async/zone.dart:1124:13)
#19     _CustomZone.run (dart:async/zone.dart:1021:19)
#20     _CustomZone.runGuarded (dart:async/zone.dart:923:7)
#21     _CustomZone.bindCallbackGuarded.<anonymous closure> (dart:async/zone.dart:963:23)
#22     _microtaskLoop (dart:async/schedule_microtask.dart:41:21)
#23     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50:5)
#24     _Timer._runTimers (dart:isolate-patch/timer_impl.dart:391:30)
#25     _Timer._handleMessage (dart:isolate-patch/timer_impl.dart:416:5)
#26     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:171:12)

Diagnostics requested after the error occurred are:
[
    {
        "name": "C:\\git\\secret_directory_name\\examples\\filipino_cuisine",
        "explicitFileCount": 3,
        "implicitFileCount": 2091,
        "workItemQueueLength": 3,
        "cacheEntryExceptions": []
    }
]

@scheglov @stereotype441 this is a crash from a recent Dart SDK, so likely still relevant (and w/ accurate the line numbers).

https://dart-review.googlesource.com/c/sdk/+/100461

For the record, here's @scheglov's analysis of what would likely trigger this exception: "It happened when something was a library, and then became a part, e.g. when you write "part of" in an empty file. I think it happens only once for such situation, i.e. does not stick in the invalid state, because subsequent changes see it as a part and ignore."


migrate from csv files to sqlite databases for downstream use in queries

All csv data are now in two .db files for the two levels of analysis (counties and msa). Prior to deployment
of dbs to the polis server, these live locally. As the db files are now too large to store on GitHub, the user needs to
run csv_to_db_pipeline.py before the first use to generate the db locally.

The original DataGrabber classes have been refactored, renamed to ...CSV and decoupled from use dowstream.

The new DataGrabberDB class has been introduced and passed on to function as generic DataGrabber and MSADataGrabber.

Additional tests for DataGrabberDB have been introduced in test_data_grabber_sql. Additionally, DataGrabberDB under the generic alias passes all the tests that the original DataGrabber did.

generate_sql.ipynb (docs/experimental) contains performance tests for both approaches. At least in the current setting the orignal method is faster. The main culprit seems to be:

13    0.013    0.001    0.013    0.001 {method 'fetchall' of 'sqlite3.Cursor' objects}

This is not too surprising, after some reflection, as illustrated by this comment from ChatGPT:

As the ultimate tests of the switch to DB would involve data updates and model retraining, I leave the original .csv files and classes until those events. Keep in mind they are now not needed for queries to work correctly (they are needed to generate the .db files and for some of the tests).

The new pytest release leads to incompatibilities that might be worth investigating later. For now, fixed the pytest version to be 7.4.3. in setup.py.

Some cleaning scripts have been moved to a subfolder, which required a small refactoring of import statements in generic data cleaning pipeline scripts.

Incorrect indentation in a DataGrabber test has been fixed.

Have been running into this issue apparently.
https://github.com/PyCQA/isort/issues/1518

still issues with what isort does between CI and locally, despite the version numbers being the same. Other people have faced this, I'm looking for a solution, but slowly considering using black w/o isort at least till switching to isort 6.0
https://github.com/PyCQA/isort/issues/1889


turn_based_test.py hangs if startMission fails

If there is an exception in startMission, turn_based_test.py calls exit(1), as per all our samples...
But in this case, the exit takes place inside a thread, which doesn't terminate the Python process, it simply kills the thread, leaving the script deadlocked.

Fixed as part of #543.


add best practices in Quick Start section

Signed-off-by: Fansong Zeng<EMAIL_ADDRESS>Ⅰ. Describe what this PR does
Ⅱ. Does this pull request fix one issue?

Ⅲ. Describe how to verify it
Ⅳ. Special notes for reviews

/approve

/lgtm


importer: use kv engine instead of raw API

What is changed and how it works?
Issue Number: Ref #12842
What's Changed:
So that it can support both v1 and v2.

To make it work with V2, we still need to use tablet registry.
Check List
Tests

Unit test
Integration test

Release note
None

/merge

/test

/test


Fetching channel emoticon metadata hanging

Hi there, first off thanks for building this it's a brilliant idea!
Running into an issue at the moment when fetching channel emoticon metadata.
I've used docker-compose up -d to run the API server on port 3000 and it seems to be fine, this is what I get from docker-compose ps:
     Name                   Command               State           Ports
--------------------------------------------------------------------------------
api_app_1        docker-entrypoint.sh npm start   Up      <IP_ADDRESS>:3000->3000/tcp
api_postgres_1   docker-entrypoint.sh postgres    Up      <IP_ADDRESS>:5432->5432/tcp

Once I use tc connect theycallmemac_ however I get the following response:
✔ Checking for emoticon api.
⠏ Fetching channel emoticon metadata.

It just seems to hang forever while it attempts to fetch the emoticon metadata. If I stop the container, it works fine again.
Any ideas why this may be happening?

Did you clone the repo with Git LFS installed? The api/seed/emotes.bak file should be around ~20mb or so. I've also updated the Docker configuration files as there seemed to be some issues with the Node server failing to connect to the database. That being said, it looks like the endpoint that I was using at the time to get the channel emotes (https://static-cdn.jtvnw.net/emoticons/v1/) seems to have changed so that will need to get updated to get the emoticons working in the CLI.

I've updated the endpoint so fetching the emotes should work again.

Running the new files and checking with docker-compose ps I get the following:
     Name                   Command               State            Ports
---------------------------------------------------------------------------------
api_app_1        docker-entrypoint.sh ./wai ...   Exit 1
api_postgres_1   docker-entrypoint.sh postgres    Up       <IP_ADDRESS>:5432->5432/tcp

If I check the logs of api_app_1 I get this:
internal/modules/cjs/loader.js:957
    throw err;
    ^

Error: Cannot find module '/app/wait-for-it.sh'
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:954:17)
    at Function.Module._load (internal/modules/cjs/loader.js:847:27)
    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:67:12)
    at internal/main/run_main_module.js:17:47 {
  code: 'MODULE_NOT_FOUND',
  requireStack: []
}
internal/modules/cjs/loader.js:957
    throw err;
    ^

Error: Cannot find module '/app/wait-for-it.sh'
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:954:17)
    at Function.Module._load (internal/modules/cjs/loader.js:847:27)
    at Function.executeUserEntryPoint [as runMain] (internal/modules/run_main.js:67:12)
    at internal/main/run_main_module.js:17:47 {
  code: 'MODULE_NOT_FOUND',
  requireStack: []
}

Let me know what you think on this :)


Mistral Embedders and Generators

Summary and motivation
Creating this issue for https://github.com/deepset-ai/haystack/issues/6928
Mistral uses the same API as OpenAI and so far we've instructed people to use the OpenAIChatGenerator and OpenAITextEmbedder to make use of the Mistral API directly.
It would be good to have components that wrap these two which makes it nicer to use with Mistral: https://haystack.deepset.ai/integrations/mistral
Detailed design
The components use the OpenAI embedders and generators. Currently mistral uses pretty much the identical API to OpenAI, but we can make changes when and if they move on to something else
Checklist
If the request is accepted, ensure the following checklist is complete before closing this issue.
### Tasks
- [ ] https://github.com/deepset-ai/haystack-core-integrations/pull/409
- [ ] Docs are published at https://docs.haystack.deepset.ai/
- [x] There is a Github workflow running the tests for the integration nightly and at every PR
- [x] A label named like `integration:<your integration name>` has been added to this repo
- [x] The [labeler.yml](https://github.com/deepset-ai/haystack-core-integrations/blob/main/.github/labeler.yml) file has been updated
- [x] The package has been released on PyPI
- [x] An integration tile has been added to https://github.com/deepset-ai/haystack-integrations
- [x] The integration has been listed in the [Inventory section](https://github.com/deepset-ai/haystack-core-integrations#inventory) of this repo README
- [x] There is an example available to demonstrate the feature
- [x] The feature was announced through social media
- [ ] Submit to Mistral Docs

@dfokina - I've prepared the docs on Notion.
Can I have a review on there please? I'm happy to publish when you see fit 👍


Enable incrementing to update binding when lostfocus.

/// <summary>
/// Controls how the increment and decrement commands work.
/// </summary>
public enum SpinMode
{
    /// <summary>
    /// Use the same as the binding i.e. LostFocus or PropertyChanged
    /// </summary>
    AsBinding,

    /// <summary>
    /// Update the binding on click.
    /// </summary>
    Update
}

I'd do something like
enum SpinUpdateMode
{
    MimickValueBinding,
    LostFocus,
    PropertyChanged
}```


Email Template not found on production build

HI - I posted a comment on the related issue: https://github.com/vue-email/nuxt/issues/13.  Creating a new issue just in case you don't get notified of comments on a closed issue.
@Flowko Hi. First, thank you for your work on this module!
I'm looking into a similar issue (to https://github.com/vue-email/nuxt/issues/13) and it has not been solved by upgrading to 0.8.8, which was the proposed solution.
I'm also seeing that the stackblitz linked in that issue (https://stackblitz.com/edit/github-usvs6j-5oeyxy?file=package.json,emails%2Ftest.vue,nuxt.config.ts) still has the same error, even with the updated version.
(If that link isn't adequate I can look into creating a separate stackblitz soon)
Could you please advise?
Thanks in advance

Hi, thanks for opening another issue, tottally missed the other one, probabaly because its closed, tbh im not even sure whats causing it to break inside of stackblitz, locally it works fine with building as well, yould you be down to make a reproduction using vercel ?

Hi, sorry for the delay, I've been out sick.
A member of my team was also looking into this and was able to solve it!
From him:

Our issue ended up being a corrupt lock file.  Our build version was missing esbuild-linux-64 which caused it to fail on deploy.

Closing this issue now.  Thanks!


How to load style file from node_modules in meteor 1.3?

<IP_ADDRESS>/:14 Resource interpreted as Stylesheet but transferred with MIME type text/html: "http://<IP_ADDRESS>:3000/node_modules/angular-material.css".

need copy style files from node_modules/XXX to client/lib or put style file into public.
how to load directly from node_modules?


Fix Fast Post

?fast post recent

For fasts that are not finished yet, PD bot should not attempt to post them! (instead post the most recent COMPLETED fast)

?fast post 2 - where fast 2 exists for user - nothing shows up

It works for my regular account, as an admin, try a dummy account


Deprecated Property Warnings in EventWebhook Signature Verification with PHP 8.2

Issue Summary
The issue involves using the convertPublicKeyToECDSA and verifySignature functions in the SendGrid PHP client library. When executing these functions, deprecated warnings are logged.
Steps to Reproduce

Call the convertPublicKeyToECDSA function with the SendGrid webhook public key.
Use the output of the first function as input to the verifySignature function along with the request content and headers.

Code Snippet
$eventWebhook = new EventWebhook();
$ecPublicKey = $eventWebhook->convertPublicKeyToECDSA($_ENV['SENDGRID_WEBHOOK_PUBLIC_KEY']);
return $eventWebhook->verifySignature(
            $ecPublicKey,
            $request->getContent(),
            $request->headers->get(EventWebhookHeader::SIGNATURE),
            $request->headers->get(EventWebhookHeader::TIMESTAMP)
        );

Exception/Log
Deprecated: Creation of dynamic property EllipticCurve\PublicKey::$pem is deprecated
Deprecated: Creation of dynamic property EllipticCurve\Signature::$der is deprecated

Technical details:

sendgrid-php version: 8.1.1
php version: 8.2.19

We are also waiting for this. I opened a SendGrid support ticket and after some struggles they agreed to "submit a JIRA" on the 16th May 2024. Its been almost a month now and still no change.
The EllipticCurve deprecations come from starkbank/ecdsa which is set to version 0.* in composer.json. There is a new version 2.1 https://github.com/starkbank/ecdsa-php which likely fixes the issue.
On a related note sendgrid/sendgrid-php prerequisites requires

PHP version 7.3, 7.4, 8.0, or 8.1

These versions are all outside of active support, and only 8.1 is still in security support. Could we please get official PHP 8.2/8.3 support!

Well Sendgrid closed my support ticket in relation to this issue as "fixed" today 😒
I am getting the distinct feeling that SendGrid is just being left to decay by Twillo with some minimal outsourcing to keep the lights on...

Dear Sendgrid, you will be pleased to learn that PHP 8.4 has been released today...
Its now been 1 year 11 months and 13 days since your library stopped explicitly supporting the latest active PHP versions. Others are starting to notice and ask about it too https://github.com/sendgrid/sendgrid-php/issues/1114
@thinkingserious I see you were a major contributor to this library in the past and still work at Twilio, Inc. Any chance you could ask the right people why this library isn't being updated anymore?

The PR has been merged. This will be available in the release on coming Thursday!


WEB Support

Is your feature request related to a problem? Please describe.
This is amazing package but there are no support web, even tho api.video supports it in JS ecosystem.
I have been trying to broadcast to rtmp on flutter web on my own using websockets as transfer protocol and ffmpeg for trancoding on a proxy server, But not able to do yet.
Describe the solution you'd like
There is a package for Native Web(JS) on NPM for doing the same @api.video/browser-to-rtmp-client .
It would be awesome, if we could port it to flutter web.
Describe alternatives you've considered
None.
Additional context
None.

Hi,
That would be really nice. Unfortunately as you said, it would required a third party server (from WebSocket to RTMP). This is not something we want to include in our live stream libraries.
So, we won't add that feature as long as it is not possible to directly stream RTMP from browser.


case-insensitive GOPATH

fixes issue #238

This change should use filepath.HasPrefix.

But, while filepath.HasPrefix() isn't perfect, is technically deprecated, and doesn't help the OSX/HFS case, it's still an incremental improvement, so works for now.

The behavior of Windows is not same as OSX. OSX's issue is filesystem things. At least, ZFS can switch case-sensitive/case-insensitive. So filepath.HasPrefix MUST work intended for non-existed files on Windows.

seems test failing. related on this commit? https://github.com/golang/dep/pull/239/commits/423e0e910e38fc80570bae6bae504ef282263166

Ugh, those are unrelated failures. I'll fix that quickly...

Should be fixed now. Closing and reopening to refresh CI.

OK, I think we're good here. Thanks, @mattn!


Model selection

Introducing the model selection subpackage and iterative stratification algorithms

@ChristianSch I've documented internal and helper functions.
@ljvmiranda921 I've adhered to your review, there's less bloat in class properties, private functions are made private, helpers are moved out of the class, lot's of docs added.
Please take a look at it, and approve the PR if it's ok, or comment if not :)


[requires.io] dependency update on master branch

Coverage remained the same at 100.0% when pulling 5737fb79e66a9739d4e9170237f32bcf9707ad8d on requires-io-master into 8185685b5afd57abb84dab5594e4c9bde69a8b3e on master.

Coverage remained the same at 100.0% when pulling 49f53bdc52cbd1558889902827ba6e97bced5228 on requires-io-master into 8185685b5afd57abb84dab5594e4c9bde69a8b3e on master.


Remove dependencies on codeblock linenumbers and line highlighting.

To investigate and remove the use of code block line numbering and code block line highlighting.
Why?
Asciidoc/Antora, for product documentation, has a simple, lean highlighting package. No line numbering or line highlighting.
These features are used in Docusaurus, for community documentation.
If the use of these features can be removed from use in Docusaurus/community then transfer to AD/Antora, and maintenance is simplified.

First proposal is in #469.

Fixed by #469


Update README.md

Add example how to install llama in openbsd (>=7.3)

Can you add the link to the package? I’d like to what version is there

The link to the CVS repo is: https://cvsweb.openbsd.org/cgi-bin/cvsweb/ports/sysutils/llama/Makefile?rev=1.1&content-type=text/x-cvsweb-markup
The package itself still not on, but it will be: http://cdn.openbsd.org/pub/OpenBSD/snapshots/packages/${ARCH}/llama-v1.2.0.tgz
We will keep it up-to-date.


Add unit test for covering 'migrate' method in ChatViewModel

Add unit test for covering 'migrate' method in ChatViewModel, specifically that uploads are migrated along with other expected fields.
MOB-2108

!merge


Unable to embed entire week in weekly note

Before Submitting: Double-check that you are running the latest version of the plugin. The bug might have already been fixed 😄

Describe the bug
I followed the instructions on the page and tried to embed the entire week in the weekly note. I added the snippet provided in the weekly note template, but it tells me that it cannot create file. I am not sure if I missed anything important and I want to have something like a weekly note with reference to other daily notes. I also installed periodic notes to check whether it is because of the problem but it did not work as well. I will include my settings as well. Thank you in advance.
Steps to reproduce

Copied the snippet from the website
Pasted it on the weekly note template
It says cannot create file

Expected behavior
When I paste the snippet provided in the weekly note template it should allow me to embed the entire week in the weekly note but it is not working at the moment.
Screenshots

Environment (please specify)
OS
MacOS
Obsidian Version (e.g. v0.10.6)
v0.12.12
Theme (if applicable):
If the bug is visual, please provide the name of the Community Theme you're using.

Update!
There is a note in the Read Me for Periodic Notes that the tags work slightly differently in Weekly Notes. I didn't catch this until I did several read-throughs of the documentation of Calendar and Periodic Notes, but I feel a bit silly now.
I resolved the issue by:

Designating my Weekly Note template file in Settings > Periodic Notes > Weekly Notes > Weekly Notes Template

Using the suggested CSS snippet:

It works!

Who knew file structures were important?


xen: polarssl extfile vulnerable to CVE-2015-1182

The xen expression fetches version 1.1.4 of polarssl as part of xenExtfiles. This version of PolarSSL is vulnerable to CVE-2015-1182. From NVD:
"The asn1_get_sequence_of function in library/asn1parse.c in PolarSSL 1.0 through 1.2.12 and 1.3.x through 1.3.9 does not properly initialize a pointer in the asn1_sequence linked list, which allows remote attackers to cause a denial of service (crash) or possibly execute arbitrary code via a crafted ASN.1 sequence in a certificate."

Thanks,
I had a quick look. http://xenbits.xensource.com/xen-extfiles/ doesn't distribute a new version. I suppose we need a subscription with them to get the proper version ?
https://security-tracker.debian.org/tracker/CVE-2015-1182 doesn't come with a patch for 1.1.4.
/cc @mixis @ts468 for more insights

The 1.1 branch is EOL in upstream: https://tls.mbed.org/download-archive

I've removed stubdom build in xen as it was breaking the compilation and also removed now unneeded dependencies (including polarssl). See https://github.com/NixOS/nixpkgs/commit/9ad60eae48c86ea2b8aa8901e87d6ba90f399400 and commit 9c36de8cb2260d1992b41555059bccd105cbd09e


EC2 Credentials Provider not used on EC2 Mac instance

Describe the bug
The default chain of credentials provider consider .ec2 only when running on Linux.
Now that Amazon provides EC2 Mac instance, .ec2 should be included in the default credential provider chain
To Reproduce
# create an AWSClient
awsClient = AWSClient(
            credentialProvider: .default
            retryPolicy: .jitter(),
            httpClientProvider: .createNew)

# Compile and Run on EC2 Mac instance  (or compile on Mac laptop and run on EC2 Mac)
# with no `~/.aws` directory, just an IAM role attached to it

# It causes the following Error
SotoCore.CredentialProviderError
No credential provider found

Expected behavior
The AWSClient should pickup IAM meta data credentials by default
Setup (please complete the following information):

OS: 12.5.1
Version of Soto : latest
Authentication mechanism IAM Instance Profile on EC2 Mac

Additional context
Add any other context about the problem here.
The line that causes the problem is here https://github.com/soto-project/soto-core/blob/main/Sources/SotoCore/Credential/CredentialProvider.swift#L77

I'm not sure this should be enabled by default.
You can already setup Soto to use an EC2 credential provider with the following
let client = AWSClient(credentialProvider: .ec2, ...)

If you want it to do the equivalent of what is the default for Linux you would provide this
let client = AWSClient(credentialProvider: .selector(.environment, .ecs, .ec2, .configfile()), ...)

This will go through the list of possible credential providers environment variables, ECS, EC2 and then the AWS config file until it finds one that works

Agreed, this is what I do now, I moved away from the .default credentials provider. So this is not a blocker or a bug. I just found it surprising that the default behaviour changes depending on the OS.
There was a good reason for that when EC2 was Linux only (I am not even considering Windows here :-) But now that EC2 has macOS too, a library (or SDK) should not provide different behaviour for different OSes.
If you prefer to close this report and keep it like it is, I am good with that. Maybe just make it clear in the doc that macOS is not behaving the same as Linux.

Documentation in SotoCore code is up to date already so closing this.


fix: issue with direct child filter partially omitting values in quickpick

This PR:

Fixes an issue in Lookup (v2) where the direct child filter would be applied after pagination has happened, which allowed some values to be omitted from the selectable items.
Adds example notes that demonstrate the fix.

lgtm for v2 changes

This PR also:

fixes a bug where triggering the direct child filter button would revert the picker value change to initial value
changes picker refresh behavior to

only refresh buttons that needs to be refreshed

this change broke the behavior of note type modifiers, which has also been fixed.

properly refresh picker after direct child filter is triggered

fyi: #1135 is branched off of this PR so this one should go in first.


SD-Card

I am not sure if I am at the right and proper adress to suggest this feature. Otherwise I would kindly ask you to move my idea forward.
What is missing

I would love to see the possibility to move the app to sd card (a feature that is on Android plattform possible for some apps).
Why should it be included

This would be great for all users of a cheap smartphone with low amount of internal memory like Motorolas Moto E 1. Gen. or other lowbuget phones. So the using of the app will be easier on lowbuget devices.
Where should it be included

In the documentation it should be included in the section ”Installation”.

Motorolas Moto E 1. Gen.

This device has no BLE.

Motorolas Moto E 1. Gen.

This device has no BLE.

Oh that's true... So maybe it (moving to sd) would be just a nice, comfortable feature, and such old/low budget smartphones are not able to use the app, anyway.
That leads me to the question:
How much percent of the smartphones used in Germany will be able to use it (circa)?

I'm think allowing the install on the SD card is not good, as we need to rely on the app to be apple to constantly download new keys, poll for updates etc (see developer documentation):

When the user enables USB mass storage to share files with their computer (or otherwise unmounts or removes the external storage), any application installed on the external storage and currently running is killed. The system effectively becomes unaware of the application until mass storage is disabled and the external storage is remounted on the device. Besides killing the application and making it unavailable to the user, this can break some types of applications in a more serious way. In order for your application to consistently behave as expected, you should not allow your application to be installed on the external storage if it uses any of the following features, due to the cited consequences when the external storage is unmounted:

@tklingbeil
USB mass storage isn't being used since Android 4.something anymore. Furthermore, it is clear that the app in general can't guarantee to work 100% of the time. Therefor I don't see any limitation why SD-card storage shouldn't be allowed.

@tklingbeil
I see your point that installing on sd card is not 100% reliable.
On the other hand it is politically wanted that as many people as possible install the app.
How many of them will buy a new mobile phone because there is no space left on the internal memory?
Wouldn't it be better if they have a not 100 % reliable app compared to they have no app?

The current size of the app is ~13MB. I understand that in some cases that might just be too large to install it - but otherwise I would consider that pretty slim.
Apart from that: It's not only about the app being unreliable: It would fail silently, i.e. the users might have a false sense of security, because they expect the app to work, even though it is not even running anymore.

Why would it fail at all? Why wouldn't it fail on internal memory?

@treysis:
Thomas already linked the android documentation:
https://developer.android.com/guide/topics/data/install-location#ShouldNot

@Lutzion And like I wrote before as well: USB mass storage mode isn't used since at least Android 5.0 anymore. This applies only if someone ejects the card.

I would like to have the possibility to move this app to the SD-Card. It makes sense also on more modern smartphones (such as Samsung A3 (2017)), having not so much memory. Nevertheless the Corona-Warn-App runs on this device on Android 8.
I belong to the people not throwing away functional devices every year. I cannot understant the constraints against shifting the app to a SD card ("security", "malfunction", and so on). I shift all applications to SD card wherever possible, and had no problem doing that.


Candy Brick Game

Project title
Candy Brick Game
Project Description
Candy Brick Game
Stack:
And in order to tick the check box just but x inside them for example - [x] like this. Please delete options that are not relevant.

[x] Html
[x] CSS
[x] JavaScript

/assign


feat(web): add device test page

feat(flat-components):  add CameraTest, SpeakerTes, MicrophoneTest, DeviceTestPanel component to flat-components

SVGs need to be optimized.
You can try with SVGO.

SVGs need to be optimized.
You can try with SVGO.

This should be done in another PR as all the svg's in this project need to be optimised.

SVGs need to be optimized.
You can try with SVGO.

Thank you for sharing your advice with me, we can create issue of the SVG optimize


Consider Documenting Configuration

I may have misunderstood the way this tool works, but at face value my impression is that I would need to dive into the source code in order to understand what configuration options are available.

Hi @Kurt-von-Laven,
Thank you for your feedback, really appreciated.
This is the first release of the plugin, I will improve the documentation and the option to customize the plugin.

Thank you for creating it!

Personally, I feel like you did this one, and it can be closed, but did you have more work planned here?

On this one I don't so I will close it :)
But I do have an expansion to this plugin coming soon.


AttributeError: 'NoneType' object has no attribute 'shape'

after set
yt_bb
crop511
train0000
train0001
...
train.json
File "/home/ty/PycharmProjects/pysot-master/pysot/datasets/dataset.py", line 218, in _get_bbox
imh, imw = image.shape[:2]
AttributeError: 'NoneType' object has no attribute 'shape'

Re-download train0056.tar. Maybe could solve your problem.

thanks

when use the new youtube_bb
AttributeError: 'NoneType' object has no attribute 'shape'
@StrangerZhang

when only use the coco
AttributeError: 'NoneType' object has no attribute 'shape'
@StrangerZhang
why?


Publish to npm

Could you please publish this to npm? It's just running npm publish from the project root.
:)

Hi, +1 for this. Looks great but I need it in npm, please. Thanks.


fix: Added Windows compatible translation of "hoist-cli" script

Description of changes
Created a script called hoist-cli-win to be used by setup-dev-win.
The new script hoist-cli-win is a translation of the hoist-cli, where:

rm -rf is replaced by rimraf
cp -r is replaced by xcopy /e /q
/ is replaced by \\ in the path

Issue #, if available
540

Description of how you validated changes
Validated by changing updating the scripts and running the setup again on Windows.
Checklist

[X] PR description included

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Description of changes
Created a script called hoist-cli-win to be used by setup-dev-win.
The new script hoist-cli-win is a translation of the hoist-cli, where:

rm -rf is replaced by rimraf
cp -r is replaced by xcopy /e /q
/ is replaced by \\ in the path

Issue #, if available
540

Description of how you validated changes
Validated by changing updating the scripts and running the setup again on Windows.
Checklist

[X] PR description included

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Thank you for your contribution! I'll verify behavior on a windows machine, then we should be good to merge this in!


reduce fn.bind(this) usage by aliasing this to self

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[x] Code style update (formatting, local variables)
[x] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Other... Please describe:

What is the new behavior?
No new behavior.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information:
Some segments of code in lib/Compiler.js and lib/Compilation.js have significant levels of callback nesting, and each callback must bind to this to be able to access the Compiler API. This is hazardous and difficult to maintain; it is better to alias this to another variable.

I'd argue that it's best to just use arrow functions instead, but that will require Node 4.

As of right now, we are only running tests against 4 and 6 (and 4 is optional fail). I think we can start with this, and then discuss the potential of dropping the pre node 4 support.

@sokra , I thought you might have some special purpose to use fn.bind(this)(or related code style) before.
But since this PR is welcomed, some of the codes can be refactored to be more readable and understandable.


Expected an assignment or function call and instead saw an expression

Hello,
Thank you for the great class!
I get the following Failed to compile error:
./src/components/PlaceDetails/PlaceDetails.js
Line 10:19:  Expected an assignment or function call and instead saw an expression  no-unused-expressions
it is for this line of cod:
if (selected) refProp?.current?.scrollIntoView({ behavior: 'smooth', block: 'start' });
I tried everything. I looked several times over the video, replaced '' with "" removed the ; tried a lot of different combinations. Nothing works. I used the exact code from git, and still the same error. On stack overflow and other sites, they say that it has something to do with the Return and some () or {} , but I am unable to figure out exactly what. And it is more frustrating as for you it worked. I had multiple issues along with the video where I was missing ; although you had them missing in the video for you the code worked but for me, it didn't until I put them at the end of various bits of code. What is the setup that you use for your VS Code?
Thank you for the help!

Here are the 2 things i added to get this to work.
In List.jsx under grid where you are passing refProp={elRefsZ[i], add ref={elRefs[i]} to Grid component.
      <Grid container spacing={3} className={classes.list}>
        {places?.map((place, i) => (
          <Grid ref={elRefs[i]} item key={i} xs={12}>
            <PlaceDetails
              selected={Number(childClicked) === i}
              refProp={elRefs[i]}
              place={place}
            />
          </Grid>
        ))}

Second, make sure in PlaceDetails.jsx under const PlaceDetails, make sure you are adding the ? in refProp?.current?.scrollIntoView({ behavior: "smooth", block: "start" });
const PlaceDetails = ({ place, selected, refProp }) => {
if (selected)
refProp?.current?.scrollIntoView({ behavior: "smooth", block: "start" });
const classes = useStyles();
If this didn't help, please post your code.

It did not work :(
Here is the PlaceDetails code:
import React from 'react';
import { Box, Typography, Button, Card, CardMedia, CardContent, CardActions, Chip } from '@material-ui/core';
import LocationOnIcon from '@material-ui/icons/LocationOn';
import PhoneIcon from '@material-ui/icons/Phone';
import Rating from '@material-ui/lab/Rating';
import useStyles from './styles.js';
const PlaceDetails = ({ place, selected, refProp }) => {
if (selected) refProp?.current?.scrollIntoView({ behavior: 'smooth', block: 'start' });
const classes = useStyles();
return (
    <Card elevation={6}>
        <CardMedia
            style={{ height: 350 }}
            image={place.photo ? place.photo.images.large.url : 'https://www.foodserviceandhospitality.com/wp-content/uploads/2016/09/Restaurant-Placeholder-001.jpg'}
            title={place.name}
        />
        <CardContent>
            <Typography gutterBottom variant="h5">{place.name}</Typography>
            <Box display="flex" justifyContent="space-between" my={2}>
                <Rating name="read-only" value={Number(place.rating)} readOnly />
                <Typography component="legend">{place.num_reviews} review{place.num_reviews > 1 && 's'}</Typography>
            </Box>
            <Box display="flex" justifyContent="space-between">
                <Typography component="legend">Price</Typography>
                <Typography gutterBottom variant="subtitle1">
                    {place.price_level}
                </Typography>
            </Box>
            <Box display="flex" justifyContent="space-between">
                <Typography component="legend">Ranking</Typography>
                <Typography gutterBottom variant="subtitle1">
                    {place.ranking}
                </Typography>
            </Box>
            {place?.awards?.map((award) => (
                <Box display="flex" justifyContent="space-between" my={1} alignItems="center">

                    <Typography variant="subtitle2" color="textSecondary">{award.display_name}</Typography>
                </Box>
            ))}
            {place?.cuisine?.map(({ name }) => (
                <Chip key={name} size="small" label={name} className={classes.chip} />
            ))}
            {place.address && (
                <Typography gutterBottom variant="body2" color="textSecondary" className={classes.subtitle}>
                    <LocationOnIcon />{place.address}
                </Typography>
            )}
            {place.phone && (
                <Typography variant="body2" color="textSecondary" className={classes.spacing}>
                    <PhoneIcon /> {place.phone}
                </Typography>
            )}
        </CardContent>
        <CardActions>
            <Button size="small" color="primary" onClick={() => window.open(place.web_url, '_blank')}>
                Trip Advisor
            </Button>
            <Button size="small" color="primary" onClick={() => window.open(place.website, '_blank')}>
                Website
            </Button>
        </CardActions>
    </Card>
);

};
export default PlaceDetails;

And here is the List code:
import React, { useState, useEffect, createRef } from 'react';
import { CircularProgress, Grid, Typography, InputLabel, MenuItem, FormControl, Select } from '@material-ui/core';

import PlaceDetails from '../PlaceDetails/PlaceDetails';
import useStyles from './styles.js';

const List = ({ places, type, setType, rating, setRating, childClicked, isLoading }) => {
  const [elRefs, setElRefs] = useState([]);
  const classes = useStyles();

  useEffect(() => {
    const refs = Array(places?.length).fill().map((_, i) => elRefs[i] || createRef());

    setElRefs(refs);
  }, [places]);

  return (

      <Typography variant="h4">Food & Dining around you</Typography>
      {isLoading ? (

          <CircularProgress size="5rem" />

      ) : (
        <>
          <FormControl className={classes.formControl}>
            <InputLabel id="type">Type</InputLabel>
            <Select id="type" value={type} onChange={(e) => setType(e.target.value)}>
              <MenuItem value="restaurants">Restaurants</MenuItem>
              <MenuItem value="hotels">Hotels</MenuItem>
              <MenuItem value="attractions">Attractions</MenuItem>
            </Select>
          </FormControl>
          <FormControl className={classes.formControl}>
            <InputLabel id="rating">Rating</InputLabel>
            <Select id="rating" value={rating} onChange={(e) => setRating(e.target.value)}>
              <MenuItem value="">All</MenuItem>
              <MenuItem value="3">Above 3.0</MenuItem>
              <MenuItem value="4">Above 4.0</MenuItem>
              <MenuItem value="4.5">Above 4.5</MenuItem>
            </Select>
          </FormControl>
          <Grid container spacing={3} className={classes.list}>
            {places?.map((place, i) => (
              <Grid ref={elRefs[i]} item key={i} xs={12}>
                <PlaceDetails
                  place={place}
                  selected={Number(childClicked) === i}
                  refProp={elRefs[i]}
                />
              </Grid>
            ))}
          </Grid>
        </>
      )}

  );
};

export default List;


Allow Timestamps to be Written with Roman Numerals

Seriously. Why not?

This can be enabled in Settings > Display with the other timestamp stuff. Closing ticket.


string_to_string has been removed warning

When attempting to implement the standard lints, I received this warning on string_to_string
lint `clippy::string_to_string` has been removed: `using `string::to_string` is common even today and specialization will likely happen soon` `#[warn(renamed_and_removed_lints)]` on by defaultrustc(renamed_and_removed_lints)
If this is believed to be valid, I can open a PR that would remove this lint

🤦 I was on the wrong version of rust, my bad


Pattern: Develop Callout - with data pattern based on the Reboot Services template requirements

User Story

As a [user role below]:
IBM.com Library developer

I need to:
create the Callout - with data

so that I can:
provide the ibm.com adopter developers patterns/components they can use to build ibm.com web pages

Additional information

This pattern will be available behind a feature flag.  Reboot will be doing user testing to decide on final design.
Story within Storybook with corresponding knobs
Utilize Carbon
See the Epic for the Design and Functional specs information
Design QA testing issue (#2824 )
Design specs Box folder

Acceptance criteria

[ ] Include README for the react pattern and corresponding styles
[ ] Designs were provided by Reboot, include Lila as a Reviewer in the Pull Request
[ ] Provide a direct link to the deploy preview for the designer in the Pull Request description
[ ] The Storybook link is added to the Design QA issue for their testing

@RaphaelAmadeu Lila finalized the designs.  I have added the Box folder link to this issue as well as the Epic.


Is it possible to publish result from 2 collections after manipulation

Is there a way to query 2 collection, manipulate with the data and publish the result?
Right now I'm publishing with JoinServer 2 queries from 2 collections and doing manipulation with the data Client Side,  but I want to send already prepared data.

Your publish-function could publish whatever value you want to be published to the client - just return the value and it will be available at the client using the key you published it with.
The data can easily be an array or an object which you define yourself. The examples in the readme file should give you the needed knowledge to get this working.


add hypcap compatibility

Lists hypcap as "currently-incompatible" and adds two test files. The first test shows that hypcap is unnecessary since the tagging code achieves its purpose by moving the anchor to the top of floats. The second test shows it errors if given one of the package options like figure that redefines the float environment.
It's possible "no-support" makes more sense.

@mbertucci47  this and other "currently incompatible" PR will show merge conflicts as I've merged the priority change so unknownandcurrently-incompatibleentries havepriority`.  If you wish to adjust the that's fine but otherwise there is no need: we can fix up the conflict in each case while merging to main, and the conflicts are just "expected adjustments" and don't affect any discussion on the PR.  (I picked this PR at random, this comment could be made on all of them)

@davidcarlisle Whatever is easiest. If you can do them without manual adjustments to each PR then let's go with that


Extract to a git-standup repo

What did I do yesterday?
This repo, but just show all commit messages you've commited.

no need for since or until, but accept a --date parameter
-- default to yesterday

Wait, it's actually this:
https://www.npmjs.com/package/git-standup

And now it's here as well: https://github.com/tomfa/git-commit-standup


Fix bugs in pfnci script

This commit fixes a bug in pfnci script.
In the previous CI, there are two tests performed, tox for pytest and
sphinx build. The CI script fails to combine all the return value
of the test commands, hence the CI status only show the status of the
sphinx build.
In this commit, the return status of both tests are combined and returned.

Please rebase all other PRs to this.


too_many_websockets causes Cannot read properties of undefined (reading 'event')

(Filling out the following with as much detail as you can provide will help us solve your issue sooner.)
@slack/bolt version
3.15.0
Refer to your package.json
Your App and Receiver Configuration
const slackApp = new App({
  token: process.env.SLACK_BOT_TOKEN,
  signingSecret: process.env.SLACK_SIGNING_SECRET,
  socketMode: true,
  appToken: process.env.SLACK_APP_TOKEN,
  port: PORT,
  logLevel: LogLevel.DEBUG
});

Node.js runtime version
v21.7.2
Steps to reproduce:
(Share the commands to run, source code, and project settings)
I just added a statement about event.user === plan.operator in current bolt app as below:
slackApp.event('reaction_added', async ({ event, client }) => {
  if (event.reaction === CHECKED_REACTION && event.user === object.operator)

It caused
[DEBUG]  socket-mode:SocketModeClient:0 Received a message on the WebSocket: {"type":"disconnect","reason":"too_many_websockets","debug_info":{"host":"applink-8"}}
[ERROR]   An unhandled error occurred while Bolt processed (type: undefined, error: TypeError: Cannot read properties of undefined (reading 'event'))

slackApp.event('reaction_added', async ({ event, client }) => {
  if (event.reaction === CHECKED_REACTION && event!.user === object.operator)

But if I apply with event!, no such error for undefined (reading 'event') produced nor too_many_websockets issue
Expected result:
(Tell what you expected to happen)
Actual result:
(Tell what actually happened with logs, screenshots)
Requirements
For general questions/issues about Slack API platform or its server-side, could you submit questions at https://my.slack.com/help/requests/new instead. :bow:
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

This should not be an error anymore in bolt v3.18 or newer.

This should not be an error anymore in bolt v3.18 or newer.

Thank you, any description link related to the fixed issue?

Please upgrade the underlying @slack/socket-mode library to this version: https://github.com/slackapi/node-slack-sdk/releases/tag/%40slack%2Fsocket-mode%401.3.5 With the upgrade, this issue should be resolved.

Thank you, I'll upgrade the version!


I use POS system, it has 'cancel' key.  the 'cancel' key send '0x0' keyCode.

please teach me How to bind ‘cancel' ?

Sorry for the delay here, I'm trying to clean up old tickets. This would be a request of the Mousetrap library, where the hotkeys are defined.


wrap shinycaas for sub

crucially related to #96

I think this now belongs in muggle, because all the actual sub-specific settings will be org secrets


Option for or default to using max_stable_version when fetching crate info

Are you experiencing an issue with...
shields.io
🐞 Description
A Rust project I've contributed to, called ucan (and ucan-key-support), recently transitioned from (higher-versioned) alpha releases to a stable 0.1.0 release. However, the badge is not getting updated (https://img.shields.io/crates/v/ucan.svg), as it uses the max_version, per this line of code: https://github.com/badges/shields/blob/master/services/crates/crates-version.service.js#L22. In viewing the crate information via https://crates.io/api/v1/crates/ucan, should it use the max_stable_version if it exists, defaulting to max_version otherwise? Here's what the crate data returns:
{
   "crate":{
      "badges":[],
      "categories":[
         "web-programming",
         "cryptography",
         "encoding"
      ],
      "created_at":"2022-03-11T05:26:14.291552+00:00",
      "description":"Implement UCAN-based authorization with conciseness and ease!",
      "documentation":"https://docs.rs/ucan",
      "downloads":1875,
      "exact_match":false,
      "homepage":"https://github.com/cdata/rs-ucan",
      "id":"ucan",
      "keywords":[
         "jwt",
         "ucan",
         "pki",
         "authz"
      ],
      "links":{
         "owner_team":"/api/v1/crates/ucan/owner_team",
         "owner_user":"/api/v1/crates/ucan/owner_user",
         "owners":"/api/v1/crates/ucan/owners",
         "reverse_dependencies":"/api/v1/crates/ucan/reverse_dependencies",
         "version_downloads":"/api/v1/crates/ucan/downloads",
         "versions":null
      },
      "max_stable_version":"0.1.0",
      "max_version":"0.7.0-alpha.1",
      "name":"ucan",
      "newest_version":"0.1.0",
      "recent_downloads":1032,
      "repository":"https://github.com/cdata/rs-ucan/",
      "updated_at":"2022-11-29T18:39:54.791532+00:00",
      "versions":[
         634470,
         552070,
         525823,
         525094,
         520718,
         519720,
         517394,
         517384,
         514425,
         514412,
         673114
      ]
   }
}

🔗 Link to the badge
https://img.shields.io/crates/v/ucan.svg
https://img.shields.io/crates/v/ucan-key-support.svg
💡 Possible Solution
No response

I'm not necessarily opposed to this - showing stable versions in preference to unstable is our usual behaviour for version badges - but I'd like to understand more first. By normal semver rules both 0.1.0 and 0.7.0-alpha.1 would both be considered 'unstable'

Major version zero (0.y.z) is for initial development. Anything MAY change at any time. The public API SHOULD NOT be considered stable.

Where we have only unstable releases to choose from, we'd usually pick the numerically latest one.
I can't seem to find any docs for it, but how does max_stable_version work? Is it entirely user-determined? Does the rust ecosystem have some concept of "some unstable versions are more stable than others"?
I'm trying to avoid specifically tagging @calebcartwright  into things, but this specific issue might be a good one for you to weigh in on.

@chris48s thanks for taking a gander. So, on crates.io, e.g. https://crates.io/crates/ucan/versions, 0.1.0 is considered stable and not a prerelease (which alpha dictates). It's really stable vs prerelease otherwise. In crates code, max_stable_version is what they actually use as the default version: https://github.com/rust-lang/crates.io/blob/master/app/models/crate.js#L35. They calculate it like so due to prerelease:
hash.max_stable_version = versionNums.find(it => !prerelease(it, { loose: true })) ?? null;

@chris48s, additionally, I think this note hints that crates have moved off max_version directly ~ https://github.com/rust-lang/crates.io/blob/20bbac9f5521cb4888ef63f8464fddb28fd973f5/src/views.rs#L214.

There's been some increased chatter this year on both internals and rfcs/cargo in the Rust ecosystem around some of the nuance and challenges of semver handling, particularly with prerelease versions.
I assume that's indeed being reflected in the different values within the api response we're seeing given the inherently necessary coupling between cargo and crates, and that we should indeed utilize max_stable_version. Will double check with the crates team to confirm though


the EAO results is not so correct

"NOTE we not use gmm to generate low, high, peak value" ------in pysot/toolkit/evaluation/eao_benchmark.py line 19.
The result evaluated by the tookit is not so correct, because the values of "low"、“high "、"peak" is fixed. It should be auto computed by KDE(GMM), because different testing result is corresponding to differente values of "low"、“high " and "peak". I will appreciate it if someone can update the compute code.

Hi @haishibei
Do you know how to determine the values of "low", "high" and "peak" for a specified dataset? Could you please provide a demo about this?
Thank you very much！