trigger command not aliasing correctly

Trying to use this to turn /trigger status-live from a datpack into /status live. It doesn't throw an error but the command just does nothing. My suspicion is that the trigger command is not working as it can't find the player that used it. I'll leave the datapack and tyhe config files I'm using left below. The config includes other status commands I'm trying to get working, but focusing on live is my goal.
status-plugin-1-1.zip
{     "schemaVersion": 1,     "commandMode": "COMMAND_CUSTOM",     "command": "status",     "permission": 0,     "message": "hello parent command",     "children": [         {         "child": "live",         "type": "argument",         "argumentType": "minecraft:word",         "suggestionProvider": {             "suggestionMode": "DATABASE_STARTS_WITH",              "suggestion": "$executor_name().home.suggestions"         },         "permission": 0,         "actions": [             {                 "command": "trigger Status-Live"             }         ]         },         {             "child": "recording",             "type": "argument",             "argumentType": "minecraft:word",             "permission": 0,             "actions": [                 {                     "command": "trigger Status-Recording"                 }             ]         },         {             "child": "reset",             "type": "argument",             "argumentType": "minecraft:word",             "permission": 0,             "actions": [                 {                     "command": "trigger Status-Reset"                 }             ]         }     ]     }

Okay, Fixed it. It was a nightmare though so I'm glad I added the issue. The commantType needed to be CLIENT

You should remove the suggestion provider if you aren't using it
"suggestionProvider": { "suggestionMode": "DATABASE_STARTS_WITH",  "suggestion": "$executor_name().home.suggestions" }


plugins/everforest: init

Adding the everforest colorscheme https://github.com/sainnhe/everforest
Fairly new to nix, I would super appreciate any feedback yall have!
I pulled all the settings info strings directly out of :help everforest

Looks like there is something wrong with the tests.
You can debug locally by entering the devshell (nix develop or using direnv).
Then, simply run the tests command.

Got it, thank you!
So it looks like the issue is due to the better_performance flag being set to 1. This causes the everforest plugin to write stuff to the disk for caching or something.
Problem is Nix's immutability is not allowing that, and it fails. Theres no setting I can find to tweak the cache location.
Seems to me an upstream problem. What would you recommend I do to resolve this? Is there way to resolve this within Nixvim? Or perhaps just modifying the docs and raising an assertion error if that setting is set?

Problem is Nix's immutability is not allowing that, and it fails. Theres no setting I can find to tweak the cache location.

If so, that should only be affecting the test and should be fine when running normally? Unless the plugin is trying to write its cache to the nix store.
If it is just the test that's affected, you can disable whatever option is needed. Along with a comment, explaining why said option causes tests to fail.

Unfortunately no its trying to write to the nix store. Can turn it off in the test, but this feature will not work in a real environment
line 5: Vim(call):E739: Cannot create directory /nix/store/0y403ixxd7jkdxvsjwl6alkayf7qxsza-vimplugin-everforest-2024-06-02/after: read-only file system

Unfortunately no, its trying to write to the nix store. Can turn it off in the test, but this feature will not work in a real environment

Frustrating. We'll just have to disable the test and report an issue upstream, recommending they use XDG_CACHE_HOME. If an issue is open before this PR is done, we can reference it in a comment.
As for the option, we don't actually have to explicitly declare all availabile settings options anyway, since settings is a freeform option, users can define any config they like, even stuff we haven't added options for.
In some respects it's better not to define all available options; once we've added them we then have to try and keep them up to date.

Cool sounds good! Ive removed the option from the settings. I'm happy to remove any others, though I'm not sure which ones are important enough to be kept in vs removed

This is an excellent first time contribution! I hope you don't mine, I left some further feedback and discussion points.

I'd prefer links to use markdown link syntax, so they are actually clickable in the docs and render nicely.
I'd prefer a comment be left about the broken option (up to you)
I'm confused as to why removing a default value has any effect; the whole point of the "plugin defaults" is they should match the plugin's defaults, as used when the option is not defined

Thank you! :) Ive addressed all those concerns.
The better_performance = 1 value is not the default, I actually intentionally made all the test values non-default to potentially catch any that I didnt find on my mostly unmodified install (which worked in this case, wahoo :smile:)
Do you expect the tests to be testing the default values? Is there a way I can create multiple tests in this file to test both the default and non-default cases?

OH, I did not realize the test values are literally named as plugin.defaults! Only after re-reading your comment did I understand. Ill update those to be default values haha

Thank you! :) Ive addressed all those concerns.

Brilliant

Do you expect the tests to be testing the default values? Is there a way I can create multiple tests in this file to test both the default and non-default cases?

Yes you can create as many test as you like, generally "empty" and "defaults" are the bare minimum, but "example" is also commonly used to test non-defaults.
Tests can be named anything, those are just the most common names we use.


Source and target pods not starting after installing tm with helm

I'm installing on docker desktop K8s 1.22.5 with the following:
kubectl apply -f https://github.com/knative/serving/releases/download/knative-v1.4.0/serving-crds.yaml
kubectl apply -f https://github.com/knative/serving/releases/download/knative-v1.4.0/serving-core.yaml
helm install -n triggermesh triggermesh triggermesh/triggermesh --create-namespace

Then trying to create an SQS source and broker like this:
tmctl dump | kubectl apply -f -

Not seeing the SQSSource pod started. Output:
kubectl get awssqssource.sources.triggermesh.io
NAME               READY   REASON         SINK   AGE
foo-awssqssource   False   SinkNotFound          4m44s

and seeing this in the TriggerMesh controller logs:
{"severity":"INFO","timestamp":"2022-12-16T12:20:28.925248218Z","logger":"triggermesh-controller","caller":"awssqssource/reconciler.go:287","message":"Returned an event","commit":"cac9c45","knative.dev/controller":"github.com.triggermesh.triggermesh.pkg.sources.reconciler.awssqssource.Reconciler","knative.dev/kind":"sources.triggermesh.io.AWSSQSSource","knative.dev/traceid":"d0ec29ce-bff4-4d9e-9f16-f10420aa607c","knative.dev/key":"default/foo-awssqssource","targetMethod":"ReconcileKind","event":"Could not resolve sink URI: failed to get lister for eventing.triggermesh.io/v1alpha1, Resource=redisbrokers: redisbrokers.eventing.triggermesh.io is forbidden: User \"system:serviceaccount:triggermesh:triggermesh-controller\" cannot list resource \"redisbrokers\" in API group \"eventing.triggermesh.io\" at the cluster scope: RBAC: clusterrole.rbac.authorization.k8s.io \"addressable-resolver\" not found"}

{"severity":"ERROR","timestamp":"2022-12-16T12:20:29.131976385Z","logger":"triggermesh-controller","caller":"controller/controller.go:566","message":"Reconcile error","commit":"cac9c45","knative.dev/controller":"github.com.triggermesh.triggermesh.pkg.sources.reconciler.awssqssource.Reconciler","knative.dev/kind":"sources.triggermesh.io.AWSSQSSource","knative.dev/traceid":"3529b8ff-b3fa-4f27-ae9c-f54aeeee2172","knative.dev/key":"default/foo-awssqssource","duration":"87.322834ms","error":"Could not resolve sink URI: failed to get lister for eventing.triggermesh.io/v1alpha1, Resource=redisbrokers: redisbrokers.eventing.triggermesh.io is forbidden: User \"system:serviceaccount:triggermesh:triggermesh-controller\" cannot list resource \"redisbrokers\" in API group \"eventing.triggermesh.io\" at the cluster scope: RBAC: clusterrole.rbac.authorization.k8s.io \"addressable-resolver\" not found","stacktrace":"knative.dev/pkg/controller.(*Impl).handleErr\n\tknative.dev/pkg@v0.0.0-20220818004048-4a03844c0b15/controller/controller.go:566\nknative.dev/pkg/controller.(*Impl).processNextWorkItem\n\tknative.dev/pkg@v0.0.0-20220818004048-4a03844c0b15/controller/controller.go:543\nknative.dev/pkg/controller.(*Impl).RunContext.func3\n\tknative.dev/pkg@v0.0.0-20220818004048-4a03844c0b15/controller/controller.go:491"}

Solved via https://github.com/triggermesh/charts/pull/181


Node doesn't have console.debug

I'm submitting a feature request

Library Version:
1.0.0

Please tell us about your environment:

Operating System:
OSX 10.13.3

Node Version:
8.9.4

NPM Version:
5.6.0

Language:
all

Current behavior:
If I use this project on the server (node), logging to a logger with level "debug" does not output anything to the console. It's a noop.
Expected/desired behavior:
const logging = require('aurelia-logging');
const { ConsoleAppender } = require('aurelia-logging-console');
logging.addAppender(new ConsoleAppender());
const logger = logging.getLogger('my logger');
logger.setLevel(logging.logLevel.debug);
logger.debug('hello');

I expect to see the following on stdout:
DEBUG [my logger] hello

What is the motivation / use case for changing the behavior?

This project states that its meant to work on the server, but not having a debug level on Node is less than complete support.

i just saw #10.
it's my opinion that delegating this to the pal-nodejs package is quite a bloated solution for what amounts to a single || console.log expression.
is there any information regarding what interface the pal* packages are abstracting to? are there specific web standards, proposals, etc?

@aoberoi what problem are you facing?
what scenario does not work for you?

@Alexander-Taran I’m not longer using this package - I don’t think my use case fits it well.
My only feedback is that I hope the maintainers consider shifting the responsibility of working around the missing console.debug in Node to this package, not the platform abstraction libraries. Another alternative would be to make a formal proposal for Node to gain a console.debug function.

Thanks for your input.
Can this issue be closed then?

Yes, closing


update listify command to work with selection and add keyboard shortcut

I originally wanted to add in just the keyboard shortcut (cmd+shift+8 I think is by far the most common, google/slack etc.) but also fixed a bug /expanded scope that allows you to listify the current line that the cursor is in, not having to select text first.
I think ideally listifying would be more like a toggle (subsequent invocations would de-listify) instead of the current behavior where you just add bullets with each invocation.
As always, open to whatever feedback and discussion!

Looks reasonable. Indeed, making it a toggle would be nicer, maybe a future change? Merging this one for now.


Complete FixedSizeList Scalar support

Is your feature request related to a problem or challenge?
Various implementations are missing:

[x] #8224
[ ] to_array_of_size for FSL
[ ] eq_array for FSL

Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context
No response

2 days and 2 PRs to fix. Nice work @wjones127  -- thank you 🙏


fix: Remove name from Angular selector name

Description
Removes , json.name} from Angular's selector template
To better comply to angular's style guide
Closes #1442

Oh this question might seem stupid but, what would be the command to update affected snapshots?
:sweat_smile:  I had just updated them manually but I'm not sure if this is optimal

