Snippets as extra arguments on Twister CLI leads to early CMake error

Describe the bug
Whenever you call Twister with the command line argument --extra-args SNIPPET="<valid-snippet-name>", CMake aborts with the error message zephyr_set(... SCOPE snippets) doesn't exist.. This happens very early on in Zephyr bootstrapping, more specifically when including cmake/modules/snippets.cmake.
The analogous usage for a simple build, on the other hand, does not result in any error message, so both west build -S <valid-snippet-name> and west build -- -DSNIPPET="<valid-snippet-name>" work as described and documented. So far, I myself was not able to find the cause of the non-existent scope for snippets in the case of Twister and need help from experts.
To Reproduce
Here is a more fancy console log:
west twister --verbose --inline-logs --cmake-only \
     --extra-args SNIPPET="cdc-acm-console" \
     --platform nucleo_f767zi \
     --test tests/drivers/uart/uart_async_api/drivers.uart.async_api

…breaks with (CMake output in build.log):
-- Application: /.../workspace-zephyr/zephyr/tests/drivers/uart/uart_async_api
-- CMake version: 3.27.4
-- Found Python3: /.../workspace-zephyr/.env/bin/python (found suitable version "3.8.10", minimum required is "3.8") found components: Interpreter
-- Cache files will be written to: /home/user/.cache/zephyr
-- Zephyr version: 3.4.99 (/.../workspace-zephyr/zephyr)
-- Found west (found suitable version "1.1.0", minimum required is "0.14.0")
-- Board: nucleo_f767zi
CMake Error at zephyr/cmake/modules/extensions.cmake:2764 (message):
  zephyr_set(...  SCOPE snippets) doesn't exists.
Call Stack (most recent call first):
  twister-out/nucleo_f767zi/tests/drivers/uart/uart_async_api/drivers.uart.async_api/zephyr/snippets_generated.cmake:25 (zephyr_set)
  zephyr/cmake/modules/snippets.cmake:94 (include)
  zephyr/cmake/modules/snippets.cmake:121 (zephyr_process_snippets)
  zephyr/cmake/modules/zephyr_default.cmake:129 (include)
  zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:75 (include)
  zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:92 (include_boilerplate)
  zephyr/cmake/package_helper.cmake:115 (find_package)

Expected behavior
The expectation here would be that Twister behaves the same way with snippets as it does when passing a shield stack via extra arguments. Both --extra-args SNIPPET="<valid-snippet-name>" and --extra-args SHIELD="<valid-shield-name>" should not lead to any CMake errors, alone or together.
Impact
Announcement for the community, but showstopper for Bridle. Currently I'm trying to convert a board-specific shield with CPU clock tweaks to a snippet so that the default Zephyr CAN timing test passes.
Logs and console output
See above.
Environment (please complete the following information):

OS: Linux
Toolchain Zephyr SDK 0.16.1
main branch, commit SHA 341590545ac80b7a2abd9902b7f10d577323ccde

Additional context
Non.

Last few days I was playing with the new test case specification required_snippets in our downstream project with similar effect. On downstream we have the bridle/tests/drivers/can/timing/testcase.yaml looks like:
tests:
  drivers.can.timing:
    tags:
      - drivers
      - can
    depends_on: can
    filter: dt_chosen_enabled("zephyr,canbus")
    required_snippets:
      - can-timing-adj

Even the snippet can-timing-adj provides a downstream board specific overlay, details in: https://github.com/tiacsys/bridle/tree/devel/snippets/can-timing-adj/snippets/can-timing-adj
With west twister --cmake-only --alt-config-root bridle/tests/drivers/can/timing --testsuite-root zephyr/tests/drivers/can/timing Twister exits early with following error message:
ERROR   - Can't find snippet 'can-timing-adj' for test 'tests/drivers/can/timing/drivers.can.timing'

That's a result of the Twister test plan logic in apply_filters() about line 839:
if ts.required_snippets:
    missing_snippet = False
    snippet_args = {"snippets": ts.required_snippets}
    found_snippets = snippets.find_snippets_in_roots(snippet_args, [Path(ZEPHYR_BASE), Path(ts.source_dir)])

found_snippets expands here to nothing, because it looks only into the snippets and test spec folder on Zephyr upstream, never on any (well known) downstream repositories, although these are declared in bridle/zephyr/module.yml.

Give https://github.com/zephyrproject-rtos/zephyr/pull/63232 a try

Give #63232 a try

Wow, quick response. Many thanks. Looks like right direction. Twister now finds downstream snippets in modules, too. BUT: now we run into exactly the same error as descript on top of this bug report. Let me give you the sample session.
Twister called by west:
west twister --cmake-only \
             --alt-config-root bridle/tests/drivers/can/timing \
             --testsuite-root zephyr/tests/drivers/can/timing

Exit now with:
INFO    - Using Ninja..
INFO    - Zephyr version: zephyr-v3.4.0-4509-g5f7fee5b9139
INFO    - Using 'zephyr' toolchain.
INFO    - Using alternative configuration from /.../workspace/bridle/tests/drivers/can/timing/testcase.yaml
INFO    - Selecting default platforms per test case
INFO    - Building initial testsuite list...
INFO    - Writing JSON report /.../workspace/twister-out/testplan.json
INFO    - JOBS: 8
INFO    - Adding tasks to the queue...
INFO    - Added initial list of jobs to queue
ERROR   - Cmake build failure: /.../workspace/zephyr/tests/drivers/can/timing for tiac_magpie
ERROR   - tiac_magpie               tests/drivers/can/timing/drivers.can.timing         ERROR : Cmake build failure
ERROR   - see: /.../workspace/twister-out/tiac_magpie/tests/drivers/can/timing/drivers.can.timing/build.log
… … …

Build log is (.../tests/drivers/can/timing/drivers.can.timing/build.log):
Loading Zephyr module(s) (Zephyr base): zephyr_default:dts
-- Application: /.../workspace/zephyr/tests/drivers/can/timing
-- CMake version: 3.27.4
Loading Bridle default modules (Freestanding).
-- Bridle version: 3.4.99 (/.../workspace/bridle)
-- ZEPHYR_TOOLCHAIN_VARIANT: zephyr
-- SDK_VERSION: 0.16.1
-- ZEPHYR_SDK_INSTALL_DIR  : /opt/zephyr-sdk-0.16.1
-- Found Python3: /.../workspace/.env/bin/python (found suitable version "3.8.10", minimum required is "3.8") found components: Interpreter
-- Cache files will be written to: /home/.../.cache/zephyr
-- Zephyr version: 3.4.99 (/.../workspace/zephyr)
-- Found west (found suitable version "1.1.0", minimum required is "0.14.0")
-- Board: tiac_magpie
CMake Error at zephyr/cmake/modules/extensions.cmake:2764 (message):
  zephyr_set(...  SCOPE snippets) doesn't exists.
Call Stack (most recent call first):
  twister-out/tiac_magpie/tests/drivers/can/timing/drivers.can.timing/zephyr/snippets_generated.cmake:27 (zephyr_set)
  zephyr/cmake/modules/snippets.cmake:94 (include)
  zephyr/cmake/modules/snippets.cmake:121 (zephyr_process_snippets)
  zephyr/cmake/modules/zephyr_default.cmake:129 (include)
  zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:75 (include)
  zephyr/share/zephyr-package/cmake/ZephyrConfig.cmake:92 (include_boilerplate)
  zephyr/cmake/package_helper.cmake:115 (find_package)

–> zephyr_set(...  SCOPE snippets) doesn't exists.

Related to https://github.com/zephyrproject-rtos/zephyr/pull/63232#issuecomment-1739387093
cat twister-out/tiac_magpie/tests/drivers/can/timing/drivers.can.timing/build.log:
… … …
-- Board: tiac_magpie
CMake Error at zephyr/cmake/modules/extensions.cmake:2764 (message):
  zephyr_set(...  SCOPE snippets) doesn't exists.
Call Stack (most recent call first):
  twister-out/tiac_magpie/tests/drivers/can/timing/drivers.can.timing/zephyr/snippets_generated.cmake:27 (zephyr_set)
… … …

cat twister-out/tiac_magpie/tests/drivers/can/timing/drivers.can.timing/zephyr/snippets_generated.cmake:
# WARNING. THIS FILE IS AUTO-GENERATED. DO NOT MODIFY!
#
# This file contains build system settings derived from your snippets.
# Its contents are an implementation detail that should not be used outside
# of Zephyr's snippets CMake module.
#
# See the Snippets guide in the Zephyr documentation for more information.

###############################################################################
# Global information about all snippets.

# The name of every snippet that was discovered.
set(SNIPPET_NAMES "can-timing-adj" "cdc-acm-console" "usb-console" "xen_dom0")
# The paths to all the snippet.yml files. One snippet
# can have multiple snippet.yml files.
set(SNIPPET_PATHS "/media/slz/lpndevel/devel/TIAC/workspace/bridle/snippets/can-timing-adj/snippet.yml" "/media/slz/lpndevel/devel/TIAC/workspace/bridle/snippets/usb-console/snippet.yml" "/media/slz/lpndevel/devel/TIAC/workspace/zephyr/snippets/cdc-acm-console/snippet.yml" "/media/slz/lpndevel/devel/TIAC/workspace/zephyr/snippets/xen_dom0/snippet.yml")

# Create variable scope for snippets build variables
zephyr_create_scope(snippets)

###############################################################################
# Snippet 'can-timing-adj'

# Common variable appends.
# Appends for board 'tiac_magpie'
if("${BOARD}" STREQUAL "tiac_magpie")
  zephyr_set(EXTRA_DTC_OVERLAY_FILE "/media/slz/lpndevel/devel/TIAC/workspace/bridle/snippets/can-timing-adj/boards/tiac_magpie.overlay" SCOPE snippets APPEND)
endif()

It seems zephyr_create_scope(snippets) has not the expected effect in case of west twister …?

It seems zephyr_create_scope(snippets) has not the expected effect in case of west twister …?

Cross check with west build … with and w/o PR #63232
west build -b tiac_magpie -S can-timing-adj \
           -d build/tiac_magpie -p always \
           zephyr/tests/drivers/can/timing
west flash -d build/tiac_magpie

Build host output:
-- west build: making build dir /.../workspace/build/tiac_magpie pristine
-- west build: generating a build system
… … …
-- Board: tiac_magpie
-- Snippet(s): can-timing-adj
… … …
-- Found BOARD.dts: /.../workspace/bridle/boards/arm/tiac_magpie/tiac_magpie.dts
-- Found devicetree overlay: /.../workspace/bridle/snippets/can-timing-adj/boards/tiac_magpie.overlay
-- Generated zephyr.dts: /.../workspace/build/tiac_magpie/zephyr/zephyr.dts
-- Generated devicetree_generated.h: /.../workspace/build/tiac_magpie/zephyr/include/generated/devicetree_generated.h
-- Including generated dts.cmake file: /.../workspace/build/tiac_magpie/zephyr/dts.cmake
Parsing /.../workspace/zephyr/Kconfig
Loaded configuration '/.../workspace/bridle/boards/arm/tiac_magpie/tiac_magpie_defconfig'
Merged configuration '/.../workspace/zephyr/tests/drivers/can/timing/prj.conf'
Configuration saved to '/.../workspace/build/tiac_magpie/zephyr/.config'
Kconfig header saved to '/.../workspace/build/tiac_magpie/zephyr/include/generated/autoconf.h'
… … …

