Do we need to close cursor in this case?

I have a code that uses cursor directly like this:
def execute(session, query):
    connection = session.connection().connection
    cursor = connection.cursor()
    cursor.execute(query)
    description = cursor.description
    if description is not None:
        rows = cursor.fetchall()
    return rows

result = execute(session, "select 1")

I get "Connection is busy" error when there is heavy load in the server. However, I don't see a possibility of this as:

I use single session per thread.
I always consume the results.
Cursor gets closed after it goes out of scope due to GC?

However, is it better to use the closing context-manager in this case? There are some edge cases where fetchall might not get called (code not shown - but I assume it is not causing this).
def execute(session, query):
    connection = session.connection().connection
    with closing(connection.cursor()) as cursor:
        cursor.execute(query)
        description = cursor.description
        if description is not None:
            rows = cursor.fetchall()
    return rows

result = execute(session, "select 1")

Kindly let me know your thoughts.

Are you also making one connection to the database per thread?

Just to be clear: session is a SQLAlchemy Session object, correct?

Thanks for checking, @v-chojas, @gordthompson . I forgot to add a small detail. Usually, the flow is like this:
def api():
result = session.query(...)
result = session.execute(...)
result = execute(session, query) # legacy api
...
So, we use SQLAlchemy's (scoped) session to execute queries. I think it uses the same connection because it uses the session.connection() connection pool under the hood? This has been my observation so far. Are there problems mixing queries like this?

I don't know what driver or DB you're using, but some don't support having more than one simultaneous active resultset on the same connection.

You mention multiple threads and "heavy load".

Yeah, but I am thinking that it is not possible for the same connection to use multiple cursors because: 1) scoped session returns the same connection always 2) executing multiple queries sequentially can't overlap over time.

I recommend you think more about what it means to have multiple threads and why it only happens under "heavy load".

I will read more on that. Appreciate your help!


[intensifier]Encrypted with tls-crypt-v2

Encrypted with tls-crypt-v2, thanks

Will have a look on it and most probably, implement


test-branch/execute-job/quoted-authorization/9f82aa23-bae6-47e8-8278-93c2c4c7a706

PR for testing ef6256f728ea8b738b28dfb9fbe967b5197a657f

/canonical/self-hosted-runners/run-workflows ef6256f728ea8b738b28dfb9fbe967b5197a657f


Clarify depth metrics reported

OPS Build status updates of commit 8694bb7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/machines/resources-estimator.md
:white_check_mark:Succeeded

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

Interesting, the checks on this passed but now that I've merged it the publish failed. @cgranade any idea why? Any relation to
Line 215: [Error] Error happened to execute the task _dependentPackages/CommonPlugins/tools/DiffFolder.ps1

Line 81: [Error] W:\rnaq-s\.optemp\packages\opbuild.scripts.1.72.0\tools\opbuild\plugin.executor.ps1 failed to execute customized script in the plugin step docset_prebuild in the docset quamtum-api

Line 138: [Error] W:\rnaq-s\.optemp\packages\opbuild.scripts.1.72.0\tools\opbuild\docset.prebuild.ps1 failed to execute

Line : [Warning] Cannot find docset information for repository: 509346e3-2203-7eb3-98f8-84fb7fa5518f and docset name: quantum-api-private. Please make sure the docset is correctly provisioned

Line : [Warning] Cannot find docset information for repository: 509346e3-2203-7eb3-98f8-84fb7fa5518f and docset name: quantum-docs-private. Please make sure the docset is correctly provisioned

Line 87: [Warning] Failed to run script W:\rnaq-s\_dependentPackages\CommonPlugins\tools\DiffFolder.ps1 with error message: Argument $output should not be null or empty.Exception.Message.


Added format constants that follow HTML5 input types

This is my first open source contribution.
I've added format constants that follow HTML5 input types based on #3928.

Nice PR.
One thing (i'm not a Moment.js contributor): I think using abbreviation MILLIS is wrong.
It should be MILLISECONDS or MS => more coherent with Moment.js style, see 1, 2, 3...

@nimshory you have to revert all your whitespace changes to the files.
Overall I like the change, but I don't like the place/names for all constants. Maybe moment.HTML5_FMT.XXXX, or moment.FMT_XXX, the first could be abbreviated with code so its a bit better.

Thanks for contributing @nimshory - but I will close this in favor of #4154


updated pomotodo (0.13.2,1453112814)

Closes #17729.

Merged as 3b127cf43f682a4829b53cabacc531595945e243.


Requesting assistance with an error while running a demo in max78000 fthr board.

Open On-Chip Debugger (Analog Devices 0.12.0-1.0.0-7) OpenOCD 0.12.0 (2023-09-27-07:53)
Licensed under GNU GPL v2
Report bugs to<EMAIL_ADDRESS>Started by GNU MCU Eclipse
Info : Listening on port 6666 for tcl connections
Info : Listening on port 4444 for telnet connections

This is the error I am facing while debugging and running the demo hello world in max78000 fthr board.
I am getting this output in my terminal of eclipse MaximSDK.
I have folllowed all the instructions specifies in the official github page of maxim and their user guide.
The build is successful and it produceses a .elf file then when i debug it it stops at the above when it should listen and work properly and give me a output.

Hello @maanasi8, that output is indicating that the OpenOCD server is launched and waiting for a GDB client to connect to it.
Can you capture a full screenshot of your Eclipse window when you launch the debug session?

@maanasi8 closing for inactivity.  Will re-open if necessary


Invalid select does not become invalid input

Instanciate a materialize select and it will not port the "invalid" class from the select to the created input element:

Resulting in:

Instead of:

@acburst are you sure that this shouldn't be the default behaviour according to material guidelines ?

This is a huge limitation for me when using Vue since I can't bind a dynamic class to the input element using single file component templates.

Custom jQuery and vanilla JS code I've created to fix this issue for my own projects.
Both work like a charm, but it would be fantastic to get proper validation similar to this on the actual framework source code.
https://codepen.io/traveltimn/full/QWyoVOm

@TravelTimN probably not happening, this package is not maintained:


fixed command formatting for pylint and pydocstyle

PR Details
Fixes the command formatting for arguments of pydocstyle and pylint. Arguments were ignored in previous version.
Description
cf above
Motivation and Context
Bugfix

Thanks for fixing it!


Go SDK + CloudQuery Plugin SDK

Hi Team, hopefully this is right place to ask, if not, I'd appreciate if you can direct me.
I'm the founder of cloudquery.io, a high performance open source ELT framework.
Our users are interested in a Pinterest plugin, but as we cannot maintain all the plugins ourselves, I was curious if this would be an interesting collaboration, where we would help implement an initial source plugin, and you will help maintain it.
This will give your users the ability to sync Pinterest APIs (ads, analytics) to any of their datalakes/data-warehouses/databases easily using any of the growing list of CQ destination plugins.
Best,
Yevgeny

Thanks Yevgeny for taking a look at our new SDK/python client! We shared your info with our integration team internally and we'll certainly reach out if there's a future opportunity for us to build a plugin in your platform.


Malformed generic method comment freezes analyzer

Analyze this (incorrect) program with --strong:
void method(dynamic/*=T extends Uri*/ t) {}

Observe that the analyzer freezes until it eventually runs out of memory and crashes. Be sad.
I looked into it a little bit. When the generic method comment is inserted into the token stream, the original comment remains. So the first time the comment is handled, you end up with something like:
void method(dynamic T extends Uri /*=T extends Uri*/ t) {}

Then it fails to parse the extends.... At some point, it ends up retrying and hits the comment again:
void method(dynamic T extends Uri T extends Uri /*=T extends Uri*/ t) {}

Or something along those lines. I'm not sure why it keeps hitting the same code path again and again, but the result is an infinite expansion in the token stream.
cc @vsmenon @jmesserly @leafpetersen @bwilkerson

Cool! (In a sarcastic sense.)

void method(dynamic T extends Uri /*=T extends Uri*/ t) {}

At the very least I would have expected the type being replaced to be removed from the token stream, producing
void method(T extends Uri /*=T extends Uri*/ t) {}

@jmesserly Is this working as expected?

Then it fails to parse the extends....

What do you mean by "fails"? Does it throw an exception?
My guess is that we're not correctly recording the failure, and as a result we attempt to re-parse, but of course the token stream has been modified by then. Removing the comments as they are processed would certainly take care of the recursion problem.

This happens in _injectTokenList in src/generated/parser.dart
https://github.com/dart-lang/sdk/blob/c5f181894074048879482547a201655f581cbc76/pkg/analyzer/lib/src/generated/parser.dart#L3994
We don't remove the tokens from comment token stream. I could swear there was a TODO there about that. Not sure what happened to it, but definitely remember being worried about the fact that we still have those comments in there. :)
Most of the time it's fine, because the injected tokens don't themselves have preceding comments. But if we failed to advance at all, not even getting to those tokens (maybe we just peek'd?) then we'd hit a problem. Probably an easy fix. As long as we consume the generic method comment.
Also, FWIW, these comments were not supposed to last long. :)
There's some pretty egregious issues in the current implementation: https://github.com/dart-lang/sdk/issues/25637

Also BTW: you don't need to write "dynamic" there. It's implicit. :)

What do you mean by "fails"? Does it throw an exception?

That's what I'm not sure about. I stepped through the parser for a while but eventually got lost and figured it was better for me to just file the bug and get back to what I was working on.
All I know is that it seems to repeatedly walk over that token stream, adding the parsed comment tokens in again and again each time.

https://codereview.chromium.org/1686173002/


Updating tests for Last/First of empty table

Last/First are returning an error for the empty table returned by Filter.  This should be a blank record, consistent with Canvas.

Merged changes into #139


move preprocessing to be main

Allows main.nf to be the only executable

Done at https://github.com/biocorecrg/master_of_pores/commit/848c592b3a02ffe24fd4217c70ecbbc7df4a0b7d


Misc test fixes

some improvements to tests

Coverage remained the same at 89.318% when pulling 7edb0be606491880223516a20f83b712d25a0368 on misc-test-fixes into a578eb82b436bcf3f2d3d7c7d8e60af43ae402ff on master.


Suggestion: Make the hover XP red if can't afford

Great job on the update, this is a good enhancement you could make.

Done.


Highlight groups not working

The highlight groups mentioned in the readme do not work.

Can you be a little more precise about that ?
They are all defined correctly and used in the code, so I wonder what you mean.

I'm trying to customize Azy groups for my colorscheme, but they do not seem to affect the colors in any way.
I'm referring to AzyMatch, AzyDim and AzyStandout.
To reproduce, I have this line right after the azy setup:
vim.api.nvim_set_hl(0, "AzyMatch", { foreground = "#151515", background = "#151515" })

The background of the matching entries remains red, unaffected.

That was a mistake from me, could you update the plugin and confirm that the issue is gone ?
Thanks for reporting this and for the help 1


TOC focus state not clearing after new active state selected

Detailed description

Describe in detail the issue you're having.

The Table of Contents is not clearing the focus state after a new active state is selected.   When the blue bar is on the focus state, the blue bar should clear.

Is this a feature request (new component, new icon), a bug, or a general
issue?

