Example output?

This project looks interesting, and maybe I'm just ignorant to the use case, but could you provide an example output of what story(s) can be generated based on the examples in the readme?

OK I added an example of output from running the example file included in the repository as well as a simple rendering of the output from the example rule given at the end of the readme. I'll probably add more later but this should at least be a little bit better :-)

I see now, thanks!


When using this example combined with websocket server the websocket server is slow

Thanks for positing this example! I tried it with a 64x32 display and it works great!
I combined it with this websocket server, in order to control the display from a web browser:
https://github.com/ThomasBarth/WebSockets-on-the-ESP32
This websocket example running alone is very responsive, it echos back text sent from the web browser almost instantly.
However, once I add this I2S display driver, the reply to the browser takes several seconds.
I am wondering if you know any reason why the parallel I2S driver on the ESP-32 would be slowing down the network packets from being sent? Thank you!

To add more information on my issue. The websocket server works fine when I comment out this line in i2s_paralell.c:
//Start dma on front buffer
dev->lc_conf.val=I2S_OUT_DATA_BURST_EN | I2S_OUTDSCR_BURST_EN | I2S_OUT_DATA_BURST_EN;
dev->out_link.addr=((uint32_t)(&st->dmadesc_a[0]));
dev->out_link.start=1;
**//dev->conf.tx_start=1;**

When this line is commented out I can ping the ESP32 from my computer and it responds in a couple ms
When this line is un-commented, the ESP32 does not reply to the pings 3/4 of the time
So I think something about starting the I2S transmission is causing the ESP to delay sending packets


Add Variables to Structure

I'd like to create  UserDefinedStuct (Structure) and add some variables to it. Is there a way to do? And how?

Hi, get latest code (albeit i think i will make binary releases in few hours) and check this script:
https://github.com/20tab/UnrealEnginePython/blob/master/examples/create_user_defined_struct.py

I get following errors and warnings when compiling (Unreal version 4.18):
Error	C4840	non-portable use of class 'FString' as an argument to a variadic function
..\Plugins\UnrealEnginePython\Source\UnrealEnginePython\Private\UObject\UEPyDataTable.cpp	32
Warning		FStringAssetReference has been renamed to FSoftObjectPath, change to #include "UObject/SoftObjectPath.h" and rename references
d:\Program Files\Epic Games\UE_4.18\Engine\Source\Runtime\CoreUObject\Public\Misc\StringAssetReference.h	5
Warning	C4996	'UPrimitiveComponent::AddAngularImpulse': Use AddAngularImpulseInRadians instead. Please update your code to the new API before upgrading to the next release, otherwise your project will no longer compile.		..\Plugins\UnrealEnginePython\Source\UnrealEnginePython\Private\UObject\UEPyPhysics.cpp	140
Warning	C4996	'UPrimitiveComponent::AddTorque': Use AddTorqueInRadians instead. Please update your code to the new API before upgrading to the next release, otherwise your project will no longer compile.
..\Plugins\UnrealEnginePython\Source\UnrealEnginePython\Private\UObject\UEPyPhysics.cpp	240
Warning	C4996	'UPrimitiveComponent::SetPhysicsAngularVelocity': Use SetPhysicsAngularVelocityInDegrees instead. Please update your code to the new API before upgrading to the next release, otherwise your project will no longer compile.	..\Plugins\UnrealEnginePython\Source\UnrealEnginePython\Private\UObject\UEPyPhysics.cpp	372

Thanks for new release!
I got the following errors though when I compile the source code. I am using UE 4.18, python 2.7 win64
Thanks in advance:)
Error	C4840	non-portable use of class 'FString' as an argument to a variadic function	MyProject	E:\UnrealProjects\MyProject\Plugins\UnrealEnginePython\Source\UnrealEnginePython\Private\UObject\UEPyDataTable.cpp	32
Error		Failed to produce item: E:\UnrealProjects\MyProject\Plugins\UnrealEnginePython\Intermediate\Build\Win64\UE4Editor\Development\UE4Editor-UnrealEnginePython.lib	MyProject	E:\UnrealProjects\MyProject\Intermediate\ProjectFiles\ERROR	1
Error	MSB3075	The command ""D:\Program Files\Epic Games\UE_4.18\Engine\Build\BatchFiles\Build.bat" MyProjectEditor Win64 Development "E:\UnrealProjects\MyProject\MyProject.uproject" -waitmutex" exited with code 5. Please verify that you have sufficient rights to run this command.	MyProject	C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\Common7\IDE\VC\VCTargets\Microsoft.MakeFile.Targets	44

Binary releases are now available: https://github.com/20tab/UnrealEnginePython/releases/tag/20171229
The compilation error should be fixed too


s/ipython_widgets/ipywidgets/

Conflict somewhere, maybe related to #6

Indeed. Rebased.

@jdfreder any objection to merging here?

Nope, thanks!


Admin::url is not using standard url helpers ?

I had issues because of this in the Admin facade
    public static function url($url)
    {
        $prefix = (string) config('admin.prefix');

        if (empty($prefix) || $prefix == '/') {
            return '/'.trim($url, '/');
        }

        return "/$prefix/".trim($url, '/');
    }

Why isn't the url laravel helper used? like this
return url("/$prefix/".trim($url, '/'));

The problem is that laravel creates absolute urls and that's what I need. An example was the login form in the form tag.

Because of this issue, deployment cases where laravel is installed inside a subfolder or a virtual document root, will cause the admin to not work at all.
The snippet suggested is my guess, but maybe there is a better place for fixing...

Has anyone find a way to fix this?

I think it's possible by overriding some classes, but involves digging deep in the source. It would make sense that the package is adjusted. I think it's a matter of replacing Admin::url() calls in the all the views.
I've tried overriding the Admin class, but the url method seems to be used internally too. So changing that there is breaking something else.

fixed in version 1.5

Could any one help me to install the library in a sub-directory on a shared hosting in easy steps. The problem that after I entered the login credentials, I redirected to
URL/admin/auth/login
but the correct path could be:
URL/subdir/public/admin/auth/login
I am using version 1.5 + laravel 5.5

@bakrianoo I have the same problem.

@omidakhavan  Thanks for login/header/menu fixing, the problem is that the issue remain for all actions.
For example in grid show / delete / edit have the wrong url.
The same for new / export button..
I have this situation in a testing environment and for now I fixed it by .htaccess in main web server root root
RewriteEngine On
RewriteRule ^admin/(.*)$ /subdir/public/admin/$1 [R=301,NC,L]

url(admin_base_path('/')) replace all admin_base_path


update flutter run --devfs

update flutter run --devfs to handle changes to the _createDevFS observatory call
add logic and timeouts to flutter run to better handle when the observatory becomes unresponsive to calls (w/o the service protocol connection itself closing)

@yjbanov

I'm afraid of leaking timed out HTTP requests. What level of robustness are we shooting for? LGTM otherwise.


Determining if a symbol is "truly unique" doesn't really make sense.

