Docs - Add tables and header fixes to primary readme

Converted block/reward information to more organized tables in primary readme for readability
Fixed sub-headers under "development tips and tricks" in primary readme for readability

Thx for your PR. To let you know, there is some change on the way for the README.md. It's in progress, your suggested change probably need to be adapted depending on merged changes, block reward may be moved somewhere else.
If you want to contribute to the documentation or see some of the ongoing work on it, take a look at Docs4Doge :)

Rebased to 1.14.4-dev, latest PR is #2395


RFM - Join.me

https://chocolatey.org/packages/join.me/<IP_ADDRESS>2500
Got a report of this being out of date, although I believe there was an issue with going to the official distribution point. If anyone is interested, I believe @mikecole can provide the details on this one.

I do not recall the issues I had. I will try to review and provide an update in a week or so.

@mikecole I looked at this a few weeks ago and the only thing I could find was a commit where you simply said your were abandoning the pkg.
https://github.com/mikecole/chocolatey-packages/commit/1a79a74885c78b1ff496c4f77717f58217601817

I'll resurrect it and try again. IIRC there was something squirrely with
the URL doing a redirect, and I didn't have the PS skills at the time to
overcome it. Round 2 coming soon!
On Mon, Mar 20, 2017 at 2:01 PM, Spunkie<EMAIL_ADDRESS>wrote:

@mikecole https://github.com/mikecole I looked at this a few weeks ago
and the only thing I could find was a commit where you simply said your
were abandoning the pkg.
mikecole/chocolatey-packages@1a79a74
https://github.com/mikecole/chocolatey-packages/commit/1a79a74885c78b1ff496c4f77717f58217601817
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/chocolatey/chocolatey-package-requests/issues/113#issuecomment-287896667,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABHL6_e7J3r8JoUprFOz079o3XfHYblAks5rnuk4gaJpZM4Mhu3D
.

I resurrected the package and played around with it a bit. I found that the only URL I could find was https://www.join.me/Download.aspx?installer=win&webdownload=true. However, the checksums on that download constantly changed. I was never able to make it pass checksum validation because I would get erratic results on the checksum of the downloaded file.

If license allows it u can embed it and then checksum problem is gone. Au can make that automatic.

IIRC the licensing doesn't allow distribution rights.

Exactly. This is where I got stuck before. I'm not sure where to go from here.

@mikecole if you're to embed the software, you'll need to contact the authors and ask them to give you the rights to redistribute the software.

I pursued that route, but it doesn't look like they will agree to letting us redistribute. Open to any other creative solutions.

There are creative solutions:

Add into the docs that you should always invoke it with --ignore-checksum
Use Get-WebFile  from core ext to download the url locally and then Install-ChocolateyInstallPackage to install it just as it is embedded. This basically circumvents chocolatey checksum config. Make sure to add that in ## Notes secction.

OK, I can give that a try. Thanks for the info.

Use Get-PackageCacheLocation to get temp dir where to download too.

Are you aware of any packages currently using this approach that I can use as an example?

We don't recommend using Get-WebFile directly. Keep in mind that if it downloads over HTTPS, there is less issue with checksums currently. However intentionally circumventing a safety mechanism is against the community repository guidelines.
I would see use Install-ChocolateyPackage with an HTTPS location and mention in notes that the checksums constantly change, so they need to run with --ignore-checksums if they have turned off the feature allowEmptyChecksumsSecure.
Note this is not only the recommended approach but also the only approach we would allow approved. It is ALWAYS the user's decision to decide to ignore checksums and never a packager's decision. Assuming otherwise puts the community at risk.

I made Rob's recommended changes and pushed to the gallery:
https://github.com/mikecole/chocolatey-packages/commit/247dad921601f5b83eac798507597175b6433abb

It was initially someone else's recommendation.

@majkinetor 's recommendation.


Fix an index in tutor.zh_tw

Codecov Report

Merging #1996 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1996      +/-   ##
==========================================
- Coverage   74.65%   74.63%   -0.02%
==========================================
  Files          77       77
  Lines      125954   125954
  Branches    28188    28188
==========================================
- Hits        94029    94009      -20
- Misses      31921    31941      +20
  Partials        4        4

Impacted Files
Coverage Δ

src/if_xcmdsrv.c
84.25% <0%> (-1.86%)
:arrow_down:

src/libvterm/src/screen.c
73.34% <0%> (-0.12%)
:arrow_down:

src/ex_cmds.c
77.57% <0%> (-0.09%)
:arrow_down:

src/channel.c
80.05% <0%> (-0.05%)
:arrow_down:

src/window.c
80.86% <0%> (-0.04%)
:arrow_down:

src/evalfunc.c
82.58% <0%> (-0.02%)
:arrow_down:

src/gui_gtk_x11.c
47.29% <0%> (ø)
:arrow_up:

src/os_unix.c
52.75% <0%> (+0.04%)
:arrow_up:

Continue to review full report at Codecov.

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

i'll include it, thanks.


Ambiguity with BSD license trove classifier

PEP 639 will make this a little better, but once documentation exists we should at least document that looping through the licenses looking for a trove classifier match is ambiguous in some very common cases.

#6 adds a test that checks all trove classifiers are covered. If any new classifiers are introduced the test will start failing indicating which should be added.


jnp.linalg.lstsq produces NaN tangents

Description
One-line repro:
jax.jvp(lambda a: jnp.linalg.lstsq(a, jnp.arange(3.)), (jnp.eye(3),), (jnp.zeros((3, 3)),))[1][0]
# Array([nan, nan, nan], dtype=float32)

