[8.x] Expose Query\Builder->from property via Eloquent\Builder->from

This PR makes the Illuminate\Database\Query\Builder->from property available to read from the Illuminate\Database\Eloquent\Builder instance.
This is done by adding a pass-through in the __get method of Eloquent\Builder that just returns the value on the base query builder.
The Current Issue
Currently, the join/subquery methods on the Eloquent builder are documented as accepting either a base query builder, or an Eloquent query builder.
However, if you attempt to do a cross-database join/subquery, the framework eventually calls prependDatabaseNameIfCrossDatabaseQuery on the Query\Builder instance. This method checks if the query is cross-database. If so, it attempts to access the ->from property of the query.
When an Eloquent builder instance is passed along, it will error out at that point, as there is no from property accessible on the Eloquent\Builder instance.
Rationale
The Eloquent builder currently throws an error when trying to access the from property, so this doesn't modify an existing accessor.
Because the PHP doc comments say you should be able to pass an Eloquent builder into the subquery/join methods, the fact that they can fail in some cases specifically with an Eloquent builder is a bug.
Adding this from property fixes the bug by making the call to prependDatabaseNameIfCrossDatabaseQuery succeed for either type of Builder.
This change is also fairly minor, and I've included an update to the class doc for Eloquent\Builder and a test for said property access.
Related Issues
This fixes the issue described in #35631 and is an improved solution on #38629.

Would it be better to add a method (that can be added to the Query Builder interface that's in 9.x) and then call the method? That feels a little better to me than using a condition in __get()…

I can modify the solution to use a method if you think that's best. I was just trying to find the solution with the smallest impact in the hopes that this could be fixed in 8.x.

Yeah I would maybe add a method or something to access that so we don't have to have this string checking in __get.

Okay, I've added a getter method to both of the Builder classes and removed the from pass-through.

I actually did end up solving this by adding a propertyPassthru array similar to the existing passthru array instead of having to invent a new method - sorry about that.


HDDS-11354. Intermittent failure in TestOzoneManagerSnapshotAcl#testLookupKeyWithNotAllowedUserForPrefixAcl

What changes were proposed in this pull request?
TestOzoneManagerSnapshotAcl#testLookupKeyWithNotAllowedUserForPrefixAcl failed in this run
Please describe your PR in detail:
Sometimes OzoneManagerDoubleBuffer doesn't work. Therefore, OM failed to create snapshot.
Set isRunning to true before starting OzoneManagerDoubleBuffer.
This is necessary because, if OzoneManagerDoubleBuffer#flushTransactions runs while isRunning is false, it wil not flush any buffers, as shown below.
  /**
   * Runs in a background thread and batches the transaction in currentBuffer
   * and commit to DB.
   */
  @VisibleForTesting
  public void flushTransactions() {
    while (isRunning.get() && canFlush()) {        // it won't go into the while loop if isRunning is false
      flushCurrentBuffer();
    }
  }

What is the link to the Apache JIRA
HDDS-11354
How was this patch tested?
Test 100*10 times and pass:
https://github.com/chungen0126/ozone/actions/runs/10878894979

@szetszwo , could you take a look? This is a small bug in OzoneManagerDoubleBuffer not just for Snapshot, and it is related to your commit. Thanks.


Add rest\GroupUrlRule; add rest\UrlRule param controllerPrefix

For more "versioned" of config of RESTful api application
All of rules of major version is one rest\GroupUrlRule;
In nested rules module prefix is omitted;
Current
return [
    'modules' => [
        'v1' => [
            'class' => 'app\modules\v1\Module',
        ],
        'v2' => [
            'class' => 'app\modules\v2\Module',
        ],
    ],
    'components' => [
        'urlManager' => [
            'enablePrettyUrl' => true,
            'enableStrictParsing' => true,
            'showScriptName' => false,
            'rules' => [
                ['class' => 'yii\rest\UrlRule', 'controller' => ['v1/user', 'v1/post']],
                ['class' => 'yii\rest\UrlRule', 'controller' => ['v2/user', 'v2/post']],
            ],
        ],
    ],
];

propose
return [
    'modules' => [
        'v1' => [
            'class' => 'app\modules\v1\Module',
        ],
        'v2' => [
            'class' => 'app\modules\v2\Module',
        ],
    ],
    'components' => [
        'urlManager' => [
            'enablePrettyUrl' => true,
            'enableStrictParsing' => true,
            'showScriptName' => false,
            'rules' => [
                // v1
                [
                    'class' => 'yii\rest\GroupUrlRule',
                    'prefix' => 'v1',
                    'rules' => [
                        [
                            'controller' => 'post',
                            'pluralize' => false,
                            'only' => ['index', 'view', 'options'],
                        ],
                       'user',
                        //another yii\rest\UrlRule of v1 module with differences in params
                    ],
                ],
                // v2
                [
                    'class' => 'yii\rest\GroupUrlRule',
                    'prefix' => 'v2',
                    'rules' => [
                        [
                            'controller' => 'post',
                            'pluralize' => false,
                            'only' => ['index', 'view', 'options'],
                        ],
                        'u' => 'user',
                        //another yii\rest\UrlRule of v2 module with differences in params
                    ],
                ],
                //...
            ],
        ],
    ],
];

Q
A

Is bugfix?
no

New feature?
yes

Breaks BC?
no

Tests pass?
yes

Fixed issues

huh, I get some error in tests with dataProvider config %)

Is it really necessary to create another group rule class? We could adjust yii\web\GroupUrlRule to work properly with rest rules. #9122

@rob006 as you can see in code this class is a sugar for module name.
yii\web\GroupUrlRule not implement module prefix for controller property of nested yii\rest\UrlRule;

yii\web\GroupUrlRule not implement module prefix for controller property of nested yii\rest\UrlRule;

It has $prefix and $routePrefix which is basically the same. You just changed name of these properties. Why not use $routePrefix and $prefix directly in $ruleConfig?

@rob006 yes of course.
2 differences:

first
$routePrefix property is not implemented in yii\rbac\UrlRule and in yii\web\CompositeUrlRule

second
additional "short form" can't be use.

For first:
I propose add the $controllerPrefix property to yii\rbac\UrlRule. This property is same at routePrefix in another rule, but is named according to meaning.
This property unbind of $prefix by BC reason (unlike $routePrefix = $routePrefix ?: $route).
For this reason the yii\rbac\GroupUrlRule process prefix and controllerPrefix and pass it both.

