Support introspection into a VersionReq

It'd be nice to be able to look directly at the set of constraints in a VersionReq. For example, the implementation of RFC 1241 wants to check if a VersionReq corresponds to a wildcard but the implementation is forced to just compare for equality agaist VersionReq::parse("*").unwrap(): https://github.com/rust-lang/cargo/pull/2005/files#diff-e23c793746a7be78139a4e37509a74acR112

+1.  I want to parse a semver and express the implied min/max constraints in a similar constraint syntax (Debian packaging metadata).  It'd be real nice if VersionReq predicates were exposed in some way to prevent me having to duplicate all that logic.

+1.  I want to parse a semver and express the implied min/max constraints in a similar constraint syntax (Debian packaging metadata).  It'd be real nice if VersionReq predicates were exposed in some way to prevent me having to duplicate all that logic.

It looks like semver-parser provides the ability to introspect into its version of VersionReq, but semver has its own version of VersionReq that doesn't allow introspection.
Could semver just use the types from semver-parser, rather than converting to its own?

Could semver just use the types from semver-parser, rather than converting to its own?

I would say yes, but in a slightly different way: they should still convert over, but we should evolve VersionReq to be more useful.
Now that we've fully converted over, I want to take a hard look at the whole interface, looking towards a 1.0. These kinds of changes are the things I might want to make.

@steveklabnik Why the separate types and conversion between them?

They're two separate crates, the conversion means that I don't leak the specific semver-parser into semver's external API.

Is there a strong reason to keep them separate rather than unifying them?

I like small crates, and keeping concerns separate. One crate for one thing; one handles the conversions, the other has all of the logic for comparisons, etc. it feels like the split between -sys and not to me. Plus, there could be other back ends: maybe that nom parser I tried would work better, or LALRPOP.
But! Part of that is the beauty of < 1.0: I'm not actually sure. We'll see.
On Aug 8, 2016, 19:23 -0400, Josh Triplett<EMAIL_ADDRESS>wrote:

Is there a strong reason to keep them separate rather than unifying them?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub (https://github.com/steveklabnik/semver/issues/59#issuecomment-238407836), or mute the thread (https://github.com/notifications/unsubscribe-auth/AABsio9cOyLsza_0So_cP-amaGhkpa0Hks5qd7pbgaJpZM4GfolK).

I don't know how it could be implemented: for now VersionReq is just a bunch of Predicate's. For me it's not clear how to get a min and max constrains.
How can VersionReq be exposed actually?

This is now available in 1.0.0. https://docs.rs/semver/1.0.0-rc.1/semver/struct.VersionReq.html


target:“node-webkit” fails with Uncaught ReferenceError: require is not defined

BUG
What is the current behavior?
target: 'node-webkit' fails with Uncaught ReferenceError: require is not defined
If the current behavior is a bug, please provide the steps to reproduce.
Download the source code here : https://github.com/react-boilerplate/react-boilerplate/releases/tag/v3.4.0
change the line in internals/webpack/webpack.base.babel.js
from

target: 'web', // Make web variables accessible to webpack, e.g. window

to

target: 'node-webkit', // Make web variables accessible to webpack, e.g. window

or

target: 'node', // Make web variables accessible to webpack, e.g. window

$npm install
$npm start
and open http://localhost:3000 in your browser.
What is the expected behavior?
Code runs good with target:'web' config. Expected behavior is not getting the "ReferenceError: require is not defined" error.
Please mention other relevant information such as the browser version, Node.js version, webpack version, and Operating System.
node Version : v9.7.1
Browser: Chrome 64 also the same result with nw.js
Webpack Version :  "2.2.0-rc.3" and "3.0.0"
Operating System : Macos High Sierra (10.13.3)
Here package json

webpack 2 and 3.0 are no longer supported. Please update to version 4.5 or 3.11.
If you still encounter this issue, please open a new issue with a reproducible example.

This happened to me, too with webpack 6.4.1 - for anyone else encountering this issue:
You have to add
"node-remote": "http://localhost/*",

to your package.json file so that files served via the dev server have access to the node and NW.js APIs as well.
If you, like me, are new to all of this and are using typescript as well, don't for get to also
npm install -D @types/webpack-env

and
npm install -D @types/node

to prevent any TS compilation errors. I hope this will save others the headache I had trying to get a setup with NW.js, webpack-dev-server and TypeScript to work.


Add an article explaining  about dropping support for backward compatible OpenSearch

What should we add to our content collections?
(more information is useful here! Were you trying to do something particular?)
Since we will execute Phase 2 of our migration from Elasticsearch to OpenSearch, this article should explain what do we mean by dropping support for backward compatible OpenSearch.
Is this something you would work on yourself?
Yes
[ X] I plan to open a pull request for this

Maybe related to this https://github.com/aiven/devportal/issues/1114


[recnet-api] Move fast-xml-parser to dependencies

Description

move fast-xml-parser to dependencies to fix module not found issue.
Related Issue

N/A
Notes

TODO

[ ] Paste the testing link
[ ] Clear console.log or console.error for debug usage
[ ] Update the documentation recnet-docs if needed
[ ] Version bump in package.json if needed

Wait, the test failed


How to set contant speed

How to apply consistent speed regardless of content length?

Would that answer your question?

Sorry but no answer yet. We have a container which content text change every time. Setting to specic speed sometime make the marquee too slow or too fast so I need a way to make the speed consistent regardless of how long the data was.

In my link the content gets a fixed width and should solve the problem as the speed is now constant.
Alternatively you can create an example?

Sorry for late response, bit busy lately but here is the example of what I mean.
https://jsfiddle.net/jariesdev/vuwxgrfn/
As you notice same configuration but different content. You can see that speed is not consistent both marquee.

Here is an example https://jsfiddle.net/0ufmqw2x
The duration depends on the width, to make the width equal I set a div with the same width. (See my first answer)

Yes, this what i wanted except there should be no white space which
"repeat" option suppose to do right?
On Wed, Sep 30, 2020, 7:53 PM EvodiaAut<EMAIL_ADDRESS>wrote:

Here is an example https://jsfiddle.net/0ufmqw2x
The duration depends on the width, to make the width equal I set a div
with the same width. (See my first answer)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/EvodiaAut/vue-marquee-text-component/issues/19#issuecomment-701341837,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEETTVRQCDUOUDE2NRCHKWLSIML4TANCNFSM4RQBIRNQ
.

In my link the content gets a fixed width and should solve the problem as the speed is now constant.
Alternatively you can create an example?
I am set fixed width that will apply  consistent speed.
But too long word will be multi line.So this issue not realy resolve.
Can help to add feature like as "consisten:true" ??

It is important to be sure that it is a pure CSS based solution.
I do not want to use JS to calculate the width.
The width indicates the speed and can be adjusted with the property duration.


Enable TypeApplications in hlint by default, update for latest haskell-lsp

Fixes #294 and https://github.com/alanz/vscode-hie-server/issues/3


[WIP] fix: enable test case of call_precompile_with_value

Description

Fix to add GAS_STIPEND_CALL_WITH_VALUE (2300) to gas-left if value is non-zero.
Fix to set reversible_write_counter to transfer_rwc_delta (for precompile).

\b run testool

\b run testool

not good.  let's switch to #774

strange.. changing To => Delta is wrong..?

strange.. changing To => Delta is wrong..?

Yes. Seems testool reports more errors.

switch to #774


Production apps should call UseHsts - why?

Do we need to state UseHsts does not guarantee traffic will be redirected to HTTPS?

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

ID: c7ddfb2f-5dea-5e31-2a27-02a4dc16104c
Version Independent ID: 12ca0b9a-2638-a4ce-c45f-d4579c83dafb
Content: Enforce HTTPS in ASP.NET Core
Content Source: aspnetcore/security/enforcing-ssl.md
Product: aspnet-core
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

@Tratcher please advise.

Is there a context for this question?
But yes, HSTS is not guaranteed, it may not be supported by your client. It also requires at least one successful HTTPS request to establish the HSTS policy. You still need a check on every request to handle HTTP requests (redirect, reject, etc). HSTS is a client side performance optimization to avoid an extra request to the server.

The doc is Enforce HTTPS in ASP.NET Core - so I think we should list the limitations.
Should I add the following to the PR on the doc?
HSTS is a client side performance optimization to avoid an extra request to the server. HSTS limitations:

HSTS is not guaranteed to limit traffic to HTTPS only.
HSTS requires at least one successful HTTPS request to establish the HSTS policy.
Application must check every HTTP request and redirect, reject, etc.

UseHttpsRedirection limitations:

Does not guarantee HTTP requests are redirected to HTTPS.
Is less robust than HSTS, but more convenient in development.

Comparing them to each other is a bit odd, an app should have both. UseHttpsRedirection does redirect to https if properly configured. The risk is that the client may have sent sensitive data on that initial request. HSTS will prevent such leaks in the future, but cannot protect the first request.
You missed the part about the client needing to support HSTS.


Create a couchbase provider for sdk 3.0

https://github.com/lukas-krecan/ShedLock/issues/316

Thanks a lot,  looks good. Will take a look at it and release ASAP.

Released as 4.14.0


Beam 8648

Deprecate OutputHints as they are no longer in use.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

Replcement for #10084

cc @dmvk
Reviewed in #10084


Sveltekit demo throws error

Internal server error
500 error is thrown when running demos/sveltekit package using vite dev.
The following error is logged in the browsers console:
ReferenceError: Prism is not defined
    at chunk-6JJXRI76.js?v=5eb5b87e:2:1
handleError @ app.js:16
handle_error @ client.js?v=5eb5b87e:1620
_hydrate @ client.js?v=5eb5b87e:2386
await in _hydrate (async)
start @ client.js?v=5eb5b87e:293
(anonymous) @ (index):2136
Promise.then (async)
(anonymous) @ (index):2135

Live svelte-lexical codebase
Next, I tried to debug the issue with live code.
I could not find instruction on how to do this, so booted up packages/svelte-lexical using vite dev, and rewrote the src/routes/+page.svelte to use the same code as the demos/sveltekit.  Is there a better way to live debug?
Unfortunately, ran into this issue:
Error: Failed to resolve entry for package "svelte-lexical". The package may have incorrect main/module/exports specified in its package.json.

The solution to this was, rewriting MarkdownShortcutPlugin.svelte references to svelte-lexical to use the local copies of getEditor and the ImageNode imports.  Resolving this allowed the live code to work within sveltekit.
npm svelte-lexical
I am unable to get the npm version of the svelte-lexical package working, however.  I don't think it's only because of the issue with MarkdownShortcutPlugin.svelte. The Prism issue seems to resolve with adding the following to the vite config:
ssr: {
	noExternal: [/^(lexical|@lexical\/.*)$/],
}

But, once this resolves the Prism error, the following error occurs;
TypeError: Cannot set properties of undefined (setting 'class-name') at Object.extend (chunk-GB7WFCTK.js?v=ce5c0a11:301:26) at chunk-SSOLOCL7.js?v=ce5c0a11:7:43 at chunk-SSOLOCL7.js?v=ce5c0a11:92:3

Additional note
Another note on sveltekit compatibility: CodeActionMenuPlugin and FloatingLinkEditorPlugin reference document at the top scope, so unable to get either of those plugins working within svelte-lexical local package, or from the npm repository.

Hi @mudcube ,
Prism is not defined
It is working fine with pnpm which I have been using for this project. That's why I didn't come across this issue.
In my testing, the Prism undefined issue happens with npm dev build (production build works fine). The reason seems to be Prism.js package being CJS (not supporting ESM). We need to check if there is any ESM build available for Prism.js to resolve this issue.
Live coding
Regarding live coding, it works with pnpm as the project is set up as a monorepo (You will have run build on svelte-lexical for the changes to be picked up automatically by HMR).
Plugins not supporting SvelteKlt
Yeah, not all plugins support SvelteKlt at the moment. I will fix the one you pointed out on priority.

That you so much for your thoughtful response 😊.
It sounds like PrismJS v2.0 is planned to have ESM support, but no word when that will be. Since PrismJS is included in the Lexical package, if there was an ESM version, would we need to get the commit upstream in Lexical proper to sort out the issue, or do you think there'd be another way around?
Do you happen to have a Patreon, or somewhere to send a thank you for your hard work on this project?

Removed the 'svelte-lexical` references from MarkdownShortcutPlugin, instead using local references. 26e52caf97112fe7950c2905650c1c78ed2777c4

Most of the packages are tested to work with SvelteKit
Prismjs issue with dev build when npm is used as a package manager is resolved by adding npm workspaces


Merge set and set with type

Postprocessing of #27
@mbtools are you ok this ?

Nice. I will adjust the diff tomorrow


Strange behaviour on BLOB field textbox

Hello,
I needed to add field "Work Description" to our new table in app in a same way how it is e.g. on Sales Header table (Field 200). I have added it to table, to page as a field (same as on the Sales Order page) and result is:
A) Regardless the Multiline is set to true, the textbox is displayed as one line
B) If the field is not set as additional, and the page is opened in view mode, it is displayed (with the bug in A). If then the page is switched to edit mode, the field is still uneditable and page stops responding. Client fail.
C) If the field is set as additional and the page is opened, and the "show more" is clicked, the group is displayed but not the text box (when the textbox is inside special Group as on page 42). If clicked "show less" group is hidden again.
It seems that textbox over BLOB field added by extension is not working correctly and cannot be used as it is in C/AL.
Used field definition:
        field(12;"Work Description"; Blob)
        {
            Caption = 'Work Description';
            DataClassification = ToBeClassified;
        }

