Need to know engagements

-- Updated requirement ---
Provide a mechanism to handle sensitive information.
Suggested approach:
Phase 1
Add an optional sensitive information text field to each report.
Information entered in that box should be visible only to authorized personel.
Define autorized personel as the following:
for each org, provide a list of default "sensitive information handling personel"
If and only if a reader is the report's author or is part of that group, he/she can have access to the sensitive content of reports from the same organization
#Phase 2
Provide a mechanism to override the default list on a report-by-report basis, i.e. exlude or include individual members for this report only

Scratchpad for proposed solution
Case 1. Nested explicit restrictions
An engagement report is visible to the user if:

the user is an admin
the user is the report author
the primary advisor of the report belongs to an org that is not restricted, neither is any of its parent orgs
the primary advisor of the report belongs to an org that that is restricted or has a restricted parent and the user is member of the most specific of these restricted org (or an indirect member thorugh a sub org)

Pros: relatively flexible
Cons: may be trickier to explain
HMI:
Any org has :

an indication of it's parent's most specific restriction (if any). Example: this org (1.1.1) is a sub organization of the restricted org 1.1
or an indication of a locally defined restriction. Example: this org restricts access to its members

Db:
Each org has a "restricted" field which can be true or false
Case 2. Flat restrictions
restricted here has a slightly different meaning
An engagement report is visible to the user if:

same first 3 cases as above
the primary advisor of the report belongs to an org that that is restricted and the user is member of in any org belonging to the entire restricted hierarchy

HMI:
Any org has :

an indication of it's parent's restriction (if any). Example: this org (1.1.1) is a sub organization of the restricted org 1.1
or an indication of a locally defined restriction. Example: this org restricts access to its members

Db:
Each org has a "restricted" field which can be true or false, but has to be true if the parent's is true
Pros: maybe easier to explain?
Cons: may be trickier to deal when admins are modifingn restriction tree. Potentiallty not as flexible (access groups can't be nested)
Case 3. Nested implicit restrictions, descending hierarchy
An engagement report is visible to the user if:

same first 3 cases as above
the primary advisor of the report belongs to an org that that is restricted and the user is member of that org or one of its sub organizations

HMI:
Any org has :

an indication of it's parent's restriction (if any). Example: this org (1.1.1) is a sub organization of the restricted org 1.1
or an indication of a locally defined restriction. Example: this org restricts access to its members

Db:
Each org has a "restricted" field which can be true or false, but has to be true if the parent's is true
Pros: maybe easier to explain?
Cons: As soon as a sub-org is created in an already restricted org, it will not be visible to members of the parent orgs

Updated requirement:
The following approach is proposed:
Engagement report (all of them to start with) to have an optional field - "sensitive information"
The content of the field is only visible to:

author
attendees
tbd

This is what we decided to build:

per organization, specify the list of positions (only positions from inside the org) who can see the sensitive part
only admin users may change this list of positions
author: sees sensitive info for his own reports
others: see the sensitive info only if they belong to the list of positions

Agreed plan of approach for sensitive information in reports:
• add table reportsSensitiveInformation with a text column of type nvarchar(max)
• ane one-to-one relationship between reports and reportsSensitiveInformation
• a report's sensitive information is only returned via the API if the user has permission (otherwise empty)
• a report's sensitive information can only be written via the API if the user has permission (otherwise ignored)
• a report's sensitive information is only shown (in display and edit forms) in the frontend if the user has permission (otherwise it would be empty anyway)
Permission set-up for sensitive information:
• add many-to-many authorizedPositions between organizations and positions
• only admins can edit the list of authorized positions for an organization
Permission to access sensitive information; either one of:
• user is the author of the report
• user currently holds a position that is in the list of authorizedPositions of the advisorOrg of the report
Audit trail:
• changes in authorizedPositions must be logged in the audit log
• changes to reportsSensitiveInformation must be logged in the audit log
• retrieves of reportsSensitiveInformation may be logged in the audit log
NOTE: do not log the sensitive information itself! only that it was inserted, updated (or retrieved) and by whom
And optionally add searching for sensitive information:
• add reportsSensitiveInformation.text column to full-text index
• column is only used in (full-text) searches if the user has permission (otherwise clause will not be added)


Provide fox default.

In order to render the fox before an error is thrown, we have to construct an element as a conditional. Combined with changes in MetaMask, this should not modify current behavior while also accounting for no WebGL.

Is 128px high enough resolution for a retina screen?

Not sure: this is what it looks like (when I do the element construction directly on the extension).


proposal Why don't you want to change docker to option?

What's going wrong?
Our team can't use docker because of bad security policies.
How can the cookiecutter-django-rest team reproduce the problem?
So, I suggest making the docker an option.

Is this a problem with a fresh install of the project?

[x] Yes
[ ] No

Thank you for your work. Stay healthy 👍 : )

Hi @binaryeast, you can choose to remove the Dockerfile and docker-compose file if you wish! It's there for your convenience, not as a requirement. 😄


Add support for client acknowledgements and QoS settings

Purpose
Add support for manual and auto acknowledgement modes and allow setting global and consumer service specific QoS settings.
Fixes #14842
Samples
Consumer with client acknowledgement
import ballerina/log;
import ballerina/rabbitmq;

rabbitmq:Connection connection = new({ host: "localhost", port: 5672 });
listener rabbitmq:ChannelListener channelListener = new(connection);

@rabbitmq:ServiceConfig {
    queueConfig: {
        queueName: "MyQueue"
    },
    ackMode: rabbitmq:CLIENT_ACK
}
service testClientAcknowledgement on channelListener {
    resource function onMessage(rabbitmq:Message message) {
        var msg = message.getTextContent();
        if (msg is string) {
            log:printInfo("The message received: " + msg);
        } else {
            log:printError("Error occurred while retrieving the message content.");
        }
        // Positively acknowledges a single message.
        var result = message->basicAck();
    }
}

Consumer with QoS settings
import ballerina/log;
import ballerina/rabbitmq;

rabbitmq:Connection connection = new({ host: "localhost", port: 5672 });

listener rabbitmq:ChannelListener channelListener = new(connection);

@rabbitmq:ServiceConfig {
    queueConfig: {
        queueName: "MyQueue"
    },
    ackMode: rabbitmq:CLIENT_ACK,
    prefetchCount: 10
}
service testClientQos on channelListener {
    resource function onMessage(rabbitmq:Message message) {
        var msg = message.getTextContent();
        if (msg is string) {
            log:printInfo("The message received: " + msg);
        } else {
            log:printError("Error occurred while retrieving the message content");
        }
        // The consumer will continue to receive messages from the server
        // once a total of 10(prefetchCount) messages are being acknowledged.
        var result = message->basicAck();
    }
}

Check List

[x] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[x] Added necessary documentation

[x] API documentation
[ ] Module documentation in Module.md files
[x] Ballerina By Examples

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@3de2644). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master   #15455   +/-   ##
=========================================
  Coverage          ?   22.61%
  Complexity        ?     5398
=========================================
  Files             ?     1457
  Lines             ?    73289
  Branches          ?     9714
=========================================
  Hits              ?    16576
  Misses            ?    54851
  Partials          ?     1862

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 3de2644...0ed44f4. Read the comment docs.


Backtracking

Can I work on Backtracking problems like Sudoku solver, N Queens and finding all subsets?

Heyy!! I would love to do bactracking problems of rat maze, color filling algorithm, n queens problem under mentorship of GSSOC'21. Will you please assign this task to me?

@noob-26 mention the language in which you want to add the code.

C++

I want to contribute to this repo as part of GSSoC 2021. So should I make the PR after 8th March, when the coding period starts?

@noob-26 Yes so that your point will be counted in GSSoC.

Thanks a lot for the help.

Can I work on Backtracking problems like Sudoku solver, N Queens and finding all subsets?

Please make separate issues for each algorithm and you will work on a sudoku solver in this issue.

@rudrakshi99 Please can you review my code for the Sudoku solver problem
Pull request #735

Hello! I'd like to add solutions to some famous backtracking problems in Python 3 as a part of GSSoC '21. It would be very helpful if you assign this issue to me.
@rudrakshi99
@raksha009

Heyy, this issue wasn't assigned to you. Plz create a pr only after the issue is assigned.
Also, if you wanna work on this problem. Plz create a different issue addressing to only 1 problem. Closing your pr for now.

Hello! I'd like to add solutions to some famous backtracking problems in Python 3 as a part of GSSoC '21. It would be very helpful if you assign this issue to me.
@rudrakshi99
@raksha009

Heyy, this issue wasn't assigned to you. Plz create a pr only after the issue is assigned.
Also, if you wanna work on this problem. Plz create a different issue addressing to only 1 problem. Closing your pr for now.
Hi, Sorry for the mistake. I have created a separate issue #1320 . Kindly do the needful.
@raksha009

Hello! I'd like to add solutions to some famous backtracking problems in Python 3 as a part of GSSoC '21. It would be very helpful if you assign this issue to me.
@rudrakshi99
@raksha009

Heyy, this issue wasn't assigned to you. Plz create a pr only after the issue is assigned.
Also, if you wanna work on this problem. Plz create a different issue addressing to only 1 problem. Closing your pr for now.

Hi, Sorry for the mistake. I have created a separate issue #1320 . Kindly do the needful.
@raksha009

Heyy, the issue is already present and assigned. Please look for other issues :)

I would like to work on this issue in Java. Kindly assign it to me @raksha009.

I would like to work on this issue in python. Kindly assign it to me @plazzy99

Hello! I'd like to add solutions to some famous backtracking problems in Python 3 as a part of GSSoC '21. It would be very helpful if you assign this issue to me.
@rudrakshi99
@raksha009

