Segmentation fault in 0.3.28 pthreads variant when running the scikit-learn tests

Originally seen in scikit-learn https://github.com/scikit-learn/scikit-learn/issues/30151. This seems like a regression in 0.3.28 since it works fine with 0.3.27.
I managed to reproduce in C:
// test_openblas.c
#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <cblas.h>
#include <pthread.h>

typedef struct {
    double *A;
    double *B;
    double *C;
    int N;
} ThreadArgs;

void* matrix_multiplication(void *arg) {
    ThreadArgs* thread_args = (ThreadArgs*) arg;
    double alpha = 1.0, beta = 0.0;
    int N = thread_args->N;
    cblas_dgemm(CblasRowMajor, CblasNoTrans, CblasNoTrans,
                N, N, N, alpha, thread_args->A, N, thread_args->B, N, beta, thread_args->C, N);
}

int main() {
    double *A, *B, *C;
    int i, j, k;

    int N = 1000;

    A = (double*)malloc(N * N * sizeof(double));
    B = (double*)malloc(N * N * sizeof(double));
    C = (double*)malloc(N * N * sizeof(double));

    for (i = 0; i < N * N; i++) {
        A[i] = 1.;
        B[i] = 1.;
    }

    openblas_set_num_threads(10);

    pid_t pid = fork();

    if (pid == -1) {
        printf("error in fork");
        return 1;
    }
    else if (pid > 0) {
        // main process

        pthread_t t1, t2;
        ThreadArgs args;
        args.A = A;
        args.B = B;
        args.C = C;
        args.N = N;

        pthread_create(&t1, NULL, matrix_multiplication, &args);
        pthread_create(&t2, NULL, matrix_multiplication, &args);

        pthread_join(t1, NULL);
        pthread_join(t2, NULL);
    }

    // Free allocated memory
    free(A);
    free(B);
    free(C);

    printf("Matrix multiplication completed successfully.\n");

    return 0;
}

The build command for OpenBLAS I am using is a simplified version of what https://github.com/conda-forge/openblas-feedstock is doing:
make shared QUIET_MAKE=0 DYNAMIC_ARCH=1 BINARY=64 NO_LAPACK=0 TARGET=PRESCOTT NO_AFFINITY=1 NUM_THREADS=128

I am compiling my test program like this (I am using rpath to make sure that my build OpenBLAS shared library is picked rather than the system one):
gcc test_openblas.c -lopenblas_haswellp-r0.3.27.dev -I. -L. -Wl,-rpath=$PWD

I run my test program with:
./a.out

A git bisect seems to point at #4741.
Observations:

if you remove NUM_THREADS=128 from the OpenBLAS build command from you don't get the issue, so my wild guess is that this may be due to #4741 interpreting MAX_CPU_NUMBER as actually available cores whereas I am guessing that conda-forge use a high NUM_THREADS so that it can be performant on a wide variety of machines.
you need the combination of openblas_set_num_threads + fork + multi-threaded BLAS call nested inside threads. This may seem somewhat of an exotic combination but this does happen when running the scikit-learn tests: we restrict the number of OpenBLAS threads with threadpoolctl, we use process-based parallelism with joblib, and some of the scikit-learn code uses multi-threaded BLAS call nested within Python threads.
this only happens on some architectures: on Linux I was able to reproduce on Haskell and SkylakeX but not Sandybridge. A colleague fails to reproduce on macOS.
this only happens with the pthreads based OpenBLAS. OpenMP variant is not affected by this. In the Python world the pthreads variant is the default (both for conda-forge and numpy/scipy wheels).
not sure at all but https://github.com/scipy/scipy/issues/21479 may also be related

gdb info
(gdb) bt
#0  0x00007ffff6ca435e in dgemm_itcopy_HASWELL () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#1  0x00007ffff604176c in inner_thread () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#2  0x00007ffff618c2d2 in exec_threads () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#3  0x00007ffff618c4f9 in blas_thread_server () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#4  0x00007ffff5ca339d in ?? () from /usr/lib/libc.so.6
#5  0x00007ffff5d2849c in ?? () from /usr/lib/libc.so.6
(gdb) thread apply all bt

Thread 25 (Thread 0x7fff874006c0 (LWP 785557) "a.out"):
#0  0x00007ffff6ca435e in dgemm_itcopy_HASWELL () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#1  0x00007ffff604176c in inner_thread () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#2  0x00007ffff618c2d2 in exec_threads () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#3  0x00007ffff618c4f9 in blas_thread_server () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#4  0x00007ffff5ca339d in ?? () from /usr/lib/libc.so.6
#5  0x00007ffff5d2849c in ?? () from /usr/lib/libc.so.6

Thread 24 (Thread 0x7fff87e006c0 (LWP 785556) "a.out"):
#0  0x00007ffff6041d50 in inner_thread () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#1  0x00007ffff618c2d2 in exec_threads () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#2  0x00007ffff618c4f9 in blas_thread_server () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#3  0x00007ffff5ca339d in ?? () from /usr/lib/libc.so.6
#4  0x00007ffff5d2849c in ?? () from /usr/lib/libc.so.6

Thread 23 (Thread 0x7fff8d0006c0 (LWP 785555) "a.out"):
#0  0x00007ffff6041d50 in inner_thread () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#1  0x00007ffff618c2d2 in exec_threads () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#2  0x00007ffff618c4f9 in blas_thread_server () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#3  0x00007ffff5ca339d in ?? () from /usr/lib/libc.so.6
#4  0x00007ffff5d2849c in ?? () from /usr/lib/libc.so.6

Thread 22 (Thread 0x7fff91c006c0 (LWP 785554) "a.out"):
#0  0x00007ffff6c9d5bf in dgemm_kernel_HASWELL () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#1  0x0000000000000000 in ?? ()

Thread 21 (Thread 0x7fff926006c0 (LWP 785553) "a.out"):
#0  0x00007ffff6ca4345 in dgemm_itcopy_HASWELL () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#1  0x00007ffff604176c in inner_thread () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#2  0x00007ffff618c2d2 in exec_threads () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#3  0x00007ffff618c4f9 in blas_thread_server () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#4  0x00007ffff5ca339d in ?? () from /usr/lib/libc.so.6
#5  0x00007ffff5d2849c in ?? () from /usr/lib/libc.so.6

Thread 20 (Thread 0x7fff930006c0 (LWP 785552) "a.out"):
#0  0x00007ffff6ca435e in dgemm_itcopy_HASWELL () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#1  0x00007ffff604176c in inner_thread () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#2  0x00007ffff618c2d2 in exec_threads () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#3  0x00007ffff618c4f9 in blas_thread_server () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#4  0x00007ffff5ca339d in ?? () from /usr/lib/libc.so.6
#5  0x00007ffff5d2849c in ?? () from /usr/lib/libc.so.6

Thread 19 (Thread 0x7fff93a006c0 (LWP 785551) "a.out"):
#0  0x00007ffff6c9f757 in dgemm_kernel_HASWELL () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#1  0x0000000000000000 in ?? ()

Thread 18 (Thread 0x7fff944006c0 (LWP 785550) "a.out"):
#0  0x00007ffff6041d53 in inner_thread () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#1  0x00007ffff618c2d2 in exec_threads () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#2  0x00007ffff618c4f9 in blas_thread_server () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#3  0x00007ffff5ca339d in ?? () from /usr/lib/libc.so.6
#4  0x00007ffff5d2849c in ?? () from /usr/lib/libc.so.6

Thread 17 (Thread 0x7fff94e006c0 (LWP 785549) "a.out"):
#0  0x00007ffff6041d51 in inner_thread () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#1  0x00007ffff618c2d2 in exec_threads () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#2  0x00007ffff618c4f9 in blas_thread_server () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#3  0x00007ffff5ca339d in ?? () from /usr/lib/libc.so.6
#4  0x00007ffff5d2849c in ?? () from /usr/lib/libc.so.6

Thread 16 (Thread 0x7fff910006c0 (LWP 785548) "a.out"):
#0  0x00007ffff6ca434a in dgemm_itcopy_HASWELL () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#1  0x00007ffff604176c in inner_thread () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#2  0x00007ffff618c2d2 in exec_threads () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#3  0x00007ffff618c4f9 in blas_thread_server () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
--Type <RET> for more, q to quit, c to continue without paging--c
#4  0x00007ffff5ca339d in ?? () from /usr/lib/libc.so.6
#5  0x00007ffff5d2849c in ?? () from /usr/lib/libc.so.6

Thread 15 (Thread 0x7fff906006c0 (LWP 785547) "a.out"):
#0  0x00007ffff6c9d7e6 in dgemm_kernel_HASWELL () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#1  0x0000000000000000 in ?? ()

Thread 14 (Thread 0x7fff8fc006c0 (LWP 785546) "a.out"):
#0  0x00007ffff618c992 in exec_blas_async () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#1  0x00007ffff618cc10 in exec_blas () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#2  0x00007ffff604127a in gemm_driver.isra () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#3  0x00007ffff6042183 in dgemm_thread_nn () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#4  0x00007ffff5f445e3 in cblas_dgemm () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#5  0x000055555555525b in matrix_multiplication ()
#6  0x00007ffff5ca339d in ?? () from /usr/lib/libc.so.6
#7  0x00007ffff5d2849c in ?? () from /usr/lib/libc.so.6

Thread 13 (Thread 0x7fff8f2006c0 (LWP 785545) "a.out"):
#0  0x00007ffff6c9d0ce in dgemm_kernel_HASWELL () from /home/lesteve/dev/OpenBLAS/libopenblas.so.0
#1  0x0000000000000000 in ?? ()

Thread 1 (Thread 0x7ffff7e6a740 (LWP 785530) "a.out"):
#0  0x00007ffff5c9fa19 in ?? () from /usr/lib/libc.so.6
#1  0x00007ffff5ca51a3 in ?? () from /usr/lib/libc.so.6
#2  0x00005555555553c9 in main ()

Not immediately reproducible with current develop, may have been fixed by #4879

Possibly the fork makes it just more likely to run into uninitialized memory ? I had some nagging doubt at the time, that I may be papering over some deeper issue with all the recent thread setup changes, but I did not find any.


Error in text in Russian tutorial

Hello we received the issue below on try .net repo. Could you please have a look ?
https://github.com/dotnet/try/issues/140

https://docs.microsoft.com/ru-ru/dotnet/csharp/quick-starts/list-collection?tutorial-step=3
Text:

Если он имеет значение 1, элемент не найден

Correct version:
Если он имеет значение -1, элемент не найден

@SonjaSaltzman Can you take a look at this issue?

@DavidPrio Could you please help to move this issue to Russian loc repo and follow up?
Russian translation is missing minus sign "-" before 1 as mentioned in first comment.

Hi @JasonCard
I can't find the ru-ru repo to move this issue in the zenhub list (confirm I'd be https://github.com/dotnet/docs.ru-ru) but we can work it from here, would that be ok?
Kind regards
Microsoft DOCS International Team
@DavidPrio