Used page definition:
                group (WorkDescription) {
                    Caption = 'Work Description';
                    field("Work Description";"Work Description")
                    {
                        MultiLine = true;
                        ShowCaption = false;
                        ToolTip = 'Specifies the products or service being offered.';
                        Importance = Additional;
                    }
                }

I have tried different thing (remove attributes, move field from the group etc.) but I have not find any combination working correctly.

Hi Kine,
It seems that you have directly added the field to the page, you should create a Text variable and then use that in the field.
Can you please try the below changes ? Hope this solves your issue.
group(WorkDescriptionGroup)
{
Caption = 'Work Description';
field(WorkDescription; WorkDescription)
{
ShowCaption = false;
ToolTip = 'Specifies the products or service being offered.';
MultiLine = true;
ApplicationArea = All;
Importance = Additional;
trigger OnValidate();
begin
SetWorkDescription(WorkDescription);
end;
}
}
trigger OnAfterGetRecord();
begin
WorkDescription := GetWorkDescription();
end;
var
    WorkDescription: Text;

True, I overlooked that the source expression is not "Work Description" but "WorkDescription". This solved the problem with adding the field to the page.
But still, failing client just because I have added blob field to the page is not correct behavior... ;-) I can kill BC through this and I think it should prevents me to do that.

Do you still have the same issue ?

Not with the Work Description when I use the variable of course.
But of course when using Blob directly as a Source Expr. of textbox, it will kill the client when trying to have editable textbox... there is no change on this. I will assume that the textbox will be still non-editable (because I cannot edit the value of the blob directly) or ALC will give me error when adding such a textbox to the page.

Hi @kine ! Using a Blob as a source expression is a valid scenario when the PageType is set to API. I think we can implement an analyzer for this scenario that gives you a warning if you are trying to use a Blob as a source expression and the PageType is not API. What do you think?

If it will give me clear info that Blob cannot be used directly on non-API page, than yes (and may be hint that I can use it through variable? or link to some documentation?). No problem with that.


syncer don't work on current layer

Motivation
Closes #2610
Changes

syncer sync up to current-1 layer
syncer doesn't validate a layer that hare plans to

Test Plan
unit tests
DevOps Notes

[x] This PR does not require configuration changes (e.g., environment variables, GitHub secrets, VM resources)
[x] This PR does not affect public APIs
[x] This PR does not rely on a new version of external services (PoET, elasticsearch, etc.)
[x] This PR does not make changes to log messages (which monitoring infrastructure may rely on)

bors try

bors try

bors try

bors try

bors merge


Add Ai Resources for beginners in Sequence

Is your feature request related to a problem? Please describe.
Add resources for Ai made for beginners to make their way into Ai and start making projects,  Please note that while uploading, the thumbnail size should be 320x180 and  structuring should be done according to other resources
For more reference, please visit this link https://github.com/hasnainmakada-99/Open-Source-With-Hasnain/blob/main/pages/devops-resources.md
Describe the solution you'd like
Describe alternatives you've considered
Additional context
https://github.com/hasnainmakada-99/Open-Source-With-Hasnain/blob/main/pages/devops-resources.md
Happy Open Sourcing 😄

@hasnainmakada-99 I am interested in resolving this issue . Please Guide me.

Yes @Abinash-bit , You have to add resources regarding Ai from which beginners can get started with it. If you want to know how to add resources, you can refer this link https://github.com/hasnainmakada-99/Open-Source-With-Hasnain/blob/main/pages/open-source-resources.md
please note that while uploading the thumbnails, set the resoulution size of thumbnails at 320x180.

Hello can i make that tabular format in a readme.md file
On Mon, 2 Jan, 2023, 6:27 pm Hasnain Makada, @.***>
wrote:

Assigned #44
https://github.com/hasnainmakada-99/Open-Source-With-Hasnain/issues/44
to @Abinash-bit https://github.com/Abinash-bit.
—
Reply to this email directly, view it on GitHub
https://github.com/hasnainmakada-99/Open-Source-With-Hasnain/issues/44#event-8137297217,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVMKYDUTEDT7SB6BMHYN2O3WQLGCBANCNFSM6AAAAAATNTQFO4
.
You are receiving this because you were assigned.Message ID:
<hasnainmakada-99/Open-Source-With-Hasnain/issue/44/issue_event/8137297217
@github.com>

Good Evening,
I have some ideas regarding AI. I can update the resource list for the community.
Thanks

Hello can i make that tabular format in a readme.md file
…
On Mon, 2 Jan, 2023, 6:27 pm Hasnain Makada, @.***> wrote: Assigned #44 <#44> to @Abinash-bit https://github.com/Abinash-bit. — Reply to this email directly, view it on GitHub <#44 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AVMKYDUTEDT7SB6BMHYN2O3WQLGCBANCNFSM6AAAAAATNTQFO4 . You are receiving this because you were assigned.Message ID: <hasnainmakada-99/Open-Source-With-Hasnain/issue/44/issue_event/8137297217 @github.com>

Yes you have to make that in tabular format

@Abinash-bit any updates on this ?

@Abinash-bit ?

Sorry my university exams were going on.
On Tue, 17 Jan, 2023, 12:34 pm Hasnain Makada, @.***>
wrote:

@Abinash-bit https://github.com/Abinash-bit ?
—
Reply to this email directly, view it on GitHub
https://github.com/hasnainmakada-99/Open-Source-With-Hasnain/issues/44#issuecomment-1384920472,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVMKYDSRTJ6B4DUU2L4VEJTWSY77LANCNFSM6AAAAAATNTQFO4
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

Please give me time till tomorrow evening.
On Tue, 17 Jan, 2023, 4:57 pm Abinash Mahapatra, <
@.***> wrote:

Sorry my university exams were going on.
On Tue, 17 Jan, 2023, 12:34 pm Hasnain Makada, @.***>
wrote:

@Abinash-bit https://github.com/Abinash-bit ?
—
Reply to this email directly, view it on GitHub
https://github.com/hasnainmakada-99/Open-Source-With-Hasnain/issues/44#issuecomment-1384920472,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVMKYDSRTJ6B4DUU2L4VEJTWSY77LANCNFSM6AAAAAATNTQFO4
.
You are receiving this because you were mentioned.Message ID:
@.***
com>

AI RESOURCES
Prerequsites

Description
Preview
Link to get started with it

1. Introduction to Artificial Intelligence

Here you go

2. Understanding Artificial intelligence

Here you go

3. PDF of the Roadmap

Here you go

Learning Python

Description
Preview
Link to get started with it

1. Python Basics in 2 hours

Here you go

2. Python Basics

Here you go

Learning Python frameworks for Artificial Intelligence.

Description
Preview
Link to get started with it

1. Jupyter Notebooks

Here you go

2. Google colab

Here you go

Learning Algorithms of Artificial Intelligence and Machine learning.

Description
Preview
Link to get started with it

1. Artificial neural network

Here you go