Go ahead with sudoku solver  in python

I would like to work on this issue in python. Kindly assign it to me @raksha009.


[REPLACED] Ah/byte-array-test

REPLACED BY PR https://github.com/AztecProtocol/barretenberg/pull/91

See #91


Unable to determine what CMake generator to use. Please install or configure a preferred

Brief Issue Summary
I installed CMake tools and CMake yesterday and it was working fine (w/ Ninja). Today when I opened VS Code all of a sudden it is giving this error:
[main] Unable to determine what CMake generator to use. Please install or configure a preferred generator, or update settings.json, your Kit configuration or PATH variable. Error: No usable generator found.

I have no idea what is its problem.
CMake Tools Diagnostics
{
  "os": "win32",
  "vscodeVersion": "1.92.2",
  "cmtVersion": "1.18.44",
  "configurations": [
    {
      "folder": "c:\\Users\\xxx\\code\\demo",
      "cmakeVersion": "unknown",
      "configured": false,
      "generator": "unknown",
      "usesPresets": false,
      "compilers": {}
    }
  ],
  "cpptoolsIntegration": {
    "isReady": false,
    "hasCodeModel": false,
    "activeBuildType": "",
    "buildTypesSeen": [],
    "requests": [],
    "responses": [],
    "partialMatches": [],
    "targetCount": 0,
    "executablesCount": 0,
    "librariesCount": 0,
    "targets": []
  },
  "settings": [
    {
      "communicationMode": "automatic",
      "useCMakePresets": "auto",
      "configureOnOpen": null
    }
  ]
}

Debug Log
[proc] Executing command: "C:\Program Files\CMake\bin\cmake.EXE" --version
[proc] Executing command: "C:\Program Files\CMake\bin\cmake.EXE" -E capabilities
[variant] Loaded new set of variants
[kit] Successfully loaded 4 kits from C:\Users\xxx\AppData\Local\CMakeTools\cmake-tools-kits.json
[proc] The command: ninja --version failed with error: Error: spawn ninja ENOENT
[proc] The command: ninja-build --version failed with error: Error: spawn ninja-build ENOENT
[proc] The command: make --version failed with error: Error: spawn make ENOENT
[main] Unable to determine what CMake generator to use. Please install or configure a preferred generator, or update settings.json, your Kit configuration or PATH variable. Error: No usable generator found.

Additional Information
No response

@siddjain Do you have your generator installed on the machine?
For us to investigate what is going on we'd need more information about your project in order to see what is happening. Thanks

Hi Greg, all I know is that the day I installed CMake tools, it worked perfectly and the very next day I started to get this error. What is CMake generator? I have following executables on my system:
C:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools>which ninja
/c/Program Files (x86)/Microsoft Visual Studio/2022/BuildTools/Common7/IDE/CommonExtensions/Microsoft/CMake/Ninja/ninja

C:\Program Files (x86)\Microsoft Visual Studio\2022\BuildTools>which cmake
/c/Program Files/CMake/bin/cmake

Hi siddjain ,
looks like Cmake can't find a generator although it seems ninja is installed , this is likely a path problem
you can try these steps and see if it solves your problem :
1- download the Ninja binaries from here (ninja-win.zip)
2- Now extract the ninja.exe binary and put in the cmake bin folder  ( C:\Program Files\CMake\bin\ )

3- now you have to add ninja to the path to the system environment variables , there are two ways to do this

start menu > environment variables > Edit the system environment variables > Environment Variables then under System variables -- Path > Edit > New > add path to the CMake executable directory > OK on all windows to save changes

or you can open Cmmand prompt with admin privleges and run this command "setx CMAKE_GENERATOR "Ninja" /M" (remove quotes)

4- now check if the path was set correctly by running this command "where ninja && ninja --version" , it should print the path and version of ninja

5- now close vs code it it's already open and reopen it
6- Reset VScode: Ctrl+shift+P and type "CMake:Reset CMake Tools for Extension State"
7- Configure project: Ctrl+shift+P and type "CMake: Delete Cache and Reonfigure" .

siddjain, could you please try to fix your issue according to the algovoid's comment? If it still reproduces, please share us a demo project with clear repro steps. Thanks.

Is there a way to disable the new check? In my case, ninja is purposely not in the PATH, and is instead set with CMAKE_MAKE_PROGRAM. This didn't cause problems until a few days ago, but now it does.

@gg-sr What version of the extension are you operating on ?

@gcampbell-msft Oops, sorry for not including this information in the first place. I'm on v1.19.49. v1.20.0 (pre-release) appears to have the same problem. v1.18.44 works. Not sure if that makes any difference, but this happens in a devcontainer (I haven't tried it outside of a devcontainer).
Diagnostics (note that this occurs both on VS Code 1.92.2 and on 1.93.0):
{
  "os": "linux",
  "vscodeVersion": "1.93.0-insider",
  "cmtVersion": "1.19.49",
  "configurations": [
    {
      "folder": "...",
      "cmakeVersion": "unknown",
      "configured": false,
      "generator": "unknown",
      "usesPresets": false,
      "compilers": {}
    }
  ],
  "cpptoolsIntegration": {
    "isReady": false,
    "hasCodeModel": false,
    "activeBuildType": "",
    "buildTypesSeen": [],
    "requests": [],
    "responses": [],
    "partialMatches": [],
    "targetCount": 0,
    "executablesCount": 0,
    "librariesCount": 0,
    "targets": []
  },
  "settings": [
    {
      "communicationMode": "automatic",
      "useCMakePresets": "auto",
      "configureOnOpen": true
    }
  ]
}

{
  "os": "linux",
  "vscodeVersion": "1.93.0-insider",
  "cmtVersion": "1.18.44",
  "configurations": [
    {
      "folder": "...",
      "cmakeVersion": "3.28.3",
      "configured": true,
      "generator": "Ninja",
      "usesPresets": false,
      "compilers": {
        "C": "/nix/store/03xd70qibzg98s6m35kq363il7zrcbm2-clang-wrapper-16.0.6/bin/clang",
        "CXX": "/nix/store/03xd70qibzg98s6m35kq363il7zrcbm2-clang-wrapper-16.0.6/bin/clang++"
      }
    }
  ],
  "cpptoolsIntegration": {
    "isReady": false,
    "hasCodeModel": false,
    "activeBuildType": "",
    "buildTypesSeen": [],
    "requests": [],
    "responses": [],
    "partialMatches": [],
    "targetCount": 0,
    "executablesCount": 0,
    "librariesCount": 0,
    "targets": []
  },
  "settings": [
    {
      "communicationMode": "automatic",
      "useCMakePresets": "auto",
      "configureOnOpen": false
    }
  ]
}

Note that cmake is similarly not in the PATH, but instead set with cmake.cmakePath.

@siddjain @gg-sr Do you have a generator defined in cmake.generator or in cmake.preferredGenerators?

@gcampbell-msft I do: "cmake.generator": "Ninja". cmake.preferredGenerators is unset.

@gg-sr Oh interesting, so this issue still reproduces when you have cmake.generator set to Ninja?
I'm currently not able to reproduce that

@gg-sr Are you setting CMAKE_MAKE_PROGRAM to an absolute path?

@gcampbell-msft Yes, "cmake.generator": "Ninja" is set. CMAKE_MAKE_PROGRAM points to an absolute path. I am using the "unspecified" kit.
I managed to fix my local installation, but unfortunately I don't know exactly what did it as I did multiple things at the same time: unset "cmake.generator": "Ninja", configure, reset the extension state, and set "cmake.generator": "Ninja" again.
The following is still printed on project load (after reloading VS Code), but not when configuring, so configuring / building works again:
[proc] The command: ninja --version failed with error: Error: spawn ninja ENOENT
[proc] The command: ninja-build --version failed with error: Error: spawn ninja-build ENOENT
[proc] The command: ninja --version failed with error: Error: spawn ninja ENOENT
[proc] The command: ninja-build --version failed with error: Error: spawn ninja-build ENOENT

I believe that this PR #4026 will solve your issues. We hope to merge by EOD tomorrow, thanks!

To confirm, @gg-sr and @siddjain, could you please download this vsix from the PR here #4026 and confirm that it fixes the issue?
You'll have to modify the extension from .zip to .vsix to install it manually. Thanks!
cmake-tools.zip

I encountered a similar issue. CMake would attempt to use MAKE the default generator even if ninja is in the path and listed by cmake --help.
{
    "version": 2,
    "configurePresets": [
      {
        "name": "RealTime-Debug",
        "displayName": "RealTime-Debug",
        "generator": "Ninja",
        "binaryDir": "${sourceDir}/build/${presetName}",
        "cacheVariables": {
          "CMAKE_BUILD_TYPE": "Debug",
          "CMAKE_INSTALL_PREFIX": "${sourceDir}/install/${presetName}",
          "CMAKE_TOOLCHAIN_FILE": "$env{AzureSphereDefaultSDKDir}/CMakeFiles/AzureSphereRTCoreToolchain.cmake"
        },
        "environment": { "BuildAllBuildsAllRoots": "true" },
        "vendor": {
          "microsoft.com/VisualStudioSettings/CMake/1.0": {
            "intelliSenseMode": "linux-gcc-arm"
          }
        }
      },
      {
        "name": "RealTime-RelWithDebInfo",
        "displayName": "RealTime-RelWithDebInfo",
        "inherits": "RealTime-Debug",
        "cacheVariables": {
          "CMAKE_BUILD_TYPE": "RelWithDebInfo"
        }
      },
      {
        "name": "RealTime-Release",
        "displayName": "RealTime-Release",
        "inherits": "RealTime-RelWithDebInfo",
        "cacheVariables": {
          "CMAKE_BUILD_TYPE": "Release"
        }
      }
    ]
  }

