multipe tests/posix & tests/lib/c_lib/thrd build failures due to missing time.h definitions

Describe the bug

tests/posix/common/portability.posix.common.newlib
tests/posix/common/portability.posix.common.tls.newlib
tests/posix/fs/portability.posix.fs.newlib
tests/posix/fs/portability.posix.fs.tls.newlib
tests/posix/headers/portability.posix.headers.newlib.with_posix_api
tests/lib/c_lib/thrd/libraries.libc.c11_threads.newlib
tests/lib/c_lib/thrd/libraries.libc.c11_threads.newlib_nano

build failure in the following platforms:

circuitdojo_feather/nrf9160/ns
actinius_icarus_bee/nrf9160/ns
<EMAIL_ADDRESS><EMAIL_ADDRESS>innblue21/nrf9160/ns
innblue22/nrf9160/ns
lpcxpresso55s69/lpc55s69/cpu0/ns
actinius_icarus_som_dk/nrf9160/ns
actinius_icarus_som/nrf9160/ns
mimxrt1010_evk
mimxrt1024_evk
mimxrt1015_evk
mimxrt1020_evk
mimxrt1040_evk
mimxrt1060_evk
mimxrt1050_evk
mimxrt1060_evk@hyperflash
mimxrt1060_evkb
mimxrt1050_evk@qspi
mimxrt1170_evk@B/mimxrt1176/cm7
mimxrt1062_fmurt6
mimxrt1170_evk/mimxrt1176/cm7
mimxrt1160_evk/mimxrt1166/cm7
mimxrt1064_evk
mm_feather
mimxrt595_evk/mimxrt595s/cm33
mm_swiftio
mimxrt685_evk/mimxrt685s/cm33
nrf5340dk/nrf5340/cpuapp/ns
nrf5340_audio_dk/nrf5340/cpuapp/ns
nrf9151dk/nrf9151/ns
nrf9160dk/nrf9160/ns
nrf9131ek/nrf9131/ns
<EMAIL_ADDRESS>nrf9161dk/nrf9161/ns
<EMAIL_ADDRESS>raytac_mdbt53_db_40/nrf5340/cpuapp/ns
raytac_mdbt53v_db_40/nrf5340/cpuapp/ns
sparkfun_thing_plus/nrf9160/ns
b_u585i_iot02a/stm32u585xx/ns
stm32l562e_dk/stm32l562xx/ns
thingy53/nrf5340/cpuapp/ns
vmu_rt1170/mimxrt1176/cm7
bl5340_dvk/nrf5340/cpuapp/ns

To Reproduce
Steps to reproduce the behavior:
twister -i -p mimxrt1170_evk/mimxrt1176/cm7 -s portability.posix.headers.newlib.with_posix_api

Expected behavior
build PASS
Impact
POSIX compatible
~190 failures in weekly CI
Failures in PRs CI when these tests are triggered
Logs and console output
modules/crypto/mbedtls/library/platform_util.c
In file included from zephyr/include/zephyr/posix/time.h:116,
                 from zephyr/include/zephyr/posix/time.h:12,
                 from zephyr/include/zephyr/net/socket_select.h:22,
                 from zephyr/include/zephyr/posix/sys/select.h:10,
                 from zephyr-sdk-0.16.8/arm-zephyr-eabi/arm-zephyr-eabi/sys-include/sys/types.h:50,
                 from zephyr-sdk-0.16.8/arm-zephyr-eabi/arm-zephyr-eabi/sys-include/stdio.h:61,
                 from modules/crypto/mbedtls/include/mbedtls/platform.h:58,
                 from modules/crypto/mbedtls/library/platform_util.c:26:
zephyr-sdk-0.16.8/arm-zephyr-eabi/arm-zephyr-eabi/sys-include/time.h:56:1: error: unknown type name 'clock_t'
   56 | clock_t    clock (void);
      | ^~~~~~~
zephyr-sdk-0.16.8/arm-zephyr-eabi/arm-zephyr-eabi/sys-include/time.h:30:1: note: 'clock_t' is defined in header '<time.h>'; did you forget to '#include <time.h>'?
   29 | #include <sys/timespec.h>
  +++ |+#include <time.h>
   30 |
In file included from zephyr/include/zephyr/posix/posix_types.h:15,
                 from zephyr/include/zephyr/posix/time.h:61:
zephyr-sdk-0.16.8/arm-zephyr-eabi/arm-zephyr-eabi/sys-include/sys/_pthreadtypes.h:182:3: error: unknown type name 'clock_t'
  182 |   clock_t  clock;             /* specifiy clock for timeouts */
      |   ^~~~~~~
zephyr/include/zephyr/posix/posix_types.h:76:9: error: unknown type name 'clockid_t'
   76 |         clockid_t clock;
      |         ^~~~~~~~~
zephyr/include/zephyr/posix/time.h:93:19: error: unknown type name 'clockid_t'; did you mean 'clock_pfd_t'?
   93 | int clock_gettime(clockid_t clock_id, struct timespec *ts);
      |                   ^~~~~~~~~
      |                   clock_pfd_t
zephyr/include/zephyr/posix/time.h:94:18: error: unknown type name 'clockid_t'; did you mean 'clock_pfd_t'?
   94 | int clock_getres(clockid_t clock_id, struct timespec *ts);
      |                  ^~~~~~~~~
      |                  clock_pfd_t
zephyr/include/zephyr/posix/time.h:95:19: error: unknown type name 'clockid_t'; did you mean 'clock_pfd_t'?
   95 | int clock_settime(clockid_t clock_id, const struct timespec *ts);
      |                   ^~~~~~~~~
      |                   clock_pfd_t
zephyr/include/zephyr/posix/time.h:96:36: error: unknown type name 'clockid_t'; did you mean 'clock_pfd_t'?
   96 | int clock_getcpuclockid(pid_t pid, clockid_t *clock_id);
      |                                    ^~~~~~~~~
      |                                    clock_pfd_t
zephyr/include/zephyr/posix/time.h:98:18: error: unknown type name 'clockid_t'; did you mean 'clock_pfd_t'?
   98 | int timer_create(clockid_t clockId, struct sigevent *evp, timer_t *timerid);
      |                  ^~~~~~~~~
      |                  clock_pfd_t
