Rich Text Form Component, Dialogue Rich Text Editor

Motivation
The Rich Text editor could use a couple more pieces so we don't need to manually create them (having to do that now).  First this tool should be usable as a Form Component for widgets and Online Forms.
Second there should be both the Inline Rich Text Editor, and a Widget Dialogue version, as many times the area where the rich text should go isn't large enough to accommodate the Froala interfaces easily.
Proposed solution
I'll post my code once done so you can incorporate

Sadly i couldn't get an Inline Form Component going, i called Html.Kentico().RichTextEditor("Html", "default"); from my FormComponent View for the Html property, but it wouldn't render in the widget dialog, no errors or anything, so something must be going on with the events.

The Html.Kentico().RichTextEditor method utilizes the Html.Kentico().BeginInlineEditor API which only work on the page directly, in order for the RTE to work in widget properties dialog, a form component with custom JS initialization is needed.

Definitely going to need widget dialog capabilities, hopefully the enhancement won't take too long, otherwise I may need to load the tiny mce editor temporarily for widget dialog.

Hi Trevor, thanks for the suggestion and your effort to make the enhancement. Unfortunately, this enhancement is currently not at the top of the backlog. It would be very helpful if you could share your most common scenarios for such enhacement.

To create the widley issue rich text widget for page builder as one primary
case. Often an area may be too small to fit awysiwig (such as a 2 column
bootstrap), having a rich text in a modal dialogue is a life saver for
that.  Also say for a widget that can contain html (maybe A map widget with
pin that you click to see info) you can't always put a wysiwyg editor where
you need the rich text.
I'll install the tiny mce to compensate, sucks for now : /
On Fri, Jan 10, 2020, 3:11 AM David Komárek<EMAIL_ADDRESS>wrote:

Hi Trevor, thanks for the suggestion and your effort to make the
enhancement. Unfortunately, this enhancement is currently not at the top of
the backlog. It would be very helpful if you could share your most common
scenarios for such enhacement.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Kentico/ems-mvc-components/issues/79?email_source=notifications&email_token=ALIDDU7GZTN3VKE47OF33BLQ5A3TFA5CNFSM4J67HZB2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEITGMTQ#issuecomment-572941902,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALIDDU7IIVWSDRMXNR3EBGTQ5A3TFANCNFSM4J67HZBQ
.

We've gotten some feedback from some of the other Kentico MVPs (4 including me, haven't heard from the rest) and they agree that having a form component of this is a must, any way to get this higher priority?
@davidkomarek  can pass along the Slack conversations and notes we provided on scenarios and need.

Any news on this? I heard it was part way about a month ago.  I see it's set to "Waiting" but do we have any indication on when/if it will be out?

Hello Trevor, we came up with several solutions and have mockups ready review. You can expect a PM shortly.
The story is still subject to groomings and therefore there is no fixed release date set.

Hello @KenticoDevTrev , I am trying to use an WYSIWYG on the TextAreaComponent, were you able to implement it? and if yes, how?
Also @davidkomarek one little tiny feature that would be super helpful would be the ability to add custom properties to the Form Sections.

I made it work @davidkomarek.
After some JS, CSS, I was able to manage the injection of an WYSIWYG form component property.

Hello @vhugogarcia thanks for suggestions! Can you share some of the scenarios for properties on form sections and the wysiwyg? Thanks!

@davidkomarek , @vhugogarcia  can speak for his items, but i will say we have used the rich text editor in the Form tab for things such as:
Description areas for elements (needing links, bulleted lists, bold texted areas)
Sometimes basic tables for things like pricing listings in a section of text.
@vhugogarcia  are you saying though you got Froala working on the form tab with your screen shot? I know others complained that right now they have to customize CKEditor for admin side and froala for mvc side.

Hello @vhugogarcia thanks for suggestions! Can you share some of the scenarios for properties on form sections and the wysiwyg? Thanks!

Thanks for asking. As @KenticoDevTrev mentioned it. Not all forms are just a simple array of fields. Complex forms requiere the ability to add Headings to outline a section of the form, for example: Billing Informacion, Payment Information, etc. Paragraphs to provide more information to the user. Tables for rendering pricing information or something else.
So having the ability to add a custom property to a custom section on the form will help us to create build complex forms without the need to create a DB field just to display rich text. Also it will be ideal from you to add a FormComponent RichTextBox so we can add them to the custom form section and then the user can edit the text of the form from the form builder page, just like I showed in the screenshot above. The screenshot above is basically doing that, but I had to create a form component that I add into a section so I can add Headings and so on. The bad thing about that is the it creates a DB field in SQL, which does not appear the be the best practice, but sadly we are limited right now on Kentico.
Let me know if there is anything else I can help with.

@vhugogarcia are you saying though you got Froala working on the form tab with your screen shot? I know others complained that right now they have to customize CKEditor for admin side and froala for mvc side.

I just implemented Summernote (A Simple WYSIWYG) into a Form component property for a "Textarea".  Basically, since the form I'm building requires to add some headings, paragraphs, etc, I created a Form Component named: "ContentFormComponent" then I add a custom property for this and then by using JS I inject the WYSIWYG editor into the text area field for that specific component. Based on Kentico documentation we can use the .admin.css and .admin.js to inject styles and scripts into the form builder area, which is basically what I am doing. Again, not ideal, due to the "ContentFormComponent" creates a DB field (null) but at least it works, until Kentico team implements Form Sections custom properties an rich text component property.

@davidkomarek Another AWESOME thing we can do if we could get Form Section properties will be to define form layouts.
So, for example, if we are using a CSS framework such as Bootstrap, we can define a custom form property in a section and set the grid system to use and then just create one custom form section controller rather that 1 per every grid system option.

I've asked for that as well, I can't make a bootstrap layout form section
without properties.
On Thu, Apr 16, 2020, 9:21 AM Victor Hugo Garcia<EMAIL_ADDRESS>wrote:

@davidkomarek https://github.com/davidkomarek Another AWESOME thing we
can do if we could get Form Section properties will be to define form
layouts.
So, for example, if we are using a CSS framework such as Bootstrap, we can
define a custom form property in a section and set the grid system to use
and then just create one custom form section controller rather that 1 per
every grid system option.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Kentico/ems-mvc-components/issues/79#issuecomment-614683674,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALIDDU6MRFQT4CY27XB47WLRM4H5HANCNFSM4J67HZBQ
.

FYI, rich text Nuget package v1.3.0 has just been released -- it delivers Rich text form component you demanded. Please, try it out and let us know if we can close this issue 😉

Sweet!!! I'll give it a shot right away!
On Wed, Aug 5, 2020, 8:54 AM Ondřej Machala<EMAIL_ADDRESS>wrote:

FYI, rich text Nuget package v1.3.0 has just been released -- it delivers
Rich text form component you demanded. Please, try it out and let us know
if we can close this issue 😉
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Kentico/ems-mvc-components/issues/79#issuecomment-669207406,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALIDDUZOGL3K34Q3XUE2FPTR7FQBNANCNFSM4J67HZBQ
.

It's working awesomely, the only last request now is for the ability to add the Configuration property to the RichTextProperties and wire it up, that way we can specify what type of configuration toolset we want for the widget (ex a "Basic" one with only a couple elements vs. a full one)

Hi @KenticoDevTrev, we've integrated the rich text widget directly into the Kentico Xperience 13 together with the form component property for specifying a custom configuration, however we won't be backporting it to the v12 version. If you need it in Kentico 12, you'll have to download the source files and add it yourself, which should be pretty straightforward. 🙂

Thanks! I'll see about back porting, may just leave it as is possibly
unless a client needs it, I'll have enough work to do to update my existing
modules for 13!
On Wed, Nov 4, 2020, 2:56 AM Peter Petrinec<EMAIL_ADDRESS>wrote:

Hi @KenticoDevTrev https://github.com/KenticoDevTrev, we've integrated
the rich text widget directly into the Kentico Xperience 13 together with
the form component property for specifying a custom configuration, however
we won't be backporting it to the v12 version. If you need it in Kentico
12, you'll have to download the source files and add it yourself, which
should be pretty straightforward. 🙂
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Kentico/ems-mvc-components/issues/79#issuecomment-721600665,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALIDDU47WFG5YTSEUPJYXULSOEJLXANCNFSM4J67HZBQ
.


[IncludeSender] Codeunit 5600 "FA Insert Ledger Entry" - OnBeforeInsertRegister

Hello,
We would need to have the IncludeSender attribute to true on event OnBeforeInsertRegister from Codeunit 5600 "FA Insert Ledger Entry".
[IntegrationEvent(**true**, false)]
local procedure OnBeforeInsertRegister(var FALedgerEntry: Record "FA Ledger Entry"; var FALedgerEntry2: Record "FA Ledger Entry")
begin
end;

We need this, because we want to be able to call global methods from this codeunit in the event subscriber.

After analyzing the code, we came to the conclusion that we need another event and we do not need anymore the IncludeSender to be true on this event.
Therefore, this ticket can be closed.
Thank you!


version bump to 1.0.1

Oh, this PR title was scare me :-)

Uhm… Sorry, looks like I'll have to ruin it all and revert the patch. We deliberately don't include all this stuff in our styles to keep them consistent with each other (jumping fonts and text measurements on style switching look ugly) and to not impose the choice of fonts on users. Of all these changes, probably only direction: ltr does make sense, but it should go in every style then, not just one.
On a related note: the new-styles branch is nearing completion so now it makes sense to do any style updates against that branch instead of master.

Ah, I thought of the possibility, but then I saw the CodePen theme using a different font as well. Nevermind, if that's policy that's fine.

I saw the CodePen theme using a different font as well

I fully admit I wasn't very diligent in keeping them all in check :-). The new-styles is going to bring a lot of unification and I'll fix those common rules for every style too.


pod is not manged in oe23.03

What happened:
In the oe2303 environment, a pod is deployed in a namespace with kmesh enabled, but it does not seem to be managed.
[root@localhost helm]# bpftool prog trace | grep 402781194            <...>-167491  [004] d..21 95722.264353: bpf_trace_printk: [KMESH] DEBUG: origin addr=[402781194:20480]            <...>-167491  [004] d..21 95722.264356: bpf_trace_printk: [KMESH] DEBUG: bpf find frontend addr=[402781194:20480]            <...>-167491  [004] d.s41 95722.264471: bpf_trace_printk: [SOCKOPS] INFO: test123 not managed by kmesh, ip:402781194            <...>-167491  [004] d.s41 95722.264479: bpf_trace_printk: [SOCKOPS] INFO: test123 not managed by kmesh, ip:402781194
check kemsh_manage map，only cookie records，without ip records

What you expected to happen:
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

Kmesh version:v0.3.0
Others:

@bitcoffeeiux

How could two kmesh_manage created?