And, before you can ask...
If we will add only contrllerPrefix property in yii\rest\UrlRule
        'urlManager' => [
            'enablePrettyUrl' => true,
            'enableStrictParsing' => true,
            'showScriptName' => false,
            'rules' => [
                // v1 use 'yii\web\GroupUrlRule' and property `controllerPrefix`
                [
                    'class' => 'yii\web\GroupUrlRule',
                    'prefix' => 'v1',
                    'ruleConfig' =>  [
                        'class' => 'yii\rest\UrlRule',
                        'prefix' => 'v1',
                        'controllerPrefix' => 'v1',
                    ],
                    'rules' => [
                        [
                            'controller' => 'post',
                            'pluralize' => false,
                            'only' => ['index', 'view', 'options'],
                        ],
                       ['controller' => 'user'],
                        //another yii\rest\UrlRule of v1 module with differences in params
                    ],
                ],
                // v2 use 'yii\rest\GroupUrlRule' use both property `controllerPrefix` and class `yii\rest\GroupUrlRule`
                [
                    'class' => 'yii\rest\GroupUrlRule',
                    'prefix' => 'v2',
                    'rules' => [
                        [
                            'controller' => 'post',
                            'pluralize' => false,
                            'only' => ['index', 'view', 'options'],
                        ],
                        'u' => 'user', // example of short syntax of nested rule controller property
                        //another yii\rest\UrlRule of v2 module with differences in params
                    ],
                ],
                //...
            ],

related #13968

$routePrefix property is not implemented in yii\rbac\UrlRule and in yii\web\CompositeUrlRule

You could implement it in the same way as $controllerPrefix. You will get unified API and less duplicated code and redundant classes.

@rob006 $controllerPrefix uses instead of $routePrefix
because logic of applied $prefix to empty $controllerPrefix in yii\rest\UrlRule and to empty $routePrefix in yii\web\GroupUrlRule is difference.
        $this->prefix = trim($this->prefix, '/');
        $this->routePrefix = $this->routePrefix === null ? $this->prefix : trim($this->routePrefix, '/');

        $this->prefix = trim($this->prefix, '/');
        $this->controllerPrefix = trim($this->controllerPrefix, '/');

You propose use $routePrefix in  yii\rest\UrlRule instead of $controllerPrefix anyway?

because logic of applied $prefix to empty $controllerPrefix in yii\rest\UrlRule and to empty $routePrefix in yii\web\GroupUrlRule is difference.

But why? Sorry, I really don't get what is the problem here and why $controllerPrefix must be different from $routePrefix.

@rob006 just BC compatibility
I am a developer.
I use yii2 for RESTful api
I have a worked api
I have config
return [
    'modules' => [
        'v1' => [
            'class' => 'app\modules\v1\Module',
        ],
        'v2' => [
            'class' => 'app\modules\v2\Module',
        ],
    ],
    'components' => [
        'urlManager' => [
            'enablePrettyUrl' => true,
            'enableStrictParsing' => true,
            'showScriptName' => false,
            'rules' => [
                ['class' => 'yii\rest\UrlRule', 'prefix' => 'api/v1', 'controller' => ['v1/user', 'v1/post']],
                ['class' => 'yii\rest\UrlRule', 'prefix' => 'api/v2', 'controller' => ['v2/user', 'v2/post']],
                // ... or any else - see tests case below
                ['class' => 'yii\rest\UrlRule', 'prefix' => 'admin', 'controller' => 'post'],
            ],
        ],
    ],
];

https://github.com/yiisoft/yii2/blob/77ad3dca3f67069efd32c9f3b29b0f10a83c0c8c/tests/framework/rest/UrlRuleTest.php#L84-L90
'api/v1/posts', 'v1/post/index'
...
'admin/posts', 'admin/post/index'

I (we, contributors of framework) add property to yii\rest\UrlRule, property is prefix of controller;

With relation of prefix (like yii\web\GroupUrlRule)
I have init()

        $this->prefix = trim($this->prefix, '/');
        $this->routePrefix = $this->routePrefix === null ? $this->prefix : trim($this->routePrefix, '/');

I have wrong resolve
'api/v1/posts', 'api/v1/v1/post/index'
...
'admin/posts', 'admin/post/index'

Unbind / independent properties

I have init()
        $this->prefix = trim($this->prefix, '/');
        $this->controllerPrefix = trim($this->controllerPrefix, '/');

I have my code, who still work

In your example you're not using GroupUrlRule, so this will not change anything. AFAIK GroupUrlRule doesn't work well with yii\rest\UrlRule so you can't really break BC for thing that doesn't work. :D
But event if I'm wrong and this will break BC, I would rather move this to 2.1 than introduce yii\rest\GroupUrlRule which do exactly the same thing as yii\web\GroupUrlRule, but in different way (for no reason) and with different names of properties. We should keep API simple and introducing two different interfaces for the same job is confusing and harder to maintain.

which do exactly the same thing as

Not same.
yii\web\GroupUrlRule set prefix and routePrefix for self
yii\rest\GroupUrlRule set prefix and controllerPrefix (aka routePrefix) for self
AND propose sugar for set controller property
AND set another default class for nested rules.
AND set controllerPrefix (aka routePrefix) for nested rules
I will change  controllerPrefix to routePrefix without null-fill logic and add todo for BC broken changes in 2.1
Also I will split changes into 2 PR.
But it is not same.

Won't be merged into 2.0.


fix: createGlobalStyles: TypeError: fn is not a function

fixes https://github.com/solidjs/solid-styled-components/issues/19

Hmm.. We need tests for createGlobalStyles clearly.


unable to scan

parser failed: i'm not sure how to parse this uri

This one which you sent to me privately should be fixed. Please lmk if you have any more.


feat(chain): add zora networks

Adds support for Zora mainnet, testnet (goerli), and sepolia

want me to fmt quick? just saw the failing job

yes, needs nightly fmt cargo +nightly fmt

@mattsse should be all set now


Needs credit to dragonfire535/xiao.

There's some obvious takes from my bot, Xiao, in here, as far as even using some of the images I have on Imgur (https://i.imgur.com/Z7NJBK2.png in Wikipedia, for example). All you need to do is add a credit line to the README and I'll be happy. Something like this:

Some commands were inspired by and based on commands from Xiao by dragonfire535.

Hey,
I have added the respective credit to the README. I should've done that sooner and for that I apologize. Your bot is really neat by the way.
Closing the issue as I think the issue has been resolved now.


Migrate to Lodash

This includes both server- and client-side code.

Things still needing to be checked from https://github.com/lodash/lodash/wiki/Migrating:

[  ] _.each

Other than that the server code should now be on Lodash. However the web UI code needs to be converted; right now we're still depending on Underscore so the web UI doesn't break.


Interface to MA57 solver

@cnpetra and I are interested in having access MA57 linear solvers for sparse symmetric linear systems from within MFEM. Is it possible to include such an interface?

It should be possible. Feel free to propose a PR if you want.

@thartland, are you able to create a PR for this, or do you need help?

@tzanio, thanks for the followup message. I should be able to give more thought to this item later this week and early next week. I will likely need help but I will have take an honest look at this before asking for help.

ping: @thartland

I have yet to be able to commit enough time to this. When I get more into this I will reach out to the MFEM team if I run into challenges. Thanks all.

Thanks @thartland


Use osp-flatcar-cloud-init for Anexia by default

What this PR does / why we need it:
This is the last missing piece for adding support for Anexia. Since Anexia only supports cloud-init with flatcar. We are now defaulting the MachineDeployments to use osp-flatcar-cloud-init.
Which issue(s) this PR fixes:

Fixes #
What type of PR is this?

/kind chore
Special notes for your reviewer:
Does this PR introduce a user-facing change? Then add your Release Note here:

NONE

Documentation:

NONE

/lgtm


Cannot read properties of undefined (reading 'main') when defining custom color variants

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
When adding new component variants, say for Button, I am allowed to create variants by declaring ButtonPropsColorOverrides and then defining the corresponding variants in a custom theme, in the components.MuiButton.variants property. While this works for the Button.variant property, it doesn't for Button.color. After setting my variants, when I use them in code, I get:
Cannot read properties of undefined (reading 'main')

Expected behavior 🤔
I would expect the above error not to happen, since defining variants for the variant property (instead of color) does not create this issue.
I suspect that MUI tries to find my custom color in the theme palette, hence looking for theme.palette.myColor.main is undefined.
Steps to reproduce 🕹
Simply use this component in a React app:
declare module '@mui/material/Button' {
  interface ButtonPropsColorOverrides {
    hello: true;
  }
}

const theme = createTheme({
  components: {
    MuiButton: {
      variants: [
        {
          props: { color: 'hello' },
          style: { background: 'red' },
        },
      ],
    },
  },
});

export function Main() {
  return (
    <ThemeProvider theme={theme}>
      <Button color="hello" size="small">Small</Button>
    </ThemeProvider>
  );
}

I tried defining the color in the theme platte, but no luck:
const theme = createTheme({
  palette: {
    hello: {
      main: 'red',
    },
  } as any,
  // ...
});

Context 🔦
I'm trying to add additional colors alongside primary, secondary, etc, without creating a custom component. I want to keep using MUI's Button, I don't want to create a proxy component as this is for a shared theme package and I want my users to keep importing Button from @mui/material instead of @my-org/my-theme.
I looked at #14185 but it's not solving my problem.
Your environment 🌎

`npx @mui/envinfo`
 System:
    OS: macOS 11.6
  Binaries:
    Node: 14.17.5 - ~/.nvm/versions/node/v14.17.5/bin/node
    Yarn: Not Found
    npm: 7.22.0 - ~/.nvm/versions/node/v14.17.5/bin/npm
  Browsers:
    Chrome: 96.0.4664.55
    Edge: Not Found
    Firefox: Not Found
    Safari: 15.0
  npmPackages:
    @emotion/react: ^11.6.0 => 11.6.0
    @emotion/styled: ^11.6.0 => 11.6.0
    @mui/base:  5.0.0-alpha.56
    @mui/lab: ^5.0.0-alpha.57 => 5.0.0-alpha.57
    @mui/material: ^5.x.x => 5.2.0
    @mui/private-theming:  5.2.1
    @mui/styled-engine:  5.2.0
    @mui/styles: ^5.x.x => 5.2.0
    @mui/system:  5.2.1
    @mui/types:  7.1.0
    @mui/utils:  5.2.1
    @types/react: ^17.0.36 => 17.0.36
    react: ^17.0.2 => 17.0.2
    react-dom: ^16.13.1 => 16.14.0
    typescript: ^4.3.4 => 4.3.4

I tried defining the color in the theme palatte, but no luck

It is unclear what you have tried. Could you share a codesandbox? Yes, the color needs to be defined in the palette, as the Button expects that:

theme.palette[yourColor].main
theme.palette[yourColor].dark
theme.palette[yourColor].contrastText

will be defined.

@mnajdova thanks for your answer ! As mentioned in my initial post, I have tried adding the color to the palette but I still get the issue:
const theme = createTheme({
  palette: {
    hello: {
      main: 'red',
    },
  } as any,
  // ...
});

I have to use as any because Mui Typings don't let me add custom colors here.

You need to use module augmentation if you want to add new things in the theme - https://mui.com/customization/theming/#custom-variables
Without a reproduction, I can't do anything.

I'll put up a sandbox, just a sec 😄

Ok, it seems the sandbox is actually working 😓  There must be something wrong with my local setup, greatest apologies.

@gempain Did you find the solution? I am having the same issue when trying to add a custom color to the theme.


Pass all running stages to DataModule.setup

🚀 Feature
Currently. DataModule.setup is only called with stages fit or test. But we have several more:
Stages:
https://github.com/PyTorchLightning/pytorch-lightning/blob/5f3372871a333c3229968f1af1b10a925d7ec3ec/pytorch_lightning/trainer/states.py#L39-L49
Note that it's a bit tricky because fit is not a RunningStage. It indicates train or eval
Motivation
Allows having custom logic for each stage
Pitch
def setup(stage: Optional[str] = None):
    assert stage in list(RunningStage)
    ...

Additional context
We are passing 'test' when predicting as seen in #5579
https://github.com/PyTorchLightning/pytorch-lightning/blob/9137b16068fe03e6db8df548235363e5f5476aac/pytorch_lightning/trainer/trainer.py#L909

I don't think the setup is not called again during the evaluation(eval).

I don't think the setup is not called again during the evaluation(eval).

def setup(self, stage):
    if self.running_stage == 'predict':
        ....

@leifdenby just a workaround for now.


Pip broken on python:2-windowsservercore container image

I have pulled python:2-windowsservercore & created a container.
python working fine, however pip is throwing error as given below
No changes made to image and no additional tool installed( its base image from github)
**C:\>pip**
Traceback (most recent call last):
  File "c:\python\lib\runpy.py", line 174, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "c:\python\lib\runpy.py", line 72, in _run_code
    exec code in run_globals
  File "C:\Python\Scripts\pip.exe\__main__.py", line 4, in <module>
  File "c:\python\lib\site-packages\pip\_internal\cli\main.py", line 10, in <module>
    from pip._internal.cli.autocompletion import autocomplete
  File "c:\python\lib\site-packages\pip\_internal\cli\autocompletion.py", line 9, in <module>
    from pip._internal.cli.main_parser import create_main_parser
  File "c:\python\lib\site-packages\pip\_internal\cli\main_parser.py", line 7, in <module>
    from pip._internal.cli import cmdoptions
  File "c:\python\lib\site-packages\pip\_internal\cli\cmdoptions.py", line 31, in <module>
    from pip._internal.utils.ui import BAR_TYPES
  File "c:\python\lib\site-packages\pip\_internal\utils\ui.py", line 64, in <module>
    _BaseBar = _select_progress_class(IncrementalBar, Bar)  # type: Any
  File "c:\python\lib\site-packages\pip\_internal\utils\ui.py", line 57, in _select_progress_class
    six.text_type().join(characters).encode(encoding)
LookupError: unknown encoding: cp0

**C:\>python -m pip install -U pip**
Traceback (most recent call last):
  File "C:\Python\lib\runpy.py", line 174, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "C:\Python\lib\runpy.py", line 72, in _run_code
    exec code in run_globals
  File "C:\Python\lib\site-packages\pip\__main__.py", line 16, in <module>
    from pip._internal.cli.main import main as _main  # isort:skip # noqa
  File "C:\Python\lib\site-packages\pip\_internal\cli\main.py", line 10, in <module>
    from pip._internal.cli.autocompletion import autocomplete
  File "C:\Python\lib\site-packages\pip\_internal\cli\autocompletion.py", line 9, in <module>
    from pip._internal.cli.main_parser import create_main_parser
  File "C:\Python\lib\site-packages\pip\_internal\cli\main_parser.py", line 7, in <module>
    from pip._internal.cli import cmdoptions
  File "C:\Python\lib\site-packages\pip\_internal\cli\cmdoptions.py", line 31, in <module>
    from pip._internal.utils.ui import BAR_TYPES
  File "C:\Python\lib\site-packages\pip\_internal\utils\ui.py", line 64, in <module>
    _BaseBar = _select_progress_class(IncrementalBar, Bar)  # type: Any
  File "C:\Python\lib\site-packages\pip\_internal\utils\ui.py", line 57, in _select_progress_class
    six.text_type().join(characters).encode(encoding)
LookupError: unknown encoding: cp0

I'm able to reproduce, but the encoding in my error message is cp65001, not cp0: :confused:
$ docker pull python:2-windowsservercore
2-windowsservercore: Pulling from library/python
Digest: sha256:2bedbd91529fa5e98354dd53d8f5bec79aba9646e6b25ff25e9b4639a7d5d2f6
Status: Image is up to date for python:2-windowsservercore
docker.io/library/python:2-windowsservercore

$ docker run -it --rm python:2-windowsservercore powershell

Windows PowerShell
Copyright (C) 2016 Microsoft Corporation. All rights reserved.

PS C:\> pip --version
Traceback (most recent call last):
  File "c:\python\lib\runpy.py", line 174, in _run_module_as_main
    "__main__", fname, loader, pkg_name)
  File "c:\python\lib\runpy.py", line 72, in _run_code
    exec code in run_globals
  File "C:\Python\Scripts\pip.exe\__main__.py", line 4, in <module>
  File "c:\python\lib\site-packages\pip\_internal\cli\main.py", line 10, in <module>
    from pip._internal.cli.autocompletion import autocomplete
  File "c:\python\lib\site-packages\pip\_internal\cli\autocompletion.py", line 9, in <module>
    from pip._internal.cli.main_parser import create_main_parser
  File "c:\python\lib\site-packages\pip\_internal\cli\main_parser.py", line 7, in <module>
    from pip._internal.cli import cmdoptions
  File "c:\python\lib\site-packages\pip\_internal\cli\cmdoptions.py", line 31, in <module>
    from pip._internal.utils.ui import BAR_TYPES
  File "c:\python\lib\site-packages\pip\_internal\utils\ui.py", line 64, in <module>
    _BaseBar = _select_progress_class(IncrementalBar, Bar)  # type: Any
  File "c:\python\lib\site-packages\pip\_internal\utils\ui.py", line 57, in _select_progress_class
    six.text_type().join(characters).encode(encoding)