Bug

Is this issue related to a specific component?

Table of contents

What did you expect to happen? What happened instead? What would you like to
see changed?

The focus state should clear once a new active state is selected.

What browser are you working in?

Mac > Chrome

What version of the IBM.com Library are you using?

What offering/product do you work on? Any pressing ship or release dates we
should be aware of?

Web simplification
Steps to reproduce the issue

Learn page
Click any of the Table of contents options that are not the first item
Scroll to another section
See focus state and active state on two different items

Additional information

Screenshots or code
Notes

@Praveen-sr @chetankan Can you have Rahm take a look at this to see whether this is an accessibility issue?

3/10/2020;  Rahm said that it is working as expected.  Closing this issue.

@RobertaJHahn Checked this with the accessibility team, focus state is clearing after new active state Clicked / selected.


Fixed datatype size + Bake defines

This PR adds changes to the setup.py script to "bake" the defines used at compile time by generating a config.h file which is included in chipmunk.h.
This allows to change the types used for cpFloat (except for apple devices) plus the sizes of cpGroup, cpLayer and so on.
Additionally it exposes these types as the biggest possible types (uint64_t / double)  on the cython side.
Currently cymunk.pxd exposes cpFloat as float while chipmunk uses by default doubles.
While cython will internally use the type defined in the .h file (double), the type defined in cython will be used to convert to/from python. Thus a number too big for a float will be returned to the python side as INFINITE. Too large integers will raise an OverflowError.
I also removed the duplicate include folder.

Doesn't work on Python 3, due to line 52 in setup.py - you use dict.iter_items, which in python 3 is dict.items


Remove autoprefixer from cssnext preprocessors

Read https://github.com/ifraixedes/web-starter-kit-postcss/commit/837589d4582ecd79d1d988b310aee5449b22abd2

@MoOx Taking a look to your comment, I'm wondering how I could pass the autoprefixer configuration to postcss-cssnext.
Can @MoOx help me?
Thanks.

@MoOx Taking a look to your comment, I'm wondering how I could pass the autoprefixer configuration to postcss-cssnext.
Can @MoOx help me?
Thanks.

Taking a look to the implementations of postcss-cssnext and gulp-postcss, I found how to pass the options to postcss-cssnext are based in this conditional and how to pass the options to gulp-postcss
The result of that has been the PR #2 which I've sent.
@MoOx I would appreciate that you have a quick look to know if that's right.
Thanks.

#2 has been merged.


Drillable-measures type chart- Loss of Visual Clarity at Lowest Data Level

when drilling down to the lowest level of data, the waterfall chart does not maintain visual clarity .

Expected Behavior

Current Behavior

Possible Solution

Steps to Reproduce (for bugs)

Context

Screenshots

On "Drillable measures" page in the simple waterfall local testing power bi report, if you will drill down on "lowest level of data" by clicking on drill down option , the chart is not looking in a proper structure.


[Improvement][DataSourcePlugin]tiny optimize of datasource plugin module

move datasource-special constant like JDBC_DRIVER to DataSourceProcessor
change the CommonDataSourceClient.setDefaultValidationQuery() to fit various datasource
override OceanBaseDataSourceClient.setDefaultValidationQuery() cause CompatibleMode is needed
this pr does not make too much sense, I just think it is a better pattern, it also fixes setValidationQuery problem too!

Purpose of the pull request

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

(or)
If your pull request contain incompatible change, you should also add it to docs/docs/en/guide/upgrede/incompatible.md

sorry, this is my first pr, im not familiar with github and conduct

https://github.com/apache/dolphinscheduler/issues/14541

#14540

#14541

Codecov Report

Merging #14540 (db94175) into dev (8020c15) will decrease coverage by 0.07%.
The diff coverage is 78.46%.

:exclamation: Current head db94175 differs from pull request most recent head 771699c. Consider uploading reports for the commit 771699c to get more accurate results

@@             Coverage Diff              @@
##                dev   #14540      +/-   ##
============================================
- Coverage     38.73%   38.66%   -0.07%
+ Complexity     4578     4556      -22
============================================
  Files          1237     1237
  Lines         43537    43489      -48
  Branches       4823     4791      -32
============================================
- Hits          16862    16816      -46
+ Misses        24804    24801       -3
- Partials       1871     1872       +1

Impacted Files
Coverage Δ

...cheduler/common/constants/DataSourceConstants.java
0.00% <ø> (ø)

.../datasource/api/client/CommonDataSourceClient.java
2.32% <0.00%> (-0.06%)
:arrow_down:

...atasource/api/provider/JDBCDataSourceProvider.java
0.00% <0.00%> (ø)

...r/plugin/datasource/api/utils/DataSourceUtils.java
0.00% <0.00%> (ø)

...lickhouse/param/ClickHouseDataSourceProcessor.java
0.00% <0.00%> (ø)

...ugin/datasource/oracle/OracleDataSourceClient.java
0.00% <ø> (ø)

...ce/azuresql/param/AzureSQLDataSourceProcessor.java
25.45% <50.00%> (ø)

...ce/redshift/param/RedshiftDataSourceProcessor.java
32.55% <50.00%> (ø)

...source/oracle/param/OracleDataSourceProcessor.java
48.14% <75.00%> (ø)

...source/athena/param/AthenaDataSourceProcessor.java
77.27% <100.00%> (ø)

... and 13 more

... and 37 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

PTAL @ruanwenjun

Closing due to inactivity.


test: add additional tests to utils/compact-adjacency-matrix

Resolves # .
Description
Test Cases for adjacency matrix

What is the purpose of this pull request?

This pull request:

a Added more test cases for adjacency matrix.
b Checking whether the matrix is null, updation of matrix, and checking for edges between the vertices.
c They have been implemented in form of test cases.

Related Issues

Does this pull request have any related issues?

This pull request:

resolves #
fixes # adding more test-cases

Questions

Any questions for reviewers of this pull request?

Please review the features and suggest changes if required.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.
Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines.

@stdlib-js/reviewers

Please review it.


Change responsive to treat breakpoints as min-width

The responsive setting currently turns input breakpoints into the below:
"(min-width: ${previousBreakpoint}px) and (max-width: ${breakpoint}px)"
Aligning with modern practices, the input breakpoints should be treated as min-width only instead:
"(min-width: ${breakpoint}px)"

released in v0.5.0


rustPlatform.fetchCargoVendor: decrease concurrency

Related PR: https://github.com/NixOS/nixpkgs/pull/356539
fetchCargoVendor had very high concurrency limits. Decreased it to 5 jobs max. Also, disabled concurrent fetching of git stuff, since that seemed to be the most flaky in https://github.com/NixOS/nixpkgs/pull/356539
This change doesn't impact the download time by much, since AFAICT we were limited by the network bandwidth anyways.

Things done

Built on platform(s)