The stated use-case for the Symbol.isWellKnown() function is to allow a library author to detect if a symbol is "truly unique". This use-case doesn't make much sense to me though. If I were to take a guess, the thought process behind the introduction of isWellKnown() is to allow API authors to create a function that accepts a symbol as a parameter, and modifies an object we'll call foo to use that symbol as a key, and they're wanting to avoid giving foo special behaviors due to accidentally using a symbol that related to a built-in protocol. However, I don't think we should privilege built-in protocols over user-defined ones, and it's very possible that some third-party library which I'll call barLib also provides protocol implementations and symbols to go with them, and you wouldn't want to use barLib's symbols as keys on your foo object either (unless, you can guarantee that the foo object and barLib will never meet, then it doesn't really matter much).

Built-in protocols are already privileged over user-defined ones, because they're cross-realm values. Thus, that needs a way to detect it.


Create controllers for handling etcd proxy pods, etcd proxy Services, and certs for core etcd

This PR implements the EtcdProxyController to handle ReplicaSet and Service creation.

Fairly minor comments.  It's a good starting point.  I'd squash into something smaller for commit history.
Once you have tests, I think this is a good point to merge and then run parallel pulls for followups.


Add multiarch support agent

close  #181
close  #187

@anbraten preview can be found as codeberg-test and codeberg-test-alpine see https://hub.docker.com/r/woodpeckerci/woodpecker-agent/tags

@isuruf  ☝

Thanks @6543

@isuruf, @fpiesche, @kvaster, @TheFrenchGhosty please test and report if it work or dont

Hi, there isn't a arm64 version of the server image yet, right?

@ktprograms that's next :)

@6543 Thanks for the info!


Cannot login with password less in version 3.3.2

Hello dears,
i create a small login component to test the passwordless and i am using version 3.3.2 as mentioned in te title and after i add email anad click sign in this is the response
{     "status": 500,     "message": "{\"message\":\"field 'roles' not found in type: 'users': {\\\"response\\\":{\\\"errors\\\":[{\\\"message\\\":\\\"field 'roles' not found in type: 'users'\\\",\\\"extensions\\\":{\\\"path\\\":\\\"$.selectionSet.users.selectionSet.roles\\\",\\\"code\\\":\\\"validation-failed\\\"}}],\\\"status\\\":200,\\\"headers\\\":{}},\\\"request\\\":{\\\"query\\\":\\\"query users($where: users_bool_exp!) {\\\\n  users(where: $where) {\\\\n    ...userFields\\\\n  }\\\\n}\\\\n\\\\nfragment userFields on users {\\\\n  id\\\\n  createdAt\\\\n  disabled\\\\n  displayName\\\\n  avatarUrl\\\\n  email\\\\n  passwordHash\\\\n  emailVerified\\\\n  phoneNumber\\\\n  phoneNumberVerified\\\\n  defaultRole\\\\n  isAnonymous\\\\n  ticket\\\\n  otpHash\\\\n  totpSecret\\\\n  activeMfaType\\\\n  newEmail\\\\n  locale\\\\n  metadata\\\\n  roles {\\\\n    role\\\\n  }\\\\n}\\\",\\\"variables\\\":{\\\"where\\\":{\\\"email\\\":{\\\"_eq\\\":\\\"messi10010@gmail.com\\\"}}}}}\",\"stack\":\"Error: field 'roles' not found in type: 'users': {\\\"response\\\":{\\\"errors\\\":[{\\\"message\\\":\\\"field 'roles' not found in type: 'users'\\\",\\\"extensions\\\":{\\\"path\\\":\\\"$.selectionSet.users.selectionSet.roles\\\",\\\"code\\\":\\\"validation-failed\\\"}}],\\\"status\\\":200,\\\"headers\\\":{}},\\\"request\\\":{\\\"query\\\":\\\"query users($where: users_bool_exp!) {\\\\n  users(where: $where) {\\\\n    ...userFields\\\\n  }\\\\n}\\\\n\\\\nfragment userFields on users {\\\\n  id\\\\n  createdAt\\\\n  disabled\\\\n  displayName\\\\n  avatarUrl\\\\n  email\\\\n  passwordHash\\\\n  emailVerified\\\\n  phoneNumber\\\\n  phoneNumberVerified\\\\n  defaultRole\\\\n  isAnonymous\\\\n  ticket\\\\n  otpHash\\\\n  totpSecret\\\\n  activeMfaType\\\\n  newEmail\\\\n  locale\\\\n  metadata\\\\n  roles {\\\\n    role\\\\n<EMAIL_ADDRESS>  at /app/node_modules/.pnpm/graphql-request@3.7.0_graphql@16.2.0/node_modules/graphql-request/dist/index.js:340:31\\n    at step (/app/node_modules/.pnpm/graphql-request@3.7.0_graphql@16.2.0/node_modules/graphql-request/dist/index.js:63:23)\\n    at Object.next (/app/node_modules/.pnpm/graphql-request@3.7.0_graphql@16.2.0/node_modules/graphql-request/dist/index.js:44:53)\\n    at fulfilled (/app/node_modules/.pnpm/graphql-request@3.7.0_graphql@16.2.0/node_modules/graphql-request/dist/index.js:35:58)\\n    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)\"}",     "error": "internal-error" }
and here is the my component code:

This looks like an issue somwhere else. Can you try the following graphql query in the console?
{
  users(limit: 1) {
    roles {
      __typename
    }
  }
}

@dbarrosop still the same issue

If that query doesn't work it means there is something wrong in your project. Are you self-hosting, using the CLI or are you using a project in our cloud?

@dbarrosop the command working and return this issue in console it's status is 200 but here is screenshot:

Yes, that means your project is broken as it's missing tables and/or relationships that should be there. Are you self-hosting, using the CLI or are you using a project in our cloud? Did you modify anything in the auth schema? Have you changed auth's metadata somehow?

i don't do any thing just copy the code from docs to create an example, andi just wake up the app it's created 3 weeks ago and i also update the package to versin 3.3.2 thats all
can you give me instructions to make it work, or should i delete it and start a new one

If you can I'd suggest starting over, your project seems to be in a strange state and a clean start is certainly going to be the faster path.


Nif telemetry

I'm again hitting some memory issues with Argon2 v1. I don't want to argue that I need to increase the available memory, but I'm wondering if it wouldn't be useful to have some way of instrumenting argon2 for detecting issues. Currently verify_pass does return a boolean and that's it in terms of how much information I can get from the library. hash_password seems to report issues in way more detail.

Will adding log error help you in your situation?

It would at least be detectable, but it would be even nicer to have for example telemetry for any "there are issues with the execution" type of errors. Personally I'd like to harness e.g. the :alarm_handler module of the :sasl application to report such issues, where a log alone is not the best place of getting notified.

Ok. Could you give me a little more info (just a few sentences, maybe with an example) about telemetry, and the alarm_handler module that you mentioned? I can then make a more informed decision.
One more thing, how much memory is currently allotted to your server and and are you still setting the memory to 64 MiB?

Ok. Could you give me a little more info (just a few sentences, maybe with an example) about telemetry, and the alarm_handler module that you mentioned? I can then make a more informed decision.

Sure. From an operational standpoint I'd really like to know if failures are because of the nif having some issue like to less memory or if it worked like it's supposed to work, but e.g. arguments where incorrect or passwords simply don't match. hash_password seems to already return the information for me (as the client of the library) to detect problems. verify_pass on the other hand doesn't, but certainly it's called way more often than I'm hashing a new password. The information could either be given via the return value like for hash_password, which would probably need an update in the comonin interface, or via a side effect like a raised event with the telemetry library. Either would allow me to wire issues into :alarm_handler, which I wouldn't expect your library to do.
The problem I'm trying to break down in this issue really is that currently a user simply cannot login if there's an issue with memory without me as an operator being able to detect the issue. It just looks like the password was wrong.

I finally had some time to look at this issue again today. I think the first thing I need to do is make sure that errors are properly reported by verify_pass. I will update verify_pass so that it raises if there is insufficient memory - which is the same  as hash_password.

The v2.1 branch has the changes mentioned above - verify_pass now raises when there is an error. If you have any opinions about whether it should raise or not, please let me know - that is open for discussion.
As for adding telemetry, I am happy to do so, but I am just not sure how to proceed with that at the moment.

Merged v2.1 to master and released version 2.1.0 to hex.

Closing this for now - adding telemetry is still an option, but at the moment I don't have enough time to look into how best to do it.


[Event Request] codeunit 4514 "SMTP Message" - OnBeforeAddToInternetAddressList

Can we please get the following event.
local procedure AddToInternetAddressList(InternetAddressList: DotNet InternetAddressList; Recipients: List of [Text])
begin
        OnBeforeAddToInternetAddressList(Recipients); //NEW
	if not TryParseInternetAddressList(InternetAddressList, Recipients) then begin
		Session.LogMessage('0000B5M', StrSubstNo(RecipientErr, FormatListToString(Recipients)), Verbosity::Error, DataClassification::EndUserPseudonymousIdentifiers, TelemetryScope::ExtensionPublisher, 'Category', SmtpCategoryLbl);
		Error(RecipientErr, FormatListToString(Recipients));
	end;
end;

[IntegrationEvent(false,false)]
local procedure OnBeforeAddToInternetAddressList(var Recipients: List of [Text])
begin
end;

Hey @pri-kise
The event looks suspicious. If you want to modify the list of recipients, it's not a good idea to do it so late in the game.
What are your desired results? Maybe there's another way to achieve them. :)

We want to be able to change the recipients this late in one specific scenario:
For OnPrem Development to set a test@email and avoid emails are sent to real customers.

Related issue #9562 closed, as workaround was provided. If this issue persists, could you open a new issue for us? Then we'll give it another go.


npm run dev doesn't work on new nuxt project. Error info: Plugin not found: D:\vue-cropper-nuxt.nuxt\components\plugin.js

Version
System: win10
create-nuxt-app: npx version
node-js: v16.15.0
npx and npm: 8.5.5
Steps to reproduce
First run: npx create-nuxt-app
Then run: npm run dev
What is Expected?
What is actually happening?

I also encountered this problem

这是来自QQ邮箱的假期自动回复邮件。你好，我最近正在休假中，无法亲自回复你的邮件。我将在假期结束后，尽快给你回复。

When I built it last week, there was no problem.
The same problem occurred during the construction today. I'm sure my construction steps are consistent

Same error here

same error, in window10

Same here, win10

same error, in window10

Windows 10, me too.

Same here, win10

Win10, me too.

same error using yarn, win10

I tried it. Yarn and Npm all have this error.

solution here https://github.com/nuxt/nuxt.js/issues/10440#issuecomment-1126714019

solution here nuxt/nuxt.js#10440 (comment)

This can solve current error. Thank you help very much.


Removed a word in the head line...

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot


Automate WSL installation

Hi! I'd like to automate WSL feature installation for testing purposes. Using the following powershell script I can enable the Dev Mode and enable WSL feature.
# Enable Developer Mode
reg add "HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows\CurrentVersion\AppModelUnlock" /t REG_DWORD /f /v "AllowDevelopmentWithoutDevLicense" /d "1"

# Enable WSL feature
Enable-WindowsOptionalFeature -Online -NoRestart -FeatureName Microsoft-Windows-Subsystem-Linux

After rebooting the computer, I run the bash command for the first time, but it stops waiting for confirmation:

Is there a parameter to assume yes? Is there any way to avoid the question?
Thanks in advance!

Sorry, I'm closing this issue. I've discovered I can use the comand lxrun instead bash:
lxrun /install /y
lxrun /setdefaultuser myusername /y

Thanks!!

does your last command set the pwd of the user too?

Hi @jmlucjav
The only way I found to set the password:
lxrun /install /y
lxrun /setdefaultuser root /y
bash -c "echo 'myuser:mypassword' | chpasswd"
lxrun /setdefaultuser myuser /y

After running the commands above, your default user will be myuser and its password will be mypassword
Anyway, it must be there any easier way to do that... :/

Thanks! as long at it automates it, I'm good with the way above.

@xezpeleta - that looks about right.  A couple notes. The first setdefaultuser is not required because root will already be the default user.  I also think you need to set the password after creating the user, correct?


evm.md, driver.md: add PETERSBURG flag

Addresses https://github.com/kframework/evm-semantics/issues/320. Did I miss anything?

Updated the schedulestring and the test repo.

I believe that this failed due to the fact that the llvm backend cannot run some of the new tests that were added by this PR. The llvm backend still does not have adequate GC so we need to exclude a few tests that would otherwise crash with insufficient memory. I am running locally to determine which tests need to be excluded. I will push a fix to this PR once I have done this.


Make set unique together idempotent upstream

This is a proposed fix for the newly filed https://github.com/go-gorp/gorp/issues/371 to avoid creating duplicate indexes when calling SetUniqueTogether with the same parameters multiple times.
We had some discussion internally on our own fork, and agreed just to deduplicate on exactly the same order of columns, in case the resulting indexes when the order was different was desirable.

@nelsam, any chance this might get reviewed? It's not blocking us, but it shows up on my /pulls and I hate leaving things there too long :)
I'm fine with closing it if it's not high priority or desired, but thought I'd share just in case :)

Oh, thanks for reminding me.  I got distracted by another PR that has a potential data race I've been trying to trigger.
I'll come back to this today.


fixes os_type errors

Simple fix for change letters case
Also related to issue #31
fixes below error:
╷
│ Error: expected os_type to be one of [Windows Linux], got linux
│
│   with module.aci-devops-agent.azurerm_container_group.linux-container-group,
│   on .terraform\modules\aci-devops-agent\main.tf line 55, in resource "azurerm_container_group" "linux-container-group":
│   55:   os_type             = "linux"
│
╵
╷
│ Error: expected os_type to be one of [Windows Linux], got windows
│
│   with module.aci-devops-agent.azurerm_container_group.windows-container-group,

│   on .terraform\modules\aci-devops-agent\main.tf line 141, in resource "azurerm_container_group" "windows-container-group":

@uk-james thanks for your contribution. I cannot see any change in your commit. Can you please double check?

Any ETA on this fix?  I'm encountering the same errors.

For some odd reason, in all versions 0.9.0, 0.9.1, & 0.9.2 the main.tf has a lower case linux & windows.  Yet that isn't what I see in the code on master.  How odd.

@clbarrett are you running the terraform plan command from a Windows machine or from a unix-like shell? I am trying to reproduce the issue.

OK, so actually the update has already been merged (#32), but new release was not done. I just did it and it fixes the issue for me to switch to version 0.9.3. @clbarret @uk-james can you confirm that it solves the issue, please?

@jcorioland sorry for the delay! I did miss it you're right, i'll hold off until @clbarrett confirms, if all is ok, i'll go ahead and close this PR.

Yes, was reproducing this on a Windows machine using Git-Bash if it matters.  Version 0.9.3 fixed the problem (and one other subsequent one).  Thanks for the speedy response!


iTerm trying to load from wrong directory

Should be fixed in by adding a preferences files that are saved through mackup e7e19570d76268666a34cd17ad2c0afe4d108f27


Use from http server

Hi,
I'm doing some experiments with jsonnet_cgo and i'm getting some issues when trying to use it from a http handler. I'm experiencing hangs/timeouts, when trying multiple requests, there's likely something i'm not aware of in it's use.
This is just a boiled down example that replicates the issue:
package main

import (
	"fmt"
	"log"
	jsonnet "github.com/strickyak/jsonnet_cgo"
	"net/http"
)

var vm = jsonnet.Make()

func handler(w http.ResponseWriter, r *http.Request) {
	out, err := vm.EvaluateSnippet(`Test_Demo`, `"dark " + std.extVar("color")`)
	if err != nil {
		log.Fatalf("Error in evaluating jsonnet: %s", err)
	}
	fmt.Fprintf(w, "Expanded: %s", out)
}

func main() {
	vm.StringOutput(true)
	vm.ExtVar("color", "purple")
	http.HandleFunc("/", handler)
	http.ListenAndServe(":9999", nil)
}

And then,
$ while true; do curl --max-time 3 http://localhost:9999 ; done
Expanded: dark purple
curl: (28) Operation timed out after 3001 milliseconds with 0 bytes received
Expanded: dark purple
Expanded: dark purple
curl: (28) Operation timed out after 3004 milliseconds with 0 bytes received

Thanks in advance.

Interesting.
So the vm does its work, you print the result, but the HTTP session doesn't finish?
I'll try and look at this later today.

I can't reproduce the problem.
I'm using go1.8.1 on "linux_amd64" on "Ubuntu 14.04.5 LTS"
with 4-core "Intel(R) Core(TM) i7-6600U CPU @ 2.60GHz" (Lenovo X1 Carbon laptop).
However, it is slow, taking 15 to 40ms to EvaluateSnippet(),
and using 75% of a CPU in the server, while just serving your shell curl loop.
(FWIW, I think JSonnet is designed more for configuring a process at StartUp time,
than for evaluating during web requests.)
Also I see that we have memory leaks (C.CString without C.free) that I should fix.
Those would also show up as a problem if it is used frequently during web requests.

Thanks for looking into it.
Forgot to mention it was on macOS, but indeed i see it works as expected on linux .. I'm aware of it's not exactly super fast, but that's fine for my purposes.
Getting the memory leek fixed, would be a nice addition. :)
I'm closing this as this doesn't seem to be jsonnet_cgo related, but rather some likely gnarly stuff in cgo.
Thanks.


RFP - flash-decompiler-trillix

http://www.flash-decompiler.com/
Trial version is available for download. Unlimited evaluation time, limited function.
For packager: consider embedding installer in package. From license:

The Software can be copied and distributed on condition that the original copyright notice and disclaimer of warranty stay intact and the Licensee does not charge money or fees for the Software product, except to cover distribution costs.

Status: Review
https://chocolatey.org/packages/flash-decompiler-trillix

Status: Published
https://chocolatey.org/packages/flash-decompiler-trillix
Can be closed.


Index Ogit Name for InsertionOrder

Can one of the admins verify this patch?

@cy303  Please can you review


Race condition in Color.ToString()

There is an unprotected access to a static dictionary which will cause an Exception when two+ threads try to generate a PDF with a Color in it.
https://github.com/empira/MigraDoc/blob/cb9690cd2ee96ae3609508f551c2765f609e7a5a/MigraDoc/src/MigraDoc.DocumentObjectModel/DocumentObjectModel/Color.cs#L448-L466
Expected Behavior
The lines https://github.com/empira/MigraDoc/blob/cb9690cd2ee96ae3609508f551c2765f609e7a5a/MigraDoc/src/MigraDoc.DocumentObjectModel/DocumentObjectModel/Color.cs#L451-L465 should be wrapped in a lock() statement, with a check whether the dictionary has already been created. For threads that waited and no longer need to do anything. This prevents global locking at https://github.com/empira/MigraDoc/blob/cb9690cd2ee96ae3609508f551c2765f609e7a5a/MigraDoc/src/MigraDoc.DocumentObjectModel/DocumentObjectModel/Color.cs#L448
Actual Behavior
When two threads call Color.ToString() at the same time, the 2nd thread will get a DuplicateKeyEception

Will be fixed in 6.0.0
https://github.com/empira/PDFsharp/blob/d751497c3484fae9da4ae263568df00dc9554773/src/foundation/src/MigraDoc/src/MigraDoc.DocumentObjectModel/DocumentObjectModel/Color.cs#L41-L58


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

Codecov Report

Merging #731 (90c794f) into main (d8e03e4) will not change coverage.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main     #731   +/-   ##
=======================================
  Coverage   95.69%   95.69%
=======================================
  Files           1        1
  Lines         395      395
  Branches       53       53
=======================================
  Hits          378      378
  Misses         17       17


Using eval on Worker constructor instead of contents of temporary files

From doc it's possibile to pass to Worker constructor the code itself

Filename: [...]  If options.eval is true, this is a string containing JavaScript code rather than a pat

Astonishing, I didn't notice that :O
Thanks!


DEVX-1881: Update microservices code to work with cloud

DEVX-1881: Update microservices code to work with cloud
Enables schema registry configuration for security
Enables kafka sasl configuration

It looks like @rspurgeon hasn't signed our Contributor License Agreement, yet.

The purpose of a CLA is to ensure that the guardian of a project's outputs has the necessary ownership or grants of rights over all contributions to allow them to distribute under the chosen licence.
Wikipedia

You can read and sign our full Contributor License Agreement here.
Once you've signed reply with [clabot:check] to prove it.
Appreciation of efforts,
clabot

[clabot:check]

@confluentinc It looks like @rspurgeon just signed our Contributor License Agreement. :+1:
Always at your service,
clabot

LGTM.


[BUG] CVEs in Versions of Current Dependencies

While packaging opensearch-k8s-operator I noticed a handful of CVEs in current dependency versions. Specially, I came across these CVEs in the noted packages:
 📦 golang.org/x/net v0.13.0 (go-module)
        High CVE-2023-39325 GHSA-4374-p667-p6c8 fixed in 0.17.0
        Medium CVE-2023-45288 GHSA-4v7x-pqxf-cx7m fixed in 0.23.0
        Medium CVE-2023-44487 GHSA-qppj-fm5r-hxr3 fixed in 0.17.0
📦 google.golang.org/protobuf v1.30.0 (go-module)
        Medium CVE-2024-24786 GHSA-8r3f-844c-mc37 fixed in 1.33.0

There appear to be fixes for all four CVEs available in newer versions of the dependencies. Would the maintainers be amenable to a PR to bump these dependencies (or all dependencies, in fact) to the newest version?

Hi @jspeed-meyers. Thanks for reporting this. We'd be happy to accept a PR for updating the dependencies.

@prudhvigodithi Can you please take a look? Thanks


How to choose the best trained model

Hi, how do I select a model for testing when I have trained it? Does it rely on the loss or the test effect on the validation set? Your code does not give how to choose the best model.

Thanks for your interest of our work. For Charades-STA which has no validation set, you can directly select the best epoch in the test set. For TACoS and ActivityNet, you can select the epoch with the best performance (e.g., a weighted sum of these metrics) on validation set. In our practice, we use a sum of these metrics on validation set in the tensorboard to manually select epoch. For simplicity of our code, we delete the tensorboard part of the released code. Please implement it by yourself.


Das_Geek: Watch brisworkz[\W_]*com(?#obfuscated domain)

Das_Geek requests the watch of the watch_keyword brisworkz[\W_]*com(?#obfuscated domain). See the MS search here and the Stack Exchange search here.

brisworkz[\W_]*com(?#obfuscated domain) has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Makyen in Charcoal HQ


Python wrapper

Instead of using bash/shell to drive terragrunt it would be nice to use a higher level language like python.
Has anyone done a python wrapper for terragrunt?

I'm a bit confused. Where is bash/shell used with Terragrunt?

I'm a bit confused. Where is bash/shell used with Terragrunt?

To launch terragrunt, we have to launch it from a shell right?

Ah, yes Terragrunt is a CLI tool, so you typically run it from some sort of shell, though compiles into standalone binaries for all major OS's: https://github.com/gruntwork-io/terragrunt/releases.
I don't think anyone had requested higher-level language bindings for it before. What's the use case?

Our scenario is as follows:

We have moved away from bash run scripts to python after having a tough time handling errors, parsing output, code modularization and business logic in bash scripts
Calling terragrunt via python wrapper would give us the above benefits and would fit nicely with our existing code
Calling semantics would be "clean" and would have some static type checking
output = terragrunt.plan(arg1, calculated arg2)
we have python libraries for kubernetes, consul and vault - so it would be easy to pass their info to terragrunt where appropriate

Ah, gotcha. Well, I haven't seen such a Python wrapper floating around, so you may have to build it from scratch. FWIW, calling shell commands from python is pretty darn easy: call(["terragrunt", "plan"]).

calling shell commands from python is pretty darn easy: call(["terragrunt", "plan"]).

I know that :)
Now add to it

timeout
log buffer overflow
handling weird exit codes ( terraform plan returns 0 for no changes 1 for valid plan 2 for actual error)
parsing non-machine friendly output (which can change with new releases)

well you get the picture :)


test: remove unmaintained third-party tslint rules

The tslint-sonarts package is both deprecated and unmaintained. The rules provided by the package are now removed from the tslint configuration for the project.

Nit: can also be removed it from https://github.com/angular/angular-cli/blob/decaef592e11a2d1e4b9a6f10246a406cc4da2ce/scripts/validate-licenses.ts#L72

@clydin, can you do a 12.0.x version as it doesn’t apply cleanly?


Reworks the prometheus metrics to adhere to best practices

This is continuing work from https://github.com/kedacore/keda/pull/5174. I have rebased on main, resolved conflicts and working on fixing the e2e bug as described in https://github.com/kedacore/keda/pull/5174#issuecomment-1956280930.
Original text from @wonko

I reworked the prometheus metric names to follow conventions: added units where needed, and appended total.
Also passed a time.Duration down to the metrics handlers, as the conversion to units needs to happen when construction the metrics.
No tests were present, none added.
No idea where to track deprecation actions (when and how will the deprecated metrics be removed?)

Checklist

[x] I have verified that my change is according to the deprecations & breaking changes policy
[x] Changelog has been updated and is aligned with our changelog requirements
[x] A PR is opened to update the documentation on (repo) (if applicable)
[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)

Fixes #4854
Closes https://github.com/kedacore/keda/pull/5174
Relates to https://github.com/kedacore/keda-docs/pull/1258

/run-e2e sequential

/run-e2e sequential

/run-e2e sequential

/run-e2e sequential

https://github.com/kedacore/keda/pull/5687/commits/ced62ff46994542696f002c0f046d5702830ede8 contains original work from https://github.com/kedacore/keda/pull/5174
https://github.com/kedacore/keda/pull/5687/commits/e8d3c37f2c1b71efcc0ae65ece13283173f6c32e fixes the duplicate metric registration for keda_scaler_error_total
https://github.com/kedacore/keda/pull/5687/commits/30e16b9aed6a6e54a72290776545a4184f908d5e adds back scaled_job_errors with proper deprecation message

/run-e2e sequential

thx for picking this up @wozniakjan

/run-e2e sequential

/run-e2e sequential

/run-e2e sequential

/run-e2e sequential

tests finally pass, ptal @kedacore/keda-core-contributors

@wozniakjan It looks like https://github.com/kedacore/keda-docs/pull/1258 is not owned by you, but linked. Is this one still up to date or do we need another one?

It looks like kedacore/keda-docs#1258 is not owned by you, but linked. Is this one still up to date or do we need another one?

correct, kedacore/keda-docs#1258 was implemented by @wonko just like the majority of this PR originates in @wonko's PR https://github.com/kedacore/keda/pull/5174
There are a couple of minor suggestions in kedacore/keda-docs#1258, perhaps I can open a new docs PR that reflects reviewer suggestions.

We'll need a new issue for this; if I understand correctly we are just renaming them and keep the old ones.

correct, the old ones remained, just in the description KEDA will inform users these are deprecated. The new ones should be in alignment with prometheus conventions.

/run-e2e sequential

There are a couple of minor suggestions in kedacore/keda-docs#1258, perhaps I can open a new docs PR that reflects reviewer suggestions.

if these suggestions are the only additions we can merge them in the existing PR

There are a couple of minor suggestions in kedacore/keda-docs#1258, perhaps I can open a new docs PR that reflects reviewer suggestions.

if these suggestions are the only additions we can merge them in the existing PR

oh, I can see the PR is opened against 2.13, we should move that to 2.14. @wozniakjan could you please open a new doc PR then?


Poor performance of Merlin on GPU

Hi all!
Recently I've been running experiments with Merlin and tried to speed-up the learning process by utilizing GPU. Unfortunatelly I could only achieve about 50% increase in learning time, which is a bit suprising, because all Theano testing scripts had shown 10x to 30x performance increase.
Some details:
OS : Ubuntu 14.04
GPU : GeForce GTX 980
Theano : 0.8.2
NVidia drivers : 361
CUDA / CUDA Toolkit : 7.5
CuDNN : 5.0
From Merlin logs I conclude that GPU / CuDNN are being used. This is also confirmed by system performance monitoring. Network that I learn is pretty trivial:  4 layers  - TANH, TANH, TANH, LSTM. Nothing fancy, just a training example.
Has anyone ever encountered any issues like that? What other information I can provide to better describe the problem?
Any other comments and shared experience concearning Merling + GPU are very welcome :)

We currently have 2 machines set up to train models in Merlin. How many
files are you training and how long does it take to train the acoustic
model? And does it say it is using the GPU when you start the script?
For me, here are some recent stats:
machine 1: gaming laptop
OS: Ubuntu 14.04
GPU: GeForce GTX860
Theano: 0.8.2
numpy 1.11.2
NVidia driver: 352.39
CUDA: 7.5
CuDNN: 5.1
slt_arctic_full acoustic db training: 39.39 minues
This db uses 6 TANH layers with 1024 nodes per layer. I have not tried
using an LSTM layer at the end yet.
machine 2: new desktop
OS: CentOS7
GPU: GeForce GTX1080
Theano: 0.8.2
numpy 1.11.2
NVidia driver: 367.44
CUDA: 8
CuDNN: 5.1
slt_arctic_full acoustic db training: 5.99 minutes
On Thu, Jan 26, 2017 at 11:55 AM, pglushkov<EMAIL_ADDRESS>wrote:

Hi all!
Recently I've been running experiments with Merlin and tried to speed-up
the learning process by utilizing GPU. Unfortunatelly I could only achieve
about 50% increase in learning time, which is a bit suprising, because all
Theano testing scripts had shown 10x to 30x performance increase.
Some details:
OS : Ubuntu 14.04
GPU : GeForce GTX 980
Theano : 0.8.2
NVidia drivers : 361
CUDA / CUDA Toolkit : 7.5
CuDNN : 5.0
From Merlin logs I conclude that GPU / CuDNN are being used. This is also
confirmed by system performance monitoring. Network that I leart is pretty
trivial 4 layers - TANH, TANH, TANH, LSTM. Nothing fancy, just a training
example.
Has anyone ever encountered any issues like that? What other information I
can provide to better describe the problem?
Any other comments and shared experience concearning Merling + GPU are
very welcome :)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/CSTR-Edinburgh/merlin/issues/78, or mute the thread
https://github.com/notifications/unsubscribe-auth/ASbibMG3NOd-wrGEWO9UChMp8S077XCXks5rWHuYgaJpZM4LugxH
.

