Stop console and Docker logging on shutdown

Left over changes from https://github.com/dotnet/aspire/pull/1578

DashboardServiceData dispose now stops any outstanding console and Docker container logging.
Unit test console logging

Microsoft Reviewers: Open in CodeFlow

Any chance you could include an update in this PR that removes the unnecessary starting of Docker Desktop, when a .NET Aspire app host starts — that doesn't contain any containers?

This PR is replicating what the resource publisher and other subscriptions do with having a centralized token for cancellation that is triggered when the app is stopping, and a token passed in by the caller. There is no way for app shutdown to explicitly force logging to stop.
For example, in publisher:
https://github.com/dotnet/aspire/blob/5ee685ee232856f25de5ae59ba0325e2d8e11448/src/Aspire.Hosting/Dashboard/ResourcePublisher.cs#L47-L49
It looks different in console/Docker logging because logging has classes that implement IAsyncEnumerable instead of a method that returns IAsyncEnumerable, e.g.
public async IAsyncEnumerator<IReadOnlyList<(string Content, bool IsErrorMessage)>> GetAsyncEnumerator(CancellationToken cancellationToken = default)
{
    using var linkedCts = CancellationTokenSource.CreateLinkedTokenSource(_cancellationToken, cancellationToken);
    var linkedToken = linkedCts.Token;

    // ...
}

But the concept is exactly the same.

I think a clear problem statement would help. If I understand correctly, your concern is that the processes that pull logs from disk and from docker might not be cleaned up when the application shuts down.
If that's the case, I think https://github.com/dotnet/aspire/pull/1633 achieves the same thing without changing the design. When the application is shut down, any enumeration of console logs is cancelled. That PR uses cancellation token rather than IDisposable, which I think makes things clearer.

This PR creates consistency of log subscriptions with resource subscriptions. Resource subscriptions support being centrally cleaned up when the app stops. Currently log subscriptions don't.
This isn't fixing an immediate bug but what is currently happening relies on every caller of logs to correctly cancel them. If they don't then the executing code isn't cleaned up. That's especially bad for Docker logging because even if the process exists, it could leave a docker process running.

rather than IDisposable

This uses cancellation:
public async IAsyncEnumerator<IReadOnlyList<(string Content, bool IsErrorMessage)>> GetAsyncEnumerator(CancellationToken cancellationToken = default)
{
    using var linkedCts = CancellationTokenSource.CreateLinkedTokenSource(_cancellationToken, cancellationToken);
    var linkedToken = linkedCts.Token;

    // ...
}

It works exactly like the resource publisher.