Hi @CeciAc
Yes, please help to manually move this issue to https://github.com/dotnet/docs.ru-ru.
I also tried ZenHub, but maybe it cannot handle moving issues between repos in different orgs.

Hi @JasonCard @CeciAc can you try with https://help.github.com/articles/transferring-an-issue-to-another-repository/
This is new feature coming directly from GitHub, just shared by @Sara-Nicolini this week. Thanks!

Hi @JasonCard
I created a new issue manually here: https://github.com/dotnet/docs.ru-ru/issues/150
@DavidPrio I tried but couldn't use that feature either, I even temporally deactivate ZenHub but the mentioned option did not appear. Let me know if we may be missing anything


Add inverse variant in Avatar element

Fixes #1869
@interactivellama, this is ready for review. Kindly review it and comment if any other changes are needed.
Add inverse variant and corresponding example and test which looks like the following.

CONTRIBUTOR checklist (do not remove)
Please complete for every pull request

[x] First-time contributors should sign the Contributor License Agreement.  If you haven't before, wait a few minutes and a bot will comment on this pull request with instructions.
[x] npm run lint:fix has been run and linting passes.
[x] Mocha, Jest (Storyshots), and components/component-docs.json CI tests pass (npm test).
[x] Tests have been added for new props to prevent regressions in the future. See readme.
[X] Review the appropriate Storybook stories. Open http://localhost:9001/.
[ ] The Accessibility panel of each Storybook story has 0 violations (aXe). Open http://localhost:9001/.
[ ] Review tests are passing in the browser. Open http://localhost:8001/.
[ ] Review markup conforms to SLDS by looking at DOM snapshot strings.

REVIEWER checklist (do not remove)

[ ] TravisCI tests pass. This includes linting, Mocha, Jest, Storyshots, and components/component-docs.json tests.
[ ] Tests have been added for new props to prevent regressions in the future. See readme.
[ ] Review the appropriate Storybook stories. Open http://localhost:9001/.
[ ] The Accessibility panel of each Storybook story has 0 violations (aXe). Open http://localhost:9001/.
[ ] Review tests are passing in the browser. Open http://localhost:8001/.
[ ] Review markup conforms to SLDS by looking at DOM snapshot strings.

Required only if there are markup / UX changes

[ ] Add year-first date and commit SHA to last-slds-markup-review in package.json and push.
[ ] Request a review of the deployed Heroku app by the Salesforce UX Accessibility Team.
[ ] Add year-first review date, and commit SHA, last-accessibility-review, to package.json and push.
[ ] While the contributor's branch is checked out, run npm run local-update within locally cloned site repo to confirm the site will function correctly at the next release.

@pradumangoyal Looks good. Thanks!


Possibility to VIEW multiple VCFs/GEMINI DBs at once

Instead of having to load them one by one?