2. Support Vector Machine
 [](https://youtube.com/playlist?list=PLKnIA16_RmvbOIFee-ra7U6jR2oIbCZBL"Support Vector Machine ")
Here you go

3. Decision-making

Here you go

4. Logistic regression

Here you go

5. Linear regression

Here you go

6. Random forest

Here you go

7. Naive Bayes classifier

Here you go

8. K-means clustering

Here you go

9. K-nearest neighbors algorithm

Here you go

10. Unsupervised learning

Here you go

11. Cluster analysis

Here you go

12. Regression analysis

Here you go

13. Semi-supervised learning

Here you go

14. Supervised learning

Here you go

15. Adaboost

Here you go

16. Boosting

Here you go

17. Ensemble learning

Here you go

18. Stochastic gradient descent

Here you go

19. Polynomial regression

Here you go

Deploying ML and AI Models.

Description
Preview
Link to get started with it

1. Jupyter Notebooks

Here you go

Storing files in Git  and    ###Github.

Description
Preview
Link to get started with it

1. Complete Git and GitHub Tutorial

Here you go

2. Git and GitHub crash course (freecodecamp)

Here you go

3. Git for Professionals (freecodecamp)

Here you go

@hasnainmakada-99 I have worked on this issue please check sir.
And i am sorry for my inconvinence.

@Abinash-bit , you have to add these resources inside the Ai-resources.md file, and after adding the resources, please raise the PR for the same


Unable to find many features like covalent/core/loading, covalent/core/media, covalent/core/steps

While trying to upgrade my project from Angular 7 to 12 which used covalent/core, I came across an issue which occured when I upgraded the covalent/core version from 2.1.1(which doesn't support Angular 12) to version 4.0.0 (which supports Angular 12), I found that many features that were present in version 2.1.1 were missing in version 4.0.0. The module like loading, media, steps and many other modules have been removed from the covalent/core versions above 2. Is there a way to access the above mentioned features in Angular 12?

hello @manasa-rangaraj the components you mentioned have been deprecated in since the last major version. You can find the details here. https://github.com/Teradata/covalent/pull/1872


[BUG] Avoid generating illegal surrogates for list query parameters

Describe the bug
There could be an error like this: UnicodeEncodeError: 'utf-8' codec can't encode character '\udf06' in position 17: surrogates not allowed.
To Reproduce
OpenApi schema to reproduce:
      schema:
        type: array
        items:
          type: string

or
      schema:
        type: array
        items:
          type: string
          pattern: "^[^\\x00]*$"

Expected behavior
Avoid illegal surrogates generation for the described case
Environment (please complete the following information):
(generally unimportant)

OS - does not matter
python - does not matter
Schemathesis version 3.12.2
Spec version 3.0.0

Hi @gluhar2006
Thank you for reporting and for a quick fix! :) Indeed, the current version allows such cases

The fix is released in 3.12.3


hack: centralized dag to queue routing

Adding a hack to do centralized DAG-to-queue routing in Airflow scheduler that routes DAGs in the whitelist to py3 queue which is consumed by the sharedairflowworkerpy3 which runs on Python 3.6.
Keeping the DAG-to-queue mapping centralized during py3 migration much easier since we don't need to update the queue field in 1000+ DAGs.

cc: @astahlman

I wonder why you haven't configured the cluster policy. This allows you to make changes to tasks/DAG.
https://airflow.readthedocs.io/en/latest/concepts.html#cluster-policy

Codecov Report

Merging #196 into data-platform-upgrade will decrease coverage by 0.04%.
The diff coverage is 11.11%.

@@                    Coverage Diff                    @@
##           data-platform-upgrade     #196      +/-   ##
=========================================================
- Coverage                  68.63%   68.59%   -0.05%
=========================================================
  Files                        143      143
  Lines                      11411    11420       +9
=========================================================
+ Hits                        7832     7833       +1
- Misses                      3579     3587       +8

Impacted Files
Coverage Δ

airflow/executors/lyft_celery_executor.py
35.89% <11.11%> (-7.44%)
:arrow_down:

Continue to review full report at Codecov.

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

@mik-laj thanks for the suggestion and I think that makes  sense. Did not know you are watching us all the time : )

😂, didn;t know our internal repo has OSS reviewer as well :)


Installs legacy packages?

When I apply this role on Debian Jessie, it will install outdated lxc-docker
and not the docker-engine package, despite this being documented:
https://docs.docker.com/engine/installation/debian/
Thank you and
with best regards

Correct, this is a bug.

So this will be fixed with v0.2.0 release?
With best regards

Yep, will get this going during the weekend.
-Jason

On Jan 15, 2016, at 4:27 PM, strarsis<EMAIL_ADDRESS>wrote:
So this will be fixed with v0.2.0 release?
With best regards
—
Reply to this email directly or view it on GitHub.


ArrayData buffers are inconsistent accross implementations

Note: migrated from original JIRA: https://issues.apache.org/jira/browse/ARROW-12223
ArrayData implementations seems to share close structure fields accross languages, but their usage is not consistent accross implementation.

Example using ListArray's offsets buffer, in C++, Rust and JavaScript implementation:
- C++: offset's buffer is the second buffer (validity bitmap is first buffer, and buffers are laid in a type-dependant way) https://github.com/apache/arrow/blob/master/cpp/src/arrow/array/array_nested.cc#L189
- Rust: offset's buffer is the first buffer (validity bitmap is not part of the collection, and buffers are laid in a type-dependant way) https://github.com/apache/arrow/blob/master/rust/arrow/src/array/array_list.rs#L235
- JavaScript: offset's buffer is the first buffer (they have fixed position) [https://github.com/apache/arrow/blob/8e43f23dcc6a9e630516228f110c48b64d13cec6/js/src/data.ts#L125]

Note that we have the same inconsistency for validity and data buffers.

This is important in my project because I would like to transport buffers list accross technologies, and ArrayData seemed the easiest structure to transport.

I will close this as won't fix since this can be accomplished via C data interface or IPC are the mechanisms to share pointers between implementations.


How to enforce anonymous member name to be defined?

Hi!
I am working on setup which includes dotnet_style_prefer_inferred_anonymous_type_member_names  for asp.net solution.
Here is global problem - team used to abuse anon object in order to return JSON result. Hovewer, I want to enforce (or encourage at least) write anon object properties of JSON repsonse explicitly. Why? Because if source field of anon object has been renamed - the JSON output will be changed, and JS-frontend would be broken and hard to fix.
To be more specific, case like:
// consider datetime as user-defined structure.
return Json(datetimes.Select(x => new
{
    x.Mounth // typo, will break js when refactored
//    Mounth = x.Month // wont break after backend refactoring.
})

Here is the sampes for the case.

editor.config
csharp code
entire solution (big picture)

Environment:
ASP.NET
.NET 4.7.1
VS 2017 15.6.6

@devovercome To clarify, are you saying that dotnet_style_prefer_inferred_anonymous_type_member_names = false:error is not producing the error you're expecting in the following code?
                    var foo = new
		{
			dateTime.Month // this should raise error.
		};

Yes,
but it could be some other setting. For separation of concerns sake.
I really believe it should be separate setting.

@devovercome This sounds like more of a feature request. If so, can you open a request at Roslyn issues instead? It sounds like this is not a documentation issue.


State testing

Just for convenience, here's the Hubble story's state testing branch. This is the companion to cosmicds#145. See the state testing wiki page.

Closing now that state testing functionality has been added to main


Bugfix: Don't log errors for successful createCreep calls

This bug exists because I forgot that createCreep returns the creep name and not the OK return code most functions use.

😢 this removes useful logging information

I don't think spamming names of creeps at error level is useful...

this is intended functionality and incredibly required for my use case

This Pull Request has been merged by GitConsensus.
Vote Totals

Yes
No
Abstain
Voters

5
0
0
5

Vote Breakdown

User
Yes
No
Abstain

tedivm
✔

PostCrafter
✔

anisoptera
✔

daboross
✔

ags131
✔

Vote Results

Criteria
Result

Has Quorum
True

Has Votes
True


Can't run neo-cli in background

@erikzhang provided that it can be done by running screen -S neo ./neo-cli --rpc, but it is not running in background, I can't close the terminal. When I close I close it the process is terminating.
For example, I run:
docker run --rm onitsoft/neo screen -S neo ./neo-cli --rpc
And I get respond:
Must be connected to a terminal.
When I run in interactive mode with -it flag (docker run -it --rm onitsoft/neo screen -S neo ./neo-cli --rpc) it works well, but the terminal stays open and I can't close it without stopping neo-cli.
Any suggestions?

Ctrl + A + D to detach screen session
screen -r neo to resume screen session
;)

@NeoEscrow Thanks! Is it possible to run it automatically when you run docker-compose up? I mean to add command in docker-compose.yml, so the container goes to detach mode.

I just ran it by adding stdin_open: true and tty: true to my docker-compose.yml file and when ran it clicked Ctrl + Z.

Is it possible to run it automatically when you run docker-compose up? I mean to add command in docker-compose.yml, so the container goes to detach mode.

docker-compose up -d


Gray Image trainning

I want to train the model with the y channel.
It means using gray image to train model.
so, how should I do?
I have already to convert my dataset into gray image,but when I trained the model, it has so many problom.
Is there a simple way to solve it?

Hi @VincentZhao1992
What error are you getting? Typically, most of the models are pretrained with RGB images, after you load your gray image, you may need to change it to 3-channel images.


Updates to Coraza v3.2.1 and several dependencies

Updates to Coraza v3.2
Udates tinygo ci and min version to 0.31.2 (aligned with the tinygo tests run in Coraza upstream)
CI: Updates envoy images
CI: sets fail-fast: false. Tests are flaky and making the two independent will help reduce the number of re-tries to have the PRs green.

This PR also unblocks https://github.com/corazawaf/coraza-proxy-wasm/pull/263
~TODO: Update coraza-wasilibs dependency once https://github.com/corazawaf/coraza-wasilibs/pull/17 is merged and a new version is tagged~ ✅

Note: I reverted tinygo update. Taking a quick look, because of https://github.com/tinygo-org/tinygo/pull/4175, looks like that we need to also upgrade Go to 1.21 to make it work

Update: I managed to upgrade to TinyGo 0.31.2 and left some comments about the next update steps


Can SwiftGit2 support building for swift package manager?

It doesn't currently, but a PR to add support would be very welcome!

I’m interested in this too! I’m too new to SPM to figure out how to get cmake and other bits working there, haha

This should be closed in favor of #172.


Can't use calendar on fields with null value

System information:

Operating system (distribution) and version Win10
DBeaver version 22.0.3

Steps to reproduce, if exist:

Open open a table with a date/time column
Click on the field with null value
Make sure that value panel has text presentation
Change value panel to calendar

Expected: Calendar shows default value or at least opens with blank fields
Actual: Error appears

Hint about current time doesn't appear when switching from text value


Breakpoint mode code questions?

Hi, thank you for the great work! I would appreciate if you could address some of my questions. For the breakpoint mode, from the code below,

The num_frames is the number of segments before the current second, and the cur_frame is the frame index in the current segment. Do I understand it correctly?
It seems that only the video fragments before the queried time are considered in the long-term memory. Is this true? If so, should it actually incorporate all fragments in the video?
Why when doing self.model.encode_short_memory_frame(video_fragment, cur_frame) in line 271, we only consider the first cur_frame frames of each segment? Should it only applies to the last segment? In other words, for the segments before the last segment, should we adopt line 273 which considers all frames in the segment?

Also, the code below is a bug which could cause infinite loop because poping does not affect cur_short_length value. Would you pls fix it?

Your understand of num_frames and cur_frame is correct.
Only the video fragments before the queried time are considered in the long-term memory, and we just incorporate fragments before the queried time.
Sorry, there is something wrong with the judgment condition on line 270. It should actually be if middle_video and (i+1)==num_frames:. That means that when the last fragment is read, it stops when cur_frame is read, instead of stopping when every fragment reads cur_frame. I will update this question in the next version.
Thanks for the reminder, I've noticed this issue. The complete code is still being compiled. You can modify the code on lines 332 to 333 as follows:

while (len(self.long_memory_buffer)+len(self.temp_short_memory)+1) > frame_position_embeddings.shape[0]:
    if len(self.temp_short_memory) != 0:
        self.temp_short_memory.pop(0)
    else:
        self.long_memory_buffer.pop(0)


Lens API docs are out-of-sync

The Followed event is documented here despite not being present in any code in main. Are the docs out of sync?
AFAICS it is only implemented in this feature branch here - https://github.com/aave/lens-protocol/pull/16.