[ ] 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/)
25.05 Release Notes (or backporting 24.11 and 25.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

[ ] Fits CONTRIBUTING.md.

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

Merged as part of https://github.com/NixOS/nixpkgs/pull/356539


Save output to ansi text file

Is there a way to save the output as an ansi text file? If I try to redirect output, it barfs during terminal size detection. If I also specify the output size with -d, it complains the terminal is too small.
Update: I was able to capture the output using script but it would certainly be nice to have a proper save-as-ansi option.

@greenseeker I'm taking a guess here, but I believe the error occurs because you're redirecting stdout to a file. The tool detects terminal size from stdout every time, which is usually the terminal, so I can see how that's an issue. Custom dimensions also won't work since they check whether the width exceeds the terminal (stdout again) as well.
I wanted to keep the flags simple for most users so I'm not sure about adding character encoding functionality, because then covering only ANSI characters set would seem inappropriate. For the time being, I believe you can change the saved text file encoding with tools like iconv or recode.
Will have to think about this. Thanks for pointing out the issues though, I'm a bit busy these days but I'll try to make a fix soon.


Cannot load anything in MXML and how to debug the problem??

I tried tol load xmls with mxmlLoadFile(NULL, fp, MXML_OPAQUE_CALLBACK);, but got 0x0000000000000000 <NULL> as result.
The whole XMLLoad is liked this:
	wCharSet cs;
	const char* filename = (const char*)cs.conv(WCS_LOCAL, pwzFileName, WCS_UNICODE);
	FILE* fp = NULL;
	if ((fp = fopen(filename, "r")) == NULL) {
		return false;
	}

	Free();
	m_pXmlRoot = mxmlLoadFile(NULL, fp, MXML_OPAQUE_CALLBACK);
	fclose(fp);

as I seen that filename can be opened and the fp is like 0x0000017a365acfa0 {_Placeholder=0x0000000000000000 }, I checked lots of information that the result means that the file can be opened successfully, and the XML file used to be read before, and no changes happened.
So, could you please tell me how to find out the real reason for why I cannot load XML with mxmlLoadFile??
My operation system is Windows10 and Visual Studio 2017, with MXML 2.9
BTW, could you tell me how to update MXML 2.9 to MXML4? I checked the log that MXML4 seems quite different from MXML3(might be also 2.9), I tried to replace the direction of dll and libs and shows that mxml_node_s is not fit for mxml_node_t.
Great thanks again!

It would be useful to see a sample XML file you are trying to load. Also, the default error callback will send any error messages off to stderr, so if you do this from a console application you'll be able to see those errors. (regular applications don't have a console so you'd need to provide your own error callback to see the messages...)
Also, the struct mxml_node_s type became fully private in Mini-XML 3 - even for Mini-XML 2.x you should be using the provided accessor functions.
Finally, Mini-XML 4 is available as source. Your own code will require some conversion to it but the migration guide that is part of the documentation should point you in the right direction.

Thanks for your reply! I tried to run the code compiled in Visual Studio, but not any errors came back, just cannot corrrectly get the XML content. Here is the content from 'tag.xml', one of my XML(yes...every XMLs cannot be read in my code...very sad):

align_horz="near" align_vert="far" sort_descend="true">

Thanks for your advice for accessing the structure, I would get more information from the migration guide!

Thanks for your reply! I tried to run the code compiled in Visual Studio, but not any errors came back, just cannot corrrectly get the XML content. Here is the content from 'tag.xml', one of my XML(yes...every XMLs cannot be read in my code...very sad):
align_horz="near" align_vert="far" sort_descend="true">
Thanks for your advice for accessing the structure, I would get more information from the migration guide!

Thanks for your reply! I tried to run the code compiled in Visual Studio, but not any errors came back, just cannot corrrectly get the XML content. Here is the content from 'tag.xml', one of my XML(yes...every XMLs cannot be read in my code...very sad):
<?xml version="1.0" encoding="utf-8"?><masses><mass name="top_left"
align_horz="near" align_vert="far" sort_descend="true"><item
tag="0010,1010" /><item tag="0010,0020" /><item tag="0010,0010" /></mass><mass
name="top_right" align_horz="far" align_vert="far" sort_descend="true"><item
tag="0008,0070" /><item tag="0008,1090" /><item tag="0008,0020" /></mass><mass
name="bottom_left" align_horz="near" align_vert="near"
sort_descend="false"><item tag="0018,0060" /><item tag="0018,7030" /></mass><mass
name="bottom_right" align_horz="far" align_vert="near"
sort_descend="false"><item tag="0028,1051" /><item tag="0018,0050" /><item
tag="0020,0013" /></mass></masses>

Thanks for your advice for accessing the structure, I would get more information from the migration guide!

with the help of Google and ChatGPT, I tried getting some information from console, and with adding a function named mxmlSetErrorCallback,  the function is called before calling mxmlLoadFile. But not any error infomation back.
	mxmlSetErrorCallback(errorCallback);
	m_pXmlRoot = mxmlLoadFile(NULL, fp, MXML_OPAQUE_CALLBACK);

with the errorCallback is:
void errorCallback(const char *message) {
	fprintf(stderr, "XML parsing error: %s\n", message);
}

Hi there! Thanks for your patience... But I still have some problems like this, I tried to simplify my problems to find out what's happening in my scene. And run with MXML4.0, With writing some test code like these:
#include <iostream>
#include <mxml.h>

void errorCallback(const char* message) {
	fprintf(stderr, "XML parsing error: %s\n", message);
}

int main()
{
	const char* filename = (const char*)"example.xml";

	FILE* fp = NULL;
	fp = fopen(filename, "r");

	mxml_node_t* tree = mxmlNewXML("1.0");
	mxml_options_t *options = mxmlOptionsNew();
	mxmlOptionsSetTypeValue(options, MXML_TYPE_OPAQUE);

	tree = mxmlLoadFilename(NULL, options, filename);

	if (!tree) {
		fprintf(stderr, "Failed to load XML file: %s\n", filename);
		return 1;
	}

	fprintf(stderr, "Element::%s\n", mxmlGetElement(tree));
	fprintf(stderr, "Value::%s\n", mxmlGetText(tree, 0));

	mxmlDelete(tree);

	std::cout << "Hello World!\n";
}

But the console shows:
Element::(null)
Value::(null)

And the example.xml I've tried is from your document website like this:
<?xml version="1.0" encoding="utf-8"?>
<data>
    <node>val1</node>
    <node>val2</node>
    <node>val3</node>
    <group>
        <node>val4</node>
        <node>val5</node>
        <node>val6</node>
    </group>
    <node>val7</node>
    <node>val8</node>
</data>

For resetting the callback output of the console, I found that there are different RANDOM errors comes out when mxmlLoadFile, here are the outputs I saw.
1. just failed loading files
2. "[bad control character 0x00 not allowed by ](mxml: Bad control character 0x00 not allowed by XML standard!)"

(but I opened the file in Notepad++ and display every symbol and got nothing wrong.)

**3. "mxml: Mismatched close tag  under parent !" ** or "mxml: Mismatched close tag  under parent !"

but the declaration is necessary, seems failed to load the correct tag? but the file is not encrypted I am sure.

4. mxml:  cannot be a second root node after

I search the xml and nothing wrong.

Here is the XML loading code which is not work.
`#include
#include <mxml.h>
void errorCallback(const char* message) {
fprintf(stderr, "XML parsing error: %s\n", message);
}
int main() {
const char* filename = "example.xml";
FILE* fp = fopen(filename, "r");
if (!fp) {
	perror("Error opening file");
	return 1;
}

std::cout << "File opened successfully." << std::endl;

char ch;
while ((ch = fgetc(fp)) != EOF) {
	putchar(ch);
}
rewind(fp);

fprintf(stderr, "\n-------------------------------------------------------------------------------\n");

mxml_node_t* tree = mxmlLoadFile(NULL, fp, MXML_OPAQUE_CALLBACK);
fclose(fp);

if (!tree) {
	fprintf(stderr, "Failed to load XML file: %s\n", filename);
	return 1;
}

mxml_node_t* root = mxmlFindElement(tree, tree, "data", NULL, NULL, MXML_DESCEND);
if (!root) {
	fprintf(stderr, "Failed to find root element\n");
	mxmlDelete(tree);
	return 1;
}

for (mxml_node_t* node = mxmlFindElement(root, root, "node", NULL, NULL, MXML_DESCEND);
	node != NULL;
	node = mxmlFindElement(node, root, "node", NULL, NULL, MXML_NO_DESCEND)) {
	fprintf(stdout, "Element::%s\n", mxmlGetElement(node));
	fprintf(stdout, "Value::%s\n", mxmlGetOpaque(node));
}

mxmlDelete(tree);

std::cout << "Hello World!\n";
return 0;

}
`
Here is the XML file. But I saved it as UTF-8 file. I tried to fix the problem by saving it as ASCII file and then re-save it as UTF-8, but still unwork...
example.zip
Great Thanks!


any chance of making a cstl decompiler

hi i'm working on Cyanotype Daydream on steam but all the English translations are on cstl files any chance of making a decompiler like the one you made foe CST files

Maybe you can try notepad++

# 用于粗略提取 CatSystem2 的 .cstl 文件中的翻译内容的脚本
# 输出在与脚本同目录下的 cstl_out.txt 文件中
# Script for rough extraction of translations in the cstl file of Catsystem2
# The output is in cstl_out.txt in the same directory as the script

import re
import sys

filename = "com04B.cstl"

f = open(filename, "r", encoding="utf-8", errors="ignore")

fo = open("cstl_out.txt","w", encoding="utf-8")

sys.stdout = fo

# 分割对话的符号
# Symbols used to split conversations
dialogSep = re.compile(r"[\x00][\x00-\xff]")

# 分割对话与角色名的符号
# Symbols used to split dialog and role names
roleSep = re.compile(r"[\x00-\xff]「|「")

unknowChar = re.compile(r"\\x[0-9a-f][0-9a-f]")

while True:
    line = f.readline()
    if not line:
        break
    line = re.sub(dialogSep, "\n", line)
    line = re.sub(roleSep, " 「", line)
    cstlStr = repr(line)

    # 替换剩余所有无法识别为文本的二进制为换行符,如\x00
    # Replace any remaining binary unrecognized as text
    # with a line break, like \x00
    cstlStr = re.sub(unknowChar, "\n", cstlStr)

    cstlStr = cstlStr.split("\n")

    for s1 in cstlStr:
        betterStr = s1.split("\\n")

        for s2 in betterStr:

            # 过滤掉含日文假名的句子,目的是尽可能提取出翻译
            # Filter out sentences with Japanese kana
            # in order to extract translations whenever possible
            if s2 and not re.search("[あ-んア-ン]", s2):
                print(f'"{s2}"\n')

this is a python script, which may be helpful.
# 用于粗略提取 CatSystem2 的 .cstl 文件中的翻译内容的脚本
# 输出在与脚本同目录下的 cstl_out.txt 文件中
# Script for rough extraction of translations in the cstl file of Catsystem2
# The output is in cstl_out.txt in the same directory as the script

import re
import sys

#输入文件
filename = "com04B.cstl"

f = open(filename, "r", encoding="utf-8", errors="ignore")

fo = open("cstl_out.txt","w", encoding="utf-8")

sys.stdout = fo

# 分割对话的符号
# Symbols used to split conversations
dialogSep = re.compile(r"[\x00][\x00-\xff]")

# 分割对话与角色名的符号
# Symbols used to split dialog and role names
roleSep = re.compile(r"[\x00-\xff]「|「")

unknowChar = re.compile(r"\\x[0-9a-f][0-9a-f]")

while True:
    line = f.readline()
    if not line:
        break
    line = re.sub(dialogSep, "\n", line)
    line = re.sub(roleSep, " 「", line)
    cstlStr = repr(line)

    # 替换剩余所有无法识别为文本的二进制为换行符,如\x00
    # Replace any remaining binary unrecognized as text
    # with a line break, like \x00
    cstlStr = re.sub(unknowChar, "\n", cstlStr)

    cstlStr = cstlStr.split("\n")

    for s1 in cstlStr:
        betterStr = s1.split("\\n")

        for s2 in betterStr:

            # 过滤掉含日文假名的句子,目的是尽可能提取出翻译
            # Filter out sentences with Japanese kana
            # in order to extract translations whenever possible
            if s2 and not re.search("[あ-んア-ン]", s2):
                print(f'"{s2}"\n')

you can easily do this with regular expressions.
this is a python script, which may be helpful.
# 用于粗略提取 CatSystem2 的 .cstl 文件中的翻译内容的脚本
# 输出在与脚本同目录下的 cstl_out.txt 文件中
# Script for rough extraction of translations in the cstl file of Catsystem2
# The output is in cstl_out.txt in the same directory as the script

import re

filename = "com04B.cstl"

f = open(filename, "r", encoding="utf-8", errors="ignore")

fo = open("cstl_out.txt","w", encoding="utf-8")

# 分割旁白的符号
# Symbols used to split narrations
dialogSep = re.compile(r"[\x00][\x00-\xff]")

# 分割对话与角色名的符号，但这可能不起作用
# Symbols used to split dialog and role names, but it may not work
roleSep = re.compile(r"[\x00-\xff]「|「")

unknowChar = re.compile(r"\\x[0-9a-f][0-9a-f]")

while True:
    line = f.readline()
    if not line:
        break
    line = re.sub(dialogSep, "\n", line)
    line = re.sub(roleSep, " 「", line)
    cstlStr = repr(line)

    # 替换剩余所有无法识别为文本的二进制为换行符
    # Replace any remaining binary unrecognized as text with a line break
    cstlStr = re.sub(unknowChar, "\n", cstlStr)

    cstlStr = cstlStr.split("\n")

    for s1 in cstlStr:
        betterStr = s1.split("\\n")

        for s2 in betterStr:

            # 过滤掉含日文假名的句子，目的是尽可能提取出翻译
            # Filter out sentences with Japanese kana
            # in order to extract translations whenever possible
            if s2 and not re.search("[あ-んア-ン]", s2):
                fo.write('"{}"\n\n'.format(s2))

fo.close()

This is my Python 3 extraction script that doesn't use regular expressions
https://github.com/masagrator/NXGameScripts/blob/main/Cyanotype Daydream/CSTL_Extractor_PC.py
Just provide folder with cstl files as argument


Enabled PDB for Envoy deployments

Q
A

Bug fix?
no

New feature?
yes

API breaks?
no

Deprecations?
no

Related tickets
none

License
Apache 2.0

What's in this PR?
Added support to create PDBs for Envoy deployments.
Why?
The scope of this PR is to enable the creation of PDBs for Envoy deployments to facilitate more resilient deployments.
Additional context
In an environment where there are rolling updates for underlying Kubernetes nodes, we might need to account for normal disruptions (safe evictions) initiated by a cluster Admin. With this in mind, we can create a PDB for envoy deployments to ensure that there isn't a big disruption for incoming requests during a rollout.
Checklist

[x] Implementation tested
[x] Error handling code meets the guideline
[x] Logging code meets the guideline
[x] User guide and development docs updated (if needed)

@stoader @baluchicken this one is ready for a review

@amuraru can you please also take a quick look on this. Thanks


Build error on Fedora 29

I'm getting the following build error on the latest commit d94e4f11bd36e6a329e9266236b5a5f7a2e10220 using stable-channel Cargo cargo 1.37.0 (9edd08916 2019-08-02), on Fedora Linux 29 :
error occurred: Command "cc" "-O0" "-ffunction-sections" "-fdata-sections" "-fPIC" "-g"
 "-fno-omit-frame-pointer" "-m64"
 "-I" "/home/chris/stuff/git2-rs/target/debug/build/libgit2-sys-be209f253bfdf532/out/include"
 "-I" "libgit2/src"
 "-I" "libgit2/deps/http-parser"
 "-I" "libgit2/deps/pcre"
 "-I" "/home/chris/stuff/git2-rs/target/debug/build/libssh2-sys-30f297c01a235695/out/include"
 "-fvisibility=hidden" "-DGIT_REGEX_BUILTIN=1" "-DHAVE_STDINT_H=1"
 "-DHAVE_MEMMOVE=1" "-DNO_RECURSE=1" "-DNEWLINE=10"
 "-DPOSIX_MALLOC_THRESHOLD=10" "-DLINK_SIZE=2"
 "-DPARENS_NEST_LIMIT=250" "-DMATCH_LIMIT=10000000"
 "-DMATCH_LIMIT_RECURSION=MATCH_LIMIT" "-DMAX_NAME_SIZE=32"
 "-DMAX_NAME_COUNT=10000" "-DSHA1DC_NO_STANDARD_INCLUDES=1"
 "-DSHA1DC_CUSTOM_INCLUDE_SHA1_C=\"common.h\""
 "-DSHA1DC_CUSTOM_INCLUDE_UBC_CHECK_C=\"common.h\""
 "-o" "/home/chris/stuff/git2-rs/target/debug/build/libgit2-sys-be209f253bfdf532/out/build/libgit2/deps/http-parser/http_parser.o"
 "-c" "libgit2/deps/http-parser/http_parser.c"
 with args "cc" did not execute successfully (status code exit code: 1).

The full command output of cargo build -v is attached: output.txt
This was first discovered at https://github.com/anderejd/cargo-geiger/issues/67

Thanks for the report! This looks like a build issue with libgit2 itself:
cargo:warning=libgit2/deps/http-parser/http_parser.h:77:60: error: unknown type name ‘size_t’
cargo:warning= typedef int (*http_data_cb) (http_parser*, const char *at, size_t length);
cargo:warning=                                                            ^~~~~~
cargo:warning=libgit2/deps/http-parser/http_parser.h:77:60: note: ‘size_t’ is defined in header ‘<stddef.h>’; did you forget to ‘#include <stddef.h>’?
cargo:warning=libgit2/deps/http-parser/http_parser.h:47:1:
cargo:warning=+#include <stddef.h>

mind opening an issue upstream with them?


renderer source map issue + works-for-me fix

[X] Review the documentation: https://docs.sentry.io/platforms/javascript/electron/
[X] Search for existing issues: https://github.com/getsentry/sentry-electron/issues
[X] Use the latest release: https://github.com/getsentry/sentry-electron/releases
[X] Provide a link to the affected event from your Sentry account
https://sentry.io/organizations/jessedp/issues/1629926466/?project=5208692

Versions + Platform

[X] SDK version -<EMAIL_ADDRESS>[X] Electron version -<EMAIL_ADDRESS>[X] Platform - All

Description
The few steps to get events into Sentry were a breeze. Using @sentry/webpack-plugin v1.10.0 got the source maps there (apologies if this should be filed there). With both, I used the standard suggested configs (dsn/org/project/release here).
Sourcemaps in the main process (main.prod.js for me) worked perfectly without changing anything. The artificat in the Release is listed as  ~/app/main.prod.js(.map)
The extra app seems weird as I've read that "~/" is shorthand to remove the protcol (app:// here).
The renderer process appears in the release artifacts as  ~/app/dist/renderer.prod.js(.map)
Same extra app , though they are properly in the dist subdir.
In the GUI, the event shows 1 error (Source code was not found) saying it is looking for app:///dist/renderer.prod.js. Once I made it look for app:///app/dist/renderer.prod.js things worked correctly.
My fix was another one adding beforeSend to the init properties, though not one I'd found other places. Here's what worked for me:
  init({
    ...
    beforeSend(event) {
      if (
        !event.exception ||
        !event.exception.values ||
        !event.exception.values[0]
      ) {
        return event;
      }

      const value = event.exception.values[0];

      if (value.stacktrace && value.stacktrace.frames) {
        value.stacktrace.frames.forEach(frame => {
          if (frame.filename.includes('dist/')) {
            // this is stupid, but it seems to work for now @sentry/electron: 1.3.0
            frame.filename = frame.filename.replace('dist/', 'app/dist/');
            // for some reason the doubled up app/ , so fix the
            frame.filename = frame.filename.replace('app/app/', 'app/');
          }
        });
      }

      return event;
    }
  });

Is this expected behaviour and something that should be necessary? If no, is it even an issue in this pacakge?

We have just released v3.0.0 of the SDK which includes a lot of changes and improvements since this issue was opened.
If you can still reproduce this with v3, please open a new issue and I'll look into it.


KMP Algorithm

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
C plus plus Implementaion of KMP Algorithm !
I am not a GSSoC'21 participant.
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
Issue #7548

KMP Algorithm


swupdate cannot find tegra234-p3737-0000+p3701-0000-nv.dtb image file L4T 36.3.0

Hello,
I am attempting to build the swupdate-image-tegra for the AGX Orin devkit on L4T 36.3.0. I am running into this error:
ERROR: swupdate-image-tegra-1.0-r0 do_swuimage: swupdate cannot find tegra234-p3737-0000+p3701-0000-nv.dtb image file

Looks like dtb's are located under the devicetree/ subfolder in the deploy directory starting with L4T 36.3.0.

I am also bumping into this problem. I have solved it by this patch in meta-swupdate:
ndex 0367f81..cf583d0 100644
--- a/classes-recipe/swupdate-common.bbclass
+++ b/classes-recipe/swupdate-common.bbclass
@@ -299,7 +299,9 @@ def swupdate_add_artifacts(d, list_for_cpio):
bb.fatal("swupdate cannot find image file: %s" % os.path.join(deploydir, imagebase + fstype))
else:  # Allow also complete entries like "image.ext4.gz" in SWUPDATE_IMAGES
if not add_image_to_swu(d, deploydir, image, s, encrypted, list_for_cpio):

           bb.fatal("swupdate cannot find %s image file" % image)

           devicetreedir = os.path.join(deploydir ,"devicetree")

           if not add_image_to_swu(d, devicetreedir, image, s, encrypted, list_for_cpio):

               bb.fatal("swupdate cannot find %s image file" % image)

But I am not sure this is the correct solution.

I've used ${DEPLOY_DIR_IMAGE}/devicetree/${DTBFILE} instead of ${DTBFILE} in swupdate-image-tegra.bb and it worked the problem here is that swupdate-image-tegra doesn't cover all the cases. So in t234 machine the DTBs are placed in devicetree directory in depoly directory.

Sorry I'm just seeing this thread now. I suspect this was already handled with https://github.com/OE4T/tegra-demo-distro/pull/305/commits/4e1cdca9936bad099db330ca5126ca4244bfc970 in https://github.com/OE4T/tegra-demo-distro/pull/305 and this just needs a backport to scarthgap.

Verified build succeeds with https://github.com/OE4T/tegra-demo-distro/pull/309 - if someone with jetson-agx-orin-devkit could help test it I would appreciate, I don't have access to that platform.

Resolved with https://github.com/OE4T/tegra-demo-distro/pull/309 based on element discussion

Thanks, verified to work for the jetson-orin-nano-devkit-nvme.


Introduce camel-quarkus-jcache-caffeine

Follows up https://github.com/apache/camel-quarkus/pull/4088
and the discussion in https://github.com/apache/camel-quarkus/issues/1846#issuecomment-1245928494

com.github.benmanes.caffeine.jcache.copy.JavaSerializationCopier is still needed;

I see. I'd say the most user friendly solution would be to create a new extension camel-quarkus-jcache-caffeine that would host this reflection registration plus it would depend on quarkus-caffeine. We should also add a hint to extensions/jcache/runtime/src/main/doc/usage.adoc that end users may want to depend on camel-quarkus-jcache-caffeine if they want to use caffeine as a jcache impl.

I submitted a PR to Quarkus https://github.com/quarkusio/quarkus/pull/2793
Once merged, all we need to do is some cleanup in camel-quarkus-jcache, and no need to introduce another extension.


Replace alive progress with tqdm

Description

Replace alive_progress with tqdm in examples since tqdm is more well-known and easier to use.
TODO

[x] Replace in sphere.py
[x] Replace in lunar_lander.py

Status

[x] I have read the guidelines in CONTRIBUTING.md
[x] I have formatted my code using yapf
[x] I have tested my code by running pytest
[x] I have linted my code with pylint
[ ] I have added a one-line description of my change to the changelog in HISTORY.md
[ ] This PR is ready to go

Addressed in #253


How to sync/access a value of a provider b/w flutter and pure dart

Alternate Title:
How to use a WidgetRef even after getting disposed or how to stop it from being disposed on page pop.
Describe the bug
Let's say initial value of a StateNotifier was x.
When I change it to y and then access it using consumer hook widget ref then it returns y.
But when I access it using ProviderContainer(), then it returns x (i.e initial value).
Used in flutter:
ref.watch(myProvider);

Used in dart:
container.watch(myProvider)

Expected behavior
They both should return the same value.

If you are creating a ProviderContainer yourself, you shouldn't do that. You are essentially creating a different universe of providers with their respective initial values. Instead, read the provider values with a ref object.

@davidmartos96  How to use ref objects with pure dart, I am writing code inside a function, so I don't have access to ref object.

Then you need to pass a ref as a parameter to your function.
Alternatively, if this function is inside a class created within a provider, you can pass the ref as a constructor argument to then use it from inside.

@davidmartos96 I tried that, but when the widget disposed the ref also got disposed, but my function contains a running task(downloading a file) and at the same time updating the value of the download.

I'm not sure what you mean. Do you have some code as an example?

Here is my code explained:

I use dio package to download a files and then update the value of the bytes received and total bytes in state notifier provider.

This is a continuous process so it will required the ref to be alive even when HookConsumerWidget is disposed.
Code here
https://github.com/prateekmedia/appimagepool/blob/10aa863308ded3d632bd74b9f0e6b3d7e6eddfd1/lib/widgets/download_button.dart#L155

You could place the download logic inside a notifier and use the ref that you get in the provider callback, that way you can make sure that the ref is alive.you shouldn't rely on a widget ref after executing any await. It's similar to the Buildcontext in that sense.

You could place the download logic inside a notifier and use the ref that you get in the provider callback, that way you can make sure that the ref is alive. You shouldn't rely on a widget ref after executing any await. It's similar to the Buildcontext in that sense.

Thanks @davidmartos96 I will look into it.


bpftrace segmentation fault for version > 0.14.1

What reproduces the bug?
I am trying to run the following:
kprobe:fib_validate_source {
    $skb = (struct sk_buff*) arg0;
    $ipheader = ((struct iphdr *) ($skb->head + $skb->network_header));
    $version = ($ipheader->version) >>4;

    printf("%d:%d | %s:%s\n", $version, $ipheader->protocol,
        ntop($ipheader->saddr), ntop($ipheader->daddr));
}

But doing so fails with a segmentation fault:
# bpftrace fibtrace.bt
Segmentation fault (core dumped)

This code works as expected with bpftrace 0.14.1, but fails with both 0.15.0 and 0.16.0.
bpftrace --info
System
  OS: Linux 4.18.0-372.19.1.el8_6.x86_64 #1 SMP Mon Jul 18 11:14:02 EDT 2022
  Arch: x86_64

Build
  version: v0.16.0
  LLVM: 15.0.0
  ORC: v2
  foreach_sym: yes
  unsafe uprobe: no
  bfd: yes
  bcc_usdt_addsem: yes
  bcc bpf_attach_uprobe refcount: yes
  bcc library path resolution: yes
  libbpf btf dump: yes
  libbpf btf dump type decl: yes
  libbpf bpf_prog_load: yes
  libbpf bpf_map_create: yes
  libdw (DWARF support): no

Kernel helpers
  probe_read: yes
  probe_read_str: yes
  probe_read_user: yes
  probe_read_user_str: yes
  probe_read_kernel: yes
  probe_read_kernel_str: yes
  get_current_cgroup_id: yes
  send_signal: yes
  override_return: yes
  get_boot_ns: yes
  dpath: yes
  skboutput: no

Kernel features
  Instruction limit: 1000000
  Loop support: yes
  btf: yes
  map batch: yes
  uprobe refcount (depends on Build:bcc bpf_attach_uprobe refcount): no

Map types
  hash: yes
  percpu hash: yes
  array: yes
  percpu array: yes
  stack_trace: yes
  perf_event_array: yes

Probe types
  kprobe: yes
  tracepoint: yes
  perf_event: yes
  kfunc: yes
  iter:task: yes
  iter:task_file: yes
  kprobe_multi: no
  raw_tp_special: yes

Platform
This is RHCOS for OpenShift 4.11 (aka RHEL 8):
NAME="Red Hat Enterprise Linux CoreOS"
ID="rhcos"
ID_LIKE="rhel fedora"
VERSION="411.86.202208031059-0"
VERSION_ID="4.11"
PLATFORM_ID="platform:el8"
PRETTY_NAME="Red Hat Enterprise Linux CoreOS 411.86.202208031059-0 (Ootpa)"
ANSI_COLOR="0;31"
CPE_NAME="cpe:/o:redhat:enterprise_linux:8::coreos"
HOME_URL="https://www.redhat.com/"
DOCUMENTATION_URL="https://docs.openshift.com/container-platform/4.11/"
BUG_REPORT_URL="https://bugzilla.redhat.com/"
REDHAT_BUGZILLA_PRODUCT="OpenShift Container Platform"
REDHAT_BUGZILLA_PRODUCT_VERSION="4.11"
REDHAT_SUPPORT_PRODUCT="OpenShift Container Platform"
REDHAT_SUPPORT_PRODUCT_VERSION="4.11"
OPENSHIFT_VERSION="4.11"
RHEL_VERSION="8.6"
OSTREE_VERSION="411.86.202208031059-0"

Sadly, I cannot reproduce on my Fedora. Could be libbpf/bcc versions - which ones do you have?

Ah, got it. You need commit d470220bcd2d07eeeca6e118aeb0f71be52ee864 which is in master (not yet released, though).

@viktormalik thanks, I'll try building from source rather than using a release build. Is the bug platform specific (kernel/llvm/something else)?

IIUC, it's Clang-specific. The problem has always been in bpftrace, just newer Clang/LLVM (14+ I think) started to segfault on it.

Thanks for the help; running a recent build seems to have fixed things!

hi, would it be possible to cut a release of bpftrace? this seems like a pretty impactful bugfix, bpftrace segfaults every time without it. and this is now manifesting in arch repos since llvm 15 has been released
not sure who to @? @fbs @danobi as you've cut the few recent releases?
> sudo bpftrace -e 'BEGIN { print("test"); }'
Segmentation fault
>

these versions cause the segfault:
> pacman -Q bcc bpftrace clang llvm llvm-libs
bcc 0.26.0-2
bpftrace 0.16.0-2
clang 15.0.7-1
llvm 15.0.7-1
llvm-libs 15.0.7-1
>

these versions work fine:
> pacman -Q bcc bpftrace clang llvm llvm-libs
bcc 0.25.0-2
bpftrace 0.16.0-1
clang 14.0.6-4
llvm 14.0.6-4
llvm-libs 14.0.6-4
>

Hi @avaunit02, we're already working on this, you can expect a new release soon :-)


[LG] from assembly for [import], fromfile etc.

Use this query to search for the most popular feature requests.
Is your feature request related to a problem? Please describe.
From assembly will help in the following:

When I am writing an SDK, these lg files must be embedded in dll itself
When I don't need hot update, it will be faster from assembly

Describe the solution you'd like

Add an option in SDK
Add an option in function
Add new grammar/function

Describe alternatives you've considered
Additional context
Add any other context or screenshots about the feature request here.
[enhancement]

@xieofxie can you clarify the requirement, did you mean compile the lgfiles in dlls?

@boydc2014 Right. When I want to use lg embedded in an dll, how to handle the import fromFile etc.?

@vishwacsena This is a P1 bug that's been sitting for 2 months. How can we push this forward?

@cleemullins  this is not a bug, this is a feature requirement, and customer is asking about directly supported the interface to deal with Assembly. As a core library, I don't think it's the library's concern of where the files actually stores. And we've already provided generic approach through delegate for user to implement on there customized storage.
So I would suggest @xieofxie to read out the file from Assembly and feed into LGParser.ParseText method, in terms of import, the last parameter in LGParser.ParseText is "importResolver" you can also customize the way import is been handled.

@boydc2014 could we customize/override fromFile? Thanks

@boydc2014 could we customize/override fromFile? Thanks

fromFile is just a function, you can create own customize function like fromMyFile, see LG tests TestForCustomizedFunctions for example, but it's not recommended to override existing pre-built function.

@boydc2014 could we customize/override fromFile? Thanks

fromFile is not a build-in mechanism, it's just a function like any other functions, we support create customize function like fromMyFile, see LG tests TestForCustomizedFunctions for example, but it's not recommended to override existing pre-built function (which is also doable via override Expression.Functions table).

@xieofxie will this help?

@boydc2014 could we customize/override fromFile? Thanks

fromFile is not a build-in mechanism, it's just a function like any other functions, we support create customize function like fromMyFile, see LG tests TestForCustomizedFunctions for example, but it's not recommended to override existing pre-built function (which is also doable via override Expression.Functions table).

@xieofxie will this help?

Got it, thanks~


Headline with code block not rendered correctly sometimes

What version of Hugo are you using (hugo version)?

$ hugo version
Hugo Static Site Generator v0.68.1 linux/amd64 BuildDate: 2020-03-23T11:18:32Z

I have some headlines which looks like this:
## records(self, zone, params={})``
While this is rendered fine with blackfriday,

with goldmark it is broken

There seems to be something special with the {} in combination with code blocks + headline.
## `records(self, zone, params={})`   -> broken
`records(self, zone, params={})`        -> works
## `records(self, zone, params={ })`   -> works

Thanks for the report. This looks like it could be an issue with Goldmark. Could you create an issue at https://github.com/yuin/goldmark/issues and link back to this issue so we could track it?

Of course, I was just not sure where to start :)
https://github.com/yuin/goldmark/issues/123