You could do it with a bash for loop like
for GEMINI_DB in repo/*.db
do
puzzle load $GEMINI_DB
done

Ok VCF's can be loaded at once as well, just that if they're compressed then you need to specify a different pattern with -p *.vcf.gz.

Could we close this one @guillermo-carrasco

but one still can't load several GEMINI dbs at once, right?

No and it does not really make sense to do so I guess...
The idea with gemini is to collect samples in one database?

Why not do a bash-loop like proposed if you really want to do it?

yeah ok, or we can merge the DBs outside puzzle.
Thanks!

:tanabata_tree:


Word in NBest is not the same as DisplayText

Hello,
I'm using speechconfig.requestWordLevelTimestamps() to retrieve word-level timestamps in reco result. It is noticed that for numbers like 1025, 16.5%, 2020, 0.05, etc., the display text is different from that of Word in NBest.
E.g. utterance "1025" in English will be recognized as "1025" in display text, while the NBest Words list are "one", "thousand", "and", "twenty", five".

Is there any way to change lexicon format in NBest Words list to Display Text format ? The purpose is to enrich post-recognition user experience.
Thank you.

Hi, the NBest Words list contains tokens based on lexical analysis of input (cf. the Lexical result attribute which contains the whole phrase) and the format is not configurable. Timestamps are assigned to these lexical tokens. If you are trying to provide visualization of recognized numerical digits with timestamps, you could try mapping between them and lexical tokens in code.


Include RMOTR in partnered schools

As discussed with Conrad, I include the RMOTR's card into the partnered schools section.

I’m going to shut down this PR. Instead of hard-coding the school into the page, I’d like to make a fix that’s needed to happen for awhile: Update API to use image links on master branch instead of code-schools-cooper-kyle
Once that's done, any future school additions will simply require a back-end YAML file update.

Quick question. What about the school logo image? Cause as far as I saw, the backend YAML file has URLs pointing to images in this repo, and the closing the PR the RMOTR image won't be available.
Example:
https://raw.githubusercontent.com/OperationCode/operationcode_frontend/code-schools-cooper-kyle/src/images/codeSchoolLogos/rmotr.jpg

@martinzugnoni I added the logo really shortly after closing your PR :)
I didn't give you a time-table in the comment, but I did it immediately afterward. Working to get the PR merged right now.

@martinzugnoni added the logo :)


Can't install imap

Info:

Docker version ($ docker --version):  Docker version 18.03.1-ce, build 9ee9f40
Laradock commit ($ git rev-parse HEAD): 0c41fceae1b23089f5360dc4f8aeb85719c7bf5a
System info (Mac, PC, Linux): MAC
System info disto/version: 10.13

Issue:
Can't instapp php-fpm imap

Expected behavior:
Imap install

Reproduce:

Relevant Code:
// place a code sample here

Problem solved by using VPN server.


How to include the copy code script into my md file

Hi there
Sorry for the silly question but how do you include the code into my md file?
I have tried
<html>
<head>

<script src="https://github.com/Mottie/GitHub-userscripts/blob/master/github-copy-code-snippet.user.js"></script>

</head>

regards
Balint

@Mottie
Thank you and happy 2019


Thursday

✅ All CI checks passed
✅ All tests passed
✅ All typeclasses saturated
✅ Enterprise build is ready to go


Questions on dataset answers.

Hi there, I am a little confused about the answers in the dataset.
In dataset nq, I can see a sample whose answers contain same elements like Carey Mulligan and Matthias Schoenaerts. Actually, other samples also have duplicated answers. It is supposed to be like that?
Thanks for your reply!

Thanks for pointing it out! We have noticed that there are some irregular patterns in the current nq dataset (luckily they do not affect the evaluation results), we will update this dataset soon!

Thanks!

Hi! We have updated the datasets in LongBench.


Compress error

When I enable django-compressor, I get this error:
FilterError at /2016/reviews/section/talks/
NameError: #grid > .core > .span is undefined in
/home/dpoirier/clients/pycon/pycon/site_media/static/less/bootstrap/navbar.less on line 195, column 3:
194 .navbar-fixed-bottom .container {
195   #grid > .core > .span(@gridColumns);
196 }
Looking at line 195 of pycon/static/less/bootstrap/navbar.less, I see this, which looks wrong to me:
.navbar-fixed-bottom .container {
    #grid > .core > .span(@gridColumns);
 }

Like selectors and styles got mixed together somehow?
What's weird is that according to Git, those lines haven't changed anytime recently. I do have the latest version of Less, 2.5.1, installed locally, so maybe that syntax used to be valid but isn't anymore?

This works fine with lessc 1.4.2, which is the Ubuntu 14.04 provided version of less, so probably won't be a problem on our servers for now.

Closing; not an issue with the system lessc, was just my dev env where I had a much newer version.


It always deploys to a new netlify

Describe the bug
It always deploys to a new netlify
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-brvlcz
Steps to reproduce
Just deploy
Expected behavior
It should just deploy to domain I have connected to already
Screen Recording / Screenshot
No response
Platform
Browser name = Chrome
Full version = <IP_ADDRESS>
Major version = 131
navigator.appName = Netscape
navigator.userAgent = Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36
performance.memory = {
  "totalJSHeapSize": 82286979,
  "usedJSHeapSize": 75999043,
  "jsHeapSizeLimit":<PHONE_NUMBER>
}
Username = shevilpanchal
Chat ID = 69353c774a5e

Additional context
No response

Appreciate the feedback! Currently, the project should continue to reflect changes to the temporary domain when you re-deploy. We are aware of the request to have continued updates to the site associated with the custom domain and are tracking the issue here: #49. In the meantime, you can download the project files by clicking the download button at the top-right of the page and manage deployment manually. Hope this helps!


Installation of the app from source does not automatically install dependencies.

I though that installing like so:
install.packages('master.tar.gz', type = 'source')

would try to install the dependencies from CRAN, but is doesn't. It fails. Need to figure out a way to install the package in R wihtout having it on CRAN and without requiring devtools.

couldn't you just install dependencies first, then install the source
package?
On Thu, Aug 20, 2015 at 4:30 PM, Jason K. Moore<EMAIL_ADDRESS>wrote:

I though that installing like so:
install.packages('master.tar.gz', type = 'source')
would try to install the dependencies from CRAN, but is doesn't. It fails.
Need to figure out a way to install the package in R wihtout having it on
CRAN and without requiring devtools.
—
Reply to this email directly or view it on GitHub
https://github.com/ucd-ipo/aip-analysis/issues/76.

Also, take a look at drat. https://github.com/eddelbuettel/drat
I don't know a whole lot about it, but we might be able to leverage that to our advantage here.

Yes, the dependencies can be installed first but what's the point of  DESCRIPTION file with all the dependencies if you have to manually install anyways. I'm new to R packaging, don't really know how it all works.
I don't understand what drat is.

One can use drat to create a local repo or one on github. We could create a
github repo that has the exact package versions the user should use with
the app. Instead of installing packages from CRAN, the user would install
the packages from the drat repo. That way we can ensure that a user
installing new packages won't install a version that is incompatible with
the app. We could include binaries for shinyace and the other packages as
well. It won't depend on devtools--we would be essentially be creating a
mini-CRAN, if you will, that the user installs from using install.packages.
Yes, the dependencies can be installed first but what's the point of
DESCRIPTION file with all the dependencies if you have to manually install
anyways. I'm new to R packaging, don't really know how it all works.
I don't understand what drat is.
—
Reply to this email directly or view it on GitHub
https://github.com/ucd-ipo/aip-analysis/issues/76#issuecomment-133193632.

I see. Cool. Typically for web app development you want to pin to exact versions of dependencies. By default R doesn't make that easy because install.packages() can only install the latest version of a package. drat sounds like a nice solution to that. I'll look into it.

This is an interesting solution for packaging R apps: https://github.com/rstudio/packrat
I've also been trying out conda since it recently started supporting R packages.

Packrat could work, but I think it would have to be installed on the flash
drive. I'm still leaning towards drat, since we could have source packages
and binaries and host the app there as well so people can install via
install.packages and they don't have to go through any extra work. I've
played around with packrat a bit about a year ago, and unless a lot has
changed (which it probably has), it isn't super user friendly. Doesn't
conda require a python instillation?
On Tue, Aug 25, 2015 at 7:40 PM, Jason K. Moore<EMAIL_ADDRESS>wrote:

This is an interesting solution for packaging R apps:
https://github.com/rstudio/packrat
I've also been trying out conda since it recently started supporting R
packages.
—
Reply to this email directly or view it on GitHub
https://github.com/ucd-ipo/aip-analysis/issues/76#issuecomment-134779408
.

Conda requires a Python install. But we're drifting here.
None of this really matters for the flash drive. You can install whatever versions of the packages you want on the flash drive. The installation issues are really just related to whether we want people to be able to install R and the package with everything working on on their on machine. It'd be nice to pin dependencies to specific number in that case.

new devtools::install_github() version installs dependencies


Fixed Typo

fail -> fall
In Site Recovery Service section

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

Hi @Nikhilpy77 - Failover and failback appear to be the correct terms for disaster recovery. See About failover and failback
in Azure Site Recovery and Define failover and failback. Thanks for contributing to the documentation. We'll close this PR.


Is the idea of a collection's modelId functionality valid?

We are running quite old version of the Backbone in a production and it has served well, thank you guys. Now some other library requires a newer version of the Backbone and we have a small issue with the new version. We can fix the issue quite easily, but as I searched a reason for it, I begun to think that is this modelId functionality of the Collection valid idea at all? (A related commit can be found here)
So the issue is that one cannot use the get function of Backbone.Collection if the model property of the collection is not set and models inside the collection have a idAttribute overridden.
E.g. following code logs undefined, although I would expect it to log the model object (code "works" in the version 1.1.2)
var MyModel = Backbone.Model.extend({
	idAttribute: 'myid'
});
var m = new MyModel({myid:'xxx'});
var c = new Backbone.Collection([m]);
var byId = c.get(m.id);
console.log(byId); // byId is undefined

If I define the model property for a collection, it works as I expect, but I think that a collection should not need to know about models it contains.
I think that modelId functionality breaks quite a fundamental behavior (get by id) to solve an edge case issue. And if this functionality is valid, I think that a API documentation should mention that the model of the Collection must be defined.
What do you think?
And thank you for your work, you have made the world better.

modelId broke backwards compatablity, see this:  https://github.com/jashkenas/backbone/pull/3966

#3966 fixed this and was even merged, but seems to have been reverted on present master. Will need to investigate what has happened.

Good news! I don't remember why I thought that #3966 was reverted when I wrote that, but if it was true, it is not true anymore. All the changes are in present master and I can confirm from a quick test in the console on https://backbonejs.org that everything works the way it should.


Minor perf items

Don't wish to merge this until I figure out how to integrate a proper JMH performance test with this gradle / kotlin-multiplatform build to prove them.

All open for discussion. There are many more changes on my local to follow these so stick to what is here rather than what isn't!
Q: I don't know how to default the centreX to centre.coordinateX because it is a property not a method). Can we just delete them (did they ever get published in this form?)

Have added some performance numbers; Below are basically before and after this PR.
I have not changed any method names or nullable types yet.
Test, Before, After
HexagonalGridBuilderPerfTest.buildTime-RECTANGULAR, 4198, 5012
HexagonalGridBuilderPerfTest.buildTime-HEXAGONAL, 3266, 4108
HexagonalGridBuilderPerfTest.buildTime-TRIANGULAR, 4183, 4463
HexagonalGridBuilderPerfTest.buildTime-TRAPEZOID, 4551, 5090

HexagonalGridCalculatorPerfTest.calculateMovementRangeFrom, 148438, 172944
HexagonalGridCalculatorPerfTest.drawLine, 232930, 254664
HexagonalGridPerfTest.getNeighborsOf, 454140, 545758
HexagonalGridPerfTest.getByPixelCoordinate100pct, 1593, 3157
HexagonalGridPerfTest.getByPixelCoordinate50pct, 2561, 4910
HexagonalGridPerfTest.iterateHexagons, 300, 314
HexagonalGridPerfTest.getHexagonsByCubeRange, 154, 340
HexagonalGridPerfTest.getHexagonsByOffsetRange, 127, 333

perf-results.zip


Rbac id filters as a scope pt1

Currently we fetch Vm ids and then run a query on those ids.
We then ship the ids to and from the server, and for a larger installation this gets quite big - in one installation the sql is 151k.
@matthewd this started getting too big. Let me know if you want smaller, or if you want bigger and I can put the rest of the work to make ids into a scope and not shipping them around

This looks good :+1:

thanks for the ping. rebased


MS0301: implement simple tools

The implementation for these tools is the same: we call
 opensim-cmd run-tool  file.xml

For now, we need only two functions: load xml file and run. When running the tool, the result file is loaded and the model should be animated in the visualizer. I will provide a complete example when we start working on this task.
This applied to all the tools menu except:

plot:
scale
convert
plugins

In a later milestone, we will visit this menu again and implement the tools in a similar way to the java interface.

This task has less priority and is ignored for now!


error_on_unformatted is ignored when set from rustfmt.toml

$ rustfmt --version
rustfmt 0.4.1-nightly (374dba83 2018-03-22)

With the following source:
main.rs:
// Lorem ipsum dolor sit amet, consectetur adipiscing elit. Duis eget ex commodo, posuere sapien non, interdum leo.
fn main() {
    println!("Hello, world!");
}

rustfmt.toml:
error_on_unformatted = true

Notice that rustfmt main.rs (erroneously) succeeds, but rustfmt main.rs --error-on-unformatted fails (as expected).

https://github.com/rust-lang-nursery/rustfmt/blob/3de184fb0e6f7161fef2f1d9288e06f57abadfc0/src/bin/main.rs#L115-L127
CliOptions::apply_to erroneously overrides the config option values even when the corresponding command line arguments are not passed. We should use Option<bool> instead of bool for every CliOptions field, and only change config when we need to, like we already do for write_mode and color.

I would like to work on this one if nobody has already started.

I see that from_matches has some logic to define unstable_features, so it makes sense to use at as a bool in apply_to, right?
https://github.com/rust-lang-nursery/rustfmt/blob/3de184fb0e6f7161fef2f1d9288e06f57abadfc0/src/bin/main.rs#L74-L84

Closed via #2600.


fix replay chat

About the PR

currently, the replay client does not display chat from server-recorded replays properly
this PR fixes the issue as observed in testing
i have not observed any issues in testing after making this change
NOTE: i do not know the full extent of what i am doing in this PR, needs to be reviewed by someone who is highly familiar with the relevant code
Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl:

fix: Replays now display chat properly.

NOTE: i do not know the full extent of what i am doing in this PR.

This PR will break things, and I'll open a separate PR to fix this bug, but I also want to try and explain why/what this bit of code is doing.
The logic around skipEffects is meant to prevent a spam of audio & visual effects when jumping forward or backward* in time. E.g., if there is a laser gun fight and you jump forward 10 seconds, it stops it from suddenly playing all the sounds & laser visuals simultaneously. Removing it entirely will reintroduce those issues.
This is a video of how its meant to work:
https://github.com/space-wizards/space-station-14/assets/60421075/e632a9ff-05d9-4f11-a6ac-8191a45c85e9
And this is how it would look if skipEffects was always false:
https://github.com/space-wizards/space-station-14/assets/60421075/cd13e388-a1cc-47e5-b46b-d32bb79480f2
Chat messages are sort of inbetween - we want them to always get logged to the chat console, but we might want to skip the pop-up, hence there is the speechBubble argument.
The bug here was caused by the fact that I thought that the ChatMessage event inherited from EntityEventArgs, but it is actually a completely separate message. So the comment about "the event get raised as a normal networked event" doesn't apply as its not a EntityEventArgs.
* Going backward 1 tick really involves going backward many ticks, and then forward N-1 ticks.

Note that there is also a separate unrelated chat bug mentioned in #17170, which is that the messages don't appear in the chat UI when jumping too far forwards in time.
Fixing that requires completely reworking how replays handle chat messages and probably writing a custom chat ui controller. Or just extending the default chat controller.


It won't work...  :(

I was REALLY excited when I found your project. Finally a cross-platform react-native audio player! I followed the directions exactly, fired it up, click on a button, and no sound...
Ok, so obviously I did something wrong. Let's run the example app. Except, it won't compile...
https://cloudup.com/czuZa2Xv_Ho
Sadness fell upon me. I need this to work! Any ideas?

@dovy to fix that, follow these steps:  http://stackoverflow.com/questions/39560610/react-native-run-ios-build-fail-after-xcode-update-to-8-0
or upgrade the react-native version.

@dovy did the suggestion above fix the issue?

Closing due to inactivity


Debug flag should be configurable in .yaml files.

This one-line modification to YAMLLoader adds support for specifying debug flag in bundle definitions.

Also, apparently I've botched my commit message ;)


feature: distributed transaction support  ComQuery request

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

/assign

update sql request has processed, and there is delete request、insert request、global lock executor need to be processed.
https://github.com/CECTC/dbpack/blob/04fb5072f94625bb2369801f36469f6ee0746379/pkg/filter/dt/exec/query_delete.go#L47
https://github.com/CECTC/dbpack/blob/04fb5072f94625bb2369801f36469f6ee0746379/pkg/filter/dt/exec/query_insert.go#L55
https://github.com/CECTC/dbpack/blob/04fb5072f94625bb2369801f36469f6ee0746379/pkg/filter/dt/exec/query_global_lock.go#L103


MongoDB and Mongoose - Challenge nº 9 (findByIdAndUpdate) - works without updating the model.

This is the challenge: https://www.freecodecamp.org/learn/apis-and-microservices/mongodb-and-mongoose/perform-new-updates-on-a-document-using-model.findoneandupdate.
The challenge is to update a "Person" model using "findByIdAndUpdate", but I notice that both of the following codes work:
var findAndUpdate = function(personName, done) {
  var ageToSet = 20;

  var person = Person.findOne({name: personName}, (err, data) => {
    if (err) return console.log(err);

    data.age = ageToSet;

    Person.findByIdAndUpdate(data._id, data, { new: true }) // THIS LINE

    done(null, data);
  })
};

var findAndUpdate = function(personName, done) {
  var ageToSet = 20;

  var person = Person.findOne({name: personName}, (err, data) => {
    if (err) return console.log(err);

    data.age = ageToSet;

    // WIHTOUT FINDBYIDANDUPDATE

    done(null, data);
  })
};

Thanks for pointing this out @LeoDalcegio. To me, this seems quite minimal and probably not worth the effort it would take to fix it. It says to use findOneAndUpdate so users should try to use that. If they can achieve the correct result in another way in this case, I don't really have a problem with it.
The challenge also mentions to find the person by name and not id. Your examples use Id. It's not perfect, but like I said, I don't think it's worth the effort in this case. Most users are going to use findOneAndUpdate because it says to right there.
But, if someone wants to dive in and try to fix it - I wouldn't object.

Yeah you are right, it is just a small thing that I notice, but it is not something that would stop people from learning.
-------- Mensagem original --------
De : Tom<EMAIL_ADDRESS>Data: 30/04/2020 12:32 (GMT-03:00)
Para: freeCodeCamp/freeCodeCamp<EMAIL_ADDRESS>Cc: Leonardo Luís Dalcegio<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Assunto: Re: [freeCodeCamp/freeCodeCamp] MongoDB and Mongoose - Challenge nº 9 (findByIdAndUpdate) - works without updating the model. (#38677)
Thanks for pointing this out @LeoDalcegiohttps://github.com/LeoDalcegio. To me, this seems quite minimal and probably not worth the effort it would take to fix it. It says to use findOneAndUpdate so users should try to use that. If they can achieve the correct result in another way in this case, I don't really have a problem with it.
The challenge also mentions to find the person by name and not id. Your examples use Id. It's not perfect, but like I said, I don't think it's worth the effort in this case. Most users are going to use findOneAndUpdate because it says to right there.
But, if someone wants to dive in and try to fix it - I wouldn't object.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/freeCodeCamp/freeCodeCamp/issues/38677#issuecomment-621928768, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ALS34IKGZ3FV6ZCH5AZ5QODRPGKX7ANCNFSM4MTUCMHA.


Ubuntu 11.10 Gnome login fail with rbenv

Hi,
I have rbenv installed system-wide in /usr/local/rbenv, and with a /etc/profile.d/rbenv.sh patching the PATH and running the whole eval rbenv init thing.
Login to Unity fails and boots me right back to login in screen. Login to text base shell works perfectly fine.
I tracked down the problem to line 74 in rbenv-init:
73: cat <<EOS
74: function rbenv() {
75:   ...

Removing the 'function' solves my problem:
73: cat <<EOS
74: rbenv() {
75:   ...

Now both login works. Havn't seen any side effects.
Cheers!

See https://github.com/sstephenson/rbenv/issues/122


Removed references to std::make_unique

This is an attempt to expand compatibility with the consumers of the Arduino library, one of whom reported that std::make_unique was causing them to be unable to compile the library.
https://github.com/Open-Agriculture/AgIsoStack-Arduino/issues/7
Interested to hear thoughts on this... I know we are targeting at least C++14 (which is when make_unique was introduced) but this seems like a fairly minor change to bring the Arduino lib back down to C++11

While it hurts me to downgrade the supported C++ version to 11, I feel like it's the only reasonable option here. The arduino-ide is widely used among hobbyists and I don't want them to have to follow some cumbersome procedure to be able to compile the stack. I had a shot at it myself, and managed to succeed in changing the version, but only after following a dozen links on google 😒.
Though I'm happy it seems like a pretty quick fix to go back to C++11 with the stack. Can you update the CMakeLists.txt as well, so Github Actions can notify us if we happen to make changes not compliant with C++11?

Can you update the CMakeLists.txt as well

Will do

I know I said will do, but then you did it in #382 😆
Sonar seems mad about C++ 17 things like std::scoped_lock and template type deduction so I'll see if I can modify those rules...


Fix Issue 19403 - Make std.string.stripLeft on char array nogc nothrow

Gah, what is Circle doing, or rather not doing?


Can i use this software on my secondary display?

Thank you in advance for your answer.
Congrats on the software, looks very nice, didn't attach it into the leds yet, need to know if it's possible to use on my secondary screen instead of laptop screen.

at the moment it only works with the primary screen.
what would be your use case? you can always make your "big screen" to the primary in windows.

True,
I will do that, thank you for the reply.

Any news on this? I would like to use Adrilight on all of my 3 monitors, so that addition would be great!

this could be technically possible, no problem, but there are way too many degrees of freedom involved:

How many monitors?
How are they arranged?
Where are the actual LEDs placed?
How are the LEDs technically connected?
...

this is too much to develop and also to test for me for such an edge case. I have 3 monitors myself, but I don't think there are many others out there having multiple monitors and want ambilight on all of them.


[feature/dashboard-oop] Add initial workload targets and manifest entry for dashboard

Adding targets and manifest entry for the Aspire Dashboard
Microsoft Reviewers: Open in CodeFlow

Publishing as a tool by default breaks the workload build

Just curious - how did it break? It would be useful to include that in the commit message (in future of course).

Publishing as a tool by default breaks the workload build

Just curious - how did it break? It would be useful to include that in the commit message (in future of course).

Build error producing the workload due to trying to package the project twice: The file 'D:\a\_work\1\s\artifacts\obj\workloads\pkg\Aspire.Dashboard.8.0.0-ci\Aspire_icon_256.png' already exists.

Build error producing the workload due to trying to package the project twice: The file 'D:\a_work\1\s\artifacts\obj\workloads\pkg\Aspire.Dashboard.8.0.0-ci\Aspire_icon_256.png' already exists.

Binlog for that build should help with debugging the issue but that is not being generated for the build workloads step currently.

Build error producing the workload due to trying to package the project twice: The file 'D:\a_work\1\s\artifacts\obj\workloads\pkg\Aspire.Dashboard.8.0.0-ci\Aspire_icon_256.png' already exists.

Binlog for that build should help with debugging the issue but that is not being generated for the build workloads step currently.

We should produce nugets with different package IDs for the workload package and any dotnet tool package, so I don't think it makes sense to set PackAsTool by default anyway. A custom build task will be needed to run a second pack build on Aspire.Dashboard with PackAsTool set to true and an updated PackageId.


Feature: Add a notes section for each dApp. Only editable by the owner.

Only editable by the owner.


Eliminate PyJITFreeFunction

Should simply return a subclass of PyJittedCode that does its own cleanup properly.

Solving #166 might mitigate the need to have this be a separate issue, so it is probably more prudent to try and solve #166 first and then only worry about this issue if #166 fails.


feat: add v3 to bol template

able to help update the readme table too? create 2 new columns, v2, v3, then indicate which template has which support


New icon: Cloudfanatic

[X] I have carefully read and followed CONTRIBUTING document.

URL of the site: cloudfanatic.net

Proof of the Aegis-compatible 2FA:

Logo / icon info: cloudfanatic-logo-blue-black.png

Background color info: Background color is from the logo (#0000ff)

Logo / icon info: cloudfanatic-logo-blue-black.png

I assume you re-created the logo from stratch, but this should be informed if you link the raster image.

I apologize, I made the icon from the favicon while I was at work and didn't remember where I got it when I got home. I've edited the post to the original vector.


Update Rust Edition to 2021

Update to latest Rust edition. The next edition is 2024.
There are no breaking changes, and it adds more consistency to the overall language. In order to miminize the risk of introducing "outdated" idioms to the implementation, updated the edition to 2021.

What's the implication of bumping up the edition wrt compat?

There are issues with the crate being unable to build on earlier compiler versions. However, this was a significant concern when it first came out in 2021. We are almost three years from the release, so this should not be a concern.
https://doc.rust-lang.org/edition-guide/editions/index.html discusses how crates of different editions have "seamless interoperability."


Solana Connectors is showing up below the page window in mobile

I couldn't reproduce that issue, even using Telegram.
Waiting on more feedback to go further.

Tagging as blocked for now.

Are you able to connect your wallet now ?

@bradstevens1 yes, no problem on my end, did you get any issues?

No just asking if you did cause i got something like this and got to contact the support team and they’re are able to solved it

@bradstevens1 oh I see, what issue did you report? is in solana connector?

Yes but since you’ve been able to connect your wallet think you don’t have any problem with it anymore?


Enzyme testing instruction out of date

Describe the bug
Writing test with enzyme as stated here will not work http://react-dnd.github.io/react-dnd/docs/testing#testing-with-enzyme
Reproduction
Write a test using enzyme,
Try to retrieve the manager
var backend = root
  .instance()
  .getManager()
  .getBackend()

It will fail because instance() will return null with React 16 (Stateless function component don't have an instance)
Expected behavior
The doc should provide the recommanded working way to achieve this.
Workaround
I figured something like this, but does feel quite convoluted:
    const mountComponent = (Component: any, testProps: TestProps) => {
        const result: any = { wrapper: undefined, manager: undefined };

        result.manager = new Promise<DragDropManager>(resolve => {

            result.wrapper = mount(
                <DndProvider backend={TestBackend}>
                    <DndContext.Consumer>
                        {dnd => {
                            resolve(dnd.dragDropManager);
                        }}
                    </DndContext.Consumer>
                    <Component
                        {...testProps}
                    />
                </DndProvider>,
            );
        });
        return (result: {|
            wrapper: ReactWrapper<DndProvider>,
            manager: Promise<DragDropManager>,
        |});
    };

usage
        let manager: DragDropManager;
        let wrapper;
        let monitor;
        let backend;

        beforeEach(async () => {
            const result = render(MyComponent, {
               /*my props*/
            });
            manager = await result.manager;
            wrapper = result.wrapper;
            monitor = manager.getMonitor();
            backend = (manager.getBackend(): TestBackend);
        });