Hi, dreamk73, thanks for your reply!
Log of training clearly states that GPU is being utilized, yes. It says:
...
Running on GPU id=0 ...
Using gpu device 0: GeForce GTX 980 (CNMeM is enabled with initial size: 95.0% of memory, cuDNN 5005)
...
We use 200 files per 1 epoch of acoustic model training, duration of each file is about 3 sec on average. Samplerate is 16000. Sizes of our layres are (512, 512, 512, 512).
One epoch of training takes about 40 seconds on a GPU. At the same time, when Theano uses CPU + OpenMP, 1 epoch of training takes about 60 seconds. So speed-up does not seem that impressive.
And how long does 1 epoch of acoustic model training take on your machines?

The average time on the laptop is 85 seconds per epoch and on the new desktop only 10 seconds. I am not sure what all contributes to the processing speed. I haven't tried to run it with the CPU but I do know that it is a lot slower.
I know that the GTX 1080 is much faster than older models. But other factors can be at play here as well. What version of numpy do you use? One of the merlin people told me that could make a difference as well.

Dear dreamk73,
current version of numpy in environment used by Merlin on our PC is 1.8.2 Will try to update it and measure the performance again.
Could you please tell me how many files do you process per 1 epoch and what is the mean duration of each file?

@pglushkov are you using local disk or network disk?
Could you please check the I/O time and CPU time?