What about this? 35a6772 (#1620)

I don't believe this PR actually changes anything (since #1633). The application shutting down token is now used, which will cancel all pending async enumerations, which will tear down any log streaming. This adds a bunch of code without changing behaviour, unless I'm missing something.

The changed behavior is if someone incorrectly doesn't cancel a console stream, then this ensures console logging logic - which includes starting a process - are canceled by the time the host shuts down.
You cannot guarantee that everyone who will ever call the API in the future does the right thing. This puts up a guard rail.

Cancellation during shutdown was added in #1633. It doesn't require users to do the right thing. The enumeration is gated on the app hosts shutting down token, so if either the app shuts down, or the client cancels the operation, things are cleaned up. I can't see a way to leak things in the code today.
In more detail, here's what I believe happens during shutdown. Let me know if I'm missing anything.

ShuttingDown token is cancelled
All pending stream subscriptions are terminated
DI instances are disposed

This PR hooks cleanup in that step 3, but everything's already cleaned up by that point (modulo race conditions).

It doesn't require users to do the right thing.

It requires everyone who uses console log publisher correctly cancel. A week ago that wasn't happening. At this moment they are. In 1 or 2 years someone who doesn't know could pick up the console log publisher, not do the right thing and then docker processes could start leaking again.
It's a guard rail against misuse.

Requiring cancellation vs. requiring disposal doesn't seem that different to me. Both require external users to do the right thing. These are internal components, so we know all usages of the code.
I can see value in adding a check that the passed cancellation token is actually cancellable (added in #1658). But offering disposal in addition to cancellation doesn't add any value that I can see, and as I commented on the first PR you offered this change in, I feel it makes things more confusing and verbose. Sorry, but I'm not in favour of this change.

I think it is confusing that the resource publisher does this, but the log publisher doesn't.
Looks like we'll have to agree to disagree. I think it is important to have centralized clean up, but I don't want to merge without consensus.
@davidfowl can you please take a look. Thanks.

I think it is confusing that the resource publisher does this, but the log publisher doesn't.

I'm not sure what you mean. The log publisher returns objects which do that. ResourcePublisher doesn't implement IDisposable either. I feel like I'm missing something.

Give @davidfowl's refactoring in this area is this PR still useful?

No these are going to be deleted soon. Streaming logs will replace these classes. @karolz-ms is working on that


Rewrite all commands from HD Admin

First

[ ] Move internal commands (like setStat, etc) to Command repo
[ ] Change 'setStat' arg from 'Text' to 'StatValue'
[ ] Update the 'userid' arg so that it accounts for the new parser params

Setup practise commands

[ ] make dino follows an official command
[ ] Setup a laser eyes the eye color = laser color!) command (and make the eye color = laser color!)
[ ] recreate poop command and test what happens with multiple people, and when you reset whilst pooping
[ ] Setup an icecream command (consider making it a bus instead with bullys, or basing off The Magic School Bus)

Commands

[ ] commandName1
[ ] commandName2

i bought the laser eyes game-pass and now i cant activate it

Hey please help can you tell us how to give Headmin perms to Pban?!

Hi Kevin, if you need your best bet is reaching out to me via DMs on the DevForum

@1ForeverHD i play family paradise and other HD admin games ever since you took clown out of HD admin a lot of people and me were sad, a few days ago i wrote roblox an email saying if they can add it back but they told me to contact you its been so long not having the clown/icecream command me and my friends and other people beg you to add it back


Trouble Calling SnapshotAsync on a Collection Reference

Here is the Program.cs file: https://gist.github.com/ryanmats/52535d2242881a58302e64c07f4e2dd3
Here is the QuickStartFirestore.csproj file: https://gist.github.com/ryanmats/8c1673a90de3d07db9c156f0352dd07b
You can run the code by running dotnet run retrieve-all-documents YOUR_PROJECT_ID from command line.
Here is the error message I'm getting: https://gist.github.com/ryanmats/59cc0567b93317dc857c112b556a9349
One potential issue we noticed was that there was a client library change fairly recently that renamed the SnapshotAsync method to GetSnapshotAsync client library change (24 days ago)
However it looks like this change isn't yet propagated to the public packages on Nuget cause when Program.cs is updated to use GetSnapshotAsync the error returned is "'...'CollectionReference' does not contain a definition for 'GetSnapshotAsync'..."
Error Message - https://gist.github.com/ryanmats/04a49f75b8806550175a8d8542d2d0fb
Thanks in advance for the help!

Yes, if you're using 1.0.0-beta02, the name change is irrelevant.
Do you get that message every time, or just sometimes? Do you have any documents in that collection?

The error message appears every time.
Yes, I have 2 documents in the users collection.

Hmm. Slightly shorter program which works fine for me - could you try this?
Project:
<Project Sdk="Microsoft.NET.Sdk">
  <PropertyGroup>
    <OutputType>Exe</OutputType>
    <TargetFramework>netcoreapp2.0</TargetFramework>
    <LangVersion>latest</LangVersion>
  </PropertyGroup>

  <ItemGroup>
    <PackageReference Include="Google.Cloud.Firestore" Version="1.0.0-beta02" />
  </ItemGroup>
</Project>

Code:
using System;
using System.Threading.Tasks;
using Google.Cloud.Firestore;

class Program
{
    static async Task Main(string[] args)
    {
        FirestoreDb db = FirestoreDb.Create(args[0]);
        CollectionReference usersRef = db.Collection("users");
        QuerySnapshot snapshot = await usersRef.SnapshotAsync();
        foreach (DocumentSnapshot document in snapshot.Documents)
        {
            Console.WriteLine(document.Reference);
            foreach (var entry in document.ToDictionary())
            {
                Console.WriteLine($"{entry.Key} = {entry.Value}");
            }
        }
        Console.WriteLine("Done");
    }
}

I'll try your exact code now...

Ah - I see what's wrong with your code - you're calling an async method, but not waiting for it to finish.
If you just change this:
RetrieveAllDocuments(project);

into
RetrieveAllDocuments(project).Wait();

... and change the async method to return Task instead of void, then it should work.
Basically at the moment you've got an async method still running while the app is shutting down - I'm not sure there's anything we can do to make that work better.

thanks @jskeet it is working now! Completely new to C# so this is helpful.


反复的 stream opend closed 是什么原因

参数核对一致，一连就是  stream opend     stream closed
2017/04/12 22:44:56 version: 20170329
2017/04/12 22:44:56 listening on: [::]:8888
2017/04/12 22:44:56 target: <IP_ADDRESS>:10128
2017/04/12 22:44:56 encryption: salsa20
2017/04/12 22:44:56 nodelay parameters: 1 20 2 1
2017/04/12 22:44:56 sndwnd: 1024 rcvwnd: 1024
2017/04/12 22:44:56 compression: false
2017/04/12 22:44:56 mtu: 1350
2017/04/12 22:44:56 datashard: 10 parityshard: 3
2017/04/12 22:44:56 acknodelay: false
2017/04/12 22:44:56 dscp: 0
2017/04/12 22:44:56 sockbuf: 4194304
2017/04/12 22:44:56 keepalive: 10
2017/04/12 22:44:56 snmplog:
2017/04/12 22:44:56 snmpperiod: 60
2017/04/12 22:44:56 pprof: false
2017/04/12 22:44:56 remote address: 183.129.---.--:43894
2017/04/12 22:44:56 stream opened
2017/04/12 22:44:56 stream opened
2017/04/12 22:44:57 stream closed
2017/04/12 22:44:57 stream closed
2017/04/12 22:44:57 stream opened
2017/04/12 22:44:57 stream closed
2017/04/12 22:45:00 stream opened
2017/04/12 22:45:00 stream closed
2017/04/12 22:45:00 stream opened
2017/04/12 22:45:00 stream closed
2017/04/12 22:45:07 stream opened
2017/04/12 22:45:07 stream closed
2017/04/12 22:45:07 stream opened
2017/04/12 22:45:07 stream closed
2017/04/12 22:47:19 broken pipe

正常现象 都这样的。

简单的讲，网络连接开始--stream opened，网络连接断开--stream closed
不过经你这么一说倒确实不太清楚这个日志存在的意义，毕竟重复信息刷屏了
可以优化下问题表述，改成日志优化建议/讨论

normal input


:bug: fix nargs + or *

Thank you @tjohnson31415 for the guidance

Codecov Report
Attention: Patch coverage is 33.33333% with 2 lines in your changes missing coverage. Please review.

Project coverage is 56.73%. Comparing base (5852586) to head (158f02a).

Files with missing lines
Patch %
Lines

src/vllm_tgis_adapter/tgis_utils/args.py
33.33%
1 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #140      +/-   ##
==========================================
- Coverage   57.26%   56.73%   -0.53%
==========================================
  Files          25       25
  Lines        1542     1544       +2
  Branches      256      257       +1
==========================================
- Hits          883      876       -7
- Misses        581      589       +8
- Partials       78       79       +1

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


Docs: re-run of the test on the existing namespace does not create pods

git install the sonubuoy and run the test as steps provided in the READ.md
If the test run fails due to network-proxy issue, it raises the GO panic trace as see from the logs of the pod created in the namesapce "heptio-sonubuoy"
Because of the above panic, in my understand the cleanup never gets executed, thereby the namespace "heptio-sonobuoy" persists
Now re-apply the yaml file (kubectl apply -f examples/quickstart/), this time the pods are not getting created. User has to delete the namespace and then re-apply the yaml.

This is the case even if sonobuoy runs sucessfully... the pod stays around so that you can kubectl cp the output tarball locally.  In order to re-run sonobuoy you need to delete the pod (or the whole namespace) and recreate it.
Re-titling this, I think our docs need to be fixed to make this more clear.

@abiogenesis-now who may be interested in updating the README here

Closing this since we have more clarity in the docs now


manager: use validator set from state

This PR changes the signed header to use validator set from last state.
Tests needed to be updated to use at least one validator (because ValidateBasic now fails if there's nil validators). Since they need to be able to handle multiple validators anyway, we're passing 2 validators for now.
Fixes #812

Codecov Report
Patch coverage: 41.66% and project coverage change: -0.08 :warning:

Comparison is base (d450698) 55.32% compared to head (ea4f1a6) 55.25%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #814      +/-   ##
==========================================
- Coverage   55.32%   55.25%   -0.08%
==========================================
  Files          54       54
  Lines        9930     9930
==========================================
- Hits         5494     5487       -7
- Misses       3629     3634       +5
- Partials      807      809       +2

Impacted Files
Coverage Δ

types/pb/dalc/dalc.pb.go
38.78% <ø> (ø)

types/pb/rollkit/rollkit.pb.go
40.01% <ø> (ø)

types/pb/rollkit/state.pb.go
41.65% <ø> (ø)

types/validation.go
57.14% <36.36%> (ø)

block/manager.go
65.17% <100.00%> (-1.57%)
:arrow_down:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

*.pb.go files should not be modified. can you remove the changes for those?

*.pb.go files should not be modified. can you remove the changes for those?

Fixed.

Skipping TestValidatorSetHandling but creating issue to follow up, as it expects multiple genesis validators and needs to be passed a rotating signing key.


feat(integrations): Add initial support for kedro

https://www.loom.com/share/7837823f092746e4936c6489c1d3a9ca?sid=74635c28-54e8-4dd3-a271-d011c66ad4cb
Description

Fixes WB-NNNNN
Fixes #NNNN

What does the PR do?
Include a concise description of the PR contents
Testing
How was this PR tested?

Hi folks, it's Juan Luis here from the Kedro team at QuantumBlack - just wanted to chime in to say that we're very excited about this and would be happy to help if there's anything we can do.

Thanks for offering @astrojuanlu , I posted a video above to show what it looks like with Spaceflight Pandas -- did you have any initial feedback?
In particular, I wonder if there's a better way to:

Pass run metadata down into the individual nodes.  Currently, it doesn't seem like a Kedro hook can pass data into a node, so we have a workaround that sets/gets the current run's metadata as a dict inside threading.local()
Track dataset inputs/outputs for each node.  In kedro, the dataset loading and saving seems to happen outside the context of a node, whereas in wandb we assume it's inside the context.  We're currently doing some hacky things to make it work, but I wonder if there's a better way to capture the entire "load dataset, run node, save dataset" group for each node?

Once this PR is merged, it would be great to have some joint marketing for this integration :)

hi folks, sorry I never followed up to this, it was hard for me to prioritise. I shared the PR with the rest of the team in case anybody has time to look at it.


LITE-28847: Support 'late' field in product capabilities

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


In Contact Us page there is no character count filed instead there is pop up message saying minimum characters should 50.

Description
Hello PA,
In Contact Us page at Message Field I will add text like saying minimum character count should be 50 and also a bar which counts the characters when user enters a character so that user will be comfortable in reaching the minimum criteria . I think it will be good rather that pop-up message.
Screenshots

Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a GSSOC'24 contributor.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

@Rakesh9100
Please assign me this work .
Thank you.

Please go ahead and create the PR, Rakesh will start assigning the issues and merging PRs soon.


Add price methods to kamino-sdk

Summary by CodeRabbit
New Features:

Enhanced the Kamino SDK with new functions for retrieving all oracle prices, price by mint, TWAP by mint, prices by mints, and TWAPs by mints.
Introduced new models EnrichedScopePrice, KaminoPrices, and OraclePricesAndCollateralInfos to better structure price and collateral data.

Refactor:

Updated getAllPrices and getOraclePricesAndCollateralInfos functions to include optional parameters for oraclePrices and collateralInfos.
Optimized getStrategyWhirlpoolPoolAprApy function to use getAllOraclePrices instead of getAllPrices.

Tests:

Added new test cases to verify the retrieval of all Kamino prices and Kamino price and TWAP by mint, improving the test coverage for the Kamino SDK.

Can you just add a test that is skipped on the pipeline that reads price and twap from mainnet and check the value is correct?

Done


install.bat not working (access denied)

when i try to press the install.bat program inside the RDPWrap it tells me the access is denied even tho ive opened it with administrator permissions what could be the issue

Duplicate of #3222.


Update NFRs, Glossary and User Stories in Developer Guide

Make updates NFRs, Glossary and User Stories.
For User Stories, add the "so that I can..." part to make the user story complete (although it can be omitted in some cases).

[x] NFRs
[x] Glossary
[x] User Stories

Leaving issue open for any changes to be made


Grafana Keeps  on restarting

Grafana container keeps on restarting and log shows timeout error

Error: ✗ Failed to send request: Get "https://grafana.com/api/plugins/repo/flant-statusmap-panel": context deadline exceeded (Client.Timeout exceeded while awaiting headers)

i can see ton of dns queries from docker to grafana

Okay, probably same issue https://github.com/geerlingguy/internet-pi/issues/8
Added dns_servers to the playbook.


Curious about the loss function

Hi, I'm sorry to have one more question again.
I saw the loss function and it does not seem like a normal triplet loss.
To me, it is more like kind of  crossentropy loss.

Could you explain why you make this formula of loss function?(dot product and sigmoid, etc)

this is the same loss function as used in word2vec, but on top of our encoder f. you find it's discussion then you google negative sampling / Noise Contrastive Estimation, see for example here https://ruder.io/word-embeddings-softmax/


Screen at 150% has fields cut off

Not sure if you can address this. I have a 4K monitor at 150%.  Thanks!

I increased the size of the program wrapper in the text file. The layout looks better but the progress bar / time remaining is still cut off.

I'm using on my 4K screen at 150% zoom. Everything is still cut off.

I'm using on my 4K screen at 150% zoom. Everything is still cut off.

Closing this. Some other people have made better apps now than this one.
Using the basic windows forms i did here does not allow for much customizaion.
See this guys new app here. Haven't tried it, but looks good:
PlexRipper


Suggestion: head detection

Hi
Thanks for a great lib!
I'm using it to automatically crop avatars. I would be great if the library could identity the coordinates of the entire head.

I'm wondering the same thing, it would be great to extract/crop the whole head, including hair, and part of the neck.
Maybe it's possible just to add 20-30% of margins, possibly more at the top, to catch the whole head. But certainly there won't be a fixed size, depending on hairstyle.


Update longhaul for v0.1.5 image

What type of PR is this?
test

What this PR does / why we need it:
Update new client module package name
Increase timeout
Which issue(s) this PR fixes:

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
854

Relevant Lines:
1093

💛  - Coveralls


Assortative matching with multiple faculties

Add functionality for having people with more than two statuses
Tests added for assortative matching

Use-Case:
Using a negative mixing_multiplier results in groups favouring people which differ more in their status.
I believe this will be helpful for users who want to create more diverse groups.

Now it is also possible to give people multiple statuses and a corresponding mixing_multiplier by providing a list.
Use Case:
We give people not only a status like faculty member or student, but also a status related to their topics of interest to create more interesting matches.

Codecov Report

Merging #22 (e832f18) into main (c6bd311) will increase coverage by 2.24%.
The diff coverage is 94.11%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main      #22      +/-   ##
==========================================
+ Coverage   94.80%   97.05%   +2.24%
==========================================
  Files           6        6
  Lines         366      407      +41
==========================================
+ Hits          347      395      +48
+ Misses         19       12       -7

Impacted Files
Coverage Δ

src/randomgroups/create_matching.py
88.09% <75.00%> (-0.07%)
:arrow_down:

src/randomgroups/algorithm.py
100.00% <100.00%> (+12.30%)
:arrow_up:

tests/test_algorithm.py
100.00% <100.00%> (ø)

tests/test_create_matching.py
100.00% <100.00%> (ø)

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

[x]  See the comments I left below
[x]  The mixing_multipler should be either a float or a dictionary where each key corresponds to one of the status columns. I think we could run into all sorts of problems if we only have a list there.


Accessibility on and off icon

Icon Request

Icon name: Accessibility
Tags of this icon: Accessibility, Accessibility off
Screenshots of similar icons:

Good point!

I second this. This logo is what is recommended for accessibility matters, not just a wheelchair icon, which is really outdated on that topic — a wheelchair isn’t the defining feature of someone who needs a screen reader or bigger fonts.
Also, this set request would be a nice addition, because Remix is severely lacking those imho https://github.com/Remix-Design/RemixIcon/issues/104


Make project place association generate rule, or restrict its role to just display of boundaries

Similar to #908, its confusing that we have a project place association both in the Location box and the Rule box on Project/edit. How about we either:

Make adding a place in the location box generate a rule (similar to #908)

OR

Make adding a place in the location box only control the display of the boundary on the map, but leave all other place related logic to the place rule?

Thoughts @pleary?

The way it's set up now, the place is not a rule, so 2 would have the least impact. If we went with path 1, then we'd need to apply place rules to lots of existing projects that may not want them.
With the current code, the project place does not have any effect on manually-added observations, but it is used by the aggregator to filter the set of aggregatable observations. If we want with path 2, then I think we'd just need to remove that filter on the aggregator, and possibly add some text to the places section of the edit page to clear up any confusion for users.


passwordPolicies

I find nothing for " passwordPolicies " , is something that we just add ?
https://msdn.microsoft.com/en-us/library/azure/ad/graph/api/entity-and-complex-type-reference#passwordprofiletype

For what I understand the password policies is fixed on Azure AD:
https://azure.microsoft.com/en-us/documentation/articles/active-directory-passwords-policy/
I'm not an expert in this specific point, you could comment the previous article to ask how to change the password policies for a given tenant in AAD. If this is possible and there is a Rest endpoint for that, we could ask for support.

I'm not an expert.
I may well pass the parameter "passwordPolicies" on this line:
https://github.com/Azure/azure-sdk-for-python/blob/3e80c20b0008ba6004753004be1462f0295c4b1b/azure-mgmt/tests/recordings/test_graphrbac.test_graphrbac_users.yaml#L59
This is a missing parameter " UserCreateParameters " ?
Simon

I asked the question to the RestApi spec group: https://github.com/Azure/azure-rest-api-specs/issues/375

Close, can use additionalProperties


METRON-861: Allow JVM args to be passed to CLI utilities

Contributor Comments
This is motivated by the fact that if one sets the acl's on the znodes that metron creates (e.g. /metron/topology/global) to read/write by metron (from the zkcli setAcl /metron/topology/global sasl:metron:crwda), then permissions prohibit the CLI tools from functioning because the JAAS config is not loaded.
This JIRA allows users to pass java properties to the CLI tools. While important for Kerberos, it's useful in general if one needs to adjust the heap required, etc.
To exercise this, do the following:

Perform steps 1 through 17 of the kerberos docs for vagrant
su to the metron user via su - metron
Validate that you can dump the configs via $METRON_HOME/bin/zk_load_configs.sh -m DUMP -z node1
Adjust the ACL for /metron/topology/global by

start zkcli by running /usr/hdp/current/zookeeper-client/zookeeper-client -server node1
adjust the acl via setAcl /metron/topology/global sasl:metron:crwda

Destroy all existing tickets via kdestroy
Validate that $METRON_HOME/bin/zk_load_configs.sh -m DUMP -z node1 gives an error
Validate that JVMFLAGS="-Djava.security.auth.login.config=/home/metron/.storm/client_jaas.conf" $METRON_HOME/bin/zk_load_configs.sh -m DUMP -z node1 works

Pull Request Checklist
Thank you for submitting a contribution to Apache Metron (Incubating).
Please refer to our Development Guidelines for the complete guide to follow for contributions.
Please refer also to our Build Verification Guidelines for complete smoke testing guides.
In order to streamline the review of the contribution we ask you follow these guidelines and ask you to double check the following:
For all changes:

[x] Is there a JIRA ticket associated with this PR? If not one needs to be created at Metron Jira.
[x] Does your PR title start with METRON-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
[x] Has your PR been rebased against the latest commit within the target branch (typically master)?

For code changes:

[x] Have you included steps to reproduce the behavior or problem that is being changed or addressed?

[x] Have you included steps or a guide to how the change may be verified and tested manually?

[x] Have you ensured that the full suite of tests and checks have been executed in the root incubating-metron folder via:
mvn -q clean integration-test install && build_utils/verify_licenses.sh

[x] Have you written or updated unit tests and or integration tests to verify your changes?

[x] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

[x] Have you verified the basic functionality of the build by building and running locally with Vagrant full-dev environment or the equivalent?

For documentation related changes:

[x] Have you ensured that format looks appropriate for the output in which it is rendered by building and verifying the site-book? If not then run the following commands and the verify changes via site-book/target/site/index.html:
cd site-book
bin/generate-md.sh
mvn site:site

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.
It is also recommened that travis-ci is set up for your personal repository such that your branches are built there before submitting a pull request.

Looks good to me.  This should be generally useful, even outside of the jaas use case.
Would it make sense to namespace the JVMFLAGS environment variable so it's specific to Metron?  Not sure it matters much but it would make it more consistent with other environment variables we use.

You know, we definitely could.  I chose JVMFLAGS because it's what zookeeper calls them, but I'm open to other options.  Any suggestions?

METRON_JVMFLAGS?

I could get behind that.  Anyone else have other ideas?

+1


Fixed link for Metrics.NET

Code Analysis and Metrics/Metrics-Net
It looks like this project has moved to a new URL. The new maintainer posted a message about it here: https://github.com/etishor/Metrics.NET/issues/122#issuecomment-184707498. The previous owner of the repo where this was forked from has passed away: https://github.com/etishor/Metrics.NET/issues/122#issuecomment-175175100. The NuGet package (https://www.nuget.org/packages/Metrics.NET/0.3.7) now also uses this url.

@dlemstra It's a bad news.
The url of this project is not correct, should be https://github.com/Recognos/Metrics.NET, please update it.


[FEATURE] Missing Indices.ReloadSearchAnalyzers

Is your feature request related to a problem?
C# Nugets
OpenSearch.Client v1.5.0
OpenSearch.net v1.5.0
We’ve upgraded from NEST/ElasticSearch.net to the OpenSearch equivalents, and I can’t seem to find any replacement/equivalent for Indices.ReloadSearchAnalyzers
In the OpenSearch docs itself, it seems to exist at the low level (Refresh search analyzer - OpenSearch documentation) but I can’t seem to locate any equivalent in the client?
We use this to reload our synonyms without refreshing the whole index.
What solution would you like?
Add the equivalent call to OpenSearch.net that exists in ElasticSearch.net
What alternatives have you considered?
Without this functionality, we will need to explicitly call the OpenSearch server via an HTTP POST /_plugins/_refresh_search_analyzers/"index"
Do you have any additional context?

While not ideal, you can work around this missing API and still take advantage of the client's configuration (e.g auth, hosts, SSL) by making the request with the low-level raw request method like so:
client.LowLevel.DoRequest<VoidResponse>(HttpMethod.POST, "/_plugins/_refresh_search_analyzers/" + indexName);

Thomas,
Thanks for the quick response.
Yes, I can confirm that using the low level call works for me for now (and
until the feature is added).
Thanks again.
Paul Ninnis
Senior Developer
M +61 412832453 <+61%20412%20832%20453>
On Wed, Oct 11, 2023 at 10:35 AM Thomas Farr @.***>
wrote:

While not ideal, you can work around this missing API and still take
advantage of the client's configuration (e.g auth, hosts, SSL) by making
the request with the low-level raw request method like so:
client.LowLevel.DoRequest(HttpMethod.POST, "/_plugins/_refresh_search_analyzers/" + indexName);
—
Reply to this email directly, view it on GitHub
https://github.com/opensearch-project/opensearch-net/issues/391#issuecomment-1756485063,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AESKY4MXQNXDBY5CBCH3BJLX6XPDBAVCNFSM6AAAAAA526PWGCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONJWGQ4DKMBWGM
.
You are receiving this because you authored the thread.Message ID:
@.***>


ExpressionFunction updated with NOT option

Howdy @russ666! Thanks for the pull request -- I chatted with the team and we'll be updated the ExpressionFunction enum with the NOT option in our next docs update.
I'll close this PR for now but please feel empowered to open one back up if you don't see these changes land within the next 2 weeks!


[Bug]: Ambuflow Logo

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
The logo in the README.md file has the old logo, it has to be updated.
Add Screenshots

What browsers are you seeing the problem on?
No response
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

@mansiruhil13
If this is the main logo, then the logo in the current website has to be changed

Which is the main logo?

@mansiruhil13 ??

1st one is for website and 2nd one for github readme file, but you can put 1st one


TOR

How do i start tor?
How do i install it and run it?
Basically what im asking is what do i do after installing ighack?

Bro on what device you want to run tor
On Wed, 24 Feb 2021, 3:54 pm BHAVIK PRAJAPATI<EMAIL_ADDRESS>wrote:

How do i start tor?
How do i install it and run it?
Basically what im asking is what do i do after installing ighack?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/noob-hackers/ighack/issues/18, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARFEAZXMS55DNVHQEV4ENCDTATHUPANCNFSM4YEHQ3VQ
.

If you running Debian like kali all you should do is install Tor with
apt install tor
and go and open /etc/tor/torrc . in this file scroll down to line that says :
#ControlPort 9051
remove the "# "  and save the changes
open a terminal and type : tor
and then start ig again

Why are this showing all the time.

Try apt install tor if not worked then pkg install tor

Why are this showing all the time.[image: Screenshot_20210506_103722.jpg]
https://user-images.githubusercontent.com/82598865/117244661-14d7b200-ae57-11eb-8463-ac36be80002b.jpg
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/noob-hackers/ighack/issues/18#issuecomment-833228089,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARFEAZUFOI5BA2LSMNFGTMDTMIP3PANCNFSM4YEHQ3VQ
.

Try apt install tor if not worked then pkg install tor

Why are this showing all the time.[image: Screenshot_20210506_103722.jpg]
https://user-images.githubusercontent.com/82598865/117244661-14d7b200-ae57-11eb-8463-ac36be80002b.jpg
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/noob-hackers/ighack/issues/18#issuecomment-833228089,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARFEAZUFOI5BA2LSMNFGTMDTMIP3PANCNFSM4YEHQ3VQ
.

Not working please help.

pkg install tor
tor

then start a new session¹ and start the tool.
¹just swipe from left to right and a side menu will show up

connect refused
I cannot use Ighack


Update text_generation.dart

#214 adds the imports in comments in each sample. Does that address your intentions with this change? That looks like what the JS SDK is doing.


Summer jam - split commits

So when I tried the tuple things I got into a reaal weird error. I've reverted the change here and am creating an issue so this isn't blocked. It's fine for the version thing we had in mind since this only removes behaviour (?).

yikes

@FredTheDino Do you want to merge this with -6% coverage or do we write some tests? The decrease is mostly because of FlatValue and the std additions.

Nah - it's fine. The extra 6% is hard to reach and probably not worth it. In the long term - it would be nice with tests for the networking. But I personally don't see how we can do that right now. But it should probably be an issue.
We need to do a coverage pass and try to raise the coverage after this bugfixing-spree!


winpr/library: fix win32 test and some cleanup

On Windows we seem to have to load the TestLibrary[AB] test libraries from in same folder the test executable runs.
Also removed the empty RemoveDllDirectory, SetDefaultDllDirectories, AddDllDirectory tests and the redundant FreeLibrary test.
TestLibrary now works and succeeds on Win32.

As far as I know the subject, it seems good to me

perhaps the tests that have been removed could have the corresponding files dropped too ?

@hardening thanks! completely forgot to rm . fixed now.


Small improvement suggestions

Hello and thank you for your work, its a neat little plugin and on my quick testing I can provide a short feedback :)

The default key map cannot be overwritten as its being hardcoded

vim.keymap.set("n", "<leader>td", function()
		ui.toggle_todo_window()
	end, { desc = "Toggle Todo List" })

Cannot remove the tag filter once such is applied. At least its not obvious how

The Todo window renders on the far right on the screen, center would be much preferable or at least the option to set its position

A usercmd might be preferable to a keybinding as well, I'd much prefer to be able to lazy load the plugin with a cmd like 'Dooing'

From your written backlog there is this bullet point - Todo List Per Project
This sounds great, but I'd also suggest to keep a global scope Todo list with the option to enable/disable that feature when this feature is done eventually. (I'd also think for a lot of use cases people would want to have a global TODO list they can access from anywhere at any time rather than the todos being tied to a specific project

I was just about to say about the same thing about the hard coded keybind, it would be really nice to call a function to toggle it on and off

Guys this is actually very good suggestions! really appreciate these observations.
gonna work on this!
@zvrkSam @GeorgiChochev

I upvote the keybinding thing. Neovim users usually have a large personalized config, with which-keys groups set, it's not a strange case to overwrite td keymap
I think that project todos can be a different command, like :DooingProject or similar

Cannot remove the tag filter once such is applied. At least its not obvious how

Working on to-do item editing and tag editing as well

From your written backlog there is this bullet point - Todo List Per Project
This sounds great, but I'd also suggest to keep a global scope Todo list with the option to enable/disable that feature when this feature is done eventually. (I'd also think for a lot of use cases people would want to have a global TODO list they can access from anywhere at any time rather than the todos being tied to a specific projec

Reconsidering this backlog item

Editings works!
But I wasn't thinking about tag editing, but canceling tag filters.

How to cancel the filtering and go back to the list of all to-dos?

ohh, totally forgot to update the help window. Press c to clear filtering

@ImmanuelHaffner can you check the latest release please

[x] Customizable Keys;
[x] New window position adjusted
[x] New CMD :Dooing
[x] Tag editing and filterd

project-related todos is in currently in work
issue will be close

Oddly, I cannot disable the default keymapping.
I am on the recent main
    ● dooing 0.99ms  start
        dir    /home/immanuel/.local/share/nvim/lazy/dooing
        url    https://github.com/atiladefreitas/dooing
        branch main
        commit 4c2847c
        readme README.md
        help   |dooing.txt|

I configure as
{
    'atiladefreitas/dooing',
    dependencies = {
        'folke/which-key.nvim',
    },
    config = function()
        require'dooing'.setup{
            keymaps = {
                toggle_window = nil,
            },
        }

        require'which-key'.add{
            { '<leader>tn', ':Dooing<cr>', desc = 'Open Todo notes' },
        }
    end,
}

I still see the <leader>td mapping:

And it's so odd as i can clearly see that you added the check here: https://github.com/atiladefreitas/dooing/blob/4c2847ce959eafe2593f1fe1c709e1ca64a40af7/lua/dooing/init.lua#L213-L218

[x]  New window position adjusted

Hey @atiladefreitas, firstly, thanks a lot for this super cool plugin. Actually, I like the right hand side position of the todo list.
I would request if we have a flexible config something like:
window = {
  position = "right" -- Or other options "left", "center"
  ...
},

What do you think ?

hello guys, reopenning the issue to work in these two adjustments @aniketgm and @ImmanuelHaffner


docs: 📚 Using Icon in angular Project

Description
Not clear instruction on how to setup icon for Angular project
Stesp has performes:
1- Install npm install --save @synergy-design-system/assets
2- Add component  component but nothing shows up.

Rationale

Acceptance criteria

After talking to @thaiquSICKAG this was fixed with the suggested changes from the demo project.


Game Missing "Borderlands 2 Game Of The Year Edition"

Fixed with this release: https://github.com/hobby-grade/Nintendo-Switch-Discord-Status/releases/tag/0.9.5


AQCU-1278: the case of the missing TS

allow dates that are greater than AND equal to the nodeStartDate

Coverage remained the same at 98.918% when pulling 362a7a7990bdfb6b5df85de277cdd25f7c9c1f59 on mhines-usgs:AQCU-1278 into 16d4190bb1e4410fe86de1fe51ed803ee105a237 on USGS-R:master.

I'm not sure this was the only problem after Laura has elaborated more. I may revert this.


Use NVM

Confirm the correct node version to use and include it in an NVM config file.

v16.20.2 | Gallium


React Nesting: Fragments Challenge

react-nesting/fragments
📄 Rendered Exercise
🔎 Review
Review via 🔗 CodeSandbox or locally by running this command in your Terminal:
npx ghcd@latest neuefische/web-exercises/tree/react-nesting-fragments/sessions/react-nesting/fragments -i

✨ Copyable Markdown for Curriculum
Solve this challenge on
[🔗 **CodeSandbox**](https://codesandbox.io/s/github/neuefische/web-exercises/tree/main/sessions/react-nesting/fragments?file=/README.md)
or locally by running this command in your Terminal:

```
npx ghcd@latest neuefische/web-exercises/tree/main/sessions/react-nesting/fragments -i
```

refers to neuefische/web-curriculum-new-format#192

It also about avoiding unnecessary div clutter in general.

Good point, I add a note in the readme about this.


Adding a file to workspace but it did not add .vscode and settings.json sub folders

We have written the needed data into your clipboard because it was too large to send. Please paste.
Issue Type: Performance Issue
Adding a file to workspace but it did not add .vscode and settings.json sub folders
This is a new, empty folder that I am adding to the workspace, but the the steps to add it listed in the Python for Dummies book implies that that those two subfolders should be there.
I am very new to this and cannot find an answer in documentation or in Google.
Can you can tell me what I need to do to resolve this?
Thank you.
Joe Nardone
<EMAIL_ADDRESS>6034869017

VS Code version: Code 1.55.2 (3c4e3df9e89829dce27b7b5c24508306b151f30d, 2021-04-13T09:35:57.887Z)
OS version: Windows_NT x64 10.0.18363

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz (12 x 2208)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onprotected_video_decode: enabledrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.86GB (10.83GB free)

Process Argv
--crash-reporter-id 521fa201-c51c-4cf1-94c1-4706361f7360

Screen Reader
no

VM
0%

Process Info
CPU %	Mem MB	   PID	Process
    0	   116	 15896	code main
    0	    52	  1552	   window (Issue Reporter)
    0	   249	  3168	   gpu-process
    0	    73	  3968	   shared-process
    0	    30	  9716	     ptyHost
    0	     6	  1548	       console-window-host (Windows internal process)
    0	     5	  4524	       console-window-host (Windows internal process)
    0	    68	 10844	       C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe
    0	    68	 12688	       C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe
    0	    65	 13544	       C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe
    0	     6	 15980	       console-window-host (Windows internal process)
    0	    24	  6280	   utility
    0	   178	 13400	   window (My Python3 workspace.code-workspace - My Python3 workspace (Workspace) - Visual Studio Code)
    0	    11	  6792	     watcherService
    0	    11	  5704	       console-window-host (Windows internal process)
    0	    75	 15748	     extensionHost
    0	    46	 13660	       "C:\Users\Josep\AppData\Local\Programs\Microsoft VS Code\Code.exe" "c:\Users\Josep\AppData\Local\Programs\Microsoft VS Code\resources\app\extensions\json-language-features\server\dist\node\jsonServerMain" --node-ipc --clientProcessId=15748
    0	    13	 14552	   crashpad-handler

Workspace Info
|  Window (My Python3 workspace.code-workspace - My Python3 workspace (Workspace) - Visual Studio Code)
|    Folder (MyCode): 0 files
|      File types:
|      Conf files:;

Extensions (10)

Extension
Author (truncated)
Version

vscode-django
bat
1.6.0

python-extension-pack
don
1.6.0

MagicPython
mag
1.1.0

python
ms-
2021.3.680753044

jupyter
ms-
2021.5.745244803

remote-wsl
ms-
0.54.6

vscode-yaml
red
0.17.0

vscodeintellicode
Vis
1.2.12

jinja
who
0.0.8

tconda
zhp
0.0.5

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vstry244:30276681
pythonvsdeb440:30248342
pythonvsded773:30248341
pythonvspyt875:30259475
pythonvsnew554:30281908
pythontb:30283811
wslfolderdoccf:30282075
vspre833cf:30267465
pythonptprofiler:30281270
vshan820cf:30276953
pythondataviewer:30285071
vscus158:30286553
vscgsv2ct:30286882

You can create them manually or when you try to add any workspace settings (from settings editor)

Thank you


[Bug]: It is not live for you yet Comming Soon when I connect to git，what is the reason

Is there an existing issue for this?

[X] I have searched the existing issues

Description
It is not live for you yet Comming Soon when I connect to git，what is the reason
Steps To Reproduce
Click git
Public Sample App
No response
Version
1.6.5

@243324195 please update your Appsmith instance to 1.6.6


add a default function to Deed contract

Since Registrar contract contains an expression newBid.send(msg.value) for a Deed newBid, the Deed contract needs a fallback function.  This pull-request adds a fallback function that just returns.

The default functionality is to just receive ether. This will change in the next version of Solidity, but only if you specify a pragma. Nevertheless, I'm going to merge this just because explicit is better than implicit.


[test_program,cp] develop reference Earlgrey PROD CP test program

The OpenTitan repo contains a CP (host) test program to drive the CP provisioning flow. A reference test program, that makes use of the provisioning infrastructure in this repo, should be developed to drive the CP flow.

opentitan Bazel repo has been vendored in; opentitanlib builds as a dependency successfully in this repo


Not underlining the errors in java code

Issue Type: Bug
when i write the code in java , it just dosen't compile and non of the errors get underlined , i checked all the neede extentions and i had everything i don't know what happend need the reply as soon as posible please , thank you
VS Code version: Code 1.66.2 (dfd34e8260c270da74b5c2d86d61aee4b6d56977, 2022-04-11T07:46:01.075Z)
OS version: Windows_NT x64 10.0.22610
Restricted Mode: No

/extJava

It looks like this is caused by the Java extension. Please file the issue to the Java extension repository. Make sure to check their troubleshooting instructions and provide relevant information such as the extension version you're using. See also our issue reporting guidelines for more information.
Happy Coding!


Add notes on interop with eslint-plugin-import to docs

Description
One thing i always forget when setting up an astro project with eslint-plugin-astro + eslint-plugin-import, is how to resolve import/no-unresolved errors for astro:content.
My usual way of handling it is adding astro:content to core modules
"settings": {
  "import/core-modules": ["astro:content"]
}

Is that approach right, and would it be worth adding it (or the actual right thing to do) as a note in the configuration docs? Does it even belong in these docs?

Pull requests are welcome!
It would probably be good to include the following settings as well.
  "settings": {
    "import/parsers": {
      "astro-eslint-parser": [".astro"],
      "espree": [".js"],
      "...": [".ts", ...]
    }
  }

After adding Astro View Transitions to my project, I encountered the following error:
error  Unable to resolve path to module 'astro:transitions'  import/no-unresolved
The above error is fixed by the following patch:
--- a/.eslintrc.js
+++ b/.eslintrc.js
@@ -66,8 +66,9 @@ module.exports = {
   settings: {
     'import/resolver': {
       typescript: {
         project: '.'
       }
-    }
+    },
+    'import/core-modules': ['astro:transitions']
   }
 }

Perhaps this will be of help to others.


Cancel when replying to thread does not seem to work

When I select a message in the scrollback, it opens into a window of 'Replying in thread'. If I then try to :cancel to go back it says 'No message currently selected'. How does one cancel a thread reply? If I use :q it closes the whole window and I lose the chat.

You can use <C-O> to move backwards through the jumplist, and <C-I> to move forwards.

I've added a Threads section to the documention, and mentioned that you can use <C-O> there.

You can use <C-O> to move backwards through the jumplist, and <C-I> to move forwards.

I think it would still be cool to have a separate keybinding to exit the current thread and go back to the parent room, as sometimes you navigate a lot inside a thread, so you would need to hit <C-O> several times to go back.


[WIP] Just use standard rails caching for inventory

As a lark, what would happen if we didn't clear our our inventory caches so much?
I don't see why we have to clear out the associations every chance we get
/cc @dmetzger57 I have a feeling this would be a big win. If this passes, I can get more concrete numbers

:+1: given how differently associations behave these days, it seems quite likely that whatever originally necessitated this has gone away.

@matthewd I modified the tests a little and got vmware to fail.
This needs more work :(

I'm not convinced the tests have full coverage either.  But we can cross that bridge when you come to it.

Well, fun experiment.
It was a pretty painless 15% memory savings. But alas, it did not pan out today.


Possible Change Detection error?

Hi Luca,
I haven't been able to figure this one out, and although it doesn't create any problems with the functionality, I'm getting complaints from management and other developers because of the constant console errors.
Whenever a form field with validation is changed, I get an error like this in the console:
Error: ExpressionChangedAfterItHasBeenCheckedError: Expression has changed after it was checked. Previous value: 'FORMS.DURATION_IN_MONTHS.ERRORS.REQUIRED'. Current value: ''.
The error points back to my CustomErrorsComponent.
I am using ngx-translate for error messages.
My AppModule is configured with the validation and translate modules:
NgxValidationErrorsModule.forRoot({       defaultContext: 'CUSTOM_GENERAL',       errorComponent: CustomErrorsComponent as any     }),     TranslateModule.forRoot({       loader: {         provide: TranslateLoader,         useFactory: HttpLoaderFactory,         deps: [HttpClient]       }     })
and I have added the translatePipeFactoryCreator function and providers:
export function translatePipeFactoryCreator(translateService: TranslateService) {   return (detector: ChangeDetectorRef) => new TranslatePipe(translateService, detector); }
{       provide: MESSAGES_PIPE_FACTORY_TOKEN,       useFactory: translatePipeFactoryCreator,       deps: [TranslateService]     },     {       provide: MESSAGES_PROVIDER,       useExisting: TranslateService     }
Do you have any ideas on why I might be getting this error in the console? Any help is greatly appreciated. Thanks!

I downloaded your sample project and ran it locally. I do not see the error that I'm getting, so I updated the project to Angular 9 + Ivy (which is what we're using) and now I get the same error message using your project.
So...the error appears to be related to changes made either from Angular 8-9 or the use of Ivy.

I disabled Ivy after upgrading your project to Angular 9 and the errors went away. So it seems to be related to Ivy.

Hi @psenechal sorry for late reply and thanks always for your contribution! I'll try to update to the angular 9 and understand how can i remove that error with ivy. I'll let you know as soon as possibile, sorry it's a busy period

I update the project to angular 9. here is the branch:https://github.com/xtreamsrl/ngx-validation-errors/tree/feature/angular-9-update. I'm experiencing the issue that you noticed in the bootstrap version, the material one is ok. I'll investigate and come back to you

Hi @psenechal I updated the library to angular 9. I moved the internal implementation to a statusChange subscription base. It should improve efficency and it seams that the error does not appear anymore. Let me know if it is OK. There are no external breaking changes

Thanks Luca...I’ll update tomorrow morning and let you know how it goes. The current version seems to work with Angular 10 so I’ll also make sure this new version also works. It doesn’t look like they introduced anything new that would be a breaking change.

Hi Luca...thanks again for the updated version.
While that appears to have resolved the issue with the ExpressionChangedAfterItHasBeenCheckedError, the form submit directive I created to mark the form fields as touched and dirty no longer works properly.
From what I can tell, the app-custom-errors container isn't being populated with the error when the form fields are marked as touched and dirty by the directive. I have to actually modify the form fields for the app-custom-errors container to get populated with the appropriate error message.
Would you have any ideas for me on how to possibly modify the form submit directive to fix this? Is there anything I can do to mark the form fields as status changed in my directive?
Thank you sir...appreciate the work and the help.

I was able to get around the issue by setting the value to the existing value which I'm guessing triggers a status change so that the validation directive executes and populates the container:
abstractControl.setValue(abstractControl.value);
It's a bit of a hack, but it gets everything working again for now. Let me know if you have any different suggestions.

@psenechal Interesting, there isn't a way to listen to touch or dirty. I'll try the solution explained here https://stackoverflow.com/questions/41337024/how-to-observe-touched-event-on-angular-2-ngform that should do the trick so i can update the component silently. I'll work on this tonight

Hi Luca,
I updated to the new version and removed the "hack" I added to the Form Submit Directive. Everything is working perfectly now. Thanks so much for looking at these issues...the team is really happy about the console errors not appearing any more too. Cheers!


Update doc on Configuring an Active-Active Deployment

Description:
The document on Configuring an Active-Active Deployment should be updated according to new config model.

Related Issues -
Following are the pages related to Active-Active Deployment documentation.

https://github.com/wso2/docs-apim/issues/268
https://github.com/wso2/docs-apim/issues/310
https://github.com/wso2/docs-apim/issues/313
https://github.com/wso2/docs-apim/issues/315


proposal: crypto/x509: Export CertPool Length

Proposal Details
Actually the only way to know if you have some into a certpool is
if (cp.Equal(&x509.CertPool{})) {
fmt.Printf("CertPool is empty")
}

This way, we only can now if empty or not, but we don't now any about the number of certificates added.
I propose to export the len method
func (s *CertPool) Len() int {
	if s == nil {
		return 0
	}
return len(s.lazyCerts)
}

this would be at best misleading. if using the system verifier, the size works be zero but there are roots that can be used.


Fix: broken links in CONTRIBUTING.md

PR description
Fixed Issue(s)
The links in CONTRIBUTING.md pointed to outdated or incorrect pages on the Hyperledger wiki.
How was it fixed?
Updated all the links to their current locations on the lf-hyperledger.atlassian.net platform:

"I just have a quick question"
"First contribution workflow"
"Did you find a bug?"
"Issues"
"DCO"
"Suggesting Enhancements"
"Pull Requests"
"Code Reviews"
"Security contributions"
"Roadmap"
"Code of Conduct"
"Governance".

Thanks for sending a pull request! Have you done the following?

[x] Checked out our contribution guidelines?
[x] Considered documentation and added the doc-change-required label to this PR if updates are required.
[ ] Considered the changelog and included an update if required.
[ ] For database changes (e.g. KeyValueSegmentIdentifier) considered compatibility and performed forwards and backwards compatibility tests

Locally, you can run these tests to catch failures early:

[ ] unit tests: ./gradlew build
[ ] acceptance tests: ./gradlew acceptanceTest
[ ] integration tests: ./gradlew integrationTest
[ ] reference tests: ./gradlew ethereum:referenceTests:referenceTests

Per our contribution guidelines we do not accept non-code contributions: https://github.com/hyperledger/besu/blob/main/CONTRIBUTING.md#guidelines-for-non-code-and-other-trivial-contributions
I have to close the PR without merging, because of that.


Auto generated tenants API example

Overview:
Experimental branch for autogenerated TypeScript types from OpenAPI spec. Work in progress.
Related Github Issues:

TUI-19

Summary of Changes:
Add npm run api command
Testing Steps:
Instructions in wiki
Notes:
Github actions may have a difficult time with this, depending on our build configuration and tests.

Starting new repo for tapis-typescript


How can I prevent superagent from following redirects?

I'm trying to call a backend service via superagent and expecting a response object with a 302 status and a location header. I need to process the response on the client, rather than having superagent assume I want the result of the GET from the location header. Is this supported?
I've tried setting superagent.post('/my-service').redirects(0).send( ... );, superagent.post('/my-service').redirects(-1).send( ... ); and superagent.post('/my-service').redirects(false).send( ... ); - none of these seem to stop superagent from following the redirect and incorrectly giving me a 200 OK when the actual response I need is the 302.
I can see the redirect being followed from my server logs.

In node it should work. In the browser it's just not possible at all.

Oh, is that a limitation of XMLHttpRequest itself?

Just checked. Yeah it's a limitation of XMLHttpRequest. That's a shame. Seems like a pretty odd design choice but it is what it is. Will work around it, thanks.


Fix 1 1 2

Types of changes

[ ] Hot fix (emergency fix and release)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Documentation (change which affects the documentation site)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Release (develop -> main)

Quality assurance (all should be covered).

[ ] My code follows the code style of this project.
[ ] Documentation for my change is up to date?
[ ] My PR meets testing requirements.
[ ] All new and existing tests passed.
[x] All commits are signed-off.

Summary
Key links:

Sonar coverage

Before you merge

Ensure it is a 'squash commit' if not a release.
Ensure CI is currently passing
Check sonar. If you are working for a fork a maintainer will reach out, if required.

Trestle upgrade to OSCAL v1.1.2 - STATUS.pdf

Specification of semantic-release 7.33.2 has been restored to the setup.cfg. This was an oversight (nicely caught by Jenn in her review) in an overzealous attempt to eliminate all warnings during make test. Upgrading to a newer version of semantic-release should be on our radar for a next release of trestle. For now, we can live with the 17 warnings that occur. Note that the prior trestle release has 54 warnings, so we are better off with respect to warnings.

Thanks for the reviews! I think all comments made thus far have been addressed. Would appreciate a re-review toward getting the next release of trestle closer to reality.


Fix tf version dimension issue

Fixes #139
@vmirly can you double check that this looks good?

Hello @rasbt! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file ch15/ch15_part1.py:

Line 278:9: E122 continuation line missing indentation or outdented
Line 280:9: E122 continuation line missing indentation or outdented
Line 281:1: W293 blank line contains whitespace
Line 285:9: E122 continuation line missing indentation or outdented
Line 287:9: E122 continuation line missing indentation or outdented


Add-Migration does not work

An unhandled exception occurred while processing the request.
SqlException: Invalid column name 'Rating'.
Microsoft.Data.SqlClient.SqlCommand+<>c.<ExecuteDbDataReaderAsync>b__164_0(Task<SqlDataReader> result)

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

ID: cca95f5e-0fe4-bbd2-fd23-13562e02f888
Version Independent ID: c3fb9673-4da6-5eeb-08bb-6f8e7bc8e0c6
Content: Add a new field to an ASP.NET Core MVC app
Content Source: aspnetcore/tutorials/first-mvc-app/new-field.md
Product: aspnet-core
Technology: aspnetcore-tutorials
GitHub Login: @Rick-Anderson
Microsoft Alias: riande

I think Update-Database is the problem. It says this:
Applying migration '20191119023842_InitialCreate'.
Applying migration '20191119023842_InitialCreate'.
t does not Create the Table with Ratings in it.

The tutorial works. I'm not sure what step you missed. You might try asking on StackOverflow,.

When I ran Update-Database, it executed the InitialCreate.cs I made in the "Add a model" section. It DID NOT run the Rating.cs I just made. Will it work if I delete InitialCreate.cs?

probably

lol.
When I right-click<PHONE_NUMBER>3842_InitialCreate.cs, the option "Exclude from Project" is available.
What will happen if I click that? Will InitialCreate.cs be prevented from running?

I solve this problem today when i work this tutorial.

Add this div in Edit.cshtml
You add just Create.cshtml, not in Edit.cshtml probably.


DBSCAN example datasets?

hi,how can i get the file 'contact-tracing.csv'，i can not download from the url"https://lab.mlpack.org/data/contact-tracing.csv"

@zoq should we add this CSV to mlpack.org/datasets/examples?  If so let me know... I don't have a copy of the file though (sorry @OberstWB)

Yes, I'm currently updating the binder jupyter instance, one step is to consolidated all the datasets into one place.

Hey @OberstWB, we got the website updated with new locations for the datasets.  The contact tracing dataset can be downloaded here:
http://mlpack.org/datasets/notebooks/contact-tracing.csv
I think in the upcoming days we might update the URLs to use mlpack.org instead of lab.mlpack.org, but in any case, all the notebook datasets are available in that directory: http://mlpack.org/datasets/notebooks/ :+1:


Cannot build on Go 1.15.5 on macOS

macOS 10.14.6
go version go1.15.5 darwin/amd64

Hello, you don't provide an executable for this for macOS, and when I try to build using go build ... after a scary amount of dependency/modules related messages, it eventually falls down with an error about nats-io. To be fair, from what I've read, the problem stems from an Modules related error at the nats-io end of things, rather than any problem with your package. This is the error I get...
go: found golang.org/x/image/font/inconsolata in golang.org/x/image v0.0.0-20200927104501-e162460cd6b5
go: found github.com/kr/pretty in github.com/kr/pretty v0.2.1
go: github.com/go-kit/kit/examples/stringsvc4 imports
	github.com/nats-io/go-nats<EMAIL_ADDRESS>parsing go.mod:
	module declares its path as: github.com/nats-io/nats.go
	        but was required as: github.com/nats-io/go-nats

I've tried a few suggested fixes. Maybe they are out of date, but none of them work. I've also left a message on the Golang forum so if I get an answer there, I'll let you know.

Weirdly, the problem occurs if I build from the command line under go 1.15.5, yet if I build from within VS Code, using go: build workspace, it builds successfully. I can work with this now. I'm just very puzzled as to why it's happening...


Add Cocoapods podspec generation.

Motivation:
A number of users have asked for the ability to install SwiftNIO
via Cocoapods. This is an entirely reasonable request, so let's
do it.
Modifications:

Added a script that can generate, and optionally upload,
podspecs.
Made a slight modification to the dependency script to provide
target-specific dependencies.

Result:
Users can use cocoapods if they prefer.

Feedback addressed.

Could this be updated for version 1.9.5? Only 1.9.2 is available for now.

@fcorneli thanks for the report, updating them right now! I'll also add that to our release checklist so we shouldn't forget in the future...

ok, 1.9.5. uploaded


Allow creating a dependency based on an operating system version

I would like to be able to add a dependency in a Chocolatey nuspec file to an OS version number using the same rules as Nuget versioning. This is primarily needed in cases where the scripts or programs rely on OS features that can't be fulfilled in earlier OS versions. e.g. Hyper-V doesn't exist in WinXP or Win7.
http://docs.nuget.org/docs/reference/versioning
Windows OS version numbers

Windows 8.1
6.3

Windows Server 2012 R2
6.3

Windows 8
6.2

Windows Server 2012
6.2

...
...

http://msdn.microsoft.com/en-us/library/windows/desktop/ms724832(v=vs.85).aspx

Shouldn't it be handled by higher-level instruments like Chef or Puppet?

@grv87 if you are using configuration managers sure. :)