I suspect this is the attribute parsing (used to set manual IDs on titles).

This has been fixed since v0.72 when we updated to goldmark v1.31.1.


Allow for multiple tiny planet conversions

Like the pattern used here:
https://github.com/mhulse/simple-gps
As it stands currently, I new this code and can't run multiple without newing again.
Need ability to call method multiple times for different images.
Also, need to get away from returning async method from constructor.

Improved API via 85e58eac14384b5f50b9691ee0d62091cbd8fe4d
Now there are getters/setters to update the options using one instance.


Do I need to create an issue to reference my open pull requests?

e.g. #170 #162
of course I'll try an keep the pull requests merge-able as often as possible.

No, there is no additional issue necessary. It will be merged soon. :-) We are working hard on reviewing ogit.

No problem, I have fixed them both to be auto-merge-able.


Add a name optional argument to #[export]

This allows exporting methods to godot under a different name.
This commit implements the feature described in #732

@toasteater  Just made a new commit addressing your comments. Let me know if there's anything else!


Flaky Splunk Platform Functional Test

We got a failed test run from a non functional change that was healed by re-running it: https://github.com/signalfx/splunk-otel-collector-chart/runs/4778137121?check_suite_focus=true
Looks like the tests are flaky and should be fixed.
@luckyj5 @chaitanyaphalak can some of you please take a look?

