HTTP 500 From Mock Server For V2 Consumer Tests

Hi, i'm trying to generate a V2 pact file as a consumer but i'm getting an exception when making a call to the mocked server and the client is getting back HTTP 500.
Any idea what's wrong? Thanks!
Exception:
System.Exception : Unable to get data
   at MyApp.PactTests.ApiClient.Get() in C:\Temp\Example\MyApp.PactTests\ConsumerTests\ApiDataTests.cs:line 81
   at MyApp.PactTests.ApiTests.get_api_data() in C:\Temp\Example\MyApp.PactTests\ConsumerTests\ApiDataTests.cs:line 58
--- End of stack trace from previous location where exception was thrown ---

[Test Class Cleanup Failure (MyApp.PactTests.ApiTests)]: System.ArgumentNullException : Value cannot be null. (Parameter 'source')
   at System.Linq.ThrowHelper.ThrowArgumentNullException(ExceptionArgument argument)
   at System.Linq.Enumerable.Any[TSource](IEnumerable`1 source)
   at ComPact.Models.V2.Response.SetEmptyValuesToNull() in C:\Temp\ComPact\Models\V2\Response.cs:line 25
   at ComPact.Models.V2.Interaction.SetEmptyValuesToNull() in C:\Temp\ComPact\Models\V2\Interaction.cs:line 29
   at ComPact.Models.V2.Contract.<>c.<SetEmptyValuesToNull>b__16_0(Interaction i) in C:\Temp\ComPact\Models\V2\Contract.cs:line 18
   at System.Collections.Generic.List`1.ForEach(Action`1 action)
   at ComPact.Models.V2.Contract.SetEmptyValuesToNull() in C:\Temp\ComPact\Models\V2\Contract.cs:line 18
   at ComPact.Builders.PactWriter.Write(IContract pact, String pactDir) in C:\Temp\ComPact\Builders\PactWriter.cs:line 24
   at ComPact.Builders.PactWriter.Write(IContract pact) in C:\Temp\ComPact\Builders\PactWriter.cs:line 19
   at ComPact.Builders.PactBuilderBase.BuildAsync(IContract pact) in C:\Temp\ComPact\Builders\PactBuilderBase.cs:line 74
   at ComPact.Builders.V2.PactBuilder.BuildAsync() in C:\Temp\ComPact\Builders\V2\PactBuilder.cs:line 46
   at MyApp.PactTests.ApiConsumerFixture.Dispose() in C:\Temp\Example\MyApp.PactTests\ConsumerTests\ApiDataTests.cs:line 27

Here's an simple example i've built of what i'm trying to do:
using System;
using System.Net;
using System.Net.Http;
using System.Threading.Tasks;
using ComPact.Builders.V2;
using ComPact.Models;
using Newtonsoft.Json;
using Xunit;

namespace MyApp.PactTests
{
    public class ApiConsumerFixture : IDisposable
    {
        public ApiClient _apiClient;

        public PactBuilder PactBuilder { get; }

        public ApiConsumerFixture()
        {
            const string mockServerBaseUrl = "http://localhost:5000";

            PactBuilder = new PactBuilder("my-consumer", "my-api", mockServerBaseUrl);

            _apiClient = new ApiClient(new HttpClient { BaseAddress = new Uri(mockServerBaseUrl) });
        }

        public void Dispose() => PactBuilder.BuildAsync().GetAwaiter().GetResult();
    }

    public class ApiTests : IClassFixture<ApiConsumerFixture>
    {
        private readonly PactBuilder _pactBuilder;
        private readonly ApiClient _apiClient;

        public ApiTests(ApiConsumerFixture fixture)
        {
            _apiClient = fixture._apiClient;

            _pactBuilder = fixture.PactBuilder;
            _pactBuilder.ClearInteractions();
        }

        [Fact]
        public async Task get_api_data()
        {
            _pactBuilder.SetUp(Pact.Interaction.Given(
                    $"An request for data")
                .UponReceiving("A get request for data")
                .With(Pact.Request.WithMethod(Method.GET)
                    .WithPath("/data"))
                .WillRespondWith(Pact.Response
                    .WithStatus((int) HttpStatusCode.OK)
                    .WithHeader("Content-Type", "application/json; charset=utf-8")
                    .WithBody(Pact.JsonContent
                        .With(Some.Element.Named("Id").WithTheExactValue("123456"))
                        .With(Some.Element.Named("SomeInt").WithTheExactValue("98765")))));

            await _apiClient.Get();
        }
    }

    public class ApiClient
    {
        private readonly HttpClient _httpClient;

        public ApiClient(HttpClient httpClient)
        {
            _httpClient = httpClient;
        }

        public async Task<ApiResponse> Get()
        {
            var request = new HttpRequestMessage(HttpMethod.Get, "/data");
            var response = await _httpClient.SendAsync(request);

            if (!response.IsSuccessStatusCode)
            {
                throw new Exception("Unable to get data");
            }

            var responseBody = await response.Content.ReadAsStringAsync();
            return JsonConvert.DeserializeObject<ApiResponse>(responseBody);
        }
    }

    public class ApiResponse
    {
        public string Id { get; set; }

        public int SomeInt { get; set; }
    }
}

Ah yes, well spotted. Because there were no matching rules defined it failed when trying to set a property to null that was already null.
I just fixed it.

Thanks @bartschotten! I've just tried with 0.3.1 of the package though i'm still getting the same HTTP 500 error back when calling the mocked servers URL using the above example. That's then leading to the api client thrown an exception as it got back a non-expected response code that was configured.
           at System.Linq.ThrowHelper.ThrowArgumentNullException(ExceptionArgument argument)
   at System.Linq.Enumerable.Any[TSource](IEnumerable`1 source)
   at ComPact.Models.V2.Response.SetEmptyValuesToNull() in C:\Temp\ComPact\Models\V2\Response.cs:line 25
   at ComPact.Models.V2.Interaction.SetEmptyValuesToNull() in C:\Temp\ComPact\Models\V2\Interaction.cs:line 29
   at ComPact.Models.V2.Contract.<>c.<SetEmptyValuesToNull>b__16_0(Interaction i) in C:\Temp\ComPact\Models\V2\Contract.cs:line 18
   at System.Collections.Generic.List`1.ForEach(Action`1 action)
   at ComPact.Models.V2.Contract.SetEmptyValuesToNull() in C:\Temp\ComPact\Models\V2\Contract.cs:line 18
   at ComPact.Builders.PactWriter.Write(IContract pact, String pactDir) in C:\Temp\ComPact\Builders\PactWriter.cs:line 24
   at ComPact.Builders.PactWriter.Write(IContract pact) in C:\Temp\ComPact\Builders\PactWriter.cs:line 19
   at ComPact.Builders.PactBuilderBase.<BuildAsync>d__13.MoveNext() in C:\Temp\ComPact\Builders\PactBuilderBase.cs:line 74
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.GetResult()
   at ComPact.Builders.V2.PactBuilder.<BuildAsync>d__4.MoveNext() in C:\Temp\ComPact\Builders\V2\PactBuilder.cs:line 46
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.GetResult()
   at MyApp.PactTests.ApiConsumerFixture.Dispose() in C:\temp\example\MyApp.PactTests\ConsumerTests\ApiDataTests.cs:line 27
   at Xunit.Sdk.ExceptionAggregator.Run(Action code) in C:\Dev\xunit\xunit\src\xunit.core\Sdk\ExceptionAggregator.cs:line 73

If it helps, i'm trying to use this in a netcoreapp3.1 test project with xunit 2.4.0.

Are you sure you updated the package? I tried to reproduce it with your exact code, and I get the error you described with 0.3.0, but with 0.3.1 it throws the exception you define in your ApiClient class "Unable to get data".

Yes, confirmed, using 0.3.1. I even tried removing the package and adding the project directly from source code and still got the issue. Here's a full copy of the CSPROJ, i'm not sure any of those other packages would create an issue, unless maybe it's an issue with .NET 3.1 and the test host'?
<Project Sdk="Microsoft.NET.Sdk">

  <PropertyGroup>
    <TargetFramework>netcoreapp3.1</TargetFramework>

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

  <ItemGroup>
    <PackageReference Include="ComPact" Version="0.3.1" />
    <PackageReference Include="Microsoft.AspNetCore" Version="2.2.0" />
    <PackageReference Include="Microsoft.AspNetCore.TestHost" Version="3.1.4" />
    <PackageReference Include="Microsoft.NET.Test.Sdk" Version="16.6.1" />
    <PackageReference Include="xunit" Version="2.4.0" />
    <PackageReference Include="xunit.runner.visualstudio" Version="2.4.0" />
    <PackageReference Include="coverlet.collector" Version="1.0.1" />
    <PackageReference Include="Moq" Version="4.13.1" />
  </ItemGroup>

</Project>

It appears its the "ClearInteractions" call in the test fixture that it's not happy about. If i removed that call from the constructor for the test class, the test then passes. With Pact we had to clear interactions after every test class to ensure there we no previous interactions carried over.

Also seeming some weird behaviour where on the first test run, it passes, but an sequential runs it fails. I wonder if the mock isn't being shutdown for some reason?

I did find a bug in ClearInteractions. Thanks for that! It isn't really necessary to call it under normal circumstances though.
The mock should definitely be shut down when calling BuildAsync. If you try to call the mock after that you should be able to see it fail.

Are you still seeing the problem with the latest version?

Latest version has resolved the issue I was getting with with consume setup, thanks! I'll try a provider now against those tests and see how I get on...


Feature: Added setting to change the background color

Resolved / Related Issues
Added color picker to the appearance page

Validation
How did you test these changes?

[x] Built and ran the app
[x] Tested the changes for accessibility

From initial look-over, it looks good!


Strava connector (Independent Publisher)

When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)

[x] I attest that the connector works and I verified by deploying and testing all the operations.
[x] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[x] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[x] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[x] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

[x] I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
[x] Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.
[x] Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.
[x] If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

Hi @RichardWie thanks so much for your FIRST Independent Publisher submission! :)
Our certification engineer, Srikanth, is currently on emergency leave. Please expect a delay with your PR. He will return January 31st, 2022. We apologize for any inconveniences.

Dear partner,
We apologize for the delay you might be experiencing while we are working on a high volume of connector requests. Rest assured that we are doing everything we can to address this volume. We are taking measures like improving our processes and hiring additional team members to guarantee resolving your connector requests in a timely manner.
Please let us know if you have any questions in the meantime.

@RichardWie We are preparing your connector for production deployment and will queue up for the next deployment schedule. Once your connector onboards the next deployment schedule, it starts to deploy your connector in our production environments that typically takes 3-4 weeks. Please note that the documentation for your connector will not be generated until the deployment process is in the final steps. Thanks.

[[certify-connector]]

Hi @RichardWie, I hope you are doing well. I want to follow up with you and let you know that your connector is in production deployment. this process will complete within 4-6 weeks.
Please let me know if you have any questions.
Thank you very much.

Hi @RichardWie, I hope you are doing well.
Apologies for the delay in the deployment process. There was an internal issue with the deployment train that prevented us from completing your connector deployment. We are working to fix this issue; we will keep you posted.
Please let me know if you have any questions.
Thank you so much for your patience and understanding.

Hi @Amjed-Ayoub, @natalie-pienkowska ,
Is there something I can do to help with the deployment?
I see it’s Not yet released.

Hello @RichardWie,
I hope you are doing well.
Congratulations, the deployment of your connector and documentation page was already completed, We have a short survey on the connector certification process that we would like for you to complete. Your feedback is appreciated and will help us improve the program.
Thank you very much for working with us.


Adds ability to group scan servers

This change will require a change in the accumulo-cluster script to handle the group names. For example, this is how the compactors are specified. So, instead of a list of addresses, like the tserver config, it will be a little more complex. I can do this in a follow-on ticket.