I'm using a devcontainer based on Ubuntu and I was able to build manually but not using the extension.

@siddjain @gg-sr @lit-af
Could you test with this vsix and let me know if this resolves your issue?
cmake-tools.zip

@siddjain @gg-sr @lit-af
Could you test with this vsix and let me know if this resolves your issue? cmake-tools.zip

I'm trying to install and test it using: code --install-extension path but it won't work. I don't see a .vsix file anywhere. Could you please provide further instructions on how to test please?

Sure, please re-download the zip as I just updated it, you can download from here:
cmake-tools.zip
, you'll need to first modify the file extension from .zip to .vsix, then you can perform the step you mentioned I believe.

I tested it and I can confirm that it's working on my end.

The fix for this issue will be released in the next hour or so in v1.20.2.

I am also hitting the same issue (running on RHEL) and can confirm that the most recently linked cmake-tools vsix^1 fixes the issue. In my case I set cmake.environment PATH (and LD_LIBRARY_PATH) for it to find the various tools that I'm using.
Looking forward to the fix in v1.20.2!


Fix "profile sample" casing

-Changed entries to lowercase in nativewifi/wireless-profile-samples and children pages
-Removes empty bullet point entry in wireless-profile-samples

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


Fixed subtitles path with spaces

When given a subtitles path with spaces the players failed to find the files.
Not tested on Windows as I don't have a Windows machine.

Thanks!

Released as 1.2.5.


Modularisation

Modularise this project so that, any other project could use the scanning/parsing capabilities of mendable.

Altho, this CLI tool has a niche use case, the underlying scanning and parsing logic is generic enough that it could be utilised to build any other tooling around the generated Jetpack Compose Compiler Metrics.
I think this project could be modularised into the following modules:

scanner : A module to host all of the logic involving scanning of the file system to find the metrics files
parser :  A module to host all of the logic involving parsing of the files found in the scanning phase
model : (If required) As the scanner & parser modules have been separated, some common models would be required for these two modules to communicate with each other. This module will host those.
mendable : This module would be the orchestrator, which would depend on the modules mentioned above and consolidate them into a simple API.
mendable-app : This module would just consume the mendable module and make it a CLI app.

Also, tests written so far would also be needed to be split and moved into respective modules.

With the changes done so far, all the things the things mentioned in the issue description are achieved as is.
All the mentioned modules (5) are created with the mentioned respective responsibilities.


let dependabot handle github actions updates

lately several github actions got updated and older versions will be deprecated.
dependabot can handle theses updates so let him do his job ;)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
208

Relevant Lines:
222

💛  - Coveralls

Good suggestion, thanks @bakito.


Server crashes when used in headless app

I created a new Solution (Windows IoT - Background) added a class library (to host my controllers) and added restup via NuGet to the class library.
In the Background app I did this:
public sealed class StartupTask : IBackgroundTask {
	private RestWebServer _webserver;
	public void Run(IBackgroundTaskInstance taskInstance) {
		BackgroundTaskDeferral deferral = taskInstance.GetDeferral();
		_webserver = new RestWebServer(8800);
		_webserver.RegisterController< DummyController>();
		ThreadPool.RunAsync(workItem => {
			_webserver.StartServerAsync();
			//server.Start();
		});
	}
}

Further I enabled Client+Server in the manifest.
Runnig this on my RapsBerry brings up the server (Message in Output).
When I make a request the background app silently (no exceptions) terminates.
Doing the same on a headed app works as expected.
I attached the sample project - maybe this helps to better understand what I'm doing.
BackgroundApplication1.zip

I'll take a look at it. I already got it working in headless mode, so it should be possible.

Maybe I just missed some setting / ref.... in my solution.
I like your project and really would like to use it.

I'm working on a headless sample, hang on.

