.Net: Function calling: Add function author role and name property

Motivation and Context

Resolves #3017 and enables the full function calling scenario as described in the OpenAI function calling documentation.
Description

Adds the "function" author role to allow adding function messages to chat history.  Function messages require a "name" parameter, so this change also adds that to chat message model.
Contribution Checklist

[x] The code builds clean without any errors or warnings
[x] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[x] All unit tests pass, and I have added new tests where possible
[x] I didn't break anyone :smile:

I found that your version has an issue. I have a CheckStock function and my prompt is "How many tables and chairs are there in stock?". Note that LLM should issue 2 function calling messages (I have added a do-while loop in Example59). However, with your version, LLM only called CheckStock once. In my original working version using native OpenAI, the Messages in the ChatCompletionsOptions keeps both the function calling assistance message and the function return user message. In your solution, only function return user
messages are kept in the abstracted ChatMessageBase. Hence, in ClientBase.cs, when CreateChatCompletionsOptions populate ChatCompletionsOptions using the abstracted ChatMessageBase, there are no function calling assistance message histories passing back to the LLM. Hence, LLM failed to relate the function result message with the original function calling message.

@gitri-ms As mentioned above, we need to add LLM function call request to the chat history before adding the function execution result. Please consider my suggestions below:

Connectors.AI.OpenAI\AzureSdk\ClientBase.cs

foreach (var message in chatHistory)
{
    var azureMessage = new ChatMessage(GetValidChatRole(message.Role), message.Content);
    string functionName = "";
    if (message.AdditionalContext?.TryGetValue("Name", out string? name) is true)
    {
        azureMessage.Name = name;
        functionName = name;
    }

    if (message.AdditionalContext?.TryGetValue("Arguments", out string? arguments) is true)
    {
        azureMessage.FunctionCall = new FunctionCall(functionName, arguments);
    }

    options.Messages.Add(azureMessage);
}

Connectors\Connectors.AI.OpenAI\ChatCompletion\ChatHistoryExtensions.cs

public static void AddAssistantMessage(this ChatHistory chatHistory, IChatResult chatResult)
{
    FunctionCall fc = chatResult.ModelResult.GetOpenAIChatResult().Choice.Message.FunctionCall;
    chatHistory.AddMessage(AuthorRole.Assistant, "", new Dictionary<string, string> {
        { "Name", fc.Name },
        { "Arguments", fc.Arguments }
    });
}

Example59_OpenAIFunctionCalling.cs

// Check for message response
var chatMessage = await chatResult.GetChatMessageAsync();
if (!string.IsNullOrEmpty(chatMessage.Content))
{
    Console.WriteLine(chatMessage.Content);

    // Add the response to chat history
    chatHistory.AddAssistantMessage(chatMessage.Content);
}
// Check for function response
OpenAIFunctionResponse? functionResponse = chatResult.GetFunctionResponse();
if (functionResponse is not null)
{
    chatHistory.AddAssistantMessage(chatResult);

@leungkimming - Thanks for the suggestions, I am incorporating them and will have updated changes soon!

@gitri-ms I am sorry that there is a minor bug in my suggested coding. In the original OpenAI function calling assistance message, the Name field should be null but in my coding, the Name field contains the function name. Only the function execution result message should have a value in the Name field. Below is the correction in the ClientBase.cs file.
foreach (var message in chatHistory)
{
    var azureMessage = new ChatMessage(new ChatRole(message.Role.Label), message.Content);
    string? functionName = null;

    if (message.AdditionalProperties?.TryGetValue("Name", out string? name) is true)
    {
        functionName = name;
    }

    if (message.AdditionalProperties?.TryGetValue("Arguments", out string? arguments) is true)
    {
        azureMessage.FunctionCall = new FunctionCall(functionName, arguments);
    } else
    {
        azureMessage.Name = functionName;
    }

    options.Messages.Add(azureMessage);
}

In addition, to demonstrate multiple function calling, I would like to suggest below changes to the Example59_OpenAIFunctionCalling.cs file.

Make the second prompt to call both the TimePlugin-Date and KlarnaShoppingPlugin
Add a do loop until CompletionsFinishReason.Stopped
When passing function execution result, change to FunctionCallAuto instead of FunctionCallNone.

// Set FunctionCall to the name of a specific function to force the model to use that function.
requestSettings.FunctionCall = "TimePlugin-Date";
await CompleteChatWithFunctionsAsync("What day is today?", chatHistory, chatCompletion, kernel, requestSettings);

// Set FunctionCall to auto to let the model choose the best function to use.
requestSettings.FunctionCall = OpenAIRequestSettings.FunctionCallAuto;
await CompleteChatWithFunctionsAsync("What day is today and What computer tablets are available for under $200?", chatHistory, chatCompletion, kernel, requestSettings);

...

private static async Task CompleteChatWithFunctionsAsync(string ask, ChatHistory chatHistory, IChatCompletion chatCompletion, IKernel kernel, OpenAIRequestSettings requestSettings)
{
    Console.WriteLine($"User message: {ask}");
    chatHistory.AddUserMessage(ask);
    IChatResult chatResult;

    // Send request and add response to chat history
    chatResult = (await chatCompletion.GetChatCompletionsAsync(chatHistory, requestSettings))[0];
    chatHistory.AddAssistantMessage(chatResult);

    await PrintChatResultAsync(chatResult);
    do
    {
        // Check for function response
        OpenAIFunctionResponse? functionResponse = chatResult.GetFunctionResponse();
        if (functionResponse is not null)
        {
            // If the function returned by OpenAI is an SKFunction registered with the kernel,
            // you can invoke it using the following code.
            if (kernel.Functions.TryGetFunctionAndContext(functionResponse, out ISKFunction? func, out ContextVariables? context))
            {
                var result = (await kernel.RunAsync(func, context)).GetValue<object>();

                string? resultContent = null;
                if (result is RestApiOperationResponse apiResponse)
                {
                    resultContent = apiResponse.Content?.ToString();
                }
                else if (result is string str)
                {
                    resultContent = str;
                }

                if (!string.IsNullOrEmpty(resultContent))
                {
                    // Add the function result to chat history
                    chatHistory.AddFunctionMessage(resultContent, functionResponse.FullyQualifiedName);

                    //Get another completion
                    requestSettings.FunctionCall = OpenAIRequestSettings.FunctionCallAuto;
                    chatResult = (await chatCompletion.GetChatCompletionsAsync(chatHistory, requestSettings))[0];
                    chatHistory.AddAssistantMessage(chatResult);

                    await PrintChatResultAsync(chatResult);
                }
            }
            else
            {
                Console.WriteLine($"Error: Function {functionResponse.PluginName}.{functionResponse.FunctionName} not found.");
            }
        }
    } while (chatResult.ModelResult.GetOpenAIChatResult().Choice.FinishReason != CompletionsFinishReason.Stopped);
}

Created a new PR here: https://github.com/microsoft/semantic-kernel/pull/3421

@leungkimming I have created a new PR here: https://github.com/microsoft/semantic-kernel/pull/3421 to continue this work. Could you try out the latest changes (just minor updates) and let me know if it works for your scenarios. Thanks

@leungkimming I have created a new PR here: #3421 to continue this work. Could you try out the latest changes (just minor updates) and let me know if it works for your scenarios. Thanks

@ markwallace-microsoft Thankyou. It works for my scenario. However, I have a minor suggestion. Using chatResult.GetOpenAIFunctionResponse to check if not null to execute functions may have problem if OpenAI set the CompletionsFinishReason other than "stop" or "function_call". Shall we add a "GetOpenAIFinishReason" in  ChatResultExtensions.cs and let us properly check below values to determine the next action to take?
private const string StoppedValue = "stop";
private const string TokenLimitReachedValue = "length";
private const string ContentFilteredValue = "content_filter";
private const string FunctionCallValue = "function_call";
I am using chatResult.ModelResult.GetOpenAIChatResult().Choice.FinishReason but not quite elegant!


Update Stock Artisan Command

Adds a new Artisan command that allows a user to update the stock value of a product.
Closes #12.

Any boundary checks required? Eg. An empty stock slot looks like it could be decremented.


Animate view to multiple nodes

Modify existing animation method to support passing multiple nodes, in which case the view is fit to the bounding box around them.
This is in preparation for this feature

Can you split this function into 2 parts:

AnimateToBound: Which takes a bounding box and animate the view to that bounding box
Convert LGraphGroup.resizeTo as a helper function and call it to get the bounding box that contains multiple canvas objects (LGraphNode / LGraphGroup)

@bezo97 I sent the collaborator invite to you. Please confirm. Thanks!


Food Fusion

Final Pull Request that has all the changes in the dev branch
Summary
We have introduced following changes in this Pull Request:

Login page and registration page built with devise.
Food List that shows a list of food added by the logged-in user and leads to a form to add new food.
Recipe List that shows a list of recipes added by the logged-in user, leads to a form to add new recipes, and also leads to recipe details page.
Public Recipe list that display a list of all public recipes ordered by newest as in the wireframe and leads to recipe details page.
Recipe details page that display details of a given recipe and also leads to a form to add new ingredients to it. this page also has a popup that compares foods in Inventory with the ingredients of recipe and then generate a missing food list.
Inventory List that displays a list of inventories added by the user and leads to a form to add new Inventory. it also leads to a inventory details page.

Hello team👋,
This is an additional review as a result of the second opinion request.
Your app works as expected, great job 👏.
The user is directed to the appropriate page when they click on the recipe, so I can see that the first change that was requested is incorrect.
The second requested change is not one of the project requirements, it can be optional.
As these were the only reasons that your project was not approved it will be approved as soon as possible.
Please ask for another review in your Dashboard and the next Code Reviewer will mark it as approve right away :)
Keep rocking 🚀!


[5.1-04-24-2019][CodeCompletion] Complete Swift only module name after 'import'