I'm having some issues running the snapupdates.
Is it suppose that take this long 10+ minutes?
So far I've changed the expected results manually.

https://github.com/BuilderIO/mitosis/issues/747
Relevant PR

can we explicitly mention by using useMetadata instead?

Sorry for taking so long to respond.
I can try pushing some changes this Sunday.
But I wouldn't mind if this PR was taken over.

@seofernando25 I can't select your fork as base:

I add the selector option to useMetadata here. Would you like me to open up a new PR from my fork? It's wired because I see other forks as base branches as well

@nmerget oh yes please, feel free!


update the footer to make it easier to find things

Suggesting an update to the developer site's footer to help guide the user in certain areas.
@tpmccallum would love your input and thoughts on what things should be made "global" in this way.
To replace:

Wireframe / suggestion:

Our left menu and site search should remain the best place for finding your way around the Documentation, the footer more so is for considering next steps for a user to find other, related parts of the Fermyon universe of things. Some caveats:

the more links we squeeze in here, the noisier it gets. Hence I've kept this to two columns / 8 links - but don't want to miss anything critical.
this goes across Cloud, Spin, Bartholomew, so should balance for both the open source developer audience and the potential Cloud customer

Comments welcome in here or in Figma (fermyon-only)

Would this remove the links to discord/github/twitter ?


Add an option to write the cell code to a temporary file

Hello,
your plugin is simple and works really good.
But i've trouble integrating some interpreters, that don't have a command argument for executing code directly (-e, -c).
They can just run files given as arguments.
Can you add a simple feature for that? Something that i can configure my interpreters like this:
"handydandy-notebook.dispatch": {
  "C": [
    "tcc",
    ["-run", "${code:tempFile}"]
]
}

If you find the string ${code:tempFile} as argument, replace it with something like: path.join(os.tmpdir(), Date.now(), language) and write the cell.document.getText() to that file.
If you find that useful, i can also try to implement that and give you a PR for this.
Thank and greetings,
Steven

It's something I've considered, though I do want to keep the extension as simple as possible and the second we start creating files we need to worry about things like:

cleaning them up (especially in the case of errors & long running commands)
where they're located (some path relative to the notebook? workspace? extension? and what about if the notebook is unsaved?)
what name they're given (some utils like to be particular about precise file extensions)
permissions
etc.

I'm not fully against the idea, but it does have potential to be a can of worms.
It should be possible to construct a shell script that accepts standard input and writes it to a file in a location of your choosing then runs an executable of your choosing against that, have you looked into such an approach?
For instance:
run-tcc.sh:
#!/bin/sh
echo "$1" > ./tmpfile.c
tcc -run ./tmpfile.c

handydandy-notebook.dispatch:
    ...
    "c": [
      "/Users/jkearl/Downloads/tcc-0.9.27/stdin-to-tcc.sh",
      [
        "${code}"
      ]
    ],
    ...

We could provide a ${cell-id}or ${execution-id} expansion of some sort to help when executing multiple cells at once.

It would be really neat to be able to create a temp file for each language so we can keep the same context between cells of the same language. Maybe even an option to give a name or id to each context of each language too so you can order stuff around. I think this really increases the number of use cases for this already great extension. Perhaps once I finish finals at university I'd be interested in collaborating and implementing this :).

@HomemadeToast57 this is also something I've considered. Inspiration dates back to Knuth's concept of Literate Programming, in which blocks of code are written out in some intuitive sequence, then concatenated together in arbitrary orderings using a form of macro expressions to form the final product to be delivered to the interpreter . Take for instance this wc implementation: https://github.com/zyedidia/Literate/blob/master/examples/wc.lit
In handy dandy notebook, that program may look something like:

 Full code:

# Word Count Program
Here, is an overview of the file wc.c that is defined by this Literate program:
@{Header files to include}
@{Preprocessor definitions}
@{Global variables}
@{Functions}
@{The main program}

Header files to include
We must include the standard I/O definitions, since we want to send formatted output to stdout and stderr.
#include <stdio.h>
#include <fcntl.h>
#include <unistd.h>

Preprocessor definitions
We define various codes to represent different exist conditions of the program
#define OK 1    /* status code for successful run */
#define usage_error 1    /* status code for improper syntax */
#define cannot_open_file 2    /* status code for file access error */

Global variables
The status variable will tell the operating system if the run was successful or not,
and prog_name is used in case there’s an error message to be printed.
int status = OK;    /* exit status of command, initially OK */
char *prog_name;    /* who we are */

The main program
Now we come to the general layout of the main() function.
int main(int argc, char **argv)
{
    @{Variables local to main}
    prog_name = argv[0];
    @{Set up option selection}
    @{Process all the files}
    @{Print the grand totals if there were multiple files}
    return status;
}

Variables local to main
int file_count;    /* how many files there are */
char *which;    /* which counts to print */
int silent = 0;    /* nonzero if the silent option was selected */

Set up option selection
If the first argument begins with a '-', the user is choosing the desired counts and
specifying the order in which they should be displayed. Each selection is given by the
initial character (lines, words, or characters). For example, '-cl' would cause just
the number of characters and the number of lines to be printed, in that order. The
default, if no special argument is given, is '-lwc'.
We do not process this string now; we simply remember where it is. It will be used
to control the formatting at output time.
If the '-' is immediately followed by 's', only summary totals are printed.
which = "lwc";    /* if no option is given, print all three values */
if (argc >1 && *argv[1] == '-') {
    argv[1]++;
    if (*argv [1] == 's') silent = 1, argv [1]++;
    if (*argv [1]) which = argv [1];
    argc--;
    argv++;
}
file_count = argc - 1;

Process all the files
Now we scan the remaining arguments and try to open a file, if possible. The file
is processed and its statistics are given. We use a do ... while loop because
we should read from the standard input if no file name is given.
argc--;
do {
    @{If a file is given, try to open *(++argv ); continue if unsuccessful}
    @{Initialize pointers and counters}
    @{Scan file}
    @{Write statistics for file}
    @{Close file}
    @{Update grand totals}
} while (--argc > 0);

Here’s the code to open the file. A special trick allows us to handle input from stdin
when no name is given. Recall that the file descriptor to stdin is 0; that’s what we
use as the default initial value.
Variables local to main +=
int fd = 0;

Preprocessor definitions +=
#define READ_ONLY 0

If a file is given, try to open *(++argv ); continue if unsuccessful
if (file_count > 0 && (fd = open(*(++argv), READ_ONLY)) < 0) {
    fprintf(stderr, "%s: cannot open file %s\n", prog_name, *argv);
    status |= 2;
    file_count--;
    continue;
}

Close file
close(fd);

Variables local to main +=
We will do some homemade buffering in order to speed things up: Characters will be
read into the buffer array before we process them. To do this we set up appropriate
pointers and counters.
char buffer[BUFSIZ];    /* we read the input into this array */
register char *ptr;    /* the first unprocessed character in buffer */
register char *buf_end;    /* the first unused position in buffer */
register int c;    /* current character or number of characters just read */
int in_word;    /* are we within a word? */
long word_count, line_count, char_count;
    /* number of words, lines, and characters found in the file so far */

Initialize pointers and counters
ptr = buf_end = buffer;
line_count = word_count = char_count = 0;
in_word = 0;

Global variables +=
The grand totals must be initialized to zero at the beginning of the program.
If we made these variables local to main, we would have to do this initialization explicitly;
however, C’s globals are automatically zeroed. (Or rather, "statically zeroed.") (Get it?)
long tot_word_count, tot_line_count, tot_char_count;
    /* total number of words, lines and chars */

Scan file
The present section, which does the counting that is wc's raison d'être, was actually
one of the simplest to write. We look at each character and change state if it begins or ends a word.
while (1) {
    @{Fill buffer if it is empty; break at end of file}
    c = *ptr++;
    if (c > ' ' && c < 177) {    /* visible ASCII codes */
        if (!in_word) {
            word_count++;
            in_word = 1;
        }
        continue;
    }
    if (c == '\n') line_count++;
    else if (c != ' ' && c != '\t') continue;
    in_word = 0;    /* c is newline, space, or tab */
}

Fill buffer if it is empty; break at end of file
Buffered I/O allows us to count the number of characters almost for free.
if (ptr >= buf_end) {
    ptr = buffer;
    c = read(fd, ptr, BUFSIZ);
    if (c <= 0) break;
    char_count += c;
    buf_end = buffer + c;
}

Write statistics for file
It’s convenient to output the statistics by defining a new function wc_print();
then the same function can be used for the totals. Additionally we must decide here
if we know the name of the file we have processed or if it was just stdin.
if (!silent) {
    wc_print(which, char_count, word_count, line_count);
    if (file_count) printf(" %s\n", *argv);    /* not stdin */
    else printf("\n");    /* stdin */
}

Update grand totals
tot_line_count += line_count;
tot_word_count += word_count;
tot_char_count += char_count;

Print the grand totals if there were multiple files
We might as well improve a bit on UNIX’s wc by displaying the number of files too.
if (file_count > 1 || silent) {
    wc_print(which, tot_char_count, tot_word_count, tot_line_count);
    if (!file_count) printf("\n");
    else printf(" total in %d file%s\n", file_count, file_count > 1 ? "s" : "");
}

Functions
Here now is the function that prints the values according to the specified options.
The calling routine is supposed to supply a newline. If an invalid option character
is found we inform the user about proper usage of the command. Counts are printed
in 8-digit fields so that they will line up in columns.
void wc_print(char *which, long char_count, long word_count, long line_count)
{
    while (*which)
        switch (*which++) {
        case 'l': printf("%8ld", line_count);
            break;
        case 'w': printf("%8ld", word_count);
            break;
        case 'c': printf("%8ld", char_count);
            break;
        default:
            if ((status & 1) == 0) {
                fprintf(stderr, "\nUsage: %s [-lwc] [filename ...]\n", prog_name);
                status |= 1;
            }
        }
}

Wow! This is a very interesting concept! Currently, with a notebook, the output is shown below the block that reflects the code that printed that output, how would you track where it came from once you concatenate all blocks? Perhaps you can do something clever with comments? Perhaps looking at the runtime stack as well? I'd have to think of it a bit more. Is too far away from the direction you want HDNB to go? I feel like it is possible. :)

In the proposed case, you'd only run the top level call and the output would be shown under that. In general the output should be shown under the cell that was triggered to execute, even if the actual print cell happens in a different cell. This is consistent with how other notebooks I'm aware of operate.
HDNB core shouldn't know anything besides how to assemble cells into a code block, with the dispatch setting determining how to run those code blocks on the machine.

Understood. Thanks for the response!


which feature would be applied by signal? what's this signal for?

Greetings. What's this signals for? I've been watching this projects for weeks and  see lots of updated  are made. I wonder what's new feature would be affected by this signal?