OS: MacOS
Browser: Chrome
Version : 75
React: 16.8.6
React-dnd : 9.3.9

How can I get getHandlerId method for my component?

.getHandlerId() is an instance method for getting the sourceId for DropTarget, which is part of the Legacy Decorator API. It may be that this part hasn't been updated

@darthtrevino Do you have any guidance here?

Hit this myself today. The library can internally just use forwardRef / useImperativeHandle to expose the methods in functional components, right?

Found a workaround that might be slightly cleaner than @sandorfr's above:
First, define this helper for yourself:
  const wrapInTestContext = Component =>
    React.forwardRef((props, ref) => {
      const dragDropManager = useRef();
      useImperativeHandle(ref, () => ({ getManager: () => dragDropManager.current }));

      return (
        <DndProvider backend={TestBackend}>
          <DndContext.Consumer>
            {ctx => {
              dragDropManager.current = ctx.dragDropManager;
            }}
          </DndContext.Consumer>
          <Component {...props} />
        </DndProvider>
      );
    });

Then, you can do something like this in enzyme:
//
const MyCompWithDnd = wrapInTestContext(MyComp);

const ref = React.createRef();
const enzymeWrapper = mount(<><MyCompWithDnd ref={ref} /></>);

ref.current.getManager(); // now works!

You have to wrap your component in a fragment and use a ref because of an enzyme bug.

@JKillian does the above require you to be using the Decorator style of react-dnd only? Ive tried the above using the Hooks style and it appears to be throwing Element type is invalid errors


Matplotlib problems

Dear Catalyst Maintainers,
Before I tell you about my issue, let me describe my environment:
Environment

Operating System: (Windows Version or $ uname --all)
Python Version: $ python --version
Python Bitness: $ python -c 'import math, sys;print(int(math.log(sys.maxsize + 1, 2) + 1))'
How did you install Catalyst: (pip, conda, or other (please explain))
Python packages: $ pip freeze or $ conda list

Now that you know a little about me, let me tell you about the issue I am
having:
Description of Issue
~\Anaconda3\envs\catalyst\lib\site-packages\matplotlib\contour.py in
16 import matplotlib.colors as mcolors
17 import matplotlib.collections as mcoll
---> 18 import matplotlib.font_manager as font_manager
19 import matplotlib.text as text
20 import matplotlib.cbook as cbook
~\Anaconda3\envs\catalyst\lib\site-packages\matplotlib\font_manager.py in
46
47 import matplotlib as mpl
---> 48 from matplotlib import afm, cbook, ft2font, rcParams, get_cachedir
49 from matplotlib.fontconfig_pattern import (
50     parse_fontconfig_pattern, generate_fontconfig_pattern)
ImportError: DLL load failed: The specified procedure could not be found.