LookupError: unknown encoding: cp65001

Ah, if I set PYTHONIOENCODING to UTF-8, it works.
However, Python 2 is unfortunately EOL, so this isn't something we'll be fixing in the published images (which are not going to be updated for any reason).
That being said, you might be interested in https://github.com/docker-library/pypy/pull/54.

set PYTHONIOENCODING to UTF-8 is working
do pypy support window image? i was looking for windows image python 2

We are working to add pypy windows images and pypy upstream still supports a python 2 interpreter.


Sample project

Hello,
When there is a lot of code like the page above, it would be more convenient if the code would be in a sample project that can be downloaded via a ZIP file.
Thanks

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

ID: 12703330-b5a0-8523-3a89-c630c8514fb2
Version Independent ID: 56f77244-5342-d30a-7ee4-ba062f5b86a5
Content: C# sample - submissions for apps, add-ons, and flights - UWP app developer
Content Source: windows-apps-src/monetize/csharp-code-examples-for-the-windows-store-submission-api.md
Service: unspecified
GitHub Login: @mcleanbyron
Microsoft Alias: mcleans

Thanks for your feedback. The engineering team that owns this code sample is aware of this request and is already considering future options for improving how they deliver this sample.


Verbose Linux build output contains double prefixes

Steps to Reproduce

Run flutter create --plaforms linux .
Run flutter build linux -v
Observe prefixes in square brackets ([    XXms]) in the build output

Expected results:
Consistent prefixes
Actual results:
The build step that calls tool_backend.sh -> dart tool_backend.dart has two prefixes. For example:
[        ] [   +5 ms] unpack_linux: Complete

