Updated to work with .Net 5.0 (Abort is obsolete), Along with other random changes/fixes.

Also pull request #5
Since abort is no longer in .net 5.0, if we remove the abort the thread throws an exception when the driver is unloaded. Instead we can flag we unloaded and check if we unloaded.
Also I think this exception was unreachable before. If for some reason we leave the while loop we call Unload which than kills itself.

Also did fixes.

#22 Should be fixed.
#21 Fixed.
#20 Should be fixed from fixing #21. (Needs more testing)
#10 Fixed.

@kevinf100 Placing this here since I can't seem to find issues tab on your fork.
Running your version, this is working in Windows 11 with 1 oddity I have found: After the first call to SendText(), the next call will be delayed until some physical input is detected (mouse or keyboard).
Messages that are queued up execute once per physical input received. For example, if I send 6 messages, the first will send, then if I press a physical key, message 2 will print on key down, message 3 on key up, etc. until all sent messaged are printed.
I have KeyboardFilterMode set to All & do not have a MouseFilterMode set at all, as I only want to deal with sending keys in my use case.
As I'm writing a remote input application, there may not be any physical input happening on the target machine.

`using System;
using System.Threading;
using Interceptor;
namespace InterSwap
{
class Program
{
static void Main(string[] args)
{
// Tell the driver to capture all input.
var input = new Input
{
//MouseFilterMode = MouseFilterMode.All,
KeyboardFilterMode = KeyboardFilterMode.All
};
        input.Load();
        Thread.Sleep(2000);
        for (var i = 0; i < 10; i++)
        {
            Console.WriteLine($"Test{i}");
            input.SendText($"Test{i}");
            input.SendKey(Keys.Enter);
            Thread.Sleep(1000);
        }
        input.Unload();

        Console.WriteLine("Done!");
    }

}

}
`
Using this and not touching mouse or keyboard SendKey works just fine and does all 10 inputs.
Might be possible your PC default _keyBoardId is not 2. Try finding that and change that yourself.. After one real keyboard press I saved it and reuse it until it gets a new keyboard input. Perhaps the new default not working?

Thank you for checking. Turns out my application flow was off. I still don't understand what the actual cause of the issue was, but after moving stuff around a bit, everything is working as expected. As this is my first real shot at using C#, no real surprise there.

Thank you for checking. Turns out my application flow was off. I still don't understand what the actual cause of the issue was, but after moving stuff around a bit, everything is working as expected. As this is my first real shot at using C#, no real surprise there.

My best guess is something to do with a new thread being made when the library loads that just on an infinite loop. Sounds like whenever that loop loops (when a mouse or keyboard input is received.) your whole program resumes. Not sure what is going on, but this uses the lower level Thread and not TaskFactory.


[Feature Request] preserve comments

First off, super cool library. I use the field alignment govet check, and it's a pain to figure out the optimal struct order manually. With gopium, I was able to get going in ~5 minutes.
I ran the following command:
gopium \
    --walker_regexp='^InvoiceRow' \
    --package_path="$PWD/erp/integrate/gsheet" \
    ast_go erp/integrate/gsheet \
    memory_pack

The struct was successfully reordered to fix the govet field aligment check. However, gopium removed all comments from the struct field. It'd be nice to keep the documentation comments.
Other minor observations. Feel free to ignore. Including since sometimes it's helpful to hear about first-time usage:

It took me a while to figure out the correct invocation. The mix of positional commands like the walker and package and flag commands was confusing.  I think flags alone might be easier to understand:

gopium \
  --package_path=erp/integrate/gsheet \ # package path should probably be relative to $PWD
  --struct_regexp='^InvoiceRow$' \
  --strategies=memory_pack,strategy2

I had to use PWD to target the correct directory since gopium prepends src/.

I support the author. Mashing comments to the fields of the structure for me is the very problem that prevents me from using this cool utility for my project. Thanks!


Update code and metadata

The script is in production for far longer than initially expected. Brushing up the code itself, the documentation, and the repo automation seems prudent.
This is a collection ticket and will be updated with individual tasks. Each task may be handled separately if required.

[ ] minor version Py3.6

[ ] related stdlib (e.g. pathlib)
[ ] type annotations

[ ] gracefully handle failing refreshs if the cache was filled before

Seeing how this project is intended for and used in the HEP context, moving it to the MatterMiners group seems fitting.
Any objections @giffels, @eileen-kuehn?


compile error on ubuntu

Hi,
I am new to marvin, trying to install it on a ubuntu workstation. I think I had followed the readme and got cuda 7.5 and cudnn v4rc installed properly, downloaded the latest marvin-master, but it seems marvin has some issue with compiling:
lidong@T7600:~/marvin$ which nvcc
/usr/local/cuda-7.5/bin/nvcc
lidong@T7600:~/marvin$ echo $LD_LIBRARY_PATH
/usr/local/cuda-7.5/lib64:/usr/local/cudnn/v4rc/lib64:
lidong@T7600:~/marvin$ ./compile.sh
marvin.hpp(6274): error: argument of type "half *" is incompatible with parameter of type "cudnnTensorDescriptor_t"
marvin.hpp(6276): error: argument of type "double" is incompatible with parameter of type "void *"
marvin.hpp(6278): error: argument of type "half *" is incompatible with parameter of type "double"
marvin.hpp(6278): error: too few arguments in function call
4 errors detected in the compilation of "/tmp/tmpxft_00006d3a_00000000-9_marvin.cpp1.ii".
Anyone has any ideas?
Thanks a lot for any suggestions.
Regards,
lidong

Hi @lidong04 Marvin has been well tested on Ubuntu and it is hard to see what's going wrong for you. If you still have the problem, can you show some more information such as ubuntu version, marvin commit number?

Hi @fyu Thanks for your reply. I understand that Marvin is well tested on ubuntu, so I think I must have missed something - and here is my ubuntu version and marvin commit number:
Distributor ID:    Ubuntu
Description:    Ubuntu 14.04.3 LTS
Release:    14.04
Codename:    trusty
commit a79defb361009162baeeb83df97ef04cae312e90
Merge: 1fe98f2 9861e29
Author: Daniel Suo<EMAIL_ADDRESS>Date:   Thu Jan 7 17:34:52 2016 -0500
If it is ok, can you share your ubuntu/cuda version and/or settings which can run Marvin correctly? I can re-install ubuntu/cuda from scratch again, follow your version and/or settings.
Thank you very much,
lidong

Also, here is my nvcc version:
lidong@T7600:~/marvin$ nvcc --version
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2015 NVIDIA Corporation
Built on Tue_Aug_11_14:27:32_CDT_2015
Cuda compilation tools, release 7.5, V7.5.17

Just for the record, I am experiencing this issue with the exact same version of Ubuntu.

@danielsuo The problem occurs at batch normalization cuDNN call. Can you have a look at it?

I can confirm that rolling back before commit 9861e29ea6c924095d88ad0ace7870bc4a9d0134 fixes the issue.

We are not able to replicate the issue on Ubuntu 14.04.1 LTS or Ubuntu
15.04. Unlikely causes, but good to check:

Do you have any other versions of cuDNN installed that may be clobbering
v4rc?
Have you modified compile.sh in anyway?
Have you followed all the path installation directions from the README
https://github.com/PrincetonVision/marvin? (For example, being able to
echo the library path in command line does not guarantee that the
executable itself has access to the variable)

It is also possible that there is some difference between 14.04.3 and
14.04.1.
On Wed, Feb 24, 2016 at 4:55 PM, Daniel Petti<EMAIL_ADDRESS>wrote:

I can confirm that rolling back before commit 9861e29
https://github.com/PrincetonVision/marvin/commit/9861e29ea6c924095d88ad0ace7870bc4a9d0134
fixes the issue.
—
Reply to this email directly or view it on GitHub
https://github.com/PrincetonVision/marvin/issues/29#issuecomment-188469456
.

Hi @djpetti and @danielsuo Thank you very much for the comment, Rolling back before 9861e29 does fix the issue. It works fine now on my Ubuntu 14.04.3:)

Hmm... What version of gcc are people using?

This is fixed in commit 9a3e9eb37aa6fd038db586735e00a46ff49cfaab


Element "Option" is missing "description" property from Slack specifications

Hi, thanks for this great library. Just starting to use it and stumbled upon one issue.
I've found the Option element for radio buttons is missing the description property as supported by the Slack specification described @  https://api.slack.com/reference/block-kit/composition-objects#option.
Current Option representation in the library:
https://api.slack.com/reference/block-kit/composition-objects#option

Good spot @zzullick - this has been fixed in 3.12.1 and 3.13.0-beta2 👍


Add sar as default pbench tool we collect data from

I will be testing this just a little bit so please hold for now.

Testing looks good, it is ready for merge.


Create validator for 'required_with' with unit tests

validator and unit test:
https://github.com/macder/wp-form-validation/commit/8b888fe8b677b82e361d3710eff2be94c270feb2
https://github.com/macder/wp-form-validation/commit/4b7bca1002e479b604846850954f249e6b552384


Update tweeter IoT tutorials to point to new versioned directories

Description
Update for https://jira.mesosphere.com/browse/DOCS-2112
Tweeter repository versions the app definitions now. Updated the paths to specify that the ones corresponding to the DC/OS versions for 1.8 and 1.9 should be used. 1.10 and 1.11 work with the top-level tweeter files so those do not need to be changed.
Urgency

[ ] Blocker
[ ] High
[x] Medium

Requirements

Test all commands and procedures.
Build content locally and test for formatting/links.
Add redirects to dcos-website/redirect-files.
Change all affected versions (e.g. 1.7, 1.8, and 1.9).
See the contribution guidelines.

Note: need to add 1.7 tutorial update PR to the enterprise docs, and also this PR needs to have 1.10 updated once the related PR in the tweeter repo is merged.


Display documentation on Hover

When hovering over classes or functions, no documentation will be displayed, but the autocompletion does.
Should look as follows

Initial implementation in 2.3.0


fix(deps): update mantine monorepo to v7.13.5

This PR contains the following updates:

Package
Type
Update
Change

@mantine/core (source)
dependencies
patch
7.13.4 -> 7.13.5

@mantine/hooks (source)
dependencies
patch
7.13.4 -> 7.13.5

@mantine/modals (source)
dependencies
patch
7.13.4 -> 7.13.5

@mantine/notifications (source)
dependencies
patch
7.13.4 -> 7.13.5

@mantine/vanilla-extract (source)
dependencies
patch
7.13.4 -> 7.13.5

Release Notes

mantinedev/mantine (@mantine/core)
v7.13.5
Compare Source
What's Changed