@dmitryax I faced this issue several times, eg: https://github.com/signalfx/splunk-otel-collector-chart/runs/4861763953?check_suite_focus=true, which passed successfully after rerunning.
Here tests are failing (as expected) because there is no data found in Splunk env (Splunk received 0 events in the last minute)
While comparing the failed run vs successful run, I noticed the below:

Seems like tests are failing because the pod is not in a healthy state/ not running.

Closing for inactivity. Please reopen if this is still occurring.


STYLE: GetGlobalDefaultSplitter replace call_once call with local static

Replaced a call_once call and two global variables with a single local static variable, to simplify the code, while still being thread-safe.

The ITK Python libraries has some requirements for how global variable should be shared and initialized. I believe @thewtex knows this the best, perhaps get a review from him too?

The ITK Python libraries has some requirements for how global variable should be shared and initialized.

Thanks for your comment @blowekamp FYI I did mark the pull request "ready" anyway, because I don't really think the choice between calling std::call_once and using thread-safe local static variable initialization would affect Python library requirements. So I just hope the PR passes the review 😃

The question I have is not that the new implementation has the same behavior as the old. The questions is if proper behavior for ITK Python should have an implementation for this variable using the custom initialization using the itkInitGlobalsMacro.

The Singleton functionality, used by ITK Python (although it could also be used in other contexts), uses a lamba; this style change (removing the lambda) is not compatible.

@N-Dekker I will create a PR to address the global default splitter properly.

Patch here: https://github.com/InsightSoftwareConsortium/ITK/pull/4122

Closed, as this pull request is superseded by:

#4122 commit 0f5b4bd3593c3626c017fa1aa3acba0507d81247 by Matt (@thewtex)


Tegra minimal init fix boot scripts

Thanks @liuming50 , looks like some good cleanup here.  I'll take a closer look at the changes when I have a bit more time.

Changes in patch set V2:
1 Add "PACKAGE_ARCH = "${MACHINE_ARCH}"" to tegra-minimal-init_1.0.bb as suggested by @madisongh
2 Drop the udev rules dependencies, change to use 'blkid' as suggested by @madisongh
3 Add one more fix to rootwait condition checking in init-boot.sh


split-memory-limit unit conversion oddity?