Well spotted! We removed this event because we already have the FollowNFTTransferred event, so it's redundant. Removing from the docs now!


Clumsy attempt to add subscript support

This PR adds support for subscripts, using a tilde notation equivalent to the superscript ^:

N~2 --> N2
H~(2)O) --> H2O

For now superscripts and subscripts are activated together by the --superscript option.
The problem is that --superscript now disables --strikethrough, which also uses tilde (e.g., ~~foo~~ --> ~foo~).
I'm worthless with C code, so I couldn't find a way to fix --strikethrough while preserving subscripts. Any help would be much appreciated, even if it's not merged here (I'd be fine with building my own tweaked version of MacDown for personal use).

Could you explain a bit the rationale of choosing ~ for this? It feels slightly foreign to me; LaTeX, which also uses ^ for superscript, uses _ instead. I am not sure what other Markdown renderers tend to use (IIRC CommonMark does not support either).


Readme for assistants should include creation of testdb database, my-todos container, and id for partition key

In the Readme.md for assistants, https://github.com/Azure/azure-functions-openai-extension/blob/main/samples/assistant/README.md#prerequisites, we should add instructions to create the testdb database, my-todos container, and id for partition key.

@eamonoreilly , Are we good to close this issue now?
I added the auto creation of db and container logic to nodejs sample as well.


ALTER TABLE issue on $db->query()

Hey there,
I tried to change a column type from INT to VARCHAR and I used this command, but I got no result/error.
$db->query("ALTER TABLE tablename CHANGE balance balance VARCHAR(300) NOT NULL;");
Would you please help me out?
Thanks.

Make sure your query is correct.
/* MySQL */
ALTER TABLE table_name RENAME COLUMN old_name TO new_name;
/* MariaDB */
ALTER TABLE table_name CHANGE COLUMN old_name TO new_name;

The above answer is supported on MySQL 8.0.
Just make sure the database can execute the query successfully based on the version. Because query() is just executing the query alone for the database.


Support for the official mimetype

tl;dr: text/markdown since March 2016
In March 2016, text/markdown was registered as RFC7763 at IETF.
Previously, it should have been text/x-markdown. The text below describes the situation before March 2016, when RFC7763 was still a draft.

There is no official recommendation on Gruber’s definition, but the topic was discussed quite heavily on the official mailing-list, and reached the choice of text/x-markdown.
This conclusion was challenged later, has been confirmed and can be, IMO, considered consensus.
This is the only logical conclusion in the lack of an official mime type: text/ will provide proper default almost everywhere, x- because we're not using an official type, markdown and not gruber. or whatever because the type is now so common.
There are still unknowns regarding the different “flavors” of Markdown, though. I guess someone should register an official type, which is supposedly easy, but I doubt anyone dares do it beyond John Gruber, as he very recently proved his attachment to Markdown.
There is a draft on the IETF for text/markdown, but the contents do not seem to describe Markdown at all, so I wouldn't use it until it gets more complete.

Merged as d096403f


LowercaseStaticReferenceFixer - Fix invalid PHP version in example

The PHP version in LowercaseStaticReferenceFixer is invalid, probably should be 70100.

@kubawerlos Done.

Thank you @HypeMC.


[MM-30183] - No more text decoration on hover and focus

Summary

No more text decoration on hover and focus on mobile webb app view
Ticket Link

https://mattermost.atlassian.net/browse/MM-30183
Screenshots

@hahmadia I saw discussion on this in the Developers:Webapp channel, but can you help confirm that this is not a recent regression?

Yo @amyblais 🌞 !
So I checked the blame of the files and the CSS class post_permalink has been applied forever and it hasn't changed since 3 years which leads me to the fact that this is not a recent regression and has been existing forever. Somehow, it was discovered only recently through regression testing.
QA can double check me on this but I'm confident this is not a regression and has been there for a long time. Just not something anyone has probably noticed.

/update-branch


Added support for custom CSS properties in the StyleWrapper

This is a pattern I use lately, and it makes a lot of sense to me.
A couple of questions:

Would it be useful/needed to also inject a className in order to hook to? I was about to do it, but I don't know if that would be required... maybe to know if a value is set or not... but you can always have a default fallback value.
Naming... what not! I chose CSSProperty as the sufix... any other suggestions?

I'm well aware that this opens the door to fine grained style customizations, if we extend the pattern not only to use custom CSS properties but also use random CSS properties (another converter should be created, then do not append the -- and disable nested support, then) I leave it up to the community. However, the initial feeling when we came up with the style wrapper stands: I wouldn't like the wrapper to be missunderstood, and that people are able to easy create blocks that resemble WordPress Elementor...
Let's iterate over this, in the meanwhile, I will work on integrate the PoC into VLT.
Update 01/01/2024: Answer to questions:

It's not really needed, since one could build a selector like this:

 .block.teaser[style*='--background-color'] {
    padding: 20px 0;
}

if required.

This PR is not using a suffix discriminator to differentiate from the others. Now it detects if the property starts by -- see the documentation.

The slope is steep and full of grease. Pretty soon we'll end up here: https://raw.githubusercontent.com/eea/volto-block-style/master/docs/screenshot.png
When I started volto-block-style I had the same intention: let me create a way to centralize block "themes" and make it possible to assign those. That's why the "Style presets" widget is placed first in that volto-block-style sidebar. Pretty soon it devolved into whatever this is: https://github.com/eea/volto-block-style/blob/1fd970881ca1d816f8db764cdb2498348f10f191/src/StyleWrapper/schema.js#L3

@tiberiuichim I know... it's not easy. It's very hard to predict which techniques and patterns are the best for the use case.
When I first pushed for the wrapper I had one thing clear: The intention was not to provide an easy way to plague Volto blocks settings with Elementor-like controls... because it's not what we are aiming for in Volto. volto-style-block was a bit of that. It's absolutely legit if you have a client that explicitly asks for this, and it's editors are real power users who knows how the web works and are capable of handling this complexity. So not enabling doing it in an easy way was important for me.
This is something in between, still with a clear purpose, and documented so people can know how to take advantage of it. A width value, a color, so you can then take it in the CSS and use it right away, one shoot only CSS.
As I said, this opens the pandora's box by perverting it could be used to really inject arbitrary CSS properties. If you really wanted to do it, you can, but on your own. Although, given the power of using custom CSS properties instead, who wants this nowadays?

These two sentences got missed again in the review. I'll create a separate PR to fix it.


Compatibility for Raknetify

https://modrinth.com/plugin/raknetify
Adding the prefix "raknet;" or "raknetl;" to the server address ends with an unknown location.

Doesn't hurt to add compatibility, I'll release an update sometime soon :)

Sounds great - thank you!

Done and uploaded!


Insert Cards

https://getbootstrap.com/docs/5.3/components/card/#header-and-footer
Maybe this one? Can have search history in the footer of the card, have the heading in the header of the card, and render the dynamic content in main card body.


Improve date deserialization

We now expect dates to be in format iso8601, but the default iOS implementation fails to decode various formats that are valid.
we should do a custom deserialization where we try to deserialize and if the deserialization fails, we should try custom deserialization.
We can inspire here: https://stackoverflow.com/questions/46537790/iso8601-date-json-decoding-using-swift4

Code link: https://github.com/wultra/networking-apple/blob/develop/Sources/WultraPowerauthNetworking/WPNHttpRequest.swift#L21


[Suggestion] Place first log on opposite side

Currently, the first log is always placed on the east, and the second is always placed on the west. For the second layer, towards the north first then the south second.
If possible, I'd like to be able to place the first log of each layer in the direction I am looking, mainly for improved usage as decoration instead of utility/immersion.

I have this railway build I used to test out the mod. In it, I use the second-layer log to provide more coherence with the slab. Unfortunately with the way the mod functions, I can not place the first log facing the opposite direction. Not sure what it would take to make this change possible, but I hope it could be considered.

Thank you for pointing this out, I will update and publish it right now.


Fix category map and link tags, remove unused card component, and add final eligibility project status

This pull request includes several fixes and improvements:

Fixed the category map when clicking on the explore page.

Fixed the hydration link of the link tag for the RetroPGF1 card.

Removed an unused card component.

Added the final eligibility project status for RetroPGF4 applications.

Summary by CodeRabbit

New Features

Introduced static routes for different sections like RetroPGF2, RetroPGF3, and RetroPGF4 for improved navigation.

Bug Fixes

Replaced the ChevronRight icon with a text element "See all" in CardRound1 to fix a display issue.

Refactor

Simplified and standardized link paths and eligibility checks across various components.
Reformatted and cleaned up code for better readability and performance.

Documentation

Updated function signatures and import statements for clarity and consistency.

Style

Improved code formatting and structure for enhanced maintainability.

@Billy19191 Next time pls delete unused branch after merge 👍


Updating with 1.6.2-cs5 notes

Signed-off-by: Mary Anthony<EMAIL_ADDRESS>

ping @docker/docs-owners

LGTM

LGTM author


Tests run on GPUs, added log10, log2, rint, fix

[x] operators test on GPUs
[x] log10
[x] log2
[x] rint
[x] fix

please rebase the change against latest nnvm


Some golint fixes

Noticed some golint errors, cleaned them up. There are more, can try to go through them as well.

Thanks. Doc fixes are always useful


The explanation of the paramter "parameters" of the function ActivatorUtilities.CreateInstance is inaccurate.

The document said:

parameters
Object[]
Constructor arguments not provided by the provider.

This is inaccurate because according to the source code (disassembled from .NET Core v2.1.1) of the function CreateInstace(IServiceProvider, Type, object[]):
	ConstructorMatcher constructorMatcher2 = new ConstructorMatcher(item);
	bool flag2 = item.IsDefined(typeof(ActivatorUtilitiesConstructorAttribute), false);
	int num2 = constructorMatcher2.Match(parameters);
...
       return constructorMatcher.CreateInstance(provider);

And the source code of the function ConstructorMatcher.Match(object[]) and the ConstructorMatcher.CreateInstance, it is quite obvious that the Match will assign the arguments with the values from the parameters first if they exist and match the types in the Match method. And then it instantiates the parameters not given in the matching stage by looking up the service provider.
However, the document suggests it will not take the value from the parameter "parameters" unless the type of any argument is not registered in the service provider.
The correct description should be something like this:
The constructor arguments. It will override the ones provided by the provider.

Making this repository Ready Only.


[5.5] null_if() helper function

I have made a pull request for this before here: https://github.com/laravel/framework/pull/20785 and was declined because I made so close to 5.5 launch and Taylor didn't want any new features in yet.
Here's the old description:
I've found it useful on occasion to have a null_if() function, which works in the same way as MySQL's NULLIF(). My main usage was for nullifying invalid dates (mostly '0000-00-00' to null)

No plans to add this to core. You can add to your own app if needed.


Executor/aggregation: refactor StreamAggExecutor and HashAggExecutor