(Cherry-pick of #24455 into swift-5.1-branch-04-24-2019 reviewed by @brentdax and @benlangmuir)
rdar://problem/39392446

@swift-ci Please test

Build failed
Swift Test Linux Platform
Git Sha - f1da9931a747aedd945e7f4c2257b1b1d3d9e680

Linux failure is unrelated (swift-corelibs-foundation)


Add rel="noopener noreferrer" to target="_blank"

rel="noopener noreferrer" should be added to links containing target="_blank" as a precaution against reverse tabnabbing. For more information, please refer to the following article:
https://www.jitbit.com/alexblog/256-targetblank---the-most-underestimated-vulnerability-ever/

+1 for this.

@auralon I think this might be as simple as adding this in to the attributes like so;
{% set attributes = {     class: 'arrow // arrow-after // arrow-sm // whitespace-no-wrap',     rel: "noopener noreferrer" } %}
{{ block.linkTo.htmlLink(attributes) }}


Feature(Analayzer): Fix type infer

Description:
{
    get x() {
      return "boolean" as const;
    },
    y: "string",
  }

correct infer type
Related issue (if exists):
#1011

regression cauused by this
// @strict: true
// @declaration: true

interface Part {
  id: number;
  name: string;
  subparts: Part[];
  updatePart(newName: string): void;
}
type NonFunctionPropertyNames<T> = {
  [K in keyof T]: T[K] extends Function ? never : K;
}[keyof T];

type DeepReadonly<T> = T extends any[]
  ? DeepReadonlyArray<T[number]>
  : T extends object
  ? DeepReadonlyObject<T>
  : T;

interface DeepReadonlyArray<T> extends ReadonlyArray<DeepReadonly<T>> {}

type DeepReadonlyObject<T> = {
  readonly [P in NonFunctionPropertyNames<T>]: DeepReadonly<T[P]>;
};

function f10(part: DeepReadonly<Part>) {
  let a = part;
  let b = {} as Part;
  let c = {} as DeepReadonly<Part>;
  let name: string = part.name;
  let id: number = part.subparts[0].id;
  part.id = part.id; // Error
  part.subparts[0].id = part.subparts[0].id; // Error
  part.updatePart("hello"); // Error
}

part.name infer any
I don't know why, but the change changes the reasoning.
Why?

The reason is that DeepReadonlyArray was not found in find_type.
It's not relevant for that PR.
  DEBUG  Analyzer.find_type('DeepReadonlyArray#2')
    at crates/stc_ts_file_analyzer/src/analyzer/scope/mod.rs:1920
    in Scope::find_type with name: DeepReadonlyArray#2
    in Scope::find_type with name: DeepReadonlyArray#2
    in Scope::find_type with name: DeepReadonlyArray#2
    in find_local_type with name: DeepReadonlyArray#2
    in find_type with name: DeepReadonlyArray#2
    in Expander::expand_ref
    in Expander::expand_type
    in expand
    in normalize with ty: DeepReadonlyArray<Part[][number]>;
    in normalize with ty: DeepReadonlyArray<Part[][number]>;
    in normalize with ty: Part#2[] extends any[] ? DeepReadonlyArray<Part[][number]> : Part[] extends object ? DeepReadonlyObject<Part[]> : Part[];
    in normalize with ty: Part#2[] extends any[] ? DeepReadonlyArray<Part[][number]> : Part[] extends object ? DeepReadonlyObject<Part[]> : Part[];
    in exclude_types_using_fact
    in Stmt with line_col: "(28:3-28:33)"
    in visit_stmts_for_return
    in Stmt with line_col: "(26:1-31:2)"

  DEBUG  Scope.find_type: failed to find type 'DeepReadonlyArray#2'
    at crates/stc_ts_file_analyzer/src/analyzer/scope/mod.rs:1254
    in find_local_type with name: DeepReadonlyArray#2
    in find_type with name: DeepReadonlyArray#2
    in Expander::expand_ref
    in Expander::expand_type
    in expand
    in normalize with ty: DeepReadonlyArray<Part[][number]>;
    in normalize with ty: DeepReadonlyArray<Part[][number]>;
    in normalize with ty: Part#2[] extends any[] ? DeepReadonlyArray<Part[][number]> : Part[] extends object ? DeepReadonlyObject<Part[]> : Part[];
    in normalize with ty: Part#2[] extends any[] ? DeepReadonlyArray<Part[][number]> : Part[] extends object ? DeepReadonlyObject<Part[]> : Part[];
    in exclude_types_using_fact
    in Stmt with line_col: "(28:3-28:33)"
    in visit_stmts_for_return
    in Stmt with line_col: "(26:1-31:2)"

[crates/stc_ts_file_analyzer/src/analyzer/scope/mod.rs:1167] &name = DeepReadonlyArray#2
[crates/stc_ts_file_analyzer/src/analyzer/scope/mod.rs:1183] 123 = 123
[crates/stc_ts_file_analyzer/src/analyzer/scope/mod.rs:1188] 123 = 123
  ERROR  (2) Failed to find type: DeepReadonlyArray#2
    at crates/stc_ts_file_analyzer/src/analyzer/scope/mod.rs:2262
    in Expander::expand_ref
    in Expander::expand_type
    in expand
    in normalize with ty: DeepReadonlyArray<Part[][number]>;
    in normalize with ty: DeepReadonlyArray<Part[][number]>;
    in normalize with ty: Part#2[] extends any[] ? DeepReadonlyArray<Part[][number]> : Part[] extends object ? DeepReadonlyObject<Part[]> : Part[];
    in normalize with ty: Part#2[] extends any[] ? DeepReadonlyArray<Part[][number]> : Part[] extends object ? DeepReadonlyObject<Part[]> : Part[];
    in exclude_types_using_fact
    in Stmt with line_col: "(28:3-28:33)"
    in visit_stmts_for_return
    in Stmt with line_col: "(26:1-31:2)"

[crates/stc_ts_file_analyzer/src/analyzer/scope/mod.rs:2314] &type_name = Ident(
    RIdent {
        node_id: NodeId(
            58,
        ),
        span: Span {
            lo: BytePos(
                293,
            ),
            hi: BytePos(
                310,
            ),
            ctxt: #2,
        },
        sym: Atom('DeepReadonlyArray' type=dynamic),
        optional: false,
    },
)
[crates/stc_ts_file_analyzer/src/analyzer/scope/mod.rs:2314] &ty = Some(
    Keyword(
        any,
    ),
)


Extending existing Data Types

Laravel Version: 5.4.30
Voyager Version: 1.0.17
PHP Version: 5.6
Database Driver & Version: MySQL

Description:
The existing data types (Posts/Pages/Users) used in Voyager are perfectly fine for many applications. However, some applications may require only adding a few fields on the tables for the data types. What is the best way to extend the existing data type?

I don't know if it's the best way but, create your model:
<?php

namespace App\Models;

class AlmostPost extends \TCG\Voyager\Models\Post
{
    //change what you want
}

Then you need to tell voyager to use your model in your AppServiceProvider boot method:
Voyager::useModel('AlmostPost', \App\Models\AlmostPost::class);

and in bread configuration.
I'm not sure if both changes are always needed but I do it like that.

In addition to what @MrCrayon suggested, I'd just like to add that Voyager doesn't ship with a User model.  Instead, we use the default one from the Laravel Auth scaffolding (from artisan make:auth), and just modify it with a contract and trait.  You're free to add whatever you need to that model since it belongs to your app.

Thanks for your input on this. Also, to update the actual database of existing datatypes would it be better through migrations or the database system in Voyager. I generally prefer migrations as they can be version control. Would this create any sort of problems in Bread system?


Since upgrade to 1.93.1 no Source control anymore, only Source Control Graph

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

VS Code Version: 1.93.1
OS Version: Windows 11

Steps to Reproduce:

Open c++ project with submodules
Open Source control tab

I only see a Source Control header, but with Source Control Graph contents
On every repo I only see the git graph with commits, I do not see the Source Control tab contents with modified and staged files

Never mind .. just found out that there are 2 possible tabs in the left menu for Source Control.... I managed to restore the original one through the menu View - Source control
Really confusing ...

Right-click on each of them. One should offer a Reset Location option. Use it.

no such option

Not on either of the two identical SCM icons in the Activity Bar?

No, they are identical

OK, looks like you hid one. Bring up the menu in your most recent screenshot and set a checkbox on the second Source Control entry. Then use the context menu of the duplicate icon when it appears.

Ahh found it in context menu of the headers

Still very confusing, to have duplicate names of views ....

Explained at https://github.com/microsoft/vscode/issues/227708#issuecomment-2332459571


Icons don't display when using  dot syntax

How to reproduce:
Run this to set up dot syntax
php artisan vendor:publish --provider="Mkocansey\Bladewind\BladewindServiceProvider" --tag=bladewind-components --force
Add a button to the code using the dot syntax and include an icon
<x-bladewind.button
radius="full"
outline="true"
color="blue"
name="btn-edit"
icon="pencil-square"
icon_right="false"
>
Edit
</x-bladewind.button>
This will cause the button text as well as 'component.icon' to be printed as the button text.
Example:
Edit component.icon

Hello @brianmcaudill I just followed your steps above to reproduce the error but I get the button displayed as it should.
<x-bladewind.button
      radius="full"
      outline="true"
      color="blue"
      name="btn-edit"
      icon="pencil-square">Edit</x-bladewind.button>

Below is what I get. Are you using v2.5.13?

Also, can you check/confirm if you have icon.blade.php in your resources > views > components > bladewind directory.

@brianmcaudill is this working for you now?

Yes it is working now. Thank you.


[dashboard] clean up usage for old filter immune metadata

CATEGORY
Choose one

[x] Bug Fix
[x] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY
After migration/convert old filter immune attribute in #9109 and #9145, dashboard front-end doesn't have to handle backward compatibility issue with old metadata. This PR is to clean up the usage for filter_immune_slices and filter_immune_slice_fields everywhere.
TEST PLAN
CI and manual tests.
ADDITIONAL INFORMATION

[x] Has associated issue: #9109 #9145

REVIEWERS
@john-bodley @serenajiang @etr2460 @mistercrunch

Codecov Report

Merging #9146 into master will increase coverage by 0.03%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master   #9146      +/-   ##
=========================================
+ Coverage   59.06%   59.1%   +0.03%
=========================================
  Files         372     372
  Lines       11922   11915       -7
  Branches     2919    2917       -2
=========================================
  Hits         7042    7042
+ Misses       4698    4693       -5
+ Partials      182     180       -2

Impacted Files
Coverage Δ

...frontend/src/dashboard/reducers/getInitialState.js
0% <ø> (ø)
:arrow_up:

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 f95a867...a7d0070. Read the comment docs.


Report bug in supported in-market versions

You can't open Support Request for Dynamics NAV/Business Central on PartnerSource anymore. This went to support.microsoft.com. Since PartnerSource will discontiuned, these are the links to create a new Support Case for Microsoft Dynamics 365 Business Central on-premises and for Microsoft Dynamics NAV.

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

ID: b9c875d8-3401-cfe3-d7d8-aac436b2b53b
Version Independent ID: 4459ce33-7e99-adf3-591a-a665e3db0067
Content: Resources for Help and Support - Business Central
Content Source: dev-itpro/help-and-support.md
Service: dynamics365-business-central
GitHub Login: @edupont04
Microsoft Alias: edupont

Thank you for your feedback! We added a new entry in the table for Business Central on-premises with the link to support.microsoft.com site.


Update core.rb

Is better to use alias method instead of defining delegator methods.

:+1:

@EhsanYousefi Merged! Thank you!!


ISPN-11226 Set the cache manager in the task context for distributed …

https://issues.redhat.com/browse/ISPN-11226
This has the side-effect of being able to run distributed tasks in the server. To simplify things, only string parameters are supported for know. This limitation will be removed once we have registration of protobuf serialization contexts in the server.

@tristantarrant Checkstyle failures on CI.

Fixed now, @ryanemerson

Thanks @tristantarrant


Changed @vitejs/app to vite

@vitejs/app is now deprecated and vite is used now

@max-programming Thanks!


Many2many Append with nested Struct

so i want to append in many2many, the association is in a nested struct in this case Target
and i want to append AgeLevel to Pub .
What version of Go are you using (go version)?
go 1.8.7 linux/amd64
Which database and its version are you using?
postgresql 9.6.8
Please provide a complete runnable program to reproduce your issue. IMPORTANT
package main

import (
	"github.com/jinzhu/gorm"
	_ "github.com/jinzhu/gorm/dialects/postgres"
)

type Pub struct {
	gorm.Model
	VideoName string `gorm:"unique;not null"`
	Target
}

//the embedded struct
type Target struct {
	Ages []AgeLevel `gorm:"many2many:targets_agelevels" `
}
type AgeLevel struct {
	Min     uint   `gorm:"not null" `
	Max     uint   `gorm:"not null" `
	Name    string `gorm:"not null;"`
	Targets []Pub  `gorm:"many2many:targets_agelevels"`
	gorm.Model
}

//set up age table
func SetUp(Db *gorm.DB) {
	ages := []AgeLevel{
		AgeLevel{
			Min:  0,
			Max:  3,
			Name: "Infancy",
		},
		AgeLevel{
			Min:  4,
			Max:  9,
			Name: "early childhood",
		},
	}

	for _, level := range ages {
		err := Db.Save(&level).Error
		if err != nil {
			panic(err)
		}
	}
}

func main() {

	var err error
	//open connexion chose yours
	Db, err := gorm.Open("postgres", "host=localhost port=5432 sslmode=disable")
	if err != nil {
		panic(err)
	}
	//uncomment below if you want to drop tables
	// Db.DropTable(
	// 	&Pub{},
	// 	&AgeLevel{},
	// 	"targets_agelevels",
	// )
	Db.AutoMigrate(
		&Pub{},
		&AgeLevel{},
	)
	SetUp(Db)
	//create pub
	this := Pub{
		VideoName: "test",
	}
	//add age_level id to pub in ages
	this.Ages = []AgeLevel{
		AgeLevel{
			Model: gorm.Model{
				ID: 1,
			},
		},
	}

	if err := Db.Omit("Ages").Create(&this).Error; err != nil {

		panic(err)
	}

	if err := Db.Model(this).Association("Ages").Append(this.Ages).Error; err != nil {

		panic(err)
	}
}

the error is invalid association Ages for main.Pub
but if i put gorm.Model in Target struct it work but it's not that practical

Same here it looks like the primary get need to be in same struct but not embed to make association work :(


🐝  Card menu fixes

Closes: https://github.com/TryGhost/Ghost/issues/8106

Fixes the keyboard selection problems of the '+' menu.
Makes the active state of the keyboard selection blank until a key is pressed.
Fixes it so that the '+' menu executes the tool on Enter.

Coverage remained the same at 75.301% when pulling d533738bf817e9244b0ddcf2b89bb6c8c2438cc4 on disordinary:fixed-koenig-menus into e87762c7236078778aceb48ed27a7be98df3c700 on TryGhost:master.

Changes Unknown when pulling 4ddbe1aa3203f2c57d66fd4b21901224b63a924c on disordinary:fixed-koenig-menus into ** on TryGhost:master**.


feat: allowing users to set the bounding box color and flag to toggle displaying confidence scores on bbox

Allowing users to set the bounding box color
For example:

nodes:
- input.visual:
    source: 0
- model.yolox
- draw.bbox:
    show_labels: True
    color_choice: [229, 255, 0] # [ B, G, R ] Turquoise
- output.screen

Added a flag to toggle displaying confidence scores on bbox
Usage - in the yml run file:

nodes:
- input.visual:
    source: 0
- model.yolox
- draw.bbox:
    show_labels: True
    show_scores: True
- output.screen

Perhaps we should consider "Squash and merger" for this PR since there are multiple commits with the same message but different changes. Also because this looks like a collaborative PR so not sure if it will be easy to rebase.

Perhaps we should consider "Squash and merger" for this PR since there are multiple commits with the same message but different changes. Also because this looks like a collaborative PR so not sure if it will be easy to rebase.

Good idea to use squash and merge to clean up this PR. The commit messages are currently messy with duplicates.

Perhaps we should consider "Squash and merger" for this PR since there are multiple commits with the same message but different changes. Also because this looks like a collaborative PR so not sure if it will be easy to rebase.

Good idea to use squash and merge to clean up this PR. The commit messages are currently messy with duplicates.

We tried to squash but it's a bit complicated since we have commits in between each other. Will need to do git reset back and forth for several rounds. I decided to just squash the most recent commits before creating the PR. Learnt how to perform the squash and will take note of this for future commits.


ndt_scan_matcher sometimes crashes when performing dynamic_map_loading

Checklist

[X] I've read the contribution guidelines.
[X] I've searched other issues and no duplicate issues were found.
[X] I'm convinced that this is not my fault but a bug.

Description
The ndt_scan_matcher occasionally crashes when performing dynamic_map_loading. This seems to be related to memory corruption caused by a data race, leading to intermittent and varied error messages.

Since this is caused by memory corruption due to a data race, it may fall as a different error.
Expected behavior
The ndt_scan_matcher should operate without crashing.
Actual behavior
The ndt_scan_matcher crashes with low probability.
Steps to reproduce
This problem is more likely to occur when dynamic map loading and pose estimation by NDT are performed at the same time.
Therefore, the config example to reproduce is

Prepare a divided map https://github.com/MapIV/pointcloud_divider
use_dynamic_map_loading: set to true (default) https://github.com/autowarefoundation/autoware_launch/blob/805756256ce392c1a7ef300251acae85a7f4d76a/autoware_launch/config/localization/ndt_scan_matcher.param.yaml#L4
Change parameters to increase NDT load

trans_epsilon: 0.01→0.00001 (to prevent NDT from terminating the loop by trans criteria) https://github.com/autowarefoundation/autoware_launch/blob/805756256ce392c1a7ef300251acae85a7f4d76a/autoware_launch/config/localization/ndt_scan_matcher.param.yaml#L20
max_iterations: 30→300 (to make the NDT loop run a lot to make it easier to run concurrently with dynamic map loading) https://github.com/autowarefoundation/autoware_launch/blob/805756256ce392c1a7ef300251acae85a7f4d76a/autoware_launch/config/localization/ndt_scan_matcher.param.yaml#L29

This will make it easier to reproduce.
Versions

OS: Ubuntu 22.04
ROS2: humble
Autoware: before https://github.com/autowarefoundation/autoware.universe/pull/5951

Possible causes
See https://github.com/autowarefoundation/autoware.universe/pull/5951
Additional context
This issue has been fixed in the pull request below.
https://github.com/autowarefoundation/autoware.universe/pull/5951
However, there is a problem that the mutex locking time is long (about 20msec to 40msec) when loading a dynamic map, so we plan to improve it in the future.

This issue was also resolved in terms of locking time through the following pull requests.

https://github.com/autowarefoundation/autoware.universe/pull/6394
https://github.com/autowarefoundation/autoware.universe/pull/6464
https://github.com/autowarefoundation/autoware.universe/pull/6480


Individual User login limit

Please advice the status for Cached login attempts for individual user.
What I understand from the article is that a user can use his cached credentials to login as long as Domain Controller isn't available even for infinite count!

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

ID: ef720207-c5e6-182e-8a34-361244c45641
Version Independent ID: 4591df7c-e474-9b85-7c09-ce572a66a0a3
Content: Interactive logon Number of previous logons to cache (in case domain controller is not available) (Windows 10) - Windows security
Content Source: windows/security/threat-protection/security-policy-settings/interactive-logon-number-of-previous-logons-to-cache-in-case-domain-controller-is-not-available.md
Product: w10
Technology: windows
GitHub Login: @Dansimp
Microsoft Alias: dansimp

@ahmadgeo It seems that a possible value of from 0 to 50 has already been included in the article under the following section,
https://docs.microsoft.com/en-us/windows/security/threat-protection/security-policy-settings/interactive-logon-number-of-previous-logons-to-cache-in-case-domain-controller-is-not-available#possible-values
Please let me know whether this clarifies your question.


Added poe tcp streaming demos

TODO: add readmes

Modify such that PoE device is the host and other relevant changes

@themarpe Both options (OAK host or client) are in the folder. What relevant changes?

Hi, my name is Ryan.
i have still not clear how to modify those files. So i have questions.
First: Which one is server? OAK-PoE?
Second: How to put the code on OAK-PoE?
If i am on the wrong way please correct me.
Best regards,

Hello @reallyedutech ,

Files can be found here
As you can see on the link above, there are 2 folders; one where OAK is host and one where OAK is the client
When you run oak.py, the pipeline will get uploaded to the (connected) OAK camera. Simultaneously you have to run the host.py script, so the TCP communication starts.


Not compatible with Anuglar 9.1 and Ivy

When upgrading from Angular 8 to 9.1 the compiler give that error.
This likely means that the library (ngx-store) which declares WebStorageModule has not been processed correctly by ngcc, or is not compatible with Angular Ivy. Check if a newer version of the library is available, and update if so. Also consider checking with the library's authors to see if the library is expected to be compatible with Ivy.
Any idea?

@patrickrioux any update on this. I am facing the same issue
node_modules/ng2-password-strength-bar/lib/passwordStrengthBar.module.d.ts:1:22 - error NG6002: Appears in the NgModule.imports of AppModule, but could not be resolved to an NgModule class.
This likely means that the library (ng2-password-strength-bar) which declares PasswordStrengthBarModule has not been processed correctly by ngcc, or is not compatible with Angular Ivy. Check if a newer version of the library is available, and update if so. Also consider checking with the library's authors to see if the library is expected to be compatible with Ivy.
I have updated my application to Angular 9. And this is what I am getting while building the application.
Angular CLI: 9.1.1
Node: 12.16.2
OS: linux x64
Angular: 9.1.1
... animations, cli, common, compiler, compiler-cli, core, forms
... language-service, localize, platform-browser
... platform-browser-dynamic, platform-server, router
Ivy Workspace: Yes
Package Version
@angular-devkit/architect 0.901.1
@angular-devkit/build-angular 0.901.1
@angular-devkit/build-optimizer 0.901.1
@angular-devkit/build-webpack 0.901.1
@angular-devkit/core 9.1.1
@angular-devkit/schematics 9.1.1
@angular/cdk 9.2.0
@angular/material 9.2.0
@angular/material-moment-adapter 8.2.3
@ngtools/webpack 9.1.1
@schematics/angular 9.1.1
@schematics/update 0.901.1
rxjs 6.5.5
typescript 3.8.3
webpack 4.42.1
I was able to fix the same error for primeng by just updating its version to latest.
Looking for a reply soon...

@patrickrioux thank you for sharing

@patrickrioux , I see this component is a jQuery component. Is it working fine with Angular 9?

https://github.com/antoantonyk/password-strength-meter
Patrick Rioux
Senior Software Developer
Ready Computing

http://www.readycomputing.com/
https://www.linkedin.com/company/ready-computing https://twitter.com/ready_computing?lang=en https://www.youtube.com/channel/UCtA7SflMXNTkY0MWL-79LDQ
Office:<PHONE_NUMBER>, Ext. 4006
Mobile:<PHONE_NUMBER>
<EMAIL_ADDRESS><EMAIL_ADDRESS>http://www.readycomputing.com/ www.readycomputing.com
150 Beekman Street, Floor 3, New York, NY, 10038
The information in this e-mail communication together with any attachments is intended only for the person or entity to which it is addressed and may contain confidential and/or privileged material. If you are not the intended recipient of this communication, please notify us immediately. Any views expressed in this communication are those of the sender, unless otherwise specifically stated. Ready Computing does not represent, warrant or guarantee that the integrity of this communication has been maintained or the communication is free of errors, virus or interference.
From: Fabricio Carvalho Ferreira<EMAIL_ADDRESS>Sent: April 28, 2020 01:43 AM
To: rnadler/ng2-password-strength-bar<EMAIL_ADDRESS>Cc: patrickrioux<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [rnadler/ng2-password-strength-bar] Not compatible with Anuglar 9.1 and Ivy (#26)
@patrickrioux https://github.com/patrickrioux  , I see this component is a jQuery component. Is it working fine with Angular 9?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/rnadler/ng2-password-strength-bar/issues/26#issuecomment-620393090 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ABCQBEAWML6KKN4QQLUVRKTROZUFJANCNFSM4LWXC6RQ .  https://github.com/notifications/beacon/ABCQBECSWVQPOG65WR4KWR3ROZUFJA5CNFSM4LWXC6R2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOET5HFAQ.gif
--
This is not a secure transmission. The information contained in this
transmission is highly prohibited from containing privileged and
confidential information, including patient information protected by
federal and state privacy laws. It is intended only for the use of the
person(s) named above. If you are not the intended recipient, you are
hereby notified that any review, dissemination, distribution, or
duplication of this communication is strictly prohibited. If you are not
the intended recipient, please contact the sender by reply email and
destroy all copies of the original message.

It appears to be impossible to create a single library that supports both pre-A9 and A9 at the same time. I just punted and created https://www.npmjs.com/package/ng9-password-strength-bar. Updating the ng2 component to ng9 is a couple of simple name changes.
If you have a chance to try it out, let me know if you run into any problems.
Thanks!

It appears to be impossible to create a single library that supports both pre-A9 and A9 at the same time. I just punted and created https://www.npmjs.com/package/ng9-password-strength-bar. Updating the ng2 component to ng9 is a couple of simple name changes.
If you have a chance to try it out, let me know if you run into any problems.
Thanks!

What other libraries normally do is that they made some breaking changes.
So for ng2-password-strength-bar (which should probably be named "ng-password-strength-bar"), maybe we can designate version 2 to support Angular 9+ & version 1 to support pre-Angular 9.
P. S. Sorry, I had been busy to go around making the PR for Angular 9 support.

I thought about ending the ng2 line, but it still gets ~2000 downloads/week and even if there's only a remote possibility of making changes to it, I didn't want to close that door (yet).


Fixes to work on modern rubies

Relaxed version constraints where too restrictive, added constraints where missing (based on versions available at the time of the last release).
Tested on macOS Sierra with Ruby 2.3.1

Thanks! I will merge this to follow up with another modernization PR. The weekly security alert gets old very fast 😅


Control submit button visibility using a fragment argument

Is your feature request related to a problem? Please describe.
At the moment we control the visibility of the submit button using styling. But perhaps we should control it a bit more explicitly using a fragment arguement. I worry that we're using a style attribute to control a very important functional behavior of the sdc library.
Describe the solution you'd like
Create a new fragment argument
Describe alternatives you've considered

current approach

Additional context
NA
Would you like to work on the issue?
maybe

@santosh-pingle fyi

@williamito fyi

can i get assigned?

#1872

@santosh-pingle you might still need to remove the style declarations in the style files.

can i get assigned?

please take an issue labelled with good first issue. thanks!


API for prestashop 1.6

Hi!
I have a version prestashop 1.6 and the module is only valid as of version 1.7, is there any solution?
Regards.

to be honest, I don't see anything specific to 1.7 release here...
How about updating the ps version compliancy to 1.6.1.? https://github.com/coinbase/coinbase-commerce-prestashop/blob/master/coinbase/coinbase.php#L21

Hey @Edabea, we've added support for 1.6 in the latest code. Stay tuned for the latest release.


Bug: ReferenceError: require is not defined in ES module scope when using @aws-lambda-powertools/tracer in ES Module

Expected Behavior
The package should work seamlessly with ES module syntax without throwing a require error.
Current Behavior
I'm encountering an issue when trying to use the @aws-lambda-powertools/tracer package in a TypeScript project configured as an ES module. The error message suggests that there is an internal use of require which is not supported in ES module scope.
Code snippet
//route.ts file
import { APIGatewayProxyResult, Context } from 'aws-lambda';
import { Tracer } from '@aws-lambda-powertools/tracer';
import { AccountController } from './controllers/AccountController.js';
import { loadConfig } from './config.js';
const tracer = new Tracer({ serviceName: 'transflowAccountService' });
export const updateAccountHandler = async (event: any, context: Context): Promise => {
tracer.getSegment();
const container = await loadConfig();
const account = new AccountController(container);
const response = await account.suspend(event.body as any);
return response;

};
Steps to Reproduce

Configure package.json with "type": "module".
Install @aws-lambda-powertools/tracer and use it in a TypeScript file with ES module syntax.
tsConfig.json
{
"compilerOptions": {
"allowUnreachableCode": false,
"allowUnusedLabels": false,
"declaration": true,
"declarationMap": true,
"forceConsistentCasingInFileNames": true,
"noEmitOnError": true,
"noFallthroughCasesInSwitch": true,
"noImplicitReturns": true,
"removeComments": true,
"esModuleInterop": true,
"experimentalDecorators": true,
"emitDecoratorMetadata": true,
"resolveJsonModule": true,
"pretty": true,
"sourceMap": true,
"strict": true,
"module": "esnext",
"moduleResolution": "node",
"target": "es2019",
"outDir": "build",
"rootDir": "src",
"lib": ["es2019", "DOM"]
},
"ts-node": {
"esm": true,
"experimentalSpecifierResolution": "node",
},
"exclude": ["node_modules", "built"],
// "include": ["src/.ts", "src/**/.ts"]
"include": ["src/**/*"],
}

Possible Solution
No response
Powertools for AWS Lambda (TypeScript) version
latest
AWS Lambda function runtime
20.x
Packaging format used
npm
Execution logs
2024-06-05T09:08:00.072Z undefined ERROR Uncaught Exception
{
    "errorType": "ReferenceError",
    "errorMessage": "require is not defined in ES module scope, you can use import instead\nThis file is being treated as an ES module because it has a '.js' file extension and '/var/task/package.json' contains \"type\": \"module\". To treat it as a CommonJS script, rename it to use the '.cjs' file extension.",
    "stack": [
        "ReferenceError: require is not defined in ES module scope, you can use import instead",
        "This file is being treated as an ES module because it has a '.js' file extension and '/var/task/package.json' contains \"type\": \"module\". To treat it as a CommonJS script, rename it to use the '.cjs' file extension.",
        "    at ProviderService.captureHTTPsGlobal (file:///var/task/node_modules/@aws-lambda-powertools/tracer/lib/esm/provider/ProviderService.js:33:9)",
        "    at new Tracer (file:///var/task/node_modules/@aws-lambda-powertools/tracer/lib/esm/Tracer.js:122:27)",
        "    at file:///var/task/build/routes.js:11:16",
        "    at ModuleJob.run (node:internal/modules/esm/module_job:222:25)",
        "    at async ModuleLoader.import (node:internal/modules/esm/loader:316:24)",
        "    at async _tryAwaitImport (file:///var/runtime/index.mjs:1008:16)",
        "    at async _tryRequire (file:///var/runtime/index.mjs:1057:37)",
        "    at async _loadUserApp (file:///var/runtime/index.mjs:1081:16)",
        "    at async UserFunction.js.module.exports.load (file:///var/runtime/index.mjs:1119:21)",
        "    at async start (file:///var/runtime/index.mjs:1282:23)"
    ]
}

Hi, thank you for opening this issue.
This is a known limitation of the AWS X-Ray SDK for Node.js that doesn't support ESM.
If you want to use ESM you need to provide a polyfill for the require keyword during your bundling process, you can find a reference on how to do this in our docs here.
Duplicate of #2555, #2523, #2464

@dreamorosi, thank you for your response. I've noted it and will make the necessary updates. This is very helpful.


Refactor

General code refactor.

Tested all the UDFs/UDAFs on a production database, everything works correctly. Going to merge it in.


Mode of Action types in WD

Based on the discussion we had previously in #3 , I would suggest the following regarding modeling the mode of actions:
There are two ways to model the info from BEL into WD I believe:

First, we could use the "physically interacts with", and then add the relevant WD ID as a subject (which would gives us more freedom in modeling, however then the model itself is less transparent and more difficult to query).
Or second, we use dedicated properties (=predicates) to model the BEL language, see for example inhibitor of (a property) which is referring to the MoA more then enzyme inhibitor (an item). This gives us less modeling freedom (because getting a property accepted is more difficult then just making an item), however the model is more understandable, inter-operable etc.

Some more relevant info for point 2 can be found in the property proposal. Specifically the link to the relation ontology is very useful to see if we are missing any properties now, and if so how these are linked to one another. The properties which are now modeled can be found here, under "See also" :

Probably quite a lot of compounds are now modeled in the old way (point 1), but I think point 2 would be more clear. @egonw and @cthoyt what do you think? Do the properties above describe what is needed (now)?

I've made a couple updates to the output (normalize HGNC genes to UniProt proteins, add FamPlex hierarchy, add more manually curated content), and the current output summary looks like this:

source_db
relation
target_type
target_db
count

chebi
activator
protein
uniprot
28

chebi
activator
protein family
ec-code
1

chebi
activator
protein family
fplx
4

chebi
agonist
protein
uniprot
281

chebi
agonist
protein family
fplx
24

chebi
antagonist
protein
uniprot
134

chebi
antagonist
protein family
fplx
16

chebi
antagonist
protein family
hgnc.genefamily
18

chebi
inhibitor
complex
fplx
42

chebi
inhibitor
complex
go
1

chebi
inhibitor
protein
prosite
1795

chebi
inhibitor
protein
uniprot
694964

chebi
inhibitor
protein family
ec-code
22417

chebi
inhibitor
protein family
fplx
6

chebi
inverse agonist
protein
uniprot
2

chebi
inverse agonist
protein family
fplx
1

chebi
negatively_regulates
pathway
go
146

chebi
positively_regulates
pathway
efo
6

chebi
positively_regulates
pathway
go
9

chebi
positively_regulates
pathway
ncit
37

It looks like the pharmacological relation for inverse agonist is missing from this list (and also RO). positively_regulates (RO:0002213) and negatively_regulates (RO:0002212) both should map to the relations in the Gene Ontology/Relations Ontology

Okay cool! I've opened up the relationship proposal topic in WD , hope to get some responses :) .

@DeniseSl22 I just got pinged about this repo by @cmungall, so I'm going to start looking back into this. No pressure from your side - enjoy baby time :)

@cthoyt okay cool! Let's hope some stuff happens on the Wikidata side; otherwise I'll make my own proposals ;) . I'm slowly getting back to working on projects, unofficially. Leave is scheduled until end of July; August I'll start with 2 days. From September on I'm available again full time. Hope you're doing well.

Hey @DeniseSl22, good to hear from you! Stuff is fine across the border, but always threatening to get worse because the inhabitants of Germany don't appreciate how (relatively) good we've had it.
I've also begun a few conversations about how the Relation Ontology might include these kinds of relations at https://github.com/oborel/obo-relations/issues/371 and https://github.com/oborel/obo-relations/issues/369
I'm a bit worried about making all of these things aligned.  However, the RO discussion seemed to stall. I think we should go ahead and make our own proposals on WikiData to really get the ball rolling

Update: I added a property proposal for "inverse agonist of" at https://www.wikidata.org/wiki/Wikidata:Property_proposal/Natural_science#inverse_agonist_of


Component Relationships Section - License

"The URL for each license link MUST be a public webpage  with the full text of the proprietary license."
How can this apply to proprietary licenses where polishing on a public website is not the preferred approach?

Licences referenced should be public but making public all licenses could reveal vendor relationships that are not desired to known.

Does https://github.com/hughsie/uefi-sbom-best-practices/commit/cf76bf475cd3420561d8f74ecec071b227f16d99 help? I think it's also fine to describe the license without naming the entity.


importing pex.variables will may look at disk and update state

https://github.com/pantsbuild/pex/blob/master/pex/variables.py#L36
When a module imports variables it creates an ENV object which checks the env variables and stores them.  It also may read files from disk.  In most cases this is probably an unexpected side effect.  Since many modules inside pex import variables the user is often more removed from this effect since they may not be directly importing variables.
Proposed solution is to change Variables to load values lazily on first access.  This will be a change in behavior and may impact users who implicitly rely on this behavior when using pex as a library.

do you mean having a deferred __init__ triggered off a __getattr__ call?
I'm just an innocent bystander, but I'm curious what behavior you want to see. I assume you are aiming to avoid code execution upon import (a clear anti-pattern, to be sure). Yeah?

I'm going to close this as obsolete since Pex discontinued support for any API in the switch to 2.x. Its only interface since then has been the CLI, with strict backwards compatibility guarantees.


Build error on Ubuntu 18.04 LTS

Bug Report
yarn build fails on Ubuntu 18.04LTS
Describe the Bug
Trying to follow the instructions here: https://docs.ohif.org/development/getting-started.html
I installed NPM and Node, installed Yarn
ran yarn config set workspaces-experimental true
running yarn install
and it fails with
[10/46] ? cypress
[-/46] ? waiting...
[-/46] ? waiting...
[-/46] ? waiting...
error /home/<username>/ohif/Viewers/node_modules/sharp: Command failed.
 System Linux 5.0.0-1028-azure
gyp ERR! command "/usr/bin/node" "/home/<username>/ohif/Viewers/node_modules/.bin/node-gyp" "rebuild"
gyp ERR! cwd /home/<username>/ohif/Viewers/node_modules/sharp

additionally, if I try npm rebuild it fails with
><EMAIL_ADDRESS>install /home/<username>/ohif/Viewers/node_modules/sharp
> (node install/libvips && node install/dll-copy && prebuild-install) || (node-gyp rebuild && node install/dll-copy)

info sharp Using existing vendored libvips v8.7.4
make: Entering directory '/home/<username>/ohif/Viewers/node_modules/sharp/build'
  TOUCH Release/obj.target/libvips-cpp.stamp
  CXX(target) Release/obj.target/sharp/src/common.o
  CXX(target) Release/obj.target/sharp/src/metadata.o
  CXX(target) Release/obj.target/sharp/src/stats.o
../src/stats.cc: In member function ‘virtual void StatsWorker::HandleOKCallback()’:
../src/stats.cc:130:37: error: no matching function for call to ‘v8::Array::Set(int&, v8::Local<v8::Object>&)’
         channels->Set(i, channelStat);
                                     ^
In file included from /home/<username>/.cache/node-gyp/13.6.0/include/node/node.h:63:0,
                 from ../src/stats.cc:19:
/home/<username>/.cache/node-gyp/13.6.0/include/node/v8.h:3547:37: note: candidate: v8::Maybe<bool> v8::Object::Set(v8::Local<v8::Context>, v8::Local<v8::Value>, v8::Local<v8::Value>)
   V8_WARN_UNUSED_RESULT Maybe<bool> Set(Local<Context> context,
                                     ^~~
/home/<username>/.cache/node-gyp/13.6.0/include/node/v8.h:3547:37: note:   candidate expects 3 arguments, 2 provided
/home/<username>/.cache/node-gyp/13.6.0/include/node/v8.h:3550:37: note: candidate: v8::Maybe<bool> v8::Object::Set(v8::Local<v8::Context>, uint32_t, v8::Local<v8::Value>)
   V8_WARN_UNUSED_RESULT Maybe<bool> Set(Local<Context> context, uint32_t index,
                                     ^~~
/home/<username>/.cache/node-gyp/13.6.0/include/node/v8.h:3550:37: note:   candidate expects 3 arguments, 2 provided
At global scope:
cc1plus: warning: unrecognized command line option ‘-Wno-cast-function-type’
sharp.target.mk:138: recipe for target 'Release/obj.target/sharp/src/stats.o' failed
make: *** [Release/obj.target/sharp/src/stats.o] Error 1
make: Leaving directory '/home/<username>/ohif/Viewers/node_modules/sharp/build'
gyp ERR! build error
gyp ERR! stack Error: `make` failed with exit code: 2
gyp ERR! stack     at ChildProcess.onExit (/usr/lib/node_modules/npm/node_modules/node-gyp/lib/build.js:194:23)
gyp ERR! stack     at ChildProcess.emit (events.js:321:20)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:275:12)
gyp ERR! System Linux 5.0.0-1028-azure
gyp ERR! command "/usr/bin/node" "/usr/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /home/<username>/ohif/Viewers/node_modules/sharp
gyp ERR! node -v v13.6.0
gyp ERR! node-gyp -v v5.0.5
gyp ERR! not ok
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: `(node install/libvips && node install/dll-copy && prebuild-install) || (node-gyp rebuild && node install/dll-copy)`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/<username>/.npm/_logs/2020-01-20T10_56_30_082Z-debug.log

yarn version: 1.21.1
node version: 13.6.0
npm version: 6.13.4
What steps can we follow to reproduce the bug?
Follow deployment instructions at https://docs.ohif.org/deployment/recipes/build-for-production.html or https://docs.ohif.org/development/getting-started.html

:warning: Reports we cannot reproduce are at risk of being marked stale and
closed. The more information you can provide, the more likely we are to look
into and address your issue.

Please review the latest code in the master branch. This issue might have been resolved. If it persists, kindly reopen the issue with updated details.
Try viewer-dev.ohif.org instead of viewer.ohif.org
Our viewer.ohif.org is deployed from release branch while viewer-dev.ohif.org is our master  branch
Read more about branch explanations here https://docs.ohif.org/development/getting-started#developing

I had the same problem. I was using node v20.0.0 with nvm. I downgraded to node v16.19.0 and yarn install worked.


Is this repo mantained?

I see there's a new version published 2 days ago, but issues are unanswered since months.
Any chance to get bugs fixed?

Most of the issues - are installation or configuration problems. I'm trying to answer some of them.

@anija people just don't follow instructions from Google, for some reason.


Update custom-dependency-properties.md

Changes C++ to C++/CX for clarity

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

#sign-off


Tdx18

Hey Josh, some of this has been done in other branches and your implementation, especially around data, differs from those branches.  Please look at some of the other branches, especially pkg2-beta.

Ignore these, they're just test pulls


Tiles don't line up exactly

Perhaps it's because I'm not using LoadImage, but I've noticed that the tiles don't always line up against eachother--there's usually a gap in-between map tiles.

I believe this is a duplicate of #9
I mean precision fixes should correct that as well

this should be fixed with double precision commits


Fails to upload to coveralls

Get's error: Exception: Unprocessable Entity {"message":"JSON file not found or failed to parse.","error":true,"url":""} when trying to upload data to coveralls. Check log output for more information: https://travis-ci.org/drager/faker/jobs/64918482

Hitting the same issue. Going back to 0.2.0 and things work fine.

Hitting the same issue too: https://travis-ci.org/domokit/cassowary/builds/68539740#L835

0.5.0 was released this morning.  Please try again.  Things should work with the latest dart now.  Please let us know if you have any trouble!

Please file a new issue and analysis if this problems persists.


Specs

It would be good to set up a policy on specs (CONTRIBUTING.md) and/or adopt a spec suite. I'm partial to trying out minitest.cr, but I'm not fully aware of the other options available, as well as specs practices in Crystal (if they differ in any way from langs like ruby w/ rspec). The crystal stdlib spec API seems a bit limited.

In other libraries I've written for REST APIs, I've created a namespace for functions that generate HTTP::Request, for example:
# request.cr
module Request
  extend self

  def get_resource(param)
    params = HTTP::Params.build do |form|
      form.add("param", param)
    end

    HTTP::Request.new("GET", "/resource?#{params}")
  end
end

# rest.cr
module REST
  @http_client = HTTP::Client.new(API_HOST, tls: true)

  def request(request : HTTP::Request)
    @http_client.exec(request)
  end

  def get_resource(param)
    response = request(Request.get_resource(param))
    Resource.from_json(response.body)
  end
end

Allowing me to create a spec:
it "forms GET /resource" do
  request = Request.get_resource("foo")
  request.method.should eq "GET"
  request.path.should eq "/resource"
  request.query.should eq "param=foo"
end

I could write all of these Request functions and specs per the API documentation, and then write my REST binding knowing that I was always executing a correctly formed request.
Would there be value in writing a similar request-building function layer for discordcr?

Is there still no mock API server, wasn't there a project to make one?

A few - meew0 was going to make one (link), as well as b1nzy at one point I think. Maybe others. None to my knowledge are functional for any serious use


Fixing current deprecation warnings and re-enabling them.

We can turn them back off if there's big integrates blocked by them but in the steady state having them off just leads to lazy inconsistency.

Ugh, of course xla builds with our settings instead of the tf settings and it has tf-specific code that pulls in deprecated things. I give up - switching to just disabling this in msvc and continuing to keep our heads in the sand.


Payment methods mismatch refund

Log:
2015-06-26T15:31:15+00:00 DEBUG (7): Array
(
    [processResponse begin] => Begin to process Notification
    [response] => Array
        (
            [amount] => Array
                (
                    [currency] => EUR
                    [value] => 9800
                )

            [eventCode] => REFUND
            [eventDate] => 2015-06-26T17:29:58+02:00
            [merchantAccountCode] => AceTateNL
            [merchantReference] => 100024235
            [originalReference] =><PHONE_NUMBER>817380
            [paymentMethod] => mc
            [pspReference] =><PHONE_NUMBER>988599
            [success] => true
        )

    [_validateNotification] => Payment method in Magento is: hpp, payment method in notification is: mc
    [info] => Order does not validate payment method in Magento don't match with payment method in notification
    [__pid] => 9245
)

Order was placed on 24-05-2015 using Adyen_Payment 2.1.1 and refund was invoked from Adyen control panel on 22-06-2015 when we were on Adyen_Payment 2.3.0. Is this the issue?

Hi Leeuwd,
This is fixed in the development branch. We will soon release this hotfix into the release of 2.3.1.
Regards,
Rik
Adyen


Changed function names to $this->functionname

We should use $this-> for local class function calling.

Good catch @brammittendorff. To be honest this is pretty much dead code at the moment in Twill because our uploader currently doesn't make use of chunked requests. So we might clean things up at some point but still makes sense to merge this in the meantime. Thank you!


Consider providing syn::parse::Nothing that parses nothing

Defined roughly as:
struct Nothing;

impl Parse for Nothing {
    fn parse(_input: ParseStream) -> Result<Self> {
        Ok(Nothing)
    }
}

This would be useful for attribute macros that want to ensure they are not provided any attribute args.
#[proc_macro_attribute]
pub fn my_attr(args: TokenStream, input: TokenStream) -> TokenStream {
    parse_macro_input!(args as Nothing);

    /* ... */
}

error: unexpected token
 --> src/main.rs:3:19
  |
3 | #[my_attr(asdf)]
  |           ^^^^

I think this is often called 'epsilon', but perhaps 'Nothing' is more clear :-)

Added in 82a3aed7ecfd07fc2f7f322b01d2413ffea6c5e7.


Address CRAN comments

Thanks,
If there are references describing the methods in your package, please
add these in the description field of your DESCRIPTION file in the form
authors (year) doi:...
authors (year) arXiv:...
authors (year, ISBN:...)
or if those are not available: https:...
with no space after 'doi:', 'arXiv:', 'https:' and angle brackets for
auto-linking.
(If you want to add a title as well please put it in quotes: "Title")
Please add \value to .Rd files regarding exported methods and explain
the functions results in the documentation. Please write about the
structure of the output (class) and also what the output means. (If a
function does not return a value, please document that too, e.g.
\value{No return value, called for side effects} or similar)
Missing Rd-tags in up to 35 .Rd files, e.g.:
assert_list_element.Rd: \value
assert_one_to_one.Rd: \value
assert_varval_list.Rd: \value
...
You have examples for unexported functions.
Please either omit these examples or export these functions.
Used ::: in documentation:
man/arg_name.Rd:
test_fun <- function(something) {
admiral:::arg_name(substitute(something))
}
etc..
Please fix and resubmit.

@pharmaverse/admiral-devs I need your help with the second point, i.e.

Please add \value to .Rd files regarding exported methods and explain the functions results in the documentation. Please write about the structure of the output (class) and also what the output means. (If a function does not return a value, please document that too, e.g. \value{No return value, called for side effects} or similar).

We have to find all exported functions which do not have a "Value" heading in the documentation page (which means it does not have a @return tag in the function header). If you find such a function please add it in a comment here.

All man files without \value{:
adae.Rd
adcm.Rd
add.Rd
adex.Rd
admiral-package.Rd
adsl.Rd
advs.Rd
arg_name.Rd
assert_list_element.Rd
assert_one_to_one.Rd
assert_varval_list.Rd
backquote.Rd
compute_map.Rd
convert_blanks_to_na.Rd
dataset_vignette.Rd
default_qtc_paramcd.Rd
derive_baseline.Rd
derive_params_exposure.Rd
derive_var_basec.Rd
derive_var_basetype.Rd
enumerate.Rd
expect_dfs_equal.Rd
ex_single.Rd
extract_duplicate_records.Rd
extract_unit.Rd
extract_vars.Rd
filter_if.Rd
get_duplicates_dataset.Rd
get_many_to_one_dataset.Rd
get_one_to_many_dataset.Rd
grapes-notin-grapes.Rd
list_all_templates.Rd
negate_vars.Rd
print.tte_source.Rd
queries.Rd
quo_c.Rd
reexports.Rd
signal_duplicate_records.Rd
squote.Rd
tte_source_objects.Rd
use_ad_template.Rd
vars2chr.Rd
warn_if_invalid_dtc.Rd
warn_if_vars_exist.Rd
what_is_it.Rd

Thanks a lot @bundfussr! How did you discover this so quickly?

I called grep -L '\\value{' *.Rd in the man folder.

I have to seriously improve my command line skills 😅
Furtunately I asked and didn't skin through all pages manually...


Access token is not removed from the server after logging out

Core Library
MSAL.js (@azure/msal-browser)
Core Library Version
2.18.0
Wrapper Library
MSAL React (@azure/msal-react)
Wrapper Library Version
1.0.0-alpha.1
Public or Confidential Client?
Public
Description
I'm using logoutRedirect to log out from the app. But seems like the user session is active after logging out because I still can use the old access token to authenticate requests that I'm sending via Postman
Error Message
No response
MSAL Logs
No response
Network Trace (Preferrably Fiddler)

[ ] Sent
[ ] Pending

MSAL Configuration
{
    auth: {
        authority: "https://login.microsoftonline.com/{tenantId}",
        clientId: "clientId",
        redirectUri: "http://localhost:3000",
    },
    cache: {
        cacheLocation: BrowserCacheLocation.SessionStorage,
        storeAuthStateInCookie: false,
    },
};

Relevant Code Snippets
export const logoutCurrentUser = async (): Promise<void> => {
    const accounts = authProvider.getAllAccounts();
    const account = accounts[0] || null;
    await authProvider.logoutRedirect({
        account: account,
        postLogoutRedirectUri: "http://localhost:3000",
    });
};

Reproduction Steps

Copy the access token 1 that is generated.
Log out of the application.
Try to send a request with token 1 in Postman

Expected Behavior
401 error will appear
Identity Provider
Entra ID (formerly Azure AD) / MSA
Browsers Affected (Select all that apply)
Chrome
Regression
No response
Source
Internal (Microsoft)

Access tokens are stateless so this is expected. After signing out you will not be able to acquire new tokens without re-authenticating, however, existing tokens will be valid until expiration. MSAL.js will clear tokens from the cache upon signout but if you're storing them somewhere else it is your responsibility to remove them.


Improper user or organization names are allowed

This issue is being fixed by a pull request #4695

Gogs version (or commit ref): <IP_ADDRESS>7
Git version: Irrelevant
Operating system: Irrelevant
Database: Irrelevant
Can you reproduce the bug at https://try.gogs.io:

[x] Yes (provide example URL)
[x] https://try.gogs.io/explore
[x] https://try.gogs.io/explore/repos.git
[x] https://try.gogs.io/explore/users.git
[x] https://try.gogs.io/explore/organizations.git
[x] https://try.gogs.io/explore/repos
[x] https://try.gogs.io/explore/users
[x] https://try.gogs.io/explore/organizations
[x] https://try.gogs.io/org/create/settings
[x] https://try.gogs.io/org/create

Log gist:

[ ] I don't have access to log files from https://try.gogs.io

Description
Users or organizations can be created with an improper name, such as "explore" or "create"

fixing it in #4695

Close by merging https://github.com/gogits/gogs/pull/4695.


upcoming: [M3-7926] - Linode Create Refactor - VLANs - Part 9

Description 📝
Adds VLANs to the new Linode Create flow ✨
Changes  🔄

Use query key factory for VLANs
Adds new VLANSelect component
Add infinite query for VLANs to power the VLAN Select
Adds VLANs section to the new Linode Create flow

Preview 📷

How to test 🧪
Prerequisites

Turn on the Linode Create v2 feature flag using our local dev tools

Verification steps

Test the new VLAN section
Test the new VLANSelect
Check new logic, utils, and UI for issues, regressions, and quality
Verify you can create a Linode with a VLAN attached
Verify VLANs throughout the app still work as expected

As an Author I have considered 🤔

[ ] 👀 Doing a self review
[ ] ❔ Our contribution guidelines
[ ] 🤏 Splitting feature into small PRs
[x] ➕ Adding a changeset
[x] 🧪 Providing/Improving test coverage
[ ] 🔐 Removing all sensitive information from the code and PR description
[ ] 🚩 Using a feature flag to protect the release
[ ] 👣 Providing comprehensive reproduction steps
[ ] 📑 Providing or updating our documentation
[ ] 🕛 Scheduling a pair reviewing session
[ ] 📱 Providing mobile support
[ ] ♿  Providing accessibility support

I observed a bit of odd behavior in the Linode Create flow with a "ghost" VLAN appearing in the dropdown:

Was there a response to this comment by @dwiley-akamai? I am not able to replicate it myself -- I think it might be different VLANs in different regions with the same name.

I wasn't able to reproduce the "ghost" issue, but the component has undergone some significant changes and it now filters by region. I think we should be good now @hkhalil-akamai @dwiley-akamai

Oops. I forgot to fix those spelling issues @dwiley-akamai. My fault 🤦
I'll make sure to fix them in the next Linode Create reactor PR


Bolse - Unplayable due to missing objects (Windows)

When entering into Bolse Defense Station stage, the player's ship as well as the energy towers, enemy ships and central fixture do not appear, rendering the stage unplayable. This occurs during normal play and when entering directly to stage with level select debug option.
Player's ship as well as crosshairs are not visible once control is gained:

The same assets are missing during the intro cutscene (the energy shield in the center is normally visible):

Hi, I’m not experiencing this issue on my end, even after multiple attempts
Are you sure you weren’t running any debug features outside of the level select at any point during your playthrough?

These were the only debug options active when this issue was encountered.

Issue was encountered on both DirectX and OpenGL rendering options as well.

Same thing happens for me in Bolse and also in Katina. No cutscene plays for either. The little pyramid in Katina is visible, but nothing in Bolse. No debug options on. Changed graphics settings to max when i first booted up the game, but didn't change anything else before I got to Katina as normal.
https://github.com/user-attachments/assets/a3259081-d651-4060-ab34-0cbcde03cbdf
https://github.com/user-attachments/assets/9888d9de-bb1c-4559-8ce8-d06466249163

Is this the release build?

I'm unable to reproduce this issue.

Yes, release build, no branches or modifications.

Yes, release build, no branches or modifications.

Same for me.


 Good spot!
On Sat, 28 Oct 2023, 17:06 Felix Wiegand, @.***> wrote:

@.**** requested changes on this pull request.
From what i have heard this sounds amazing thanks so much for this!
I think we might need to introduce a .gitignore file though that contains
*/.DS_Store and */.vscode/.
—
Reply to this email directly, view it on GitHub
https://github.com/wgd-modular/loewenzahnhonig-firmware/pull/2#pullrequestreview-1702811838,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACKZSUOJ5AQA6LEREPMETN3YBUUQVAVCNFSM6AAAAAA6N5XHZCVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTOMBSHAYTCOBTHA
.
You are receiving this because you authored the thread.Message ID:
@.***>


gambler script.

✋❤
On 6 Oct 2019 22:29, "Craig"<EMAIL_ADDRESS>wrote:

Merged #578 https://github.com/geekcomputers/Python/pull/578 into
master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/geekcomputers/Python/pull/578?email_source=notifications&email_token=ALWRAPD4ZZXGHQESSUZ4TYLQNIYPTA5CNFSM4I53Q6A2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOUBLWBNA#event-2690080948,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ALWRAPGT7TXWNTIF3DZ3XQTQNIYPTANCNFSM4I53Q6AQ
.


OSSM-4855 Add support to build operator on MacOs

/retest

/retest

/retest

Tested on MacOs and working


Use lazy logging when reporting memcached errors

Context being: Sentry generates message fingerprints depending on how the message is constructed, because the string is interpolated before sending it to the logger it considers every message unique (because the message is always different). This way Sentry will collate them under the same entry.
Also this is how the Python docs document suggested logging (see: https://docs.python.org/2/library/logging.html#logging.Logger.debug)
logger.warning('Protocol problem: %s', 'connection reset', extra=d)

@jwhitlock any chance of a look-see on this one. Not sure it matters for us anymore, but it's still a sensible thing to merge.

Looks good to me as well. @iserko, you may have an opinion on logging vs exceptions, re https://github.com/django/django/pull/8681 .


Optionally dispatch callbacks in calling threads

Wrap callbacks from db_service::begin and query::run_async in strands to dispatch then to the calling thread. Create a flag not to use this behavior.

Wrapped callbacks in strands for connections. That means callbacks in a single transaction are called in one thread and in a strict order, in which the queries were enqueued.


VSCode server can not start on WSL (Ubuntu 19.10)

VSCode Version (insider):

Version: 1.40.0-insider (system setup)
Commit: 29ffbfe3f1c21b01a2d0670170855669dd2b3b4f
Date: 2019-10-25T14:03:31.497Z
Electron: 6.0.12
Chrome: 76.0.3809.146
Node.js: 12.4.0
V8: 7.6.303.31-electron.0
OS: Windows_NT x64 10.0.18362

VSCode Version (regular):

Version: 1.39.2 (system setup)
Commit: 6ab598523be7a800d7f3eb4d92d7ab9a66069390
Date: 2019-10-15T15:35:18.241Z
Electron: 4.2.10
Chrome: 69.0.3497.128
Node.js: 10.11.0
V8: 6.9.427.31-electron.0
OS: Windows_NT x64 10.0.18362

OS Version:

Windows_NT x64 10.0.18362

Steps to Reproduce:

use WSL with Ubuntu 19.10
run VSCode
install Remote - WSL extension
pick Remote - WSL: New Window
get errors in terminal:

[2019-10-25 15:12:00.985] Starting VS Code Server inside WSL (Ubuntu)
[2019-10-25 15:12:00.985] Extension version: 0.39.9, Windows build: 18362. Multi distro support: enabled. WSL path support: enabled
[2019-10-25 15:12:01.122] Probing if server is already installed: C:\WINDOWS\System32\wsl.exe -d Ubuntu -e sh -c "[ -d ~/.vscode-server/bin/6ab598523be7a800d7f3eb4d92d7ab9a66069390 ] && echo found || ([ -f /etc/alpine-release ] && echo alpine || echo x64)"
[2019-10-25 15:12:01.220] Server install found in WSL
[2019-10-25 15:12:01.220] Launching C:\WINDOWS\System32\wsl.exe -d Ubuntu sh -c '"$VSCODE_WSL_EXT_LOCATION/scripts/wslServer.sh" 6ab598523be7a800d7f3eb4d92d7ab9a66069390 stable .vscode-server 0  --disable-telemetry' in c:\Users\▮▮▮▮▮▮▮▮▮▮▮▮\.vscode\extensions\ms-vscode-remote.remote-wsl-0.39.9
[2019-10-25 15:12:01.421] Setting up server environment: Looking for /home/▮▮▮▮▮▮▮/.vscode-server/server-env-setup. Not found.
[2019-10-25 15:12:01.496] WSL version: 4.4.0-18362-Microsoft Ubuntu
[2019-10-25 15:12:01.496] Updating server...
[2019-10-25 15:12:01.496] Starting server: /home/▮▮▮▮▮▮▮/.vscode-server/bin/6ab598523be7a800d7f3eb4d92d7ab9a66069390/server.sh  --port=0 --fileWatcherPolling=0 --disable-telemetry
[2019-10-25 15:12:01.496] /c/Users/▮▮▮▮▮▮▮▮▮▮▮▮/.vscode/extensions/ms-vscode-remote.remote-wsl-0.39.9/scripts/wslServer.sh: 60: /home/▮▮▮▮▮▮▮/.vscode-server/bin/6ab598523be7a800d7f3eb4d92d7ab9a66069390/server.sh: not found
[2019-10-25 15:12:01.496] VS Code Server for WSL closed unexpectedly.
[2019-10-25 15:12:01.496] For help with startup problems, go to
[2019-10-25 15:12:01.496] https://code.visualstudio.com/docs/remote/troubleshooting#_wsl-tips

Does this issue occur when all extensions are disabled?: Problem is closely related to Remote - WSL extension

May be important: https://github.com/andy-5/wslgit – I use v.0.7.0.

Can you do a ls -al /home/▮▮▮▮▮▮▮/.vscode-server/bin/6ab598523be7a800d7f3eb4d92d7ab9a66069390 to see if there really is an installation or wheter the installation got corrupted?
The log says it discovered an existing installation, is that possible?

Can you do a ls -al /home/▮▮▮▮▮▮▮/.vscode-server/bin/6ab598523be7a800d7f3eb4d92d7ab9a66069390 to see if there really is an installation or wheter the installation got corrupted?
The log says it discovered an existing installation, is that possible?

total 0
drwxrwxrwx 1 console console 4096 Oct 21 12:18 .
drwxrwxrwx 1 console console 4096 Oct 21 12:18 ..

So the 6ab598523be7a800d7f3eb4d92d7ab9a66069390  folder is there, but all empty?

So the 6ab598523be7a800d7f3eb4d92d7ab9a66069390  folder is there, but all empty?

Yep.

Sure. I'll do it tomorrow.


chore: Expose mocks files

closes #1179
Signed-off-by: Firas Qutishat<EMAIL_ADDRESS>

The mocks were moved inside the internal folder so that consumers wouldn't use it in actual code. As @Moopli suggested, may be we can move these mocks to _test.go file, so that only consumer test code can access it.

Now that we start using aries components, we should expose mocks for them. Maybe we don't need to expose all of them but certainly we need mocks for core services such as VDRI, KMS etc. Otherwise you have to implement mocks in every single project. We can definitely discuss which approach to take.

@fqutishat @sandrask @rolsonquadras do we really need to expose every mock? maybe there is a smaller subset?


std::thread::hardware_concurrency() always returns 1

I don't think it is a good idea to always return 1, since:

It's not true on pretty much any modern hardware.
The spec says: If this value is not computable or well defined an implementation should return 0.

Fixed. Thanks for reporting.


Install any node version from nvm + global dependencies?

I got curious into how node was installed and found it on the code. Why did you take the approach of
RUN (apt-get remove -y nodejs || true) && (apt purge nodejs || true)
RUN curl -sSL --fail https://deb.nodesource.com/setup_14.x >/tmp/setup_node.sh ||  curl -sSL --fail https://deb.nodesource.com/setup_14.sh >/tmp/setup_node.sh
RUN bash /tmp/setup_node.sh && apt-get install nodejs && npm config set unsafe-perm true && npm install --global gulp-cli yarn

Why not simply do it through nvm and allow ANY version on nodejs_version? Also I wonder if something like this: https://docs.platform.sh/languages/nodejs.html#2-specify-any-global-dependencies could be added to config which could work for node/php and potentially others in the future?
I know we can still use nvm to install any other version, but that means adding a post-start and the more I can set up on the image the better/faster it is to start it up. After all one need a specific version on the environment.

nodejs_version is done with the official packages. It predated the nvm support. So I guess it's just historical.
Note that nodejs when used from packages is done at Dockerfile time (so no overhead on second start), and nvm is done at start time (so it always has some overhead).
I don't know how nvm could set up at image build time and still be changeable by the user, which I'd assume they'd expect.

Different thoughts:

I can't think of a reason one would want to switch node version within the same project other than on build time. It is, if anything, very very rare. But let's assume you want to.
If you were to change versions, I assume you can sudo. I wonder if you install several versions from Dockerfile (like: nvm install 16, nvm install 8) you can then nvm use without sudo. Or you can just give user ownnership to nvm.
I believe you can use NVM in Dockerfile AND nvm in the user space. Obviously a bit of tweaking into how and where you install nvm and how you use the cache needs work, but I'd think it very similar to https://docs.platform.sh/languages/nodejs/node-version.html#use-nvm.

Not sure about #3, but I am doing something very similar to
https://github.com/hanoii/ddev-nvm-issue-4250/blob/main/.ddev/web-build/Dockerfile.nvm And it works, I can even install another version because of the chmod
This doesn't use the cache, but I challenge that as well, do we need the cache at all if we are installing necessary version on the image? I really really don't think you need to nvm install in the lifetime of a project, only initially.

to add to the cache question, we are caching nvm per project, so not sure if the cache there is worth it

I think if we can keep backward compatibility, a PR would be fine. I do think people expect to be able to use nvm and ddev nvm on an existing project.

Is there a status update here?
It would be great if we could specify a more specific Node version. The corresponding version is also reused by us in other contexts (f.e. in our GitLab Build-Chain) and sometimes there is unexpected behaviour because the node or npm version in DDEV is slightly different than expected due to an update - even if the major version is still identical.
If we could specify the exact version as with the Composer version, both Node version and NPM version are fixed and we can use the same version everywhere.
I have tried to solve the problem with the currently available NVM integration but have not found a practical solution that may not be at least somewhat confusing (e.g. because the specified nodejs_version does not match the nvm version).

Hi @kevin-appelt - Why don't you just use ddev nvm install <version> ? See https://ddev.readthedocs.io/en/latest/users/basics/cli-usage/#nodejs-npm-nvm-and-yarn

Hi @rfay
My issue is that the change has to be done not only for me, but also for all my colleagues. In addition, the version is not identical in all projects. So the best would be to have the version in the DDEV configuration and in the Git repository.
There is already the nodejs_version, so it seems to be a good idea to use it.
I tried a post_start hook that uses ddev nvm install <version>, but it seems to work only at the first start of the project. On each subsequent start I get an error message that the version is already installed and when I then run node --version in the web container I seem to get the version defined by nodejs_version.
All in all it feels more like a workaround than a real solution (for us), because with nodejs_version a basically working configuration already exists - only limited to the major version.
A change of the version at runtime is (again: for us) not necessary either. The container runs with the specified version until this version gets updated.

If you want an explicit patch version of node,  you can put it in a post-start hook and check in your .ddev/config.yaml as is normally done.
hooks:
  post-start:
  - exec: "nvm install <version> && nvm use <version>"

If you really want an explicit version of the nodejs package, you can do it with a .ddev/web-build/Dockerfile.nodejs, which would do it at the package level.

This is my Dockerfile.node on a project I need node 8
ENV NVM_DIR=/usr/local/nvm
ENV NODE_DEFAULT_VERSION=8.17

RUN (apt-get remove -y nodejs || true) && (apt purge nodejs || true)

RUN curl -sL https://raw.githubusercontent.com/nvm-sh/nvm/v0.39.1/install.sh -o install_nvm.sh
RUN mkdir -p $NVM_DIR && bash install_nvm.sh
RUN echo "source $NVM_DIR/nvm.sh" >>/etc/profile
RUN bash -ic "nvm install $NODE_DEFAULT_VERSION && nvm use $NODE_DEFAULT_VERSION"
RUN chmod -R ugo+w $NVM_DIR

hooks:
  post-start:
  - exec: "nvm install <version> && nvm use <version>"

When trying to use nvm from a post-start hook command I always get:
bash: line 1: nvm: command not found

If I do ddev nvm ... it works and likewise if I SSH into the web container and run nvm it's also OK, is there anything special that needs to be done beforehand to make this command work?

So nvm is unfortunately not a regular command that can be executed, it's a bash alias. But please do open new issues instead of commenting on unrelated issues. You can work around your issue like this:
hooks:
  post-start:
  - exec-host: ddev nvm install 16

@rfay My apologies, as this issue was still open and tangibly related to the subject, I didn't want to create a whole other issue for fear of further noise ironically.
Thank you for the workaround.

So nvm is unfortunately not a regular command that can be executed, it's a bash alias. But please do open new issues instead of commenting on unrelated issues. You can work around your issue like this:
hooks:
  post-start:
  - exec-host: ddev nvm install 16

@rfay so this doesn't really work. I need to use Node 10, don't get me started on the EOL of it, and simply trying to use NVM and even having it set 10 as the default doesn't work when later I attempt to have a command that runs in the web service do npm ci for example, it's still running Node 14(what's defined in the config.yml). I need a solution that isn't going to require a huge number of manual steps every time we need to install packages are run builds of the code.

Could you please try latest HEAD version of DDEV? A significant fix was made (fixing a regression in v1.21.4). You can get latest at https://nightly.link/drud/ddev/workflows/master-build/master
See

#4620

I confirm that this works with HEAD:
hooks:
  post-start:
    - exec: set +u && nvm install 10 && nvm alias default 10
    - exec: node --version

@rfay so the HEAD version is working however there seems to be some other issue in HEAD. I'm not sure if this has been reported another issue but I'm getting tons of duplicate output during a ddev start & ddev stop
 Network ddev-<project>_default  Creating
 Network ddev-<project>_default  Created
 Container ddev-<project>-db  Creating
 Container ddev-<project>-redis  Creating
 Container ddev-<project>-web  Creating
 Container ddev-<project>-dba  Creating
 Container ddev-<project>-web  Created
 Container ddev-<project>-db  Created
 Container ddev-<project>-dba  Created
 Container ddev-<project>-redis  Created
 Container ddev-<project>-redis  Starting
 Container ddev-<project>-web  Starting
 Container ddev-<project>-dba  Starting
 Container ddev-<project>-db  Starting
 Container ddev-<project>-web  Started
 Container ddev-<project>-dba  Started
 Container ddev-<project>-db  Started
 Container ddev-<project>-redis  Started
 Container ddev-router  Creating
 Container ddev-router  Created
 Container ddev-router  Starting
 Container ddev-router  Started

 Container ddev-<project>-web  Stopping
 Container ddev-<project>-redis  Stopping
 Container ddev-<project>-db  Stopping
 Container ddev-<project>-dba  Stopping
 Container ddev-<project>-redis  Stopped
 Container ddev-<project>-web  Stopped
 Container ddev-<project>-dba  Stopped
 Container ddev-<project>-db  Stopped
 Container ddev-<project>-redis  Stopping
 Container ddev-<project>-redis  Stopping
 Container ddev-<project>-web  Stopping
 Container ddev-<project>-db  Stopping
 Container ddev-<project>-web  Stopping
 Container ddev-<project>-db  Stopping
 Container ddev-<project>-dba  Stopping
 Container ddev-<project>-dba  Stopping
 Container ddev-<project>-redis  Stopped
 Container ddev-<project>-redis  Removing
 Container ddev-<project>-db  Stopped
 Container ddev-<project>-dba  Stopped
 Container ddev-<project>-db  Removing
 Container ddev-<project>-dba  Removing
 Container ddev-<project>-web  Stopped
 Container ddev-<project>-web  Removing
 Container ddev-<project>-redis  Removed
 Container ddev-<project>-db  Removed
 Container ddev-<project>-dba  Removed
 Container ddev-<project>-web  Removed
 Network ddev-<project>_default  Removing
 Network ddev-<project>_default  Removed

I think there's too much of that, please open an issue, thanks.


Move server functions to separate submodule

Improves readability and is overall a much better module structure

When I run pip install -r requirements.txt --break-system-packages; pip uninstall salve_ipc -y --break-system-packages; pip install . --break-system-packages --no-build-isolation; pytest . locally I get no errors.

Well its not python version

Why in the world did the tests start working there?

If it works it works I guess


how to set playing game while running?

how to set playing game while running?

event.getJDA().getPresence().setGame(Game.of("Your game here"));

As a note, you can only set the presence of a bot 5 times per minute.

cd


Questions about dataset preprocessing and train configuration for unbalanced classes

I am training a custom dataset using the Pointcept codebase. My dataset consists of a large point cloud of a building facade spanning ~10 meters in width and height. I divided it into two files train.txt and val.txt including coordinates, colors, normals and label. The total amount of points is ~8M. I have a few questions.

Given the scale of my point cloud, should I tune some parameter related to the model?
Should I normalize the colors between 0 and 1?
I didn't understand how the trainer processes the input shape. It seems that the model is fed with inputs of size 200K. Should I divide the point clouds into multiple files?
How should I change the configuration file if I want to ignore one or more classes?
The dataset is unbalanced. Does the code manage this? Should I assign different weights to the classes? If so, how?

Given the scale of my point cloud, should I tune some parameter related to the model

10M^2 is not large compared with the outdoor scene. Just adjust the grid size (in both augmentations and model).

Should I normalize the colors between 0 and 1?

We have a unified strategy to normalize color; please refer to "NormalizeColor" in the data.transform. Just make sure your color range is [0, 255]. But this is important only when you want to engage multiple datasets for training.

I didn't understand how the trainer processes the input shape. It seems that the model is fed with inputs of size 200K. Should I divide the point clouds into multiple files?

Currently, we don't need to try to adjust the grid size, and if we fail to train the model efficiently, we can further chunk the data.

How should I change the configuration file if I want to ignore one or more classes?

In this case, you need to rewrite the dataset to map the classes you want to ignore to -1.

The dataset is unbalanced. Does the code manage this? Should I assign different weights to the classes? If so, how?

Lovasz loss, Weighted CE (provide a weight), CAC Segmentor.

Hi, thank you for your detailed reply.

Lovasz loss, Weighted CE (provide a weight), CAC Segmentor.

Should I implement the Weighted CE? As far the CAC, can I just put it instead of the default segmentor, or should I change some parameter?

I think initially default is good enough as a baseline.


run file for ACAS-XU

Hello,
I need to run the script  for all experiences on ACAS-XU ?
How can I specify the verifier ?
Which command should be launched ?
Thank you.

What script do you mean? DNNV is benchmark agnostic. It does not include any benchmarks or make any assumptions about what benchmark is being run.
I have made the ACAS Xu benchmark available here, separate from DNNV. I believe all of the properties can be run by using the script run.sh included in that archive, but I have not tested it recently. It believe it runs most of the verifiers. You can modify the run script to choose a specific verifier to run. There is also a version of the ACAS Xu benchmark (specified in VNNLIB) in this repo, in benchmarks/acasxu, but I don't think they provide a run script for DNNV, although they do provide a CSV file with the combinations of property and network combinations which you could parse.


[Fix] Install Fonts Script

Reviewer: @brindy
Asana:
CC:
Description:
Steps to test this PR:

<STEP 1>

Reviewer Checklist:

[ ] Ensure the PR solves the problem
[ ] Review every line of code
[ ] Ensure the PR does no harm by testing the changes thoroughly
[ ] Get help if you're uncomfortable with any of the above!
[ ] Determine if there are any quick wins that improve the implementation

PR DRI Checklist:

[ ] Get advice or leverage existing code
[ ] Agree on technical approach with reviewer (if the changes are nuanced)
[ ] Ensure that there is a testing strategy (and documented non-automated tests)
[ ] Ensure there is a documented monitoring strategy (if necessary)
[ ] Consider systems implications (Database connections, Grafana stats, CPU)

Have merged this with a minor change to generate an Xcode warning when fonts aren't there.

This also closes #261


IE bug: On donejs.com, when clinking on the side menu links, the page isn't scrolling to that section

Reproduces on IE (all IE all windows)
Steps:

go to http://donejs.com/Guide.html (or any other page with the side menu)
click on a title
notice that the page isn't scrolling to that section

I experienced that also in opera and its related to the scroll script probably as there is one I think at present
else this is really a browser bug that also happens some times in different browsers some one needs to look if there is a script involved when scrolling if there is one please share location so i can look and test and fix that


Observed white flickering page transition to Scene with Tabs

Version
Tell us which versions you are using:

react-native-router-flux v4.0-beta.24
react-native v0.50.3

Expected behaviour
During transition Entering a scene with Tabs defined should be no white flickering page
Actual behaviour
During transition entering a scene observed a white flickering page only if the Tabs under the Scene has property  <animationEnabled = true>
Steps to reproduce
For non-obvious bugs, please fork this component, modify Example project to reproduce your issue and include link here.

Defined a Scene with some Tabs inside the Scene, Set property <animationEnabled = true>
Observed white flicker page before entering that Scene that just created.
We did not observer this white flicker page if  property <animationEnabled = false>

Are you working in an emulator or a physical device?

This seems to be a problem with react-navigation, they are planning to resolve this issue on IOs in v2, but RNRF only supports v1..
source:
https://github.com/wix/react-native-navigation/issues/358

I have the same issue, and I am not using tabs, this both happens on Android and IOS, emulator and physical device. This is not easy to notice when you have a light color theme, my app has a night theme, when everything goes to dark, this white flickering became very annoying.
Anyone have a solution yet?

Please try to reproduce it with Example project and latest version 4.0.0-beta.40. Feel free to open if the issue still exists

I had the exact same issue mentioned above while using v4.2.0
and fixed it by setting the wrap property to false on the Tabs Component.
Since i use my own navbar solution this works fine, but could lead to problems if this is not the case.

Anyone have fixed this issue? I'm getting the same issue. My app has dark mode support so in dark mode, while changing the Tabs white screen flickering and It's looking very ugly. If anyone has fixed this issue with router flux library then please let me know. Thanks in advance.

Please try to reproduce it with Example project and latest version 4.0.0-beta.40. Feel free to open if the issue still exists

Anyone have fixed this issue? I'm getting the same issue. My app has dark mode support so in dark mode, while changing the Tabs white screen flickering and It's looking very ugly. If anyone has fixed this issue with router flux library then please let me know. Thanks in advance.
Here is an example: https://drive.google.com/file/d/1aGpVJBaJMCNeduVTCuT3MJCP1FEqEBf5/view?usp=sharing

I am getting the same issue on android. Anyone have fixed this issue on android?


For Ecipse ?

How can I add this in my eclipse project ?

If your eclipse android project uses gradle, you can include it as a dependency in your gradle build file. I don't really know how to add maven dependencies in Eclipse without gradle, though.
Eclipse's ADT is no longer in active development, the Android Developer Blog suggests you migrate to Android Studio:
http://android-developers.blogspot.com/2015/06/an-update-on-eclipse-android-developer.html
https://developer.android.com/sdk/installing/migrate.html

You may be able to clone this repo, build it with ./gradlew build, and look in the circularprogressview/build folder and see if it gives you a .jar or .aar you can use.


[INSTALLATION PROBLEM] - program wont start

When attempting to launch any of the py files it does not work. Intructions followed exactly. An error is given when I edit and run the module:
Traceback (most recent call last):
File "C:\Users\Matt\Downloads\UVR_V4GUI_All_IN_ONE_12_10\UVR-V4GUI\inference_v4.py", line 5, in
import cv2
ModuleNotFoundError: No module named 'cv2'

hopefully someone can help

Found similar post and resolved


[DevTestLabs] Adding scenario test to create simple Linux + Windows VM in lab

This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines

[ ] The PR has modified HISTORY.rst with an appropriate description of the change.

Command Guidelines

[x] Each command and parameter has a meaningful description.
[x] Each new command has a test.

(see Authoring Command Modules)

Codecov Report

Merging #2767 into master will increase coverage by 1.74%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2767      +/-   ##
==========================================
+ Coverage   61.15%   62.89%   +1.74%
==========================================
  Files         480      480
  Lines       25743    25744       +1
  Branches     3898     3898
==========================================
+ Hits        15742    16191     +449
+ Misses       9068     8540     -528
- Partials      933     1013      +80

Impacted Files
Coverage Δ

...li-lab/azure/cli/command_modules/lab/validators.py
25.35% <ø> (+11.97%)
:arrow_up:

...-cli-lab/azure/cli/command_modules/lab/commands.py
100% <100%> (ø)
:arrow_up:

...vtestlabs/operations/global_schedule_operations.py
8.08% <0%> (+2.12%)
:arrow_up:

.../lab/sdk/devtestlabs/operations/disk_operations.py
7.79% <0%> (+2.16%)
:arrow_up:

.../sdk/devtestlabs/operations/schedule_operations.py
8.37% <0%> (+2.2%)
:arrow_up:

.../operations/virtual_machine_schedule_operations.py
9.18% <0%> (+2.55%)
:arrow_up:

...labs/operations/notification_channel_operations.py
9.18% <0%> (+2.7%)
:arrow_up:

.../lab/sdk/devtestlabs/operations/user_operations.py
9.82% <0%> (+2.89%)
:arrow_up:

...ab/sdk/devtestlabs/operations/policy_operations.py
10% <0%> (+3.12%)
:arrow_up:

...vtestlabs/operations/artifact_source_operations.py
10% <0%> (+3.12%)
:arrow_up:

... and 33 more

Continue to review full report at Codecov.

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


myers/middle_snake: correct safety doc

Hello, one last thing that my colleague noticed. This PR is assuming that bounds_check is correct - I haven't scrutinized the algorithm enough to determine what the exact bounds are.
Again, feel free to change to meet your project's style. (This passes cargo fmt, but I don't know how long you prefer your documentation lines to be.)
Commit message follows:
In the documentation for MiddleSnakeSearch::new, the safety constraints described on the data pointer do not exactly match how the class uses it. It is stored as the kvec field, and looking at the write_xpos_at_diagonal, x_pos_at_diagonal, and pos_at_diagonal functions (a search of kvec in the source code revealing that all access to kvec go through one of these functions):

writes could happen, not only reads
the range of access is as described in the bounds_check function

Therefore, update the documentation accordingly.

I think I wrote this safety comment very easily when I started working on this and then didn't update it as the code evolved (as it's just an internal function). I did a patch release for the last fix since that could feasibly have negative effects but since this is just internal documentation and doesn't affect users I will merge this without cutting a release

Makes sense. Thanks for taking a look.