Why when running under Azure Devops Pipelines does this happen

NOTE: Remove any parts of the template you do not need and replace the default text in each section with your own, including this.
Describe the bug or question
Running on Dev machines all is good. i.e. via ReSharper NUnit, or Visual Studio Test Explorer
But when running within DEvops Azure Pipelines it randomly does this:
System.InvalidOperationException : We didn't consume any input
Example:
[Test]
        public void A010_LoadATabbedPage()
        {
            var component = RenderComponent<CascadingAuthenticationState>(x => x.AddChildContent<Index>());
            // TODO: Navigate to a page !
        }

@page "/"

<AuthorizeView Roles="LRAdmin,LREngineer">
    <Authorized>
        <AdminstratorDashboard></AdminstratorDashboard>
    </Authorized>
    <NotAuthorized>
        <BSJumbotron IsFluid="true">
            <BSContainer>

Welcome to the LR Administration dashboard for FLOAT

Please sign in as an LR Administrator or LR Engineer to gain access.

            </BSContainer>
        </BSJumbotron>
    </NotAuthorized>
</AuthorizeView>

Expected behavior:
For it to load, so that eventually Navigation tests can be written
Version info:

Library version:
Latest
.NET Core/Blazor version:
Latest

Additional context:
Dev-Ops VM is "Windows-Latest" with VisStudio 16.7 (Latest)

Haven't heard of that before. Can you provide the full stacktrace and related logs.
Ps. for testing auth code, look at https://bunit.egilhansen.com/docs/test-doubles/faking-auth.html