Taking a quick gander at jax.make_jaxpr(jnp.linalg.lstsq)(jnp.eye(3), jnp.arange(3)), I don't see any custom_jvps. I'm guessing this is something to do with duplicate eigenvalues in the SVD decomposition, which IIRC pose problems for autodiff.
(Not a high-priority problem for me -- I have my own lstsq primitive wrapper I'll be publishing soon, I just stumbled across this whilst testing it against core JAX.)
What jax/jaxlib version are you using?
JAX 0.4.5, jaxlib 0.4.4
Which accelerator(s) are you using?
CPU
Additional system info
Linux
NVIDIA GPU info
No response

For anyone else getting bit by this bug -- I now recommend Lineax, which is our new library for linear solves and linear least squares. In particular it doesn't suffer from this bug.


AddAuthorization Suggested Actions do not Work

In the section "Authorization: AddAuthorization overload moved to different assembly" it is stated that:
New behavior
AddAuthorization methods exist in Microsoft.AspNetCore.Authorization.Policy. AddAuthorizationCore is the new name for the old methods.

And that the recommended action is:
Either add a reference to Microsoft.AspNetCore.Authorization.Policy or use AddAuthorizationCore instead.

The package Microsoft.AspNetCore.Authorization.Policy does not contain the method AddAuthorization as of the latest version (2.2.0). I need my authorization library to support both .NET Core 2.2, and .NET Core 3.0 apps, so it's important to have a solution to this problem.

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

ID: df2066cb-244d-0b1b-cf46-428b198e8601
Version Independent ID: e52631de-28de-77c2-082f-e79b89b17f04
Content: Breaking changes, version 2.2 to 3.0 - .NET Core
Content Source: docs/core/compatibility/2.2-3.0.md
Product: dotnet-core
GitHub Login: @gewarren
Microsoft Alias: mairaw

Actually, I just found this function in the github for this package here: https://github.com/aspnet/AspNetCore/blob/master/src/Security/Authorization/Policy/src/PolicyServiceCollectionExtensions.cs#L47
However, despite the fact that I can see class PolicyServiceCollectionExtensions, it does not add the AddAuthorization method. Is this code not available in version 2.2.0?

Since the suggested workaround isn't working for me, I'm using the following workaround:
#if NETSTANDARD2_1
        public static IServiceCollection AddAuthorization(this IServiceCollection services, Action<AuthorizationOptions> configure)
               => services.AddAuthorizationCore(configure);
#endif

.net standard 2.1 is only supported on core 3.0 so if you build for that target, your library will not work on 2.2

You can create a standard 2.0 library and it will work on 3.0 and 2.2, and then I believe you would only need to reference one of the two libraries with the method in it.

I opened the library and I can confirm I don't see that method there.

Related to #14525
@ajaybhargavb can you comment? I only see AddAuthorizationPolicyEvaluator in that library.

@HaoK can you help?

So its a breaking change in 3.0, AddAuthorization was moved into the Policy dll.
For 2.2, its still in the base dll, you will need to have a different binary for 2.2 as compared to 3.0

.net standard 2.1 is only supported on core 3.0 so if you build for that target, your library will not work on 2.2
You can create a standard 2.0 library and it will work on 3.0 and 2.2, and then I believe you would only need to reference one of the two libraries with the method in it.

Yes. My use case is a library that compiles to both .net standard 2.0 and 2.1 (to use both core SDK 2.2 and 3.0). The conditionally compiled code in my sample that works around the problem only compiles in .net standard 2.1, because that's the only target framework that suffers this problem (it wouldn't compile in net standard 2.0 since AddAuthorizationCore doesn't exist in 2.0).

So its a breaking change in 3.0, AddAuthorization was moved into the Policy dll.
For 2.2, its still in the base dll, you will need to have a different binary for 2.2 as compared to 3.0

I don't see AddAuthorization in version 2.2 (the latest on nuget) for the policy library. That's the issue. The documentation claims that this is available, but presumably there needs to be a released version 3.0 of the policy library (which doesn't exist yet).

As the title of the documentation is breaking change document from 2.2 to 3.0, this change doesn't happen until 3.0

@HaoK I believe this issue is reporting that the policy library downloaded from nuget doesn't seem to have the method the documentation says it should have. I grabbed it and threw it in a decompiler and I only see AddAuthorizationPolicyEvaluator in that library, I don't see AddAuthorization.

@Alexei-B I'm curious as to why you would want your library compiling both for 2.0 and 2.1 instead of just 2.0?

Oh so the issue is really the 3.0 package isn't showing up on nuget

In 3.0 Microsoft.AspNetCore.Authorization.Policy is part of the shared framework. See https://docs.microsoft.com/en-us/aspnet/core/migration/22-to-30?view=aspnetcore-3.0&tabs=visual-studio#framework-reference for referencing shared framework APIs from libraries.

Perhaps @Alexei-B isn't referencing the Microsoft.NET.Sdk.Web SDK and that is why it is missing?

@Thraka right, library developers don't reference the web SDK. That's what the framework reference is for.

Oh duh, I missed that part ahahaha

Suggested text:
The core AddAuthorization methods that used to reside in Microsoft.AspNetCore.Authorization were renamed to AddAuthorizationCore. The old AddAuthorization methods still exist, but are in the Microsoft.AspNetCore.Authorization.Policy assembly instead. Apps using both methods should see no impact. Note that Microsoft.AspNetCore.Authorization.Policy now ships in the shared framework rather than a standalone package as discussed in #shared-framework-assemblies-removed-from-microsoftaspnetcoreapp

As the title of the documentation is breaking changes from 2.2 to 3.0, this change doesn't happen until 3.0

My problem is this is documentation that recommends a fix that doesn't exist yet:

Documentation shouldn't recommend suggested workarounds that don't work.

@Alexei-B I'm curious as to why you would want your library compiling both for 2.0 and 2.1 instead of just 2.0?

The library adds authorization behavior and policy common to the platform we are developing. Some of our developers will be working with .NET Core 2.2, and some with .NET Core 3.0. We need to support both. When trying to use our library (unmodified and just built to .netstandard 2.0) with ASP.NET Core 3.0, I get the exception for the missing method AddAuthorization.
Ideally, the fix suggested in the documentation would work, then I could simply modify our csproj to build to .netstandard 2.0 and 2.1, then change the packages for 2.1 to include the Microsoft.AspNetCore.Authorization.Policy library at version 3.0. However, since this doesn't fix I am using the #if based workaround I posted earlier for now.

Suggested text:
The core AddAuthorization methods that used to reside in Microsoft.AspNetCore.Authorization were renamed to AddAuthorizationCore. The old AddAuthorization methods still exist, but are in the Microsoft.AspNetCore.Authorization.Policy assembly instead. Apps using both methods should see no impact. Note that Microsoft.AspNetCore.Authorization.Policy now ships in the shared framework rather than a standalone package as discussed in #shared-framework-assemblies-removed-from-microsoftaspnetcoreapp

I'm fine with this text being used when (and only when) Microsoft.AspNetCore.Authorization.Policy actually solves the problem.

To be more specific, what I'd like to see in the documentation is a notice that the ideal fix of using the policy library isn't available yet, so that other people don't also lose a few hours going down the same rabbit hole I did. It's extremely misleading and quite disheartening.

While I'm ranting about this, the reason for the change comes off as a bit strange:

AddAuthorization is a better method name for adding all common services needed for authorization.

This is ambiguous wording, but aside from that, did we really need a breaking change to go from AddAuthorization to AddAuthorizationCore? Sure, maybe AddAuthorization isn't the best possible name. But then how is AddAuthorizationCore more preferable? Something like AddAnAuthorizer isn't really in the naming convention but might express the meaning a bit better. AddAuthorizationCore is just confusing.

@alexei-B a Microsoft.AspNetCore.Authorization.Policy 3.0 package will not be published to nuget.org. The advice I gave in https://github.com/dotnet/docs/issues/15745#issuecomment-552000932 should already work.

Can you elaborate on the suggestion?
I tried making an ASP.NET application using .NET Core 3.0 using our library without the fix I used and it did not work. I got a missing method exception for AddAuthorization.
I also tried simply compiling our library to .NET Standard 2.1, which failed to compile saying that the method AddAuthorization did not exist.
If this is part of the shared framework, shouldn't the first solution work fine?

Libraries that want to use 3.0 functionality need to cross compile to netcoreapp3.0 and add <FrameworkReference Include="Microsoft.AspNetCore.App" />

OK - so the official line is that it's now not possible to use previously working .NET Standard 2.0 libraries with .NET Core 3.0 if they used any feature that is on the list of breaking changes?
Excellent :|

Correct.

Well, thank you for being straightforwards about it.
I'm dumbfounded that .NET Standard forward compatibility is this fragile.
Since this isn't actually an issue with the documentation and is more an issue with the choice to break forward compatibility, I can't justify holding this issue open.

Note this is unrelated to .NET standard, these libraries were never part of the .NET standard, they only complied against it. The primary breaking change was when these libraries moved from targeting netstandard2.0 to targeting netcoreapp3.0.

That's definitely a more accurate way of putting it. That said, the effect is still that you can't tell if a .NET Standard 2.0 library will work with a .NET Core 3.0 application until it crashes at runtime, unless you rebuild that library for .NET Standard 2.1, which isn't ideal.


Fix profiler accept null.

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

Tests pass?
✔

Thanks for pull request. It helped to see that the change should be made elsewhere.


Resource detection incorrect

Not as much as a bug since I understand how it happens but wondering how to solve this.
As in the logs below it detects the resources available of the host system and not what the docker container is limited to.
The docker resource limits are set so I wonder how I can set the correct max_server_memory_usage for the instances.
2021.05.03 13:26:04.150734 [ 48 ] {} <Information> Application: Available RAM: 251.84 GiB; physical cores: 24; logical cores: 48.
2021.05.03 13:26:14.167103 [ 98 ] {} <Information> Application: Setting max_server_memory_usage was set to 226.65 GiB (251.84 GiB available * 0.90 max_server_memory_usage_to_ram_ratio)

You can specify 'max_server_memory_usage' in server settings.
  configuration:
    settings:
      max_server_memory_usage: <type_disired_value_here>

Thank you. This solved my issue. Sorry for the delay in confirming.


Fix generate_password command on Django 5.1

Fix #1897
The make_random_password method has been removed in Django 5.1. Looking at the most recent implementation, it was calling get_random_string under the hood, and that seems to exists in all versions of Django supported by django-extensions

Any idea of why the CI is failing? All jobs are hitting GitHub's timeout. I would say a test in test_managestate.py is doing shady things, network related I guess.
The test setup could use some simplification, a tox that calls a makefile that calls pytest is too much, the Makefile could be gone, wdyt?

I noticed the problem but didn't spent any time investigating... I noticed that you opened a PR to look at it, but haven't yet seen the results of the run (they seem to be queue'd?)

I need to workflow approval from a maintainer. If you are one, please approve :)


Fix: typo and some mistakes in tests

Here some fixes typos and mistakes in tests

Thanks for the contribution


FW-4504 use person crud apis

Description of Changes

Implemented use of people api for "speaker" crud in Dashboard
Standardized the means of obtaining sitename across all dataHooks
created custom useMutation hook for mutation with notification

Checklist

[ ] README / documentation has been updated if needed
[x] PR title / commit messages contain Jira ticket number if applicable
[ ] Tests have been updated / created if applicable

Reviewers Should Do The Following:

[x] Review the changes here
[ ] Pull the branch and test locally

Additional Notes
N/A

Love this standardized approach


ENSEMBL (Mouse) gene reference must be downloaded and prepared

The following gene annotation dependency is pinned for this version of the schema.

Organism
Required version
Download

ENSEMBL (Mouse)
Mouse reference mm10 (GENCODE vM23/Ensembl 98) matching cellranger 2020-A (July 7, 2020) release
gencode.vM23.primary_assembly.annotation.gtf

Parsing requirement:

Extract the ENSEMBL gene identifier and its human-readable label

Implemented in cellxgene-schema v2.0.0


Create a new release

It looks like Django 1.10a1 is slated for next month. Would it be possible to get a release that includes the removal of features deprecated in 1.10?
I think the relevant existing changes were in https://github.com/carljm/django-model-utils/pull/207 and https://github.com/carljm/django-model-utils/pull/208.
Thanks!

Released 2.5.

Thanks, my build is much cleaner now!


Add test editor keyboard shortcuts

GSK-83
⌘+Enter (ctrl+Enter) = Run test
⌘+S (ctrl+S) = Save
Project: Giskard
Reporter: aavtomonov
Created: 2022-04-28

done already


[web]: Flutter web not working in for existing Flutter app

We have existing Flutter app, want to add Flutter web support. but it gives the error
$ flutter create .

Ambiguous organization in existing files: {com.mansh.purplle, com.mansh.purplle.fluttermodule}. The --org command line argument must be specified to recreate project.

if We run
$flutter create --org  com.manash.purplle .
it won't give an error  but web folder is not created
If create any new sample project flutter web is working perfectly.
Flutter doctor
[✓] Flutter (Channel master, v1.10.7-pre.23, on Linux, locale en_IN)
    • Flutter version 1.10.7-pre.23 at /home/ag-purplle/Downloads/flutter
    • Framework revision e0247442d0 (30 hours ago), 2019-09-26 02:49:41 -0400
    • Engine revision 3713ecf107
    • Dart version 2.6.0 (build 2.6.0-dev.2.0 c6a14f4459)

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /home/ag-purplle/Android/Sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.2
    • Java binary at: /home/ag-purplle/Downloads/android-studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)
    • All Android licenses accepted.

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

[✓] Android Studio (version 3.5)
    • Android Studio at /home/ag-purplle/Downloads/android-studio
    • Flutter plugin version 39.0.3
    • Dart plugin version 191.8423
    • Java version OpenJDK Runtime Environment (build 1.8.0_202-release-1483-b49-5587405)

[✓] Connected device (3 available)
    • Android SDK built for x86 • emulator-5554 • android-x86    • Android 9 (API 28) (emulator)
    • Chrome                    • chrome        • web-javascript • Google Chrome 76.0.3809.132
    • Server                    • web           • web-javascript • Flutter Tools

• No issues found!

Modules in pubsec.yml

# This section identifies your Flutter project as a module meant for
  # embedding in a native host app.  These identifiers should _not_ ordinarily
  # be changed after generation - they are used to ensure that the tooling can
  # maintain consistency when adding or modifying assets and plugins.
  # They also do not have any bearing on your native host application's
  # identifiers, which may be completely independent or the same as these.
  module:
    androidPackage: com.mansh.purplle.fluttermodule
    iosBundleIdentifier: com.mansh.purplle.flutterModule

/cc @jonahwilliams

@rahuldevanavar91 can you provide more detail here about your specific project?
@jonahwilliams is there a way to get more debug info in the failure case?

@jkurtw
I already edited the question please check.

Ah, got it. @jonahwilliams their existing mobile app is Add-to-App. Is adding web in that case expected to work?

No

Ok. Sounds like we should close this issue and think about the right project structure for this use case. If you have an Add-to-App mobile app with some screens in Flutter and want to bring those screens / features to the web, what is the best way to structure your project(s)?

The normal way to share code would be to have a second project which depends on a shared package, or imports the code from the first.

@rahuldevanavar91 can you try that approach? Close this now, since this is expected.

@jkurtw
Then we have to maintain two different project for web and app .

I'll follow up via email

I had the same issue, I fixed with this command.
please try this command,
flutter create --org com.example.app .


SharePoint Admin Api Access page broken

Target SharePoint environment
SharePoint Online
What SharePoint development model, framework, SDK or API is this about?
💥 SharePoint Framework
Developer environment
Windows
What browser(s) / client(s) have you tested

[X] 💥 Internet Explorer
[X] 💥 Microsoft Edge
[X] 💥 Google Chrome
[X] 💥 FireFox
[X] 💥 Safari
[ ] mobile (iOS/iPadOS)
[ ] mobile (Android)
[ ] not applicable
[ ] other (enter in the "Additional environment details" area below)

Additional environment details

browser version
SPFx version
Node.js version
etc

Describe the bug / error
After follow guideline from @lucabandMSFT My Sharepoint Admin Access token has been broken.
https://github.com/SharePoint/sp-dev-docs/issues/7640#issuecomment-1021519426

Steps to reproduce

log in to Azure portal as a Global Administrator
go to "App registrations"
search for "SharePoint Online Client Extensibility Web Application Principal" object
click there and select "Certificates and secrets"; you should end up in a UX like the following and you should have just one client secret
image
at that point click on the trash can icon and delete that secret. Once you are done there. go back to SPO tenant admin and ensure you are both Global Admin AND SPO teantn admin. Then go to the "API Access" page: that will trigger generation of a new client secret (you can see that in the AAD portal after you visited the page in SPO tenant admin and refresh).
Expected behavior
It should generate new secret and spfx app work as expected on Team Desktop client.

Here is my log of the page
ApiAccessLog.zip

thanks @ntdong1889. looking at that logs now.

hey @ntdong1889 , can you please go to the AAD Portal and for that principal tell me what the identifierUris is? you can find it under "manifest" section.

@ntdong1889  - Is this actually continuing to fail?  I noticed you sent me a trace from January 27th.
The issue you are seeing looks like it is because the identifier uri already exists... however, if it already exists we shouldn't try to create a new one.  This can happen if the service principal was just created and we query for it, so we don't find it (due to propagation being slow), then when we try to create it you get an error that it already exists.  However, when that happens refreshing the page fixes that as it has now had time to propagate.  Is it still occuring?

hey @ntdong1889 , can you please go to the AAD Portal and for that principal tell me what the identifierUris is? you can find it under "manifest" section.

@ntdong1889 - Is this actually continuing to fail? I noticed you sent me a trace from January 27th.
The issue you are seeing looks like it is because the identifier uri already exists... however, if it already exists we shouldn't try to create a new one. This can happen if the service principal was just created and we query for it, so we don't find it (due to propagation being slow), then when we try to create it you get an error that it already exists. However, when that happens refreshing the page fixes that as it has now had time to propagate. Is it still occuring?

Yes it's a continuing to fail. I ask all the SP admin to try but same result. :(

@GrahamMcMynn @lucabandMSFT  Do we have any news on this please :). Thanks for all the support

