docs(String) add some documentation for String

I saw "Contributing.md" said you were interested in more docs for the standard library.
I want to learn more about Crystal and I like writing docs so I thought I'd try. I've tried to take some clues from the surrounding documentation. What do you think of these?
Also, if you have the time, I have some questions:

How do you like these docs? Is there anything I can do to make them better?
How do you want to handle cases for methods with the same behavior but different signatures (eg, one accepts a Char and the other accepts a String, or one yields but the other returns an Array)? I see that the Ruby API docs group these methods, which seems good to me. I've often documented just one of the methods, but included the different signatures in the example.
For future documentation, is there anything that you'd like documented more quickly? Or anything that should be left undocumented for now?

Thanks!
I think the time has come to decide how to document things. I usually prefer to use the 3rd person, as in "Tests whether str matches..." instead of "Test whether". I always imagine myself asking "What does this method do?" and the answer being "(it) tests...".
Opinions? We can include this decision in the Contributing.md so everyone follows this rule.

:+1: for third person, it also matches is_a?, responds_to?, includes?, starts_with?
I'm happy to go back and update my docs & Contributing.md.
(By the way, THANK YOU for using third-person method names in the standard library, I think it's much better!)

I just added this section to the docs so we are all one the same page :-)
@rmosolgo Thanks for documenting this and wanting to document even more! Documentation is so, so important. It reduces the amount of questions made and answers needed, and makes everything look much more professional. So your help is really needed and appreciated.

Thanks for your feedback, I've updated to follow that styleguide (very helpful, good idea!):
https://github.com/rmosolgo/crystal/commit/321d5700a5592c73dd1363ce59f24fb2afe15337
Let me know what you think, or if it looks good, I'll squash them!

LGTM :+1:, squashing would be much appreciated ;)

rebased!

Thanks!


Bot is not running

Hey everyone, I want to run this example with bun but it didn't work, why?
code
import { ChatInputApplicationCommandBuilder, Session } from "@biscuitland/core";
import { GatewayIntents } from "@biscuitland/api-types";

const session = new Session({
  token: "TOKEN",
  intents: GatewayIntents.Guilds,
});

const commands = [
  new ChatInputApplicationCommandBuilder()
    .setName("ping")
    .setDescription("Replys with pong!")
    .toJSON(),
];

session.events.on("ready", async ({ user }) => {
  console.log("Logged in as:", user.username);
  await session.upsertApplicationCommands(commands, "GUILD_ID");
});

session.events.on("interactionCreate", (interaction) => {
  if (interaction.isCommand()) {
    if (interaction.commandName === "ping") {
      interaction.respond({ with: { content: "pong!" } });
    }
  }
});

session.start();

output:

Bun does not have a proper polyfill for the websocket module, we have to deliver somehow a webspec version to the npm registry which is really hard since Node doesn't have webspec websockets
Please look at the third known issue or write an http application with no gateway


Need help with convert query from Mysql to Mongo

public function calculationsCount()
    {
        return $this->calculations()->select([
            DB::raw('count(*) as count'), 'user_id'
        ])->groupBy('user_id');
    }

Thanks!

public function calculationsCount()
{
    return $this->raw(function($collection)
    {
        return $collection->aggregate([
            [
                '$group' => [
                    '_id' => '$user_id',
                    'count' => [
                        '$sum' => 1
                    ]
                ],
            ],
        ]);
    })["result"];
}

@alexeyriva I cant use it in this case User::with('calculationsCount')->get()

@vanchelo This is not a Mongo support forum!


Error When importing

Hi There,
I have been using these 2 commands for awhile and they work fine
Export-Startlayout –path C:\Temp\StartMenu.xml
Import-StartLayout –LayoutPath C:\Temp\StartMenu.xml -MountPath:C:
I have changed the way we are using the StartMenu with stacked icons.
The export command works fine but the import command returns an error: Import-StartLayout : The file C:\temp\StartMenuLayout.xml is not a valid layout file.
This article doesn't mentioned that you cannot use stacked icons, is this an oversight, because this is a great feature and I'm sure other end users would follow. please contact me if you require more information.
Thanks
Nathan

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

ID: 79fccecd-5484-0304-d82b-9425f1889a22
Version Independent ID: 4df6b138-c1b5-b551-22c9-19d3d1f9105d
Content: Customize and export Start layout (Windows 10)
Content Source: windows/configuration/customize-and-export-start-layout.md
Product: w10
GitHub Login: @jdeckerMS
Microsoft Alias: jdecker

@nathanallan-geelongcollege What are "stacked icons"?