[Feature] Get component output(static rendered version)

Hi, It's possible to get a complete static render version of the current opened component ?
My use case: I deliver plain html samples of components for backend implementation, they don't actually use my components, they look for a specific component and get the html.
Would be nice have a panel "html version"(like action logger and notes), when the user clicks, it renders the component in this section.
Why not just press f12 and get from dev tools: I prefer a clean output with just the current component inside, without any other data.
Let me know if is possible to implement this.

@mateuspv We won't do a SSR rendered version of storybook pretty soon. We might do it someday.
You could customize addons as you want.
See: https://getstorybook.io/docs/react-storybook/addons/introduction

Thanks for de feedback guys...It's on my backlog, but I'm not with too much free time now ^^

I am preparing a little workshop about storybook for my colleagues, and decided to implement this plugin as an example: https://github.com/evgenykochetkov/react-storybook-addon-static-markup
Enjoy:)

@evgenykochetkov this is cool.
Add it to here: https://getstorybook.io/docs/react-storybook/addons/addon-gallery
(click on the "EDIT THIS PAGE" link on the top)

@arunoda, thank you! But are you sure about adding it to addons gallery? I think that this use case is quite unique and the plugin will be useless for 99.9% of people. At least let's see if @mateuspv likes it😄


Webview cache not being used (iOS)