This is is started as jotai-signal project. It's inspired by preactjs/signals-react, and had two goals (a) implementing less hacky with jsx-transform, and (b) utilizes jotai atoms as data sources.
It doesn't add any new use cases compared to normal jotai usage, it's just a different syntax.
This library is a spin-off, and provides the common feature for jotai-signal, valtio-signal and zustand-signal. So, the primary goal is (a) only.
Lately, we had a new idea about bypassing react rendering (which had been experimented in jotai-uncontrolled project), and it's implemented now. So, the benefit of this library is both (1) a different syntax that doesn't require react hooks (thus no need to follow hooks rules) and (2) occasional performance improvement (uncontrolled components don't work for all cases).

wow, I'm looking forward to these releases. Always appreciate with your OSS works !

note that what's described is something already released.


SCIM Ask password account does not get locked until user reset the password

When creating a user with the AskPassword option using SCIM, the user account is not getting locked until the user resets their password. However, it's expected to have a random password created from the application when creating the user with the AskPassword option, so that the user won't be able to login to the application. As an improvement, we can lock the user until the user resets their password.
Relared to #5651

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


K8SPG-465 add certs generation command

What makes me doubt about this pull request is the output of the following command, which contains substantially more tls secrets than we mention:
$ kubectl get secrets
NAME                            TYPE     DATA   AGE
cluster1-cluster-cert           Opaque   3      23m
cluster1-instance1-9rz9-certs   Opaque   6      23m
cluster1-instance1-fsfp-certs   Opaque   6      23m
cluster1-instance1-nxdd-certs   Opaque   6      23m
cluster1-pgbackrest             Opaque   5      23m
cluster1-pgbouncer              Opaque   6      23m
cluster1-pguser-cluster1        Opaque   12     23m
cluster1-replication-cert       Opaque   3      23m
pgo-root-cacert                 Opaque   2      23m

$ kubectl describe secret/cluster1-instance1-5gjm-certs
Name:         cluster1-instance1-5gjm-certs
....
Data
====
pgbackrest-server.crt:  952 bytes
pgbackrest-server.key:  227 bytes
dns.crt:                952 bytes
dns.key:                227 bytes
patroni.ca-roots:       583 bytes
patroni.crt-combined:   1179 bytes


(AUTOMATIC) opensource update

(AUTOMATIC) opensource update

Thank you for the update! Is there a plan to help fix the Travis CI build? The open-source community relies on that for making sure the code is correct.

It looks like it started failing a while back when INSTANTIATE_TEST_CASE_P was replaced by INSTANTIATE_TEST_SUITE_P so maybe a dependency to a newer version of googletest?


how to extract alignment file from poppunk

Hi John,
I would like to build ML tree with IQ tree, but it requires an align_variants.aln.
How can I export the alignment file from Poppunk output.
Thank you,
Apiwat

This should be produced by ska, which will take the assemblies as input. PopPUNK itself is alignment-free
If you run this pipeline with snakemake it should be automated - but please not this is still experimental. You will need to change at least the following lines in the snakefile:
https://github.com/johnlees/PopPIPE/blob/master/Snakefile#L87-L92
https://github.com/johnlees/PopPIPE/blob/master/Snakefile#L233-L242

Thank you John, I will go for the ska first.
Just curious, in case of running the whole pipeline, it requires poppunk database file in ".h5" format, but I don't see it from poppunk result. How to generate this file?
Thanks a lot,
Apiwat

Are you using PopPUNK v2 (with pp-sketchlib >=v1.3.0), which generates a .h5 rather than .msh files? You'll need to for this pipeline to work

Yes, I currently used PopPUNK v1. I will try version 2.
Thank you


Update sbt-scoverage to 1.9.3

Updates org.scoverage:sbt-scoverage from 1.9.2 to 1.9.3.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scoverage", artifactId = "sbt-scoverage" } ]

labels: sbt-plugin-update, early-semver-patch, semver-spec-patch, commit-count:1

Codecov Report

Merging #270 (1312b3e) into master (ef25dd6) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #270   +/-   ##
=======================================
  Coverage   96.19%   96.19%
=======================================
  Files           4        4
  Lines         105      105
  Branches        4        4
=======================================
  Hits          101      101
  Misses          4        4

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 ef25dd6...1312b3e. Read the comment docs.


Overriding the default fonts so that Ubuntu is never rendered

Hi, I'm most likely missing something but I was wondering how to override the default fonts so that I am not stuck with Ubuntu. I'm hoping to use Roboto.

Hi @muxica, thanks for sharing!
We made a design proposition in #80 about this use-case. Please have a look at it and feel free to comment with your feedback there.

@muxica actually, you can already use Roboto, using the font-family attribute see the doc. Based on that #80 will be useful for you only if you need non supported fonts.


Add https support for minreq_http of jsonrpc

This way, bitcoind-json-rpc-client can use jsonrpc with feature minreq_http for https connection.

The bitcoind-json-rpc-client is blocking by design and is explicitly provided for integration testing. Please see https://github.com/bitcoindevkit/bdk/issues/1589 for relevant discussion.


Added DOUBLE option

It was reading DOUBLES as DOUBLE PRECISION (which is not a valid DataType) so added in this case.

Merged, big thanks for your pull request.


Release 2.0.0

This is the release 2.0.0 PR, that merges multiple disparate branches on different forks, including thermal_update from jfahlen and speedups from pgbrodrick, along with other changes outline on the project board.
Release 2.0.0 is not fully backward compatible.  Config files have a slight reorganization, and old files will need to be updated.  Working configuration files can be found in the examples and test.
This is currently a work in progress, that as of now includes a merge between thermal_update and speedups, with some bug fixes, that works on a full VSWIR line.  Please attach closed issues to this PR, and we will release when the V2.0 Planned board is cleared.

@pgbrodrick, thanks for organizing this release!  I love the use of the project boards for tracking related issues and organizing the outstanding ones.  It took me all of ten seconds to see what features and issues will be included in the release as well as its current state. Very cool!


Remove stray CustomBuilder references

Codecov Report

Merging #467 into master will increase coverage by 0.07%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #467      +/-   ##
==========================================
+ Coverage   67.51%   67.59%   +0.07%
==========================================
  Files          74       74
  Lines        3408     3419      +11
==========================================
+ Hits         2301     2311      +10
- Misses        836      837       +1
  Partials      271      271

Impacted Files
Coverage Δ

pkg/reconciler/builder/builder.go
46.15% <ø> (ø)

pkg/reconciler/clusterbuilder/clusterbuilder.go
46.15% <ø> (ø)

pkg/cnb/builder_builder.go
84.31% <100.00%> (ø)

pkg/reconciler/build/build.go
74.38% <0.00%> (+1.65%)
:arrow_up:

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 eaf78e9...4e53b48. Read the comment docs.


Update Credential Issuance Guide

This improves the documentation and adds JS & KT tests for the tbDEX Credential Issuance guide.
issuance of verifiable credentials. The changes include updating package dependencies, adding new documentation, and implementing new tests.
Package dependencies:

Added express as a new dependency
Added ktor-server-test-host as a new kotlin dependency

Documentation:

site/docs/tbdex/issuer/vc-issuance.mdx: Updated instructions for environment setup and API service creation, and adding new code snippets for credential issuance.

Testing:

site/testsuites/testsuite-javascript/__tests__/tbdex/issuer/credentialIssuance.test.js: Added a new test file for credential issuance. This file tests the functionality of JWT verification, credential creation, and credential signing.
site/testsuites/testsuite-kotlin/src/tests/kotlin/docs/tbdex/issuer/credentialIssuanceTest.kt: Added a new test file for credential issuance. This file tests the functionality of JWT verification, credential creation, and credential signing.

To see the specific tasks where the Asana app for GitHub is being used, see below:

https://app.asana.com/0/0/1206393561850035

build is failing and has merge conflicts.
be sure to run pnpm build before running your tests. this error occurs locally, so you can catch it before committing.
Module not found: Error: Can"t resolve<EMAIL_ADDRESS>in "/opt/build/repo/site/docs/tbdex/issuer"
[ERROR] Client bundle compiled with errors therefore further build is impossible.

and then pnpm test exposes this:
 FAIL  site/testsuites/testsuite-javascript/__tests__/tbdex/issuer/credentialIssuance.test.js > Direct method functionality tests > .sign() method signs the credential and includes expected fields
TypeError: Cannot read properties of undefined (reading 'verificationMethodKeys')

I'm pushing some fixes

@angiejones thanks I'll do that going forward. I noticed the build fail when I pushed I was going to create an issuerDid in the test file to resolve it.
But I noticed with your changes .sign() and .create() methods are no longer in the guide for Kotlin, did you want to show the entire code block like JS instead of breaking them up?

@angiejones thanks I'll do that going forward. I noticed the build fail when I pushed I was going to create an issuerDid in the test file to resolve it.
But I noticed with your changes .sign() and .create() methods are no longer in the guide for Kotlin, did you want to show the entire code block like JS instead of breaking them up?

yes @EbonyLouis, the original had everything but sign() in one block. The problem with your approach is the snippets were broken up and therefore weren't compiling. For example, the api.get() wasn't complete with a closing brace in its snippet file because another snippet was basically embedded in it, cutting it off. And the other snippets had return statements but were no longer apart of the api.get() function in their respective snippet files. So, it all got really messed up and I couldn't even get it to build.


Single Matrix 128x64 with FM6126A

Hi There,
Anyone has succeed to use a 128x64 single panel with FM6126A?
Display is correct but missing part, or superposed part, can't have full area.
As one single was not working, then tried to set it up with chained mode (virtual)

2 x 64x64 2 rows 1 col
2 x 64x64 1 row 2 cols
4 x 64x32 2 rows 2 cols

No success at all, of someone got the correct config, I would give a try (I'm using latest dev branch)
Thanks

Refer to this example, see if this works: https://github.com/mrfaptastic/ESP32-HUB75-MatrixPanel-I2S-DMA/tree/dev/examples/3_FM6126Panel

ah ah ah that did the trick as expected, not sure why it does not works with my code since I used the same init.
indeed I love the new init stuff which allow just to setup the overwritten pins.
// It's my config to use with electrodragon I2S DMA
// https://www.electrodragon.com/product/rgb-matrix-panel-drive-interface-board-for-esp32-dma/
#if defined (ARDUINO_ESP32_PICO)
#define CH_D   22
#define CH_E   9
#define CLK    21
#elif defined (ARDUINO_ESP32_DEV)
#define CH_D   17
#define CH_E   18
#define CLK    16
#endif

  HUB75_I2S_CFG mxconfig( PANEL_RES_X,  PANEL_RES_Y, PANEL_CHAIN  );

#if defined (CH_D) && defined (CH_E) && defined (CLK)
  // override default pin mappings
  mxconfig.gpio.d = CH_D;
  mxconfig.gpio.e = CH_E;
  mxconfig.gpio.clk = CLK;
#endif

  // in case that we use panels based on FM6126A chip, we can set it here before creating MatrixPanel_I2S_DMA object
  mxconfig.driver = HUB75_I2S_CFG::FM6126A;

  // OK, now we can create our matrix object
  dma_display = new MatrixPanel_I2S_DMA(mxconfig);

Glad it works :-)
Can you post a link to the panel you bought?

Sure, but take care looks very fragile got one with an issue and looks the price raised, now 2 panels cost $61.4 and I paid 2 for $51.4 same seller one month ago.
https://www.aliexpress.com/item/1005001623824711.html

And here a picture of the back

Hmmm, interesting. So am I right that it is just two 64x64 chained in one case?

Hmmm, interesting. So am I right that it is just two 64x64 chained in one case?

My guess is yes. They are generally speaking highly predictable. Outdoor and high density panels generally have the fun stuff in them. This is kind of mid range density panel. There are a few problems that pop up around them.
These modules have a few variations in terms of multiplex ratio, LED drivers, row scan topologies, etc. The panels target applications like outdoor (low multiplex for high brightness), high density, etc. Generally speaking there will be two RGB data chains. GCLK could be OE for memory mapped drivers. A, B, C, D, E can mean decoder, direct, shift register, or hybrid. Sometimes the multiplexing with full panel while on others it per data chain. Outdoor modules will sometimes strip the panels to in a few configurations to increase brightness and refresh traits.


Bugfixing2

Tasks to be done:

[x] https://github.com/openpoke/decidim-module-reporting-proposals/issues/38
[x] https://github.com/openpoke/decidim/issues/49
[x] https://github.com/openpoke/decidim-module-reporting-proposals/issues/47

closed by testing


nixos/steam: fix CJK fonts on client, fix gamescope

Description of changes

Add a new option programs.steam.extraPackages to add to the Steam environment.
Add config.fonts.packages to extraPackages to fix all Chinese and Japanese text in the Steam interface display as boxes ("tofu"). As we use the existing fonts on the system, this avoids closure impact. It was first came up by #302591 that reverted by #302748
Add dependencies required by gamescope, to fix undefined symbols in X11 session. Came up by @BillHuang2001 in https://github.com/NixOS/nixpkgs/issues/162562#issuecomment-1229444338 .

fix #214275

Format nixos/modules/programs/steam.nix by nixfmt-rfc-style
Add a new option programs.steam.fontPackages, just let user to choose whatever font they like, to end argue in #302862

close #302862
fix #178121

This PR has tested on my machine.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

CC:

Steam maintainer team: @Atemu @eclairevoyant @jonringer @K900 @mkg20001
#302591 author: @eclairevoyant
@BillHuang2001

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

Please just add those two packages unconditionally into the standard Steam fhsenv, and please check if fonts.fontconfig.cache32Bit without any of the changes here fixes your Steam fonts.

On my machine, fonts.fontconfig.cache32Bit not works for me.
fontconfig:
    fontconfig = {
      defaultFonts = {
        emoji = [
          "Source Han Serif SC"
          "JetBrainsMono Nerd Font"
          "Noto Color Emoji"
        ];
        monospace = [
          "Source Han Serif SC"
          "Sarasa Mono SC"
          "JetBrainsMono Nerd Font Mono"
        ];
        sansSerif = [ "Source Han Serif SC" ];
        serif = [ "Source Han Serif SC" ];
      };
      cache32Bit = true;
    };

Is there an easy way to test this locally? I don't see anything wrong with that screenshot.

Is there an easy way to test this locally? I don't see anything wrong with that screenshot.

I set Source Han Serif SC. All fonts should be serif whatever the application actually asked. For example, all CJK in kitty is serif. But steam is still sans. To proof that steam is not following font-config.

And adding the font to the fhsenv actually makes it work? Can you run fc-match in the fhsenv?

For fonts, it's very likely we just need to enable fonts.fontconfig.cache32Bit and Steam will work automagically with system fonts.

This doesn't work for me.

Feel free to check https://store.steampowered.com/app/1647250/KUUKIYOMI_3_Consider_It_More_and_More__Father_to_Son/ as an example.

In case of providing font packages to steam and setting as fontconfig:

And in case of not providing, steam is still Tofu block.

And adding the font to the fhsenv actually makes it work? Can you run fc-match in the fhsenv?

Adding the fonts to the fhsenv just make steam to pick up a lucky font , still not following font-config. It was discussed in https://github.com/NixOS/nixpkgs/pull/302862#issuecomment-2046308403 . @CoelacanthusHex reports that steam follows font-config on her Arch, that's wired on NixOS......

log.txt
The log of the case of not providing font packages to the fhsenv, and font-config like:

    fontconfig = {
      defaultFonts = {
        emoji = [
          "Source Han Serif SC"
          "JetBrainsMono Nerd Font"
          "Noto Color Emoji"
        ];
        monospace = [
          "Source Han Serif SC"
          "Sarasa Mono SC"
          "JetBrainsMono Nerd Font Mono"
        ];
        sansSerif = [ "Source Han Serif SC" ];
        serif = [ "Source Han Serif SC" ];
      };
      cache32Bit = true;
    };

That is still wrong. Please add the packages directly to the fhsenv.

I've submitted the correct version of the gamescope change here: https://github.com/NixOS/nixpkgs/pull/313697
Please test that and we can reduce the scope of this PR to fixing the font setup.

Please test that and we can reduce the scope of this PR to fixing the font setup.

Though, the option programs.steam.extraPackages is needed insetad of override package, in case of user would like to do other things to steam.

Latest progress:
Case: not providing font packages to steam FHS
NixOS config:

  fonts = {
    packages = with pkgs; [
      sarasa-gothic
      (nerdfonts.override { fonts = [ "JetBrainsMono" ]; })
      source-han-serif
      source-han-sans
      noto-fonts-emoji
      shanggu-fonts
    ];

    fontconfig = {
      defaultFonts = {
        emoji = [
          "JetBrainsMono Nerd Font"
          "Noto Color Emoji"
        ];
        monospace = [
          "Sarasa Mono SC"
          "JetBrainsMono Nerd Font Mono"
        ];
        sansSerif = [ "Sarasa Gothic SC" ];
        serif = [ "Source Han Serif SC" ];
      };
      cache32Bit = true;
    };
  };

Extra fontconfig:
  fonts.fontconfig.localConf = ''
<?xml version="1.0"?>
<!DOCTYPE fontconfig SYSTEM "urn:fontconfig:fonts.dtd">
<fontconfig>
  <description>Load local customization file</description>
	<!-- Load local system customization file -->
	<match>
	    <test name="prgname" compare="contains" ignore-blanks="true">
		    <string>steam</string>
    	</test>
		<test name="family" compare="eq">
			<string>Arial</string>
		</test>
		<edit binding="same" mode="prepend" name="family">
			<string>sans-serif</string>
		</edit>
 	</match>
</fontconfig>

strace -f steam :
[pid 11930] access("/etc/fonts/local.conf", R_OK) = -1 ENOENT (No such file or directory)

fully strace has a so big size 686.0 MB, grep font upload here:
grep.txt

Why couldn't steam open the existing file /etc/fonts/local.conf ?

Does that file actually exist outside the sandbox? Can you reach it from steam-run?

Add font packages to steam FHSenv is just boken all font-config, force it to use only god knows which lucky font randomly. It's not the right dirction.

Does that file actually exist outside the sandbox?

Yes.
I also managed to launch a kitty inside steam, file is existing, font is correct:

Can you reach it from steam-run?

Yes

Steam do access the true font file
cat strace.txt |grep sarasa

[pid 11553] statx(AT_FDCWD, "/nix/store/xw74k7lqbbn993qpsx7xxd7hwhigcg53-sarasa-gothic-1.0.12", AT_STATX_SYNC_AS_STAT|AT_NO_AUTOMOUNT, STATX_BASIC_STATS, {stx_mask=STATX_ALL|STATX_MNT_ID, stx_attributes=0, stx_mode=S_IFDIR|0555, stx_size=10, ...}) = 0
[pid 11553] openat(AT_FDCWD, "/nix/store/xw74k7lqbbn993qpsx7xxd7hwhigcg53-sarasa-gothic-1.0.12", O_RDONLY|O_LARGEFILE|O_CLOEXEC) = 27
[pid 11553] openat(AT_FDCWD, "/nix/store/xw74k7lqbbn993qpsx7xxd7hwhigcg53-sarasa-gothic-1.0.12/.uuid", O_RDONLY|O_LARGEFILE|O_CLOEXEC) = -1 ENOENT (No such file or directory)
[pid 11553] openat(AT_FDCWD, "/nix/store/xw74k7lqbbn993qpsx7xxd7hwhigcg53-sarasa-gothic-1.0.12/.uuid", O_RDONLY|O_LARGEFILE|O_CLOEXEC) = -1 ENOENT (No such file or directory)
[pid 11553] statx(AT_FDCWD, "/nix/store/xw74k7lqbbn993qpsx7xxd7hwhigcg53-sarasa-gothic-1.0.12/share", AT_STATX_SYNC_AS_STAT|AT_NO_AUTOMOUNT, STATX_BASIC_STATS,  <unfinished ...>
[pid 11553] openat(AT_FDCWD, "/nix/store/xw74k7lqbbn993qpsx7xxd7hwhigcg53-sarasa-gothic-1.0.12/share", O_RDONLY|O_LARGEFILE|O_CLOEXEC <unfinished ...>
[pid 11553] openat(AT_FDCWD, "/nix/store/xw74k7lqbbn993qpsx7xxd7hwhigcg53-sarasa-gothic-1.0.12/share/.uuid", O_RDONLY|O_LARGEFILE|O_CLOEXEC <unfinished ...>
[pid 11553] openat(AT_FDCWD, "/nix/store/xw74k7lqbbn993qpsx7xxd7hwhigcg53-sarasa-gothic-1.0.12/share/.uuid", O_RDONLY|O_LARGEFILE|O_CLOEXEC <unfinished ...>
[pid 11553] statx(AT_FDCWD, "/nix/store/xw74k7lqbbn993qpsx7xxd7hwhigcg53-sarasa-gothic-1.0.12/share/fonts", AT_STATX_SYNC_AS_STAT|AT_NO_AUTOMOUNT, STATX_BASIC_STATS, {stx_mask=STATX_ALL|STATX_MNT_ID, stx_attributes=0, stx_mode=S_IFDIR|0555, stx_size=16, ...}) = 0
[pid 11553] openat(AT_FDCWD, "/nix/store/xw74k7lqbbn993qpsx7xxd7hwhigcg53-sarasa-gothic-1.0.12/share/fonts", O_RDONLY|O_LARGEFILE|O_CLOEXEC <unfinished ...>
[pid 11553] openat(AT_FDCWD, "/nix/store/xw74k7lqbbn993qpsx7xxd7hwhigcg53-sarasa-gothic-1.0.12/share/fonts/.uuid", O_RDONLY|O_LARGEFILE|O_CLOEXEC <unfinished ...>
[pid 11553] openat(AT_FDCWD, "/nix/store/xw74k7lqbbn993qpsx7xxd7hwhigcg53-sarasa-gothic-1.0.12/share/fonts/.uuid", O_RDONLY|O_LARGEFILE|O_CLOEXEC <unfinished ...>
[pid 11553] statx(AT_FDCWD, "/nix/store/xw74k7lqbbn993qpsx7xxd7hwhigcg53-sarasa-gothic-1.0.12/share/fonts/truetype", AT_STATX_SYNC_AS_STAT|AT_NO_AUTOMOUNT, STATX_BASIC_STATS, strace: Process 11665 attached
[pid 11553] openat(AT_FDCWD, "/nix/store/xw74k7lqbbn993qpsx7xxd7hwhigcg53-sarasa-gothic-1.0.12/share/fonts/truetype", O_RDONLY|O_LARGEFILE|O_CLOEXEC <unfinished ...>
[pid 11553] openat(AT_FDCWD, "/nix/store/xw74k7lqbbn993qpsx7xxd7hwhigcg53-sarasa-gothic-1.0.12/share/fonts/truetype/.uuid", O_RDONLY|O_LARGEFILE|O_CLOEXEC) = -1 ENOENT (No such file or directory)
[pid 11553] openat(AT_FDCWD, "/nix/store/xw74k7lqbbn993qpsx7xxd7hwhigcg53-sarasa-gothic-1.0.12/share/fonts/truetype/.uuid", O_RDONLY|O_LARGEFILE|O_CLOEXEC <unfinished ...>
[pid 11553] access("/nix/store/xw74k7lqbbn993qpsx7xxd7hwhigcg53-sarasa-gothic-1.0.12/share/fonts/truetype/Sarasa-Regular.ttc", F_OK <unfinished ...>
[pid 11553] openat(AT_FDCWD, "/nix/store/xw74k7lqbbn993qpsx7xxd7hwhigcg53-sarasa-gothic-1.0.12/share/fonts/truetype/Sarasa-Regular.ttc", O_RDONLY|O_LARGEFILE <unfinished ...>
[pid 11553] access("/nix/store/xw74k7lqbbn993qpsx7xxd7hwhigcg53-sarasa-gothic-1.0.12/share/fonts/truetype/Sarasa-Bold.ttc", F_OK <unfinished ...>
[pid 11553] openat(AT_FDCWD, "/nix/store/xw74k7lqbbn993qpsx7xxd7hwhigcg53-sarasa-gothic-1.0.12/share/fonts/truetype/Sarasa-Bold.ttc", O_RDONLY|O_LARGEFILE <unfinished ...>
[pid 11553] access("/nix/store/xw74k7lqbbn993qpsx7xxd7hwhigcg53-sarasa-gothic-1.0.12/share/fonts/truetype/Sarasa-SemiBold.ttc", F_OK <unfinished ...>
[pid 11553] openat(AT_FDCWD, "/nix/store/xw74k7lqbbn993qpsx7xxd7hwhigcg53-sarasa-gothic-1.0.12/share/fonts/truetype/Sarasa-SemiBold.ttc", O_RDONLY|O_LARGEFILE <unfinished ...>

As a temporary mitigation measure, we can require users to pass in only one font package, forcing Steam to use and only use this font package. Passing in more than two packets will inevitably cause greater confusion.

The fonts issue is unrelated to this change. Does this resolve your gamescope issue?

The fonts issue is unrelated

The fonts issue is related with this PR.

Sorry, wrong PR. Can you maybe get this down to a minimal version?

All clear:
Reason
Accroding to https://github.com/ValveSoftware/steam-for-linux/issues/10422

The steamwebhelper (Steam UI) in the beta client is using a vaguely Flatpak-like container (the same Steam Linux Runtime containers used by Proton, Dota 2, CS2) to get a consistent library stack and runtime environment across all Linux distros

It isn't currently possible to override files in /etc in the Steam Linux Runtime container

That's why fonts.fontconfig not work —— we all use nixosModules.

The container that is now used to run the steamwebhelper is meant to have behaviour similar to Flatpak: it will find fonts in /usr/share/fonts and ~/.local/share/fonts automatically, but those fonts will only be used if an application asks for them by name.

That's why providing font packages to FHSenv works, it links all font files to the fake path /usr/share/fonts
Solution

Providing only one font package to FHSenv, that would work, but may not satisfied all users, especially East Asian users who would like to use multi fonts for cmn, jp, even emoji.
Linking all fonts.packages into FHSenv, by default to at least barely works out of the box. Document it somewhere to teach user how to write the fontconfig in their own home directory, may could linkage with home-manager. But it's hard to warn CJK users precisely, without bothering the latin world.
Pass font-config by environment variable, but causes steam boot loop, issue open upstream https://github.com/ValveSoftware/steam-for-linux/issues/10915

CC @eclairevoyant What's your point.

Can you drop the reformat change? There's likely going to be a treewide format at some point, and it's going to cause merge conflicts on other changes.

All right, let's merge it and we can iterate later. Thanks!

Actually the current version doesn't pass EditorConfig. Please fix that.

Re-tested on my computer, all looks good

Actually the current version doesn't pass EditorConfig. Please fix that.

Apologize, fixed

This caused a build failure for me:
A definition for option `programs.steam.fontPackages."[definition 1-entry 24]"' is not of type `package'. Definition values:
       - In `/nix/store/mw4dx9syd793x0s5z4rq88mrh2i65vxm-source/nixos/modules/programs/steam.nix': "/nix/store/r4qnpfh6s2s6d0kw938syll5pp3gy76c-ghostscript-with-X-10.02.1/share/ghostscript/fonts

probably caused by my config:
fonts = {
    enableGhostscriptFonts = true;
}

Which adds
config = mkIf config.fonts.enableGhostscriptFonts {
    fonts.packages = [ "${pkgs.ghostscript}/share/ghostscript/fonts" ];
  };

source

@gador Could you please open a new issue and mention me by @Cryolitia , this thread has too many participants.


Getting fields of report, before actual rendering

Hi!
How can i get read all variables
referenced in my document before creating context and render a document?
For example, i had next template:
"Hello {name}!
{for tag in tags:
{tag.Name}
}"
How can i get list of: name, tags(Name)?
I always get dynamic report from frontend. And for creating context with data for it i must build dynamic sql query. But i cant do this without list of fields in report.

For now, i see fields in non-public members, but can't access them in code.

Hello @veantm, I'm not sure I get your question exactly, but I think this is what is described under section "Spying values" of the documentation. You'll need to render your document at least once with a blank context (just pass Context.Blank) and you'll get the list of global variables used in your document.
Note you won't get any dynamic field from these values, as they would require rendering the document with some actual context (in your example the iteration over tags would be no-op), but it may be OK given your description.

Yes, is work's. Thank you.

Hello @veantm, I'm not sure I get your question exactly, but I think this is what is described under section "Spying values" of the documentation. You'll need to render your document at least once with a blank context (just pass Context.Blank) and you'll get the list of global variables used in your document.
Note you won't get any dynamic field from these values, as they would require rendering the document with some actual context (in your example the iteration over tags would be no-op), but it may be OK given your description.


import-dbacsv and import-dbacsvtosql

both error out with WARNING: [19:17:47][Import-DbaCsv] Error occurred while establishing connection to  | The connection was not closed. The connection's current state is open.
This is when using azure sql database.  I can do normal dba write operations.

as a work around you can use the following to import csvs
  Write-DbaDataTable -SqlInstance $sqlinstance -SqlCredential $SqlCredential -Database $database -Table $table -InputObject $(import-csv -Path C:\temp\waka.csv) -AutoCreateTable

Thank you for your report. We'll take a look when we get a moment.

hello, please provide more information - what command did you type, how large is the CSV? does it happen immediately? anything else you can think that would be helpful for me to repro. thanks

what version of dbatools are you on? please let me know if you had issues seeing the bug report template and if so, what process did you follow to open the issue? we will likely auto-close issues that don't follow it in the future because it asks for information that is very helpful to us.

End Fix was to use convertto-dbadatatools, as far as version I update it every day with whatever is the latest.  Write-DbaDataTable -SqlInstance $SQLDatabase -SqlCredential $SqlCredential -Database $database -Table $table -InputObject (ConvertTo-DbaDataTable -InputObject $file) -AutoCreateTable -Truncate -Confirm:$false


Update README.md

Removed .default from the SideMenuManager.default as it's no longer available.

@hemangshah I appreciate the gesture, but this is a new change to the repo, not an outdated one. Run pod update to see the new code.


Test failure on OpenBSD - tests/test-pe.c:390: rule does not match contents

Bug

Build of YARA v4.5.1 on OpenBSD current/amd64 (future version 7.6) OK
Error when running test with make check

tests/test-pe.c:390: rule does not match contents of'tests/data/079a472d22290a94ebb212aa8015cdc8dd28a968c6b4d3b88acdd58ce2d3b885' (but should)

Expected behavior
All tests should pass.
Please complete the following information:

OS: OpenBSD current/amd64 (OpenBSD 7.5-current (GENERIC.MP) #93)
YARA version: 4.5.1 built from sources
LibreSSL 3.9.0

This looks like a difference between openssl and libressl. I don't have an installation with libressl around, could you try to comment out lines in the test below until you get a more minimalistic test that reproduces the issue?
https://github.com/VirusTotal/yara/blob/8616165472424d9949c434a5da26858e7880affb/tests/test-pe.c#L294-L388

After a lot of iterations (modify test-pe in tests/test-pe.c, rebuild and test with make check), I have 2 cases of minimalist test that reproduces the issue:

pe.signatures[0].signer_info.length_of_chain == 2
pe.signatures[0].countersignatures[0].length_of_chain == 2

After some searchs in issues, mine seems to be a duplicate of #2046.

I would like to try to build Yara on OpenBSD using OpenSSL lib instead of LibreSSL.

OpenSSL version 3.1.6 installed via openssl OpenBSD packages
includes in /usr/local/include/eopenssl31/openssl/
libs in /usr/local/lib/eopenssl31/

I don't find in configure script how to use OpenSSL instead of LibreSSL. Is there an options/flag in configure script to do this ?
I checked configure.ac file for AC_CHECK_HEADERS / AC_CHECK_LIB for openssl/crypto but I don't find how to modify theses checks.

FYI, I succeeded to compile and test Yara with OpenSSL instead of LibreSSL on OpenBSD (amd64).

Install of OpenSSL version 3.1.6 via pkg_add openssl-3.1.6v0

$ /usr/local/bin/eopenssl31 version
OpenSSL 3.1.6 4 Jun 2024 (Library: OpenSSL 3.1.6 4 Jun 2024)

Build of Yara with OpenSSL

$ ./configure --enable-cuckoo --enable-magic --enable-dex --enable-macho --with-crypto CPPFLAGS=-I/usr/local/include/eopenssl31 LDFLAGS=-L/usr/local/lib/eopenssl31
$ make
(...)
$ LD_LIBRARY_PATH=/usr/local/lib/eopenssl31/ ./yara -v
4.5.1

Tests of Yara => no error for test-pe

$ LD_LIBRARY_PATH=/usr/local/lib/eopenssl31/ make check
(...)
make  check-TESTS
PASS: test-arena
PASS: test-alignment
PASS: test-atoms
PASS: test-api
PASS: test-rules
PASS: test-pe
PASS: test-elf
PASS: test-version
PASS: test-bitmask
PASS: test-math
PASS: test-stack
PASS: test-re-split
PASS: test-async
PASS: test-string
PASS: test-exception
PASS: test-macho
PASS: test-dex
PASS: test-dotnet
PASS: test-magic
make  all-am
============================================================================
Testsuite summary for yara 4.5.1
============================================================================
# TOTAL: 19
# PASS:  19
# SKIP:  0
# XFAIL: 0
# FAIL:  0
# XPASS: 0
# ERROR: 0
============================================================================


Problem with client using streamer

I'm having a problem with a client failing on reading certain bytes. I don't know much about the problem, but it happens with a python script which worked perfectly before.
I'm trying it on WSL and the streamer starts and generates the files containing the tweets propertly for a while, but after a couple of hours working it stops and displays the following error:
Traceback (most recent call last):
  File "/usr/lib/python3.6/http/client.py", line 546, in _get_chunk_left
    chunk_left = self._read_next_chunk_size()
  File "/usr/lib/python3.6/http/client.py", line 513, in _read_next_chunk_size
    return int(line, 16)
ValueError: invalid literal for int() with base 16: b''

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib/python3.6/http/client.py", line 578, in _readinto_chunked
    chunk_left = self._get_chunk_left()
  File "/usr/lib/python3.6/http/client.py", line 548, in _get_chunk_left
    raise IncompleteRead(b'')
http.client.IncompleteRead: IncompleteRead(0 bytes read)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/luisc/.local/lib/python3.6/site-packages/urllib3/response.py", line 397, in _error_catcher
    yield
  File "/home/luisc/.local/lib/python3.6/site-packages/urllib3/response.py", line 479, in read
    data = self._fp.read(amt)
  File "/usr/lib/python3.6/http/client.py", line 449, in read
    n = self.readinto(b)
  File "/usr/lib/python3.6/http/client.py", line 483, in readinto
    return self._readinto_chunked(b)
  File "/usr/lib/python3.6/http/client.py", line 594, in _readinto_chunked
    raise IncompleteRead(bytes(b[0:total_bytes]))
http.client.IncompleteRead: IncompleteRead(0 bytes read)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "startStreaming.py", line 109, in <module>
    stream.filter(locations=locations, languages=languages)
  File "/home/luisc/.local/lib/python3.6/site-packages/tweepy/streaming.py", line 474, in filter
    self._start(is_async)
  File "/home/luisc/.local/lib/python3.6/site-packages/tweepy/streaming.py", line 389, in _start
    self._run()
  File "/home/luisc/.local/lib/python3.6/site-packages/tweepy/streaming.py", line 320, in _run
    six.reraise(*exc_info)
  File "/home/luisc/.local/lib/python3.6/site-packages/six.py", line 693, in reraise
    raise value
  File "/home/luisc/.local/lib/python3.6/site-packages/tweepy/streaming.py", line 289, in _run
    self._read_loop(resp)
  File "/home/luisc/.local/lib/python3.6/site-packages/tweepy/streaming.py", line 339, in _read_loop
    line = buf.read_line()
  File "/home/luisc/.local/lib/python3.6/site-packages/tweepy/streaming.py", line 200, in read_line
    self._buffer += self._stream.read(self._chunk_size)
  File "/home/luisc/.local/lib/python3.6/site-packages/urllib3/response.py", line 496, in read
    raise IncompleteRead(self._fp_bytes_read, self.length_remaining)
  File "/usr/lib/python3.6/contextlib.py", line 99, in __exit__
    self.gen.throw(type, value, traceback)
  File "/home/luisc/.local/lib/python3.6/site-packages/urllib3/response.py", line 415, in _error_catcher
    raise ProtocolError('Connection broken: %r' % e, e)
urllib3.exceptions.ProtocolError: ('Connection broken: IncompleteRead(0 bytes read)', IncompleteRead(0 bytes read))

Any idea what is happening? As I say the same script worked perfectly before and no changes has been made to it, but if it was necessary I can upload the script I'm using.

Same Error I face and the problem was I process thousand of tweets in a data structure which took a lot of memory. Due to this tweepy disconnect connection from Twitter, I just resolve it by just process tweets chunk by chunk, not whole tweets. Actually, it's a dynamic approach to solve algorithmic problems. Hope this will work for you.

Do you mean by retrieving thousand of tweets or by saving them? I ask tweepy for the full metadata although I only save some of the fields, is there a way to ask only for those specific fields in order to avoid that error?

is there a way to ask only for those specific fields in order to avoid that error?

No, but that isn't the direct cause of the issue, which is a duplicate of #237 (and #448).


Overzooming not working when adding multiple mbtiles

I have created Vector tiles with Tippecanoe in mbtiles format. I am using tileserver-gl for serving the Vector tiles. Firstly I am creating 20 mbtiles separately as different layers needed different Tippecanoe options. After that I join tiles with tile-join feature. In tilestats of mbtiles json file I am getting different minzoom and maxzoom for each layer. The problem is the layer became invisible after the maxzoom level, the overzooming is not working though I have assign minzoom and maxzoom in style json file. It's working if I create vectortiles with a single script by providing multiple json layer as the zoom is the same for each layers. But few of my layers need much precision level data so that I have to create maxzoom z15 for each layer. As a result mbtiles becoming too high in size and taking a lot of time to create vector tiles.
Moreover when creating vector tiles with multiple json a lot of data lost in zoom scale 10-11. Again point layers also behaving weirdly. Such as, a single point feature is showing 5-6 times (See the attached image below).
What will be the best options for creating vector tiles with tippecanoe. Am I in the right track for establishing a self-hosted vector tiles server with Tippecanoe and Tileserver-GL?

This is unfortunately a limitation of how tile compositing and rendering works, and can't be fixed in Tippecanoe itself. If any layers are defined for a tile, it will use only the layers that are explicitly part of that tile rather than going up to the parent tiles to overzoom data for layers that are not defined in the higher-zoom tile. (tile-join now gives a warning when joining tilesets that have different zoom ranges.)
If you use Mapbox tile serving and upload the mbtiles for each layer separately and then use server-side compositing of the layers, it will overzoom the data from tilesets that have a restricted zoom range when serving composite tiles at higher zooms. It may be possible to make your tile server do something similar but I don't know anything about Tileserver-GL and can't give advice about it.


fix coordinator bug

Purpose or design rationale of this PR
Describe your change. Make sure to answer these three questions: What does this PR do? Why does it do it? How does it do it?
PR title
Your PR title must follow conventional commits (as we are doing squash merge for each PR), so it must start with one of the following types:

[ ] build: Changes that affect the build system or external dependencies (example scopes: yarn, eslint, typescript)
[ ] ci: Changes to our CI configuration files and scripts (example scopes: vercel, github, cypress)
[ ] docs: Documentation-only changes
[ ] feat: A new feature
[ ] fix: A bug fix
[ ] perf: A code change that improves performance
[ ] refactor: A code change that doesn't fix a bug, or add a feature, or improves performance
[ ] style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc)
[ ] test: Adding missing tests or correcting existing tests

Deployment tag versioning
Has tag in common/version.go been updated or have you added bump-version label to this PR?

[ ] No, this PR doesn't involve a new deployment, git tag, docker image tag
[ ] Yes

Breaking change label
Does this PR have the breaking-change label?

[ ] No, this PR is not a breaking change
[ ] Yes

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 52.67%. Comparing base (07dc2c9) to head (d910571).
Report is 1 commits behind head on feat/upgrade4.

Additional details and impacted files
@@              Coverage Diff               @@
##           feat/upgrade4    #1455   +/-   ##
==============================================
  Coverage          52.67%   52.67%
==============================================
  Files                157      157
  Lines              12527    12527
==============================================
  Hits                6599     6599
  Misses              5367     5367
  Partials             561      561

Flag
Coverage Δ

coordinator
17.53% <ø> (ø)

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

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


IUserTwoFactorTokenProvider

We're currently implementing the "Traditional Prompt" and part of this is implementing IUserTwoFactorTokenProvider for Duo.
How do we migrate this to the new "universal prompt"? There seems to be no mention of this anywhere.
Many thanks.

Yep sure, thanks. I guess since this is a .NET repo that has a example app I was expecting it to use the standard .NET APIs for 2FA integration. Or if the standard APIs are not to be used with the new Universal Prompt then why?

@mausch The Duo prompt in general is not a "token" provider suitable for use with the built-in abstractions.  It is a web-based flow that involves redirecting the user out of the application and back.  While we do have a data object we refer to as a 'token' it is not the type of token that IUserTwoFactorTokenProvider deals with, as far as I can tell.

Thanks for the quick reply!
It seems the developer that implemented this in our code used the wrong APIs then :slightly_smiling_face:

Well, I can see how the old method (what we call "Web SDK 2") could be stretched to be considered a 'token', subject to special validation rules.  But I don't think the new scheme can unfortunately.


PlaceholderAPI

Hey,
When i using PlaceHolder API with my Scoreboard. Its shows me Error1
Screen: http://prntscr.com/nwtrrw
Logs are clean. I have PlaceholderAPI on my Spigot Server. But wich Expansions i need to run this ?

What version of PlaceholderAPI du you use? (/version PlaceholderAPI)
My plugin is build against PlacegholderAPI version 2.10.2, if your version is older, try updating it.
Placeholders should work with just PlaceholderAPI and my Plugin installed, my Plugin should register its own expansion (called onlinetime).
Also, which version of Spigot do you use?

I am using Spigot 1.12.2 and Placeholder 2.9.2. Its works now with 2.10.2 but its update to slow. i set with admin 500h, 501h and 502h
My Scoreboard updates always in 5 secs

Do you use Bungee? If so, increase the poll interval by changing save-interval inside the OnlineTime config.yml to a lower value.
If not, try retrieving the value with /papi parse me %onlinetime_string% (instead of watching the scoreboard) and see if it updates there. If so, your scoreboard isn't reading the value every time it updates, if not, this is a bug.

@DesperoKF Did this solve your issue or is it still present?


Some issue about model saving

When I use save function of policy: self.policy.save()，I encounter a issue as following:
"_generator_ctor() takes from 0 to 1 positional arguments but 2 were given."
However, when i use the saving function of policy without  'data':self_get_constructor_parameters(), It works fine.
Can I just ignore the 'data':self_get_constructor_parameters() when saving the model?

Hi @wslgqq277g . Thanks for using DexArt. I've add a simple callback function for saving the model while training. Please refer here for more information.


get tests to run locally + docs.

Get NJSW tests to run on my machine & document the process.

This is ready to merge.


Updated success message (curl and TP) for deleting a DS

Related: #4351

Which Traffic Control components are affected by this PR?

Traffic Ops
Traffic Portal

What is the best way to verify this PR?

Create a DS or region or server
Delete a DS (see updated message), region and server (have same message as before i.e. server/region was deleted. on TO and server/region deleted on TP.
Updated message for DS
curl -X DELETE -k -H "Accept: application/json" --cookie "mojolicious=<mojolicious cookie>" https://localhost:8443/api/4.0/deliveryservices/<ds id>

{"alerts":[{
       "text":"ds was deleted. Perform a CDN snapshot then queue updates on all servers in the cdn for the changes to take affect.",
       "level":"success"
}]

If this is a bugfix, which Traffic Control versions contained the bug?

PR submission checklist

[x] This PR has tests
[ ] This PR has documentation
[x] This PR has a CHANGELOG.md entry
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Should this new message maybe go in an info-level alert instead of appending it to the success message?


add progress bars for various building, compilation, MCMC running steps

Anything that might take more than 15-30 seconds or so might have a progress bar.
A first step would be to have a general function to help with this. There might be some code available on the web.

We agreed for now to have some more limited progress info -
building model
defining model
checking model (and note that check=FALSE will avoid this)
also make check() warnings into print statements and not formal warnings.

Done. The output looks like this now.

m=nimbleModel(code)
Defining model...
Building model...
Building node functions...
Setting data and initial values if supplied...
Checking model (set 'check = FALSE' in nimbleModel() to skip this step) ...
NAs were detected in model variables: mu, logProb_mu, y, logProb_y.
Modeling building finished.

Closing this issue but feel free to reopen if you don't like how this looks.

Actually, the issue was more general than model building, so I'll leave it open as we may want progress info for compiling and other steps.

moved to wiki


Sometimes VC libraries are not copied

https://github.com/YehudaKremer/msix/blob/5c574b5935adb5da66452971901a82ce618b6ee8/lib/src/msixFiles.dart#L213
Hi, it's first time to raise an issue on here.
I realized sometimes vcruntime140.dll and vcruntime140_1.dll is missing when I build MSIX package.
It looks like it starts "packing.." before it finishes copyVCLibsFiles()
I guess it's from the delay in async-foreach process.
ref: https://stackoverflow.com/questions/42467663/async-await-in-list-foreach
Could you check the parts, please?

Hello @innim98
thank you for this bug report
i switch to use normal for loop to resolve this bug: https://github.com/YehudaKremer/msix/commit/dfbabd5d54c9ba9dc9129da53f2de3b6332e7e06#diff-603f8a2333c376bb15387757439652de01f1d668623f1943622fb474c1c59bfe
this fix include in version 0.1.17

Hello @YehudaKremer Thank you for quick update!
I realized there is one more point.
listen
Sometimes, the callback for listen() method works later than processing VC libraries. As of result, vcruntime140_1.dll is missed. (1 out of 3 times in my environment)
Could you change it as well? Thank you.

Done:) now the files actions are Sync.
fix include in version 0.1.18: commit
@innim98 thank you for this important report,
please check if now all the files are copied properly

@YehudaKremer Thank you for the great job.
The files are packed normally.


{CDN} Deprecate CDN WAF related commands

Related command

az cdn waf
Description

Deprecate CDN WAF related commands
Testing Guide

az cdn waf policy custom-rule list -g afdx-bugbash --policy-name CDBWAFbug
History Notes

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

CDN

Hi @necusjz , could you help to merge the PR? Another question, will the doc update for https://docs.microsoft.com/en-us/cli/azure/cdn/waf?view=azure-cli-latest requires another release for CLI?

Another question, will the doc update for https://docs.microsoft.com/en-us/cli/azure/cdn/waf?view=azure-cli-latest requires another release for CLI?

Yes, it is.


nixos/tests/home-assistant: improve type handling

The testdriver is being typechecked since #171280 which broke the home-assistant test, because I simply assumed a regex pattern would never not match.
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

@ofborg test home-assistant


Add title to sort fields

For the Policy finder (https://www.gov.uk/government/policies) sorting
by public_timestamp is meaningless as content changes don't bubble up
to the Policy. Until we can do that, it makes more sense to sort the
Policies by title. This commit adds title to the ALLOWED_SORT_FIELDS.

I have some performance worries about this; sorting by an analyzed text field which isn't stored is going to require fetching and parsing the full stored "source" of every matching document.  We can configure elasticsearch to index the field better for allowing sorting.  It would be good to do some timings of a search for all documents with this sort enabled, compared to the default sort order; and I'd like to look into updating the title field's configuration before this is merged.

There are ~230 Policies and this isn't likely to grow anytime soon. I can see a case where certain Finders other than Policies would need this though (Contacts is another example) so we should definitely get this sorted.

Merging this for now, but please keep an eye on performance in preview, and we can tweak elasticsearch configuration if it turns out to be a problem.


add axis support to cwt

Users requested batched operation for cwt in #445. This can be done by adding an axis argument as in this PR. This PR allows the input data to be n-dimensional with batched operation over all axes aside from the specified cwt axis. For 1D data, the behavior is unchanged from before.
The final shape of the output for n-dimensional data becomes:
(len(scales),) + data.shape  (i.e. the scales dimension is always first as it was for the 1D case previously)
For the 'conv' case implementation is via a simple for loop, but for the 'fft' case we do not have to repeat the FFT of the wavelet filter for each item in the batch, so there is a performance benefit to batched operation.
a subset of benchmark results.
first, for non-batch cases
              -------------------------------------------- ---------------------
                n     wavelet   max_scale       dtype         conv       fft
              ====== ========= =========== =============== ========== ==========

               128      mexh        16      numpy.float32   2.01±0ms   5.13±0ms
               128      mexh        16      numpy.float64   4.32±0ms   5.99±0ms

               2048     shan       256      numpy.float32   397±0ms    68.0±0ms
               2048     shan       256      numpy.float64   864±0ms    100±0ms

a few batch (n_batch=5) cases
               128      mexh        16      numpy.float32   6.68±0ms   6.67±0ms
               128      mexh        16      numpy.float64   5.46±0ms   7.39±0ms

               2048     shan       256      numpy.float32   1.87±0s    178±0ms
               2048     shan       256      numpy.float64   4.32±0s    291±0ms

Summary for the 2048/shan/float32 case:
with conv: n_batch=5 batch takes 4.7 times longer than the non-batch case

with fft:  n_batch=5 batch takes 1.8 times longer than the non-batch case

for the non-batch case, mode 'fft' is 5.8 times faster than 'conv'

for the n_batch=5 batch case, mode 'fft' is 10.5 times faster than 'conv'

closes #445

I should note that the FFT results in the benchmarks above are for single-threaded scipy.fftpack as the fft backend. It is likely multi-threaded FFT backend would give additional benefit in batched operation.

Backwards-compatible, and seems like a nice usability improvement.
I mostly just reviewed the tests and description. axis=-1 is the right default. Going to merge this, looks good. Thanks @grlee77


Check if a ship is really available when loading a fleet

A ship might destroyed, dissolved, or used as a rebuilding material -- anyways, it may not be available at the time of loading.

IIUC this is already resolved in many places.


Latest version not available in Maven Central

We would like to use new features from the latest version, Unfortunately maven has 2.0.3
Please push the latest version to Maven Central

i've come up with the same problme


Unpin sqlalchemy once issue is fixed

Once the source issue is fixed:

pandas-dev/pandas#51015

we should revert the pin introduced in:

#5476

@albertvillanova It looks like that issue has been fixed so I made a PR to unpin sqlalchemy!

The source issue:

https://github.com/pandas-dev/pandas/issues/40686

has been fixed:

https://github.com/pandas-dev/pandas/pull/48576

The fix was released yesterday (2023-04-03) only in pandas-2.0.0:

https://github.com/pandas-dev/pandas/releases/tag/v2.0.0

but it will not be back-ported to pandas-1.
Also note that pandas-2.0.0 dropped support for Python 3.7:

https://github.com/pandas-dev/pandas/issues/41678
https://github.com/pandas-dev/pandas/pull/41989

Therefore, we cannot unpin sqlalchemy until we drop support for Python 3.7 (these Python users cannot use pandas-2).


feat: Use tracing-subscriber for ockam_app log output during development

Current behavior
tauri-plugin-log does not seem to support coloring or trivially runtime-configurable filtering, so we get a full firehose of internal details' log emissions while running the app. tauri-plugin-log's primary perceived advantage is that it lets us log to disk on end users' machines so that we could collect that info out of band when troubleshooting issues.
Proposed changes
In development workflows, use tracing-subscriber to provide the more familiar and runtime-configurable log stream. In release artifacts, make sure to enable the release crate feature which will fall back to tauri-plugin-log.
My suggested starting point for using the tracing features to zoom in on logging statements that are just within our ockam_app code:
env OCKAM_LOG=ockam_app=trace,ockam_node=warn,ockam_identity=error,info cargo tauri dev

Checks

[x] All commits in this Pull Request are signed and Verified by Github.
[x] All commits in this Pull Request follow the Ockam commit message convention.
[x] I accept the Ockam Community Code of Conduct.
[x] I have accepted the Ockam Contributor License Agreement by adding my Git/Github details in a row at the end of the CONTRIBUTORS.csv file in a separate pull request to the build-trust/ockam-contributors repository. The easiest way to do this is to edit the CONTRIBUTORS.csv file in the github web UI and create a PR, this will mark the commit as verified.

😌

😌


Should Instrument.del() and close() not exist?

I have been getting somewhat random python errors due to dereferencing deleted objects.  The answer seems to be not to have either __del__() or close() functions in the Instrument class (and thus not call usb.util.dispose_resources()) because Python deletes things in unpredictable orders.  Just calling close() causes the same problems even with __del__() removed.
See:  https://stackoverflow.com/questions/50574201/memory-corruption-on-exit-or-stupid-user-error

Is that actually the correct solution?  Just don't do any clean up at all and hope things will just work out?  Seems like the linked question says to use __enter__ and __exit__, but this then makes it extremely cumbersome to do much of anything as then the whole script that uses usbtmc has to be wrapped in a bunch of nested with statements and I believe this precludes any sort of GUI style operation where the connection is maintained in the background.

So, it looks like contextlib.ExitStack can alleviate some of the issues with opening multiple connections, but this still requires operating inside a with statement.
I also found https://stackoverflow.com/questions/36981177/python-with-statement-in-gui-pyqt-program , which seems to indicate that it may not be possible for python to do this sort of cleanup properly.

Hmm, I also found atexit, maybe that's a way to get some of this manual cleanup sorted out properly.

Looks like atexit might be ideal, with the one unfortunate problem being atexit does not support unregister in python 2, so some sort of callable weak reference is required for python 2 compatibility.  This could solve a number of issues, though, for more than just python-usbtmc.

The place where I had problems was when close called usb.util.dispose_resources.
I tried removing the __del__ function and calling close from main.  That produced similar errors when exit ran, it complained of trying to access things that had already been deleted.
It looks like pyusb goes to a lot of effort to support explicit freeing of resources, but as it notes, Python does not really provide for this.
I think one of the limitations of using a language that hides all of its memory management is that you have to trust it to handle things for you.  Instead of freeing an object you have to clear anything that refers to it and the runtime should then free it.

I also wonder why I seem to be the only one seeing this.  Am I doing something that different???

Also, would you mind trying something?
Add
import atexit

at the top, and
atexit.register(self.close)

at the bottom of open. Let me know if you're still seeing these cleanup issues with that.

Using atexit prevents errors in a simple test where I just open and then call exit.  If I access the device to read data then I still get warnings like this from down in pyUSB.
Exception ignored in: <bound method Device.del of <DEVICE ID 1ab1:04ce on Bus 003 Address 042>>
Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/usb/core.py", line 1029, in del
File "/usr/lib/python3/dist-packages/usb/core.py", line 225, in dispose
File "/usr/lib/python3/dist-packages/usb/core.py", line 220, in release_all_interfaces
TypeError: 'NoneType' object is not callable
Or like this:
Exception ignored in: <bound method _Device.del of <usb.backend.libusb1._Device object at 0x7fad44d4b390>>
Traceback (most recent call last):
File "/usr/lib/python3/dist-packages/usb/backend/libusb1.py", line 561, in del
AttributeError: 'NoneType' object has no attribute 'libusb_unref_device'


[MNT] Update wheels CI/CD workflow after dropping C extensions and Cython

Reference Issues/PRs

What does this implement/fix? Explain your changes.

Does your contribution introduce a new dependency? If yes, which one?

What should a reviewer concentrate their feedback on?

Any other comments?

PR checklist

For all contributions

[ ] I've added myself to the list of contributors.
[ ] Optionally, I've updated sktime's CODEOWNERS to receive notifications about future changes to these files.
[ ] I've added unit tests and made sure they pass locally.

For new estimators

[ ] I've added the estimator to the online documentation.
[ ] I've updated the existing example notebooks or provided a new one to showcase how my estimator works.

just so I understand correctly: with C gone, we no longer need cibuildwheel, which in turn allows us to have a single (platform agnostic) wheel only?

Yup :grinning:


Passing array as partial parameter

This doesn't seem to work when I try it but Handlebars doesn't give me any errors.
{{> myPartial test=["hello", "world"] }}

It doesn't seem to parse this as a string nor a array since iterating or echoing it doesn't seem to yield anything.

That is not support and should be a syntax error... Can you post a fiddle (template provided in the contributing.md doc in the repo) that shows this live?

Can't show this on jsfiddle since the {{> path/to/file.hbs }} uses an external file.

I missed this previously, this is using the segment literal notation, so it ends up evaluating to something like context["\"hello\", \"world\""], thus no syntax error.

@kpdecker Do you know why this is not supported, I mean sending arrays to a partial?

You can use a simple helper to achieve this. Its not part of the core to keep the core small.
(...args) => args

An example would be nice, let's say we want to call a partial with two parameters, one that is a string and one that is an array, something like this:
{{> myPartial test=["hello", "world"] id="abcd" }}
How would a helper do this?

I was on my cell, when I wrote my last comment, so I couldn't write a lot of text:
I've created a fiddle with the helper that I am using in some projects: https://jsfiddle.net/9D88g/811/
String-splitting is also possible, of course.

The EcmaScript 5 version of the helper is
// Array helper. Remove the last item, because that is the "options" object
Handlebars.registerHelper("arr", function () {
	return Array.prototype.slice.call(arguments, 0, -1)
})

@dhollenbeck Your solution breaks if array members have semicolon in them. I can not make such assumption over value put into the array.

This entire thread including your request for an example is regarding how to hardcode array data inside a template syntax. Since you are hardcoding the array data there is no assumptions regarding the array data. If your desire is pass an array into a partial then just pass the array into the partial.
var template = '{{> myPartial test=myArray id="abcd" }}';
var fnc = Handlebars.compile(html);
var context = {myArray: []};
var html = fnc(context);

@dhollenbeck Actually there is, otherwise I would't be asking this question. We are using handlebars in a build framework that is used by other developers. I have no idea what parameters they pass to their templates or what they put in their arrays.


Documentation: Create docs on how to establish and run a local BPD Chapter

In /how_to/create_a_local_bpd_chapter.md, we need to document the process of how community members can establish a local BPD Chapter. The created docs should outline the steps for starting a chapter, organizing meetings, and engaging with the local community.
Experienced chapter leaders or anyone who is willing to collaborate on our Discord to get the appropriate information for the docs can take on this task.

folks can reach out to me to learn more about this process

I'll work on this @activus-d

Thanks for volunteering to work on this @mihrab34. I have assigned the issue to you


modify year of license in pkg/ddc/alluxio/engine.go

Ⅰ. Describe what this PR does
Ⅱ. Does this pull request fix one issue?

fixes #XXXX
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

/lgtm


Support stdin and stdout in CLI

The compiler should support compiling code that does not reside in a file, and support outputting directly to stdout instead of to a file on disk.

:+1:


Add cz email templates

Before submitting this PR:
Checklist

[x] No breaking changes
[x] Tests pass

Breaking changes
Avoid breaking changes and regressions. If you feel it is unavoidable, make it explicit in your PR comment so we can review it and see how to handle it.
Tests

please make sure your changes pass the current tests (Use the make test or the make watch command).
if you are introducing a new feature, please write as much tests as possible.

Documentation
Please make sure the documentation is updated accordingly, in particular:

Workflows. Workflows are Mermaid sequence diagrams
Schema. The schema in a Mermaid ER diagram
Environment variables. Please adjust the .env.example file accordingly
OpenApi specifications. We are using inline JSDoc annotations

That's awesome!
Děkuji mnohokrát!


TOMEE-4164 JSON-B TCK Setup

Closing this one because it's merged into https://github.com/apache/tomee/pull/1005

@jeanouii Guess it is time to rebase? :)

Signature tests are passing.
Currently only one test fails:

ee.jakarta.tck.json.bind.cdi.customizedmapping.adapters.AdaptersCustomizationCDITest.testCDISupport

This should be good now (both modules pass). @jeanouii can you have a look?


Fix some issue with the client preview for the tag and task preview widgets

@JavaCafe01
Some other issues:

I think these widgets should be turned into proper objects, with methods like :show() :hide() and :toggle() instead of the current signals.
When there isn't an available client.content to preview, the tag preview displays a black background with the client icon in the cener. The task preview before this PR would always show the client.content, so there wasn't a need for that - but maybe we should at now do it here too?
The task preview height differes between clients. I think it has to be with the title not being limited in size?
What about using #129 to the get the icons for the running clients? It will retrive the icon for the running client from the currently system selected icon theme, or whatever theme was passed to the constructor

@Kasper24

I think this is a great idea. I believe Bling is overusing signals :smile:. I think we can wait for #99 to finish or add this to the list for that PR.

The client icon is already shown in the preview, so I think if there is nothing to show we just don't show an image (just the icon and title), or we can show the AwesomeWM logo as a fallback.

That, and maybe also because the client previews are different sizes?

Seems like a good idea! But for a different PR, not this one.


Add docker compose file with PostgreSQL included

Hi,
I've added a simple docker compose file, so that a PostgreSQL container will be started, which can be used by the application.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: FuriKuri sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


add pylint to catch unused args

fixes #2761.

don't have a unit test

this utility function is simple enough that we would basically just be writing the implementation which doesn't add a lot of extra assurance imo
there is also plenty of transitive test coverage via things that use the Validator so i think it is fine w/o


[WEBI] - Sort internal tables

Function module WEBI_GET_OBJECT returns several internal tables. Sometimes, the sorting order of these internal tables is different in each system ex: S/4HANA and non-S/4HANA. This fixes #5389.

Good find. Thx


Save and display both stdout and stderr for simulations interleaved together

Currently runBioSimulations is setup to display one of stdout or stderr. Even when a simulation fails, it would be useful to display the stdout (up to the point of failure).
Currently, runBioSimulations displays stdout and stderr separately. This makes it hard to understand when the stderr was written relatively to messages in stdout. In contrast, consoles typically display both stdout and stderr (in different colors). CI systems such as GitHub actions do the same. We should as well.

[ ] Save and display both stdout and stderr
[ ] Display stdout and stderr interleaved together, with stderr in a different color (red)

This idea is superseded by #1851.

This idea is superseded by #1851.


v1.3 backports 2019-03-05

v1.3 backports 2019-03-05
Included

#7265 -- api: Return 500 "Internal Error" when API handlers panic (@joestringer)

Not included

#7246 -- Protect remaining local keys from kvstore delete events (@tgraf)
#7249 -- daemon: Remove old health EP state dirs in restore (@joestringer)

Dropped

#7258 -- contrib: $a corrupted patch in cherry-pick (@nebril)

Once this PR is merged, you can update the PR labels via:
$ for pr in 7265; do contrib/backporting/set-labels.py $pr done 1.3; done

@joestringer #7249 depends on e1b6ebb2500608620fad58fba4234974e81a3b7e (ep.StateDirectoryPath()). Maybe we can define the method separately and include into the backport?
@tgraf 234b257166ad5ca6dfbc5981c2ef31130c0bfad7 of #7246 depends on a5f8515478065477a9bf83da9eb494bd8646ae1d (Configuration.Observer) for testing. Maybe we can define the field separately?

This change is

test-me-please

test-me-please

test-me-please

test-me-please

Does it make sense to backport #7258 if it isn't the right fix?

Does it make sense to backport #7258 if it isn't the right fix?

As #7286 got merged and I hit the case which #7258 does not fix, it doesn't make sense to backport. Same applies to the v1.4 backport. I'm going to update both PRs.

test-me-please

@joestringer #7249 depends on e1b6ebb (ep.StateDirectoryPath()). Maybe we can define the method separately and include into the backport?

@brb sure, that would be a simple change and easy to compare and see it's correct against master. Let me know if you want me to do the backport.