Target console output:
*** Booting Zephyr OS build zephyr-v3.4.0-4592-g2205092ae00e ***
Running TESTSUITE can_timing
… … …
===================================================================
START - test_timing
testing bitrate 20000, sample point 87.5% (valid): prop_seg = 0, phase_seg1 = 13, phase_seg2 = 2, prescaler = 150 OK, sample point error 0.0%
testing bitrate 50000, sample point 87.5% (valid): prop_seg = 0, phase_seg1 = 13, phase_seg2 = 2, prescaler = 60 OK, sample point error 0.0%
testing bitrate 125000, sample point 87.5% (valid): prop_seg = 0, phase_seg1 = 13, phase_seg2 = 2, prescaler = 24 OK, sample point error 0.0%
testing bitrate 250000, sample point 87.5% (valid): prop_seg = 0, phase_seg1 = 13, phase_seg2 = 2, prescaler = 12 OK, sample point error 0.0%
testing bitrate 500000, sample point 87.5% (valid): prop_seg = 0, phase_seg1 = 13, phase_seg2 = 2, prescaler = 6 OK, sample point error 0.0%
testing bitrate 800000, sample point 80.0% (valid): prop_seg = 0, phase_seg1 = 15, phase_seg2 = 4, prescaler = 3 OK, sample point error 0.0%
testing bitrate 1000000, sample point 75.0% (valid): prop_seg = 0, phase_seg1 = 11, phase_seg2 = 4, prescaler = 3 OK, sample point error 0.0%
testing bitrate 125000, sample point 90.0% (valid): prop_seg = 0, phase_seg1 = 13, phase_seg2 = 2, prescaler = 24 OK, sample point error 2.5%
testing bitrate 125000, sample point 80.0% (valid): prop_seg = 0, phase_seg1 = 11, phase_seg2 = 4, prescaler = 24 OK, sample point error 5.0%
testing bitrate 125000, sample point 100.0% (invalid): OK
testing bitrate 1000001, sample point 75.0% (invalid): OK
 PASS - test_timing in 0.122 seconds
===================================================================
… … …
===================================================================
TESTSUITE can_timing succeeded

------ TESTSUITE SUMMARY START ------

SUITE PASS - 100.00% [can_timing]: pass = 3, fail = 0, skip = 3, total = 6 duration = 0.127 seconds
 - SKIP - [can_timing.test_set_timing_data_max] duration = 0.001 seconds
 - SKIP - [can_timing.test_set_timing_data_min] duration = 0.001 seconds
 - PASS - [can_timing.test_set_timing_max] duration = 0.001 seconds
 - PASS - [can_timing.test_set_timing_min] duration = 0.001 seconds
 - PASS - [can_timing.test_timing] duration = 0.122 seconds
 - SKIP - [can_timing.test_timing_data] duration = 0.001 seconds

------ TESTSUITE SUMMARY END ------

===================================================================
PROJECT EXECUTION SUCCESSFUL

Result, the autogenerated file snippets_generated.cmake in build folder is exactly the same:
md5sum build/tiac_magpie/zephyr/snippets_generated.cmake \
       twister-out/tiac_magpie/tests/drivers/can/timing/drivers.can.timing/zephyr/snippets_generated.cmake

9c6a6a7cf63cc083a4722094853e4061  build/tiac_magpie/zephyr/snippets_generated.cmake
9c6a6a7cf63cc083a4722094853e4061  twister-out/tiac_magpie/tests/drivers/can/timing/drivers.can.timing/zephyr/snippets_generated.cmake

That leads me to the assertion that zephyr_create_scope(snippets) from zephyr/cmake/modules/extensions.cmake has side effects, if west twister … is the entry point.

Here a last cross check that shows the unexpected different behaviour between west build … and west twister …. This check was done without our 3rd party downstream project – Zephyr main branch only (ddd2b490b006e36c2565ac5d53eda272a7a68488).
Also we do not try to use --extra-args SNIPPET="<valid-snippet-name>", but the official recommended way with the test case spec required_snippets was used. Thus, to reproduce the CMake bug with original UART API test on board nucleo_f767zi we had add the following to lines locally:
diff --git a/tests/drivers/uart/uart_async_api/testcase.yaml b/tests/drivers/uart/uart_async_api/testcase.yaml
index e9914d48d9..8b26a7db84 100644
--- a/tests/drivers/uart/uart_async_api/testcase.yaml
+++ b/tests/drivers/uart/uart_async_api/testcase.yaml
@@ -13,6 +13,8 @@ tests:
     harness_config:
       fixture: gpio_loopback
     depends_on: gpio
+    required_snippets:
+      - cdc-acm-console
   drivers.uart.wide:
     filter: CONFIG_UART_CONSOLE and CONFIG_SERIAL_SUPPORT_ASYNC and not CONFIG_UART_MCUX_LPUART
     harness: ztest

To reproduce the CMake bug, it's enough to run with --cmake-only in both cases, west build … and west twister ….
The good case west build …
west build --cmake-only -p always \
           -b nucleo_f767zi -S cdc-acm-console \
           zephyr/tests/drivers/uart/uart_async_api

results in:
-- west build: generating a build system
… … …
-- Board: nucleo_f767zi
-- Snippet(s): cdc-acm-console
… … …
-- Found devicetree overlay: /.../workspace/zephyr/snippets/cdc-acm-console/cdc-acm-console.overlay
… … …
Merged configuration '/.../workspace/zephyr/snippets/cdc-acm-console/cdc-acm-console.conf'
… … …
-- Build files have been written to: /.../workspace/build

ALL FINE
The buggy path west twister …
Note, the patch (above) was applied:
west twister --verbose --inline-logs \
             --cmake-only --platform nucleo_f767zi \
             --test tests/drivers/uart/uart_async_api/drivers.uart.async_api

results in:
INFO    - Using Ninja..
INFO    - Zephyr version: zephyr-v3.4.0-4625-gddd2b490b006
… … …
ERROR   - Cmake build failure: /.../workspace/zephyr/tests/drivers/uart/uart_async_api for nucleo_f767zi
INFO    - 1/1 nucleo_f767zi             tests/drivers/uart/uart_async_api/drivers.uart.async_api  ERROR Cmake build failure (cmake)
INFO    - /.../workspace/twister-out/nucleo_f767zi/tests/drivers/uart/uart_async_api/drivers.uart.async_api/build.log
ERROR   - Loading Zephyr module(s) (Zephyr base): zephyr_default:kconfig
… … …
-- Board: nucleo_f767zi
CMake Error at zephyr/cmake/modules/extensions.cmake:2764 (message):
  zephyr_set(...  SCOPE snippets) doesn't exists.
… … …

NOT EXPECTED

All this is in serious contradiction to the test for extra_args: SNIPPET="<valid-snippet-name>" that shows no errors: west twister --verbose --inline-logs --testsuite-root zephyr/tests/cmake/snippets.

@rexut could you try this fix: #63372

@rexut could you try this fix: #63372

Great, it fixes the origin of the issue:
west twister --verbose --inline-logs --cmake-only \
     --extra-args SNIPPET="cdc-acm-console" \
     --platform nucleo_f767zi \
     --test tests/drivers/uart/uart_async_api/drivers.uart.async_api

… works now as expected –> zephyr,console = &snippet_cdc_acm_console_uart;
Also, with a local patched zephyr/tests/drivers/uart/uart_async_api/testcase.yaml:
index e9914d48d9..8b26a7db84 100644
--- a/tests/drivers/uart/uart_async_api/testcase.yaml
+++ b/tests/drivers/uart/uart_async_api/testcase.yaml
@@ -13,6 +13,8 @@ tests:
     harness_config:
       fixture: gpio_loopback
     depends_on: gpio
+    required_snippets:
+      - cdc-acm-console
   drivers.uart.wide:
     filter: CONFIG_UART_CONSOLE and CONFIG_SERIAL_SUPPORT_ASYNC and not CONFIG_UART_MCUX_LPUART
     harness: ztest

… and simple call:
west twister --cmake-only \
     --platform nucleo_f767zi \
     --test tests/drivers/uart/uart_async_api/drivers.uart.async_api

–> zephyr,console = &snippet_cdc_acm_console_uart;


Use DC manager instead of configuring the test setup manually

