Odd behavior of "Get-Help" and "man"

Disclamer: this is my first time reporting a bug, so please tell me what to add/change to help :)
I have found out an odd output change within powershell

as you can see, I ran the firsrt command as "man" and there was no output, and if the second command is "man" than it outputs the content to the screen.
I don't think this is big deal, but just let you developers know + (I wanted to submmit a bug for a while)

I believe you  are using Windows PowerShell and not PowerShell 6+. I am not able to repro in both the envs. Can you share the output of $PSVersionTable

follow up question: just in case this is not the fourm to submmit bugs. where is the fourm to do so?

@MomIsBestFriend Please install PowerShell Core latest build and try reproduce the issue.
For Windows PowerShell you could report to UserVoice site.

Windows PowerShell user voice: https://windowsserver.uservoice.com/forums/301869-powershell

Thank you both. I will reopen this issue if I ever install PowerShell Core.
Good Luck in feature development <3


Add HMRC webchat to ‘manage your tax credits’ page

Request from HMRC
To be merged and deployed on Friday 20 May 2016.

👍

This looks like what we did the last time we had to add it to a page, so :+1:.

FTFY


Matter device cannot be group controlled

Reproduction steps
I'm trying to do group control on matter device.
I have tested the connectedhomeip/examples/lighting-app/esp32 example using the CHIPTool v1.0-branch.
I implemented the code below.

Add Group permission to ACL using writeAttributeACLWithValue.
Set the group key using keySetWriteWithParams.
Add a map using writeAttributeGroupKeyMapWithValue.
Added to a group using addGroupWithParams.

But I can't do group control.
I would appreciate it if someone could show me how to implement group control in CHIPTool.
Bug prevalence
Always
GitHub hash of the SDK that was being used
58cd5fb
Platform
darwin
Platform Version(s)
No response
Anything else?
No response

@Akihiro-gif Are you using the Darwin CHIPTool app?  That does not support group control, and neither does the Matter.framework it's using.

Yes, I am trying to implement a group control function in the Darwin CHIPTool app. Doesn't it also support matter.framework?

@Akihiro-gif Darwin CHIPTool uses Matter.framework.  Matter.framework currently does not support group communication.

Thank you. Does that mean that you still can't control multiple devices using Matter.framework as it is?

You can control multiple devices, but not using group messages.

Thank you.
I thought it could be done by Matter.framework,because it seems to be controlled by the command line like # 20979.
When will group control become possible?

because it seems to be controlled by the command line like # 20979.

Nothing in that issue involves Matter.framework...

When will group control become possible?

@Akihiro-gif When someone implements it...  There is no concrete time line for that right now, and no one is working on it that I'm aware of.  There was work in progress in https://github.com/project-chip/connectedhomeip/pull/18375 that is not being actively pursued right now

Thank you.
I'll wait until it's implemented.
Is there any other way to implement automation between devices besides group control? (For example, turn LightApp on when you unlock LockApp, etc.)
If so, please let me know. Is this possible with the Scene and Binding clusters?

Is there any other way to implement automation between devices besides group control?

Having something track the automation and do unicast communication with the relevant devices?

when you unlock LockApp, etc.

This can't be done with groups on its own, right?  Either you're the one unlocking the lock, and that cannot be done via group communication per spec (because a door lock device is not allowed to implement the Groups cluster), or someone else is unlocking the lock and then you have to observe the unlock and communicate with the light(s).

Yes, I understand. Thank you. I'll try to find another way.

Matter Support Framework is unstable, also run into some unknow error,  no more logs, very frustration, can't do anything.

Special under version < IOS 16.4, HomKit Error Code: 2 , and 23(OperationCancel), anybody know the reason?

@uascent:

Special under version < IOS 16.4, HomKit Error Code: 2 , and 23(OperationCancel), anybody know the reason?

For Matter Support or Home issues, please file a feedback assistant ticket on Home as follows:
Feedback assistant ticket:

File a feedback assistant ticket.
https://developer.apple.com/bug-reporting/#feedback-assistant
Click on the link below using your account:
https://feedbackassistant.apple.com/new-form-response
Choose iOS/iPadOS to file an issue
Choose the area as Home App and fill in the details of the type of issue and the description

Technical Information
Please provide the following
a) Sysdiagnose logs for iPhone / iPad with the timestamp / timeframe of the issue
b) Sysdiagnose logs for all the Home Hubs (HomePod mini / HomePod / Apple TV) with the timestamp / timeframe of the issue
c) Accessory logs with the timestamp / timeframe of the issue
d) mDNS network logs with timestamp / timeframe of the issue
e) Video or screenshot describing the issue (optional but beneficial)
Please make sure you download and install the profiles for iOS and tvOS as provided here. The instructions are also in these links.
iOS:

https://developer.apple.com/bug-reporting/profiles-and-logs/?platform=ios&name=homekit
https://developer.apple.com/bug-reporting/profiles-and-logs/?platform=ios&name=network
https://developer.apple.com/bug-reporting/profiles-and-logs/?platform=ios&name=mdns

tvOS:

https://developer.apple.com/bug-reporting/profiles-and-logs/?platform=tvos&name=homekit
https://developer.apple.com/bug-reporting/profiles-and-logs/?platform=tvos&name=network
https://developer.apple.com/bug-reporting/profiles-and-logs/?platform=tvos&name=mdns
For installation instructions, refer to https://developer.apple.com/bug-reporting/profiles-and-logs.
Note: Once you have the profile on the iPhone, it will ask you if you want to install it on other devices. Follow the prompts to install on the HomePod, Apple TV, etc., where you have the described issue and reproduce it.

Matter Commission  demo video ： https://www.bilibili.com/video/BV1Nx421y7sa/


Typescript CollapsibleProps doesn't contain childre

There is no property children?: React.ReactNode; in CollapsibleProps.

Hi @teyalite Did you find a potential workaround for this?

@MikeyAlmighty yes, i added it in the source file in node_modules

FWIW ~
diff --git a/node_modules/react-native-collapsible/index.d.ts b/node_modules/react-native-collapsible/index.d.ts
index 26f8d56..7acf736 100644
--- a/node_modules/react-native-collapsible/index.d.ts
+++ b/node_modules/react-native-collapsible/index.d.ts
@@ -93,6 +93,12 @@ export interface CollapsibleProps {
    * Function called when the animation finished
    */
   onAnimationEnd?: () => void;
+
+  /**
+   * Children! https://github.com/oblador/react-native-collapsible/issues/436
+   */
+  children?: React.ReactNode;
+
 }

 export default class Collapsible extends React.Component<CollapsibleProps> {}

This is due to the update to React 18, where children props need to be stated in the type explicitly.
@oblador Would you accept a PR with this single change? AFAIK it does not affect older versions. Thanks for the reply :)

@oblador could you add that fix?

I'm also facing the issue and would appreciate a fix for this one. I'm not considering changing the index.d.ts from node_modules as a proper solution.

Same. Please fix! I'll put @ts-expect-error for now... 👿

I see this has been merged in a PR.  https://github.com/oblador/react-native-collapsible/pull/432.  Could it please be published?

@oblador This is still causing a problem, would it be possible to publish a new version including the fix that was merged in #432 ?

this has been fixed?

ii saw is merged but when i install the package is not fixed

Hi, any plans on fixing this?


Release v0.62.0

Merged PRs

Refactor ruff #1155 (@RhetTbull)
Added shared_library to asdict() #1154 (@RhetTbull)
Configure Sweep #1152 (@sweep-ai)
Added shared moment, syndicated, shared library to inspect #1150 (@RhetTbull)
Feature shared icloud library 860 #1149 (@RhetTbull)
Feature post function return 1136 #1147 (@RhetTbull)
Added API_README to docs #1146 (@RhetTbull)
Feature post command options 1142 #1145 (@RhetTbull)
Feature catch template error #1141 (@RhetTbull)
Feature keep file 1135 #1139 (@RhetTbull)
Feature custom sidecar zero length #1138 (@RhetTbull)
Refactored --sidecar-template options #1137 (@RhetTbull)
Release v0.61.0 #1132 (@RhetTbull)

Contributors to this release
(GitHub contributors page for this release)
@neilpa | @RhetTbull | @sweep-ai


Update URLs based on redirects

Also fix the link to VirusTotal (https://virustotal.com doesn't resolve without www.)

10x for the link fixes!!


AdminQueries Output 'ApiId' not found in stack

Note: If your issue/bug is regarding the AWS Amplify Console service, please log it in the
Amplify Console GitHub Issue Tracker

Describe the bug
A clear and concise description of what the bug is.
I am trying to update a function with permissions to the AdminQueries api. The operation fails because the  AdminQueries api does not output it's Id.
The error:
UPDATE_FAILED   <function name> AWS::CloudFormation::Stack <time> Output 'ApiId' not found in stack 'arn:aws:cloudformation:<regionName>:<accountId>:stack/<projectId>-apiAdminQueries-<uuid>'

Outputs from admin-queries-cloudformation-template.json
    "Outputs": {
        "RootUrl": {
            "Description": "Root URL of the API gateway",
            "Value": {"Fn::Join": ["", ["https://", {"Ref": "AdminQueries"}, ".execute-api.", {"Ref": "AWS::Region"}, ".amazonaws.com/", {"Fn::If": ["ShouldNotCreateEnvResources","Prod", {"Ref": "env"} ]}]]}
        },
        "ApiName": {
          "Description": "API Friendly name",
          "Value": "AdminQueries"
        }
    }

Amplify CLI Version
You can use amplify -v to check the amplify cli version on your system
4.41.2
To Reproduce
Steps to reproduce the behavior or terminal output if applicable
$ amplify update function
? Select which capability you want to update: Lambda function (serverless function)
? Select the Lambda function you want to update <function name>
? Which setting do you want to update? Resource access permissions
? Select the category api
? Api has # resources in this project. Select the one you would like your Lambda to access AdminQueries
? Select the operations you want to permit for AdminQueries create, read, update, delete
You can access the following resource attributes as environment variables from your Lambda function
        API_ADMINQUERIES_APIID
        API_ADMINQUERIES_APINAME

amplify push then fails.
Expected behavior
A clear and concise description of what you expected to happen.
I expect the cloud formation step to reach UPDATE_COMPLETE
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. Mac/Windows/Ubuntu]

WSL2

Node Version. You can use node -v to check the node version on your system

v14.15.3

Log output
Include any relevant log output under ~/.amplify/logs/amplify-cli-<issue-date>.log
Additional context
Add any other context about the problem here.

I am working on this and I was trying to reproduce your issue. However, I am getting a different error due to a cyclic dependency on the latest Amplify CLI version 4.44.2.

Finding a fix for this.


type object is not subscriptable

Hi. I am trying to run your demo code on a new VPS that I have. It runs very well on lots of other VPS'es and PCs of mine.
#!/usr/bin/env python3

from simple_term_menu import TerminalMenu

def main():
    options = ["entry 1", "entry 2", "entry 3"]
    terminal_menu = TerminalMenu(options)
    menu_entry_index = terminal_menu.show()
    print(f"You have selected {options[menu_entry_index]}!")

if __name__ == "__main__":
    main()