Not sure if this is related to recent fix
int record_tuple(struct bpf_sock_ops *skops)
{
    if (skops->family != AF_INET && !ipv4_mapped_addr(skops->local_ip6))
        return 0;

How could two kmesh_manage created?

Because it's kind mode, I'm querying this on the host machine. there are two kind node with kmesh，so have two map

Not sure if this is related to recent fix
int record_tuple(struct bpf_sock_ops *skops)
{
    if (skops->family != AF_INET && !ipv4_mapped_addr(skops->local_ip6))
        return 0;

it is suspected that the problem is caused by this reason. The 2303 has a special judgment.
 static inline bool conn_from_cni_sim_add(struct bpf_sock_ops *skops) {     // cni sim connect <IP_ADDRESS>:929(0x3a1)     // 0x3a1 is the specific port handled by the cni for enable Kmesh #if !OE_23_03     return ((bpf_ntohl(skops->remote_ip4) == 1) && (bpf_ntohl(skops->remote_port) == 0x3a1)); #else     return ((bpf_ntohl(skops->remote_ip4) == 1) && (bpf_ntohl(skops->remote_port) == 0xa1030000)); #endif }

/close


support open-simulator

Codecov Report

Merging #60 (f96015a) into main (48276fc) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #60   +/-   ##
=======================================
  Coverage   10.44%   10.44%
=======================================
  Files          11       11
  Lines        1168     1168
=======================================
  Hits          122      122
  Misses       1043     1043
  Partials        3        3

Flag
Coverage Δ

unittests
10.44% <ø> (ø)

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

Impacted Files
Coverage Δ

pkg/scheduler/algorithm/cache/types.go
61.66% <ø> (ø)

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 48276fc...f96015a. Read the comment docs.


FATAL ERROR: This version of the anchor-data API is no longer supported. Please update Bikeshed.

What I did:
$ bikeshed update
Downloading anchor data...
FATAL ERROR: This version of the anchor-data API is no longer supported. Please update Bikeshed.

This is after updating to the most recent Bikeshed manually using git pull in the repo. There's a long delay before the error, so I half suspect that this is a network timeout that isn't getting reported as such.

I just updated, and it got through fine, so yeah, probably a network timeout.  Looking at the code, not getting a resource at all will trigger that error message.
The Shepherd server was down for a bit this morning, so that's probably what it was.

Would it be hard to report it as a network error instead of suggesting to update?

Nah it wouldn't; I suspect that if I don't get a resource at all, it's probably always a network error.

Do you want a new feature request, or is this issue fine as it is?

If the apiclient fails for any reason, the exception is silenced and None is returned. The 406 error is a red herring here. In my case it was missing SSL certificates.


Add calibration view

Add CalibrationView and samples for AR toolkit. WIP

The calibration view implementation, used to fix the origin camera. This view contains 4 sliders (latitude, longitude, altitude and heading). This view is implemented in QML and can be used with C++ and QML APIs.
cc @JamesMBallard @ldanzinger @tdunn @DanCobb1111
This PR will be merged to the ArcGISArView branch, not v.next or master.

For documentation, I was not consistent for inmodule and ingroup. For calibration view, I used:
    \ingroup ArcGISQtToolkit
    \ingroup ArcGISQtToolkitCppApi
    \ingroup ArcGISQtToolkitQmlApi
    \inqmlmodule Esri.ArcGISRuntime.Toolkit.AR

What is the best values to use in the classes and items provide in AR toolkit?
As a reminder, the public classes and items are:

For C++ API: ArcGISArView (C++), LocationDataSource (C++), CalibrationView (QML)
For QML API: ArcGISArView (QML), LocationDataSource (C++), CalibrationView (QML)

@tdunn I didn't update AR.md with CalibrationView. Do you think I should do that?

@GuillaumeBelz - I think that would be a good idea.

Todo:

[ ] change opacity of the scene

Sorry for delay. I found a bug in the calibration (conflict between the origin camera and the location camera).
@JamesMBallard This is a big commit in the code to calculate the camera. Can you review again? (Especially updateTmccOriginCamera functions)


Footer should be better

Describe the feature
The current footer can be better in terms of look and for better user experience.
Add ScreenShots

In this footer it is difficult to use bottom to top  arrow and the alignment of item can be better for better user experience purpose.
Kindly assign me this task under GSSOC24
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

issue already assigned.


errors when authentication is needed

when using predict with connect, a common pain point is receiving a cryptic error, usually requests.exceptions.JSONDecodeError: Expecting value: line 1 column 1 (char 0) when needing authorization. it would be helpful to add in a better error message to get people to the desired outcome quicker

duplicate of #71


lerna-publisher should publish process.cwd() by default

Currently requires a dot.

fixed in https://github.com/wixplosives/lerna-publisher/commit/76196f741feb8b4c23c5b808856694fe1207a56e


1902 Changes/Improvements

I noticed the Changes section for 1902 has been renamed to "Improvements" section. During initial 1902 update release (7-March-2019), there's only one (1) changes that highlight about new interface for creating plans, offers, quotas and etc.
Now, we are one week after the initial 1902 release (13-March-2019), and MS update this section with additional 3 improvements.
My question is why MS did not highlight about this during the initial 1902 releases?
Are these new improvements release after 1902?
Please be more transparent to customers. This needs to be improve further. TQ.

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

ID: 9ceb9a8e-febf-8c7d-1ff7-ac16b93573c2
Version Independent ID: d0e85dfd-2b6c-20ff-db81-9a194a63cead
Content: Azure Stack 1902 update
Content Source: articles/azure-stack/azure-stack-update-1902.md
Service: azure-stack
GitHub Login: @sethmanheim
Microsoft Alias: sethm

@easonlauys , Thanks for your feedback. We are looking into this query and will update you as soon as possible.

Guess what, just one-day passed, there's additional 6 bullets under "Improvements" section. Can you please use numbering instead of blank bullet? Now there's a total of 10 bullets under that section. Tomorrow another 4?

@easonlauys Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

Yes, please. We are taking about Improvements/Changes section. Not the known issues or post-installation issues. I guess you know why I feedback about this. Thank you very much.

@easonlauys Due to scheduling issues, some of these known issues were added to the release notes rather late. However, with every Azure Stack release, issues can come in after the release and be added to the release notes. Please check back often for updates. Thank you for the feedback! #please-close this issue.

@easonlauys We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


[WIP] [macOS] Handle keyboard shortcuts using NSMenuItem

This allows keyboard shortcuts to be handled by macOS rather than by godot which correctly handles the keyboard layout "Dvorak - QWERTY ⌘".
Fixes #4386
This solution may seem a bit hacky but reading the macOS developer docs, it's the only way I could figure see.
As far as I can tell:

there is no way to get info on the shortcut remapping done for "Dvorak - QWERTY ⌘" from the macOS API.
there is no way to create a keyboard shortcut ("keyEquivalent") without adding it to the menu bar.

This code still has a few issues that need worked out:

the shift modifier is incorrectly being added to shortcuts that don't have shift.
godot shortcut handling should be disabled so that unremapped keyboard shortcuts don't get triggered.
shortcuts cannot yet be changed or removed.

Adding menus to just hold shortcuts seems to be too hacky, in case this is really only way we should at least replicate actual Godot menus with shortcuts, but if I understand this issue correctly following patch works for me, please check it out.
diff --git a/platform/osx/os_osx.mm b/platform/osx/os_osx.mm
index 113c6636f..c6852c67e 100644
--- a/platform/osx/os_osx.mm
+++ b/platform/osx/os_osx.mm
@@ -1001,7 +1001,7 @@ static const _KeyCodeMap _keycodes[55] = {
 	{ '/', KEY_SLASH }
 };

-static int remapKey(unsigned int key) {
+static int remapKey(unsigned int key, unsigned int state) {

 	if (isNumpadKey(key))
 		return translateKey(key);
@@ -1023,7 +1023,7 @@ static int remapKey(unsigned int key) {
 	OSStatus err = UCKeyTranslate(keyboardLayout,
 			key,
 			kUCKeyActionDisplay,
-			0,
+			(state >> 8) & 0xFF,
 			LMGetKbdType(),
 			kUCKeyTranslateNoDeadKeysBit,
 			&keysDown,
@@ -1052,7 +1052,7 @@ static int remapKey(unsigned int key) {
 		ke.osx_state = [event modifierFlags];
 		ke.pressed = true;
 		ke.echo = [event isARepeat];
-		ke.scancode = remapKey([event keyCode]);
+		ke.scancode = remapKey([event keyCode], [event modifierFlags]);
 		ke.unicode = 0;

 		push_to_key_event_buffer(ke);
@@ -1105,7 +1105,7 @@ static int remapKey(unsigned int key) {
 		}

 		ke.osx_state = mod;
-		ke.scancode = remapKey(key);
+		ke.scancode = remapKey(key, mod);
 		ke.unicode = 0;

 		push_to_key_event_buffer(ke);
@@ -1121,7 +1121,7 @@ static int remapKey(unsigned int key) {
 		ke.osx_state = [event modifierFlags];
 		ke.pressed = false;
 		ke.echo = false;
-		ke.scancode = remapKey([event keyCode]);
+		ke.scancode = remapKey([event keyCode], [event modifierFlags]);
 		ke.unicode = 0;

 		push_to_key_event_buffer(ke);

Wow, yup, that works perfectly!
I think I must have misunderstood the docs for UCKeyTranslate

Wow, yup, that works perfectly!

Neat, I have opened PR: #29481


Xml2js

Reopen this PR since --
The v1.0-dev branch is no more. Please re-open to master.

Originally posted by @kadler in https://github.com/IBM/nodejs-itoolkit/pull/53#issuecomment-573921742

Ah...Seems it has been merged already.


fix(spark-lineage): default timeout for future responses

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

Summary by CodeRabbit

New Features

Enhanced timeout functionality for waiting on responses, improving application responsiveness.
Introduced a default timeout for metadata change proposals, preventing indefinite blocking during processing.

Bug Fixes

Improved error handling by incorporating new exception management for timeouts in metadata emission.

@treff7es, I have added responseLatch.countDown(); in FutureCallback methods

Thanks @treff7es


Query: Compiler crash for invalid Include path

(As asked by @anpete.)
Please support eager loading when the root entity ends up in a projection.
There is not a good way to do what I need to do: load a list of entities with multiple child collections on each entity populated, plus an arbitrary calculation associated to each entity.
This would be perfect:
var results = dbContext.Entities
    .Select(entity => new
    {
        Entity = entity,
        ComplicatedServerSideCalculation = Entity.NavProperty.OtherInfo.Sum(x => x * 2)
    })
    .Include(info => info.Entity.ChildCollection1)
    .Include(info => info.Entity.ChildCollection2)
    .Include(info => info.Entity.ChildCollection3)
    .ToListAsync();

Intuitively, I'd also expect that this would work:
var results = dbContext.Entities
    .Include(entity => entity.ChildCollection1)
    .Include(entity => entity.ChildCollection2)
    .Include(entity => entity.ChildCollection3)
    .Select(entity => new
    {
        Entity = entity,
        ComplicatedServerSideCalculation = Entity.NavProperty.OtherInfo.Sum(x => x * 2)
    })
    .ToListAsync();

Have you checked if the second code listing works or not? We think that will already work.
Also, what happens with the first example (exception, ignored, etc.)?

@rowanmiller You're right about the second code sample! I'm surprised and happy! Given the response to the exact same question for EF6 I thought this was a path you decided against. This will be a major win for us when we finally get to switch to EF Core.
As for the first code sample with EF Core, I get:
System.InvalidCastException occurred
  HResult=0x80004002
  Message=Unable to cast object of type 'System.Linq.Expressions.NewExpression' to type 'System.Linq.Expressions.MemberExpression'.
  Source=Microsoft.EntityFrameworkCore
  StackTrace:
   at Microsoft.EntityFrameworkCore.Query.ResultOperators.Internal.IncludeResultOperator.GetQuerySource(MemberExpression expression)
   at Microsoft.EntityFrameworkCore.Query.ResultOperators.Internal.IncludeResultOperator.GetQuerySource(MemberExpression expression)
   at Microsoft.EntityFrameworkCore.Query.ResultOperators.Internal.IncludeExpressionNode.CreateResultOperator(ClauseGenerationContext clauseGenerationContext)
   at Remotion.Linq.Parsing.Structure.IntermediateModel.ResultOperatorExpressionNodeBase.ApplyNodeSpecificSemantics(QueryModel queryModel, ClauseGenerationContext clauseGenerationContext)
   at Remotion.Linq.Parsing.Structure.IntermediateModel.MethodCallExpressionNodeBase.Apply(QueryModel queryModel, ClauseGenerationContext clauseGenerationContext)
   at Remotion.Linq.Parsing.Structure.QueryParser.GetParsedQuery(Expression expressionTreeRoot)
   at Microsoft.EntityFrameworkCore.Query.Internal.QueryCompiler.CompileQueryCore[TResult](Expression query, INodeTypeProvider nodeTypeProvider, IDatabase database, ILogger logger, Type contextType)
   at Microsoft.EntityFrameworkCore.Query.Internal.QueryCompiler.<>c__DisplayClass19_0`1.<CompileQuery>b__0()
   at Microsoft.EntityFrameworkCore.Query.Internal.CompiledQueryCache.GetOrAddQueryCore[TFunc](Object cacheKey, Func`1 compiler)
   at Microsoft.EntityFrameworkCore.Query.Internal.CompiledQueryCache.GetOrAddQuery[TResult](Object cacheKey, Func`1 compiler)
   at Microsoft.EntityFrameworkCore.Query.Internal.QueryCompiler.CompileQuery[TResult](Expression query)
   at Microsoft.EntityFrameworkCore.Query.Internal.QueryCompiler.Execute[TResult](Expression query)
   at Microsoft.EntityFrameworkCore.Query.Internal.EntityQueryProvider.Execute[TResult](Expression expression)
   at Remotion.Linq.QueryableBase`1.GetEnumerator()
   at Microsoft.EntityFrameworkCore.EntityFrameworkQueryableExtensions.IncludableQueryable`2.GetEnumerator()
   at System.Collections.Generic.List`1..ctor(IEnumerable`1 collection)
   at System.Linq.Enumerable.ToList[TSource](IEnumerable`1 source)
   at Test_EF_eager_loading.Program.Main() in c:\users\jmusser\documents\visual studio 2017\Projects\Test EF eager loading\Test EF eager loading\Program.cs:line 46

We decided not to support the first scenario. Include only works on entities.

Instead of throwing from compiler crash, we should print out exception message that include is not applied on entity type.

We already warn for ignored Includes. Agree we should never crash though.

Fixed dbe6b2846bc81e2653605eb4c2b6b4a7987848e5 in a sense that we no longer throw for this case and log Ignored Include warning instead

Reactivating and sending for re-triage - we might end up supporting this scenario in the future

Modified the original fix in 2e4e138572404ed936eb6e75cde4ed40a8ba6096. Now we throw NotSupportedException with a better explanation, rather than just log a warning.


add watchdog http server

Add a http server that return the codes:

Alive => StatusCode::OK (200)
Graceful => StatusCode::NO_CONTENT (204)
Critical => StatusCode::SERVICE_UNAVAILABLE (503)

2 signals are implemented:

the scheduler loop doesn't run
the mempool has more than 0 Tx after pooling.

Add a timer to detect that the mempool pick a task is not called after a certain time.

I've still some change. I repoen a new PR.


Momentum

Hi,
I'm developing a trading system and I'm trying your API.
I got some trouble to understand MOM (Momentum indicator) parameter. Your documentation says:

Number of data points used to calculate each MOM value. Positive integers are accepted (e.g., time_period=60, time_period=200)

The question is:
time_period is the same as the window parameter used to calculate Momentum?
I was using an own function to calculate Momentum and the results are totally different.
Thanks.

Not sure what the window parameter on your function is. Could you please ask this question on a forum like StackOverflow? Issue tracking should be for issues with the repo.

https://www.fmlabs.com/reference/default.htm?url=Momentum.htm

The time_period is the number of times you use the interval. Let's say you have interval=1min, and time_period=60, this would get you the momentum for 60 minutes (aka one hour).


Update Chroma example notebooks

Updating Chroma's example notebooks to reflect recent API changes.

Will assume this is good (I saw you've contributed to Chroma as well)


boards: arm: nrf5340: Add pwm-led0 alias

The pwm-led0 alias is required for building fade_led and blinky_pwm
samples.
Signed-off-by: Sigurd Olav Nevstad<EMAIL_ADDRESS>

Would like a review by @anangl


Reduce number of raw: interpolations in CodeGeneration

CodeGeneration still uses many raw: interpolations, which should be avoided unless it’s really necessary. Some of these raw: interpolations don’t need to the raw: label anymore. For some others, we could restructure how nodes are represented in SyntaxSupport to use TokenSyntax instead of String, similar to what I’ve done in https://github.com/apple/swift-syntax/pull/1914.

Tracked in Apple’s issue tracker as rdar://113237456

I've grabbed the lowest-hanging fruit and and removed all the no-longer-needed raw: interpolations. Further reductions in the use of raw: will require changing types.

So actually, if I've already started doing things related to this issue, I can do the whole thing.

We've already eliminated most of the raw: interpolations in CodeGeneration. I have a few more potential places in mind where we could eliminate them, but some of them seem to be incorrect. I would love to hear your feedback @ahoppen @kimdv

Node.getter:parserFunction - I'm thinking of changing the type to TokenSyntax. If I'm not mistaken, the function identifier is of this type, right?
let optionalMark = child.isOptional ? "?" : "" into let optionalMark: TokenSyntax = child.isOptional ? TokenSyntax.postfixQuestionMarkToken() : TokenSyntax(""). The second branch of this condition seems to be incorrect. TokenSyntax = "" doesn't feel right, does it?
let iuoMark = child.isOptional ? "" : "!" into let iuoMark: TokenSyntax = child.isOptional ? TokenSyntax("") : TokenSyntax.exclamationMarkToken().

Let me know if there are any issues with these changes or if you can identify other potential usages of raw: that could be eliminated but I've missed them.

Node.getter:parserFunction - I'm thinking of changing the type to TokenSyntax. If I'm not mistaken, the function identifier is of this type, right?

Yes, that seems like a good change to me

let optionalMark = child.isOptional ? "?" : "" into let optionalMark: TokenSyntax = child.isOptional ? TokenSyntax.postfixQuestionMarkToken() : TokenSyntax(""). The second branch of this condition seems to be incorrect. TokenSyntax = "" doesn't feel right, does it?
let iuoMark = child.isOptional ? "" : "!" into let iuoMark: TokenSyntax = child.isOptional ? TokenSyntax("") : TokenSyntax.exclamationMarkToken().

You are correct, TokenSyntax("") doesn’t make any sense. What we could do, is add a new appendInterpolation method to SyntaxStringInterpolation that takes an optional syntax nodes. If the node is nil, it just doesn’t add any interpolation. In that case you could have let optionalMark = child.isOptional ? TokenSyntax.postfixQuestionMarkToken() : nil.


Incompatibility with SD

Hello! There is the following: although you do not claim backward compatibility with the official SD from AUTOMATIC1111, can you fix it so that it works there too. We tried different combinations of checkboxes and other settings, but the extension is simply silent and does not work.
I also even tried to put the following positions - the result is the same.

There are no errors in the console, except that by turning the sliders to maximum and setting all the filters, the following is displayed:
████████████████████████████████████████████████████████████████████████████| 20/20 [00:37<00:00,  1.88s/it]
*** Error running postprocess_image: D:\AI\NS\extensions\sd-extension-nudenet\scripts\nudenet-ext.py<00:00,  1.85s/it]
    Traceback (most recent call last):
      File "D:\AI\NS\modules\scripts.py", line 675, in postprocess_image
        script.postprocess_image(p, pp, *script_args)
      File "D:\AI\NS\extensions\sd-extension-nudenet\scripts\nudenet-ext.py", line 58, in postprocess_image
        process(p, pp, enabled, metadata, copy, score, blocks, censor, method, overlay)
      File "D:\AI\NS\extensions\sd-extension-nudenet\scripts\nudenet-ext.py", line 35, in process
        info = processing.create_infotext(p)
    TypeError: create_infotext() missing 3 required positional arguments: 'all_prompts', 'all_seeds', and 'all_subseeds'

Example image:
woman
Negative prompt: EasyNegativeV2
Steps: 20, Sampler: DPM++ 2M Karras, CFG scale: 7, Seed:<PHONE_NUMBER>, Size: 512x512, Model hash: 71e14760e2, Model: lazymixRealAmateur_v30b, TI hashes: "EasyNegativeV2: 339cc9210f70", Version: v1.6.0-2-g4afaaf8a
I would like to be able to use it there too, as it is convenient, simple and fast.
Hope for your help, thank you.

making it compatible with a1111 should be relatively easy, but i dont have or plan to install a1111 to test.
however, contributions and/or PRs would be welcome.


[Marketplace] add bookmarks clickable icons in Marketplace to add/remove from My Workspace

Description
add bookmarks clickable icons in Marketplace to add/remove from My Workspace
Use case/motivation
No response
Related issues
No response
Confidential information

[X] I confirm that do not share any confidential information

Please fix also hover on icon
Now:

Expected:

verified on staging
related issues #2449


Removed duplicated code

This removes duplicate if for certificateChain verification.

@swift-server-bot test this please


Problem with eslint

When I ran the tests I was experiencing some troubles with eslint example:

I'll Send a PR With the solution.

Fixed linting issues


Task System for Bevy

Currently bevy depends on rayon for multithreaded dispatching of systems. This PR replaces rayon with a custom designed task system that consists of several threaded "TaskPools."
Why replace Rayon

Rayon has long-standing performance issues with smaller workloads on machines with more than a few cores. (#111)
Rayon is not async-friendly
Rayon is somewhat of a closed box, it owns the threads and it would be more difficult to upstream changes that are tuned for games as it’s a general purpose library
Rayon has a lot more in it than we really need. Our alternative has fewer dependencies, compiles faster, and is substantially less code

Bevy Tasks
bevy_tasks does three things:

Fire-and-forget `static tasks
Fork-join for non-static tasks
Chunked parallel iteration of slices

These are enough to replace all current rayon functionality within bevy, as well as to provide additional functionality that can partially supplant parts of rayon's parallel iteration support.
Exposed API

Exports IOTaskPool, ComputePool, and AsyncComputePool.
bevy_app exports DefaultTaskPoolOptions now.
Documentation mirrored at https://charted.space/notes/bevy_tasks/bevy_tasks/index.html.

Future work:

Continue distributing more work onto task pools (e.g. asset handlers).
More optimization of cpu usage.

Resolves #318

I imagine this will also improve compile times. Any idea by how much?

Just to crudely illustrate how much of an improvement this is, when I run the breakout example I get the following cpu usage (average across cores):
rayon 1.3: ~50%
rayon 1.4: ~40%
bevy_tasks: ~20%

This also cuts our dependency count by 7 and significantly decreases compile times.
For example, before breakout clean-compiled in release mode on my machine in 2 minutes 3 seconds
Now it compiles in 1 minute 34 seconds!

Actually I ran a few more tests and I dont think the difference is quite that stark. Probably closer to 5-10 seconds.

Great stuff!
I think it would be very cool to be able to spawn tasks on a thread-local executor too. Sometimes you have things that are !Send, and you can then still use them in an async context by wrapping them in an Rc or something. It should be quite easy to support:

Create a thread-local var like this:

thread_local! {
    static EXECUTOR: RefCell<LocalExecutor> = RefCell::new(LocalExecutor::new());
}

Each worker thread polls its LocalExecutor in addition to its Ticker within its loop.
Expose a public function called spawn_local like this: https://github.com/async-rs/async-std/blob/master/src/task/executor.rs#L19

To protect from people trying to spawn_local on a non-worker thread, may want the thread_local to be an Option<RefCell<LocalExecutor>>?

@kabergstrom yeah that would be a welcome feature. when this gets merged, can you create an issue for it? id prefer to keep this pr constrained to its current scope.
@lachlansneff  some of the schedule tests are failing because ComputeTaskPool doesn't exist. we'll need to fix them before merging.

Just to crudely illustrate how much of an improvement this is, when I run the breakout example I get the following cpu usage (average across cores):
rayon 1.3: ~50%
rayon 1.4: ~40%
bevy_tasks: ~20%

On a 3950x with 32 cores, I see a bigger difference (and rayon 1.3 and 1.4 give the same results for me):
Debug mode: 900% -> 110%
Release mode: 450% -> 40%

I think I addressed everything - I PR'ed it to @lachlansneff's branch so he can review it before we ask @cart to take another look.

@cart I think this is ready for a second look, the two main issues remaining are:

“I'm not convinced bevy_tasks should be hard-coded into apps...”
“Ordering determining "precedence" is maybe a little hard to reason about...“

Ok I think this ready to merge now. The "bevy_tasks hard coded into apps" issue can be resolved separately. I'd rather merge this and then iterate than wait longer.
One more huge thanks to everyone involved. This is a big step forward!


Question about making effect representing wave

Hello and thank you for this project, I've been trying to intergrate your code into my project which can also represent naval data like wave or current. For now I set the lineWidth parameter wider and make it change based on frame number, so it would expand along the way until it disappears, but I also want it to show like a circular shape, like in https://www.ventusky.com/?p=-56.02;54.71;6&l=wave&w=0SJdjA9AA ,instead of a straight line. How exactly can I do that? I'm fairly new to GLSL programming, I think I should make the edge move slower than the center, but how can I do that?

well, I don't have experience on rendering such shape, but after some googling I think maybe GL_TRIANGLE_FAN can help you render the circular shape you want?

Thank you, I'll give it a try


New groups don't consume old messages

Maybe this is a missunderstanding as to how this is supposed to work, sorry if so.
I have the following usecase:

N messages are published to topic A (potentially by an external service)
Service X starts listening to topic A using a fresh group ID (let's say a random UUID V4)

In this scenario, I will not receive any messages. Only if I publish new messages after having started to listen.
I've already tried to set config.Consumer.Offsets.Initial = sarama.OffsetOldest, expecting to start at 0, as it should be the oldest. Especially since no one has ever read from the topic before. And offsets should be separate for groups either way, if I understood correctly?
A workaround I found, was manually resetting the offsets to 0, however, this isn't really a good solution I think?
If you think that this sounds like a bug, I can provide you with an extensive example repository.

@Bios-Marcel config.Consumer.Offsets.Initial = sarama.OffsetOldest should have been sufficient, but crucially only if the consumer hasn't already committed an offset newer than that. As long as you start a brand new consumer group with that then it should begin from the first available offset

Well, since I couldn't figure out what was going on, I actually went for franz-go instead of Sarama. But thanks for the answer anyway.


Add new command to manage.py

Description:
Added new command popdb to managment/command. This command is supposed to populate database with fake users, offers and organizations. Appropriate test is attached
Unit Tests:

apps/volontulo/tests/test_popdb.py

Codecov Report

Merging #773 into master will increase coverage by 0.27%.
The diff coverage is 96.55%.

@@            Coverage Diff             @@
##           master     #773      +/-   ##
==========================================
+ Coverage   91.55%   91.83%   +0.27%
==========================================
  Files          99      101       +2
  Lines        2830     2999     +169
  Branches        7        7
==========================================
+ Hits         2591     2754     +163
- Misses        230      236       +6
  Partials        9        9

Impacted Files
Coverage Δ

backend/apps/volontulo/tests/test_popdb_command.py
100% <100%> (ø)

backend/apps/volontulo/tests/test_factories.py
100% <100%> (ø)
:arrow_up:

...ackend/apps/volontulo/management/commands/popdb.py
84.61% <84.61%> (ø)

backend/apps/volontulo/factories.py
97.7% <96.07%> (-2.3%)
: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 0ce3af7...b8b01f8. Read the comment docs.


Allow event name symbols in events$EventEmitter.

events$EventEmitter methods support Symbol as eventName https://nodejs.org/api/events.html#events_emitter_addlistener_eventname_listener - make sure flow library definition does match nodejs.

This change looks correct, thank you for the PR! This does change the output of some tests, would you mind regenerating the expected test outputs for node_tests (with ./runtests -r) and adding that to the PR?


call php in exec

Hello and thank you for the valuable product you produced :)
I decide to start a file server inside and access the internal server. But this cannot be done.
server.php
<?php  exce("php -S localhost:10000");
Considering that it is not installed php on the target system, it displays an error. Please tell me how I can first run a server to display information graphically?

cat micro.sfx server.php > single

If you are using microsfx and package PHP and PHP code in one file, you cannot use the built-in PHP web server. The PHP built-in server is only supported with cli mode and micro is not CLI. (Micro is indeed launched from the command line, but it is still different from the official cli SAPI).
If you want to use a built-in web server, it's best to use standalone php-cli instead of standalone micro.

thanks jerry, but how can i use them and protect code source ?  need copy php-cli.exe along side compiled micro.sfx ? and how in server.php call the php.exe ?

https://static-php.dev/en/faq/#can-i-protect-my-source-code-with-micro
micro.sfx has never been able to protect code. The static-php-cli and phpmicro projects do not involve any functionality for protecting, obfuscating, or encrypting code.


Change log of v32.1 - v38 is missing.

Please could you provide a change log for the new releases?

Unfortunately the release logs have not been updated, you can check the recently merged PR's for a quick overview of what's happened in the last few releases.


2FA with sms code

Hello, I have read #168 and #99.
But i am still struggling with the 2FA and sms code in nodejs.
Could you please provide a full example of login with this ?

Yes please!


[FLINK-3761] Introduction of key groups

This pull request introduces the concept of key groups to Flink. A key group is the smallest assignable unit of key-value state to a stream operator. Differently said, it is a sub set of the key space which is assigned to a stream operator. With the introduction of key groups, it will be possible to dynamically scale Flink operators which use partitioned (=key-value) state.
In order to support key groups, the following components were added/changed:
Introduction of KeyGroupStateBackend
In order to make the state backends aware of the key groups, we had to introduce a new type of state backend KeyGroupStateBackend. A key group state backend behaves similarly to the old AbstractStateBackend just with the difference that it is aware of key groups. This means that it generates a snapshot for each key group upon snapshotting.
In order to leverage the existing implementation of state backends, the AbstractStateBackend was split up into the new AbstractStateBackend which is responsible for managing non-partitioned state and the PartitionedStateBackend which is responsible for managing the partitioned state.
Furthermore, the PR comes with a GenericKeyGroupStateBackend implementation which is the standard for the KeyGroupStateBackend. The GenericKeyGroupStateBackend simply creates for every key group a PartitionedStateBackend which manages this key group. Upon snapshotting, each PartitionedStateBackend is snapshot.
Introduction of KeyGroupAssigner
In order to assign keys to a key group, the KeyGroupAssigner interface is introduced. The KeyGroupAssigner implementation is used by the GenericKeyGroupStateBackend to select the proper PartitionedStateBackend for the current key. Furthermore, the former HashPartitioner, which is renamed now into KeyGroupStreamPartitioner uses the KeyGroupAssigner to distribute the streaming records in a consistent way wrt to the key group mappings. The key groups itself are mapped in a round robin fashion (key group index modulo #channels = out-going channel) to the downstream tasks.
Introduction of MaxParallelism to user API
In order to scale programs up or down, it is necessary to define the maximum number of key groups. The maximum number of key groups denotes the maximum parallelism of an operator, because every operator needs at least one key group to get elements assigned. Thus, in order to specify this upper limit, the ExecutionConfig allows to set a job-wide max parallelism value via ExecutionConfig.setMaxParallelism. In addition to that the SingleOutputStreamOperator allows to set similarly to the parallelism a max parallelism value on an operator basis. If the max parallelism has not been set and there is no job-wide max parallelism set, the parallelism of the operator will be taken as the max parallelism. Thus, every operator would then receive a single key group.
However, in order to scale jobs up and down, one has to set a max parallelism for the operators which shall be scaled up/down. The upper limit for the scaling is the max parallelism value. The max parallelism mustn't change when scaling the job, because it would destroy the mapping of keys to key groups.

[X] General

The pull request references the related JIRA issue ("[FLINK-XXX] Jira title text")
The pull request addresses only one issue
Each commit in the PR has a meaningful commit message (including the JIRA id)

[X] Documentation

Documentation has been added for new functionality
Old documentation affected by the pull request has been updated
JavaDoc for public methods has been added

[X] Tests & Build

Functionality added by the pull request is covered by tests
mvn clean verify has been executed successfully locally or a Travis build has passed

Very cool stuff! I was wondering did you do any benchmarks for the performance impact of this change? For instance it would be good to know how well RocksDB behaves with a large number of instances etc.

No I haven't run performance benchmarks, yet. Should definitely do that as a follow-up.
You're right that the behaviour of RocksDB is interesting in particular. It might make sense for the future to implement a RocksDB specialised KeyGroupStateBackend implementation, which uses only a single RocksDB instance for all key groups.

@aljoscha would be great to get some feedback from you :-)

So if I understand currently there is no way to scale jobs with non-partitioned states. This also means that window operations (that register timers) will not be scalable right?

@gyfora Yes, this is correct. We'll have to put the timers into partitioned state and upon restarting iterate over the saved timers to re-set any callbacks at the StreamTask.
Doing this naively would slow things down. What I had in mind is to still keep the timers in an in-memory Map but before snapshotting put them into a partitioned state. This way, we don't have the overhead every time that we deal with timers but just when checkpointing.

In order to support scaling for non-partitioned state, the next step could be the introduction of a kind of union state. The idea would be to aggregate the non-partitioned state of each subtask and then sending it to all subtasks upon recovery. Then every subtask can pick from the union state what it needs for its execution

Yeah, we were also wondering wether it would make sense to allow the state itself to be repartitioned, i.e. union and then split into the new parallelism. In this way we wouldn't read all state in every operator.

I started looking into it, but man this is one big change... 😃
I have some first remarks about API and internals:
Whats the reason for the introduction of PartitionedState? The Javadoc for State already says that it is the base class for partitioned state and that it is only usable on a KeyedStream.
The signature of KeyGroupedStateBackend and PartitionedStateBackend is exactly the same. AbstractStateBackend has both, method createPartitionedStateBackend and createKeyGroupStateBackend. Users of an AbstractStateBackend should only ever call the latter while the former is reserved for internal use by the default implementation for KeyGroupedStateBackend which is GenericKeyGroupStateBackend. Also, AbstractStreamOperator has the new method getKeyGroupStateBackend that should be used by operators such as the WindowOperator to deal with partitioned state. Now, where am I going with this? What I think is that the AbstractStateBackend should only have a method createPartitionedStateBackend that is externally visible. This would be used by the AbstractStreamOperator to create a state backend and users of the interface, i.e. WindowOperator would also deal just with PartitionedStateBackend, which they get from AbstractStreamOperator.getPartitionedStateBackend. The fact that there are these key groups should not be visible to users of a state backend. Internally, state backends would use the GenericKeyGroupStateBackend, they could provide an interface to it for creating non-key-grouped backends.
Above, "exactly the same" is not 100 % correct, since the snapshot/restore methods differ slightly but I think this could be worked around. Also, I found it quite hard to express what I actually mean but I hope you get my point. 😅

Thanks for the initial feedback @aljoscha :-)
The introduction of PartitionedState is indeed not strictly necessary for this PR. The idea was that we will have partitioned and non-partitioned state in the future. PartitionedState is the key-value state backed by the PartitionedStateBackend whereas non-partitioned state is backed by the AbstractStateBackend. The first non-partitioned state (apart from the state serialized via CheckpointStateOutputStream) could be the redistributable non-partitioned state necessary for the KafkaSources, for example. Thus, the PartitionedState is more of a logical separation and it lays the foundation so that also non-keyed stream operators can use a proper state abstraction. But I can revert it, if you deem it redundant or pre-mature.
It is true that the PartitionedStateBackend and the KeyGroupStateBackend have almost the same signature. However, the changes you've mentioned are imho crucial and made the whole refactoring of the state backends necessary in the first place. The difference is that the KeyGroupStateBackend is aware of the key groups and, consequently, is able to snapshot and restore each key group individually. Trying to work around this would mean that the PartitionedStateBackend always has a single key group associated. But for that, it would have to know the sub task index of the enclosing StreamOperator to assign a sensible key group index. Furthermore, it wouldn't make sense to use any other PartitionedStateBackend than the KeyGroupStateBackend (given that it respects the KeyGroupAssigner) for the AbstractStreamOperator, because the data is shuffled according to the key group assignments. In general, I think the notion of key groups are touching too many parts of the Flink runtime so that it makes no longer sense to try to unify the KeyGroupStateBackends and PartitionedStateBackends. The state backends used by the AbstractStreamOperator have to be aware of that notion.
You can regard the PartitionedStateBackend as an internal class which was introduced to reuse the existing state backend implementations via the GenericKeyGroupStateBackend. In the future it might make sense to directly implement the KeyGroupStateBackend interface to decrease the key group overhead. It's just unfortunate that Java does not allow to specify package private methods. Otherwise, I would have declared the createPartitionedStateBackend as package private. But since the GenericKeyGroupStateBackend resides in a sub-package of o.a.f.runtime.state, it cannot access this method. But I think we could refactor it the following way: Remove createPartitionedStateBackend, make createKeyGroupStateBackend abstract, let the implementations of AbstractStateBackend implement a PartitionedStateBackendFactory interface and define the createKeyGroupStateBackend method for all AbstractStateBackend implementations with creating a GenericKeyGroupStateBackend which requires a PartitionedStateBackendFactory. That would be probably a better design.

Jip, this last paragraph with the Factory is what I hinted at with my comment. 😃 This may be somewhat academic but if there is a method getPartitionedStateBackend the likelihood of it being wrongly used is somewhat high.

Yes definitely: better be safe than sorry. Will remove the createPartitionedStateBackend method from AbstractStateBackend.

I think this is subsumed by #2376
Should we close this Pull Request?

Yes will close the PR because it has been subsumed by #2440.


Detect CUDA and nvcc compiler

Can you add information for the detection of Nvidia's CUDA compiler (nvcc)? See https://docs.nvidia.com/cuda/cuda-compiler-driver-nvcc/index.html#nvcc-identification-macro
It would be nice to also be able to detect the CUDA compilation phases (host and device code generation). A caveat is that clang can also compile CUDA and uses different macros, see https://www.llvm.org/docs/CompileCudaWithLLVM.html#detecting-clang-vs-nvcc-from-code

Sounds like a good idea.

Will give it a shot as we migrate to Github Action, we will then reuse the nvidia docker directly.


[proposal] clarify scheduler framework extender

What is your proposal:
The internal concepts of the frameworkext package of koord-scheduler are somewhat confusing and need some clarification.

ExtenderFactory is responsible for creating Extender.
Extender implements ExtendedHandle, which means the extension of framework.Handle, and implements some functions of framework.Framework, responsible for intercepting some function calls. The Extender corresponds to each profile, and some transformers inside each Extender are initialized through the type assert mechanism when the plugin is built.
design document describes the new extension points named Transformer but now named Hook in the current implementation.
And the Hook should be registered in the main function, but we can use type assert with plugin instance to determine whether the plugin implements the Hook
And we should add PreBindReservation extension point to clearly indicate whether a plugin needs to update the Reservation in the PreBind phase

Why is this needed:
Is there a suggested solution, if so, please add it:

[ ] refactor the scheduler framework extender.  Already in review.. #1066
[ ] implements the PreBindReservation
[ ] update the doc

/assign

/milestone v1.2


Need to get several indexes when doing a box/ lasso selection from a plotly graph

Description
When using the new feature to integrate plotly charts, we do get the index of the selected point on the chart.
However, when performing a box or lasso selection, the selected_state is empty instead of returning a list of indexes corresponding
to the selected points.
How to reproduce
You can check using the following code below

A code fragmentfrom taipy.gui import Gui
import plotly.express as px

df = px.data.gapminder()
df_first = df[df['year']==2007].reset_index(drop=True)
bubble_fig = px.scatter(df_first, x="gdpPercap", y="lifeExp",size="pop", hover_name="country", log_x=True, size_max=60)

selected_state = []

def on_change(state,var,val):
 print("val = ", val)
 if var == 'selected_state':
    print(state.selected_state)
    print("selected state", val, "row:", state.df_first.iloc[val])

md = """
<|chart|figure={bubble_fig}|selected={selected_state}|>
  ```
Gui(md).run(port=4563)

Expected behavior
selected_state variable shouldn't be empty, it should contain a list of indexes corresponding to the selected points.
Runtime environment
Taipy develop branch
Windows 11

What I see is that there are 2 events fired on box selection:

the first one contains the list of indexes
the second one is empty ...

Yes, not sure why we get 2 events here. This happens for both box and lasso selection.


CONTRIBUTING doc changes

What does this PR do?
Minor changes: update of the templates links and Detekt/Ktlint versions used.
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

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

Project coverage is 70.06%. Comparing base (ce17163) to head (099f99a).
Report is 2 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #2275      +/-   ##
===========================================
+ Coverage    70.05%   70.06%   +0.01%
===========================================
  Files          732      732
  Lines        27209    27214       +5
  Branches      4579     4579
===========================================
+ Hits         19060    19065       +5
+ Misses        6876     6869       -7
- Partials      1273     1280       +7

see 29 files with indirect coverage changes


Is all content migrated here already?

Hi,
I cannot find the contents of the actual spec in the website. Has the PDF/Word document been migrated to asciidocs already?
Thanks,
Sebastian

Hi Seb,
You can pull down the .docx or .pdf from http://restfulobjects.org.
Funny you should mention having this spec in asciidoc ... been meaning to do this for a long while, and have just started on that; see https://github.com/restfulobjects/spec (work in progress, will take a while...)
Cheers
Dan


Update empty state illustrations for balances and activity

Please update empty states illustrations with new illustration system:

Balances: https://www.figma.com/file/VupGh90FJtT0dcBj2Sh7bb/📏Spec?node-id=603%3A17688

Activity: https://www.figma.com/file/VupGh90FJtT0dcBj2Sh7bb/📏Spec?node-id=603%3A16925

Browser not supported: https://www.figma.com/file/GShZH28L05eyKjRZxPb8SK/✏Illustration-System?node-id=1%3A9751

I am going to move the last illustration here to a new issue in the Connect repo.

Test passed.Moving it out of QA


How to get list of collections?

I am using Map types to store collections in a Map object of databases. But being able to get the list of collection names and directly creating references to those collections like dbInsance.collection('collectionName') would be great. Or is there? I searched through documents and couldn't find it.

Very basic jsdoc docs have been added, hopefully in the future people will at least be aware of the existence of the method.


v6 Direction for path param parsing

We started playing with this package for v6 (which is looking great 👍 ) when it was using parsers like numberParser these threw if it was bad input.
Now with the move to numberType you either pass a fallback or the returned value could be undefined which requires checking for bad input in every route or (just swallowing it by passing -1 or NaN)
Whilst the errors were jarring at first, using an ErrorBoundary you could handle them globally or per nested set of routes, especially if the error thrown was rich.
Curious on what you are thinking for handling parse errors going forward (just rely on fallbacks, is there an option to still throw)?

There also doesn't seem to be a way to specify fallbacks for oneOf?

There also doesn't seem to be a way to specify fallbacks for oneOf?

Something like this should work:
oneOfType(...Object.values(Direction))(Direction.Ascending)

Granted, it may be a bit confusing.oneOfType is not a type, but a function returning a type.
Regarding throwing vs returning undefined. Throwing was rather a legacy from the previous implementation than a thoughtful decision. In v5, there were regexps for path params, and parsers were used to properly type them. React-router v5 wouldn't match paths with invalid params, and therefore throwing was perfectly fine, because it meant that there is a developer error and the parser in question doesn't match the regexp. (Or that the component is being rendered on an unexpected path.)
In contrast, react-router v6 will happily match routes with any parameters. In other words, we now have to deal with invalid parameters in our components. I think that returning undefined enables more fine-grained control in this situation. It's also similar to what react-router docs propose, and it also aligns with how types (like numberType) work for search params and state.
Note that implicit params will never be undefined because there is no parsing involved.
For now, if you need to throw errors, you can do it like this:
// Define a helper function
function assertIsDefined<T>(value: T | null | undefined): asserts value is T {
  if (value === null || value === undefined) {
    throw new Error('Assertion failed: Expected value to be defined.');
  }
}

// In some component
const { id } = useTypedParams(ROUTES.MY_ROUTE);
assertIsDefined(id); // After this line TS knows that id is not undefined (or null)

While I'm certain that returning undefined should be enabled by default, I understand that some users might want to always throw errors instead. I'll think about providing a way to configure it.

It's worth noting that I updated the library to return undefined fields instead of omitting them, so now it's possible to assert the whole params object:
function required<T extends Record<string, unknown>>(obj: T): Required<T> {
    if (Object.values(obj).includes(undefined)) throw new Error("Unexpected undefined");

    return obj as Required<T>;
}

// In some component
const { id } = required(useTypedParams(MY_ROUTE)); // TS knows that id is not undefined

Also, version 0.4.0 is now released 🥳.
The above-mentioned option is delayed for some future release, because I'm still not sure how to properly implement it 😅

Thanks, I'm in the process of upgrading now.
It really feels like the types should have the optionality baked in. i.e. requiredNumberType or numberType({required:true})
Or even allow fallback to be a function which could throw and allow us to make a requiredNumberType

It really feels like the types should have the optionality baked in.

Yeah, that's what I've eventually settled with. In v0.4.3 it's now possible to specify a special throwable fallback like this:
import { throwable } from "react-router-typesafe-routes/dom"; // Or /native

const ROUTE = route("route/:id", { params: { id: numberType(throwable) } });

This way, in case of a parsing error, the original error will be thrown, and TS is aware of that.

Closing as this seems to be resolved by #16. Please reopen if I'm missing something.

Amazing, thanks!


RFE: support passing through existing env var using -e for exec command

podman run supports simply doing -e FOO to pass through whatever the current value of FOO is. It looks like that's not the case for exec:
$ sudo podman exec -t --env TERM pet /bin/bash
exec failed: container_linux.go:348: starting container process caused "invalid environment 'TERM'"
exit status 1

As expected -e TERM=$TERM does work.

This seems like it should be an easy enough fix. We can probably reuse the environment handling from podman run.

Since 552 is merged I am going to close this issue.


build: Update Vulkan-Headers and VUL to 1.3.291

CI Vulkan-ExtensionLayer build queued with queue ID 226560.

CI Vulkan-ExtensionLayer build # 934 running.

CI Vulkan-ExtensionLayer build # 934 passed.


top gadgets: output flickers with empty results

Description
The output is refreshed every second but some of the seconds have an empty output. See this video:
https://user-images.githubusercontent.com/27575/186199963-8cc8b9de-2bd0-4332-ad14-688f54067bef.mp4
Impact
It does not feel reliable. I don't know if the CLI misses some events or if they are accumulated to the next second's output.
Environment and steps to reproduce

Set-up:
a. AKS cluster (1 node)
b. Install Falco with ebpf driver:

git clone https://github.com/falcosecurity/charts.git
vim falco/values.yaml # replace "kind: module" by "kind: ebpf"
vim falco/Chart.yaml # delete dependency on falcosidekick
helm upgrade --install \
        falco ./falco \
        --create-namespace \
        --namespace falco \
        --values ./falco/values.yaml

Task:
Action(s):
a. ./kubectl-gadget top ebpf -o custom-columns=node,progid,type,name,runtime,runcount,totalruncount,totalruntime --sort totalruntime
Error: See video above

Expected behavior
The output does not get empty results from time to time.
Additional information

This is probably also a more generic issue that the top gadgets have; this could happen whenever stats are not reported exactly within the same interval specified from all the nodes (so most likely a timing/latency issue).
See https://github.com/kinvolk/inspektor-gadget/blob/main/cmd/kubectl-gadget/top/block-io.go#L215
When printing the stats, the previously received stats will be cleared (https://github.com/kinvolk/inspektor-gadget/blob/main/cmd/kubectl-gadget/top/block-io.go#L215) - if the next PrintStats() happens, before new stats are received, it will result in this "flickering". A possible solution would be to not clear the map on printing (or maybe cache the results for one interval) - however, that could actually lead to more unreliable stats, even if it feels more reliable ;)
@mauriciovasquezbernal Any other ideas?

This is probably also a more generic issue that the top gadgets have

That's correct. It's a generic issue for all of them.
It's a problem with the top gadgets design itself where each gadget pod sends an update and kubectl-gadget collects all of them. Since there is not any synchronization amount the client and the gadget pods, some times the client receives two updates from a pod in a single interval, causing this effect. This is same cause of https://github.com/kinvolk/inspektor-gadget/issues/716.
I think a possibility is to switch to a "pull" approach where the kubectl-gadget asks the gadgets pods for an update instead of them pushing it. In this way, the client handles all the synchronization and we avoid missing or receiving two updates on the same interval.

I think a possibility is to switch to a "pull" approach where the kubectl-gadget asks the gadgets pods for an update instead of them pushing it. In this way, the client handles all the synchronization and we avoid missing or receiving two updates on the same interval.

I don't think it's worth switching to the pull approach, as you would run into the same issue when there's more than one node. At one point you have to decide when to output the stats and that may be a time when node 1 answered but node 2 has not yet. For UX reasons, I would suggest that we cache times for one interval - if there's no new data, we would use the old values (for one more interval).

I don't think it's worth switching to the pull approach, as you would run into the same issue when there's more than one node. At one point you have to decide when to output the stats and that may be a time when node 1 answered but node 2 has not yet.

I don't know if I am missing something but I don't understand why it should be a problem. If at that time only node 1 answered, then we simply output its stats. It is not mandatory to have stats from both nodes to output them, or is it? Am I missing something?
In any case, the problem persists if no nodes generate stats. In such a situation, we will have a "flickering" and that's where StatsCombiner could help. See my comment on the PR.

Reopening as https://github.com/inspektor-gadget/inspektor-gadget/pull/898 only contains the "snapshotcombiner" but it's not used yet.

Fixed by https://github.com/inspektor-gadget/inspektor-gadget/pull/1396


Script Packt not working on the first time

Hello,
I'm using the lastest release and having an minor issue. every day I need to execute 2 times the packt-cli -gd command to get the ebook. You can see the error below.
You can ask me anything, logs, troubleshoot. I'm at your disposal.
root@computer:/opt/Packt-Publishing-Free-Learning# packt-cli -gd
[INFO] - JWT token has been fetched successfully!
[INFO] - Start grabbing ebook...
[INFO] - Getting your books data...
[INFO] - Books data has been successfully fetched.
[INFO] - Started solving ReCAPTCHA on Packt Free Learning website...
[INFO] - Waiting for completion of the 177670279 task...
[SUCCESS] - Solution found for 177670279 task.
[INFO] - A new Packt Free Learning ebook "Hands-On Penetration Testing on Windows" has been grabbed!
[INFO] - Title: "Hands-On Penetration Testing on Windows"
[INFO] - Downloading ebook: "Hands-On Penetration Testing on Windows" in pdf format...
[ERROR] - Invalid URL 'None': No schema supplied. Perhaps you meant http://None?
[INFO] - Title: "Hands-On Penetration Testing on Windows"
[INFO] - Downloading ebook: "Hands-On Penetration Testing on Windows" in epub format...
[ERROR] - Invalid URL 'None': No schema supplied. Perhaps you meant http://None?
[INFO] - 0 ebooks have been downloaded!
[SUCCESS] - Good, looks like all went well! :-)

root@computer:/opt/Packt-Publishing-Free-Learning# packt-cli -gd
[INFO] - JWT token has been fetched successfully!
[INFO] - Start grabbing ebook...
[INFO] - Getting your books data...
[INFO] - Books data has been successfully fetched.
[INFO] - You have already claimed Packt Free Learning "Hands-On Penetration Testing on Windows" offer.
[INFO] - Title: "Hands-On Penetration Testing on Windows"
[INFO] - Downloading ebook: "Hands-On Penetration Testing on Windows" in epub format...
[PROGRESS] - [################################################# ] 100.0%
[SUCCESS] - Ebook "Hands-On Penetration Testing on Windows" in epub format downloaded successfully!
[INFO] - Title: "Hands-On Penetration Testing on Windows"
[INFO] - Downloading ebook: "Hands-On Penetration Testing on Windows" in pdf format...
[PROGRESS] - [################################################# ] 100.0%
[SUCCESS] - Ebook "Hands-On Penetration Testing on Windows" in pdf format downloaded successfully!
[INFO] - 2 ebooks have been downloaded!
[SUCCESS] - Good, looks like all went well! :-)
root@computer:/opt/Packt-Publishing-Free-Learning#

Looks like duplicate of #135. Unfortunately I couldn't help then, I'll think again what solution we could devise, but I cannot promise anything.
The (?) temporary solution may be cloning the repository, adding time.sleep(60) (maybe a few seconds will be enough, maybe waiting 60 seconds is not necessary) before download_products(api_client, download_directory, formats, [product_data], into_folder=False) in line 88 and installing the script from local repository (pip3 install -e .).
Note that I couldn't reproduce that issue before, so I'm only suggesting ranrinc's solution. Does his solution work for you?

Yes it looks like good now. I'll see in the future.
Thank you mjenczmyk ;)

Hello,
Sorry to bring this up again, but the time.spleep(60) didn't help. Tell me if you want logs or anything, thanks ;)
root@computer:~# packt-cli -gd --cfgpath /opt/Packt-Publishing-Free-Learning/configFile.cfg
[INFO] - JWT token has been fetched successfully!
[INFO] - Start grabbing ebook...
[INFO] - Getting your books data...
[INFO] - Books data has been successfully fetched.
[INFO] - Started solving ReCAPTCHA on Packt Free Learning website...
[INFO] - Waiting for completion of the 5200506 task...
[SUCCESS] - Solution found for 5200506 task.
[INFO] - A new Packt Free Learning ebook "Mastering Python for Networking and Security" has been grabbed!
[INFO] - Title: "Mastering Python for Networking and Security"
[INFO] - Downloading ebook: "Mastering Python for Networking and Security" in pdf format...
[ERROR] - Invalid URL 'None': No schema supplied. Perhaps you meant http://None?
[INFO] - Title: "Mastering Python for Networking and Security"
[INFO] - Downloading ebook: "Mastering Python for Networking and Security" in epub format...
[ERROR] - Invalid URL 'None': No schema supplied. Perhaps you meant http://None?
[INFO] - 0 ebooks have been downloaded!
[SUCCESS] - Good, looks like all went well! :-)

OK, I may have an idea - we can write a function executing another function and retrying a few times after some time if condition taking function result is not satisfied (and timeout after given time).
def wait_until(action, predicate, timeout, retry_after)

This way we could wrap (or decorate) lambda: get_product_download_urls(api_client, product_id) with wait_until, so it would wait as long as it's needed. I'll write that code someday (maybe next week, I cannot promise it'll be this one) and let you know so you can test it.

It's been a while since Packt doesnt provide other formats than html (kinda bookdown). My question is that this script allows downloading books in this format too?
Thanks

@danmaxis I've moved the discussion to another issue, as it's an off topic here.

@luk6xff Let's leave it open for a bit longer, I've an idea how to solve this problem (although I don't have any idea why I don't encounter this problem).

@VictorJ76 Please try installing script with pip install git+https://github.com/luk6xff/Packt-Publishing-Free-Learning.git@timeout command and see if it helps. If not we may try to increase the timeout value.
If that helps install script with pip install packt and confirm that it does not work - that way we'll make sure whether #166 helps.

@VictorJ76 Please try installing script with pip install git+https://github.com/luk6xff/Packt-Publishing-Free-Learning.git@timeout command and see if it helps. If not we may try to increase the timeout value.

That has worked for me. Excellent!

Glad I could help. Writing this wait_for_computation decorator was interesting itself, I haven't wrote in Python for some time until this pull request ;)
I'll merge it, release the new version and close the issue.

Resolved by #166.


Author entity page needs to handle authors with limited data

Ensure the author/agent page can properly display authors that have no Wikidata or DbPedia information and only limited catalog data.

Created this retroactively for tracking purposes, as the work has already been completed.


Add an "insert at cursor" option for Wildcards

Feature Idea
Currently when a Wildcard is clicked to add it to a prompt, it gets added automatically to the end of the prompt.  This behavior is fine for tag and keyword based text interpreters but with the advent of Flux and NLP interpreters, the location and context of the wildcard output becomes much more critical.  Adding a toggle option to allow wildcards to be inserted at the cursor location in the prompt box would eliminate a lot of manual cutting and pasting when formulating Flux prompts using wildcards.
Other
No response

wildcards now go to your cursor pos


The language cheat sheet

It would be nice to have a list of all syntax, maybe with some examples.

Though https://github.com/agentm/project-m36/blob/master/docs/tutd_tutorial.markdown probably is such a cheat sheet.

Indeed, I can imagine a more condensed version of examples for each construct. I've actually been using C.J. Date's now outdated "Database In Depth" which has a final page with Tutorial D examples as a cheatsheet. I'll put something like that together to share.

Let me know what you think of the new cheatsheet.
I didn't recall some of the syntax I myself had implemented, so the cheatsheet will be useful to me, as well.


Receiving wrong values in arrEcgContent array

I am using the SDK with pulsebit ex device. Device is connecting correctly with bluetooth. After I retrieve a detail Ecg object as "VTProEcgDetail" , the problem is that the values in arrEcgContent array is wrong while the other values like heart rate , ecgResult  , date , timeLength etc  are correct.
Can you please tell me what's going wrong here?
Also the values in arrEcgContent array are in range between -5 to 5.

The parser not support Pulsebit EX device.
I will add support for EX in the new version.


[HOTFIX] Constraints Any semantics should preserve false

Tickets addressed: N/A
Review: By commit
Merge strategy: Merge (no squash)

Description

This case was discovered by @camargo in preparation for a demo. A constraint similar to the following was never returning violations:
export default (): Constraint => {
  return Constraint.ForEachActivity(ActivityType.grandchild,
    instance => Discrete.Resource("/flag").equal("B").if(instance.window()));
}

As a reminder, the x.if(y) is syntactic sugar for Any(not y, x).
I stepped through the debugger and found that the Any node was converting false into null in the following case:
(Assume that /flag is never equal to "B")

Instance.window produces three intervals: (Duration.MIN, start time, FALSE), (start time, end time, TRUE), and (end time, Duration.MAX, FALSE) (I'm being imprecise with my 'clusivity because it's irrelevant to my point)
The Invert correctly swaps TRUE with FALSE and vice versa above, resulting in (Duration.MIN, start time, TRUE), (start time, end time, FALSE), and (end time, Duration.MAX, TRUE)
We get up to the Any node, and we immediately OR these three intervals with the empty Windows object (which is null everywhere). This drops the FALSE interval, losing the crucial information we needed to check this constraint.
We emerge from Any with (Duration.MIN, start time, TRUE), (end time, Duration.MAX, TRUE) and we say that there are no violations, since there are no FALSE intervals.

The change I'm proposing is that the Any node should start with the interval FOREVER, FALSE (the identity value for "OR"). This seems reasonable to me, but I'd like feedback from people who have spent more time thinking about this @JoelCourtney @Twisol .
Verification

I updated testOr to check that a false interval in both left and right should be preserved by Any(left, right). This test fails until the next commit, after which it passes.
Documentation

Do we have documentation for how nullable windows behave?
Future work

None

I have these changes in my branch for gaps in external profiles. There's a couple erroneous and calls that should be select too. Its this commit

What I mean to say is, if you could take all of those changes and use them in this PR, that would be great. Then I can drop that commit from my branch, cuz it should've been a separate PR to begin with, like this. Thanks!

What I mean to say is, if you could take all of those changes and use them in this PR, that would be great. Then I can drop that commit from my branch, cuz it should've been a separate PR to begin with, like this. Thanks!

Thanks for the quick review :+1: I'm super down to pull your changes. I tried cherry-picking that commit, and it's failing my "false pass through" test case. It seems like select drops the false as well, and replaces it with null - so maybe Any should use and, not select? Or am I misunderstanding select?

Force push replaces new Windows(Segment.of(Interval.FOREVER, false)); with new Windows(false).

Nevermind! I had neglected to extend the bounds of the simulation results. select was doing its job just fine :+1:

What I mean to say is, if you could take all of those changes and use them in this PR, that would be great. Then I can drop that commit from my branch, cuz it should've been a separate PR to begin with, like this. Thanks!

Done :+1:


--prof-startup leaks PII

VSCode Version: 1.13 master
OS Version: Windows

Steps

Start code with --prof-startup
create a GitHub issue out of it

The issue has absolute path names on my machine hence it could leak PII.

@dbaeumer did you run this out of dev or from the build version? There is a difference between and for ease of dev we don't remove PII when running out of dev: https://github.com/Microsoft/vscode/blob/master/src/vs/base/node/profiler.ts#L28

I ran this out of dev this morning. Let me check with insider.

Tested with insider and it contains the same PII

Ah, that you mean. I thought about us rewriting the paths in the actual profiles. So, the full paths are in the issue because we cannot upload the files and because people forget that in 20% of the cases. Unsure what to do, it is very much in your face and the idea is that people replace that message with actual files

Actually I only noticed it by looking at the preview and I only did this to verify another bug :-) Normally you see this:

Most people will simply press submit new issue.

What's about changing the text to something like 'don't forget to attache xxxx from your home directory'.


Add Batch Create and Batch Get Measurements Methods.

Large Reports can result in a significant amount of Measurements and individual method calls for each one of those Measurements is a performance bottleneck.

This change is


[Study Plan(ER)] Set up Basic Database Schema

to be solved together with @bsweeyee since he has made similar script for miniMongo at the client side


Review: services list from GOV.UK site search

I remembered that we added a /services finder to GOV.UK site search in 2019, which lists ~7,300 services currently. Should this list be reviewed at some point for possibly adding to the x-gov list?
This is more of a discussion than an issue, sorry @frankieroberto!

@stevenjmesser thanks – do you know how that differs to https://www.gov.uk/api/search.json?filter_format=transaction?


Extension issue

Issue Type: Bug
Extension Name: html-css-class-completion
Extension Version: 1.19.0
OS Version: Windows_NT x64 10.0.19042
VSCode version: 1.48.2

:warning: We have written the needed data into your clipboard. Please paste! :warning:

Already reported here: https://github.com/zignd/HTML-CSS-Class-Completion/issues/155


Enemy check treats allied factions as enemies

When trying to !hangar save or load a grid, the enemy detection radius seems to be including factions you are allied with in game ("Peace") as "enemies" and prevents using the hangar

It will ignore allies now

As of Sept 19th players still reported not being able to load from hangar if an allied faction is nearby

Pretty sure this has been fixed. Just need to update it


Problem during make install: failed to create symbolic link '/usr/lib/x86_64-linux-gnu/libclang.so': File exists

ln -s /usr/lib/x86_64-linux-gnu/libclang-*.so.1 /usr/lib/x86_64-linux-gnu/libclang.so
Error
ln: failed to create symbolic link '/usr/lib/x86_64-linux-gnu/libclang.so': File exists
Ubuntu 16, Anaconda with Python 3.6

Thank you for pointing this out Tyler. This has been fixed in the latest commit.

Thank you!


rolling_update: fix mon+rgw/multisite collocation

When monitors and rgw are collocated with multisite enabled, the
rolling_update playbook fails because during the workflow, we run some
radosgw-admin commands very early on the first mon even though this is
the monitor being upgraded, it means the container doesn't exist since
it was stopped.
This block is relevant only for scaling out rgw daemons or initial
deployment. In rolling_update workflow, it is not needed so let's skip
it.
Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1970232
Signed-off-by: Guillaume Abrioux<EMAIL_ADDRESS>

jenkins test centos-non_container-switch_to_containers

jenkins test centos-container-all_daemons

jenkins test centos-container-all_daemons

jenkins test centos-container-all_daemons

jenkins test centos-non_container-switch_to_containers

failure is unrelated


Allow applying prompt template to the input for huggingface models.

Summary
The current Garak treats all huggingface model as base models and does not consider the prompt template. However, for those instruct tuned models, templates are necessary to let them generate proper responses. So I am wondering whether it will be good if we can add a parameter in the yaml config file to configure whether we apply the prompt template to the input to distinguish instruct model with base model.
Basic example
If the proposal involves a new or changed API, include a basic code example. Omit this section if it's not applicable.
Motivation
Why are we doing this? What use cases does it support? What is the expected outcome?
Insturct models need prompt template to maximize its performance and users also use prompt template to interact with those models, so allowing prompt template will simulate the real use scenario better.

Thanks Zijia! Is is possible to edit the issue to include a very basic example?

Sure!
An very naive example to apply template to the prompt is to add
prompt = self.generator.tokenizer.apply_chat_template([{"role":"user", "content": prompt}], tokenize=False, add_generation_prompt=True)

before this line. This code works for most probes, but it is not applicable to some probes such as atkgen because those probes somehow lack tokenizer.chat_template. Here is the error I encountered when testing atkgen probes after adding the above code to apply template
  File "/usr/lib/python3.10/runpy.py", line 196, in _run_module_as_main                                        | 0/10 [00:00<?, ?it/s]
    return _run_code(code, main_globals, None,
  File "/usr/lib/python3.10/runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "/home/zijiac/.local/lib/python3.10/site-packages/garak/__main__.py", line 14, in <module>
    main()
  File "/home/zijiac/.local/lib/python3.10/site-packages/garak/__main__.py", line 9, in main
    cli.main(sys.argv[1:])
  File "/home/zijiac/.local/lib/python3.10/site-packages/garak/cli.py", line 530, in main
    command.probewise_run(
  File "/home/zijiac/.local/lib/python3.10/site-packages/garak/command.py", line 237, in probewise_run
    probewise_h.run(generator, probe_names, evaluator, buffs)
  File "/home/zijiac/.local/lib/python3.10/site-packages/garak/harnesses/probewise.py", line 107, in run
    h.run(model, [probe], detectors, evaluator, announce_probe=False)
  File "/home/zijiac/.local/lib/python3.10/site-packages/garak/harnesses/base.py", line 114, in run
    attempt_results = probe.probe(model)
  File "/home/zijiac/.local/lib/python3.10/site-packages/garak/probes/atkgen.py", line 137, in probe
    challenge = self.redteamer.generate(query)[0]
IndexError: list index out of range

And in the garak.log, the corresponding error message is as below:
ERROR  Cannot use chat template functions because tokenizer.chat_template is not set and no template argument was passed! For information about writing templates and setting the tokenizer.chat_template attribute, please see the documentation at https://huggingface.co/docs/transformers/main/en/chat_templating


Add Waveterm Icon to Linux

Issue #97 brought up that the icon does not appear on linux.  This change provides the wave-logo-dark.png to the Electron Browser Window to fix that.

Ah, we should only conditionally add this if the platform is linux.  We don't want this icon to override the "icns format" MacOS icon.  can check for unamePlatform == "linux"


feat: add cwgo docs

What type of PR is this?
feat

What this PR does / why we need it (en: English/zh: Chinese):

en: Add cwgo docs.
zh: 新增 cwgo 文档。
Which issue(s) this PR fixes:

NA

PATL @Duslia  @welkeyever @GuangmingLuo


TypeError in brigade

I got the following error..
± b build logs 01cp3at17bpwnkars7yng7dh74

==========[  brigade-worker-01cp3at17bpwnkars7yng7dh74  ]==========
prestart: no dependencies file found
prestart: no script override
TypeError: Cannot read property 'toString' of undefined
error Command failed with exit code 1.

note, b is an alias to brig in the namespace I use
Notice, this came before the message I usually see of:
[brigade] brigade-worker version: 0.17.0

This happened for the charts issue at https://github.com/helm/charts/pull/7424. Is this because brigade.js didn't exist on the PR branch? If so, it would be nice if there was an easier to follow message so I could easily pick this up.

I found the source of the TypeError.
https://github.com/Azure/brigade/blob/c5907e734358eaec08cf4cb83a9d059c1c622647/brigade-worker/prestart.js#L31-L34
It's the e.toString() call here. Happens when there is no brigade.js or default file.

fixed via #621


Fixes visual DatePicker bug

Long text exits the screen on the right, making the Tooltip out of the display. The "Siste dagen i måneden" button also had zero margin on the bottom, making it too close to the "Opprett avtale" button.
#615

Tested on devices

[ ] Desktop 💻
[ ] Mobile 📱

Tests

[ ] All tests are running ✔
[ ] Test are updated 🧪
[ ] Code Review 👩💻
[ ] QA 👌

⏲ Time spent on CR:
⏲ Time spent on QA:

Warning: don't merge before fixing vulnerabilities in the dependencies. npm audit fix does not automatically solve them, and it may require manual fixing. Unfortunately, I don't have time to investigate it before this weekend.


Question: Database type and engine compatibility?

Ideally the database type (SQL, NoSQL) and database engine used should not have any impact on the funtionality of graphene-subscriptions. However just to make sure, is there any known compatibility issue w.r.t. database engine (database type covered implicitly)?

@fkromer I'm not aware of any compatibility issues, but I haven't done wide testing. graphene-subscriptions should work with any database that is supported by the Django ORM, but it may work for other ORMs as well. As long as the ORM implementation that's being used is compatible with Django's JSON serializer, then it should work properly.

@jaydenwindle Thanks!


Is there a way to debug the code using an IDE?

I like the idea of developing my script locally, but it seems that there is no way to debug it, since the google functions are not accessible locally.

You can write a local mock server that simulates the Google Script API calls and can be invoked only when the proces.env. NODE_ENV environment is not production.

See https://github.com/dan-kushnaryov/gas-mock-globals


gateway-api: Sync up with the latest upstream v1.2.0-rc1

Support for https://github.com/kubernetes-sigs/gateway-api/releases/tag/v1.2.0-rc1

ci-gateway

ci-gateway-api

/ci-gateway-api

/ci-gateway-api

/test

/ci-gateway-api

Seems like something is broken after rebase with main branch, marking this as draft.

/ci-gateway-api

/ci-gateway-api

/test


coredump: add config/API for registering memory for dump

Introduce new config and API to allow for dynamically adding
(and removing) regions of memory to be collected at time of coredump.
Add coredump_memory test to verify registered memory is collected in
dump when configured.
Signed-off-by: Mark Holden<EMAIL_ADDRESS>

The pushes after dcpleung's review have just been to rebase and make sure the change still merges/functions ok, no code changes.

@tejlmand please take another look

Resolving merge conflicts

@tejlmand, @gmarull do the latest changes look ok?

Would love to see a coredump pseudo-device with DT bindings 😀

@mrkhldn  - This would be the perfect candidate for a pseudo-device driver + DT bindings 👀

@mrkhldn - This would be the perfect candidate for a pseudo-device driver + DT bindings 👀

@cfriedt, this is an interesting idea. Can you help me understand the thinking a bit more? What advantage do we get changing this into a pseudo-device driver?
@dcpleung, any thoughts on the suggested approach?

Hm... maybe the idea is to use device tree to define all the mappings at build time, and skip the runtime stuff? From a security standpoint this is (arguably) better since the region list can reside in read-only memory, and the list can't be manipulated externally to dump other addresses.

@mrkhldn - This would be the perfect candidate for a pseudo-device driver + DT bindings 👀

@cfriedt, this is an interesting idea. Can you help me understand the thinking a bit more? What advantage do we get changing this into a pseudo-device driver?
@dcpleung, any thoughts on the suggested approach?

@mrkhldn, as @dcpleung mentioned, with a pseudo-device driver, addresses are stored in ROM as constants. I still think it's important to support the dynamic use case, of course.
However, a kind of nice part  about it is that you get multi-instance for free. Additional coredump areas can be specified with complete independence and flexibility for dedicated memory areas in heterogenous layouts (or potentially also DMA memory, or shared bus memory, or really any logged or buffered data). This type of information is fairly helpful in post-mortem analysis.


login with google in signup page

Is your feature request related to a problem? Please describe.
I think there should login with google functionality in signUp page

Assign this issue to me under GSSOC'23

This Feature is not using in this project !
Thank you !


support multiple paths in dist --

Much feature work involves multiple files. It would be nice to be able to build multiple files without building the whole feature set.

This should work already. For example, if I comment out a compat feature from a.yml and base.yml, then run npm run dist -- features/a.yml features/base.yml, those files (and only those files) are updated.
Can you provide reproduction steps for a case where this is not working?


fix(scan-dirs): add file pattern to the dir glob's basename is *

fix #399

Maybe don‘t need to handle this case? The option named dirs, so need user to make sure it must be the dir globs


PhotosLibrary().selection hangs if nothing selected

At least on Catalina, PhotosLibrary.selection appears to hang (never returns) if nothing was selected.

On Ventura this works fine -- returns an empty list.


Add :cached to override bind mounts

Similar to 7b1aed6305d07dbcc38ec9812e6cb90b073fa192, default to using
:cached mode on mounted volumes in
docker-compose-dist.override.yml to avoid high CPU usage on
Docker4Mac.  See [1], [2].  If you've already created a
docker-compose.override.yml, manual append :cached to all volumes
except those under emberosf and rabbitmq.
[1] https://docs.docker.com/docker-for-mac/osxfs-caching/#cached
[2] https://github.com/docker/for-mac/issues/1759#issuecomment-366632793

Purpose

Changes

QA Notes

Documentation

Side Effects

Ticket

Ah, good catch!


Invalid links in the README.md

for example cli commands

Below is the list of valid links for update:
NeuraLegion’s solutions
Repeater
www.nexploit.app
setting up an API key
Setting up a New Repeater
a full list go here
[NexPloit CLI polling] (http://localhost:3000/#/guide/np-cli/commands/checking-scan-status)

Here are valid links
Repeater
setting up an API key
Setting up a New Repeater
NexPloit CLI polling


docs: add explainations to generate migrations with TypeORM

Description
As the title says.
To-Dos

[ ] Successful build pnpm build
[ ] Translation keys pnpm i18n:extract
[ ] Database migration (if required)

:tada: This PR is included in version 2.2.0 :tada:
The release is available on:

v2.2.0
GitHub release

Your semantic-release bot :package::rocket:


partners[anthropic]: update docstring of the ChatAnthropic and AnthropicLLM classes

Description: Update docstring of the ChatAnthropic and AnthropicLLM classes
Issue: Not applicable
Dependencies: None

update with changes after pr review of a similar change langchain-ai/langchain#18637

update - add a model name in the call to ChatAnthropic() based on review comment received for langchain-ai/langchain#18868

@efriis
please review this pr when convenient

@efriis  Please review this pr. The change is minor but it (slightly) improves the usability of the system.


kmod package isn't installed

I tried to run lsmod (which is part of kmod package) but it was missing in opensuse:13.2 image. This package was part of image before.
Is this change made by mistake or it is on purpose? Thank you for your answer.

We deliberately removed this package from the official images because 99% of the times there's no reason to allow a container to mess with the kernel of the host.
You can add the package back by installing it manually inside of your Dockerfile.