Hi @ntdong1889 - sorry for missing your original response. The issue is the identifier is not updated (as can be seen by the image you previously sent) . You need a global admin to visit the web api access page (not just a SP admin). If you are visiting as a global admin please send a trace of that to me so I can take a look at it if it continues not to work.

Thank I will contact my boss and use the highest level account to check this out. I will let you know about this.
Thank for all the support.

Hi @ntdong1889 - sorry for missing your original response. The issue is the identifier is not updated (as can be seen by the image you previously sent) . You need a global admin to visit the web api access page (not just a SP admin). If you are visiting as a global admin please send a trace of that to me so I can take a look at it if it continues not to work.

Hi I just check with Top account and still the same issue.

HI @ntdong1889  - I understand your concern.  Has the identifier URI been updated?  Please go to the AAD Portal and for that principal tell if it has changed.  You can find it under "manifest" section.
If it still continues to be https://microsoft.spfx3rdparty.com that is still the issue.  If that is the case can you at a minimum have your boss visit the api access page and tell me the UTC time that he does that at.  I can pull logs from my side, but I need that information.

HI @ntdong1889 - I understand your concern. Has the identifier URI been updated? Please go to the AAD Portal and for that principal tell if it has changed. You can find it under "manifest" section.
If it still continues to be https://microsoft.spfx3rdparty.com that is still the issue. If that is the case can you at a minimum have your boss visit the api access page and tell me the UTC time that he does that at. I can pull logs from my side, but I need that information.

Hi @GrahamMcMynn ,
It is still https://microsoft.spfx3rdparty.com/ like the picture.

My Boss Access around 08-03-2022 2:30pm to 3:00pm GMT+7 (7:30am-8:00amUTC)

Thank you alot for all the support 👍  💯 💯 💯 💯

@ntdong1889 - Thank you for sending that info.  I was able to look up in our logs and see what is going on.  Our code is failing to set the identifierUri (I guess that could have been expected seeming as it has the old value).
It's getting the following error:
Unexpected failure while adding resource identifier. Error: Property identifierUris is invalid.
That happens when trying to set the identifierUris to be:
"identifierUris": [
"api://ce57ebe3-a63d-4708-b5cf-c274b48bd26c/microsoft.spfx3rdparty.com
]
I am looking into it from our side why that won't update... however, you will be able to unblock yourself if you manually modify that identifier URI.
Thanks,
Graham

@ntdong1889  - could you do the following:

Visit https://developer.microsoft.com/en-us/graph/graph-explorer
Run the following get query:
https://graph.microsoft.com/v1.0/applications?$filter=identifierUris/any(r:r+eq+ 'api://ce57ebe3-a63d-4708-b5cf-c274b48bd26c/microsoft.spfx3rdparty.com')

If there are results copy the results and carry on, if there are no results please let me know.

Run the following get query:
https://graph.microsoft.com/v1.0/applications?$filter=identifierUris/any(r:r+eq+ 'https://microsoft.spfx3rdparty.com')

See that those values are different.  If they are different the issue is that both exist.

The first query return empty.

The second return value like picture

Hi @ntdong1889 - Ok that is not what I expected.  Would it be possible for you to update the identifier URI manually to the value I gave via portal.azure.com and if that fails send a trace for that?

@ntdong1889 - yes, update it as suggested.  If you hit an error updating would it be possible to send a trace?  If so please send it to grahamc at microsoft.com.

@ntdong1889 - yes, update it as suggested. If you hit an error updating would it be possible to send a trace? If so please send it to grahamc at microsoft.com.

I successes update to the new value "api://ce57ebe3-a63d-4708-b5cf-c274b48bd26c/microsoft.spfx3rdparty.com"
but this part still not work

at that point click on the trash can icon and delete that secret. Once you are done there. go back to SPO tenant admin and ensure you are both Global Admin AND SPO teantn admin. Then go to the "API Access" page: that will trigger generation of a new client secret (you can see that in the AAD portal after you visited the page in SPO tenant admin and refresh).

@ntdong1889 - I'm afraid I don't understand. Why are you deleting the service principal? I actually need more information if it still doesn't work... could you send me a time when you tried it, a correlation id, or even better a trace and description of what didn't work? I'm sorry you're running into this problem and I would really like to help you and figure out what is wrong so we can fix it for everyone affected by this issue.

After doing it All ours app use graph API stop working even on web browser. And I can not revert it back to previous value

@GrahamMcMynn Finally I have a solution that We delete the hold app SharePoint Online Client Extensibility Web Application Principal and SharePoint Online Client Extensibility Web Application Principal Helper and let the AAD create them again. After that request permission from app and go to API access page to grand again. Now all my app work as expected.
Thank for all your help this idea could help other people to solve the Custom SPFX app not work on Teams Clients


Cannot connect to Billing Service

Hello, I have some issues with connecting to the Billing API. The singleton gets loaded, it tries to connect but then it says something about an API version 3 and I don't know where this is coming from. Should be version 7, I guess. I mean, that's the goal of all this.
My log:
I/godot   (26634): AndroidIAPP singleton loaded
I/godot   (26634): Billing: start connection
W/BillingClient(26634): In-app billing API version 3 is not supported on this device.
W/BillingClient(26634): Billing service disconnected.
I/godot   (26634): Billing disconnected
Do I miss some setup? Do I have to include some additional plugins or dependencies in my build.gradle or permissions in AndroidManifest?
Thanks for any help.

This is very strange, I just made sure that I am using version 7.
No additional settings are needed, just provide the application with Internet access to connect to the server.
Try updating the plugin files from the releases section.

Hi, thanks for your prompt reply. I used the files of the release V1.2 for Godot 4.2 as I was using Godot 4.2. Now, I tried to merge to Godot 4.3, deleted the android/build folder, reinstalled the according Android Build Template and then downloaded the files of the release V1.2 for Godot 4.3. That seems to work fine. There might have been some settings from an earlier approach, I tried out some other plugins before. Anyway, thanks for your work!


Add new unit tests for 'helpers'

Description
Includes new tests for getActiveNoteFromShortUuid on helpers that check if it gets the right NoteItem on a list by a given Short Uuid.
Closes #419
Browser checklist
This PR has been tested in the following browsers:

[X] Chrome
[  ] Firefox
[  ] Safari

Testing checklist

[ ] End-to-end tests have been created if necessary

@all-contributors add @marcosspn for tests


按钮的有效区域太小了

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
左侧功能栏的按钮，默认状态下并没有很明显的边界，需要使用的时候，才发现必须把鼠标移到那个小小的范围内才可以，使得使用起来并不流畅。
建议向win10的ui学习，按钮有效区直接就是大方块，贼爽。
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

你可以使用快捷键：D


20.10

Description
This PR will add some mechanisms to standardize the project
Requirements
Editorconfig is widely used and has plugins for many software, such as vscode, atom and vim
Benefits
Standardization in the editing of files, improving the reading and writing of codes.
Ignores some files generated in the project that should not be committed

Hello,
Please note that you've made a pull against 20.10 and not master (20.10 is intended as a side-branch for building against Ubuntu Mate 20.10). If this was intended only for this branch, let me know, otherwise I will also mirror it onto master once the pull is complete.
Thanks for submitting your pull request!

Hey! Thanks you! This project helped me a lot in my pi4.
Only 20.10 branch works here (ubuntu 20.10 64 bit)
If you want, you can use it on the master, I don't see any problems!
thank you again!

Rebased onto 20.10 and pushed; in rebase two of the .editorconfig change commits were compacted into the one that introduced it. The same changes were also merged into the main branch.
Thanks again for taking the time to submit your pull request!