The docs for --split-memory-limit say that this number should be provided in units of megabytes. However, it looks like it is actually being interpreted as kilobytes. Perhaps there is an inconsistant conversion somewhere, e.g. multiplying by 1024 instead of 1024^2?
[saladi1@comet-ln3 mmdb]$ mmseqs search -h | grep mem
  --split-mode          	2         	0: split target db; 1: split query db;  2: auto, depending on main memory
  --split-memory-limit  	0         	Maximum system memory in megabyte that one split may use. Defaults (0) to all available system memory.

When I specify --split-memory-limit 1000000 (1 TB) to mmseqs search, it looks like mmseqs prefilter is called carrying this parameter through, however, it dies with the following message:

Can not fit databased into<PHONE_NUMBER> byte. Please use a computer with more main memory.

This is odd because,<PHONE_NUMBER> bytes = 1 GB not 1 TB (off by a factor of 1e3). See call and output below.
I'm not sure how many places this unit conversion is done, but in the context of mmseqs prefilter it does look like the parameter is multiplied by 1024 (as opposed to 1024^2).
https://github.com/soedinglab/MMseqs2/blob/d32e75025c1bd419188d4892db863a0f02b30ddf/src/prefiltering/Prefiltering.cpp#L184
MMseqs Output

[saladi1@comet-30-01 mmdb]$ mmseqs search \
>     euk_screen/screenDB \
>     uref_euk85_rep/uref_euk85_rep \
>     euk_screen_prof \
>     mmtemp \
>     --local-tmp /scratch/saladi1/22776241/ \
>     --remove-tmp-files 1 \
>     --split-memory-limit 1000000
search euk_screen/screenDB uref_euk85_rep/uref_euk85_rep euk_screen_prof mmtemp --local-tmp /scratch/saladi1/22776241/ --remove-tmp-files 1 --split-memory-limit 1000000

MMseqs Version:                                                           	fac81fa7a63c1205209d4acbc46ce2209abcaa8a
Substitution Matrix                                                       	blosum62.out
Add backtrace                                                             	false
Alignment mode                                                            	2
E-value threshold                                                         	0.001
Seq. Id Threshold                                                         	0
Min. alignment length                                                     	0
Seq. Id. Mode                                                             	0
Alternative alignments                                                    	0
Coverage threshold                                                        	0
Coverage Mode                                                             	0
Max. sequence length                                                      	65535
Compositional bias                                                        	1
Realign hit                                                               	false
Max Reject<PHONE_NUMBER>
Max Accept<PHONE_NUMBER>
Include identical Seq. Id.                                                	false
Preload mode                                                              	0
Pseudo count a                                                            	1
Pseudo count b                                                            	1.5
Score bias                                                                	0
Gap open cost                                                             	11
Gap extension cost                                                        	1
Threads                                                                   	64
Compressed                                                                	0
Verbosity                                                                 	3
Seed Substitution Matrix                                                  	VTML80.out
Sensitivity                                                               	5.7
K-mer size                                                                	0
K-score<PHONE_NUMBER>
Alphabet size                                                             	21
Max. results per query                                                    	300
Offset result                                                             	0
Split DB                                                                  	0
Split mode                                                                	2
Split Memory Limit                                                        	1000000
Diagonal Scoring                                                          	1
Exact k-mer matching                                                      	0
Mask Residues                                                             	1
Mask lower case residues                                                  	0
Minimum Diagonal score                                                    	15
Spaced Kmer                                                               	1
Spaced k-mer pattern
Local temporary path                                                      	/scratch/saladi1/22776241/
Rescore mode                                                              	0
Remove hits by seq.id. and coverage                                       	false
Sort results                                                              	0
In substitution scoring mode, performs global alignment along the diagonal	false
Mask profile                                                              	1
Profile e-value threshold                                                 	0.1
Use global sequence weighting                                             	false
Filter MSA                                                                	1
Maximum sequence identity threshold                                       	0.9
Minimum seq. id.                                                          	0
Minimum score per column                                                  	-20
Minimum coverage                                                          	0
Select n most diverse seqs                                                	1000
Omit Consensus                                                            	false
Min codons in orf                                                         	30
Max codons in length                                                      	32734
Max orf gaps<PHONE_NUMBER>
Contig start mode                                                         	2
Contig end mode                                                           	2
Orf start mode                                                            	1
Forward Frames                                                            	1,2,3
Reverse Frames                                                            	1,2,3
Translation Table                                                         	1
Use all table starts                                                      	false
Offset of numeric ids                                                     	0
Add Orf Stop                                                              	false
Chain overlapping alignments                                              	0
Merge query                                                               	1
Search type                                                               	0
Number search iterations                                                  	1
Start sensitivity                                                         	4
Search steps                                                              	1
Run a seq-profile search in slice mode                                    	false
Strand selection                                                          	1
Disk space limit                                                          	0
Sets the MPI runner
Force restart using the latest tmp                                        	false
Remove Temporary Files                                                    	true

prefilter euk_screen/screenDB uref_euk85_rep/uref_euk85_rep mmtemp/150669805093381426/pref_0 --sub-mat blosum62.out --seed-sub-mat VTML80.out -k 0 --k-score<PHONE_NUMBER> --alph-size 21 --max-seq-len 65535 --max-seqs 300 --offset-result 0 --split 0 --split-mode 2 --split-memory-limit 1000000 -c 0 --cov-mode 0 --comp-bias-corr 1 --diag-score 1 --exact-kmer-matching 0 --mask 1 --mask-lower-case 0 --min-ungapped-score 15 --add-self-matches 0 --spaced-kmer-mode 1 --db-load-mode 0 --pca 1 --pcb 1.5 --local-tmp /scratch/saladi1/22776241/ --threads 64 --compressed 0 -v 3 -s 5.7

Using 64 threads.
Can not fit databased into<PHONE_NUMBER> byte. Please use a computer with more main memory.
Error: Prefilter died

The description text in MMseqs2 is indeed wrong. We are planing to rework and improve all help texts and descriptions for the next release.


README, CONTRIBUTING, and LICENSE files now throw INFO if they are not UPPERCASE

Some projects use readme.md or Readme.md - I think this should not be an error state but should throw some kind of indication that the ideal filename would be README.md.
I updated the testing to match, but I'm not super happy with the test changes on RemoteProject. Feels a little sloppy. Can we discuss how I can make them better? Thanks!

#21


[DOC-3089] Deleted topic for CyberArk Secret Manager

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ x] Maintenance/Chore

Deleted topic for CyberArk secret manager from FirstGen.
House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[x ] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app


Add license matching Rust

As discussed in https://github.com/rust-lang/rust/issues/25664 and https://github.com/rust-lang/rfcs/issues/1259 , this repo is basically part of Rust but needs to make its licensing information explicit.

r? @brson

For the record, in theory, everyone who already contributed to the repository needs to agree to the license.

Yeah, we can't just do this.

I'm going to close this. If we think it's worth pursuing now we can discuss relicensing, but it's a mess. With the RFC repo in particular there's also the important question of what license people's comments have since they are a pretty important part of the overall artifact.


Copy Call Stack doesn't work when doing C/C++ debugging with a multithreaded process

VSCode Version: 1.38.1
OS Version: Windows (others?)

Steps to Reproduce:

Use the C/C++ extension to attach to a multithreaded process, break into the debugger, and right click a call stack and choose "Copy Call Stack" and then paste somewhere.

Bug: Copy silently has no effect. It works with TypeScript debugging. I don't think our C/C++ debugger extension is to blame.

Does this issue occur when all extensions are disabled?: Yes

@sean-mcmanus thanks for reporting this. I have pushed a fix. You can try it out in vscode insiders from Thursday and onwards and let us know if the issue is trully fixed for you. Thanks

My verification steps:
Install C++ extension
Add this debug config:
		{
			"name": "(Windows) Attach",
			"type": "cppvsdbg",
			"request": "attach",
			"processId": "${command:pickProcess}"
		},

Launch stable VS Code so that I have something good to attach to.
Run the debug config and choose a Code.exe process
Hit the pause button
Expand a thread in the Call Stack
Right click -> Copy Call Stack
Paste but 🐛 the call stack isn't what pastes, it's instead the last thing from my clipboard.

This should be fixed with latest with https://github.com/microsoft/vscode/pull/82235
@alexr00 since you have nice steps, can you be so kind to try it out?
In case it does work not can you ping me so we investigate? Or you can put a breakpoint here

@alexr00 thanks a lot for the help. I believe i have hunted down the issue.
Argument undefined was getting lost for the unavailbe stack Frames. Now ti should work


Deleting an image doesn't actually remove the image from the filesystem

The \OptimistDigital\NovaSettings\Http\Controllers\SettingsController::deleteImage method doesn't actually delete the image from the filesystem it only sets the setting value to null.
We could force a delete like this example below but it would be better to hook into the Nova Image field instance and delete it that way, does anyone have any idea how to do that? I will file a PR if we can figure it out!
Storage::disk('public')->delete($existingRow->value);

Whoops just realised this is pretty much a dupe of #125 - I had searched for "image" in the issues so not noticed that issue was already open.

Thanks for reporting this! Fixed in version 5.0.3. ;)

@Tarpsvo thanks, would you consider back-porting this to v3 branch too? I could make a PR if you like.

Heya! Released the Nova 3 fix in version 3.5.7.

Thank you @Tarpsvo amazing!


Identify important third-party integrations requiring documentation

To date, our docs only talk about integrating prometheus/grafana/alertmanager for external monitoring. Those docs need to be improved, but we should also identify the most important other third-party integrations requiring some form of mention/tutorial in our documentation.

Lightstep

@knz, @andreimatei, @mberhault, @bdarnell, @nstewart, @dianasaur323, please add any ideas you have.
Also, perhaps we should categorize these (monitoring, sql tools, tracing)?

zipkin
systemd

Closing this in favor of https://github.com/cockroachdb/docs/issues/2979.


Solutions for assignments (robotics and software dev)

It would be good to have at least one copy of solutions for each of the assignments that can pass the unit tests (of course, not stored in a way that learners can access).
The way I've been doing this before is accepting an assignment as a learner, and then adding the solution there.

https://github.com/SissiFeng/solutions_microcourse


Closing SSH terminal after idle period

After the last update, all SSH tabs are closed after an idle period.
I didn't find any timeout config that could fix this.
OS: Windows 10 Pro build 19041.84
Current version: 1.0.103

Looks like it has been fixed with the 1.0.104 release. Closing this issue.


Fix falcosidekick chart linting

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

If this PR will release a new chart version please make sure to also uncomment the following line:

/kind chart-release

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area falco-chart

/area falco-exporter-chart

/area falcosidekick

What this PR does / why we need it:
The previous PR wasn't successful, an error with linting in Chart.yaml crashed the CI before to release.
Which issue(s) this PR fixes:
N/A
Fixes #
Special notes for your reviewer:
Checklist

[ ] Chart Version bumped
[ ] Variables are documented in the README.md
[ ] CHANGELOG.md updated

@Issif: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign nibalizer
You can assign the PR to them by writing /assign @nibalizer in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/assign @leogr