Requirements missing in readme

From my experience, at least the following are required:

Java (clearly)
python2 (not having it results in error:

Jul 20, 2020 1:19:02 PM it.unical.acr.qasp.Utilities runAndGetList
INFO: Command: /bin/bash -c /tmp/fmla28211205352332702261595243941821 /tmp/tempfile13974259926850230945.tmp -read-qcir -write-gq | /tmp/qcir-conv.py100199615952460303061595243941861 - -prenex -write-gq | /tmp/rareqs-nn131160796793846413811595243941863 -  returned code 134
Jul 20, 2020 1:19:02 PM it.unical.acr.qasp.Utilities runAndGetList
INFO: /bin/bash: /tmp/qcir-conv.py100199615952460303061595243941861: /usr/bin/python: bad interpreter: No such file or directory
Exception in thread "main" java.lang.IndexOutOfBoundsException: Index -1 out of bounds for length 0
	at java.base/jdk.internal.util.Preconditions.outOfBounds(Preconditions.java:64)
	at java.base/jdk.internal.util.Preconditions.outOfBoundsCheckIndex(Preconditions.java:70)
	at java.base/jdk.internal.util.Preconditions.checkIndex(Preconditions.java:248)
	at java.base/java.util.Objects.checkIndex(Objects.java:372)
	at java.base/java.util.ArrayList.get(ArrayList.java:458)
	at it.unical.acr.qasp.QAsp.solveQCIRProgram(QAsp.java:220)
	at it.unical.acr.qasp.QAsp.solve(QAsp.java:213)
	at it.unical.acr.qasp.QAsp.main(QAsp.java:101)

installing python resolves the line
INFO: /bin/bash: /tmp/qcir-conv.py100199615952460303061595243941861: /usr/bin/python: bad interpreter: No such file or directory

but still, soemthing seems to be missing. Should I put RareQS somewhere?
Jul 20, 2020 1:26:27 PM it.unical.acr.qasp.Utilities runAndGetList
INFO: Command: /bin/bash -c /tmp/fmla21855170453043390111595244387523 /tmp/tempfile15287017388391027071.tmp -read-qcir -write-gq | /tmp/qcir-conv.py96048812566246600861595244387556 - -prenex -write-gq | /tmp/rareqs-nn10866392572933433191595244387559 -  returned code 134
Jul 20, 2020 1:26:27 PM it.unical.acr.qasp.Utilities runAndGetList
INFO: c RAReQS-NN, Beta
Exception in thread "main" java.lang.IndexOutOfBoundsException: Index -1 out of bounds for length 0
	at java.base/jdk.internal.util.Preconditions.outOfBounds(Preconditions.java:64)
	at java.base/jdk.internal.util.Preconditions.outOfBoundsCheckIndex(Preconditions.java:70)
	at java.base/jdk.internal.util.Preconditions.checkIndex(Preconditions.java:248)
	at java.base/java.util.Objects.checkIndex(Objects.java:372)
	at java.base/java.util.ArrayList.get(ArrayList.java:458)
	at it.unical.acr.qasp.QAsp.solveQCIRProgram(QAsp.java:220)
	at it.unical.acr.qasp.QAsp.solve(QAsp.java:213)
	at it.unical.acr.qasp.QAsp.main(QAsp.java:101)

Was probably my problem.
Extra requirement to be added to readme:

%@constraint line should be present (even if there is no constraint)


Permission denied

~$ echo -e "\ndeb https://www.mindforger.com/debian stretch main" >> /etc/apt/sources.list
bash: /etc/apt/sources.list: Permission denied

solved:
Devian/Mint install is confusing though because I use Mint


C#: tests.Grpc.Core.Tests flake

https://grpc-testing.appspot.com/job/gRPC_master/config=dbg,language=csharp,platform=linux/7727/console

The correct link seems to be
https://grpc-testing.appspot.com/job/gRPC_master/7727/config=opt,language=csharp,platform=linux/console

Stacktrace:
at  <0xffffffff>
at (wrapper managed-to-native) Grpc.Core.Internal.CompletionQueueSafeHandle.grpcsharp_completion_queue_destroy (intptr) <0xffffffff>
at Grpc.Core.Internal.CompletionQueueSafeHandle.ReleaseHandle () <0x00013>
at System.Runtime.InteropServices.SafeHandle.RunRelease () <0x00107>
at System.Runtime.InteropServices.SafeHandle.Dispose (bool) <0x00035>
at System.Runtime.InteropServices.SafeHandle.Dispose () <0x00015>
at Grpc.Core.Internal.GrpcThreadPool.Stop () <0x000f5>
at Grpc.Core.GrpcEnvironment.Close () <0x00027>
at Grpc.Core.GrpcEnvironment.Release () <0x000bb>
at Grpc.Core.Server/c__async0.<>m__0 () <0x0000b>
at System.Threading.Tasks.Task.InnerInvoke () <0x0004a>
at System.Threading.Tasks.Task.Execute () <0x00056>
at System.Threading.Tasks.Task.ExecutionContextCallback (object) <0x00054>
at System.Threading.ExecutionContext.Run (System.Threading.ExecutionContext,System.Threading.ContextCallback,object) <0x0006f>
at System.Threading.ExecutionContext.Run (System.Threading.ExecutionContext,System.Threading.ContextCallback,object,bool) <0x0001e>
at System.Threading.Tasks.Task.ExecuteWithThreadLocal (System.Threading.Tasks.Task&) <0x0016d>
at System.Threading.Tasks.Task.ExecuteEntry (bool) <0x000dd>
at System.Threading.Tasks.Task.System.Threading.IThreadPoolWorkItem.ExecuteWorkItem () <0x00016>
at System.Threading.ThreadPool.m__0 (object) <0x000aa>
at (wrapper runtime-invoke) .runtime_invoke_void__this___object (object,intptr,intptr,intptr) <0xffffffff>
Native stacktrace:
mono() [0x4b23dc]
/lib/x86_64-linux-gnu/libpthread.so.0(+0xf8d0) [0x7f3fd152a8d0]
/lib/x86_64-linux-gnu/libc.so.6(gsignal+0x37) [0x7f3fd11a7107]
/lib/x86_64-linux-gnu/libc.so.6(abort+0x148) [0x7f3fd11a84e8]
/var/local/git/grpc/libs/opt/libgpr.so.0(+0x7f3d) [0x7f3fcd5cff3d]
/var/local/git/grpc/libs/opt/libgrpc.so.0(grpc_completion_queue_shutdown+0x26) [0x7f3fcd1b70b6]
/var/local/git/grpc/libs/opt/libgrpc.so.0(grpc_completion_queue_destroy+0x9) [0x7f3fcd1b7189]
[0x4135eb10]

Debug info from gdb:
E09/30/2015 12:44:35<PHONE_NUMBER>40736 src/core/support/sync_posix.c:48: assertion failed: pthread_mutex_lock(mu) == 0
=================================================================
Got a SIGABRT while executing native code. This usually indicates
a fatal error in the mono runtime or one of the native libraries
used by your application.
tools/run_tests/run_csharp.sh: line 52:  4168 Aborted                 $NUNIT_CONSOLE -labels "$1/bin/$MSBUILD_CONFIG/$1.dll"
FAILED: Grpc.Core.Tests [ret=134, pid=4162]

[BUILDPONY]still happening, same output[/BUILDPONY]

[BUILDPONY]still happening, same output[/BUILDPONY]

Marking P0 as this flakes frequently enough to show up on pull requests:
https://grpc-testing.appspot.com/job/gRPC_pull_requests/3484/config=dbg,language=csharp,platform=linux/testReport/junit/(root)/tests/Grpc_Core_Tests/

Reassigning to me: I have a fix.


When a user adds a Board Game to an Event, ask to add it to his/her library as well

Give the possibility to the user to add a Board Game to his/her library when adding it to an Event.

I feel it is cleaner (and it makes sense) to issue two API calls for this one.

Would it be more efficient to have an API that checks if a board game (from BGG ID or BGWE ID or both ?) is already in a user’s library? Or should the UI query for all the bgs in a user’s lib?

Sorry, not what I meant :)
Let's say that a User adds a bg to an event. 2 cases:

User doesn't have it in its lib, we ask to add it to his/her lib.
User does have it, in which case we skip that question.

How do we determine if BG is already in User's lib? :)

Via an API (like GET /user/library_games/:bgid
Use GET /user/library_games, then see if it is there

You don't have to check that. If you add a game that is already in an event/a library, the server won't change the database and will return a 200 OK response.

What I would like is to not propose to add it if it is already in the lib :)

Ok ! I see two options:

Issue a "GET /user/library_games" call when landing on the event games page. You need it anyway for displaying games to list/filter games to be added to the event (we talked about this more or less few weeks ago)
Avoid the problem by adding a checkbox "Add to my library if not in it yet". It is not so much burden to leave a checkbox I guess

I don't like the idea of issuing a request only for checking that a game is in the library.

Ask user to add to library via Snackbar


Update polish translations and add information about additional themes to README

I've updated polish translations (a few were missing in comparsion to english translations) and moved "by" word in forums/index into translations.
I've also added to the README information about other themes (twist and orange).

Thanks for taking the time to do this!

No problem, thanks for you for making Forem! :-)


When wallet attestation is revoked, wallet should display a disactivated message with a button to close the wallet. remove option to bypass

replace "skip" by "close"
user must not be able to skip

Close should close the application as user must not access the wallet

No that is not what we want!
we want 2 options to user
Delete Account
Close
if close, the app must be closed.

@bibash28


IGNORE: testing CI fix for e2e

IGNORE: testing CI fix for e2e

Codecov Report

Merging #382 (a578b12) into main (f08b506) will decrease coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #382      +/-   ##
==========================================
- Coverage   26.95%   26.91%   -0.05%
==========================================
  Files          45       47       +2
  Lines        6276     6345      +69
==========================================
+ Hits         1692     1708      +16
- Misses       4446     4495      +49
- Partials      138      142       +4

Impacted Files
Coverage Δ

module/x/gravity/keeper/test_common.go
65.09% <0.00%> (-0.19%)
:arrow_down:

module/x/gravity/module.go
2.59% <0.00%> (-0.15%)
:arrow_down:

module/app/app.go
0.62% <0.00%> (-0.03%)
:arrow_down:

module/x/gravity/keeper/migrations.go
0.00% <0.00%> (ø)

module/app/upgrades/v2/upgrades.go
42.85% <0.00%> (ø)


⚠ ZV800.com has degraded performance