Hi Jeanie
When you move another icon on top of another icon you can stack them into a group/folder.
I have attached a screen shot.
Nathan
From: Jeanie Decker<EMAIL_ADDRESS>Sent: Monday, 30 July 2018 10:49 PM
To: MicrosoftDocs/windows-itpro-docs<EMAIL_ADDRESS>Cc: Nathan Allan<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/windows-itpro-docs] Error When importing (#1377)
@nathanallan-GeelongCollegehttps://github.com/nathanallan-GeelongCollege What are "stacked icons"?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/windows-itpro-docs/issues/1377#issuecomment-408851463, or mute the threadhttps://github.com/notifications/unsubscribe-auth/An8iei-k61wgMcuMC3PLp8MQ2njfqxVvks5uLwCpgaJpZM4VlyIr.

IMPORTANT (1) The contents of this email from The Geelong College and any attachments are confidential and intended solely for the person to whom it is addressed. The contents represent the views or opinions of the author only and are not necessarily made on behalf of the School. If you are not the intended recipient of this message or their part, or if this message has been addressed to you in error, please immediately alert the sender by reply email and then delete this message and any attachments. (2) Before opening or using attachments, check them for viruses and defects. The contents of this email and its attachments may become scrambled or altered in transmission. Please notify us of any anomalies. The school does not accept liability for any damage caused by a transmitted virus. (3) Any personal information in this email must be handled in accordance with the Privacy Act 1988. #g*87 (4) Please consider the environment before printing this email.

@nathanallan-GeelongCollege What version of Windows 10 are you applying the Start layout to?

Hi Jennie
We are using version 1803 Enterprise
Thanks
From: Jeanie Decker<EMAIL_ADDRESS>Sent: Wednesday, 1 August 2018 2:57 AM
To: MicrosoftDocs/windows-itpro-docs<EMAIL_ADDRESS>Cc: Nathan Allan<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/windows-itpro-docs] Error When importing (#1377)
@nathanallan-GeelongCollegehttps://github.com/nathanallan-GeelongCollege What version of Windows 10 are you applying the Start layout to?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/windows-itpro-docs/issues/1377#issuecomment-409292673, or mute the threadhttps://github.com/notifications/unsubscribe-auth/An8ietMJe3w417JuVcNiqelPfM33Mxeaks5uMIxXgaJpZM4VlyIr.

IMPORTANT (1) The contents of this email from The Geelong College and any attachments are confidential and intended solely for the person to whom it is addressed. The contents represent the views or opinions of the author only and are not necessarily made on behalf of the School. If you are not the intended recipient of this message or their part, or if this message has been addressed to you in error, please immediately alert the sender by reply email and then delete this message and any attachments. (2) Before opening or using attachments, check them for viruses and defects. The contents of this email and its attachments may become scrambled or altered in transmission. Please notify us of any anomalies. The school does not accept liability for any damage caused by a transmitted virus. (3) Any personal information in this email must be handled in accordance with the Privacy Act 1988. #g*87 (4) Please consider the environment before printing this email.

Hi Jennie
Just wondering if you have any news as of yet?
Thanks
Nathan
From: Nathan Allan
Sent: Wednesday, 1 August 2018 8:12 AM
To: 'MicrosoftDocs/windows-itpro-docs'<EMAIL_ADDRESS>Subject: RE: [MicrosoftDocs/windows-itpro-docs] Error When importing (#1377)
Hi Jennie
We are using version 1803 Enterprise
Thanks
From: Jeanie Decker <notifications<EMAIL_ADDRESS>Sent: Wednesday, 1 August 2018 2:57 AM
To: MicrosoftDocs/windows-itpro-docs <windows-itpro-docs<EMAIL_ADDRESS>Cc: Nathan Allan <Nathan.Allan<EMAIL_ADDRESS>Mention <mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/windows-itpro-docs] Error When importing (#1377)
@nathanallan-GeelongCollegehttps://github.com/nathanallan-GeelongCollege What version of Windows 10 are you applying the Start layout to?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/windows-itpro-docs/issues/1377#issuecomment-409292673, or mute the threadhttps://github.com/notifications/unsubscribe-auth/An8ietMJe3w417JuVcNiqelPfM33Mxeaks5uMIxXgaJpZM4VlyIr.

IMPORTANT (1) The contents of this email from The Geelong College and any attachments are confidential and intended solely for the person to whom it is addressed. The contents represent the views or opinions of the author only and are not necessarily made on behalf of the School. If you are not the intended recipient of this message or their part, or if this message has been addressed to you in error, please immediately alert the sender by reply email and then delete this message and any attachments. (2) Before opening or using attachments, check them for viruses and defects. The contents of this email and its attachments may become scrambled or altered in transmission. Please notify us of any anomalies. The school does not accept liability for any damage caused by a transmitted virus. (3) Any personal information in this email must be handled in accordance with the Privacy Act 1988. #g*87 (4) Please consider the environment before printing this email.

@nathanallan-GeelongCollege Sorry, I don't have an answer yet but I'm still working on it.

@nathanallan-GeelongCollege Did you name the folder of stacked icons in the XML?

Hi Jennie
Looks like I did not name the folder.
I have attached a copy on the startmenu export
Nathan
From: Jeanie Decker<EMAIL_ADDRESS>Sent: Tuesday, 21 August 2018 12:58 AM
To: MicrosoftDocs/windows-itpro-docs<EMAIL_ADDRESS>Cc: Nathan Allan<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/windows-itpro-docs] Error When importing (#1377)
@nathanallan-GeelongCollegehttps://github.com/nathanallan-GeelongCollege Did you name the folder of stacked icons in the XML?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/windows-itpro-docs/issues/1377#issuecomment-414346614, or mute the threadhttps://github.com/notifications/unsubscribe-auth/An8ieqk36mp2Td0Fcg-jQsTqkF-DQAfzks5uSs6DgaJpZM4VlyIr.

IMPORTANT (1) The contents of this email from The Geelong College and any attachments are confidential and intended solely for the person to whom it is addressed. The contents represent the views or opinions of the author only and are not necessarily made on behalf of the School. If you are not the intended recipient of this message or their part, or if this message has been addressed to you in error, please immediately alert the sender by reply email and then delete this message and any attachments. (2) Before opening or using attachments, check them for viruses and defects. The contents of this email and its attachments may become scrambled or altered in transmission. Please notify us of any anomalies. The school does not accept liability for any damage caused by a transmitted virus. (3) Any personal information in this email must be handled in accordance with the Privacy Act 1988. #g*87 (4) Please consider the environment before printing this email.

@nathanallan-GeelongCollege  Nathan, I don't see an attachment, could you try again?

Hi Jennie
I have zipped the file and attached, if that has not worked here is the code for the file:

[cid:logo4823.jpg]
Nathan Allan
IT Technical Systems & Support Officer
Phone: +613 5226 3150
PO Box 5, GEELONG VIC 3220
www.geelongcollege.vic.edu.auhttp://www.geelongcollege.vic.edu.au/
From: Jeanie Decker<EMAIL_ADDRESS>Sent: Tuesday, 21 August 2018 11:55 PM
To: MicrosoftDocs/windows-itpro-docs<EMAIL_ADDRESS>Cc: Nathan Allan<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/windows-itpro-docs] Error When importing (#1377)
@nathanallan-GeelongCollegehttps://github.com/nathanallan-GeelongCollege Nathan, I don't see an attachment, could you try again?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/windows-itpro-docs/issues/1377#issuecomment-414682113, or mute the threadhttps://github.com/notifications/unsubscribe-auth/An8ievyxV-3Hh-3NIS8fy6SSUsBeoJPPks5uTBEcgaJpZM4VlyIr.

IMPORTANT (1) The contents of this email from The Geelong College and any attachments are confidential and intended solely for the person to whom it is addressed. The contents represent the views or opinions of the author only and are not necessarily made on behalf of the School. If you are not the intended recipient of this message or their part, or if this message has been addressed to you in error, please immediately alert the sender by reply email and then delete this message and any attachments. (2) Before opening or using attachments, check them for viruses and defects. The contents of this email and its attachments may become scrambled or altered in transmission. Please notify us of any anomalies. The school does not accept liability for any damage caused by a transmitted virus. (3) Any personal information in this email must be handled in accordance with the Privacy Act 1988. #g*87 (4) Please consider the environment before printing this email.

@nathanallan-GeelongCollege Thanks, Nathan. I see two issues - one is StartTileGroupCellWidth being set to 8. If you look at the XML reference (https://docs.microsoft.com/en-us/windows/configuration/start-layout-xml-desktop), they're limited to 6 in the LayoutModification.xml. Also, a number of the tiles have a position of -1 for column and row -- I'm not sure how that happened in your export, but there isn't a -1 column or row in which to display the tile.

Hi Jeanie
So I have retried the setup/layout of the start menu on another laptop and the export and import process is now working fine.
Its still odd why the GroupCellWidth is still listing 8 when I have resized the menu to 4, I have attached a screen snip of the startmenu and below is the code of the xml.

From: Jeanie Decker<EMAIL_ADDRESS>Sent: Wednesday, 22 August 2018 8:27 AM
To: MicrosoftDocs/windows-itpro-docs<EMAIL_ADDRESS>Cc: Nathan Allan<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/windows-itpro-docs] Error When importing (#1377)
@nathanallan-GeelongCollegehttps://github.com/nathanallan-GeelongCollege Thanks, Nathan. I see two issues - one is StartTileGroupCellWidth being set to 8. If you look at the XML reference (https://docs.microsoft.com/en-us/windows/configuration/start-layout-xml-desktop), they're limited to 6 in the LayoutModification.xml. Also, a number of the tiles have a position of -1 for column and row -- I'm not sure how that happened in your export, but there isn't a -1 column or row in which to display the tile.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/windows-itpro-docs/issues/1377#issuecomment-414841884, or mute the threadhttps://github.com/notifications/unsubscribe-auth/An8ielI-xIbfMhv3fPoqdOcZXKXqeNFHks5uTIlFgaJpZM4VlyIr.

IMPORTANT (1) The contents of this email from The Geelong College and any attachments are confidential and intended solely for the person to whom it is addressed. The contents represent the views or opinions of the author only and are not necessarily made on behalf of the School. If you are not the intended recipient of this message or their part, or if this message has been addressed to you in error, please immediately alert the sender by reply email and then delete this message and any attachments. (2) Before opening or using attachments, check them for viruses and defects. The contents of this email and its attachments may become scrambled or altered in transmission. Please notify us of any anomalies. The school does not accept liability for any damage caused by a transmitted virus. (3) Any personal information in this email must be handled in accordance with the Privacy Act 1988. #g*87 (4) Please consider the environment before printing this email.

@nathanallan-GeelongCollege Glad it's working now! I don't understand the width of 8 either, but if it's working now, obviously that wasn't the problem. It looks like getting rid of the -1 values helped.

Hi Jeanie
I know you closed the job, but I thought I would send this through.
Every now and then when you pin the icons, the icon will be placed very oddly I think that’s where the negative values are coming from.
I have attached a screen shot and the code is below.
Also I was wondering if you could point me the in correct direction with a SCCM question about activation the option “Show more tiles on Start” during a deployment sequence.
I have had no luck finding any information on this.
Thanks

[cid:logo4823.jpg]
Nathan Allan
IT Technical Systems & Support Officer
Phone: +613 5226 3150
PO Box 5, GEELONG VIC 3220
www.geelongcollege.vic.edu.auhttp://www.geelongcollege.vic.edu.au/
From: Jeanie Decker<EMAIL_ADDRESS>Sent: Thursday, 23 August 2018 10:35 PM
To: MicrosoftDocs/windows-itpro-docs<EMAIL_ADDRESS>Cc: Nathan Allan<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/windows-itpro-docs] Error When importing (#1377)
@nathanallan-GeelongCollegehttps://github.com/nathanallan-GeelongCollege Glad it's working now! I don't understand the width of 8 either, but if it's working now, obviously that wasn't the problem. It looks like getting rid of the -1 values helped.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/windows-itpro-docs/issues/1377#issuecomment-415398475, or mute the threadhttps://github.com/notifications/unsubscribe-auth/An8iel6KmjDlAhh2TgDH8sU8nQBtLKpXks5uTqGAgaJpZM4VlyIr.

IMPORTANT (1) The contents of this email from The Geelong College and any attachments are confidential and intended solely for the person to whom it is addressed. The contents represent the views or opinions of the author only and are not necessarily made on behalf of the School. If you are not the intended recipient of this message or their part, or if this message has been addressed to you in error, please immediately alert the sender by reply email and then delete this message and any attachments. (2) Before opening or using attachments, check them for viruses and defects. The contents of this email and its attachments may become scrambled or altered in transmission. Please notify us of any anomalies. The school does not accept liability for any damage caused by a transmitted virus. (3) Any personal information in this email must be handled in accordance with the Privacy Act 1988. #g*87 (4) Please consider the environment before printing this email.

@nathanallan-GeelongCollege Sorry, I don't have any info on SCCM and didn't find anything specific to your question on search. You might try the IT Pro forum for Win10 deployment, others may have had a similar question or may have worked through it before: https://social.technet.microsoft.com/Forums/en-US/home?forum=win10itprosetup


Make passwords required (in most cases)

Description
This updates the code in the Admin API controller that handles password changes. Now, passwords are always required for new admins.
There are numerous issues with the original password checking code that will require a lot of refactoring, but these are all outside of the scope of the ticket associated with this work.
Motivation and Context
Passwords have been practically required for a while, but the code doesn't enforce the fact that they are.
Affects: https://www.notion.so/lyrasis/Make-password-a-required-field-when-creating-an-account-on-the-CM-ab4313f97fc4465687dfbafc9f64dc1e
How Has This Been Tested?
I created a new admin locally, and ensured all of the existing tests pass.
Checklist

[x] I have updated the documentation accordingly.
[x] All new and existing tests passed.

I should point out that the Admin web interface won't show the password field as being required. I believe it's a one line change in that project to make it show up as required, but I don't have permission to push to the circulation-admin project:
diff --git a/src/components/IndividualAdminEditForm.tsx b/src/components/IndividualAdminEditForm.tsx
index 4e7652d5..cbf281fe 100644
--- a/src/components/IndividualAdminEditForm.tsx
+++ b/src/components/IndividualAdminEditForm.tsx
@@ -134,7 +134,7 @@ export default class IndividualAdminEditForm extends React.Component<
             name="password"
             label="Password"
             ref={this.passwordRef}
-            required={this.context.settingUp}
+            required={true}
             error={this.props.error}
           />
         )}


[DNM]Keyspace on tikv master metrics

Please don't review. Open this PR to make arm64 image.

/run-build-arm64 comment=true

/run-build-arm64 comment=true

/run-build-arm64 comment=true


Fix upgrade-only-k8s for etcd_client_cert_serial

This fixes the following error when running the upgrade-only-k8s playbook:
TASK [kubernetes/master : Write kube-apiserver manifest]
fatal: FAILED! => {"changed": false, "msg": "AnsibleUndefinedVariable: 'etcd_client_cert_serial' is undefined"}

are there any workarounds for this issue?


Document does not mention if we can continue to use Outlook 2007 even if the support for it has ended in October 2017

[This document does not mention if we can continue to use Outlook 2007 even if the support for it has ended in October 2017. Please note that we do understand the fact that it will not get updated with product issues and features.]

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

ID: 60388d66-91f6-0eb6-e857-d6663d170437
Version Independent ID: 3af34a6f-4953-5e52-b79b-c1196dc7d8ab
Content: Office 2007 end of support roadmap
Content Source: DeployOffice/office-2007-end-support-roadmap.md
Product: office-perpetual-itpro
GitHub Login: @DHB-MSFT
Microsoft Alias: danbrown

You can use any of the Office 2007 apps (Word, Excel, Outlook, etc) even though Office 2007 is no longer supported. But we strongly recommend against that since Office 2007 is no longer receiving security updates. Also keep in mind that Outlook 2007 isn't supported by Exchange Server 2016 or Exchange Server 2019.


Remove forced printing

https://github.com/ritajie/requests_to_curl/blob/c3f89134db77a3cfcf78257e85aa8eb44b560f9d/requests_to_curl/requests_to_curl.py#L45
Please consider removing forced printing since there may be sensitive information (e.g. access tokens, passwords, etc.) in the request or response.
Also return_it=True should be the default behavior to avoid boilerplate.

Agreed!
Thanks for the suggestion, around this week I'll find time to finish this update🙌

Just FYI, I wanted to open up a PR but it was not possible to do so. You might want to look into that if you would like to get contributions.
Have a great day.

@cemkavuklu The update is complete: Version 1.1.0 🚀


[BugFix] Support audit for insert into statement executing from follower

Fixes #issue
What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[x] Yes, this PR will result in a change in behavior.
[ ] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.1
[x] 3.0
[x] 2.5
[ ] 2.4

[FE Incremental Coverage Report]
:disappointed: fail : 2 / 32 (06.25%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/qe/DefaultCoordinator.java
0
27
00.00%
[887, 888, 889, 890, 891, 892, 893, 894, 897, 898, 899, 900, 901, 902, 903, 904, 905, 907, 908, 909, 910, 912, 913, 914, 915, 916, 917]

:large_blue_circle:
com/starrocks/qe/StmtExecutor.java
1
4
25.00%
[483, 484, 1376]

:large_blue_circle:
com/starrocks/sql/ast/InsertStmt.java
1
1
100.00%
[]

[BE Incremental Coverage Report]
:disappointed: fail : 0 / 35 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
src/exec/pipeline/result_sink_operator.h
0
3
00.00%
[110, 112, 121]

:large_blue_circle:
src/exec/pipeline/sink/export_sink_operator.cpp
0
3
00.00%
[195, 196, 225]

:large_blue_circle:
src/exec/pipeline/sink/memory_scratch_sink_operator.h
0
5
00.00%
[44, 45, 89, 91, 99]

:large_blue_circle:
src/exec/pipeline/sink/iceberg_table_sink_operator.cpp
0
2
00.00%
[64, 65]

:large_blue_circle:
src/exec/pipeline/olap_table_sink_operator.h
0
1
00.00%
[106]

:large_blue_circle:
src/exec/pipeline/result_sink_operator.cpp
0
1
00.00%
[67]

:large_blue_circle:
src/exec/pipeline/sink/export_sink_operator.h
0
6
00.00%
[39, 40, 82, 83, 84, 92]

:large_blue_circle:
src/exec/pipeline/olap_table_sink_operator.cpp
0
5
00.00%
[141, 142, 200, 203, 206]

:large_blue_circle:
src/exec/pipeline/sink/iceberg_table_sink_operator.h
0
7
00.00%
[51, 108, 109, 110, 111, 112, 120]

:large_blue_circle:
src/exec/pipeline/sink/memory_scratch_sink_operator.cpp
0
2
00.00%
[49, 50]

[FE Incremental Coverage Report]
:disappointed: fail : 2 / 32 (06.25%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/qe/DefaultCoordinator.java
0
27
00.00%
[887, 888, 889, 890, 891, 892, 893, 894, 897, 898, 899, 900, 901, 902, 903, 904, 905, 907, 908, 909, 910, 912, 913, 914, 915, 916, 917]

:large_blue_circle:
com/starrocks/qe/StmtExecutor.java
1
4
25.00%
[483, 484, 1376]

:large_blue_circle:
com/starrocks/sql/ast/InsertStmt.java
1
1
100.00%
[]

[BE Incremental Coverage Report]
:disappointed: fail : 0 / 35 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
src/exec/pipeline/result_sink_operator.h
0
3
00.00%
[110, 112, 121]

:large_blue_circle:
src/exec/pipeline/sink/export_sink_operator.cpp
0
3
00.00%
[195, 196, 225]

:large_blue_circle:
src/exec/pipeline/sink/memory_scratch_sink_operator.h
0
5
00.00%
[44, 45, 89, 91, 99]

:large_blue_circle:
src/exec/pipeline/sink/iceberg_table_sink_operator.cpp
0
2
00.00%
[64, 65]

:large_blue_circle:
src/exec/pipeline/olap_table_sink_operator.h
0
1
00.00%
[106]

:large_blue_circle:
src/exec/pipeline/result_sink_operator.cpp
0
1
00.00%
[67]

:large_blue_circle:
src/exec/pipeline/sink/export_sink_operator.h
0
6
00.00%
[39, 40, 82, 83, 84, 92]

:large_blue_circle:
src/exec/pipeline/olap_table_sink_operator.cpp
0
5
00.00%
[141, 142, 200, 203, 206]

:large_blue_circle:
src/exec/pipeline/sink/iceberg_table_sink_operator.h
0
7
00.00%
[51, 108, 109, 110, 111, 112, 120]

:large_blue_circle:
src/exec/pipeline/sink/memory_scratch_sink_operator.cpp
0
2
00.00%
[49, 50]

[FE Incremental Coverage Report]
:disappointed: fail : 17 / 32 (53.12%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/qe/StmtExecutor.java
1
4
25.00%
[483, 484, 1376]

:large_blue_circle:
com/starrocks/qe/DefaultCoordinator.java
15
27
55.56%
[904, 905, 907, 908, 909, 910, 912, 913, 914, 915, 916, 917]

:large_blue_circle:
com/starrocks/sql/ast/InsertStmt.java
1
1
100.00%
[]

[BE Incremental Coverage Report]
:disappointed: fail : 0 / 35 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
src/exec/pipeline/result_sink_operator.h
0
3
00.00%
[110, 112, 121]

:large_blue_circle:
src/exec/pipeline/sink/export_sink_operator.cpp
0
3
00.00%
[195, 196, 225]

:large_blue_circle:
src/exec/pipeline/sink/memory_scratch_sink_operator.h
0
5
00.00%
[44, 45, 89, 91, 99]

:large_blue_circle:
src/exec/pipeline/sink/iceberg_table_sink_operator.cpp
0
2
00.00%
[64, 65]

:large_blue_circle:
src/exec/pipeline/olap_table_sink_operator.h
0
1
00.00%
[106]

:large_blue_circle:
src/exec/pipeline/result_sink_operator.cpp
0
1
00.00%
[67]

:large_blue_circle:
src/exec/pipeline/sink/export_sink_operator.h
0
6
00.00%
[39, 40, 82, 83, 84, 92]

:large_blue_circle:
src/exec/pipeline/olap_table_sink_operator.cpp
0
5
00.00%
[141, 142, 200, 203, 206]

:large_blue_circle:
src/exec/pipeline/sink/iceberg_table_sink_operator.h
0
7
00.00%
[51, 108, 109, 110, 111, 112, 120]

:large_blue_circle:
src/exec/pipeline/sink/memory_scratch_sink_operator.cpp
0
2
00.00%
[49, 50]

[FE Incremental Coverage Report]
:disappointed: fail : 17 / 32 (53.12%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/qe/StmtExecutor.java
1
4
25.00%
[483, 484, 1376]

:large_blue_circle:
com/starrocks/qe/DefaultCoordinator.java
15
27
55.56%
[904, 905, 907, 908, 909, 910, 912, 913, 914, 915, 916, 917]

:large_blue_circle:
com/starrocks/sql/ast/InsertStmt.java
1
1
100.00%
[]

[BE Incremental Coverage Report]
:disappointed: fail : 0 / 35 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
src/exec/pipeline/result_sink_operator.h
0
3
00.00%
[110, 112, 121]

:large_blue_circle:
src/exec/pipeline/sink/export_sink_operator.cpp
0
3
00.00%
[195, 196, 225]

:large_blue_circle:
src/exec/pipeline/sink/memory_scratch_sink_operator.h
0
5
00.00%
[44, 45, 89, 91, 99]

:large_blue_circle:
src/exec/pipeline/sink/iceberg_table_sink_operator.cpp
0
2
00.00%
[64, 65]

:large_blue_circle:
src/exec/pipeline/olap_table_sink_operator.h
0
1
00.00%
[106]

:large_blue_circle:
src/exec/pipeline/result_sink_operator.cpp
0
1
00.00%
[67]

:large_blue_circle:
src/exec/pipeline/sink/export_sink_operator.h
0
6
00.00%
[39, 40, 82, 83, 84, 92]

:large_blue_circle:
src/exec/pipeline/olap_table_sink_operator.cpp
0
5
00.00%
[141, 142, 200, 203, 206]

:large_blue_circle:
src/exec/pipeline/sink/iceberg_table_sink_operator.h
0
7
00.00%
[51, 108, 109, 110, 111, 112, 120]

:large_blue_circle:
src/exec/pipeline/sink/memory_scratch_sink_operator.cpp
0
2
00.00%
[49, 50]

[FE Incremental Coverage Report]
:disappointed: fail : 17 / 32 (53.12%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/qe/StmtExecutor.java
1
4
25.00%
[483, 484, 1376]

:large_blue_circle:
com/starrocks/qe/DefaultCoordinator.java
15
27
55.56%
[904, 905, 907, 908, 909, 910, 912, 913, 914, 915, 916, 917]

:large_blue_circle:
com/starrocks/sql/ast/InsertStmt.java
1
1
100.00%
[]

[BE Incremental Coverage Report]
:disappointed: fail : 0 / 35 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
src/exec/pipeline/result_sink_operator.h
0
3
00.00%
[110, 112, 121]

:large_blue_circle:
src/exec/pipeline/sink/export_sink_operator.cpp
0
3
00.00%
[195, 196, 225]

:large_blue_circle:
src/exec/pipeline/sink/memory_scratch_sink_operator.h
0
5
00.00%
[44, 45, 89, 91, 99]

:large_blue_circle:
src/exec/pipeline/sink/iceberg_table_sink_operator.cpp
0
2
00.00%
[64, 65]

:large_blue_circle:
src/exec/pipeline/olap_table_sink_operator.h
0
1
00.00%
[106]

:large_blue_circle:
src/exec/pipeline/result_sink_operator.cpp
0
1
00.00%
[67]

:large_blue_circle:
src/exec/pipeline/sink/export_sink_operator.h
0
6
00.00%
[39, 40, 82, 83, 84, 92]

:large_blue_circle:
src/exec/pipeline/olap_table_sink_operator.cpp
0
5
00.00%
[141, 142, 200, 203, 206]

:large_blue_circle:
src/exec/pipeline/sink/iceberg_table_sink_operator.h
0
7
00.00%
[51, 108, 109, 110, 111, 112, 120]

:large_blue_circle:
src/exec/pipeline/sink/memory_scratch_sink_operator.cpp
0
2
00.00%
[49, 50]

[FE Incremental Coverage Report]
:disappointed: fail : 17 / 32 (53.12%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/qe/StmtExecutor.java
1
4
25.00%
[483, 484, 1376]

:large_blue_circle:
com/starrocks/qe/DefaultCoordinator.java
15
27
55.56%
[904, 905, 907, 908, 909, 910, 912, 913, 914, 915, 916, 917]

:large_blue_circle:
com/starrocks/sql/ast/InsertStmt.java
1
1
100.00%
[]

[BE Incremental Coverage Report]
:disappointed: fail : 0 / 35 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
src/exec/pipeline/result_sink_operator.h
0
3
00.00%
[110, 112, 121]

:large_blue_circle:
src/exec/pipeline/sink/export_sink_operator.cpp
0
3
00.00%
[195, 196, 225]

:large_blue_circle:
src/exec/pipeline/sink/memory_scratch_sink_operator.h
0
5
00.00%
[44, 45, 89, 91, 99]

:large_blue_circle:
src/exec/pipeline/sink/iceberg_table_sink_operator.cpp
0
2
00.00%
[64, 65]

:large_blue_circle:
src/exec/pipeline/olap_table_sink_operator.h
0
1
00.00%
[106]

:large_blue_circle:
src/exec/pipeline/result_sink_operator.cpp
0
1
00.00%
[67]

:large_blue_circle:
src/exec/pipeline/sink/export_sink_operator.h
0
6
00.00%
[39, 40, 82, 83, 84, 92]

:large_blue_circle:
src/exec/pipeline/olap_table_sink_operator.cpp
0
5
00.00%
[141, 142, 200, 203, 206]

:large_blue_circle:
src/exec/pipeline/sink/iceberg_table_sink_operator.h
0
7
00.00%
[51, 108, 109, 110, 111, 112, 120]

:large_blue_circle:
src/exec/pipeline/sink/memory_scratch_sink_operator.cpp
0
2
00.00%
[49, 50]

[FE Incremental Coverage Report]
:disappointed: fail : 17 / 32 (53.12%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/qe/StmtExecutor.java
1
4
25.00%
[483, 484, 1376]

:large_blue_circle:
com/starrocks/qe/DefaultCoordinator.java
15
27
55.56%
[904, 905, 907, 908, 909, 910, 912, 913, 914, 915, 916, 917]

:large_blue_circle:
com/starrocks/sql/ast/InsertStmt.java
1
1
100.00%
[]

[BE Incremental Coverage Report]
:disappointed: fail : 0 / 35 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
src/exec/pipeline/result_sink_operator.h
0
3
00.00%
[110, 112, 121]

:large_blue_circle:
src/exec/pipeline/sink/export_sink_operator.cpp
0
3
00.00%
[195, 196, 225]

:large_blue_circle:
src/exec/pipeline/sink/memory_scratch_sink_operator.h
0
5
00.00%
[44, 45, 89, 91, 99]

:large_blue_circle:
src/exec/pipeline/sink/iceberg_table_sink_operator.cpp
0
2
00.00%
[64, 65]

:large_blue_circle:
src/exec/pipeline/olap_table_sink_operator.h
0
1
00.00%
[106]

:large_blue_circle:
src/exec/pipeline/result_sink_operator.cpp
0
1
00.00%
[67]

:large_blue_circle:
src/exec/pipeline/sink/export_sink_operator.h
0
6
00.00%
[39, 40, 82, 83, 84, 92]

:large_blue_circle:
src/exec/pipeline/olap_table_sink_operator.cpp
0
5
00.00%
[141, 142, 200, 203, 206]

:large_blue_circle:
src/exec/pipeline/sink/iceberg_table_sink_operator.h
0
7
00.00%
[51, 108, 109, 110, 111, 112, 120]

:large_blue_circle:
src/exec/pipeline/sink/memory_scratch_sink_operator.cpp
0
2
00.00%
[49, 50]

[BE Incremental Coverage Report]
:disappointed: fail : 0 / 38 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
src/exec/pipeline/result_sink_operator.h
0
3
00.00%
[110, 112, 121]

:large_blue_circle:
src/exec/pipeline/sink/export_sink_operator.cpp
0
3
00.00%
[195, 196, 225]

:large_blue_circle:
src/exec/pipeline/sink/memory_scratch_sink_operator.h
0
5
00.00%
[44, 45, 89, 91, 99]

:large_blue_circle:
src/exec/pipeline/sink/iceberg_table_sink_operator.cpp
0
2
00.00%
[64, 65]

:large_blue_circle:
src/exec/pipeline/olap_table_sink_operator.h
0
1
00.00%
[106]

:large_blue_circle:
src/exec/pipeline/pipeline_driver_executor.cpp
0
2
00.00%
[348, 349]

:large_blue_circle:
src/exec/pipeline/result_sink_operator.cpp
0
1
00.00%
[67]

:large_blue_circle:
src/exec/pipeline/sink/export_sink_operator.h
0
6
00.00%
[39, 40, 82, 83, 84, 92]

:large_blue_circle:
src/exec/pipeline/olap_table_sink_operator.cpp
0
5
00.00%
[141, 142, 200, 203, 206]

:large_blue_circle:
src/exec/pipeline/sink/iceberg_table_sink_operator.h
0
7
00.00%
[51, 108, 109, 110, 111, 112, 120]

:large_blue_circle:
src/runtime/query_statistics.cpp
0
1
00.00%
[58]

:large_blue_circle:
src/exec/pipeline/sink/memory_scratch_sink_operator.cpp
0
2
00.00%
[49, 50]

[FE Incremental Coverage Report]
:disappointed: fail : 26 / 67 (38.81%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/qe/DefaultCoordinator.java
0
31
00.00%
[888, 889, 890, 892, 893, 894, 895, 896, 897, 898, 899, 902, 903, 904, 905, 906, 907, 908, 909, 910, 911, 912, 913, 914, 915, 917, 918, 919, 920, 921, 922]

:large_blue_circle:
com/starrocks/qe/StmtExecutor.java
14
24
58.33%
[1388, 1389, 1390, 1507, 1508, 1509, 1510, 1511, 1512, 1513]

:large_blue_circle:
com/starrocks/qe/LeaderOpExecutor.java
8
8
100.00%
[]

:large_blue_circle:
com/starrocks/sql/ast/InsertStmt.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/qe/ConnectProcessor.java
3
3
100.00%
[]

[BE Incremental Coverage Report]
:disappointed: fail : 0 / 38 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
src/exec/pipeline/result_sink_operator.h
0
3
00.00%
[110, 112, 121]

:large_blue_circle:
src/exec/pipeline/sink/export_sink_operator.cpp
0
3
00.00%
[195, 196, 225]

:large_blue_circle:
src/exec/pipeline/sink/memory_scratch_sink_operator.h
0
5
00.00%
[44, 45, 89, 91, 99]

:large_blue_circle:
src/exec/pipeline/sink/iceberg_table_sink_operator.cpp
0
2
00.00%
[64, 65]

:large_blue_circle:
src/exec/pipeline/olap_table_sink_operator.h
0
1
00.00%
[106]

:large_blue_circle:
src/exec/pipeline/pipeline_driver_executor.cpp
0
2
00.00%
[348, 349]

:large_blue_circle:
src/exec/pipeline/result_sink_operator.cpp
0
1
00.00%
[67]

:large_blue_circle:
src/exec/pipeline/sink/export_sink_operator.h
0
6
00.00%
[39, 40, 82, 83, 84, 92]

:large_blue_circle:
src/exec/pipeline/olap_table_sink_operator.cpp
0
5
00.00%
[141, 142, 200, 203, 206]

:large_blue_circle:
src/exec/pipeline/sink/iceberg_table_sink_operator.h
0
7
00.00%
[51, 108, 109, 110, 111, 112, 120]

:large_blue_circle:
src/runtime/query_statistics.cpp
0
1
00.00%
[58]

:large_blue_circle:
src/exec/pipeline/sink/memory_scratch_sink_operator.cpp
0
2
00.00%
[49, 50]

[FE Incremental Coverage Report]
:disappointed: fail : 62 / 117 (52.99%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/common/util/AuditStatisticsUtil.java
46
95
48.42%
[27, 30, 40, 41, 42, 43, 44, 45, 46, 47, 54, 58, 64, 70, 76, 82, 88, 93, 94, 96, 97, 98, 99, 100, 102, 103, 104, 105, 106, 108, 110, 111, 112, 114, 117, 118, 120, 126, 136, 148, 149, 150, 151, 153, 154, 156, 157, 159, 160]

:large_blue_circle:
com/starrocks/qe/StmtExecutor.java
5
10
50.00%
[1409, 1410, 1411, 1544, 1545]

:large_blue_circle:
com/starrocks/qe/LeaderOpExecutor.java
3
4
75.00%
[122]

:large_blue_circle:
com/starrocks/sql/ast/InsertStmt.java
1
1
100.00%
[]

:large_blue_circle:
com/starrocks/qe/ConnectProcessor.java
3
3
100.00%
[]

:large_blue_circle:
com/starrocks/qe/DefaultCoordinator.java
4
4
100.00%
[]

[BE Incremental Coverage Report]
:disappointed: fail : 0 / 38 (00.00%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
src/exec/pipeline/result_sink_operator.h
0
3
00.00%
[114, 117, 125]

:large_blue_circle:
src/exec/pipeline/sink/export_sink_operator.cpp
0
3
00.00%
[195, 196, 225]

:large_blue_circle:
src/exec/pipeline/sink/memory_scratch_sink_operator.h
0
5
00.00%
[44, 45, 89, 91, 99]

:large_blue_circle:
src/exec/pipeline/sink/iceberg_table_sink_operator.cpp
0
2
00.00%
[64, 65]

:large_blue_circle:
src/exec/pipeline/olap_table_sink_operator.h
0
1
00.00%
[106]

:large_blue_circle:
src/exec/pipeline/pipeline_driver_executor.cpp
0
2
00.00%
[348, 349]

:large_blue_circle:
src/exec/pipeline/result_sink_operator.cpp
0
1
00.00%
[68]

:large_blue_circle:
src/exec/pipeline/sink/export_sink_operator.h
0
6
00.00%
[39, 40, 82, 83, 84, 92]

:large_blue_circle:
src/exec/pipeline/olap_table_sink_operator.cpp
0
5
00.00%
[141, 142, 200, 203, 206]

:large_blue_circle:
src/exec/pipeline/sink/iceberg_table_sink_operator.h
0
7
00.00%
[51, 108, 109, 110, 111, 112, 120]

:large_blue_circle:
src/runtime/query_statistics.cpp
0
1
00.00%
[58]

:large_blue_circle:
src/exec/pipeline/sink/memory_scratch_sink_operator.cpp
0
2
00.00%
[49, 50]

@Mergifyio backport branch-3.1

@Mergifyio backport branch-3.0

@Mergifyio backport branch-2.5

@Mergifyio backport branch-3.2


Fix HTML/Markdown typo in ModularizationLearningJourney.md

Good spot, thanks. Force of habit when writing markdown :)


Kafka source oauth new

Description

OAuth functionality implemented for the Kafka Source Consumer

Issues Resolved

Github issue https://github.com/opensearch-project/data-prepper/issues/254

Check List

[ ] OAuth functionality includes testing.
[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Please add tests for all these files.

@ajeeshakd ,
I think we need the auth config something like this.  And OAUTH config goes under oauth: and these changes can go there.
authentication:
  protocol: # plaintext/ssl (more may be added later)
     plaintext:
     ssl:
  mechanism: # plain/iam/oauth (more like - scram, gssapi may be added later)
     plain:
     iam:
     oauth:

Let me know if this makes sense.

Closing this PR because the review comments change has been done as part of the PR https://github.com/opensearch-project/data-prepper/pull/2935


v2024.12.02 is missing doclicense-CC-by-88x31.pdf

v2024.12.02 is missing the file doclicense-CC-by-88x31.pdf at least in nime-paper-latex-template.zip

That file should be coming from your LaTeX distribution unless I am badly mistaken. It does seem to work on our local systems, on Overleaf and the GitHub Action builds it correctly (which I think uses TeXlive 2024). What's your build environment like?

I am using texlive 2024 locally, I searched for the file in my computer and it's not there. I installed the texlive with the minimal distribution (TinyTex?) so I might not ship with that. Everything compiles if I provide the doclicense-CC-by-88x31.pdf file myself. For general use I think it would be good to include the file!

adding \usepackage{ccicons} fixed the issue


Add new section in documentation for explanation, examples, and permissions set of optional multiuser and RBAC functionality

Context
This PR add documentation for the multiuser and role based access control functionality in the core node: https://github.com/smartcontractkit/chainlink/pull/6843
Description
The core node will now optionally support multiuser sessions with role backed authorization. Update documentation with usage examples and state what each role can do.
Changes
This PR adds a section on the miscellaneous page describing how to set it up and enumerates all actions that each role is permitted to do.

Rebased to main

Rebased to release/chainlink-next


Add Emacs process communication layer

@alanz: Can you review and merge this one?

Looks fine, great to see the first IDE integration


Deploy process

Hi,
Firstly, I want to thank you for this plugin, It is excellent developed, and It works great for me.
My problem is when I tried to deploy a process.bpmn from camunda modeler
It'is always with the error "Should point to a running camunda platform REST API "
I tried with http://localhost:8087/engine-rest as in application.yml
spring:
jersey:
application-path: http://localhost:8087/engine-rest
camunda.bpm:
history-level: audit
authorization:
enabled: true
filter:
create: All tasks
webapp.application-path:
server:
port: 8087
servlet.context-path: /camunda
Any idea?
thanks,

Hi @Ghada-Ch,
looks like the Camunda Modeler cannot connect / or authenticate against the REST API of Camunda.
If you have secured the REST API the Camunda Modeler will need a valid token from Keycloak in order to deploy  a process...
Now before we speculate what could be wrong, can you check if the REST API is callable via browser with REST plugin or via Postman etc.? The recommended endpoint for such a simple check is https://docs.camunda.org/manual/7.16/reference/rest/engine/

Hi,
you are right, The REST API a token here :
Camunda Rest API
rest.security:
enabled: true
provider: keycloak
required-audience: camunda-rest-api
But I just enabled it and everything works great.
Camunda Rest API
rest.security:
enabled: false
provider: keycloak
required-audience: camunda-rest-api
Thanks


disabling auth in v3 API

This PR a little bit cleaning and test enhance for disabling auth in v3 API.

updated the test. It depends on https://github.com/coreos/etcd/pull/5725 so I'll rebase this on the master after the merge of the PR.

rebase?

@xiang90 rebased, could you review?

lgtm

@xiang90 updated for your comments, PTAL

lgtm


Change need rdf structure to "is/seeks"

we are changing the rdf structure of needs to have 2 parts "is" and "seeks"
"is" describes the properties that relate to the offering of the need
"seeks" describes the properties that describe potential other needs which should be matched with this one
this structure replaces the need types (e.g. demand, supply, dotogehter, critique) that we were using and became too inflexible for our purposes now
further each need can have a MatchingBehavior entry with the following values: Mutual, Steahly, Lazy, DoNotMatch.
Mutual: is the default => hints get send to both needs if one seeks caluse matches the other needs is clause
DoNotMatch: do not sent hints neither to current need nor to other need
Steahly: send hints only to current need, not to other need
Lazy: send hints only to other need, not to current need

It is not clear for me how these changes would be reflected in the GUI.

Currently in the GUI nothing changes. For now these are just structural changes. The GUI can be adapted later

The pull-request #1025 adapts the create-need GUI to the is-seeks-structure and thus enables creating needs with the GUI again.
As of a78a6ec on need-structure-impl_GUI, the needs arrive in the state. "All that remains" is to look for all usages of won:hasContent and won:hasBasicNeedType in selectors, reducers and components and adapt them to the new structure.


[terraform-ibm-scc-da] Create WP agent flavour

@ctolon22

https://github.com/terraform-ibm-modules/terraform-ibm-scc-da/pull/91


Implement Health Checks

https://docs.microsoft.com/en-us/aspnet/core/host-and-deploy/health-checks?view=aspnetcore-3.1
https://docs.microsoft.com/en-us/dotnet/architecture/microservices/implement-resilient-applications/monitor-app-health

https://docs.microsoft.com/en-us/aspnet/core/host-and-deploy/health-checks?view=aspnetcore-3.1
https://docs.microsoft.com/en-us/dotnet/architecture/microservices/implement-resilient-applications/monitor-app-health


Upgrade from 1.1.7 to 1.1.8 causing crash-loop on shared storage.

PR to add port 80 was actually a replace port 80. Breaking change.

Hi @KenBurk,
Could you please let us know which env you are using? NGINX or Kong?
Thank you!

Hi @KenBurk ,
have you downloaded the latest helm chart?

We did not, our charts have to be customized to set the proper namespace and storage configuration, so breaking changes are kind of a big deal.

Hi @KenBurk ,
My sincere apologies, you can upgrade safely by adding
--set appsec.learning.image.tag=1.1.7 --set appsec.storage.image.tag=1.1.7
to the upgrade command.
I will update once a BC image is available.

backward compatible image is published


pypy37: reinit

Motivation for this change
pypy38 breaks bootstrapped-pip (and more or less all the pypy3Packages)
No motivation for switching to 3.8 was given during the update
Will self-merge if CI passes (this is a pure addition of pypy37, not switching the default)
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

PyPy3.8 fails test earlier, the tests are written for CPython, withPackages being broken for PyPy is known issue.


chore: update typescript

Summary
https://devblogs.microsoft.com/typescript/announcing-typescript-4-7
ts-eslint will complain the version is too new, but that doesn't really matter: https://github.com/typescript-eslint/typescript-eslint/issues/4800

Test plan
Green CI

@mrazauskas any ideas? 😀

Sure. I was looking at it already. Seems easy to fix. I will send a PR in few days. Unfortunately got sick today 😐


Colemak layout support

Colemak is a highly efficient keyboard layout.
Would be great if this includes a 5 row keyboard with that layout.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 23 Mar 2013 at 4:34

I'd also love this, please let me know when this will be on f-droid!

I'd love Colemak support as well!
Where in my filesystem should I put the patch file?

Make a PR like #491 or #500, & then 🙏!

PR created:#618

Is there any way for end users to gain access to this layout?
It's the only impediment for me to use this awesome keyboard again.


Have you tought of making a Kickstarter campaign for making free assets for FreeHL & CS?

Hey, I think this project is awesome, I think more people should know about it and I had an idea. You know OpenARENA? A clone of Quake, based on Quake's engine, same physics, same engine, same gameplay, new assets, it'd be awesome if FreeCS & FreeHL had financing to make free assets. Tell me what you think about it, also, if you don't like the idea, could you guide me in the right direction, so I can try to?

Yeah, ever since the project's inception in 2016 someone came around asking about it. I very dearly welcome it and even suggested it in the issue here: https://github.com/eukara/freecs/issues/23
The problem is that while I am very good at cloning game logic, I am not qualified to make replacement content by myself (nor do I have the time to do so).
This is where anyone else can come in and contribute, although it should be a separate project by itself so if people want to use mine to play it with the original data, they can.
You can make models using any of the formats supported by FTEQW, and levels with any of the available Quake BSP ones.
I'd highly suggest not using the Half-Life formats though, as they're restricted by the LICENSE of the original Half-Life SDK (you'll undoubtedly be using Valve's tools to create those, so that can't happen).

Okay, I'm not a map modder, but I can try, if I come up with something useful, I'll make you know

Yeah this game would explode with popularity if it has free assets packed in like openarena

Yup, it has been done before, with FreeDOOM, LibreQuake, OpenARENA, Xonotic, Unvanquished, Red Eclipse, etc.


draft: initial terminal logging

Overview
Problem
Enable logging for easy debugging
Solution
calling the recently added DBTTerminal output stream logging serv
Screenshot/Demo
How to test

TBD

Checklist

[ ] I have run this code and it appears to resolve the stated issue
[ ] README.md updated and added information about my change

@svipulrao98 : Can you try adding a decorator instead that logs the function name + arguments and return value. See https://www.meziantou.net/aspect-oriented-programming-in-typescript.htm
I think that could be the base and if we need more logging we add it it inline in the method.
@saravmajestic , @AdiGajbhiye : WDYT?

@mdesmet can you check the latest commit just an idea for decorator

What about the decorator?

had a discussion with @AdiGajbhiye, we decided to just add the logs for now & then think about it later

Decorators will be added later


Writing .bag stream to video gives inconsistent frame counts

Required Info

Camera Model
D435

Firmware Version
latest (no info tab in viewer)

Operating System & Version
Ubuntu 18

Kernel Version (Linux Only)
5.0.0-27-generic

Platform
PC

SDK Version
2

Language
python

Segment

Issue Description
Using the python wrapper, when I write the bag file to a video file (e.g., mp4), the resulting video varies in the total frame count by a few frames. For example, the video from the same bag file will have 250 frames on the first run, then 252, then 255, 255, 253, 251, etc. I looked to see if it was simply dropping frames at the end, but the last frame was the same every time, so it is dropping somewhere else. Is this typical behavior? I am using rs.config.enable_device_from_file(config, bag_fname, repeat_playback=False) to finish the video and a try and except block to handle frames = pipeline.wait_for_frames(timeout_ms=100).
As some context, I am concerned about this because I first get the 2D stream, then I do 2D pose estimation on that mp4 file, which returns 2D coordinates of body joints, and then I use the pyrealsense2 wrapper to align the color and depth frames to extract the 3D coordinate of those 2D locations. If the inconsistency is inherent, would it instead be recommended to align the depth and color streams and write them to video files before I do pose estimation, and then align those videos using opencv after?
Related issues: #4344
EDIT: I would use the conversion tools, but when I try to compile rs-convert.cpp on Ubuntu 18 after cloning this repository I get the following error:
$ g++ rs-convert.cpp -o rs-convert
rs-convert.cpp:8:10: fatal error: tclap/CmdLine.h: No such file or directory
 #include "tclap/CmdLine.h"
          ^~~~~~~~~~~~~~~~~
compilation terminated.

Are you setting the playback not to play in real-time?
    profile = pipe.start(config)
    playback=profile.get_device().as_playback()
    playback.set_real_time(False)

I have noticed before that I was loosing much more than just a few frames just because I was not running not in real-time.

I'm streaming from a file though, which shouldn't have that issue (although that is a good point). Here is how I'm starting the stream:
config = rs.config()
rs.config.enable_device_from_file(config, bag_fname, repeat_playback=False)
pipeline = rs.pipeline()
config.enable_stream(rs.stream.color)
pipeline.start(config)

Give it a try to include the set_real_time(False). Here is my setup that worked:
    pipe = rs.pipeline()
    config = rs.config()

    rs.config.enable_device_from_file(config, bagfile)
    config.enable_stream(rs.stream.depth, 640, 480, rs.format.z16, 30)
    config.enable_stream(rs.stream.color, 640, 480, rs.format.rgb8, 30)
    profile = pipe.start(config)
    playback=profile.get_device().as_playback()
    playback.set_real_time(False)

@bobdavies2000 That did the trick! Thank you!


Redesign pages generated from makeModule CMD

Description
Currently the generated pages from the makeModule command are looking quite basic. The design will be updated to look better for both React and Vue
Tasks (generated page updates)

[ ] Update Details page components (React)
[x] Update Details page components (Vue)
[x] Update Index page components (React)
[x] Update Index page components (Vue)

Vue Index page redesign

Vue Details page redesign

React Index page redesign

React Details page redesign


Added renaming of gpio registers

GPIO registers are renamed to remove common field prefix.
GPIO registers are bundled into arrays (saves a lot of code in esp32, and allows for significant simplification in esp32_hal).
This is dependent on update of svdtools: https://github.com/stm32-rs/svdtools/pull/28

PR https://github.com/stm32-rs/svdtools/pull/28 is merged (not yet published as new version however). Can this PR be merged?

Easier once its released, I've created https://github.com/stm32-rs/svdtools/pull/29 which should do it.


after upgrade flutter latest version still failed builing apk

Command
flutter build apk

Steps to Reproduce

...
...
...

Logs
ProcessException: Process exited abnormally:
FAILURE: Build failed with an exception.

What went wrong:
Project 'app' not found in root project 'android'.

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 1s Command: D:\development\qc_opname1\android\gradlew.bat, OS error code: 1
#0      RunResult.throwException (package:flutter_tools/src/base/process.dart:124:5)
#1      _DefaultProcessUtils.run (package:flutter_tools/src/base/process.dart:276:19)
<asynchronous suspension>
#2      flavorUndefinedHandler.<anonymous closure> (package:flutter_tools/src/android/gradle_errors.dart:294:38)
<asynchronous suspension>
#3      AndroidGradleBuilder.buildGradleApp (package:flutter_tools/src/android/gradle.dart:469:42)
<asynchronous suspension>
#4      AndroidGradleBuilder.buildApk (package:flutter_tools/src/android/gradle.dart:231:5)
<asynchronous suspension>
#5      BuildApkCommand.runCommand (package:flutter_tools/src/commands/build_apk.dart:105:5)
<asynchronous suspension>
#6      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1043:27)
<asynchronous suspension>
#7      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#8      CommandRunner.runCommand (package:args/command_runner.dart:196:13)
<asynchronous suspension>
#9      FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:284:9)
<asynchronous suspension>
#10     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#11     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:232:5)
<asynchronous suspension>
#12     run.<anonymous closure>.<anonymous closure> (package:flutter_tools/runner.dart:62:9)
<asynchronous suspension>

[✓] Flutter (Channel stable, 2.2.1, on Microsoft Windows [Version 10.0.10586], locale en-ID)
    • Flutter version 2.2.1 at D:\development\flutter
    • Framework revision 02c026b03c (3 days ago), 2021-05-27 12:24:44 -0700
    • Engine revision 0fdb562ac8
    • Dart version 2.13.1

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at D:\development\android-sdk
    • Platform android-30, build-tools 30.0.3
    • ANDROID_SDK_ROOT = D:\development\android-sdk
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at C:\Users\user\AppData\Local\Google\Chrome\Application\chrome.exe

[✓] Android Studio (version 4.0)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 51.0.1
    • Dart plugin version 193.7547
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[✓] VS Code (version 1.56.2)
    • VS Code at C:\Users\user\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.22.0

[✓] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 90.0.4430.212

• No issues found!

Flutter Application Metadata
Type: app
Version: 1.0.0+1
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: stable
Creation framework version: 2e540931f73593e35627592ca4f9a4ca3035ed31
Plugins
flutter_plugin_android_lifecycle-1.0.11
google_maps_flutter-0.5.33
image_picker-0.6.7+22
location-2.5.4
path_provider-1.6.28
path_provider_linux-0.0.1+2
path_provider_macos-0.0.4+8
path_provider_windows-0.0.5
sqflite-1.3.2+4

@denisciala
There's similar open issue describing your case, https://github.com/flutter/flutter/issues/63933
Please follow-up there for updates.
Closing this as duplicate.


@php

@php
(!empty($event->competition) && $event->competition == 2 ? ($registrationClosed = true) : ($registrationClosed = false));
switch(true){
....

changes to
@php(!empty($event->competition) && $event->competition == 2 ? ($registrationClosed = true) : ($registrationClosed = false));
switch(true){
....

and breaks all the code,

I think you could write it like this for work-around.
@php
$registrationClosed = !empty($event->competition) && $event->competition == 2;
switch (true) {
~~~
}
@endphp


Return additional fields?

Hi - is there any way to include additional certificate fields in the response JSON object?

Sure, what kind of information would you like?


Add estimating guide

Tips and tricks for accurate project estimating.

Good spot for some notes about calibration exercises, e.g., Douglas Hubbard and How to Measure Anything.
http://www.hubbardresearch.com/category/facilitating-calibrated-estimates/


[10.x] Allow for stricter lazy loading check

Overview
We just ran into an issue where a potential Eloquent lazy load violation was missed during local development, but once the release hit production it started logging violation errors.
The reason for this is a check in the Builder to only check for lazy loading violations if the model is part of a query that returned multiple models. See Builder.hyrdate here and PR #37503 for the original change to add the restriction.
I am proposing a new strict setting for Model::preventLazyLoading which removes this restriction.
If the application calls Model::preventLazyLoading('strict'), then we will no longer check that the model belongs to a set of more than one model. This will make lazy loading violations more common, so should be opt-in only, and the default will remain as is.
Why?
The query that caused this issue in our application would return 1 model 90% of the time, but in more complex scenarios would return multiple models. We didn't pick up on this lazy loading violation locally because we mostly test in the single model case. Once the app hit production and saw more complex scenarios with multiple models the violation triggered.
We would prefer to catch all lazy loading violations in local testing so that this doesn't occur.
To add to the issue, another developer was doing a demo using their local system and hit the violation during the demo which we had never seen before (and locally it threw an actual exception, so the demo showed the app breaking).
What's next
This is a quick draft of the change that I am proposing. I am happy to add a unit test to cover it if there is agreement for this change in principle.
As another commenter on the original PR mentioned - the best solution here would be to figure out not if more than one model was returned, but what the intention of the query was ie. was it a query for a single item, or a query that could return multiple models. If we could figure out a way to pass this down to hydrate then that would be ideal. It would be possible in some places, but in others it looks like it would be more difficult.

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


adds file-based tests, test runner, and README for review

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Change already applied in master; closing


fix typo

Thanks! I have changed the base branch to develop, which is where we are collecting changes until we make a new release.


Local IP host

Hi,
I'm trying to use your docker's image on a x86 board with dual ethernet.
The Idea is to configure one ethernet as WAN and the other one as a LAN.
Following your compose file, I configured either LAN and WAN on the two specific physical Ethernet interface.
After on OpenWRT, I configured WAN as a dhcp client, and LAN with static address with dhcp server enabled.
If I connect something on LAN port it receives correctly the ip address from dhcp server on OpenWRT.
My problem is on how to give an IP address of the LAN to the host machine, otherwise the host is cutted off.
Thank you for your work!

My problem is on how to give an IP address of the LAN to the host machine, otherwise the host is cutted off.

Currently, this is not possible. Both WAN and LAN Ethernet interfaces are added to OpenWrt exclusively. The only option you have is to add a third Ethernet interface e.g. with a USB-Ethernet adapter to have access to the host.

Sorry,
I want the host that can reach client connected on Openwrt LAN

You have two options:

As mentioned above add a third physical Ethernet interface and connect the LAN OpenWrt Ethernet interface and the host Ethernet interface via an external Ethernet switch together.
Use the veth options to create a virtual LAN Ethernet interface at host side. Then create a Ethernet bridge at the host side and add the virtual OpenWrt Ethernet interface and the host physical Ethernet interface to this bridge. Details about that you can find at https://developers.redhat.com/blog/2018/10/22/introduction-to-linux-interfaces-for-virtual-networking or https://wiki.archlinux.org/title/Network_bridge.

Thank you for your answer.
I' ve tried your second option, but when I use veth as LAN on Openwrt and I change the subnet of LAN in something like 192.168.xx.xx/24 the veth interface remains on the old address. It is like the address of veth is hardcoded.
Thank you

Yes you are right. In run_openwrt.sh the IP address is hard-coded. See: https://github.com/AlbrechtL/openwrt-docker/blob/2b2910eb2b95689a21d777104be9523f86705d3c/src/run_openwrt.sh#L72

Nevertheless the hard-coded address does option 2 solves your issue?
What would be a better approach in your option?

Hi,
thank you for your time.
I will try your solution.
If possible, I think it will be better that Openwrt will configure veth, since veth is the LAN of OpenWRT.
Thank you very much

Just to be clear LAN_IF: "veth,noautoip" is not implemented, currently.

In https://github.com/AlbrechtL/openwrt-docker/commit/aaab668d70e189620bdf2915e039bfd4eaf04761 I added the option nofixedip. See docker-compose.yml for details
https://github.com/AlbrechtL/openwrt-docker/blob/aaab668d70e189620bdf2915e039bfd4eaf04761/docker-compose.yml#L28
I hope that with this new feature, you can work.


find_by_* cannot find ajax result

i  am a new user of this tool, it's so cool
but
when the "Browser.html" doesnot have the ajax part
i can  use the is_element_present_by_*  to chech if  the 'ajax' element  is present
but
i want to click the 'ajax' part,such as button or href
hope u can understand my 'chinese' english
么么哒
<EMAIL_ADDRESS>

@halent you can use find_by if is_element_present returns True


[Feature] Integration with Azure ML Studio

This is a great package. I'd suggested (politely) that if there are not any plans to bring XGBoost within the Azure ML Studio GUI, this would be a good module to add as an option and provide a much needed tool within the interface

hi @PerformanceGenetics . we would happy to talk to Azure ML team to make this happen.

We are actively collaborating with Azure ML team on bring this onboard. And internally there is another GBDT tool available on AzureML, which will be even more powerful in the future with our help.


#[bugs] Cannot specify None for path argument

After installing from Package control, I saved my tex file, then the error appears: can't specify None for path argument.
Thanks!

Closing as dup of #2


Using patch/apply on update pods.

What type of PR is this?
/kind bug
/kind feature

What this PR does / why we need it:
Using patch/apply on update pods.
Which issue(s) this PR fixes:

Fixes #2056
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Use PATCH updates for pods. This fixes support for Pods when using the latest features in Kubernetes v1.29

/ok-to-test

/test all

/cc @alculquicondor

/lgtm
/approve
/retest

/release-note-edit
/kind bug
Use PATCH updates for pods. This fixes support for Pods when using the latest features in Kubernetes v1.29

@alculquicondor Could we cherry pick this to 0.6?

/cherry-pick release-0.6

@mbobrovskyi Could you submit a cherry pick PR manually? This cherrypicker would be helpful: https://github.com/kubernetes-sigs/kueue/blob/main/hack/cherry_pick_pull.sh

/release-note-edit
Pod Integration: Use PATCH updates for pods. This fixes support for Pods when using the latest features in Kubernetes v1.29


Kanban Setup

https://github.com/NabilHY/space-hub/projects/2?query=is%3Aopen+sort%3Aupdated-desc
Hello, Hope you have a good day, this project has two members including myself.

Hi @NabilHY @ppmarq1 👋 ,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉
Great work setting up the Kanban board. You can start executing the project. 👍
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


[Question] Logging middleware doesn't work with Prompts

SDK Platform
Node SDK
Code Sample
https://github.com/Microsoft/BotBuilder-Samples/tree/master/Node/capability-middlewareLogging
Problem
Hi,
I'm using a middleware this way in order to send messages from a user to my live chat.
    // Middleware to send messages to Crisp
    bot.use({
        botbuilder: function (session, next) {
            logIncomingMessage(session, next, crispClient, process.env.CRISP_WEBSITE_ID);
        }
    })

The logIncomingMessage function is the following.
    // Messages sent by user
    const logIncomingMessage = (session, next, crispClient, websiteId) => {
        const message = session.message.text;
        const sessionId = session.userData.crispSessionId;

        if (session.userData.isLiveChat && session.options.dialogId !== '/endLiveChat') {
            crispUtils.sendMessage(crispClient, websiteId, sessionId, message, 'user', session.userData);
        } else {
            next();
        }
    }

So this function send the messages from the user to Crisp and then if the user is not in a live chat session, his actions are handled by next();
The problem is that when I'm using a Prompt the following way, it isn't working and the bot is restarting...
    bot.dialog('/getUsername', [
        function (session) {
            builder.Prompts.text(session, "Hi! What's your name?");
        },
        function (session, results) {
            session.send("Hello %s.", results.response);
        }
    ]);

So I was wondering if someone knows a way to handle Prompts in my middleware. I've tried several solutions but none were working. I guess I need to add a condition in my middleware to handle it but I couldn't find which one.
I've also implemented the function to log the messages from the bot but it's not working either with it.
Thank you for your help!

Nevermind, my implementation of a custom Redis store was in fault.
I recommend you the following package that implements it well!
suttna/botbuilder-redis-storage


Dragging of points in scatter is not working

jupyterlab = 3.0.15
bqplot = 0.12.29
I was trying to build out a plot with some ipywidgets and went to add some interactivity of dragging points - but they would not move on the plot.
I then returned to the documentation and was executing this notebook to see what I was missing:
https://github.com/bqplot/bqplot/blob/master/examples/Marks/Object Model/Scatter.ipynb
I noticed in that notebook, the points also aren't moving.  I have reduced the notebook down to the section that has the example for the moving points:
import numpy as np
import pandas as pd
from bqplot import (
    Axis, ColorAxis, LinearScale, DateScale, DateColorScale, OrdinalScale,
    OrdinalColorScale, ColorScale, Scatter, Lines, Figure, Tooltip
)
from ipywidgets import Label

size = 100
x_data = range(size)
y_data = np.cumsum(np.random.randn(size) * 100.0)

## Enabling moving of points in scatter. Try to click and drag any of the points in the scatter and
## notice the line representing the mean of the data update
sc_x = LinearScale()
sc_y = LinearScale()

scat = Scatter(x=x_data[:10], y=y_data[:10], scales={'x': sc_x, 'y': sc_y}, colors=['orange'],
               enable_move=True)
lin = Lines(x=[], y=[], scales={'x': sc_x, 'y': sc_y}, line_style='dotted', colors=['orange'])

def update_line(change=None):
    with lin.hold_sync():
        lin.x = [np.min(scat.x), np.max(scat.x)]
        lin.y = [np.mean(scat.y), np.mean(scat.y)]

update_line()
# update line on change of x or y of scatter
scat.observe(update_line, names=['x'])
scat.observe(update_line, names=['y'])

ax_x = Axis(scale=sc_x)
ax_y = Axis(scale=sc_y, tick_format='0.2f', orientation='vertical')

fig = Figure(marks=[scat, lin], axes=[ax_x, ax_y])
fig

I am looking for someone to try to reproduce - if they move for them or not?
here is an animated give of what I am seeing:

In the chrome web console I am seeing lots of errors

Error: widget model not found
    at x.get_model (523.fa256ee012d38a89b65a.js:1)
    at u.renderModel (523.fa256ee012d38a89b65a.js:1)
102568.2ffd16368ca616eca759.js?v=2ffd16368ca616eca759:1 Uncaught TypeError: Cannot read property 'x' of null
at h.on_drag (568.2ffd16368ca616eca759.js?v=2ffd16368ca616eca759:1)
at SVGGElement. (568.2ffd16368ca616eca759.js?v=2ffd16368ca616eca759:1)
at ct.apply (693.bdce231db58bf853b2df.js?v=bdce231db58bf853b2df:1)
at yn (693.bdce231db58bf853b2df.js?v=bdce231db58bf853b2df:1)
at Object.a [as mouse] (693.bdce231db58bf853b2df.js?v=bdce231db58bf853b2df:1)
at p (693.bdce231db58bf853b2df.js?v=bdce231db58bf853b2df:1)
at 693.bdce231db58bf853b2df.js?v=bdce231db58bf853b2df:1

A few new environments later and it is working - not sure why it seems to be so sensitive on install or if it is something local.  I will reopen if I get more details on what causes it.


Is "3D feature(by-slice)"   the average of  2d-slice features derived from the whole 3d ROI?

Is "3D feature(by-slice)"   the average of  2d-slice features derived from the whole 3d ROI?
thx

What do you mean by 3D feature(by-slice)? Where does it mention this?
PyRadiomics has both 3D and 2D extraction, with the difference being that for 2D, no offsets that moves between slices are used. Therefore, there is no difference in firstorder or shape features and for texture features, all slices are combined, e.g. GLSZM only defines 2D zones, GLRLM just the in-plane runs and GLCM/NGTDM/GLDM only consider in-plane adjacent voxels to be neighbors.

"force2D [False]: Boolean, set to true to force a by slice texture calculation. Dimension that identifies the ‘slice’ can be defined in force2Ddimension"
Given 3D ROI  and  setting "force2d=True ",  how will  the feature values  combined? every slices have their onw texture feature value,  e.g. if A ROI has 32 slices and force2D is True, there will be 32 GLSZM values , right? ,how will  these values combined?  average ?

No, slices are combined. E.g. in case of GLSZM, 1 matrix will be built using voxels from all slices. The difference is that in 3D, adjacent voxels on other slices are considered neighbors, in force 2D only adjacent voxels on the same slice are neighbors. This causes the zones to be 3D and 2D, respectively.
In case of GLCM, GLRLM, there is one matrix created for each angle (offsets indicating where the neigbor is). Features are then calculated for each matrix and an average is returned. In case of 3D extraction, those angles describe offsets that move in-plane, out-of-plane or both. In case of 2D, only the in-plane angles are retained. I.e. less matrices are calculated, but those that are, are identical to those calculated in 3D

@JoostJM I think it would be good to add this explanation to the documentation, if it's not already there! I think this question came up before.

Understand know, thank u


PR with updated tutorial

Changes - adapted to cds watch, and other minor modifications
Note: Image changed to new-cds-commands.PNG. Please check if this image is fine for the tutorial

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

@LakshmiCR & @lalithmanokar : Thanks for the changes, please sign the CLA as mentioned above. I´ll do a review of course, but only Daniel can approve this PR.

@thecodester : Could you please approve this PR that we can review the content on QA with the dev team? THANKS

You are trying to update the production repo without updating the QA repo first. Please contact me offline if you need help.

@thecodester Will do the changes provided in this PR on QA first. @LakshmiCR & @lalithmanokar: once the changes are on QA I´ll let you know so we can final review the content.

You may want to use the Atom plugin that we provide which makes committing and pushing much easier.

Looks good to me content-wise


Apply loose versioning to JCasC package

Because of the new non-semantic versioning scheme used by Jenkins
plugins, renovate has a hard time identifying new versions. To get
around this, we must tell renovate to use loose versioning.

FWIW, the solution is via https://github.com/renovatebot/renovate/discussions/18449#discussioncomment-3931340.


The position is not correct After the blocklyDraggable is dragged to blocklyFlyout

Expected Behavior
The blocklyDraggable can be dragged to the corresponding position with the mouse.
Actual Behavior
When drag the blocklyDraggable  into blocklyFlyout area , The blocklyDraggable unable to reach the specified position. instead, it will back to the original position. And it also affects other blocklyDraggable position.

Steps to Reproduce
I want blocklyFlyout is hidden  after the blocklyDraggable  is dragged or clicked, So I do the following steps:

Modify flyout_vertical.js file  to make blocklyFlyout can autoClose  :
Blockly.VerticalFlyout.prototype.autoClose = true

Drag the blocklyDraggable into blocklyFlyout area, then release the mouse, The blocklyDraggable will back to the original position

Operating System and Browser
Window10 Chrome 67.0.3396.99

Hi, I found a hack solution, and it may be useful for temp.
Just change Blockly.Toolbox.prototype.getWidth  function, and it works.
/**
 * Get the width of the toolbox.
 * @return {number} The width of the toolbox.
 */
Blockly.Toolbox.prototype.getWidth = function() {
  if (this.flyout_ && !this.flyout_.isVisible) {
    return this.HtmlDiv.clientWidth;
  }
  return this.width;
};

@cwxyz007 The problem still exists!

@cwxyz007 It should be isVisible_ , And I try it,  The problem has been solved, Thanks a lot.
But I still have a small problem that when I first time drag the blocklyDraggable , it will be attached to the edge. Why does this happen?

After you do what you say，have problem：
Cannot read property 'id_' of null
const categoryId = this.workspace.toolbox_.getSelectedCategoryId();
have you？

@yi-jy Have you found a workaround for the blocks attaching to the edge the first time?

@yi-jy Have you found a workaround for the blocks attaching to the edge the first time?
我不明白你的意思
I don't understand what you are saying。
Blockly.Toolbox.prototype.getWidth = function() {
if (this.flyout_ && !this.flyout_.isVisible_) {
return this.HtmlDiv.clientWidth;
}
return this.width;
};
as this ，Solved the problem


Errno::ENOTDIR Exception in fsevents_to_vm/watch.rb:23

May be a dupey, and this may already be fixed, but I thought I'd post about this.  This exception appears to be causing my fsevents daemon to barf (on dinghy 4.4.1):
/Library/Ruby/Gems/2.0.0/gems/fsevents_to_vm-1.1.2/lib/fsevents_to_vm/watch.rb:23:in `stat': Not a directory - /Users/dstaudigel/Library/Caches/com.tinyspeck.slackmacgap.ShipIt/update.axaVf3l/Slack.app/Contents/Frameworks/ReactiveCocoa.framework/Headers (Errno::ENOTDIR)
	from /Library/Ruby/Gems/2.0.0/gems/fsevents_to_vm-1.1.2/lib/fsevents_to_vm/watch.rb:23:in `build_event'
	from /Library/Ruby/Gems/2.0.0/gems/fsevents_to_vm-1.1.2/lib/fsevents_to_vm/watch.rb:13:in `block (2 levels) in run'
	from /Library/Ruby/Gems/2.0.0/gems/fsevents_to_vm-1.1.2/lib/fsevents_to_vm/watch.rb:12:in `each'
	from /Library/Ruby/Gems/2.0.0/gems/fsevents_to_vm-1.1.2/lib/fsevents_to_vm/watch.rb:12:in `block in run'
	from /Library/Ruby/Gems/2.0.0/gems/rb-fsevent-0.9.6/lib/rb-fsevent/fsevent.rb:45:in `call'
	from /Library/Ruby/Gems/2.0.0/gems/rb-fsevent-0.9.6/lib/rb-fsevent/fsevent.rb:45:in `run'
	from /Library/Ruby/Gems/2.0.0/gems/fsevents_to_vm-1.1.2/lib/fsevents_to_vm/watch.rb:17:in `run'
	from /Library/Ruby/Gems/2.0.0/gems/fsevents_to_vm-1.1.2/lib/fsevents_to_vm/cli.rb:24:in `start'
	from /Library/Ruby/Gems/2.0.0/gems/thor-0.19.1/lib/thor/command.rb:27:in `run'
	from /Library/Ruby/Gems/2.0.0/gems/thor-0.19.1/lib/thor/invocation.rb:126:in `invoke_command'
	from /Library/Ruby/Gems/2.0.0/gems/thor-0.19.1/lib/thor.rb:359:in `dispatch'
	from /Library/Ruby/Gems/2.0.0/gems/thor-0.19.1/lib/thor/base.rb:440:in `start'
	from /Library/Ruby/Gems/2.0.0/gems/fsevents_to_vm-1.1.2/exe/fsevents_to_vm:4:in `<top (required)>'
	from /usr/local/bin/fsevents_to_vm:23:in `load'
	from /usr/local/bin/fsevents_to_vm:23:in `<main>'

Seeing as there already was a rescue call for ENOENT, I added one for ENOTDIR:
    def build_event(filepath)
      mtime = Event.format_time(File.stat(filepath).mtime)
      Event.new(filepath, mtime, Time.now)
    rescue Errno::ENOENT
      # TODO: handling delete events is tricky due to race conditions with rapid
      # delete/create.
      nil
    rescue Errno::ENOTDIR
      nil
    end

I'm not sure if this will actually fix my relatively flakey fsevents, but it might!

Oh good catch thanks. I've pushed out a new version of https://github.com/codekitchen/fsevents_to_vm
You can force dinghy to upgrade to the new version without waiting for a new dinghy release by running sudo /System/Library/Frameworks/Ruby.framework/Versions/Current/usr/bin/gem uninstall fsevents_to_vm and then dinghy restart.

Cool - that was fast.  Thanks!


Fix typo

Typo: Processthreadapi.h was wrong. Correct file name is Processthreadsapi.h.

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

#sign-off


[console] fix address name validation

Type of change

Bugfix

Description
Fix input validation issue for address name
Checklist

[ ] Update/write design documentation in ./documentation/design
[ ] Write tests and make sure they pass
[ ] Update documentation
[ ] Check RBAC rights for Kubernetes / OpenShift roles
[ ] Try your changes from Pod inside your Kubernetes and OpenShift cluster, not just locally
[ ] Reference relevant issue(s) and close them after merging
[ ] Update CHANGELOG.md

@enmasse-ci run tests testcase=FirefoxConsoleTest#testValidAddressNames

:heavy_check_mark: Test Summary :heavy_check_mark:
TEST_PROFILE: pr
TEST_CASE: FirefoxConsoleTest#testValidAddressNames
TOTAL: 26
PASS: 26
FAIL: 0
SKIP: 0
BUILD_NUMBER: 1053
BUILD_ENV: oc cluster up


Apparent false alarm in Bolt project

I have a Bolt-based project, installed through their composer-install method. Running dependency-guard on it gives me one violation:

[ERROR] Package "passwordlib/passwordlib" is installed, but never used.

However, that lib is actually used in Bolt, in Bolt\AccessControl\Login for example.
How to reproduce:
# create-project bolt/composer-install:^3.5 bolt-dependency-test --prefer-dist
Installing bolt/composer-install (3.5.0)
<snip - installation of Bolt>

# cd bolt-dependency-test
# composer require --dev mediact/dependency-guard
<snip - installation of Dependency Guard>

# composer dependency-guard
 [ERROR] Package "passwordlib/passwordlib" is installed, but never used.
 [ERROR] Number of dependency violations: 1

# composer why passwordlib/passwordlib
bolt/bolt              v3.5.5  requires  passwordlib/passwordlib (^1.0@beta)
bolt/composer-install  -       requires  passwordlib/passwordlib (^1.0@beta)

Not sure if this is an issue with Bolt or with Dependency Guard, but I thought I'd bring it up.

Hello @doenietzomoeilijk,
First of all, thanks for reporting this issue. We are appreciative of any issues you find in DependencyGuard or help solving problems you might unravel because of using it.
I've had a look at the code inside the package bolt/composer-install and found that their autoloader is set to load in the src/ folder. Inside that folder is the single file CustomisationExtension.php. If we open up that file, it is clear it uses no code being installed because of passwordlib/passwordlib. I've looked deeper and went inside the source of the passwordlib/passwordlib package and found it is not exposing an application through the bin configuration option. This means you can safely remove passwordlib/passwordlib as dependency from your project.
You mention it is being used within Bolt itself. However, Bolt also has an explicit dependency on that package, so it will still install the package, even if you remove the requirement in your project. In this instance, you have used Dependency Guard to successfully spot a piece of wrong configuration inside the package bolt/composer-install. I suggest you open an issue for bolt/composer-install and refer to the current issue so they may have some context for the problem at hand.
If you could verify this satisfies your issue, I will follow up by closing it.
Cheers for bringing this up :]

Yep, that about clears it up, I'll take it up with the Bolt team. Thanks! :)


(bug/feature request) Hybrid operator not compatible with aggregations

HI,
It looks like aggregations are not supported with hybrid search.
Search:
  search: Get {
    Test1(
      limit: 20
      offset: 0
      hybrid: {query: "helo", alpha: 0.750000}
    ) {
      wpsolr_id
      wpsolr_title
    }
  }
  search_count: Aggregate {
    Test1(
      hybrid: {query: "helo", alpha: 0.750000}
    ) {
      meta {
        count
      }
    }
  }
}

gives:
{
  "errors": [
    {
      "locations": [
        {
          "column": 7,
          "line": 14
        }
      ],
      "message": "Unknown argument \"hybrid\" on field \"Test1\" of type \"AggregateObjectsObj\".",
      "path": null
    }
  ]
}

I would say this is more of a feature request than a bug. But let me check with the team if this is trivial to add. If yes, then we could indeed call this a bug and release the "fix" with a patch soon.

I forgot to mention that aggregation is the #1 feature for e-commerce.

:)


README notes automatic release version pinning

The resource updates the manifest to pin releases to the versions of releases provided, but this was not clear in the README.

Fixed


Patch for Grove to support new version of PyQuil

This removes all of the util methods.

This will fix #106


error for test seg task

During run torchpack dist-run -np 1 python tools/test.py configs/nuscenes/seg/fusion-bev256d2-lss.yaml pretrained/bevfusion-seg.pth --eval map
I had the same problem as #461at first, but I added the following code after line 185 in nusences_converter.py
 file location = nusc.get(             "log", nusc.get("scene", sample["scene_token"])["log_token"]         )["location"]
And added location to the info
'location': location,

Rerun nusecenes_converter to regenerate the PKL file, the following error was encountered
AttributeError: 'dict' object has no attribute 'in_range_bev'

Have you solved this problem? I have the same problem...

Have you solved this problem? I have the same problem...

I have also got this problem during run torchpack dist-run -np 1 python tools/test.py configs/nuscenes/seg/fusion-bev256d2-lss.yaml pretrained/bevfusion-seg.pth --eval map.
I don't know how to evaluate the pretrained model for the segmentation variant of BEVFusion.

Thank you for your interest in our project. This repository is no longer actively maintained, so we will be closing this issue. Please refer to the amazing implementation at MMDetection3D. Thank you again!


fix return_token_log_probs on vLLM > 0.3.3 endpoints

Pull Request Summary
since vLLM > 0.3.3, logprobs are returned as as type Logprob instead as direct float's. it's an API-breaking change (see https://github.com/vllm-project/vllm/pull/3065#issuecomment-1984761326)
change vllm_server to return output in the same previous format
Test Plan and Usage Guide
create new llama-2-7b with vLLM==0.4.0.post1
Completion.create(
    model="llama-2-7b",
    prompt="Hello ",
    max_new_tokens=10,
    temperature=0.0,
    return_token_log_probs=True,
)

thanks for the fix! i think we need to do some backfills also?

will update endpoints to latest image post merge


[Task] Increment project version for release

Purpose

Increase project version in Xcode to 1.1.0-beta.1

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[x] Other... Please describe:
    - Project settings update

Other Information

I think there are other parts in the read me / documentation (like the code snippets) that will need the version bumped as well.

I think there are other parts in the read me / documentation (like the code snippets) that will need the version bumped as well.

I see there's a separate task for read me/documentation update. Do you think I should include those update in this PR or leave it for the read me/ documentation task? (John's taking that one)

I think there are other parts in the read me / documentation (like the code snippets) that will need the version bumped as well.

I see there's a separate task for read me/documentation update. Do you think I should include those update in this PR or leave it for the read me/ documentation task? (John's taking that one)

yea, I am handling it. Posted a discussion in teams channel, but got no response yet. But I guess I will create a PR later.


Persist scheduled container size in packing plan

Persist actual container size in packing plan as containerPlan.scheduledResource
Rename containerPlan.resource to containerPlan.requiredResource: this is the minimum resource requirement as suggested by packing plan

Fixes #1396

Also what about the resource usage in com.twitter.heron.packing.Container? Are those required or scheduled or should those resources remain generic?

com.twitter.heron.packing.Container is a helper class used by packing algorithms. This change does not impact or is impacted by that class.

Re com.twitter.heron.packing.Container yes, that class in completely independent. I was just wondering if the naming change from resource to required and scheduled resource was something that that class should also adopt. I'm really not sure if it should or not.

One last public -> private comment, then lgtm! Thanks!

👍 once CI completes


chore: adding keploy to the list of shadow testing tools

Pull Request Template
What are you trying to address
As discussed in this PR, it would be fantastic if Keploy could be added to the list above to assist developers in solving the shadow testing challenge.
Checklist
READY TO PR? Use the check-list below to ensure your branch is ready for PR.

[x] Changes follow the repo structure and land in the appropriate folder and section
[x] No confidential information
[x] No duplicated content
[x] Labeled appropriately
[x] This PR was reviewed by at least one subject matter expert
[x] No lint check errors related to your changes

Note: You may see link check errors on pages you have not touched. This is normal, and due to either broken links or sites that reject link checker bots. The reviewer will help you get to a green state on these.

Hi @shiranr can you please review this one? It's a small change. :)


As a GP Clinic Management Staff with poor eyesight I can customise the font size and style of the application

... so that I can easily read the information on the screen.

Closed as decided not to implement.


kodi: fix youtube addon

Fixes https://github.com/NixOS/nixpkgs/issues/226613.
Add infotagger plugin and use it.
Two notes on the infotagger plugin:

I used the GitHub source, and not https://kodi.tv/addons/nexus/script.module.infotagger/ which links to http://ftp.halifax.rwth-aachen.de/xbmc/addons/nexus/script.module.infotagger/script.module.infotagger-0.0.7.zip. Not sure if this is a problem (or a feature).

The infotagger plugin has a weird Python path resources/modules. I expect this path may change in the upstream module in the future (it is lib for all other modules as far as I can see).

I can watch YouTube videos here.
However, this PR does not address the relatively new dependency
<import addon="inputstream.adaptive" version="20.3.1"/>

I do not get any errors though. I am not sure how to proceed (open another issue?; ignore?).
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

Two more cents:

I think the automatic updater should scan for dependency changes, and fail if there are new or dependencies or if dependencies have been removed.
This change was not caused by a dependency update of the YouTube addon, but by a change in base. (I think). The infotagger module seems to ensure some form of backwards compatibility.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2154

Please follow the contributing guide when naming your commits.

Hi @SuperSandro2000. I was not completely sure if I should then split this into two commits (one adding the infotagger addon, the other one actually using it). As you see, I decided to go for one commit with a changed message, as you requested. Let me know if this is Ok.

one adding the infotagger addon, the other one actually using it

It is a new package so it should get an init commit.

I see. I didn't know this was so strict. Here you go!