What have you changed? (mandatory)
This PR is based on https://github.com/pingcap/tikv/pull/3359. It refactors StreamAggExecutor and HashAggExecutor, define an inner struct AggExecutor for them to process the repeated logic before.
It's also prepared for return arrow chunk since the field type of return columns in the above aggregation executors share the same logic.
What are the type of the changes? (mandatory)

Improvement (non-breaking change which is an improvement to an existing feature)

How has this PR been tested? (mandatory)
unit tests
Does this PR affect documentation (docs/docs-cn) update? (mandatory)
no
Does this PR affect tidb-ansible update? (mandatory)
no
Refer to a related PR or issue link (optional)
https://github.com/pingcap/tikv/pull/3359
@hicqu @breeswish PTAL

/run-all-tests

/rebuild

/rebuild


For ST-Link v3, read the version from the correct byte and then ignore it

Signed-off-by: Daniel Egger<EMAIL_ADDRESS>

LGTM. Thanks :)


Add 2 games to the README

Add Wormo and Ripple Effect Puzzle games to the Readme.

@impulse do you intend to update this PR?

Sorry tagged the wrong person, @tommybuonomo is the correct one of course.

do you intend to update this PR?


In Angular Service, any way to get the same functionality from onTranslationChange?

In previous Tolgee release 4.X have a method onTranslationChange
onTranslationChange
EventEmmitter<TranslationData> emitting when translation is changed.

type TranslationData
{
    key: string; // the key which was changed
    translations: { // new translations for languages
        [key: string]: string;
    };
}

e.g.

{
    key: "i_am_a_key"
    translations: {
      "en": "English translation",
      "de": "German translation"
    };
}

But this method does not exist in the new version and I don't see any similar one in the API.
Is it possible to get the same functionality or some workaround?

Hey, this is done differently in the new version. You can achieve a similar result with on('cache', (e) => ...) handler which is called every time something in the cache changes and the descriptor will tell you which part of a cache has been changed (e.g. { language: 'en', namespace: '', key: 'test_key' }). The key property is there only if an individual translation was changed, which is the same case that you are describing above, I think.
If you want to get an actual value of the changed translation, you can read it from the cache like this:
tolgee.on('cache', ({ value }) => {
  if (value.key) {
    console.log(tolgee.getRecord(value)?.get(value.key));
  }
});

The event system was changed in v5, because before there were two kinds of events for languageChange and keyChange and we always had to subscribe for both of them in order to keep translations updated and it still was not reflecting all the usecases very well.

Thanks, I'll try then to use the events to get the changes.


Multiple FE Sleeper awakening issues?

A bug was reported in the sleeper edition (https://steamcommunity.com/workshop/filedetails/discussion/2036461392/2146469724197004978/) where subsequent Fallen Empires could not be awakened after the first one was triggered. This is apparently an ongoing/inherited issue from the previous version of Crisis Manager?
Confirm/reproduce this bug, and fix it if possible.

I've looked in to his save. The only bug there is, you can choose the option for WiH when no requirements are valid. In fact there was only 1 valid FE.  And also with good will there was also no valid AE, as the 3 other FE were as follows: 1 Machine, 2 already dead without army (there is in fact a trigger for this) and 3rd AE in war. So I'll simply extend the validity trigger for this option.

Should be fixed with the new version (even it was a 50:50 chance, as the first FE (SleeperA)  had no navy but gets some on awaken): https://steamcommunity.com/sharedfiles/filedetails/?id=2577705378


[enhancement] Export log to file

It would be great to be able to export current log into file.
Or even maybe combine it with filter, and export only filtered log.

Hey @samuraii, sorry by the late answer, the past week was a madness! i just release the 1.1.0 version, with filters and exporting features.. thanks for suggestions!


Struggling with SSL setup

Do you have any tutorials about how to properly create certificates for a TLS setup on VerneMQ? Unfortunately I am struggling with the TLS setup and I only succeeded connecting to VerneMQ using TLS when I do a port-forward from my kubernetes pod to localhost. If I try to connect to the cluster using the ingress hostname or loadbalancer IP I always get ECONREFUSED.
I am not sure why this is happening, I am convinced by LoadBalancer is properly configured as I was able to access the metrics endpoint for instance.
Apparently VerneMQ doesn't show any logs for failed handshakes / connection attempts with invalid certs or similiar.

I'm afraid we don't have any tutorials on that. Can you get it to work on a local setup without a load balancer etc - with less moving parts? That way you may be able to draw some conclusions where the problem may be.
There's also a lot of hints in the closed issues here on github and you're of course also welcome to ask on our slack forum http://slack-invite.vernemq.com/

I would like to reopen this issue. I have been struggling for some time now to get a simple SSL listener set up with the VerneMQ test certificates. It would help a lot of people out if you were able to provide a simple working example of how to set up SSL.

Hi @nickfloop allright, thanks!
For a simple configuration with our test certificates, the following lines in you vernemq.conf should be enough. If this works, you can then configure with your own certs and find out where any differences might be in case your own certs don't work.
listener.ssl.default = <IP_ADDRESS>:8883

listener.ssl.cafile = /etc/vernemq/ssl/all-ca.crt
listener.ssl.certfile = /etc/vernemq/ssl/server.crt
listener.ssl.keyfile = /etc/vernemq/ssl/server.key

Do you see any specific errors or behaviours?

I have opened a new issue and put together my own SSL example as a Github repo in order to demonstrate the issues I am having. Please see here: https://github.com/erlio/vernemq/issues/856 . Thanks so much for your quick reply!


custom_xxx_args ignored?

Hiyas
My SnapFile contains the two lines:
custom_run_args "ch.abacus.abaclik.demo YES"
custom_args "GCC_PREPROCESSOR_DEFINITIONS='$(inherited) SCREENSHOTS=1'"
but, for whatever reasons, when I do a snapshot, neither is SCREENSHOTS defined NOR is the NSUserDefaults property set when I read it. Is there maybe a sample or so on how to use this properly? I'm quiet sure that the issue is somewhere in my code / files, but I can't figure out where.
Best regards & thanks
Rog

Nevermind.. found it.. my bad..

What was the issue?

I think it's because he used "custom_args" instead of "custom_build_args"
The description can be confusing:
"Use the custom_args directive to prepend custom statements to the build command."

I used custom_run_args but I forgot the dash at the beginning of the value. I think I tried to use custom_build_args, but as skrew mentioned, it might be possible that I've totally misread, I'm not sure. I solved it using custom_run_args so I closed the issue again. Thanks anyway, I'm currently only using snapshot, but it's already real timesaver!


updated dependencies

Updated vulnerable dependencies

@benhayehudi @mheap Looks like this has upgraded the nexmo-ruby gem from v4.6.0 to v5.5.0 without updating any of the nexmo-ruby method calls (see #1)


Add a basic keyboard shortcuts overview dialog

Is your proposal related to a problem?
There is a longer term desire to add a fully fledged keyboard shortcuts dialog, with the ability to be customised and potentially support 'quick type' actions throughout the admin. See https://github.com/wagtail/wagtail/issues/3949
However, we have discussed with some other core team and agreed we should start with something simple first.
This issue is to set up a very basic keyboard shortcuts overview dialog that's accessible within the Wagtail admin menu's help section.
Describe the solution you'd like
There should be a simple clickable button in the help section of the admin menu that allows the user to access a new dialog. It should have the label 'Shortcuts' and use the regex icon for now, we can enhance this later.

Inside the dialog, it should have a basic set of shortcut groups as per the schedule below.

Section
Label
Windows
MacOS (if different)

Actions
Save
Ctrl + s
⌘ + s

Actions
Preview
Ctrl + p
⌘ + p

Common actions
Copy
Ctrl + c
⌘ + c

Common actions
Cut
Ctrl + c
⌘ + c

Common actions
Paste
Ctrl + c
⌘ + c

Common actions
Paste without formatting
Ctrl + c
⌘ + c

Common actions
Undo
Ctrl + z
⌘ + z

Common actions
Redo
Ctrl + shift + c
⌘ + shift + c

Text formatting
Bold
Ctrl + b
⌘ + b

Text formatting
Italic
Ctrl + i
⌘ + i

Text formatting
Underline
Ctrl + u
⌘ + u

Text formatting
Monospace (code)
Ctrl + j
⌘ + j

Text formatting
Strike-through
Ctrl + shift + x
⌘ + shift + x

Text formatting
Superscript
Ctrl + .
⌘ + .

Text formatting
Subscript
Ctrl + ,
⌘ + ,

Text content
Insert or edit a link
Ctrl + k
⌘ + k

Text content
Insert horizontal line
---

(Also add other text content; blockquote, code block apply heading style, numbered list, bullet list)
Mockup

Beyond that we want to keep things simple, we may need to review the keyboard shortcuts on the PR and whoever picks this up should manually test them also.
Finally, we will want a way to show the meta key differently on MacOS-like devices or Windows-like.
Proposed implementation
1. Add a new menu item, something like this should be a good starting point.
wagtail/admin/wagtail_hooks.py

@hooks.register("register_help_menu_item")
def register_keyboard_shortcuts_menu_item():
    return MenuItem(
        _("Shortcuts"),
        icon_name="regex",
        order=1200,
        attrs={
            "role": "button",
            "data-a11y-dialog-show": "keyboard-shortcuts-dialog",
            "data-action": "w-action#noop:prevent:stop",
            "data-controller": "w-action",
        },
        name="keyboard-shortcuts-trigger",
        url="#",
    )

# add before @hooks.register("register_admin_menu_item")

2. Add a new template tag to link to a new shared template include
Something like below, I have not fully tested but this should be an OK starting point to pass the meta key contextually to the template.
Feel free to approach this differently.
wagtail/admin/templatetags/wagtailadmin_tags.py

@register.inclusion_tag("wagtailadmin/shared/keyboard_shortcuts_dialog.html", takes_context=True)
def keyboard_shortcuts_dialog(context):
    user_agent = context["request"].headers.get("User-Agent", "")
    is_mac = re.search(r"Mac|iPod|iPhone|iPad", user_agent)
    return {
        "meta": "⌘" if is_mac else "Ctrl",
    }

# add before @register.inclusion_tag("wagtailadmin/shared/human_readable_date.html")

3. Add the new template tag to the admin template
wagtail/admin/templates/wagtailadmin/base.html
{% extends "wagtailadmin/admin_base.html" %}
{% load wagtailadmin_tags wagtailcore_tags i18n %}

{% block furniture %}
    <template data-wagtail-sidebar-branding-logo>{% block branding_logo %}{% endblock %}</template>
    {% sidebar_props %}

    {% keyboard_shortcuts_dialog %}
# ...

4. The detail part, build out the initial keyboard shortcuts template content
Below is a basic starting point but it has no styles and we may want to be smarter about generating the content. It's OK if the PR for this issue has lots of repeated HTML, we want to start simple and then refine the content as we build more features.
We can probably use the tailwind classes to build out most of the styling but feel fre to add a standalone stylesheet within client/scss/components if needed.
wagtail/admin/templates/wagtailadmin/shared/keyboard_shortcuts_dialog.html
{% load wagtailadmin_tags %}
{% dialog icon_name='regex' id='keyboard-shortcuts-dialog' title=_("Keyboard shortcuts") %}

            All keyboard shortcuts

                Section
                Keyboard shortcut
                Keyboard shortcut

                Actions

              Save
              {{ meta }} + s

              Preview
              {{ meta }} + p

                Common actions

              Copy
              {{ meta }} + c

              Cut
              {{ meta }} + x

                Paste
                {{ meta }} + v

{% enddialog %}

5. Unit tests
It's important there is a basic unit test for the dialog content showing in the admin template and the trigger in the menu.
Mainly to ensure that the ids of the dialog and the data attributes on the menu trigger match and both are being rendered.
Testing the menu content is a bit tricky as it's all JSON (to render the React sidebar), but there should be some tests for the dismissable 'what's new in Wagtail' attributes and we can just do something similar in new tests.
Describe alternatives you've considered

We could do nothing, but this kind of feature has been a long requested one and it's also incredibly useful for ATAG compliance (an accessible content management system).
We could try to solve all the desired features in one PR such as a new keyboard shortcut to trigger the modal and a standalone page but for now let's start small.

Additional context

The keyboard shortcuts suggested above and in the Figma need to be validated, it's OK that some of those shortcuts will not work everywhere (e.g. Cmd + p is for previewing). We may look at future enhancements where context specific shortcuts can be shown but that can come later.
Be sure to review the Figma designs for the long term goal here https://www.figma.com/proto/h67EsVXdWsfu38WGGxWfpi/Wagtail-CMS?type=design&node-id=14420-153315&viewport=6289%2C-12853%2C1&t=Sk9k7iALDZCA1bu7-0&scaling=scale-down-width&starting-point-node-id=14420%3A153315 (note that we will not be able to do that with the current base dialog but for now it's best to keep the PR simple).
https://github.com/wagtail/wagtail/issues/10167 - This is a related issue where we want to make it easier to add keyboard shortcuts within Wagtail's admin with Stimulus. Once that issue is resolved, we can look at adding more keyboard shortcuts and then documenting them in this dialog.

Working on this

Anyone can contribute to this.
The aim of this issue is to be a really simple starting point, anyone with basic knowledge of the Django template system and a willingness to do a bit of manual HTML writing and checking of keyboard shortcuts should be able to do.
Once this issue is covered, there will be much more to build on in future PRs as we work towards the final goal laid out (mostly) in https://github.com/wagtail/wagtail/issues/3949
View our contributing guidelines, add a comment to the issue once you’re ready to start.

I will not label this a good first issue but this is a great second or third issue for someone to pick up. We have a clear goal with an implementation proposal and small scope. This involves mostly usage of Django templates and Wagtail's own hooks system.
@thibaudcolas @laymonage  - feel free to edit, we want to keep the scope as small as possible though. We can build on this, as discussed, with more enhancements such as a better icon, activation of this dialog with a separate keyboard shortcuts and maybe review how this content can be customised per view but let's keep future enhancements listed on https://github.com/wagtail/wagtail/issues/3949

@lb- Can I work on this?

Go for it, I think you messaged on Slack right (checking your the same person :) ) @kituuu

Go for it, I think you messaged on Slack right (checking your the same person :) ) @kituuu