[@mantine/core] Update peer dependencies range for react to allow react and react-dom 19 as dependcy
[@mantine/core] Fix error in Next.js with React 19 related to ref prop usage in Tooltip, Popover and Combobox components (#7028)
[@mantine/core] FileButton: Fix resetRef throwing error if the component is contidionally rendered (#7025)
[@mantine/core] Button: Fix incorrect focus styles of Button.Group (#6992)
[@mantine/charts] CompositeCharts: Fix missing key prop error (#7020)
[@mantine/core] NumberInput: Fix min/max value being bypassed if 0 has been entered as the first digit (#7021)
[@mantine/form] Add useCallback wrapper to form.resetDirty (#7029)
[@mantine/core] Combobox: Fix incorrect logic of selected options when the dropdown is closed without selecting value (#7039)
[@mantine/charts] BarChart: Add barLabelColor prop support
[@mantine/charts] BarChart: Fix bar label being positioned incorrectly with horizontal orientation (#7042)
[@mantine/charts] RadarChart: Fix incorrect series prop type (#7046)
[@mantine/form] Add additional type exports from the package (#7062)
[@mantine/core] Tabs: Fix tabIndex not being overridden by Tabs.Tab props  (#7081)
[@mantine/dates] DatePickerInput: Fix nextLabel and previousLabel props not being handled correctly (#7082)
[@mantine/charts] Update recharts dependency to the latest version to improve Next.js 15 and React 19 support

New Contributors

@pfo-omicsstudio made their first contribution in https://github.com/mantinedev/mantine/pull/7062
@zaphire12 made their first contribution in https://github.com/mantinedev/mantine/pull/7046
@fsd-niraj made their first contribution in https://github.com/mantinedev/mantine/pull/7042
@MariaBanaszkiewicz made their first contribution in https://github.com/mantinedev/mantine/pull/7039
@anthony-dandrea made their first contribution in https://github.com/mantinedev/mantine/pull/7029
@owenmoogk made their first contribution in https://github.com/mantinedev/mantine/pull/7021
@mariakax3 made their first contribution in https://github.com/mantinedev/mantine/pull/7020

Full Changelog: https://github.com/mantinedev/mantine/compare/7.13.4...7.13.5

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about these updates again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update artifacts related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: packages/mantine/package.json
Command failed: BUN_VERSION=v1.1.34 && BUN_INSTALL=/home/ubuntu/.local bash -c "$(curl -fsSL https://bun.sh/install)" -- "bun-$BUN_VERSION" 2>/dev/null

File name: packages/mantine/package.json
Command failed: bun --revision
/bin/sh: 1: bun: not found

File name: packages/mantine/package.json
Command failed: bun install --silent
/bin/sh: 1: bun: not found

File name: packages/mantine/package.json
Command failed: bun run syncpack
/bin/sh: 1: bun: not found

File name: packages/mantine/package.json
Command failed: bun run syncpack:format
/bin/sh: 1: bun: not found

File name: packages/mantine/package.json
Command failed: BUN_VERSION=v1.1.34 && BUN_INSTALL=/home/ubuntu/.local bash -c "$(curl -fsSL https://bun.sh/install)" -- "bun-$BUN_VERSION" 2>/dev/null

File name: packages/mantine/package.json
Command failed: bun --revision
/bin/sh: 1: bun: not found

File name: packages/mantine/package.json
Command failed: bun install --silent
/bin/sh: 1: bun: not found

File name: packages/mantine/package.json
Command failed: bun run syncpack
/bin/sh: 1: bun: not found

File name: packages/mantine/package.json
Command failed: bun run syncpack:format
/bin/sh: 1: bun: not found

File name: packages/mantine/package.json
Command failed: BUN_VERSION=v1.1.34 && BUN_INSTALL=/home/ubuntu/.local bash -c "$(curl -fsSL https://bun.sh/install)" -- "bun-$BUN_VERSION" 2>/dev/null

File name: packages/mantine/package.json
Command failed: bun --revision
/bin/sh: 1: bun: not found

File name: packages/mantine/package.json
Command failed: bun install --silent
/bin/sh: 1: bun: not found

File name: packages/mantine/package.json
Command failed: bun run syncpack
/bin/sh: 1: bun: not found

File name: packages/mantine/package.json
Command failed: bun run syncpack:format
/bin/sh: 1: bun: not found

File name: packages/mantine/package.json
Command failed: BUN_VERSION=v1.1.34 && BUN_INSTALL=/home/ubuntu/.local bash -c "$(curl -fsSL https://bun.sh/install)" -- "bun-$BUN_VERSION" 2>/dev/null

File name: packages/mantine/package.json
Command failed: bun --revision
/bin/sh: 1: bun: not found

File name: packages/mantine/package.json
Command failed: bun install --silent
/bin/sh: 1: bun: not found

File name: packages/mantine/package.json
Command failed: bun run syncpack
/bin/sh: 1: bun: not found

File name: packages/mantine/package.json
Command failed: bun run syncpack:format
/bin/sh: 1: bun: not found

File name: packages/mantine/package.json
Command failed: BUN_VERSION=v1.1.34 && BUN_INSTALL=/home/ubuntu/.local bash -c "$(curl -fsSL https://bun.sh/install)" -- "bun-$BUN_VERSION" 2>/dev/null

File name: packages/mantine/package.json
Command failed: bun --revision
/bin/sh: 1: bun: not found

File name: packages/mantine/package.json
Command failed: bun install --silent
/bin/sh: 1: bun: not found

File name: packages/mantine/package.json
Command failed: bun run syncpack
/bin/sh: 1: bun: not found

File name: packages/mantine/package.json
Command failed: bun run syncpack:format
/bin/sh: 1: bun: not found


Create function that can call itself

Description:
I think it is currently not possible to create a function that it allowed to call itself. I am trying to achieve this with the following template.
Resources:
  MyFunction:
    Type: AWS::Serverless::Function
    Properties:
      CodeUri: aws_lambda
      Handler: handler.run
      Runtime: python3.7
      MemorySize: 2048
      Policies:
        - LambdaInvokePolicy:
            FunctionName: !Ref MyFunction

This fails with the following message:
FAILED. Reason: Circular dependency between resources: [MyFunction, MyFunctionRole]

It looks to me like I cannot break this circular dependency. The function needs to know which role it should be created with and role defines the policy, which needs to know what function it refers to.
I'd be glad to hear that there is a way to do it currently. Otherwise, this would be a great feature to add.

To avoid the circular dependency issue, you will need to pass in the actual name of the function to the FunctionName parameter of LambdaInvokePolicy. There are two ways of doing this:

You can add a name to the function in the FunctionName parameter of the Serverless::Function resource and pass the same name into the FunctionName parameter of the LambdaInvokePolicy policy template.
You can deploy the function without the policy, copy the physical id of the function, then put the policy back in the template with the actual name of the function (without using !Ref).

Example:
  MyFunction:
    Type: 'AWS::Serverless::Function'
    Properties:
      FunctionName: MyFunctionNameAbc
      CodeUri: ./
      Handler: hello.handler
      Runtime: python3.6
      Policies:
        - LambdaInvokePolicy:
            FunctionName: MyFunctionNameAbc

This creates the following policy:
{
    "Statement": [
        {
            "Action": [
                "lambda:InvokeFunction"
            ],
            "Resource": "arn:aws:lambda:us-west-2:123456789012:function:MyFunctionNameAbc*",
            "Effect": "Allow"
        }
    ]
}


Use new test fixture in Cassandra controls

I'm trying to get rid of the Cassandra specific test fixture in ./pkg/controllers/cassandra/testing
In this branch I've switched:

ServiceAccounts
Roles
RoleBindings

I'll deal with Pilot and NodePool controls in separate branches.
Release note:
NONE

/test e2e

/test e2e v1.10

/retest

/lgtm
/approve


Blog App - creating a Data Model

Hey there 👋
This is my Blog_app, built using RoR. In thin branch, I used migration files to build my schema.
Here are the project requirements:

Be sure to reference the ERD diagram from the sneak peek.

Create a new app. Give it a meaningful name.
Initialize your project with Git.
Make sure that your project has Postgres database set up.
Build your project schema.
Create and run the necessary migration files.
Table and column names should match the ERD diagram.

note: photo for users table should be a link to an image

Foreign keys should be included.

Pay attention that in posts table the column holding the foreign key to the users table should be named author_id

All columns that are foreign keys should have a corresponding index.

Thanks for your kind review and feedback

Thanks @IngDavidTM !


Incorrect upper and lower values in the last example of the BitSlider demo page

Incorrect upper and lower values in the last example of the BitSlider demo page:

this issue is not valid anymore


[T2A5][W11]Ong Heng Le

Good work @initialshl .. There is inconsistency indentation in toLowerCaseStrings function in c08bb4d


chore: github action to run daily xcm tests

@sander2 , am I correct in my understanding that this completes #54 , or is there still some work missing to close that issue?


Matomo integrations

Hi,
many thanks for your great plugin. I have now added it to the list of Matomo integrations:
https://matomo.org/integrate/#programming-language-platforms-and-frameworks
Would it maybe be possible to rename the plugin to Matomo (even though that would be a breaking change)?
When I find some time I'll try it out as I am currently tracking my vue app with a kind of hackish setup.

Sure. I'll get right at it. Thanks for adding the plugin to your website.

It's done. https://github.com/AmazingDreams/vue-matomo


Send code coverage to Coveralls.io

Blocked on Adracus/dart-coveralls#17

Done.
Coverage data can be found here.


Revise the unit testing framework

The current unit tests depend on a lot of external dependencies. For example, some unit tests depend on packagematplotlib having mpl_toolkits in top_level.txt.
This is bad practice, it would be a lot better to not have these external dependencies for our unit tests, e.g. by creating mock '.venv' directories.

Omitted matplotlib, now the state of the unit tests seems quite good. Added codecov.


TTS issue within DLGaS (French LP)

Can't hear TTS within the DLG in demo client (TTSaaS only tab works fine). French language.
Project: 14901 (TTS modality is on in my project)
I do see the following error:

Any idea?
Thanks.
Caroline-

To verify...

What experience simulation is selected?
Did you provide the appropriate channel name that has the nlg message segments?

Hi Nirvana,
Simulate Experience: IVR: Voice & DTMF Input with Voice Output.
As regards channel names, I tried Default, Digital VA or IVR/Voice VA (channels that I have defined in my project and have TTS modality on). Can't hear any voice.


Query: add support for correlated collection optimization for queries with GroupBy

Currently, if query contains group by on the top level, correlated collection optimizations won't be applied - we will still produce N+1 queries.
example:
ctx.Gears.Select(g => new { g.Rank, g.Weapons }).GroupBy(g => g.Rank).ToList();

produces:
            AssertSql(
                @"SELECT [g].[Rank], [g].[FullName]
FROM [Gears] AS [g]
WHERE [g].[Discriminator] IN (N'Officer', N'Gear')
ORDER BY [g].[Rank]",
                //
                @"@_outer_FullName1='Augustus Cole' (Size = 450)

SELECT [w0].[Id], [w0].[AmmunitionType], [w0].[IsAutomatic], [w0].[Name], [w0].[OwnerFullName], [w0].[SynergyWithId]
FROM [Weapons] AS [w0]
WHERE @_outer_FullName1 = [w0].[OwnerFullName]",
                //
                @"@_outer_FullName1='Garron Paduk' (Size = 450)

SELECT [w0].[Id], [w0].[AmmunitionType], [w0].[IsAutomatic], [w0].[Name], [w0].[OwnerFullName], [w0].[SynergyWithId]
FROM [Weapons] AS [w0]
WHERE @_outer_FullName1 = [w0].[OwnerFullName]",
                //
                @"@_outer_FullName1='Damon Baird' (Size = 450)

SELECT [w0].[Id], [w0].[AmmunitionType], [w0].[IsAutomatic], [w0].[Name], [w0].[OwnerFullName], [w0].[SynergyWithId]
FROM [Weapons] AS [w0]
WHERE @_outer_FullName1 = [w0].[OwnerFullName]",
                //
                @"@_outer_FullName1='Dominic Santiago' (Size = 450)

SELECT [w0].[Id], [w0].[AmmunitionType], [w0].[IsAutomatic], [w0].[Name], [w0].[OwnerFullName], [w0].[SynergyWithId]
FROM [Weapons] AS [w0]
WHERE @_outer_FullName1 = [w0].[OwnerFullName]",
                //
                @"@_outer_FullName1='Marcus Fenix' (Size = 450)

SELECT [w0].[Id], [w0].[AmmunitionType], [w0].[IsAutomatic], [w0].[Name], [w0].[OwnerFullName], [w0].[SynergyWithId]
FROM [Weapons] AS [w0]
WHERE @_outer_FullName1 = [w0].[OwnerFullName]");

Not needed. We don't have correlated collection "optimization" anymore.
Linking to client group by issue #17068


Create new folder wiht Ctrl+Shift+N shortcut

Resolved / Related Issues

[x] Were these changes approved in an issue or discussion with the project maintainers? In order to prevent extra work, feature requests and changes to the codebase must be approved before the pull request will be reviewed. This prevents extra work for the contributors and maintainers.
Closes Feature: #10396

Validation
How did you test these changes?

[ X] Did you build the app and test your changes?
[ ] Did you check for accessibility? You can use Accessibility Insights for this.
[ ] Did you remove any strings from the en-us resource file?

[ ] Did you search the solution to see if the string is still being used?

[ ] Did you implement any design changes to an existing feature?

[ ] Was this change approved?

[x] Are there any other steps that were used to validate these changes?

Open app ...
Click settings button ...

Screenshots (optional)
Add screenshots here.
Made the changes to enable auto create folder when using ctrl + shift + n shortcut: Feature: #10396
Before making the changes:

After making the changes:

@kworede582 thank you for contributing and congrats on your first PR to Files! We still haven't approved the feature request in #10396 but I'll take a look and let you know what we decide.

Requirements fulfilled:

'Ctrl + Shift + n' shortcut will no longer display the new item dialog box with multiple/different options
Shortcut will now auto prompt user to enter the folder name

That's the current proposal but it's still in the planning stage (you can check the status in the right pane). In the meantime, let's leave this open and I'll let you know once we finish reviewing the request.


Herb v3 herb2

This branch has been used for demos on herb2. @pengjujin added launch files for Kinect, Multi Sense, and AprilTags. I added the launch file for R-CNN+MOPED module and new script files for working with tmux sessions.
I renamed starting scripts, so I think this branch should be merged after merging WAM branch #39 .

I addressed issues and updated README so that anybody can start HERB machines by using this repo.


Browser-compatible version

Description
I need to use a minimal subset of Sigstore function, only for validation, inside a browser extension. While there are polyfills and stuff like browserify, I did not have much success with them, and I think that when doing delicate crypto I should not rely on automatic fillings.
I would like to work on achieving this, and could use some guidance and insights on the possible issues and, if there is interest, how to structure things in order to have them integrated here.
So far, I have started porting the core package, and these are my ideal steps on which I have experimented a bit:

Swapping Buffer for the both node and browser native Uint8Array
Porting the crypto to the Web Crypto API
Build using vite
Port the tests to something vite-compatible

I think that overall, having a verifying only minimal implementation should not be a very long job.

Sample porting of core to Uint8Array only here: https://github.com/lsd-cat/sigstore-js/commit/934a73a769b6ff68f1389b5d23b125cd033f657f

I have worked both on points 2 and 3, also porting the already existing tests to vitest as possible.
At the moment, the Sigstore core module successfully builds with vite. Also, almost all tests are passing, with the exception of some verify cases returning false instead of true that I will investigate soon.
The modified crypto.ts contains the crypto logic using the Web Crypto API. It's not perfect, and it probably cannot ever natively supports all algorithms that are currently supported by Sigstore, however most standard cases (P-256, P-384, P-521 - SHA-256, SHA-384, SHA-512) are either already covered or should be doable.

Next step would be to port the verify package, and my understanding is that it depends on:

core -> we have it!
bundle -> if protobuf bindings works, then should be trivial
tuf -> depends on local files or external requests: it needs to be decided how to handle those
rekor-types -> should be trivial
protobuf-specs -> just need to find out if the currently built files work in browsers out of the box


  The meaning of the indexOrDestination is then determined by the value of the EmberOutgoingMessageType.
Proposed Solution
It might be better to have a struct that has an EmberOutgoingMessageType member and then a union of whatever things indexOrDestination is supposed to be able to represent, with accessors to get at various information.
I think indexOrDestination can be a NodeId or a GroupId or an index into the binding table.  @selissia does that sound about right?

I think indexOrDestination can be a NodeId or a GroupId or an index into the binding table. @selissia does that sound about right?

Yes, that's right. We discussed this on the #chipsilabsintegration a while ago (for some reason I can no longer see the old messages).
￼

for some reason I can no longer see the old messages

Because we're not paying Slack (enough?) so they only keep a certain number of messages.

I'm working on this, this will be a subtask of integrate ember with exchange layer.
I'll add a EmberContext to capture all short live variables, and it will replace indexOrDestination. The binding and the destination node id will be the member of EmberContext

Please don't use "Ember" in new names; we are generally aiming to eliminate that naming convention from the code, but it's waiting on us having control of the generator for the generated files so we can change those in sync with the non-generated ones.
Also, "Context" is kind of generic.... the right thing here is "MessageDestination" or the like, right?

Let's continue the discussion in the linked PR.

@kghost is this still required? Please re-open if needed! (Linked PR is closed)

@kghost is this still required? Please re-open if needed! (Linked PR is closed)

Yes, this is still needed.

Yes, this is still needed.

This is block by #4019, see https://github.com/project-chip/connectedhomeip/issues/4170#issuecomment-771352570


Upgrade adapter failed to load AMD module extending Array.prototype

My application is running in mixed mode using upgrade adapter provided by Angular team. I have the following AMD module defined:
define(function () {
    'use strict';

    Array.prototype.pushArray = pushArray;
    Array.prototype.clear = clear;
    Array.prototype.fill = fill;

    return;

    function pushArray() {
        this.push.apply(this, this.concat.apply([], arguments));
    }

    function clear() {
        this.length = 0;
    }

    function fill() {
        this.clear();
        this.pushArray.apply(this, arguments);
    }
});

When I load that using SystemJS module loader I got the following exception:
Unhandled Promise rejection: Error: Cannot resolve all parameters for 'Console'(?). Make sure that all the parameters are decorated with Inject or have valid type annotations and that 'Console' is decorated with Injectable.
at NoAnnotationError.BaseException [as constructor] (http://localhost:4200/vendor/@angular/core/src/facade/exceptions.js:17:23)
at new NoAnnotationError (http://localhost:4200/vendor/@angular/core/src/di/reflective_exceptions.js:217:16)
at _dependenciesFor (http://localhost:4200/vendor/@angular/core/src/di/reflective_provider.js:182:15)
at resolveReflectiveFactory (http://localhost:4200/vendor/@angular/core/src/di/reflective_provider.js:72:24)
at resolveReflectiveProvider (http://localhost:4200/vendor/@angular/core/src/di/reflective_provider.js:96:97)
at Array.map (native)
at Object.resolveReflectiveProviders (http://localhost:4200/vendor/@angular/core/src/di/reflective_provider.js:104:31)
at Function.ReflectiveInjector.resolve (http://localhost:4200/vendor/@angular/core/src/di/reflective_injector.js:357:38)
at Function.ReflectiveInjector.resolveAndCreate (http://localhost:4200/vendor/@angular/core/src/di/reflective_injector.js:387:62)
at Object.browserPlatform (http://localhost:4200/vendor/@angular/platform-browser/src/platform_browser.js:38:57)
Evaluating http://localhost:4200/main.js
Error loading http://localhost:4200/main.js ; Zone:  ; Task: Promise.then ; Value: Error: Error: Cannot resolve all parameters for 'Console'(?). Make sure that all the parameters are decorated with Inject or have valid type annotations and that 'Console' is decorated with Injectable.(…)
I have tried to debug the code and it looks like the error is comming from the following module:
resolvedDeps = _dependenciesFor(useClass); //useClass = function Console()
function _dependenciesFor(typeOrFunc: any): ReflectiveDependency[] {
var params = reflector.parameters(typeOrFunc);
if (isBlank(params)) return [];
if (params.some(isBlank)) {
throw new NoAnnotationError(typeOrFunc, params);
}
return params.map((p: any[]) => _extractToken(typeOrFunc, p, params));
}
reflective_provider.ts - resolveReflectiveFactory
reflective_provider.ts - resolveReflectiveProvider
reflective_providers.ts - resolveReflectiveProviders
reflective_injector.ts - ReflectiveInjector.resolve
reflective_injector.ts - ReflectiveInjector.resolveAndCreate
platform_browser.ts - browsePlatform
upgrade_adapter.ts - UpgradeAdapter.bootstrap

Sorry sir, this is not a support forum.
Is better for you to go to stackoverflow for that.
Thanks!


[4.25] Singleton Issues

This should be a PIE only issue.
Since the singleton isn't deconstructed between PIE sessions, the login-state can lead to errors.
We should be able to call the Logout function in our Initialization function, because it will be called during every PIE run too.
void UGameJolt::Initialize(const int32 game_id, const FString private_key, const FString server, const FString version)
{
    UGameJolt& API = UGameJolt::Get();
    API.Logout();
    API.GameID = game_id;
    API.PrivateKey = private_key;
    API.Server = server;
    API.Version = version;
}


[GSSoC'23] Crossword Game Completed

PR Description 📜
Title - Cross word Game
Tech used - HTML, CSS, and JS
Fixes #1572

Mark the task you have completed ✅

[x] I follow CONTRIBUTING GUIDELINE & CODE OF CONDUCT of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I have followed proper naming convention showed in CONTRIBUTING GUIDELINE
[x] I have added screenshot for website preview in assets/images
[x] I have added entries for my game in main README.md
[x] I have added README.md in my folder
[x] I have added working video of the game in README.md (optional)
[x] I have specified the respective issue number for which I have requested the new game.

Add your screenshots(Optional) 📸

Thank you soo much for contributing to our repository 💗

Thank you @Avdhesh-Varshney ,for creating the PR and contributing to our GameZone 💗
Review team will review the PR and will reach out to you soon! 😇
Make sure that you have marked all the tasks that you are done with ✅.
Thank you for your patience! 😀

@kunjgit please review this pull request and requested to merge it into the main branch of thie repository for a successful contribution under GSSoC'23 program.

Thank you  @Avdhesh-Varshney , for your valuable time and contribution in our GameZone 💗.
It’s our GameZone, so Let’s build this GameZone altogether !!🤝
Hoping to see you soon with another PR again 😇
Wishing you all the best for your journey into Open Source🚀

Hey @Avdhesh-Varshney,
You can't work on a closed issue 👀!
Follow the proper guideline and make a new PR again 😀.
Happy Hacking 💗

@kunjgit i have not worked on the closed issue.
I am always first see the issue in one tab before creating the pull request.
I had not broken any guideline.
Kunj, I requested to add the levels on this pull request so that it will comes on the leaderboard and count as a contribution under GSSoC'23.

@kunjgit @gurjeetsinghvirdee please add labels on this pr


Hello, when will you release the rest of the code and data?

Hi @weikang-wang ,
sorry for my late answer and thanks for pointing out the missing code! I will upload the full code on the weekend.
Thanks for your patience.
Best Regards,
Tilman

Hi, I believe that there is still some code missing from the repo. Will this be added soon?

@weikang-wang @tilmanbeck Please have a look at the updated code.


Add client connection check

Closes #215
Adds a check as part of the connect method to perform a test query. The BoM Analytics service provides the Yaml endpoint which just returns the docs, so the test checks that this query can be performed without raising an exception.
If an exception is raised, then separate error messages are raised depending on whether it's a 404 (service not installed) or something else (probably a 500, check the service layer logs).

Codecov Report

Merging #239 (414d702) into main (b2557fe) will increase coverage by 0.11%.
The diff coverage is 91.66%.

@@            Coverage Diff             @@
##             main     #239      +/-   ##
==========================================
+ Coverage   95.21%   95.32%   +0.11%
==========================================
  Files          10       10
  Lines        1358     1369      +11
==========================================
+ Hits         1293     1305      +12
+ Misses         65       64       -1

Impacted Files
Coverage Δ

src/ansys/grantami/bomanalytics/_item_results.py
90.86% <ø> (ø)

src/ansys/grantami/bomanalytics/indicators.py
93.10% <ø> (ø)

src/ansys/grantami/bomanalytics/queries.py
98.20% <50.00%> (+0.26%)
:arrow_up:

src/ansys/grantami/bomanalytics/_connection.py
88.28% <100.00%> (+1.15%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


refactor: decouple AuthoredObject configuration and from Tasks

Description

In some instances AuthoredObject types are used outside of the scope of tasks. Allowing the the tasks to directly take the AuthoredObject type allows these configured objects to be reused. It also decouples the task from the model specific AuthoredObject configuration.
One such use case is in PR #90
Type of change

[X] Breaking change (fix or feature that would cause existing functionality to not work as expected)

no end-user features added
How has this been tested?

[X] Unit test updates/existing cover these changes

Checklist

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

@JimFuller-RedHat PTAL


Failed to deploy

Hi I am deploying a Ruby script that connects to Telegram API.
To dokku@t.cloudapp.net:dev-0
 ! [remote rejected] master -> master (pre-receive hook declined)
error: failed to push some refs to 'dokku@t.cloudapp.net:dev-0'

Why am I receiving this error and how can I fix them?
Thanks

Check out the docs regarding troubleshooting and post a gist of your deployment output with dokku trace on. We'll need that to help you further.

Here's the log
Counting objects: 596, done.
Delta compression using up to 8 threads.
Compressing objects: 100% (182/182), done.
Writing objects: 100% (596/596), 115.76 KiB | 0 bytes/s, done.
Total 596 (delta 306), reused 596 (delta 306)
-----> Cleaning up...
-----> Building dev-0 from buildstep...
-----> Adding BUILD_ENV to build environment...
-----> Ruby app detected
-----> Compiling Ruby
       !
       !     An error occurred while installing Ruby ruby-2.0.0
       !     For supported Ruby versions see https://devcenter.heroku.com/articles/ruby-support#supported-runtimes
       !     Note: Only the most recent version of Ruby 2.1 is supported on Cedar-14
       !     Command: 'set -o pipefail; curl --fail --retry 3 --retry-delay 1 --connect-timeout 5 --max-time 30 https://s3-external-1.amazonaws.com/heroku-buildpack-ruby/cedar-14/ruby-2.0.0.tgz -s -o - | tar zxf - ' failed unexpectedly:
       !
       !     gzip: stdin: invalid compressed data--format violated
       !     tar: Unexpected EOF in archive
       !     tar: Unexpected EOF in archive
       !     tar: Error is not recoverable: exiting now
       !
To dokku@t.cloudapp.net:dev-0
 ! [remote rejected] master -> master (pre-receive hook declined)
error: failed to push some refs to 'dokku@t.cloudapp.net:dev-0'

related issues: #1304 #909 (i'm sure others)

make sure you're on the latest release
try setting $CURL_TIMEOUT to something high
try using the latest buildpack by setting $BUILDPACK_URL

@michaelshobbs I will try that out. So for the CURL_TIMEOUT, I can set that before the app exist am I right?

Something like this
dokku apps:create app_name
dokku config:set app_name CURL_TIMEOUT=600

Ok I will try that later. Hopefully that works

@michaelshobbs I keep trying what the other issues have suggested. I still get the following errors.
Counting objects: 3, done.
Delta compression using up to 8 threads.
Compressing objects: 100% (3/3), done.
Writing objects: 100% (3/3), 376 bytes | 0 bytes/s, done.
Total 3 (delta 0), reused 0 (delta 0)
-----> Cleaning up...
-----> Building notifycenters from buildstep...
-----> Adding BUILD_ENV to build environment...
-----> Fetching custom buildpack
To dokku@jumpset.com:notifycenters
 ! [remote rejected] master -> master (pre-receive hook declined)
error: failed to push some refs to 'dokku@jumpset.com:notifycenters'

It looks like it's failing to download the custom buildpack? Make sure the buildpack_url is valid.

I am pretty sure it is valid.

If I were you, I would spin up a docker container on that host and validate you can download it then. This seems quite similar to other issues we've seen when IPv6 is enabled or possibly some other networking issue.

hmm alright I'll try it. thanks!

@kennetham did you get any further?

@michaelshobbs I haven't had time to deploy to docker yet. I will update you once I have the results. Thanks for asking.

Closing as there hasn't been any movement in a month. @kennetham let us know if you find out what was occurring.

sure thanks!


Unable to scroll normally when clicking on the map tab (safari 9.0.3)

This happens many times. When I click on the map tab, I cannot scroll down the list on the right. But I can scroll up normally.

It seems that the "menu" button displays abnormally as well.

cc @hallaathrad

More details:
Initially, I open this page, and then click on the map tab.
Now, I can scroll up and down normally.

Then I click on one of the challenges.
In the new page, I can only scroll up. But I cannot scroll down.

@hallaathrad It seems to be related to the height applied to the iframe element at:
.map-aside .iframeWrapper iframe
If I change 'height: 100%tomin-height: 500px`, it will scroll.

hahaha that sidecar thing has been a nightmare. My thoughts if iframes had mothers.
I've been able to replicate and will work on a fix :+1:
Thanks for your patience, all! We're almost there.

Status update for all those affected/interested: I am still working on a fix for this, which includes rebuilding the actual layout for the map and the new wiki. Thanks for your patience!

@hallaathrad a little off topic. But what was the "Oops of Doom"?

@M1kep RE https://github.com/FreeCodeCamp/FreeCodeCamp/issues/6960 and similar.

James Bryson


perf: remove unrequired cloning of ctx.res

Response constructor's 2nd argument does not require a cloned response as all it does is read these fields and does not modify anything:

status
statusText
headers

https://fetch.spec.whatwg.org/#initialize-a-response

Hi @JakeChampion !
Thank you!  I'll merge it.


Add some unit test

Screenshot

@zploskey done.

Overall this looks great. There are a couple of minor issues that probably would have been caught by CI. That's something I plan to set up.

@zploskey All fixed.


[Bug]: selecting a combobox option with Enter submits the form

Environment
Radix Vue version: 1.9.5
Vue version: 3.4

Link to minimal reproduction
https://stackblitz.com/edit/ubaikc?file=src%2FApp.vue
Steps to reproduce

Create a <form> element that has a <button type="submit" /> inside it
Add a combobox inside the form
Navigate combobox options using keyboard, then press Enter to make a selection

Describe the bug
Pressing Enter when making a selection in the combobox will also submit the form, which is an unexpected behavior.
Expected behavior
Pressing Enter to make a selection should not submit the underlying form.
Combobox Root context should stop event propagation, when user is making a selection. In other cases, it should not handle enter keypress.
For reference, this is what Headless UI does - it stops the propagation, and prevents the default handling.
Context & Screenshots (if applicable)
No response

@ragulka  thanks for the isuse and PR. However we don't want to prevent default event nor stop propagation event unnecessarily.
In your case you should be able to have @keydown.enter.stop.prevent for ComboboxInput easily right?

In your case you should be able to have @keydown.enter.stop.prevent for ComboboxInput easily right?

Unfortunately, that won't work. The event should stop propagation only - and only if - the user is making a selection from the options. The rules are simple:

Combobox input has focus, suggestions is open and user presses Enter (makes a selection): stop propagation and prevent default.
Combobox input has focus, suggestions are not open and user presses Enter: do not stop propagation or prevent default.

Unfortunately, when adding @keydown.enter.stop.prevent, it will stop propagation and prevent default on all enter keydown events, regardless if user is making a selection, or not. Even trying to handle this conditionally in my own code is not possible, because the event carries no information wether the user made a selection or not (this is only determined by the keydown handler in ComboboxRoot, but not exposed in any way that would help us in userland code in the keydown event handler).
The only place where handling this makes sense, is in the library code, as I've implemented it in the PR.
This is also how HeadlessUI for example handles enter keydown events - it only stops propagation and prevents default when user makes a selection, never on any other occasion.

Appreciate the clear differentiation here @ragulka . I understand the issue you are facing now 😁


Increase coverage with closing #271

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

Tests pass?
✔

Fixed issues
#271

👍


Ignore mypycache

This simply ignores the cache dir created by the mypy checks.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4939

Relevant Lines:
6093

💛  - Coveralls


Run tutorial about gz-sim-yarp-plugins

https://github.com/robotology/gz-sim-yarp-plugins/tree/main/tutorial

I'm trying to run the Camera model tutorial, following https://github.com/robotology/gz-sim-yarp-plugins/tree/main/tutorial/camera/model.
When I launch the command gz sim model.sdf  gazebo sim open but the terminal writes the following errors:
[Err] [SystemLoader.cc:119] Failed to load system plugin: (Reason: Could not find shared library)
- Requested plugin name: [gzyarp::Camera]
- Requested library name: [gz-sim-yarp-camera-system]
- Library search paths:
  - /home/acroci/.gz/sim/plugins/
  - /home/acroci/mambaforge/envs/gz7/lib/gz-sim-7/plugins/
  - /home/acroci/.ignition/gazebo/plugins/
[Err] [SystemLoader.cc:119] Failed to load system plugin: (Reason: Could not find shared library)
- Requested plugin name: [gzyarp::RobotInterface]
- Requested library name: [gz-sim-yarp-robotinterface-system]
- Library search paths:
  - /home/acroci/.gz/sim/plugins/
  - /home/acroci/mambaforge/envs/gz7/lib/gz-sim-7/plugins/
  - /home/acroci/.ignition/gazebo/plugins/

It is similar to the errors I got in gazebo classic tutorials when the environment variables could not find the models and worlds of the tutorials.
@traversaro do you know if gazebo sim works through the same environment variables used in gazebo classic? I could not find helpful documentation on this topic.

Did you set the GZ_SIM_SYSTEM_PLUGIN_PATH env variable as described in the readme of the repo?

Did you set the GZ_SIM_SYSTEM_PLUGIN_PATH env variable as described in the readme of the repo?

Thank you that was the problem! I forgot this step when I created the new environment. I put the export command in my env activation scripts.

Here two things that happened when launching gz sim model.sdf:
Segmentation fault
gz sim model.sdf
[INFO] gz-sim-yarp-camera-system: configuration of sensor  camera_sensor  loaded from yarpConfigurationString :  (yarpDeviceName camera_plugin_device) (parentLinkName link_1) (sensorName camera_sensor) (display_timestamp)
[DEBUG] |yarp.dev.PolyDriver|gazebo_camera| Parameters are (device gazebo_camera) (display_timestamp) (parentLinkName link_1) (sensorName camera_sensor) (sensorScopedName "model/camera_model/link/link_1/sensor/camera_sensor") (sensor_name camera_sensor) (yarpDeviceName camera_plugin_device)
[INFO] |yarp.dev.PolyDriver|gazebo_camera| Created device <gazebo_camera>. See C++ class CameraDriver for documentation.
[INFO] gz-sim-yarp-camera-system: Registered YARP device with instance name: model/camera_model/link/link_1/sensor/camera_sensor/camera_plugin_device
[DEBUG] Reading file camera_nws.xml
[DEBUG] yarprobotinterface: using xml parser for DTD v3.x
[DEBUG] Reading file camera_nws.xml
[INFO] Yarprobotinterface was started using the following enable_tags:
[INFO] Yarprobotinterface was started using the following disable_tags:
[DEBUG] List of all enable attributes found in the include tags:
[DEBUG] List of all disable attributes found in the include tags:
[DEBUG] Preprocessor complete in:  7.15256e-07 s
[INFO] startup phase starting...
[INFO] Opening device camera_nws_yarp with parameters [("robotName" = "camera"), ("name" = "/camera"), ("period" = "0.033333"), ("capabilities" = "COLOR")]
[DEBUG] |yarp.dev.PolyDriver|camera_nws_yarp| Parameters are (capabilities COLOR) (device frameGrabber_nws_yarp) (id camera_nws_yarp) (name "/camera") (period 0.033333000000000001517) (robotName camera)
[INFO] |yarp.os.Port|/camera/rpc| Port /camera/rpc active at tcp://<IP_ADDRESS>:10002/
[INFO] |yarp.os.Port|/camera| Port /camera active at tcp://<IP_ADDRESS>:10003/
[INFO] |yarp.device.frameGrabber_nws_yarp| Running, waiting for attach...
[INFO] |yarp.dev.PolyDriver|camera_nws_yarp| Created wrapper <frameGrabber_nws_yarp>. See C++ class FrameGrabber_nws_yarp for documentation.
[INFO] Entering action level 5 of phase startup
[INFO] Executing attach action, level 5 on device camera_nws_yarp with parameters [("device" = "camera_plugin_device")]
[WARNING] |yarp.device.frameGrabber_nws_yarp| Targets has not IVisualParamInterface, some features cannot be available
[INFO] All actions for action level 5 of startup phase started. Waiting for unfinished actions.
[INFO] All actions for action level 5 of startup phase finished.
[INFO] startup phase finished.
Stack trace (most recent call last) in thread 22541:
#6    Object "", at 0xffffffffffffffff, in
#5    Source "../sysdeps/unix/sysv/linux/x86_64/clone3.S", line 81, in __clone3 [0x7f07cc92684f]
#4    Source "./nptl/pthread_create.c", line 442, in start_thread [0x7f07cc894ac2]
#3    Source "../../../../../libstdc++-v3/src/c++11/thread.cc", line 104, in execute_native_thread_routine [0x7f07c7dd9e94]
#2    Object "/home/acroci/mambaforge/envs/gz7/lib/libgz-sim7.so.7.6.0", at 0x7f07c6dcc592, in gz::sim::v7::SimulationRunner::ProcessSystemQueue()::{lambda()#1}::operator()() const
#1    Object "/home/acroci/mambaforge/envs/gz7/lib/libgz-sim-yarp-camera-system.so", at 0x7f0797d4ee20, in non-virtual thunk to gzyarp::Camera::PostUpdate(gz::sim::v7::UpdateInfo const&, gz::sim::v7::EntityComponentManager const&)
#0    Source "../sysdeps/x86_64/multiarch/memmove-vec-unaligned-erms.S", line 801, in __memmove_sse2_unaligned_erms [0x7f07cc8c4c4d]
Segmentation fault (Address not mapped to object [0x7f07c62c25e0])

Closing and reopening the terminal tab seems to resolve the issue.
Strange jitter in Gazebo sim GUI
I noticed a strange behavior in gazebo sim GUI in which the play/pause button of the simulation seemed to change state in milliseconds and does not allow me to start the simulation properly. Maybe an issue with the gpu or some drivers?
https://github.com/robotology/gz-sim-yarp-plugins/assets/57228872/b6919412-b6c2-4581-9212-0a34c8e82bd3

Strange jitter in Gazebo sim GUI
I noticed a strange behavior in gazebo sim GUI in which the play/pause button of the simulation seemed to change state in milliseconds and does not allow me to start the simulation properly. Maybe an issue with the gpu or some drivers?

Rebooting my PC this is not happening anymore.

https://github.com/robotology/gz-sim-yarp-plugins/blob/main/tutorial/forcetorque/model_two_sensors/README.md?plain=1#L25-L41
Maybe it's more clear to name the two /read ports differently in order to not have clashing names if the user wants to read the two torque sensors simultaneously.

https://github.com/robotology/gz-sim-yarp-plugins/blob/main/tutorial/forcetorque/model_two_sensors/README.md?plain=1#L25-L41
Maybe it's more clear to name the two /read ports differently in order to not have clashing names if the user wants to read the two torque sensors simultaneously.

Good idea, feel free to open a PR!

I've launched all the tutorials. They all worked as expected.
Now I'm looking through the code of the plugins, for specific topics I will open new dedicated issues.

Turials have been ran successfully. The issues reported in https://github.com/robotology/gz-sim-yarp-plugins/issues/57#issuecomment-1886923601 have not reproduced. Closing.


Data Platform: Offer users ideas on missing indexes

Problem / Solution
This is not an easy problem, but a feature I would be willing to pay for in the proxy/data platform you are building would be a query analysis that offers suggestions on missing indexes that would speed the optimization process.
Better even if I can see the query plans with problematic sections (usually sequential scans), fork the db (with copy on write so it's instant), add the new index and see the query plan.
Alternatives
Currently I am using the GCP query insight which is decent, but I still need to analyse the query plan, clone my db (pretty slow on gcp), connect with an external tool, create the index and then re-try my query.

Hello Emile!
Thank you for your feedback and for providing details about your current workflow, this helps us categorize and understand insights like yours. This is indeed a candidate solution that we are considering for the mid-term (think "a couple of quarters" time horizon) and I would love to jump in a quick call with you to learn more about your current workflow and also how you would imagine this working for you: https://calendly.com/martzoukos/45-min-discovery-call
Otherwise, if you don't have time now, may we get in touch with you in the future once we have something like a beta version of such a feature for you to try it out?

Hello @martzoukos!
Happy to take a call with you, I am also in the prisma company slack if you want to reach me (channel #collaboration-emile). This is just a vague idea I had while working on optimisation for a client (I opened a bunch of issues related to that at the same time as this one). I thought it would be valuable to share since it's a problem I expect others will encounter and an argument in favour of using your managed service.


Added preset for allotments=plot.

This PR adds preset for allotments=plot.
Related #4917.

Thanks @vershwal!
From reading the wiki page, I think this preset is supposed to be its own feature, not combined with landuse=allotments.
so move:
data/presets/presets/landuse/allotments/plot.json
to
data/presets/presets/allotments/plot.json
..and remove all references to landuse in that file.  It should just set the allotments=plot tag by itself.

Hey @bhousel!!
Thanks for reviewing. I have made the required changes. Please check.

Perfect @vershwal - thanks again!


Connecting Reseting

Ver 0.2.2
Single Streamdeck connected over LAN to companion.
Every 60 secs a reconnection seems to occur.

I don't know why this would be happening. Have you checked the companion and companion remote logs to see if any errors are reported?

@JPratchett are you still experiencing this issue?

There are enough details here to be able to reproduce anything


[OFFICIAL] Generated Issue: Error

Please fill out the details below.
Summary:
Steps to reproduce:
Additional Details
Jikan Parser Version: v4.0.5@6be2fa68bacf7abe7c51d5a0633ffc218aea078b
PHP: 8.1.18
Redis: Connected
Exception: Error
Code: 0
Message: Call to undefined method Jikan\MyAnimeList\MalClient::getRecentReviews()
Trace: /var/www/jikan-rest/app/Http/Controllers/V4DB/ReviewsController.php on line 76
Request: GET /v4/reviews/anime

Duplicate https://github.com/jikan-me/jikan-rest/issues/344


Missing return in drivers.html

makeSockDriver should return something, but I'm not 100% that this is right.

Yes, this is correct. Thanks!


Bump knative/pkg and cosign dependencies

Bumps to knative/pkg @ release-1.6 and cosign@latest (1.10.1)
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Has Docs included if any changes are user facing
[ ] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[x] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

This is running into the same problem as https://github.com/tektoncd/chains/pull/514 😭
We need Go 1.18 but the CI images are on 1.17. I made a PR to bump it, but it was causing issues with other tekton projects - https://tektoncd.slack.com/archives/CJ4ERJWAU/p1660211899070299 for

This is running into the same problem as #514 😭
We need Go 1.18 but the CI images are on 1.17. I made a PR to bump it, but it was causing issues with other tekton projects - https://tektoncd.slack.com/archives/CJ4ERJWAU/p1660211899070299

Yeah, sounds like there's some ongoing work @abayer is doing to get us up onto 1.18. I'm going to stop pushing these things for now until these land.

--- FAIL: TestNoDeps (0.59s)
    --- FAIL: TestNoDeps/github.com/tektoncd/chains/cmd/controller (0.59s)
        depcheck.go:126: CheckNoDependency() = github.com/tektoncd/chains/cmd/controller depends on banned dependency github.com/golang/glog
            github.com/tektoncd/chains/cmd/controller
            github.com/tektoncd/chains/pkg/reconciler/taskrun
            github.com/tektoncd/chains/pkg/chains
            github.com/tektoncd/chains/pkg/chains/signing/x509
            github.com/sigstore/cosign/cmd/cosign/cli/fulcio  # Also: [github.com/sigstore/cosign/cmd/cosign/cli/options github.com/sigstore/cosign/pkg/cosign/git github.com/sigstore/cosign/pkg/cosign/git/github github.com/sigstore/cosign/pkg/cosign/git/gitlab github.com/sigstore/cosign/pkg/cosign/kubernetes github.com/sigstore/cosign/pkg/signature github.com/tektoncd/chains/pkg/chains github.com/tektoncd/chains/pkg/chains/signing/x509]
            github.com/sigstore/cosign/pkg/cosign
            github.com/sigstore/cosign/cmd/cosign/cli/fulcio/fulcioverifier/ctl
            github.com/google/certificate-transparency-go/x509util
            github.com/golang/glog
FAIL
FAIL	github.com/tektoncd/chains/cmd/controller	1.094s
FAIL

😭

Closing this in favor of https://github.com/tektoncd/chains/pull/514
If there are more bumps you want to do, feel free to open another PR!


Added load/save support

Hi, I've added support for load/save, with the accompanying tests. A few notes:

DockerClient.load allows loading the tarball of an image as a stream. An image name must be passed in, and the load implementation internally tags the just loaded image with the provided name. The load implementation is based on the pull resource of the  Docker API, which takes a fromSrc parameter, indicating it to treat the request's body as the .tar payload of the image.

DockerClient.save allows pulling the .tar stream of an image (and saving it locally or whatever).

The above mimick the load and save commands of Docker's command-line (hence the naming).

Hmm. Got CI environment-related failures methinks: https://circleci.com/gh/spotify/docker-client/1019
I've hack the tests and will submit another pull request.


Fix issues with database; make it easier to test

These changes fix compatibility issues with common SQL databases including MySQL, MariaDB and Postgres.

Fixes #147 - datetime now ANSI SQL timestamp
Fixes #160 - AccessToken and RefreshToken type change from varchar to text
Adds Docker Compose files to make it easier to test database integration.

TODO:

Update documentation with configuration examples and latest compatibility info
Create DB URI parser (currently only object config works)
Database table/collection name prefix (will default to next-auth_)
MongoDB support

MongoDB has some issues which mean it will require additional work and refactoring to support (while preserving SQL DB support, which is important).
It's going to take some thinking about to get right; MongoDB support might have to be dropped from 2.0 (and follow in a subsequent release) but I'm going to review options and consider the impact before making a call.

I've been leaving this till nearer release as knew the internal API would change a bit.
Still needs a shakedown and would appreciate anyone who sees this testing it and checking out the tables, etc that get created, but is not as broken now!
I need to add this to the docs, but for now:

npm run test:db:start it will start MySQL, Postgres and MongoDB running in isolated Docker containers for testing.

Ctrl-C it will stop the databases.

Example database objects that can be used with the test databases:
// npm i mysql
const mysql = {
  type: 'mysql',
  host: "<IP_ADDRESS>",
  port: 3306,
  username: "nextauth",
  password: "password",
  database: "nextauth",
  synchronize: true
}

// npm install pg
const postgres = {
  type: 'postgres',
  host: "<IP_ADDRESS>",
  port: 5432,
  username: "nextauth",
  password: "password",
  database: "nextauth",
  synchronize: true
}

Am sure will be other updates to come in like 'created' and 'last modified' timestamps. I'm waiting to see how much hassle parallel MongoDB support is going to prove in practice before I decide that; I think I know how to handle it though…

Published as<EMAIL_ADDRESS>


add token authentication

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@c8c0c6b). Click here to learn what that means.
Patch has no changes to coverable lines.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff            @@
##             master      #14   +/-   ##
=========================================
  Coverage          ?   63.93%
=========================================
  Files             ?        3
  Lines             ?       61
  Branches          ?        0
=========================================
  Hits              ?       39
  Misses            ?       22
  Partials          ?        0

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


Fix 'port info' output for maintainers and long_description

After 30c27d5d3ad169ffa5f55465cf9663dbd1ff7537, the output of port info for maintainers is broken for some cases. This tries to resemble the old output of 2.3.4 as closely as possible.
While I am at it, also fix the output for long_description that should not contain multiple lines unless the output is pretty-printed.
See: https://trac.macports.org/ticket/52928
Review-Request: @neverpanic

Random thought. I would find it useful to be able to use something like --email to get just the list of valid emails and --github to get just the list of valid github handles, without any Email: prefix and alike.

@raimue I know you just mentioned it on IRC, but could you provide an example of the new output here, for others' benefit?

$> port info
...
Maintainers:          Email<EMAIL_ADDRESS>GitHub: neverpanic
                      Policy: openmaintainer
$> port info --maintainer
maintainer<EMAIL_ADDRESS>neverpanic, openmaintainer
$> port info --maintainer --line
<EMAIL_ADDRESS>neverpanic,openmaintainer

For maintainers:
$ port info --maintainer nginx
maintainer<EMAIL_ADDRESS><EMAIL_ADDRESS>openmaintainer
$ port info --line --maintainer nginx
cal@macports.org,mps@macports.org,openmaintainer
$ port info --pretty --maintainer nginx
Maintainers:          Email<EMAIL_ADDRESS>                      Email<EMAIL_ADDRESS>                      Policy: openmaintainer

For long_description:
$ port info --long_description dummy
long_description: This is the first line. \nAnd this is the second line.
$ port info --line --long_description dummy
This is the first line. \nAnd this is the second line.
$ port info --pretty --long_description dummy
Description:          This is the first line.
                      And this is the second line.

For @mojca's idea, maybe we could add new fields such as --maintainers-email, --maintainers-github, --maintainers-policy. Although a regex on the result would usually be enough to separate emails from other values. In any case, I would consider this an enhancement that should be handled after we have merged this bugfix. Please open a new ticket for it, so we do not forget about it.


Update Test Matrix Rubies

add 2.5.0

I don't recall what the policy on version numbers in the test matrix is, but the existing entries have all fallen behind their branch's stable release.

bump 2.2.8 to 2.2.9 (soon to be EOL'd)
bump 2.3.5 to 2.3.6
bump 2.4.2 to 2.4.3

Closed by #605.


Code equals nil in WorkOS::MFA.challenge_factor response

Hey, I'm using WorkOS MFA in my customer project & utilise Ruby gem. Everything works as expected but I was able to find one confusing response:
When running a challenge_factor request I can see an attribute code: nil for WorkOS::Challenge in response (and it returns nil for this attribute for any auth_factor_id), example:
irb(main):003> auth_factor_id = Member.find(2).mfa_sms_auth_factor_id
=> "auth_factor_01J50D363JX20ZVRZE2M2P****"

irb(main):004> challenge = WorkOS::MFA.challenge_factor(authentication_factor_id: auth_factor_id)
=>
#<WorkOS::Challenge:0x00000001223d0248
...

irb(main):005> challenge
=>
#<WorkOS::Challenge:0x00000001223d0248
 @authentication_factor_id="auth_factor_01J50D363JX20ZVRZE2M2P****",
 @code=nil,
 @created_at="2024-08-12T12:39:09.808Z",
 @expires_at="2024-08-12T12:49:09.809Z",
 @id="auth_challenge_01J539JZ7MH51DWBTDJ1W6****",
 @object="authentication_challenge",
 @updated_at="2024-08-12T12:39:09.808Z">

I can see that Code is NOT covered in the Authentication Challenge documentation as well:
https://workos.com/docs/reference/user-management/mfa/authentication-challenge

I would suggest to either remove Code from response or to document it properly.

Thanks for the report! That's indeed an unused parameter, I'll make a note to remove this in the next major version.


Add my own silly chrome remote debugging project.

This is a PR to add the python remote debug protocol wrapper I've been poking about with for the last 8+ months or so.
It's half horrible python AST abuse, but it should allow the protocol wrapper to be regenerated on-the-fly for the active chromium version once https://codereview.chromium.org/2867073006/ (Chromium 60) lands.
At this point, I'm actively using it, and a lot of the higher-level automation is still under development, but as a convenient way to just use chromium in a single-tab manner (multiple tabs are present, very WIP), it's quite nice. I even have docs!

I signed it!

?

@fake-name i am not very happy with having 4 python projects in here with fairly similar goals. However, I'm not a python developer, so I'm not in the best place to evaluate the strengths of each project. Can you suggest a project or two that i should remove in exchange for yours?

Looking at them, I don't see them being that similar at all, really..

ChromeWhip is the most similar to what my project implements, but they're seeking to re-implement a specific API, rather then just exposing the devtools API itself (which is my goal).
PyChrome is much lower level, it primarily looks to allow calling arbitrary functions via the websocket transport.
PyChromeDevTools is very similar to PyChrome.

Realistically, I could probably actually place my project on top of either PyChrome or PyChromeDevTools, using them as the underlying communication transport. However, I've also recently been putting a fair bit of work into the effort of managing executing chrome from python, and managing it's life-cycle and proper tear-down. From a quick skim of the other options, none of them do this.
If I were forced to chose, I'd say that PyChrome and PyChromeDevTools very much replicate each-other more then the other projects.
Realistically, I'd group them into two groups: "Transport", which handle the banal internals of talking to chromium, but don't do things like function name/parameter checking, and "High-Level", which seek to expose the fully structured debugging interface in python. There's another transport chrome module chrome_remote_shell, which I admit I actually started with as a base.
I'd group PyChrome, PyChromeDevTools, and chrome_remote_shell as "transport" libraries, while ChromeWhip and my own project are "high-level" interface wrappers.
I'm not sure if this helps.

@fake-name this does help. thanks for that breakdown.

?


Flow Types not displaying in PropTypes table in addon-info

If you are reporting a bug or requesting support, start here:
Bug or support request summary
Flow types don't show up in addon-info props table.
Please provide issue details here - What did you expect to happen? What happened instead?
The documentation seems to suggest Flow types should work analogously to PropTypes.
Steps to reproduce
Please provide necessary steps for reproduction of this issue. Describe the exact steps a maintainer has to take to make the problem occur. If the problem is non-trivial to reproduce, please link a repository or provide some code snippets.
(A screencast can be useful for visual bugs, but it is not a substitute for a textual description.)
Please specify which version of Storybook and optionally any affected addons that you're running

storybook/addon-info ^3.4.0
storybook/react ^3.4.0

Affected platforms

If UI related, please indicate browser, OS, and version
If dependency related, please include relevant version numbers
If developer tooling related, please include the platform information

Chrome on Mac
Screenshots / Screencast / Code Snippets (Optional)
// @flow

import * as React from 'react';
import styled from 'styled-components';

type Button = {
  margin?: string,
  bgColor?: string,
  hoverColor?: string,
}

export const StyledButton = (styled.button`
  margin: ${props => (props.margin ? props.margin : '10px')};
  height: 40px;
  max-width: 254px;
  text-transform: uppercase;
  text-decoration: none;
  text-align: center;
  cursor: pointer;
  border-radius: 2px;
  background-color: ${props => (props.bgColor ? props.bgColor : 'rgb(255, 168, 39)')};

  &:hover {
    background: ${props => (props.hoverColor ? props.hoverColor : '#81A2CA')};
  }
`: React.ComponentType<Button>);

End bug report support request - delete the rest below

will be fixed by #3403, we currently don't support named exports.

It doesn't seem to work with default exports either:
type Button = {
  /** Button label */
  children: string,
  /** The color for the button */
  color?: string,
  /** The size of the button */
  size?: string,
  /** Gets called when the user clicks on the button */
  onClick: () => void
};
/**
 * The only true button.
 */
const Button = ({ color, size, onClick, children }) => {
  const s = size ? size : "small";
  const styles = {
    color: color || "cornsilk",
    fontSize: Button.sizes[s]
  };

  return (
    <button className="button" style={styles} onClick={onClick}>
      {children}
    </button>
  );
};

export default Button;

yeah.. flow support is kind of wonky right now. once I get that version in and published, please test it for me.

any news about this bug?

@ruanmer it should be available in the 4.0.0-alpha builds

I was able to make it work putting the react-docgen plugin in my babelrc (#3481).
//.babelrc
{
  ...
  "plugins": [
    "react-docgen"
  ]
}

@danielduan should it work in v3.4.10?


Option to set a separate direction for baseline screenshot

Hello developers.
I hope you're doing well. I'd like to request the addition of a feature that allows to specify a separate path for storing baseline screenshots in the configuration file.
This feature would be helpful for organizing a project, as it gives the opportunity to keep and share only the baseline screenshots more efficiently. E.g.,  I faced the issue that I needed to store baseline screenshots in a cloud, where everybody can get them,  and other screenshots can be stored locally because there is no sense in sharing them
Thank you for considering this request. Looking forward to your response.
Best regards,
Anastasiya

Hi @Nastya9js , I'm not sure if I get you right, but I assume you could just push only the baseline changes to the cloud and ignore diff and comparison folders altogether, just like you would with your version control system. Am I missing something here?


Don't show money transfers in ticker

Currently, money transfers are shown in the ticker at the bottom of the page. This is bad for obvious privacy reasons.

also dont show checkouts


chore: deprecate blog routes

The blog is moving to the new infrastructure. This PR deletes all the
posts, leaves a note, and makes sure the related routes do not render
anything even though they will never be hit in production after Fastly
is updated.

Before merging this we need to deploy Fastly's configuration!

@erickzhao I've done a bit more cleanup: basically deleted more code and changed the feed location.


Should push not do anything if there aren't any local changes?

[kind/Feature]
Which functionality do you think we should add?
push should not do anything and return immediately if there are no local changes.
Why is this needed?
Not needed but would be nice! :)

How would odo know if there are any local changes since the last push?

Timestamp comparison? Not sure that it would indeed be worth the effort…

Sounds like a nice to have, but I am worried about that it could create problems for us :)

This is already implemented
/close


Wrong conditional type using "extends" never

Dear Typescript team,
Using conditional types that check against never, are always fall through to the last else branch.
TypeScript Version:  3.7.x; 3.8.x
Search Terms:
conditional type, never, extends
Code
interface ActionWithType<TType> {
    readonly type: TType
}

interface ActionWithPayload<TType, TPayload> extends ActionWithType<TType> {
    readonly payload: TPayload
}

interface ActionWithPayloadAndMeta<TType, TPayload, TMeta> extends ActionWithPayload<TType, TPayload> {
    readonly meta: TMeta
}

type Action<TType, TPayload = unknown, TMeta = unknown> = TPayload extends never
    ? ActionWithType<TType>
    : TMeta extends never
        ? ActionWithPayload<TType, TPayload>
        : ActionWithPayloadAndMeta<TType, TPayload, TMeta>;

enum ActionType {
    ActionTypeA = "ActionTypeA",
    ActionTypeB = "ActionTypeB"
}

interface PayloadType {
    readonly id: string;
}

type MySimpleAction = Action<ActionType.ActionTypeA>;
type MyPayloadAction = Action<ActionType.ActionTypeB, PayloadType>;

Expected behavior:
// expected MySimpleAction  === ActionWithType<ActionType.ActionTypeA>;
// expected MyPayloadAction === ActionWithPayload<ActionType.ActionTypeA, PayloadType>;

Actual behavior:
// actual   MySimpleAction  === ActionWithPayloadAndMeta<ActionType.ActionTypeA, unknown, unknown>;
// actual   MyPayloadAction === ActionWithPayloadAndMeta<ActionType.ActionTypeA, PayloadType, unknown>;

Playground Link:
hovering over the MySimpleAction and MyPayloadAction shows the  actual signature
Playground example
Info:
if I change the conditional type to the following it is working as I expected.
Maybe I totally not got the gist of the never and unknown types.
// proper working conditional type
type Action<TType, TPayload = undefined, TMeta = undefined> = TPayload extends undefined
    ? ActionWithType<TType>
    : TMeta extends undefined
        ? ActionWithPayload<TType, TPayload>
        : ActionWithPayloadAndMeta<TType, TPayload, TMeta>;

And thank you for this perfect Language.

unknown extends never is false, so TS chooses the false branch.

unknown extends never is false, so TS chooses the false branch.

Thank you @RyanCavanaugh for such a simple explanation, I think i got it.
everything is extending unknown, and unknown extends nothing else, as unknown is the top type of everything.
never is the bottom type, so nothing extends never.
Never the less i stumbled over something else that goes allongside which i do nit understand it is a bit more complex as its using infer.
Code:
interface ActionWithType<TType> {
    readonly type: TType
}

interface ActionWithPayload<TType, TPayload> extends ActionWithType<TType> {
    readonly payload: TPayload
}

interface ActionWithPayloadAndMeta<TType, TPayload, TMeta> extends ActionWithPayload<TType, TPayload> {
    readonly meta: TMeta
}

type Action<TType, TPayload = undefined, TMeta = undefined> = TPayload extends undefined
    ? ActionWithType<TType>
    : TMeta extends undefined
        ? ActionWithPayload<TType, TPayload>
        : ActionWithPayloadAndMeta<TType, TPayload, TMeta>;

enum ActionType {
    ActionTypeA = "ActionTypeA",
    ActionTypeB = "ActionTypeB"
}

interface PayloadType {
    readonly id: string;
}

interface MetaType {
    readonly status: string;
}

type FnWithType<TType> = () => Action<TType>;
type FnWithTypeAndPayload<TType, TPayload> = ( payload: TPayload ) => Action<TType, TPayload>;
type FnWithTypeAndPayloadAndMeta<TType, TPayload, TMeta> = ( payload: TPayload, meta: TMeta ) => Action<TType, TPayload, TMeta>;

// conditional type for the creator function type definition
type CreateFn<TType, TPayload = undefined, TMeta = undefined> = TPayload extends undefined
    ? FnWithType<TType>
    : TMeta extends undefined
        ? FnWithTypeAndPayload<TType, TPayload>
        : FnWithTypeAndPayloadAndMeta<TType, TPayload, TMeta>;

// infered types
export type ExtractType<T> = T extends Action<infer TType> ? TType : undefined;
export type ExtractPayload<T> = T extends Action<unknown, infer TPayload> ? TPayload : undefined;
export type ExtractMeta<T> = T extends Action<unknown, unknown, infer TMeta> ? TMeta : undefined;

// factory method
function create<TAction extends Action<ExtractType<TAction>>>(type: ExtractType<TAction>):
    CreateFn<ExtractType<TAction>, ExtractPayload<TAction>, ExtractMeta<TAction>> {

    return ( ( ...args: unknown[] ) => {
        const [
            type,
            payload,
            meta
        ] = args;
        return { type, payload, meta } as unknown as TAction;
    } ) as unknown as CreateFn<ExtractType<TAction>, ExtractPayload<TAction>, ExtractMeta<TAction>>;
}

type MyAction = Action<"MyAction", boolean, MetaType>;
// factory methods returns a union type, as it looks like infer boolean is handled as true | false
const myActionFn = create<MyAction>("MyAction");

// call with boolean value true or false throws type error,
// true | false not assignable to type never
myActionFn(true, { status: "error" });

// expected myActionFn  === FnWithTypeAndPayloadAndMeta<"MyAction", boolean, MetaType>;
// actual   myActionFn  === FnWithTypeAndPayloadAndMeta<"MyAction", true, MetaType> | FnWithTypeAndPayloadAndMeta<"MyAction", false, MetaType>;;

Playground


Add issue link info to PR template

While reviewing pull requests I noticed there are a lot of PRs that are addressing open issues, but the PRs are not properly linked. This results in a lot of issues being left open, even after the fix has been merged.
This updates the PR template to give a short instruction on adding the expected line in the PR description so that the issue and PR can be linked automatically and closed on merge.

Thanks @stmcginnis
/lgtm

That makes sense. I've pushed up another format that has worked really well for me on other projects. It makes it more clear what sections/information to fill out in the PR template.
It does not shorten things as you mention, but I wanted to throw it out there as another potential approach. If you prefer the original way but more concise, just let me know and I can switch things back to that.

I liked the concise format Gaurav proposed -

"If this PR solves an open issue, please add a closing tag (Closes #[issue_number]) to the PR description."

and if needed then a hyperlink for newcomers to learn more about it can be added at bottom of this whole description.

/remove-lifecycle stale

/lgtm

@tengqm, could you please take a look?

/approve


[iOS] TestID for StatusBar component is not visible

Description
After adding testID, the new name attribute does not appear on the iOS platform, thanks to which I could use this element in my e2e tests. On the Android platform, this testID is visible as a resource-id attribute.
For example, in the Button component, after adding testID, the name attribute appears on the iOS platform, and for the Android platform, testID is visible as the resource-id attribute.
A screenshot from the Appium inspector tool, where all the attributes of the hidden element are visible (missing testID for iOS platform):

A screenshot from the Appium inspector tool, where all the attributes of the hidden element are visible (testID is visible as resource-id for the Android platform):

React Native Version
0.72.4
Output of npx react-native info
System:
OS: macOS 13.0.1
CPU: (8) arm64 Apple M1 Pro
Memory: 223.94 MB / 32.00 GB
Shell:
version: 5.8.1
path: /bin/zsh
Binaries:
Node:
version: 18.17.0
path: ~/.nvm/versions/node/v18.17.0/bin/node
Yarn:
version: 1.22.19
path: /usr/local/bin/yarn
npm:
version: 9.6.7
path: ~/.nvm/versions/node/v18.17.0/bin/npm
Watchman:
version: 2<IP_ADDRESS>
path: /opt/homebrew/bin/watchman
Managers:
CocoaPods:
version: 1.12.1
path: /Users/elzbietaszulc/.rvm/gems/ruby-2.7.6/bin/pod
SDKs:
iOS SDK:
Platforms:

DriverKit 22.1
iOS 16.1
macOS 13.0
tvOS 16.1
watchOS 9.1
Android SDK: Not Found
IDEs:
Android Studio: 2021.3 AI-213.7172.25.2113.9123335
Xcode:
version: 14.1/14B47b
path: /usr/bin/xcodebuild
Languages:
Java:
version: 17.0.8
path: /usr/bin/javac
Ruby:
version: 2.7.6
path: /Users/elzbietaszulc/.rvm/rubies/ruby-2.7.6/bin/ruby
npmPackages:
"@react-native-community/cli": Not Found
react: Not Found
react-native: Not Found
react-native-macos: Not Found
npmGlobalPackages:
"react-native": Not Found
Android:
hermesEnabled: false
newArchEnabled: false
iOS:
hermesEnabled: true
newArchEnabled: false

Steps to reproduce

Add testID="status_bar_hidden_button" to the StatusBarExample.js component (hidden element)
Reload RNtester app
Run the Appium Inspector tool
Check if name attribute has been added for the iOS platform

Snack, screenshot, or link to a repository
A link to the StatusBar component that I defined:
https://github.com/facebook/react-native/commit/1a0aa4deccf8e4906b6cabb6ca1c256144c1905d#diff-b18fc0c0f0855b6d50574152d4c1169b6e17ca8058d1cdf6aa0ff35203b03183
A link to StatusBarExample.js where I added testID:
https://github.com/elzszulc/react-native/commit/a8e91190646ee31d925d08e9916f707530319f87#diff-2b00a1862df8983883889bfdc2e01f3ea7ca3970edab508da39c26419f9447e3

cc. @cipolleschi mind looking into this issue? 🙏

Do not reap


change(ci): Combine high and low Mergify queues into a batched queue

Motivation
This PR combines the high and low queues into a single batched queue.
The urgent queue remains the same.
This reduces Google Cloud costs.
Review
This PR is not urgent, but we should merge it soon anyway, to increase CI speed.
We will need to merge this PR manually, Mergify doesn't automatically merge changes to its own configs.
Reviewer Checklist

[ ] CI passes
[ ] Config format is correct

@mergifyio update


Arrow does not compile

flatbuffers had non backwards compatible API changes, that lead to arrow-rs not building anymore if . This is the case for the latest master, but I assume also for the released version on crates.io

Closed by #239


Diagnostic doesn't work

Description:
See below

Tried the sample and I could get the diagnostics. @shafreenAnfar could you confirm if you can get diagnostics for the syntax errors?

Checked with the latest master and could not reproduce the issue. Hence closing the issue. Please feel free to reopen if the issue can still be reproduced


(new) google-analytics doesn't work?

I just created a google analytics account for my new blog and the code is not UA-XXXXXXXX-X but instead G-XXXXXXXXXX (new system? I don't see how to get the UA version), it doesn't seems to count my first visits, is this change the cause of it?

It is working on my side.
https://github.com/HEIGE-PCloud/DoIt/blob/326f5aae7f8bb49206ce3009c0353cd6074b6130/layouts/partials/plugin/analytics.html#L3-L11
The config for ga is the one to config the new G-XXXXXXXXXX id.
Check:

Whether you have enabled it correctly

  [params.analytics]
    enable = true
    # Google Analytics
    [params.analytics.google]
      id = "G-XXXXXXXXXX"

Whether your browser block analytics somehow.


template for future interviews

 Issue by Erkan-Yilmaz
Monday Feb 06, 2017 at 08:14 GMT
Originally opened as https://github.com/Erkan-Yilmaz/Gridcoin-tasks/issues/81

to be used for:

interview guide with BOINC admins

questions:

What is your BOINC project doing ?

research-wise: publishing papers, ... ?
company-wise
How long do you plan to run your project for?
Does your project have any milestone goals in terms of time, WU completed etc.

How are you rewarding your users ? (If your project is on the whitelist: rewards come via Gridcoin)

about your WUs/applications:

recommendations based on:

OS type

Have you noticed any apps run better on certain processors and why?

research we users want to contribute to

What were the problems when you didn't have WUs (anymore) ?

BOINC server questions:

server load / user situation
self-install vs docker
experience as BOINC admin over the years: tips to fresh BOINC admins

Do you perform regular maintenance on your BOINC project? Are you able to inherit changes from the main BOINC repo easily?

What kind of help do you need for your BOINC project ? (from Gridcoin or other groups)

What kind of security do you provide ?

SSL implementation/improvement planned for the future? (If they don't have an 'A' grade cert).
WU protection ? (spot-checking, replication, tracking trust/reputation of workers)
virtualbox apps ?
Have you had issues with spammers?
Would you consider implementing 2FA? #102

What do you do against cheating ?

Do you use spot-check tasks?
If so, spot-checking within real tasks (unique spot-check calculations for a work unit's replicated tasks) or dedicated spot-check tasks?
Do you track trustworthiness/reputation of volunteers? (Based on outcome of work unit voting pools, or likelihood of collusion).
Do you utilize task replication?

background of the researcher:

What field did they study in?
How did they get started with the project that they are in?
Were they involved with distributed computing in the past?

What do you think of the following boinc issue: https://github.com/BOINC/boinc/issues/1644 (improve password security) ?

What are highest priority issues regarding BOINC development in your opinion?

Are you aware of the BOINC project governance? https://boinc.berkeley.edu/trac/wiki/ProjectGovernance

Do you regularly communicate with the project governance?

What do you think of the 'monetization of BOINC computation'?

Gridcoin related:

Before being contacted for the interview, had you heard of Gridcoin?

Have you had much experience with other cryptocurrencies like bitcoin?

Are you aware of Gridcoin's leaderboard position on your BOINC project? (most projects we're number 1)
What do you think of Gridcoin? Anything that we could be doing better?
Do you accept Gridcoin donations ?
The teamfounder can be transferred to a trusted person ?
Can you add a Gridcoin donation button to your website? Preferably via coinpayments?
What do you think of the BOINC team requirement?

Do you feel it helps or hinders Gridcoin as a cryptocurrency, and as a BOINC team?

Gridcoin makes always challenges: do you have any preferences ? e.g. duration: 3 days long or 1 week ?
Would you entertain the idea of implementing a Gridcoin field within your BOINC project's profile page, so as to improve proof of CPID ownership? #101

 Comment by ghost
Monday Feb 06, 2017 at 08:34 GMT

What is your BOINC project doing ?
How long do you plan to run your project for?
Does your project have any milestone goals in terms of time, WU completed etc.

 Comment by Erkan-Yilmaz
Monday Feb 06, 2017 at 09:37 GMT

@AffectionGC thx I'll add these above

 Comment by grctest
Tuesday Feb 07, 2017 at 17:21 GMT

"Can you add a Gridcoin donation button to your website? Preferably via coinpayments?"

 Comment by grctest
Tuesday Feb 07, 2017 at 22:56 GMT

Before being contacted for the interview, had you heard of Gridcoin?
What do you think of Gridcoin? Anything that we could be doing better?

Regarding security:

SSL implementation/improvement planned for the future? (If they don't have an 'A' grade cert).
Have you had issues with spammers?.

 Comment by ghost
Wednesday Feb 08, 2017 at 18:44 GMT

If the project is brand new it would be nice to hear a little bit about the background of the researchers as well. What field did they study in? How did they get started with the project that they are in? Were they involved with distributive computing in the past?
Might be a little fluffy but it could be of interest to people outside of gridcoin for higher visability. The arguement could be made that if there are enough questions about the actual science of the project and what the creators are doing then the podcast could be posted around in areas outside of gridcoin/crypto currency fourms which in turn could bring in new users.

 Comment by Erkan-Yilmaz
Wednesday Feb 08, 2017 at 19:00 GMT

thanks @grctest @AffectionGC
your suggestsions were all added above

 Comment by Peppernrino
Saturday Feb 11, 2017 at 09:28 GMT

What do you think of the Gridcoin BOINC team requirement? Do you feel it helps or hinders Gridcoin as a cryptocurrency, and as a BOINC team?

 Comment by grctest
Sunday Feb 12, 2017 at 01:39 GMT

What do you think of the following boinc issue: https://github.com/BOINC/boinc/issues/1644
What are highest priority issues regarding BOINC development in your opinion?
Are you aware of the BOINC project governance? https://boinc.berkeley.edu/trac/wiki/ProjectGovernance
Do you regularly communicate with the project governance?
What do you think of the 'monetization of BOINC computation'?

 Comment by Erkan-Yilmaz
Sunday Feb 12, 2017 at 09:56 GMT

thanks @grctest, added those and more questions from yesterday's session

 Comment by grctest
Monday Feb 13, 2017 at 09:09 GMT

We could potentially expand the following:

What do you do against cheating? (perhaps also covers "WU protection ? (spot-checking, replication, tracking trust/reputation of workers)" ) --> we could consolodate 'cheating/wu-protection' headings.

With:

Do you use spot-check tasks?

If so, spot-checking within real tasks (unique spot-check calculations for a work unit's replicated tasks) or dedicated spot-check tasks?

Do you track trustworthiness/reputation of volunteers? (Based on outcome of work unit voting pools, or likelihood of collusion).
Do you utilize task replication?

 Comment by grctest
Monday Feb 13, 2017 at 09:49 GMT

Are you aware of Gridcoin's leaderboard position on your BOINC project? (most projects we're number 1)

 Comment by grctest
Sunday Apr 09, 2017 at 22:06 GMT

Would you entertain the idea of implementing a Gridcoin field within your BOINC project's profile page, so as to improve proof of CPID ownership? https://github.com/Erkan-Yilmaz/Gridcoin-tasks/issues/101
Would you consider implementing 2FA? https://github.com/Erkan-Yilmaz/Gridcoin-tasks/issues/102
Do you perform regular maintenance on your BOINC project? Are you able to inherit changes from the main BOINC repo easily?

 Comment by Erkan-Yilmaz
Wednesday Apr 12, 2017 at 13:19 GMT

added last 3 comments by @grctest into template, thx


Default schema

Having something like
function MLJBase.fit(model, verb, X, y)
    features = MLJBase.schema(X).names
    ...
end

will fail if, for instance, X is just a matrix. It seems that the right thing to do would be to give the default (:x1, :x2, ...) as DataFrame does?
(happy to PR if relevant)

So you are requesting that we overload MLJBase.schema (a re-export of Tables.schema, if I recall) to work on matrices?
The Tables.jl people are quite insistent that matrices are not tables (and now provide a way to convert matrices to tables). While I don't see any particular harm in overloading schema, I wonder what your use case is. At present, no model accepts matrix input - only tabular (in the sense of Tables.jl) input. If there is a call to have models that accept matrix input, what would be the need to extract schema? Or are you suggesting all models that accept tabular input should also accept matrix input? Although there is a small performance cost, it is simpler to stick with tables, I think. (Or, many moons ago, we should have decided only matrix input was allowed : - ) )

Another reason I generally don't like the implicit labelling (:x1, :x2, ...) is that it can have unintended consequences later on. For example, when two such objects are combined and we get a name clash. Currently we do this in our dimension reduction algorithms (invent labels for the output) and I've been wondering if it might be better to output a matrix and have the user explicitly provide the labels (which she can do with MLJ.table).

I think these are all good reasons and so we can shelf this issue.
Maybe to explain where this question came from, I can imagine users new to MLJ and willing to give it a quick shot broadly wanting to do:

generate random X, y (matrix/vector)
wanting to quickly try to do a regression
getting bitten

and possibly being annoyed as a result.
Of course this goes away if we have clear examples of how to do exactly that, that don't require immediately understanding learning networks etc.


Fix unwanted listener endpoint creation and record creation failure for multiple services

Purpose

Fixes https://github.com/ballerina-platform/ballerina-lang/issues/21088
Fixes https://github.com/ballerina-platform/ballerina-lang/issues/22361

Approach

Change messageList -> messageMap
Use flag to check the endpoint already created or not.

Samples
Generate service files using the following protobuf file.
syntax = "proto3";

import "google/protobuf/wrappers.proto";
service Register {
	rpc registerPerson (Person) returns (google.protobuf.BoolValue);
}

service Provider {
	rpc fetchPerson (google.protobuf.StringValue) returns (Person);
}

message Person {
    string name = 1;
}

It will generate two service files that share a common endpoint.
import ballerina/grpc;

listener grpc:Listener ep = new (9090);

service Register on ep {

    resource function registerPerson(grpc:Caller caller, Person value) {
        // Implementation goes here.

        // You should return a boolean
    }
}

import ballerina/grpc;

service Provider on ep {

    resource function fetchPerson(grpc:Caller caller, string value) {
        // Implementation goes here.

        // You should return a Person
    }
}

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

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

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

[ ] Increased Test Coverage
[ ] Added necessary documentation

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

Codecov Report

Merging #22374 into master will not change coverage by %.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #22374   +/-   ##
=======================================
  Coverage   14.59%   14.59%
=======================================
  Files          51       51
  Lines        1411     1411
  Branches      219      219
=======================================
  Hits          206      206
  Misses       1189     1189
  Partials       16       16

Continue to review full report at Codecov.

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


Making everything private except the start method

Huge rewrite 😅
(I've put the already private functions also inside the constructor in order to keep everything together)

Whoops, ignore, for now. All tests are broken 😑

Tests fixed! Please, check that you agree with the structure they have now!

Nice work! :100:
I like the way the test are structured, much more natural :)


10.0.22621.2066 not supported

termsrv.dll.zip

https://github.com/sebaxakerhtc/rdpwrap.ini/commit/f17b4659ddb594a80cec86b89e92db393407ff7b

Reopened. Not included in official INI (by Stas'M) :)