python312Packages.lcn-frontend: 0.1.6 -> 0.1.7

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python312Packages.lcn-frontend is: LCN panel for Home Assistant
meta.homepage for python312Packages.lcn-frontend is: https://github.com/alengwenus/lcn-frontend
meta.changelog for python312Packages.lcn-frontend is: https://github.com/alengwenus/lcn-frontend/releases/tag/0.1.7
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.1.7 with grep in /nix/store/8dlyz60p0651nfk29nlf971nbvqcmh4v-python3.12-lcn-frontend-0.1.7
found 0.1.7 in filename of file in /nix/store/8dlyz60p0651nfk29nlf971nbvqcmh4v-python3.12-lcn-frontend-0.1.7

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath

home-assistant-component-tests.lcn
python311Packages.lcn-frontend
python312Packages.lcn-frontend

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/8dlyz60p0651nfk29nlf971nbvqcmh4v-python3.12-lcn-frontend-0.1.7 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python312Packages.lcn-frontend https://github.com/r-ryantm/nixpkgs/archive/cc94ad8684658f1e51b74003dfb0922444ae1742.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/cc94ad8684658f1e51b74003dfb0922444ae1742#python312Packages.lcn-frontend

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/8dlyz60p0651nfk29nlf971nbvqcmh4v-python3.12-lcn-frontend-0.1.7
ls -la /nix/store/8dlyz60p0651nfk29nlf971nbvqcmh4v-python3.12-lcn-frontend-0.1.7/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

5 packages built:

home-assistant-component-tests.lcn
python311Packages.lcn-frontend
python311Packages.lcn-frontend.dist
python312Packages.lcn-frontend
python312Packages.lcn-frontend.dist

Maintainer pings
cc @dotlambda for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

@ofborg eval


Show more path if several same-named references exists

nice )

Nice, but could we just add 'library' to the tokens ignored (like 'src')? That is using the mechanism which already exists.

@elsassph: May be yes, how to do that ("library" is duplicated only in my projects - it is not a common practice; other developers can have other duplicated names)?

Well 'library' seen a fairly generic term, like 'src' so it would be also be worth not counting as a useful folder name.
Your solution is somehow smarter but it works only if you have two projects using 'library' as the last folder name.

Many "library" - it is a problem for me, one "library" - is not! :)))

This list can be managed from ProjectManager settings -> Filtered Directory Names. If you want to improve this, please modify the current logic in ClasspathNode.


Samples for getting MSI from App Service should use IDENTITY_ENDPOINT

This is a doc issue, not a code issue.
The sample showing how to login using an MSI from App Service contains the following snippet:
const options: msRestNodeAuth.MSIAppServiceOptions = {
  msiEndpoint: "http://<IP_ADDRESS>:41741/MSI/token/";
}

However, per the App Service documentation, one is supposed to use the environment variable IDENTITY_ENDPOINT to get the URL for the token service.
It took me a little bit to figure this out. The sample code in the README would be more effective if it used this in the example (and perhaps linked to the App Service page for more details), e.g.:
const options: msRestNodeAuth.MSIAppServiceOptions = {
  msiEndpoint: process.env["IDENTITY_ENDPOINT"]
}

@dpolivy Thank you, Dan!
We will be improving the documentation based on your feedback. We appreciate your interest in our projects!

@sadasant Thanks! I just took a look at the linked PR and added one quick comment for you, hope it is useful :) But yes, looks like the changes may address my feedback.

Thank you @dpolivy !

With the latest version 3.0.5 of the package, IDENTITY_ENDPOINT and IDENTITY_SECRET environment variables will be respected when logging in using MSI from App Service.
Thanks for reporting @dpolivy


Adding flag for ensuring http headers on non-http records

See #12

Coverage increased (+0.0001%) to 99.897% when pulling ddc466119a707f6ff06a82c762b324cc6ba48818 on ensure-http-headers into 8d46c1896752e606fcf64e6b2e9531b65eac69fc on master.

Coverage increased (+0.0001%) to 99.897% when pulling ddc466119a707f6ff06a82c762b324cc6ba48818 on ensure-http-headers into 8d46c1896752e606fcf64e6b2e9531b65eac69fc on master.

Coverage increased (+0.0001%) to 99.897% when pulling 46debf5311a85bcda0defb9cd66926d51180917c on ensure-http-headers into 8d46c1896752e606fcf64e6b2e9531b65eac69fc on master.

Coverage increased (+0.1%) to 100.0% when pulling 5b1e14cc18ef6603754dbc5deef4bd7adad7da25 on ensure-http-headers into 8d46c1896752e606fcf64e6b2e9531b65eac69fc on master.


Visual Studio 2019 support

The Bot Builder framework template does not yet support Visual Studio 2019.

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

ID: 1f1718fa-0745-3fee-6ef5-18e36b61fabf
Version Independent ID: fd9ef97f-4540-e624-d459-c076e94a50ad
Content: Create a bot with the Bot Builder SDK for .NET - Bot Service
Content Source: articles/dotnet/bot-builder-dotnet-sdk-quickstart.md
Service: bot-service
Product: bot-framework
GitHub Login: @Kamrani
Microsoft Alias: kamrani

Could you then add support for Vs 2019 so that we can test it?

@Kaiqb it's disappointing to see that you closed this issue with a oneliner and without a path going forward, especially seeing that your marketplace page clearly states

Visual Studio 2017 or greater

as a prerequisite - so that would mean that VS 2019 should be supported already.
I created https://github.com/Microsoft/BotBuilder-Samples/pull/1072 as a PR to fix it.
cc @JonathanFingold


Add Script.signal() to signal a process that it should terminate

PTAL, I added the requested tests and addressed feedback, the xargs thingy in the LineStream was interesting to deal with listen rather than await for, and the Completer would immediately throw unless I "wait" to release the event from within the Script.capture callback 🤔

I have updated the branch to remove the workaround now that the script.capture bug is fixed, thanks!


Integrate with yasnippet was support by company

Hello,
The first, thanks for your wonderful work.
I'd like to add yasnippet support by company-mode It's working for me.
Example about snippet template: https://github.com/AndreaCrotti/yasnippet-snippets/tree/master/snippets/terraform-mode

Thank you for your contribution, but I believe such customization changes, which are subject to user's taste, should be done by each user on their own. The company-terraform-init function is just a convenience, instead of using it you can just as well build your own 'company-backends list in your init.el.


Error on first install

Maybe the docs just need to be updated, but this is what happens if I follow the instructions on npm...
$ npm insall -g redis-commander
$ redis-commander
No config found or was invalid.
Using default configuration.
No Save: false
listening on  <IP_ADDRESS> : 8081
Redis error Error: Redis connection to <IP_ADDRESS>:6379 failed - connect ECONNREFUSED
    at RedisClient.on_error (/usr/local/lib/node_modules/redis-commander/node_modules/redis/index.js:149:24)
    at Socket.<anonymous> (/usr/local/lib/node_modules/redis-commander/node_modules/redis/index.js:83:14)
    at Socket.EventEmitter.emit (events.js:95:17)
    at net.js:441:14
    at process._tickCallback (node.js:415:13)
Connection closed. Attempting to Reconnect...
Redis error Error: Redis connection to <IP_ADDRESS>:6379 failed - connect ECONNREFUSED
    at RedisClient.on_error (/usr/local/lib/node_modules/redis-commander/node_modules/redis/index.js:149:24)
    at Socket.<anonymous> (/usr/local/lib/node_modules/redis-commander/node_modules/redis/index.js:83:14)
    at Socket.EventEmitter.emit (events.js:95:17)
    at net.js:441:14
    at process._tickCallback (node.js:415:13)
Redis error Error: Redis connection to <IP_ADDRESS>:6379 failed - connect ECONNREFUSED
    at RedisClient.on_error (/usr/local/lib/node_modules/redis-commander/node_modules/redis/index.js:149:24)
    at Socket.<anonymous> (/usr/local/lib/node_modules/redis-commander/node_modules/redis/index.js:83:14)
    at Socket.EventEmitter.emit (events.js:95:17)
    at net.js:441:14
    at process._tickCallback (node.js:415:13)
Redis error Error: Redis connection to <IP_ADDRESS>:6379 failed - connect ECONNREFUSED
    at RedisClient.on_error (/usr/local/lib/node_modules/redis-commander/node_modules/redis/index.js:149:24)
    at Socket.<anonymous> (/usr/local/lib/node_modules/redis-commander/node_modules/redis/index.js:83:14)
    at Socket.EventEmitter.emit (events.js:95:17)
    at net.js:441:14
    at process._tickCallback (node.js:415:13)
Redis error Error: Redis connection to <IP_ADDRESS>:6379 failed - connect ECONNREFUSED
    at RedisClient.on_error (/usr/local/lib/node_modules/redis-commander/node_modules/redis/index.js:149:24)
    at Socket.<anonymous> (/usr/local/lib/node_modules/redis-commander/node_modules/redis/index.js:83:14)
    at Socket.EventEmitter.emit (events.js:95:17)
    at net.js:441:14
    at process._tickCallback (node.js:415:13)
Redis error Error: Redis connection to <IP_ADDRESS>:6379 failed - connect ECONNREFUSED
    at RedisClient.on_error (/usr/local/lib/node_modules/redis-commander/node_modules/redis/index.js:149:24)
    at Socket.<anonymous> (/usr/local/lib/node_modules/redis-commander/node_modules/redis/index.js:83:14)
    at Socket.EventEmitter.emit (events.js:95:17)
    at net.js:441:14
    at process._tickCallback (node.js:415:13)
Redis error Error: Redis connection to <IP_ADDRESS>:6379 failed - connect ECONNREFUSED
    at RedisClient.on_error (/usr/local/lib/node_modules/redis-commander/node_modules/redis/index.js:149:24)
    at Socket.<anonymous> (/usr/local/lib/node_modules/redis-commander/node_modules/redis/index.js:83:14)
    at Socket.EventEmitter.emit (events.js:95:17)
    at net.js:441:14
    at process._tickCallback (node.js:415:13)
Redis error Error: Redis connection to <IP_ADDRESS>:6379 failed - connect ECONNREFUSED
    at RedisClient.on_error (/usr/local/lib/node_modules/redis-commander/node_modules/redis/index.js:149:24)
    at Socket.<anonymous> (/usr/local/lib/node_modules/redis-commander/node_modules/redis/index.js:83:14)
    at Socket.EventEmitter.emit (events.js:95:17)
    at net.js:441:14
    at process._tickCallback (node.js:415:13)
Redis error Error: Redis connection to <IP_ADDRESS>:6379 failed - connect ECONNREFUSED
    at RedisClient.on_error (/usr/local/lib/node_modules/redis-commander/node_modules/redis/index.js:149:24)
    at Socket.<anonymous> (/usr/local/lib/node_modules/redis-commander/node_modules/redis/index.js:83:14)
    at Socket.EventEmitter.emit (events.js:95:17)
    at net.js:441:14
    at process._tickCallback (node.js:415:13)