LGTM label has been added.  Git tree hash: 41e84836439028819c1e30d3291c52a1474585a0

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Explictly stating 'from' and 'to' parameters for fade-out

Took me sometime to realize that when declaring a mixin like mui-fade($state: out, $duration: 200ms), one had to also explicitly state $from and $to. Not sure if other animation features have this same issue, but in this case defining $state: out should be enough to execute out.

See https://github.com/zurb/motion-ui/pull/103


#2901 fixed confirmation dialog of approval

Fixes #2901
Signed-off-by: Klaus Strießnig<EMAIL_ADDRESS>

Codecov Report

Merging #3015 (7bf080a) into master (17c08f4) will decrease coverage by 14.42%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           master    #3015       +/-   ##
===========================================
- Coverage   36.32%   21.89%   -14.43%
===========================================
  Files         105       51       -54
  Lines        6762     1215     -5547
  Branches      220      225        +5
===========================================
- Hits         2456      266     -2190
+ Misses       3980      919     -3061
+ Partials      326       30      -296

Flag
Coverage Δ

moduleA
21.89% <0.00%> (-0.22%)
:arrow_down:

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

Impacted Files
Coverage Δ

bridge/client/app/_models/root.ts
0.00% <0.00%> (ø)

bridge/client/app/_models/trace.ts
33.33% <0.00%> (-2.20%)
:arrow_down:

bridge/client/app/_services/data.service.ts
0.87% <0.00%> (-0.01%)
:arrow_down:

helm-service/controller/handler_base.go

helm-service/pkg/mesh/istio.go

shipyard-controller/api/project.go

...ouse-service/event_handler/evaluate_sli_handler.go

helm-service/controller/mock_handler.go

...-service/event_handler/start_evaluation_handler.go

api/handlers/evaluation.go

... and 47 more

Continue to review full report at Codecov.

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

Codecov Report

Merging #3015 (7bf080a) into master (17c08f4) will decrease coverage by 14.42%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           master    #3015       +/-   ##
===========================================
- Coverage   36.32%   21.89%   -14.43%
===========================================
  Files         105       51       -54
  Lines        6762     1215     -5547
  Branches      220      225        +5
===========================================
- Hits         2456      266     -2190
+ Misses       3980      919     -3061
+ Partials      326       30      -296

Flag
Coverage Δ

moduleA
21.89% <0.00%> (-0.22%)
:arrow_down:

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

Impacted Files
Coverage Δ

bridge/client/app/_models/root.ts
0.00% <0.00%> (ø)

bridge/client/app/_models/trace.ts
33.33% <0.00%> (-2.20%)
:arrow_down:

bridge/client/app/_services/data.service.ts
0.87% <0.00%> (-0.01%)
:arrow_down:

helm-service/controller/handler_base.go

helm-service/pkg/mesh/istio.go

shipyard-controller/api/project.go

...ouse-service/event_handler/evaluate_sli_handler.go

helm-service/controller/mock_handler.go

...-service/event_handler/start_evaluation_handler.go

api/handlers/evaluation.go

... and 47 more

Continue to review full report at Codecov.

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


Endpoint crashed exit code: 101

The previous version of the endpoint can run normally, but latest version of compiled endpoint will crash when game starts.
I think it's a rust problem but I can't solve it. Could you please help me find a solution to this problem?

It seems to be MSC related problem.
MSC(script/mahjong_soul.js) is updated in last commit.
Did you update the script in tempermonkey?

Apparently I mistyped some lines in js when I modify the link. My bad, the issue is resolved after I replaced js. Sorry to be disturbing you with this trivial matter.


ask before delete waypoint

before a week I have the situation that I have create a waypoint for a cache and when I want to navigate to it I accidentally hit delete and there wher no way to restore the WP

Every deletion should ask first, there are also other cases (e.g. whole cache).

I'm not a fan of confirming every single action as I typically know what I do and confirming just makes the whole workflow much slower. So if something like "confirm on every deletion" is introduced I really want to have an option to disable it.


chore(JSON-RPC,execution): consolidate repeating test code

Pull Request type
Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Does this introduce a breaking change?

[ ] Yes
[ ] No

Other information

This change is

crates/papyrus_execution/src/objects.rs line 128 at r2 (raw file):

/// The execution trace of an L1Handler transaction.
#[derive(Clone, Debug, Serialize, Deserialize, Eq, PartialEq)]

I'm not sure why reviewable is showing this like a new code - I just added Eq + PartialEq to the derive


Improve prompts in 'ddev clean'

What does this PR do?

Don't ask for "force clean" if we're only removing compiled assets, because force_clean_root doesn't have any effect in that case.
Improve wording of "force clean" prompt.
Add hint on what actually gets cleaned.
Use existing constants when documenting which files are cleaned.

Motivation

I was confused by the fact that ddev clean -c asked me to confirm removing root assets too, while -c is designed to remove compiled assets only.
The "Should we continue?" prompt was misleading, as the command would continue regardless of the answer. What the prompt was really asking was whether root assets should be removed too — hence the change in this PR.
The undifferentiated output ("Cleaning X...") made it unclear what was actually going to happen, even though we passed -c or agreed to remove root assets.

Additional Notes

Example output:

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

Thanks for your reviews @hithwen and @AlexandreYang, updated the PR based on your suggestions. :+1:


Structurise testing

Currently, the tests for GHEtool are somewhat all over the place, with examples, validation, unittesting and model testing all combined.
This issue will try to seperate all these tests, so future versions can be more easily checked and verified.

Optimised GitHub actions and code coverage reports so that the same tests are not run twice.


AttributeError: 'module' object has no attribute 'has'

I just checked out the latest PyZMQ master and installed it from source by running:
python setup.py install --zmq=bundled

in a new virtual environment.  Then, when I import zmq, I get this error:
>>> import zmq
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Users\ACaron\Desktop\develop\pyzmq\site\lib\site-packages\zmq\__init__.py", line 49, in <module>
    from zmq import backend
  File "C:\Users\ACaron\Desktop\develop\pyzmq\site\lib\site-packages\zmq\backend\__init__.py", line 41, in <module>
    reraise(*exc_info)
  File "C:\Users\ACaron\Desktop\develop\pyzmq\site\lib\site-packages\zmq\backend\__init__.py", line 29, in <module>
    _ns = select_backend(first)
  File "C:\Users\ACaron\Desktop\develop\pyzmq\site\lib\site-packages\zmq\backend\select.py", line 38, in select_backend
    ns[key] = getattr(mod, key)
AttributeError: 'module' object has no attribute 'has'

Everything seems to have built fine.  Any idea what's going wrong?

I dug a bit and noticed that the generated .c file for utils.pyx did not contain the definition for the has function.   I reinstalled Python 2.7.9, virtualenv, cython, cleaned everything and the error disappeared...

I think I've seen this before when upgrading pyzmq while it's already in use, and it doesn't successfully overwrite all files (a Windows filesystem problem). Can you pip uninstall pyzmq then pip install pyzmq?

Since I already fixed it be reinstalling everything, it will be hard to confirm whether the uninstall fixes it.  If I run into it again, I'll try to use that instead of reinstalling everything.  Thanks!

Okay, thanks.


Squadra Technologies SecRMM Azure Sentinel Data Connector

Fixes #
Proposed Changes

added file SquadraTechnologies.SecRMM.json to directory DataConnectors.
added file SquadraTechnologiesLogo.svg to directory Logos
added file SquadraTechnologies.SecRMM.SampleData.json to directory "Sample Data"
added file SquadraTechnologiesSecRMM.json to directory Workbooks
modified file WorkbooksMetadata.json in directory Workbooks
added file SquadraTechnologiesLogo.svg to directory Workbooks\Images\Logos
added file SquadraTechnologiesSecRMMBlack.PNG to directory Workbooks\Images\Preview
added file SquadraTechnologiesSecRMMWhite.PNG to directory Workbooks\Images\Preview

 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: anthonylamark sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

Hi Preeti,
OK, I fixed what you requested.
Thanks!
Tony