X A010_LoadATabbedPage [646ms]

  Error Message:

   System.InvalidOperationException : We didn't consume any input.

  Stack Trace:

   at Bunit.Htmlizer.RenderChildren(HtmlRenderingContext context, ArrayRange`1 frames, Int32 position, Int32 maxElements)

   at Bunit.Htmlizer.RenderElement(HtmlRenderingContext context, ArrayRange`1 frames, Int32 position)

   at Bunit.Htmlizer.RenderCore(HtmlRenderingContext context, ArrayRange`1 frames, Int32 position)

   at Bunit.Htmlizer.RenderFrames(HtmlRenderingContext context, ArrayRange`1 frames, Int32 position, Int32 maxElements)

   at Bunit.Htmlizer.RenderChildren(HtmlRenderingContext context, ArrayRange`1 frames, Int32 position, Int32 maxElements)

   at Bunit.Htmlizer.RenderElement(HtmlRenderingContext context, ArrayRange`1 frames, Int32 position)

   at Bunit.Htmlizer.RenderCore(HtmlRenderingContext context, ArrayRange`1 frames, Int32 position)

   at Bunit.Htmlizer.RenderFrames(HtmlRenderingContext context, ArrayRange`1 frames, Int32 position, Int32 maxElements)

   at Bunit.Htmlizer.RenderCore(HtmlRenderingContext context, ArrayRange`1 frames, Int32 position)

   at Bunit.Htmlizer.RenderFrames(HtmlRenderingContext context, ArrayRange`1 frames, Int32 position, Int32 maxElements)

   at Bunit.Htmlizer.RenderChildComponent(HtmlRenderingContext context, ArrayRange`1 frames, Int32 position)

   at Bunit.Htmlizer.RenderCore(HtmlRenderingContext context, ArrayRange`1 frames, Int32 position)

   at Bunit.Htmlizer.RenderFrames(HtmlRenderingContext context, ArrayRange`1 frames, Int32 position, Int32 maxElements)

   at Bunit.Htmlizer.RenderChildComponent(HtmlRenderingContext context, ArrayRange`1 frames, Int32 position)

   at Bunit.Htmlizer.RenderCore(HtmlRenderingContext context, ArrayRange`1 frames, Int32 position)

   at Bunit.Htmlizer.RenderFrames(HtmlRenderingContext context, ArrayRange`1 frames, Int32 position, Int32 maxElements)

   at Bunit.Htmlizer.RenderCore(HtmlRenderingContext context, ArrayRange`1 frames, Int32 position)

   at Bunit.Htmlizer.RenderFrames(HtmlRenderingContext context, ArrayRange`1 frames, Int32 position, Int32 maxElements)

   at Bunit.Htmlizer.RenderChildComponent(HtmlRenderingContext context, ArrayRange`1 frames, Int32 position)

   at Bunit.Htmlizer.RenderCore(HtmlRenderingContext context, ArrayRange`1 frames, Int32 position)

   at Bunit.Htmlizer.RenderFrames(HtmlRenderingContext context, ArrayRange`1 frames, Int32 position, Int32 maxElements)

   at Bunit.Htmlizer.GetHtml(ITestRenderer renderer, Int32 componentId)

   at Bunit.Rendering.RenderedFragment.RetrieveLatestMarkupFromRenderer()

   at Bunit.Rendering.RenderedFragment..ctor(IServiceProvider services, Int32 componentId)

   at Bunit.Rendering.RenderedComponent`1..ctor(IServiceProvider services, Int32 componentId, TComponent component)

   at Bunit.TestContext.RenderComponent[TComponent](Action`1 parameterBuilder)

   at Admin.UI.Tests.BUnitTestContext.RenderComponent[TComponent](Action`1 parameterBuilder) in C:\agent\_work\2\s\Administration\Test\Admin.UI.Tests\Utils\BunitTestContext.cs:line 47

   at Admin.UI.Tests.UploadPageTests.A010_LoadATabbedPage() in C:\agent\_work\2\s\Administration\Test\Admin.UI.Tests\UploadPageTests.cs:line 33

What version of bUnit?
What's inside the Admin.UI.Tests.BUnitTestContext?
Are you rendering the authorized or not authorized portion of the test?
If authorized, what's inside AdministatorDashboard?

What version of bUnit?

I did state the latest

What's inside the Admin.UI.Tests.BUnitTestContext?

public abstract class BUnitTestContext : ITestContext
    {
        public Bunit.TestContext Context;

        public ITestRenderer Renderer => Context?.Renderer ?? throw new InvalidOperationException("NUnit has not started executing tests yet");

        public TestServiceProvider Services => Context?.Services ?? throw new InvalidOperationException("NUnit has not started executing tests yet");

        public void Dispose()
        {
            Context?.Dispose();
            Context = null;
        }

        [OneTimeSetUp]
        public void Init() => Context = new Bunit.TestContext();

        [OneTimeTearDown]
        public void Cleanup() => Dispose();

        public IRenderedComponent<TComponent> RenderComponent<TComponent>(params ComponentParameter[] parameters) where TComponent : IComponent
            => Context?.RenderComponent<TComponent>(parameters) ?? throw new InvalidOperationException("NUnit has not started executing tests yet");

        protected IRenderedComponent<TComponent> RenderComponent<TComponent>(Action<ComponentParameterBuilder<TComponent>> parameterBuilder) where TComponent : IComponent
            => Context?.RenderComponent<TComponent>(parameterBuilder) ?? throw new InvalidOperationException("NUnit has not started executing tests yet");

        protected static IHtmlInputElement FindInputElement(IRenderedComponent<CascadingAuthenticationState> cut, string findFrom = @"input")
        {
            return ((AngleSharpWrappers.Wrapper<AngleSharp.Dom.IElement>)cut.Find(findFrom)).WrappedElement as AngleSharp.Html.Dom.IHtmlInputElement;
        }

        protected async Task<FileInfo> AddFileToAsync(IRenderedComponent<CascadingAuthenticationState> cut, string fileName, string findFrom = @"input", string type = @"text/csv")
        {
            var inputLoading = FindInputElement(cut, findFrom);
            (Context.Services.GetService<IFileReaderService>() as MockIFileReaderService).SetInput(inputLoading);
            var fi = new FileInfo(fileName);
            inputLoading.Files.Add(new InputFile(fi.Name, type, fi.OpenRead(), fi.LastWriteTimeUtc));
            await inputLoading.ChangeAsync(new ChangeEventArgs());
            return fi;
        }
    }

Are you rendering the authorized or not authorized portion of the test?

As the following has been set CascadingAuthenticationState I expect the authorized portion

If authorized, what's inside AdministatorDashboard?

Sorry wrong index, here is the correct one:
@page "/data"
@using LR.Float.Common.BlazorUI.Shared.Managers
@using LR.Float.AdministrationAPI.Shared
@using LR.Float.Common.Provisioning.Proto
@using GetAssetDetailsRequest = LR.Float.AdministrationAPI.Shared.GetAssetDetailsRequest

@inject AuthMetadataManager authMetadataManager
@inject AssetsManagement.AssetsManagementClient client

<AuthorizeView Roles="LRAdmin,LREngineer">

 Data

                    <BSTabGroup>

                            <BSTabList IsPills="true" IsList="true">
                                <BSTab>
                                    <BSTabLabel>Structures</BSTabLabel>
                                    <BSTabContent>
                                        <UploadsTable FileType="FileType.AllStructures" ShowUploadButton="@ShouldShowUploadStructuresButton" UploadHref="@UploadHrefStructures" />
                                    </BSTabContent>
                                </BSTab>
                                <BSTab>
                                    <BSTabLabel>Operations</BSTabLabel>
                                    <BSTabContent>
                                        <UploadsTable FileType="FileType.AllOperations" ShowUploadButton="true" UploadHref="@UploadHrefOperations" />
                                    </BSTabContent>
                                </BSTab>
                                <BSTab>
                                    <BSTabLabel>Environment</BSTabLabel>
                                    <BSTabContent>
                                        <UploadsTable FileType="FileType.AllEnvironment" ShowUploadButton="true" UploadHref="@UploadHrefEnvironment" />
                                    </BSTabContent>
                                </BSTab>
                            </BSTabList>

                            <BSTabSelectedContent />

                    </BSTabGroup>

</AuthorizeView>

@code {
    [CascadingParameter]
    private Task<AuthenticationState> AuthenticationStateTask { get; set; }

    private long AssetId { get; set; }

    private bool ShouldShowUploadStructuresButton { get; set; }

    private string UploadHrefStructures => $"/data/uploadstructures/{AssetId}";
    private string UploadHrefOperations => $"/data/uploadoperations/{AssetId}";
    private string UploadHrefEnvironment => $"/data/uploadenvironment/{AssetId}";

    protected override async Task OnInitializedAsync()
    {
        var user = (await AuthenticationStateTask).User;
        ShouldShowUploadStructuresButton = user.IsInRole("LRAdmin") || user.IsInRole("LREngineer");

        await authMetadataManager.CallApiAsync(async metadata =>
        {
            var reply = await client.GetAssetDetailsAsync(new GetAssetDetailsRequest(), metadata);
            AssetId = reply.Asset.Id.GetValueOrDefault();
        });

    }
}

Thanks.
The first thing that jumps out at me is this:
        [OneTimeSetUp]
        public void Init() => Context = new Bunit.TestContext();

        [OneTimeTearDown]
        public void Cleanup() => Dispose();

This means you are sharing a TestContext between tests, if I understand NUnit correctly. Try changing this to:
        [SetUp]
        public void Init() => Context = new Bunit.TestContext();

        [TearDown]
        public void Cleanup() => Dispose();

My guess is that Azure Pipelines are running tests in parallel, and this is what results in the issue to show up.
There is currently no support for sharing a TestContext between tests, but it is something I am considering investigating.

Ps. I am also curious as to why you need that convoluted code to mimik uploading files. That seems way to complicated. Can you please raise another issue on that so we can figure out how to make that easier for everyone, i.e. these bits:
        protected static IHtmlInputElement FindInputElement(IRenderedComponent<CascadingAuthenticationState> cut, string findFrom = @"input")
        {
            return ((AngleSharpWrappers.Wrapper<AngleSharp.Dom.IElement>)cut.Find(findFrom)).WrappedElement as AngleSharp.Html.Dom.IHtmlInputElement;
        }

        protected async Task<FileInfo> AddFileToAsync(IRenderedComponent<CascadingAuthenticationState> cut, string fileName, string findFrom = @"input", string type = @"text/csv")
        {
            var inputLoading = FindInputElement(cut, findFrom);
            (Context.Services.GetService<IFileReaderService>() as MockIFileReaderService).SetInput(inputLoading);
            var fi = new FileInfo(fileName);
            inputLoading.Files.Add(new InputFile(fi.Name, type, fi.OpenRead(), fi.LastWriteTimeUtc));
            await inputLoading.ChangeAsync(new ChangeEventArgs());
            return fi;
        }

Thanks.

likewise for the fast responses

The first thing that jumps out at me is this:
        [OneTimeSetUp]
        public void Init() => Context = new Bunit.TestContext();

        [OneTimeTearDown]
        public void Cleanup() => Dispose();

This means you are sharing a TestContext between tests, if I understand NUnit correctly.

That is correct and is not a problem.
In this case the A010_ is the first test to run and is not actually doing any work yet.

My guess is that Azure Pipelines are running tests in parallel, and this is what results in the issue to show up.

Nope; purposely disabled that as "we" have tests that update databases, so need to be repeatable as they clean out states to have repeatable tests

There is currently no support for sharing a TestContext between tests, but it is something I am considering investigating.

Currently not a problem for all the other tests, and this is the first test, and Like I said it sometimes fails in Azure; nowhere else.

Ps. I am also curious as to why you need that convoluted code to mimik uploading files. That seems way to complicated.

This is just "setting" the upload file values, not actually initiating the upload.
That is handles by calls to a form submit, that will take each of the individual "sets" and calls wrapped functionality within Tewr.Blazor.FileReader
e.g.
        await using var stream = await fileInfo.FileSelect.OpenReadAsync();
        // Let any exception thrown be caught by the parent of all these uploads,
        // otherwise there could be a toast for every file, which is not the correct way !
        var reply = await SendPayloadAsync(formModel, fileInfo, stream, cancellationToken);

where FileSelect is from Tewr

Thanks.

likewise for the fast responses

The first thing that jumps out at me is this:
        [OneTimeSetUp]
        public void Init() => Context = new Bunit.TestContext();

        [OneTimeTearDown]
        public void Cleanup() => Dispose();

This means you are sharing a TestContext between tests, if I understand NUnit correctly.

That is correct and is not a problem.
In this case the A010_ is the first test to run and is not actually doing any work yet.

Can you please try to switch to SetUp and TearDown and see if it fixes the problem. The error you are getting indicates sharing of the renderer and concurrent renders, so that is my best guess without getting access to all your code and experimenting myself.
If the problem persists, please see if you can extract just the faulty test and related component and share it with me in a repo so I can debug the problem locally.
As for the file uploader scenario, it should not be that complex for you regardless, so I would still love an issue that explains what you have to do and why. We might be able to come up with a general purpose solution that makes this easier for everyone.

The first thing that jumps out at me is this:
        [OneTimeSetUp]
        public void Init() => Context = new Bunit.TestContext();

        [OneTimeTearDown]
        public void Cleanup() => Dispose();

This means you are sharing a TestContext between tests, if I understand NUnit correctly.

That is correct and is not a problem.
In this case the A010_ is the first test to run and is not actually doing any work yet.

Can you please try to switch to SetUp and TearDown and see if it fixes the problem. The error you are getting indicates sharing of the renderer and concurrent renders, so that is my best guess without getting access to all your code and experimenting myself.

To reiterate:
The A010_ is the first test to run within bUnit and is not actually doing any work yet.
So the context is as fresh as it can be

The A010_ is the first test to run within bUnit and is not actually doing any work yet.
So the context is as fresh as it can be

I get that, but it's still something I would like to eliminate as an option.
The second thing we need to look at is this:
protected override async Task OnInitializedAsync()
    {
        var user = (await AuthenticationStateTask).User;
        ShouldShowUploadStructuresButton = user.IsInRole("LRAdmin") || user.IsInRole("LREngineer");

        await authMetadataManager.CallApiAsync(async metadata =>
        {
            var reply = await client.GetAssetDetailsAsync(new GetAssetDetailsRequest(), metadata);
            AssetId = reply.Asset.Id.GetValueOrDefault();
        });

    }

It is not clear to me if the awaits in the OnInitializedAsync method ever completes, or if it just blocks forever (can't see if you are injecting services). Everytime an await completes, the component renders again, which should not be a problem though.

Looks like I cut too much out of the context create...
            var mockIAccessTokenProvider = new Mock<IAccessTokenProvider>();
            mockIAccessTokenProvider.Setup(m => m.RequestAccessToken())
                .ReturnsAsync(new AccessTokenResult(
                    AccessTokenResultStatus.Success,
                    new AccessToken { Value = @"hello" },
                    null));

            Context.Services.AddSingleton(mockIAccessTokenProvider.Object);

            MockNavigationManager.AddMockNavigationManager(Context.Services);

            var authContext = Context.Services.AddTestAuthorization();
            authContext.SetAuthorized(@"UnitTester").SetRoles(UserRoleNames.LR_ADMIN);

            serviceProvider = Context.Services.BuildServiceProvider();

AuthMetadata class parts:
        public AuthMetadataManager(IAccessTokenProvider accessTokenProvider,
            NavigationManager navigationManager)
        {
            this.accessTokenProvider = accessTokenProvider;
            this.navigationManager = navigationManager;
        }
        public async Task CallApiAsync(Func<Metadata, Task> action)
        {
            try
            {
                var metadata = await GetAuthMetadataAsync();
                if (metadata != null)
                {
                    await action(metadata);
                }
            }
            catch (Exception e)
            {
                ProcessUnexpectedException(e);
            }
        }
        public async Task<Metadata> GetAuthMetadataAsync()
        {
            try
            {
                var tokenResult = await accessTokenProvider.RequestAccessToken();
                if (tokenResult.Status != AccessTokenResultStatus.Success)
                {
                    navigationManager.NavigateTo($"authentication/login?returnUrl={Uri.EscapeDataString(navigationManager.Uri)}");
                    throw new DeclineApiRequestException();
                }

                if (tokenResult.TryGetToken(out var token))
                {
                    var metadata = new Metadata
                    {
                        {"Authorization", $"Bearer {token.Value}"}
                    };
                    return metadata;
                }
            }
            catch (DeclineApiRequestException)
            {
                return null;
            }
            catch (Exception e)
            {
                ProcessUnexpectedException(e);
                Console.WriteLine(e.Message);
            }
            return new Metadata();
        }

OK, so if I read your code right, all awaits should return immediately, since all the mocks has a value set before the call to RenderComponent?

Not quite, the following will make a call out to an actual pre-running service, and will return "moments later" with a default result
 var reply = await client.GetAssetDetailsAsync(new GetAssetDetailsRequest(), metadata);
 AssetId = reply.Asset.Id.GetValueOrDefault();

Hmm ok. I am thinking about a minimal example that has the same properties as your scenario, but without the complexity. Any suggestions?

Within OnInitializedAsync have a await Task.sleep equivalent and keep increasing it (i.e. upto 5 seconds) and see if it still works with one of your existing tests for Authentication mocking?

OK. Lets see if I can break down what is going on. The key is that when there is an async operation in does not complete immediately, the first render will complete, producing some output, and then, when the async operation does complete, a second render is kicked up automatically, and new output is produced.
So, if understand correctly, you have one async operation that is not completed when the OnInitializedAsync method runs:
    protected override async Task OnInitializedAsync()
    {
        // returns immediately since user is provided to the
        //TestAuthProvider during test context creation
        var user = (await AuthenticationStateTask).User;
        ShouldShowUploadStructuresButton = user.IsInRole("LRAdmin") || user.IsInRole("LREngineer");

        // this call is awaited, causing the renderer to render when the call is awaited
        // and again when the async operation completes
        await authMetadataManager.CallApiAsync(async metadata =>
        {
            var reply = await client.GetAssetDetailsAsync(new GetAssetDetailsRequest(), metadata);
            AssetId = reply.Asset.Id.GetValueOrDefault();
        });
    }

Is this correct? If so, how long does it take before authMetadataManager.CallApiAsync completes, and does it do its work on a different thread?

If the "first render completes"
Then why the
System.InvalidOperationException : We didn't consume any input.
It always seems to complete when running in Visual Studio (Debug or Release)
It "Mostly" completes on Dev-Ops
So the theory you have put forward is that if the await inside the Async initialise "Takes some time" then the exception is thrown.
you have the code for CallApiAsync above, and it calls the "moq"ed accessTokenProvider.RequestAccessToken(); as shown.
The only thing that isn't "moq"ed is the client.GetAssetDetailsAsync which might takes a variable amount of time (But it is on the same machine) so is expected to be quick (i.e. the test normally completes in 200ms)

So the theory you have put forward is that if the await inside the Async initialise "Takes some time" then the exception is thrown.

Right. It only has to take 1ms for the renderer to continue. This is the timeline I think we are seeing:

RenderComponent is called (FIRST RENDER)
OnInitiazliedAsync is called
authMetadataManager.CallApiAsync returns an incompleted task, because client.GetAssetDetailsAsync returns an incomplete task, which is awaited.
The TestRenderer finishes rendering returning to RenderComponent.
RenderComponent creates an instance of the RenderedComponent type
RenderedComponent calls GetHtml which asks the TestRenderer for the current render tree (RetrieveLatestMarkupFromRenderer call in stacktrace)
While GetHtml is walking through the render tree from the first render, the authMetadataManager.CallApiAsync callback completes, and the renderer starts a SECOND RENDER on another thread (due to async/await).
The second render modifies the render tree that GetHtml is currently iterating over, and the change to the render tree causes to exception "We didn't consume any input".

This scenario is related to the one described here: https://bunit.egilhansen.com/docs/interaction/awaiting-async-state
Can you describe what changes happens between the first and second render in the rendered output? E.g. are you adding or removing elements.

Rendering will be setting the UploadHref
As the AssetId is used as a URL redirect when set

Just to confuse even more (Maybe) here is the UploadsTable file
@using LR.Float.Common.Provisioning.Proto
@using LR.Float.Common.BlazorUI.Shared.Managers
@using LR.Float.AdministrationAPI.Shared
@using LR.Float.Common.BlazorUI.Shared.Components.Download @*Do not remove as it is need for the reference below*@
@using LR.Float.AdministrationAPI.AdminWebUI.Services
@inject AuthMetadataManager AuthMetadataManager
@inject ProvisioningAdminSvcApis.ProvisioningAdminSvcApisClient PClient

@if (ShowUploadButton)
{

             Upload

}

<BSDataTable IsHoverable="true" IsResponsive="true" IsStriped="true" Items="tableFiles" Context="file" >
    <HeaderTemplate>

            Uploaded
            Filename
            Data type
            Status

    </HeaderTemplate>
    <LoadingTemplate>
        Loading ...
    </LoadingTemplate>
    <NoDataTemplate>
        No Data Available
    </NoDataTemplate>
    <ItemTemplate>
        @file.UploadUtc?.ToDateTime().ToLocalTime()
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>@file.OptionalMessage

            <DownloadSelect FileDetails="@file" PClient="@fileDownloader"/>

    </ItemTemplate>
</BSDataTable>

    <PagedListPager Items="@pagedFiles" OnPageNumberChangedCallback="@OnPageNumberChanged"></PagedListPager>

@code {
    private const uint DEFAULT_PAGE_SIZE = 16;  // Make it fit on a single browser view ??

    [Parameter]
    public FileType FileType { get; set; }

    [Parameter]
    public bool ShowUploadButton { get; set; }

    [Parameter]
    public string UploadHref { get; set; }

    public uint PageNumber { get; set; } = 1;
    public uint PageSize { get; } = DEFAULT_PAGE_SIZE;

    private List<FileDetails> tableFiles => Files?.ToList();
    private IPagedList<FileDetails> pagedFiles => Files ?? new StaticPagedList<FileDetails>(new List<FileDetails>(), 1, (int)DEFAULT_PAGE_SIZE, 0);

    private FileDownloader fileDownloader;

    public IPagedList<FileDetails> Files;

    protected override async Task OnInitializedAsync()
    {
        fileDownloader = new FileDownloader(PClient);
        await LoadData();
    }

    private async Task LoadData()
    {
        await AuthMetadataManager.CallApiAsync(async metadata =>
        {
            var request = new GetHistoryOfFilesRequest { OptionalPageSize = PageSize, OptionalStartNumber = PageNumber };
            request.FileTypes.Add(FileType);
            var reply = await PClient.GetHistoryOfFilesAsync(request, metadata);
            var fileDetails = reply.FileDetails.ToList();
            Files = new StaticPagedList<FileDetails>(fileDetails, (int)reply.PageNumber, (int)reply.PageSize, (int)reply.TotalItemCount);
        });
        StateHasChanged();
    }

    private async Task OnPageNumberChanged(uint pageNumber)
    {
        Files = null;   // Allow the "Loading..." message
        StateHasChanged();
        PageNumber = pageNumber;
        await LoadData();
    }

}

OK, hmm, I think I have a very good idea of what is going on. Now I need to find the time to fix it :)
Thanks for the reporting this.

Just a bit more info.. I have just enabled a demo version of DotCover in the ReSharper license I have, and it is causing the
System.InvalidOperationException : We didn't consume any input. mentioned in the OP.
The Azure pipeline also has coverage turned on via coverlet.console.
So it appears that the title should be changed to "When running with Coverage ......."
HTH

Just a bit more info.. I have just enabled a demo version of DotCover in the ReSharper license I have, and it is causing the
System.InvalidOperationException : We didn't consume any input. mentioned in the OP.
The Azure pipeline also has coverage turned on via coverlet.console.
So it appears that the title should be changed to "When running with Coverage ......."
HTH

Does it do this consistently with dotCover or is it only once in a while like with azure pipelines?
Either way, interesting that the instrumentation coverage tools do causes this race condition to surface.

Hey @Smurf-IV
I think the issue is fixed in the latest nightly. That said, I have not been able to reproduce the problem. So I wonder if you would be so kind to try out the latest bits available here and report back: https://github.com/egil/bUnit/packages

FYI: I'll get back to it next week, I'm currently "away from work"

Ok, I'll close this for now then and we can open it once more if you discover that the big is still there.

FYI: 1.0.0-beta-9 still has this issue when running under Debug - ReSharper and NUnit (No perf or coverage involved)

@Smurf-IV did you try the prerelease of beta-10 available on the GitHub Package Repository?
I did not update beta-9, so I expect it to still have the issue.

Adding the following to my csProj
<PackageReference Include="bunit.web" Version="1.0.0-beta-10-g9ed7f33174" />
does not retrieve the beta version as suggested by your link, I get this:
Severity	Code	Description	Project	File	Line	Suppression State Error	NU1603	Admin.UI.Tests depends on bunit.web (>= 1.0.0-beta-10-g9ed7f33174) but bunit.web 1.0.0-beta-10-g9ed7f33174 was not found. An approximate best match of bunit.web 1.0.0-beta-7 was resolved.	Admin.UI.Tests	blah\blah\Administration\Test\Admin.UI.Tests\Admin.UI.Tests.csproj	1

You have to do a few setup steps before dotnet restore can pull from GPR:
https://docs.github.com/en/packages/using-github-packages-with-your-projects-ecosystem/configuring-dotnet-cli-for-use-with-github-packages#installing-a-package

If I remember correctly, you have to:

Go into your github settings under security and generate a new access token (https://github.com/settings/tokens). The token should have read:packages rights.
Create a nuget.config file and put your project the following content:

<?xml version="1.0" encoding="utf-8"?>
<configuration>
    <packageSources>
        <clear />
        <add key="github" value="https://nuget.pkg.github.com/egil/index.json" />
    </packageSources>
    <packageSourceCredentials>
        <github>
            <add key="Username" value="USERNAME" />
            <add key="ClearTextPassword" value="TOKEN" />
        </github>
    </packageSourceCredentials>
</configuration>

do a dotnet restore

Thanks for the above.. Then I get this:
C:\Program Files\dotnet\sdk\5.0.100-preview.8.20417.9\NuGet.targets(128,5): error : Failed to retrieve information about 'Microsoft.TestPlatform.TestHost' from remote source 'https://github.com/egil/bUnit/Packages/FindPackagesById()?id='Microsoft.TestPlatform.TestHost'&semVerLevel=2.0.0'. [blah\blah\Administration\Test\Admin.UI.Tests\Admin.UI.Tests.csproj] C:\Program Files\dotnet\sdk\5.0.100-preview.8.20417.9\NuGet.targets(128,5): error :   Response status code does not indicate success: 404 (Not Found). [blah\blah\Administration\Test\Admin.UI.Tests\Admin.UI.Tests.csproj]

Ahh, we removed nuget.org, which has that package. try this:
<?xml version="1.0" encoding="utf-8"?>
<configuration>
    <packageSources>
        <add key="nuget.org" value="https://api.nuget.org/v3/index.json" protocolVersion="3" />
        <add key="github" value="https://nuget.pkg.github.com/egil/index.json" />
    </packageSources>
    <packageSourceCredentials>
        <github>
            <add key="Username" value="USERNAME" />
            <add key="ClearTextPassword" value="TOKEN" />
        </github>
    </packageSourceCredentials>
</configuration>

I've got that, otherwise you get every package as an error ;-)

So you were able to restore now?

So you were able to restore now?
Sadly no.. See edited above..

Thanks for sticking with me with this.. It must feel annoying for you.

 I will investigate.

OK, try just referencing both explicitly:
<PackageReference Include="bunit.core" Version="1.0.0-beta-10-g9ed7f33174" />
<PackageReference Include="bunit.web" Version="1.0.0-beta-10-g9ed7f33174" />

I just tried locally with a new nunit test project (dotnet new nunit -o BunitNightly)
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <TargetFramework>net5.0</TargetFramework>

    <IsPackable>false</IsPackable>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="NUnit" Version="3.12.0" />
    <PackageReference Include="NUnit3TestAdapter" Version="3.16.1" />
    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="16.5.0"/>
    <PackageReference Include="bunit.core" Version="1.0.0-beta-10-g9ed7f33174" />
    <PackageReference Include="bunit.web" Version="1.0.0-beta-10-g9ed7f33174" />
  </ItemGroup>

</Project>

Thanks for sticking with me with this.. It must feel annoying for you.

Thank you for sticking with me on this as well. I want bUnit to work correctly for this, so thanks for testing and reporting the issue.

Btw. when I have to try to get issues like this to surface, I run the tests in a with this little powershell script:
$maxRuns = $args[0]
$mode = $args[1]
$filter = $args[2]

dotnet build ..\bunit.sln -c $mode --nologo

for ($num = 1 ; $num -le $maxRuns ; $num++)
{
    Write-Output "### TEST RUN $num ###"

	if($filter)
	{
		dotnet test ..\bunit.sln -c $mode --no-restore --no-build --blame-hang --blame-hang-timeout 100s -f net5.0 --nologo --filter $filter
	}
	else
	{
		dotnet test ..\bunit.sln -c $mode --no-restore --no-build --blame-hang --blame-hang-timeout 100s -f net5.0 --nologo
	}
}

usage:

runtests.ps1 100 debug runs the tests 100 times in debug mode
runtests.ps1 100 release runs the tests 100 times in release mode

The 3rd argument, filter, uses the syntax specified here: https://docs.microsoft.com/en-us/dotnet/core/testing/selective-unit-tests?pivots=nunit, e.g.:

runtests.ps1 100 release MyTestMethod runs the tests that has the MyTestMethod in their FullyQualifiedName 100 times in release mode

Yay - "This" test now passes with ReSharper coverage enabled in DevStudio..
Now onto the next problem


Navigator and UITabbarController has some conflict on iOS.

react-native 0.43.4:
node 4.5.0:
npm 3.10.7:

when i create a UIWindow with rootController is UITabbarController, and the UITabbarController contain some UINavigationConller . then , i push a new vc contain RCTRootView with full screen , native UINavigationBar is hiden , and hidesBottomBarWhenPushed is YES . final appear RCTView bounds is not correct .
this is layer preview
image
this is my some code
`
UINavigationController = UINavigationController.rootVC(new vc);
UITabbarController = new UITabbarController(@[@UINavigationController,@xx])
window.rootViewController = UITabbarController
RNVC = UIViewController.addSubView(RCTRootView)
RNVC.navigationBarHidden=yes
`
maybe don't make me clear. sorry

This issue looks like a question that would be best asked on StackOverflow.
StackOverflow is amazing for Q&A: it has a reputation system, voting, the ability to mark a question as answered. Because of the reputation system it is likely the community will see and answer your question there. This also helps us use the GitHub bug tracker for bugs only.
Will close this as this is really a question that should be asked on StackOverflow.


Use precision argument on cmd

Precision gets default to 0 when using as cmd argument.
I guess it's related to https://github.com/pytest-dev/pytest-cov/issues/403

The test doesn't seem a very good one... As it passes without my modifications.

Thanks.  Can you show the behavior you are fixing?  It looks like it works to me:
$ coverage report --precision=1
Name      Stmts   Miss    Cover
-------------------------------
igor.py     274    233    15.0%
-------------------------------
TOTAL       274    233    15.0%

$ coverage report --precision=4
Name      Stmts   Miss       Cover
----------------------------------
igor.py     274    233    14.9635%
----------------------------------
TOTAL       274    233    14.9635%

I guess on your example, would be:
$ coverage report --fail-under=15.0 --precision=4
Name      Stmts   Miss       Cover
----------------------------------
igor.py     274    233    14.9635%
----------------------------------
TOTAL       274    233    14.9635%

It's the total coverage that gets round using precision=0.

I see what you mean about the test. I'll fix up the test after merging this. Thanks.

Thank you. :)

This is now released as part of coverage 6.3.1.


[Proposal] Add functions remove_empty_fields remove_null_fields

Proposal for adding built-in functions remove_empty_fields and remove_null_fields which would be useful to remove fields of an object that are empty or have null value.
e.g. if we have a json like this
{'a':1, 'b':'', 'c': null}

we call
remove_empty_fields(@)

Result
{'a':1}

if we call
remove_null_fields(@)

Result
{'a':1, 'b':''}

Hi, any update on this?

Discarding null values is a surprisingly frequently requested feature from JMESPath.
The canonical answer is to use the merge() function and let external deserializers discard / irgnore those values.
That said, we are considering adding this to JMESPath Community.
Thanks to the new items(), from_items() and zip() functions, however, you can do this today:
Given:
{ "a": 1, "b": "", "c": null }

The following expressions can be useful:

from_items( items(@) [? @[1]!=`null` ]  ) returns `` { "a": 1, "b": "" }
from_items( items(@) [? @[1] ]  ) returns `` { "a": 1 }

@deenie1 i think you inadvertently uploaded exported files.


Test Case ID: TC_004 (Test Case 4: Combine Filters (Name + Location))

Title: Search by name and filter by location.

Precondition: The user is on the Rick & Morty Directory application homepage.

Test Steps:

Enter a first name (e.g., "Morty") in the search bar.
Select a location filter (e.g., "Citadel of Ricks").
Click the search button.

Expected Result: The list shows all characters named "Morty" who are from the "Citadel of Ricks."

Actual Result: the characters matching both the name and location filters were displayed as expected.

The manual test for searching by name and filtering by location was conducted successfully. Both filters worked as expected, and the relevant characters were displayed accurately.
Environment:
Device: MacBook Air (Intel)
Browser: Chrome Version 128.0.6613.138 (Official Build) (x86_64)
Operating System: macOS 12.4
Rick & Morty Directory URL: https://d28dp6fycxce58.cloudfront.net


filter out repos without mit, apache-2.0, or 0bsd

let's just skip those repos in separate filter step called license_filter. It should go first, before filter.

@rultor release, tag is 0.0.15

@rultor release, tag is 0.0.15

@h1alexbel OK, I will release it now. Please check the progress here.


Added the scorecard github action and its badge

Which issue(s) this PR fixes:

Issue #8990
What this PR does / why we need it:
PR to add the Scorecard GitHub Action and its badge in the README file.
Special notes for your reviewer:
None
Additional documentation or context
None

Thanks @harshitasao

The scorecard is only added to longhorn/longhorn.
Do we need to add it to all repos? @harshitasao @innobead


Bad rendering of Radio button with readonly attribute

Bug Report
Steps
Visit https://react.semantic-ui.com/addons/radio/#types-radio . Review the first example titled, "Radio. A radio for checking.". Click to check the radio button. Clicking again does not toggle the radio off.
Expected Result
Be able to toggle the radio button on and off.
Actual Result
It will not toggle off.  The html is rendered as
<input type='radio' class='hidden' readonly='" tabindex='0" />.

Version
0.82.5
Testcase
[Fork, update, and replace this pen to show the bug]:
https://codesandbox.io/s/pypo05yqnq

It's not possible to toggle radio, native input has same behaviour:
https://developer.mozilla.org/en-US/docs/Web/HTML/Element/Input/radio
https://codepen.io/anon/pen/YObxZQ


pull request for e2e test at 12/26/2016 21:26:11

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


_controller.back for dislike button not working

When configuring the bottom button to have a direction of left when uses dislikes does not animate and does not work when clicking the button using the code _controller.back() . The animation and code for _controller.forward() does work when it is configured to the like button

When configuring the bottom button to have a direction of left when uses dislikes does not animate and does not work when clicking the button using the code _controller.back() . The animation and code for _controller.forward() does work when it is configured to the like button

Updated, please install new version 🥺


aws/asg-defaults-1: add support for price-capacity-optimized

This new strategy strikes a balance between cost and interruption risk.
https://aws.amazon.com/blogs/compute/introducing-price-capacity-optimized-allocation-strategy-for-ec2-spot-instances/

/retest

/retest

Will open a PR in qontract-reconcile first, as we need to be able to specify spot_instance_pools when updating the strategy.
Error updating Auto Scaling Group: ValidationError: SpotInstancePools option is only available with the lowest-price allocation strategy.

Terraform seems to set this one by default, and we need to be able to unset it (to 0).

Looking at qontract-reconcile, @ondrejbudai remarked that all instances_distribution values are just copied over in bulk, including spot_instance_pools. So changing the strategy up should work as is.


Set peer dependency for hedera

📦 Pull Request
[Provide a general summary of the pull request here.]
https://app.shortcut.com/magic-labs/story/77214/blockchain-update-hedera-sdk-package
We need to set peerDependency for hedera so that we can make sure a developer uses over 2.31.0 version
✅ Fixed Issues

[List any fixed issues here like: Fixes #XXXX]

🚨 Test instructions
[Describe any additional context required to test the PR/feature/bug fix.]
⚠ Don't forget to add a semver label!
Please only add one label:

patch: Bug Fix?
minor: New Feature?
major: Breaking Change?
skip-release: It's unnecessary to publish this change.

📦 Published PR as canary version: Canary Versions

:sparkles: Test out this PR locally via:
npm install<EMAIL_ADDRESS># or
yarn add<EMAIL_ADDRESS>

:rocket: PR was released in<EMAIL_ADDRESS>:rocket:


Auditor always includes the last copy of the report, even if it doesn't correspond to the latest run

Via supervisor, auditor includes credentials.REPORT_BASE_PATH as an attachment in the email. However, this file won't be updated if there is an error that causes the checks/fixes to bomb out before they're finished. Thus, an error email will have an out-of-date report that causes confusion when troubleshooting.
One solution is to atomize the report writing so that auditor writes to the report for every item instead of after everything is finished (see TODO before def log_report()).

One cause of this may be when the AGOL connection times out before completing all the fixes, and when it goes to write the log report dictionary it doesn't have all the expected keys so it fails:
#: ...

Exception: Invalid token.
(Error Code: 498)

During handling of the above exception, another exception occurred:

#: ...

  File "c:\dev\auditor\src\auditor\auditor.py", line 79, in log_report
    item_list.extend([str(item_report_dict[col]) for col in columns])

  File "c:\dev\auditor\src\auditor\auditor.py", line 79, in <listcomp>
    item_list.extend([str(item_report_dict[col]) for col in columns])

KeyError: 'metadata_result'

We could either pre-populate the keys with empty data or make log_report() more robust.


Podcast list doesn't load

podcasts-export.txt
Launching the app using the instructions worked fine but no list of podcasts ever appears, even after hitting the refresh button a bit. I also tried the Python script, and installed the pip modules there, thinking that might help, but the script did not work, and the pip modules did not help. I am enclosing what might be relevant lines from the logs, though there are no outright errors or faults. I saw the one closed issue and also tried the solution there, even though it was not really relevant, but it also did not help. Running macOS 13.1 on an M1 processor.

Hi,
Thanks for the bug report and the logs.
I'm still procrastinating on my update to OS 13, so I unfortunately can't reproduce the issue yet. I'll check back when I get around to launching the update!


822 Act & Warden Precompiles

Summary by CodeRabbit

New Features

Enhanced installation and deployment processes for local blockchain environments to utilize unstable versions.
Introduced a new interface for managing keys and keychains within the Warden protocol, including various functions for administration and request handling.
Added comprehensive event handling and logging capabilities for actions within the Warden protocol.
Established a robust framework for managing Ethereum-related events and interactions.
Implemented a new interface for the x/act module, facilitating action and template management.
Introduced new query and transaction methods for managing actions, templates, and keychains.
Comprehensive test suites added for both Warden and Act precompile functionalities, validating interactions and state changes.
Updated changelog to reflect new features, bug fixes, and enhancements across modules.

Bug Fixes

Improved handling of gas consumption and error management in transaction processing.
Corrected typographical errors in method calls across various test cases to ensure proper execution.

Documentation

Updated protocol buffers to include new RPC methods for keychain administration.

Closing this one as here too many duplicated comments


Update steganopy.py

Merge changes on experimental into the main branch

A bug also existed in the pypi release when decoding information.


[ign-sensors3] Workflow updates

Add labeler workflow
Add triage workflow
Add Ubuntu CI workflow with Codecov upload
Update badges
Port CODEOWNERS

Codecov Report

:exclamation: No coverage uploaded for pull request base (ign-sensors3@bf6cae8). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff               @@
##             ign-sensors3      #21   +/-   ##
===============================================
  Coverage                ?   59.63%
===============================================
  Files                   ?       16
  Lines                   ?      976
  Branches                ?        0
===============================================
  Hits                    ?      582
  Misses                  ?      394
  Partials                ?        0

Continue to review full report at Codecov.

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


feat: new Azure Traffic Manager Module

Description
Closes #245
If you haven't already, read the full contribution guide. The guide may have changed since the last time you read it, so please double-check.
Enable GitHub Worksflows in your fork to enable auto-generation of assets with our GitHub Action.
To trigger GitHub Actions after auto-generation, add a GitHub PAT as a secret in your forked repository called PAT.
Adding a new module

[x] A proposal has been submitted and approved.
[x] I have included "Closes #{module_proposal_issue_number}" in the PR description.
[x] I have run brm validate locally to verify the module files.
[x] I have run deployment tests locally to ensure the module is deployable.

Closes #245

@shenglol  also ready for review

/azp run

/azp run


tirocks-sys: update titan

wrap titan into a separate cargo crate

I would rather not. Instead, we can make it an optional feature.

I would rather not. Instead, we can make it an optional feature.

So crates.io will cache package with different features?

So crates.io will cache package with different features?

No, all sources will be published to crates.io. And build script can build part of sources according to features.


feat: avm/res/desktop-virtualization/scaling-plan

Description
Closes avm/res/desktop-virtualization/scaling-plan #145
Adding a new module

[x] A proposal has been submitted and approved.
[x] I have included "Closes #{module_proposal_issue_number}" in the PR description.
[ ] I have run brm validate locally to verify the module files.
[x] I have run deployment tests locally to ensure the module is deployable.

Don't forget to add a workflow status badge, more details see here.


 I just really wanted to make a custom bot for my discord server and it seems to have blown up in my face. Like many others, I went through that Davao JS walkthrough to make my bot, double and triple checked everything, but this is all Command Prompt is giving me. A quick google suggested running as admin, so I tried that too and no difference.

I am getting the same error


[BUG] ApacheHttpClient5Transport hangs when content is too large

What is the bug?
When using java client with ApacheHttpClient5Transport, large requests that exceeds AWS network limit (e.g. 10MB on m6g.large.search) will cause the request to hang and permanently blocks the thread.
In actuality, AWS responds with 400 (or 413 if the content-length header is present in the request, which is missing in this case, but that's another issue).
This bug only affects ApacheHttpClient5Transport, but works correctly on the legacy RestClientTransport (i.e. throws an exception). It also works with curl (i.e. returns a 400 or 413 response).
How can one reproduce the bug?
Steps to reproduce the behavior.

Create an AWS OpenSearch Service domain, with data nodes of an instance-type with 10MB network quota, e.g. m6g.large.search
Makes a bulk request >10MB, with the following junit test:

public class BulkLimitTest {
    private static final String OPENSEARCH_URL = "https:/my-domain.ap-southeast-2.es.amazonaws.com";
    private static final String INDEX = "sheep-index";
    private OpenSearchClient client;

    @Before
    public void setup() throws IOException, URISyntaxException {
        ConnectionConfig connectionConfig = ConnectionConfig.custom()
                .setConnectTimeout(5L, TimeUnit.SECONDS)
                .setSocketTimeout(5, TimeUnit.SECONDS)
                .build();
        PoolingAsyncClientConnectionManager connectionManager = PoolingAsyncClientConnectionManagerBuilder
                .create()
                .setDefaultConnectionConfig(connectionConfig)
                .build();
        ApacheHttpClient5Transport transport = ApacheHttpClient5TransportBuilder.builder(HttpHost.create(OPENSEARCH_URL))
                .setHttpClientConfigCallback(builder -> builder.setConnectionManager(connectionManager))
                .build();

        client = new OpenSearchClient(transport);
        client.indices().delete(d -> d.index(INDEX).ignoreUnavailable(true));
        client.indices().create(r -> r.index(INDEX));
    }

    @Test
    public void testBulkLimit() throws IOException {
//        final int SIZE_MB = 8; // ---> this works
        final int SIZE_MB = 10; // ---> this hangs
        BulkRequest request = generateRequest(SIZE_MB * 1024 * 1024);
        client.bulk(request);
    }

    private BulkRequest generateRequest(int size) {
        BulkRequest.Builder builder = new BulkRequest.Builder();
        final String SAMPLE = """
            Lorem ipsum dolor sit amet, consectetur adipiscing elit. Sed efficitur in metus quis lacinia.
            Nullam vel blandit lacus. Nam ornare purus nibh, et varius nunc finibus non.
        """;

        int unitSize = SAMPLE.length();
        for (int i=0, s=0; s < size; i++, s+=unitSize) {
            String id = "sheep-" + i;
            builder.operations(o -> o.index(d -> d
                    .index(INDEX)
                    .id(id)
                    .document(Map.of("content", SAMPLE))));
        }
        return builder.build();
    }
}

What is the expected behavior?
Throws OpenSearchException or ResponseException with 400 or 413 error.
What is the actual behavior?
The thread hangs forever. No exceptions, no timeouts.
What is your host/environment?

Tested on:

Mac OS (M1)
EC2 Amazon Linux

AWS OpenSearch Service

engine version: 2.11
data-nodes: 3 x m6g.large.search

opensearch-java client version: 2.10.1

Do you have any screenshots?
No. See the junit code above.
Do you have any additional context?
Only affects ApacheHttpClient5Transport.
Works correctly with RestClientTransport, curl, postman, etc, i.e. produces 400 or 413 errors.

Looks like a bug :( Do you think you can try to turn your repro into a (failing) unit test with a server mock response?

The issue seems to be really related to quota setting (it is not reproducible locally at least):

Create an AWS OpenSearch Service domain, with data nodes of an instance-type with 10MB network quota, e.g. m6g.large.search

@dblock I may need your help here, wondering if we could have an AWS OpenSearch Service for a few hours to troubleshot the issue?

@reta I can help offline, yes.

Do you think you can try to turn your repro into a (failing) unit test with a server mock response?

Unfortunately it's not easily replicable with a server mock that returns a specific error code. The bug only occurs with the specific low level network behavior of AWS when we reach the network quota, in which case we'll get a specific sequence of events that's different from normal, e.g. it closes the request stream (preventing it from sending more data) before writing the status to the response.
So I think the best way would be to test this bug is with actual AWS.

Force-closing the request stream in an intercepted call is usually a solution, but I am talking without trying.

@hendryluk @dblock interesting, the issue is related to HTTP protocol chosen:

RestClientTransport for 2.x is always using HTTP/1.1, (HC4), but
ApacheHttpClient5Transport talks HTTP/2.0 and there is no response callback (looking why now)

Setting HttpVersionPolicy for the client fixes the issue, but I am not sure where exactly the problem is: OS 2.x does not support HTTP/2.0, why protocol is negotiated this way? gateway or LB in front?
ApacheHttpClient5TransportBuilder.builder(HttpHost.create(OPENSEARCH_URL))
     .setHttpClientConfigCallback(builder -> builder.setVersionPolicy(HttpVersionPolicy.FORCE_HTTP_1))

Thanks @reta for looking into the issue and for the workaround - we'll give that a try.


check for git lock

Description
I keep getting erros like this more and more often. Running the command again fixes.

I have a feeling this is happening because git is busy when I commit, which could be due to vscode using git.
Steps to reproduce
Should happen if you can start a lint-staged process at the same time git is busy doing something else.
Debug Logs

expand to view
COPY THE DEBUG LOGS HERE

Environment

OS: mac os mojave
Node.js: 13.12.0
lint-staged: 10.1.3

this could be addressed by checking if $PWD/.git/index.lock exists and sleeping a bit to see if git finishes being used. I could be wrong tho so leaving issue issue to get some ideas/feedback

Please post your config and debug logs, so that we can speculate what might be causing this.
I’m not aware of any race conditions with git locking caused by lint-staged itself (last was recently fixed), but another process running at the same time might affect it.

I see this a bunch too on Windows 10. I've resorted to shutting down webpack and possibly Atom whenever I run lint-staged because of these conflicts.

@kaiyoma Please post your config and debug logs.

Is there a need for lint-staged to be doing a full backup of the entire index?
Could it not create and store a stash with the matched files set added in 10.1.3 to make sure you are only stashing the files that a task will run on?
It feels like this would be safest and is the solution I was going to suggest to a different issue.
Maybe I should create the separate issue anyway as it is entirely unrelated in it's cause (possibly)

@nokternol we could use the matched file set with git stash create and  git stash push, but this is something I haven’t experimented with yet.
Thanks for the suggestion!

raised #844 to track this separately

Does the actual error in this scenario never get surfaced?
The OP picture shows the lint-staged output but right after this logger statement it throws the error..
This does not appear in the image and wondered whether it is ever surfaced...
Should the logger also write out the actual git error to the out also?
Have to say, I have not seen this error personally. If git is busy (which is the error being fixed in 10.1.3 from the git add) I was seeing the actual error from git stating that the git.lock file was there.
Without an actual error being displayed it is difficult to know exactly what is happening here

The code falls through and the cause is not 100% going to be related to what lint-staged is printing.
if (stash)... is just to protect against printing this message if the stash was not created.
But it could still be misleading to the cause.
It is just a message saying that you can recover using the stash.
There is no debug code in this area so I'm not sure how you get the error if not displayed by throw..

I refer back to @iiroj's original comment
Please post your config and debug logs, so that we can speculate what might be causing this.

@nokternol @iiroj updated OP with screenshot of index.lock error, sometime I get it sometimes no error, and my debug ouput. Config seems to be included in debug output. I'll run lint-staged in debug all the time, and when I get a fail I'll paste logs in this issue. 👍

This is exactly the same error that 10.1.3 fixed.
I wonder whether this is actually an npm cache issue...
Have you recently upgraded to 10.1.3?
Have you tried deleting node_modules and re-installing or maybe even running npm cache verify.
Just smells far too much like the fixed issue where git add was being run concurrently causing a race condition with the lock.
You have removed the git add from your package.json configuration as well as this is handled internally now

The reason it works the second time is that the files have already been modified so the git add is empty and doesn't hit the lock.
I honestly struggle to believe this is a code issue still existing in 10.1.3 but I'll leave it to the contributors to handle this as I am just the person who raised the lock issue that was fixed in 10.1.3 (same error)

Have you recently upgraded to 10.1.3
possibly, I upgrade deps on this project every few days
npm cache verify:

npm cache verify
Cache verified and compressed (~/.npm/_cacache):
Content verified: 15237<PHONE_NUMBER> bytes)
Content garbage-collected: 8771 (534168728 bytes)
Missing content: 3
Index entries: 26227
Finished in 25.653s

I use yarn though. I'll run in debug to see if I catch a good log, if not then I may re-clone the project to see if it fixes.

obtained debug logs of the lock problem and added to OP

https://github.com/okonet/lint-staged/pull/850 could help here, since files no longer disappear from the disk and file watchers etc shouldn't trigger.

Sweet. I update deps regularly so I'll pick it up eventually when it's released. I could install from a branch too if that helps.

@Thomazella it was just released, actually, in v10.1.7.

Since updating haven't run into the problem 🚀 closing, will reopen if needed 🙏


Optuna_implementation

Hi @j4nberner please create a bit more verbose PRs. In addition, please ensure to not check in files on git that are not needed.
Otherwise, nice progress ⭐!


[tool_crash] ProcessException: The requested operation requires elevation. Command: "C:\Program Files\Google\Chrome\Application\chrome.exe", OS error code: 740

Hello,
I've created a Flutter project. It works if I run it on Android. But on Chrome it does not start (same on Edge), see details below.
I'm using VSCode (in Admin mode).
If I create a new Flutter project (the default one), it works.
Any suggestions?
Command
flutter run -d Chrome --web-port 5000 --release

Steps to Reproduce

...
...
...

Logs
ProcessException: The requested operation requires elevation.
Command: "C:\Program Files\Google\Chrome\Application\chrome.exe", OS error code: 740
#0      _ProcessImpl._start (dart:io-patch/process_patch.dart:390:33)
#1      Process.start (dart:io-patch/process_patch.dart:36:20)
#2      LocalProcessManager.start (package:process/src/interface/local_process_manager.dart:39:20)
#3      ErrorHandlingProcessManager.start.<anonymous closure> (package:flutter_tools/src/base/error_handling_io.dart:638:33)
#4      _run (package:flutter_tools/src/base/error_handling_io.dart:532:20)
#5      ErrorHandlingProcessManager.start (package:flutter_tools/src/base/error_handling_io.dart:638:12)
#6      ChromiumLauncher._spawnChromiumProcess (package:flutter_tools/src/web/chrome.dart:247:53)
#7      ChromiumLauncher.launch (package:flutter_tools/src/web/chrome.dart:225:35)
<asynchronous suspension>
#8      ChromiumDevice.startApp (package:flutter_tools/src/web/web_device.dart:133:17)
<asynchronous suspension>
#9      _ResidentWebRunner.run.<anonymous closure> (package:flutter_tools/src/isolated/resident_web_runner.dart:521:9)
<asynchronous suspension>
#10     asyncGuard.<anonymous closure> (package:flutter_tools/src/base/async_guard.dart:111:24)
<asynchronous suspension>

[✓] Flutter (Channel beta, 2.0.1, on Microsoft Windows [Version 10.0.19042.867], locale en-NL)
    • Flutter version 2.0.1 at C:\src\flutter
    • Framework revision c5a4b4029c (11 days ago), 2021-03-04 09:47:48 -0800
    • Engine revision 40441def69
    • Dart version 2.12.0

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
    • Android SDK at C:\Users\jeroe\AppData\Local\Android\sdk
    • Platform android-30, build-tools 30.0.3
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✓] Android Studio (version 4.1.0)
    • Android Studio at C:\Program Files\Android\Android Studio
    • 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-b01)

[✓] Connected device (2 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 89.0.4389.82
    • Edge (web)   • edge   • web-javascript • Microsoft Edge 89.0.774.50

• No issues found!

Flutter Application Metadata
Type: app
Version: 1.0.0+1
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: beta
Creation framework version: 044f2cf5607a26f8818dab0f766400e85c52bdff
Plugins
cloud_firestore-1.0.1
cloud_firestore_web-1.0.1
firebase_analytics-7.1.1
firebase_analytics_web-0.2.0+1
firebase_auth-1.0.1
firebase_auth_web-1.0.2
firebase_core-1.0.1
firebase_core_web-1.0.1
google_sign_in-5.0.0
google_sign_in_web-0.10.0
path_provider-2.0.1
path_provider_linux-2.0.0
path_provider_macos-2.0.0
path_provider_windows-2.0.0
shared_preferences-2.0.4
shared_preferences_linux-2.0.0
shared_preferences_macos-2.0.0
shared_preferences_web-2.0.0
shared_preferences_windows-2.0.0
sqflite-2.0.0+2

same here

I attempted to reproduce this on the latest stable version but I was not able to reproduce this.
Given that this issue has not had any activity in quite some time, I will be closing this as stale.
If anyone faces this issue, kindly file a new issue and provide detailed information as to how to reproduce.
Thank you

flutter doctor -v
```
[√] Flutter (Channel stable, 3.3.4, on Microsoft Windows [Version 10.0.19043.2006], locale en-GH)
    • Flutter version 3.3.4 on channel stable at c:\src\sdks\stable\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision eb6d86ee27 (7 days ago), 2022-10-04 22:31:45 -0700
    • Engine revision c08d7d5efc
    • Dart version 2.18.2
    • DevTools version 2.15.0
[√] Android toolchain - develop for Android devices (Android SDK version 32.1.0-rc1)
• Android SDK at C:\Users\Deana\AppData\Local\Android\sdk
• Platform android-33, build-tools 32.1.0-rc1
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)
• All Android licenses accepted.
[√] Chrome - develop for the web
• Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe
[√] Visual Studio - develop for Windows (Visual Studio Community 2022 17.2.3)
• Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
• Visual Studio Community 2022 version 17.2.32526.322
• Windows 10 SDK version 10.0.22000.0
[√] Android Studio (version 2021.2)
• Android Studio at C:\Program Files\Android\Android Studio
• 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 11.0.12+7-b1504.28-7817840)
[√] IntelliJ IDEA Community Edition (version 2022.1)
• IntelliJ at C:\Users\Deana\AppData\Local\JetBrains\Toolbox\apps\IDEA-C\ch-0\221.5921.22
• 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
[√] IntelliJ IDEA Ultimate Edition (version 2022.1)
• IntelliJ at C:\Users\Deana\AppData\Local\JetBrains\Toolbox\apps\IDEA-U\ch-0\221.5921.22
• 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
[√] VS Code (version 1.72.1)
• VS Code at C:\Users\Deana\AppData\Local\Programs\Microsoft VS Code
• Flutter extension version 3.50.0
[√] Connected device (4 available)
• M2007J20CG (mobile) • 5dd3be00 • android-arm64  • Android 12 (API 31)
• Windows (desktop)   • windows  • windows-x64    • Microsoft Windows [Version
10.0.19043.2006]
• Chrome (web)        • chrome   • web-javascript • Google Chrome 106.0.5249.103
• Edge (web)          • edge     • web-javascript • Microsoft Edge 106.0.1370.37
[√] HTTP Host Availability
• All required HTTP hosts are available
• No issues found!
</details>


Terminus 3.0 compatibility

When the plugin is installed with the terminus 3 command (terminus self:plugin:install terminus-plugin-project/terminus-pancakes-plugin ), every subsequent terminus command generates multiple PHP warnings:
PHP Warning:  include(/Users/jmann/.terminus/terminus-dependencies-2512c16/vendor/terminus-plugin-project/terminus-pancakes-plugin/src/Commands/../../vendor/autoload.php): failed to open stream: No such file or directory in /Users/jmann/.terminus/terminus-dependencies-2512c16/vendor/terminus-plugin-project/terminus-pancakes-plugin/src/Commands/PancakesCommand.php on line 13

The plugin still works as expected without this include statement, so I propose simply removing it.

I'm guessing that this was included for Terminus 1 (or Terminus 0?) compatibility. It would be best to remove it now, as those old Terminus versions are no longer supported on the Pantheon platform.

@jeffm2001:  Good call!  Thank you for the confirmation @greg-1-anderson.


Add .editorconfig

Consider adding a .editorconfig to the project.
More info here.
# Editor configuration, see https://editorconfig.org
root = true

[*]
charset = utf-8
omdemt_style = space
indent_size = 4
trim_trailing_whitespace = true
# Unix-style newlines with a newline ending every file
end_of_line = lf
insert_final_newline = true

# Tab indentation (no size specified)
[Makefile]
indent_style = tab

[*.md]
max_line_length = off
trim_trailing_whitespace = false

Added! Thank you!:)


Importing heroku env variables

Issue story
As a FE developer
I need to be able to import system variables added in heroku
So that I can provide API keys dynamically based on dyno environment

Additional information
I would like to know if there's any chance to send variables via node command line such as:
ng build --target=production --env=prod --myvar=1 --myvar2=2

This should be either as node args or via a an external file while not touching the existing environment files.
The addition should not create any additional files in the process and should use pre-existing hooks combined with webpack DefinePlugin methodology
Thank you

The functionality we have for this scenario is environment files: https://github.com/angular/angular-cli#build-targets-and-environment-files


Added autotools support

I added autotools supports, some of the systems I used does not have cmake and I got no permissions so with autotools user just needs a bash intepreter to install our package from the tar ball.
I also modified the travis.yml file to also test the autotools build.
Note that from autotools is easy to create a package :+1:
EDIT: I had to add unittest++ since gtest with Autotools is a nightmare. I created a second unit test.

We will not use autotools.

@lamerman that is your call to decide what technologies to use or not. However, Cmake and Autotools can coexist together, is it there any particular reason to not to include autotools?
In my case, I added autotools since for my use case as an user it is needed.


⚠ ClamAV has degraded performance

In dfc05a5, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 460 ms

Resolved: ClamAV performance has improved in e6ff979.


Black screen on initial load

Great tutorial, and thank you for sharing this code.
On a Mac machine, getting a scene where it appears the camera is not correctly moving.
Using the arrow keys, I can see red parts of the robot move across the screen

Did a bit more digging, it seems to be related to the CSGBox, when the csgbox3d is hidden, simulation works
Just downloaded the latest release candidate - must be an issue or update with that


Feature/scale enhancements

[hipBLASLt] add Sample for ScaleA/B
Enhanced sample_hipblaslt_scale_a.cpp and sample_hipblaslt_scale_b.cpp with additional diagnostics and matrix size tests, updated CMakeLists.txt.

Are you sure that scale A/B can take effect?

Yes, based on the code implementation and the successful execution results, I am sure that the scale A/B operations are taking effect. Both simpleGemmScaleA and simpleGemmScaleB functions correctly set the scale pointers and successfully perform the matrix multiplication with scaling. The execution results showed in the picture prove that all heuristic solutions are successful, indicating that the operations, including scaling, are working as expected.

Are you sure that scale A/B can take effect?

Yes, based on the code implementation and the successful execution results, I am sure that the scale A/B operations are taking effect. Both simpleGemmScaleA and simpleGemmScaleB functions correctly set the scale pointers and successfully perform the matrix multiplication with scaling. The execution results showed in the picture prove that all heuristic solutions are successful, indicating that the operations, including scaling, are working as expected.

return success doesn't mean scale A can take effect.

@jichangjichang
do you mean there should be a validation section in this sample?

@jichangjichang do you mean there should be a validation section in this sample?

No, you just need to validate it atlocal side to make sure the sample is correct.

@jichangjichang do you mean there should be a validation section in this sample?

No, you just need to validate it atlocal side to make sure the sample is correct.

Got it, I will validate it and return the results for you to review soon.

Remove this PR and send a new PR soon


The 'Preserve Blob Tags' part is not localized on the Settings panel

Storage Explorer Version: 1.30.0-dev
Build Number: 20230521.1
Branch: main
Platform/OS: Windows 10/Linux Ubuntu 20.04/MacOS Ventura 13.3.1 (Apple M1 Pro)
Reproduce Languages: All languages
Architecture: x64/arm64
How Found: From running test cases
Regression From: Not a regression
Steps to Reproduce

Launch Storage Explorer -> Open 'Settings' -> 'Application -> Regional Settings'.
Select 'Français' -> Restart Storage Explorer.
Open Settings -> Click 'Transfer'.
Check whether the 'Preserve Blob Tags' part is localized.

Expected Experience
The 'Preserve Blob Tags' part is localized.
Actual Experience
The 'Preserve Blob Tags' part is not localized.

This string is localized now.

Verified this issue on the build 20230603.6. Fixed.


upgrade RestAssured version (FINERACT-809)

Co-authored-by: percyashu<EMAIL_ADDRESS>

This is just a rebase of  #719 from @xurror and @percyashu and I'm just curious if this passes... I ran this locally x3 times, and it failed with https://issues.apache.org/jira/browse/FINERACT-855 every time - may be that is not just a flaky test, but really something that this PR breaks, for some (strange, yes) reason.
NB that other PRs passed today (e.g. #723 and #725) so if this still fails, perhaps the new RestAssured version is somehow changing some timing or something which causes IT test failures?!

Now this failed due to yet another new IT failure, see FINERACT-857..

@percyashu or @xurror ..Please can we look into this?

#719


fix: compatibility with storybook < 6.4

In https://github.com/creevey/creevey/pull/187 the<EMAIL_ADDRESS>was added as a new dep. But it depends on another<EMAIL_ADDRESS>packages. This results in conflicts while trying to use<EMAIL_ADDRESS>in a project with storybook < 6.4.
I see two possible solutions:

replace @storybook/testing-library package with original @testing-library/dom and probably lose some debugging features (implemented in this PR);
officially drop support of storybook < 6.4 starting from<EMAIL_ADDRESS>

I suggest the 2nd variant.

Closing in favor of https://github.com/creevey/creevey/pull/239


Local llm fixes

Description

Related Issues

Solution and Design

Test Plan

Type of change

[ ] 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)
[ ] Docs update

Checklist

[ ] My pull request is atomic and focuses on a single change.
[ ] I have read the contributing guide and my code conforms to the guidelines.
[ ] I have documented my changes clearly and comprehensively.
[ ] I have added the required tests.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: rounak610:x: UbuntuUbuntu seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Add LLMs-related findings to search functionality document

@alandefreitas This is ready to merge. I would appreciate your suggestions if you come across any areas in the text that are unclear.

@alandefreitas This is ready to merge. I would appreciate your suggestions if you come across any areas in the text that are unclear.

The text is fine. It's just weird to have two commits for this single thing. Could you squash this?


Truncate long failure reasons in activity worker

Closes #161
Logging format is very consistent... wasn't sure how to make the truncate log message fit within the pattern.

Looks like the CI build needs to be rerun.

:+1:


[core] Batch of fixes

Fix a prop type warning when rendering on the server
Fix a UI jump on the home page
Fix a table break of line issue with the demos
Remove a flaky test from Argos
Remove dead code (thanks to code coverage)
Use US Engish over UK.
Remove wrongly translated markdown files (in the premium themes)

Could you write a summary what you fixed?

Yes, I have updated the pull request description.


llama3.1:70b <|eom_id|> missing as stop in template

What is the issue?
llama3.1:70b outputs <|eom_id|> as a string rather than an input. To test this, make a function call to the model without a 'tool' response message multiple times. 8B hallucinates, 70B wants to refuse, returns an <|eom_id|> but ollama doesn't recognize it.
OS
Linux
GPU
Nvidia
CPU
Intel
Ollama version
latest

This should be fixed now as per @MaxJa4's comment


fix: format_number() can't be used on CLI

Description
format_number() depends on IncommingRequest::getLocale(), and causes Error Call to undefined method CodeIgniter\HTTP\CLIRequest::getLocale().
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

Thank you @kenjis


Passter

Add https://github.com/WardPearce/paaster

There is no need for this as it is currently supported in vaultwarden


CartJS & Shopify multiple languages store ?

Hi,
With the native translation API from Shopify, we can now have this kind of URL : site.com and site.com/fr or site.com/de
But the response from CartJS is always using this structure :

site.com/cart.js,
site.com/cart/add.js
site.com/cart/change.js
site.com/cart/update.js
site.com/cart/clear.js

It would be great to add the current language in the structure : site.com/fr/cart.js (or site.com/de/cart.js)
Currently, when we are using CartJS.addItem, whichever language is being used in the front-office, the response from CartJS is awlays to /cart.js instead of /fr/cart.js (as an example)
Thanks in advance.

Hey @msev
Just realized this was also affecting some of our stores.
We use CartJS with Rivets.
And whenever a line item is updated, the front actually shows a cart in the root locale !
Here is my fix, works like a charm.

Declare routes in theme.liquid in a script markup

window.kercambreRoutes = {
      root_url : "{{ routes.root_url }}",
      add_url: "{{ routes.cart_add_url }}",
      cart_change_url : "{{ routes.cart_change_url }}",
      cart_clear_url: "{{ routes.cart_clear_url }}",
      cart_update_url: "{{ routes.cart_update_url }}"
    }

In Cart.min.js, find the functions declarations and simply use the variable above to ping the correct url.
For instance :

getCart: function (a) {
    console.log("get CART");
    return null == a && (a = {}), (a.type = "GET"), (a.updateCart = !0), c.Queue.add( "/cart.js", { v: new Date().getTime() }, a);
            },

and change it to :
getCart: function (a) {
      console.log("get CART");
      return null == a && (a = {}), (a.type = "GET"), (a.updateCart = !0), c.Queue.add( window.kercambreRoutes.cart_url + ".js", { v: new Date().getTime() }, a);
            },

That worked for me :)
The best of course should be to start the CartJS.init() function with the sufficient variables. Might try to do a pull request.


python3Packages.pyzmq: fix build

Description of changes
Tested building a few packages that depend on it too: circus, jupyter-client, logbook, partd, rpcq, tornado, zerorpc.
Fixes: e597af558361 ("zeromq: enable curve support")
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Idiotic Darwin and its full path in the NEEDED section:
$ otool -L result/lib/python3.12/site-packages/zmq/backend/cython/_zmq.cpython-312-darwin.so
result/lib/python3.12/site-packages/zmq/backend/cython/_zmq.cpython-312-darwin.so:
        /nix/store/lr8xrdglvzhp0rsykrpl4bhs5d5pxs2l-zeromq-4.3.5/lib/libzmq.5.dylib (compatibility version 5.0.0, current version 5.2.5)
        /nix/store/wkn35lg99z83sf9x4qwncx0vyzx6r716-libsodium-1.0.20/lib/libsodium.26.dylib (compatibility version 29.0.0, current version 29.0.0)
        /usr/lib/libSystem.B.dylib (compatibility version 1.0.0, current version 1238.60.2)

I completely missed that when testing #338064 just on Darwin.
But now I think there must be a better solution that adding now libsodium to every zeromq dependency. Shouldn't it be propagated automatically if we add libsodium to propagatedBuildInputs?
I'll need to find a way to setup a Linux testing env.

Does everything that uses zeromq need libsodium?  If so, we might want to just link zeromq against it.  (This is nicer than propagation, because propagation is exceptionally difficult to remove down the line if it becomes unnecessary.)

Okay, well, we need some fix for the next staging cycle.  I would still be happy to find a better one.

@OfBorg build python3Packages.pyzmq


Q: Whats the reason behind custom rpc client?

I was looking through your code and how you interact with nodes, and I wonder why you handcraft RPC requests instead of using the client node has?

Option for client

You also could reuse the types we define

Hi, we use our own API wrapper because your wrapper does not realize a rate limit. It's required because we allow to request any blob by our API and it can be potential weak point.

Why do you need a client side rate limit? The node could have a rate limit on the server side

Almost all services temporarily blocks client which makes too many requests. It would be bad if node blocks our back-end. I don't know the behavior of node so it's easier to limit requests on client side.

I see. Currently node doesn't have any rate-limiting. It will have it optionally, but not now. Hence you don't really need it to have client side one, as there is nothing blocking for you. Also, the rate-limiting we will do won't be blocking, but slowing request processing. Btw, you ask me anything about the node next time to validate your assumption, I am happy to help you.

Got it. Thank you for the information.


Heredoc doesn't stop

this seems to be because of the fact that hilbish does not append a newline to the end of input in continued input.


Implement server action with mongoose and mongoDB integration

[x] quote fetching logic
[x] validation
[ ] mongo append

example how to separately store mongoDb model

https://medium.com/@aniruddh622003/setting-up-mongodb-with-mongoose-and-nextjs-13-3a598609c5d1

Previously i came up to quotes structure as:
quotes: [
      {
        usd: [
          { amd:<PHONE_NUMBER>5 },
          { brl: 4.94543198 },
          { usd: 1 },
        ],
      },
]

but this seems like not best structure due to dynamic key names (currencies). This is not convenient for schemas. I.e. for mongoose scheme.
Better to use static key names:
quotes: [
    {
        baseCurrency: "usd",
        rates:[
             {currency: "brl", rate: 4.94543198},
             {currency: "amd", rate:<PHONE_NUMBER>5},
        ]
    }
]

MongoDB supports subdocuments, so for example quotes can be stored separately as a subdocument of record. For the sake of simplicity will not be use em and store whole nested record inside single document

Implemented base functionality.
Remaining

add error handling to ui
add success handling to UI
prevent adding multiple records per month. Ask to override


events.js:167 throw er; // Unhandled 'error' event

Hi,
I get this weird unhandled 'error' event and node server just crash.
Is this node or routeros-client related?
"node": "v10.4.1"
"routeros-client": "^0.11.0"
events.js:167
throw er; // Unhandled 'error' event
^
RosException: Timed out after 10 seconds
at Connector.onTimeout (/var/app/node_modules/node-routeros/dist/connector/Connector.js:181:30)
at Object.onceWrapper (events.js:273:13)
at Socket.emit (events.js:182:13)
at Socket._onTimeout (net.js:444:8)
at ontimeout (timers.js:427:11)
at tryOnTimeout (timers.js:289:5)
at listOnTimeout (timers.js:252:5)
at Timer.processTimers (timers.js:212:10)
Emitted 'error' event at:
at RouterOSAPI.rosApi.once (/var/app/node_modules/routeros-client/dist/RouterOSClient.js:33:53)
at Object.onceWrapper (events.js:273:13)
at RouterOSAPI.emit (events.js:182:13)
at Connector.connectedErrorListener (/var/app/node_modules/node-routeros/dist/RouterOSAPI.js:103:30)
at Object.onceWrapper (events.js:273:13)
at Connector.emit (events.js:182:13)
at Connector.onTimeout (/var/app/node_modules/node-routeros/dist/connector/Connector.js:181:14)
at Object.onceWrapper (events.js:273:13)
at Socket.emit (events.js:182:13)
at Socket._onTimeout (net.js:444:8)
Thanks for help!

I haven't tested with node v10, but the error is related to routeros-client are you catching the login promise error?
There are two errors you must catch, one from the promise when you try to connect:
api.connect().then((client) => {
    // Connected
}).catch((err) => {
    // Connection error
});

The other error occurs when you are connected but for some reason the connection fails:
const api = new RouterOSClient({
    ...
});

api.on('error', (err) => {
    // Connection error
});

Thanks. I will add api.on('error', ..) and let you know, if no more app crashing with Unhandled exception error.
I was catching errors for connect, but not for the connected connection fail situation.

Closing due to inactivity.


Create ndmc.txt (Notre Dame of Midsayap College, INC)

Good afternoon, JetBrains team,
I hope this message finds you well. I would like to propose an update by adding my current college to the list of schools.
Here are the details:
• Official website: Notre Dame of Midsayap College
• Official School Email<EMAIL_ADDRESS>• Undergraduate programs: NDMC Undergraduate Programs
• Proof of legitimacy: [Notre Dame of Midsayap College - Wikipedia (https://en.wikipedia.org/wiki/Notre_Dame_of_Midsayap_College)
Thank you for your time and consideration!
Best regards,
zeykdev

@zeykdev Thank you for your patience while waiting for a review and processing your request. Pull request merged. Please start applying for JetBrains educational subscription at https://www.jetbrains.com/shop/eform/students in an hour, to allow the changes to propagate through JetBrains systems.

Thank you very much! I’ll apply in an hour. ✨


Makes the sort stable even if the underlying Arrays.sort() is unstable

Since Arrays.sort() is unstable for large (> 10) arrays in Chrome, I needed the sorting to compare previous array index if several elements in the array has the same value. That way they will keep their previous sorting order.

Hey there.
Thanks for the PR. Can you cite or demo this instability? Thanks!

I created a simple web page and put in the same directory as stupidsort.js
Code:

Skola
Skolform

AFsk
BFsk
CFsk
DFsk
EFsk
FFsk
GFsk
HFsk

A	Gr

B	Gr

C	Gr

D	Gr

E	Gr

F	Gr

G	Gr

H	Gr

https://code.google.com/p/v8/issues/detail?id=90

I'll take a look, thanks for the issue

Hi. Did you not like my solution?

Hi @egoon . Thanks for sending this PR a while back. The commit provided in this PR dc50bd5c41 was added to this merge https://github.com/joequery/Stupid-Table-Plugin/commit/8f035673cc140ef366396c9c7d23dfda577fccbc
I added unittests to verify stability.
Thanks again!


GD-42: Error after installing.

The used GdUnit4 version
4.2.2 (Latest Release)
The used Godot version
4.2.1
Operating System
MacOs 14.4 M1 Mac.
Describe the bug
After installing from AssetLib, the plugin fails to load with the following errors:
CS0246: The type or namespace name 'GdUnit4' could not be found (are you missing a using directive or an assembly reference?) addons/gdUnit4/src/mono/GdUnit4CSharpApi.cs(4,7)

CS0246: The type or namespace name 'GdUnit4' could not be found (are you missing a using directive or an assembly reference?) addons/gdUnit4/src/mono/GdUnit4CSharpApi.cs(15,16)

I've since tried later versions via manual installation, but have not been successful.
Steps to Reproduce

Install the plugin.
Activate the plugin.
Observe the issue.

Minimal reproduction project
No response

Which Godot version you are using?
Please fill in the full version information.
I guess you are using the Godot mono version without a c# project, if you run Godot Mono you need to create the c# project.

Please fill in the full version information.

Oh, my apologies - I thought I had! v4.2.1.stable.mono.official [b09f793f5]
I guess you are using the Godot mono version without a c# project, if you

run Godot Mono you need to create the c# project.

I have a C# project - in that I've got other C# code in the project.
But this pointed me in the right direction - I needed to install/activate
the plugin and then re-create the C# project after, and then everything
worked.
Thanks!
Unless I missed it, is it worth adding this step to the docs? I'm happy to
do that if it's helpful!
On Fri, 22 Mar 2024 at 08:14, Nullpointer @.***> wrote:

Which Godot version you are using?
Please fill in the full version information.
I guess you are using the Godot mono version without a c# project, if you
run Godot Mono you need to create the c# project.
—
Reply to this email directly, view it on GitHub
https://github.com/MikeSchulze/gdUnit4/issues/377#issuecomment-2014564842,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHDB667WD5OOHJVMXKGUGLYZPR7HAVCNFSM6AAAAABFCJ3KZKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJUGU3DIOBUGI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Feel free to improve the documentation ;)
here is the repository https://github.com/MikeSchulze/gdUnit4/tree/documentation

Hmm, pre-caffeine brain spoke too soon; regenerating the csproj didn't fix
it.
What did fix it was properly reading the C# section of the manual, which
I had missed, and manually adding the required bits the .csproj, as
described here: https://mikeschulze.github.io/gdUnit4/faq/C%23/
Silly me. My apologies, and thanks!
Simon
On Fri, 22 Mar 2024 at 09:41, Nullpointer @.***> wrote:

Feel free to improve the documentation ;)
here is the repository
https://github.com/MikeSchulze/gdUnit4/tree/documentation
—
Reply to this email directly, view it on GitHub
https://github.com/MikeSchulze/gdUnit4/issues/377#issuecomment-2014713534,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHDB64WADRIQMZKA3Y4JGDYZP4GLAVCNFSM6AAAAABFCJ3KZKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJUG4YTGNJTGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

ok no problem, i will close it as clarified


Create types for environment variables and prefix with PROSOPO

There a loads of different environment variables currently used to populate configs. These names should be typed and prefixed with PROSOPO to avoid collisions with other client variables. In the case of a React App, the typed variables names should be prefixed with REACT_APP so that they are picked up by REACT. Same goes for NEXT, and and other frameworks that require certain known prefixes.

Follow up to https://github.com/prosopo/captcha/pull/652

Also
API_PORT is one env variable, but API_BASE_URL contains the port too. It would be better to have a single variable to control the port, i.e. eliminate the port from API_BASE_URL or remove API_PORT altogether


Add Nginx

Adds nginx to support deploying behind a load balancer that does SSL termination. Nginx is configured to provide the X-Forwarded type headers to rails which address ActionController::InvalidAuthenticityToken (ActionController::InvalidAuthenticityToken): errors.
References:

https://github.com/rails/rails/issues/22965

I forgot to update the README with instructions on how to run. I'll do that shortly.

Mention reference this PR in Issue #89 after merging.

Check that iis is populated when running locally, with docker, and with docker compose.

Check that iis is populated when running locally, with docker, and with docker compose.

Confirmed that iss is populated to http://localhost:3000 when running all 3 ways.


upgrade guzzle to ^7.0

You are using guzzle version 6.. Modern Application uses ^7.0.. Its good to have guzzle version ^7.0.. It will solve many issues while installing this packages

Could this pull request be merged ? This is the only package in my project still preventing me from upgrading to Guzzle 7.

Yes, this pull request can be merged..

Dear team. Please merge. Or release a new version


Add Mesos hyperkube minion server

The minion hyperkube server will launch the executor and the proxy and rotate their logs, as a replacement of a S6 setup.

integration test failing in shippable for one of the two builds (so, unrelated?):
I0728 20:31:47.262371 32362 config.go:253] Setting pods for source file : {[] 0 file}
I0728 20:31:48.262781 32362 config.go:253] Setting pods for source file : {[] 0 file}
F0728 20:31:48.575741 32362 integration.go:879] FAILED: (Scheduler probably didn't process deletion of 'phantom.bar') Pod never started running: timed out waiting for the condition
!!! Error in ./hack/test-integration.sh:51

reviewed, a few minor nits. otherwise lgtm. have not tested

@jdef PTAL
The command line handling in the scheduler is not really elegant right now. This really asks for a multiflags package (not in the limits of the PR) which can handle command lines like
km scheduler --logv=2 executor --logv=3 proxy --logv=9

with scheduler as the primary command which somehow launches the executor and proxy.

few minor comments. otherwise lgtm.

Addressed all remaining minor nits. Rebased and squashed (as far as it makes sense).
@jdef or @karlkfi Ready to be merged. Can you assign to @davidopp?
This PR only modified contrib/mesos files with the exception of the new lumberjack.v2 logging library as godep.

Rebased to master and addressed @tsenart's comment (if not state otherwise above).
Ready for merge.

This broke 'make quick-release'


enable distributed rate limiter in pingora limit

What is the problem your feature solves, or the need it fulfills?
distributed rate limiter with pingora limit, so that we can deploy pingora in a group of hosts to handle larger amount of traffics.
Describe the solution you'd like
maybe, an async trait that can invoke external service, like redis

there is the storage trait that you can implement for caching, and make your own caching strategy

This is a great idea, but it isn't something we are likely to add to pingora. That sort of cross-process coordination isn't a goal for the core library. If there was a distributed rate limiter (or just an agreement on how to use some distributed storage to store rates), it could be called within pingora just like the existing rate-limit constructs.


fix: improve meteor method signatures

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Bug fix

What is the current behavior? (You can also link to an open issue here)

Some methods fail to execute, failing type checks on incoming arguments.

What is the new behavior (if this is a feature change)?

Signatures were changed to expect NULL as a possible value on optional arguments (undefined arguments are changed to null for sending over the wire in Meteor).

Other information:

Status

[x] Code documentation for the relevant parts in the code have been added/updated by the PR author
[x] The functionality has been tested by the PR author
[ ] The functionality has been tested by NRK

Codecov Report

Merging #669 (2248d4c) into release40 (6990b93) will decrease coverage by 0.00%.
The diff coverage is 26.66%.

@@              Coverage Diff              @@
##           release40     #669      +/-   ##
=============================================
- Coverage      64.02%   64.01%   -0.01%
=============================================
  Files            295      295
  Lines          18552    18553       +1
  Branches        4401     4403       +2
=============================================
  Hits           11877    11877
- Misses          6334     6335       +1
  Partials         341      341

Impacted Files
Coverage Δ

meteor/lib/api/migration.ts
100.00% <ø> (ø)

meteor/lib/api/triggeredActions.ts
100.00% <ø> (ø)

meteor/lib/api/userActions.ts
100.00% <ø> (ø)

meteor/server/api/userActions.ts
22.98% <0.00%> (-0.07%)
:arrow_down:

meteor/server/migration/api.ts
82.35% <100.00%> (ø)

Continue to review full report at Codecov.

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


Add monochrome icon

Please support the Android 13 "Themed App Icons" feature.

Thanks 🙏

The next release will have support for monochrome
It will release whenever I can get a screen in the app to show connected hotspot devices.
Thanks!

Thanks 🙏

Version 22 is in testing and should release on the store in the next couple of days, this should bring suport for monochrome icons.
I do not have a monochrome icon supported launcher, so please test when the new version comes to your device and see if this is resolved. Thank you!

Thanks, please release this version on the GitHub.
I don't have access to Play Store.
Thanks.

The latest release is out on github

Thanks but why the background is not simple?
The purpose of this feature is the uniformity of the icons, in this case the purpose will not be achieved.

Again, since I don't actually have a launcher that supports monochrome I was unable to actually test.
Thus, I didn't know what it looked like at all. I will see about removing the background mask from the iconography

Ok, you can use this launcher:
https://github.com/MM2-0/Kvaesitso


Yank all 0.7 versions older than 0.7.6

Due to #1388. If the standard library ever do change the memory layout of SocketAddr, usage of this crate of versions before 0.7.6 will have undefined behaviour and lead to segfaults. And even if the standard library never do change the layout of those types, those older versions of this crate still makes assumptions they should not.
This is exactly what yanking is for. Marking a release as "should not be used". I think all versions of mio from 0.7.0 through 0.7.5 should be yanked from crates.io.
Related issues on other crates having the same problem: https://github.com/deprecrated/net2-rs/issues/107, https://github.com/rust-lang/socket2-rs/issues/139

Yanked all v0.7.x versions expect v0.7.6: https://crates.io/crates/mio/versions. I'm leaving v0.6.x as the dependencies are at fault there.


Removed outdated comment.

closes #78

I had expected that there should be more obsolete code to remove regarding the functionality of this "load" button, but I can't find it so it might have been dropped already. So LGTM.