zephyr/include/zephyr/posix/time.h:105:21: error: unknown type name 'clockid_t'; did you mean 'clock_pfd_t'?
  105 | int clock_nanosleep(clockid_t clock_id, int flags,
      |                     ^~~~~~~~~
      |                     clock_pfd_t
In file included from zephyr/include/zephyr/sys/fdtable.h:13,
                 from zephyr/include/zephyr/net/socket_select.h:26:
zephyr/include/zephyr/fs/fs.h:368:1: error: unknown type name 'ssize_t'; did you mean '_ssize_t'?
  368 | ssize_t fs_read(struct fs_file_t *zfp, void *ptr, size_t size);
      | ^~~~~~~
      | _ssize_t
zephyr/include/zephyr/fs/fs.h:389:1: error: unknown type name 'ssize_t'; did you mean '_ssize_t'?
  389 | ssize_t fs_write(struct fs_file_t *zfp, const void *ptr, size_t size);
      | ^~~~~~~
      | _ssize_t
zephyr/include/zephyr/fs/fs.h:409:36: error: unknown type name 'off_t'; did you mean '_off_t'?
  409 | int fs_seek(struct fs_file_t *zfp, off_t offset, int whence);
      |                                    ^~~~~
      |                                    _off_t
zephyr/include/zephyr/fs/fs.h:425:1: error: unknown type name 'off_t'; did you mean '_off_t'?
  425 | off_t fs_tell(struct fs_file_t *zfp);
      | ^~~~~
      | _off_t
zephyr/include/zephyr/fs/fs.h:447:40: error: unknown type name 'off_t'; did you mean '_off_t'?
  447 | int fs_truncate(struct fs_file_t *zfp, off_t length);
      |                                        ^~~~~
      |                                        _off_t
zephyr/include/zephyr/sys/fdtable.h:48:17: error: expected specifier-qualifier-list before 'ssize_t'
   48 |                 ssize_t (*read)(void *obj, void *buf, size_t sz);
      |                 ^~~~~~~
zephyr/include/zephyr/sys/fdtable.h:52:17: error: expected specifier-qualifier-list before 'ssize_t'
   52 |                 ssize_t (*write)(void *obj, const void *buf, size_t sz);
      |                 ^~~~~~~
ninja: build stopped: subcommand failed.```

Environment (please complete the following information):

OS: (e.g. Linux)
Toolchain (e.g Zephyr SDK, ...) zephyr-sdk-0.16.8
Commit SHA or Version used: main after b82b5b0734c30490368e21627423f10036487343

Additional info
Introduced by #73978
PR is not bisectable
Edited by @aescolar to encompass all cases of the same issue

It's not really clear to me how commit a9a909c558c8c2069742aeee55aba831740826cd could affect anything to do with mbedtls or posix headers (so I think there was a misstep in the git bisect, likely missing a west update), but this test does fail to build using the command below on main.
twister -i -p mimxrt1170_evk/mimxrt1176/cm7 -s portability.posix.headers.newlib.with_posix_api

A bisect at my end identified 94d712e5cfe001996d7280c028e0e0303cf7b747 as the offending commit, which is also just as arbitrary and unrelated.
The search continues!

I ran a bisect and it points to https://github.com/zephyrproject-rtos/zephyr/pull/73978 which does affect posix related implementation.

The bisect I ran points the same PR however, the first commit of that PR does not break posix thing but fdtable.c.
The error is:
.../zephyr/lib/os/fdtable.c: In function 'zvfs_rw':
.../zephyr/lib/os/fdtable.c:315:41: error: 'const struct fd_op_vtable' has no member named 'write_offset'; did you mean 'write_offs'?
  315 |                 if (fdtable[fd].vtable->write_offset == NULL) {
      |                                         ^~~~~~~~~~~~
      |                                         write_offs
.../zephyr/lib/os/fdtable.c:319:51: error: 'const struct fd_op_vtable' has no member named 'write_offset'; did you mean 'write_offs'?
  319 |                         res = fdtable[fd].vtable->write_offset(fdtable[fd].obj, buf, sz,
      |                                                   ^~~~~~~~~~~~
      |                                                   write_offs
.../zephyr/lib/os/fdtable.c:327:51: error: 'const struct fd_op_vtable' has no member named 'read_offset'; did you mean 'read_offs'?
  327 |                         res = fdtable[fd].vtable->read_offset(fdtable[fd].obj, buf, sz,
      |                                                   ^~~~~~~~~~~
      |                                                   read_offs
.../zephyr/lib/os/fdtable.c:335:1: error: label 'unlock' defined but not used [-Werror=unused-label]
  335 | unlock:
      | ^~~~~~

I have the same problem. The sys/types.h from Newlib includes sys/select.h (provided by Zephyr), when __BSD_VISIBLE is set to 1. The sys/select.h later includes zephyr/net/socket_select.h which includes time.h and zephyr/sys/fdtable.h, which need off_t, ssize_t and clockid_t.
Missing types should be provided by sys/types.h, but sys/select.h is included before these types are defined.
Including posix_features.h using -imacro doesn't solve the issue. We can't also rely on POSIX headers from Newlib, because some types have different representation (e.g. pthread_rwlockattr_t).
For me, creating sys/types.h with the following content worked (sys/types.h from Newlib never includes sys/select.h).
diff --git a/include/zephyr/posix/sys/types.h b/include/zephyr/posix/sys/types.h
new file mode 100644
index<PHONE_NUMBER>0..9a13cfbda64
--- /dev/null
+++ b/include/zephyr/posix/sys/types.h
@@ -0,0 +1,19 @@
+/*
+ * Copyright (c) 2024 Google LLC
+ *
+ * SPDX-License-Identifier: Apache-2.0
+ */
+
+#ifndef ZEPHYR_INCLUDE_POSIX_SYS_TYPES_H_
+#define ZEPHYR_INCLUDE_POSIX_SYS_TYPES_H_
+
+#ifdef CONFIG_NEWLIB_LIBC
+
+#include <sys/features.h>
+#undef __BSD_VISIBLE
+
+#endif /* CONFIG_NEWLIB_LIBC */
+
+#include_next <sys/types.h>
+
+#endif /* ZEPHYR_INCLUDE_POSIX_SYS_TYPES_H_ */

Just got back (I have quite a commute :-/). Looking at it again.

I think part of the issue is that posix types are being used where they probably shouldn't be.
off_t, ssize_t and <sys/types.h> are all part of posix but in fdtable.h, which is below the posix api.
I have a fairly simple workaround (just define the types locally).

Very strange how this somehow managed to go through CI without triggering any failures on the original set of PRs...


UI added, Deno fresh server is used and code is restructured accordinlgy.

We added a basic UI and a playground to test the app. Implemented #20
Used Deno Fresh server for routing and rendering components and UI.
Restructured the code in line with the fresh framework guidelines

@karthikchiru12 raise another pull request to allow force merge. Thanks.

@sripravan :) That is really strange, it`s not letting it merge even after resolving conflicts.
Here is a new pull request: https://github.com/yakshavingdevs/datadance/pull/30


Supporting more BERT-like models

Hi HF team!
I am extending our UForm repository of multimodal models to support Swift and mobile deployments, and along that way I've noticed that several classes for a broad range of BERT-like models are not yet supported by swift-transformers. So I've added a WordPieceDecoder class and aliases for BertPreTokenizer and BertProcessing.
Moreover, are you are well aware config.json and tokenizer.json come in all shapes and sizes. So I've added fallback mechanisms to handle different tuple order in vocabulary listings.
The current main-dev branch of UForm is already using this functionality from my fork. I am looking into integrating more Hub functionality next. Please let me know what you think about this PR 🤗

Another open problem that I've recently discovered is the way strings are compared in Swift. By default, the language uses UTF8-aware normalization techniques when comparing strings. This is great for some applications, but horrible for tokenization, especially with multilingual models. I've solved that by introducing a LiteralString wrapper for String, that uses the literal comparators:
    struct LiteralString: Hashable {
        let value: String

        static func ==(lhs: LiteralString, rhs: LiteralString) -> Bool {
            return lhs.value.compare(rhs.value, options: .literal) == .orderedSame
        }

        func hash(into hasher: inout Hasher) {
            hasher.combine(value)
        }
    }

I believe it should be applicable in other places as well. Let me know what you think, @pcuenca 🤗

Any chance of this being merged in soon? I'm trying to use a BERT model and this PR would be a huge help :)

I was trying out these changes since have DistilBERT model I wanted to test, but getting thrown unsupportedTokenizer("Distilbert") on try await AutoTokenizer.from(pretrained:..
Is there a chance support for DistilBERT is also added?

Superseded by #137


translate

https://github.com/sunfish-shogi/electron-shogi/issues/376

Codecov Report
Patch coverage: 85.52% and project coverage change: +0.12 :tada:

Comparison is base (75f118f) 71.46% compared to head (596689f) 71.58%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #453      +/-   ##
==========================================
+ Coverage   71.46%   71.58%   +0.12%
==========================================
  Files          69       69
  Lines       14655    14720      +65
  Branches     1894     1894
==========================================
+ Hits        10473    10538      +65
  Misses       2315     2315
  Partials     1867     1867

Impacted Files
Coverage Δ

src/common/settings/app.ts
75.71% <0.00%> (ø)

src/renderer/store/analysis.ts
67.65% <20.00%> (+0.13%)
:arrow_up:

src/common/i18n/translate.ts
97.79% <100.00%> (+0.12%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


configurable match rule for dashboard

as suggested in this comment https://github.com/containous/traefik-helm-chart/issues/143#issuecomment-610795188
If the dashboard tends to be exposed insecurely using usual Service chances high that it will be hosted using own URL.
So IngressRoute, if needed, should be able to filter traffic by Host header too.

@mmatur any specific reason why this is not getting merged?

Hi @den-is,
Thanks for your contribution and sorry for the late answer.
Unfortunately we will not be able to merge your PR.
We have documented how to expose the dashboard and we don't want to allow user to customized the dashboard exposition through the helm chart.
Thanks again for your contribution.


Rectangles in place of "inline code" text

The problem

Inline code text appears as what I presume are "missing character" rectangles.
Context
Operating system
Kubuntu Linux 20.04 x64
Affected projects
Markdown.AvaloniaDemo and Markdown.AvaloniaFluentDemo
A hypothesis
I'm not entirely certain as to why this happens, but I did notice that, of the fonts listed in FontFamilyCollecter.cs (menlo, monaco, consolas, droid sans mono, inconsolata, courier new, monospace,droid sans fallback), none are installed on my setup except the last one (droid sans fallback)...and uh, something seems a bit...off about that one...
Droid Sans Fallback weirdness
While I was checking which fonts were installed, I noticed that the font manager's preview of droid sans fallback looked rather peculiar, as shown here:

For comparison, here's a more typical font preview, for comparison (specifically of FreeSans):

Testing in LibreOffice Writer
Oddly enough though, Droid Sans Fallback is perfectly readable in LibreOffice Writer, as shown here:

(...though, given what comes next, I'm not fully convinced that LibreOffice Writer is actually rendering Droid Sans Fallback at all here...)
Testing in Microsoft Office Word 2010
...after seeing this bizarre discrepancy, I decided to try comparing the fonts in some other programs. First pick was Microsoft Office Word 2010, which produces some rather...telling results:

A potential fix
Just before submitting this issue, an idea popped into my head. I decided to try adding a font I knew I had installed - and which I knew would not misbehave the way Droid Sans Fallback did - to the list of fonts in FontFamilyCollecter, to see if that would help. First, I tried FreeMono (which I spelled as freemono in the code)...but this produced the same results as Droid Sans Fallback, no matter where I put it in the list. After reverting this, I instead tried adding DejaVu Sans Mono (which I spelled as dejavu sans mono in the code) to the list, placing it at index 0. I was pleasantly surprised when I built and ran the sample app with the changes, and was met by this:

Further experimentation showed that I could make dejavu sans mono the second-last entry in the list, and the text would still be displayed correctly as shown just above. To avoid any possibility of ambiguity, that final test (which was successful) had the font list modified as follows:
            string[] RequestFamilies = {
                "menlo",
                "monaco",
                "consolas",
                "droid sans mono",
                "inconsolata",
                "courier new",
                "monospace",
                "dejavu sans mono",
                "droid sans fallback"
            };

Closing note
I hope this gives you whatever you need to look into this, if/whenever you're able and willing. If you need anything more from me, feel free to @Splitwirez and I'll see what I can do next time I have a spare moment. Pretty cool library otherwise.
Oh and uh...I've not seen it spelled "CollectEr" before, only "CollectOr"...is that a regional thing?

Just tried https://github.com/whistyun/Markdown.Avalonia/tree/dd13a2d3459fa0e5ec7986309f9ef1b4be274122

Works without a hitch now :ok_hand:

I released v0.10.5 for fix it.

You alse could rewrite the style of the mardownscrollviewer control to custom the font family of your inline code

Thanks for reporting the issue and checking my commit. I released v0.10.5 for fix it.

Shouldn't the issue be closed?


missing PDO::MYSQL_ATTR_SSL_VERIFY_SERVER_CERT

file: CodeIgniter-3.1.8/system/database/drivers/pdo/subdrivers/pdo_mysql_driver.php
PDO::MYSQL_ATTR_SSL_VERIFY_SERVER_CERT is not set based on value in encrypt['ssl_verify']

I've searched case-insensitively for "verify" and it doesn't exist in that file. Please explain what you mean.

I guess what he meant:
He want the set the ssl_verify (based on PDO::MYSQL_ATTR_SSL_VERIFY_SERVER_CERT), thats missing in this block:
/system/database/drivers/pdo/subdrivers/pdo_mysql_driver.php Line ~ 163
$ssl = array();
empty($this->encrypt['ssl_key'])    OR $ssl[PDO::MYSQL_ATTR_SSL_KEY]    = $this->encrypt['ssl_key'];
empty($this->encrypt['ssl_cert'])   OR $ssl[PDO::MYSQL_ATTR_SSL_CERT]   = $this->encrypt['ssl_cert'];
empty($this->encrypt['ssl_ca'])     OR $ssl[PDO::MYSQL_ATTR_SSL_CA]     = $this->encrypt['ssl_ca'];
empty($this->encrypt['ssl_capath']) OR $ssl[PDO::MYSQL_ATTR_SSL_CAPATH] = $this->encrypt['ssl_capath'];
empty($this->encrypt['ssl_cipher']) OR $ssl[PDO::MYSQL_ATTR_SSL_CIPHER] = $this->encrypt['ssl_cipher'];

But MYSQL_ATTR_SSL_VERIFY_SERVER_CERT seems to be missing in the documentation. Check the comment from Desislav Kamenov on http://php.net/manual/de/ref.pdo-mysql.php


Update iopipe to 1.7.2 version (Python)

updated iopipe-python to latest version (https://github.com/iopipe/iopipe-python/releases)

@kolanos thoughts on removing the version pinning here entirely?


script context functions

closes #27

What's the status of this? Is something blocking?

What's the status of this? Is something blocking?

It was blocked on the CI pr I've rebased it now, diff should be pretty similar to your last review

Could you incorporate initMockState from #49?


How customer can be aware with upcoming changes in LCS?

Hello team,
Do you update this source regularly? I can see the last update is from 2nd of December 2019. Where customers can see Dynamics 365 LCS upcoming updates? We have a case where customer complained about not being informed that PG decided to remove File size information from Asset library and he is complaining about lack of transparency in LCS changes done by PG, also not announcing this change in any monthly releases. Can someone clarify? Thank you!

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

ID: 8d8d91ac-1947-9637-3b39-53d832483355
Version Independent ID: 69495807-2269-65d3-0825-2f4174d6096c
Content: What's new in Lifecycle Services (LCS)
Content Source: articles/fin-ops-core/dev-itpro/lifecycle-services/whats-new-lcs.md
Service: dynamics-ax-platform
Product: ****
Technology: ****
GitHub Login: @kfend
Microsoft Alias: kfend

@xbizzit, thanks for posting here!
The topic you commented on is no longer being updated, as mentioned in the "Important" note on that page.  For information about the latest features in Lifecycle Services, please see the Release plans.


Running multiple docker4drupal

It should be noted somewhere in documentation that if someone wishes to run two or more docker4drupal that they should just use other ports for all containers in docker-compose.yml (if I understood well. I was not familiar with docker before this and that enabled me to have more than one system running).
If you have some other suggestions how it should be done, I'd be glad to hear.
Here is my second configuration, just for reference (I used 7000 ports instead of 8000 which I used for first one):
version: "2"

services:
  mariadb:
    image: wodby/drupal-mariadb
    environment:
      MYSQL_RANDOM_ROOT_PASSWORD: 1
      MYSQL_DATABASE: drupal
      MYSQL_USER: drupal
      MYSQL_PASSWORD: drupal
    volumes:
      - ./docker-runtime/mariadb:/var/lib/mysql
#      - ./docker-runtime/mariadb-init:/docker-entrypoint-initdb.d # Place init .sql file(s) here.

  php:
    image: wodby/drupal-php:5.6 # Allowed: 7.0, 5.6.
    environment:
      PHP_SENDMAIL_PATH: /usr/sbin/sendmail -t -i -S mailhog:1025
      PHP_XDEBUG_ENABLED: 1 # Set 1 to enable.
    volumes:
      - ./:/var/www/html

  nginx:
    image: wodby/drupal-nginx
    environment:
      NGINX_UPSTREAM_NAME: php
      NGINX_DOCROOT: drupal # Relative path inside the /var/www/html/ directory.
      DRUPAL_VERSION: 7 # Allowed: 7, 8.
    volumes_from:
      - php
    ports:
      - "7000:80"

  pma:
    image: phpmyadmin/phpmyadmin
    restart: always
    environment:
      PMA_HOST: mariadb
      PMA_USER: drupal
      PMA_PASSWORD: drupal
      PHP_UPLOAD_MAX_FILESIZE: 1G
      PHP_MAX_INPUT_VARS: 1G
    ports:
     - "7001:80"

  mailhog:
    image: mailhog/mailhog
    ports:
      - "7002:8025"

  redis:
    image: redis:3.2-alpine

#  memcached:
#    image: memcached:1.4-alpine

#  solr:
#    image: solr:5.5-alpine
#    volumes:
#      - ./docker-runtime/solr:/opt/solr/server/solr/mycores
#    ports:
#      - "8003:8983"

Hi @macmladen,
Yes, the easiest way is using other ports for all containers.

I've added the description in https://github.com/Wodby/docker4drupal/commit/e2fbbc558e0332f4940a18192280bce02dd025a7
Later we might add a special container for routing


Consider a 'query' input?

Hello! 👋
I notice that right now there's no real way to say "apply to these repos that match my stuff" besides doing a big regex. I think a GitHub Search query input would be a good idea? 🤔
- uses<EMAIL_ADDRESS>  with:
    query: (user:${{ github.repository_owner }} OR org:my-org) fork:false node
    ...

This would eliminate the complicated solution of having a "repositories" and "repositories_list_regex" input in the readme; you could condense them to just a GitHub Search query that most users are already familiar with.

I'd be interested in porting some of the things that I did in https://github.com/jcbhmr/set-secrets-action to this project if this project is open to PRs.

📫 @jpoehnelt friendly ping  I saw you were the top-ranked contributor so I figured I'd ping you

Sorry, I missed this. Feel free to send prs!

@jcbhmr  are you still planning on working the list above? Asking as am particular interested in

Allow setting different app secrets #51

@DanyC97 as a tide-over until I do something, you can use the building gh CLI that's available on all GitHub Actions runners (windows, macos, ubuntu; they all have it) to set secrets/variables:
https://cli.github.com/manual/gh_variable_set
https://cli.github.com/manual/gh_secret_set
eg:
echo "$SECRET" | gh secret set -b -R octocat/my-project -a dependabot

And to answer your question: yes I do want to contribute.

Closing this as "completed" since @jpoehnelt you answered my question. ❤


TargetRubyVersion should be pulled from the Gemfile or .ruby_version files if available

We already have standard locations where the desired Ruby version lives.  Adding yet another to this list seems unnecessary.
My request is that if a Gemfile Ruby requirement is set, use that, otherwise fall back to .ruby_version if available and as a last resort, look for the TargetRubyVersion.

This was discussed most recently here: https://github.com/bbatsov/rubocop/issues/2956

@mikegee I did a search for TargetRubyVersion and didn't see that one.  Thank you!


Denovo Signature and Assigning Signature Matching Error

Hello, I am a student studying bioinformatics.
I am encountering an error while using SigProfilerExtractor and am posting an issue here.
I generated 1 denovo signature and decomposed it against COSMIC v.3.4, which indicated the presence of SBS5 and SBS21. However, the columns in 'COSMIC_SBS96_Activities.txt' file are labeled as SBS1, SBS5, and SBS96A. How can this error be resolved?
I am attaching an image.
Thank you.

Hi @KANGseungseok,
The de novo signature SBS96A is present because it was not accurately reconstructed (cosine similarity was less than 0.8) with the COSMIC signatures. You can read more about this in the following posts:
https://github.com/AlexandrovLab/SigProfilerExtractor/issues/128
https://github.com/AlexandrovLab/SigProfilerExtractor/issues/171
Importantly, this analysis is not recommended on samples that have total mutations counts of 2 to 4. The resulting analysis will be highly inaccurate due to the reconstruction that is performed. It may help to check the reconstructed samples, these can be found in the Solutions_Stats folder, and filter out samples that were poorly reconstructed (ie. reconstructed samples that have less than 0.9 cosine similarity).
Please reach out if you have any additional questions.
Thanks!


how to handle connection timeout?

Hello,
first of all, thank you for this library which had saved me time to "just implement rate limitation" :-).
I am currently struggling with the problem to handle connection timeouts.
Simple said, if I try to connect to non existing redis host, I don't get an error message. All I get is a hanging application for more than 10 Minutes.
I would prefer to throw an HTTPException to send my client a "Internal server error" or "Service temporary not available" message.
I could not solve this issue so far. Following my current way how I connect to the redis server.
@app.on_event("startup")
async def startup():

    try:
        redis_connection = redis.Redis(
            host=os.getenv('REDIS_HOST'),
            port=int(os.getenv('REDIS_PORT')),
            db=int(os.getenv('REDIS_DATABASE')),
            socket_connect_timeout=5,   # socket_connect_timeout applies only to blocking during the initial TCP handshake
            socket_timeout=5,           # socket_timeout applies to all blocking socket operations, ref: https://github.com/redis/redis-py/issues/722#issuecomment-607509357
            health_check_interval=10
        )

        logging.debug('Redis connection established',\
                      extra={'context': {'connection_parameters':\
                                             'host >>{0}<<, port >>{1}<<, db >>{2}<<.'.format(os.getenv('REDIS_HOST'),\
                                                                                os.getenv('REDIS_PORT'),\
                                                                                os.getenv('REDIS_DATABASE'\
                                                                                ))}})
    except redis.ConnectionError as exception:
        log_excepted_exception(exception, 'startup', 500, 'redis_connection_error', 'SYSTEM')
    except redis.TimeoutError as exception:
        log_excepted_exception(exception, 'startup', 500, 'redis_timeout_error', 'SYSTEM')

    await FastAPILimiter.init(
        prefix='fastapi_limiter_',
        redis=redis_connection
    )

Non of the three settings (socket_connect_timeout, socket_timeout or health_check_interval) could stop the application from "hanging".
I hope there is a big missing link I ran into. Have you ever tried to implement my kind of requirements?
Thanks in advance,
Stev

I'm not sure what you are trying to achieve.
If you want to let your users know that your FastAPI application is not setup, this is a FastAPI issue. startup events are setup before FastAPI accepts any requests, so you might need throw your own custom error upon catching the redis errors.
If you want to retry your connection to redis if the initial connection failed, you can use Retry and any kind of backoff.
See below for an example.
from redis.retry import Retry
from redis.backoff import ExponentialBackoff

import redis.asyncio as redis
from fastapi import Depends

app = FastApi()

@app.on_event("startup")
async def startup():
	url = "redis://..."
    retry = Retry(backoff=ExponentialBackoff(), retries=10)
    redis_instance = await redis.from_url(url, encoding="utf-8", decode_responses=True, retry=retry)
    await FastAPILimiter.init(redis_instance)

If you want to make FastApiLimiter work after redis went down and back up again, you'll have to wait until https://github.com/long2ice/fastapi-limiter/pull/33 gets merged.

@tcrasset thank you very much for all your shared knowledge.
I will give it a try and will give feedback if useful.
Best regards!


Loop all words to apply a class

I'm looking for a clean way to loop through the words of an element, where I wanted to apply individual CSS Classes to the words.
I tried to adapt the code from https://github.com/timdown/rangy/issues/130:
function printWords() {
  var it = rangy.createWordIterator(document.body, 0, 'forward');
  while ( (word = it.next()) ) {
    if (word.isWord) {
      console.log(word.toString());
    }
  }
}

window.onload = function() {
  rangy.init();
  printWords();
};

I tried to insert my Class Applier code (with applyToRange or applyToSelection), but everything has failed.
Meanwhile I think this is, because it's not possible to create a range or a selection with the token word or the iterator it.
Is there a solution to do this with the word iterator or is there a solution with a different iterator?

The above shown function printWords prints every word of a page.
Therefore, it seems to me a good starting point for doing what I want.
I have tried other ways like the following:
function printWords() {
	var range = rangy.createRange();
	range.collapseBefore(document.body);
	do {
		range.expand("word", { wordOptions: { includeTrailingSpace: false }});
		console.log(range.toString());
	}
	while (range.move("word", 1, { wordOptions: { includeTrailingSpace: true }}));
}

However, this version omits the first word of every phrase.

The problem with createWordIterator is that it relies on the DOM staying the same between calls to next(). You could use it to store a list of character ranges and iterate over this list, creating a range and applying a class to it for each character range, but Rangy doesn't make this easy because the object returned by next() does not contain a convenient range representing the word.
Your second example seems to work for me: http://jsbin.com/liqewoxowa/2/edit

Thank you very much for your reply and for the jsbin!
In my previous comment I mentioned, that it omits the first word of every phrase (I meant sentence – sorry for my bad English).
I cloned your jsbin to show what I meant: http://jsbin.com/yuxegomuva/1/
It seems that the punctuation marks cause the problem.

@timdown, can you elaborate more on how to do it with the createWordIterator way? That works best for my purpose, but as @Solaner mentioned, I can't apply a class on words.

NAVER - http://www.naver.com/
<EMAIL_ADDRESS>님께 보내신 메일 <Re: [rangy] Loop all words to apply a class (#309)> 이 다음과 같은 이유로 전송 실패했습니다.

받는 사람이 회원님의 메일을 수신차단 하였습니다.


iwd: revert test that's broken on aarch64-linux

And maybe others
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Proposed iwd fix as https://github.com/NixOS/nixpkgs/pull/274824


Rename segment_size in Layout and make it public

Problem
Currently segment_size is only available to use internally. I think we should make it public:
// without segment_size

let target = Rect {
    x: 0,
    y: 0,
    width: 10,
    height: 1,
};

let chunks = Layout::default()
    .direction(Direction::Horizontal)
    .constraints([Constraint::Min(0), Constraint::Min(0)].as_ref())
    .split(target);

dbg!(chunks);

// [src/layout.rs:764] chunks = [
//     Rect {
//         x: 0,
//         y: 0,
//         width: 0,
//         height: 1,
//     },
//     Rect {
//         x: 0,
//         y: 0,
//         width: 10,
//         height: 1,
//     },
// ]

// With segment_size

let target = Rect {
    x: 0,
    y: 0,
    width: 10,
    height: 1,
};

let chunks = Layout::default()
    .direction(Direction::Horizontal)
    .constraints([Constraint::Min(0), Constraint::Min(0)].as_ref())
    .segment_size(SegmentSize::EvenDistribution)
    .split(target);

dbg!(chunks);

// [src/layout.rs:764] chunks = [
//     Rect {
//         x: 0,
//         y: 0,
//         width: 5,
//         height: 1,
//     },
//     Rect {
//         x: 5,
//         y: 0,
//         width: 5,
//         height: 1,
//     },
// ]

Additional context
This was introduced in this PR with the caveat that the name will be reconsidered: https://github.com/ratatui-org/ratatui/pull/395#issuecomment-1685139772

I think @joshka's suggestion in that comment makes sense. I can take this one if you also agree.

I think there's possibly a little better name for this to be found - but I'm not certain.
Perhaps start with the user in mind and what they'd want this functionality to expose. SegmentSize doesn't seem obvious, and even Allocation doesn't obviously align with how to allocate the excess space available (or fill it, or ...)

Maybe something that has "preference" or "preferred" in the name?

preferred_allocation(AllocationPreferences::DistributeEvenly)
segment_size_preference(SegmentSizePreference::LastTakesRemaining)

We have to figure out 2 names:

Name for the function segment_size + name of the enum SegmentSize.
Name for the variants in the enum, only EvenDistribution and LastTakesRemainder at the moment

We may want to add more variants to SegmentSize in the future. One common one I've found wanting is TakesRemainder(usize) where TakesRemainder(segments.len() - 1) would do the same thing as LastTakesRemainder.
I group all these variants under the name "preference" because I imagine this is always going to be a weak constraint and is going to never be enforced.

One concern I have about this is that it bolts more features to a fairly limited layout algorithm (cassowary). We could instead consider whether it's more beneficial to put effort into replacing this with something taffy based.
I say this is "limited" not in the sense that cassowary can't do everything, but in the sense that if we want it to do something the we have to specifically implement that code. I.e. it's possible to add our own growing, shrinking, weighted values, etc. on top of cassowary, but once we write that, then we have to maintain our particular way of laying something out. If instead we accept that flexbox / grid is a reasonable layout approach, and we move towards that then we reduce our maintenance burden.
I case it's not obvious, my favorite type of feature is one that can be implemented by deleting more code than you write.
Either way, I'd suggest that it would be worth listing out the use cases for layout and then go from there rather than evolving it piecemeal.

I like the idea of moving to taffy. I did even look into it briefly to use taffy directly in my project to work up what a interface would be for users (but didn't get very far).
Do you think we'll be able to maintain the same Layout interface with taffy? If yes, then we can pause adding any more features to this. If not, I imagine we might have to integrate taffy with a TaffyLayout struct, while keeping the existing Layout as is (with deprecated warnings). And then after a few versions or after reasonable amount of time (e.g. 6months?) we can remove the existing Layout. My reasoning is that if the interface for users with taffy is going to be different than the existing Layout interface, I imagine that will break practically everyone's program with ratatui.

Do you think we'll be able to maintain the same Layout interface with taffy?

https://github.com/ratatui-org/ratatui/pull/385 does just that, but the rationale remains that we should think in terms of the use cases that we want to support - it may make sense to replace cassowary and add functionality, or it may make more sense to add Taffy as a secondary option with less focus on backwards compatible interfaces. Designing for the use cases first helps support either option.


gl.ClearCommand does not work on Mac

OS: MacOS Catalina
GPU: AMD

When gl.ClearCommand is run the resulting framebuffer is always blue: 0x0000ff3b
examples/tools/clear/clear.go is extremely laggy

Similar problems can be observed on Intel GPU.
More information about failing tests can be found in the CI build:
https://buildkite.com/pixiq/multiplatform-test/builds/8

Fixed by #94


shellescape buffer name

I have a system created path that has a space in it and which I unfortunately cannot get around of. I had naively copied the config in the readme which I'm sure works almost every time, but doesn't escape the path.
function()
	return {
		exe = "prettier",
        args = {"--stdin-filepath", vim.api.nvim_buf_get_name(0), '--single-quote'},
        stdin = true
	}
end

It's not really an issue per se, rather just a heads up for people that may come across something like this.
Anyway, I know almost nothing about nvim api or lua, so my solution right now is this:
local escape_path = function(path)
   return vim.api.nvim_eval('shellescape(\'' .. path .. '\')')
end

You can close this, or mention it, or conjure a more robust example for the readme.

Since the config is mostly in user space, we should probably document this instead. A way to deal with this is
function()
	return {
	  exe = "prettier",
          args = {"--stdin-filepath", vim.fn.fnameescape(vim.api.nvim_buf_get_name(0)), '--single-quote'},
          stdin = true
	}
end

Since not every formatter needs to have the file name in the args, there's not a whole lot we can do to verify that it's escaped.


Update expandrive

Switch to versioned url

Thank you for the contribution. It was merged directly as ad48223f8759dd056eb0518efeb40e97e717fc19 to keep commit history cleaner. Your contribution is still credited to you.


Dedicated API for controlled inputs

See gitter for previous discussion and some proposed solutions.
Basically, if you have a "controlled" input element (to borrow a React.js term), and you want to change its value (e.g. remove non-digits) after the user types, the most straightforward way to do that in Laminar doesn't work if input value is feeding from a signal or a var. Two compounding reasons for this:

you can't preventDefault onInput, it's not a cancelable event in the JS DOM, and

signal doesn't emit a value that is == to previous value.

So when user types in an "invalid" character like "n", the browser updates the input value, but the desired input value, controlled by the signal, doesn't change (if it was "12" and the user typed an invalid character like "n", it will remain "12"), so the signal doesn't emit anything, and so what the user typed is allowed to remain in the input, and the input's value diverges from the Var that is supposed to feed it.
I'm trying to find a reasonable solution to this. My latest thinking is, broadly speaking, to force-set the input's value to the controlling signal's current state after the onInput event has "finished processing" (unless at this point the input's value already matches the controlling signal's current value of course, in which case we don't need to do anything).
I think, if we define "finished processing" as "the transaction in which onInput event happened, and any descendant transactions created from it have completed", which in turn probably resolves to "wait until the transaction queue is empty" in case of transactions started from browser events, this should net us the right outcome. I'm still experimenting, and might run into issues, e.g. with multiple onInput listeners (because each onInput listener will create a separate transaction...), but at least it's looking promising.
After looking into this, I am rather annoyed by the status quo, so I'll try to get this into 0.12.0, although I might publish 0.12.0-m1 before that.

If you're wondering how React does controlled inputs, their state does not have a == check, which is a prerequisite for this problem. Also their event system is custom built so they have more control over it (Laminar just uses native DOM events). But if you're curious you can start here I think.

Alright, after a lot of experimentation, looks like the following API will probably make it:
val textState = Var("")
val boolState = Var(false)
div(
  // Text input
  input(
    controlled(
      value <-- textState.signal,
      onInput.mapToValue.map(_.filter(Character.isDigit)) --> textState.writer
    )
  ),
  // Checkbox
  input(
    typ("checkbox"),
    controlled(
      checked <-- boolState.signal,
      onClick.mapToChecked --> boolState.writer
    )
  )
)

What it does is make sure the value property always matches textState.signal, so basically behaves like React controlled components. This requires some magic (not too bad tho), so the new API is scoped to this controlled construct that accepts two parameters. Only one controlled block per element is allowed. Existing usage patterns of Laminar will be still available and unaffected.
I'm also considering adding a custom onEdit key so that users don't need to think whether they need to listen to onClick or onInput or onChange on checkboxes and stuff, but not sure if I'll be able to get it to work. Maybe later.

Explanation for why a dedicated controlled block is needed on gitter.


Recursive pipeline submission

Add functionality as requested in #24

Codecov Report
Patch coverage: 76.00% and project coverage change: -0.20% :warning:

Comparison is base (8ac7a58) 98.07% compared to head (2fa3e6b) 97.87%.
Report is 7 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #28      +/-   ##
==========================================
- Coverage   98.07%   97.87%   -0.20%
==========================================
  Files          28       28
  Lines        2805     2826      +21
  Branches      468      473       +5
==========================================
+ Hits         2751     2766      +15
- Misses         34       38       +4
- Partials       20       22       +2

Flag
Coverage Δ

unittests
97.87% <76.00%> (-0.20%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Changed
Coverage Δ

gaps/cli/pipeline.py
91.13% <76.00%> (-7.14%)
:arrow_down:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add new blog post issue template

Description of change
A short template for issues related to create new blog post content.
Checklist

[ ] npm test passes
[ ] documentation is changed or added (if applicable)
[ ] permission has been obtained to add new logo (if applicable)
[ ] contribution guidelines followed here

Think this needs a rebase @smlambert


fzf change directory not working in pop-os (debian)

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info

OS

[x] Linux
[ ] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[ ] bash
[x] zsh
[ ] fish

Problem / Steps to reproduce

install fzf sudo apt-get install fzf
cd ** <TAB> as per manual is not giving list of directories with fuzzy search option. Instead is giving all directory list as default shell(zsh).
Please let me know if you require more details.

See https://github.com/junegunn/fzf/issues/1866


0.5.4 Release Proposal

@ofrobots @cristiancavalli @JustinBeckwith @misterpoe PTAL.

LGTM

LGTM
On Jul 3, 2016 7:21 PM, "Justin Beckwith"<EMAIL_ADDRESS>wrote:

LGTM
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GoogleCloudPlatform/cloud-trace-nodejs/pull/272#issuecomment-230192568,
or mute the thread
https://github.com/notifications/unsubscribe/AAE0qWpc-zQcSstSG3ElfwlB5WIRVhg4ks5qSG45gaJpZM4JDtTf
.


version 4

hello,
would you like to upgrade to dropzone version 4?

Hey Vianney,
See #13, will try to get to it before the end of the week.

oups, indeed, i didn't see it, I close this one. thanks! there's not urgency :)


Update Layout.js

What kind of change does this PR introduce?

Does this PR introduce a breaking change?

What needs to be documented once your changes are merged?

👷 Deploy Preview for fluffy-croissant-f26e87 processing.

Name
Link

🔨 Latest commit
c9921307da4d5b99f72eb9e405fd402761dd02c2

🔍 Latest deploy log
https://app.netlify.com/sites/fluffy-croissant-f26e87/deploys/65cb1147e8f2c70008840803


Way to set token from UI in new version

Hi,
In older version we can set token from UI using:
window.swaggerUi.api.clientAuthorizations.add("key", new SwaggerClient.ApiKeyAuthorization("Authorization", "Bearer " + r.access_token, "header"));
In new version, i changed the index.html:
app.UseSwaggerUI(c =>
{
c.IndexStream = () => new FileStream($"{AppDomain.CurrentDomain.BaseDirectory}/wwwroot/index.html", FileMode.Open);
});
How can we set token from this custom index.html using javascript ?
or
How can we append more inputs in authentication page and make custom authentication ?

Please take a look at the code I added for another issue: https://github.com/domaindrivendev/Swashbuckle.AspNetCore/issues/971#issuecomment-604488724
It's the code I use to be able to specify a JWT beare token.
Should be adaptable to your case ;)

@ronaldhoek's solution seems like it should work for this usecase

@ronaldhoek's solution seems like it should work for this usecase


Throw copy exception on invalid utf8 string

Closes #2148

If you add Closes #2148 to your commit body, you needn't manually close the linked issue on merge.

I'm fairly certain CSV reading isn't the only place invalid UTF-8 can enter a string column. We need to make sure that it every cast function fails on invalid UTF-8.
This is yet another reason to create a harmonized type system (besides better support across all parts of the system).

See #2148 for why this PR is not sufficient. We should probably aim to fix this before the next major release, it's not a small task in my opinion.


Allow dynamic scaling of clusters

Right now clusters are statically configured and thus can't take advantage of new machines that are added to the cluster. This is a basic piece of functionality that needs to be added to pfs.
This issue will serve as the design discussion for this feature. (Details to come later).
We should plan on having this implemented for 0.4.

Right now this is done by de-scheduling and rescheduling services in fleet. I imagine this will be improved with Kubernetes integration and will eventually result in a "rebalance" command of some sort.

This has been subsumed by replication work being done now and isn't serving much purpose.


feat bucket replicate: added flag to ignore blocks marked for deletion

Hi, we bumped into some race conditions with tools bucket replicate  from bucket with a compactor running, such as replication of a block that just gets deleted by compactor.
Also, this can save some bandwidth, since it's mostly useless to sync blocks that will be deleted eventually.
Would you be ok accepting this? I'm open to any suggestions., thanks!
Changes

added a new flag --ignore-marked-for-deletion to the thanos bucket replicate command

Verification

added tests
tested manually

Thanks, @GiedriusS for the swift review!
The white space changes were caused by the make docs, not sure why. Rolled it back.

Passing through the docs test is a task on it's own I see :smile:

Only got one question, other than that, LGTM! :)

Not sure why but the PR diff broke GitHub somehow :sweat_smile:
Getting 500 on the changes page..


Reservation-form

In this PR

I created a form that can make reservation

@jahongiry  Can you check this out?

Approved!!!
Looks nice and clean 👍
You can merge to Dev branch now


The view dbo.vTableSizes never returns results.

[Enter feedback here]

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

ID: 473726a0-19e3-a144-56ca-a66058f9ff67
Version Independent ID: 820466f5-2ec1-a3f8-51ae-8ab02fed9395
Content: Designing tables - Azure Synapse Analytics
Content Source: articles/synapse-analytics/sql-data-warehouse/sql-data-warehouse-tables-overview.md
Service: synapse-analytics
Sub-service: sql-dw
GitHub Login: @WilliamDAssafMSFT
Microsoft Alias: wiassaf

I executed this view with a where clause for a schema that contains ~100 tables.  The query ran for almost 2 hours and never returned any results.

Ensure that index maintenance occurs on a regular basis and after large data changes. Do that, then try the vTableSizes again.
If you have any deeper technical questions, this may not be the best forum, as this space is for documentation updates. I recommend Microsoft Q & A (Azure Synapse Analytics), Azure Synapse Analytics feedback, Microsoft Support, or your support partners. If this persists, open a support ticket. This isn't the space for technical support.
Regardless, thanks again for this feedback, your suggestions help improve our documentation.
#please-close

#assign:WilliamDAssafMSFT


Revert "[SW-199559] use native checkpointing under compile mode"

This reverts commit c39a0b7d778eaf53a3b605e4260c4a742b5905a2.
Change-Id: Iafaacd11faf5eff6bd68ed146b2978683e295a9b
What does this PR do?

Fixes # (issue)
Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests?

@astachowiczhabana created a revert PR since it causes regression. Will reland after fixing them.


silicon: 0.5.2 -> 0.5.3

Description of changes
Upgrade package to 0.5.3 (changes)
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

result/bin/silicon pkgs/by-name/si/silicon/package.nix -o test.png -l nix

Result of nixpkgs-review pr 344634 run on x86_64-linux 1

1 package built:

silicon


OneToOne relationship - unique and exclude

Laravel Version: 5.4
Voyager Version: 0.11.5
PHP Version: 7.0
Database Driver & Version: MySQL

Description:
I have an table that stores all of my RFID cards, and a table with my users that references the RFID card in one-to-one relationship. This works as expected.
I need to build upon this and add two things

Unique
When adding an RFID card to a user the dropdown should be filled only with those RFID cards that are not already assigned to a user

Exclude
RFID table has an enabled column. I wish to exclude those RFID cards from the list that are disabled ( enabled == 0 )

Is this possible without upgrading Voyager?

This is a Laravel question, not a Voyager one.  Just update your model's relationship to limit the results: https://laravel.com/docs/5.4/eloquent-relationships


Remove refer_secret_id from corpora terraform variables

Context: https://github.com/chanzuckerberg/single-cell-infra/pull/75#discussion_r434850348
This secret variable ends up being sent over plaintext which means that it can be snooped, effectively making the S3 bucket that the secret is supposed to protect, entirely public. The solution here is to ensure that the s3 bucket that is serving the website does not contain anything restricted and that everything is fine for public consumption. Then this variable needs to be removed altogether (requires PR in single-cell-infra repo).

I think this one can be closed. The refer_secret still serves an important purpose. It prevent user from access files directly from s3, which would be more expensive for us and slower for users. The worst case is a user discovers the secret and accesses the bucket, but this is an edge case that doesn't help anyone.

fixed by #354


Non-interactive dev mode output

Original Discussion: https://www.pika.dev/npm/snowpack/discuss/417
/cc @abumalick, @FredKSchott
Currently, our dev dashboard is live-updating, and will clear the terminal regularly to keep the output meaningful. The only way to clear the terminal is to clear the ENTIRE terminal, so that you lose your history / ability to scroll up. This is especially harmful when running Snowpack in the context of a workspace, with something like ws-run.
We should support a non-interactive mode, where you can run Snowpack dev without owning the entire terminal. This would mean outputting logs but not clearing the page. This would be noisy (ex: TypeScript would reprint every type error on every save). but for some use cases that would still be preferable.

A naive approach would be to only clear the terminal if process.stdout.isTTY is true. But, we then re-output the entire dashboard, so that would be VERY noisy..
A better approach would be to have a non-interactive mode to print.ts that skips printing the dashboard entirely if process.stdout.isTTY, and just prints new messages to the console as they show up. We should add a log for every file that is saved, to split up the logs for the viewer.

Solution 2 shouldn't actually be too tricky. I'd love some help if anyone is interested!

Superseded by https://github.com/pikapkg/snowpack/issues/648


Dynamic range for weapons

Add a new range attribute which can be used to define a range for a weapon.

Is added?


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_b2a3y56esf/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_b2a3y56esf/clone/recipe/meta.yaml:37: http_url: http://euske.github.io/pdfminer/index.html is not https
ERROR: /tmp/abs_b2a3y56esf/clone/recipe/meta.yaml:37: missing_license_family: The recipe is missing the about/license_family key.
ERROR: /tmp/abs_b2a3y56esf/clone/recipe/meta.yaml:37: missing_documentation: The recipe is missing a doc_url or doc_source_url
WARNING: /tmp/abs_b2a3y56esf/clone/recipe/meta.yaml:37: missing_description: The recipe is missing a description
ERROR: /tmp/abs_b2a3y56esf/clone/recipe/meta.yaml:37: missing_dev_url: The recipe is missing a dev_url
Errors were found


[BugFix] Fix bug query INFORMATION_SCHEMA when eq database()

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #11361
Problem Summary(Required) ：

This was introduced in #5124, mainly because the judgment was not complete, and it was thought that the right side could only be ConstantOperator, but it might actually be CallOperator
Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

run starrocks_fe_unittest

run starrocks_fe_unittest

[FE PR Coverage Check]
:heart_eyes: pass : 2 / 2 (100.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/plan/PlanFragmentBuilder.java
2
2
100.00%
[]

@Mergifyio backport branch-2.4

https://github.com/Mergifyio backport branch-2.3


openstack-crowbar: add SES integration support (SCRD-7076)

Add SES integration test coverage to the openstack-crowbar job.
This PR adds SES integration support for the Crowbar cloud
automated deployment:

[x] update the Jenkinsfile pipeline definition to conditionally
create a one-node virtual SES cluster running alongside the
Crowbar virtual cloud infrastructure
[x] update the deploy-crowbar.yml playbook to conditionally
generate the SES artifacts (users/keys/pools) and cloud
configuration via the SES salt API and store the configuration
on the Crowbar admin node
[x] load the SES configuration using the Crowbar CLI (requires
this crowbar-client PR)
[x] create a batch scenario with SES enabled in the glance/cinder
barclamps, to be used with the openstack-crowbar Jenkins job

This PR includes a slight refactoring of the SES Ardana integration artifacts
(roles, playbooks, variables) necessary to prepare them to be reused for
Crowbar:

rename the ardana-ses-integration.yml playbook  to cloud-ses-integration.yml
rename the ses_ardana_integration role to ses_cloud_integration
rename role variables
make the ansible user dependent on cloud type

This change also requires moving the task that disables the
OpenStack DHCP server, because the SES cluster also needs it.

do we maybe want to adda  new toplevel dir (common/) or the like instead of ardana/ ? this way it might be easier to navigate going forward. Please keep in mind that we'll have to replicate / integrate also cloud10 ..

do we maybe want to adda new toplevel dir (common/) or the like instead of ardana/ ? this way it might be easier to navigate going forward. Please keep in mind that we'll have to replicate / integrate also cloud10 ..

@dirkmueller the Crowbar ECP CI reuses a lot of ansible code from Ardana. Yes, I agree we should have a common location for shared code (that we may also share with cloud10), but as the Crowbar ECP CI is still under heavy development it's too early to tell what that common code is and how (or if) we should do the split.
Based on what I've seen so far, I lean towards having one location (not ardana, not crowbar, but cloud) that covers both Ardana and Crowbar. As for cloud10, I doubt we'll be reusing this ansible code.

Testing a fully automated SES Crowbar deployment with: https://ci.suse.de/blue/organizations/jenkins/openstack-crowbar/detail/openstack-crowbar/177/pipeline

Testing for regression against openstack-ardana with: https://ci.suse.de/blue/organizations/jenkins/cloud-ardana9-job-std-min-gerrit-x86_64/detail/cloud-ardana9-job-std-min-gerrit-x86_64/345/pipeline


fix(HLS): Avoid get basic info when the first segment is a gap

Do we have to get the basic info from the first segment? If the presentation starts on a gap, couldn't we just request the second segment? Or the third segment if the second segment is also a gap, etc.

Yes, it could be done, but first I have to do a refactor which is not trivial, this is the simple short-term solution, the other I need one or two weeks to implement.


pi4 64bit os not connecting to 5g wifi network (2.4 okay)

Using latest OS. Devices cannot connect to 5g wifi networks. I have tried multiple networks on multiple PI4's. 2.4 seems to work fine.
dmesg
[   19.192358] brcmfmac: power management disabled
[   44.008818] IPv6: ADDRCONF(NETDEV_UP): wlan0: link is not ready
[   44.557136] IPv6: ADDRCONF(NETDEV_UP): wlan0: link is not ready
nmcli info
GENERAL.DEVICE:                         wlan0
GENERAL.TYPE:                           wifi
GENERAL.HWADDR:                         F6:55:AA:AA:AA:AA
GENERAL.MTU:                            1500
GENERAL.STATE:                          30 (disconnected)
GENERAL.CONNECTION:                     --
GENERAL.CON-PATH:                       --
nmcli con show
Wired connection 1  8f677aa3-eedb-39ed-97d5-b11eafcc443c  ethernet  eth0
supervisor0         cd69f527-996b-48ca-a728-8e23d4b3024a  bridge    supervisor0
myasus5g            6669052a-13ae-44a3-a55d-abfb577bee02  wifi      --
system configuration
root@a46aff9:~# uname -a
Linux a46aff9 4.19.71 #1 SMP PREEMPT Fri Oct 25 14:00:45 UTC 2019 aarch64 aarch64 aarch64 GNU/Linux
root@a46aff9:~# nmcli -v
nmcli tool, version 1.20.2
root@a46aff9:~# wpa_supplicant -v
wpa_supplicant v2.7
Copyright (c) 2003-2018, Jouni Malinen<EMAIL_ADDRESS>and contributors
root@a46aff9:~# cat /etc/os-release
ID="balena-os"
NAME="balenaOS"
VERSION="2.44.0+rev3"
VERSION_ID="2.44.0+rev3"
PRETTY_NAME="balenaOS 2.44.0+rev3"
MACHINE="raspberrypi4-64"
VARIANT="Production"
VARIANT_ID="prod"
META_BALENA_VERSION="2.44.0"
RESIN_BOARD_REV="a4f4d1a"
META_RESIN_REV="7fed82f"
SLUG="raspberrypi4-64"

Can you also try with the new firmware? https://github.com/RPi-Distro/firmware-nonfree/issues/5
We have not yet included it but we will

Mine isn't displaying when using the newest development image I downloaded today. Was there something I needed to enable? Or a certain wifi configuration needed from my router?

Hi @Poltergeisen
I checked now with the 2.54.3+rev3 Pi4 image from staging and I'm able to connect to a 5GHz ap:
root@f60831f:~# iw dev
phy#0
        Interface wlan0
                ifindex 3
                wdev 0x1
                addr dc:a6:32:c2:20:fb
                ssid Balena5GHz
                type managed
                channel 36 (5180 MHz), width: 40 MHz, center1: 5190 MHz
                txpower 31.00 dBm

This image has an updated firmware as well as the 5.4 kernel, and the boot firmware on my Pi is the stable one from 31 July - pieeprom-2020-07-31.bin.
Does the 2.54.3+rev3 image work for you?


feat: add logging (PROOF-788)

Rationale for this change
Set up a logging framework to more quickly diagnose issues. This PR only adds some basic logging for the inner product proof. I will follow up with another PR to add more extensive logging.
What changes are included in this PR?

adds spdlog as a dependency
read an environmental variable BLITZAR_LOG_LEVEL to control the logging level
adds an info log function that is a light wrapper around spdlog

Are these changes tested?
Verified logging locally

:tada: This PR is included in version 1.31.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[Drawer] component overlayClassName

Drawer component overlayClassName not working
The overlayClassName prop does not get applied to the Drawer component.  Can we get this fixed?
www.voicerecordpro.com

Versions 0.16.7

Material-UI:
React:
Browser:

I can't see a overlayClassName property provided.

Please provide a reproduction test case next time. For instance, you can use http://www.webpackbin.com/4yeaqq5_z. Thanks!


Assist to convert for loop to Iterator::for_each

For example:
// Before
for (pixel_mut_ref, pixel_value) in pixels
    .get_frame()
    .iter_mut()
    .zip(screen_pixel_buf.subpixels())
{
    *pixel_mut_ref = pixel_value;
}

// After
pixels
    .get_frame()
    .iter_mut()
    .zip(screen_pixel_buf.subpixels())
    .for_each(|(pixel_mut_ref, pixel_value)| *pixel_mut_ref = pixel_value);

It might also be useful to have an assist that goes the other way.

Since #7741 has been merged, should this issue be closed and a new one be created for ‘for_each to for’, or should we keep this one and repurpose it?

I did say it should be a new issue.

I did say it should be a new issue.

Oops, sorry, I must’ve missed it.


useSingleQuotes replaces double quotes in KSS documentation(commented section)

useSingleQuotes rule will format double quote from commented text . We are using KSS documentation in which we have defined our markup in scss files and this rule replace all double quote from commented markup. e.g.
before running formatter:
/* <ex-input-form labelText= "this doesn't format" */
after running formatter:
/* <ex-input-form labelText= 'this doesn't format'> */
which leads error while rendering KSS documentaion.
please help on this.

Sorry for the delay in replying, I was on vacaton. I'll try to fix this tomorrow.

I updated extension and if you're using useSingleQuotes setting now, double quotes in comments will remain untouched. I tested multiple different cases (default CSS comments - single line/multi line, Sass comments, inline comments) and everything works fine.
For testing purposes I used examples from here with double quotes inside of comments:
https://developer.mozilla.org/en-US/docs/Web/CSS/Comments
http://sass-lang.com/documentation/file.SASS_REFERENCE.html#Comments________and_____comments
http://warpspire.com/kss/syntax/
Please update extension to the latest version on your side and let me know if everything is good. Thanks.

I'll consider this as done. If there is any additional issue on your side, feel free to reopen this issue or create new one. Thanks.

Hi Sasa,
Sorry for the delay, today I have tried same issue with latest version and found it working fine for us. Thanks for help.

Hi Sasa,
I also have one observation regarding sass formatter i.e. formatter is not splitting &.abc, &.def into below given format:
&.abc,
&.def {
//sample code
}
Please look if you can help us in this.

Hi saxena-ashutosh,
Thanks for checking. I'm glad to help you.
About 2nd topic, that is how sass-convert console tool work. For those cases formatter will fix only spacing and indention, in this way formatter will not break user coding style. Someone and/or some css frameworks like to put all selectors in single line, and others like to put each selector in new line. I can't do anything around this.
Best


Reveal open when existing modal is open fails

Since 1c966e7, opening a reveal modal when an existing modal is open fails with the error:
TypeError: undefined is not an object (evaluating 'target.attr')
The problem line of code in foundation.reveal.js is:
159: if (modal.hasClass('open') && target.attr('data-reveal-id') == modal.attr('id')) {
When triggered with $someContainer.foundation('reveal', 'open'), target is undefined, causing that line of code to fail.

When instantiating the reveal with options, the target becomes indeed the options object as @sveetch commented. E.g.
$dialog.foundation( 'reveal', 'open', {
    animation: 'none'
} );

Otherwise target is always undefined for me.
With an options object the error is:  Uncaught TypeError: undefined is not a function (because the target has no attr() method...)
Without an options object the target is undefined so the error is:  Uncaught TypeError: Cannot read property 'attr' of undefined
What seems pretty clear when looking at the code is that target [can be falsy] (https://github.com/zurb/foundation/blob/master/js/foundation/foundation.reveal.js#L140)
Without setting a value for target an attempt is made, a few lines further, to read an attribute of this falsy target if the modal is open.
So if target can indeed be falsy, this is a bug.
I pasted the complete code below with 2 comments.
      if (target) {
        if (typeof target.selector !== 'undefined') {
          // Find the named node; only use the first one found, since the rest of the code assumes there's only one node
          modal = self.S('#' + target.data(self.data_attr('reveal-id'))).first();
        } else {
          modal = self.S(this.scope);

          ajax_settings = target;
        }
      } else {
        // ** TARGET IS FALSY AND REMAINS SO  *
        modal = self.S(this.scope);
      }

      var settings = modal.data(self.attr_name(true) + '-init');
      settings = settings || this.settings;

      // AN ATTEMPT IS MADE TO READ THE ATTRIBUTE OF FALSY TARGET IF MODAL IS OPEN
      if (modal.hasClass('open') && target.attr('data-reveal-id') == modal.attr('id')) {
        return self.close(modal);
      }

I ran into this today as well. +1

+1, please, I am also running into this.

Much simpler workaround for now (haven't tested thoroughly yet though).
$("#ContentModal").removeClass("open");
This will short circuit the if statement causing the problem - but it won't close the modal, so you'll have to close and refill it yourself.

There is a recent pull request that I think will solve this issue. https://github.com/zurb/foundation/commit/586fcb24779ac82bab22e178428b15b9b3dd256a

Hi Guys,
I believe this bug is still not resolved. Please take a look at my simple example: http://www.kabasinskas.lt/stuff/Reveal_test/ you will see JS error after clicking on link inside first reveal window.

It is an issue. You can fix it by changing the reference of this to self. See https://gist.github.com/FineLineAutomation/e1c06142e5afaa98cfba#file-foundation-reveal-js-L210-L212 for the lines I changed.

@FineLineAutomation I am using foundation version from github in my example.
if you would look at foundation.reveal.js file in my example, you would see these lines in place: http://www.kabasinskas.lt/stuff/Reveal_test/foundation/js/foundation/foundation.reveal.js
Still even with these lines, issue does exist.

Also running into this issue.

+1

still an issue...

Also ran into this lately on 5.5.2

+1

i think this is fixed in #6517

I still have it;
if (modal.hasClass('open') && target.attr('data-reveal-id') == modal.attr('id')) {
        return self.close(modal);
      }

Uncaught TypeError: Cannot read property 'attr' of undefined

+1

I still have it too


Resource

Description
Adding resource parameter. Running resource.create() in the distro. This will greatly speed up startup since it will only be called once.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API change check
API changes are not detected in this pull request.


Firebase/MLVisionBarcodeModel was resolved to 5.0.0,

firebase_core: Using Firebase SDK version '7.3.0' defined in 'firebase_core'
firebase_crashlytics: Using Firebase SDK version '7.3.0' defined in 'firebase_core'
firebase_messaging: Using Firebase SDK version '7.3.0' defined in 'firebase_core'
firebase_ml_vision: Using Firebase SDK version '7.3.0' defined in 'firebase_core'
[!] CocoaPods could not find compatible versions for pod "Firebase/CoreOnly":
In Podfile:
Firebase/MLVisionBarcodeModel was resolved to 5.0.0, which depends on
Firebase/CoreOnly (= 5.0.0)
firebase_core (from `.symlinks/plugins/firebase_core/ios`) was resolved to 0.7.0, which depends on
  Firebase/CoreOnly (= 7.3.0)

firebase_messaging (from `.symlinks/plugins/firebase_messaging/ios`) was resolved to 7.0.3, which depends on
  Firebase/CoreOnly (~> 7.3.0)

CocoaPods could not find compatible versions for pod "FirebaseCore":
In Podfile:
firebase_core (from .symlinks/plugins/firebase_core/ios) was resolved to 0.7.0, which depends on
Firebase/CoreOnly (= 7.3.0) was resolved to 7.3.0, which depends on
FirebaseCore (= 7.3.0)
firebase_ml_vision (from `.symlinks/plugins/firebase_ml_vision/ios`) was resolved to 0.1.1, which depends on
  Firebase/MLVision (= 7.3.0) was resolved to 7.3.0, which depends on
    FirebaseMLVision (~> 7.3.0-beta) was resolved to 7.3.0-beta, which depends on
      FirebaseCore (~> 7.0)

CocoaPods could not find compatible versions for pod "nanopb":
In Podfile:
firebase_core (from .symlinks/plugins/firebase_core/ios) was resolved to 0.7.0, which depends on
Firebase/CoreOnly (= 7.3.0) was resolved to 7.3.0, which depends on
FirebaseCore (= 7.3.0) was resolved to 7.3.0, which depends on
FirebaseCoreDiagnostics (~> 7.0) was resolved to 7.4.0, which depends on
nanopb (~> 2.30907.0)
firebase_crashlytics (from `.symlinks/plugins/firebase_crashlytics/ios`) was resolved to 0.4.0-1, which depends on
  Firebase/Crashlytics (= 7.3.0) was resolved to 7.3.0, which depends on
    FirebaseCrashlytics (~> 7.3.0) was resolved to 7.3.0, which depends on
      nanopb (~> 2.30906.0)

Seems like a duplicate of #4625
Still not fixed though...

Seems like a duplicate of #4625
Still not fixed though...

There are some other incompatibilities but the issue is quite vague and this will depend on https://github.com/FirebaseExtended/flutterfire/issues/4625 anyway.
Closing this as a duplicate of https://github.com/FirebaseExtended/flutterfire/issues/4625

There are some other incompatibilities but the issue is quite vague and this will depend on https://github.com/FirebaseExtended/flutterfire/issues/4625 anyway.
Closing this as a duplicate of https://github.com/FirebaseExtended/flutterfire/issues/4625


[New chain]: Cronos Mainnet 1.4.1

Summary

https://github.com/safe-global/safe-deployments/pull/621
https://github.com/ethereum-lists/chains/blob/master/_data/chains/eip155-25.json

Chain ID
25
Chain detail URL
https://chainlist.org/chain/25
RPC URL
https://cronos.drpc.org
Blockscout Client URL
No response
Etherscan Client URL
No response
Etherscan Client API URL
No response
Version
1.4.1
Address (Master copy)
0x41675C099F32341bf84BFc5382aF534df5C7461a
Deployment Tx hash (Master copy)
0x0f4db513375b44f78d26b96bc8f5274020de52efe7d534e11bf2458d1a6f5006
Block explorer URL (Master copy)
No response
Address (Master copy L2)
0x29fcB43b46531BcA003ddC8FCB67FFE91900C762
Deployment Tx hash (Master copy L2)
0x865de5ea21c7309234c71b056ff6bbc1e3d393add28f8b5ad3b6f83490bf22ba
Block explorer URL (Master copy L2)
No response
Address (Proxy factory)
0x4e1DCf7AD4e460CfD30791CCC4F9c8a4f820ec67
Deployment Tx hash (Proxy factory)
0xb9ef22cbde010b9a4bb95a928bc32a0b4640ef0a40e6dfec295590b30977f4ff
Block explorer URL (Proxy factory)
No response

/execute


Exception - Microsoft.Bot.Builder.AI.Orchestrator.OrchestratorRecognizer: "Failed to find or load Model with path" Exception during initialization: bad allocation

Describe the bug
After the bot web app running for a few days, started getting this exception.
This consistently repros for web app services that have multiple instances, and we have to scale up and down to get rid of the bad nodes.
Would you please help to understand what could be the possible reason? and how should we investigate?
Exception:
Failed to find or load Model with path C:\home\site\wwwroot\generated\orchestratorModels\english
Call Stack:
System.InvalidOperationException
Microsoft.Bot.Builder.AI.Orchestrator.OrchestratorRecognizer.b__39_0
[external code]
Microsoft.Bot.Builder.BotAdapter+d__23.MoveNext
Inner exception System.ApplicationException handled at Microsoft.Bot.Builder.AI.Orchestrator.OrchestratorRecognizer.b__39_0
Microsoft.BotFramework.Orchestrator.Orchestrator..ctor
Microsoft.Bot.Builder.AI.Orchestrator.OrchestratorRecognizer.b__39_0

I can confirm this happens in the centos stream. I don't find any reasonable way to find the reason.


Upgrade package dependencies to fix security issues

Attempts to fix #9.

Added another upgrade to fix selenium-standalone to work with the latest Selenium driver.


C26149: HTML content in MD

Hello, @rloutlaw   ,
Localization team has reported source content issue that causes issue for localization. The file is written in HTML. It should be in MD syntax.
Please review the comment tag added and reply with explanation if fix is needed or not. If you make related fix in another PR then share your PR number so we can confirm and close this PR.
Many thanks in advance.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/index.experimental.md
:white_check_mark:Succeeded
View (azure-node-latest)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

No fix is needed. The HTML in this content is intentional as part of a en-us A/B experiement.

Thanks @rloutlaw ,
I will notify localization team.
Regards.


Travis: Use codecov instead of coveralls

Changes modeled after:
https://github.com/codecov/example-python/blob/3f28e93cbc1d266c30a5bdf43126ed44fbeb009d/.travis.yml

Current coverage is 93.15% (diff: 100%)

No coverage report found for master at 5ee5439.

Powered by Codecov. Last update 5ee5439...9a96b83


List, add & edit school type

Codecov Report

Merging #51 (46e62dd) into master (12f28db) will decrease coverage by 0.15%.
The diff coverage is 92.48%.

@@            Coverage Diff             @@
##           master      #51      +/-   ##
==========================================
- Coverage   93.06%   92.91%   -0.16%
==========================================
  Files          74       89      +15
  Lines         707      833     +126
  Branches       29       34       +5
==========================================
+ Hits          658      774     +116
- Misses         49       59      +10

Impacted Files
Coverage Δ

...tructure/School/Repository/SchoolTypeRepository.ts
35.71% <35.71%> (ø)

api/src/Domain/School/School.entity.ts
93.87% <88.88%> (-1.37%)
:arrow_down:

.../Application/School/Command/CreateSchoolCommand.ts
100.00% <100.00%> (ø)

...ation/School/Command/CreateSchoolCommandHandler.ts
100.00% <100.00%> (ø)

...ion/School/Command/Type/CreateSchoolTypeCommand.ts
100.00% <100.00%> (ø)

...ool/Command/Type/CreateSchoolTypeCommandHandler.ts
100.00% <100.00%> (ø)

...ion/School/Command/Type/UpdateSchoolTypeCommand.ts
100.00% <100.00%> (ø)

...ool/Command/Type/UpdateSchoolTypeCommandHandler.ts
100.00% <100.00%> (ø)

.../Application/School/Command/UpdateSchoolCommand.ts
100.00% <100.00%> (ø)

...ation/School/Command/UpdateSchoolCommandHandler.ts
100.00% <100.00%> (ø)

... and 26 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #51 (46e62dd) into master (12f28db) will decrease coverage by 0.15%.
The diff coverage is 92.48%.

@@            Coverage Diff             @@
##           master      #51      +/-   ##
==========================================
- Coverage   93.06%   92.91%   -0.16%
==========================================
  Files          74       89      +15
  Lines         707      833     +126
  Branches       29       34       +5
==========================================
+ Hits          658      774     +116
- Misses         49       59      +10

Impacted Files
Coverage Δ

...tructure/School/Repository/SchoolTypeRepository.ts
35.71% <35.71%> (ø)

api/src/Domain/School/School.entity.ts
93.87% <88.88%> (-1.37%)
:arrow_down:

.../Application/School/Command/CreateSchoolCommand.ts
100.00% <100.00%> (ø)

...ation/School/Command/CreateSchoolCommandHandler.ts
100.00% <100.00%> (ø)

...ion/School/Command/Type/CreateSchoolTypeCommand.ts
100.00% <100.00%> (ø)

...ool/Command/Type/CreateSchoolTypeCommandHandler.ts
100.00% <100.00%> (ø)

...ion/School/Command/Type/UpdateSchoolTypeCommand.ts
100.00% <100.00%> (ø)

...ool/Command/Type/UpdateSchoolTypeCommandHandler.ts
100.00% <100.00%> (ø)

.../Application/School/Command/UpdateSchoolCommand.ts
100.00% <100.00%> (ø)

...ation/School/Command/UpdateSchoolCommandHandler.ts
100.00% <100.00%> (ø)

... and 26 more

Continue to review full report at Codecov.

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


SRP server is disabled when the OTBR loses WLAN connectivity

Describe the bug
SRP server is disabled when the OTBR loses WLAN connectivity.
To Reproduce Information to reproduce the behavior, including:

Git commit id: 569d0733c6d76d0b55f1db40aabd40eaf7ac4fb0
IEEE 802.15.4 hardware platform: Nordic nrf52840
OTBR services built with WEB_GUI=1 NAT64=1 INFRA_IF_NAME=eth0 OTBR_MDNS=avahi ./script/setup
Raspberry Pi (OTBR) is connected to a wifi AP running on my laptop, which allows me to easily turn it on and off. I am logged in on the OTBR through a USB-UART adapter.

Expected behavior
The goal was to make sure the Thread devices can communicated with the LwM2M server running on the OTBR, even when it is not connected to the internet or any other network. Following the great advice given here I wanted to make the LwM2M server discoverable with SRP on the Thread network, instead of relying on Avahi like I was until now.
When the Wifi AP is up I get the following on the OTBR:
$ sudo ot-ctl netdata show
Prefixes:
fdfa:f767:79d8:1::/64 paos low 8400
Routes:
fc00::/7 sa med 8400
fdfa:f767:79d8:2:0:0::/96 sn low 8400
Services:
44970 01<PHONE_NUMBER>0e10 s 8400
44970 5d fd4200bc7d7f4bd4bf8d233440068760d123 s 8400
Contexts:
fdfa:f767:79d8:1::/64 1 c
Done

$ sudo ot-ctl srp server state
running
Done

As soon as I bring the Wifi AP down it seems like the SRP server is disabled:
$ sudo ot-ctl netdata show
Prefixes:
Routes:
Services:
44970 01<PHONE_NUMBER>0e10 s 8400
Contexts:
fdfa:f767:79d8:1::/64 1 -
Done

$ sudo ot-ctl srp server state
disabled
Done

Is this expected? Am I supposed to manually restart the SRP server? What would you recommend to make sure that the services that I want to be advertised by the OTBR's SRP client are persisted through state changes in the otbr-agent, or reboots of the OTBR?

This is a feature/behavior of SRP srver (AutoEnableMode).
It can be controlled using SPR servers's API: otSrpServerSetEnabled() and otSrpServerSetAutoEnableMode():
https://github.com/openthread/openthread/blob/e958f8827ede37d543e962513d332d00dab6b1c3/include/openthread/srp_server.h#L244-L276
/**
 * Enables/disables the SRP server.
 *
 * On a Border Router, it is recommended to use `otSrpServerSetAutoEnableMode()` instead.
 *
 * @param[in]  aInstance  A pointer to an OpenThread instance.
 * @param[in]  aEnabled   A boolean to enable/disable the SRP server.
 *
 */
void otSrpServerSetEnabled(otInstance *aInstance, bool aEnabled);

/**
 * Enables/disables the auto-enable mode on SRP server.
 *
 * Requires `OPENTHREAD_CONFIG_BORDER_ROUTING_ENABLE` feature.
 *
 * When this mode is enabled, the Border Routing Manager controls if/when to enable or disable the SRP server.
 * SRP sever is auto-enabled if/when Border Routing is started and it is done with the initial prefix and route
 * configurations (when the OMR and on-link prefixes are determined, advertised in emitted Router Advertisement message
 * on infrastructure side and published in the Thread Network Data). The SRP server is auto-disabled if/when BR is
 * stopped (e.g., if the infrastructure network interface is brought down or if BR gets detached).
 *
 * This mode can be disabled by a `otSrpServerSetAutoEnableMode()` call with @p aEnabled set to `false` or if the SRP
 * server is explicitly enabled or disabled by a call to `otSrpServerSetEnabled()` function. Disabling auto-enable mode
 * using `otSrpServerSetAutoEnableMode(false)` will not change the current state of SRP sever (e.g., if it is enabled
 * it stays enabled).
 *
 * @param[in] aInstance   A pointer to an OpenThread instance.
 * @param[in] aEnabled    A boolean to enable/disable the auto-enable mode.
 *
 */
void otSrpServerSetAutoEnableMode(otInstance *aInstance, bool aEnabled);

Well thank you! I should have looked a little deeper into OpenThread's documentation... Do you know why this would be desirable? The SRP server might be used by devices attached to the Thread network to learn about each other, without requiring any off-mesh connectivity.

AutoEnableMode serves two purposes:

It enables the SRP server after the Border Router (BR) has completed the initial prefix and route configuration. This ensures that clients do not need to re-register due to receiving a new OMR prefix and address. It also ensures that services are advertised on the AIL (adjacent infrastructure link) only after the routes are properly set and there is end-to-end reachability between devices on the AIL and Thread mesh. Without this, we could face issues where a service is discoverable on the AIL, but we cannot communicate with it because the routes are not yet advertised and configured.

It disables the SRP server when the device is no longer acting as a BR. This is because we want clients that registered with this server to switch to another server/BR that is still acting as a BR and connected to the AIL. This ensures that the clients are still advertised on the AIL (and thus discoverable on the AIL).

Yes it makes sense! Thanks for the explanation.


docs: restructure migration guide

Summary
Create a "Migrate" section and move the "Migrating to Keptn" guide underneath it.  This allows us to add additional migration guides as needed.  Need to get this done so we are ready for the Context Migration docs and so Stacey has the new URL to reference from her 2023 summary newsletter.
Sorry, I thought this was refined in the 1/17 Community meeting.  Only after I did the work did I see that it is tagged for refinement.
Fixes https://github.com/keptn/lifecycle-toolkit/issues/2835
Checks

[x] My PR fulfills the Definition of Done of the corresponding issue and not more (or parts if the issue is separated
into multiple PRs)
[xI used descriptive commit messages to help reviewers understand my thought process
[x] I signed off all my commits according to the Developer Certificate of Origin (DCO)(
see Contribution Guide)
[x] My PR title is formatted according to the semantic PR conventions described in
the Contribution Guide
[x] My content follows the style guidelines of this project (YAMLLint, markdown-lint)
[n/a] I regenerated the auto-generated docs for Helm and the CRD documentation (if applicable)
[x] I have performed a self-review of my content including grammar and typo errors and also checked the rendered page
from the Netlify deploy preview
[ ] My changes result in all-green PR checks (first-time contributors need to ask a maintainer to approve their test runs)

Yup, this PR has run amok and it's too late to have this in place before Ondrej starts writing the migration to KeptnAppContext and all so i'm closing it and we can revisit this later if necessary


unallowed value docker

molecule examples from the book, and this repo do not work anymore. at least not on OSX+Docker Desktop
molecule converge stops with

CRITICAL Failed to pre-validate.
{'driver': [{'name': ['unallowed value docker']}]}

Thansk,

$ molecule --version
molecule 3.2.3 using python 3.9
ansible:2.10.5
delegated:3.2.3 from molecule
vagrant:0.6.1 from molecule_vagrant
$ docker --version
Docker version 20.10.2, build 2291f61

no it works. thanks @wells 👍


Withdrawing doesn't work unless you refresh the page

Upload / download a file
Click withdraw, or cancel then withdraw (if you're downloading)
Wallet pops up, asks you to sign transaction in metamask. No transaction in metamask
Refresh the page
Click withdraw again
Then the transaction is in metamask

Liam, George and I all witnessed this behaviour in our test this afternoon.

Haven't seen this issue recently.


Database: Clubs.Discrim every club has the same discriminator?

Database: Clubs.Discrim every club has the same discriminator?
Not totally sure if that's normal but i don't guess it is at all, when joining a club you need to specify you're applying for "Whatever Club#1" and everyone is the same "SomeOther#1" "Another#1"
Otherwise, applying with "Whatever Club" or "SomeOther" doesn't work, gives the error That club doesn't exist.
🤔

Because someone else can create Whatever Club, and it will be Whatever Club#2


Error with Get and Get(int id)

I'm getting an error using the following controller:
namespace WebApi.Controllers
{
    [ApiVersion("1.0")]
    [Route("v{version:apiVersion}/values")]
    public class ValuesController : ApiController
    {
          [HttpGet]
          public IEnumerable<string> Get()
          {
              return new string[] { "value controller 1", "value controller 2" };
          }

          [HttpGet]
          [Route("{id:int}")]
          public string Get(int id)
          {
              return "value " + id;
          }
          //removed methods
    }
}

and the following WebApiConfig.cs:
namespace DocFxWebApi
{
    public static class WebApiConfig
    {
        public static void Register(HttpConfiguration config)
        {
            // Web API configuration and services
            config.AddApiVersioning();

            var constraintResolver = new DefaultInlineConstraintResolver()
            {
                ConstraintMap = {
                    ["apiVersion"] = typeof( ApiVersionRouteConstraint )
                }
            };

            // Web API routes
            config.MapHttpAttributeRoutes(constraintResolver);
        }
    }
}

Going to v1.0/values/5 always return 404

Can you elaborate on the error. What's the message and against which routes?  Also, I notice that your route attributes are incongruent. For clarity, I would recommend:
namespace WebApi.Controllers
{
    [ApiVersion("1.0")]
    [RoutePrefix("v{version:apiVersion}/values")]
    public class ValuesController : ApiController
    {
          [HttpGet]
          [Route]
          public IEnumerable<string> Get()
          {
              return new string[] { "value controller 1", "value controller 2" };
          }

          [HttpGet]
          [Route("{id:int}")]
          public string Get(int id)
          {
              return "value " + id;
          }
          //removed methods
    }
}

That might even solve the issue.

Yes, sorry it was quite lacking in details.
If i add [Route] like in your code i get for the first Get:
<Error>
<Error>
UnsupportedApiVersion
<Message>
The requested resource with API version '1' does not support HTTP method 'GET'.
</Message>
<InnerError>
<Message>
No route providing a controller name with API version '1' was found to match HTTP method 'GET' and request URI 'http://localhost:1032/v1/values'.
</Message>
</InnerError>
</Error>
</Error>

and 404 for the second one
Removing the first Route like in my sample, i get the correct answer for the first get, and 404 for the second one

Did you also change the [Route] on the controller to [RoutePrefix]? I haven't been able to reproduce your scenario - yet. I've attached a working example based on your information (ApiVersionRepro.zip).
The route provides the following results:

Request URL
Status Code
Body

http://localhost/v1/values
200
["value controller 1", "value controller 2"]

http://localhost/v1/values/42
200
"value 42"

This would seem to be the expected results. Perhaps you can compare it against your configuration.

Thanks, i didn't notice RoutePrefix instead of Route , i didn't see that in the various docs.
It solved everything


error opening build log

Hello. I used an older ninja version 120715 on Windows this year.
After upgrade to 1.4.0 I get permanent error messages:
ninja: error: opening build log: No such file or directory
It is because this version does not create buildlog. If I create this directory manually, all works.
After experiments I found a solution: add "" to the directory name.
i.e.:
builddir = temp     - does not work
builddir = temp\    - works
Probably it is a Windows-related bug, because many ninja file examples in the internet contain no "" or "/" chars.

Yes, this is something else. I experience this under macOS as well (although inside own project).

@andrewrevvo Is this still reproducible with Ninja 1.10.0?

@andrewrevvo Is this still reproducible with Ninja 1.10.0?

Hello. It is strange that this issue is still open on github.
It was fixed many years ago. I just tested it again with 1.9.0 and with 1.10.0.
Both cases: "temp" or "temp" are worked correctly, and also with a sub-tree paths like "temp\aaa\bbb".
PS. I had no any issues with ninja all these years, thanks for a quality software. I use it every day in my work.