We are using local disk. Sorry, what exactly do you mean by I/O and CPU time?

@dreamk73
Today I've tried the training with numpy 1.11.2 - no speed-up unfortunately.

Could you please only calculate the computation time for this line of code?
this_train_error = train_fn(current_finetune_lr, current_momentum)
Other codes are related to reading and processing data, not actual neural network training

@zhizhengwu
Hello! If you mean the line in the most inner loop of train_DNN() function, then average time for 1 batch execution is about 0.2 sec on average.


WIP: Future changes to the site

Hi,
After revisiting the site, and getting wiser, I plan for the future some new changes.

[ ]  Deploy static site from a Travis build-job to gh-pages instead
[ ]  Strip down the Node parts... 🎉
[ ]  Clean up files/structure

Suggestions are welcome! 😄
Suggestion

[ ] Update screenshots

seems like sensible changes :) There was one thing I've been thinking about for a little while, albeit not directly related to the site, is updating the screenshots. They are a bit out of date, not much and maybe not worth updating but maybe worth considering.

Good one.  Added to the list. 👍

Also maybe increase column size for the text content, a bit narrow I think. I can take the new screenshots, so we get consistency with the resolution and the font selection.

I needed to change the repo name, as when using the organization URL layout, the site can only be served from master, and not from e.g.gh-pages.  Hoping the changes will be swift.
The new setup should be more flexible.

Been looking at trying out Parcel.js instead of Webpack.  But had no luck the first attempt.
There have been a new version between, so maybe I'll try it again.  Also, TailwindCSS is soon also out with its 1.0 release.  So changes it required.
I've also yet to push the new images that you supplied. 👍  Been busy with school.  Hopefully some time clears up soon.

Parcel looked pretty neat. Push whenever you can, no worries.


Add retry strategy to increase stability

Add a retry policy to increase stability when calling OpenAI API.
This PR is a workaround for #55 , as explained here: https://github.com/arc53/DocsGPT/issues/90#issuecomment-1430688573
This PR closes: #90

The wait time can be increased by retry times.
What's more, I saw there are some index store stuff in the except block, maybe should also add them to retry function


SentinelSCGAutoConfiguration Unable to register correctly (csp.sentinel.app.type)，

Which Component
Sentinel
Describe the bug
com.alibaba.cloud.sentinel.gateway.scg.SentinelSCGAutoConfiguration#initAppType
Didn't refresh com.alibaba.csp.sentinel.config.SentinelConfig#appType after init SentinelSCGAutoConfiguration，cause machine hearbeat message can't get the right app.type
初始化SentinelSCGAutoConfiguration配置没有刷新SentinelConfig#appType，导致发送心跳包时无法获得正确的app.type
Additional context
Version 2.1.0 RELEASE

The initialization of application type in SentinelSCGAutoConfiguration#initAppType before SentinelConfig#resolveAppType by default.
But if you configure with spring.cloud.sentinel.eager=true, the initialization action will after SentinelConfig#resolveAppType.

thx, when i use default config spring.cloud.sentinel.eager = false is ok

i think it could be enhance and i reopen it.

Hi, has it been solved yet?

Any progress on this? cc @yuhuangbin @theonefx


[BUG] Initializing library 'Browser' with no arguments failed

Describe the bug
Hello,
Robotcode cannot import the browser library as it fails to find node
To Reproduce
Steps to reproduce the behavior:
*** Settings ***
Library             Browser