Use antidote_dc_manager to setup systests
Removed unused code in test_utils
Only create a ring and connect dcs if nodes are started for the first time (can be removed once #396 is fixed)

The logging_vnode function set_sync_log(Value) is not called anymore after changing to the antidote_dc_manager calls.

Branch is ready to be merged into master.


CI hardware check doesn't fail on hardfault

Describe the bug
if during the Hardware tests there is a hardfault, the test pass and doesn't fail.
To Reproduce
for example, this  and this tests has been failed but the test passed.
from the tests done on https://github.com/PX4/Firmware/pull/14097, all the nxp tests fails since  commit 156cf16 there is hardfault.
Expected behavior
Fail on hardware fault, for sure.
the hardware test is great tool, but if it really check the CPU load, driver proper initializations, RAM and of course hardfault.
I think that in addition to the tests there should be some addons such scrapping the shell output, looking for ERROR/FAIL from any module that might send such message.
maybe analyze the top, for any issue, and so on.
Ideas?
I opened this thread to enable sharing ideas, or maybe on-going-work that is already done on this issue

fixed on CI - exit from failed hardware check with error, so the check will fail #14215


Tpay payment methods only for orders with PLN currency

At the moment, the plugin supports payments in PLN currency only. However, if the order is in another currency, such as USD, it's still possible to select one of Tpay's payment methods.
The proposed solution is to display Tpay payment methods at checkout only if the order is in PLN currency. Additionally, we should prevent the completion of an order when the Tpay payment method is selected if the currency is not PLN.

We won't support other currencies for now, as new Tpay gateway supports PLN only. I'm closing it for now, to reopen once it will be possibl


Fixed th Navigation Bar Highlight and Alignment Issues in FAQ Section

Related Issue
None
Description
As mentioned in issue #862, I have solved the highlight and alignment issue please go through it and if require any changes kindly let me know.
Type of PR

[x] Bug fix
[x] Feature enhancement
[ ] Documentation update
[ ] Other (specify): _______________

Screenshots / videos (if applicable)
Before:

After:

Checklist:

[x] I have performed a self-review of my code
[x] I have read and followed the Contribution Guidelines.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers, screenshots, and videos after making the changes.
[x] I have commented my code, particularly in hard-to-understand areas.

Additional context:
Please label it under gssoc with level.

@Suchitra-Sahoo If possible please give level 2.

Sorry i can't give you level2 tag based on the amount of changes you have made.


Fixes and documentation

This PR moves few private keys or secrets to ENV file where they should be and also integrates the base code required for testing.

utility.Helper basically mock Functions for testing ?


Spectre.Console doesn't respect Microsoft.Extensions.DependencyInjection singletons

Information

OS: MacOS
Version: 0.49.1
Terminal: iTerms

Describe the bug
When dependency injecting a service registered as a singleton into my command, the instance of the service isn't the same as the singleton. Spectre seems to instantiate a new instance of injected parameters outside of the ServiceCollection.
To Reproduce
I've created this example repo: https://github.com/macdonaldr93/spectre-di-debug
It's a minimalist implementation, so you should be able to read all of the code. The main two spots are:

HashCode output from the HostedService: https://github.com/macdonaldr93/spectre-di-debug/blob/main/SpectreDI/Program.cs#L23
HashCode output from the Command: https://github.com/macdonaldr93/spectre-di-debug/blob/main/SpectreDI/Commands/ExampleCommand.cs#L22

You can run the console app with:
dotnet run --project SpectreDI example

and this will output:
HostedService->IExampleService hash code: 1816341
info: Microsoft.Hosting.Lifetime[0]
      Application started.
Additional context
The real use case is that we have a background authentication hosted service. We want this available as a singleton when the command runs because it knows about the auth token.

Please upvote :+1: this issue if you are interested in it.

I'm sorry, but this is not a minimal reproducable example, this is a whole framework. You will need to reduce the scope a bit, perhaps create an example using just the TypeRegistrar and TypeResolver and show us what is wrong.

I actually had a few seconds to look at this and I've been trying to wrap my head around DI so I thought I'd dig in some more. Oo me this seems like just a misunderstanding of how MEDI is gonna work along with the TypeRegistrar you've written. That Registrar builds its own service provider, as does hosted service. So the code relating to DI is no different than this which has a similiar output.
var services = new ServiceCollection();
services.AddSingleton<IExampleService, ExampleService>();

var provider1 = services.BuildServiceProvider();
var exampleService1 = provider1.GetRequiredService<IExampleService>();
Console.WriteLine(exampleService1.GetHashCode());

var provider2 = services.BuildServiceProvider();
var exampleService2 = provider2.GetRequiredService<IExampleService>();
Console.WriteLine(exampleService2.GetHashCode());

That being said, this probably falls under #1657 as first class Microsoft.Extensions.DependencyInjection would probably allow Spectre.Console to play nicely with whatever HostedService is doing to grab that singular service provider

@phil-scott-78 that's super helpful context! You're right, I assumed that Build would only be called once here https://github.com/macdonaldr93/spectre-di-debug/blob/main/SpectreDI/Program.cs#L35.
It looks like https://github.com/spectreconsole/spectre.console/issues/1657 already captures our use case. I'm happy to close this and keep an eye on that issue. Our example just gives you another use case of how folks are attempting to make this work.
In the meantime, do you know of any workarounds? We'll internally work on it, but if there's someone who's already solved this, it would help a lot. From what I've seen online, most folks take a similar approach to us. @patriksvensson just realizing now that your repo (https://github.com/patriksvensson/spectre.console-di-sample/blob/main/Infrastructure/TypeRegistrar.cs) was the one I had originally referenced. At the time I didn't realize you were an author on this.


Build fails missing libisl when /usr/local is blocked from sandbox access on MacOS x86_64

Observed behavior
Build fails on MacOS, complaining of libisl not being available under /usr/local when --sandbox_block_path=/usr/local is provided to build command.  See https://gist.github.com/akesling/2b2d059cd6fc1e5257026e562b7f933a for build error output.
Note that --sandbox_block_path=/usr/local helps mitigate non-hermetic library inclusion on MacOS.  See https://github.com/bazelbuild/bazel/issues/10472#issuecomment-6785287 for context.
Reproduction
See musl-isl-repro branch in reproduction repo for a full reproduction.
Steps

Include musl-toolchain in WORKSPACE
Add build --sandbox_block_path=/usr/local to .bazelrc
Configure a build transition for @io_bazel_rules_go//go/toolchain:linux_amd64_cgo
Build cross-compiling target

This was fixed in https://github.com/bazel-contrib/musl-toolchain/pull/23 and released in 0.1.13 :)


Cluster-Autoscaler: fix logging in scale down

This will have to be cherry-picked to 1.5.
cc: @fgrzadkowski @piosz @jszczepkowski

This change is

LGTM


Do delete work?

Elasticsearch-dump 3.1.0
Elasticsearch 5.2.2
Command:
./bin/elasticdump --httpAuthFile /tmp/auth.tmp   --format=human  --input=http://<IP_ADDRESS>:9200/logstash-*-nginx-access-live   --output=jumia.json  --searchBody ' {  "query": {  "bool": { "must": [ { "query_string": { "query": "create AND  mobapi", "analyze_wildcard": true } } ], "must_not": []  }  } } ' --limit 1000 --delete=true
I run this, i got about 380k events, all info is  perfect... but the delete didn't happen, i can still search for that same query in kibana and the 380k hits
So i'm doing something wrong or --delete is not working in this ES version or for file output?
Thanks for the help and for elasticsearch-dump

Use --delete, not --delete=true. Please re-open if this is still an issue.


v3.1.11 commits and release notes

Patch release with two small build fixes/features:

Add explicit std:: namespace for isfinite in ImathFun.cpp
Add uninstall target

This release also introduces the practice of signing release artifacts via sigstore.

Needs cmake 3.14

I've removed the uninstall target and the resulting need for cmake 3.14, which will wait for the next minor release.


Remove the notion of FencedPropertySet

Currently, the limitation for transformation passes to write the property set is too strong. In particular, EnlargeWithAncilla needs to write both the DAG and property_set[layout] to keep them "in sync". If in the future we remove named qubits from the DAG, this feature can be include back. For now, I think it needs to be removed.
This PR borrows some code from https://github.com/Qiskit/qiskit-terra/pull/2007

I don't think we need to do this anymore. In #2157 I updated the passes so that the analysis pass only updates the layout and the transformation pass only updates the DAG. So I think we should still keep the properties fenced for transformation passes, as this creates a much better isolation of responsibilities among passes. Closing, but feel free to reopen if you disagree.


Text gets cut off in the Welcome to Dev Home Page

Dev Home version
0.401.226.0
Windows build number
10.0.25948.1000
Other software
OS Build Version: 10.0.25948.1000.amd64fre.rs_we_adept.230905-1700
.NET Version: .NET 6.0.20
Steps to reproduce the bug
When horizontally shrinking the Dev Home window, the header text and the corresponding description get cut-off.
Expected result
The text should be stacked (i.e. the size of the textbox changes, but the text itself is still visible)
Actual result
Some of the text is cut off.

Included System Information
Physical Memory: 47.71GB (18.61GB free)
Processor Architecture: x64
Included Extensions Information
Extensions:
Microsoft.Windows.DevHome_0.401.226.0_x64__8wekyb3d8bbwe
Microsoft.Windows.DevHomeGitHubExtension_0.400.226.0_x64__8wekyb3d8bbwe
Microsoft.Windows.DevHome.Dev_<IP_ADDRESS>_x64__8wekyb3d8bbwe

Closed by #1512


Issue with WebUI

Hello,
I've followed this tutorial https://github.com/herlesupreeth/docker_open5gs and deployed the script "sa-deploy.yaml" where all containers are running with the exception of webUI, that exits automatically.
Do you have any suggestion on how to solve this issue?
Thanks in advance,
Best regards,
João Silva

It could be related to this issue in open5gs - https://github.com/open5gs/open5gs/issues/1824
Most likely be fixed in upcoming commits

Is fixed in open5gs repo so closing this


Validate env support dynamic ssh port

What changes are proposed in this pull request?
Please outline the changes and how this PR fixes the issue.
Why are the changes needed?
Please clarify why the changes are needed. For instance,

If you propose a new API, clarify the use case for a new API.
If you fix a bug, describe the bug.

Does this PR introduce any user facing changes?
Please list the user-facing changes introduced by your change, including

change in user-facing APIs
addition or removal of property keys
webui

@alluxio-bot check


Merz - Overlaping combs produce "knockout" effect instead of an expected layering effect

When combs overlap they make a white area, rather than a layered effect, which makes it hard to read what is going on with a curve. @RafalBuchner mentions on Discord this is an issue trying to get all the curves to work on one layer with Merz. Maybe @typesupply could offer a suggestion.

Can you give me a link to the code that generates these shapes? I have some ideas but I need to test.

I read through the code and I think I see what is happening.
There is one pen to which all of the data is drawn:
https://github.com/RafalBuchner/curvatureVisualizator/blob/main/source/code/curvatureVisualizator/curvatureVisualizatorSubscriber.py#L180
That data is then rendered to a single Path layer:
https://github.com/RafalBuchner/curvatureVisualizator/blob/main/source/code/curvatureVisualizator/curvatureGlyph_merz.py#L322
The knockout is probably happening because the data is on one CGPath (a Quartz Bezier path) and the default winding rule is even/odd or whatever. There are a couple of ways you could resolve this:

Change the winding rule on your Path layer to the other option. pathLayer.getCALayer().setFillRule_(Quartz.kCAFillRuleEvenOdd) (untested code) I don't think this ill solve the problem, but it's an easy thing to try so I'd try it.
Draw all paths in the same direction when creating the data for the Path layer.
Draw each comb to a different Shape layer.

My hunch is that option 3 is going to give you the desired result. However, you will need to be mindful of the speed and create/destroy layers at select times. Looking at your code quickly, I would strategically do this in anywhere you currently call self.drawPath() except glyphEditorDidMouseDrag. Frequently recurring methods need to execute quickly and adding/removing layers is expensive.
If you'd like, we could do a discussion on how to optimize the Merz display of this kind of data on Discord (in the Merz channel). There are a few possible strategies and this would be a great example for people to learn from. I don't think I've written about these anywhere yet so I'd be happy to work through it there.

References:

https://developer.apple.com/documentation/quartzcore/cashapelayer/1522146-fillrule?language=objc
https://developer.apple.com/documentation/quartzcore/cashapelayer/shape_fill_mode_values?language=objc
https://developer.apple.com/library/archive/documentation/Cocoa/Conceptual/CocoaDrawingGuide/Paths/Paths.html#//apple_ref/doc/uid/TP40003290-CH206-BAJIJJGD

Thanks! I will check it soon, lateley I'm busy


How to create two charts sharing the same x axis

Hello, how can I make that two verticaly stacked charts share the same x axis, so when I hover one chart y can see the values in the same x axis on the other chart. The width of the charts is the same.
Something like this buth with the share x axis issue would be great:

Thank you!!

Hi. I created an example that might help: jsbin.

vertically stacked: There is an option to an axis to have the data stacked, see here: docs
hover-thing: There is an option for the tooltip-mode called 'index', this will show all data at the index you are hovering on. See docs.

To conclude: you add more datasets to have more data in the way you described on your chart. See JSBin linked above for example on that.
Hope this helps!
Also: take a look at guidelines, this question might be better off on slack for example.
Cheers

Wait, just realised I might have read your question a bit too fast. You do not want a single chart, but multiple charts? And when something is being hovered on one chart it should do the same on the others?

See the linked charts example: https://chartjs-plugin-crosshair.netlify.com/samples/

See the linked charts example: https://chartjs-plugin-crosshair.netlify.com/samples/

updated link: https://chartjs-plugin-crosshair.netlify.app/samples/


Fail test if conversation is empty

If conversation is empty it isn't testing anything, so it should fail.
This change should help to understand issue caused by using \n on Windows easier.
It is very confusing for new users like me.

Hi, this is basically a very fine addition.
Would you mind to rewrite your code to use the "readConvoReject(err)"-Callback instead of throwing an exception - just to follow the general coding style.

That exception is caught 3 lines down.

Solved with latest commit! Thanks for your input!


#161832708 Add endpoint for getting users

What does this PR do?
This PR enables a user to view all users
Description of Task to be completed?
Create an endpoint for getting all users and an individual user
How should this be manually tested?
Clone this repository using git clone https://github.com/Paulstar200/Store-Manager-API-V2.git
Create a database on your local machine called store_manager_api
Switch to the branch ft-get-users-161832708 by doing git checkout ft-get-users-161832708
On your terminal, install a virtual environment using pip install virtualenv
Create a virtual environment using python -m virtualenv env
Activate the virtual environment using env\scripts\activate for windows
For Mac use env/bin/activate
On your terminal install the dependencies using pip install -r requirements.txt
Do set FLASK_APP=run.py
Run the app in your terminal using python run.py or flask run
Copy the localhost link and test it using Postman.
use localhost or the heroku link https://storemanagerapi2.herokuapp.com/ to test
Login first using the endpoint https://storemanagerapi2.herokuapp.com/api/v2/auth/login
Use the following {             "email"<EMAIL_ADDRESS>             "password": "manu2012" }
Use the following endpoint to get the users https://storemanagerapi2.herokuapp.com/api/v2/users
Any background context you want to provide?
Ensure the setup of the database is complete before you run the application
What are the relevant pivotal tracker stories?
#161832708
Screenshots (if appropriate)
None
Questions:
Feel free to ask any questions

Pull Request Test Coverage Report for Build 157

42 of 43 (97.67%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+1.2%) to 81.524%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

app/api/V2/models.py
21
22
95.45%

Totals

Change from base Build 155:
1.2%

Covered Lines:
503

Relevant Lines:
617

💛  - Coveralls


Add 8ball command

Adds an 8ball command which takes a query and returns a magic 8 ball response, with the color of the embed representing the nature of the answer given.

noi

no

yes, very bad. I see a couple issues just glancing at it. pls remove.

Code should be cleaned lol. Why are you initializing the array EVERY TIME the command is ran?  Why are you initializing the array EVERY TIME the command is ran? Also some brackets are there that don't need to be like in the exception handling (which should be two lines)

How do I like init the array only once lol

There's superior commands to look at.

There's superior commands to look at.

banned

no

lmao

WHY did you initialize the array in Bot (which you didn't push)
also namespaces...


Fix error checking when opening files.

We used to have Errno() take a bool, and return an error if it was
false.  Now, we have Errno() take an int, and return an error if that
int is < 0.  However, we still appear to have a few cases floating
around where it is passed a bool.  Since a bool is either 0 or 1, this
means we never return an error. :(
Fixed in two places, and updated comment.

lgtm


Python 3.8.1 virtualenv installation problem

Expected behaviour
In an virtualenv the pip install is not working neither the precompiled wheel installation

create a virtualenv venv
run

pip install opencv-python

I did expect the library to install successfully.
Actual behaviour
Error with:
Collecting opencv-python
  Could not find a version that satisfies the requirement opencv-python (from versions: )
No matching distribution found for opencv-python

I did expect the library to install. In a virtualenv with Python 3.7.4 it works.
Steps to reproduce

Create a virtual environment
try to execute pip install in that environment:
example code:
pip install opencv-python
operating system: Windows 10
architecture: x64
opencv-python version: 4.2.0

Further infos:
To see the supported wheel tags inside my Python 3.8.1 virtualenv i ran:
<path_to_my_repository>\venv\Scripts> .\python.exe
Python 3.8.1 (tags/v3.8.1:1b293b6, Dec 18 2019, 23:11:46) [MSC v.1916 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> from setuptools.pep425tags import get_supported
>>>
>>> for t in get_supported():
...     print(str(t))
...
('cp38', 'cp38m', 'win_amd64')
('cp38', 'none', 'win_amd64')
('py3', 'none', 'win_amd64')
('cp38', 'none', 'any')
('cp3', 'none', 'any')
('py38', 'none', 'any')
('py3', 'none', 'any')
('py37', 'none', 'any')
('py36', 'none', 'any')
('py35', 'none', 'any')
('py34', 'none', 'any')
('py33', 'none', 'any')
('py32', 'none', 'any')
('py31', 'none', 'any')
('py30', 'none', 'any')

I noticed that the naming of the wheel file changed:

opencv_python-4.2.0-cp37-cp37m-win_amd64.whl
compared to:
opencv_python-4.2.0-cp38-cp38-win_amd64.whl

But i am not sure how this correlates to the supported wheel tags.
Anyway i am using python 3.7.4 for now. I just wanted to let u guys know.

I cannot replicate this. Just downloaded Python 3.8.1 official installer and tested:
PS C:\Users\> py -3.8
Python 3.8.1 (tags/v3.8.1:1b293b6, Dec 18 2019, 23:11:46) [MSC v.1916 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> exit()
PS C:\Users\> py -3.8 -m venv test
PS C:\Users\> cd .\test\
PS C:\Users\\test> .\Scripts\Activate.ps1
(test) PS C:\Users\\test> pip install opencv-python
Collecting opencv-python
  Downloading https://files.pythonhosted.org/packages/4a/58/58513225ca637292ae5aada194e5c641f891218145e94f3a59b15c823e75/opencv_python-<IP_ADDRESS>-cp38-cp38-win_amd64.whl (33.1MB)
     |████████████████████████████████| 33.1MB 6.4MB/s
Collecting numpy>=1.17.3 (from opencv-python)
  Downloading https://files.pythonhosted.org/packages/95/47/ea0ae5a778aae07ede486f3dc7cd4b788dc53e11b01a17251b020f76a01d/numpy-1.18.1-cp38-cp38-win_amd64.whl (12.8MB)
     |████████████████████████████████| 12.8MB 3.2MB/s
Installing collected packages: numpy, opencv-python
Successfully installed numpy-1.18.1 opencv-python-<IP_ADDRESS>

@hOPFEN1 Make sure that your version of pip is updated. I encountered this exact same issue, but it was resolved upon upgrading pip in my venv.


Support yazi flavors

Hi, here's an idea.

Flavors (BETA)
The "flavor" is a pre-made Yazi theme, while what we typically refer to as a "theme" is the user's own theme, i.e. ~/.config/yazi/theme.toml file.
The purpose of separating them is to allow users to customize their preferences more conveniently on top of an existing flavor, without having to modify those flavor files. This makes it easier to update, as there won't be conflicts when pulling from Git.
Behind the scenes, Yazi merges the user's theme.toml with the flavor's flavor.toml automatically, and the user's always takes precedence over the flavor.
https://yazi-rs.github.io/docs/flavors/overview

If you'd welcome a PR about this, please let me know and I can maybe contribute it.

Duplicate of https://github.com/catppuccin/yazi/issues/1.


[T3A2][F12-A2]Lim Huan Hock

@huanhock Accepting this PR without a review because another of your PRs for this tutorial was reviewed in detail. Do follow the PR instructions of adding Ready to review to indicate that you are done with this activity.


Update ProgressIndicator.java

package name was wrong in java file.

Thanks @saaksshi.

@johncblandii WC

It has 1 more mistake in iOS I'll update it today.
then you can merge it.

That works for me. Tag me when you do.

Hi I have updated iOS code also.
Please have a look @ it.
On Thu, Aug 13, 2015 at 1:23 PM, John C. Bland II<EMAIL_ADDRESS>wrote:

That works for me. Tag me when you do.
—
Reply to this email directly or view it on GitHub
https://github.com/johncblandii/cordova-progressIndicator/pull/1#issuecomment-130568158
.

--
Regards,
Saaksshi Tyagi https://in.linkedin.com/in/saaksshityagi


Dropped body part on Linux w/ Dahua

Hi!
I have been testing alarm server with Dahua model ITC415-PW6M-IZ-C2, which has cgi-bin/snapManager.cgi API route for event management.
On Windows 10, go version 1.19.2 everything works like a charm.
On Linux Ubuntu Server, go version 1.19.3 the function part.Read from lib multipart reads out only about a half of data from response body. The other half of byte array is filled with zeros.
The issue only appears for bigger payloads (i.e. 3925 bytes, the n value from calling part.Read is only 1281).
Could you please give some advice how to address this issue?
Thank you in advance.

Managed to get full request body by using io.ReadFull in function readEvents of server.go in dahua folder as so:
_, err = io.ReadFull(part,body) instead of _, err = part.Read(body).


Configurable CNI support

This will allow people to launch tasks under a CNI network if they don't want to use the host's network stack.

Can one of the admins verify this patch?

@mesosphere-ci: retest this please

@mesosphere-ci: retest this please

BTW this bug causes issues with running everything (including the framework itself) in a CNI network https://dcosjira.atlassian.net/browse/DCOS-429
In our case we deploy the framework using Marathon and we will be putting the framework in our CNI network via the ipAddress JSON that you can include. We also use a bunch of DNS entries for some of the configs like the seeds URL which is how we noticed that DNS was returning host IPs and causing most of the communication to break.

Apart from minor comments, the only real question from me is, is this backwards compatible with Mesos 0.28 clusters?  We spoke on Slack and are verifying this.

@aaronjwood:
WARN  [2016-10-14 18:10:52,454] org.eclipse.jetty.util.component.AbstractLifeCycle: FAILED com.mesosphere.dcos.cassandra.scheduler.CassandraScheduler@7cb2651f: java.lang.UnsatisfiedLinkError: Mesos JAR version 1.0.0 is not backwards compatible with Mesos native library version 0.28.1
! java.lang.UnsatisfiedLinkError: Mesos JAR version 1.0.0 is not backwards compatible with Mesos native library version 0.28.1

@gabrielhartmann is there a way to work around this? I need 1.0.0 to be able to set the network name in NetworkInfo.

I found a workaround for the DNS issue. Using cassandra-framework.mesos instead of cassandra-dev-cassandra-framework.marathon.mesos will give me the container IP.
Looks like I still need to fix some issues with seeds not being detected. https://github.com/mesosphere/dcos-cassandra-service/blob/master/cassandra-scheduler/src/main/java/com/mesosphere/dcos/cassandra/scheduler/seeds/SeedsManager.java#L149 shows 0 when having the framework and the tasks it launches under CNI.

We spoke about this on Slack but posting this here in case this helps anyone else in the future:
https://github.com/mesosphere/dcos-cassandra-service/blob/master/cassandra-scheduler/src/main/java/com/mesosphere/dcos/cassandra/scheduler/seeds/SeedsManager.java#L151
The host name is "" when using CNI which is causing the issue.

@gabrielhartmann I've fixed the gossip issue by supporting CNI IPs for the seeds. Can you take a look at my changes when you have a chance and let me know what you think?

@mesosphere-ci: retest this please

@mesosphere-ci: retest this please

retest this please

retest this please (was just a network flake during the gradle build)

Is there something I need to fix? I don't have access to the Jenkins server.

Currently failing to build with:
commons/src/test/java/com/mesosphere/dcos/cassandra/common/tasks/CassandraDaemonTaskTest.java:191: error: no suitable method found for create(String,List<String>,double,int,int,int,String,URI,URI,URI)
        ExecutorConfig updatedTestExecutorConfig = ExecutorConfig.create(
                                                                 ^
    method ExecutorConfig.create(String,List<String>,double,int,int,int,String,String,String,URI,URI,URI) is not applicable
      (actual and formal argument lists differ in length)
    method ExecutorConfig.create(String,List<String>,double,int,int,int,String,String,String,String,String,String,boolean) is not applicable
      (actual and formal argument lists differ in length)
1 error

Should be able to repro locally with ./gradlew check (from the root of the repo)

Thanks! Looks like I missed this one, it should be fixed now.

@mesosphere-ci ok to test

Ok, should just be waiting for the mesos lib packaging in PR #338 at this point

Awesome! Thanks for the update :)

Could someone tell me what the new errors are?

retest this please

retest this please

test_cpus_increase_slightly failed in test:rootdisk, but test:mountdisk succeeded, so it's likely a flaky test


Unable to use different CAs for server / client ingress mTLS

Bug description
I am trying to build on the secure ingress docs and configure Istio to use a different CA for client verification than that which signed the server cert. Istio does not seem to support this.
Specifically, I have an ingress gateway with a server block like so:
...
  - hosts:
    - 'foo.dev'
    port:
      name: https
      number: 443
      protocol: HTTPS
    tls:
      mode: MUTUAL
      credentialName: foo-tls

foo-tls is created by cert-manager and has the fields tls.crt, tls.key, and ca.crt. ca.crt is not the ca.crt I want to use for clients -- it is the cert of the CA which signed tls.crt.
I figured, then, that I could put the alternate CA crt at foo-tls-cacert, as the docs indicate. However, even when I do that, istio still prefers the ca.crt in foo-tls. It is only once I delete the ca.crt from foo-tls that foo-tls-cacert is used and my client certs are accepted.
I would expect the more specific foo-tls-cacert to override the ca.crt in foo-tls. Is there a way to support a separate CA cert for ingress clients without mangling cert-manager's output?
[ ] Docs
[ ] Installation
[x] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
[ ] Upgrade
Expected behavior
Most simply, that istio would prefer  foo-tls-cacert over the ca.crt field within foo-tls for client auth. More generally, that istio would support specifying another CA for client auth that played nice with cert-manager.
Steps to reproduce the bug

Create the ingress using MUTUAL and credentialName: foo-tls, where foo-tls has the fields tls.crt, tls.key and ca.crt, where ca.crt is the cert of the CA that signed tls.crt -- this mirrors cert-manager's output.
Put the ca.crt that we want to verify client certs at foo-tls-cacert in the field cacert
Try using client certs signed by foo-tls-cacert -- confirm failure
Run kubectl patch secret -n istio-system foo-tls --type=json -p='[{"op": "remove", "path": "/data/ca.crt"}]' to delete the obscuring ca.crt -- now the curl works.

Version (include the output of istioctl version --remote and kubectl version --short and helm version --short if you used Helm)
$ istioctl version --remote
client version: 1.8.1
control plane version: 1.8.1
data plane version: 1.8.1 (1 proxies)
$ kubectl version --short
Client Version: v1.19.3
Server Version: v1.19.1

How was Istio installed?
Via istioctl
Environment where the bug was observed (cloud vendor, OS, etc)
Local env on kind

I agree this is the desired behavior, and it turns out this actually was the behavior prior to a change we made in Istio 1.8. It also turns out the old behavior was sort of broken as well, if you removed the -cacerts Secret it would not fallback to the main Secret - this was already fixed by the change in 1.8

I agree this is the desired behavior, and it turns out this actually was the behavior prior to a change we made in Istio 1.8. It also turns out the old behavior was sort of broken as well, if you removed the -cacerts Secret it would not fallback to the main Secret - this was already fixed by the change in 1.8

Awesome, thanks @howardjohn.
Qq while I've got you -- do you happen to know if the name from an ingress client cert should be visible to the authorization policies in source.principals? There is no mention of it in the docs or anything, but it seems like it should be feasible.

Awesome, thanks @howardjohn.
Qq while I've got you -- do you happen to know if the name from an ingress client cert should be visible to the authorization policies in source.principals? There is no mention of it in the docs or anything, but it seems like it should be feasible.


Validate ~/ in void local functions and @functions results in error.

Title says it all.
Verifying https://github.com/aspnet/AspNetCore/issues/10734

Verified


Yandex icon from all-the-icons replaces cyrillic letter Я

What did you expect to happen?
When I type a ‘Я’, I expect to see the glyph from my font.
What actually happened?
Instead, I get the Yandex logo from file-icons.ttf from all-the-icons. This is also referenced (with a screenshot) in domtronn/all-the-icons.el#299 (but only seems valid in Doom).
Describe your attempts to resolve the issue
I disabled all-the-icons in packages.el, the issue went away.
It seems it's related to this bit in core-ui.el:
(add-hook! 'after-setting-font-hook
    (defun doom-init-all-the-icons-fonts-h ()
      (when (fboundp 'set-fontset-font)
        (dolist (font (list "Weather Icons"
                            "github-octicons"
                            "FontAwesome"
                            "all-the-icons"
                            "file-icons"
                            "Material Icons"))
          (set-fontset-font t 'unicode font nil 'append)))))

after which file-icons takes precedence over the normal font.
Steps to reproduce

Install Doom
Open Emacs
Put ‘Я’ in any buffer

System Information
https://pastebin.com/1mM60SvB

This must be resolved resolved upstream. Please report it to https://github.com/domtronn/all-the-icons.el. As this is not really a Doom issue, I'll close it.
Otherwise, use
(add-hook! 'after-setting-font-hook :append
  (set-fontset-font t 'unicode X nil 'append))

Where X is any font that will provide the Я glyph you want.

Thank you. Didn’t work, though
(add-hook! 'after-setting-font-hook :append
  (set-fontset-font t 'unicode (font-spec :family "FiraCode Nerd Font" :size 11.0) nil 'prepend))

worked (with 'prepend).


Loading spinner not visualized correctly in ie11

While I was testing my angular web application in internet explorer 11, I noticed that loading spinner is not visualized correctly. In all other browsers it is ok. Here is my code:
<ng4-loading-spinner [timeout]="10000"> </ng4-loading-spinner>
Attached below is the loading spinner.

Dependencies
angular: 6.0.3,
bootstrap: 4.0.0
ng4-loading-spinner: 1.2.3

Me also having the same issue.

Solved this problem by adding this line of code (ie11 target only style):
_:-ms-fullscreen, :root .spinner {   background: rgba(0, 0, 0, 0.99); }


CI-build participant's PR

If non-members / non-contributors of SuperElastix do a pull request we want to be able to build it on Jenkins.
Solution?:
https://wiki.jenkins.io/display/JENKINS/GitHub+pull+request+builder+plugin

Seems to be no issue. PRs #222  #226 by external were build on jenkins.

as @kaspermarstal mentioned, after an update, the build has to be launched manually...


add support for changing the color mode

Allow setting the color mode for images.
This can help on some e-ink displays that use color information to assist with dithering.

Thanks for this PR. I decided to use a grayscale Eink specific lovelace theme for myself, but for all other cases this might be really helpful!


Allow gridModel.emptyText to be set dynamically

It can be useful to customize / update the empty text message displayed by Grid beyond the initial configuration of the GridModel. Currently GridModel.emptyText is provided in the ctor only and is treated as immutable.

IIRC the problem here is that the underlying agGrid does not provide a way to change the emptyText dynamically without remounting a new version of the Grid

Peter -- perhaps you could just verify that this is the case -- assuming it is, I don't think we have any great options

Not to stress out about this feature overly much, but it takes a custom overlay component to render the emptyText.
Could we always provide the grid with a custom component that gets its inner text from the model, and would internally re-render as needed?

I was looking into the custom overlay components, and there seems to be some inconsistency in the ag-grid docs.
This page says the component is called noRowsOverlayRenderer.
This page says it's noRowsOverlayComponent.
I believe the correct name is noRowsOverlayComponent.

PR is here:
https://github.com/xh/hoist-react/pull/1519

👍


Test

Type: Incorrect
Summary:
Test
Details:
Test
Resolution:
Test
Affected definitions:

node-releases 1.1.13

Automatically added versions:

1.1.10
1.1.11
1.1.12

Matching license file(s): package/LICENSE
Matching metadata: registryData.manifest.license: 'MIT'

You can review the change introduced to the full definition at ClearlyDefined.


Feature/pester

Adding pester for unit testing

nifty lol

@ChrisTitusTech The repository might need additional configuration to get actions to run properly on PRs
https://docs.github.com/en/actions/managing-workflow-runs/approving-workflow-runs-from-public-forks

Digging these tests. Let me know when to squash and merge.
Adding these will help some bad PRs that will inevitably coming through.

feel free to merge this in, I think I have enough for a start. It will check the config files, check the gui to ensure they match and test to see if the main script runs with no errors. I have to play around with the button clicks to ensure those work properly.

@DeveloperDurp how can I contact you personally? Linkedin or any other

links are in my profile


code review

chore(format): format files with go fmt
Fix import orders
Avoid type juggling
Fix typo
Fix deep exit
Remove dead code
Fix and format code
refactor code
Refactor to display the Usage only when needed

@IamDaedalus please read commit by commit
It should improve the code quality

@IamDaedalus please read commit by commit
It should improve the code quality

Hi thank you again for taking time for this. I'm currently reading through each commit and the changes. I woke up not too long ago so pardon the time

My pleasure. I discovered cobra library by reviewing your project.


API changes report for Eureka Client

Hi,
I'd like to share my results on API changes analysis of the library: https://abi-laboratory.pro/java/tracker/timeline/eureka-client/
The report is generated by the https://github.com/lvc/japi-compliance-checker tool for jars found at http://central.maven.org/maven2/com/netflix/eureka/eureka-client/ according to https://wiki.eclipse.org/Evolving_Java-based_APIs_2.
Thank you.

Hi @lvc Thank you for the information, this look quite interesting. Appreciate the report.


Fallback mechanism for Property Manager installation

I am creating this issue as per Akamai Tech Support. There was an issue a few week's back when the command  was failing continuously for almost a day (09/23 to be precise) due to NPM error, within ADO pipelines, as follows:
unable to install package manager: npm
ERROR[0018] unable to execute package manager: npm command=install
ERROR[0018] INSTALL error: Unable to install selected package command=install

I am using the recommended way of brew install akamai before installing the Property Manager. So we are not sure if Brew was facing any regressions that given day, as well!
I would like to understand if there is any fallback mechanism with which one could install Property Manager and all similar packages of Akamai (like Onboard, CPS, Cloudlets) from command line just in case  command fails.
Thank you for response, in advance.

Thank you for providing a workaround here. Closing this issue.
Regards,
Sarat


fread slow when consuming a message

Good morning,
For some reason the line $s = @fread($this->stream, $this->frameMax); in AbstractClient::read() just waits for data for some messages being consumed. Bunny is used in a project which communicates with a DNS Management Service by RabbitMQ using the RPC pattern. For some request (retrieving a DNS zone) the consumer from the reply queue waits for a long (10s for example). First I thought that the consumer in the DNS Management was slow, but this puts the reply message onto the reply queue within 25ms. When I look what is going one with the reply queue in RabbitMQ admin panel, I see that the message is consumed, but the state is unacked. The strange thing is, that this only happens for a few zones at a specific environment. Zone A on production is slow, but the same zone on acceptance is fast. The only difference is that the environments are on a different server, but with the same software and configuration. The both use the same RabbitMQ server.
My question is, what can I do to see why the process waits at the fread call? Is there a way to see in RabbitMQ if the message is consumed and the data is send to it?
Hopefully you can give me some pointers. I would appreciate that.
Thank you in advance.
Cheers,
Frank

After some further testing, the delay only happens when the rabbitmq server is not running on localhost, the message is larger than 128kb and the connection to rabbitmq is with SSL.
The delay seems somehow to be related to the heartbeat interval. For example if the heartbeat interval is set to 30 seconds, the delay is 15 seconds. When set to 10 seconds, the delay is 5 seconds.
Hopefully you can find the time to give me some pointers.

One way of looking into those things would be to add metrics, or some other way of observability. Maybe through middleware.
But one of the things I want to change is to fully rely on react/socket and react/stream for I/O instead of using our own. That on it's own might already solve this as we tackled lots of tiny bugs like this there.

Thank you for your reply.
I see that my initial message wasn't clear. The slow fread occurs within Bunny itself. I'm not using my own implementation to consume from the queue.
Here is the exact line in Bunny. https://github.com/jakubkulhan/bunny/blob/e03224f076a21fa386b1da1b29381c3e0acab603/src/Bunny/AbstractClient.php#L298
I'll try to create a small project to reproduce this. Is it perhaps possible for you to look into this?

I'll try to create a small project to reproduce this. Is it perhaps possible for you to look into this?

Currently spending time to work out/on some issues/updates with/for Bunny, this will be one of them.

Great thanks! I appreciate you taking the time!
I've got a POC which shows the delay when using RabbitMQ on an external server with SSL.
After a composer install ofcourse. Run the script consumer.php with the number of bytes as the first argument.
In another terminal run the script rpc-publisher.php. This script will show the time it took to finish reading the message.
I found that the delay occurs if the number of bytes is set to at least 12214.
When you connect to RabbitMQ without SSL, then there is no delay.
The delay seems to be related to the number of seconds for the heartbeat in roc-publisher.php. When this is set for example to 30 seconds, the delay is 14.9 seconds. If it is set to 10 seconds, the delay is 4.9.
I've included a small video which shows this behaviour.
The behaviour is the same for from PHP 7.1 and to 8.0.
Hopefully you can find something.

slow-fread.zip


Bug: CONNECT_UPDATE_PROFILE cannot be found

Actual behavior
CONNECT_UPDATE_PROFILE cannt be found though I added this const to the NativeMethods.txt file.
Expected behavior
It must be found.
Repro steps

NativeMethods.txt content:

WNetCancelConnection2
CONNECT_UPDATE_PROFILE

NativeMethods.json content (if present):

Any of your own code that should be shared?

Context

CsWin32 version: 0.1.647-beta
Win32Metadata version (if explicitly set by project):
Target Framework: .NET 8
LangVersion: 9

CONNECT_UPDATE_PROFILE is a value on an enum in the metadata. When you compile your project, a warning should have been produced telling you to switch asking for CONNECT_UPDATE_PROFILE out for asking for the enum type name itself. Do you see that when you build?

I could not find the enum name. Looking at the docs about WNetCancelConnection2A, I have thought this is defined by #define directive.

When I put CONNECT_UPDATE_PROFILE into my NativeMethods.txt file, I see this warning when building, which tells you the name of the enum to use:

CsWin32Sandbox\NativeMethods.txt(1,1,1,23): warning PInvoke004: Use the name of the enum that declares this constant: NET_USE_CONNECT_FLAGS

How do I know that enum class before building?

You might not know before building, but since building emits a warning that tells you the enum type name, you can then fix it.


💅🏾 Material You implementation!

Material You implementation
Description
Overhauled Home Screen design and implemented Material You across all screens inside the app
Refer #30
Type of Change

Home Screen redesign.
Material You implementation based on Google's Material Design 3 principle.

Update Preview
https://user-images.githubusercontent.com/78308067/198932000-15604553-33cb-4906-a3ea-2623e4fcc3d4.mp4
More Information
Device: OnePlus 9 (LE2115)
OS: OxygenOS 12 (Android 12)
App Version: 1.0.3
Tested by me and @HaikalAkif
Side Note
I had a lot of free time last weekend and tried to redesign the User Interface of this app

Hi. May I ask which Flutter channel you are on? Because in stable, some changes in material_color_utilities are not available yet.

One minor issue though, the cards in validator page is dark when device in light mode.

Looks good. Thank you.

I'm happy to contribute to your project and to the IIUM Community!

Also I've been wondering how do you generate the color_schemes.g.dart file?

I generated the color_schemes file using Google's online version of their Material Theme Builder. Anyways, I am still finding ways on how to use the user's current theme instead of choosing only one color for the Material 3 theme version of the app.

For the Material You implementation, I have successfully configure the app to use user's dynamic colour palette instead of using the hardcoded colour!
Refer https://github.com/iqfareez/iium_schedule/commit/e64a13c2924e66bc5cab8b32b96fac887f96b05d


feature/BRIDGE-941

THIS PR IS DEPENDENT ON https://github.com/Sage-Bionetworks/ResearchKit/pull/13 WHICH MUST BE MERGED FIRST
Added in flow for forgetting passcode to AppCore.  Also, added it as default behavior for the APCPasscodeViewController.

Closing in preference to https://github.com/Sage-Bionetworks/AppCore/pull/108


Continuation with cpu_bound functions

Using a cpu_bound function in a continuation chain currently fails. Example:
from unsync import unsync
import asyncio
import time

@unsync()
async def download_data(url):
    await asyncio.sleep(1)
    return 'data'

@unsync(cpu_bound=True)
def process_data(task):
    data = task.result()
    time.sleep(1)
    return 'processed data'

@unsync()
def store_processed_data(task):
    data = task.result()
    time.sleep(1)
    print('Stored')

tasks = [
    download_data(url).then(process_data).then(store_processed_data)
    for url in ['url1', 'url2', 'url3']
]

[task.result() for task in tasks]

# TypeError: cannot pickle '_asyncio.Task' object

I think this is a fairly large drawback. It could easily be solved by passing the result() to the continuation rather than the Unfuture. If there are good reasons for passing the Unfuture, perhaps it could somehow be made picklable, or the unpicklable parts could be stripped / swapped out before passing it on.

I started #41, not sure exactly why I chose this API in the first place. It doesn't really seem like any reason to pass the Unfuture.
Let me think about this a bit since it's a breaking change, but looks fine at first glance.

Great! It’s worth remembering that it will still only work with a result that can be picked.


support for custom tokenizer added in Predictor.predict method

I tested it locally by modifying the package files. It works. Pls merge or let me know if i have to add something

Thanks.  Looks good except that you've prevented the ability to use a custom tokenizer for Chinese.   It should probably be changed to something like this:
if custom_tokenizer:
    tokens = custom_tokenizer(s)
elif TU.is_chinese(lang, strict=False):
    tokenize_chinese = lambda text:[c for c in text]
    tokens = tokenize_chinese(s)
else:
    tokens = TU.tokenize(s)

When this tag(0.19.3) will get uploaded to pypi?

It is there now.


Add CVE support in SnykDataSource

Implement datasource_advisory_from_cve in SnykDataSource
For example, see GithubDataSource.datasource_advisory_from_cve:
https://github.com/nexB/vulnerablecode/blob/0a8f7a863de3062deabcb3cff347f64bee1e776e/vulntotal/datasources/github.py#L54-L73

Hey @keshav-space, can I take this up?

Sure, go ahead.

Hey @keshav-space, while working on this issue, I found a minor bug in the datasource_advisory_from_cve function, for which I have created an issue (#1398) and pull request (#1399) as well
Coming to this issue, I think the code for Snyk will be slightly different from the Github one as for Github we use GraphQL to connect to Github API but for Snyk we would have to use BeautifulSoup (ofcourse other than that I think logic would be same). To add this feature, I am planning to follow the steps below

Go to the URL https://security.snyk.io/vuln?search={cve_id} and parse the HTML page
Loop through all vulnerabilities in the page and for each vulnerability parse the affected version and fixed version from the respective urls (which will be of the form https://security.snyk.io/vuln/{snyk_id})
Return the data as VendorData instance

I will try to write the code using the above steps. In the meanwhile, if there is anything I have missed, do let me know.

Coming to this issue, I think the code for Snyk will be slightly different from the Github one as for Github we use GraphQL to connect to Github API but for Snyk we would have to use BeautifulSoup (ofcourse other than that I think logic would be same). To add this feature, I am planning to follow the steps below

Go to the URL https://security.snyk.io/vuln?search={cve_id} and parse the HTML page
Loop through all vulnerabilities in the page and for each vulnerability parse the affected version and fixed version from the respective urls (which will be of the form https://security.snyk.io/vuln/{snyk_id})
Return the data as VendorData instance

I will try to write the code using the above steps. In the meanwhile, if there is anything I have missed, do let me know.

@shravankshenoy Sounds good to me 👍🏻

Hi everyone, I have raised a PR to add CVE support in Snyk data source. Let me know if any changes are required.
PR #1405


Add LA Designer panel for token selecting

Added to tester:

Panel:

Unit tests:

Issue with E2E test, are these still used? Maybe issue on my end:

Hi @preetriti1 I am closing this in favor of this PR. Changing to just exporting a few more things for our repo.


Handle Python 3.12

Conan version is upgraded to 1.62, Python 3.12 is excluded from supported versions because subunit 1.4.0 dependency of hyperonc is incompatible with Python 3.12. This is hotfix to unblock Python packages release.

See https://github.com/trueagi-io/hyperon-experimental/issues/584 for the issue description


Date disabled attribute in ng2-bootstrap date Picker is not working

Date disable attribute is not working.
Added below code in .ts file.
this.testDateDisabled = [
{date: (new Date("October 13, 2016 11:13:00")), mode: 'day'}
];
October 13 is not disable in the datepicker.

Facing same issue, anyone know about it please share your answer, thanku

it's marked as "todo" in the source code, doesn't seem to be implemented fully

+1

When could we expect a release of this feature?

Hi
I see the feature implemented but it is not available in npm version.
Any update on that front?
Thanks

available in old datepicker
not yet in new one


Multi-highlight Support?

Does your control support adding multi-highlights to various data in different colors?

Does your control support adding multi-highlights to various data in different colors?

Apologies, for some reason I had custom notifications on this repo so I didn't see your comment. Currently it does not support multiple highlighting. There would need to be some refactoring done to support such a feature.


Improve cli container exec behaviour

This PR redirects container stderr to console stderr and exits with container exit code.
Fixes #32

@nevalla ping

Looks good!


Wordcount errors in visual mode

Environment

vim: 8.1-1799
vim-airline: c213f2a
OS: Linux Mint 18.3 Cinnamon 64-bit
minimal vimrc:

    set nocompatible
    filetype off
    set rtp+=~/.vim/bundle/Vundle.vim
    call vundle#begin('~/.vim/bundle')
    Plugin 'VundleVim/Vundle.vim'
    Plugin 'vim-airline/vim-airline'
    call vundle#end()
    filetype plugin indent on

airline configuration: default
terminal: GNOME Terminal 3.18.3
$TERM variable: xterm-256color
color configuration: t_Co=256

Error Messages
Error detected while processing function airline#extensions#wordcount#get[2]..<SNR>39_get_wordcount:
line    2:
E315: ml_get: invalid lnum: 13

Reproduce

Create two windows with filetype=text
The following is the content of these window:

Press Ctrl-V to start visual mode blockwise in right window line 3

Move cursor

I have tried

Disabling wordcount will solve this:

let g:airline#extensions#wordcount#enabled = 0

Copy function! s:get_wordcount() in vim-airline/autoload/airline/extensions/wordcount.vim to vimrc. Then set statusline=%{My_get_wordcount(1)}. However, wordcount works correctly.

this is actually a Vim bug I believe. Nevertheless I think af96e97  fixes it by only trying to access the wordcount if we are in the correct buffer. (however to work properly, it might need https://github.com/vim/vim/releases/tag/v8.1.1372 not exactly sure).

af96e97 works perfectly on my machine (vim 8.1.1779), thanks a lot!


Failure to Link Child Record to Parent Record When Adding a New Record

I can't get a child record to be linked to parent record when adding a new record.
The old modules and pages works perfectly.
Version : 2023.9.5
Steps to Reproduce:

Create a parent module named "a1" with a field named "parentName" of type string.
Create a child module named "a2" with fields "childName" (string) and "parentID" (record selector) referencing module "a1".
Create a record page for the module "a1" and add a record list block for the child module "a2" with a prefilter condition: "parentID = ${recordID}".
Create a record page named "a2".

Expected Behavior:
When adding a new record within a record of module "a1" by clicking "+Add" in the record list block for module "a2", the new record in module "a2" should automatically inherit the parentID field value equal to the parent's record.
Actual Behavior:
Upon adding a new record in module "a2" from within a record of module "a1", the parentID field in the new child record remains empty, failing to establish the expected link to the parent record.
Additional Information:
In the documentation, there is no clear steps talking about linking parent modules to child modules. I used keywords like "linking" "parent", or "child" but I didn't succeed.

This is an edge case that will be addressed in 2024.3.x. For now to avoid this, I would recommend creating a record page and then a record list page for module a1, doing the same for a2, and after this adding page blocks to the desired pages. I would delete the currently broken pages and follow the steps in the previous sentence.

Reproduction steps


fix: reset amount fields after swap or withdraw

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Add Traditional Chinese translation

Hello author, I hope to add traditional Chinese translation to your module, I have prepared the file
After downloading the file, change the file format from .txt to .json
zh_tw.txt
P.S. :Since I won't be using pull requests I can only put here

Added with https://github.com/trainb0y/FabriZoom/commit/ad4d68c4ff538700b3195ec57251c033ad1fe742.
Thanks for the translation!
It will be in the next release.


fix(packages/core): core repl pipe-stage splitter fails for \ line co…

…ntinuations
Fixes #7279

Description of what you did:

My PR is a:

[ ] 💥 Breaking change
[x] 🐛 Bug fix
[ ] 💅 Enhancement
[ ] 🚀 New feature

Please confirm that your PR fulfills these requirements

[x] Multiple commits are squashed into one commit.
[x] The commit message follows Conventional Commits, which allows us to autogenerate release notes; e.g. fix(plugins/plugin-k8s): fixed annoying bugs
[x] All npm dependencies are pinned.

/lgtm


README.md: better plugins' sorting

Maybe it's my OCD, but I feel like sorting alphabetically is more natural. In the plugins table, I kept the enabled ones at the top.
Let me know what you think :)

Plugins are specified in order they are run which is critical for optimisation and correctness.

Yeah, but in the README, it's harder to find which plugin is enabled if they are sorted randomly...

Copy pasted config will not work so I'd prefer to not confuse anybody.

This is more confusing, trying to find a plugin randomly. The CLI help text sorts them by name for example.
Sorting alphabetically is more natural for human beings than random sorting.

@XhmikosR I agree with that. But providing broken config is not human friendly.


Consideration to allow users to Unshield Assets direct from the Shield pool out to IBC chains - Currently users can only unshield to NAM transparent

The current user flow requires users to make 2 transactions to get their assets out of the shield pool back to an IBC chain

Transfer from Nam shielded to Nam transparent
Transfer from NAM transparent out through IBC

Current Unshield UI

UI Changed needed to the IBC tab

Add the toggle to allow to switch between shield and unshield. Shield would be the default


fix comment in annotation

fix comment in annotation

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

Can one of the admins verify this patch?

no error in fact

Can one of the admins verify this patch?

no error in fact


[#97] Allow use of wildcard ('*') groupIds

Fixes #97

Limitations on wildcard usage would still apply - no wildcards in any channel sent for testing or production.

I do not see how this would be used. If the channel specifies one stream:
groupId=; artifactId=; versionPattern=.*.Final
It seems to me that installer would have to download every artifact ending in "Final" from every repository. Is that correct?


Cutscene freeze entering pharmacy

Version
3.0.3 (0b004de)
Seed
R730-LP7H-ZTLF19N53Y700N92EJM0WCB07HEUCZZZ5XZ
Scenario
RE 3 (Jill)
Player Character
Claire (RE2)
What happened?
Currently saved the game inside the room before water test. Taking the door that would go down to water test goes to the pharmacy entrance where the cutscene would start (either Nicholai or Carlos killing Murphy), and the cutscene just freezes there with Claire standing quietly infinitely without the cutscene proceeding
re3freeze (2).zip

Is this the same problem as #278?
And can you skip the cutscene?

Is this the same problem as #278? And can you skip the cutscene?

It is the same problem. You cannot skip the cutscene.


feat: IRSA enabled by default

Description

Enable IRSA by default

Motivation and Context

Within the EKS ecosystem, it is difficult to find a scenario where the EKS OIDC provider (IRSA) should not be enabled

Breaking Changes

No - users who wish to opt out of IRSA enablement can set enable_irsa = false

How Has This Been Tested?

[x] I have tested and validated these changes using one or more of the provided examples/* projects

This PR is included in version 18.4.0 :tada:


 This might not work if that function calls another function on a different scope. Is it possible to loop over all scope functions and pre-bind them, at init time?

Done.


What does a shimple shnippet mean I don't know what this mean

Buffalo buffalo Buffalo buffalo buffalo buffalo Buffalo buffalo


scoop install not working: Couldn't find manifest for 'pragtical'.

Installing pragtical via powershell not working in windows.
I tried following the installation from the docs but it seems to be not working.

I'm not familiar with scoop my self but maybe running this:
scoop bucket add extras

Then try again with scoop install pragtical if that works the website docs will need an update.

Thanks it worked!

Thanks it worked!!


[alf] files

For the Alf Language , I suppose we need to write the alf.y and alf.l files, but for the alf.js file can we use your parser (the one from tp5 for example)?

alf.js is generated from the .y and .l files.
On Wed, Mar 27, 2019 at 7:56 PM diana-ghindaoanu<EMAIL_ADDRESS>wrote:

For the Alf Language , I suppose we need to write the alf.y and alf.l
files, but for the alf.js file can we use your parser (the one from tp5 for
example)?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/UPB-FILS/alf/issues/30, or mute the thread
https://github.com/notifications/unsubscribe-auth/ACtOoQF_kXzrdAVj60H2CESlggOPwIAgks5va7DigaJpZM4cObqX
.

--
Alexandru RADOVICI
Universitatea "Politehnica" din Bucuresti
e-mail<EMAIL_ADDRESS>telefon:<PHONE_NUMBER>
www.ipworkshop.ro

You can start from the .y and .l files from the lab.
On Wed, Mar 27, 2019 at 8:02 PM Alexandru Radovici<EMAIL_ADDRESS>wrote:

alf.js is generated from the .y and .l files.
On Wed, Mar 27, 2019 at 7:56 PM diana-ghindaoanu<EMAIL_ADDRESS>wrote:

For the Alf Language , I suppose we need to write the alf.y and alf.l
files, but for the alf.js file can we use your parser (the one from tp5 for
example)?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/UPB-FILS/alf/issues/30, or mute the thread
https://github.com/notifications/unsubscribe-auth/ACtOoQF_kXzrdAVj60H2CESlggOPwIAgks5va7DigaJpZM4cObqX
.

--
Alexandru RADOVICI
Universitatea "Politehnica" din Bucuresti
e-mail<EMAIL_ADDRESS>telefon:<PHONE_NUMBER>
www.ipworkshop.ro

--
Alexandru RADOVICI
Universitatea "Politehnica" din Bucuresti
e-mail<EMAIL_ADDRESS>telefon:<PHONE_NUMBER>
www.ipworkshop.ro

great, thank you


[camera] Pause/resume video recording for Android(update of PR 1370)

Description
Replace this paragraph with a description of what this PR is doing. If you're modifying existing behavior, describe the existing behavior, how this PR is changing it, and what motivated the change.
Related Issues
Replace this paragraph with a list of issues related to this PR from the issue database. Indicate, which of these issues are resolved or fixed by this PR. Note that you'll have to prefix the issue numbers with flutter/flutter#.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[ ] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[ ] All existing and new tests are passing.
[ ] I updated/added relevant documentation (doc comments with ///).
[ ] The analyzer (flutter analyze) does not report any problems on my PR.
[ ] I read and followed the Flutter Style Guide.
[ ] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[ ] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy.
[ ] I updated CHANGELOG.md to add a description of the change.
[ ] I signed the CLA.
[ ] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md and increment major revision).
[ ] No, this is not a breaking change.

I signed it!

Thank you for your patience and for helping out. #1370 has just been updated again and is in the process of being landed, so I'm closing this in favor of the original.


[REQUEST] Add a loading icon

Would it be possible to add a circular progress dialog in the right corner? It should overlap the voice icon.

Everything is possible, but why? :)

In my case I want to load suggestions from internet. While we are retrieving the data we want it to show a progressdialog icon so the user knows it is loading data from the internet :)

Ok, I will try to add something like that in next release. It is not a problem, but give me some time cause right now I am doing my diploma thesis :)

Got that, were can I donate?

Probably only here:
https://www.paypal.com/cgi-bin/webscr?cmd=_s-xclick&hosted_button_id=2W9YDH84RKA4W

Sure thing, I will buy you some beers for the next release party :)

:D thanks.

This function will not be a part of next version. I have done a lot of changes in library, but I do not see reason why put progress circle to library. You should use something else or do a custom version of my lib.

@lapism Do you have a callback after searching is complete so the progress circle could be stopped?

Not now :).  In my prepared version I have these interfaces right now.
public interface OnQueryTextListener {
    boolean onQueryTextChange(String newText);
    boolean onQueryTextSubmit(String query);
}

public interface OnSuggestionListener {
    boolean onSuggestionClick(int position);
    boolean onSuggestionSelect(int position);
}

public interface OnOpenCloseListener {
    void onClose();
    void onOpen();
}

public interface OnMenuClickListener {
    void onMenuClick();
}


Add JCDS direct chunking method of upload which is what jamf cloud normally does.

This is the method by which the UI uploads packages to JCDS now, they don't use dbfileupload except in the old jamf admin gui client. Its a bit of a rough cut but you get the idea.

Merging to a test branch for a good look!


Update docs, cairo-lang-runner references

Update docs
Specify that the implementation of certain Cairo 1 functions are inspired by the cairo-lang-runner crate.
Reference:

cairo-lang-runner

Checklist

[ ] Linked to Github Issue
[ ] Unit tests added
[ ] Integration tests added.
[ ] This change requires new documentation.

[ ] Documentation has been added/updated.
[ ] CHANGELOG has been updated.

I think the module also needs to be referenced.

I think the module also needs to be referenced. It's the dict_manager.rs

Nice catch @FrancoGiachetta !
done 89178e8


Custom HTML should work in Markdown

Custom HTML styliing should work, and if possible custom colors as well.

Fixed as of last commit. This introduces a new issue, but that's not essential.


Clarify host.name attribute and host registry definition

Area(s)
area:host
Is your change request related to a problem? Please describe.
The current semantic conventions' documentation for the “host” registry is defined as:

A host is defined as a computing instance. For example, physical servers, virtual machines, switches or disk array.

Reference: https://github.com/open-telemetry/semantic-conventions/blob/main/docs/attributes-registry/host.md
Based on this definition, a Linux/K8s container (cgroup) is not explicitly considered a host. However, this raises an important question: isn’t a cgroup an isolated instance with its own dedicated computing resources? What truly distinguishes a virtual machine from a container?
If containers are not classified as hosts but as services running within a host, it brings up another critical question: what should the value of the host.name attribute be when retrieved by SDKs operating inside a container? Should this attribute even be populated in such cases?
Currently, the Go SDK utilizes the os.Hostname function from the internal package to retrieve and populate the host.name resource attribute. Meaning that a Go SDK container will report as host.name the container’s hostname, not the hostname value of the virtual machine it is running on. If a container is not considered a host, should SDKs running on containers report this value?
A workaround to fulfill the host’s semantic convention description, would be to send all container’s signals to a collector which overrides/sets the host.name value with the actual virtual machine hostname the container is running on. The following diagram shows an architecture that overrides all container's host.name value with the k8s.node.name value, which normally corresponds to the virtual machine's hostname:

The previous is a possible user interpretation of the host.name value to simplify service correlation—ensuring all service.name instances sharing the same host.name are identified as running on the same machine— but, it comes with notable downsides:

Requires of an additional processor (either on-site or on the backend) to override the host.name value. In addition, it requires the ability to gather the value of the host.name in the upper virtualization layer.  The latest is feasible to achieve if there is a service like the OpenTelemetry collector that has resource detectors to do so. But what would happen for standalone agents running in a container and directly sending OTLP data to another node’s OTLP endpoint or even a remote endpoint?

The hostname within a container/cgroup describes useful information: “Containers within the Pod see the system hostname as being the same as the configured name for the Pod.”
The container’s hostname is used for container’s communication, either in K8s or standalone Docker deployment.

Describe the solution you'd like
Proposal: Include container in the host definition + new host.hostname attribute
Modify the host semantic conventions registry definition to explicitly include containers. This adjustment will narrow the scope of the definition, reducing ambiguity for containerized environments.
The current description of host.name is highly permissive:

“Name of the host. On Unix systems, it may contain what the hostname command returns, or the fully qualified hostname, or another name specified by the user.” - Reference

This flexibility leads to a lack of determinism in the value of host.name. In some cases, it reflects the value returned by the gethostname system call, while in others, it may represent a user-defined custom value. However, in distributed systems, the actual service networking hostname plays a critical role in enabling reliable entity and signal correlation.
To address this issue and align with established conventions, such as those in Elastic ECS, a new attribute could be introduced to explicitly reference the hostname used for networking communications. This proposal differentiates between the network hostname and the general host name as follows:

host.hostname: The hostname of the host as used for networking communications. Typically, this is the value returned by the hostname command on the host machine.

host.name: The name of the host. This attribute may contain what the hostname command returns on Unix systems, the fully qualified domain name (FQDN), or a name specified by the user. The recommended value is the lowercase FQDN of the host.

Describe alternatives you've considered
Proposal: Define the value of host.name dependent on the environment
To ensure consistency across different deployment scenarios, we could define the value of host.name in a way that depends on the environment in which the service is running. Suggested guidelines for various scenarios are as follows:

Virtual Machine Deployments: In virtual machine environments, the value of host.name for any running service should be equal to the virtual machine's fully qualified domain name (FQDN).

Cloud instances: TBD (instance-id? FQDN hostname?)

For Docker deployments, container’s host.name should correspond to the FQDN of the host machine is running on, not the hostname of the service's container. Services (SDKs) should have a resource detector in place to retrieve the “host” hostname, similar to the OpenTelemetry Collector docker resource detector: https://github.com/open-telemetry/opentelemetry-collector-contrib/blob/main/processor/resourcedetectionprocessor/internal/docker/documentation.md
In Kubernetes environments, the approach should mirror that for Docker: either don’t provide the host.name value if SDKs don’t have a “host” resource detector (which will need an outer enricher) or be equal to the “host” FQDN hostname (or even the k8s.node.name value).

This second proposal complements the first by further emphasizing the need to distinguish between the actual networking hostname of the computing instance and the value of host.name as defined by semantic conventions.
Additional context
No response

In Kubernetes environments, the approach should mirror that for Docker: either don’t provide the host.name value if SDKs don’t have a “host” resource detector (which will need an outer enricher) or be equal to the “host” FQDN hostname (or even the k8s.node.name value).

I had shared some related confusion at https://github.com/open-telemetry/semantic-conventions/issues/761#issuecomment-1978262862


[WIP][#169336585] Update AWS RDS certificate bundle

What
Describe what you have changed and why.
How to review
Describe the steps required to test the changes.
Who can review
Describe who can review the changes. Or more importantly, list the people
that can't review, because they worked on it.

Using this command, the contents of the existing bundle are:
openssl crl2pkcs7 -nocrl -certfile <(yq read manifests/cf-manifest/data/000-aws-rds-combined-ca-bundle-pem.yml 'aws_rds_combined_ca_bundle') | openssl pkcs7 -print_certs -text -noout | grep -e 'Subject:' -e 'Not After'
            Not After : Mar  5 09:11:31 2020 GMT
        Subject: C=US, ST=Washington, L=Seattle, O=Amazon Web Services, Inc., OU=Amazon RDS, CN=Amazon RDS Root CA
            Not After : Mar  5 22:03:06 2020 GMT
        Subject: C=US, ST=Washington, L=Seattle, O=Amazon Web Services, Inc., OU=Amazon RDS, CN=Amazon RDS ap-northeast-1 CA
            Not After : Mar  5 22:03:19 2020 GMT
        Subject: C=US, ST=Washington, L=Seattle, O=Amazon Web Services, Inc., OU=Amazon RDS, CN=Amazon RDS ap-southeast-1 CA
            Not After : Mar  5 22:03:24 2020 GMT
        Subject: C=US, ST=Washington, L=Seattle, O=Amazon Web Services, Inc., OU=Amazon RDS, CN=Amazon RDS ap-southeast-2 CA
            Not After : Mar  5 22:03:31 2020 GMT
        Subject: C=US, ST=Washington, L=Seattle, O=Amazon Web Services, Inc., OU=Amazon RDS, CN=Amazon RDS eu-central-1 CA
            Not After : Mar  5 22:03:35 2020 GMT
        Subject: C=US, ST=Washington, L=Seattle, O=Amazon Web Services, Inc., OU=Amazon RDS, CN=Amazon RDS eu-west-1 CA
            Not After : Mar  5 22:03:40 2020 GMT
        Subject: C=US, ST=Washington, L=Seattle, O=Amazon Web Services, Inc., OU=Amazon RDS, CN=Amazon RDS sa-east-1 CA
            Not After : Mar  5 21:54:04 2020 GMT
        Subject: C=US, ST=Washington, L=Seattle, O=Amazon Web Services, Inc., OU=Amazon RDS, CN=Amazon RDS us-east-1 CA
            Not After : Mar  5 22:03:45 2020 GMT
        Subject: C=US, ST=Washington, L=Seattle, O=Amazon Web Services, Inc., OU=Amazon RDS, CN=Amazon RDS us-west-1 CA
            Not After : Mar  5 22:03:50 2020 GMT
        Subject: C=US, ST=Washington, L=Seattle, O=Amazon Web Services, Inc., OU=Amazon RDS, CN=Amazon RDS us-west-2 CA
            Not After : Mar  5 00:05:46 2020 GMT
        Subject: C=US, ST=Washington, L=Seattle, O=Amazon Web Services, Inc., OU=Amazon RDS, CN=Amazon RDS ap-northeast-2 CA
            Not After : Mar  5 21:29:22 2020 GMT
        Subject: C=US, ST=Washington, L=Seattle, O=Amazon Web Services, Inc., OU=Amazon RDS, CN=Amazon RDS ap-south-1 CA
            Not After : Mar  5 19:58:45 2020 GMT
        Subject: C=US, ST=Washington, L=Seattle, O=Amazon Web Services, Inc., OU=Amazon RDS, CN=Amazon RDS us-east-2 CA
            Not After : Mar  5 00:10:11 2020 GMT
        Subject: C=US, ST=Washington, L=Seattle, O=Amazon Web Services, Inc., OU=Amazon RDS, CN=Amazon RDS ca-central-1 CA
            Not After : Mar  5 17:44:42 2020 GMT
        Subject: C=US, ST=Washington, L=Seattle, O=Amazon Web Services, Inc., OU=Amazon RDS, CN=Amazon RDS eu-west-2 CA
            Not After : Mar  5 21:39:26 2020 GMT
        Subject: C=US, ST=Washington, L=Seattle, O=Amazon Web Services, Inc., OU=Amazon RDS, CN=Amazon RDS eu-west-3 CA
            Not After : Mar  5 00:55:42 2020 GMT
        Subject: C=US, ST=Washington, L=Seattle, O=Amazon Web Services, Inc., OU=Amazon RDS, CN=Amazon RDS ap-northeast-3 CA
            Not After : Mar  4 22:42:27 2020 GMT
        Subject: C=US, ST=Washington, L=Seattle, O=Amazon Web Services, Inc., OU=Amazon RDS, CN=Amazon RDS preview-us-east-2 CA

We see that the bundle includes a certificate for each region that existed in 2015, expiring near midnight on 2020-03-05, allowing for time zone differences. Whoever reviews this should check the new cert bundle to ensure it still contains these certs.


Avoid mutating jumpstateDefaults

The config Object.assign was adding properties into the jumpstateDefaults object.
By switching the assign order, mutation is avoided.

You're the man @robertpenner!  Thanks for the help :)

No problem, thanks for Jumpstate @tannerlinsley!


[DCOS-12416] Use ConfigurationMap in PodDebugContainer

This PR implements the ConfigurationMap components in place of the HashMapDisplay component.

run unit tests

Looks good!


Enhancement request: Add support for an interactive distance measurement tool in the analysis results report

The film analysis report interactive figure should have allow the use of a distance tool to measure physical distances in any of the subfigures. This would be quite useful to measure for example the difference in the measured vs planned beam profiles.

I added this functionnality in the isodoses plot. Could not get it to work in the analysis report, as mouse interaction is already used for setting up profiles.

Solved in latest PR #48


Question: why jQuery dependency?

I understand Box.com might be using jQuery features extensively, but if T3's idea is to stay modular, then why not decouple jQuery as well.
Looking at the documents (apologize if I missed something), it seems jQuery would be used for data-binding, but there can be lots of approach to data-binding, not to mention virtual-dom approach (where binding is handled differently)
Seems like T3 can be done without jQuery, or at least with alternatives like DOMtastic.
Either way, I am just very excited to see a framework that takes progressive enhancement into account and I am very interested in using it for my current project.

Thanks, we actually do cover this in our README:
https://github.com/box/t3js#why-is-there-a-dependency-on-jquery
It's basically for event normalization across browsers.

missed it :)
If that's the case though, there must be something that can cover your need without loading jQuery into it right? How about this:
https://github.com/desandro/eventie

Also note that DOMtastic mentioned above use the same api as jQuery, and you can build event system independent of other components.
Is it possible to allow developers to substitute jQuery? Just a suggestion.

If you want to use a library that has the same event interface as jQuery ($().on), then you can use it already. There are literally only two references to jQuery in T3:
https://github.com/box/t3js/blob/master/lib/application.js#L378
https://github.com/box/t3js/blob/master/lib/application.js#L431
So if you can make those methods work with another library, there's nothing stopping you from doing so right now.

Ah, many thx for your answers.

If that's the case, you should abstract this so:
a) People who don't need IE8 don't need to add a dependency or shim an API,
b) People who do can explicitly plug in a microlibrary like Gator to solve the dependency, and
c) Future versions don't accidentally presume jQuery.
T3.EventAPI = userProvidedAPI || $;

function listenTo (target, eventType, listener) {
  if (T3.EventAPI) {
    T3.EventAPI(target).on(eventType, listener);

  } else if (target.addEventListener) {
    target.addEventListener(eventType, listener);

  } else {
    throw new Error("T3 requires an event library like Gator or jQuery for IE8 support");
  }
}

+1 to @appsforartists, though I don't know the exact reason why polyfill can't be used in this case, since IE8 appear to be the source of problem.

We are looking at removing jQuery once we can EOL IE8. We were looking at further extracting the jQuery-ness before open sourcing and just ran out of time. If you're interested, please file a separate issue suggesting building such an abstraction and we'll see if we either get some time or can find someone to contribute.

We've opened https://github.com/box/t3js/issues/22 to continue discussion about how to abstract out event handling.


Process Sync requests

The API can send a sync request. This should force the clients to resync.

[ ] Handle a sync request
[ ] Create a notification when a favorited event has changed
[ ] MAYBE: create a notification if a new announcement is posted.

done in 387d51