Thanks, I'm the same person 🙃

Hey @lb-

Basic implementation is done so anyone can work on this in the future. Should I add all the shortcuts you mentioned above?

@kituuu yes please, try to add the sections/keyboard shortcuts described in the issue.
However, I have not manually tested all these in Wagtail (specifically the text editor shortcuts). For example, I think the ctrl/CMD+K may not work yet.
The generic browser/os shortcuts like cut and paste should be good to go.
It would be great to also ensure there is some styling included. To align with the designs for the content within the dialog. Such as the table row heights, section title text, border under rows.

Resolved via https://github.com/wagtail/wagtail/pull/11717


Is it possible to have different return types on a single call?

Is it possible to return different model types on one response?  For instance if we have a "GeneratePet" call that takes "animal" as a parameter.
if cat, return CatModel
if dog, return DogModel
perhaps something like
@Success 201 {object} CatModel DogModel
Thank you.

I'm afraid not


When master receive a SDP offer from viewer, ICE failed from master.

Hi, I have testing my application using KVS signaling channel. And code are referred as same code here. When I create master using firefox browser and viewr with chrome then I test that a viewer connects and disconnects while a master is connected. After some number of success or may be sometime, suddenly firefox browser of the master display below error.
--- here is viewr's console log ---
[VIEWER] Creating SDP offer
...
[VIEWER] All ICE candidates have been generated
--- here is master's console log ---
[MASTER] Creating SDP answer
[MASTER] Sending SDP answer
...
[MASTER] All ICE candidates have been generated for client
WebRTC: ICE failed, your TURN server appears to be broken, see about:webrtc for more details
I think this error is related with ICE failure, so I want to know why master encounter this problem.
Could you take a look for debug? I attach the about:webrtc log for your help.
Firefox_ICEfailed.log

Closing due to inactivity.


Bad Autocomplete Output When Tab Completing a File With a Space

I am currently experiencing an issue with tab completion on an installed application running PicoCLI 4.6.1 and java 11 on Fedora 34.
Here is a snippet of the command line class for this project:
...
import picocli.CommandLine.*;
import picocli.CommandLine.Model.*;
...

@Command(name = "bcc")
public class BccCommands implements Runnable{

    @Spec
    CommandSpec spec;

    @Option(names = {"-s", "--snapshot"}, description = "Loads the specified snapshot file immediately upon launch.  Use full paths.")
    public File snapshotFile;

   ...
}

...
	_cmd = new BccCommands();
       	_cmdParser = new picocli.CommandLine(_cmd);
        _cmdParser.setCaseInsensitiveEnumValuesAllowed(true);
        _cmdParser.setOverwrittenOptionsAllowed(true);
...

I am currently generating the AutoComplete using Maven and picocli.AutoComplete to generate a bcc11_completion script.
With the generated completion script in /etc/bash_completions.d/ I can  do the following:
bcc11 --snap<TAB><TAB>

and get
bcc11 --snapshot

as expected
However when I try to add the file path and use tab completion to fill in the file path like this:
bcc11 --snapshot /home/zpater/Snapshots/2<TAB><TAB>

I get the following output:

but I would expect something similar to this:

If I tab complete on a combination that doesn't find a file with a space in it, I get the expected result:

There seems to be an issue with the completion script I generated and spaces. Is there anything I can do to resolve this issue?

Hi @zpater345, thank you for raising this!
Picocli 4.6.2 has just been released, can I ask you to check if the problem still exists in the latest version?

I updated this morning and the behavior didn't change


[Net] Make TCP poll explicit, drop is_connected_to_host.

In this PR:

Make StreamPeerTCP::_poll_connection explicit.
No longer hacked into get_status and renamed to poll.
The old poll (for *nix poll, win select) is now called wait.

Drop is_connected_to_host for TCP and UDP.
The UDP method is now called is_socket_connected to limit confusion with the actual host connection status which doesn't make sense in UDP.
The TCP method is completly dropped, just use get_status instead.
The only one left is the WebSocketPeer one, which should be fine as is for now.

Fixes #22605
Fixes #18323

Thanks!


Always prefer vaccination certificates over recovery certificates if the recovery cert. is older than 90 days

It seems like CWA currently prefers recovery certificates over vaccination certificates until 180 days have passed since the positive test.
This should be adjusted and CWA should only prefer recovery certificates for 90 days, else users with a valid 2G+ status (and a recovery which is valid 180 days (but CovPassCheck can't detect)) will be rejected because their certificate expired.

@mlenkeit
There are multiple users on Twitter reporting that their status disappeared with the update to 2.19.
In 2 cases the app merged vaccination & recovery certificates, now prefers the recovery certificate although it is older than 90 days and shows no status anymore.
This is IHMO a top priority issue. Could you please comment so that we know you are aware of this issue?

Can you please give details what vaccination are used. Is there no status or from 2G+ to 2G. A screenshot would be helpful.

@thomasaugsten I will ask the users however wouldn't the change I suggest in the OP make sense anyways?

I think CovPass will change the acceptance of recover certificate independent of the valid to date in the certificate

@thomasaugsten
This is not the problem. Let me try to explain it again:
The user has a

1/2 vaccination certificate
2/2 vaccination certificate
1 recovery certificate

in their app.
The app now prefers the recovery certificate for 180 days over the 2/2 certificate although the recovery certificate is only valid for 90 days.
This then results that when the user shows their certificate on the "Certificates Overview screen", as suggested by the app, CovPassCheck rejects the certificate as invalid, as it doesn't know that this is a recovery certificate after basic immunisation.
How to fix this?
Prefer a recovery certificate over other certificates only if the certificate isn't older than 90 days.

This is difficult because the recovery certificate is a proof for his 2G+ status. He has to show it manually until the covpass includes this in the 2G+ check.

@thomasaugsten
Yeah but there are two options here
a) show a valid vaccination certificate
b) show an invalid recovery certificate
Do you really think it is better when the user shows an invalid recovery certificate (always, even if the person controlling is checking for 3G) then showing a valid vaccination certificate?
Keep in mind that most federal states are lifting their 2G+ restrictions.

Ok we will discuss this and update the ccl according to the decision

@Ein-Tim will be covered by #55, release date is not yet decided.

@mlenkeit
Trank you! I can't test this fix, though.


Gamepad Support!

Hello everybody,
the past week I worked on gamepad support for GeonBit.UI. If someone is interested in taking a look then please do: https://github.com/sqrMin1/GeonBit.UI/tree/gamepad
Included is a sample project (GeonBit.UI.Gamepad). You should start it first to get a clue how it works (of course with your Xbox 360 controller plugged in).
Everything should be self-explaining, because I described everything in the sample and in the code.
I didn't make a pull request so far, because I used MonoGame 3.6 Windows DX (through NuGet) and the main GeonBit.UI-repo is using MonoGame 3.5 DesktopGL. But in case you are interested in having this in the main repo @RonenNess then you could create a seperate branch for it (eg. gamepad36) and I will make a PR on this then.
I hope this is helpful.
Cheers,
Marcel
By the way: GeonBit.UI is very awesome! I had a lot of fun using it and I will use it in my main project. Thanks for your hard work!

Awesome work, thank you for sharing! :)
I'll probably do some GeonBit.UI dev this weekend so I could take a look at this as well as the other PR waiting. I don't have an xbox but I'm sure there are plenty of emulators out there, just hope they play nice with MG.
Will keep you updated once I'm on it.

Hello, do you have any plans to move forward with implementing gamepad functionality within the GeonBit UI?