Maybe I just need to add the config settings first?

As the error message shows it tries to connect to local redis instance if nothing else is given.
If you need to connect to a different redis server you must provide all neccessary informations via command line or config file as stated in the README.


Test A11Y Accessibility with Cypress and Axe

https://github.com/component-driven/cypress-axe

First example here: https://github.com/riccardogiorato/cypress-for-everything/tree/main/examples/accessibility


feat: add support for generics

[x] class templates class Foo<T> {}
[x] interface templates interface Foo<T> {}
[x] method templates public function bar<T>(T $s): T {}
[x] function templates function bar<T>(T $s): T {}
[x] function call generics foo::<string>()
[x] method call generics Foo::bar::<string>()
[x] static method call generics $bar->foo::<string>()
[x] instance class with generics new Foo::<string>()
[x] generic types function foo(Bar<string> $s) {}
[x] covariance templates +T
[x] contravariance templates -T
[x] template sub-type constraint T as Foo
[x] template super-type constraint T super Foo
[x] template equal-type constraint T = Foo

This looks absolutely badass. I'm looking for this parser to be finished and for you guys to start working on a PHP superset. This can be a new era in PHP development (ala typescript).
I'd love to be able to contribute but my skills are not good enough yet.
I'll follow the whole organization really close 👍 !

There's a lot of things to do in the parser first but yes, that is a long term goal.

Going to re-visit this with PXP.


kubectl-kcp: add ws alias for workspace

Add ws as an alias for the workspace command.
mentioned in https://github.com/kcp-dev/kcp/issues/633
Signed-off-by: jbpratt<EMAIL_ADDRESS>

:+1:


Optional Contact Storage Refactor

Refactor the current Hubspot code to include a base class definition which will allow us to implement Brevo as a solution for the NC while maintaining the Hubspot functionality in the Colorado instance.

@bespinosa01, @CalebPena could y'all see if this ticket is ok? feel free to add better acceptance criteria! tysm :)

Something like
class CrmIntegration:
    def __init__(self, user: User, screen: Screen):
         ...

    def add(self):
         ...

    def update(self):
        ...

    # if you want to have different criteria for integrating,
    # then we can both have a method to determine if a user should be uploaded to the CRM
    def should_integrate(self):
        ...

@bespinosa01 @mexi-cano we need to start on this in the new sprint (starting Monday) to enable the Brevo integration - any thoughts on who I should assign it to?


Regex match issue when string contains  or

There are backslashes in the failing cases, but this is not the same issue as https://github.com/intuit/karate/issues/988.
When there are '\r' or '\n' characters in the string being matched, the regex match fails. This is probably because Pattern is not being compiled with the MULTILINE flag. See https://github.com/intuit/karate/blob/ae199b270b9f566847d275760d779456ff7d5289/karate-robot/src/main/java/com/intuit/karate/robot/StringMatcher.java#L45
Without testing, it seems that passing the MULTILINE flag should fix this issue.
A possible workaround might be to trim the string before the problem characters. See below.
Here is a simple feature file showcasing 2 cases that pass, 2 cases that fail, and a sample workaround.
(Could not attach .feature file.)
Feature: regex bug

  Scenario: pass 1 - no special characters

    * def body = 'Word'
    * match body == '#regex ^.*o.*$'

  Scenario: pass 2 - no new line characters

    * def body = 'Word\\\\'
    * match body == '#regex ^.*o.*$'

  Scenario: fail 1 - carriage return

    * def body = 'Word\r'
    * match body == '#regex ^.*o.*$'

  Scenario: fail 2 - line feed

    * def body = 'Word\n'
    * match body == '#regex ^.*o.*$'

  Scenario: pass 3 - workaround - trim before trouble character

    * def originalBody = 'Word\n'
    * def indexOfTroubleCharacter = originalBody.indexOf('\n')
    * def body = originalBody.substring(0, indexOfTroubleCharacter)
    * match body == '#regex ^.*o.*$'

Thanks for the great tool.
If I am able, I'll try to submit a PR.

@arnaldop yes please submit a PR - I'm also wondering if we should add a karate.trim() helper function, let me know and I can take care of that.

@arnaldop on second thoughts I'm closing as won't fix as I think this is a very rare use-case, this is the first time this has come up in 4 years. I've made the change to add karate.trim() as a helper function.
you can attempt a PR and request to re-open but I need to be convinced this doesn't impact any existing use of #regex in the wild

@ptrthomas, I missed your last message until now.
Here is an additional workaround, for anyone looking. I think this one is actually a better workaround. This works with every failing case I mentioned above.
Scenario: Better Workaround

    # initial string - can contain \r, \n or \r\n
    * def body = 'Word\r'
    # split text by individual lines
    * def splitBody = body.split(/\r\n|\n|\r/)
    # see if the array of split strings contains the line with the regular expression
    * match splitBody contains [ '#regex ^.*o.*$' ]


cmd/operator-sdk/olmcatalog: scaffold package.yaml for a set of operator manifests

Description of the change:

cmd/operator-sdk/olmcatalog/gen-csv.go: Added --csv-channel to add the CSV name with version supplied to --csv-version to a package manifest deploy/olm-catalog/operator-name/{operator_name}.package.yaml, and --default-channel to specify that the supplied channel is the default package manifest channel.
internal/pkg/scaffold/olm-catalog/package_manifest*.go: Added a PackageManifest scaffold to create and update a package manifest file at the file location mentioned above.

Motivation for the change: automatically creating and updating a package manifest for OLM users during CSV generation more easily onboards them.

@LiliC @hasbro17 @AlexNPavel PTAL

Update the CHANGELOG for the new flags --csv-channel and --default-channel.


Utilize the output of another node in data quality

Is your feature request related to a problem? Please describe.
Say you want to ensure your input data is the same shape as your output data. This should be easy enough, but is not currently feasible. You could work around it if you want by joining them into a single tuple and writing a custom check to see if the two items match, but that's awkward and requires the joined one to be on the blocking path.
Describe the solution you'd like
TBD exactly -- will write out more later, but I think we can use source and value, defaulting to value.
@check_output(index_matches=source('input_data'))
def output_data() -> pd.DataFrame:
    # ...