Browser is highlighted and following error is displayed (note that I manually edited the playwright.py file to dump the content of the PATH, which shows /usr/bin:/bin:/usr/sbin:/sbin and not my virtual env):
Import definition contains errors.robotcode.namespace(ImportContainsErrors)
playwright.py(72, 1): Initializing library 'Browser' with no arguments failed: PATH: /usr/bin:/bin:/usr/sbin:/sbinCouldn't execute node. Please ensure you have node.js installed and in PATH. See https://nodejs.org/ for instructions. Original error is [Errno 2] No such file or directory: 'node'
Traceback (most recent call last):
File "/Users/***/Documents/project/.venv/lib/python3.8/site-packages/Browser/playwright.py", line 69, in ensure_node_dependencies
run(["node", "-v"], stdout=DEVNULL, check=True)
File "/Users/***/.pyenv/versions/3.8.19/lib/python3.8/subprocess.py", line 493, in run
with Popen(*popenargs, **kwargs) as process:
File "/Users/***/.pyenv/versions/3.8.19/lib/python3.8/subprocess.py", line 858, in __init__
self._execute_child(args, executable, preexec_fn, close_fds,
File "/Users/***/.pyenv/versions/3.8.19/lib/python3.8/subprocess.py", line 1720, in _execute_child
raise child_exception_type(errno_num, err_msg, err_filename)
FileNotFoundError: [Errno 2] No such file or directory: 'node'

During handling of the above exception, another exception occurred (note that I manually edited the playwright file to display the PATH environment variable in the error message):

Traceback (most recent call last):
File "/Users/***/Documents/project/.venv/lib/python3.8/site-packages/Browser/browser.py", line 875, in __init__
self.playwright = Playwright(
File "/Users/***/Documents/project/.venv/lib/python3.8/site-packages/Browser/playwright.py", line 53, in __init__
self.ensure_node_dependencies()
File "/Users/***/Documents/project/.venv/lib/python3.8/site-packages/Browser/playwright.py", line 72, in ensure_node_dependencies
raise RuntimeError(
RuntimeError: PATH: /usr/bin:/bin:/usr/sbin:/sbinCouldn't execute node. Please ensure you have node.js installed and in PATH. See https://nodejs.org/ for instructions. Original error is [Errno 2] No such file or directory: 'node'

Expected behavior
Library is imported
Logs
Robotcode output:
client for file:///Users/***/Documents/project stopped.
executeRobotCode: /Users/***/Documents/project/.venv/bin/python -u -X utf8 /Users/***/.vscode/extensions/d-biehl.robotcode-0.85.0/bundled/tool/robotcode --format json --no-color --no-pager --default-path . profiles list
executeRobotCode: exit code 0
create Language client: RobotCode Language Server mode=pipe for folder "project"
trying to start Language client: RobotCode Language Server mode=pipe for folder "project"
client for file:///Users/***/Documents/project starting.
executeRobotCode: /Users/***/Documents/project/.venv/bin/python -u -X utf8 /Users/***/.vscode/extensions/d-biehl.robotcode-0.85.0/bundled/tool/robotcode --format json --no-color --no-pager --default-path . profiles list
executeRobotCode: exit code 0
client for file:///Users/***/Documents/project running.
executeRobotCode: /Users/***/Documents/project/.venv/bin/python -u -X utf8 /Users/***/.vscode/extensions/d-biehl.robotcode-0.85.0/bundled/tool/robotcode --format json --no-color --no-pager --default-path . profiles list
client for file:///Users/***/Documents/project started.
executeRobotCode: /Users/***/Documents/project/.venv/bin/python -u -X utf8 /Users/***/.vscode/extensions/d-biehl.robotcode-0.85.0/bundled/tool/robotcode --format json --no-color --no-pager discover info
executeRobotCode: /Users/***/Documents/project/.venv/bin/python -u -X utf8 /Users/***/.vscode/extensions/d-biehl.robotcode-0.85.0/bundled/tool/robotcode --format json --no-color --no-pager --default-path . discover --read-from-stdin all
executeRobotCode: exit code 0
executeRobotCode: exit code 0
executeRobotCode: exit code 0

Robotcode LSP output:
[Error - 11:06:21 AM] Server process exited with code 0.

Desktop (please complete the following information):

VS Code Version 1.89.1
RobotCode Version 0.85.0
OS: MacOS Ventura 13.3.1
Python Version 3.8.19
RobotFramework Version 7.0.1
robotframework-browser 18.7.0
robotframework-robocop 5.3.0

Additional context
I did as you suggested here but this did not change anything :/
Note that I'm using nvm, can this interfere?
Thanks

nvm is based on node but it is not node.
did you install node? and where did you install node?
did you call rfbrowser init after installing the Browser library? does this work?
if you start vscode, is node in PATH?

From vscode terminal (with my venv):
$ which node
/Users/***/.nvm/versions/node/v18.19.0/bin/node

$ rfbrowser init chromium                                                                                [11:14:29]
2024-08-22 11:14:52,792 [INFO    ] ==============================================================================================================
2024-08-22 11:14:52,792 [INFO    ] Installing node dependencies...
2024-08-22 11:14:53,701 [INFO    ] Installing rfbrowser node dependencies at /Users/***/Documents/project/.venv/lib/python3.8/site-packages/Browser/wrapper
2024-08-22 11:14:53,869 [INFO    ] npm WARN config production Use `--omit=dev` instead.
2024-08-22 11:14:55,333 [INFO    ]
2024-08-22 11:14:55,333 [INFO    ] added 67 packages, and audited 68 packages in 1s
2024-08-22 11:14:55,333 [INFO    ]
2024-08-22 11:14:55,333 [INFO    ] 8 packages are looking for funding
2024-08-22 11:14:55,333 [INFO    ]   run `npm fund` for details
2024-08-22 11:14:55,334 [INFO    ]
2024-08-22 11:14:55,334 [INFO    ] found 0 vulnerabilities
2024-08-22 11:14:55,338 [INFO    ]
2024-08-22 11:14:55,339 [INFO    ] Installing browser chromium binaries to /Users/***/Documents/project/.venv/lib/python3.8/site-packages/Browser/wrapper
2024-08-22 11:14:55,339 [INFO    ] npx --quiet playwright install chromium
2024-08-22 11:15:22,759 [INFO    ] |■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■■| 100% of 1 MiB
2024-08-22 11:15:22,820 [INFO    ]
2024-08-22 11:15:22,820 [INFO    ] rfbrowser init completed
2024-08-22 11:15:22,820 [INFO    ] ==============================================================================================================

So yes I have node in my path and I ran rfbrowser init, I'll give a try on the reboot

yup multiple times
Also rfbrowser clean-node followed by rfbrowser init

ok, because nvm is running on the shell. did you call the nvm command before calling rfbrowser init?
did you try to open vscode from terminal/shell where you call nvm before?
I do not know nvm, so that's why I'm asking. Is there a way to put you node installation to the system/userpath without starting a shell? I know this from pyenv, there is a shim folder, where you can manage the default user python, the only thing you have to do ist, to put it into the userpath. There is an nvm alias command, maybe this?

If I run a python from my terminal with my venv it is working fine:
$ python
Python 3.8.19 (default, Aug 20 2024, 11:17:47)
[Clang 14.0.3 (clang-14<IP_ADDRESS>.1)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> from subprocess import run
>>> run(["node", "-v"], check=True)
v18.19.0
CompletedProcess(args=['node', '-v'], returncode=0)

I suspect the issue on the PATH not being properly set up in the LSP

Actually nvm is setting for me the node globally in my system, so from any terminal I got the node with the version I've set with nvm (here 18.19.0)

yes, because nvm is running in the .bashrc or .zshrc, but if you start vscode, vscode is started without a shell, so the nvm script is not started. And yes, if you open a terminal in vscode a full shell is started, but vscode/lsp does not know what is started in this shell.
So you need to set/update the paths with nvm before you start vscode.

FYI https://github.com/MarketSquare/robotframework-browser/pull/3758


clasp loginfails with--creds` option

(Note: Non-breaking issues are likely not to be prioritized. Please consider a PR in addition to your issue)

Expected Behavior
Success clasp login --creds creds.json and it is able to log in with my own credential.
Actual Behavior
Run clasp login --creds creds.json and get the following error message.
Logging in locally...

No valid .clasp.json project file. You may need to `create` or `clone` a project first.

Steps to Reproduce the Problem

Go to APIs & Services > Credentials of my GCP project.
Create OAuth 2.0 Client IDs (with type of Desktop)
Download the credential json file
run npm i clasp -g and install clasp globally
run clasp login --creds path_to_credential.json

Specifications

Node version (node -v): v12.13.0
Version (clasp -v): 2.3.0
OS (Mac/Linux/Windows): Mac

@saito-ya can you test with the #791 unofficial release?
npm uninstall -g @google/clasp && npm install -g forked-clasp
clasp --version
# should be 2.4.0 or more

Intended behavior to require an existing project. --creds is really only used to set up auth for the run command and it does some other things in the associated GCP project.
Since it's a bit more specialized than the login command implies, may revisit this in 3.0 and break it out to a separate command to configure the run command.


Change Designate subnet

The current Designate subnet conflicts with the default serviceNetwork in OpenShift:
https://github.com/openshift/installer/blob/0eafdbb77fd62f8311ba8d9abda145f3280c5f79/docs/user/customization.md\?plain\=1\#L56
This change moves the designate subnet we're using from <IP_ADDRESS>/24 to <IP_ADDRESS>/24 to avoid this conflict.

/lgtm

/test install-yamls-deploy


feat(chain-adapters): add getRpcUrl to EVMBaseAdapter

Description
This adds a rpcUrl class property in EVMBaseAdapter, and a getRpcUrl getter.
Note that this hasn't been added to the other adapters, as the current need for getting the rpcUrl is to add an ethereum chain in https://github.com/shapeshift/hdwallet/pull/547 - if we do end up architecturally needing it in other adapters, we could add it there, but IMO it shouldn't be added just for the sake of consistency.

conceptually good - but agree with others to pop this on the EVMBaseAdapter


BYOS on Linux AppService limited to 5 volumes

Can you elaborate on the five mount points per app limitation? Why is this limitation? And why is it global regardless the AppService SKU ?
Thank you

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

ID: 3cd70fac-5d7c-7637-8385-f3f4af3ba0ad
Version Independent ID: 15d95fc5-f08e-a26c-ce06-8ead78474661
Content: Mount Azure Storage as a local share - Azure App Service
Content Source: articles/app-service/configure-connect-to-azure-storage.md
Service: app-service
GitHub Login: @msangapu-msft
Microsoft Alias: msangapu

Apologies for the delay! Limitation of 5 volumes for BYOS mounts is built into the backend API for this feature and yes it is consistent across all the pricing tiers. At this time, there is no doc update required. Thanks for sharing your feedback.

My question is about why is it on the API Backend. What is the Linux limitations that are behind this limitation ? Why this limitation is not on Windows Plans ?
Thank you

Shessane, Thanks for the follow-up. The limitation is same for Linux, Windows Code and Windows Containers. As mentioned in the same documentation -  Mount Azure Storage as a local share - Azure App Service | Microsoft Docs


www.ldu.edu.cn

Add Ludong University

@gitccms Thank you for your patience while waiting for a review and processing your request. Pull request merged.
Please start requesting the licenses in about an hour, to let the changes to propagate through our system.


Godot doesn't show components that exist in the .tscn file and deletes them when I save the scene

Godot version: 3.3
OS/device including version:
MacOS Catalina
Issue description:
My .tscn file appears (by checking it with a text editor) to have a couple components that don't show up in Godot when I open it in Godot.
When I save the scene in Godot, those components are deleted.
Steps to reproduce:
Save a scene
Minimal reproduction project:
You can check out my project https://github.com/johnfn/ld48/commit/b0f8e6263e403efe69efe7633d5e3413951e6311 - specifically at the hash provided
Then just open it in Godot 3.3 and save RoundTree.tscn

Unfortunately, it's very difficult to figure out what's going on exactly without a proper minimal reproduction project.

Nevermind - this was my own issue. Had a tool script and didn't realize it was deleting stuff!


Adds transport_id variable assignment back to outbound enqueue method

Signed-off-by: Akiff Manji<EMAIL_ADDRESS>

Codecov Report

Merging #1776 (211af2b) into main (27b238a) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1776   +/-   ##
=======================================
  Coverage   95.23%   95.23%
=======================================
  Files         525      525
  Lines       33056    33057    +1
=======================================
+ Hits        31482    31483    +1
  Misses       1574     1574


Add scenario for OVS-DPDK

Currently there is no scenario to pick ovs-dpdk scenario from architecture repository - https://github.com/openstack-k8s-operators/architecture/tree/main/examples/va/nfv/ovs-dpdk.

/lgtm

/ok-to-test
/approve


Name and Version empty for Java package when scanning provided image

What happened:
When using syft from the tip of main for image caphill4/syft-manifest-bug:latest the following behavior was experienced:

Name field was blank for multiple discovered package
Version field was blank for multiple discovered package
Java Metadata provided a manifest version, but no other manifest details were made available

syft -o json caphill4/syft-manifest-bug:latest

Example package below - note there were multiple of these:
  {
   "id": "40358cd756d70d11",
   "name": "",
   "version": "",
   "type": "java-archive",
   "foundBy": "java-cataloger",
   "locations": [
    {
     "path": "/opt/asserts/api-server/enterprise-server.jar",
     "layerID": "sha256:4d8a814cf85fcbdaa2cff3f001c705392dcc05e1bf659fcaac718b84e9dfc662",
     "annotations": {
      "evidence": "primary"
     }
    }
   ],
   "licenses": [],
   "language": "java",
   "cpes": [],
   "purl": "pkg:maven/",
   "metadataType": "JavaMetadata",
   "metadata": {
    "virtualPath": "/opt/asserts/api-server/enterprise-server.jar:BOOT-INF/lib/1-555680818.jar",
    "manifest": {
     "main": {
      "Manifest-Version": "1.0"
     }
    },
    "digest": [
     {
      "algorithm": "sha1",
      "value": "4c1415ccb35494ea281446ce12463ff40263c910"
     }
    ]
   }
  },

What you expected to happen:
Syft should have an option or config to eliminate packages after the fact if there is not enough identifying information.
Alternatively, the file cataloger could be enhanced to show nested jar information so this information is not lost, but instead moved from package information to file information.
Example:
syft -o json --prune caphill4/syft-manifest-bug:latest

The offending jars also had some warnings, but these seem to be related to regex matching. Packages are still being created for these jars, but given they have almost no identifying information the package is blank besides the path and virtualPath fields showing their location
[0003]  WARN unexpectedly empty matches for archive 'BOOT-INF/lib/57-30595491.jar'
[0003]  WARN unexpectedly empty matches for archive 'BOOT-INF/lib/67-1304492339.jar'
[0003]  WARN unexpectedly empty matches for archive 'BOOT-INF/lib/67-1304492339.jar'
[0003]  WARN unexpectedly empty matches for archive 'BOOT-INF/lib/32-409283951.jar'
[0003]  WARN unexpectedly empty matches for archive 'BOOT-INF/lib/32-409283951.jar'
[0003]  WARN unexpectedly empty matches for archive 'BOOT-INF/lib/6-710714459.jar'
[0003]  WARN unexpectedly empty matches for archive 'BOOT-INF/lib/6-710714459.jar'
[0003]  WARN unexpectedly empty matches for archive 'BOOT-INF/lib/17-2051466981.jar'
......

Steps to reproduce the request:
syft -o json caphill4/syft-manifest-bug:latest
Inspect the output for the above characteristics
Anything else we need to know?:
Built from syft main as of - a46d12270f1e49d9bddb8bb4c082dcec34a8e95b
Environment:

Output of syft version: a46d12270f1e49d9bddb8bb4c082dcec34a8e95b
OS (e.g: cat /etc/os-release or similar): OSX

Alright - I’ve hit the end of investigating this and have this update -
Currently the behavior is correct in that syft is identifying the main parent jar enterprise-server . A package does exist in the SBOM for that main package along with the manifest information.
The confusion about it possibly not existing comes from path and virtualPath fields being conflated. This might lead the user to incorrectly believe that blank information is being inserted for path=/opt/enterprise-server.jar . If we look at the virtualPath  it shows that these blank entries actually come from nested jars with limited identifying information virtualPath=/opt/enterprise-server.jar:BOOT-INF/lib/1-555680818.jar
Potential solutions:

A Prune option which eliminates packages in a post processing step that do not have both name and version fields. This presents some challenges in that the file cataloger by default does not account for nested jar paths. This kind of option would remove any kind of detection or representation of these nested jars leading to an incomplete SBOM

While the catalogers are logically detached at the moment, I would be more in favor of the above pruning option if I knew the results showed up somewhere else on the SBOM. The files field could be enhanced via the fileCataloger to show something like below, with an option to also create a relationship to the parent package.
{
"id": "e82d211f6cc65681",
"location": {
"path": "/opt/enterprise-server.jar:BOOT-INF/lib/1-555680818.jar",
"layerID": "sha256:4693057ce2364720d39e57e85a5b8e0bd9ac3573716237736d6470ec5b7b7230"
}
},

Let me know in this thread comments or thoughts :smiley: but I've put this into our backlog for future discussion during community/team sync

@spiffcs I think this has come up with a couple of other users as well, so probably worth restarting the discussion on what the correct answer is for Syft here.
My (current) 2c is that a jar without metadata or any other identifiers as a java artifact is really just the same as a tar file or zip file. In terms of artifact relationships it should be treated like an archive that is recursed into by the cataloger to find other artifacts rather than an identified package. The file cataloger can gather digests etc, but the java cataloger should probably skip if it cannot identify the actual java software in the jar file.
This would also be a good candidate for any "known-unknown" classification logic, to identify to the SBOM reader that theres is content that is known to be likely a part of an application or artifact but that cannot be identified.


node get command needs heath info

This should provide access to whatever we get after bloqpriv/cloud-nodes#15

Not relevant.


fixed sets

I realized that there are incomplete sets shown in the edit sets dialog:

A short search showed that they come from your tokens.xml file!
None of these sets show up in mtgjson or on gatherer...
What I could find about them:

FNM: promotional flip Token for Friday Night Magic
(http://archive.wizards.com/Magic/magazine/article.aspx?x=mtg/daily/arcana/938)
Avacyn Restored (The Helvault Experience): some kind of promotional pre release token prizes for Avacyn Restored
(http://mtgsalvation.gamepedia.com/Token#Double-faced_tokens)
(http://archive.wizards.com/Magic/magazine/article.aspx?x=mtg/daily/arcana/963)
League: promotional token prizes for in-store leagues for different sets(http://mtgsalvation.gamepedia.com/League)

http://magiccards.info/extras.html
Suggestions:

change all of the special sets above in your file to promotional sets, since they are promotional (e.g. FNM to pFNM)

There is a set for Friday Night Magic promo cards (pFNM) already in mtgjson.
This will at least make the FNM set disappear.
Since you do not provide additional information in your file about the sets, Oracle doesn't know about them and can't include this information in the cards.xml.
I found out that there are sets information in cards.xml... maybe we can include them in the token.xml as well... and it worked! :+1:
    <sets>
        <set>
            <name>ARH</name>
            <longname>Avacyn Restored (The Helvault Experience)</longname>
            <settype>Promo</settype>
            <releasedate>2012-04-28</releasedate> (i put the pre-release date from avacayn restored here!)
        </set>
        <set>
            <name>LGE</name>
            <longname>League</longname>
            <settype>Promo</settype>
            <releasedate>2012-07-13</releasedate> (i put the release date of m13 here, since that was the start of this tournament types)
        </set>
    </sets>

after the change (i didn't add the small p's already for the pic)

Maybe you want to tell the user that they are artificial sets and no real sets somehow...

use "Token" instead of "Promo" as settype would work for example
use tFNM, tLGE and tARH instead the small p
this would separate the sets better from normal/real sets

selfnote:

need to fix example set (one  too much)

Those tokens were in the file before I started working on it, so if they cause trouble then I'm inclined to just comment them out.
The Avacyn Restored one might belong to pREL Release Events or pLPA Launch Parties.
@Sembiance
What would the mtgjson set code for these would be if those 'sets' used your conventions?

Avacyn Restored (The Helvault Experience) fits nicely to the pPRE set
(not pREL ! it's was a pre-release special as you can read in my first post, no release special)
I wasn't sure about pLPA exactly... but hey:
http://archive.wizards.com/Magic/magazine/article.aspx?x=mtg/daily/feature/181a
Launch Parties = former Release Events (not Pre-Release!)
(there are mentions about the "store League" as well in the article!)
So just the League 'set' needs to get resolved...

This PR can't be merged because:

It is out of date.
I don't want to put  tags in the file.
I didn't like the arbitrary white-space changes.


C2C Networking: Diego

Summary
The C2C communication would allow apps to communicate with each other directly.
Intended Outcome
The C2C should allowed the direct communication between app instances when networking policies are defined, in a security way (TLS enabled).  It should also support app service discovery that allows apps to find each other internal addresses.
3rd party components may be plugged where necessary (CoreDNS instead of BOSH-DNS as an example).
Note
The C2C networking on Diego is a well known core feature that is missing on KubeCF.

@viovanov maybe below could work for the internal app domain resolving:

install bosh-dns (have it listen on port 53) and bosh-dns-adaptor into one POD, naming it as c2c-dns-adaptor, and create service (port 53) for these PODs
config the coredns with below, this could be implemented with this feature: https://github.com/cloudfoundry-incubator/quarks-operator/issues/1036

    forward  apps.internal <c2c-dns-adaptor-service-IP>

Then the internal app domain name resolving flow is: app queries domain, like app1.apps.internal, this query goes to coredns, coredns forward this query to bosh-dns, and bosh-dns forwards this query to bosh-dns-adaptor.

@viovanov maybe I am missing something (most probably), but way would we need both BOSH-DNS and coreDNS on the same setup?

BOSH DNS requires the BOSH director, a dependency we can't satisfy.
@JimmyMa the problem with that solution is that there can be more than one internal app domain.
We have found a solution for this, with what we call "App DNS" - a "CoreDNS based" replacement for the BOSH DNS Adapter.

Can we get a status update on this, and close if appropriate?

@troytop it's GA which means available to the public. It was not my plan to close it from the roadmap board but to set the right label.


upstream: update edsServiceName to return a ref rather than a string to avoid unnecessary memory allocation and copy

Commit Message: upstream: update edsServiceName to return a ref rather than a string to avoid unnecessary memory allocation and copy
Additional Description:
As title.
Risk Level: low.
Testing: n/a.
Docs Changes: n/a
Release Notes: n/a.
Platform Specific Features: n/a.

/retest

/wait-any


router.back()

when I navigate to "/profile" from "/" and then click to change language, route changes to "it/profile" and then when I click on back button: <Button @click="router.back()">{{ t('back') }} the page doesn't navigate to "/it" it just back to "/profile"
I want when in homepage I navigate to "/profile" and then click to change language the route changes to "it/profile" and then when I click on back button it navigate to "/it/"

I'm not sure if we should change the navigation history (and if it is possible to accurately do so) but I'm open to look into if there's more demand.


Minimum Rust version

Add minimum supported Rust version in Cargo.toml

Added MSRV 1.73 to workspace cargo.toml - development branch

Awesome 😎


[2.1] UNDERTOW-1622: Implement blocking IO timeouts

Added two new UndertowOptions:

BLOCKING_READ_TIMEOUT: The amount of time in milliseconds a single
blocking read operation may take before a timeout IOException is thrown.
Unlike Options.READ_TIMEOUT this only applies to blocking operations
which can be helpful to prevent the worker pool from becoming saturated
when clients stop responding.
BLOCKING_WRITE_TIMEOUT: The amount of time in milliseconds a single
blocking write operation may take before a timeout IOException is
thrown. Unlike Options.WRITE_TIMEOUT this only applies to blocking
operations which can be helpful to prevent the worker pool from becoming
saturated when clients stop responding.

A version of xnio-api containing the fix for XNIO-356 is required
for these options to work correctly, otherwise erronious timeouts
may be reported when channel.await{Readable,Writable} returns
spuriously.
This commit does not apply the write timeout to Channels.flushBlocking
invocations because that will require either a local utility class
to support flushBlocking with a timeout, or a backport of XNIO-357.

The proxy test cases are failing on the new tests, I'll investigate when I have a moment.

I've run benchmarks locally with and without blocking timeouts, differences appear to be within the margin of error so I've not updated existing benchmarks to test this case:
Benchmark                                    (blockingTimeout)  (listenerType)  Mode  Cnt     Score     Error  Units
SimpleBenchmarks.benchmarkBlockingEmptyGet            DISABLED            HTTP  avgt    3    30.998 ±   0.678  us/op
SimpleBenchmarks.benchmarkBlockingEmptyGet            DISABLED           HTTPS  avgt    3    61.496 ±   4.240  us/op
SimpleBenchmarks.benchmarkBlockingEmptyGet             ENABLED            HTTP  avgt    3    31.622 ±   1.179  us/op
SimpleBenchmarks.benchmarkBlockingEmptyGet             ENABLED           HTTPS  avgt    3    59.764 ±   2.326  us/op
SimpleBenchmarks.benchmarkBlockingEmptyPost           DISABLED            HTTP  avgt    3    31.605 ±   0.124  us/op
SimpleBenchmarks.benchmarkBlockingEmptyPost           DISABLED           HTTPS  avgt    3    61.610 ±   2.446  us/op
SimpleBenchmarks.benchmarkBlockingEmptyPost            ENABLED            HTTP  avgt    3    31.811 ±   0.538  us/op
SimpleBenchmarks.benchmarkBlockingEmptyPost            ENABLED           HTTPS  avgt    3    59.757 ±  10.410  us/op
SimpleBenchmarks.benchmarkBlockingLargeGet            DISABLED            HTTP  avgt    3  2912.159 ± 418.062  us/op
SimpleBenchmarks.benchmarkBlockingLargeGet            DISABLED           HTTPS  avgt    3  4925.408 ± 741.017  us/op
SimpleBenchmarks.benchmarkBlockingLargeGet             ENABLED            HTTP  avgt    3  2873.870 ± 416.579  us/op
SimpleBenchmarks.benchmarkBlockingLargeGet             ENABLED           HTTPS  avgt    3  4924.962 ± 482.183  us/op
SimpleBenchmarks.benchmarkBlockingLargePost           DISABLED            HTTP  avgt    3   612.207 ±  35.712  us/op
SimpleBenchmarks.benchmarkBlockingLargePost           DISABLED           HTTPS  avgt    3  2629.098 ± 145.288  us/op
SimpleBenchmarks.benchmarkBlockingLargePost            ENABLED            HTTP  avgt    3   615.682 ±  58.484  us/op
SimpleBenchmarks.benchmarkBlockingLargePost            ENABLED           HTTPS  avgt    3  2614.834 ± 130.408  us/op

@fl4via I think I have a simpler solution using a custom HttpHandler to apply request and response conduit wrappers. This way we can measure time spent in awaitReadable/awaitWritable, and reset the timer when a read/write method returns non-zero.
This approach does not require any new UndertowOptions, nor modification of existing code. Unlike the async-aware implementations, this approach doesn't risk violating the undertow thread model because callers of awaitWritable must own the conduit.
The drawback is that we end up calling System.nanoTime() twice for each await* invocation, which is about double what we expect using this approach, but I think the simplicity is worth the trade-off. I can put together a PR with this proposal for comparison.

PR is up: https://github.com/undertow-io/undertow/pull/837

@carterkozak I'll be tagging 2.1.0.CR1 this week, soon I hope. Both PRs will be reviewed and I'll give you feedback as soon as I finish working on some new features for Undertow/WF

Thanks :-)

@carterkozak  I am approving the general idea proposed by this commit. However, I'll have to follow a process since it adds a new feature to Undertow. You can expect it to be released within a month, and probably we will tag a CR before that.

Thanks, I’m also happy to close this out in favor of the approach used in #837 which I expect to be easier to support, and matches the undertow handler wrapper architecture more cleanly.


Add support for AlmaLinux & Rocky

This change will move all module default values into OS specific hiera data. This is done because not all module default values are really default values for all OS flavours. Now we have one file per OS flavour which makes adding support for new OS flavours very easy.
The usage of os.family is migrated to use os.name. This allows more fine granular settings whenever needed.
The last commits will finally add support for AlmaLinux and Rocky.
Acceptance tests are only added for AlmaLinux and Rocky version 8. Puppet Litmus doesn't provide containers for newer versions yet. FacterDB doesn't provide facts for Rocky 9 yet, but it should work once it gets added to FacterDB.

Thank you @anders-larsson, released as v2.3.0


[GEARPUMP-262] Invoke setup/teardown methods of SingleInputFunction i…

…n all tasks
Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[x] Make sure the commit message is formatted like:
[GEARPUMP-<Jira issue #>] Meaningful description of pull request
[x] Make sure tests pass via sbt clean test.
[x] Make sure old documentation affected by the pull request has been updated and new documentation added for new functionality.

Current coverage is 70.54% (diff: 100%)

Merging #132 into master will increase coverage by 0.05%

@@             master       #132   diff @@
==========================================
  Files           191        191
  Lines          6016       6020     +4
  Methods        5496       5501     +5
  Messages          0          0
  Branches        518        517     -1
==========================================
+ Hits           4241       4247     +6
+ Misses         1775       1773     -2
  Partials          0          0

Powered by Codecov. Last update a23a40f...0d5e8a8

+1


Setup tests in react file based routes

test workflow for node 20
test workflow for node 21
test workflow for node 22
coveralls usage setup

example: https://github.com/idrinth-api-bench/issues/blob/277fc348e116ff08f5a6ca629217aeeb9a9924ee/.github/workflows/global.test.yml


[opentitanlib] Activate LED driver on HyperTeacup 1.5

I noticed that running //sw/device/examples/teacup_demos:teacup_leds_demo on a HyperTeacup rev 1.5.0 board doesn't work as-is (the board's LEDs don't light up at all). This PR fixes the issue, although I'm not sure whether this is the best solution and would appreciate feedback from folks more familiar with the board design and this part of the codebase.
Explanation:
The latest rev of the HyperTeacup board (v1.5.0) connects the active-low shutdown pin of the LED driver to HyperDebug pin CN10.15. The board pulls this pin low with a resistor, so the HyperDebug needs to actively drive it high to enable the LED driver. (See signal LED_DRV_SD_L on pages 5 and 7 of the schematic).
This commit updates opentitanlib's HyperTeacup configuration to drive this pin high by default, ensuring that the driver is always active.
CN10.15 is not connected on HyperTeacup rev 1.3.0, so this change should be backwards compatible with this rev of board. I'm not sure what other revs might be in the wild though, and whether this backwards compatibility holds for all of these revs.

@jesultra thanks for taking a look! This is ready to merge, but I don't have committer rights - would you mind pressing the button for me? Thanks!


Display "Start an evaluation" only when there are testsets

There is no sense to display "start an evaluation" when there are no testsets.

@aakrem I would like to take up this issue.

I want to take up this issue. @mmabrouk

Hi @okieLoki it's assined for you!

@aakrem please check  PR#515

Fixed in #515


feat: new command to cms sync

What's the purpose of this pull request?
Create a new command on Faststore CLI called cms-sync in order to prevent the user from entering the temporary folder so that the script can run.
How it works?
faststore cms-sync

How to test it?
running the script above
Starters Deploy Preview

References

Emerson, could you also make sure that on your way to completing this feature you'll update our documentation?

@gvc the doc https://faststore-site-qi2r15q2t-faststore.vercel.app/docs/component-customization/creating-a-new-section


Convert pkg/secrets to module

What does this PR do?
PR converts pkg/secrets to go module.  In the process we do the following

Convert pkg/util/winutil to a module
Move pkg/util/winutil/crashreport to pkg/util/crashreport - to simplify creating pkg/util/winutil module
Move pkg/util/winutil/pdhutil to pkg/util/pdhutil  - to simplify creating pkg/util/winutil module

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

create a windows vm
install IIS
powershell: Install-WindowsFeature -name Web-Server -IncludeManagementTools
install the agent
enable USM & NPM in system probe
ensure you see your host in NPM
after generating some http traffic, make sure you see defaultapppool in services for your host in USM

Reviewer's Checklist

[x] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[x] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[x] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[x] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

Heads up that secrets is being turned into a component by https://github.com/DataDog/datadog-agent/pull/20927. This will impact this PR by both moving the source out of pkg/secrets (into comp/core/secrets/secretsimpl) and also by slightly changing the method signatures of secrets. Aside from that, it should not be changing any functionality.

@hush-hush i have updated the PR title.
@hush-hush @dustmop I will update the PR to convert comp/core/secrets  to a module.

We were able to verify it is working as expected


Using a game engine

Hello! I saw your game on the Pygame website and it caught my eye - the concept is great and I'm impressed by your progress already! I've made my fair share of games in Pygame and I'm aware of how bare-bone it is - as you may know, it isn't really a game engine. Due to a lack of spare time I've moved on from dabbling in gamedev, but since, I've come to realise that using a "proper" game engine is almost critical if you want your game to scale - writing your own game engine (which you practically must do when using Pygame) is really not a good idea, unless you are really experienced.
I'm not trying to put you off, but in my humble opinion I think using a game engine will get your game a lot further, far quicker, especially when it comes tasks like making the terrain, etc. (your note about it being difficult and long is very accurate!). My personal choice would be Godot for many reasons, amongst others that it is open source, has a very active and productive community (https://www.reddit.com/r/godot/top/?t=all) and has most of the features of professional game engines. It's native scripting language is not Python, but I believe there are addons which allow you to script it using Python. There are also plugins, demos and tutorials for (procedural) terrain generation. Some examples of plugins are https://github.com/Zylann/godot_heightmap_plugin and https://github.com/SirRamEsq/SmartShape2D)
Whatever you decide (and I'm aware the decision will not be a simple one - one that even I would find difficult due to how much I enjoy working with Pygame despite the challenges), I wish you all the best with your project, and I can't wait to see how it turns out!

Hello and thank you for the suggestion on Godot engine, I will keep that into consideration for sure. I will definitely encounter performance issues when the game is getting bigger especially in ver 5.0. For now, I will see how far pygame can get me without having to cut too much corner.

Came across this - a Godot game with a very similar concept to this!
https://kaetjaatyy.github.io/2020/10/27/battles.html

Indeed, seem like we both have the same idea especially in term of unit structure. Too bad there is not video showcase yet but seem like the way battle progress is probably also in the same way. Definitely worth following.

If you want to continue using python, try arcade. I haven't personally used it, but I am thinking about using it instead of pygame for one of my games. Here's the link: https://arcade.academy/

Look interesting. Wonder how it fares against Pygame 2.0 now though. But the library does look somewhat simpler to use than Pygame. In any case, thanks for the suggestion never know about it before.


Trivial pytest fixes

Add zigpy dependency in requirements_test_all.txt.

Codecov Report

Merging #1521 (e9377a7) into dev (b848521) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #1521   +/-   ##
=======================================
  Coverage   80.31%   80.31%
=======================================
  Files         227      227
  Lines        6773     6773
=======================================
  Hits         5440     5440
  Misses       1333     1333

Continue to review full report at Codecov.

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

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 80.319%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5440

Relevant Lines:
6773

💛  - Coveralls


Fix normalization on data

otherwise it returns ndarray which could not be serialized by json

/assign @xuanyang-cn


ENH: Enhance @Index to support mulitple columns, unique flag and hence also be able to put on the class (annotation target TYPE)

A the moment @Index is limited and it might be useful to provide a simply mechanism for this. Need to review this question once the updated DB migration / DDL generation rolls out.

Note this is 'breaking api' in the sense that @Index had a value() attribute that has been renamed to name() ... and this being the index name.


Fix markdown lists on pagination component page

When there's no empty line before the list the Markdown renderer is treating it like it's part of the preceding paragraph.

Before
After

Thanks @peteryates – it looks like this was introduced by us updating to the latest version of Marked. v4.0.15 includes an intentional change which affects lists not preceded by whitespace.
I'll do a check to see if there are any other pages that need updating.

@36degrees thanks, I had a quick scan through and didn't spot any - wasn't an exhaustive search though.

Thanks for raising this @peteryates – it helped us to identify that something wasn't quite right.
We've now identified that quite a few pages are affected, so we're opting to roll the change back (#2288).

We're going to go ahead and merge this anyway, as it's generally good practise to have a blank line before a list in Markdown.


Feat:Add a cool stylish button in darkmode toggle button

Describe the feature
Now the button is not styled
Add ScreenShots
.
Record

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

@RamakrushnaBiswal which type of button you are talking about?

@RamakrushnaBiswal No, this icon looks a bit weird

solved #472


Use uinput on Linux

It may be desirable (especially with people moving to stuff like Wayland) to go through the kernel via uinput so that there is no runtime dependency on X11, making this work on systems that don't use X11.
libevdev has some wrappers for using uinput, but you can totally use it directly.

awesome thanks for this info! Ill work on implementing it.

oh, appearantly it doesn't work well with multiuser systems w/ multiple X11 sessions, and can be a bit of a keylogger risk?
more research would probably be a good thing.


Adding force removal to swarmctl

@aaronlehmann
Signed-off-by: Diogo Monica<EMAIL_ADDRESS>

LGTM


Azure GitHub deployment | Remove additional files at destination folder.

Hello Team,
When we use GitHub as a deployment option in Azure web app deployment then it first deploys content to the Repository folder and then deploys to WWWROOT which doesn’t delete the additional content available in the WWWROOT folder.
If I have any additional files under WWWROOT folder which is not the part of GitHub committed deployment then that file will not be deleted.
While in Visual Studio deployment we can “remove additional files at destination” through setting tab.
Below it the GitHub deployment cycle,
https://github.com/projectkudu/kudu/wiki/Deploying-from-GitHub
Is there any option available in GitHub which can delete the additional files available at the destination.
Thanks in advance!

Kudu will only delete the files which are part of previous deployment but removed from current one.  See https://github.com/projectkudu/KuduSync.    Is that you observed?   I don't think we have option currently to remove anything not in current deployment.   It seems like a good option, IMO.
If you are adventurous and have time, try improving KuduSync (introduce new knob) and send us PR.   We can try to integrate and light up end-to-end.

Any update on this? It's been 2.5 years.

Any update on this? It's been 2.5 years.

I made two deployments using the github action azure/webapps-deploy@v2:

First deployment added a src file: DEPLOYMENT_MARKER
The second deployment added a new src file DEPLOYMENT_MARKER_1, and removed the old DEPLOYMENT_MARKER file

After the second deployment I SSH into our app service files and check /home/sites/wwwroot and this is what I see:

The old files are still there. This means that if you have a bug in an old file that is causing your app to crash on startup and you try to solve the problem by removing the file. The problem will persist. This one issue is basically making Azure App Service unusable in any enterprise or professional capacity.
@suwatch will this issue ever be fixed?

Linux or Windows?   Assuming this is the test site - provide the sitename, utc times of both deployments.

This is a Linux App Service running Node 12. Is it okay if I email you the details? I am currently also in contact with microsoft support, they suggest setting this option in Kudu: SCM_ZIPDEPLOY_DONOT_PRESERVE_FILETIME=1 is there a way to use this option with this action?

I was able to solve my issue by adding this to my app service configuration: WEBSITE_RUN_FROM_PACKAGE="1" more info can be found on it here: https://docs.microsoft.com/en-us/azure/app-service/deploy-run-package#enable-running-from-package.
Note: this works for me because our app is stateless and does not require writing to files

Just dropping a note here that this is still an issue when using zip deploy.

Trying to wake this: still an issue. WEBSITE_RUN_FROM_PACKAGE doesn't help. In fact, my only resolution has been to set it to 0 so I can use SSH to rm -rf the whole wwwroot folder and then deploy over the top, which is crazy.

Hi
If the problem persists and is related to running it on Azure App Service, please open a support incident in Azure:
https://learn.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request
This way we can better track and assist you on this case
Thanks,
Joaquin Vano
Azure App Service

Hi
If the problem persists and is related to running it on Azure App Service, please open a support incident in Azure: https://learn.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request
This way we can better track and assist you on this case
Thanks,
Joaquin Vano Azure App Service

I am another who suffers from this bug/design error.
How can I track this issue over at learn.microsoft.com ?
Or are you saying that every individual that is waiting for this issue to be solved should raise individual support requests, instead of something collaborative such as this issue which perfectly explains the problem?

Yes, please file your individual support request. This will allow us to know more information about your case like your site id, etc. This is why it's important you file you support request.

Yes, please file your individual support request. This will allow us to know more information about your case like your site id, etc. This is why it's important you file you support request.

I fear there might be some miscommunication in this matter.
The issue is not specific to us. Anyone who uses github actions to deploy a web app as described earlier in this issue will be affected. So a specific site id etc is 100% irrelevant.
There is no way to deploy to a "fresh" filesystem. If your app's behavior is to write new files, these files are now stuck forever.
Unless there was an option to deploy with a setting to delete all previous files (no matter if a previous deployment are responsible for them or not).
Are you 100% we still should open support tickets for this even though it is irrelevant? I don't want to accidentally end wasting anybody's time by doing so.

The site id was just an example of information we might need. We will need some information for us to filter the logs and find the site, region, scale unit, time ranges, etc. so please file an azure support request if you need support.

The site id was just an example of information we might need. We will need some information for us to filter the logs and find the site, region, scale unit, time ranges, etc. so please file an azure support request if you need support.

I do not have permission to file an azure support request according to the azure portal.