@smiracle I don't know if you mean me or @RonenNess but i'll answer from my perspective, since this is posted in my old issue thread.
I created a repo which shows GeonBit.UI with GamePad support roughly before 2 Years. It seems that much happened since then, because my old repo is 189 commits behind the current master of GeonBit.UI.
I can deffinitley imagine to take a second look and create a clean GamePad support branch based on the current master, but as you can think of this takes much time and preperation (need to figure out all the new features and such). Despite that i'm also busy with different projects.
But yes, this project is really awesome as well as Ronen is and I can imagine to do it. I just can't tell when this will happen, but it will surely take some time.
So, don't expect it to soon (from my side).

Hi @smiracle,
Sorry but currently I have no plans to add gamepad, not even sure I'll work on GeonBit.UI any time soon, but that could change.
You can try and take @sqrMin1's repo and work from there, I'll try to be available and responsive for questions if you have any.
Thanks :)

@RonenNess
Oh, no worries! Everything is fine :)

Happy to hear from you both, I've been spending some time merging the latest UI changes with RonenNess' old gamepad code. So far it seems to be mostly working, although there are some problems with the way entities are selected when tabs are present in the panel (and likely other bugs). I'll likely be looking into this more over the next couple of weeks.

I went ahead and forked the repo and put all of my changes in a feature branch. There's still a lot of problems to iron out that I've been struggling with (primarily selection between nested panels) but the functionality that sqrMin1 added is all there in conjunction with the latest code and all of the original examples. No extra libraries were added.
Here's the forked branch in case anyone wants to take a look at it:
https://github.com/smiracle/GeonBit.UI/tree/gamepad-support

Thanks @smiracle. I took a look and it's fine to see gamepad functionallity in the newest GeonBit.UI iteration.
But as you said there a some problems which makes my old gamepad code unstable. Espacially when you think of new features like MessageBoxes, where the gamepad didn't even respond to. I'm also receiving crashes here and there.
Nothing special - this is in the end just exactly what I would expect from using pretty old code in a heavenly evolving library like GeonBit.UI.
I think it's neccessary to completly start from scratch again and implement the gamepad support in a clean and fresh (maybe even new) way.
Sure this is a bit of work but we should do it to comply with the high quality of GeonBit.UI!
Anyway thanks for your refreshed sample. It puts me in a positive mood to work on full gamepad support somewhat in the future.

@sqrMin1, awesome! I agree, I think maybe taking a more event-driven approach to gamepad input rather than checking for button presses under certain conditions on each update iteration might be worth thinking about.

GeonBit.UI - GamePad-Support
It was about time...

Click on this image to watch the YouTube-Video!

Implemented using the Microsoft.Xna.Framework.Input namespace from the MonoGame.Framework!
Available on my GamePad-Branch

Hello Com!
4 years after my original GamePad-Implementation for GeonBit.UI, I'm ready to show you my new one. But this time every single
usually interactable entity from the GeonBit.UI library is supported! No mouse-roaming anymore for "complex" entities like SelectList or DropDownButton or even complex UI-Scenarios with nested Panels and complex entities.
How does it work under the hood?
First of all I created the IGamePadInput interface of course and added its functionallity to the DefaultInputProvider. But instead of imitating the regular IMouseInput techniques, I created a specialized, GamePad-Supported version of each usually interactable entity of GeonBit.UI.
To enable the important core events for each entity in relation to the users GamePad-Input, I created an IEntityGamePad interface and traversed specialized GamePad-Events to the original GeonBit.UI-IMouseInput ones. This has the great advantage that I can precisley decide what action on each GamePad-Input-Supported-Entity really has to happen before the input get traversed to its final destination.
This also means that I nearly don't needed to modify the original source code of GeonBit.UI, despite the general GamePad implementation and one single modification.
The one single core-change I did was on this line in the Entity class to make it possible to search in hidden entities (children):
https://github.com/sqrMin1/GeonBit.UI/blob/gamepad/GeonBit.UI/Source/Entities/Entity.cs#L743
The reason for this is that it wasn't possible to find my own custom entites, when they were added to a PanelTabs-Entity, because its internal _panelsPanel (which hosts the added entites), got automatically skipped on Entity.Find() (because they are hidden by default).
However, this change will not break your existing code, because the "skipHidden" parameter   automatically sets to false, which is the standard behavier of GeonBit.UI.
Also on board is a GamePadSetup class, which makes it easy to configure the deadzones of your GamePad-Thumbsticks for example, but it also initializes a set of RichParagraphStyleInstruction with original GamePad-Colors from Xbox-Controllers, so that it perfectly integrates with GeonBit.UI.
Where to find?
I know that the GeonBit.UI library is made of high-quality and so it should stay. In the moment you will only find this GamePad-Implementation on my GamePad-Branch.
Feel free to share your (improvment)-suggestions here and if you think that my work could meet the high quality expectations of this library, you could maybe give this post a like and show @RonenNess your interest in an official GamePad-Integration directly into the GeonBit.UI library.
In the case this should ever happen, I am willing to create a "clean" version of my GamePad-Branch which only contains the most important commits and a well integrated sample project.

Thanks for reading and have fun using your GamePad with GeonBit.UI! 😁
Cheers,
Marcel
PS: TRY THE SAMPLE PROJECT!
- Made with ❤ -


systemd service name collisions

On an Ubuntu 16.04 system, I observed that an application created via this cookbook will attempt to create multiple systemd services with the same name.
Am I misunderstanding the README by mixing a gunicorn and celery_worker resource in the same application resource?
Given the following cookbook code:
application '/opt/myapp' do
  git 'git@github.com:myorg/myapp.git'
  pip_requirements
  python '2.7'
  gunicorn do
    port 9001
  end
  celery_worker do
   app_module 'app.tasks'
  end
end

this will happen:
* application_gunicorn[/opt/myapp] action enable
  * python_package[gunicorn] action install (up to date)
  * poise_service[/opt/myapp] action enable
    * execute[systemctl daemon-reload] action nothing (skipped due to action :nothing)
    * template[/etc/systemd/system/myapp.service] action create
      - create new file /etc/systemd/system/myapp.service
      - update content in file /etc/systemd/system/myapp.service from none to bd37a3
      --- /etc/systemd/system/myapp.service	2017-05-29 17:46:47.668598009 +0000
      +++ /etc/systemd/system/.chef-myapp20170529-24061-1i17040.service	2017-05-29 17:46:47.636598009 +0000
      @@ -1 +1,15 @@
      +[Unit]
      +Description=myapp
      +
      +[Service]
      +Environment=
      +ExecStart=/usr/bin/python2.7 -m gunicorn.app.wsgiapp --bind <IP_ADDRESS>:9001 setup
      +ExecReload=/bin/kill -HUP $MAINPID
      +KillSignal=TERM
      +User=root
      +WorkingDirectory=/opt/myapp
      +Restart=on-failure
      +
      +[Install]
      +WantedBy=multi-user.target
      - change mode from '' to '0644'
      - change owner from '' to 'root'
      - change group from '' to 'root'
    * execute[systemctl daemon-reload] action run
      - execute systemctl daemon-reload
    * service[myapp] action enable
      - enable service service[myapp]
    * service[myapp] action start
      - start service service[myapp]

* application_celery_worker[/opt/myapp] action enable
  * poise_service[/opt/myapp] action enable
    * execute[systemctl daemon-reload] action nothing (skipped due to action :nothing)
    * template[/etc/systemd/system/myapp.service] action create
      - update content in file /etc/systemd/system/myapp.service from bd37a3 to a61b3d
      --- /etc/systemd/system/myapp.service	2017-05-29 17:46:47.636598009 +0000
      +++ /etc/systemd/system/.chef-myapp20170529-24061-1fov4ul.service	2017-05-29 17:46:50.100598009 +0000
      @@ -3,7 +3,7 @@

[Service]
Environment=
      -ExecStart=/usr/bin/python2.7 -m gunicorn.app.wsgiapp --bind <IP_ADDRESS>:9001 setup
      +ExecStart=/usr/bin/python2.7 -m celery --app=app.tasks worker
ExecReload=/bin/kill -HUP $MAINPID
KillSignal=TERM
User=root
    * execute[systemctl daemon-reload] action run
      - execute systemctl daemon-reload
    * service[myapp] action enable (up to date)
    * service[myapp] action start (up to date)

I believe I've self-resolved here by specifying a unique 'service_name' for each of those.

Yeah, though I should probably figure out a better way to address this or at least make it an error.


Update jest editor version

Related to https://github.com/kavod-io/vscode-jest-test-adapter/issues/26
It is said the the latest version handles parametrised tests properly.
As far as I can see by my tests, it does not fixes the problem in #26 out of the box. But since I've already updated it, decided to make a PR.

Sorry, sent a PR to the wrong repo.


grid: allow Tuple, add order - fix termshow on 1x1 Matrices

grid: lift restriction on types: allow to use a Tuple of Renderables;
grid: allow iterating in row major or col major layout using the new order keyword;
don't use eval(...): a const NamedTuple called BOXES is added instead (can be indexed using a Symbol e.g. BOXES[:SQUARE]);
replace repeat([x], n) patterns with fill(x, n) (shorter, faster, less allocs);
replace var::Vector{Type} = [...] patterns with var = Type[...];
replace max(x...) patterns with maximum(x);
use === on Symbols (akin to is kw of Python);
added 23_test_issues.jl to test for reported issues (good practices);
more syntax simplifications/enhancements (modernization).

Fix https://github.com/FedeClaudi/Term.jl/issues/127.
Fix https://github.com/FedeClaudi/Term.jl/issues/129.
Fix https://github.com/FedeClaudi/Term.jl/issues/130.

Codecov Report

Merging #128 (70c50cb) into master (86d5fac) will decrease coverage by 0.72%.
The diff coverage is 98.95%.

@@            Coverage Diff             @@
##           master     #128      +/-   ##
==========================================
- Coverage   94.68%   93.96%   -0.73%
==========================================
  Files          34       34
  Lines        1938     1889      -49
==========================================
- Hits         1835     1775      -60
- Misses        103      114      +11

Impacted Files
Coverage Δ

src/compositors.jl
93.61% <ø> (ø)

src/layout.jl
94.79% <97.50%> (-0.17%)
:arrow_down:

src/_progress.jl
97.95% <100.00%> (-0.03%)
:arrow_down:

src/_tables.jl
92.50% <100.00%> (ø)

src/boxes.jl
98.14% <100.00%> (-0.07%)
:arrow_down:

src/dendograms.jl
98.70% <100.00%> (-0.05%)
:arrow_down:

src/grid.jl
100.00% <100.00%> (ø)

src/logs.jl
90.51% <100.00%> (ø)

src/panels.jl
95.83% <100.00%> (-0.17%)
:arrow_down:

src/repr.jl
85.10% <100.00%> (-10.69%)
:arrow_down:

... and 11 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 86d5fac...70c50cb. Read the comment docs.

Hey, I’m on holiday until the third so I can’t do a proper review yet. This looks great though thank you!