What did you expect to happen?
What happened instead?

Here is how you can reproduce this issue on your machine:
Reproduction Steps

...
What steps have you taken to resolve this already?
...
Anything else?
...
Sincerely,
$ whoami

I am having the same issue, did anyone respond here on how to resolve


Change payout address to reward me in transfer

In transfer dialkog, kodadot.xyz/transfer/

We rename "Copy Payout Address" to "Copy Reward Me link",
Show Copy Reward Me link without need adding amount

the payout might get lost in translation and to ease the process we should be showing it despite the need to enter an amount to make it easier for first-time devs.
Once this is done, #2717 can be updated accordingly
Originally posted by @yangwao in https://github.com/kodadot/nft-gallery/issues/2717#issuecomment-1085761331

@yangwao I'll do this


X和YouTube帐户加载错误

Describe the issue
Problems that have always existed in your latest construction

Instructions to reproduce
After the software is opened, the account loads abnormally, and you need to log in manually every time.
What version of LiveContainer are you using?
Version 3.1.0-release (main/8c21128)
Other
No response

Suddenly appear


Reveal vs. Focus (Activate)

Currently, to make widgets visible they get activated and it is implied that a widget sets the focus on some of its elements on activate.
What is the best approach to only reveal a widget but not focus it?
For a use-case imagine debugging, where different editors and lines get revealed highlighted while stepping, but the focus should not jump to the editors.

I'm a assuming you mean when a tab is clicked. That action (clicking a tab) causes the tabActivateRequested signal to be emitted. In response, the TabPanel and DockPanel will call .activate() on the widget associated with the tab which was clicked. That method is a convenience method for posting an activate-request message to the widget, which will (when processed) call the onActivateRequest message handler of the widget. The default implementation of that method is a no-op. Widget subclasses are expect to reimplement that method to "do the right thing" when they get that message. So Phosphor is not focusing these widgets automatically, it's something the subclass is doing.


Several MIPS regression tests fail with "decoder.cpp: Assertion `isa(res.branchCall->getNextNode())' failed"