Or...
@check_output.custom(CustomIndexMatcher(index_matches=source('input_data'))
def output_data() -> pd.DataFrame:
    # ...

Describe alternatives you've considered
See above, but nothing that clear
Additional context
In a talk with an OS user.

Two resources:

Pydantic passing context info to validators: https://docs.pydantic.dev/latest/usage/validators/#validation-context
Deal ensure decorator receives function inputs/ouputs: https://deal.readthedocs.io/basic/values.html#deal-ensure

Changes one might need to make:

Change this to add any non-static dependencies
Loosen the validation here -- instead wire through the type as the validator type -- we can take the applies_to and make the parameter in (1) a union of the applicable types
Change that to return validators in a delayed manner (E.G. a function that gives them given the constructor argument).

Basically we need to push this further downstream -- we only need to know (a) which validators to build and (b) which parameters they take in to build the DAG, then we can construct them at runtime. So, probably a complex change but not too many lines of code.


Question: Is it possible to AWS Redis ElasticCache?

Hi.  I'm planning a Peatio deployment in Amazon, but I'd like to know if it is possible to use ElasticCache with Redis or peatio use some features that are not available in ElasticCache?
Thanks

@arausseo Hi
Since ElasticCache API is compliant with Redis it should work well. Peatio doesn't use something specific.


Enable use on BSD distros.

I only tested this on FreeBSD, but I added a broad match for any BSD. If you'd prefer a specific match for FreeBSD, just let me know. It seems safe enough to allow any BSD to use it, though.

This is a very simple change. Any chance of getting it merged?


[bug]: patch/dl annex not working anymore

Environment
Arch Linux, zsh 5.9
Reproduction steps
I am trying to install FZF + extras using the snippet there:
https://wiki.zshell.dev/community/gallery/collection/programs#gh-r-junegunnfzf--extras
It doesn't work anymore :(
Expected behavior
Installation results (working state):

Downloading: junegunn/fzf…
(Requesting `fzf-0.35.0-linux_amd64.tar.gz'…)
####################################################################################################################################################################### 100.0%
ziextract: Unpacking the files from: `fzf-0.35.0-linux_amd64.tar.gz'…
ziextract: Successfully extracted and assigned +x chmod to the file: `fzf'
patch-dl annex: File _fzf_completion downloaded correctly
patch-dl annex: File key-bindings.zsh downloaded correctly
patch-dl annex: File fzf-tmux.1 downloaded correctly
patch-dl annex: File fzf.1 downloaded correctly
bin-gem-node-annex: Created the fzf shim and set {auto}`+x` on the fzf binary
Installed 0 completions. They are stored in $INSTALLED_COMPS array.

Current behavior
Results in:
Downloading: junegunn/fzf…
(Requesting `fzf-0.35.0-linux_amd64.tar.gz'…)
####################################################################################################################################################################### 100.0%
ziextract: Unpacking the files from: `fzf-0.35.0-linux_amd64.tar.gz'…
ziextract: Successfully extracted and assigned `+x` chmod to the file: `fzf'
bin-gem-node-annex: Created the fzf shim and set `+x` on the fzf binary
.zi-load-plugin:source:90: no such file or directory: /home/niflo/.cache/zconfig/plugins/junegunn---fzf/key-bindings.zsh
zsh-syntax-highlighting: unhandled ZLE widget 'fzf-history-widget'

The other annexes work, but the dl'...' hook doesn't run (nothing printed).
Worked beautifully until I upgraded today.

### Code snippet

```sh
# Annexes
zi light z-shell/z-a-bin-gem-node
zi light z-shell/z-a-patch-dl
zi light z-shell/z-a-submods
# FZF + widgets
zi for from'gh-r' \
    dl'https://raw.githubusercontent.com/junegunn/fzf/master/shell/completion.zsh -> _fzf_completion' \
    dl'https://raw.githubusercontent.com/junegunn/fzf/master/shell/key-bindings.zsh -> key-bindings.zsh' \
    dl'https://raw.githubusercontent.com/junegunn/fzf/master/man/man1/fzf-tmux.1 -> $ZI[MAN_DIR]/man1/fzf-tmux.1' \
    dl'https://raw.githubusercontent.com/junegunn/fzf/master/man/man1/fzf.1 -> $ZI[MAN_DIR]/man1/fzf.1' \
    id-as'junegunn/fzf' nocompile pick'/dev/null' sbin'fzf' src'key-bindings.zsh' \
      junegunn/fzf

Additional information
Since commit #25a22 it doesn't work anymore (I bisected it).
Self-service

[ ] I'd be willing to address this documentation request myself.

Have you read the Contributing Guidelines?

[X] I have read the Contributing Guidelines.

Are you familiar with the Contributor Covenant Code of Conduct?

[X] I have read the Contributor Covenant Code of Conduct.

Contact Details
No response

Hi, @niflostancu could you provide the output of echo $ZI[CACHE_DIR], echo $ZSH_CACHE_DIR and echo $XDG_CACHE_DIR.

Hello, I think you meant XDG_CACHE_HOME, here you go:
➜ echo '|' $ZI[CACHE_DIR] '|' $ZSH_CACHE_DIR '|' $XDG_CACHE_HOME
| | ~/.cache/zconfig | ~/.cache

Oh, yes, I recall, I added this to my zsh config:
typeset -Ag ZI
export ZI[HOME_DIR]="$ZSH_CACHE_DIR"
export ZI[BIN_DIR]="${ZI[HOME_DIR]}/bin"

Thanks for pointing this out :+1: A fix has been applied. Please update Zi, reload the shell, and re-open if the issue persists :dart:

That was quick. Thanks, it works now ;)


Update README for new org

[ ] Make sure new sponsoring companies are up to date
[ ] Verify any links are correct
[ ] Include the new logo

Here again the new logo


[Violation] 'requestAnimationFrame' handler took ms

First, nice work @ericblade !
I'm getting this error when usign Quagga with type LiveStream with frequency 20. Could this affect to performance? How can we avoid this warning?

Thanks!

that seems to be an indication that it is in fact affecting performance.  I'm honestly not sure about what exactly the frequency parameter represents, if it's a target frames per second, or a demand for frames per second, or what.
I'd try increasing it by quite a lot and see what happens, if it remains same, then try decreasing it by quite a lot, and see what happens.
Please report back :) thanks

I've tried to decrease the frequency to 1 and it still shows the warning, and my handler only has a console.log

ok, so... i just looked into the actual code that runs with requestAnimationFrame ..
so.. what it does, is it runs a frame update every 1000 / (config.frequency) milliseconds.  if frequency is not there, it is 60.  The samples I inherited with this code use 10.
so, 1000 / 10 should be updating with 100ms intervals between. captures.
the requestanimationframe function calls update, which calls framegrabber to grab the image, and then locateAndDecode, to attempt to run the locator and the decoder.
eventually, if anything is located/decoded, it would also pass that into your callbacks, which look like they would be executed in the same run, so this should include the entire locate/decode process, as well as any handlers you have attached to handle detection or successful decode.
i haven't timed this directly in a browser, but 70-100ms for a complete decode operation seems normal, from what I remember of test runs outputs.
SO .. this seems like it's probably a recent addition to a browser.  What browser/version are you using?  and is it a desktop or a mobile?

please reopen if you have any further input @gpalomar .  I have been unable to reproduce this in anything I use, so I'd need more information to try and chase it down.


Advice on sharding for data parallelism

I'm using code similar to the 8-way batch data parallelism example here: https://jax.readthedocs.io/en/latest/notebooks/Distributed_arrays_and_automatic_parallelization.html#way-batch-data-parallelism
mesh = Mesh(mesh_utils.create_device_mesh((8,)), 'batch')
sharding = NamedSharding(mesh, P('batch'))
replicated_sharding = NamedSharding(mesh, P())
batch = jax.device_put(batch, sharding)

So I think if batch is shaped (B, H, W, C) like an image, then if B is a multiple of 8, then the data is nicely distributed among the 8 devices. Is Grain able to prepare a batch like this? I haven't been able to figure it out from looking at the ShardOptions.

Is there a solution with ShardByJaxProcess?


www/nginx: add server ssl directives to GUI

Hi
may close https://github.com/opnsense/plugins/issues/2472 i hope

Can you remove sslv2, sslv3 and tls1.0 as no one should be using these and not to mention the fact openssl does not even support sslv2 and  V3

ssl3 is still supported in 1.1.1 afaik? although, of course, I would remove it if the maintainer does not mind.
and i know real-life scenarios when tls1.0 is still can be needed
so I would remove the ssl and leave the tls, as it is done for upstreams

I believe OpenSSL is built without any SSL support.

On 29. Jul 2021, at 07:24, kulikov-a @.***> wrote:

ssl3 is still supported in 1.1.1 afaik? although, of course, I would remove it if the maintainer does not mind.
and i know real-life scenarios when tls1.0 is still can be needed
so I would remove the ssl and leave the tls, as it is done for upstreams
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

you are right. misled by the openssl ciphers output again

Another strategy despite specifying individual ciphers would be offering different policies like I have done here → https://github.com/opnsense/plugins/pull/2255 for os-postfix.
There I have implemented 'Old', 'Intermediate' and 'Modern' policies as recommended by Mozilla.
They have those recommendations ready to use for nginx.
They even have a hbs template, which could easily be transformed into a Jinja template.
Users who manually adjust cipher settings typically want to maintain a certain security level. That is typically better to achieve through certain security policies than through occasional tinkering with individual cipher suites.
(I proposed here → https://github.com/opnsense/core/issues/5060 to make such security policies available for all plugins to use.)

@windgmbh
hi. interesting (especially with the json presence). I would prefer this to be offered as an alternative to manual selection rather than a replacement (so that it doesn't break existing configs and allows custom configs).
so it can be done outside of this PR:

client-side js with ajax call  for mozilla fresh guidelines and values applying at plugin level?
update guidelines with OPN updates and use it globally (huge work)

using at the plugins level seems more realistic to me.

@kulikov-a: If you want to download something from mozilla, you can likely not access it due to the same origin policy and need to proxy it in the backand via an OPNsense API endpoint. However that should be easy to implement.
But keep that discussion to the core team if you want to support that in multiple plugins.

@fabianfrz
thanks for the detailed review. I answer in conversations with small pauses (sorry)

@fabianfrz imho all comments are taken. as I said, the use of the ssl_ prefix in the config is due to the better reflection of nginx directives. if you insist, I will remove the prefix

This PR now has merge conflicts that need resolving

@L1ghtn1ng thanks!

@fabianfrz thanks!!

@fichtner sorry, any chance for this before 22.* (just clarifying my migration plans)?

Is this such a big change that I will only make it to a major release?
Waiting on this also...

@marcelmah No, it is not. It can be releases any time (in general plugins are rolling release except when there are really breaking changes)

Hmm alright, I hope someone can review and merge this soon!

@kulikov-a ok ready to roll?

looks good for me )

Merged, thanks!

thanks a lot!


Feature/encrypted preferences

encrypt authentication related user preference data
add a preference to enable the encryption for credential

wrong base (should be merge to develop) first


Hybrid rendering

Implementation PR: https://github.com/withastro/astro/pull/6991
Stage 3 Proposal: https://github.com/withastro/roadmap/pull/565

Needs

Note that this feature will likely be released as experimental at first.

The config reference page will include details on about how to turn on this feature.
Currently prerendering is explained on the server-side rendering page. Since this new feature is for SSG it wouldn't work well there.
How it works
See the RFC for full details. In short:
defineConfig({
  output: 'hybrid'
})

Then you can opt-out of prerendering on a per-page basis, for ex:
index.astro
---
export const prerender = false;
---

Thanks @matthewp ! Just to confirm, this will release as experimental next Wednesday, right? in 2.5? We'll get started on some docs in prep for that!

The plan is experimental in 2.5, yes.
Where's the right place to document this, do you think? Currently prerendering is explain on the SSR page, but this new feature is for SSG.

I'm going to have to think about this one. We've talked as a longer-term goal of SSR eventually not having its own page, but more SSR and SSG interwoven as appropriate throughout Docs as just two different options when discussing other topics. This is another reason why that would make sense.
I'll come up with some thoughts!

OK, so to figure out where to place this (maybe something about rendering added to our routing page?):

when you configure server: hybrid, I'm checking that this does not require the user to also install an adapter integration in order to make use of some pre-rendering?  Because I don't see mention of needing to do that. (i.e. Is hybrid actually a sneaky way of setting server but with default rendering set to false?) If it requires an adapter, and is "stealth server" mode, then we could add it to SSR for now to augment the existing pre-rendering content, and I make a note (maybe on the routing page?) that pre-rendering for some pages is available for your routes using server: hybrid and link to SSR page for more details.
We could consider a new "rendering" page that will include everything about SSR vs SSG, pre-rendering etc. to replace the existing SSR page. This is a little more ambitious and not necessary for this 2.5 release (but, I could accomplish it on my end if we felt it was high enough priority). BUT, that would also lead me to want to include this information on the existing SSR page, because then it will be right there for when I redesign the page.

So, I think the answer I seem to be converging on is, it's least disruptive to what we have now and probably most convenient for you to add your docs to the SSR page.
And it'll be on me to first, make a logical link from elsewhere in the docs (again, I'm thinking routing or endpoints or something... I'll discuss with @delucis, too). And then, maybe in time for 2.5 release, maybe not, I'll repurpose the SSR page into a "rendering" page where we can also have better SSG vs SSR content, all the options etc. We knew that the SSR page was outgrowing its use, and now's as good a time as any to tackle that!

@sarah11918 Good point, hybrid does also require an adapter. I'll update the RFC to convey that. Given that, I like option A here, for now anyways, it becomes a secondary way to do SSR and can be documented there.


[Cherry pick] Fix Starlette Denial of service (DoS) via multipart/form-data (#4006)

chore:	Fix CVE-2024-47874
Cherry-picks https://github.com/kserve/kserve/pull/4006

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Type of changes
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Feature/Issue validation/testing:
Please describe the tests that you ran to verify your changes and relevant result summary. Provide instructions so it can be reproduced.
Please also list any relevant details for your test configuration.

[ ] Test A

[ ] Test B

Logs

Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

Checklist:

[ ] Have you added unit/e2e tests that prove your fix is effective or that this feature works?
[ ] Has code been commented, particularly in hard-to-understand areas?
[ ] Have you made corresponding changes to the documentation?

Release note:

Re-running failed tests

/rerun-all - rerun all failed workflows.
/rerun-workflow <workflow name> - rerun a specific failed workflow. Only one workflow name can be specified. Multiple /rerun-workflow commands are allowed per comment.

@hdefazio can you please open a jira for this?

Yes - there is already a JIRA for this: RHOAIENG-14604

Yes - there is already a JIRA for this: RHOAIENG-14604

Isn't this one:? https://issues.redhat.com/browse/RHOAIENG-15961


Added many_until(p, end) function

We could improve this by adding min=0, max=Inf and consume_end=True kwargs. It's already useful as is, though.
I'm not sure about the name of this function, but I couldn't find a better name.

Codecov Report

Merging #31 into master will not change coverage.
The diff coverage is 100%.

@@          Coverage Diff          @@
##           master    #31   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files           2      2
  Lines         283    297   +14
  Branches       63     65    +2
=====================================
+ Hits          283    297   +14

Impacted Files
Coverage Δ

src/parsy/__init__.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

As in our other discussion, I think this would be better as a method so it chains like many etc.

Closed in favour of #59


minor KTO setting changes + KL batch size

What does this PR do?

change the default learning rate from 5e-7 to 1e-6; for models 7B and up (which is what most people use w/ or w/o LoRA), i this works better with the default beta=0.1 than an lr=5e-7, which can lead to slow learning
add explanations in the docs on choosing the learning rate
(important) change the batch size of the KL data to be the per-step batch size, not the effective batch size; the current implementation leads to worse results when gradient accumulation is used
allow dropout to be used, like in DPOTrainer

cc @kashif @qgallouedec

in the kto_config.py docstrings can you kindly add:
disable_dropout (`bool`, *optional*, defaults to `True`):
            Whether to disable dropout in the model.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

in the kto_config.py docstrings can you kindly add:
disable_dropout (`bool`, *optional*, defaults to `True`):
            Whether to disable dropout in the model.

done!


Tutorial fixes

Updates to the developer tutorial for creating a first dapp

@adamspofford-dfinity
When I open the frontend in my browser, I get the error message.

poll_frontend/src/index.html

The above may not be necessary.

When I click on the "Rest Poll" button I get an error message. I am using Chrome.

This is due to "pollResults = 0;" in index.js.

When I open the frontend in my browser, I get the error message.
poll_frontend/src/index.html

The above may not be necessary.

Thanks for this.

Good catches @tokuryoo ! Fixed the tutorial thanks to these bug reports.

Thank you !


Don't ignore color profile

It seems that w(a)ifu2x ignores color profiles. Using the website, I put in an image using Adobe RGB, and the colors I got back were very muted. When I converted the original image to sRGB before using waifu, the colors were correct.
If waifu uses 16-bit color internally, the solution is obvious. Read the color profile and use it to map it correctly to the internal profile. Then, before output, map it back to the same profile that was originally used.
If waifu uses 8-bit sRGB internally, then there are two solutions, both with advantages and drawbacks. In this case, it might be good to give the user two options:

Keep the color profile. (Ignore the color profile at first, but then  attach the same profile to the output image.)
Convert to the sRGB color profile.

The first option is easy, and makes sure that no color information is lost. The problem is that the waifu engine would be working with false colors. The second option allows waifu to have the right colors, but can result in color information loss.

I heard this problem from many users. I will add support for ICC profile.


Dashboard Token Listing: SAVEx

Discord Handle of Submitter: goldenageguide#5455
Token Symbol: SAVEx
Network: Gnosis Chain
Token Address: 0x83ccc929c1131d2d18b001c05cfeed9f3364a506
Network Explorer Link: https://blockscout.com/xdai/mainnet/address/0x83cCC929C1131d2d18b001C05CFeED9f3364a506/token-transfers#address-tabs
Is a Super Token? true
Address of Underlying Token:  0x338D4527c9E7F7316c43581684Bc510b851F832a
SVG Logo Link: https://drive.google.com/file/d/1bdmU7h8BPH2hTP7D_cQIk96hXCNK3o_K/view?usp=sharing
Coingecko or Hardcoded Price?: Hardcoded
Coingecko ID:
Harcoded Price (if Coingecko ID is unavailable): 10
Token HEX Color: #59a21a
Info on Listing: Save Impact Dao.   Save money, Save the planet.   All Rights Reserved 2022 By Adbongo Group LLC www.adbongo.io
Zero Waste Protocol White Paper: https://docs.google.com/document/d/1j2klDUYwekXK10Tm-kyU-dPfseaCmicKn8RTYtOupSY/edit?usp=sharing
Discord:  https://discord.gg/Ym4xcdPVqA
Please close only after token has been added to resolver!

Resolver listing done in https://blockscout.com/xdai/mainnet/tx/0xe5dd0c3f1affce3f75b72199cd1bf823d3a9abea8f60fa7dee7a769c5eb7ab2a
See https://console.superfluid.finance/xdai/supertokens/0x83ccc929c1131d2d18b001c05cfeed9f3364a506?tab=streams


Chrisfromwork/calibrationUXImprovements

This change has a few UX improvements for the calibration app:

A console window is created to output debug strings whether or not the application is running under the debugger.
An overlay is drawn over the camera frames to relay to the user where chess board patterns have been identified.
Text is rendered to relay the number of useable images captured as well as the available application commands.

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


To add a calculator to find 7's and 8's compliment

Is your feature request related to a problem? Please describe.
Adding calculator to find 7's and 8's complement of binary number under binary calculations.

Please assign me this issue

Go ahead
Deadline 1 April,2021 (11:59 PM IST)


Disable sign up feature in self-hosted app version

Description
We need to disable sign up feature for self hosted app instances
Steps to reproduce
Open signup page in configurator
Expected behavior
Feature disabled, page returns 404
Actual behavior
You can create new account via sign up mechanism

Hello! Is this one fixed already? Cannot find any configuration


Add networking symbols for uclibc/x86_64

@bors: r+

:pushpin: Commit dee3699 has been approved by alexcrichton

:hourglass: Testing commit dee3699228c07fcd2aa56d2dfb19bc53f0293fce with merge 28952efd8bd3c9da737f598da11a984af809d6cf...


Readme edits

Hi @ClareMSYanGit, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

It looks like you're a Microsoft contributor (Clare Yan (Aquent LLC)). If you're full-time, we DON'T require a Contribution License Agreement. If you are a vendor, please DO sign the electronic Contribution License Agreement. It will take 2 minutes and there's no faxing! https://cla.microsoft.com.

TTYL, MSBOT;

@ClareMSYanGit Thanks for fixing these.


Support more formats and add new method returning deleted fragments

See #1431 for related issue.
Basically, I did the following:

I created kind of enumeration Format. This enumeration is a third argument in the class constructor.
I've extracted the logic in StripMol, changed it a bit so that it is able to return the deleted fragments.
My new method StripMolAndReturnRemoved and public StripMol use then the extracted logic.

My main concern was backward compatibility. I did find any test for this class. Is there any one? Should I write one?
I hope, I did not break anything. Any discussion point welcome.
Best,
christian

Hi,
Thanks for the meaningful suggestions. Looks better now, no?
https://github.com/rdkit/rdkit/pull/1432/files
Regards,
christian

Yes it does. Thank you.
Peter
On Fri, May 26, 2017 at 10:56 AM Christian Ribeaud<EMAIL_ADDRESS>wrote:

Hi,
Thanks for the meaningful suggestions. Looks better now, no?
https://github.com/rdkit/rdkit/pull/1432/files
Regards,
christian
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/rdkit/rdkit/pull/1432#issuecomment-304304653, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AIUQvwRzB2DWnnFMcfFJQ6b_NRo46gElks5r9ugYgaJpZM4NnNUh
.

Hi,
I will add support for SMILES very soon, today or tomorrow.
I am a bit in a hurry. Cheers and have a nice day,
christian

Good morning,
This would be my last changes for a couple of weeks. Today we are going on vacation.
Kindly feel free to finalize/close this. Wish you a great day.
Best regards,
christian

Hi Peter,
I saw that the changes have been approved. Cool! Do you know when the changes will be merged to RDKit master branch resp. when these changes will be available in the next public RDKit release?
By the way, I should send Bernd-Jürgen Freitag's greetings.
Have a beautiful day. Kind regards,
christian

@ribeaud: this is blocked on me going through the code; I'll try to finish that off in the next few days.

Hi Greg,
You meant like this?
Cheers,
christian

Thanks @ribeaud !


How to download a recording you can "join anonymously"

Hi. I would like to download a recording that has been made available without logging in to Microsoft.
destreamer takes me to the Login page, bypassing the page with the "login anonymously" option that is in the original link, so making it impossible for me to access the video.
How can I solve this problem, please?
Thank you.

Having the same issue, have you figured out any solution @glispiger


FileNotFoundError when running run_basics.py and no output

I didn't install dlib and opencv2 yet, idk if that's the issue
This is the output:
2018-07-05 23:05:51.604383: I T:\src\github\tensorflow\tensorflow\core\common_runtime\gpu\gpu_device.cc:1356] Found device 0 with properties:
name: GeForce GTX 1060 6GB major: 6 minor: 1 memoryClockRate(GHz): 1.7715
pciBusID: 0000:01:00.0
totalMemory: 6.00GiB freeMemory: 4.97GiB
2018-07-05 23:05:51.612427: I T:\src\github\tensorflow\tensorflow\core\common_runtime\gpu\gpu_device.cc:1435] Adding visible gpu devices: 0
2018-07-05 23:05:52.415988: I T:\src\github\tensorflow\tensorflow\core\common_runtime\gpu\gpu_device.cc:923] Device interconnect StreamExecutor with strength 1 edge matrix:
2018-07-05 23:05:52.420490: I T:\src\github\tensorflow\tensorflow\core\common_runtime\gpu\gpu_device.cc:929]      0
2018-07-05 23:05:52.423549: I T:\src\github\tensorflow\tensorflow\core\common_runtime\gpu\gpu_device.cc:942] 0:   N
2018-07-05 23:05:52.427180: I T:\src\github\tensorflow\tensorflow\core\common_runtime\gpu\gpu_device.cc:1053] Created TensorFlow device (/job:localhost/replica:0/task:0/device:GPU:0 with 4740 MB memory) -> physical GPU (device: 0, name: GeForce GTX 1060 6GB, pci bus id: 0000:01:00.0, compute capability: 6.1)
Traceback (most recent call last):
  File "run_basics.py", line 51, in <module>
    np.savetxt(os.path.join(save_folder, name + '.txt'), kpt)
  File "C:\Users\fede\env\PRnet\lib\site-packages\numpy\lib\npyio.py", line 1307, in savetxt
    open(fname, 'wt').close()
FileNotFoundError: [Errno 2] No such file or directory: 'TestImages/AFLW2000_results\\AFLW2000\\image00050.txt'

Also there are no images in the \TestImages\AFLW2000_results folder.

There is a funky little issue with the how program works.
Try going to the AFLW2000_results folder and create a new folder named AFLW2000
That should get the program working.


Bluetooth: Mesh: Local network interface related fixes

A few fixes related to packet handling when the local network interface is enabled.

@rymanluk FYI

Looks good!


removed references to intr/nointr (deprecated)

Both an example and a recommendation included 'intr' text. Both references have been removed.

@ronhogue : Thanks for your contribution! The author(s) have been notified to review your proposed change.


How to limit Async uploads to x at a time?

I've spent much time on this without success. What I'd like to do is upload 1,000 files. The way that the async script works now is that all 1,000 will upload concurrently. What I'd like to do is upload (for example) 10 files concurrently, then repeat for the next 10, and so on until done. I believe my issue is that I'll need to modify<EMAIL_ADDRESS>(included in index.html). I've already gotten the files split into chunks of 10, but when doing a chunk.forEach..., the async nature of<EMAIL_ADDRESS>still uploads them all in parallel.
function ResumableUploadForGoogleDrive(accessToken) {
  const f = document.getElementById("file");
  const chunkSize = 10;
  const totalChunks = Math.ceil(f.files.length / chunkSize);
  console.log("The totalChunks are: " + totalChunks);
  for (let i = 0; i < totalChunks; i++) {
    const startIndex = i * chunkSize;
    console.log("The startIndex is: " + startIndex);
    const endIndex = startIndex + chunkSize;
    console.log("The endIndex is: " + endIndex);
    const chunk = Array.from(f.files).slice(startIndex, endIndex);
    console.log("The chunk is: " + chunk);
    chunk.forEach((file, i) => {
...

Sovled https://stackoverflow.com/questions/78876835/how-to-limit-async-uploads-to-x-at-a-time


Cagliostro Brave Counter only shows 1 active frame

When performing Brave Counter (M+H) while in blockstun, all other characters show 3 active frames:

Seems to be accurate to game data (no hitbox on those two frames). Closing until someone can prove otherwise w/ a gameplay clip.


Getting error in combination with MMM-Modulebar

When I installed MMM-Modulebar I'm getting "Undefined" message on the mirrorscreen for the Youless2 module.
Looking into the mirror dev tools I see this error information: (I don't know how to fix this.)
`TypeError: wrapper.gaugeMeter is not a function
at Class.getDom (:8080/modules/MMM-YouLess2/MMM-YouLess2.js:78)
at main.js:113
at new Promise ()
at updateDom (main.js:111)
at main.js:53
at Array.forEach ()
at createDomObjects (main.js:20)
at Object.modulesStarted (main.js:502)
at startModules (loader.js:55)
at loader.js:38
MMM-YouLess2.js:78 Uncaught (in promise) TypeError: wrapper.gaugeMeter is not a function
at Class.getDom (:8080/modules/MMM-YouLess2/MMM-YouLess2.js:78)
at main.js:113
at new Promise ()
at updateDom (main.js:111)
at main.js:53
at Array.forEach ()
at createDomObjects (main.js:20)
at Object.modulesStarted (main.js:502)
at startModules (loader.js:55)
at loader.js:38 `

Looks like the GaugeMeter.js is not found. Have you done a „npm install“ after checkout this modul repo?

Hello Markus,
Yes I've done the npm install, but it gave me some errors (5 moderate, 1
high, 1 critical)  - So I executed 'npm audit' to get more detail.
High          │ Regular Expression Denial of Service in tough-cookie
│
│ Package       │ tough-cookie
│
│ Patched in    │ >=2.3.3
│
│ Dependency of │ request
│
│ Path          │ request > tough-cookie
│
│ More info     │ https://github.com/advisories/GHSA-g7q5-pjjr-gqvp

Critical      │ Insufficient Entropy in cryptiles
│*│ Package       │ cryptiles
│
│ Patched in    │ >=4.1.2
│
│ Dependency of │ request
│
│ Path          │ request > hawk > cryptiles
│
│ More info     │ https://github.com/advisories/GHSA-rq8g-5pc5-wrhr

I looked at those websites but I don't know how to fix these errors. What
exactly need to be installed. It's a bit like chinees for me.  (Sorry for
the expression)
I hope you can give me more details how to fix or solve this.
Thanks in advance
Kind regards,
Johan
Op zo 7 nov. 2021 om 14:49 schreef Markus @.***>:

Looks like the GaugeMeter.js is not found. Have you done a „npm install“
after checkout this modul repo?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/eckonator/MMM-YouLess2/issues/1#issuecomment-962613916,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AH5SRJB6MFWXLP5FKVOC4DTUKZ7WVANCNFSM5HQZQ7XA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Difficult to help without watching the problem in your enviroment. It is possible to have a teamviewer session with you to debug your problem?

Hello Markus,
I have installed Putty so I can remotely access the PI now.
Next I installed teamviewer on my laptop.
It's already a bit late now but if OK for you,  would it be possible to
connect tomorrow around 08 PM ?
Thanks.
Regards,
Johan
Op ma 8 nov. 2021 om 21:06 schreef Markus @.***>:

Difficult to help without watching the problem in your enviroment. It is
possible to have a teamviewer session with you to debug your problem?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/eckonator/MMM-YouLess2/issues/1#issuecomment-963528213,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AH5SRJGKSQLVARZSFGQ6A2DULAUVFANCNFSM5HQZQ7XA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Yes, this should be possible. You can contact me via whatsapp. You find my number on markuseckert.de


Make detection of disconnected agents less aggressive

Additionally make checks for upgrading ecs agent more frequent since we are using older version of ami

lgtm


Just ignore it.


When I installed after this package, error on react-native run-ios command

Hello, I have one question.
why the error on "react-native run-ios" command after installed the package?

so I wish to fix this error and I test to 'npm install' command. So great!, begin running "react-native run-ios" command.
But next error :(

How to fix this error? please, Help me???

Hi @DevSanSon, this error looks unrelated to react-native-button. If you find a bug specifically with this package, feel free to open another issue.


Azure CPI use env.bosh.group instead of env.bosh.group_name

This doc is change for azure cpi https://github.com/cloudfoundry-incubator/bosh-azure-cpi-release/issues/205

Thanks - changes made via #215 instead.


Sort Support

why is there no support for sorting!. Elasticsearch does have sorting available. am i missing something here?

Hi @muhamin, right now Scout is a very simple package for full text search, afaik it doesn't have sorting.
I'm planning to extend scout query builder to support more elastic search features in the near future.

Do you have an update for this feature?


CNV-29502: Move YAML switcher out of tabs in Review and create VM

📝 Description
Fixes:
https://issues.redhat.com/browse/CNV-29502
Design:
https://docs.google.com/document/d/1j5SeV-4ZxJnOvJAhnVn6pjF7A-5TJgx4UQD-6brTn64/
Move the YAML switcher out of the tabs in Review and create VirtualMachine page to the new place in the right - above the tabs, same as it is for VM or Template tabs, for consistency reasons. Display the switcher only for relevant pages that contained the switcher also before this change (all tabs in the page except YAML one). Make this change according to the recent design updates.
Make also some minor css updates for some pages to achieve more consistent look:
align sidebar editor properly in Disks and NICs tabs to be the same as in the other pages (padding-top).
🎥 Screenshots
The following screenshots are displayed as pairs - before and after, for better visual comparison:
Overview tab:

Scheduling tab:

Environment tab:

Network interfaces (a.k.a NICs) tab:

Disks tab:

Scripts tab:

Metadata tab:

@avivtur @metalice @pcbailey @upalatucci @vojtechszocs please review

/lgtm


Resize multiple elements

The plan
As an initial version we will support resizing multiple elements, but always keeping the initial aspect ratio.

[x] Resize from NW
[x] Resize from NE
[x] Resize from SE
[x] Resize from SW
[x] Resize curved lines/arrows
[ ] Resize 2-points lines/arrows
[ ] Resize to negative values (invert)
[ ] What to do with the text? Right now it's not resizable

Rationale
It would be totally fine, if we didn't support rotation, but since we support rotation that becomes an issue. If you try to resize a rotated rectangle in random proportions you will end up in a "irregular" element that we don't support (at the moment). Keeping the initial ratio will not have this issue.
Example of random ratios from Figma
Notice how the rotated rectangle and ellipse are losing their initial properties.

More in #672

@lipis Here's my plan for the remaining items.

I should be able to work on two-point lines some time soon.
I don't see good use cases of resize-to-negative (we can't create a mirror diagram), and its implementation will be very complicated to maintain (unless someone have good ideas/refactorings). I wouldn't do it in this iteration. Would you remove it from this issue and create a separate one for tracking?
I am interested in supporting texts. We need to do support text resizing alone first. We could create a separate issue if it will take time.

Agreed on all three points.

For the last but not least one, there's a serious issue with resizing texts.
As we found https://github.com/excalidraw/excalidraw/pull/1650#issuecomment-634324144, the width and height of an text element is discrete numbers changed by 1.
This prevents resizing multiple elements with texts. I can think of a workaround if a selection has at most one text element, but if there's multiple of them, the resizing behavior won't be consistent.

So, what I propose is when resizing, we make font size to fit in the bounding box, and not change width/height after measurement. We only change width/height from the measurement if the font size is explicitly changed. I think this would work without breaking anything (only we need to be careful that font size and width/height are not always in sync.)


chore: ref to allow sync storage friendlier

Problem
Tempt to fix https://github.com/PostHog/posthog-js-lite/issues/96
Changes

Release info Sub-libraries affected
Bump level

[ ] Major
[ ] Minor
[ ] Patch

Libraries affected

[ ] All of them
[ ] posthog-web
[ ] posthog-node
[x] posthog-react-native

Changelog notes

Added support for X

Definitely breaking changes but in good time if the approach is better for sync storages.
cc @filipef101 WDYT?

I think this is good to go

Teamwork and the community jumping in cc @filipef101
Thanks!


Write a C++ program to find sum of gp series

Description
Write a C++ program to find sum of gp series
A geometric progression (GP) is a sequence of numbers where each term after the first is found by multiplying the previous one by a fixed, non-zero number called the common ratio. The nth term of a GP is given by the formula an = a1rn-1 where a1 is the first term and r is the common ratio.
Input  : 2, 3, 3
Output : 26

How to contribute

Save the solution in FindSumOfGpSeries.cpp file
Add FindSumOfGpSeries.cpp file in program/find-sum-of-gp-series folder

!assign


Test syncing Docker container attributes with database

@kklin , this also needs a rebase. Let me know if you think I should take another look after you rebase