I'm very sorry for this...
Just checked my code one again - and found the problem.
I did:
public sealed class StartupTask : IBackgroundTask {
private RestWebServer _webserver;
public void Run(IBackgroundTaskInstance taskInstance) {
BackgroundTaskDeferral deferral = taskInstance.GetDeferral();
_webserver = new RestWebServer(8800);
...
The problem here is that deferral is a local variable.
And (for some reason) this thing was alive till I made a call to the WebServer.
Several assemblies got loaded - which must have triggered GC - and this (of course) remove the deferral - shutting down the app.
A simple change to
public sealed class StartupTask : IBackgroundTask {
private RestWebServer _webserver;
BackgroundTaskDeferral deferral;
public void Run(IBackgroundTaskInstance taskInstance) {
deferral = taskInstance.GetDeferral();
_webserver = new RestWebServer(8800);
...
fixed everything - I confirm that WebServer is running in headless applications without problems.

Thanks for the info and I keep it open, because I'm doing the same thing you did with the deferral. Will fix it in the demo app.

Fixed in branche

Do you have a timeline when you merge / update nuget?

Merge already happened and the nuget package is updated to beta1 (indexing now, so could take a few hours before you can install).


Pool creation for users who are not connected

Users who haven't connected their wallets have an access to pool creation flow which ends up with a complicated error message. In order to improve user experience, is it possible not to allow users who haven't connected their wallets to be able to progress in pool creation flow? I would change main button to "connect a wallet" on the first step.
Zeplin: https://app.zeplin.io/project/5d70f6905eefcd18e1c40502/screen/5f4f320c8fe8fdb1a8fa0868/.
Thanks!

@katyarozenfeld thanks for bringing this to my attention.
On swap we open the login modal when a user clicks the button but isnt logged in. To have a consistent UX we should do either or for all action buttons. My suggestion is to open the login modal. What are your thoughts about that?

Yes, I think consistently displaying login modal on early steps when user wants to continue with a specific task is a good solution. The intention is not to allow users to progress within specific flow before they connect their wallets in order to avoid frustration. Thanks!

fixed.


Diagram starts to freeze when it gets larger

The drawio is an excellent tool. I really recommend and appreciate it. It has helped a lot in drawing many block diagrams, flowcharts, etc.  However, I am facing some annoying freezing issues. When I initially create new diagrams, it looks fine. I can quickly make diagrams.  But when my diagram gets larger, it starts to freeze and does not respond quickly. If I select or add or move blocks (rectangle, circle, etc...), draw lines, etc., the entire diagram freezes for 2 to 3 secs or more sometimes. It has reduced my productivity. I am still looking solution for such an issue.
I am currently using the 21.2.9 version in Chrome browser (Version 105.0.5195.125). I used to sync up my design using Google Drive but later, I started to save on the local device. I thought it would solve the freezing issue but it is still right there.
I also followed this information https://www.drawio.com/blog/clear-diagrams-net-cache but still having the problem.

Browser extensions? System specifications? Diagram that recreates the problem?

CDMA
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Nikil Thapa @.>
Sent: Tuesday, May 9, 2023 5:51:47 AM
To: jgraph/drawio @.>
Cc: Subscribed @.***>
Subject: Re: [jgraph/drawio] Diagram starts to freeze when it gets larger (Issue #3571)
I am a little bit confused about browser extensions. You mean drawio chrome extension? No, I don't have it.
I am using Ubuntu 20.04.5 LTS. I have the following system specifications
Architecture: x86_64
CPU(s): 4
Thread(s) per core: 2
Core(s) per socket: 2
Socket(s): 1
Vendor ID: GenuineIntel
Model: 61
Model name: Intel(R) Core(TM) i5-5200U CPU @ 2.20GHz
CPU MHz: 700.000
CPU max MHz: 2700.0000
CPU min MHz: 500.0000
BogoMIPS: 4389.81
Virtualization: VT-x
L1d cache: 64 KiB
L1i cache: 64 KiB
L2 cache: 512 KiB
L3 cache: 3 MiB
I cannot share the diagram that gave me such a freezing issue. But, I have a screenshot of it.
[2023-05-09_14-29]https://user-images.githubusercontent.com/81352102/237055101-cc97784f-6105-4cde-93db-ab56062089d6.png
—
Reply to this email directly, view it on GitHubhttps://github.com/jgraph/drawio/issues/3571#issuecomment-1539776895, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AO7MXRWJ5Z3Y3M7HUO45TUTXFIHTHANCNFSM6AAAAAAXYYBUOI.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

exmple freezes.zip
this is example of one of the smaller files with less freezing behaviour, some of the others I have are ie 6 pages and it is very frustraing. Sadly I still did not find any alterative do draw.io. This one is free to boot, so please do not take it as complaining, I'm rather looking for suggestions.
Ie. to not use pictures or make pictures much smaller before putting them to the diagram
that i can do differently or even better for a way to get rid of the freezes while using the SW as it is now.

Able to repro with this diagram:
https://drive.google.com/file/d/1ELjT0wOBBjzQXtIY8trRNmSLu-WMTH2G/view?usp=sharing

The issue is going to be autosave taking a long time because of the size of the diagram and/or the transmission speed of saving.
"exmple freezes" I cannot recreate the issue.
The last file on Google Drive, I can recreate. That's a 39MB file being saved repeatedly, yeah, that's going to be slow for most people. There is a very beta tool at https://github.com/jgraph/drawio/tree/dev/etc/imageResize. It performs resizing by image width value or image width percentage resizing.
You could add a console.log that tells you the size of each image, also.

We've an improvement in 24.1.0 specifically for diagrams with lots of images.
I'm going to close the issue up, as we might have a few issues tied into one in this thread. If you do still experience problems please open a new ticket with version, environment and sample diagram (i.e. follow the template).


Allow changing 'Free' label in the extension gallery

In Visual Studio Code marketplace/gallery all extensions are currently labeled as 'Free':

However ours is not free and requires a license. Is there a field in package.json or some other way to change the label?

@joaomoreno @seanmcbreen do you know? I'll also check with the market place team.

We had some discussion on this. Our conclusion was that the wallaby extension is free, it is the backend service that requires a license. So the market place shows the right information about the extension. Our suggestion is to clarify this in the Readme.md. You can also try to add a hint in the description in the package.json.
My understanding that support for paid extensions is on the backlog of the market place.

Thanks Erich, makes sense. Will update the readme file.

Though it'd be great to be able to specify a 'Trial' label for the extension like they can in Visual Studio marketplace:

@ArtemGovorov fyi we have added support to label an extension with preview https://github.com/Microsoft/vscode-vsce/issues/82#issuecomment-196791767

@egamma Great, thanks for the update!


Add the OpenSSF Scorecard GitHub Action

Fixes #52043.
As per the linked issue, this PR adds the OpenSSF Scorecard GitHub Action. This Action serves as a "meta-analysis" of a repository's security posture and adds objective suggestions to the Security Dashboard whenever it finds something that can be improved.
I can already say that the project's score of 6.5/10 puts it in the top 20% of important projects and top 2% of GitHub projects overall.
I have also taken the liberty of adding a badge to the README. However, this badge is strictly optional and I'll be more than happy to remove it, just say the word!

@microsoft-github-policy-service agree [company="Google"]

@microsoft-github-policy-service agree company="Google"


"This shop already exists" error for a new store

When I signed into a new account for the first time, clicked "add store" and was prompted to name my shop, I received the error "this shop already exists" for the first two times, and it only went through on the 3rd try.
However when I logged in again, all 3 stores had been created under the account.

Setting priority to P1 since that impacts new users and could be a very bad first-time experience.
@gullible1 Was there any error in the browser javascript console?

Fixed in next deploy


Allow for noisy observations in LSS class

Added the H matrix like here.
It's optional with default argument 0, so it shouldn't change any existing code.

Codecov Report

Merging #167 into master will increase coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #167      +/-   ##
==========================================
+ Coverage   90.29%   90.31%   +0.02%
==========================================
  Files          23       23
  Lines        1442     1446       +4
==========================================
+ Hits         1302     1306       +4
  Misses        140      140

Impacted Files
Coverage Δ

src/lss.jl
100% <100%> (ø)
: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 dbb35ef...6f6f3eb. Read the comment docs.

@vgregory757 Thanks! It's great to have this functionality and the code looks good.


CPLP-1212 services

In this PR the Tasks CPLP-1214 CPLP-1215 CPLP-1216 CPLP-1213 CPLP-1413 & CPLP-1427 are resolved.
Please review the database changes in PR as well

@ntruchsess in my opinion it will make sense to finish the #183 first. The Vulnerability, Security Hotspot and duplication will disappear than.

ok - I'm done so far, but in the devenv-PR https://github.com/tractusx-team-portal-onboarding/devenv/pull/114 we have 'id uuid DEFAULT gen_random_uuid() NOT NULL,' in table 'offer_subscriptions'. This auto-creation of id is not configured in other regular entities and I cannot find why it is the case here.
As one can see in the Migration Designer-files all regular entities have ValueGeneratedOnAdd() set, but in the database-export this apparently did not create the default value creation as it does in offer_subscriptions. Any idea why this is the case?

ok - I'm done so far, but in the devenv-PR tractusx-team-portal-onboarding/devenv#114 we have 'id uuid DEFAULT gen_random_uuid() NOT NULL,' in table 'offer_subscriptions'. This auto-creation of id is not configured in other regular entities and I cannot find why it is the case here. As one can see in the Migration Designer-files all regular entities have ValueGeneratedOnAdd() set, but in the database-export this apparently did not create the default value creation as it does in offer_subscriptions. Any idea why this is the case?

As we found out, came with an previous migration. It's removed now.


ALL_FACE contour positions wrong on iOS

Same flutter code, using the same contour indexes (based on official doc) - iOS clearly has an issue. The code worked fine with the dead firebase_ml_vision plugin.
The code below looks okay to me, since it's pretty much the same as on Android. But I guess there is some issue with the contour parts 🤔 order.https://github.com/brianmtully/flutter_google_ml_vision/blob/e8dfedc8335fdd4bde861008cf290df6523f94e9/ios/Classes/FaceDetector.m#L123

Android
iOS

Can you provide your drawing function for the contours that you are using?

Sure, here is a gist with the CustomPainter and snippet of its usage, should be easy to add into the plugin sample.
https://gist.github.com/shliama/f1ae6add1c879a0b0404e91fc1ae2f39

So after further inspection, the contours array returned on iOS are not in the order that you are expecting. The iOS MLKit doesn't seem to follow the order from the documentation when I grab all of the contours of a face. In order to get the result you expect I probably have to go one by one and input them into the resulting array.
allPoints isn't actually a contourType if you look through the documentation. I am just pulling all the points by looping through all the actual contours.

Changed the code based on your idea - https://github.com/brianmtully/flutter_google_ml_vision/pull/16
Tested with my mask and everything looks perfectly aligned now between iOS & Android.

thank you, I merged your pull request.


Update to latest mixer api with gogoproto

Jenkins job mixerclient/presubmit passed

Jenkins job proxy/mixerclient-pr passed

Jenkins job proxy/mixerclient-pr passed


Can't run as root

--- crash start ---
[command] logcat -b crash -t '09-27 18:55:40.000' -d
--- crash end ---
--- brevent start ---
[command] logcat -b main -t '09-27 18:55:40.000' -d -s BreventLoader BreventServer
--- brevent end ---

黑域版本号？


Equivalent for sub_entity?

I use it for a sensor, but maybe it can be applied to other widgets types.
Is there a similar function?

You can use state for that. Here is an example of how I'm showing my  gas usage with the cost from a different sensor:
                  {
                     position: [1, 2],
                     type: TYPES.SENSOR,
                     title: 'Gas usage',
                     id: 'sensor.gas_usage_this_month',
                     value: '&sensor.gas_usage_this_month.state',
                     state: '&sensor.gas_cost_this_month.state',
                     unit: 'kWh',
                     filter: numberFilter(1)
                  }

I assume we can close this one.


Add vision GAPIC

Support for doc generation coming

Is the TOC incorrect, or the gapic client name?

@jdpedrie Good spot, the TOC was incorrect, updated.
@dwsupplee I don't think we need anything else.

LGTM!


No such member errors for ic2 & mekanism

Intro:
I'm trying to add/modify some recipes for mekanism and ic2 and I'm getting No such member: ... errors for any import statements I attempt (modeled after the modtweaker wiki pages since you don't have any examples besides JEI for the 1.12.x wiki.
What happens:
No such member: ... errors
What you expected to happen:
I expected the imports to work, recipes to be usable, etc.
Script used
https://pastebin.com/x4ut2LHt
The errors are being thrown starting from the imports
import mods.mekanism;
import mods.jei.JEI.removeAndHide as hide;
import mods.ic2;

Minetweaker.log file (Please Pastebin or gist your file, posting an unpasted or ungist'd file will automatically close this issue):
Output of /ct mods and /ct syntax
https://pastebin.com/rWd0jyiN
Where the issues are starting
[SERVER_STARTED][SERVER][ERROR] recipeTweaks.zs:1 > No such member: mekanism
[SERVER_STARTED][SERVER][ERROR] recipeTweaks.zs:1 > Not a valid type
[SERVER_STARTED][SERVER][ERROR] recipeTweaks.zs:3 > No such member: ic2
[SERVER_STARTED][SERVER][ERROR] recipeTweaks.zs:3 > Not a valid type

Affected Versions (Do not use "latest"):

Minecraft: 1.12.1
Forge: <IP_ADDRESS>5
Crafttweaker: 1.12-4.0.6
Mekanism: 9.4
IC2: 2.8.16-ex112

We don't add support for either of those.

Mekanism and IC2 support is not updated yet unfortunately.


@angular/material entry-point causes very slow builds with Ivy

Reproduction
Steps to reproduce:
yarn global add @angular-cli@next
ng new foo --enable-ivy
ng add @angular/material
ng generate @angular/material:nav my-navbar
ng build

Expected Behavior
What behavior were you expecting to see?
ngcc will compile just a handful of entrypoints needed for the navbar. This should take just a few seconds.
Actual Behavior
What behavior did you actually see?
ngcc compiles all entrypoints within the @angular/material package as well as some/all entry-points within @angular/cdk package.
This delays the initial build by as much as 2 minutes.
Environment

Angular: 8.1.0-next.0
CDK/Material: 8.0.2
Browser(s):
Operating System (e.g. Windows, macOS, Ubuntu):

Proposed solution

update schematics to use deep secondary entry-points instead of the main entry point
import { MatToolbarModule, MatButtonModule, MatSidenavModule, MatIconModule, MatListModule } from '@angular/material';
execute migration to secondary entry-points and remove the @angular/material entry-point

The schematics have been already updated to use the secondary entry-points. For reference: https://github.com/angular/components/commit/b823dbd8b66db47e5f7d0a646020e9f77e87c270

that's great! do we have a plan for migrating all of the existing code to these entry-points?

The current state is that nothing in the components repository uses the primary entry-point anymore.
With the deprecation of the primary entry-point in version 8, we also provided a migration schematic that updates existing CLI projects to the new imports.
The plan is to remove the primary entry-point in version 9 (according to the changelog. see here)

The related deprecation guidelines were discussed in today's sync.

Deprecation period: When an API or a feature is deprecated, it will still be present in the next two major releases. After that, deprecated APIs and features will be candidates for removal.

Perhaps since these are not changes to the APIs and only changes to the paths of the imports, this may not require 2 majors before removal?

Yeah it's quite ambiguous as the changelog says that we'll do it in version 9. I don't see any problem with doing that in version 9 as we deprecated in version 8 and provided a migration schematic.
Regardless of the deprecation guideline, this would definitely also help with Ivy and version 9. Doing it  in version 10 (as the deprecation guideline proposes) will probably result in bad experience w/ Ivy.

Closing this because the schematics and examples have already been changed, the top-level entry-point is deprecated as of version 8, and we can't remove it sooner while respecting Angular's breaking change policy (which you authored :wink:).
Might be worth seeing if someone wants to add a lint check for codelyzer for it separately.


minidump_dump: Support dumping the stack as 32/64bit words

This would make it a lot easier to read.
i.e. something like
@@ -1518,22 +1519,22 @@ Memory
 ",
             self.desc.start_of_memory_range, self.desc.memory.data_size, self.desc.memory.rva,
         )?;
         self.print_contents(f)?;
         writeln!(f)
     }

     /// Write the contents of this `MinidumpMemory` to `f` as a hex string.
     pub fn print_contents<T: Write>(&self, f: &mut T) -> io::Result<()> {
         write!(f, "0x")?;
-        for byte in self.bytes.iter() {
-            write!(f, "{:02x}", byte)?;
+        for bytes in self.bytes.chunks_exact(8) {
+            writeln!(f, "{:016x}", u64::from_le_bytes(bytes.try_into().unwrap()))?;
         }
         writeln!(f)?;
         Ok(())
     }

     pub fn memory_range(&self) -> Option<Range<u64>> {
         if self.size == 0 {
             return None;
         }
         Some(Range::new(

Seems fine and a good starter given the OP basically include a patch already.
I'm assuming this is "standard" for a hexdumper (don't use them enough to recall).


Add dynamic condition to inject or not awarness_config block

The solution is to dynamically inject awarness_config block based on the variable.
Sources: https://www.hashicorp.com/blog/hashicorp-terraform-0-12-preview-for-and-for-each/#dynamic-nested-blocks
Fixes #40

This is exactly what I need! I have this patch applied on my tree and can confirm it's working.

Would be great to get this merged!

Great PR. Would be good to get in?

Here we go! v1.3.0 of this module has been just released.

Feels good to see this
No changes. Infrastructure is up-to-date.


estimate time in the issue conflict

When the plug-in installed, the estimated time field in the issue has conflict with the pug-in, even if the estimated time is not blank,when creating the issue, there is message said: the estimated time is blank.
and I found redmine speed is slow when the plug-in installed.

Can you send some screenshots ? I am not sure if I understand what you mean.


chore: add data-table, dropdown, and date-picker

The data table knobs are working correctly now 🎉 👍  the selection is still not working however. It looks like the row isn't getting the class it needs when it's selected. I wonder if this is because it's added via prop... but I can't figure out where that prop is updated...
  const className = cx(props.className, {
    [`${prefix}--data-table--selected`]: props.isSelected,
  });

https://github.com/carbon-design-system/carbon/blob/e78bbb5cc6321c54eecd67af1d26c3bef77ad68e/packages/react/src/components/DataTable/TableRow.js#L20
expected:

actual:

@jnm2377 looks like I was missing a prop-getter for the row, fixed now.


Convert failed

When I convert nnet3 model, it came up this problem.
Seems that the converter cannot handle  component correctly.
`
Traceback (most recent call last):
File "/home/kaldi-onnx/converter/utils.py", line 268, in read_matrix
f = float(tok)
ValueError: could not convert string to float: ''
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "converter/convert.py", line 364, in
main()
File "converter/convert.py", line 324, in main
onnx_model, configs, trans_model = converter.run()
File "converter/convert.py", line 81, in run
self.parse_configs()
File "converter/convert.py", line 173, in parse_configs
self._components, self._transition_model = parser.run()
File "/home/kaldi-onnx/converter/parser.py", line 617, in run
self.parse_component_lines()
File "/home/kaldi-onnx/converter/parser.py", line 1073, in parse_component_lines
component_type)
File "/home/kaldi-onnx/converter/parser.py", line 1109, in read_component
action_dict)
File "/home/kaldi-onnx/converter/parser.py", line 1148, in read_generic
obj, pos = func(line, pos, line_buffer)
File "/home/kaldi-onnx/converter/utils.py", line 273, in read_matrix
.format(sys.argv[0], pos, tok), file=sys.stderr)
File "/usr/lib/python3.5/logging/init.py", line 1308, in error
self._log(ERROR, msg, args, **kwargs)
TypeError: _log() got an unexpected keyword argument 'file'
`

I have exactly the same problem. read_matrix() in utils.py got stuck in output.affine layer. Did you solve the problem?


Improve support es6 features for node v4

Background
I've used MeteorJS framework where the version of node is 4 and it can be upgraded only by framework's developers. And I found your files don't work properly with node v4.
It throws error "regeneratorRuntime is not defined"
I prepared changes to supporting node v4.
Changes

Removed old package babel-polyfilland install new plugin that works properly with async\await
Recompile files

Hi! Thanks for the great PR! Unfortunately didn't see the notification about it, so didn't respond earlier.

Published as v1.1.5

@yarax Cool :+1: Attention please on another your library -- https://github.com/yarax/node-request-by-swagger/pull/1

Yep, merged. The new dependency is included to v1.1.6


Update virtual environment on the Jülich HPC-systems

The virtual environment based on Stages/2022 runs into several version issues with the Python packages metpy, setuptools and typing_extensions. To solve the issues, an update to Stages/2023 is performed which also includes that Tensorflow version 2.11 will be used in the future.

Some issues when importing metpy (version: 1.4) persist in conjunction with the typing_extensions-package.
In particular, the typing_extensions-version as part of the GCCcore/11.3.0-software module does not provide the deprecated-decorator.
However, since this decorator is just used to raise warning for future deprecation in the code of the pint-issue (part of the virtual environment), the issue can be bypassed as follows.

remove from typing_extensions import deprecated-statement in pint/compat.py
remove the import of the deprecated-decorator and its usage in the following Python-scripts of the pint-package under [...]/lib/python3.10/site-packages/pint/facets/plain:
- quantity.py
- unit.py
- registry.py


Please upload the code

Up

It's great to have cleaned up code.  Even if it's not so clean code it's better than no code. :-)

It's great to have cleaned up code.  Even if it's not so clean code it's better than no code. :-)

I would by happy to clean it up myself tbh, but it has to be open-sourced for that

Hey @kksinghal @MathisFederico @watts4speed thank you for the patience and for the nagging! 😄
I've uploaded the code now - let me know if you run into any issues!


Remove individual params

Related PR: https://github.com/tier4/autoware_launcher.iv.universe/pull/100
Remove package and move to a different repository.

The composition is good.
I don't have the environment for ros2, so I haven't been able to confirm that it works.

@yukkysaito Thank you! I think it's okay that we merge this now and that if there is any problem we'll fix it.
We'll test this on a real vehicle this weekend.


Move sign up form into app from static site

Related Issue or Background Info

Resolves #2045

This was very much a team effort with @zdeveloper and I!
Changes Proposed

At long last, ditch the sign up form in the static site and implement it in the app frontend
Form contains PROPER validation using yup. No more submissions with invalid phone numbers

Additional Information

On success, instead of redirecting back to the thanks page on the static site, I added a Thanks component
Once Experian goes live, we will redirect to the Experian flow instead

Screenshots / Demos

Checklist for Author and Reviewer
UI

[x] Any changes to the UI/UX are approved by design
[x] Any new or updated content (e.g. error messages) are approved by design

Testing

[x] Includes a summary of what a code reviewer should verify

Changes are Backwards Compatible

[ ] Database changes are submitted as a separate PR

[ ] Any new tables that do not contain PII are accompanied by a GRANT SELECT to the no-PHI user
[ ] Any changes to tables that have custom no-PHI views are accompanied by changes to those views
(including re-granting permission to the no-PHI user if need be)
[ ] Liquibase rollback has been tested locally using ./gradlew liquibaseRollbackSQL or liquibaseRollback

[ ] GraphQL schema changes are backward compatible with older version of the front-end

Security

[ ] Changes with security implications have been approved by a security engineer (changes to  authentication, encryption, handling of PII, etc.)
[ ] Any dependencies introduced have been vetted and discussed

Cloud

[ ] DevOps team has been notified if PR requires ops support
[ ] If there are changes that cannot be tested locally, this has been deployed to our Azure test environment for verification

https://github.com/CDCgov/prime-simplereport-site/pull/207 will point the static site to this new form

@nickclyde

Looks like there's a period missing at the end of the email/sentence in the success message (make sure the period is not hyperlinked).
For the org name we have helper text that I think can now be removed (we've taken out the questions for the testing facility name and info. from this form, correct? If so we can remove that text).
The sign up header at the top should be a larger size since it's basically the title of this form (compare size to the org admin section below, which should be smaller in comparison).

Thanks!

@Rebecca-LM @emmastephenson Alright, made the suggested copy changes and updated the screenshots above!

Update: got rid of the Thanks component and changed form to redirect to the Experian flow, since it just went live!


build xbb-3.4 on mac os

The system environment is mac10.13.6, and the xcode version is 10.1
This command executes normally,
caffeinate bash "${HOME}/Work/xpack-build-box.git/macos/build-xbb-bootstrap-v3.4.sh
The following error occurs when executing this command
command  :
caffeinate bash "${HOME}/Work/xpack-build-box.git/macos/build-xbb-v3.4.sh"
error:
cp: cannot lseek '/System/Library/Frameworks/Tk.framework/Resources/Wish.app/Contents/MacOS/Wish': Inappropriate ioctl for device
make[1]: *** [Makefile:194: Git Gui.app] Error 1
make: *** [Makefile:2073: all] Error 2
Can you give me some advice？ thanks

I'm not sure, but it might require the Command Line Tools, not Xcode.
Anyway, why do you need v3.4?
It was deprecated long time ago, and the current build scripts no longer use it.

My goal is to build riscv none elf gcc xpack v12.2.01. When running the script, it shows that an XBB environment is required. Therefore, I used the script build xbb v3.4. sh in xpack build box/mac os. If this is an error, what should I do? Thank you and look forward to your reply

v12.2.0-1 was indeed using the old XBB environment :-(
The reason this environment was replaced by XBB v5.x was exactly the complexity and the difficulty to create and manage it.
If you really want v12.2.0-1, try to remove xcode and install the command line tools. But I cannot guarantee it'll work, you need an absolute pristine environment, without anything else installed, like HomeBrew or MacPorts.
If your need is v12.2.0, I suggest you go for the latest v12.2.0-3, which uses the new build environment:

https://xpack.github.io/xbb/prerequisites/

An even better choice would be v12.3.0-1, just released, or even v13.2.0-1.

Thank you very much. I'll give it a try.

Just curious, why do you need to rebuild the toolchain and not use the provided binaries?


.dll are not in export

Hello,
I have a problem. Whenever I try to build the project, the resulting application is missing the dlls. After manually copying everything works. However, I would like to use the compilation into a single exe file or apk, which is not possible now. (GODOT 4.2.2)

Hi, have you tried including .dlls in the export settings?:

I'm currently on Linux or I'd just test that. I'll add it to the README if that works since I could see how people might not know that. Godot only exports specific files (Like .res, .tres, etc) by default, and you have to tell it to do the rest.

I didn't know about this option. However, unfortunately it didn't help.
The problem is the missing "libvlc" folder. The strange thing is that it is in the temporary files, together with the other .dll files that appear in the export. All .dll files including some VLC libraries will appear in the project data. But the "libvlc" folder does not.

can you add libvlc/* to that same section as well if it's missing the folder?

Ok, while not pretty, I was able to get it to work by manually copying the nuget package and renaming the x64 folder to win-x64. On Linux with Wine, it seems path name need to begin with a Z: (and isn't perfect)

I've tried everything but nothing works. It seems that Godot simply ignores all .dll files that are not part of its Mono version. It is strange that after compilation a functional library appears in the project in the tmp folder, but godot does not export it. By the way, the folder with VLC is already named win-x64 and win-x86.
I can't think of anything wrong, most likely the error is directly in GODOT.

I downloaded GODOT 4.3 today and tried exporting the project in it. Everything works, the libraries have been transferred correctly. So the bug was really in the older version of GODOT and it is now fixed by them.

I downloaded GODOT 4.3 today and tried exporting the project in it. Everything works, the libraries have been transferred correctly. So the bug was really in the older version of GODOT and it is now fixed by them.

Awesome! Sorry my best temporary solution was to just copy the VLC library into the exported project's folder. While it worked and could be automated with a command line script export, it still wasn't an ideal solution. When I published this repo, I figured with the licensing constraints of using libVLC and how limited the implementation I got working was, that it would just save others time, either if they chose to expand on this for something more usable in some end product, or if they chose to go a different route with how the drawing target was done, they were aware of a possible way it was done with HWND.


Added FpdfRenderer

This renderer adds the ability to render into a PDF vector like format, it requires a FPDF(like) object.

This is pixel perfect in print, and in most viewers (Acrobat, Sumatra). But some browsers might display tiny white lines.

Thank you!


[KYUUBI #1032][FOLLOWUP] Refine session hande in AbstractSession

Why are the changes needed?

Remove session hande in AbstractSession
How was this patch tested?

[ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Codecov Report

Merging #1046 (1d7543a) into master (347c932) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #1046      +/-   ##
============================================
- Coverage     78.44%   78.43%   -0.01%
  Complexity       61       61
============================================
  Files           167      167
  Lines          6202     6201       -1
  Branches        724      724
============================================
- Hits           4865     4864       -1
  Misses          914      914
  Partials        423      423

Impacted Files
Coverage Δ

...la/org/apache/kyuubi/session/AbstractSession.scala
95.40% <ø> (-0.11%)
:arrow_down:

...kyuubi/engine/spark/session/SparkSessionImpl.scala
100.00% <100.00%> (ø)

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 347c932...1d7543a. Read the comment docs.

merged to master


[ServiceBus] [Azure Functions] Fix early executions cancelling on listener stop

During closing ServiceBus message processor new messages can still be delivered to the processing handler but we already canceled the token which is used in execution pipeline before closing the processor. It causes multiple deliveries and TaskCanceledExceptions due:
azure-functions-host/GrpcWorkerChannel.cs at 428215c25052dfde38e66fd064b9981dffbda562 · Azure/azure-functions-host (github.com)
CRI: 381018457

API change check
API changes are not detected in this pull request.

Replaced by https://github.com/Azure/azure-sdk-for-net/pull/38063


skip training tasks where human demonstrator gave a bad input

i think this is easier and cleaner than the individual task saving idea in #946, and so we should only do that more complicated thing if we really find it necessary in the future.
closes #946

i'm fixing that in a separate PR. my plan is to make changes here: https://github.com/Learning-and-Intelligent-Systems/predicators/blob/master/src/main.py#L199
to search for other amounts of data first before giving up entirely.


Create an installer script

This PR adds an installer script and changes the shebang from /usr/bin/python2 to /usr/bin/env python2 for systems where python may be installed in other locations, e.g. FreeBSD / OSX.

Nice, thanks!


chore: migrate api node-sass to dart-sass

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Proposed changes
Migrate from legacy api (node-sass) to new api (dart-sass).
https://sass-lang.com/documentation/js-api/#md:legacy-api
Effectively only replaces the function call.
Reference
#4723
Checklist

[x] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the forum/slack team.
[x] I have run npm run test to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

#3743 does that (and some other stuff). Should I close it in favor of this one?

@bonanitech as you're probably the main user of the script, I'd value your input on which PR we should proceed with. We don't need both, but if you feel your PR addresses more items you'd like to see addressed, then we can go with that.

Thank you @knolleary
To be honest I don't use it anymore. I have one in the theme-collection repository.
But I think #3743 can benefit other people trying to create themes. With the changes made in that PR, the script will generate a file in the format Node-RED uses nowadays, containing only the CSS variables.

Thanks @bonanitech - we'll close this one in favour of your other PR.


Just a question

I try to understand the translation process. Getting contents from the connector seems to be a little bit complicated and redundant and I am not sure if I understand that.

I have an API to get the the identifiers from /cache, aka

UniqueId
GroupId

UniqueId must be unique so it very likely just something like ${GroupId}_{FieldName}

Next you make an API call with /cache/items, here I have to define

GroupTitle
Title
Every extra field I want to show to the user.

Next you get the actual content from items with /translate ... is this correct?

Technically this is just 3 times the same call for me (more or less) and I just do not get why. When do you call which endpoint?

@andrewLab-lokalise We should add a Mermaid.js diagram for the expected flow.

Hey @SebastianStehle !
That is right! The idea behind cache endpoint is just to list the items, without any extra data. Usually just using the listing API’s.
The cache items is used to get more details, so it might require extra calls to the API and we can chunk these requests not to reach API limits.
translate is used to get translations selected by the customer
Hope this makes it clearer :)


Updating snapshots resets --watch state

🐛 Bug Report / To Reproduce

run tests in --watch mode.
press a to run all tests.
press u to update snapshots.

a option is forgotten.

Expected behavior
Should not forget watch mode options after updating snapshots.
Run npx envinfo --preset jest
  System:
    OS: Linux 4.13 Ubuntu 17.10 (Artful Aardvark)
    CPU: x64 Intel(R) Core(TM) i7-8550U CPU @1.80GHz
  Binaries:
    Node: 9.5.0 - ~/.nvm/versions/node/v9.5.0/bin/node
    npm: 6.0.0 - ~/.nvm/versions/node/v9.5.0/bin/npm
  npmPackages:
    jest: ^22.4.3 => 22.4.3

This was fixed in #5696


Support saving/loading for zip file

Overview
It's part of specs implementation requirements:

zip a Data Package descriptor and its co-located references (more generically: "zip a data package")
read a zip file that "claims" to be a data package
save a zipped Data Package to disk

http://specs.frictionlessdata.io/implementation/

My 2c: this sort of stuff should not go into this core lib - we'll get really bloated. Instead we have help libraries to do writing / serializing of data package to specific forms.
Also i didn't realize this was part of implementation guide. We don't yet have an agreed spec for compressing or bundling - https://github.com/frictionlessdata/specs/issues/132 + https://github.com/frictionlessdata/specs/issues/290. I think it best to get those complete before people start implementing.
/cc @pwalsh as czar of implementations

@roll what happened re my comment on this? /cc @vitorbaptista

@rufuspollock
It's a part of the specs implementation requirements - https://frictionlessdata.io/specs/implementation/. At the moment datapackage implementations on 8 languages support saving/loading a data package to zip. So this implementations just follows the general approach.
Also it's a key feature for the project we work now with our partners.
I would say bloating is a problem only for client-side JavaScript. Server-side libraries don't really sensitive for this like this. So if datapackage-py could load/save zip there is no reason to remove this capability. It's good for users and don't really affect installation speed/size etc.
And for the client-side JavaScript lib we just could do bundle-size optimization/splitting. I've done a lot of profiling and for now I would say the most problematic size-wise is shipping profiles not deps like this.

@roll my point was there was no response to the comment left on 13 sep until you shipped the feature now. In my original comment i pointed out that i did not understand why this was part of the implementation specs when it is not part of the spec itself.
But the real point is about how we factor this library - something i've mentioned a lot. /cc @vitorbaptista

@rufuspollock

In my original comment i pointed out that i did not understand why this was part of the implementation specs when it is not part of the spec itself.
That's the question to @pwalsh. As mentioned it's a part the the implementation specs. Also it was a part of every tool fund implementation contract. I could agree on having it (and I do) or disagree but I work on the implementations level. High level requirements like this are something to decide on the yours/@pwalsh's specs level.

The implementation document was designed to make it easier for new implementors (people not working on the Python or Javascript libraries).In particular, for those implementing for the tool fund.
I personally would not add zip support to the javascript library, unless we already have a good story in place for removing certain code paths in a browser distribution of the lib (I can't imagine reading or writing zips in the browser is very useful). On the other hand, I don't think we should keep talking about bloat without looking at the facts (distribution size of lib, etc.).

I’ll be exploring the zip feature for use in an electron app.

I’ll be exploring the zip feature for use in an electron app.

Great! Will this be handled via a Node.js process, or the client-size Javascript?

@Stephen-Gates
I'll be finishing this work around Jan 7 (when I'll be back from the break) and also I'd like to sync with you/your team on some questions like Paul's one.

@roll before you complete could we consider whether stuff like this should go out of package as a separate mini library. /cc @pwalsh @vitorbaptista
See the data library analysis and recommendation document here:
https://hackmd.io/CwZgnOCMDs0LQEMAckBMdSsnMAjaIcADKgKYBmArGOSACbRJhA==?view
@pwalsh the bloat i'm talking about is less about package zip and more about conceptual complexity and library structure.


Viber CDN not working with -f

Operating system / операционная система
Windows 10 22h2
Running as service / Запуск программы как сервис
I run it as a regular program / Запускаю программу обычным образом
Describe the bug / Опишите ошибку программы
When subj started with -f option (f.e. in 5/6 modes) - downloading from Viber CDN not working:
http://download.cdn.viber.com/cdn/desktop/windows/ViberSetup.exe
http://download.cdn.viber.com/cdn/desktop/windows/ViberSetup.msi
Additional information / Дополнительная информация
No response

Works for me, doesn't seem like a software bug.
To debug, use Wireshark to capture the traffic and check what's going on.
https://github.com/ValdikSS/GoodbyeDPI/wiki/How-to-make-traffic-dump-with-Wireshark


AspNetClaims updating a claim from a role

The fact that an extension method is needed to remove claims in either the aspnetuserclaim and aspnetuserroleclaims is not the best way to interact with the system. It would be best if
just like there is
pseudo code
RoleManager _roleManager =
Exist today
_roleManager.AddClaimAsync
_roleManager.RemoveClaimAsync
My proposal:
There should also be UpdateClaimAsync
http://stackoverflow.com/questions/24587414/how-to-update-a-claim-in-asp-net-identity
Can I do a PR?

@haok didn't we do this already? maybe only for user and not for role?

Yeah someone added ReplaceClaim for UserManager, but we didn't add anything for role claims.
@jvelezc feel free to submit a PR
https://github.com/aspnet/Identity/blob/dev/src/Microsoft.AspNetCore.Identity/UserManager.cs#L978

Bumping to post RTM

Easy fix to make

This is breaking change unfortunately as we need to add ReplaceClaim to IRoleClaimStore, its a fairly straightforward change, but needs to wait for now


fix: remove redundant table shifts

Overview
Shifts of the TABLE columns are required only when dealing with plookup, since we have moved lookups to use logup, we no longer
need to include evaluations of these entities, since they are used in no relations.
This should save a few scalar muls in gemini.
TODO:
update solidity

#9504  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @Maddiaa0 and the rest of your teammates on  Graphite

taken over by https://github.com/AztecProtocol/aztec-packages/pull/9559


[MINOR][DOC] Add missing call of update() in examples of PeriodicGraphCheckpointer & PeriodicRDDCheckpointer

What changes were proposed in this pull request?
forgot to call update() with graph1 & rdd1 in examples for PeriodicGraphCheckpointer & PeriodicRDDCheckpoin
How was this patch tested?
existing tests

Test build #81659 has started for PR 19198 at commit 6f3859c.

Test build #81659 has finished for PR 19198 at commit 6f3859c.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81659/
Test FAILed.

retest this please

Test build #81663 has started for PR 19198 at commit 6f3859c.

Test build #81663 has finished for PR 19198 at commit 6f3859c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81663/
Test PASSed.


main_a2c.py doesn't run because of missing gym_pomdps module

Hello, thanks for providing code for the paper. I'm able to install all the requirements, but I still get an error while trying to run main_a2c.py
python main_a2c.py

Traceback (most recent call last):
  File "/home/fishy/python_ws/aais-baisero/code/asym-rlpo/main_a2c.py", line 16, in <module>
    from asym_rlpo.algorithms import A2C_ABC, make_a2c_algorithm
  File "/home/fishy/python_ws/aais-baisero/code/asym-rlpo/asym_rlpo/algorithms/__init__.py", line 3, in <module>
    from asym_rlpo.envs import Environment
  File "/home/fishy/python_ws/aais-baisero/code/asym-rlpo/asym_rlpo/envs/__init__.py", line 12, in <module>
    from .env_gym import make_gym_env
  File "/home/fishy/python_ws/aais-baisero/code/asym-rlpo/asym_rlpo/envs/env_gym.py", line 8, in <module>
    import gym_pomdps
ModuleNotFoundError: No module named 'gym_pomdps'

Not sure where to get this module exactly! I'm just trying to reproduce some experiments from your paper:
https://arxiv.org/abs/2105.11674
Thanks!

Hi, thanks for raising this issue to my attention;  this code relies on some of my other repositories, but I just noticed that this is not documented anywhere;  shortly, I'll update the readme to include this information.

Thank you, this helps. Could you also please tell me how I could use the environments in ;
https://github.com/abaisero/gym-pomdps/tree/master/gym_pomdps/pomdps
I've tried giving the filename as follows:
python main_a2c.py heavenhell_3 a2c
But it does not pick up the environment name. Might be nice to have this in the readme too ;)

Hello Amit,
I'm assuming you found the solution to this, since I could not see a github
issue on this topic;  if you still need help on this, please open a github
issue, it makes it a lot easier to track questions, both for me and for
future users.
Thanks!
Andrea
On Mon, Oct 10, 2022 at 10:56 PM Amit Sinha @.***>
wrote:

Thank you, this helps. Could you also please tell me how I could use the
environments in ;
https://github.com/abaisero/gym-pomdps/tree/master/gym_pomdps/pomdps
I've tried giving the filename as follows:
python main_a2c.py heavenhell_3 a2c
But it does not pick up the environment name. Might be nice to have this
in the readme too ;)
—
Reply to this email directly, view it on GitHub
https://github.com/abaisero/asym-rlpo/issues/1#issuecomment-1274023447,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANWG6TEFNMQPEVMBNIDPA3WCTJPHANCNFSM6AAAAAARB2I4MI
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Yes I did! I didn't raise another issue because your corrections were sufficient to get things to work.
Thanks!
Amit


✏ Fix typo in docs/tutorial/select.md

Thanks for the interest! But if you read it closely, you will see that it is written as intended, it is referring to different sections in the code, and how each section needs one session.
So I'll pass on this one, but thanks for the effort! ☕


raidboss: fix classical1 bug with noflip checkbox

Closes #5771.

Sorry for not catching this earlier either.


Ref values for address params in publish

Problem solved
Short description of the bug fixed or feature added

PR-Codex overview
This PR focuses on enhancing the contract deployment forms by introducing new dynamic input fields for parameters, allowing for better management of reference contracts and implementation constructor parameters.
Detailed summary

Updated trusted-forwarders-fieldset.tsx to cast value as string.
Added DynamicParams type in deploy-metadata.ts for dynamic contract parameters.
Introduced RefInputFieldset and RefInputImplFieldset components for managing reference contracts.
Enhanced ContractPublishForm to support new input fields and state management.
Implemented dynamic input toggling in ContractParamsFieldset and ImplementationParamsFieldset.
Updated handling of deploy parameters to include dynamic values in CustomContractForm.
Modified deployment logic in deploy-published.ts to process reference deployments.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@kumaryash90 could you link a linear ticket in the description? Need context to review this

@kumaryash90 could you link a linear ticket in the description? Need context to review this

https://linear.app/thirdweb/project/[contract-tooling]-deployment-publish-flow-revamp-0194e04e2a84/overview

Merge activity

Dec 11, 1:05 PM EST: The merge label 'merge-queue' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.


SDK-143 Java SDK: Create test coverage for the class utils.FormBuilder

Ticket
SDK-143
Description
Added test for package utils.FormBuilder. I also used xml files for long form output for better readability and maintainability.

@AmHubert256 could you run this test class using Run 'FormBuilderTest' with Coverage in IntelliJ and post the result here?

Result of the FormBuilderTest coverage in IntelliJ


Sound Not Audible

Description: There seems to be no notification audio regardless of the state of the device (locked+display off / Riot.iOS App Open). This issue is similar to this: https://github.com/vector-im/riot-ios/issues/2264#issue-411527145
The iPhone 8 has audio notification enabled in the device settings.
Here is a video replicating the issue on a device. Sorry about the background I could not find a quiet environment. In the video I toggle the notification slider on the side of the iPhone so notifications can be heard. iMessage notifications are heard just fine whereas Riot.IM notifications could not be heard.
https://sabe.dev/index.php/s/6tg7GyH9kmkgAzo
App Version: 0.11.4

I am experiencing the same issue, but I do hear audio when I am being tagged.  Is this WAI?

I am experiencing the same issue as well. I have bridges setup for WhatsApp but without audible notifications, I am not able to use the element iOS client as I don't get audible notifications on my phone. Not sure where to look first.

Still present for me and a friend on
App Version: 1.1.1


ib interface

(1)ib-sriov-cni
https://github.com/k8snetworkplumbingwg/ib-sriov-cni
(2) ipoib
https://github.com/Mellanox/ipoib-cni

done in v0.9


Update how-to-verify-that-strings-are-in-valid-email-format.md

Summary
escape *

Close/open to get fresh build.

In the preview of the changes, the table formatting was lost. It was too hard to get the formatting of the table right, since it contained | pipe characters; so, I removed the table altogether.


Cards effect: translate3d not working properly in Safari on swiper element.

Check that this is really a bug

[X] I confirm

Reproduction link
https://codesandbox.io/p/sandbox/ppvng9?file=/index.html
Bug description
The tz positioning for translate3d doesn't have any effect on slides if they are in a shadow dom element. I am not sure if it is a swiper issue or a Safari issue.

Expected Behavior
translate3d should move slides back based on perspective.
Actual Behavior
The tz parameter of translate3d doesn't have any effect.
Swiper version
10.1.0
Platform/Target and Browser Versions
MacOS Ventura 13.3.1 (a), Safari 16.4 (186<IP_ADDRESS>.1)
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

https://swiperjs.com/blog/swiper-slider-in-webflow

@nolimits4web is it possible to use the lazy loading with the slots? I tried <div class="slide-content" slot="slide-3" lazy="true"> and it didn't work.


Make generate-js.js ts clean

While working on #429 I found that having @ts-check enabled on the new files was very helpful, so I decided to see how hard it would be to turn on for generate-js.js. This is the result.
I found several typing errors (either errors in jsdoc comments, or in peg.d.ts) - but these were just incorrect declarations - the code itself was fine (also, most of them were introduced by me, when I added source map support to generate-js.js).
I also found one possible bug (again, it was me that did it): A SourceNode was constructed with a child array consisting of strings and an array of SourceNodes (ie Array<string|Array<SourceNode>>). The type info says it should be a Array<string|SourceNode>. Apparently the code did work though, so who knows.
In any case, if this seems worthwhile, I'll be happy to put up similar pull requests for other files as I get time.

I think this is the next one I want to land.  I'll look at it later today, but could you do a quick rebase please?

Rebased

JFYI there's a full rewrite in TS here
Feel free to steal anything from there, as I don't have time to update it to most recent version

@reverofevil Nice.
From my point of view a full typescript rewrite would be much better. But I'd assumed that sticking with code that "just works" without a build step was probably a project requirement. If thats not the case, then sure...

@markw65 Whether it's a full rewrite or not, we still have to figure out how to make types work, and it's the same in either case.
As far as I understand, you plan to continue cleaning up peggy's codebase. In that case feel free to take anything from a (differently) cleaned up version of peggy.

@hildjj Is there anything else you want here? I have the changes to stack.js if you'd like me to roll those into this pull request; or I can do that separately.

@hildjj ping

Sorry, have been traveling. Will get to this when I can, it's still first on the list.

All this needs is a CHANGELOG.md entry, and it's ready to go.


this undefined when using @validate with a function

Hey there. It seems that we get an undefined this when we're trying to do that:
@Validate(() => {
    return this.username;
  })

any idea ?

Nevermind I had to do it like that:
@Validate(function (this: Class) {
  return this.username;
})


Add support for Phoenix Live Dashboard 0.7.0, Elixir 1.14

Thanks again for your work on the project and your help.
This PR also drops support in GitHub actions of earlier versions of Elixir and Erlang.

@walter awesome! Thank you! :purple_heart:


Resource.duplicate(true) doesn't duplicate subresources in Array or Dictionary

Related to #4.
Because of this behavior, set_point_array(make_unique=true) is not working as expected. SS2D_Point resources are not duplicated, when SS2D_Point_Array is duplicated. Only dictionary that holds them is duplicated, preserving SS2D_Point references that are stored as dictionary values.
This is a part of deeper design issue involving copy-pasting of shapes, meta shapes and sharing/not sharing shape properties. At this moment, copy-paste of a shape produces a shape copy in a state that is not easily fixable via UI.

So, the engine internal implementation of Resource does not pass the deep copy parameter of duplicate to the duplicate call of its array/dictionary properties?
That sounds like an engine bug.

 I found a related issue here: https://github.com/godotengine/godot/issues/37162. The documentation could really be more clear on this.
Alright, so we need to add duplicate back, at least in some classes, under a different name to not override the internal duplicate, manually deep copy affected properties, and then adapt other code to use the custom duplicate function.


Update maximum model init time

This PR is in response to https://github.com/mlcommons/training_policies/pull/485/files and moves the requirement under Run Results section.

I believe we discussed this on 5/5/22, and all agreed<EMAIL_ADDRESS>could you merge?

Eric does not have access at the moment, so I'm merging just to get this done. I believe we've discussed this one enough.


Using yaSingletons for NON-MONOBEHAVIOUR Classes, like Utility classes?

As mentioned in the title, I would love to know, am I able to use ur SIngletones as constant undismissable classes which keeps their data alive even when editor/runtime refreshes, since I have a current static class called "Utility" which has some int and lists of objects which gets their content from assertdatabase calls and which it should keep persistent for the entire lifetime of the unityEngine as such. Is this possible and if so, how can I do this, since it seems to me that its only useful if it should do mono stuff or am i wrong (hopefully i might add ^haha)
Best regards
Shpend

To be more specific:
I have a Tile subclass (MyTile: Tile {...}) and in this MyTile i have a "MenuItem" which creates a new mytuile with specifc settings and within that i need a counter, persistent counter, which only shall notify when the creation of the tile is finished at a given amount, like when i created 10 amount of that tile it should stop making new ones. But when i do:
public class Utility : yaSingleton.Singleton {...}
it tells me that ur singleton is calling DontDestroyOnLoad which wont work for editor scripts and hence my question how can I have a scritpablesingleton of my utlility class its so important, any help much appreciated!

Heya,
A basic ScriptableObject should do the trick for what you're describing. I also have a bunch of tooling around ScriptableObjects in my main library Elarion. Just extend the SavedObject class and then add a menu item so you can create it.
There's also a bunch of things you can use to store small specific things - e.g. values or arrays of objects.
PS: The library can't really create singletons in the Editor (I've planned those forever, but still haven't implemented them) because it lacks the Update methods.


Support inner join in the filterMapping in Engine

Support inner join in the filterMapping in Engine

The code review has been reviewed/approved by @akphi.
Updated based on the comment made in https://github.com/finos/legend-engine/pull/432


Improved error handling

Thanks for the update. I was hesitant to access the next-auth routes manually but was able to signin as you suggested above successfully.
One follow-up question, though. Is there a way to capture errors thrown in the next-auth.functions signin() method? For example, if signin() fails, next-auth redirects to error.js and passes '/auth/error?action=signin&type=credentials' in the query string. Is there a way to capture more detailed information regarding the failure - perhaps the error originally thrown during signin()??
Originally posted by @DomenicoCammarota in https://github.com/iaincollins/next-auth/issues/45#issuecomment-417373260

Hi @adeshkumar1201,
Short answer, no there isn't.
I've just made an announcement about version 2.0 which should drop later this month.
It's much simpler to configure and this is definitely something that should be address in 2.x - if not at release, then it should be on the roadmap for 2.1.

Moving this into version 2.0 milestone as this needs to be addressed.
There is a chance this will slip into a subsequent release (e.g. v2.1) but the issue is still relevant.

@iaincollins I saw you mention somewhere you'd like to have URLs to the docs in the error messages. I like this idea a lot too and I noticed you have error codes for most things already built in (i.e. CLIENT_USE_SESSION_ERROR.)
Maybe we could use redirects to get those error messages to point to the right page, i.e. next-auth.js.org/err/CLIENT_USE_SESSION_ERROR could point to the client page. And we could append these to the end of the error output on the console.
If your into the idea, I could take a look at this a little further.

Yes, I would love that! I am a huge fan of errors that point to a URL.
Maybe we could even make specific docs for each error so they are easy to link to and describe?
e.g.
https://next-auth.js.org/errors/CLIENT_USE_SESSION_ERROR -> www/docs/errors/CLIENT_USE_SESSION_ERROR.md?
Or I guess we could just have a single page for all errors and use anchor links, if that approach feels like overkill?
e.g.
https://next-auth.js.org/errors#CLIENT_USE_SESSION_ERROR
Either would be better than no documentation :D

Okay so this has been added with #222, however @iaincollins I just wanted to remind you, the errors docs page needs filling out eventually. Currently I only have one of the errors descriptions written out.
I've gathered all the named errors I could find in the application in the document already, they just need descriptions now. See: https://next-auth.js.org/errors or www/docs/errors.md. Some of them are pretty obvious as to what the error is and I can help you write those, but others you're going to have to decipher haha

@ndom91 Oh no how is _GET_OAUTH_ACCESS_TOKEN_ERROR a real error 🙈
That list is great, thank you! I guess I have some cleaning up and documenting to do. :-)

We have excellent debug logging, a place for online documentation (and integration that directs users to the website when an error is logged) as well as new events system (which is great for audit logging), so I'm going to close this off as resolved.
We still need to expand on some of the errors and provide additional debug logging, but we now have the mechanisms to do that easily.