I get the following error:
Traceback (most recent call last):
  File "./test.py", line 14, in <module>
    main()
  File "./test.py", line 10, in main
    menu_entry_index = vis.show()
  File "/usr/local/lib/python3.7/dist-packages/simple_term_menu.py", line 1515, in show
    self._init_term()
  File "/usr/local/lib/python3.7/dist-packages/simple_term_menu.py", line 975, in _init_term
    cast(list[bytes], self._new_term[6])[termios.VMIN] = b"\x01"
TypeError: 'type' object is not subscriptable

I have tried lots of iterations, but nothing works to get this fixed. Please can you try and shed some light on this for me?
I'm running Debian 10 with python 3.7

Thanks for your report. The last release added the line you are referring to. By accident, I used list[bytes] instead of List[bytes]. list[bytes] is only compatible with Python >= 3.9. I have fixed this in the latest release, that will be published in the next minutes. Could you please try to upgrade to version 1.6.6?

1.6.6 Worked for me, thank you very much!


Using [ instead of [[ with (maybe) empty bash vars can cause errors.

e.g.
When I tried to run devenv build, this caused this error:
/nix/store/[...]-devenv-profile/bin/devenv: line 118: [: -eq: unary operator expected

... even though the code suggested it should default to running as devenv build shell

The error seems to have come back?
Did this PR's commit get dropped/over-written in the upstream cachix/devenv#build-container, @domenkozar ?

https://github.com/cachix/devenv/blob/build-container/src/devenv.nix#L125

Seems like I fixed the wrong one before force-pushing. Should be fixed now!

Seems like I fixed the wrong one before force-pushing. Should be fixed now!

No worries!
I wasn't sure if you had discovered a better way to fix it, anyway, TBH.


Symfony process passes ENV variables to child process.

Symfony process passes ENV variables to child process.

Laravel Version: 9.28.0
PHP Version: 8.1.10
Database Driver & Version: 10.9.3-MariaDB (any is affected)

Description:
I had noticed this awhile ago when using Symfony processes but wanted to bring it up now with the attention of the Process convenience layer being added. When this addition lands Symfony process will probably get more use in Laravel.
Right now when using Symfony processes all of the Laravel environment variables get passed through to the child process.
Steps To Reproduce:
You can see that with this example:
// Symfony processes example.
$process = new \Symfony\Component\Process\Process(['env']);
$process->run();
dd($process->getOutput());
// Or Facade example.
$result = \Illuminate\Support\Facades\Process::run('env');
dd($result->output());

For most things, this probably isn't much of an issue, although DB_PASSWORD is included in the environment.
What I did find was an issue was it made it impossible to run Laravel artisan command from one application inside of another. You run the process command in Application One calling Application Two's artisan command. And as you can see from the result, even though it runs the code from Application Two, it uses the environment and database from Application One.
// Symfony processes example.
$process = new \Symfony\Component\Process\Process(['php', '/path/to/another/laravel/app/artisan', 'db:show']);
$process->run();
dump($process->getOutput());
$process = new \Symfony\Component\Process\Process(['php', '/path/to/another/laravel/app/artisan', 'about']);
$process->run();
dump($process->getOutput());
// Or Facade example
$result = \Illuminate\Support\Facades\Process::run('php /path/to/another/laravel/app/artisan db:show');
dump($result->output());
$result = \Illuminate\Support\Facades\Process::run('php /path/to/another/laravel/app/artisan about');
dump($result->output());

Resolutions:
I'm not sure if this should be changed in Symfony's Process in the getDefaultEnv function, although any change there would be a breaking change. With Laravel's process facade not landed yet it is probably best to address it there before any breaking changes are needed.
My assumption is that the Laravel environment variables would not need to be passed to the child process in most cases so should be excluded by default and when they are needed they can be added with the setEnv function on Process.
My workaround so far was to extend Symfony process and remove the Laravel environment variables, although this is just a hack and is not a good solution for everyone's environment.
<?php

namespace App\Support;

use Dotenv\Dotenv;
use Illuminate\Support\Facades\File;
use Symfony\Component\Process\Process as SymfonyProcess;

class Process extends SymfonyProcess
{
    private $laravel_env = null;
    private $backup_env = null;
    private $backup_server = null;

    public function start(callable $callback = null, array $env = [])
    {
        $this->sanitizeEnv();

        parent::start($callback, $env);

        $this->restoreEnv();
    }

    public function sanitizeEnv()
    {
        if (is_null($this->laravel_env)) {
            $this->laravel_env = Dotenv::parse(File::get(base_path('.env')));
        }
        $this->backup_env = $_ENV;
        $this->backup_server = $_SERVER;
        foreach ($this->laravel_env as $key => $_value) {
            unset($_ENV[$key]);
            unset($_SERVER[$key]);
        }
    }

    public function restoreEnv()
    {
        $_ENV = $this->backup_env;
        $_SERVER = $this->backup_server;
    }
}

We've taken your feedback in consideration. By default, Laravel processes inherit all the env vars defined in your system / application, and the same happens when regular Symfony processes.
We see our Process facade as an API for the most common use cases. For more advanced scenarios, such as the case your describe, we advice you to use the Symfony Process class directly, and unset the system / application environment variables using something like so:
$process = new Process(['...'], null, [
    'APP_ENV' => false,
    'CUSTOM_ENV' => false,
]);

Thanks for your your consideration here. I agree it is more of an edge case and won't affect most use cases of it.
I brought it up since you are making Process a first class part of Laravel now and not just using Symfony process as a dependency and it may unknowingly leak .env variables to other processes.
Thanks for the example as well, that will help me improve my workaround that I'm already using.


Split Crude Bionics and Integrated Weapons into standalone mod

I've decided to split the Crude Bionics and Integrated Weapons mechanics to a separate mod. Rationale:

Smaller mods are more compatible with other mods
Focused mods are easier to troubleshoot
Smaller mods are more resilient to changes in the vanilla game

Fixed in 951e90f9d6ede62cee0ed2b892e703c0a20ac6dd


OrderItem

I have tried to achieve with the special functions the items in a certain order, but I get no response from the function, you could help me understand what's the problem?
function getAmazonItemOrders($ord) {
    require_once('../includes/classes.php'); //autoload classes, not needed if composer is being used
    try {
        $amz = new AmazonOrderItemList("Passiontech"); //store name matches the array key in the config file
        $amz->setOrderId($ord);
        $amz->setUseToken(); //tells the object to automatically use tokens right away
        return $amz->fetchItems(); //this is what actually sends the request
    } catch (Exception $ex) {
        echo 'There was a problem with the Amazon library. Error: '.$ex->getMessage();
    }
}

Yes, I see what the problem is. The fetchItems() method indeed retrieves the items from Amazon, but it stores the data in the object and instead gives an indication of whether the operation was successful or not, as indicated by the PHPDoc. To access the data in the object, use the getItems() method.
If you have any questions about how to use a particular class, I recommend checking the PHPDoc at the top of the class file. For each class, I put information about which functions to use in the class and the order in which to use them.

Thank you, it works.

You're welcome. The whole library uses the same design, so hopefully you will find the rest of the library as easy to use.


Possible to give us access to chrome/chromium flags ?

I have an issue with my HDR settings being applied on all apps/websites.
While some websites support it, a lot of chat apps and other don't which over-brightens the screen/window a lot when doing some actions.
The workaround is to change the chrome://flags/#force-color-profile value to sRGB for people who make use of HDR.
Any way to do that in Ferdi, or to give us a way to access the flags of the Chromium/Electron version you're using ?

This is not possible. Please log a feature request against Electron - and if they implement this, we can discuss the need and how it gets implemented.


Failed to create NAT chain!

Hey @jfrazelle
So I upgraded to 4.7.4 of Linux as well as enabled NAT and OverlayFS as modules.
I'm wondering if you ever encountered this issue when bringing up the network (vidalia) via this plugin?
Error response from daemon: NetworkDriver.CreateNetwork: Setup iptables chains failed: Failed to create NAT chain: Iptables not found

If not, I'll continue to search away. Can't seem to find a solution at the moment.
I'm using --iptables=true within the Docker daemon, despite having the NAT module + iptables module enabled ¯_(ツ)_/¯

Hmmm. Just debugging a bit and I'm starting from here: https://github.com/jfrazelle/onion/blob/master/tor/driver.go#L129

You need iptables
On Saturday, September 24, 2016, Charlie Drage<EMAIL_ADDRESS>wrote:

Hmmm. Just debugging a bit and I'm starting from here:
https://github.com/jfrazelle/onion/blob/master/tor/driver.go#L129
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/jfrazelle/onion/issues/2#issuecomment-249366598, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABYNbAKnikLY1QNVQJbLrMHotRjhqEuvks5qtS5agaJpZM4KFpgv
.

--
Jessie Frazelle
4096R / D4C4 DD60 0D66 F65A 8EFC  511E 18F3 685C 0022 BFF3
pgp.mit.edu http://pgp.mit.edu/pks/lookup?op=get&search=0x18F3685C0022BFF3

Have you installed iptables?

I fix it https://github.com/jfrazelle/onion/blob/master/Dockerfile

@jfrazelle That would do it. Needed the iptables within the container.
Thanks! Confirms it works from here.


webUI update for time and temp

changed the UTime to be a value and now handle all formatting in the webUI
render the CPU temp in the webUI

I updated the version constant in gdl90 but I now see I use the wrong test. "0.2pre+" should be "0.2b1+".
I likely won't matter since you will change it for the very next posted build.


Investigate Docpad integration

Per the note in https://github.com/ajam/pulp/issues/41#issuecomment-147701388, Docpad is a build tool with an easy to use interface — if I'm understanding it correctly.
Some current things a build step could help with

Setting animation durations, which currently have to be changed in the config and in the stylesheet
I haven't finished the different zoom modes, but configuring those easily would also be nice since they will have different options

@nylnook will you post your ongoing investigations here and we can see how best to integrate?

Sure ;)

I've made a few improvements to the build process.

You no longer need to set your animation times twice. Set them in config.json and those values will be sent to both the javascript and the stylesheet
You no longer need install stylus and nib globally to compile the stylesheet. Instead, do npm install and then npm run build.

@nylnook In looking more at DocPad, it might be too-opinionated of an approach for people who aren't used to using that. Are there other build process issues that could be simplified from a workflow standpoint? I can integrate them into the new process.
I'll rename this issue...

Hello Michael !
Sorry, I'm really busy with COP21 in Paris theses 2 weeks ;p
Yes Docpad is quite special... but also allow you to choose your preferred languages...
Let me one month : I'll dig into this integration in Januray I think.
Thanks for your work in any way !


Upgrade to detective 4.0.0 for better ES6 compatibility.

The existing version (~2.3.0) of detective breaks when it encounters ES6 artifacts like yield*. Upgrade to 4.0.0 fixes them (at least the ones I have). All tests pass.

Not sure why Travis didn't run on this, but I can confirm everything passes. Will merge w/ your go-ahead @thlorenz

LGTM

@thlorenz This should be able to be a patch

Published as 0.4.0, thanks.
@bendrucker remembering what happened to browserify at the time they updated the esprima parser I opted for releasing as minor just to be on the safe side.


Allow Prisma to work with foreign key that uses different (native) types on each side

Bug description
prisma introspect with native types produces invalid schema.
How to reproduce

Deploy the following SQL

CREATE TABLE public.grading_periods (
    id bigint NOT NULL,
    grading_period_group_id integer NOT NULL
);

ALTER TABLE ONLY public.grading_periods
    ADD CONSTRAINT grading_periods_pkey PRIMARY KEY (id);

CREATE TABLE public.grading_period_groups (
    id bigint NOT NULL
);

ALTER TABLE ONLY public.grading_period_groups
    ADD CONSTRAINT grading_period_groups_pkey PRIMARY KEY (id);

ALTER TABLE ONLY public.grading_periods
    ADD CONSTRAINT fk_rails_9cc118401a FOREIGN KEY (grading_period_group_id) REFERENCES public.grading_period_groups(id);

prisma introspect (with nativeTypes flag) produces

model grading_period_groups {
  id              BigInt            @id @db.BigInt
  grading_periods grading_periods[]
}

model grading_periods {
  id                      BigInt                @id @db.BigInt
  grading_period_group_id Int                   @db.Integer
  grading_period_groups   grading_period_groups @relation(fields: [grading_period_group_id], references: [id])
}

prisma validate returns

(base) divyendusingh [prisma-explicit-panic]$ yarn prisma validate
yarn run v1.22.4
$ /Users/divyendusingh/Documents/prisma/triage/prisma-explicit-panic/node_modules/.bin/prisma validate
Prisma schema loaded from schema.prisma
Error: Schema parsing
error: Error parsing attribute "@relation": The type of the field `grading_period_group_id` in the model `grading_periods` is not matching the type of the referenced field `id` in model `grading_period_groups`.
  -->  schema.prisma:19
   |
18 |   grading_period_group_id Int                   @db.Integer
19 |   grading_period_groups   grading_period_groups @relation(fields: [grading_period_group_id], references: [id])
20 | }
   |

Prisma information
(base) divyendusingh [prisma-explicit-panic]$ yarn prisma --version                                                                                130 ↵
yarn run v1.22.4
$ /Users/divyendusingh/Documents/prisma/triage/prisma-explicit-panic/node_modules/.bin/prisma --version
@prisma/cli          : 2.16.0-dev.17
@prisma/client       : 2.16.0-dev.17
Current platform     : darwin
Query Engine         : query-engine d621d9b01ebb4c8fcd3adeaf3421e0149c676c58 (at node_modules/@prisma/engines/query-engine-darwin)
Migration Engine     : migration-engine-cli d621d9b01ebb4c8fcd3adeaf3421e0149c676c58 (at node_modules/@prisma/engines/migration-engine-darwin)
Introspection Engine : introspection-core d621d9b01ebb4c8fcd3adeaf3421e0149c676c58 (at node_modules/@prisma/engines/introspection-engine-darwin)
Format Binary        : prisma-fmt d621d9b01ebb4c8fcd3adeaf3421e0149c676c58 (at node_modules/@prisma/engines/prisma-fmt-darwin)
Studio               : 0.343.0
Preview Features     : nativeTypes
✨  Done in 1.75s.

This should become a datamodel parser validation.

This should become a datamodel parser validation.

Yeah, we could make the validation there more lenient. Although I wonder whether the db would reject the inverted case. You seem to be constraining a bigint column to values from an int column. That should always work, but the other way around could fail

We looked into the issue and had a discussion about this. The databases to some degree allow differing types between the underlying and the referenced column in foreign keys. However not all the combinations where they do not error out are also safe. On Postgres for example you can reference a Text column into a Char(10) column. This can of course fail at run time.
Since we currently don't know of a use case where you would need the ability for differing types we're going to stay with the strict validation. We think this will protect the user from accidentally creating migrations that are valid SQL schemas, but might later fail at runtime.
Our proposal for this case would be to align the types in the original database which migrate should be able to do.
I'm leaving this issue open for others who might run into this. If you have a specific use case where you need to be able to have different types please let us know and we might reconsider. For now we're not going to change the validation.

thanks for reporting.  i think the validation is great and in the db i am working with i absolutely agree that the types should be consistent.  i have a similar issue to OP here, poor db design mixed references to bigint with int.
however - as you know, tech debt cannot always be paid off immediately.  i would very much like to continue with these as warnings instead of it stopping.  much like how the earlier versions comments out models when there are problems

is it possible to get prisma to skip the validation check on a particular line?  along the lines of /* eslint-disable no-eval */
i guess at the moment, the quick solution is to rollback a version.  i know the best/correct solution is for me to fix the columns in this db.  i realise these checks are all so you can make sane migrations.  and if the underlying database has these sloppy type mistakes, it might make it harder/impossible for the code that generates migrations.
if i can dream a bit, it would be amazing if prisma could generate a migration to fixes these type incompatibilities (:pray:) by changing the columns to the same type (eg, upgrade int columns to bigint).

answering my own question

is it possible to get prisma to skip the validation check on a particular line? along the lines of /* eslint-disable no-eval */

i just commented out the lines in question.  obviously, this means i cannot use prisma to access any of these values, however, it means that i can fix them as i need to.  (i am slowly porting a legacy code base over to prisma)

Hey @carchrae, thanks for providing your input. As you figured out, commenting out the offending fields is currently the only solution. I think you could even comment out just the relationfield and then still write to the underlying scalar fields directly. Just mentioning it in case you are commenting out both.
model User {
  id              BigInt   @id @db.BigInt
  // Posts       []
}

model Posts {
  id             BigInt   @id @db.BigInt
  user_id   Int         @db.Integer
  // user        User     @relation(fields: [user_id], references: [id])
}

I like your idea of a cleanup helper to find Sql schema smells and offer a migration to clean them up. I'll open a separate issue about this.

thanks @do4gr.  yes, i've been learning how to overcome these different errors and warnings.  a lack of valid unique ids is the other main curse, in particular, compound ids that can be made up of nullable values (but are still unique).  sometimes i need to go into the woods and shout a bit.
hmm - i just realised that maybe views can help in some of these cases https://www.prisma.io/docs/guides/general-guides/database-workflows/sql-views-postgres - yes, kind of ugly, but maybe functional.  eg, in a view, null becomes 0, or you can cast a column type.  yes, you can't update views, but queries with prisma could then work.
i do appreciate you taking up the migration fix idea, but i do have a 'developer relations' recommendation, which is that instead of errors, introspect should continue to comment out models (or fields) that have issues.  when i ran introspect using 2.16 it gave me a working, but incomplete, schema.  when checking out if a tool is going to work with your project or not, that was a far more rewarding developer experience than lots of errors - each of which take some searching and research to resolve.  being able to see it work asap in 2.16 gave me some proof it was worth investing more time using prisma and the commented out errors shown in the schema comments were clear.   ymmv - but on a bigger team, proving something works well is required to invest more time (or even consider running migrations to allow the tool to work)

An alternative workaround here might be to lie to Prisma. Instead of trying to generate a client with two non matching columns in the relation, change the bigger one to actually have the type of the smaller one. This will not reflect reality, but e.g. for Int and BigInt this should be fine assuming your data confirms to that and does not include BigInts. Then the error should go away, you should be able to use the full API of Prisma on your models.

We looked into the issue and had a discussion about this. The databases to some degree allow differing types between the underlying and the referenced column in foreign keys. However not all the combinations where they do not error out are also safe. On Postgres for example you can reference a Text column into a Char(10) column. This can of course fail at run time.
Since we currently don't know of a use case where you would need the ability for differing types we're going to stay with the strict validation. We think this will protect the user from accidentally creating migrations that are valid SQL schemas, but might later fail at runtime.
Our proposal for this case would be to align the types in the original database which migrate should be able to do.
I'm leaving this issue open for others who might run into this. If you have a specific use case where you need to be able to have different types please let us know and we might reconsider. For now we're not going to change the validation.

Hi @do4gr, we are facing the exact same issue. Here's the scenario, we've got an application which is in charge of database migration e.g. Rails, after many years, it develope two tables like:
CREATE TABLE author (
    id bigint NOT NULL
);

CREATE TABLE book (
    id bigint NOT NULL
    author_id int NOT NULL
)

The type of author.id was INT, and it's migrated to BIGINT successfully, but the book.author_id wasn't.
As you can see, the original application doesn't complain about this, and we are trying to use Nest.js with Prisma to read the existing database schema only, so Rails does the actual migration, and we use the prisma db pull to do the one-way sync-up here.
Hope we can find a way to ignore this validation error without commenting out the relation line.


AWS SDK for JavaScript version 3 (v3) Upgrade

Issue #, if available:
Description of changes: Migrating your code to SDK for JavaScript V3
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Check any applicable:

[ ] Were any files moved? Moving files changes their URL, which breaks all hyperlinks to the files.

Any chance this could get merged as well @taoyong-ty  ? We are seeing warnings in workflows using this action.

I rebased my changes
$ git rev-parse --short HEAD
642ba69
$ npm run test

><EMAIL_ADDRESS>test
> mocha

  logName
    ✔ return the logGroupName and logStreamName from an ARN
    ✔ return undefined when the group and stream are null
    ✔ return undefined when the Arn is undefined

  githubInputs
    ✔ build basic parameters for codeBuild.startBuild
    ✔ a project name is required.
    ✔ can process env-vars-for-codebuild
    ✔ skips override when parameter is set to true
    ✔ can handle pull requests
    ✔ will not continue if there is no payload
    ✔ can handle configuring update call-rate
    ✔ can hide cloudwatch logs when the parameter is set to true

  inputs2Parameters
    ✔ build basic parameters for codeBuild.startBuild
    ✔ build override parameters for codeBuild.startBuild
    ✔ can process env-vars-for-codebuild
    ✔ can process disable-source-override
    ✔ can process disable-github-env-vars

  waitForBuildEndTime
    ✔ basic usages
got one
    ✔ waits for a build endTime **and** no cloud watch log events
    ✔ waits after being rate limited and tries again
    ✔ dies after getting an error from the aws sdk that isn't rate limiting

  buildSdk
    ✔ return the codebuild client when env variable AWS_CONTAINER_CREDENTIALS_FULL_URI exists
    ✔ return the codebuild client when env variable AWS_CONTAINER_CREDENTIALS_RELATIVE_URI exists

  22 passing (48ms)


False positive in "variable is never read" when using object.defineProperty

I was originally looking at the migration guide on how to fix accessors overriding properties in TS -> https://github.com/microsoft/TypeScript/pull/33509#
Part of that guidance was using object.defineProperty and defining the getter/setter over there but hit a strange compiler error.
TypeScript Version:  3.8.3

Search Terms:

object define property false positive
unread variable define property
Code

export class Foo {
  private _value: string | undefined;

  constructor() {
    Object.defineProperty(this, "value", {
      get() {
        return this._value;
      }
    });
  }
}

Expected behavior:
No compiler errors as I am defining a setter for the _value so the value is being read somewhere
Actual behavior:
'_value' is declared but its value is never read.
If I remove _value I get
Property '_value' does not exist on type 'Foo'. Did you mean 'value'?
Playground Link:
https://www.typescriptlang.org/play?ts=3.8.3#code/KYDwDg9gTgLgBAYwDYEMDOa4DEITgbwFgAoOOMKASwDcUZg4B9WpAV2AC440YqA7AOZwAPnFZ8AJsABmlPsAkBuEiTIIIfHlFYIY0ABQBKAqrJwA8gCMAVsF0A6KbPkAFKBDDBYAT30wAFpRoADRwAEQs7GGhRKRmZALAMEYmcfFkUEmsUHxwAUH2zChswMppZgC+pmQVhmU1JBVAA
Related Issues:

Typescript don't understand that this in the getter definition will be the class. As a workaround, define the type of this inside the getter:
export class Foo {
  private _value: string | undefined;

  constructor() {
    Object.defineProperty(this, "value", {
      get(this: Foo) {
        return this._value;
      }
    });
  }
}

Playground

👆


allow simple dot notation in alias?

not using set-value or something, but some more basic and simple.

I want to do this as well, but IMHO we should hold off on it until the community is a little better educated on what lazy-caching is and how it works. Some devs already (mistakenly) think this module is doing magic, so anything we add to this lib will make it seem more magical.

but let's keep this open and maybe implement when lazy-cache hits 50-100m downloads or some number where we can more easily make a case: "We've had X million downloads an X issues". idk, just thinking out loud

You can also implement this in your library for the specific case that you want to use...

something.js

var something = require('lazy-cache')(require);
something('foo');
something('bar');
something('baz');

module.exports = something;

utils.js

var utils = require('lazy-cache')(require);

utils('./something', 'something');
module.exports = utils;

index.js

var utils = require('./utils');
utils.something.foo('blah');

So, after typing that out, I can see how the dot notation can be useful in the alias.

So, after typing that out, I can see how the dot notation can be useful in the alias.

Indeed. I can do it, but don't want to create whole new file, even more when I need dot in only 2 aliases.
And I think this also will work?
var utils = require('lazy-cache')(require);

require('foo')
require('bar')

utils.plugins = {}
utils.plugins.fooBar = require('qux-fez')

require = fn
module.exports = utils

I'm convinced :)
this would be really nice to do


BUG: RSA example is skipped

http://mne-tools.github.io/dev/auto_examples/decoding/decoding_rsa.html?highlight=rsa
doesn't create the image, we probably need to just change the name of the example.

Yes plot_

I think we left this without the plot because of the size of the dataset.

yes it's too much for Circle CI

ok

We should add a note


Empty photo block items set returned as null

Might be an easier to use api if it returned []. Experiencing this behavior calling getPhotoBlocks from the mobile native layer.

Hmm, that doens't sound right... the core api returns []repo.Block, the mobile wrapper converts that to json with an object wrapper:
type Blocks struct {
	Items []repo.Block `json:"items"`
}

Could possibly be an issue in the bridges, or just a bug. Can try and take a look today... but PRs are welcome or course... no departmental lines here :)

Yea, and it's not just as simple as making sure the Go code works as expected... We're generating the Android and iOS apis using gobind, so it's also a matter of understanding how that works.

totally


Null safety

Any thoughts as to when null safety will be implemented?

https://github.com/desxz/Flutter-FFNavigationBar/tree/4d94bd37d1ced07531e0f5ff317c1d35dbc5670d

I want to offer up my Null-Safety migrated version too, with some extra features I added for myself about 2 years ago..
ff_navigation_bar:
git: https://github.com/skquark/Flutter-FFNavigationBar.git
Here's the bonus customizations:

barShadow: Optional BoxShadow effect of navigation bar
selectedItemBackgroundGradient: Optional LinearGradient of selected circle background (default uses selectedItemBackgroundColor)
unselectedItemIconGradient: Optional LinearGradient of unselected icon (default uses unselectedItemIconColor)
selectedItemBorderWidth: The thickness for the selected item's circle border (default = 4.0)
selectedItemIconSize: The size of the selected icon (default to theme)
unselectedItemIconSize: The size of the unselected icons (default to theme)
selectedItemTopOffset: The y offset of the selected icon's circle overhang (default = 10.0)
onClick: Optional function call when clicking the already selected item.

If you'd like me to make a pull request, would be glad to contribute.  These customizations made it a lot nicer looking in my app, and probably make others out there happy..

I want to offer up my Null-Safety migrated version too, with some extra features I added for myself about 2 years ago..
  ff_navigation_bar:
    git: https://github.com/skquark/Flutter-FFNavigationBar.git

Here's the bonus customizations:

barShadow: Optional BoxShadow effect of navigation bar
selectedItemBackgroundGradient: Optional LinearGradient of selected circle background (default uses selectedItemBackgroundColor)
unselectedItemIconGradient: Optional LinearGradient of unselected icon (default uses unselectedItemIconColor)
selectedItemBorderWidth: The thickness for the selected item's circle border (default = 4.0)
selectedItemIconSize: The size of the selected icon (default to theme)
unselectedItemIconSize: The size of the unselected icons (default to theme)
selectedItemTopOffset: The y offset of the selected icon's circle overhang (default = 10.0)
onClick: Optional function call when clicking the already selected item.

If you'd like me to make a pull request, would be glad to contribute. These customizations made it a lot nicer looking in my app, and probably make others out there happy..

Thanks bro @Skquark


Docfix: lt-head and column-type blueprint

closes https://github.com/offirgolan/ember-light-table/issues/198

👍 Thank you! 😺


Area3D created in Physicsserver3D dont work

Godot version
4.0 stable
System information
Windows 10 64bit RTX-3060
Issue description
I create a Area3D via Physicsserver3D, including callback.
On input press, a second area was created the same way in same position, but the callbacks was not trigered.
I created a Area3D with nodes and move it over a "server-area", but wether the "server area" nor the "node area" callbacks was trigered
Steps to reproduce
create a new project scene in 3d, paste below code on a script on the root node,
press ui_right to move the "node area" towards where the "server area" should be : no trigger
press ui_down to create a second area on same position where the first is: no trigger
edit: added the .gd script for reproduce the problem
Minimal reproduction project
extends Node3D

var area_node

var area_rid
var collider

var area_rid2

func _ready():

	collider = PhysicsServer3D.box_shape_create()

	area_node = Area3D.new()
	var colsh = CollisionShape3D.new()
	colsh.shape = BoxShape3D.new()
	area_node.add_child(colsh)
	area_node.transform.origin.x = -2
	add_child(area_node)
	area_node.area_entered.connect(area_enter)

	area_rid = PhysicsServer3D.area_create()

	PhysicsServer3D.shape_set_data(collider, Vector3.ONE)

	PhysicsServer3D.area_set_space(area_rid, get_world_3d().space)
	PhysicsServer3D.area_add_shape(area_rid, collider, transform)
	PhysicsServer3D.area_set_collision_mask(area_rid, 1)
	PhysicsServer3D.area_set_collision_layer(area_rid, 1)
	PhysicsServer3D.area_set_monitorable(area_rid, true)
	PhysicsServer3D.area_set_monitor_callback(area_rid, area_enter2)

func _process(delta):
	if Input.is_action_just_pressed("ui_right"):
		area_node.translate(Vector3(0.5,0,0))

	if Input.is_action_just_pressed("ui_down"):

		area_rid2 = PhysicsServer3D.area_create()

		PhysicsServer3D.area_set_space(area_rid2, get_world_3d().space)
		PhysicsServer3D.area_add_shape(area_rid2, collider, transform)
		PhysicsServer3D.area_set_collision_mask(area_rid2, 1)
		PhysicsServer3D.area_set_collision_layer(area_rid2, 1)
		PhysicsServer3D.area_set_monitorable(area_rid2, true)
		PhysicsServer3D.area_set_monitor_callback(area_rid2, area_enter2)

func  area_enter(area:Area3D):
	print("entered")

func  area_enter2(a, b, c, d, e):
	print("entered")

RID_Test_.zip

PS: Code blocks should use triple backticks like this (with an optional language name for syntax highlighting):
```gdscript
code here
```
I edited your post accordingly, but remember to do this in the future :slightly_smiling_face:

Thanks a lot Calinou!

The signal area_entered is triggered from the area_set_area_monitor_callback callback, the area_set_monitor_callback is for bodies

So beyond the above, the area_entered signal won't be fired because it only fires when a real Area3D enters, as you can see because the signal has an Area3D parameter with it
And if you use the correct monitor function it all works correctly

No problem! The new documentation should help indicate that the two different callbacks are for different uses


arming and takeoff with mavros !

Hello
I am successfully able to launch mavros_posix_sitl.launch, but i am unaware of using mavros command like arming and takeoff. However i tried using rosservice call /mavros/cmd arming and set it to true but its returning me FALSE.
I don't think mavros  launch needs anything else for me to use service call for arming and takeoff.
please guide me on how can i arm and takeoff my iris in gazebo.
Thanks

rosrun mavros mavsafety arm
rosrun mavros mavcmd takeoff 0 0 0 0 5 (being 5 the altitude)

Hi
Thanks for the comment, although I tried these rosrun and it's showing up request failed

So you are probably missing something more fundamental here. Also it's not clear what's your current framework (meaning PX4 Firmware version, etc). Have you tried to launch the SITL simulation without ROS, by issuing make posix_sitl_default gazebo and after it opens (and gets a position lock), issue commander takeoff on the same window you launched the previous commad?

Please also mark here:

your ROS and MAVROS version;
you Gazebo version;
Also, try launching QGC and check for the data coming from the vehicle and if there are any log error msgs being reported

Did try it , commander take off without ROS is showing no position lock.. I waited for much longer..

Did try it , commander take off without ROS is showing no position lock.. I waited for much longer..

OK are you able to show us a video/screencapture of the procedure you take to launch the simulation and then the behavior you see on the simulator? Are you using the latest upstream master of PX4 or a recent stable release?

This is my console output now without ROS
-- http://pastebin.com/Hqx4sbxs

Here is my screen shot with mavros posix_sitl.launch
-- http://pasteboard.co/HnN6Yih.png


[POC] Add chromatic to the project

Add chromatic to the project so we can experiment with visual regression testing.
https://www.chromatic.com/docs/setup

You can add chromatic to you github account here: https://www.chromatic.com/builds?appId=646dd42469efabe23410bb59

We're probably going to need this: https://www.chromatic.com/docs/monorepos#combine-multiple-projects-into-a-single-storybook

We're probably going to need this: https://www.chromatic.com/docs/monorepos#combine-multiple-projects-into-a-single-storybook

I'm not sure if that is possible: one uses @storybook/web-components, the other @storybook/angular. If I had to choose, I definitely would choose the web components one.

We can't turn on multi-browser testing (yet) since you need at least a "STANDARD" account for that. According to Jeroen (and the chromatic terms & conditions 😁) we should be able to get a STANDARD or PRO account for free, since our project is open source.
Waiting for confirmation on that.


Updated most offsets for new version

Okay, didn't mess up the PR this time. I updated most of the offsets for the latest patch, all of the major features work without crashing the client, and fog still works. I'm still not sure about the 0xNAN addresses, but the one thing I know needs updating is the debug pointer.

Hey @nightlark ! thank you for all the effort you put on this!
I'm going to close this issue as I had prepared some changes that I didn't commit because I found something strange in the latest patch.
They added a new set of instructions that change the look at vector so when you move the camera it flickrers the game screen. Don't you have problems in your machinima studio build? I'm curios to see you didn't reported anything.
Also, I think anet added a new security layer because when you move the camera far away from the player, the game crashes, is it a problem I just have or are you able to reproduce it?
I'll keep this open until you answer my but I won't merge it as I made some important changes that have conflicts with this PR, also I added a new method to resolve one of the three 0x00000NAN values, so now we only have 2 0x00000NAN values more to resolve 😄 I want to write a wiki page on how to update all the offsets....
Btw, I saw you've implemented the highlighting function, but it is not included in this PR, what happened?

I only had crashes with the free camera and stopping animations, but updating the offsets fixed them. Just tried going to the opposite end of the map, no crash, and performance seemed okay (a couple spots of lag, probably not the flickering you're referring to?). Video: http://sendvid.com/q68qewnc
On a different system I had some interesting flicker stuff going on related to position the camera thought it should be at, but that only happened when I tried using the mouse to move the camera. I also can't reproduce that bug anymore.
I'm seeing if there is a better place to disable highlighting. The current patch works, but anything highlighted will remain highlighted after checking the box to disable highlighting; it's a minor issue (maybe some people would even find it useful), so with or without a fix I'll send another PR after merging it with your next update.

@karliky Is the flickering something like in this ( http://sendvid.com/qsa8ykre ) video? I was able to reproduce the issue I had before - tilting the camera down slightly makes the view clip to below the water surface, in some other areas without the water it jumps to a view under the map.

I know a way to select npcs by setting the ID (int32 4 bytes) and that activates the high tough times so maybe setting it to 0 disables the highlight.....
I'm experiencing an issue with the latest patch, I saw your video and everything seems to work fine so I'm really surprised.
Can you move your player to Lion's arch? Then activate the freebie camera and move it far away (holding W pressed for 20 seconds should be enough)
when I do this my game crashes :(

@nightlark yes, I know that problem, it also happens with the PVP spectate camera, when you introduce the lookat vector inside the water the game does a movement and applies the drops water effect in your screen.
No easy way to fix that...

If your game doesn't crashes when move the camera in Lion's arch, would you mind sharing with me your gw2.exe? Also your graphic settings would be good

I wasn't so much worried about the water drops, but more how when the camera position is above the water, rotating the camera down shouldn't make it suddenly jump underwater, and then rotating back up makes it return to where it was before.
I went to the opposite end of the map in LA, and then flew up into the sky for another 20 seconds. I'm going to see if I can reproduce the crash you are getting.

@karliky I figured out the crash. The new patch 3 has the same offset as patch 2 and is overwriting an instruction that should be left intact.
Also, the new patch 8 is identical to the old patch 3 (offset 0x00675623 is what the old patch 3 offset should be updated to).

I've updated the files in my fork. Patches 6 & 7 were also identical.
Even after trying the latest upstream patch I didn't notice any flickering. Maybe ANet made a change that your graphics card doesn't like? I've uploaded a copy of my gw2.exe to http://www.filedropper.com/gw2_1
Graphics settings I'm using are:
Refresh Rate - Default
Frame Limiter - Unlimited
Antialiasing - FXAA
Reflections - All
Render Sampling - Native
With Ambient Occlusion, Best Texture Filtering, Depth Blur, Effect LOD, Light Adaptation, and Vertical Sync enabled, and all other settings on High.

I'm at the hospital right now in really bad condition, I've sent you an invitation as a collaborator so you can take care of the project, and commit directly or accept your own PRs, I won't be able to work on this while I'm sick and recovering at home, if you haven't receive the link send me an email to<EMAIL_ADDRESS>

Thank you so much for your support

@nightlark not sure If you read my las 2 comments

@karliky Yea, I've just been trying to figure out what to say as this news is rather sudden. I hope that your condition improves quickly and that you have a swift recovery.

Hope you get well soon !


Tests: relax tolerance for gpu/cpu agreement in image case

Our first GPU test is failing because the GPU and CPU are not giving final losses within tolerance of 0.0001, in the case of images.
In this PR the tolerance is relaxed by one order of magnitude, which is probably still acceptable.
I am not currently able to test locally, so this is "suck-it-and-see" PR.

Looks like it fixed the test failures.

?? https://buildkite.com/julialang/mljflux-dot-jl/builds/10

That's concerning. Seems to be intermittent.
Can you open an new dummy PR to trigger CI to see if we can reproduce?

Here's the log for the build that passed: https://buildkite.com/julialang/mljflux-dot-jl/builds/9


Database port is returned  as int and unable to convert it to sting

Hello!

Vote on this issue by adding a 👍 reaction
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already)

Issue details
While creating a Database, the output for the port is int, I want to use that port in Kubernetes configmap which only accepts the str format, I tried multiple ways to convert int to sting but they don't work due to pulumi's output format. is there any way i can change it to str or any workaround to convert int to str in pulumi.
Steps to reproduce

Create a database
Extract Database port in variable to use in Configmap (DB_PORT = mysql.port.apply(lambda port: port))
Use same variable in Configmap, "binary_data={"DB_PORT" : DB_PORT } ,"
pulumi up will now give bellow error.

"""
error: resource default/vault-port-gqg1dja7 was not successfully created by the Kubernetes API server : ConfigMap in version "v1" cannot be handled as a ConfigMap: v1.ConfigMap.BinaryData: base64Codec: invalid input, error found in #10 byte of ...|"DB_PORT":25060},"ki|..., bigger context ...|{"apiVersion":"v1","binaryData":{"DB_PORT":25060},"kind":"ConfigMap","metadata":{"annotations|
"""
Thanks!!

Could you please provide more of your code? Your lambda as written currently is a no-op and I would've expected
DB_PORT = mysql.port.apply(lambda port: str(port))

to actually make it a string.

Hey, thanks! this worked for port, no-op with the issue.
but when I use the get_database_ca like
certificate_new = do.get_database_ca(cluster_id=mysql_cluster.id.apply(lambda id: str(id))).certificate
then I get the below error
Exception: invoke of digitalocean:index/getDatabaseCa:getDatabaseCa failed: Required attribute is not set ()
Thanks!

Hey @leezen, it worked like charm, my no-op lamda was the issue, Thanks a ton.


update side bar

Why are these changes needed?

Show all the blog posts on the website
Related issue number

Closes #687
Checks

[x] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[x] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[x] I've made sure all auto checks have passed.

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

Comparison is base (fe00925) 28.85% compared to head (d3a7285) 25.88%.
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #702      +/-   ##
==========================================
- Coverage   28.85%   25.88%   -2.98%
==========================================
  Files          27       26       -1
  Lines        3393     3400       +7
  Branches      764      767       +3
==========================================
- Hits          979      880      -99
- Misses       2342     2451     +109
+ Partials       72       69       -3

Flag
Coverage Δ

unittests
25.88% <ø> (-2.92%)
:arrow_down:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Return twiml from webhook

Judging by this doc it looks like we want to return twiml directly, rather than a payload:
https://www.twilio.com/docs/sms/tutorials/how-to-receive-and-reply-python-amazon-lambda
An empty MessagingResponse() stringifies to:
<?xml version="1.0" encoding="UTF-8"?><Response />
I'm 90% confident this should work

There are two types of lambda integrations with API gateway. We're using a "lambda proxy" integration, and so we need to return a dictionary that includes the body and headers keys. We need this integration type to get access to the request headers that we're using to validate the payload.
The other integration is a "lambda" integration, which I think Twilio's guide refers to.
https://medium.com/@lakshmanLD/lambda-proxy-vs-lambda-integration-in-aws-api-gateway-3a9397af0e6d


Modified RandomUserSwift to remove "Swift 3" from description

Project URL

RandomUserSwift
Category

SDK - Unofficial
Description

I finally got around to updating RandomUserSwift for Swift 4. Thought I'd come by and fix my description. :)
Why it should be included to awesome-ios (optional)
Checklist

[x] Only one project/change is in this pull request
[ ] Addition in chronological order (bottom of category)
[x] Supports iOS 9 / tvOS 10 or later
[x] Supports Swift 4 or later
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

          1 Warning

:warning:
Found 2 link issues, a project collaborator will take care of these, thanks :)

Link issues by awesome_bot

Line
Status
Link

1247
301
https://pusher.com/push-notifications redirects tohttps://pusher.com/beams

3016
Error
https://weeklycocoa.news  No route to host - connect(2) for "weeklycocoa.news" port 443

  Generated by :no_entry_sign: Danger

Thanks ❤


All python tests ported to use ragger

There is a pylint config but no CI checks? Is that normal?
There is one in Checks on the Python client / Type checking (pull_request) Failing after 23s


Large API Multipart support

Large API Multipart support

New PR: https://github.com/cisco-pxgrid/cloud-sdk-go/pull/31


Recording error on Jenkins slave

Can you please look into this?
08:30:23  INFO -Trying to execute the following command: [cmd, /c, for /f "tokens=2" %i in ('tasklist ^| findstr "ffmpeg"') do @echo %i]
08:30:23  INFO -Trying to execute the following command: [SendSignalCtrlC.exe, ]
08:30:23  INFO -Process kill output: "" is not a valid PID.
0x00000057 == (FormatMessage failed)
08:30:23 DEBUG -Recording output log: ffmpeg version N-81784-g92de2c2 Copyright (c) 2000-2016 the FFmpeg developers
built with gcc 5.4.0 (GCC)
configuration: --enable-gpl --enable-version3 --disable-w32threads --enable-dxva2 --enable-libmfx --enable-nvenc --enable-avisynth --enable-bzlib --enable-libebur128 --enable-fontconfig --enable-frei0r --enable-gnutls --enable-iconv --enable-libass --enable-libbluray --enable-libbs2b --enable-libcaca --enable-libfreetype --enable-libgme --enable-libgsm --enable-libilbc --enable-libmodplug --enable-libmp3lame --enable-libopencore-amrnb --enable-libopencore-amrwb --enable-libopenh264 --enable-libopenjpeg --enable-libopus --enable-librtmp --enable-libschroedinger --enable-libsnappy --enable-libsoxr --enable-libspeex --enable-libtheora --enable-libtwolame --enable-libvidstab --enable-libvo-amrwbenc --enable-libvorbis --enable-libvpx --enable-libwavpack --enable-libwebp --enable-libx264 --enable-libx265 --enable-libxavs --enable-libxvid --enable-libzimg --enable-lzma --enable-decklink --enable-zlib
libavutil      55. 30.100 / 55. 30.100
libavcodec     57. 58.100 / 57. 58.100
libavformat    57. 51.100 / 57. 51.100
libavdevice    57.  0.102 / 57.  0.102
libavfilter     6. 63.100 /  6. 63.100
libswscale      4.  1.100 /  4.  1.100
libswresample   2.  2.100 /  2.  2.100
libpostproc    54.  0.100 / 54.  0.100
[gdigrab @<PHONE_NUMBER>c32cc0] Capturing whole desktop as 1920x1080x32 at (0,0)
[gdigrab @<PHONE_NUMBER>c32cc0] Failed to capture image (error 5)
[gdigrab @<PHONE_NUMBER>c32cc0] Could not find codec parameters for stream 0 (Video: bmp, none, 1988680 kb/s): unspecified size
Consider increasing the value for the 'analyzeduration' and 'probesize' options
Input #0, gdigrab, from 'desktop':
Duration: N/A, bitrate: 1988680 kb/s
Stream #0:0: Video: bmp, none, 1988680 kb/s, 29.97 fps, 1000k tbr, 1000k tbn, 1000k tbc
Output #0, mp4, to 'D:\Jenkins\YO\workspace\Development\VideoEnroll\Framework\target\video\temporary_recording_2016_13_10_08_29_38.mp4':
Output file #0 does not contain any stream

Could you provide more details? What operation systems is?
From logs I see 08:30:23 INFO -Process kill output: "" is not a valid PID.
It means that ffmpeg process id detected incorrectly. By to understand why? We need to get more information

Windows Server 2012
It's a remote vm. It's configured to be Jenkins slave.
I have access there via RDP.
I tured to execute this command line through build job:
ffmpeg -y -video_size 1920x1080 -f gdigrab -i desktop -an -framerate 24 D:\Jenkins\YO\workspace\Multi-Product-Enrollment\Framework\target\video\temporary_recording.mp4
It's interesting that it starts to record only if I have active RDP session going on.
If session is off or session window is minimized, it fails with the error I posted earlier : [gdigrab @<PHONE_NUMBER>c32cc0] Failed to capture image (error 5)

So it's not a bug in VideoRecorder. I think you should google about ffmpeg and gdigrab and RDP. Probably when you singn off machine it closes session and disconect the screen. So for ffmpeg it looks like you machine doesn't have monitor

Maybe you have workaround ideas?

I would suggest to use VNC instead of RDP,. Second variant is to Google about this problem :)

VNC is no better. I googled about it. No luck. One of the popular solutions - redirect session to console but then i get another error: [gdigrab @<PHONE_NUMBER>c028a0] Capture area (0,0),(1920,1080) extends outside window area (0,0),(1024,768)desktop: I/O error

On one of my previous projects we used bat script that exit RDP without closing session. You may google such solution and try

Can't find it. Tell me, why does recorder sets resolution to (1920,1080)  when it should be (1024,768) ?

VideoRecorder defines screen resolution automatically. Probably I should add ability to set screen resolution in configuration section

That would be nice

Agree, will plan it for next realease

fixed https://github.com/SergeyPirogov/video-recorder-java/commit/4131c66b865138833b6685d8091923018ca88314

@da-yaroslav-orel I've found such a tool https://technet.microsoft.com/ru-ru/sysinternals/bb963905.aspx . It can help to keep RDP connection alive

I can't see how this tool should help me. Session seems to be disconnected whenever I'm not logged in manually.


Add childless html functionality

Added a utils.ts for the can have children function that way it can be used by other renderers in the future

Thank you!


GIES: intervention targets

I may be missing something but I think GIES algorithm may not work properly. It takes a “score” input,  that has “targets” in pcalg R implementation, i.e., intervened nodes.
Only difference between GES and GIES implementation in this package seems to be the “score”, obs and int, as it should be.  However,
https://github.com/FenTechSolutions/CausalDiscoveryToolbox/blob/master/cdt/causality/graph/GIES.py
doesn’t seem to have “target”  for 'int' score. Could you elaborate whether something is missing in GIES implementation, or I am missing something. Thanks.

Hello @bvarici, thanks for noting this point, I'll be fixing ths in the next release !

Hi Diviyan, was this fixed? I don't think so, but I might be missing something.
Have a good day!


HtmlRenderer (Razor) removes significant space

Is there an existing issue for this?

[x] I have searched the existing issues

Describe the bug
The razor engine of HtmlRenderer removes whitespaces which it considers insignificant.
However, it seems to remove significant spaces, too.
Example:

    @GetHello() @Name

    @if (!string.IsNullOrWhiteSpace(Name))
    {
        @GetHello() @Name
    }

    @if (!string.IsNullOrWhiteSpace(Name))
    {
        <text>@GetHello() @Name</text>
    }

@code {
    public string Name { get; set; } = "World";

    private static string GetHello()
    {
        return "Hello";
    }
}

Result:

Hello World

HelloWorld

Hello World

Note that in the second example, the space is missing between 'Hello' and 'World'.
Expected Behavior
I would expect all three examples to output the same:

Hello World

Hello World

Hello World

Steps To Reproduce
No response
Exceptions (if any)
No response
.NET Version
8.0.403
Anything else?
No response

One could find reasons why the second example is missing the space.
However, I found another example (example 4):

    @if (!string.IsNullOrWhiteSpace(Name))
    {
        @GetHello()<text> </text>@Name
    }

And the result:

HelloWorld

In this example, the space is missing, too, although the space is embedded within a <text> tag.

Thank you, Jan, for your detailed explanation!
The rules for Razor seem to be very complicated (probably because Razor wants to enable a convenient writing experience)
The reason why I might be confused is that some years ago, the behavior was different (no removal of whitespace). Then this changed and the  @preservewhitespace directive was introduced. I thought that this directive would only be necessary for edge cases, but it seems that is not the case.
I have learned that surrounding an existing block of content with e.g. an @if can change how the content is rendered (because the meaning of whitespace changes when switching from content mode to code mode).
Ok, thank you! I think you can close this issue.

Closing as by-design. Please comment if you think this is in error. Thanks!


Crash after 1-5s [crash.txt]

Circumstances:

This is only when I run BeardLib and BeardLib-Editor in the mods folder together.

I have disabled/uninstalled other mods with only these 2 running with the crash reccuring. [Including mod_overrides]

I have disabled BeardLib-Editor with and without mods installed (not including BeardLib ofc) and I still crash.

Running the game with all mods except BeardLib-Editor seems to work every time.
Mods:
   1. BeardLib
   2. BeardLib-Editor [Cause of Crash]
   3. FreeFlightCamera
   4. RestorationMod
   5. PocoHud3
   6. Silent Assasin Mod
   7. WolfHUD

At the time these were:
    1. Enabled
    2. Attempted Both
    3. Disabled
    4. Enabled
    5. Disabled
    6. Disabled
    7. Disabled
    8. Disabled

`Application has crashed: C++ exception
[string "lib/utils/game_state_machine/gamestatemachi..."]:601: attempt to index a nil value
SCRIPT STACK

Callstack:
     payday2_win32_release  (???)     ???
     payday2_win32_release  (???)     ???
     payday2_win32_release  (???)     ???
     payday2_win32_release  (???)     ???
     payday2_win32_release  (???)     zip_get_name
                     ntdll  (???)     RtlFreeHeap
                       ???  (???)     ???
                       ???  (???)     ???
                     ntdll  (???)     wcstok_s
                KERNELBASE  (???)     SetUnhandledExceptionFilter
                KERNELBASE  (???)     SetUnhandledExceptionFilter
     payday2_win32_release  (???)     ???
     payday2_win32_release  (???)     zip_get_name

Current thread: Main

System information:
Application version : 1.68.209
CPU : Intel(R) Core(TM) i3-4150 CPU @ 3.50GHz (1 core); SSE; SSE2; SSE3; SSSE3; SSE4.1; SSE4.2
DirectX : 12.0
GPU : NVIDIA GeForce GTX 950 / nvd3dum.dll[<IP_ADDRESS>49]
Language : @ID94cca2fdaccfc2cd@
Memory :     8079MB 332KB
OS : 6.2.9200 () 0x300-0x1 (64 bits)
Physics : threaded
Renderer : DX9 threaded
Sound : Logitech (Speakers (2- Logitech G430 Gaming Headset))
`

Are you running the dev branch or master?

At the moment, the master.

Uhh.... I went in game and check the BeardLib Main Menu and there wasn't anything, anywhere about the editor like I've seen.

Install beardLib dev as well.


cross-compilation llvm_12 for aarch64 broken (ncurses linking)

Describe the bug
It appears that cross-compiling llvm_12 (with pkgsCross.aarch64-multiplatform.llvm_12 doesn't work due to an issue where linking ncurses fails.
This is both on staging and staging-next, but not on master, because llvmPackages_latest is bumped to 12.

[x] Broken on staging (bcc720d9c5ddc6e2ac90370d0be694e3e60c1495)
[x] Broken on staging-next (d8950b1fad5e542500bb15e9a94d7dda4b16a766) -> probably broken
[ ] Still building on master (997977a64e6501e68543ac9491987556aa38b5c8)

To Reproduce
Steps to reproduce the behavior:

nix-build -A pkgsCross.aarch64-multiplatform.llvm_12

Expected behavior
A clear and concise description of what you expected to happen.
Build log
Error in the build log
[100%] Building CXX object lib/Support/CMakeFiles/LLVMSupport.dir/Signals.cpp.o
[100%] Building CXX object lib/Support/CMakeFiles/LLVMSupport.dir/TargetRegistry.cpp.o
[100%] Building CXX object lib/Support/CMakeFiles/LLVMSupport.dir/ThreadLocal.cpp.o
[100%] Building CXX object lib/Support/CMakeFiles/LLVMSupport.dir/Threading.cpp.o
[100%] Building CXX object lib/Support/CMakeFiles/LLVMSupport.dir/Valgrind.cpp.o
[100%] Building CXX object lib/Support/CMakeFiles/LLVMSupport.dir/Watchdog.cpp.o
[100%] Linking CXX static library ../libLLVMSupport.a
[100%] Built target LLVMSupport
Scanning dependencies of target llvm-config
[100%] Building CXX object tools/llvm-config/CMakeFiles/llvm-config.dir/llvm-config.cpp.o
[100%] Linking CXX executable ../../bin/llvm-config
/nix/store/6wfnavjv56bagkd4q81lq5sayy4xf8p8-binutils-2.35.1/bin/ld: /nix/store/7b3w6rdk782yhj3gd4n6mrmwsqzwnhf7-ncurses-6.2-aarch64-unknown-linux-gnu/lib/libtinfo.so: error adding symbols: file in wrong format
collect2: error: ld returned 1 exit status
make[6]: *** [tools/llvm-config/CMakeFiles/llvm-config.dir/build.make:107: bin/llvm-config] Error 1
make[5]: *** [CMakeFiles/Makefile2:26969: tools/llvm-config/CMakeFiles/llvm-config.dir/all] Error 2
make[4]: *** [CMakeFiles/Makefile2:26976: tools/llvm-config/CMakeFiles/llvm-config.dir/rule] Error 2
make[3]: *** [Makefile:7368: llvm-config] Error 2
make[2]: *** [tools/llvm-config/CMakeFiles/NativeLLVMConfig.dir/build.make:80: NATIVE/bin/llvm-config] Error 2
make[1]: *** [CMakeFiles/Makefile2:27192: tools/llvm-config/CMakeFiles/NativeLLVMConfig.dir/all] Error 2
make: *** [Makefile:171: all] Error 2

Additional context
This may or may not be related to this change: https://reviews.llvm.org/D85820.
This bump put llvm_12 as a default, but it's most probably not the culprit: 336eb64ae56 (https://github.com/NixOS/nixpkgs/pull/116646). Might be somewhere in the packaging, or upstream.
Notify maintainers
@primeos @Ericson2314 @sternenseemann @thefloweringash
Metadata

system: "x86_64-linux"
host os: Linux 5.12.10-hardened1, NixOS, 21.11pre295280.fa0326ce523 (Porcupine)
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.4pre20210601_5985b8b
channels(root): "nixos-21.11pre295280.fa0326ce523, nixpkgs-21.11pre293089.1c2986bbb80"
channels(rick): ""
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos

Maintainer information:
attribute:
- pkgsCross.aarch64-multiplatform.llvm_12

To me this looks like a failure to compile the native llvm-config which would run on the build platform ($dev/bin/llvm-config-native). This also makes sense since ncurses is in propagatedBuildInputs, so its host version would be used.
Not sure what the precise change to fix this would need to be, can't look into it today.


Area3D shape doesn't follow CollisionShape3D

Tested versions
4.3rc2
System information
Godot v4.3.rc2 - Nobara Linux 40 (GNOME Edition) - Wayland - Vulkan (Mobile) - dedicated NVIDIA GeForce RTX 3050 Ti Laptop GPU - 11th Gen Intel(R) Core(TM) i7-11370H @ 3.30GHz (8 Threads)
Issue description
This orange box should align with the blue box, I'm told.
(Also, the blue box is very, very faint... it's very difficult for me to see it.)

Steps to reproduce

Create a new Scene with Area3D as root node
Add a CollisionShape3D which uses the built in box shape
Move the CollisionShape3D using the red dot handles only

(Oh, another comment here, I can't seem to find a button that hides the big transform gizmo, so sometimes I transform using that gizmo rather than the red dots... a UX papercut.)
Minimal reproduction project (MRP)
area3dtest.zip

The orange box is the bounds of the node, this includes various things, in this case collision shapes doesn't have any bounds to use, so it just takes the origin of that node, I'm not sure how it is supposed to work, not sure if this is a bug or not as it normally covers visible nodes

That is the "remember to not scale your collision shapes, resize them" box.
... sly jokes aside this is the default Node3D spatial gizmo AABB box, tiny by default, and only really visible when you start to scale stuff.

Ok, thanks for the quick replies. Guess it's just a slightly unintuitive thing we just have to live with (it starts making sense once you add meshes to scene, so not sure of the value of seeing it until you do that). I'm not the only one puzzled by it, because I asked in the official Godot Discord and seems this is new for 4.3?

Well if it's new in 4.3 that's more implying it is a bug (would be good information to add to the report), if you can confirm this then that's something to investigate

It seems its changed around 4.3dev3
Maybe https://github.com/godotengine/godot/pull/87132

What changed is that previously when a node was inherited from VisualInstance (which many nodes do), it would just straight use the visual instance aabb. This aabb ended up being zero size most of the time. Without a size it had no real visible geometry to render.
The change in the linked PR fixed parts of a wrong aabb calculation and made all zero sized aabbs fall back to the default size. There was always a default AABB, by default the size is 0.4 units when the Node3D had no parent.


Latest std documenation does not seem to be loading any methods/API

Going to<EMAIL_ADDRESS>:

Compare that with<EMAIL_ADDRESS>/<EMAIL_ADDRESS>:

This weird state where the left-hand panel seems to be affecting other modules/versions randomly:

<EMAIL_ADDRESS>(Ok)
<EMAIL_ADDRESS>(Bad)
<EMAIL_ADDRESS>(Ok)

The specific issue with std is fixed.  The "problem" is that some modules are too CPU intensive to be able to complete the documentation "lazily" on Deno Deploy.  We are going through a fair amount of change internally on how we handle this, and we haven't integrated the updating as part of release process yet.
Since this started out with the std issue and the title reflects that, I am going to close this now the 1.155.0 has been manually loaded and open a new issue around the allo_arguments as it seems to be unreleated (a forced reload didn't resolve the issue, like it did for std).


PersonListPanelTest#createXmlFileWithPersons(): Fix incorrect path

createXmlFileWithPersons() saves the location of the temporarily
generated .xml file to:

    src/test/data/sandboxmanyPersons.xml

instead of:

    src/test/data/sandbox/manyPersons.xml

It is likely that the original author of the test intended the saving
location to be the latter, given that it has always resolved to the
latter until we revamped the path handling logic in AddressBook to use
java.nio.file.Path API, which introduced this discrepancy (commit
9eac635d9da1a71c81587ba66cab1911977a863b).

Saving the temporary .xml file to the 'sandbox' folder is also ideal as
the folder is git-ignored, so even if JVM crashes and does not clean
up the temporary file, students would not accidentally commit this
potentially _enormous_ .xml file into the repository.

Let's fix the incorrect path.

So, I took a closer look at the code in PersonListPanelTest, and I realized that we actually don't even need to go through the trouble of creating an XML file with persons, since that's not what the performance test is testing for :-P


New version: TTKN.CNKIExpress version 0.2.23

Result: Installation Successful

Manifest
Add/Remove Programs

Name
全球学术快报

Version
0.2.23

Publisher
同方知网(北京)技术有限公司

ProductCode

Auto-updated by vedantmgoyal2009/winget-pkgs-automation in workflow run 2786
Microsoft Reviewers: Open in CodeFlow

bump

@denelon please assign to an engineer...

@vedantmgoyal2009 - I investigated bit on this installer validation failure. On our local sandbox environment installation is failing with following exception per logs and I do see same exception with the validation pipeline logs.
2022-07-31 14:14:21.676 [CLI ] Completed installer selection.
2022-07-31 14:14:21.729 [CLI ] Caught std::exception: No mapping for the Unicode character exists in the target multi-byte code page.
not sure if this is related to resources file content. Will check with team for the additional understanding.

Looks like a client bug triggered by the Installer URI.
Debugging I found that this is throwing while generating the download path for the installer here:
https://github.com/microsoft/winget-cli/blob/1a6ca17040a0577dceb25af3232649e4bd271b81/src/AppInstallerCLICore/Workflows/DownloadFlow.cpp#L72
At this point, from the installer URI https://download.cnki.net/cnkiexpress/%E5%85%A8%E7%90%83%E5%AD%A6%E6%9C%AF%E5%BF%AB%E6%8A%A5%20Setup%200.2.23.exe we have extracted the file name "全球学术快报%20Setup%200.2.23.exe", which seems to be causing that line to throw.
I haven't looked into why it's throwing, so I have no idea for a possible fix.

Oh, there's already an issue in the client for this...
microsoft/winget-cli#2057


Can I create common classes in shared projects that can be utilized by many other models? In other words can I setup remote dependencies to other projects/repos and branches?

We do support this feature internally but didn’t push it externally yet. In order for this to work we need to finish the SDLC build pipeline which is currently scheduled for end Q1. We will notify everyone when this feature is supported externally.

Is this now supported @beekemarie  & @agitana ?

Hi @AJ73! Quick update on this, we are running slightly behind on the SDLC build pipeline but that work should be completed in the next 2-3 weeks. However, we realized that we cannot fully support this use case without open sourcing a first version of the metadata server as well.  We're still discussing prioritization and the timeline internally. Once we finalized the roadmap for this, I'll share more details with you.

Moved this issue to the Legend FAQ wiki page. https://github.com/finos/legend/wiki/Legend-FAQs


Gulp task fails on imagemin-gifsicle, vendor folder not installed

Hi,
I tried to reinstall several times and get the pre-build test success. But gulp keeps having this error:
Error: spawn /node_modules/gulp-imagemin/node_modules/imagemin/node_modules/imagemin-gifsicle/node_modules/gifsicle/vendor/gifsicle ENOENT     at exports._errnoException (util.js:746:11)     at Process.ChildProcess._handle.onexit (child_process.js:1053:32)     at child_process.js:1144:20     at process._tickCallback (node.js:355:11)
tried emptying npm cache and reinstall. Tried installing gifsicle seperately. Fixed my permissions on the directory, but nothing works.
version gulp: ^3.8.5
version gulp-imagemin: ^2.3.0
version imagemin-pngquant: ^4.1.0

did the whole step by step installation again and now it worked (probably because I fixed permissions after the first installation, now installed with fixed permissions)


Implement missing MediaControl methods

Fixes https://github.com/ConnectSDK/Connect-SDK-iOS/issues/170

sendNotSupportedFailure method is a common for all services / session and now it is duplicated. I think it's a good idea to extract it into one place.


Since v70.1.1 can no longer use Java 8

Looks like from this version you're using the Java 9 version of the Arrays util class. Using Java 8 I get the following error:
Caught: java.lang.NoSuchMethodError: java.util.Arrays.equals([BII[BII)Z
java.lang.NoSuchMethodError: java.util.Arrays.equals([BII[BII)Z
	at com.sigpwned.chardet4j.ByteOrderMark.detect(ByteOrderMark.java:61)
	at com.sigpwned.chardet4j.Chardet.detectCharset(Chardet.java:52)
	at com.sigpwned.chardet4j.Chardet.detectCharset(Chardet.java:44)
	at com.sigpwned.chardet4j.Chardet$detectCharset.call(Unknown Source)

No worries if this library is Java 9, but you might want to change the About description which says "Simple, compact charset detection for Java 8+".
Thanks!

Great flag! Looking into this now.

Fixed by #35


Improve About / Help page

Description:
Improve About Help page
Layout
Structure
Links to support articles etc
Issue for iOS:  #1183

QA passed


Add note to update packages

When running govuk_puppet to update the development-vm, sometimes there are errors when installing system packages where these packages can't be found. It may simply require getting the latest package list.

Nope, we didn't. You could maybe put an echo in https://github.com/alphagov/govuk-puppet/blob/master/tools/puppet-apply-dev for people who forget to read docs

@surminus I've added an echo but this is like the second time in my life I've ever bashed so hope it's ok.


[Do not merge] Integrate SCR rubric into delivery process

Do not merge until the SCR rubric PR is merged (https://github.com/18F/cg-site/pull/1059). This also needs review by the cloud.gov Program Manager before merge.
Changes to feature lifecycle:

Before we put a feature into the meeting agenda for discussion with JAB TRs, we review it against our SCR rubric. Some changes do not need any discussion with TRs.
Move the threat modeling guide to this stage, as a supplement to SCR analysis.

Changes to story lifecycle:

If a story has not received security impact analysis as part of a feature, it needs security impact analysis at the story stage. This is to catch "gravel" stories that pop up outside of features.
The steps about reviewing with the System Owner or the AO belong in the Security Impact Analysis and formal SCR rubric, not in this grooming stage.

Non-blocking questions before merge, mainly for Program Manager and System Owner:

Question: is this the right place for the threat modeling guide? How might we integrate the threat modeling guide in a more practical way that is required at the appropriate step, rather than as an optional supplement jammed in there?
CM-3 part b and CM-7 a require System Owner review and approval before significant changes. Do we need to add anything more to these processes to ensure we're getting proper System Owner review and approval, or is this already covered by having the SCRs signed by the System Owner or their delegate?
Is there anything relevant we should integrate into https://github.com/18F/cg-product/blob/master/ConMonChecklist.md or link from there?

Question: is this the right place for the threat modeling guide? How might we integrate the threat modeling guide in a more practical way that is required at the appropriate step, rather than as an optional supplement jammed in there?

I think this is fine until we're doing it more consistently and less optionally.

CM-3 part b and CM-7 a require System Owner review and approval before significant changes. Do we need to add anything more to these processes to ensure we're getting proper System Owner review and approval, or is this already covered by having the SCRs signed by the System Owner or their delegate?

They're going to see it when it goes through the SIA stage. They're also going to see it right before implementation during PI planning.

Is there anything relevant we should integrate into https://github.com/18F/cg-product/blob/master/ConMonChecklist.md or link from there?

That's separate work, I think.

This is fine to review and merge.


Please invite me to the GitHub Community Organization

Name
Dipesh Babu
Discord Username (if applicable)
dipesh.babu#3698
Additional Context
I had heard about this community through the Eddie Jaoude's youtube channel and have been following it since a month. I hope joining into this community will make me more familiar with open sourcing. I will be pleased being part of this community.

Welcome to the EddieHub community, @dipeshbabu! Happy you've joined us! 😄👋🎉


[T3A3][F12-A3]Wang Yu

work in progress

@WangYu-g some comments added, please check
please close the PR once you have noted the suggestions.


[BUG] File open in another program error when using GOBIG

Describe the bug
When using GOBIG sometimes the GUI will try to load one of the splits as a preview image which will cause it to be locked and cause the operation to fail
To Reproduce
Steps to reproduce the behavior:

Create an image with GOBIG enabled
Wait for it to eventually error out.
Observe that the image is question is being previewed in the GUI
See error

Expected behavior
The GUI should not just open the newest image in the target folder. It should look for the proper name of the output file and wait for that to appear before previewing anything.
Desktop (please complete the following information):

OS: Windows 11
GPU and VRAM: 3090 24GB
CUDA Version 11.7

Additional context
The image size I'm generating is 1728x960 with a GOBIG scale of 4

I'll look into it. I did find errors running gobig but I was able to get the final image. I was super concerned with proper functionality of displaying the images of gobig in the GUI since, at least on my desktop anything larger than 768 along with the GUI starts to track off the screen.

I haven't been able to reproduce the results. Are you using prsgui.py or makegudpics.py?

I haven't been able to reproduce the results. Are you using prsgui.py or makegudpics.py?

prsgui

I haven't been able to reproduce the issue. I tried many sessions. I did however rewrite the way that the images are displayed. It also now resizes the images if they are larger than 768 in either direction. Please let me know if your issue is solved with the new code.


Browser tests support

Fixed URL parsing in browser. Our logic for detecting relative URLs was broken in the browser, where a default base URL of window.location is used. Instead, simply use url-parse's optional base URL argument.
Passes an 'include' argument to fetch which allows cross-origin cookies in the browser for testing LROs. Not sure if this is better implemented as a RequestPolicy--this will fail on any service which sends back the CORS header Access-Control-Allow-Origin: *, as CORS forbids storing cookies for a service with a wildcard in the allow-origin header.

See Azure/ms-rest-js#35


the abstract employee class, the commission employee class and the sa…

…lary employee class were created

the abstract employee class, the commission employee class and the salary employee class were created


Custom Claims not available

I'm following your example of how to add custom claims from the windows login. I can confirm the below code is correctly returning a claim:
app.UseMixedAuth(new MixedAuthOptions
{
    Provider = new MixedAuthProvider
    {
        OnImportClaims = identity =>
        {
            var claims = new List<Claim>();

            using (var principalContext = new PrincipalContext(ContextType.Machine)) // todo: Change to ContextType.Domain  ?
            {
                using (var userPrincipal = UserPrincipal.FindByIdentity(principalContext, identity.Name))
                {
                    if (userPrincipal != null)
                    {
                        claims.Add(new Claim(ClaimTypes.WindowsAccountName, userPrincipal.Name ?? string.Empty));
                        claims.Add(new Claim(ClaimTypes.Email, userPrincipal.EmailAddress ?? string.Empty));
                        claims.Add(new Claim(CustomClaimTypes.DisplayName, userPrincipal.DisplayName ?? userPrincipal.Name ?? string.Empty));
                    }
                }
            }
            return claims;
        }
    }

}, cookieOptions);

Debugging reveals that DisplayName is being correctly set, however when I try to access this value in the view with @ClaimsPrincipal.Current.FindFirst(CustomClaimTypes.DisplayName)?.Value I get an empty string.
Am I missing another step that will correctly include the claims?

Imported claims will be added to the claims list of the external identity (windows identity, Not the local identity).
Upon user registration or account linking you need to read those claims from the external identity and save them to the database as part of the local identity claims.


Refine json schema with support for templates

Document how Schema validation can still be applied for goss.yaml files using inline templates, by using render preprocessing
Checklist

[ ] make test-all (UNIX) passes. CI will also test this
[ ] unit and/or integration tests are included (if applicable)
[x] documentation is changed or added (if applicable)

Description of change
Doc update
Would be nice to add some automated tests by using a json schema validator, e.g. one available from https://json-schema.org/implementations.html such as https://github.com/santhosh-tekuri/jsonschema yet its yaml support is pending, see https://github.com/santhosh-tekuri/jsonschema/pull/52

Thanks for the contribution, noticed a few commits came in after the PR was opened. Let me know when this is ready to merge and I'll merge it.

thanks @aelsabbahy
It would be good for me to add automated tests for the schema to avoid future regressions. I might be able to find time to add this in 2/3 weeks. I may open a new PR if you prefer merging this one sooner.

I'll merge this now. I'm a bit mixed on automated tests if they'll fail the build, can you check if the v4 branch changes would break this? If not, then it's fine.

Can't remember if you can comment after I merge or not, I'll keep it open until your next response and then merge it

thanks @aelsabbahy
Might be better for you to merge this branch now, and that I later submit additional tests against v4 branch or after v4 gets merged.

Thanks for the update and great contribution. Much appreciated!


scantailor-universal: init at 0.2.14

Description of changes
Adds scantailor-universal, a fork based on Enhanced+Featured+Master versions of ScanTailor to the pkgs

Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

We use 2 spaces for indentation, so please fix that.
Dangit! I was 100% sure I had fixed that. My bad.

As for the rest of the changes, do I fix each thing in its own commit or do I just end up squashing them all into the original commit? Also, do I mark you as a co-author of the changes or not?

As for the rest of the changes, do I fix each thing in its own commit or do I just end up squashing them all into the original commit?

Yes, it's the preferred way, unless you need to split up a big PR.

Also, do I mark you as a co-author of the changes or not?

No, that's not necessary.

alright I actually used the formatting tool this time, thought I wouldn't need it because I thought that was the only change left to do, oops

should be all good

Result of nixpkgs-review pr 260983 run on x86_64-linux 1

1 package built:

scantailor-universal


OneBranch Pipeline for Kusto Cluster Deployments

What this PR does / why we need it:
Creates a OneBranch pipeline for generating and deploying Kusto Ev2 config files
Relevant work item:
Deploy Kusto Cluster to Public INT
Test plan for issue:
OneBranch build pipeline is already deploying Kusto clusters to public INT from fork. Build pipeline will be modified to target this main repo.
Is there any documentation that needs to be updated for this PR?
No

/azp run ci

@jewzaam Im big far from all EV2 pipelines so will leave it for you. No idea what and how this gonna work :/

/azp run CI


issue: Provide better errors for unsupported combination of elements

If you have a rect that was set to position: absolute the app will still run. But as soon as give it some content, it crash the app. A minimal example:
#![cfg_attr(
    all(not(debug_assertions), target_os = "windows"),
    windows_subsystem = "windows"
)]

use freya::prelude::*;

fn main() {
    launch(app);
}

fn app() -> Element {
    rsx!(
        rect {
            position: "absolute",
            position_top: "10",
            position_left: "10",
            "Just some text"
        }
    )
}

That will crash with this error:
thread 'main' panicked at crates/core/src/dom/dom_adapter.rs:35:14:
This should exist.
stack backtrace:
   0: rust_begin_unwind
             at /rustc/7cf61ebde7b22796c69757901dd346d0fe70bd97/library/std/src/panicking.rs:647:5
   1: core::panicking::panic_fmt
             at /rustc/7cf61ebde7b22796c69757901dd346d0fe70bd97/library/core/src/panicking.rs:72:14
   2: core::panicking::panic_display
             at /rustc/7cf61ebde7b22796c69757901dd346d0fe70bd97/library/core/src/panicking.rs:196:5
   3: core::panicking::panic_str
             at /rustc/7cf61ebde7b22796c69757901dd346d0fe70bd97/library/core/src/panicking.rs:171:5
   4: core::option::expect_failed
             at /rustc/7cf61ebde7b22796c69757901dd346d0fe70bd97/library/core/src/option.rs:1988:5
   5: core::option::Option<T>::expect
             at /rustc/7cf61ebde7b22796c69757901dd346d0fe70bd97/library/core/src/option.rs:894:21
   6: <freya_core::dom::dom_adapter::DioxusDOMAdapter as torin::dom_adapter::DOMAdapter<shipyard::entity_id::EntityId>>::get_node
             at ./crates/core/src/dom/dom_adapter.rs:33:26
   7: torin::measure::measure_inner_nodes
             at ./crates/torin/src/measure.rs:378:26
   8: torin::measure::measure_node
             at ./crates/torin/src/measure.rs:193:13
   9: torin::measure::measure_inner_nodes
             at ./crates/torin/src/measure.rs:381:48
  10: torin::measure::measure_node
             at ./crates/torin/src/measure.rs:193:13
  11: torin::measure::measure_inner_nodes
             at ./crates/torin/src/measure.rs:381:48
  12: torin::measure::measure_node
             at ./crates/torin/src/measure.rs:193:13
  13: torin::torin::Torin<Key>::measure
             at ./crates/torin/src/torin.rs:257:52
  14: freya_core::layout::layout_measurer::process_layout
             at ./crates/core/src/layout/layout_measurer.rs:24:9
  15: freya_renderer::app::App<State>::process_layout
             at ./crates/renderer/src/app.rs:305:13
  16: freya_renderer::DesktopRenderer::launch
             at ./crates/renderer/src/lib.rs:77:9
  17: freya::launch::launch_cfg
             at ./crates/freya/src/launch.rs:226:5
  18: freya::launch::launch
             at ./crates/freya/src/launch.rs:36:5
  19: absolute::main
             at ./examples/absolute.rs:9:5
  20: core::ops::function::FnOnce::call_once
             at /rustc/7cf61ebde7b22796c69757901dd346d0fe70bd97/library/core/src/ops/function.rs:250:5
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

Oh my bad. It only crash if the child is a text & not a rect

It's actually not a bug, to display text you have to use the label or paragraph elements. That said, I should add a friendlier error heh

Or maybe fail silently? What do you think it's better?

I think I will just fail silently, will improve it asap, it's easy

Nice! Yeah it's definitely better to fail silently. In JS you'd have a warning log in the console that warn you about the invalid nested element.
Another cool option is to catch these errors at compile time but that'd be pretty difficult (I imagine) 😄

Another cool option is to catch these errors at compile time but that'd be pretty difficult (I imagine) 😄

I would love that, but Dioxus doesn't support it :(


:bug: Fix: remove major from a changeset

Description
Concise description of proposed changes
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

#1256  👈
main

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

Join @roninjin10 and the rest of your teammates on  Graphite