Verbose build & Flutter doctor output
$ flutter build linux -v
[ +139 ms] executing: [/home/jpnurmi/Flutter/] git -c log.showSignature=false log -n 1 --pretty=format:%H
[  +60 ms] Exit code 0 from: git -c log.showSignature=false log -n 1 --pretty=format:%H
[        ] cf6c33e58a05b6bf45d8636cb27342709043e745
[   +1 ms] executing: [/home/jpnurmi/Flutter/] git tag --points-at cf6c33e58a05b6bf45d8636cb27342709043e745
[  +18 ms] Exit code 0 from: git tag --points-at cf6c33e58a05b6bf45d8636cb27342709043e745
[   +2 ms] executing: [/home/jpnurmi/Flutter/] git describe --match *.*.* --long --tags cf6c33e58a05b6bf45d8636cb27342709043e745
[  +34 ms] Exit code 0 from: git describe --match *.*.* --long --tags cf6c33e58a05b6bf45d8636cb27342709043e745
[        ] 1.24.0-7.0.pre-343-gcf6c33e58a
[  +98 ms] executing: [/home/jpnurmi/Flutter/] git rev-parse --abbrev-ref --symbolic @{u}
[   +5 ms] Exit code 0 from: git rev-parse --abbrev-ref --symbolic @{u}
[        ] origin/master
[        ] executing: [/home/jpnurmi/Flutter/] git ls-remote --get-url origin
[   +4 ms] Exit code 0 from: git ls-remote --get-url origin
[        ] git@github.com:flutter/flutter.git
[  +64 ms] executing: [/home/jpnurmi/Flutter/] git rev-parse --abbrev-ref HEAD
[   +4 ms] Exit code 0 from: git rev-parse --abbrev-ref HEAD
[        ] master
[  +71 ms] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterWebSdk' is not required, skipping update.
[   +3 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[  +90 ms] Artifact Instance of 'MaterialFonts' is not required, skipping update.
[        ] Artifact Instance of 'GradleWrapper' is not required, skipping update.
[        ] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterWebSdk' is not required, skipping update.
[        ] Artifact Instance of 'FlutterSdk' is not required, skipping update.
[        ] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[   +2 ms] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FontSubsetArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'PubDependencies' is not required, skipping update.
[ +104 ms] Skipping pub get: version match.
[ +214 ms] Building Linux application...
[  +16 ms] executing: [build/linux/release/] cmake -G Ninja -DCMAKE_BUILD_TYPE=Release /home/jpnurmi/Temp/verbose_linux_build/linux
[ +335 ms] -- The CXX compiler identification is Clang 9.0.1
[   +2 ms] -- Detecting CXX compiler ABI info
[ +304 ms] -- Detecting CXX compiler ABI info - done
[  +29 ms] -- Check for working CXX compiler: /usr/bin/clang++ - skipped
[        ] -- Detecting CXX compile features
[   +1 ms] -- Detecting CXX compile features - done
[  +11 ms] -- Found PkgConfig: /usr/bin/pkg-config (found version "0.29.2")
[   +1 ms] -- Checking for module 'gtk+-3.0'
[  +34 ms] --   Found gtk+-3.0, version 3.24.23
[ +181 ms] -- Checking for module 'glib-2.0'
[  +24 ms] --   Found glib-2.0, version 2.66.2
[  +63 ms] -- Checking for module 'gio-2.0'
[  +24 ms] --   Found gio-2.0, version 2.66.2
[  +79 ms] -- Checking for module 'blkid'
[  +23 ms] --   Found blkid, version 2.36.0
[  +63 ms] -- Checking for module 'liblzma'
[  +25 ms] --   Found liblzma, version 5.2.4
[  +79 ms] -- Configuring done
[  +15 ms] -- Generating done
[  +18 ms] -- Build files have been written to: /home/jpnurmi/Temp/verbose_linux_build/build/linux/release
[  +26 ms] executing: ninja -C build/linux/release install
[  +12 ms] ninja: Entering directory `build/linux/release'
[+38462 ms] [1/6] Generating /home/jpnurmi/Temp/verbose_linux_build/linux/flutter/ephemeral/libflutter_linux_gtk.so,
/home/jpnurmi/Temp/verbose_linux_build/linux/flutter/ephemeral/flutter_linux/fl_basic_message_channel.h, /home/jpnurmi/Temp/verbose_linux_build/linux/flutter/ephemeral/flutter_linux/fl_binary_codec.h,
/home/jpnurmi/Temp/verbose_linux_build/linux/flutter/ephemeral/flutter_linux/fl_binary_messenger.h, /home/jpnurmi/Temp/verbose_linux_build/linux/flutter/ephemeral/flutter_linux/fl_dart_project.h,
/home/jpnurmi/Temp/verbose_linux_build/linux/flutter/ephemeral/flutter_linux/fl_engine.h, /home/jpnurmi/Temp/verbose_linux_build/linux/flutter/ephemeral/flutter_linux/fl_json_message_codec.h,
/home/jpnurmi/Temp/verbose_linux_build/linux/flutter/ephemeral/flutter_linux/fl_json_method_codec.h, /home/jpnurmi/Temp/verbose_linux_build/linux/flutter/ephemeral/flutter_linux/fl_message_codec.h,
/home/jpnurmi/Temp/verbose_linux_build/linux/flutter/ephemeral/flutter_linux/fl_method_call.h, /home/jpnurmi/Temp/verbose_linux_build/linux/flutter/ephemeral/flutter_linux/fl_method_channel.h,
/home/jpnurmi/Temp/verbose_linux_build/linux/flutter/ephemeral/flutter_linux/fl_method_codec.h, /home/jpnurmi/Temp/verbose_linux_build/linux/flutter/ephemeral/flutter_linux/fl_method_response.h,
/home/jpnurmi/Temp/verbose_linux_build/linux/flutter/ephemeral/flutter_linux/fl_plugin_registrar.h, /home/jpnurmi/Temp/verbose_linux_build/linux/flutter/ephemeral/flutter_linux/fl_plugin_registry.h,
/home/jpnurmi/Temp/verbose_linux_build/linux/flutter/ephemeral/flutter_linux/fl_standard_message_codec.h, /home/jpnurmi/Temp/verbose_linux_build/linux/flutter/ephemeral/flutter_linux/fl_standard_method_codec.h,
/home/jpnurmi/Temp/verbose_linux_build/linux/flutter/ephemeral/flutter_linux/fl_string_codec.h, /home/jpnurmi/Temp/verbose_linux_build/linux/flutter/ephemeral/flutter_linux/fl_value.h,
/home/jpnurmi/Temp/verbose_linux_build/linux/flutter/ephemeral/flutter_linux/fl_view.h, /home/jpnurmi/Temp/verbose_linux_build/linux/flutter/ephemeral/flutter_linux/flutter_linux.h, _phony_
[  +21 ms] [ +188 ms] executing: [/home/jpnurmi/Flutter/] git -c log.showSignature=false log -n 1 --pretty=format:%H
[        ] [  +59 ms] Exit code 0 from: git -c log.showSignature=false log -n 1 --pretty=format:%H
[        ] [        ] cf6c33e58a05b6bf45d8636cb27342709043e745
[        ] [   +1 ms] executing: [/home/jpnurmi/Flutter/] git tag --points-at cf6c33e58a05b6bf45d8636cb27342709043e745
[        ] [  +18 ms] Exit code 0 from: git tag --points-at cf6c33e58a05b6bf45d8636cb27342709043e745
[        ] [   +2 ms] executing: [/home/jpnurmi/Flutter/] git describe --match *.*.* --long --tags cf6c33e58a05b6bf45d8636cb27342709043e745
[        ] [  +37 ms] Exit code 0 from: git describe --match *.*.* --long --tags cf6c33e58a05b6bf45d8636cb27342709043e745
[        ] [        ] 1.24.0-7.0.pre-343-gcf6c33e58a
[        ] [ +110 ms] executing: [/home/jpnurmi/Flutter/] git rev-parse --abbrev-ref --symbolic @{u}
[        ] [   +4 ms] Exit code 0 from: git rev-parse --abbrev-ref --symbolic @{u}
[        ] [        ] origin/master
[        ] [        ] executing: [/home/jpnurmi/Flutter/] git ls-remote --get-url origin
[        ] [   +4 ms] Exit code 0 from: git ls-remote --get-url origin
[        ] [        ] git@github.com:flutter/flutter.git
[        ] [  +67 ms] executing: [/home/jpnurmi/Flutter/] git rev-parse --abbrev-ref HEAD
[        ] [   +5 ms] Exit code 0 from: git rev-parse --abbrev-ref HEAD
[        ] [        ] master
[        ] [  +68 ms] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[        ] [        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] [        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[        ] [        ] Artifact Instance of 'FlutterWebSdk' is not required, skipping update.
[        ] [   +3 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] [        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[        ] [        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[        ] [        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] [        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] [        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[        ] [        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[        ] [  +83 ms] Artifact Instance of 'MaterialFonts' is not required, skipping update.
[        ] [        ] Artifact Instance of 'GradleWrapper' is not required, skipping update.
[        ] [        ] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[        ] [        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] [        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[        ] [        ] Artifact Instance of 'FlutterWebSdk' is not required, skipping update.
[        ] [  +13 ms] Artifact Instance of 'FlutterSdk' is not required, skipping update.
[        ] [        ] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] [        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[        ] [        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[        ] [        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] [        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] [        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[        ] [        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[        ] [        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] [        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] [        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] [        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] [        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] [        ] Artifact Instance of 'FontSubsetArtifacts' is not required, skipping update.
[        ] [        ] Artifact Instance of 'PubDependencies' is not required, skipping update.
[        ] [ +156 ms] Initializing file store
[        ] [  +38 ms] Skipping target: gen_localizations
[        ] [  +10 ms] unpack_linux: Starting due to {}
[        ] [ +104 ms] kernel_snapshot: Starting due to {}
[        ] [   +5 ms] unpack_linux: Complete
[        ] [+1695 ms] /home/jpnurmi/Flutter/bin/cache/dart-sdk/bin/dart --disable-dart-dev /home/jpnurmi/Flutter/bin/cache/artifacts/engine/linux-x64/frontend_server.dart.snapshot --sdk-root
/home/jpnurmi/Flutter/bin/cache/artifacts/engine/common/flutter_patched_sdk_product/ --target=flutter --no-print-incremental-dependencies -Ddart.developer.causal_async_stacks=false -Ddart.vm.profile=false
-Ddart.vm.product=true --aot --tfa --packages /home/jpnurmi/Temp/verbose_linux_build/.dart_tool/package_config.json --output-dill
/home/jpnurmi/Temp/verbose_linux_build/.dart_tool/flutter_build/4cf4fecc7e2dfc5b6e635efc3e96ba3d/app.dill --depfile
/home/jpnurmi/Temp/verbose_linux_build/.dart_tool/flutter_build/4cf4fecc7e2dfc5b6e635efc3e96ba3d/kernel_snapshot.d package:verbose_linux_build/main.dart
[   +3 ms] [+22586 ms] kernel_snapshot: Complete
[        ] [ +635 ms] aot_elf_release: Starting due to {InvalidatedReason.inputChanged}
[        ] [  +11 ms] executing: /home/jpnurmi/Flutter/bin/cache/artifacts/engine/linux-x64-release/gen_snapshot --deterministic --snapshot_kind=app-aot-elf
--elf=/home/jpnurmi/Temp/verbose_linux_build/.dart_tool/flutter_build/4cf4fecc7e2dfc5b6e635efc3e96ba3d/app.so --strip --no-causal-async-stacks --lazy-async-stacks
/home/jpnurmi/Temp/verbose_linux_build/.dart_tool/flutter_build/4cf4fecc7e2dfc5b6e635efc3e96ba3d/app.dill
[   +1 ms] [+10992 ms] aot_elf_release: Complete
[        ] [  +52 ms] linux_aot_bundle: Starting due to {InvalidatedReason.inputChanged}
[        ] [  +80 ms] linux_aot_bundle: Complete
[        ] [  +53 ms] release_bundle_linux_assets: Starting due to {}
[        ] [ +200 ms] release_bundle_linux_assets: Complete
[        ] [  +64 ms] Persisting file store
[        ] [  +10 ms] Done persisting file store
[        ] [   +8 ms] build succeeded.
[        ] [  +15 ms] "flutter assemble" took 36,841ms.
[        ] [   +7 ms] Running shutdown hooks
[        ] [        ] Shutdown hooks complete
[        ] [        ] exiting with code 0
[+1302 ms] [2/6] Building CXX object CMakeFiles/verbose_linux_build.dir/main.cc.o
[  +52 ms] [3/6] Building CXX object CMakeFiles/verbose_linux_build.dir/flutter/generated_plugin_registrant.cc.o
[ +311 ms] [4/6] Building CXX object CMakeFiles/verbose_linux_build.dir/my_application.cc.o
[ +266 ms] [5/6] Linking CXX executable intermediates_do_not_run/verbose_linux_build
[        ] [5/6] Install the project...
[  +14 ms] -- Install configuration: "Release"
[        ] -- Installing: /home/jpnurmi/Temp/verbose_linux_build/build/linux/release/bundle/verbose_linux_build
[        ] -- Set runtime path of "/home/jpnurmi/Temp/verbose_linux_build/build/linux/release/bundle/verbose_linux_build" to "$ORIGIN/lib"
[        ] -- Installing: /home/jpnurmi/Temp/verbose_linux_build/build/linux/release/bundle/data/icudtl.dat
[   +1 ms] -- Installing: /home/jpnurmi/Temp/verbose_linux_build/build/linux/release/bundle/lib/libflutter_linux_gtk.so
[  +88 ms] -- Installing: /home/jpnurmi/Temp/verbose_linux_build/build/linux/release/bundle/data/flutter_assets
[        ] -- Installing: /home/jpnurmi/Temp/verbose_linux_build/build/linux/release/bundle/data/flutter_assets/version.json
[        ] -- Installing: /home/jpnurmi/Temp/verbose_linux_build/build/linux/release/bundle/data/flutter_assets/NOTICES
[        ] -- Installing: /home/jpnurmi/Temp/verbose_linux_build/build/linux/release/bundle/data/flutter_assets/FontManifest.json
[        ] -- Installing: /home/jpnurmi/Temp/verbose_linux_build/build/linux/release/bundle/data/flutter_assets/packages
[        ] -- Installing: /home/jpnurmi/Temp/verbose_linux_build/build/linux/release/bundle/data/flutter_assets/packages/cupertino_icons
[        ] -- Installing: /home/jpnurmi/Temp/verbose_linux_build/build/linux/release/bundle/data/flutter_assets/packages/cupertino_icons/assets
[        ] -- Installing: /home/jpnurmi/Temp/verbose_linux_build/build/linux/release/bundle/data/flutter_assets/packages/cupertino_icons/assets/CupertinoIcons.ttf
[        ] -- Installing: /home/jpnurmi/Temp/verbose_linux_build/build/linux/release/bundle/data/flutter_assets/fonts
[        ] -- Installing: /home/jpnurmi/Temp/verbose_linux_build/build/linux/release/bundle/data/flutter_assets/fonts/MaterialIcons-Regular.otf
[   +1 ms] -- Installing: /home/jpnurmi/Temp/verbose_linux_build/build/linux/release/bundle/data/flutter_assets/AssetManifest.json
[        ] -- Installing: /home/jpnurmi/Temp/verbose_linux_build/build/linux/release/bundle/lib/libapp.so
[  +25 ms] Building Linux application... (completed in 41.9s)
[   +1 ms] "flutter linux" took 42,402ms.
[   +8 ms] Running shutdown hooks
[        ] Shutdown hooks complete
[   +1 ms] exiting with code 0

$ flutter doctor -v
[✓] Flutter (Channel master, 1.24.0-8.0.pre.343, on Linux, locale en_DK.UTF-8)
    • Flutter version 1.24.0-8.0.pre.343 at /home/jpnurmi/Flutter
    • Framework revision cf6c33e58a (11 hours ago), 2020-11-21 14:04:01 -0800
    • Engine revision 23a8e027db
    • Dart version 2.12.0 (build 2.12.0-62.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at /home/jpnurmi/Android/sdk
    • Platform android-29, build-tools 29.0.3
    • ANDROID_SDK_ROOT = /home/jpnurmi/Android/sdk
    • Java binary at: /usr/lib/jvm/default-java/bin/java
    • Java version OpenJDK Runtime Environment (build <IP_ADDRESS>+1-post-Debian-1)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at google-chrome

[✓] Linux toolchain - develop for Linux desktop
    • clang version 9.0.1-15
    • cmake version 3.18.4
    • ninja version 1.10.1
    • pkg-config version 0.29.2

[!] Android Studio (not installed)
    • Android Studio not found; download from https://developer.android.com/studio/index.html
      (or visit https://flutter.dev/docs/get-started/install/linux#android-setup for detailed instructions).

[✓] VS Code (version 1.51.1)
    • VS Code at /usr/share/code
    • Flutter extension version 3.16.0

[✓] Connected device (3 available)
    • Linux (desktop)  • linux      • linux-x64      • Linux
    • Web Server (web) • web-server • web-javascript • Flutter Tools
    • Chrome (web)     • chrome     • web-javascript • Google Chrome 87.0.4280.66

! Doctor found issues in 1 category.

Reproducible on MacOS builds as well.
[        ] origin/master
[        ] executing: [/Users/pedromassango/dev/SDKs/flutter_master/] git ls-remote --get-url origin
[  +19 ms] Exit code 0 from: git ls-remote --get-url origin
[        ] https://github.com/flutter/flutter.git
[  +83 ms] executing: [/Users/pedromassango/dev/SDKs/flutter_master/] git rev-parse --abbrev-ref HEAD
[  +17 ms] Exit code 0 from: git rev-parse --abbrev-ref HEAD
[        ] master
[  +10 ms] executing: sw_vers -productName
[  +66 ms] Exit code 0 from: sw_vers -productName
[        ] Mac OS X
[        ] executing: sw_vers -productVersion
[  +44 ms] Exit code 0 from: sw_vers -productVersion
[        ] 10.15.7
[        ] executing: sw_vers -buildVersion
[  +33 ms] Exit code 0 from: sw_vers -buildVersion
[        ] 19H2
[   +2 ms] executing: sysctl hw.optional.arm64
[   +9 ms] Exit code 1 from: sysctl hw.optional.arm64
[   +1 ms] sysctl: unknown oid 'hw.optional.arm64'
[  +88 ms] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterWebSdk' is not required, skipping update.
[   +5 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[   +1 ms] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[ +103 ms] executing: /Users/pedromassango/Library/Android/sdk/platform-tools/adb devices -l
[  +15 ms] executing: /usr/bin/xcode-select --print-path
[  +13 ms] Exit code 0 from: /usr/bin/xcode-select --print-path
[   +2 ms] /Applications/Xcode.app/Contents/Developer
[   +4 ms] executing: sysctl hw.optional.arm64
[   +7 ms] Exit code 1 from: sysctl hw.optional.arm64
[        ] sysctl: unknown oid 'hw.optional.arm64'
[        ] executing: xcrun xcodebuild -version
[+1200 ms] Exit code 0 from: xcrun xcodebuild -version
[   +1 ms] Xcode 12.1
           Build version 12A7403
[   +2 ms] executing: xcrun xcdevice list --timeout 2
[   +6 ms] xcrun simctl list --json devices
[        ] executing: xcrun simctl list --json devices
[  +17 ms] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[   +3 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[  +25 ms] Changing current working directory to: /Users/pedromassango/dev/Projects/flutter_example
[  +84 ms] executing: /Users/pedromassango/Library/Android/sdk/platform-tools/adb devices -l
[        ] executing: xcrun xcdevice list --timeout 2
[   +1 ms] xcrun simctl list --json devices
[        ] executing: xcrun simctl list --json devices
[  +34 ms] Skipping pub get: version match.
[ +341 ms] Generating

flutter doctor -v
[✓] Flutter (Channel master, 1.24.0-8.0.pre.365, on Mac OS X 10.15.7 19H2 darwin-x64, locale en)
    • Flutter version 1.24.0-8.0.pre.365 at /Users/pedromassango/dev/SDKs/flutter_master
    • Framework revision 2ceb371d21 (9 hours ago), 2020-11-24 17:18:06 -0800
    • Engine revision 6ed357c8b3
    • Dart version 2.12.0 (build 2.12.0-74.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/pedromassango/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.1, Build version 12A7403
    • CocoaPods version 1.9.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] IntelliJ IDEA Community Edition (version 2020.2.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 51.0.3
    • Dart plugin version 202.8070

[✓] VS Code (version 1.51.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.16.0

[✓] Connected device (4 available)
    • sdk gphone x86 arm (mobile) • emulator-5554 • android-x86    • Android 11 (API 30) (emulator)
    • macOS (desktop)             • macos         • darwin-x64     • Mac OS X 10.15.7 19H2 darwin-x64
    • Web Server (web)            • web-server    • web-javascript • Flutter Tools
    • Chrome (web)                • chrome        • web-javascript • Google Chrome 86.0.4240.198

• No issues found!


RFC: Fix an unsoundness issue around stripping optional properties

Two RFCs that combine to fix an unsoundness issue with table subtyping.

.Could we not just evaluate this to an unsealed type? It was my understanding before testing this that the only way to declare sealed types is to provide a type of an indexer, and any specific keys in a type will make it sealed

User-supplied type annotations are always sealed. Type inference seals tables when they are returned from functions. (We should also be doing alias analysis to ensure unique access to unsealed tables, but that's another story.)

Just a process note... we should probably use one RFC per PR. When two orthogonal features are submitted as one PR, it makes it impossible for us to merge one but not the other. When the features are coupled and must come together, they should ideally come as a single .md file. I'd recommend merging the .md files in this case, especially since they reference each other. (I understand that the RFC proposes two separate changes to the type system but since they both need to happen I think it's better to have fewer .md files and have 1-1 relationship).

These really are different (but related) features, I'd like to keep them as two md files. I'll split into two PRs if you like.

Can we at least remove cross-referencing then? Making all table literals unsealed seems to stand on its own at least.

One downside to avoiding sealing non-empty table literals is that it leads to typos in assignment being difficult/impossible to detect:
local t = {x = 1, y = 2}
if foo then
t.z = 3 -- is z a typo or intentional 2-vs-3 choice?
end

Which I suppose is why we're sealing them currently. Of course it's unclear what to do about this in absence of type annotations or record types, so it's probably fine - the code above could easily be valid or invalid depending on the intent. My understanding is that we will still detect uses of keys that were never written to.

Yes, not much we can do about that though. I'll add this point to the downsides section.

I removed the cyclic cross-referencing, so the subtyping proposal refers to the table literals proposal, but not the other way round.

Control flow analysis will be able to keep an eye on properties that are definitely-or-maybe assigned. The same problem still arise even when the tables are empty anyway regardless of this RFC.
local t = {}
if cond()
  t.x = 5
end

local some_number: number = t.x


Fix a few lint issues

Code Coverage

File
Coverage [0.00%]
:x:

app/src/main/java/com/example/myapplication/RecipesApp.kt
0.00%
:x:

app/src/main/java/com/example/myapplication/di/AppModule.kt
0.00%
:x:

app/src/main/java/com/example/myapplication/recipes/data/datasource/remote/RecipeBackend.kt
0.00%
:x:

app/src/main/java/com/example/myapplication/recipes/domain/usecase/DeleteRecipe.kt
0.00%
:x:

app/src/main/java/com/example/myapplication/recipes/domain/usecase/Recipes.kt
0.00%
:x:

Total Project Coverage
14.74%
:white_check_mark:


Modify Snapshot Event to publish updated local changes

What this PR does / why we need it?
This PR updates the behavior of the Snapshot Event to ensure that it publishes the most recent data reflecting local changes. Additionally, it modifies the CodeMirror example to update the cursor position when a snapshot occurs, which addresses an issue where the cursor momentarily jumps to the top.

Before: Cursor is moved to unintended position when the snapshot event is published (--backend-snapshot-threshold=1)
https://github.com/user-attachments/assets/ec3b74d9-33c3-4f3b-90bf-c12faaa72482

After: Cursor is not moving when the snapshot event is published (--backend-snapshot-threshold=1)
https://github.com/user-attachments/assets/13769761-f218-4bd8-8819-77411f4c4d56

Any background context you want to provide?
The changes made in this PR aim to resolve this issue by ensuring the local changes are reflected in the snapshot and that the cursor position is updated accordingly during the snapshot process.

Problem: When dragging text with mouse, the anchor is moved to 0. (This issue will be handled in https://github.com/yorkie-team/codepair/issues/402)
https://github.com/user-attachments/assets/0c406ef7-676b-46f6-b737-76f173300ba5
I suspect that the issue occurs because the cursor briefly moves to the top when the code below runs.
view.dispatch({
  changes: { from: 0, to: view.state.doc.length, insert: text.toString() },
  annotations: [Transaction.remote.of(true)],
  selection: {
    anchor: ...,
    head: ...,
  }
});

What are the relevant tickets?

Fixes #922
Checklist

[x] Added relevant tests or not required
[x] Didn't break anything

Summary by CodeRabbit
Summary by CodeRabbit

New Features

Enhanced collaborative editing capabilities with improved management of selections and operations.
Introduced a new type, YorkiePresence, to better handle user presence and selection states.

Bug Fixes

Improved logic for applying local changes and publishing snapshot events, ensuring accurate document state representation.

Documentation

Updated type definitions for better clarity and functionality.

@hackerwins I checked your change that moves the call to removePushedLocalChanges.
It seems there’s a condition corresponding to the else part, making it different from the previous logic. What do you think about this?
if (hasSnapshot) {
  this.applySnapshot(
    pack.getCheckpoint().getServerSeq(),
    pack.getVersionVector()!,
    pack.getSnapshot()!,
    pack.getCheckpoint().getClientSeq(),
  );
} else if (pack.hasChanges()) {
  this.applyChanges(pack.getChanges(), OpSource.Remote);
  this.removePushedLocalChanges(pack.getCheckpoint().getClientSeq());
} else {
  // Here
}


need to add ABSTRACT for report

@thenukagunawardana You can use the following link to edit the report.
https://docs.google.com/document/d/1Gj0j69Go88hPZGo5Q1cJfuL9tE2skUHn/edit?usp=sharing&ouid=105856518530847106049&rtpof=true&sd=true

Done


82-nav-menu-open-close-animation

closes #82

I was studying that html course. I'll inform you once this one gets completed.


manageiq container is missing an rpm

Per commit https://github.com/ManageIQ/manageiq-appliance-build/pull/116/files
openscap-scanner package is required to run openscap on a container image.
when it is not present the SSA for containers will fail.
This is relevant only for upstream

@miq-bot add_label build/docker

@bazulay Anything else needed besides the openscap RPMs on image?

Fixed by https://github.com/ManageIQ/manageiq/pull/13469


Removed color: inital from the caret element #5076

Proposed changes
Remove the
color: inital

from the .caret element. #5076
Types of changes

[x] Bug fix (non-breaking change which fixes an issue).
[ ] New feature (non-breaking change which adds functionality).
[ ] Breaking change (fix or feature that would cause existing functionality to change).

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Why?

It isn't intended that the caret is seen in select. So this change is not necessary


catt cast always fails with the same error

when I try to cast a local mp4 file I get
Exception thrown when calling load failed callback
Traceback (most recent call last):
  File "/home/kevin/.local/lib/python3.13/site-packages/pychromecast/controllers/media.py", line 771, in _fire_load_failed
    listener.load_media_failed(queue_item_id, error_code)
    ^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'MediaStatusListener' object has no attribute 'load_media_failed'
Error: Playback of local file has failed.

it looks like MediaStatusListener is defined here https://github.com/skorokithakis/catt/blob/b4e33f91e44daab87ae26a4d3f437a62e9cce6f9/catt/controllers.py#L198 and it is definitely missing the load_media_failed method.

Hm, I think PyChromecast changed its API again... Any PRs would be welcome.


Bloom query doesn't accept arbitrary String value as arg of search phrase?

It happens "WITH" search suggestions:

Or "WITHOUT" search suggestions too:

Hi @complyue! Thank you for reporting this.
It appears there's indeed a bug in the search bar when it comes to handling search phrase parameters. When pasting or fully typing out search phrase parameters, Bloom's search does not seem to recognize the parameter as fully provided. Until this is fixed, a workaround is to press the tab key after pasting the message id and before running the search. I hope this helps in the meantime until the bug is fixed.
Please let me know if you have other questions. Thank you again for taking the time to report this.

Hi @complyue, I just tested with the latest release and can confirm that a fix for this has been released.
I will be closing this issue, but if you have any other issues with Bloom, please feel free to raise a new issue here or on Canny https://neo4j-aura.canny.io/explore

Hi @ckanz, could you be more specific about the "latest" release? The publicly available bloom atm, via Neo4j Desktop 1.5.9 is still Bloom 2.11.0, doesn't seem to have the fix incorporated.


Add option to turn off total count

I ahve an issue with an api I'm working on that queries against a collection wit millions of records. I want to return the Link header but not the total to avoid the extra query call for total count.
It would be nice if this was a configurable option.

Hmm... This seems reasonable but I feel like I need to ask: how long does the SELECT COUNT(*) ... query take for your collection? Is it really enough that generating the “Total” header is a noticeable performance loss?

On the one query that's looking at 2.7M records, it takes almost a minute for the api call to complete.
I'm not sure it's a big concern for anyone else, so I just hacked something together here:
https://github.com/bitium/api-pagination/commit/dcdf526a283f885a9cebfea500949c7438c1d87b
Basically allows paginate to take a skip_total option and modifies the way the data is returned. Can probably be improved.

You could try https://github.com/desktoppr/cached_counts also

I'd like this as well.

is this already implemented? I set config.include_total = false but the pagination is still querying the count.

@Phifo It looks like the patch I included only skips a query for calculating the Total header. But since the number of records is still necessary for calculating the last page… It's gonna execute a COUNT query regardless 😞
Unfortunately, at this point I think this is a WONTFIX for me. I'd rather not support this sort of edge case in the api-pagination gem and need to implement functionality that prevents what I feel like are all of the necessary links in the Link header. I'll likely deprecate the config.include_total patch before v5.0 as well because of this.
I strongly recommend looking into ways to speed up query performance on that table instead. I know that's not a great answer. Sorry!

I think it's a pretty common thing to paginate across large result sets, I wouldn't consider it an edge case.

"Edge case" probably wasn't the best phrase, but what it comes down to is that the case of paginating across a large result set where the COUNT query takes unreasonably long is not a case I'd like api-pagination to try to deal with… In the case of this gem, it means leaving out the Total header and the link to the last page in the Link header. I was okay with the first result. I'm not really okay with the latter one.

Why are you not ok with skipping the last header? I am. :)

@davidcelis your answer makes a lot of sense, I should find a way to speed up the count query. Leaving out the last page in the link header will not follow the standard.


How do I get the dlls in my ADLS account?

I'm running the script almost exactly as listed above - I just changed the FROM path for the first extract. When I try to run this job, however, I get a message saying "Assembly 'master.ImageCommon' does not exist.". I've tried creating a new DB, and still getting the same error, but with the new DB in the Assembly name.
Do I need to somehow enable Cognitive extensions on my ADLA account?

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

ID: 333230aa-2763-16eb-d2f8-8baa95e242ec
Version Independent ID: 7f3c4f63-2870-a5ec-d42e-142578f99ff6
Content: Using U-SQL Cognitive capabilities in Azure Data Lake Analytics | Microsoft Docs
Content Source: articles/data-lake-analytics/data-lake-analytics-u-sql-cognitive.md
Service: data-lake-analytics
GitHub Login: @saveenr
Microsoft Alias: saveenr

i also just went into my ADLA account and clicked the link for "Add U-SQL extensions". Tried my job again, and got the same failure.

After clicking the link for "Add U-SQL extensions", I had to wait for a job to run that registered those extensions.

@zartcosgrove
Thanks for the feedback! We are currently investigating and will update you shortly.

@zartcosgrove
I have assigned the issue to the content author to investigate further and update the document as appropriate.
@saveenr
I was able to replicate the issue and resolve it by installing the U-SQL extensions from the Data Lakes Analytics - "Sample scripts" blade, and checking on the installation job completion from the "Job management" blade.

@saveenr
A link to the installation instructions for the Cognitive Extensions in U-SQL can be found here:
https://msdn.microsoft.com/en-us/azure/data-lake-analytics/u-sql/cognitive-capabilities-in-u-sql

Seems like the installation instructions on the the docs page would help. https://msdn.microsoft.com/en-us/azure/data-lake-analytics/u-sql/cognitive-capabilities-in-u-sql#registeringExtensions
@zartcosgrove please let us know if this didn't resolve your issue.
#please-close

I agree, it seems like the installation instructions on the docs page would
help - why isn't this link included on the Getting Started page? How can
someone get started without first installing the extensions? The first step
for any "getting started" page, since it's assuming that people are just
getting started, should be to quickly link to prerequisites like
installation.
I got to this page by googling "Getting started with Cognitive Services
U-SQL". This is how many people are going to find this page. We should make
sure that it'll have what's necessary for people to get up and running.
On Tue, Jun 5, 2018 at 8:51 AM, Jason H<EMAIL_ADDRESS>wrote:

Seems like the installation instructions on the the docs page would help.
https://msdn.microsoft.com/en-us/azure/data-lake-analytics/
u-sql/cognitive-capabilities-in-u-sql#registeringExtensions
@zartcosgrove https://github.com/zartcosgrove please let us know if
this didn't resolve your issue.
#please-close
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/5507#issuecomment-394762011,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AI5XkVJ8fZGEuZn4QeknvByTeH-gOtVEks5t5qkMgaJpZM4SjiR9
.

--
I must not fear.
Fear is the mind-killer.
Fear is the little-death that brings total obliteration.
I will face my fear.
I will permit it to pass over me and through me.
And when it has gone past I will turn the inner eye to see its path.
Where the fear has gone there will be nothing.
Only I will remain.

Plus - I first issued this comment March 8th. It's three months later, and
no one has updated the documentation? That's not very agile.
On Tue, Jun 5, 2018 at 9:07 AM, Zart Cosgrove<EMAIL_ADDRESS>wrote:

I agree, it seems like the installation instructions on the docs page
would help - why isn't this link included on the Getting Started page? How
can someone get started without first installing the extensions? The first
step for any "getting started" page, since it's assuming that people are
just getting started, should be to quickly link to prerequisites like
installation.
I got to this page by googling "Getting started with Cognitive Services
U-SQL". This is how many people are going to find this page. We should make
sure that it'll have what's necessary for people to get up and running.
On Tue, Jun 5, 2018 at 8:51 AM, Jason H<EMAIL_ADDRESS>wrote:

Seems like the installation instructions on the the docs page would help.
https://msdn.microsoft.com/en-us/azure/data-lake-analytics/u
-sql/cognitive-capabilities-in-u-sql#registeringExtensions
@zartcosgrove https://github.com/zartcosgrove please let us know if
this didn't resolve your issue.
#please-close
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/5507#issuecomment-394762011,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AI5XkVJ8fZGEuZn4QeknvByTeH-gOtVEks5t5qkMgaJpZM4SjiR9
.

--
I must not fear.
Fear is the mind-killer.
Fear is the little-death that brings total obliteration.
I will face my fear.
I will permit it to pass over me and through me.
And when it has gone past I will turn the inner eye to see its path.
Where the fear has gone there will be nothing.
Only I will remain.

--
I must not fear.
Fear is the mind-killer.
Fear is the little-death that brings total obliteration.
I will face my fear.
I will permit it to pass over me and through me.
And when it has gone past I will turn the inner eye to see its path.
Where the fear has gone there will be nothing.
Only I will remain.

@zartcosgrove Sorry for the frustration. I plan on fixing it today. Your feedback is appreciated. Thanks, Jason

Thanks Jason!
On Tue, Jun 5, 2018 at 9:16 AM, Jason H<EMAIL_ADDRESS>wrote:

@zartcosgrove https://github.com/zartcosgrove Sorry for the
frustration. I plan on fixing it today. Your feedback is appreciated.
Thanks, Jason
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/5507#issuecomment-394770707,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AI5XkYcscICJLx4JA7kbWuCYb9dipFRSks5t5q7wgaJpZM4SjiR9
.

--
I must not fear.
Fear is the mind-killer.
Fear is the little-death that brings total obliteration.
I will face my fear.
I will permit it to pass over me and through me.
And when it has gone past I will turn the inner eye to see its path.
Where the fear has gone there will be nothing.
Only I will remain.

Why was this closed? The documentation has not been updated.
On Tue, Jun 5, 2018 at 10:27 AM, Shilpa Sharma<EMAIL_ADDRESS>wrote:

Closed #5507 https://github.com/MicrosoftDocs/azure-docs/issues/5507
via 00b1d2017e36fbcb6e0b33892cc05e001ca1e2c0.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/5507#event-1664291239,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AI5XkQ-igRo_xWwLCJy_3GgFCtLeKnQKks5t5r-DgaJpZM4SjiR9
.

--
I must not fear.
Fear is the mind-killer.
Fear is the little-death that brings total obliteration.
I will face my fear.
I will permit it to pass over me and through me.
And when it has gone past I will turn the inner eye to see its path.
Where the fear has gone there will be nothing.
Only I will remain.

@zartcosgrove It was updated in the backend, but takes a few hours to publish.


Error while running AMP validation using API end point

_http_outgoing.js:491
throw new Error('Can't set headers after they are sent.');
^
Error: Can't set headers after they are sent.
at validateHeader (_http_outgoing.js:491:11)
at ServerResponse.setHeader (_http_outgoing.js:498:3)
at ServerResponse.header (F:\ampbench\ampbench\node_modules\express\lib\resp
onse.js:767:10)
at ServerResponse.send (F:\ampbench\ampbench\node_modules\express\lib\respon
se.js:170:12)
at ServerResponse.json (F:\ampbench\ampbench\node_modules\express\lib\respon
se.js:267:15)
at on_output (F:\ampbench\ampbench\ampbench_routes.js:534:29)
at IncomingMessage.res.on (F:\ampbench\ampbench\ampbench_lib.js:961:25)
at emitNone (events.js:111:20)
at IncomingMessage.emit (events.js:208:7)
at endReadableNT (_stream_readable.js:1064:12)
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: node ampbench_main.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional log
ging output above.

Hi amandv,
Thank you for raising this issue.
Could you please provide the test details so that we can actually reproduce this outcome in order to investigate.
Thanks //pieter


zqd: run suricata-update at startup

closes #1552

In the future, we could let Brim trigger/request a rules update, obtain
info abou the current rules via zqd apis. Getting this auto-update in now
lays some groundwork for that, and ensures we have some mechanism to keep
the rules updated.

@henridf, my preferred approach would be to have the suricata updater run by brim instead. Is the reason why zqd needs to do this because the update might be occurring while a pcap ingest is started, and this could cause something weird to happen?


Improve depth entry recognition for cases such as '.80'.

And here the PR to include '.80' and the like as well.

I made two changes:

Apply individual corrections to a single depth value, instead of trying all possible corrections at once. For example, from a value of "40", we would like to try the possibilities "0.40" as well as "10", but with the original implementation, we would only try "0.10". This currently does not make a difference for our datasets, but I think it makes more sense to apply corrections individually, especially as we will likely add more possible corrections in the future.
Allow for more than one depth value to be corrected in a depth column. This improves the results for 680244005-bp.pdf (where two corrections of the 4->1 type are needed), while for 16005.pdf the multiple corrections lead to a column of 3 different values to be incorrectly recognized as a depth column (though it does not make a difference for the F1 score, as this document scores 0% anyway). Maybe allowing a certain number of corrections that is proportional to the total number of entries in the depth column, would be most robust? However, with so few documents where this makes a difference at all, I don't really think that it's worthwhile implementing such a rule with a additional parameter that needs to be tuned. @redur what do you think, should we stick to allowing one correction per column for now, or shall we drop that limit?

I made two changes:

Apply individual corrections to a single depth value, instead of trying all possible corrections at once. For example, from a value of "40", we would like to try the possibilities "0.40" as well as "10", whereas with the original implementation, we would only try "0.10". This currently does not make a difference for our datasets, but I think it makes more sense to apply corrections individually, especially as we will likely add more possible corrections in the future.
Allow for more than one depth value to be corrected in a depth column. This improves the results for 680244005-bp.pdf (where two corrections of the 4->1 type are needed), while for 16005.pdf the multiple corrections lead to a column of 3 different values to be incorrectly recognized as a depth column (though it does not make a difference for the F1 score, as this document scores 0% anyway). Maybe allowing a certain number of corrections that is proportional to the total number of entries in the depth column, would be most robust? However, with so few documents where this makes a difference at all, I don't really think that it's worthwhile implementing such a rule with a additional parameter that needs to be tuned. @redur what do you think, should we stick to allowing one correction per column for now, or shall we drop that limit?

Thank you for the improvements!
I agree to your first point. This makes a lot of sense.
Regarding your second point, I initially did not want to do this as I had the opinion that "corrections" should be applied scarcely to minimize the risk of "correcting" something that was correct.


feat(behavior_velocity_planner): add new 'dynamic_obstacle_stop' module

Description

Add parameter file for a new behavior_velocity_planner module.
Required for https://github.com/autowarefoundation/autoware.universe/pull/5835
Tests performed

PSim
Evaluator (TIER IV INTERNAL LINK): https://evaluation.tier4.jp/evaluation/reports/36edca70-1018-5410-9957-4412db637bbc?project_id=prd_jt
Effects on system behavior

Not applicable.
Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

@satoshi-ota

Please add launch flag in default_preset.yaml.

Done: d83fbda


WIP - indexTracker custom cycle times

closes #357

Pull Request Test Coverage Report for Build 472004808

29 of 33 (87.88%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.3%) to 34.242%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/tracker/index.go
29
33
87.88%

Files with Coverage Reduction
New Missed Lines
%

pkg/tracker/psrs.go
2
81.94%

Totals

Change from base Build 468747351:
0.3%

Covered Lines:
1382

Relevant Lines:
4036

💛  - Coveralls

Pull Request Test Coverage Report for Build 472004808

29 of 33 (87.88%)  changed or added relevant lines in 1 file are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.3%) to 34.242%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/tracker/index.go
29
33
87.88%

Files with Coverage Reduction
New Missed Lines
%

pkg/tracker/psrs.go
2
81.94%

Totals

Change from base Build 468747351:
0.3%

Covered Lines:
1382

Relevant Lines:
4036

💛  - Coveralls

Also I just remembered that there should be some check that no index interval should be smaller than the global tracker cycler.

Also I just remembered that there should be some check that no index interval should be smaller than the global tracker cycler.

added

added


feat: helm

Helm!

Thanks for the PR!
Any successful deployments (not always required) will be available below.
Backend available
Frontend available
Once merged, code will be promoted and handed off to following workflow run.
Main Merge Workflow

This PR is a mess.  The work can go into something cleaner.  Closing.


Add sorting to lists.

Allow sorting users and repos.
Default to alphabetic.

Could also add some simple filtering.


added flutter_gen for automatic asset generation

This change will create a folder named "generated" under lib/product/ which contains all of the assets of the app.

@cem256 thank you so much for supprot, actullay we were thinking this feature but we could not find a time thank u


Update AMC regression

This PR includes minor internal refactoring of the AMC regression mechanics.

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

Comparison is base (2bda6b6) 98.97% compared to head (d92b368) 98.97%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #51      +/-   ##
==========================================
- Coverage   98.97%   98.97%   -0.01%
==========================================
  Files          59       59
  Lines        2934     2933       -1
==========================================
- Hits         2904     2903       -1
  Misses         30       30

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


Recent item home

References
Add references/links to any related issues or PRs. These may include:

Fixes #[issue-number]
Requires DSpace/DSpace#[pr-number] (if a REST API PR is required to test this)

Description
Short summary of changes (1-2 sentences).
Instructions for Reviewers
Please add a more detailed description of the changes made by your PR. At a minimum, providing a bulleted list of changes in your PR is helpful to reviewers.
List of changes in this PR:

First, ...
Second, ...

Include guidance for how to test or review your PR. This may include: steps to reproduce a bug, screenshots or description of a new feature, or reasons behind specific changes.
Checklist
This checklist provides a reminder of what we are going to look for when reviewing your PR. You need not complete this checklist prior to creating your PR (draft PRs are always welcome). If you are unsure about an item in the checklist, don't hesitate to ask. We're here to help!

[ ] My PR is small in size (e.g. less than 1,000 lines of code, not including comments & specs/tests), or I have provided reasons as to why that's not possible.
[ ] My PR passes TSLint validation using yarn run lint
[ ] My PR doesn't introduce circular dependencies
[ ] My PR includes TypeDoc comments for all new (or modified) public methods and classes. It also includes TypeDoc for large or complex private methods.
[ ] My PR passes all specs/tests and includes new/updated specs or tests based on the Code Testing Guide.
[ ] If my PR includes new, third-party dependencies (in package.json), I've made sure their licenses align with the DSpace BSD License based on the Licensing of Contributions documentation.

@DSquareTechnologies : I was just reviewing all open PRs for possible inclusion in 7.3 and stumbled on this PR without any description.  Is this an accidental PR, or is this the "Recent Submissions" work you've been working on?
If it's the "Recent Submissions" work, we'd need a description added to this PR, including details on how testers can test this feature. We'd also need the failing tests to be fixed in this PR.
As a sidenote, our deadlines for 7.3 are rapidly approaching.  PRs which should be considered for 7.3 must be created by April 28 (or very shortly after), that allows reviewers enough time to review them before May 19.  All PRs for 7.3 MUST be merged by May 27, or they will be rescheduled for 7.4 (due in Oct).  If you have any other outstanding work you'd like considered for 7.3, please create PRs as soon as reasonably possible. Thanks!

Gaurav has updated required details in the PR. Please confirm if that's
fine?
On 21-04-2022 21:45, Tim Donohue wrote:

@DSquareTechnologies https://github.com/DSquareTechnologies : I was
just reviewing all open PRs for possible inclusion in 7.3 and stumbled
on this PR without any description. Is this an accidental PR, or is
this the "Recent Submissions" work you've been working on?
If it's the "Recent Submissions" work, we'd need a description added
to this PR, including details on how testers can test this feature.
We'd also need the failing tests to be fixed in this PR.
As a sidenote, our deadlines for 7.3 are rapidly approaching. PRs
which should be considered for 7.3 must be created by April 28 (or
very shortly after), that allows reviewers enough time to review them
before May 19. All PRs for 7.3 MUST be merged by May 27, or they will
be rescheduled for 7.4 (due in Oct). If you have any other outstanding
work you'd like considered for 7.3, please create PRs as soon as
reasonably possible. Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/DSpace/dspace-angular/pull/1558#issuecomment-1105431389,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABQHNKUYA2XBPZB4VF4FWVTVGF5L5ANCNFSM5Q4FKN5Q.
You are receiving this because you were mentioned.Message ID:
@.***>

Is it possible to assign this PR to gauravd2t?

From: Tim Donohue @.>
Sent: Tuesday, April 26, 2022 8:51:37 PM
To: DSpace/dspace-angular @.>
Cc: DSquare Technologies @.>; Mention @.>
Subject: Re: [DSpace/dspace-angular] Recent item home (PR #1558)
@DSquareTechnologieshttps://github.com/DSquareTechnologies : I answered @GauravD2thttps://github.com/GauravD2t in his repository over at GauravD2t/dspace#2https://github.com/GauravD2t/dspace/pull/2 It appears that a formal PR has not been submitted to DSpace/dspace-angular... unless this PR is your "formal" PR.
If this is your formal PR, then please update the description above to detail how to test it, etc. We'd also need to ensure the existing automated tests succeed, as several of them are currently failing
So, currently, I don't have a PR which is in a reviewable state. I don't have a way to move this forward yet.
If you or @GauravD2thttps://github.com/GauravD2t need help on creating a PR, see https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request-from-a-fork Please make sure that when you create the PR you fill out a description which helps testers understand the PR, how to test it (and enable it if it requires configurations to enable), etc. Thanks!
—
Reply to this email directly, view it on GitHubhttps://github.com/DSpace/dspace-angular/pull/1558#issuecomment-1109931747, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABQHNKXQZQPURR2JSQRJ4ZDVHACYDANCNFSM5Q4FKN5Q.
You are receiving this because you were mentioned.Message ID: @.***>

Okay he will create a new PR in morning. Hope his account has rights to do so. Thanks

From: Tim Donohue @.>
Sent: Wednesday, April 27, 2022 12:50:20 AM
To: DSpace/dspace-angular @.>
Cc: DSquare Technologies @.>; Mention @.>
Subject: Re: [DSpace/dspace-angular] Recent item home (PR #1558)
@DSquareTechnologieshttps://github.com/DSquareTechnologies : No, GitHub doesn't allow you to give a PR from one account to another. What we could do is to close this PR & have @GauravD2thttps://github.com/GauravD2t send a new PR to DSpace/dspace-angular. That way he's the owner of the PR & can update it, fix tests, etc.
—
Reply to this email directly, view it on GitHubhttps://github.com/DSpace/dspace-angular/pull/1558#issuecomment-1110162169, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABQHNKRR54UUH5OKNRB6XCDVHA6XHANCNFSM5Q4FKN5Q.
You are receiving this because you were mentioned.Message ID: @.***>

Thanks @DSquareTechnologies !  I'll close this PR then as it seems like a new PR is the easiest approach.  @GauravD2t definitely has the rights to create a new PR, as any GitHub account can send us PRs (there's no restrictions, we accept PRs from anyone who wants to send them).

Here is the URL passed by Gaurav
https://github.com/GauravD2t/dspace-angular/pull/3

From: DSquare Technologies @.>
Sent: Wednesday, April 27, 2022 12:51:30 AM
To: DSpace/dspace-angular @.>; DSpace/dspace-angular @.>
Cc: Mention @.>
Subject: Re: [DSpace/dspace-angular] Recent item home (PR #1558)
Okay he will create a new PR in morning. Hope his account has rights to do so. Thanks

From: Tim Donohue @.>
Sent: Wednesday, April 27, 2022 12:50:20 AM
To: DSpace/dspace-angular @.>
Cc: DSquare Technologies @.>; Mention @.>
Subject: Re: [DSpace/dspace-angular] Recent item home (PR #1558)
@DSquareTechnologieshttps://github.com/DSquareTechnologies : No, GitHub doesn't allow you to give a PR from one account to another. What we could do is to close this PR & have @GauravD2thttps://github.com/GauravD2t send a new PR to DSpace/dspace-angular. That way he's the owner of the PR & can update it, fix tests, etc.
—
Reply to this email directly, view it on GitHubhttps://github.com/DSpace/dspace-angular/pull/1558#issuecomment-1110162169, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABQHNKRR54UUH5OKNRB6XCDVHA6XHANCNFSM5Q4FKN5Q.
You are receiving this because you were mentioned.Message ID: @.***>

@DSquareTechnologies : Unfortunately, it seems that @GauravD2t has misunderstood how to send us a PR.  That link you shared is in Guarav's own GitHub repository. As you can see from the link it starts with "https://github.com/GauravD2t/dspace-angular/" which is not correct.  It needs to be a PR under https://github.com/DSpace/dspace-angular/
I've added a comment to Guarav with more information at https://github.com/GauravD2t/dspace/pull/3#issuecomment-1111240104


Improve api testing

Codecov Report

Merging #188 (f597ba7) into main (2f59e65) will increase coverage by 0.61%.
The diff coverage is n/a.

:exclamation: Current head f597ba7 differs from pull request most recent head db759c7. Consider uploading reports for the commit db759c7 to get more accurate results

@@             Coverage Diff              @@
##               main     #188      +/-   ##
============================================
+ Coverage     79.05%   79.66%   +0.61%
  Complexity      110      110
============================================
  Files            42       42
  Lines          2115     2115
  Branches        314      314
============================================
+ Hits           1672     1685      +13
+ Misses          304      289      -15
- Partials        139      141       +2

Impacted Files
Coverage Δ

...tor/kamelet/KameletDeploymentGeneratorService.java
66.66% <0.00%> (+2.22%)
:arrow_up:

...va/io/kaoto/backend/deployment/ClusterService.java
78.67% <0.00%> (+3.67%)
:arrow_up:

...elroute/IntegrationDeploymentGeneratorService.java
55.73% <0.00%> (+11.47%)
:arrow_up:

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


Bump path to regexp

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

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 100.0%

Totals

Change from base Build<PHONE_NUMBER>8:
0.0%

Covered Lines:
400

Relevant Lines:
400

💛  - Coveralls


Test function

Would it be possible to implement a test feature that sends an SMS / Email to see if everything is working correctly? I don't believe I'm getting an e-mail.

There is always a test for smtp config, it tries to connect to mail server on startup to check credentials. But sending actual test mail is also a good idea.
I will think about it, maybe some shell argument like
 ./crawler.py -test

Besides, if you are looking for KS-1 then its about one mail per week.

Yeah that would be awesome :)
I am indeed looking for KS-1, it's pretty hard to get it :(

Has this been implemented yet? It looks like all my settings are correct but I yet again missed the server I wanted to buy and I did not get an email when it was available.

It's implemented; SMTP config is always checked automatically during startup. If credentials are wrong you will get an error message in terminal. If you have doubt, the easiest way to double-check: put some more popular servers in your config (eg KS-4) and see if you get notifications for it.


How to run the unit tests ? - do not see any help option

Hello,
I was trying to run some unit tests built default with libspdm compilation but I am not sure how to run them exactly ? I don't see any help option as well on how to run these tests, specifically looking for details what types of arguments needs to be supplied, if it's a file then what should be the contents of those files which will be provided as arguments to some of these test binaries. Is there any document with details on this ?
For example - there are quite of responder related tests,  please see below a test was run but it's asking about some file args.
So unsure what contents should be put in file and what it represent to the test.
root@dpdk-1:~/manish/libspdm/build/bin# ls test_spdm_responder*
test_spdm_responder                 test_spdm_responder_encap_challenge        test_spdm_responder_finish_rsp     test_spdm_responder_psk_exchange_rsp
test_spdm_responder_algorithms      test_spdm_responder_encap_get_certificate  test_spdm_responder_heartbeat_ack  test_spdm_responder_psk_finish_rsp
test_spdm_responder_capabilities    test_spdm_responder_encap_get_digests      test_spdm_responder_if_ready       test_spdm_responder_version
test_spdm_responder_certificate     test_spdm_responder_encap_key_update       test_spdm_responder_key_exchange
test_spdm_responder_challenge_auth  test_spdm_responder_encap_response         test_spdm_responder_key_update
test_spdm_responder_digests         test_spdm_responder_end_session            test_spdm_responder_measurements
/root/libspdm/build/bin# ./test_spdm_responder_version
error - missing input file
/root/libspdm/build/bin# ./test_spdm_responder_version --help
file error
/root/libspdm/build/bin# ./test_spdm_responder_version -h
file error
Thanks,
Manish

The unit test is test_spdm_requester and test_spdm_responder
The test_spdm_requester_get_version is for fuzzing, and it expects a seed.

For unit test, please refer to https://github.com/DMTF/libspdm#run-unit_test
For test related topic in libspdm, please refer to https://github.com/DMTF/libspdm/blob/main/doc/test.md
You may also refer to spdm_emu, please refer to https://github.com/DMTF/libspdm#run-spdm_emu


Add ability to split expenses

As a user, I want to be able to split an expense equally among all group members or specify different shares for each person, so that the app accurately reflects our agreement on how costs are shared.

Duplicate of #1


Code workflow

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: sakaldeepyadav sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Bugfix/equal weight checkbox on edit

#532 also, currently the "weighted equally" checkbox is always false when editing an existing experiment. This is probably because the backend isn't sending a value for this, so I added some logic in edit flow to mark it as true manually if all the conditions are actually equal.

I'm not sure if this is the bug you tried to fix but updating the "Weight Equally" checkbox still doesn't seem to work properly.
I tried it in this branch.
https://user-images.githubusercontent.com/90279765/189432431-30a91965-a8a8-4607-8375-38af32b2c158.mov

The scenario in your video would be if the user 1) unchecked weighted equally, then 2) weighted things equally anyway. If that user went in to edit, they would see 'weighted-equally' checked. True, that's not what they technically selected... but it is weighted equally, so does it matter?
(The weighted-equally checkbox exists for the UI only, no value is sent to the backend for this, so it will never "remember" this selection. This seemed like a good workaround vs updating the API for something pretty small).

@danoswaltCL I think you are right, it doesn't matter and in some ways, it makes more sense to enable the checkbox when the weights are equal.

But one problem though is that it doesn't seem to work the same way when there are 3 equally-weighted conditions. The checkbox is disabled when I open the stepper the next time which I think is a little confusing (because the "Weight Equally" checkbox is disabled while actually, they are equally weighted - unlike when there are 2 conditions).
https://user-images.githubusercontent.com/90279765/189671081-17e66864-5e74-4672-a96a-28b91c29b3f4.mov

@zackcl bah you're right, these darn decimals. I pushed a change.


Broken link in Readme

The "definitions directory" link in the readme looks broken.

thx, I'll have a look to fix it


fix: initial visible bug

Without setting defaultSizes, when a pane is initially visible, toggling visibility results in a weird bug where the layout is shifted as if there were an additional pane.
To fix this, I added a "default" descriptor for views when defaultSizes is not set.
It's not really clear to me why this works, because it pretty much does the same thing as the for (const enterKey of enter) {...} block.

Hi @adri1wald. I'm not ignoring this PR I just need to find the time to take a proper look (hopefully today). Thanks for your help!

@adri1wald I wonder if this was being caused by an incorrect return statement (should have been a continue). I've fixed this in d06931b7ba37248f4c61f0768d3ba571141b4fdf and published a new release 1.11.1.

Thanks for making the fix!


pythonPackages.webdavclient3: init at 3.14.5

Motivation for this change
I needed a Python-based WebDAV client library, that can be imported as a module.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Determined the impact on package closure size (by running nix path-info -S before and after) 0 -> 127593712
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Thank you 😀


Encoding site: http://code-cracker.github.io

Which site encoding used http://code-cracker.github.io by default?
What encoding can I save files?

I checked the diagnostic files, they all seem to be using UTF8 without the byte order mark.

@giggio   Ok.


ResourceIterator flatten doesn't work anymore

Hi.
Recently I've tried to upgrade my project from kantan-csv 0.5.1 to 0.6.0 and got compile errors like:
value flatten is not a member of kantan.codecs.resource.ResourceIterator[Option[(String, Array[String])]]

We use Scala 2.12, so it looks like it doesn't converts to TraversableOnce any more.

Ah, yes, that is entirely possible. I had to break ResourceIterator out of the standard collection API because of Scala 2.13, so it still supports most collection primitives, but not all. I doubt I'll go through the trouble of making it compatible with the 2.12 and 2.13 API (nor with any collection API, given how much of a maintenance nightmare this turned out to be), so the only workaround I can suggest is to use one of the toXyz methods that let you go from ResourceIterator to a standard collection - toIterator, for example.


bridge

What version of Wallet are you using?
fuel wallet
What is happening?
I withdrew more than 1 eth from Sepolia to fuel, and the transaction was successful. However, I withdrew 1 eth from FUEL to Sepolia, and it has been 4 days and the account has not yet been paid.
steps to reproduce
https://app.fuel.network/tx/0xc36ae74ba9130d4ec37f6968a7f927a0067fddbb39c14c74ee947519085a52d0
How it should behave?

What os are you seeing the problem on?
None
What browser?
None
What is your browser version?
No response

It is taking this long currently, but working fine even switched off

The withdrawal process needs a last step which is relaying message on ethereum side. Did you do that ? @yz-ux

closing due to lack of response