In e9ec9de, ZV800.com (https://www.zv800.com//) experienced degraded performance:

HTTP code: 200
Response time: 2000 ms

Resolved: ZV800.com performance has improved in 35652d8 after 28 minutes.


/arcane Command

I would like a simple /arcane command to show you what it does, for e.g.
/arcane grace
Rank 1:
ON DAMAGED: 1.5% chance for +1% Health Regeneration Per Second for 1.5 seconds
Rank 2:
ON DAMAGED: 3% chance for +2% Health Regeneration Per Second for 3 seconds
etc.
And then Drop Sources which for Arcane Grace would be LOR/NMLOR
It could just show a picture of the stats and the source if that's any easier for you.
--EDIT--
Also could you possibly add the rarity of said arcane in the command?
Thank you for your time and consideration of this suggestion!
-Silarious

done 👍

didn't do by-rank because it would've spread it out a lot more

I even colored them by rarity:

Ok awesome thanks!


tests of fluent-plugin-config-expander fails with v0.14 (master) branch (uninitialized constant Fluent::Input/Fluent::Output)

HOW TO REPRODUCE
git clone https://github.com/tagomoris/fluent-plugin-config-expander
cd fluent-plugin-config-expander
# fix Gemfile
# + gem 'fluentd', git: 'https://github.com/fluent/fluentd'
bundle update
bundle exec rake

ERROR1. uninitialized constant Fluent::Input (NameError)
lib/fluent/plugin/in_config_expander.rb:3:in `<top (required)>': uninitialized constant Fluent::Input (NameError)

REASON
It looks this is caused because fluent/test now does not require 'fluent/input`
Adding require 'fluent/input' into test/helper.rb of the plugin made tests succeed.
ERROR2. Fluent::ConfigError: '' sections are required
Error: test_emit(ConfigExpanderOutputTest): Fluent::ConfigError: '<store>' sections are required
/Users/seo.naotoshi/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/bundler/gems/fluentd-0b6e5048da40/lib/fluent/config/section.rb:160:in `block in generate'
/Users/seo.naotoshi/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/bundler/gems/fluentd-0b6e5048da40/lib/fluent/config/section.rb:146:in `each'
/Users/seo.naotoshi/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/bundler/gems/fluentd-0b6e5048da40/lib/fluent/config/section.rb:146:in `generate'
/Users/seo.naotoshi/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/bundler/gems/fluentd-0b6e5048da40/lib/fluent/configurable.rb:68:in `configure'
/Users/seo.naotoshi/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/bundler/gems/fluentd-0b6e5048da40/lib/fluent/plugin/base.rb:39:in `configure'
/Users/seo.naotoshi/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/bundler/gems/fluentd-0b6e5048da40/lib/fluent/plugin_id.rb:34:in `configure'
/Users/seo.naotoshi/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/bundler/gems/fluentd-0b6e5048da40/lib/fluent/log.rb:423:in `configure'
/Users/seo.naotoshi/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/bundler/gems/fluentd-0b6e5048da40/lib/fluent/plugin_helper/event_emitter.rb:52:in `configure'
/Users/seo.naotoshi/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/bundler/gems/fluentd-0b6e5048da40/lib/fluent/plugin/multi_output.rb:53:in `configure'
/Users/seo.naotoshi/src/github.com/tagomoris/fluent-plugin-config-expander/lib/fluent/plugin/out_config_expander.rb:33:in `configure'
/Users/seo.naotoshi/.rbenv/versions/2.3.0/lib/ruby/gems/2.3.0/bundler/gems/fluentd-0b6e5048da40/lib/fluent/test/base.rb:71:in `configure'
/Users/seo.naotoshi/src/github.com/tagomoris/fluent-plugin-config-expander/test/plugin/test_out_config_expander.rb:22:in `create_driver'
/Users/seo.naotoshi/src/github.com/tagomoris/fluent-plugin-config-expander/test/plugin/test_out_config_expander.rb:65:in `test_emit'

REASON
I am not investigating yet.

ERROR2 was fixed via https://github.com/fluent/fluentd/pull/974, but ERROR1 still remains

@tagomoris For ERROR1, do you fix fluentd itself (fluent/test) for compatibility, or should each plugin be fixed?

IMO, each (test) code should require files which are used from code in that file.
I'll add that require in v0.14.1 or so if there are too many build errors in various plugins. Let me see how it goes.

With this change https://github.com/tagomoris/fluent-plugin-config-expander/pull/14, all tests of fluent-plugin-config-expander passed on v0.14. Let me close.


Flex 2748 convert add device fitnesse tests to cucumber

Test removed, fitnesse test ran because the test called the endpoint directly.

All is well.

All is well.


CHIP Device Controller Python Script Binding will crash on error

Problem
ble-scan-connect xxxxx
connect xxxx
ble-disconnect
(crash)

Proposed Solution
<suggested fix, suggested enhancement>

Is this still happening @erjiaqing post https://github.com/project-chip/connectedhomeip/pull/4542 ?

Is this still happening @erjiaqing post https://github.com/project-chip/connectedhomeip/pull/4542 ?

After 4542 is merged, we no longer use this API, so we close it now.

After 4542 is merged, we no longer use this API, so we close it now.


test_run_env_systemroot seems to be failing on python 3.12.8

Current Behavior
nox' test test_run_env_systemroot fails on python 3.12.8
Expected Behavior
test_run_env_systemroot should not fail on >=python 3.12.8
Steps To Reproduce
I am reporting this as a failure of a current rebuilding on NixOS that bumped python 3.12.7 -> 3.12.8.
The build logs from CI can be found here: https://hydra.nixos.org/build/281820678/nixlog/1
I tried to dig into the source code changes of 3.12.8 for shutil.which and i think the test is failing because of the following change:
https://github.com/python/cpython/commit/8899e85de100557899da05f0b37867a371a73800
The change intention on cpython is to fix behavior for which on windows systems.
The test_run_env_systemroot is running shutil.which with mocked sys.platform = "win32" so should be affected by the change.
In Lib/shutil.py the line 1558 change removes cmd from the windows lookup path thus the test is failing in our ci suite.
Environment
- OS: NixOS 25.05 (nixos-unstable)
- Python: 3.12.8
- Nox: 2024.10.09

Anything else?
In addition os.pathsep (line 1553) does return an unexpected  pathseperator when having the mock sys.platform = "win32" active. So i kind of have the feeling that the overall test setup is not correct when building in linux systems
I am actually not sure why sys.platform is mocked on the test at all, shouldnt it work the same on any platform?

thx for the patch :slightly_smiling_face:


Windows CI is broken

Bug description:
The Windows CI has been broken for 5 months, we should clean things up and get it back up and running.
To Reproduce:
Send a PR, Windows build is broken going back to November-ish 2020
Looks like this started with PR #1623
Expected behavior:
Windows CI passes
Screenshots/Videos:

@cglein FYI

I expect there will need to be continuous contribution to keep Windows CI working correctly. I know there was been some internal discussion about this before, since maintenance hasn't been super hands-on since the original contribution.
Some of the CI breaks that RNW sees that would apply to modules as well:

Visual Studio Breaks (and associated patch releases)
Major version upgrades for WIndows, to keep CI working
RNW build instability on less powerful machines

Should be back to working


Update avro, avro-compiler, avro-mapred to 1.10.1

Updates

org.apache.avro:avro
org.apache.avro:avro-compiler
org.apache.avro:avro-mapred

from 1.8.2 to 1.10.1.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.apache.avro" } ]

labels: library-update, semver-minor

Superseded by #401.


Alcatraz doesn't work on Xcode 6.2 beta

The Package Manager item doesn't seem to show in the Window menu.

I can confirm that after installing for Xcode-Beta 6.2 that Xcode crashed the first time tapping the menu item. The next time the menu item was gone. But after running the install script again it was there and tappable with no further crashes.

just updated to official release of Xcode 6.2. Alcatraz isn't showing up.

The app store official Xcode 6.2 UUID is A16FF353-8441-459E-A50C-B071F53F51B7.
Add that to the DVTPlugInCompatibilityUUIDs key in all your plugins in:
~/Library/Application Support/Developer/Shared/Xcode/Plug-ins/plugin.xcplugin/Show Package Contents/Contents/Info.plist
There are scripts floating about to help speed this up or you can write your own. I'll leave that as an exercise for the reader.
For future reference, to get Xcode's UUID, in terminal:
defaults read /Applications/Xcode.app/Contents/Info DVTPlugInCompatibilityUUID

EPIC ANSWER ianmaster \o/
Works like a charm. Thank you!

thank ianmaster, works for me as well.

Thanks for your solution @ianmasters

work like a magic, thanks @ianmasters

It works! Thanks you a lot, @ianmasters

find ~/Library/Application\ Support/Developer/Shared/Xcode/Plug-ins -name Info.plist -maxdepth 3 | xargs -I{} defaults write {} DVTPlugInCompatibilityUUIDs -array-add A16FF353-8441-459E-A50C-B071F53F51B7
For quick copy paste open your terminal and copy the command above. Searches for info plists recursively and adds the UUID to the UUIDs array.

@fuxx you beaut. Forgotten how painful Xcode was without Fuzzy autocomplete.

Seems like this was added 2 hours ago to the repo, but an uninstall and reinstall didn't work for me.  Thanks @ianmasters for the UUID and thanxxx @fuxx for the find snippet!

@clariontools the script doesn't install from master, it installs from deploy. I will try to release a new version soon

@fuxx Thank you! (And while I'm at it, thank you to the Alcatraz team, any Xcode plugin authors who might see this, and heck, even Google for pointing straight to this thread when I needed it. Everyone is awesome!)

@ianmasters fantastic!

:+1: @fuxx

Thank you, Stefan!

@ianmasters thanks

Thanks @fuxx

New command for App Store Xcode 6.3:
find ~/Library/Application\ Support/Developer/Shared/Xcode/Plug-ins -name Info.plist -maxdepth 3 | xargs -I{} defaults write {} DVTPlugInCompatibilityUUIDs -array-add 9F75337B-21B4-4ADC-B558-F9CADF7073A7

Just updated the UDID from fuxx's solution.

For anyone who finds this on Google, here is the command to insert the UUID of whatever XCode version you are using:
find ~/Library/Application\ Support/Developer/Shared/Xcode/Plug-ins -name Info.plist -maxdepth 3 | xargs -I{} defaults write {} DVTPlugInCompatibilityUUIDs -array-add $(defaults read /Applications/Xcode.app/Contents/Info DVTPlugInCompatibilityUUID)


[UNDERTOW-2149] - cookie config paths fix on UNDERTOW-2082

Jira: https://issues.redhat.com/browse/UNDERTOW-2149
Depends on https://github.com/undertow-io/undertow/pull/1366
Replace: https://github.com/undertow-io/undertow/pull/1380

"next release" based on reviews.


Close #687, pass sources path in through the command line

~Edit: Blocked by https://github.com/SuffolkLITLab/ALKiln/issues/760.~
~Will have to update~ Have updated ALKilnInThePlayground.
Is this format what we want?
npm run test @tag --sources=path/to/sources/dir

In this PR, I have:

~Added tests for any new features or bug fixes (if relevant)~
[x] Added my changes to the CHANGELOG.md at the top, under the "Unreleased" section
[x] Ensured issues that this PR closes will be automatically closed

Reason for this PR
This allows the caller to specify the path to the sources folder, something that's very environment specific and hard for ALKiln itself to work out. See the issue.
Links to any solved or related issues
Closes #687
Any manual testing I have done to ensure my PR is working
Tested locally with:
npm run cucumber @none -- --sources=./none
npm run cucumber @none -- --sources=./none --sources=./docassemble/*/data/sources
npm run cucumber @none -- --sources=./docassemble/*/data/sources

Tested from ALKilnInThe Playground as well.

Conclusion: That is indeed the desired format for the command line args.

This will have some merge conflicts with #774. I'd like to prioritize that one, so I'm holding off until that one's merged.

I think this may be unblocked now, and ready for review.

I think this may be unblocked now, and ready for review. If you have bandwidth, @BryceStevenWilley, would love your input.


Improve messages when setting the pwd

Reported by Govind K. Mane:
When password criteria are not met the error message should be clearer, e.g. "Your password does not meet the criteria, please re-enter your password."
The message displayed with successful  account creation rather than saying "password setting completed" should say e.g. "Your account password has been successfully created."

I guess some generic error messages like "invalid input" are given here, which are applied by default to all input components. So that's probably not solvable by just rephrasing them. To hit the mentioned use cases, it may require dedicated JSF validators with separate error handling.
Please check.

I think a good idea would be also to have 2 fields when setting the password, "re-enter" password, to make sure that the correct password is set.

@mdehn , @SylviaDehn  What is the process for reproducing? The 'set password' fields that I have found do not fit this description.

Govind Mane to comment.

Whenever any new user account build in catalog manager, it sent email notification to respective user to reset the password. Also if any existing customer want reset his/her password, email notification received to respective user to reset the password. This email has a link to access reset password page. On password rest page, user need to enter the userid, new password and retype of password. This is a page were validation is not taken care in case password criteria doesn’t meet, also on successful password reset message should be some meaning. I seems, given message has some meaning and it gives generic understanding on user action to reset password.

The issue is not in the code of catalog manager application.


OpenSCENARIO 2.0 supported

Description
We supported OpenSCENARIO 2.0 standard in this feature.
We implemented a compilation system to compile and execute osc2's code, and implemented part of the domain model.
We wrote some scenario files with .osc file extensions and each passed the test.
Please read README_OpenSCENARIO_2.0.md for more details.
Where has this been tested?

Platform(s): Ubuntu 20.04
Python version(s): 3.8.10
Unreal Engine version(s): UE4
CARLA version: 0.9.13

Possible Drawbacks
The support for the domain model is not complete, but some simple scenarios can already be implemented, such as the. osc file example in the example folder.

This change is

@glopezdiest We have re-created the PR for openscenario 2.0 support, please ignore previous one #990

@joel-mb could you review this, please?

We have updated the code to address the static code check.


Get Travis CI working.

@dhh I already configured it but I think someone with administrative rights have to enable in the Travis application

This issue can now be closed (even though build is currently failing, Travis CI is configured and running).


[NTR] TEMRA: terminally differentiated effector memory cells re-expressing CD45RA

@addiehl please
Currently in CL "TEMRA" is a broad synonym for both
CL:0001062 effector memory CD8-positive, alpha-beta T cell, terminally differentiated
and
CL:0001087 effector memory CD4-positive, alpha-beta T cell, terminally differentiated
As far as I can see, there isn't a broader term in CL to represent TEMRA cells.
Based on usage in literature, would you consider adding one please?
If the specs below are correct, I can add it myself.
Preferred term label:
effector memory CD45RA-positive, alpha-beta T cell, terminally differentiated
Synonyms
terminally differentiated effector memory cells re-expressing CD45RA (exact) (https://en.wikipedia.org/wiki/T_cell#Memory_T_cells)
terminally differentiated effector memory CD45RA+ T cells (exact) (PMID:25104859)
TEMRA cell (exact) (PMID:25104859)
effector memory RA T cell (broad? if not exact) (PMID:16237082)
TEMRA (broad or related, with has_synonym_type=abbreviation) (PMID:16237082)
Definition (free text, please give PubMed ID in format PMID:XXXXXX)
An alpha beta memory T cell with the phenotype CD45RA-positive.
Parent cell type term (check the hierarchy here https://www.ebi.ac.uk/ols/ontologies/cl):
CL:0000791 mature alpha-beta T cell
Your nano-attribution (ORCID)
https://orcid.org/0000-0002-2825-0621
Any additional notes or concerns
To become parent of
CL:0001062 effector memory CD8-positive, alpha-beta T cell, terminally differentiated
and
CL:0001087 effector memory CD4-positive, alpha-beta T cell, terminally differentiated
Thank you.
Paola

I don't mind if you add this term.  Is there a specific request for it?
We haven't provided general classes such as 'alpha-beta memory T cell, CD45RO-positive', 'central memory alpha-beta T cell', 'effector memory alpha-beta T cell', either, in case you want to fill out the full lattice of possibilities.

@addiehl

I don't mind if you add this term. Is there a specific request for it?

It's found in text mining, and TEMRA is in CL as a synonym for 2 sibling classes, which suggests that a parent may be useful.

We haven't provided general classes such as 'alpha-beta memory T cell, CD45RO-positive', 'central memory alpha-beta T cell', 'effector memory alpha-beta T cell', either, in case you want to fill out the full lattice of possibilities.

Thanks, I might request/add those on an ad-hoc basis.
Paola


Error on example code

Selenium extensions version: current, fresh install
Python version: 3.6.5
Operating System: MacOS 10.12 (Sierra)

Description
Tried to run example code (as written, with minor change of chromedriver location to /usr/local/bin
Got error.
What I Did
$ python test.py
Traceback (most recent call last):
  File "test.py", line 18, in <module>
    bot.goto_google()
  File "test.py", line 14, in goto_google
    self.populate_text_field(searchbox_locator, 'query')
  File "/Users/scohn/.pyenv/versions/3.6.5/lib/python3.6/site-packages/selenium_extensions/core.py", line 211, in populate_text_field
    input_element.send_keys(text)
  File "/Users/scohn/.pyenv/versions/3.6.5/lib/python3.6/site-packages/selenium/webdriver/remote/webelement.py", line 350, in send_keys
    'value': keys_to_typing(value)})
  File "/Users/scohn/.pyenv/versions/3.6.5/lib/python3.6/site-packages/selenium/webdriver/remote/webelement.py", line 499, in _execute
    return self._parent.execute(command, params)
  File "/Users/scohn/.pyenv/versions/3.6.5/lib/python3.6/site-packages/selenium/webdriver/remote/webdriver.py", line 297, in execute
    self.error_handler.check_response(response)
  File "/Users/scohn/.pyenv/versions/3.6.5/lib/python3.6/site-packages/selenium/webdriver/remote/errorhandler.py", line 194, in check_response
    raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.WebDriverException: Message: unknown error: call function result missing 'value'
  (Session info: headless chrome=65.0.3325.181)
  (Driver info: chromedriver=2.33.506106 (8a06c39c4582fbfbab6966dbb1c38a9173bfb1a2),platform=Mac OS X 10.12.6 x86_64)

Never mind, turned out to be older version of Chromedrive causing the issue.  Works now.


Fix slice panic

avoids crash mentioned in #267
(do not know if this is a fix)

Hey @FaKod I didn't realize this was merging into the release branch not master, could you repush with a PR to master?

@stevesloka Sorry, my bad. -> #282


Inconsistency in annotation form

Hi, I've noticed that action annotation have different keys in different datapoints:
In most samples, the annotation of action events is defined in "events": [{"startTime":,"endTime":, }], which is the same as specified in the instructions file.
However, in some datapoints(for example, "ZY20210800003/H3/C13/N39/S140/s04/T1/"), the annotation is given in "markResult":{"marks":[{"hdTimeStart":,"hdTimeEnd":}]. There are dozens of datapoints annotated like this. FYI, I download the dataset from Baiduyun link.
I'm a little bit confused and wonder if it's a mistake or if I understood it wrong.
Could you check it please? Thanks, looking forward to your reply.

Hi liy1shu,
I'm sorry for the ambiguity of data definition. The action annotation is stored in "markResult" for some videos. The usage for these annotations is the same as which for the annotation stored in "events".
We recommend to use the data packaged in our ActionSeg Challenge (https://www.hoi4d.top/).


The X.Org Endless Vacation of Code (EVoC)

If you are adding a SoC/WoC, please make sure

[ ] You have added the SoC/WoC in README.md file in correct table
[ ] You have added the SoC/WoC in index.html in correct table
[ ] ensure Website is tested on a deployment and looks okay

If you are fixing an issue please add

[ ] fixes issue #issue_number to PR description

Some of the internship present in the readme doesn't have a timeline, am adding the timeline or shouldn't i?

This is GSoC timeline I think https://summerofcode.withgoogle.com/how-it-works/#timeline.

The dotted has no timeline, but all has a standard timeline. I was trying to add it, shouldn't i?

ok go for it! :) Sorry my bad. :zap:

ok go for it! Sorry my bad. ⚡

ok go for it! My bad. ⚡

Ok 😊


Micro-ros agent create_participant DDS error

Issue template

Hardware description: STM32
RTOS: freeRTOS
Installation type: micro_ros_setup
Version: foxy

Steps to reproduce the issue
I was following this guide on https://micro.ros.org/docs/tutorials/core/first_application_rtos/freertos/
I connected the olimex and my pc to a switch via ethernet cables and an ethernet to USB converter
I ran theses commands:
ros2 run micro_ros_setup configure_firmware.sh int32_publisher -t udp -i <IP_ADDRESS> -p 8888
ros2 run micro_ros_setup build_firmware.sh
ros2 run micro_ros_setup flash_firmware.sh
docker run -it --rm --net=host -v /dev/shm:/dev/shm --privileged microros/micro-ros-agent:foxy udp4 --port 8888 -v6
Expected behavior
No DDS error
Actual behavior

Apologies for bumping an old thread, but I just ran into this problem myself, and it took me quite some time to figure out the Agent was not compatible with the version of the Client running on the device.
Would there be some way for the Agent to print a more informative error message, or something on the client side?
The client reported something like "Issues running micro XRCE-DDS session", which is rather generic as well.

Yes, probably your client is using the binary entity creation mode and the agent is an old version that only supports XML and references.
I'll check how to include an error message that alerts about this.
Thanks.


Separate train and test augmentation

Hello,
in some settings it might be useful to use other image augmentation methods for training than for testing. E.g. in order to make a model rotational invariant, rotating during training might be useful, whereas at test time it might not be necessary.
If you find this a useful contribution for the main branch, i'll create a PR dealing with this issue.

Hi @alexriedel1, yes, that would be really useful in certain usecases!

should I assign this issue to you then?

should I assign this issue to you then?

yes, thats fine


 Removing them, we once again ease the code with useless boilerplate.

@tetienne I will have a look later today, this is something I will need to test a bit more. I am not 100% sure what the previous code did, since this is coming from the original component.

@iMicknl Normally the data I removed are not used in the cover behavior. They are here only to be used as HA attributes where you can put what you want.

I don't have covers and can't really appreciate those changes, so I put my faith in you :)

@tetienne true, but I am not sure if those value are used by automations of people using the current component. I have never used those values, thus I would need to see if this is something where people can depend on.
However, I am up for making the cover platform simpler, since there is still some legacy inside.

I don't see any reason to use these weird variables. It's not something familiar within HA.


Improve SSL setup documentation for real certs

Automated message from Dropbox CLA bot
@timabbott, it looks like you've already signed the Dropbox CLA. Thanks!

Merged.


Fix spec links in error reporting to use the appropriate spec version

Currently links are to v1.0 regardless of the object or storage root version, e.g.
ocfl-py> ./ocfl-validate.py fixtures/1.0/bad-objects/E025_wrong_digest_algorithm
[E001a] OCFL Object root contains unexpected file: inventory.json.md5 (see https://ocfl.io/1.0/spec/#E001)
...
OCFL v1.0 Object at fixtures/1.0/bad-objects/E025_wrong_digest_algorithm is INVALID

ocfl-py> ./ocfl-validate.py fixtures/1.1/bad-objects/E025_wrong_digest_algorithm
[E001a] OCFL Object root contains unexpected file: inventory.json.md5 (see https://ocfl.io/1.0/spec/#E001)
....
OCFL v1.1 Object at fixtures/1.1/bad-objects/E025_wrong_digest_algorithm is INVALID

Now:
> ./ocfl-validate.py fixtures/1.0/warn-objects/W004_uses_sha256
[W004] OCFL Object root inventory SHOULD use sha512 but uses sha256 as the DigestAlgorithm (see https://ocfl.io/1.0/spec/#W004)
OCFL v1.0 Object at fixtures/1.0/warn-objects/W004_uses_sha256 is VALID
>
> ./ocfl-validate.py fixtures/1.1/warn-objects/W004_uses_sha256
[W004] OCFL Object root inventory SHOULD use sha512 but uses sha256 as the DigestAlgorithm (see https://ocfl.io/1.1/spec/#W004)
OCFL v1.1 Object at fixtures/1.1/warn-objects/W004_uses_sha256 is VALID


Delete relay agent from target cluster while cluster deletion

What does this PR change?

Deletes relay agent from target cluster while cluster deletion

Does the PR depend on any other PRs or Issues? If yes, please list them.

No

Checklist
I confirm, that I have...

[x] Read and followed the contributing guide in CONTRIBUTING.md
[ ] Added tests for this PR
[x] Formatted the code using go fmt (if applicable)
[ ] Updated documentation on the Paralus docs site (if applicable)
[x] Updated CHANGELOG.md

@mabhi for your verification

Codecov Report

Merging #120 (cbc6368) into main (cd55d2d) will decrease coverage by 0.03%.
The diff coverage is 2.08%.

@@            Coverage Diff             @@
##             main     #120      +/-   ##
==========================================
- Coverage   34.48%   34.45%   -0.04%
==========================================
  Files          73       74       +1
  Lines       11573    11573
==========================================
- Hits         3991     3987       -4
- Misses       7048     7052       +4
  Partials      534      534

Impacted Files
Coverage Δ

pkg/event/event.go
0.00% <ø> (ø)

pkg/sentry/authz/authz.go
0.00% <0.00%> (ø)

pkg/service/deleterelayservice.go
0.00% <0.00%> (ø)

pkg/service/cluster.go
31.66% <8.33%> (+0.77%)
:arrow_up:

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


chore: add script to add "Edit this page" button in User Guide

Description

make PRs easier

After generation of the User Guide.
$ user_guide_src/add-edit-this-page user_guide_src/build/html/

"Edit this page" buttons are added in the HTML files.

Click the button, go to GitHub edit page.

Checklist:

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

@paulbalandan I've changed the way of outputting filenames.
And squashed the commits.
Sorry to be difficult to review.


Implement /staticroutes post request

implement the /staticroutes post request!
https://github.com/kubeshop/kuskgateway-api-server/pull/38

I see this has a merged pull request. Is this done?

@crjones this has been completed


Fix: no match controller version

Description of your changes

🤖 Generated by Copilot at a492ee5
Summary
🆕🎛🧪

Add controller version check for application reconciliation. This feature lets users choose the controller version for their applications by setting the app.oam.dev/controller-version annotation on the Application resource.

controller-version
a new annotation for apps
spring of OAM features

Walkthrough

Add a condition to check controller version requirement (link)

fix: https://github.com/kubevela/kubevela/issues/6090

Fixes #
I have:

[ ] Read and followed KubeVela's contribution process.
[ ] Related Docs updated properly. In a new feature or configuration option, an update to the documentation is necessary.
[ ] Run make reviewable to ensure this PR is ready for review.
[ ] Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested

Special notes for your reviewer

/retest go

I left some comment on the issue this PR related. The implementation way might not be correct for fixing the issue.


go get problem

Hi!
This is a bug in the read.me.
I am following the install and get this warning:
Go-> ~/go/src/github.com: go get -v -u github.com/okkur/reposeed/cmd/reposeed/...
github.com/okkur/reposeed (download)
go: warning: "github.com/okkur/reposeed/cmd/reposeed/..." matched no packages
Go-> ~/go/src/github.com: go version
go version go1.12.5 darwin/amd64

This works:
go get -u -v github.com/okkur/reposeed/cmd/...

Thanks for reporting. We were reworking some things within reposeed and wanted to move towards a vanity url to pull, which should also fix the README issue. Happy for you to create a PR to fix this specific issue until we get the iteration working.

That's ok. No need for a PR. I'll check back in a while.

We now have a vanity url: go get -u go.okkur.org/reposeed
Changing the docs right now.
Happy for more feedback and even PRs in the future.


Is it possible to add the multiple fragments?

Is it possible to add the multiple fragments in skackview?

I don't think I understand. Can you elaborate a little bit more on that?


Naive ort update to fix Travis CI failure (#2844)

Cherry-pick of https://github.com/pytorch/vision/pull/2844

@seemethere this PR just makes CI green for ONNX by updating the version used during testing. It's not a bugfix for the release

this PR just makes CI green for ONNX

Since the ONNX tests aren't failing on the CI right now, I'll close this PR. Please feel free to re-open if I'm missing something.
Thanks @vfdev-5 !


Editor toolbar inline menu: paragraph style

Will you remove the feature of editor toolbar inline menu?
An inline style selection does what expected. A  paragraph style selection does not have an effect as the class is not set.

This helps:

If editor toolbar inline menu will still be a feature, I would be happy and I would place a PR. What are your plans with Slate?

@ksuess It's a bug. I don't plan on removing that feature from Slate, it's an optional addon, it doesn't bother anyone.


Application for code ownership

This is my formal application for code ownership

Actually, @stephanwampouille can you adjust this to exclude probes for now. Keep that with me


Hamiltonian cycle in cpp, java, python

🛠 Issue #1058

Issue no #1058
👨💻 Changes proposed

✔ Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

📄 Note to reviewers

📷 Screenshots

This issue was assigned in C, C++ and Python, so kindly implement in these languages only

@Kumar-laxmi ok

This issue was assigned in C, C++ and Python, so kindly implement in these languages only

@Kumar-laxmi Changes have been done.

You can give same name to all program files

@Kumar-laxmi Changes have been done.

You can give same name to all program files

@Kumar-laxmi Changes have been done.

@Kumar-laxmi please check it already pass the 3 approval.


TypeError: The "path" argument must be of type string or an instance of Buffer or URL. Received undefined

Extension
https://www.raycast.com/asubbotin/shell
Raycast Version
1.79.1
macOS Version
14.5 (23F79)
Description
Error:
TypeError: The "path" argument must be of type string or an instance of Buffer or URL. Received undefined

Bn:index.tsx:276:41

---
273: }

### Steps To Reproduce

Uninstalled extension and then reinstalled. Same error

### Current Behaviour

Get a "something went wrong" error when trying to run command.

### Expected Behaviour

Should run a command.

@dtomasch sorry, I've tried to reproduce but I'm not able to get the same error with the steps you said.
Could you provide more info?

I wish I could — the error was happening all day today and has now vanished so I can't provide any more details.
I guess I'll close this.


Rarefaction diversity switched mean and quantile.

🐛 Bug
Hi there,
I stumbled upon a bug in the rarefaction diversity estimation.
In
https://github.com/immunomind/immunarch/blob/9dce2ea78f4fb15557e17664910f14c47caa8e63/R/diversity.R#L409-L410
the "Mean" and Quantile columns are wrongly named. The column "Mean"  should be the second column in my opinion.
Also, as far as I can tell, the quantile seems to be hardcoded in
https://github.com/immunomind/immunarch/blob/9dce2ea78f4fb15557e17664910f14c47caa8e63/R/diversity.R#L402
although it is possible to specify custom quantiles.
To Reproduce
Steps to reproduce the behavior:
data(immdata)
test_raref <- immunarch::repDiversity(immdata$data, .method = "raref", .col = "aa")
test_raref

The "mean" is always lower than the the "Q0.025".
Expected behavior
The "mean" should be between the quantile values.

Hello @TobiTekath  and thank you so much for signing up on GitHub to open an issue, I truly appreciate it!
I'm deeply sorry for the very late response. We've been extremely busy raising investments and hiring the team after finishing UC Berkeley startup accelerator. We are finally set up for now: we grew the team from 4 to 15 amazing professionals, and raised a large investment round from great investors. Thank you so much for your patience!
We made changes to the Immunarch code to fix the issue you described. Please update to the latest development version of the package and let us know how it works. To install the development version, execute the following code:
devtools::install_github("immunomind/immunarch", ref="dev")
Improving upon Immunarch, we will soon release our online analytics platform for fast, effortless and code-less reproduction of impactul single-cell transcriptomics and immune repertoire publications. It's designed for researchers who spend a lot of time or energy reproducing sophisticated pipelines or analysing datasets from publications. Feel free to send us a link to publications of your interest so we can reproduce them on our platform and give you an access.
We are more than happy to grant you an early access as our gratitude for you being a longstanding immunarch user.
We have a list of datasets from around 30 publications that we are going to make available for your exploration and analysis. As an example, we recently reproduced on a platform a paper focused on scRNAseq-based estimation of efficacy of CAR-T immunotherapy.
Please send links to<EMAIL_ADDRESS>The direct link to the publication datasets would be also greatly appreciated.


Update release note 1.13

opendistro-for-elasticsearch/security pull request intake form
Please provide as much details as possible to get feedback/acceptance on your PR quickly

Category: (Enhancement, New feature, Bug fix, Test fix, Refactoring, Maintenance, Documentation)
Maintenance

Github Issue # or road-map entry, if available:

Description of changes:
Update the release note 1.13 with the newly merged changes.
Note that this PR itself will be included in the update to be included in the release 1.13 release note

Why these changes are required?
Keep release note updated.

What is the old behavior before changes and new behavior after changes? (Please add any example/logs/screen-shot if available)
No changes in behavior.

Testing done: (Please provide details of testing done: Unit testing, integration testing and manual testing)
UT

TO-DOs, if any: (Please describe pending items and provide Github issues# for each of them)

Is it backport from master branch? (If yes, please add backport PR # and commits #)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

I don't see need for this, it is not critical for 1.13 release. If there are any other critical issues required for 1.13, we will include the latest commits from master as well.

I don't see need for this, it is not critical for 1.13 release. If there are any other critical issues required for 1.13, we will include the latest commits from master as well.

You mean Fix AuthCredentials equality ([#876](https://github.com/opendistro-for-elasticsearch/security/pull/876)) is not a must-have in release 1.13? If you are not strongly aganst including this one, I'd keep it since it's already been in the master. I'm about to cut the git tag for v<IP_ADDRESS>-rc1 as soon as this release note updates get merged.
IMO, the other 2 commits should be included since they are related to 1.13.

It is OK to keep the commit on the master, but it is not required for 1.13 release. Please tag 87ff85af3ac3f6b7929ad3a16bca8fd5c10c73ba as 1.13 RC.

It is OK to keep the commit on the master, but it is not required for 1.13 release. Please tag 87ff85a as 1.13 RC.

Done.


interesting

etherium long time planning to upgrade to 2.0, but in theory, if it happens, what will happen to SilentETHminer? Will the project be closed?

You would still be able to mine other Ethash coins or ETC.


Add env object to WASIContext to pass an externally defined Memory object into the WebAssembly context

My first typescript code. My type-fu is not great. Feel free to suggest alternatives.

Hey mate! Thanks for the PR.
This is a great suggestion! Being able to specify the Memory is important.
I don't want to mix up the env variables for WASI with the memory to give to WebAssembly.
I think a refactor is in order to make the interface better match the NodeJS WASI interface.

I think a refactor is in order to make the interface better match the NodeJS WASI interface.

I'm happy to refactor. Can you be a bit more specific? Not sure I'm understand. I'm not familiar with the NodeJS WASI interface. is that any different than the wasi sdk?
I'm building C to WASM and running in a browser environment. Can see how I'm building and linking. It looks the --import-memory flag of the linker somehow binds to the env.memory but still learning how all the moving parts work

So I think the confusion here is between env in WASI and env in WebAssembly. In WASI env represents the environment variables to a CLI command (like a dotenv file). In WebAssembly env is things like memory (what you're passing through).
The NodeJS API makes this more clear because you create a WASI instance and then pass it to the WebAssembly instantiate method. https://nodejs.org/api/wasi.html
So in that API you would be able to specify the memory during the WebAssembly Instantiate and not how Runno forces you to do it (from the WASI layer).
My goal with a refactor would be to better match that NodeJS API.

After looking at the changes properly, I'm a bit confused about what you're trying to achieve here. I thought you were trying to specify a custom memory import, but it looks like you're overriding the memory that Runno writes to.
What problem are you trying to solve?

Could you please look at the documentation in my refactor: https://github.com/taybenlor/runno/pull/278
Does that achieve what you want?

After looking at the changes properly, I'm a bit confused about what you're trying to achieve here. I thought you were trying to specify a custom memory import, but it looks like you're overriding the memory that Runno writes to.
What problem are you trying to solve?

The application might wanna define and have control over the memory. Might be JS code that wants to manipulate it.

Okay so with the changes I just released in 0.6.0 you can specify your own memory instance when initialising the WebAssembly runner. Check out packages/wasi/README.md for instructions.
If you'd just like to access the memory object exported by the binary, that is available on the WASI instance when it is instantiated with new WASI().
Have a look at the PR referenced above for what I mean.
Does that solve your problem?

Could you please look at the documentation in my refactor: #278
Does that achieve what you want?

Tried the PR and get the error:
Uncaught TypeError: WebAssembly.instantiate(): Import #0 module="env" error: module is not an object or function
My WASM code expect the memory passed through the env object. I don't have control over it since that comes imposed from the wasi-sdk and compiler / linker (clang - llvm) when using the --target=wasm32-wasi option. Don't see a way to change that.
Ideally and for max flexibility I would want control on what it's passed to WebAssembly.instantiateStreaming (another need I'll have soon is to pass an object with JS functions to be called from wasm. that object can have any arbitrary name) but I imagine It doesn't fit with the runno abstraction. Not sure what the right solution is. Still in the process of understanding all the moving parts.

And that's using the method I've outlined in the README?
// Then instantiate your binary with the imports provided by the wasi object
const wasm = await WebAssembly.instantiateStreaming(
  fetch("/binary.wasm"),
  {
    ...wasi.getImportObject(),

    // Your own custom imports (e.g. memory)
    memory: new WebAssembly.Memory({ initial: 32, maximum: 10000 });
  }
);

I don't understand why the memory would be passed through the env variable in the context. That is for environment variables (https://en.wikipedia.org/wiki/Environment_variable#:~:text=An environment variable is a,in which a process runs.) which can only be strings.

And that's using the method I've outlined in the README?
// Then instantiate your binary with the imports provided by the wasi object
const wasm = await WebAssembly.instantiateStreaming(
  fetch("/binary.wasm"),
  {
    ...wasi.getImportObject(),

    // Your own custom imports (e.g. memory)
    memory: new WebAssembly.Memory({ initial: 32, maximum: 10000 });
  }
);

I don't understand why the memory would be passed through the env variable in the context. That is for environment variables

I tried your refactor and in the doc you gotta do.
     const wasm = await WebAssembly.instantiateStreaming(
     ...
     env: {memory: new WebAssembly.Memory({ initial: 32, maximum: 10000 }
    .... )

Looking at my wasm code I see:
memory $env.memory (;0;) (import "env" "memory")

I see the same error in the context of emscripten so not exclusive to the wasm-sdk toolchain.
Also need a reference to the passed memory (as I did in my PR). Notice that if the memory is externally defined it's not longer in part of the instance.exports object:
https://github.com/taybenlor/runno/blob/1fe106293daa3a640231980b98259cbd378a3268/packages/wasi/lib/wasi/wasi.ts#L88

Ahhh okay, that makes sense. Thanks for the extra context, super helpful!
I've just merged another PR which should fix those issues for you: https://github.com/taybenlor/runno/pull/281
Feel free to file a bug or discussion topic if you have any other problems.
Cheers!

Thanks for the patience. It works now! 👏

Yay!! Thanks for the feature request.


[BUG] Not able to cdk deploy . Error Insufficient Lake Formation permission(s): Required Create Database on Catalog (Service: AWSGlue; Status Code: 400

Describe the bug
Filling out this field will help us investigate the issue efficiently. Providing detailed information allows us to set the appropriate priority. We appreciate your cooperation.
I created an amazon linux 2023 instance and installed docker, npm, cdk etc.
I am trying to do a cdk deploy with the latest code .
The "cdk bootstrap aws://<ACCOUNT_ID>/us-east-1" ( with the right 12 digit value of the ACCOUNT_ID subsituted) runs fine
However "cdk deploy --require-approval never --all"  errors out
7:29:24 PM | CREATE_FAILED        | AWS::Glue::Database                             | UsageAnalysisDatabase203EEF4F
Insufficient Lake Formation permission(s): Required Create Database on Catalog (Service: AWSGlue; Status Code: 400; Error Code: AccessDeniedException; Request ID: 1ddc191c-b7c8-4f5b-a035-7da398c28263; Proxy: null)
To Reproduce
Filling out this field will help us investigate the issue efficiently. Providing detailed information allows us to set the appropriate priority. We appreciate your cooperation.
Steps to reproduce the behavior:

check out the latest code
run the commands as listed in the "Deploy with CDK " section
There is no mention of that lakeformation should be activated anywhere in the notes
In my account LakeFormation is already activated in us-east-1.
I added the role used in the ec2-instance is added as an LakeFormation administrator  with permission to create database as well as data lake administrator.
I got the error
7:29:24 PM | CREATE_FAILED        | AWS::Glue::Database                             | UsageAnalysisDatabase203EEF4F
Insufficient Lake Formation permission(s): Required Create Database on Catalog (Service: AWSGlue; Status Code: 400; Error Code: AccessDeniedException; Request ID: 1ddc191c-b7c8-4f5b-a035-7da398c28263; Proxy: null)

Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

To overcome this error  from cloudtrail , i found the eventname="createdatabase" and found the userName = cdk-hnb659fds-cfn-exec-role-ACCOUNT_ID-us-east-1 . Then I added that to the LakeFormation permissions for cdk-hnb659fds-cfn-exec-role-ACCOUNT_ID-us-east-1 as a database creator and it allowed the deployment to continue.

In my account LakeFormation is already activated in us-east-1.

As you mentioned, this looks the root cause. Doc says:
The user/role must be a data lake administrator.

cdk-hnb659fds-cfn-exec-role-ACCOUNT_ID-us-east-1

This role is used to provision resources on your account, so your workaround looks reasonable.

There are 2 options #1. update the instruction to just add this user as a data lake administrator and retry
#2. split the cdk deploy into 2 segments  where the user "cdk-hnb659fds-cfn-exec-role-ACCOUNT_ID-us-east-1" is created in 1st segment and add a step in between where user has to add it manually and then run the 2nd part of the cdk deployment.

@vishallakhotia Thank you for detailed workaround. let us close this issue


I can't get into the discord thread.

Because I don't have an invitation to it.

Please join the Atlyss discord to access this thread.
Atlyss Discord
I will add it alongside the thread link.


Use Multiple CSS Themes

If you are reporting a bug or requesting support, start here:
I am writing stories for a component library where it is possible to use multiple CSS themes to style the same component. I would like to know if there's a way to use the
Here are my two stories for the same button (with different CSS Themes)
import React from 'react';
import { storiesOf } from '@storybook/react';
import { action } from '@storybook/addon-actions';
import { Button } from '../../src/v2/components/Button';
import '../../dist/themes/ant.css';

storiesOf('ant/Button', module)
  .add('Default Button', () => <Button onClick={action('clicked')}>Default Button</Button>);

import React from 'react';
import { storiesOf } from '@storybook/react';
import { action } from '@storybook/addon-actions';
import { Button } from '../../src/v2/components/Button';
import '../../dist/themes/partner-portal.css';

storiesOf('partner-portal/Button', module)
  .add('Default Button', () => <Button onClick={action('clicked')}>Default Button</Button>);

When I load these stories, the two CSS files are bundled together and the styles of the story which is imported later take precedence.
So, I was wondering if it's possible to use different CSS themes in storybook. And if yes, how would I go about achieving it.
Please specify which version of Storybook and optionally any affected addons that you're running

@storybook/react 3.4.8

We decided to simply spin different static websites for each theme (we merely want to provide a showcase for our components).
Hopefully we can have the possibility of switching themes later.


result of kitti sequence 00 and 10 is worse than A-LOAM

I run your code in kitti sequence 00 and 10, I found the result was worse than A-LOAM
how about your result?

the white trajectory is groundtruth, and the worse trajectory is produced by lego-loam, the better trajectory is produced by A-LOAM

From my experience, generally, A-loam had better performance compared to the LeGO loam with Kitti data set


Move deprecations along

@jmdavis I created a PR to fix Dscanner -> https://github.com/dlang-community/D-Scanner/pull/761
We can bump the dscanner hash in posix.mak once this has been merged and then this PR should pass.

@wilzbach Thanks!

Bumped DScanner to https://github.com/dlang-community/D-Scanner/commit/e5a73fa0ef0bf790a56fbcb34df407878454c965 which hopefully makes this PR pass.

Okay, so this three remaining projects fail:

[ ] dlang-bot: https://github.com/dlang/dlang-bot/pull/230 - a bit tricky because upgrading to a newer vibe.d breaks the test suite
[ ] dls (the problem is that it uses dub 1.15 and the original dub PR was against master. Hence, I cherry-picked it to stable in https://github.com/dlang/dub/pull/1710, s.t. the next point release contains this fix).
[ ] ae (CC @CyberShadow)

sys/install/common.d(202,27): Error: module `digest` is in file 'std/digest/digest.d' which cannot be read


Crash in 3.0 & 3.1 exporting glb of animated model

Describe the bug
Crash exporting to gltf with animated rigged model.
To Reproduce
Animate a rigged model (Minion model from turbosquid: https://www.turbosquid.com/3d-models/3d-model-minion/1005881)
Export as glb
Blender crashes
Expected behavior
GLB is created
Version
blender-3.0.0-beta+v30.945a99386b8c-linux.x86_64-release
Also happens on Windows 3.0beta & 3.1alpha
minionrig_anim.crash.txt

Hello,
The model is a non free model from turbosquid, so we can't use it to reproduce the error.
Seems this is a blender crash, not a python error, so it may make it difficult to find the root cause.
So, some question to help find where is the issue:

Did you try to export from another computer, with another installation of Blender?
Do you have issue when trying to export another model (for example the Robot from this ticket:
Did you try to export with 2.93 version?

Hi,
I can probably get you a copy of the model for debugging purposes. How best to contact you?

I've tried it under Windows and Linux. It also happens on another PC.
Exporting RobotExpressive works ok
I get the issue as mentioned here: https://blender.stackexchange.com/questions/207840/export-to-gltf-fails-due-to-child-data-shape-keys-path-resolve-error - which is fixed (if I understand correctly) in 3.0.0 (https://github.com/KhronosGroup/glTF-Blender-IO/issues/310)
Thx!

Hi,

How best to contact you?

Please send me the file or a link on the following email: contact (AT) julienduroure [DOT] com

You have a non valid driver on the model:

Remove it, and export is no more crashing.
I will add a check to avoid crash on invalid drivers.
Here is a simple file to reproduce on 3.0+
bug_invalid_driver.blend.zip
Note that there is no crash on blender 2.93 with current master branch

Thankyou for looking in to this. Really appreciate your help and time!


Automated tests

https://docs.unrealengine.com/en-us/Programming/Automation

Added in commit cc457baa8e841914079ca6f88729d502baf01b81.


Maintain schema meta locally or support merge local config with upstream config.

Currently, lsp setting schema very dependent on upstream updates, some schema not exist for some reason, eg: gopls, but I try update in my machine, and no error as upstream says.
So, maintain schema meta locally is a flexible way, not bound by upstream.

@tamago324 any thoughts ?


Update 386.txt

The 2 operators names have changed, Mobitel to Telekom Slovenije and Tušmobil to Telemach.

I signed it!

@penmetsaa
Only those with write access to this repository can merge pull requests.

Oh ok. Sorry!
I have merged it now :)

Thnx man ;)


lodash: signatures of the method _.ceil changed

https://lodash.com/docs#ceil
https://lodash.com/docs#_ (description of the chainable wrapper)

thanks mate!


Release v1.10.0-alpha.1 Tracking

Kubernetes v1.10.0-alpha.1 has been built and pushed.
The release notes have been updated in CHANGELOG-1.10.md with a pointer to it on github
cc

Kubernetes v1.10.0-alpha.1 has been built and pushed.
The release notes have been updated in CHANGELOG-1.10.md with a pointer to it on github

Kubernetes v1.10.0-alpha.1 has been built and pushed.
The release notes have been updated in CHANGELOG-1.10.md with a pointer to it on github

testing.


The SSL connection could not be established, see inner exception.

@qcjxberin
我遇到了同样的问题，调试后发现 Inner Exception 是：远程主机强迫关闭了一个现有的连接。主要原因是 https://raw.githubusercontent.com/Redth/dotnet-maui-check/main/manifests/maui-preview.manifest.json 这个文件获取失败。
解决办法是上VPN。

I have this too. Visual Studio 2022 Preview on Windows Server 2022 preview.
Is there a solution?

You can just download https://raw.githubusercontent.com/Redth/dotnet-maui-check/main/manifests/maui.manifest.json locally and then use:
maui-check -m your-downloaded-file

You can download two files
file1: https://raw.githubusercontent.com/Redth/dotnet-maui-check/main/manifests/maui.manifest.json
file2: https://raw.githubusercontent.com/dotnet/maui/release/6.0.1xx-preview5/eng/Versions.props
Replace https://raw.githubusercontent.com/dotnet/maui/release/6.0.1xx-preview5/eng/Versions.props in file1 with the local file path of file2.
maui-check -m file1

Tanx @albertwoo Worked for me

Thank you very much @albertwoo

Thank you


Np 22588/01 new fields to customer

That depends on AWS as CustomerHandler<CustomerDto> allready have been deserialized by AWS, by the time
 protected CustomerDto processInput(CustomerDto input, RequestInfo requestInfo, Context context) this runs

That is not correct. AWS gives us a com.amazonaws.services.lambda.runtime.RequestStreamHandler, and we do all the serialisation and deserialisation in nva.commons.apigateway.RestRequestHandler<I, O> and nva.commons.apigateway.ApiGatewayHandler<I, O>. But ignore testing it if you want. But check manually afterwards to make sure we generate an appropriate response :)


.gitignore updated and .DS_Store removed

I have updated the .gitignore file adding macOS files that are not necessary to push to the origin (e.g DS_Store).

Are there reasons why this PR is still in pending?

@filograno sorry for taking so long. Thanks for contributing 👍


Redirection of IPFS blogpost to local gateway leads to download (of the HTML page)

Describe the bug
I tried to open one of the most recent blogposts on the ipfs blog which was redirected to my local gateway by my ipfs browser plugin. My browser then started to download the HTML file, instead of rendering it.
To Reproduce
Steps to reproduce the behavior:

Open this URL

Expected behavior
Rendering of webpage
Screenshot

Desktop (please complete the following information):

OS: ArchLinux
Browser Brave
Browser version: 1.8.96 Chromium: 81.0.4044.138 (Official Build) unknown (64-bit)
Plugin version: 2.11.0

Opening the url http://blog.ipfs.io.ipns.localhost:8080/2020-05-20-gossipsub-v1.1 with Firefox(78.0a1) or Chromium (83.0.4103.61) without IPFS Companion installed downloads the page too, so that could be unrelated to redirection.
My local IPFS daemon (0.6.0-dev/53739faee) seems to be responding with Content-Type: application/x-troff-man. ipfs.io responds with text/html type for the same URL.

I suspected this is unexpected side-effect of recent change in go-ipfs:
https://github.com/ipfs/go-ipfs/pull/7262  but I was unable to reproduce this with latest master of go-ipfs:
$ docker run --rm -it --net=host ipfs/go-ipfs:master-2020-05-25-707b550
...
Daemon is ready

local gateway returns text/html
$ curl -sD - http://blog.ipfs.io.ipns.localhost:8080/2020-05-20-gossipsub-v1.1 | grep -i content-type
Content-Type: text/html

@RubenKelevra @sinkuu I assume you both run bleeding edge – are you able to update to the latest master of go-ipfs and check if the issue is still there?

Upgraded to https://github.com/ipfs/go-ipfs/commit/707b5506ba285432e6a628888574b3380d3cd23d, still the same. This behavior may be system dependent (I'm using ArchLinux), because mime.TypeByExtension function utilizes local mime.types entries.
On ArchLinux:
application/x-troff-man                            man 1 2 3 4 5 6 7 8

On Ubuntu (https://packages.ubuntu.com/focal/mime-support):
application/x-troff-man				man

Thank you @sinkuu, I was able to reproduce by updating /etc/mime.types on my Linux box.
This confirms its go-ipfs issue, unrelated to ipfs-companion – I'm transferring this issue to respective repo.

@Stebalien @hsanjuan @gowthamgts I am unsure if this is a bug or a feature introduced in https://github.com/ipfs/go-ipfs/pull/7262, but this type of OS-dependent / nondeterministic behavior feels like a bug to me.
Perhaps we should disable use of OS-level /etc/mime.types and instead ship implicit list with go-ipfs itself? In needed, we could support overrides via $IPFS_PATH/mime.types, but the default content-types should be deterministic, and not depend on OS-level configs.

I agree that disabling OS level mime type checking is a good solution.
Note: Ultimately, the problem was that we weren't generating posts as /foo.1/index.html. Instead, we were outputting files like foo.1. See https://github.com/ipfs/blog/issues/423.

I've filed https://github.com/ipfs/go-ipfs/issues/7418 to summarize this issue.

Thank you!
This issue was originally filled about inability to open http://blog.ipfs.io.ipns.localhost:8080/2020-05-20-gossipsub-v1.1 and that bug was fixed by https://github.com/ipfs/blog/issues/423 (I am no longer able to replicate), so I am closing this as fixed.
Impact of /etc/mime.types on Content-Type returned by go-ipfs continues in https://github.com/ipfs/go-ipfs/issues/7418


Roll Engine from 2d8b3571d074 to 07ba8fd075d6 (6 revisions)

https://github.com/flutter/engine/compare/2d8b3571d074...07ba8fd075d6
2021-10-09<EMAIL_ADDRESS>[fuchsia] Add Launcher & Resolver to Dart JIT CMX. (flutter/engine#29091)
2021-10-09<EMAIL_ADDRESS>Update comments (flutter/engine#29088)
2021-10-08<EMAIL_ADDRESS>Roll Skia from 89457e9237fc to 6030e0a2c5bd (1 revision) (flutter/engine#29089)
2021-10-08<EMAIL_ADDRESS>Reland Android systrace (flutter/engine#29080)
2021-10-08<EMAIL_ADDRESS>Roll Skia from cd2f207a2e3f to 89457e9237fc (1 revision) (flutter/engine#29087)
2021-10-08<EMAIL_ADDRESS>Roll Skia from 8fa6dbffff50 to cd2f207a2e3f (1 revision) (flutter/engine#29085)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


HTTPS alpha interface + stopgap implementation

I think I do not understand spans completely yet, but currently there isn't enough time for me to figure it out. We probably shouldn't take both response_buffer and out_response as parameters.

There should now be powershell on mac and linux, so this might even work on other platforms, haha.

Will be obsolete soon


[ios][android] support return value for native module APIs someway

For example getItem(key) in storage module. It's more simple and familiar with web developers.
Of course from the perf aspect, this kind of API is suitable for non-frequent calls only.

It's a very very bad idea:
0. Break 'js-native' bridge.
0. Performance issue.
0. Nobody will like do IO operation in sync way.

Maybe we can refer to the design of JSON-RPC.
1)Notification. (no return)
2)Request and Response.

If this issue is resolved, then the timer api module will be valid to use again.
var timer = require('@weex-module/timer')
var id = timer.setTimeout(function() { /*none*/ }, 1000)
timer.clearTimeout(id)

even though the timer can be set and clear by calling setTimeout/clearTimeout directly, still we like the require way because the consistency of using 'require' to require a API module.
And maybe there occurs another using case in another day. I think the ability should be prepared to solve such problems.

The more valuable thing of this is we provide  module developers the ability of both mechanism for sync & async pattern. Not switch all module APIs into sync mode.
Take storage module for example, this is a very simple k-v pairs data storage, not necessary to implemented by a SQLite database and async/callback APIs. This could be more easily to implement and use.
Thanks.

This discussion is moved to #1677
Thanks.


[BUG] Unable to train on multiple GPUs in Sagemaker Notebook Terminal

[ X] I have checked that this bug exists on the latest stable version of AutoGluon
[ ] and/or I have checked that this bug exists on the latest mainline of AutoGluon via source installation

Describe the bug
Autogluon 0.4.0 TextPredictor training on p3.8xl 4-GPU instance in Sagemaker Notebook terminal, with env.num_gpus: 4 setting.  I get an error in spawning multiprocessing.  When I train with everything the same, but only on a single GPU within the same instance and setup, it trains without a problem.
Expected behavior
Train across all 4 GPUs in the p3.8xl instance with no errors.
To Reproduce

SageMaker Notebook p3.8xl instance
python 3.7.12.
pip install torch==1.10.0 autogluon.text==0.4.0 awswrangler pandas autofluon.features==0.4.0
python train.py

Code:
in train.py file

import pandas as pd
import awswrangler as wr
from autogluon.text import TextPredictor

args = Namespace(
    train_filename = "s3://ccds-asin-drc/eu/modeling-data/mf2_no_emb/train/0/train.parquet",
)

model_config = {
    "eval_metric": "accuracy",
    "time_limit": 60*60*3,
    "features": ['label', 'item_name_orig']
}

hyperparameters = {
    'model.hf_text.checkpoint_name': 'microsoft/mdeberta-v3-base',
    'optimization.top_k': 1,
    'optimization.lr_decay': 0.9,
    'optimization.learning_rate': 1e-4,
    'env.precision': 32,
    'env.per_gpu_batch_size': 4,
    'env.num_gpus': 4
}

train_df = wr.s3.read_parquet(args.full_train_filename)
print(train_df.info())

predictor = TextPredictor(
    label='label',
    eval_metric=model_config['eval_metric']
)

predictor.fit(
    train_data=train_df[model_config['features']],
    hyperparameters=hyperparameters,
    time_limit=model_config['time_limit']
)

Screenshots
Error:
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/home/ec2-user/anaconda3/envs/JupyterSystemEnv/lib/python3.7/multiprocessing/spawn.py", line 105, in spawn_main
    exitcode = _main(fd)
  File "/home/ec2-user/anaconda3/envs/JupyterSystemEnv/lib/python3.7/multiprocessing/spawn.py", line 114, in _main
    prepare(preparation_data)
  File "/home/ec2-user/anaconda3/envs/JupyterSystemEnv/lib/python3.7/multiprocessing/spawn.py", line 225, in prepare
    _fixup_main_from_path(data['init_main_from_path'])
  File "/home/ec2-user/anaconda3/envs/JupyterSystemEnv/lib/python3.7/multiprocessing/spawn.py", line 277, in _fixup_main_from_path
    run_name="__mp_main__")
  File "/home/ec2-user/anaconda3/envs/JupyterSystemEnv/lib/python3.7/runpy.py", line 263, in run_path
    pkg_name=pkg_name, script_name=fname)
  File "/home/ec2-user/anaconda3/envs/JupyterSystemEnv/lib/python3.7/runpy.py", line 96, in _run_module_code
    mod_name, mod_spec, pkg_name, script_name)
  File "/home/ec2-user/anaconda3/envs/JupyterSystemEnv/lib/python3.7/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/home/ec2-user/SageMaker/rubinome_labs/lab/202203_drc_multilingual/train_textonly.py", line 46, in <module>
    time_limit=model_config['time_limit']
  File "/home/ec2-user/myagenv/lib/python3.7/site-packages/autogluon/text/text_prediction/predictor.py", line 248, in fit
    seed=seed,
  File "/home/ec2-user/myagenv/lib/python3.7/site-packages/autogluon/text/automm/predictor.py", line 410, in fit
    enable_progress_bar=self._enable_progress_bar,
  File "/home/ec2-user/myagenv/lib/python3.7/site-packages/autogluon/text/automm/predictor.py", line 561, in _fit
    ckpt_path=self._ckpt_path,  # this is to resume training that was broken accidentally
  File "/home/ec2-user/myagenv/lib/python3.7/site-packages/pytorch_lightning/trainer/trainer.py", line 741, in fit
    self._fit_impl, model, train_dataloaders, val_dataloaders, datamodule, ckpt_path
  File "/home/ec2-user/myagenv/lib/python3.7/site-packages/pytorch_lightning/trainer/trainer.py", line 685, in _call_and_handle_interrupt
    return trainer_fn(*args, **kwargs)
  File "/home/ec2-user/myagenv/lib/python3.7/site-packages/pytorch_lightning/trainer/trainer.py", line 777, in _fit_impl
    self._run(model, ckpt_path=ckpt_path)
  File "/home/ec2-user/myagenv/lib/python3.7/site-packages/pytorch_lightning/trainer/trainer.py", line 1199, in _run
    self._dispatch()
  File "/home/ec2-user/myagenv/lib/python3.7/site-packages/pytorch_lightning/trainer/trainer.py", line 1279, in _dispatch
    self.training_type_plugin.start_training(self)
  File "/home/ec2-user/myagenv/lib/python3.7/site-packages/pytorch_lightning/plugins/training_type/ddp_spawn.py", line 173, in start_training
    self.spawn(self.new_process, trainer, self.mp_queue, return_result=False)
  File "/home/ec2-user/myagenv/lib/python3.7/site-packages/pytorch_lightning/plugins/training_type/ddp_spawn.py", line 201, in spawn
    mp.spawn(self._wrapped_function, args=(function, args, kwargs, return_queue), nprocs=self.num_processes)
  File "/home/ec2-user/myagenv/lib/python3.7/site-packages/torch/multiprocessing/spawn.py", line 230, in spawn
    return start_processes(fn, args, nprocs, join, daemon, start_method='spawn')
  File "/home/ec2-user/myagenv/lib/python3.7/site-packages/torch/multiprocessing/spawn.py", line 179, in start_processes
    process.start()
  File "/home/ec2-user/anaconda3/envs/JupyterSystemEnv/lib/python3.7/multiprocessing/process.py", line 112, in start
    self._popen = self._Popen(self)
  File "/home/ec2-user/anaconda3/envs/JupyterSystemEnv/lib/python3.7/multiprocessing/context.py", line 284, in _Popen
    return Popen(process_obj)
  File "/home/ec2-user/anaconda3/envs/JupyterSystemEnv/lib/python3.7/multiprocessing/popen_spawn_posix.py", line 32, in __init__
    super().__init__(process_obj)
  File "/home/ec2-user/anaconda3/envs/JupyterSystemEnv/lib/python3.7/multiprocessing/popen_fork.py", line 20, in __init__
    self._launch(process_obj)
  File "/home/ec2-user/anaconda3/envs/JupyterSystemEnv/lib/python3.7/multiprocessing/popen_spawn_posix.py", line 42, in _launch
    prep_data = spawn.get_preparation_data(process_obj._name)
  File "/home/ec2-user/anaconda3/envs/JupyterSystemEnv/lib/python3.7/multiprocessing/spawn.py", line 143, in get_preparation_data
    _check_not_importing_main()
  File "/home/ec2-user/anaconda3/envs/JupyterSystemEnv/lib/python3.7/multiprocessing/spawn.py", line 136, in _check_not_importing_main
    is not going to be frozen to produce an executable.''')
RuntimeError:
        An attempt has been made to start a new process before the
        current process has finished its bootstrapping phase.

        This probably means that you are not using fork to start your
        child processes and you have forgotten to use the proper idiom
        in the main module:

            if __name__ == '__main__':
                freeze_support()
                ...

        The "freeze_support()" line can be omitted if the program
        is not going to be frozen to produce an executable.

Installed Versions
Which version of AutoGluon are you are using?
If you are using 0.4.0 and newer, please run the following code snippet:

INSTALLED VERSIONS
------------------
date                 : 2022-04-06
time                 : 15:22:16.975165
python               : 3.7.12.final.0
OS                   : Linux
OS-release           : 4.14.252-131.483.amzn1.x86_64
Version              : #1 SMP Mon Nov 1 20:48:11 UTC 2021
machine              : x86_64
processor            : x86_64
num_cores            : 32
cpu_ram_mb           : 245845
cuda version         : 11.450.142.00
num_gpus             : 4
gpu_ram_mb           : [8404, 8476, 16160, 16160]
avail_disk_size_mb   : 11391

autogluon.common     : 0.4.0
autogluon.core       : 0.4.0
autogluon.features   : 0.4.0
autogluon.text       : 0.4.0
autogluon_contrib_nlp: None
boto3                : 1.21.34
dask                 : 2021.11.2
distributed          : 2021.11.2
fairscale            : 0.4.6
matplotlib           : 3.5.1
nptyping             : 1.4.4
numpy                : 1.21.5
omegaconf            : 2.1.1
pandas               : 1.3.5
PIL                  : 9.0.1
psutil               : 5.8.0
pytorch_lightning    : 1.5.10
ray                  : None
requests             : 2.27.1
scipy                : 1.7.3
sentencepiece        : None
skimage              : 0.19.2
sklearn              : 1.0.2
smart_open           : 5.2.1
timm                 : 0.5.4
torchmetrics         : 0.7.3
tqdm                 : 4.64.0
transformers         : 4.16.2

Additional context
Add any other context about the problem here.

I see a similar issue while training the text predictor 0.4 on an instance with multiple GPUs on SageMaker Studio.

@Abhinavb08 @melrubino  We need to write the code like follows (protect .fit() with if __name__ == "__main__":):
import pandas as pd
import awswrangler as wr
from autogluon.text import TextPredictor

args = Namespace(
    train_filename = "s3://ccds-asin-drc/eu/modeling-data/mf2_no_emb/train/0/train.parquet",
)

model_config = {
    "eval_metric": "accuracy",
    "time_limit": 60*60*3,
    "features": ['label', 'item_name_orig']
}

hyperparameters = {
    'model.hf_text.checkpoint_name': 'microsoft/mdeberta-v3-base',
    'optimization.top_k': 1,
    'optimization.lr_decay': 0.9,
    'optimization.learning_rate': 1e-4,
    'env.precision': 32,
    'env.per_gpu_batch_size': 4,
    'env.num_gpus': 4
}

train_df = wr.s3.read_parquet(args.full_train_filename)
print(train_df.info())

if __name__ == "__main__":
    predictor = TextPredictor(
        label='label',
        eval_metric=model_config['eval_metric']
    )

    predictor.fit(
        train_data=train_df[model_config['features']],
        hyperparameters=hyperparameters,
        time_limit=model_config['time_limit']
    )

Marking as complete, please create a new issue if issue persists in v0.7 release / pre-release when using autogluon.multimodal.


Add "no-log" modus to binary

When starting the dht as a node it will automatically log requests
https://github.com/hyperswarm/dht/blob/153d0fb498cc8ba106e64ba23612e28ee250a447/bin.js#L11-L21
this might fill up the hard disc and it should be possible to switch it of.
Note: I am not sure how to implement this best, else I would send a PR

to implement:

accept a flag on the command line (e.g. -q)
const quiet = process.argv.indexOf('-q') > -1
every log if (quiet) console.log('blah')

This was resolved in #28.


I doen't need to have informed wimpy_aws_keypair

HI,
I want to deploy without key_name.
regards

I don't know why I put doen't in tittle... sorry for that

Thank you for the changes!


Unable to start container after it was killed

Image: openvpn/openvpn-as:2.13.1-d8cdeb9c-Ubuntu22
Steps to repro:

Setup openvpn via docker & admin UI
Kill the container
Restart the container

openvpn-as-1  | + '[' '!' -c /dev/net/tun ']'
openvpn-as-1  | + mkdir -p /dev/net
openvpn-as-1  | + mknod /dev/net/tun c 10 200
openvpn-as-1  | + '[' '!' -f /openvpn/etc/docker-init ']'
openvpn-as-1  | + rm -rf /usr/local/openvpn_as/etc
openvpn-as-1  | + ln -s /openvpn/etc /usr/local/openvpn_as/etc
openvpn-as-1  | + '[' '!' -f /openvpn/etc/docker-init ']'
openvpn-as-1  | + exec /usr/local/openvpn_as/scripts/openvpnas --nodaemon
openvpn-as-1  | Another twistd server is running, PID 1
openvpn-as-1  |
openvpn-as-1  | This could either be a previously started instance of your application or a
openvpn-as-1  | different application entirely. To start a new one, either run it in some other
openvpn-as-1  | directory, or use the --pidfile and --logfile parameters to avoid clashes.
openvpn-as-1  |

Seems like something isn't cleaned up correctly on startup.

Hello,
Thanks for reporting this.
Yeah we do need to clean the old PID files
The fix is coming with the next image release

fixed with the commit above

Thanks for the quick turn around. When can we expect the next image release?

Same issue here with Access Server 2.14.0 running in Docker container. The PID file in question is located in container root directory. Here is its structure:
docker exec openvpn-as ls /
bin
boot
dev
docker-entrypoint.sh
etc
home
lib
lib32
lib64
libx32
media
mnt
openvpn
opt
ovpn
proc
root
run
sbin
srv
sys
tmp
twistd.pid
usr
var

There is no pid directory in the path referenced in entrypoint:
docker exec openvpn-as ls /usr/local/openvpn_as/etc
PLATFORM
VERSION
as.conf
as_templ.conf
config-local.json
config.json
db
docker-init
exe
licenses
sock
tmp
upgrade-flags
web-ssl


KAFKA-8120 Getting NegativeArraySizeException when using Kafka Connect

Trigger buffer expansion when buffer does not extract a line
Limit buffer size to Integer.MAX_VALUE
Using single loop inside FileStreamSourceTask.poll
Close the stream before throw ConnectException
Move stream and buffer related function to a separate class FileStreamBuffer
Check the JIRA comment for the steps to verify.

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

call for reviewers @kkonstantine @phstudy @ewencp  @hachikuji
should have separate refactoring code with the bug fix code. but it is a fairly safe fix.

@wj1918 thanks for opening a PR!
I'd definitely recommend fleshing out the bugfix in a separate PR from the rest of the refactoring since the latter does not seem trivial. Additionally, w/r/t the refactoring, I'd suggest thinking whether it is essential. Keep in mind that this connector is available for demonstration purposes only and therefore it's maintenance is not very heavy. But, back to the first point, before reviewing any changes, it'd be good to have them in separate PRs, since that will significantly help - at least myself - to review.

@kkonstantine thanks for the comment. understood, will close this PR and create separate PRs when ready.

Thanks @wj1918 !
Happy to review after they are ready.


Delete pr branch test

test pr

Repoman Generation Results
Repoman pushed changes to remotes for the following projects:
Project: ****

Repoman Generation Results
Repoman pushed changes to remotes for the following projects:
Project: todo-nodejs-mongo
Remote: azure-samples
Branch: pr/16
You can initialize this project with:
azd init -t kaizentm/todo-nodejs-mongo -b pr/16

View Changes | Compare Changes

Project: todo-python-mongo
Remote: azure-samples
Branch: pr/16
You can initialize this project with:
azd init -t kaizentm/todo-python-mongo -b pr/16

View Changes | Compare Changes