When running regression tests with the Debug build of RetDec on the current master (1da7146b9a4d4c777eff73a975337d84f7ab0e17), several MIPS tests fail with the following error:
retdec-bin2llvmir: src/bin2llvmir/optimizations/decoder/decoder.cpp:1424:
    void retdec::bin2llvmir::Decoder::handleDelaySlotLikely():
    Assertion `isa<BranchInst>(res.branchCall->getNextNode())' failed.
retdec-decompiler.sh: line 1264:  6781 Aborted   "$BIN2LLVMIR" "${BIN2LLVMIR_PARAMS[@]}" -o "$OUT_BACKEND_BC"
Error: Decompilation to LLVM IR failed

Here is a list of failing tests:
bugs.error-no-instructions-were-decoded.TestMips (mips-elf-e8e3b30ff295c1eac0cc06e1c8b1dc5e -k)     [ FAIL ]  (0.96s)
bugs.gcc-function-detection.gzip.mips.Test (gzip-strip -k)                                          [ FAIL ]  (1.45s)
bugs.llvmir-syntax-obj-defined-with-type.Test (mips-elf-fd27a58c009f1700499804d6f8a35981)           [ FAIL ]  (0.92s)

Steps to reproduce

Checkout the current master and build RetDec in the Debug mode.
Run the following commands:

$ wget https://github.com/avast-tl/retdec-regression-tests/raw/ec7d65bc95741aff954ae195ee9b1d247557867e/bugs/gcc-function-detection/gzip/mips/gzip-strip
$ retdec-decompiler.sh gzip-strip

The decompilation fails with the above error. The decompilation is expected to succeed (see this regression test).
Notes

This error only appears with the Debug build of RetDec (in the Release build, assertions are disabled).
This error is also present in the v3.1 release.
This error was introduced by 3764c1654160b90de7fba5be07e681ed690d000c (prior to this commit, the decompilation finishes successfully).

Configuration

Commit: 1da7146b9a4d4c777eff73a975337d84f7ab0e17 (current master)
64b Arch Linux, GCC 8.1.0, Debug build of RetDec

When it says "partial bin2llvmir refactorization", does it mean there is more refactorization on the way?


Move Orchest database backend to postgres

This PR changes the database backend of Orchest from sqllite to postgres. A new service, orchest-database, is launched on start.
The PR affects orchest-ctl and services that were persisting their data: the auth-server, orchest-webserver, orchest-api, celery-worker.

Agreed that this is the right moment (and PR) to change schemas to avoid issues down the line, will do.


FloatParameter default<0.01 changes to 0

When setting the default value of a FloatParameter to values smaller than 0.01, the initial value in the GUI just states 0. This happens also for exponential notations, e.g. 1e-6. When setting the value in the GUI everything works as expected. So it has to be related to the initial setting of the value.
I tried tracing down the error but I had no success so far. The change happens in the set_parameter method of the Input class in inputs.py. The value passed to self.setValue(parameter.default) is fine. Reading the set value with self.value() directly afterwards gives 0.
relevant part of the Input class code:
class Input(object):
    """
    Mix-in class that connects a :mod:`Parameter <.parameters>` object to a GUI
    input box.
    :param parameter: The parameter to connect to this input box.
    :attr parameter: Read-only property to access the associated parameter.
    """

    def __init__(self, parameter, **kwargs):
        super().__init__(**kwargs)
        self._parameter = None
        self.set_parameter(parameter)

    def set_parameter(self, parameter):
        """
        Connects a new parameter to the input box, and initializes the box
        value.
        :param parameter: parameter to connect.
        """
        self._parameter = parameter

        if parameter.default is not None:
            # here the parameter.default value is correct
            self.setValue(parameter.default)
            # here self.value() gives 0

        if hasattr(parameter, 'units') and parameter.units:
            self.setSuffix(" %s" % parameter.units)

Going further down into QtGUI's setValue method exceeds my python/programming skills... I have no idea how this works.
Its interesting that the number of decimals of the QDoubleSpinBox is set to 2. (by default?) This would explain the rounding to 0. But on the other hand, this value remains 2 when values with higher precision are entered in the GUI and these are displayed and passed correctly.
It's actually not a huge problem, since the values can still be set in the GUI. But it would be handy...

should be fixed in #278, so closing this issue


API key

Please improve your documentation on using an API key with your library.

Hi @jonhassall, this is a great request. The API key docs are hard to find because not every API supports them. For the APIs which ARE supported, here's how you could take advantage of them:
use Google\Cloud\RecaptchaEnterprise\V1\Client\RecaptchaEnterpriseServiceClient;
use Google\Cloud\RecaptchaEnterprise\V1\ListKeysRequest;
use Google\ApiCore\InsecureCredentialsWrapper;

// Create a client.
$recaptcha = new RecaptchaEnterpriseServiceClient([
    // STEP 1: use insecure credentials wrapper to bypass application default creds
    'credentials' => new InsecureCredentialsWrapper(),
]);

// Prepare the request message.
$formattedParent = RecaptchaEnterpriseServiceClient::projectName('[PROJECT]');
$request = (new ListKeysRequest())->setParent($formattedParent);

// Call the API and handle any network failures.

/** @var PagedListResponse $response */
$response = $recaptchaEnterpriseServiceClient->listKeys($request, [
    // STEP 2: Pass in the API key with each RPC call as a "Call Option"
    'headers' => ['x-goog-api-key' => ['[API-KEY]']],
]);


Event Handlers Singleton only?

Nice library. Am I reading the code correctly? It appears Event Handlers must be singletons and other lifetimes are not supported.

Currently the lifetime of the event bus is Singleton, but it would not be difficult to add a parameter to specify other lifetimes. I'm happy to add this.

I will add this to the next version.


Information Panel is not updating

Describe the bug
The information Panel does not update when changing the level

Working on it!


Azure Learn - DevOps Exercise - Include variable group

In the Exercise - Run load tests in Azure Pipelines - section Modify the pipeline configuration, I think the code highlighted should add the variables group Release, otherwise we get a "variable STAGING_HOSTNAME not found" error during the tests. The following images, Summary execution and Average response time also shows zero time for execution and response time perhaps due to the same issue. Sample as following.
- job: RunLoadTests
  dependsOn: Deploy
  displayName: 'Run load tests'
  pool:
    vmImage: 'ubuntu-16.04'
  variables:
  - group: Release
  - name: jmeterVersion
    value: '5.2'
  steps:
  - script: |
      wget -c ftp.ps.pl/pub/apache/jmeter/binaries/apache-jmeter-$(jmeterVersion).tgz
      tar -xf apache-jmeter-$(jmeterVersion).tgz
    displayName: 'Install Apache JMeter'
  - script: apache-jmeter-$(jmeterVersion)/bin/./jmeter -n -t LoadTest.jmx -o Results.xml -Jhostname=$(STAGING_HOSTNAME)
    displayName: 'Run Load tests'
  - script: |
      sudo apt-get update
      sudo apt-get install xsltproc
      xsltproc JMeter2JUnit.xsl Results.xml > JUnit.xml
    displayName: 'Transform JMeter output to JUnit'
  - task: PublishTestResults@2
    inputs:
      testResultsFormat: JUnit
      testResultsFiles: JUnit.xml

@tpetchel can you take a look at this?

@tpetchel can you take a look and let us know if you can address this issue?

@fonsecamar we're looking into this.

Hi @fonsecamar, sorry for the late response. And thank you for letting us know about this.
As you suggest, we fixed this issue by adding the Release variable group to the job. That enables the job to find the STAGING_HOSTNAME variable. You'll find that we updated the module to reflect that.
Again, sorry for the late response. In the future, you can click the feedback link at the bottom of the page when you find an error or think something needs to be clarified. That puts your comment right in our queue. Or you can file an issue on the reference code repo if you want to have a longer conversation.
Thanks!

@fonsecamar I'm going to close this based on @tpetchel  's note. Thank you for pointing out the issue.


New Hama colors 95 - 98

Hama has added some new colors recently, namely 95 - 98
https://www.hama.dk/media/102647/colour-chart.pdf
https://www.hama.dk/media/102654/99199_artikel1.pdf
I was looking into adding them myself and submitting a PR, but I am unsure what method should be used to properly get an approximate color. I tried color picking the beads in the second PDF in gimp to compare them with the values for the current colors but they were a bit off so I am uncertain if this is an acceptable approach or if there is a more proper way? Also the images in the PDF is kinda low quality so depending on where you pick the color it can vary.

Thanks for notifying this ! I wasn't aware of this new colors.
Sadly we don't really have a proper way to infer color reference (rgb or whatever) from these.
I guess the best way is still to get them in hand and try to look for reference that looks good enough to be satisfied (be careful of your own screen, since it could not display color properly as well).
I also heard of people using advanced tools like color muse, but I never got my hands on those.
Ideally this repository is here to be adjusted over time, so I don't really have a problem of adding those colors even if the reference are approximated for now. Hopefully people could refine those over time and find a agreement.
It might also be worth it to try to contact Hama color reference contributor for better advice about sampling :

@jeppewl (looks like I can't tag him)
LThanda (which is @dpalay if I'm not mistaken)

I've created pull request #29 with approximate colors using color picker on the Hama color chart for now

Hi @maxcleme I haven't worked with beads for a while so I wasn't aware there were new colors. I have just ordered them and will sample their RGB values from a photography, as I have done previously https://docs.google.com/spreadsheets/d/1Ordv1eQV0HszqxWJf5oM4alch4Q2v5dtiJb-Vui8Rh0/edit?usp=sharing

@galaxyAbstractor Thanks for contribution, I'll accept your MR and release the new version of Beadcolors (and they should be available directly from Beadifier as well)
@jeppewl Thanks in advance for this, you are going to merge the current sample in order to make them available asap, however feel free to correct those once your sample is done if you think it is necessary.

Here are my photo-based samples of the four new hama colors, as well as no. 33 which I found out is also a normal, opaque color.
33 | Cherise |  #fe1327
95 | Pastel rose | #dd9ba3
96 | Pastel lilac  | #b491ad
97 | Pastel ice blue | #8aafc2
98 | Pastel mint | #94cca4

@jeppewl Thx for contribution, I'll update them asap.


'governmental organization' (OOSTT:00000111) has no definition

Hello @neilotte,
@mbrochhausen  asked me to reach out to you and assign this issue since you were listed as term editor for 'governmental organization.'  We were unsure of the intended meaning and need your input for a definition.  Thanks.

@jmwhorton @mbrochhausen Thanks for the ping here. I think the goal was to establish a contrast class with nongovernmental organization. This will involve some modest hierarchy updates. I've open a pull request for you review. Thanks!

Updated and resolved with https://github.com/OOSTT/OOSTT/pull/21


zerver/lib/test_events: Refactor realm update tests.

Created do_set_realm_property_test, which individual realm tests use
to test updating realm properties.
Still working on how to best use the Realm.property_types structure;
ideally, do_set_realm_property_test would iterate over Realm.property_types
and no changes would need to be made to test_events.py when new realm
properties are added.
I'm also wondering about the best way to validate types -- I'm not sure
the current code best addresses type checking and validating for the different
possible realm property values and would love feedback if there are better ways
to do this.
Addresses part of issue #3854

Automated message from Dropbox CLA bot
@sarahcstringer, it looks like you've already signed the Dropbox CLA. Thanks!

(The test failures are an unrelated issue with a dependency and will go away if you upgrade)

Thanks! I'll take a look at test_realm.py next :)

Great, merged, thanks @sarahcstringer!
I guess another thread worth working on while you're at it is updating the "new feature tutorial" to discuss the fact that these pieces are happening automatically now.  I think we can still discuss those parts of the code path -- we still want to explain how this fits together -- we'll just want to reference that this is happening magically for you and where the code is so you can read it and adapt it for your feature :).  The content is in docs/new-feature-tutorial.md.

Thanks for all this! I'll now start work test_realm and looking at updating the new feature tutorial.
@timabbott, did you create the follow up piece to make the tests fail if someone adds a new setting that isn't tested, or did you want me to look into that?

No, I haven't.  The idea I had is that we'd replace the N individual tests in test_events with a single test that loops over the values in Realm.property_types, and then looks up the values to set using dict mapping field name (in Realm.property_types) to a list of values to test.  Then if a value gets added to Realm.property_types but the tests aren't updated, the test can detect that (using the dict) and throw an exception.

If that idea makes sense to you, go for it!  Also I did 06a814f6d58b729e678c525c290f2b490685e3bc recently, which is basically the same thing for one of our classes of UserProfile settings.  We could probably do the same thing for notification settings as well.


Fix some issues with simple patterns

Replaces #82
Using this system should make it more friendly to also handle record patters or whatever they are gonna throw at us next, which might need multiple passes which wouldn't have worked with #82 I think

ugh, validation isn't passing

The latest commit fixes compilation issues within Minecraft, so I think it should be good to merge now.


IOProxy read/write error handling improvements

IOMemReader::pread catch out-of-range read positions.
Fixes #3711
Along the way, I noticed that the system pread & pwrite return a ssize_t rather than a size_t as our IOProxy methods do, so I need to catch the negative value returns that they use to indicate errors.

Thank you @lgritz


Authentik support?

Hi,
The old OpenID library seems to be deprecrated.
Can we translate the openid part to support authentik as well?  Not sure if both OpenID specs / code is doing the same.
https://github.com/goauthentik/authentik/issues/4543
I wouldn't like to fork your code and make a entire new lib from it, without your help to speed up the process.
Regards

Hello my friend,
I didn't reply yet, work and private projects first.
To your comments:
I can understand that you had trouble with following the specs, implemented differently for every "ID service". Also, I understand it's a huge time investment to get everything working.

for free on features that do not benefit me

Despite the contradicting statement and a simple "no" was absolutely sufficient.
I initially offered my help and at first I was even motivated to pay you for the work.  Latter changed due to the issues in your code (see below) and your (unfriendly, almost rude) words here.
People are more motivated to pay (or pay back by filing a PR / coding or docs) when they are approached friendly and wise. Usually there is the good-will and hard-work spirit of Github that is flourishing us developers.
Authentik (Python) instead of Keycloak (Java)  could be of interest to you as well. Didn't you want to get rid of non-pythonic stuff (see your comp. website) ?
Please take the following advice for the future:

refactor your code
whenever you are re-using code snippets, don't copy & paste them, move them in a method and use that one
use methods and classes (to abstract the multiple usage into a single use)
add more comments, if needed
keep an eye on security flaws as well
keep it readable

I had not only to fix a minor sec flaw in your code. I had to refactor it heavily, especially moving a piece of code you used over and over without putting that into a method, which I did. Prob you had trouble doing so. I had them too but with a bit google-fu that issue was easily solvable or you got lazy, which doesn't fit your description though.
I am aware that you linked to this repo on your company's website. Dear future reader: People develop and they their code. The CV of kroketio is impressive, despite this minor issue.
Good luck! I wish a great time and business.


docs: move demo video under correct header

Currently, the demo video is under the JSON section, but it's actually for the select section :)

Oops, I was wrong. Ignore me 🙈


Service retries and handling rate limit (429) and forbidden error (403)

The service error handling should match sand-ruby now.
Also change to just one global cache instead of several with different default expiration time. The default expiration time of a cache actually shouldn't matter to us, since we always specify the expiration duration in every cache Set call.

[ ] @johnny-lai
[x] @abdulchaudhrycoupa
[ ] @supriya

Generated Reviewers

:warning: Do not modify anything in this section! :warning:
The content in this section is managed automatically, and any manual changes to it will be ignored and overwritten the next time the code review status is refreshed! View this PR on Cody to see the current code review status.

AMER-M-611:sand-go johnwu$ go test ./... -cover
ok  	github.com/coupa/sand-go	15.597s	coverage: 91.7% of statements
ok  	github.com/coupa/sand-go/cache	0.315s	coverage: 100.0% of statements
AMER-M-611:sand-go johnwu$

AMER-M-611:sand-go johnwu$ go test ./... -cover
ok  	github.com/coupa/sand-go	15.311s	coverage: 91.4% of statements
ok  	github.com/coupa/sand-go/cache	0.235s	coverage: 100.0% of statements
AMER-M-611:sand-go johnwu$

[lockit v4.9.3] merged via Lockit CLI with --skip-pending-status on 2021-09-06 16:04:20.4283 +0800 CST m=+3.894638589


[Automation] Generate Fluent Lite from confluent#package-2023-08-22

[Automation] Generate Fluent Lite from confluent#package-2023-08-22

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-resourcemanager-confluent


Add support of Job Names with folders

#203 added support of arbitrary Job names, but it is not possible to set a folder using the current approach. It would be great to add support of nested paths like folder1/folder2/job1 so that users of Jenkinsfile Runner can emulate standard Jenkins structures more easily.
CC @rinckm  @alxsap

Would like to give it a try!

@oleg-nenashev, after looking at the code, it seems like this does not allow for folders.
Jenkins.checkGoodName(bootstrap.jobName);

Would removing this be enough ?

I would rather create a virtual folder structure inside the JFR runtime.
It is not just enough to have an item with slashes to emulate folders for
some Jenkins plugins
On Tue, Oct 29, 2019 at 5:42 PM Guillaume Le Floch<EMAIL_ADDRESS>wrote:

@oleg-nenashev https://github.com/oleg-nenashev, after looking at the
code, it seems like this does not allow for folders.
Jenkins.checkGoodName(bootstrap.jobName);
Would removing this be enough ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jenkinsci/jenkinsfile-runner/issues/210?email_source=notifications&email_token=AAW4RICCR2X7HNIYVK4QGV3QRBDWFA5CNFSM4JDUBT32YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECQX7NA#issuecomment-547454900,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAW4RIGAJE4JMJ6HBBJJAXTQRBDWFANCNFSM4JDUBT3Q
.

Also environment variable JOB_BASE_NAME should work correctly in the pipeline.

I was just going through the issue @oleg-nenashev The above PR seems to have closed this issue, is there something pending here ?

I do not think so @sladyn98 . There are definitely some possible improvements (e.g. for multibranch), but they can be handled separately. Thanks to @addihorn for the patch!


Opening edit page view with preview-panel closed results in 3 preview requests

Issue Summary
Currently, if you open a page for editing (so the side panel is closed), this results in 3 preview requests (beside other xhr requests related to the preview) in the background! I've expected 0 preview requests ;)
I think those requests should only be executed, when the preview panel is opened by the user.

On one of our website, which has many active editors, this leads to many, I think unnecessary requests, because often those people don't use the preview at all.
Steps to Reproduce

Start a new project with wagtail start myproject
./manage.py makemigrations && ./manage.py migrate && ./manage.py createsuperuser && ./manage.py runserver
Login to the CMS
Create a new draft page (you can publish it and open the edit page again, same result)
Find the request orgy in your dev tools

I have confirmed that this issue can be reproduced as described on a fresh Wagtail project: yes

Technical details

Python version: 3.10
Django version: 3.2, 4.1
Wagtail version: 4.0.2
Browser version: Firefox 105

I would Like to take up this issue.

@AnishaSingh0118 go for it. Please ensure you understand and can reproduce the issue first. Maybe follow the steps and take a screenshot - this way we can remove the unconfirmed label.

Yes, Thank you for your response :)

Hi @AnishaSingh0118, are you still working on this? I'm interested in taking a look if not

Hi @AnishaSingh0118, are you still working on this? I'm interested in taking a look if not

Actually I have been stuck at the set up part. After following the documentation it is saying that wagtail command not found, did you faced it too?


Manage Groovy dependencies with groovy-bom import

Relates to https://github.com/apache/camel-k/issues/2418.

Let's try for 2.0.0


Distributed training with kvstore crashes if worker has different number of data batches

Description
I'm following here https://mxnet.apache.org/api/faq/distributed_training to run distributed training. The examples goes fine. But in case that the data are not divided equally among workers, once the first worker finishes training, the whole program hangs and crashes.
Is it the expected behavior for data parallelism training?
In my case, each worker reads several files and most likely the number of total records for a worker differs. Then how to deal with this case for distributed training using dist kvstore? Is there any other option than splitting the whole data to exactly n parts?
Environment settings: I'm using several CPU server nodes, and using mxnet-mkl 1.6.0b20191025
Thanks so much in advance.
Error Message
The program crashes and never exits or go on.
To Reproduce
../../tools/launch.py -n 2 -s 2 -H hosts --launcher ssh --sync-dst-dir /tmp/mxnet_job/ 'python image_classification.py --dataset dummy --model resnet18_v1 --kvstore dist_sync --log-interval 2 --batch-size 128 --epochs 3'
and in data.py, generate a random batch every time creating a DummyIter instance.

Hi @apeforest Thanks for taking time to look into this issue. Feel free to tell me if you need more information. Thanks a lot for your help!

@apeforest @leezu

mxnet-mkl 1.6.0b20191025 is very old. Try recent version from https://repo.mxnet.io/dist/index.html
CC @eric-haibin-lin

Could you post your script to reproduce? Thanks

@leezu I upgraded to the just recent release 1.6.0 and the problem still exists.
@apeforest I'm running the image classification example: https://github.com/apache/incubator-mxnet/blob/master/example/gluon/image_classification.py
You can reproduce using the following command:
../../tools/launch.py -n 2 -s 2 -H hosts --launcher ssh --sync-dst-dir /tmp/mxnet_job/ '/opt/work/client/anaconda3/envs/mxnet/bin/python image_classification.py --dataset dummy --model resnet18_v1 --kvstore dist_sync --log-interval 2 --batch-size 128 --epochs 3'

And also modify https://github.com/apache/incubator-mxnet/blob/master/example/gluon/data.py#L138 to generate a random number of batch for each DummyIter instance:
import random
self.batches = random.randint(10, 20)

The program hangs once the worker having smaller batches finishes the training.
Hope to get your help on this issue. Feel free to point out if I'm doing anything wrong. If it is truly a bug, then is there any workaround for this? Thanks.

@hkvision what data iterator/dataloader are you using? if you use data iterator, you can use io.ResizeIter to make sure the size is balanced. For dataloader, you can use the SplitSampler with even_size=True: https://gluon-nlp.mxnet.io/_modules/gluonnlp/data/sampler.html#SplitSampler

@eric-haibin-lin Hi, I'm using NDArrayIter. I checked ResizeIter, so in order that all my data are trained, I need to set the size to the largest batch among all workers and for workers with less batches, after finish all the data, it will iterate from the very beginning until the target size is reached. Point out if I'm wrong. :)
Thanks so much and this can be a workaround for me.

I think that's correct. Please let me know if you come across issues using that.


Serial message queue

While the serial editor is invisible, it saves messages in a queue and displays them upon opening.
Removed all logic surrounding "source" - now graphs are simply separated by variable name.  Separating graphs by source (which is runtime ID for simulator) led to conflict: queued data and live data from the same stream were going to different graphs because the runtime ID changes when you open the serial editor.
Should go into master too.

@pelikhan This should be ready to merge.

Needs cherry-pick to master.


New Crowdin translations

Enjoy!

Closing so that the new config will hopefully generate a fresh PR and put the files in the correct directories


chore: upgrades

Codecov Report

Merging #134 (faab178) into master (e6e1434) will decrease coverage by 0.14%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #134      +/-   ##
==========================================
- Coverage   79.27%   79.13%   -0.15%
==========================================
  Files         123      123
  Lines       11394    11394
  Branches     1230     1228       -2
==========================================
- Hits         9033     9017      -16
- Misses       2361     2377      +16

Flag
Coverage Δ

unittests
79.13% <ø> (-0.15%)
:arrow_down:

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

Impacted Files
Coverage Δ

...kages/lib/components/auto-suggest/auto-suggest.tsx
85.28% <0.00%> (-6.93%)
:arrow_down:

Continue to review full report at Codecov.

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


 in ptipython does not do history substring search

In ipython, I can do the following:
In [1]: x = 1 + 1

In [2]: y = 2 + 2

In [3]: x = <up arrow key on keyboard>

and ipython completes it to x = 1 + 1. I can't see any way of doing this in ptipython.

Hi @Wilfred,
In ptipython, go to the Menu (Press F2) and turn history search on.
To make this setting persistent, save this in the config: https://github.com/jonathanslenders/ptpython/blob/master/examples/ptpython_config/config.py#L76
Hope that works,
Jonathan

Cool, I didn't know about that feature. It seems like the enable_history_search and complete_while_typing features could be compatible if history search was disabled in insert mode since completion is already insert-mode-only.

Thanks, that works nicely.


Move IHeaderDictionary to Features to reduce wrapping.

#426 Starting with just IHeaderDictionary because it has cross-repo impact.
https://github.com/aspnet/Hosting/commit/12770e2b1569f7ff028dba873bbbf7c1bf0cc8da
https://github.com/aspnet/KestrelHttpServer/commit/4195aced5fb266c9050dc608cef4d7879a7328b7
https://github.com/aspnet/WebListener/commit/1befc3b39df886cd96d0f70ab44a07aea54ff34b
@muratg @davidfowl @lodejard @benaadams

Looks good :shipit:


Refactored Chromecast module usage

@vidible/mobile-sdk-team: Ready for review.
JIRA

Please, provide meaningful description for ticket


[Project Addition] : Insect Detection and Classification using DL

Pull Request for DL-Simplified 💡
Issue Title : Insect Detection and Classification using DL

Info about the related issue (Aim of the project) : To detect and classify insects using mentioned dataset below.
Name:  Aaradhya Singh
GitHub ID: https://github.com/kyra-09
Email ID<EMAIL_ADDRESS>Idenitfy yourself: (Mention in which program you are contributing in. Eg. For a JWOC 2022 participant it's, JWOC Participant)  GSSOC-2024 Contributor

Closes: #720
Describe the add-ons or changes you've made 📃

Utilized 5 models such as EfficientNetB0 , Xception, VGG16, ResNet50, and InceptionV3 for image classification.
Applyed data augmentation (rotation, zooming, flipping, shearing, brightness) to enhance dataset robustness.
Compared model performance using accuracy scores, loss/accuracy graphs, and confusion matrices.
Conducted EDA for dataset insights, including image distribution, quality, class imbalances, and sample visualization.
Documented the process in a comprehensive README file.

Type of change ☑
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested? ⚙
Describe how it has been tested
Describe how have you verified the changes made
Checklist: ☑

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[x] I have made corresponding changes to the documentation.
[ ] My changes generate no new warnings.
[x] I have added things that prove my fix is effective or that my feature works.
[ ] Any dependent changes have been merged and published in downstream modules.

@abhisheks008 kindly review my pr and if posiible kindly assign the 3 issues I have opened , you'll get the pull requests of all of those issues by the morning please if possible.


cloud_factory only parses azs

Instead of parsing the whole cloud config cloud_factory
now parses only the azs, because only the azs are needed.
 A deployment previously failing with task timeout ran successful with this change.


Feature: Add JSON2CSV app specification

Add JSON2CSV app specification
Resolves: N/a
See also: N/a

Note that the basic requirements make this a Beginner app, but the bonus requirements push it to an Intermediate level app.

Good one @jdmedlock !


Fresh app doesn't connect to nodes or connects strangely

Fresh app => app on first run. Subsequent runs don't produce the same behavior.

It fails to connect to the same nodes which actually work on subsequent runs.
Fails to connect to most nodes (works on subsequent runs).
Even if it connects, it disconnects after 10 seconds or so
Some nodes work okay though even on the first run

No errors visible, but in myst_supervisor.log I constantly get these:
2020/04/14 17:15:33 Running myst supervisor daemon
2020/04/14 17:15:33 Waiting for connections...
2020/04/14 17:15:33 Waiting for connections...
2020/04/14 17:15:33 Client connected: unix
2020/04/14 17:15:33 myst [err=exec: already started] output:
2020/04/14 17:15:33 Error running myst: exec: already started
2020/04/14 17:22:35 Client disconnected: unix
2020/04/14 17:22:40 Waiting for connections...
2020/04/14 17:22:40 Client connected: unix
2020/04/14 17:22:40 myst [err=exec: already started] output:
2020/04/14 17:22:40 Error running myst: exec: already started

exec: already started means exactly that - you have a conflicting instance of node running on your machine. Close it (kill it) and it will work just fine.

No, it was clean run.
I've tried with Mysterium v1.0 release https://github.com/mysteriumnetwork/mysterium-vpn-desktop/releases/download/SNAPSHOT/MysteriumVPN.1.0.dmg
Same behavior.
Doesn't connect to the node. After I relaunch the app, it works.
Try this proposal:
0xa790b42ed41898122fb485c79267cdcf53482412 | GB | openvpn

I hope you understand that this is in active development and things are expected to break often. it's clearly written in the readme. Contributions are welcome. 😎

I hope you understand that I'm building business on this app, I'm not playing around.
Please talk to @chompomonim

Also, that open source project is in active development is not a reason to close issues without resolving them.

Created issue on node project, as I guess it's node problem: https://github.com/mysteriumnetwork/node/issues/2051

It's great that you are building on the foundation of our apps and platform. But let's get one thing straight - this is an open-source project. You're welcome and free to use the source, modify it and adapt it to your needs, but ultimately I don't own you anything. If you continue to communicate in an aggressive manner and make demands, your rights will be limited.

I'll create issues in node project from now on.

node v0.30.0-pre which is used in this desktop app build is not very stable. It's pre-release and there are a couple of problems to fix. I think if you would try to connect into same node via CLI it would also not work :(
We're fixing issues with this pre-release and should release a stable version (or at least another, more stable pre-release) until the end of this week. I do believe that a desktop app will also work more stable then.

The strange thing here is that this occurs only on the first run (and the most important one for new customers), the subsequent runs works really well.


Rewrote the Scarlet Crusade's quest flavour with an eye for epicness

Is there a comma missing there? After Other


Consider a more explicit message about missing outputs when detected...

Testing #129463

Imagine a new user is opening a notebook with VS Code. No Python or Jupyter is installed and are perusing a set of notebooks for the first time.
They either:

Notice the notification asking whether they want to install extensions for Python and ignore or decline
Don't notice the notification

They are looking for a notebook with some set of output

As they scroll, they don't notice some of the cell outputs that indicate that a renderer can't be found. However, they may also see some cell output rendered, so they have no reason to believe anything is amiss.
They continue looking for output, but eventually discover they need to do an extra step and install the Jupyter extension at least.
Possible suggestions:

Add color and/or icon to the message indicating a renderer cannot be found to make it stand out.
Detect the situation and automatically display the "install additional renderers from the marketplace" quick pick
Show a modal dialog box indicating that some cell output may not be visible until additional renderers are installed.
Possibly other ideas... the above are just possibilities.

@connor4312 I think you did this? Do you have an issue to dupe against?

Yea, there's now a button for outputs:

https://github.com/microsoft/vscode/issues/123581


Implement to_owned on all key types that has a lifetime, and avoid using the stack in the boxed functions

We only had to_owned on the PublicKey. There might be reasons why people want 'static versions of the SecretKey and SharedSecret as well. So let's implement that.
I also added tests for these conversions.

Not stack overflows on all platforms? Oh god, this library is really not usable without heap allocation :sweat_smile:.. I'll see if this hack solves it or if I need to heap store the buffer for this test. The latter is fine, since the test is only for the alloc feature anyway.

I really don't understand why this stack overflows when very similar tests don't :eyes:

Ok, ready for review :+1:

I found out why the Windows tests fail... I knew this but had forgotten it. Box::new([0u8; HUGE]) still puts the huge array on the stack temporarily, even with optimized compilation. So I had to introduce this ugly hack to avoid touching the stack. I also adedd a test that will make sure that the allocating version of the kem can be performed on only 512 kiB of stack.

@dkales Ready for review (again). Sorry for mixing so many fixes into the same PRs. But I think we are making great progress here.

The doctests in the readme still fail on windows, but this seems kind of unavoidable? we could mark them as no_run, but then this kind of defeats the point?
Maybe also duplicating them as normal tests with a cfg(not(windows))?

Yeah. I did not focus on fixing those tests in this PR. I felt that this PR was already doing too much.
Yeah. I wanted to avoid the no_run part. I'd like to find a better solution. Maybe we can make github actions only run the tests for the small variants on Windows specifically or something.

I/we can look at that later. But I don't think it should block this PR.


Integrations: Improvements to dropdown menu behaviour on docs home page.

In this PR we have commits which migrate integrations docs page to use font awesome 4.7 prefixes and fixes buggy behaviour around the categories drop down and its alignment relative to the integration lozenges.
Below I attach a number of pics relating to the alignment improvements I did. I am gonna attach pics for the page in different resolution as it would look after merging this PR rather than doing an old vs new on the images. I am not attaching screens at some resolutions (of course each screenshot represents a change in resolution level in the sense what we define using media queries) because things around the drop down don't change at those levels.

I merged the first commit.
@adnrs96 can you please include a prose explanation for why the second and third commits are correct?  I don't understand a number of things about them.
Once that's done, we should get a review from @synicalsyntax or @jackrzhang.

@synicalsyntax can you review this?

@synicalsyntax Thanks for reviewing. Addressed your comments above and updated this PR.
I think this should be ready for a merge @timabbott

I rebased and merged this, thanks @adnrs96!  This improves that page's mobile experience a lot.
The only thing I couldn't reproduce as fixed was the pointer behavior being correct; can you do another round of QA on that page's mobile responsive behavior just to make sure I wasn't testing things incorrectly?  Thanks!


Merge pull request #16829 from shajrawi/fix_dynamic_cast

Assume we have:
Protocol P
An intenal extension to Optional - conforming to P
Compiling under whole module mode
Use of Optional type T?
Given a dynamic cast from T? to P, that cast might succeed even if T itself does not conform to P
rdar://problem/40500142

@swift-ci Please test and merge


missing docstring in Law2021-model

https://github.com/jonescompneurolab/hnn-core/blob/d6b08ff60d9a74d517582755d5e0d6d4dbd3df89/hnn_core/network_models.py#L197
Didn't spot this in review :) @ntolley ?

Addressed in #333 (along with many many more fixes)
https://github.com/jasmainak/hnn-core/blob/90f33f3a397005ee300de98909a6e4603e93a22d/hnn_core/network_models.py#L198


refactor share page to be a modal component

The separate page for /share is unnecessary and causes conversation state to be lost.

hey @kasumi-1 can i work on this

@agrim0312 sure thing!

@kasumi-1 are you using any UI library or i need to build it from scratch using divs??

tailwind

@kasumi-1 please check the PR and please guide me if any changes required as i am new to open source.


⚠ Samsung has degraded performance

In e746f7e, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2273 ms

Resolved: Samsung performance has improved in 3a8bae7.


How to create training data for custom character image

I performed the OCR training in the matlab for custom character image. so recognition correct rate becomes good
matlab ocr is based on tesseract
But I can't get the .traineddata for custom character image

I'd search for the newly generated .traineddata file on your computer's storage.
And please use the Tesseract user forum for questions.


Pull Request Template Typo

Motivation
Description
The pull request template has a typo: restructoring

I guess this was made from a blank template. I was wondering why this looked weird.


Multiple Access-Control-Allow-Origin Headers

Hi There,
just hit a Problem - would be nice if you could implement it.
Multiple "Access-Control-Allow-Origin"-Headers are not allowed:
https://developer.mozilla.org/en-US/docs/Web/HTTP/CORS/Errors/CORSMultipleAllowOriginNotAllowed
Instead of
foreach($allowedOrigin as $allowed_origin) { // to support multiple origins
	header("Access-Control-Allow-Origin: $allowed_origin");
}

do
header("Access-Control-Allow-Origin: ".implode(", ",$allowed_origin);

yes - this should work for me - will get back if i integrate a new version of your rest-server into my project - but my workaround looks and works the same:
$server = new \Jacwright\RestServer\RestServer($config["mode"]);
$server->useCors = false;
$server->allowedOrigin = $config["allowedOrigin"];

$server->addClass(...);

header("Access-Control-Allow-Origin: ".implode(", ",$config["allowedOrigin"]));
header('Access-Control-Allow-Methods: GET,POST,PUT,DELETE,OPTIONS');
header('Access-Control-Allow-Credential: true');
header('Access-Control-Allow-Headers: X-Requested-With, content-type, access-control-allow-origin, access-control-allow-methods, access-control-allow-headers, Authorization');
if ($server->getMethod() == 'OPTIONS')
{
    //Exit because of Bug in RestServer that makes it crash when not using cors
    exit;
}

$server->handle();


Precompute a zone context to avoid always having to compute them on t…

…he fly.
@bratseth PR

Try to counter performance regression due introduced in #23080


Feature/119 inifield file

Fixes #119. Both the iniFieldFile, as well as 1D field INI file format (see UM sec. D.2 and D.4.1).

Code smell now also solved.


Add docstrings to all methods

Currently, several docstrings are missing. Now that we've implemented a semi-automated API documentation, all docstrings need to be filled.

We started a discussion about the docstring style here: i156.
But I do not think that a decision has been taken.
Also reflected in the repo', as the two formats are present:

example of Napoleon
example of Sphinx

Note: I think only Sphinx docstring have been imported in the ReadTheDoc for now.
(I have a preference for Napoleon's clarity.)

Suggestion of split:
@andreanne-lemay

loader folder
models

@olix86

testing folder
losses
metrics

@charleygros

main (and derivatives in coming PR #230 )
transforms
utils

Does it sound fair?
Happy to discuss this, we could swap etc.

Note: I think only Sphinx docstring have been imported in the ReadTheDoc for now.

Fixed in #258

Suggestion of split:
@andreanne-lemay

loader folder
models

@olix86

testing folder
losses
metrics

@charleygros

main (and derivatives in coming PR #230 )
transforms
utils

Does it sound fair?
Happy to discuss this, we could swap etc.

Seems good to me :)


Add invite feature

As you said:
I've also recently created an invite system built on this and supabase edge functions which allows group owners to generate a token which other users can use to join their group with a specific pre-selected role. I have not yet extensively tested it but if you are interested in this code as well

Please add this feature. Thanks

I have one question. In this following code:
set check_function_bodies = off;

CREATE OR REPLACE FUNCTION public.set_group_owner()
 RETURNS trigger
 LANGUAGE plpgsql
 SECURITY DEFINER
AS $function$
	begin
		IF auth.uid() IS not NULL THEN
		insert into public.group_users(group_id, user_id, role) values(new.id, auth.uid(), 'owner');
		end if;
		return new;
	end;
$function$
;

CREATE TRIGGER on_insert_set_group_owner AFTER INSERT ON public.groups FOR EACH ROW EXECUTE FUNCTION set_group_owner();

Why need to set check_function_bodies = off;

Why need to set check_function_bodies = off;

Actually, we don't. Feel free to remove or skip that. It gets automatically added by the diff tool when I use supabase's cli to create the migration scripts.

Just popping in to say I haven't forgotten this. Just been busy. This is on my to-do list.

this would be great addition

I have added this in the dev branch. An updated TLE is coming soon but is currently blocked by this bug.

@point-source First of all - thank you, great work! I am looking forward to this extension!
Is it possible to extend the invitation logic to allow multiple roles when creating the invitation? This would allow more complicated role setups right after invitation.


[Feature]: Allow plugin specific logical implementation on client side

Is there an existing issue for this?

[X] I have searched the existing issues

Summary
Make sure that the client can support features not just on the basis of config, but in code.
Why should this be worked on?
Allows us to start supporting feature hierarchies on the client side as well.

issue unclear; if this is still needed, please reopen with more detailed description including the problem statement and the solution.


Run example error

Hi, when i run the run_gym.py, it reports the following error:

could you give me some help?

Seems there is an incompatible version of numba to python3.7. Please upgrade your py to 3.8 then have another try. @nuaasgq

Thanks! I upgrade py to 3.7.13 and the problem has been solved.

@nuaasgq the training pipeline of MALib has been reconstructed, please follow the changes in master branch.


Add some sort of toast notifications and update ResourceLanguagePreferenceSwitcher to use it for errors

Working on this one.

https://github.com/euvl/vue-notification/

Also, update language switcher to function like the resource language preference switcher functions, where it is an ajax call to preferences, after which the page changes. Unlike the pref switcher though it'll be forwarding to a new URL.


Replace Launch Configurations with Launch Templates

AWS is retiring launch configurations:

The standard going forward will be launch templates.

I second this request.


readme updates to include Invoice Example.

Co-authored-by: jsandov<EMAIL_ADDRESS>

Looks promising!

Let's land it!


HC-546: Update logstash to move a failed job over to a new job_failed index

This PR is in conjunction with the following other PRs:

sdscli: https://github.com/sdskit/sdscli/pull/109
lightweight jobs: https://github.com/hysds/lightweight-jobs/pull/34

This PR updates the logstash configuration such that when it sees a failed job, it will make sure to move it from job_status to job_failed. Note that we don't want to introduce any rollover since the intent is to preserve this failed jobs for further analysis and put the onus on the end user to purge this failed jobs eventually.
Testing
See comments in https://hysds-core.atlassian.net/browse/HC-546

Nice. Didn't realize logstash had a way of deleting documents at the same time it indexes one.

Ya Hook suggested that this solution could be done through logstash and so had to look up in the docs on how to do this.


pause-bin: remove signal handlers

Remove signal handlers from pause-bin to prevent it from being
killed by other members of the pid namespace, causing all processes
in the same pid namespace being killed.
The agent can always kill the pause-bin with SIGKILL from outside.
Signed-off-by: Peng Tao<EMAIL_ADDRESS>

LGTM

LGTM

How to make codecov/project — 27.77% (-0.25%) passed?

Need unit tests...

added unit test.