I noticed, in iOS, that the initial load after an app kill the site was taking too long to load and then noticed all resources were loaded from the external site not using the cache.
I found this bug comment, which helped.
Adding this to the config.xml, the cache does get used and initial load is much faster:
<preference name="AppendUserAgent" value="[my_app_name_here]" />

This is interesting.  Does this bug only happen when you are using a custom user agent or does it happen even when your not tryin to change the useragent?

As far as I can tell its the default behaviour for any Cordova app, it might be only on Cordova 6.3.* that's what I'm using.
I created a HelloWorld app with an iframe and it didn't seem to use caching.
From what I understood the cache is indexed with the user agent which has a unique Id appended every time the app starts, this config key replaces the unique Id with a custom string which allows caching to work in different sessions.
Another thing I noticed is that you can't rely on the safari devtools to tell if it was loaded from cache or not, best just to measure loading time with the network throttled.

okay, we'll keep our eye on this.   I appreciate you letting us know about it so we can share with others who run into the same issue.  I'm going to close this issue, and watch the cordova one

adding it to the next release.  If this is still the default behavior at that point, we will likely want to include it to just make things run faster for everyone


Created UTC timestamping  functionality

A few things I should probably note:
Not entirely too sure if this would work, I mainly based it off my conversation with Dhruv and Tony.
The timer setup is the thing I'm most concerned about - I'm really not sure if I did it right.
Also, the period of the timer hasn't been set yet