From: Preeti Krishna<EMAIL_ADDRESS>Sent: Monday, February 10, 2020 12:49 PM
To: Azure/Azure-Sentinel<EMAIL_ADDRESS>Cc: Anthony LaMark<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Azure/Azure-Sentinel] Squadra Technologies SecRMM Azure Sentinel Data Connector (#475)
@preetikr requested changes on this pull request.

In DataConnectors/SquadraTechnologies.SecRMM.jsonhttps://github.com/Azure/Azure-Sentinel/pull/475#discussion_r377298689:

@@ -0,0 +1,119 @@
+{

"id": "SquadraTechnologiesSecRMM",
"title": "Squadra Technologies secRMM",
"publisher": "Squadra Technologies",
"descriptionMarkdown": "Use the Squadra Technologies secRMM Data Connector to push USB removable storage security event data into Azure Sentinel Log Analytics.",
"graphQueries": [
{

Please modify the graphQueries as follows: You can have the current graphQuery mentioned as a sample query. That's because we have code in the connector framework that uses specifically this to add more to and the custom graphQuery you've added breaks that and we don't see the rendering right.
"graphQueries": [
{
"metricName": "Total data received",
"legend": "secRMM_CL",
"baseQuery": "secRMM_CL"
}
],

In DataConnectors/SquadraTechnologies.SecRMM.jsonhttps://github.com/Azure/Azure-Sentinel/pull/475#discussion_r377306509:

{

 "description": "Show mobile devices that are being USB mounted but are not MDM (Microsoft Intune) enrolled",

 "query": "secRMM_CL | where ((Event_s == 'ONLINE') and (AdditionalProgramInfo_s contains 'Mobile device is not MDM enrolled'))"

}
],
"dataTypes": [
{

 "name": "secRMM_CL",

 "lastDataReceivedQuery": "secRMM_CL | summarize Time = max(TimeGenerated) | where isnotempty(Time)"

}
],
"connectivityCriterias": [
{

 "type": "IsConnectedQuery",

 "value": [

   "secRMM_CL | summarize LastLogReceived = max(TimeGenerated) | project IsConnected = LastLogReceived > ago(30d)"

This query is not formed properly. Please modify as follows to include '\n':
"secRMM_CL\n | summarize LastLogReceived = max(TimeGenerated)\n | project IsConnected = LastLogReceived > ago(30d)"

In DataConnectors/SquadraTechnologies.SecRMM.jsonhttps://github.com/Azure/Azure-Sentinel/pull/475#discussion_r377307604:

   "providerDisplayName": "Workspace",

   "scope": "Workspace",

   "requiredPermissions": {

     "write": true,

     "read": true,

     "delete": true

   }

 }

],
"customs": [
]
},
"instructionSteps": [
{

 "title": "",

 "description": "Follow the step-by-step instructions provided in the 'Squadra Technologies configuration guide for Azure Sentinel' at http://www.squadratechnologies.com/StaticContent/ProductDownload/secRMM/<IP_ADDRESS>/secRMMAzureSentinelAdministratorGuide.pdf",

Please make Squadra Technologies configuration guide for Azure Sentinel as a clickable link to the PDF guide and please remove the long URL to the PDF guide. You can remove the quotes from 'Squadra Technologies configuration guide for Azure Sentinel' as well
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Azure/Azure-Sentinel/pull/475?email_source=notifications&email_token=AAYPUWTZGQC5HHFWNWKJPULRCG4U3A5CNFSM4KSDWR4KYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCU54UIQ#pullrequestreview-356239906, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAYPUWVZ5JPZG3YV3LUEWXLRCG4U3ANCNFSM4KSDWR4A.

Hi Preeti,
OK, I fixed what you have requested.
Thanks!
Tony

From: Preeti Krishna<EMAIL_ADDRESS>Sent: Monday, February 10, 2020 12:59 PM
To: Azure/Azure-Sentinel<EMAIL_ADDRESS>Cc: Anthony LaMark<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [Azure/Azure-Sentinel] Squadra Technologies SecRMM Azure Sentinel Data Connector (#475)
@preetikr requested changes on this pull request.

In Workbooks/SquadraTechnologiesSecRMM.jsonhttps://github.com/Azure/Azure-Sentinel/pull/475#discussion_r377311799:

@@ -0,0 +1,160 @@
+{

"version": "Notebook/1.0",
"items": [
{

 "type": 1,

 "content": {

   "json": "## Removable Storage security via Squadra Technologies secRMM\n---\nFile writes to removable storage by User\n\n"

Remove captitalization on Storage - "Removable storage security via Squadra Technologies secRMM"
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/Azure/Azure-Sentinel/pull/475?email_source=notifications&email_token=AAYPUWX6P4A6ENNDZLWE4YLRCG53DA5CNFSM4KSDWR4KYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCU6AZIA#pullrequestreview-356256928, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAYPUWT25ETMAWTEWCOS2MLRCG53DANCNFSM4KSDWR4A.


swift-package-manager comatibility

Since Turnstone is currently not compatible with the Swift Package Manager, it can't currently be integrated with Curassow on platforms where CocoaPods is not available (Linux).
As far as I see nothing but the Podfile and the name of the source files directory would need to change, but I am just getting started with the whole Swift package management thing.

@Ahti This library won't work on Linux until Foundation on Linux get's regex support, so Linux support is blocked until that happens first.

@kylef looks like there is some progress on NSRegularExpression.


Ethernet Link down automatically after some random time when using W5500. (IDFGH-6262)

Environment

Development Kit: ESP32-S2-Kaluga-1
Kit version (for WroverKit/PicoKit/DevKitC): V1.3
Module or chip used: ESP32-S2-Wrover
IDF version: v4.3 (Stable)
Build System: idf.py
Compiler version: xtensa-esp32-elf-gcc (crosstool-NG esp-2020r3) 8.4.0
Operating System: Linux
Using an IDE?: No
Power Supply: external 5V-2A

Problem Description
I am using WIZ850io with ESP32S2 and using sample basic application provided with ESP-IDF with W5500.
Expected Behavior
Ethernet Link should be active all the time. Even if it get down, it should automatically gets connected.
Actual Behavior
Ethernet link is down after some random time. It also get Link Up but it does not get IP. After some debug, I come to know that Ethernet module does not reply to SPI command. I checked that by reading Ethernet MAC address from PHY register. When link was OK, it was giving correct MAC address. When link is down automatically, it stops giving correct MAC address.
Steps to reproduce

Take basic ethernet example and configure it for W5500 using menuconfig
Run the example and keep it running for while. (approx 1-2 hours, if possible overnight)

Code to reproduce this issue
 Same code, no change

Debug Logs
rst:0x1 (POWERON),boot:0xb (SPI_FAST_FLASH_BOOT)
SPIWP:0xee
mode:DIO, clock div:1
load:0x3ffe6100,len:0x18d8
load:0x4004c000,len:0xa04
load:0x40050000,len:0x2b80
entry 0x4004c1f4
I (21) boot: ESP-IDF v4.3.1-dirty 2nd stage bootloader
I (21) boot: compile time 14:08:49
I (21) boot: chip revision: 0
I (25) boot.esp32s2: SPI Speed      : 80MHz
I (29) boot.esp32s2: SPI Mode       : DIO
I (34) boot.esp32s2: SPI Flash Size : 2MB
I (39) boot: Enabling RNG early entropy source...
I (44) boot: Partition Table:
I (48) boot: ## Label            Usage          Type ST Offset   Length
I (55) boot:  0 nvs              WiFi data        01 02 00009000 00006000
I (62) boot:  1 phy_init         RF data          01 01 0000f000 00001000
I (70) boot:  2 factory          factory app      00 00 00010000 00100000
I (77) boot: End of partition table
I (81) esp_image: segment 0: paddr=00010020 vaddr=3f000020 size=106a0h ( 67232) map
I (103) esp_image: segment 1: paddr=000206c8 vaddr=3ffbed50 size=0298ch ( 10636) load
I (106) esp_image: segment 2: paddr=0002305c vaddr=40022000 size=0cd44h ( 52548) load
I (122) esp_image: segment 3: paddr=0002fda8 vaddr=50000000 size=00010h (    16) load
I (122) esp_image: segment 4: paddr=0002fdc0 vaddr=00000000 size=00258h (   600)
I (128) esp_image: segment 5: paddr=00030020 vaddr=40080020 size=2e0d8h (188632) map
I (180) boot: Loaded app from partition at offset 0x10000
I (181) boot: Disabling RNG early entropy source...
I (192) cac��f<��3?�.���,W��
                            �V�,ZY�B��size 8KB, 4Ways, cache line size 32Byte
I (192) cpu_start: Pro cpu up.
I (204) cpu_start: Pro cpu start user code
I (204) cpu_start: cpu freq: 160000000
I (204) cpu_start: Application information:
I (208) cpu_start: Project name:     ethernet_basic
I (214) cpu_start: App version:      v4.3.1-dirty
I (219) cpu_start: Compile time:     Nov 18 2021 14:08:45
I (226) cpu_start: ELF file SHA256:  59c995876fe73212...
I (231) cpu_start: ESP-IDF:          v4.3.1-dirty
I (237) heap_init: Initializing. RAM available for dynamic allocation:
I (244) heap_init: At 3FF9E000 len 00002000 (8 KiB): RTCRAM
I (250) heap_init: At 3FFC2FD8 len 00039028 (228 KiB): DRAM
I (257) heap_init: At 3FFFC000 len 00003A10 (14 KiB): DRAM
I (263) spi_flash: detected chip: generic
I (268) spi_flash: flash io: dio
W (271) spi_flash: Detected size(4096k) larger than the size in the binary image header(2048k). Using the size in the binary image header.
I (288) cpu_start: Starting scheduler on PRO CPU.
I (290) w5500-mac: version=4
I (300) esp_eth.netif.glue: 64:1a:ba:00:00:02
I (300) esp_eth.netif.glue: ethernet attached to netif
I (300) eth_example: Ethernet Started
I (2300) eth_example: Ethernet Link Up
I (2300) eth_example: Ethernet HW Addr 64:1a:ba:00:00:02
I (3290) esp_netif_handlers: eth ip: <IP_ADDRESS>, mask: <IP_ADDRESS>, gw: <IP_ADDRESS>
I (3290) eth_example: Ethernet Got IP Address
I (3290) eth_example: ~~~~~~~~~~~
I (3290) eth_example: ETHIP:<IP_ADDRESS>
I (3300) eth_example: ETHMASK:<IP_ADDRESS>
I (3300) eth_example: ETHGW:<IP_ADDRESS>
I (3310) eth_example: ~~~~~~~~~~~
I (1226300) eth_example: Ethernet Link Down
I (1228300) eth_example: Ethernet Link Up
I (1228300) eth_example: Ethernet HW Addr 64:1a:ba:00:00:02 <This is not read using SPI >
E (1346300) eth_example: Ethernet Lost its IP Address

Hi @nilesh-dryad

IDF version: v4.3 (Stable)

Which commit id are you on? Is it the release branch or v4.3 tag?
Note, that the important fixes to w5500 hadn't been included in v4.3 tag, but were added to v4.3.1
Please see the relevant commit history: https://github.com/espressif/esp-idf/commits/release/v4.3/components/esp_eth and make sure these are included d09f6cac247, 1f2af241188.

@david-cermak I apologies for mentioning incorrect version in Environment section. I was using 4.3 then I switched to 4.3.1 as soon as it was released.
Still issue persists. I am able to reconnect it via following workaround but issue still persists. There is a setting of 120 seconds for getting lost ip handler callback. So I have to wait till 120 seconds to reconnect.
Note: Ethernet Lost IP handler logic I took it from the master branch.

static void lost_ip_event_handler(void *arg, esp_event_base_t event_base,
int32_t event_id, void *event_data)
{
ESP_LOGE(TAG, "Ethernet Lost its IP Address");
esp_eth_stop(eth_handle);
esp_eth_del_netif_glue(ptr);
esp_eth_driver_uninstall(eth_handle);
vTaskDelay(100);
ESP_ERROR_CHECK(esp_eth_driver_install(&config, &eth_handle));
ESP_ERROR_CHECK(esp_netif_attach(eth_netif, esp_eth_new_netif_glue(eth_handle)));
esp_eth_start(eth_handle);
}

@nilesh-dryad May I ask some questions so that I could easily reproduce your problem? I've been running ESP32-S2 + w5500 setup for 7 hours already, the link is up, IP address valid and responding.

How do you connect to w5500, wires? or a custom PCB using the kaluga board?
Any other device on the SPI? What is the frequency?
How do you test it, flood it with packets or just leave it respond to some periodic packets, such as pings or keep it completely silent?

It also get Link Up but it does not get IP
Ethernet module does not reply to SPI command.

So you got the Link up event, but the module didn't respond to SPI?
Anyway, I'll keep running my test over the weekend to see if I'm about to see any link-downs.

I connect w5500 using wires on Kaluga board.
No other device on SPI but we are planning to add other device. Earlier I was testing with other SPI device but issue was there only with Ethernet SPI, other SPI device was working even Ethernet was not working.
20 Mhz Frequency I am using.
I continuously pinging Kaluga board from my PC. But even if I don't ping, it came down after some time.

I am using wires to connect w5500 module in my setup. Can it be a wire problem? I used brand new wires and Ethernet also work for some hours using those wires. See below image.

I also have my custom board with ESP32-S2 and W5500 which I just got yesterday from fabrication. When I started this board I got link down two times but since then Ethernet is running since 15 hours without link break. Ethernet module is mounted on PCB so no wires on this board. Still I am going to put my PCB over the weekend to test the connectivity. See below image.

I think that those wires could easily cause issues in a noisy environment, it not only about frequency, but mainly how the signal (edges) really look like.
I'm running a similar test, continuous pinging, using 36MHz, but connected similarly to your PCB (have a simple breakout board stacking a SAOLA kit to w5500 EVB, running for > 12 hours with no issues)

@david-cermak Ethernet on my new PCB worked without any link down successfully without a single break. It seems like it was cable issue with Kaluga board. I hope yours also worked over the weekend. I am closing this issue as it appears to be cable issue when using module with Kaluga board even though cables were brand new but we can not ignore the fact of noisy environment while using cables.