As a side note, it would be better to avoid try/catch in the codes as much as possible (https://docs.julialang.org/en/v1/manual/control-flow/#The-try/catch-statement), and rely on type dispatch instead (if possible of course). That's left for another PR.


Correct wav format?

Im trying to extract mfcc features from audio of a video file.
I tried FFMPEG:
def extractAudioFromVideo(video, audio_out="out.wav"):
	cmd="ffmpeg -i {} -acodec pcm_s16le -ac 1 -ar 16000 {}".format(video, audio_out)
	os.system(cmd)
	return audio_out

def extractAudioMFCC(file_name="out.wav"):
	fs, signal = wav.read(file_name)
	signal = signal[:,0]

	############# Extract MFCC features #############
	mfcc = speechpy.feature.mfcc(signal, sampling_frequency=fs, frame_length=0.020, frame_stride=0.01,num_filters=40, fft_length=512, low_frequency=0, high_frequency=None)
	mfcc_cmvn = speechpy.processing.cmvnw(mfcc,win_size=301,variance_normalization=True)
	print('mfcc(mean + variance normalized) feature shape=', mfcc_cmvn.shape)

extractAudioMFCC("test.mp4", audio_out="out.wav")
extractAudioMFCC("out.wav")

The error I get:

Traceback (most recent call last):
File "TWK.py", line 99, in
extractAudioMFCC()
File "TWK.py", line 22, in extractAudioMFCC
signal = signal[:,0]
IndexError: too many indices for array

Am I using the wrong wav format?

Would you please tell me why you are using signal[:,0] format?
The error does not seem to come from the package. It's a pythonic mismatch. In the test example of this repository, the reason behind using signal[:,0] is the dual-band nature of the example Wav files. Yours may only have a single channel. Try to remove signal = signal[:,0] and run the code again.

hey @astorfi . Ah, you were right.. i was blindly copy pasting the tutorial code and i completely forgot to check the shape. Thank you
Apologies in advance but im new to audio processing. I'm trying to do this in syncet, where the audio inputs are mfcc features and the audio model takes this input (according to model.summary())

conv1_audio (Conv2D)         (None, 13, 20, 64)        640

According to the owner of syncnet repo,

I have used a library called speechpy to extract MFCC features. The function to extract MFCC features from a .wav file according to the instructions using speechpy is:
speechpy.feature.mfcc(signal, sampling_frequency, frame_length=0.010, frame_stride=0.010, num_cepstral=13)
Audio features are computed over a duration of audio. In the paper, it is mentioned that features are computed at 100 Hz => for every 0.010 seconds. Hence, frame_length=0.010, frame_stride=0.010 (no overlap).
According to the paper, audio features and video features are extracted for every 0.2 seconds.
Lip: 0.2 seconds => 0.2 * 25fps = 5 video frames
Audio: 0.2 seconds => 0.2 / 0.01(frame duration) = 20 audio frames
Hence, a 112x112x5 matrix is input to the lips model, and a 13x20 matrix is input to the audio model.

Can you help me understand how to shape peechpy.feature.mfcc return value to be 13x20 matrix?


Define a recommendation for graphql-java-extended-scalars

Pull Request type

[x] Feature

Changes in this PR
This commit adds a recommendation for graphql-java-extended-scalars that will accommodate the requirements of compatibly in relationship with graphql-java. The latter is already part of the recommendations.
Per graphql-java-extended-scalars

use 1.0.1 or above for graphql-java 14.x and above
use 15.0.0 or above for graphql-java 15.x and above
use 16.0.0 or above for graphql-java 16.x and above

Reference https://github.com/graphql-java/graphql-java-extended-scalars
Note
The graphql-java-extended-scalars 16.0.0 artifact is not published in Maven Central.
For this reason we are recommending 15.0.0, with a preferred scope, until  https://github.com/graphql-java/graphql-java-extended-scalars/issues/38 is fixed.
Alternatives considered
We could not have a recommendation, but this will mean that developers will then have to figure out which version of graphql-java-extended-scalars to use according to the version of graphql-java that we do recommend.

The graphql-java-extended-scalars 16.0.0 artifact is not published in Maven Central. We can't merge this as is before this gets addressed.
Ref https://github.com/graphql-java/graphql-java-extended-scalars/issues/38

I assume it is compatible to use graphql-java 16 with graphql-java-extended-scalars 15?

It is as far as our examples and internal usage, in fact internally folks are using graphql-java-extended-scalars 1.0 and 1.1 while using graphql-java 16. That said in the graphql-java-extended-scalars page there is no references on backward compatibility and the statement so far is to use the version that matches the version used for graphql-java.

use 1.0.1 or above for graphql-java 14.x and above
use 15.0.0 or above for graphql-java 15.x and above
use 16.0.0 or above for graphql-java 16.x and above


Test case execution with multiple data scenarios

Execution
All data combinations for a test case are executed.  Adjust the test case description in the "Run Tests" dialog to indicate the number of data scenarios that will be run:

@brogreenmoon  - test cases do not have data sets, only test routines.  You need to create a test case and then add a test routine and specificy which data set to run

nope, not quite happy yet jan.  a test case can call multiple test routines with multiple datasets defined for each test routine, so there is a wider concept of data scenarios that applies to a test case, where a data scenario is a combination of test routine -level datasets that in totality is a dataset/data scenario that is used for one instance of the test case.  this "data scenario" does not exist in terms of the underlying data model being persisted, but it is a key construct in terms of what is actually being executed, as per my comment in #161.
we're going to need to visualise this in #118 too - i think it's a bit silly if you can't actually see the data that applies to each executing instance of the test case - you have no idea what's going on otherwise, such as when a test case calls three test routines with 2, 3 and 5 datasets respectively.  you need to be able to see the combinations that are in play in each instance, and in terms of this ticket it means identifying that 5 scenarios are in play and showing "5" to the user.

this is not going to be done in this milestone - no time and also it does not make sense for the currrent construct.  Data sets are releated to test routines and not test cases.  You can add more than 1 routine and with more than 1 data set for each to a test case.  This means that any listing of routine and data set really does not make sense unless you see the context of the entire test case.  To do this, you might as well open and view the test case file.
As I mentioned before, you you want to be granular about it, create a test case for each test routinue data set combo.
Moving out of this milestone.


Vivado Simulation

I have generated file occamy_mesh_floo_noc.sv successfully and I want to make a Simulation in Vivado. File floo_narrow_wide_chimney.sv is a submodular of occamy_mesh_floo_noc.sv and it includes registers.svh、assertions.svh and assign.svh,but I can't find those files in project.

Would you please tell me how to find the missing file?Thank you !

Those are header files of the common_cells and axi dependencies, that are automatically fetched by bender. For vivado you probably want to create a compilation script like this:
bender script vivado -t rtl -t test > compile.vivado.tcl

which will automatically set the correct include directories of the dependencies

I have found header files in file Bender.yml. However hw/floo_rob_wrapper.sv still lacks axi_demux_id_counters.sv and hw/floo_rob.sv lacks tc_sram_impl.

I am not quite sure what you mean with you found the header files in the Bender.yml. The include headers are part of the bender dependencies which define the include directory paths that need to be added to the compilation in their respective Bender.yml (see here for example). Bender will automatically handle this for you when you generate a compilation script as I showed before.
axi_demux_id_counters and tc_sram_impl are modules from the bender dependencies axi and tech_cells_generic, so they should be listed as well in the generated compilation script. Maybe you could check the output of the last command that I sent you to check that they are there.
Also, maybe it helps if you have a look at the Bender documentation to understand how dependencies are managed with bender.

They are declared inside the axi_demux_simple (see here)


[Bug]:  Prototype Pollution Vulnerability Affecting chartist  module, versions >=1.0.0 <=1.3.0

Would you like to work on a fix?

[ ] Check this if you would like to implement a PR, we are more than happy to help you go through the process.

Current and expected behavior
Overview
A Prototype Pollution vulnerability Affecting chartist , versions >=1.0.0 <=1.3.0, due to missing check if the argument resolves to the object prototype. This allow the attacker to inject malicious object property using the built-in Object property __proto__ which recursively assigned to all the objects in the program.
Reproduction
sent directly to the maintainer's email
Chartist version
1.3.0
Possible solution

Freeze the root prototype using Object.freeze
Require schema validation of JSON input.
Avoid using unsafe recursive merge functions.
Consider using objects without prototypes (for example, Object.create(null)), breaking the prototype chain and preventing pollution.
As a best practice use Map instead of Object

Fix:

#1433

Is there any chance this will be accepted soon?  And would a new release be forthcoming afterwards?


Various Patches which Complete v1.6

Proposal
Various bug fixes, patches, and improvements which complete v1.6. This release fixes #85, fixes #83, fixes #82, fixes #80, fixes #79, and fixes #27.
Improvements

Added support for NBD (network block device)
Added debug mode negation via CLI options
Removed short CLI options
Added support for static DHCP lease assignments and DHCP whitelist
Improved CLI argument groupings to help without output out --help in pypxe-server.py

Bug Fixes

Now able to enable/disable services via CLI when using a JSON configuration file
DHCP options are now client-specific

@psychomario before we do any actual testing perhaps we should take care of any missing documentation for whatever patches/fixes we have here?

@mmattioli Probably, I was just about to comment regarding this. So far I've noticed that debug negation is missing from --help and the README, I'm just about to go through the rest to find other missing sections.

The following kwarg table items are missing from DOCUMENTATION.md:

DHCP
whitelist
static_config

I believe that is it. Do you want me to write a section under Background for NBD and a similar table for the kwargs?

@psychomario yeah if you handle the NBD stuff I'll take care of the DHCP stuff

I've got something written up in #91, accepting any criticism.

Do you think --dhcp-fileserver and --dhcp-whitelist should be somehow separated from the rest of the --dhcp-* arguments, as they are the two that are available in both --dhcp and --dhcp-proxy modes?

Not really, if someone has an understanding of how DHCP works then they should have an understanding that these two options would inherently work in both modes.

Fair enough.

@psychomario is this ready for testing or are we missing something? Pretty sure we should be ready to start testing but just wanted to check with you.

I think we're ready for testing. I'll be able to test everything in the ~morning, although I'm not expecting to find anything as I've been testing as I've been going along.

Sounds good, let me know if you run into anything because once this is merged and we tag the new release I think we can take a break for a while lol

Well that's all the long standing issues done, except for NFS, which I really ought to get motivated to continue as I have the summer off.

I've done some testing, and my fixes are present in #92, there's also a query regarding log messages. Once that PR is resolved then I think we're ready to be merged.

@psychomario LGTM Ready to go?

Other than the query in #92, I think so:
With the initial debug logging block, there are a lot of lines that duplicate the service name in the log format string and the actual log message. For example:
2015-05-12 17:30:51,190 [DEBUG] PyPXE.TFTP TFTP Server IP: <IP_ADDRESS>
2015-05-12 17:30:51,190 [DEBUG] PyPXE.TFTP TFTP Server Port: 69
2015-05-12 17:30:51,190 [DEBUG] PyPXE.TFTP TFTP Network Boot Directory: .

Should we remove the second TFTP, as it is already present in the logging namespace?

@psychomario just fixed, good catch :+1: Ready?

@mmattioli Thanks! I think we're ready now, yes.