Fixed some of the formatting issues. A few things still need to be done:
Changing up the period & switching from tim4 to something else
Will hopefully have that up by tomorrow!

Switched to timer12 instead of 4 (this way we can count to at least 1000 with some cushion)
I set the period to 0.001 (1000hz0. The idea behind this is each tick represents 1 millisecond. Let me know if this works because I'm not even entirely sure.

Thanks for getting this in. I have a few comments.
Do you have a dev board that you can test this on?

Unfortunately not, is there a way someone could test it for me?

Here are the people who currently have dev boards. You can ask around and see if someone is willing:

Aadi
Dhruv
Gordon
Anni
Anthony

Your best bet is probably Aadi if I recall correctly

Here are the people who currently have dev boards. You can ask around and see if someone is willing:

Aadi
Dhruv
Gordon
Anni
Anthony

Your best bet is probably Aadi if I recall correctly

I'm not exactly sure how I'd test something like this.... is it safe to assume they'd know?
Also, the module isn't exactly complete until sensor fusion passes me in the GPS struct, and IIRC that hasn't been done yet.

A few new things...
Switched over to timer14 (this hasn't been configured in ioc yet!)
Fixed the way I set the recentUTCTime (accounted for HHMMSS.00 format)
And other small little details.


Fix parsing Issue 63 https://github.com/3rd-Eden/useragent/issues/63

Proposed solution to issue 63.
Tested with nearly 3000 agents. (using updated reg-ex)

NO "$" left over.
No changes to browser
Changes in "os"

  CloudyTabs/1.4 CFNetwork/720.1.1 Darwin/14.0.0 (x86_64)
  Safari/106<IP_ADDRESS> CFNetwork/720.2.4 Darwin/14.1.0 (x86_64)
  com.apple.WebKit.WebContent/10600.4.8 CFNetwork/720.2.5 Darwin/14.1.1 (x86_64)

used to be "Mac OS X" version: 10
are now "Mac OS X" version: Network
   MobileSafari/600.1.4 CFNetwork/711.0.6 Darwin/14.0.0
   Twitter/6.18.3 CFNetwork/711.0.6 Darwin/14.0.0
   Web/1.0 CFNetwork/711.1.12 Darwin/14.0.0

used to be "iOS version: 7 (or 8)
are now "iOS" version: Network
The version is wrong. This is because, I kept the behaviour of using the 2nd, 3rd and 4th match, if there is no "os_v1_replacement" (see file update.js, last argument to getReplacement())
Most of them only have "os_replacement":
  - regex: '(CFNetwork)/(5)48\.0\.3.* Darwin/11\.0\.0'
    os_replacement: 'iOS'
  - regex: '(CFNetwork)/(5)48\.(0)\.4.* Darwin/(1)1\.0\.0'
    os_replacement: 'iOS'

Changes to "device"

Quit a lot of changes. the data is now much better allocated to
  ua.device.family
  ua.device.major
  ua.device.minor

Some example for changes (old => new)
iPhone5,2 / 0 / 0 => Apple / iPhone5,2 / 0       // applies to all iPad, ipod, IPhone
XT1068 / 0 / 0 => Motorola / XT1068 / 0
Samsung SM-T805 / 0 / 0 => Samsung / SM-T805 / 0
Lumia 925 / 0 / 0 => Nokia / Lumia 925 / 0
D6503 / 0 / 0 => Sony / D6503 / 0
BlackBerry 9000 / 0 / 0 => BlackBerry / 9000 / 0
LG- / E610 / 0 => LG / E610 / 0
Kindle Fire / 0 / 0 => Amazon / Kindle Fire / 0
R831K / 0 / 0 => Generic_Android / R831K / 0
ONE TOUCH 4015D / 0 / 0 => Generic_Android / ONE TOUCH 4015D / 0
ALCATEL ONE TOUCH 4030X / 0 / 0 => Generic_Android / ALCATEL ONE TOUCH 4030X / 0

Nokia / 0 / C1-01 => Nokia / C1-01 / C1-01

The Nokia now has some duplication. Probably due to keeping $2, $3, $4 / see above

It looks great, but I do wonder about the potential performance impact of the newly introduced function call.

replaceRegExResult()
is only called once when a user-agent is found. And then up to 2 extra times, when "os" or "device" are accessed.
Given that to find the result for each user-agent many regexp against the UA have to be checked, this will IMHO hardly matter.
Accessing "os" or "device" also runs more of the regex matching the UA. So again the time goes to matching those.
It would be possible to reduce the inner loop
for(var i=1; i<=9; i++) { // $1..$9

currently the maximum is $4.
Limiting it to far might break future update. But setting it to max 5 might be ok,
If you have an idea how to improve that otherwise, let me know.

If performance is a big issue, here is an idea (but not one I will do any work on)
Several regex start the same
parser[0] = new RegExp("(Firefox)
new RegExp("(Fennec)
new RegExp("(Opera

But detecting them must take into account, that the term is not optional (no ? or | ). With a good regex analyser terms can also be found in the middle of the regex.
For those a "guardian" could be created (a regex or indexOf) checking if the term in the regex does exist.
If the first such regex is reached, the guardian is run. If the guardian failed all of the guarded regex can be skipped.

Btw, I thought about avoiding the duplication.
Nokia / 0 / C1-01 => Nokia / C1-01 / C1-01

I tried keeping track (in update.js) whih $n are used. But then I ended up getting other issues.
e.g. LG has a pattern that only uses $2. When the code detected $1 was not used and put it on the first free field, it was replaced by "LG-". Not good.either.
I might still look at something where the fallback stays as it is ($2 in 2nd field, $3 in 3rd. $4 in 4th), but is est to empty if that replacement was used before. That way it will not introduce using $1 or $5... except where they are in the downloaded data.

I added some more to optimize the speed and remove duplicate fields.
This also pushed another commit, which is unrelated.
Revision: da66fbe56dcfd013013175c967ae1385ba10eb82
Author: User4martin <n>
Date: 11/08/2015 19:44:52
Message: Allow setting values for unknown family or version

That way one can localize the "Other" value (or in my case set it to undefined.

*** Important
This needs a rebuild of the regex file, as the format changes
Author: User4martin <n>
Date: 11/08/2015 21:44:44
Message: check for duplicate replacements / optimize replacements in found result

Does 2 thinks (sorry one commit)

For each field it stores what $n needs to be replaced. This avoids calling replace on none existing $n in the live code

If a $n was already used in a previous field, then it is not used as default (default then is empty)
So if

  - regex: 'Android Application[^\-]+ - (Sony) ?(Ericsson)? (.+) \w+ - '
    device_replacement: '$1 $2'
    brand_replacement: '$1$2'
    model_replacement: '$3'

Makes the 2nd field "$3".
The default for the 3rd field is also $3, therefore the 3rd field will be empty.

@3rd-Eden Ping. Is there a reasons this hasn't been merged? Just wondering if this isn't a valid improvement/fix then some feedback could probably move it in the right direction.

Because I'm enjoying my free time.

@3rd-eden apologize, didn't mean to make that sound as snarky as I'm sure it came across. Could this PR be merged though?


Unsupported operand types

$books = Book::searchByQuery([
'multi_match' => [
'query' => 'moby dick',
'fields' => ['name^3', 'description']
]
])->paginate(10);
returns Error
Unsupported operand types
screenshot

The problem is from the ElasticquentPaginator class
you need to change line 25 to this:
$this->lastPage = (int) ceil((int)$total / (int)$perPage);
Note: don't change the vendor directly, either fork this repo and make the changes to it, then use it on your application.
or make a pull request to this repo


How do the three input folders correspond to each other?

In this code, i found there have three input folders: images_all, masks_all and labels_all,
could U please help me in detail how to set the original images and masks to these folders?
because i found that a image correspond many masks.

images_all - images in flat structure
masks_all - generated masks (3 channels masks) with borders. See create_masks file for details
labels_all object labels, i.e. single channel masks where each object has different int value for its pixels

#7 may answers your question. Let me know if it does. @lpengc


🐛 Bump go modules for getting rid of CVEs

Co-Authored by: @tuminoid

/retest

/test-ubuntu-integration-release-1-1

/assign @mboukhalfa


Fix large file writes under preview 2 implementation

Fixes #7390

Should this be backported to 14.0.x as well?

Yes. I was going to write more tests and I was chasing down some other potential improvements as well, but the bug here is obvious now, so I think we should just land this patch in main and in the next 14 release to go out.


Jainammm as primary maintainer of k6 artifacthub integration listing

I have already listed you as a CODEOWNER for the k6 integration on artifacthub. When https://github.com/keptn-contrib/artifacthub/issues/183 is complete, please start migrating documentation across to artifacthub or at least modify the content to point to this repository.
At this point you will "own" the k6 integration for Keptn as you are now the most knowledgeable on this area, it makes sense for you to be the primary maintainer.
This issue can be closed when:

https://github.com/keptn-contrib/artifacthub/issues/183 is done
The artifacthub listing points to (or removes the need for) this repo

This issue is outside the GSoC scope and is a separate, follow-on activity.

thanks @agardnerIT


Output Artifact -> meta container

Today we produce images for VM infra, but for on prem environments there are a set of use cases that could benefit from a meta-container that lumps all of the repos deps into a local container.
@luxas created a container like this in the past, but the purpose of using wardroom is to allow others to generate the artifact(s).
/cc @detiber

https://github.com/xakraz/kubeadm-installer

Closing as I believe this falls outside the scope of what wardroom should do. Reopen if necessary.


improve a number matcher in watched keywords

don't match an ipv6 format, make sure we only start matching on the first digit.

some more info - I expect this to cause at least one less FP, and it should not remove any TP. If it does then I'll review

Metrics:
Before: TP: 1067 FP: 8 NAA: 0
After: TP: 1067 FP: 7 NAA: 0

I'm confused. This says that it's "improve a number matcher", which implies that it's changing something, but the change to the watchlist is to only add an entry. Was something already removed somewhere else?

Seems to have gotten lost in the rebasing, will check this @makyen

actually, this may have been something that I noticed for two reasons, it looks like one of the migrations to watched_numbers makes this PR redundant

relevant info: #13187


[Fix #7358] Fix an incorrect autocorrect for Style/NestedModifier

Fixes #7358.
This PR fix an incorrect autocorrect for Style/NestedModifier when parentheses are required in method arguments.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

Nicely done!


Fermi GBM DoL (Daughter of LocBurst)

The legacy DoL ported to python including documentation and tutorials.

I did a quick review of all the changes, and verified that all the unit tests passed. I still question why dol4.exe simply calls the legacy_dol script as another process, but don't consider that design to be enough to prevent the initial merge and release of the DoL to main.


Update free-programming-books-langs.md

What does this PR do?
Add resource(s) | Remove resource(s) | Add info | Improve repo
fixes #7980
For resources
Description
Why is this valuable (or not)?
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[ ] Read our contributing guidelines.
[ ] Search for duplicates.
[ ] Include author(s) and platform where appropriate.
[ ] Put lists in alphabetical order, correct spacing.
[ ] Add needed indications (PDF, access notes, under construction).
[ ] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

Thanks, and welcome to free-programming-books!


Awesome package - quick question

This looks amazing and solves two major issues I am having, tile loading and shared config code.
I am having issues where the marker images are not loading, have I missed something? (I have used the hello world examples)
Also, how nicely does this play with other leaflet js files, markercluster for example?

Also, is it possible to pass more than one tilelayer through from the leafletOptions helper?

Where is the map looking for the markers? You could load them externally through something like mapbox or just have local files. I added awesomemarkers & markercluster plugin capabilities so if you want to use those just follow the readme. multiple tilelayers (like clouds, traffic) should be doable, this package doesn't do anything fancy, it basically just makes sure leaflet loads without race conditions.

Thanks again for your work on this.
The map is looking in "http://localhost:3000/lib/config/images/marker-icon.png" for the markers, there are no markers there.
I am never sure how to do contributions on github as I pulled this in through atmosphere. However, two suggestions that I would work on if I could do it in a constructive way.
Rather than having defined plugins perhaps it could just spool through all js in a defined folder, then there is no need to define it, just point it to the addons folder. This would allow for custom shares code.
Multiple tile layers is tricky. This requires one in the leafletOptions object but I am currently loading a set into a group to allow for them to be toggled.
var osm = new L.TileLayer('http://{s}.tile.openstreetmap.org/{z}/{x}/{y}.png');
var os = new L.BingLayer("xxxxx", {type: 'OrdnanceSurvey'});
var road = new L.BingLayer("xxxx", {type: 'Road'});
var aerial = new L.BingLayer("xxxx", {type: 'Aerial'});

var baseLayers = {
    'OS': os,
    'Roads': road,
    'OpenStreetMap': osm,
    'Aerial': aerial
};

Icons I have resolved by adding
L.Icon.Default.imagePath = '/images/leaflet';

I like your idea about just a dump folder for all the plugins! I'm just not sure how to implement it, can JS view the contents of the public folder? If so shoot me a gist & I'll add it.
for the layers, I'd save them to your map namespace. so var os = leafletMaps.maps.myMap.os = new L.BingLayer. That's where I save all my map related stuff.


flash  android

when switch the keyboard , android will  flash.

YES SAME ISSUE

Same thing at me, only on android, it happens randomly, not always.

Hi @jiangchunyun,
Please add more information if this is still relevant.

Does this happen in the demo app?
a. If yes, on what device - real\emulator, OS and type?
b. If not, please provide an example that it happens on.
A video of the problem if you can.

I made a little video, you can watch it below.
The device is a Samsung S5, running android 6. The keyboard flickers starting second 20 of video (until second 20 do not flickered, flickering is random at me). On iOS I did not meet this problem.
https://vimeo.com/417267483

It looks to me like the height of the keyboard is changed, is that what you mean?
I was unable to reproduce with my code on my device. If one of you can provide an example in which this is reproduced on I'll try to investigate further.

No, it's a very fast flash immediately after second 20.
I will try to setup an example.

@M-i-k-e-l
I was able to setup an example repo:
https://github.com/ggunti/math-keyboard-example
You can simply run it for android using the react-native run-android command.
In order to reproduce the keyboard flash, for example, try to switch multiple times between Text and Basic Math keyboards:

Sometimes, randomly, you should see the flash (like in the video I linked above).

Hi @ggunti,
I think I was able to reproduce the problem with your example (thanks for the effort!), however I do not think the problem is within react-native-keyboard-input.
If what I saw was what you meant, then I think this has to do with the FlatList.
If you allow YellowBox (remove these), you will receive a warning about "Each child in an array should have a unique "key" prop.".
I can't say for sure but when I changed the usage from FlatList to ScrollView (see here) the problem did not occur.
I suggest that you create a unique key mechanism for your items and hopefully that will solve the problem with FlatList.
I will not close the ticket for now, with the hope that I am correct and you will close it once you solved the issue. Please either close it or let me know this is not resolved and you still think the problem lies within react-native-keyboard-input.

@M-i-k-e-l Thank you for your feedback.
The "Each child in an array should have a unique "key" prop." warning was not coming from the FlatList, it was because of the renderRow function, I forgot to pass a key to the renderButton function:

Now it's fixed.
So, now we do not have the "Each child in an array should have a unique "key" prop." warning, but the flickering still happens. I will check what happen if I change the FlatList to ScrollView, as you suggested. I will come back soon.

@M-i-k-e-l
I think you are right, if we use ScrollView instead FlatList, there is no flickering anymore. The problem is that in my use-case I will have to use the FlatList component because I will need to load more comments when end of the list is reached (infinite-scroll).
So, I'm wondering what could cause the flickering in case of FlatList component. It makes some additional undesired re-rendering when keyboard changes?
FlatList is used frequently, so that's why also other people reported 'flash bug'. Anyhow, it looks that the bug is not in the react-native-keyboard-input package. Anyhow, it would be nice to find a solution also for the case when FlatList is used.

Since it's not the key, then I'm not sure what's the correct way to approach this.
I agree that using FlatList should be possible and not cause issues.
I'll try to take a look at this again at a later date, please do not close it.


add alert to monitor the observability service on managedcluster side

tested in my cluster, when I disable the obs for some managed clusters, I receive this alert:

I will add the runbook later /hold

/hold I will add the runbook later

/unhold


Open cypher

Are there plans to support Neo 4js open cypher query language?

Let's track this issue in #340. That was the same question actually. Will update it shortly.

@pjebs Would you mind to close the issue and rename the old one?


Add test case for get ssh_authorized_keys

Related issue:
https://github.com/rancher/os/issues/2579
============================= test session starts ==============================
platform linux -- Python 3.5.2, pytest-3.9.2, py-1.7.0, pluggy-0.8.0
rootdir: /opt/os-test_project, inifile:
plugins: xdist-1.24.1, forked-0.2, cov-2.6.0
collected 1 item

engine/test_ros_config.py Formatting '/opt/os-tests/images/BQB4PNSZ.qcow2', fmt=qcow2 size=10737418240 encryption=off cluster_size=65536 lazy_refcounts=off refcount_bits=16
.

========================== 1 passed in 137.99 seconds ==========================

Process finished with exit code 0

============================= test session starts ==============================
platform linux -- Python 3.5.2, pytest-3.9.2, py-1.7.0, pluggy-0.8.0
rootdir: /opt/os-test_project, inifile:
plugins: xdist-1.24.1, forked-0.2, cov-2.6.0
collected 1 item

engine/test_ssh_key_merge.py Formatting '/opt/os-tests/images/AJKQWHMI.qcow2', fmt=qcow2 size=10737418240 encryption=off cluster_size=65536 lazy_refcounts=off refcount_bits=16
.

========================== 1 passed in 288.53 seconds ==========================

Process finished with exit code 0

LGTM


order of operations

Check whether the selecting order of operations questions work properly + have an option “other” where a person can describe a more complex procedure.
source

Copied from Suzy who opened a duplicate bug: Hope we can figure out a way to leverage the data input to facilitate completion of this section. Eg. a button to Auto-populate text answers from above questions (like in the PDF (in the default Q order), so a person can manually arrange into a list. As a stretch goal, could create a drag-and-drop interface where each named process can be dragged into a sequence.

In this version, this is resolved in the simplest way by asking people to report their order of steps in an open-ended question. In the future, we can think of better ways such as selecting entire steps from a menu, which will allow stuff like having an unlimited number of artifact removal decisions, multiple filters...