meshmate-member.html fixed stlye

Description

Increased the bottom padding to 30px and top padding to 10px for the "Join the Community" button to make it fit properly.
Adjusted the button width to 250px.
Centered the button horizontally within its container.

This PR fixes
#1413
Signed commits

[x] Yes, I signed my commits.

I would like to work more with Meshery💪

@dhiraj0911 , Thanks for this .
However please refrain from raising direct PRs without getting assigned . If any of issue remain unassigned and you wish to work on them , ping us on Slack .
Please take care of these things from near future


Account registration email should not be controlled by send_welcome_emails setting

At present, the send_welcome_emails organization setting controls all welcome emails, including followup_day1, which tells users that their account has been created. This is not ideal, as that first email is an account security email (an account with your email address has been created), not an email "introducing Zulip to new users" email. We should change it so that followup_day1 is always sent, regardless of the configuration of this setting.

@laurynmm could you please add this one to your todo list? It is not a release goal.


ax.set_zorder not working

Description
ax.set_zorder does not work in proplot.
This becomes an issue when we want to change the zorders of twinx axes.
Steps to reproduce
import numpy as np
import pandas as pd
import proplot as pplt

x = np.linspace(0, 2*np.pi, 100)

fig, ax1 = pplt.subplots()
ax1.plot(np.sin(x), lw=3, c='b')

ax2 = ax1.twinx()
ax2.plot(np.sin(2*x), lw=3, c='r')

ax1.set_zorder(ax2.get_zorder()+1)
ax1.patch.set_visible(False)

Expected behavior: [What you expected to happen]
Red line to be placed beneath the blue line
Actual behavior: [What actually happened]

Equivalent steps in matplotlib
import numpy as np
import pandas as pd
import matplotlib.pyplot as plt

x = np.linspace(0, 2*np.pi, 100)

fig, ax1 = plt.subplots()
ax1.plot(np.sin(x), lw=3, c='b')

ax2 = ax1.twinx()
ax2.plot(np.sin(2*x), lw=3, c='r')

ax1.set_zorder(ax2.get_zorder()+1)
ax1.patch.set_visible(False)

Proplot version
3.4.3
0.9.4

It seems this is because (unlike matplotlib) proplot implements twin axes as "children" of the original axes rather than separate figure axes. This approach simplifies the tight layout algorithm, but means that the drawing order of the twin axes is controlled by the zorder of content inside the original axes rather than the original axes itself. So in principle, you should just need to set zorder below that of the blue line with e.g. ax1.twinx(zorder=1) (see below). I should have explained this in the documentation -- will add a disclaimer.
Your example also directed me to a bug -- zorder=1 only works in matplotlib < 3.4. In matplotlib >= 3.4, it seems the twin axes is getting drawn twice, once as a "child" of the original axes and once as a separate figure axes. So, for your purposes you need to use zorder=-1 instead. I'll fix this in the next version.
import numpy as np
import proplot as pplt

x = np.linspace(0, 2*np.pi, 100)

fig, ax1 = pplt.subplots()  # axes with default zorder 0
ax1.plot(np.sin(x), lw=3, c='b')  # line with default zorder 2
ax2 = ax1.twinx(zorder=1)  # less than the line, more than the subplot
ax2.plot(np.sin(2*x), lw=3, c='r')

This is now fixed -- you should get the above result in matplotlib 3.4+, and I've added a disclaimer to the documentation noting the different zorder behavior here.


Tooltip is still present when series is collapsed while y-axis showAlways is true

Bug report
Description
Tooltip is still present when series is collapsed
Steps to Reproduce

In any multi series chart set one y-axis to showAlways: true
Collapse one or all series
Hover mouse at the hidden series

Expected Behavior
The tooltip should be hidden with the series data?
Actual Behavior
Tooltip is still present over the collapsed series.
Screenshots

Reproduction Link
Link: https://codepen.io/5less/pen/mdwyGzV
Working version in 3.19.0: https://codepen.io/5less/pen/XWgJPyy

Updated with Codepen from an earlier release where the tooltip is hidden as expected.

Any updates on this issue?

Hi, I believe I am also experiencing this same issue or very similar and started to follow this post and am curious on updates as well. However to note, I am using the Angular wrapper version of the ApexCharts library on an Angular 12 project.
Similar to @alekbless, at one point this wasn't an issue as the sample demo code on the website shows it working in codepen with a older version on Angular and apexchart library: https://apexcharts.com/angular-chart-demos/mixed-charts/multiple-y-axis/ (in this version, it is working as expected if you modify the code to set the showAlways flag to true for one or any of yaxis).
However, I copied the multiple-y-axis demo code to a new project I created (in Angular 12, using newer versions of the library "apexcharts": "^3.32.0" and "ng-apexcharts": "^1.6.0") and set the showAlways flag to true, but it looks very similar to the same bug described above:

Here, the Income series shows up in the tooltip when the series has been disabled through clicking the legend.
Let me know if I can help provide additional information, otherwise, I will continue to follow this issue for any updates, thanks.


Add methods for remaining VisualElement properties

For discussion #140

Description of Change

Add methods for properties of VisualElement that do not yet have them.
Also updates the Sample app to use these new methods (where appropriate).
Also fixes the image not displaying on the sample app settings page.
Linked Issues

Fixes #140

PR Checklist

[ ] Has a linked Issue, and the Issue has been approved(bug) or Championed (feature/proposal)
[x] Has tests (if omitted, state reason in description)
[x] Has samples (if omitted, state reason in description)
[x] Rebased on top of main at time of PR
[x] Changes adhere to coding standard
[x] Documentation created or updated: https://github.com/MicrosoftDocs/CommunityToolkit/pull/166

Additional information

Thanks @mrlacey!
Before we can merge this PR, there's a couple procedural steps we first need to do:

Submit a New Feature Proposal
Get Approval Vote from Core Maintainer Team

When you get a chance, go ahead and open a Proposal. Then, I'll bring this Visual Element Proposal to a vote at our next monthly standup.
We have some more info in the README about the new feature process. It's a bit of a pain, but it ensures that every new feature we add is super high quality (which this one looks to be!).
https://github.com/CommunityToolkit/Maui.Markup#submitting-a-new-feature


Implement system-provided file selection display on iOS

Closes https://github.com/ppy/osu/issues/20581
Closes https://github.com/ppy/osu/issues/25966
Closes https://github.com/ppy/osu/issues/30395

I'll preface this with a warning that I'm not quite experienced in the world of UIKit, and the implementation in this PR came from a one/two-days of researching Apple's documentations. I'm still unsure about the way I'm disposing the view controller on Dispose but I don't sense any issues on real-world testing.
This is implemented in a manner where if a FileSelector is loaded, it checks whether the platform supports system-based file selection, and hides the contents of the FileSelector in favour of that.
This is accompanied with osu!-side diff for the final behaviour shown in the video below.

osu!-side diff
diff --git a/osu.Game/Graphics/UserInterfaceV2/FormFileSelector.cs b/osu.Game/Graphics/UserInterfaceV2/FormFileSelector.cs
index 81023417a5..924a2dabfe 100644
--- a/osu.Game/Graphics/UserInterfaceV2/FormFileSelector.cs
+++ b/osu.Game/Graphics/UserInterfaceV2/FormFileSelector.cs
@@ -255,6 +255,8 @@ private partial class FileChooserPopover : OsuPopover
             protected override string PopInSampleName => "UI/overlay-big-pop-in";
             protected override string PopOutSampleName => "UI/overlay-big-pop-out";

+            private readonly FileSelector fileSelector;
+
             public FileChooserPopover(string[] handledExtensions, Bindable<FileInfo?> currentFile, string? chooserPath)
                 : base(false)
             {
@@ -264,7 +266,7 @@ public FileChooserPopover(string[] handledExtensions, Bindable<FileInfo?> curren
                     // simplest solution to avoid underlying text to bleed through the bottom border
                     // https://github.com/ppy/osu/pull/30005#issuecomment-2378884430
                     Padding = new MarginPadding { Bottom = 1 },
-                    Child = new OsuFileSelector(chooserPath, handledExtensions)
+                    Child = fileSelector = new FileSelector(chooserPath, handledExtensions)
                     {
                         RelativeSizeAxes = Axes.Both,
                         CurrentFile = { BindTarget = currentFile }
@@ -273,24 +275,43 @@ public FileChooserPopover(string[] handledExtensions, Bindable<FileInfo?> curren
             }

             [BackgroundDependencyLoader]
-            private void load(OverlayColourProvider colourProvider)
+            private void load()
             {
-                Add(new Container
+                if (fileSelector.UsingSystemFileSelector)
                 {
-                    RelativeSizeAxes = Axes.Both,
-                    Masking = true,
-                    BorderThickness = 2,
-                    CornerRadius = 10,
-                    BorderColour = colourProvider.Highlight1,
-                    Children = new Drawable[]
+                    // keep present for FileSelector scheduling to still work.
+                    Body.AlwaysPresent = true;
+                    Body.Hide();
+                }
+            }
+
+            private partial class FileSelector : OsuFileSelector
+            {
+                public FileSelector(string? initialPath = null, string[]? validFileExtensions = null)
+                    : base(initialPath, validFileExtensions)
+                {
+                }
+
+                [BackgroundDependencyLoader]
+                private void load(OverlayColourProvider colourProvider)
+                {
+                    TopLevelContent.Add(new Container
                     {
-                        new Box
+                        RelativeSizeAxes = Axes.Both,
+                        Masking = true,
+                        BorderThickness = 2,
+                        CornerRadius = 10,
+                        BorderColour = colourProvider.Highlight1,
+                        Children = new Drawable[]
                         {
-                            Colour = Color4.Transparent,
-                            RelativeSizeAxes = Axes.Both,
-                        },
-                    }
-                });
+                            new Box
+                            {
+                                Colour = Color4.Transparent,
+                                RelativeSizeAxes = Axes.Both,
+                            },
+                        }
+                    });
+                }
             }
         }
     }
diff --git a/osu.Game/Graphics/UserInterfaceV2/OsuFileSelector.cs b/osu.Game/Graphics/UserInterfaceV2/OsuFileSelector.cs
index addea5c4a9..8c6477090b 100644
--- a/osu.Game/Graphics/UserInterfaceV2/OsuFileSelector.cs
+++ b/osu.Game/Graphics/UserInterfaceV2/OsuFileSelector.cs
@@ -29,7 +29,7 @@ public OsuFileSelector(string? initialPath = null, string[]? validFileExtensions
         [BackgroundDependencyLoader]
         private void load(OverlayColourProvider colourProvider)
         {
-            AddInternal(new Box
+            TopLevelContent.Add(new Box
             {
                 RelativeSizeAxes = Axes.Both,
                 Colour = colourProvider.Background5,

Preview:
https://github.com/user-attachments/assets/cc5ef56f-c480-40d2-bf27-31a8b84fa70e

@frenzibyte conflicts 🙏

Conflicts should be resolved. Also added minor window API to do away with some of the awkward hookings and the sprinkled null forgiving operators everywhere in the code.


Add JSCS code style validation

I've added JSCS validation to the UI Toolkit, and fixed up the validation errors. I decided to base our rules on the Google presets, but let me know if you disagree with any of this.
Note: I'm introducing JSCS because JSHint has deprecated all of their code style validations. They are switching to linting for code correctness only. See http://jshint.com/docs/options/#maxlen
@AlasdairSwan @dsjen Please review.
FYI @peter-fogg @dan-f @talbs
P.S. I added JSCS validation with the same set of rules to ORA2: https://github.com/edx/edx-ora2/pull/794

I'm not a huge fan of forcing someFunction( variable ); to be someFunction(variable);. But for consistency I may be convinced that it's okay. I just find it easier to read longer, chained lines when there is more space within the ();

:+1: I like the idea of standardizing style, especially in a centralized toolkit repository. It makes new code easier to approach by making it more familiar. I prefer no extra padding inside of parens.

@AlasdairSwan It looks like all the built-in presets have no spaces inside parenthesis enabled. You can see them all here:
https://github.com/jscs-dev/node-jscs/search?utf8=✓&q=disallowSpacesInsideParentheses
My preference is to break chained functions onto separate lines for clarity, although I know what you mean about it getting harder to read.

@andy-armstrong fair point about keeping individual lines more simple, I can live with it :+1:

@andy-armstrong should we consider this kind of change for edx-platform, then? thanks for being on top of this.

@benpatterson Yes, I would like to see us do this for edx-platform too.

:+1: I like the effort to standardize and these makes sense to me.  I actually like the spacing inside of parentheses and have done that in the past, but don't follow that style these days.

@AlasdairSwan @dsjen Any ideas why this fails on Travis but works locally for me? I'm thinking it is a JSHint version issue but I've failed to force it to the right version somehow.

@andy-armstrong -- that's odd.  It looks like it's failing on the bower install step.  2.6.0 (https://github.com/jscs-dev/node-jscs/releases) was released 5 days ago, so there shouldn't be a latency issue...  I'll restart it just in case.

@andy-armstrong -- I think I see the issue.  https://github.com/markelog/node-jscs/releases is a fork of node-jscs and doesn't have a v2.6.0.

@dsjen I tried to downgrade but that didn't seem to work. Would you be able to try fixing this since you're much more adept with Travis than I am? Thanks!

@andy-armstrong I think your fix worked, but there are now some linting errors.  I think this is good!

@dsjen Ah, I see. The linting errors were what I was trying to solve in the first place before I broke the dependencies! I don't get these errors locally, and they seem wrong to me. It makes me think it is using a different version of JSHint, but I don't know how to fix that.

@andy-armstrong -- could we switch back to the node-jscs rather than that fork?  That way we could upgrade to 2.6.0 instead.

@andy-armstrong @dsjen - see https://github.com/edx/edx-ui-toolkit/pull/14

@andy-armstrong - I merged #14. Go ahead and rebase :)


[move][tests] Use clever errors for tests

Description

We no longer need a code with assert! thanks to clever errors. Cleaned up the tests to reflect this

Test plan

Ran tests

Release notes
Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.
For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates.

[ ] Protocol:
[ ] Nodes (Validators and Full nodes):
[ ] Indexer:
[ ] JSON-RPC:
[ ] GraphQL:
[ ] CLI:
[ ] Rust SDK:

test_random.move There are two places in the file name, which causes an error in executing test. Please handle it.
sui-framework\tests\test_random.move:5:13
sui-framework\sources\test\test_random.move:5:13


Disable Secret Generation / Provide external secret names

Hello and thanks for all of your hard work on enabling stackstorm-k8s.
Our team uses GitOps to deploy and update our instance of Stack Storm.
We would like to use Sealed Secrets to store the secrets used in our deployment.
However, the Helm chart currently does not support that.
I am happy to submit a PR to disable secret generation if turned off in the Values file.
But I would like to confirm if that type of change would be appreciated?

We have a lot of the configuration at this moment where the secrets could be injected into StackStorm, like https://github.com/StackStorm/stackstorm-k8s/blob/a2a9e1c6e5a2c6e5d8cf03f0b6086c7864688d4a/values.yaml#L90-L94 and more.
Would be definitely great to make the chart work nicely with the https://github.com/bitnami-labs/sealed-secrets.
Looks like sealed-secrets can manage the existing secrets:
https://github.com/bitnami-labs/sealed-secrets#managing-existing-secrets
Did you try to explore that? Let us know what's still missing and let's find something that would work.
Looking for ideas on how to make that with minimal changes and consistent with the Helm Values.

I will test it out using sealed secrets.
In particular I was looking at disabling the following file:

secrets_st2auth.yaml

So I just confirmed that the secrets are regenerated/change each time a release occurs.
The problem is that Sealed Secrets does not update/revert the secret after it is changed.
Because I deploy my solution via GitOps using ArgoCD.
That means i will have to revert the password on each change.
So consideration of disabling the generation of secrets for st2auth and datastore-crypto-key would help resolve this issue

Thanks for looking closer at this!
I see overall we're generating values for password, datastore_crypto_key and ssh_key:
https://github.com/StackStorm/stackstorm-k8s/blob/a2a9e1c6e5a2c6e5d8cf03f0b6086c7864688d4a/values.yaml#L49-L66
@bmarick any other secrets you're overriding in your environment?
https://github.com/bitnami-labs/sealed-secrets#managing-existing-secrets allows managing existing secrets if there is an annotation sealedsecrets.bitnami.com/managed: "true".
What if we change the behavior to skip auto-generation if this specific annotation is present for these secrets?
https://github.com/StackStorm/stackstorm-k8s/blob/master/templates/secrets_st2auth.yaml#L20
This also means there's a need to allow injecting custom annotations for the secrets.
Will it help or does an option to disable secrets autogeneration altogether looks better?

@cognifloyd any other ideas on how to compose it nicely together?
Maybe providing custom secret names in Helm values that are managed outside of the chart?
Let's think about the options.

I actually use this process for other helm harts I use/manage.

Nice! Yeah, it sounds like a conventional way of doing things in the Helm chart community. +1 then
@cognifloyd thoughts?

I'm not opposed to allowing definition of external secrets. We would just need to document requirements for those secrets so that things like our initContainers can still work.

I'm not opposed to allowing definition of external secrets. We would just need to document requirements for those secrets so that things like our initContainers can still work.

I will work on a PR based on other helm charts I have used that allow external secrets

@bmarick How is it going? Have you been able to work on adding external secret support to the chart? Is anything blocking you? Thanks!

@bmarick How is it going? Have you been able to work on adding external secret support to the chart? Is anything blocking you? Thanks!

My work took me in a different direction for a while. I am about to start working on this right now

I would be willing to work on adding external secret support for the  datastore_crypto_key if we can remove the autogeneration on install.
I would add documentation on how to generate the secret to the docs as well so it can be used on installation.
Having the datastore_crypto_key deleted on uninstall and then recreated on install is a pain .
I recently had to uninstall and reinstall due to ingress changes that wouldn't apply using helm upgrade
I ended up setting the datastore_crypto_key manually from a back up of it. (I had read the docs that it gets recreated on installation)

Ah, I see your note here
I can add a new key to values.yaml for datastore_crypto_key_secret_name and if that is set skip creation of the secret altogether and just use the supplied name everywhere.


Exit codes on failure

I cannot see if there is an feature to make it fail with a exit code >0.
That would be a good feature so if you use s3-cli in a command that you know if the command failed or not.

I agree that would be a good feature.
Note to self, this could probably be done with cli.Exit as seen here -
https://cli.urfave.org/v2/examples/exit-codes/

Any updtes?

I need to pick this up.


loading tests from csv

I've found the ability to load tests from a csv file.   The example does not show how to handle the asserts if the test is loaded from csv.  Any help here?
Thanks!

Have a look at this example: https://promptfoo.dev/docs/configuration/expected-outputs#load-assertions-from-csv


Treating Beta APIs as experimental in Java

Since we are going GA, this needs to be done...

Codecov Report

Merging #1157 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #1157      +/-   ##
============================================
+ Coverage     86.16%   86.16%   +<.01%
- Complexity     3653     3654       +1
============================================
  Files           369      369
  Lines         14385    14386       +1
  Branches       1603     1603
============================================
+ Hits          12395    12396       +1
  Misses         1556     1556
  Partials        434      434

Impacted Files
Coverage Δ
Complexity Δ

...api/codegen/transformer/java/JavaSurfaceNamer.java
86.79% <100%> (+0.25%)
26 <0> (+1)
: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 0e49ab2...1842fb4. Read the comment docs.

Won't the beta APIs be in their own, non-GA packages?

So this is the release level setting for GAPIC clients, which can be at a different release level than the corresponding handwritten layer. For logging in particular, the GAPIC client will be beta and the handwritten layer will be GA.

Got it. LGTM


limit() and distinct() doesn't work together in ActiveQuery

What steps will reproduce the problem?
With any model:
$query = Model::find()->select('attribute')->distinct()->limit(20)->orderBy('attribute');

What is the expected result?
A query that searches for 20 distinct values of attribute and returns it.
What do you get instead?
Repeated values are being returned in my result. The reason is because $query->createCommand()->getRawSql() returns:
SELECT TOP 20 * FROM (SELECT DISTINCT rowNum = ROW_NUMBER() over (ORDER BY [attribute]), [attribute] FROM [Model]) sub

Making the results look like this:

rowNum
attribute

1
value1

2
value1

3
value2

4
value3

I didn't had time to look at the code and make a sugestion, sorry.
And I found a similar issue here, not sure if we are talking about the same problem (I'm using mysql)

SELECT TOP 20 * FROM

how do you get this query when using MySQL? MySQL should be using LIMIT clause after the quey.

Ops... you are correct, my bad!
This query is using another connection that I have in my project. This is actually Microsoft SQL Server.

SELECT DISTINCT rowNum = ROW_NUMBER() I am not familiar with MSSQL but it sounds like only rowNum should be distinct here.

can you try to get a query that returns the expected result when you run it directly on the dbms?

Something like:
SELECT DISTINCT TOP 20 attribute FROM Model ORDER BY attribute

or maybe:
SELECT DISTINCT TOP 20 attribute FROM (SELECT rowNum = ROW_NUMBER() over (ORDER BY attribute), attribute FROM Model) sub

Would do. I'm not very familiar with MSSQL either. But its a external connection, so I'm stuck with it.
By the way, this specific connection is a Microsoft SQL Server 2008 R2 (SP1).

Just in case someone stumbles here: This behavior does NOT occurs with mysql (as I stated when i opened the issue ). The sql generated with mysql is:
SELECT DISTINCT `attribute` FROM `Model` ORDER BY `attribute` LIMIT 20

And it's correct.

This happens only for old version MSSQL Servers.
yii\db\mssql\QueryBuilder
Method oldBuildOrderByAndLimit
This line causes problem:
$sql = preg_replace('/^([\s(])*SELECT(\s+DISTINCT)?(?!\s*TOP\s*\()/i', "\\1SELECT\\2 rowNum = ROW_NUMBER() over ($orderBy),", $sql);
It replaces
SELECT DISTINCT TOP  2  [attribute] FROM [tablename]
with query pointed in head of this thread.
I think, it should return
SELECT TOP 20  [attribute] FROM (SELECT DISTINCT rowNum = ROW_NUMBER() over (ORDER BY [attribute]), [attribute] FROM [Model]) sub
I have newer MSSQL version on my project so I use other method of QueryBuilder and  I can just simulate.
If you guys confirm it works (I mean last query), I can play around this replacement,
Exact list of fields should be selected instead of *

Hi @Patricy , Thanks for your reply!
I made a test here and the second query still returning repeated rows. But you are right about that method, this is the one my server uses.
I was playing with the sql and I found what I think the problem is: the DISTINCT clause is being used in the internal select when we have a ROW_NUMBER() that is always unique, making the clause useless.
Since this ROW_NUMBER() is used when we have a offset in that method, I can only think in two possible solutions:

Make the rowNum outside the distinct clause somehow ( I tried to google this, but with no luck... maybe someone who works with MSSQL can find a way):

SELECT TOP 20 * FROM (
    SELECT rowNum = ROW_NUMBER() over (ORDER BY [attribute]), DISTINCT ([attribute]) FROM [model]
) sub /* wrong sql just to give the general idea */

Remove the DISTINCT there and add it outside, just like the LIMIT  does:

SELECT DISTINCT TOP 20 [attribute] FROM (
   SELECT rowNum = ROW_NUMBER() over (ORDER BY [attribute]), [attribute] FROM [model]
) sub

The problem with the second solution is that I think will affect the offset, since is being checked at the end. Not sure, I didn't tested.
Anyway, I moved on this problem in my project removing the LIMIT and filtering the results, but I will try take a look at this in the weekend and return with more info. Let me know if I can help you with anything.


[RESOLVED] S100-Primario has been restarted (uptime < 10m)

Problem started at 07:48:29 on 2023.05.18
Problem name: S100-Primario has been restarted (uptime < 10m)
Host: S100-Primario
Severity: Warning
Operational data: 00:06:07
Original problem ID: 20812
Event details in Zabbix: http://<IP_ADDRESS>/tr_events.php?triggerid=23452&eventid=20812

Problem has been resolved in 4m 0s at 07:52:29 on 2023.05.18
Problem name: S100-Primario has been restarted (uptime < 10m)
Host: S100-Primario
Severity: Warning
Original problem ID: 20812


Publish new version to crates

Kinda self explanitory, just publish the new code to crates.io

Published v4.1.0


article content without summary

I used .ContentWithoutSummary in my old archetypes, but there's no provision for that with the new recommendation to use  {{ partial "article-content.html" . }}. It seems like just forgoing the "article-content.html" partial and putting {{ .ContentWithoutSummary | safeHTML}} into my article.html works for now, but I'm not sure what problem "article-content.html" is solving with it's note about nested gotmpl. I'm probably reintroducing whatever problem that is.

Migration of archetypes is underdocumented. See this post for instructions #927
Just don't touch article-content.html

I'm not touching article-content, but I'm not using it at all from article as a workaround. I've replaced it with .ContentWithoutSummary for now, which did feel wrong but it's the only way I've found to keep my eye functionality (only summary in list pages, no summary on single pages).

article-content.html encapsulates special handling for the print layout with headless bundles. If you don't care about that, use what ever fits you best.


Failures if patch committed too fast...

In https://reviews.llvm.org/D71395 I got approval and committed before the test infrastructure was able to build and instead of recognizing that the patch was committed already it failed trying to apply the patch and errored out.
Perhaps we could add some code to notice?

The link you provided has a passing build, did you link the right one?
Beyond that: I think this is a duplicate of #72.

It initially failed :)
Harbormaster failed remote builds in B42366: Diff 233518!Wed, Dec 11, 11:57 PM
Harbormaster completed remote builds in B42364: Diff 233514.
That said, perhaps #72 is the right one to use here.


Support verification of responses and errors from Node 8 async handlers using return/throw

When implementing a lambda in JavaScript with Node 8, the handler can be implemented as an async function. One can then simply return a response (which is passed back to the client when doing RequestResponse-type invocation) or use throw to return an error (see https://docs.aws.amazon.com/lambda/latest/dg/nodejs-prog-mode-exceptions.html).
lambda-tester currently supports verification of responses and errors when they were returned by calling callback(error, response), but not when a return or throw was used. It would be nice to have support for this, too.

I just saw that https://github.com/vandium-io/lambda-tester/pull/41 already addresses this issue - nice! So this can be closed once it's merged.

Closed by #41, we're already using it. Thanks!


Can Ammo.js be used ?

Trying to build a pwa with three.js and found using react-three-fiber is the right option .
My project uses ammo.js for physics simulations. Can I use ammo.js in this .
And I am pretty new to react-three-fiber can anyone suggest how to get started with it .
If this is not the right place to ask question please direct me in the right way .
Thanks

Do you specifically need ammo.js or you just wanted some physics? There is an option to useCannon with https://github.com/react-spring/use-cannon designed to bring physics to r3f. If you needed specifically to use ammo.js you could I guess see how they wrapped up cannon and create your own useAmmo hook.

I have closed the topic as it is probably better as a discussion. But feel free to reply here if need be, would like to hear what you decide.


Switch to torchvision transforms

What is the motivation for this task?
Anomalib currently uses albumentations for data transforms. As torchvision v2 introduces the features we mainly use from albumentations, it might be an idea to switch to torchvisions now.
Describe the solution you'd like

use torchvision transforms.
[optional] use model-specific transforms that will be applied within the model forward-pass.
wrap the transforms within the torch model so that they could be automatically applied during inference.

Additional context
No response

Addressed by: https://github.com/openvinotoolkit/anomalib/pull/1706


[BUG] tabular-nums and slashed-zero are conflicting, but should not.

Describe the bug
Using tabular-nums and slashed-zero gives linting error for conflicting classnames.
To Reproduce
Steps to reproduce the behavior:
Add both tabular-nums and slashed-zero as classnames to same element with tailwindcss/no-contradicting-classname enabled.
Expected behavior
It should not give error/warning. The documentation clearly states they can be combined since it uses css variables under the hood.
eslint config file or live demo
"tailwindcss/no-contradicting-classname": "warn"

@einarlove, indeed some can be combined and some cannot, I made a quick Tailwind Play to document this for later use
https://play.tailwindcss.com/QStyY11EAJ

I'll close once released 😉

Please try it via npm i<EMAIL_ADDRESS>-D and let me know how it goes. 😉

@einarlove Please try this beta version:
npm i<EMAIL_ADDRESS>-D
and give me feedbacks.

Fixed in the latest release<EMAIL_ADDRESS>


The Codelab code for 'SearchRepositoriesViewModel.searchRepo' is not the Same as GitHub Code

In task 12 "Adding List Separators", for method SearchRepositoriesViewModel.searchRepo we are given code has undeclared variables:
val lastResult = currentSearchResult
if (queryString == currentQueryValue && lastResult != null) {
    return lastResult
 }
currentQueryValue = queryString

at the being of of the method and
currentSearchResult = newResult
return newResult

at the end.
These lines are not in the GitHub code for this exercise.

Fixed.


docs(deployment) Create uWSGI + NGINX Deployment Documents

This is initial shot at some deployment documentation using uWSGI and NGINX. If you find this helpful, both content and writing style please let me know and I will get it polished. Please read through point out spelling / grammar errors, I have looked at it long enough, I can't see them any more.
My objective was not to have a fully comprehensive guide and I am a little hesitant to be giving advice for setting up uWSGI since it is such a dynamic piece of software. Regardless, I would look to others to supply comments / best practices / additional thoughts that would improve this... hopefully I can learn along the way. :)
Kindly, NZ

It would be great to have a (Nginx+?) Gunicorn and Gunicorn+PyPy variants covered too, but as a fellow uWSGI user I don't feel contributing these without much real-world experience. Maybe @kgriffs or @jmvrbanac could lend a hand? :slightly_smiling_face:

Gunicorn and Gunicorn+PyPy variants

Added #1369

Codecov Report

Merging #1280 into master will not change coverage.
The diff coverage is n/a.

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

LGTM now!


Just want to say thanks for putting this up on npm!

Thanks for your kind word!
I'm glad this library is useful for you.


Error extending native elements in Safari

Core issue: create a custom element which is a subclass of a native DOM element using the ES5 prototype chain. Current problem is the instantiation using new().

see iVis-at-Bilkent/cytoscape.js-context-menus#55
see https://github.com/WebReflection/document-register-element/issues/159
see https://stackoverflow.com/questions/41514964/what-does-reflect-construct-let-us-do-that-wasnt-doable-before
see https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Operators/new.target
https://github.com/webcomponents/polyfills/issues/108#issuecomment-403936697

Looks like lit.dev does not  support it. One more reason for us to do it!

There is a statement.

I've updated the linked documentation to at least be up to date with custom elements v1. It still is on old shadow DOM however, and in general https://developer.mozilla.org/en-US/docs/Web/Web_Components looks very, very outdated and confusing. If anyone has time to update all the web components docs to the latest specs, that would be a great help to developers everywhere, I am sure.

Actually, I am in doubt if extending native elements works at all in Safari. I guess, the current Candid implementation of ES5-style classes is correct.
Here is code that uses ES6-style classes and should work according to the spec.
Chrome

Safari

It looks like I am right - Safari does not (and probably will never) support customizable built-in elements.

Quick fix: include this polyfill
<!-- this should be on top of your HTML <head> scripts -->
<!-- it fixes customized built-in element in Safari and older browsers -->
<script src="//cdn.jsdelivr.net/npm/@ungap/custom-elements"></script>

Strategy / design decision
Candid will not ship a polyfill for Safari or other browsers. For now, an existing polyfill can be used.
However, maybe it makes sense not to support customized built-in elements at all. We might remove the support and simplify the implementation in an upcoming PR.


HTML/WebSocket previewer "transport" + file watcher "transport"

dotnet exec --runtimeconfig ./Avalonia.BattleCity.runtimeconfig.json --depsfile ./Avalonia.BattleCity.deps.json ~/.nuget/packages/avalonia/0.8.999-cibuild0005045-beta/tools/netcoreapp2.0/designer/Avalonia.Designer.HostApp.dll --transport file:///home/kekekeks/Projects/Avalonia.BattleCity/MainWindow.xaml --method html --html-url http://<IP_ADDRESS>:6001 ./Avalonia.BattleCity.dll

@kekekeks, hello. I've integrated this into AvaloniaRider, it is fine. I believe this could be merged now.
Thanks!


Tab close button + display policies + application on Scenes Tabs

Added close button to tabs [x].
Added 4 display policies for this close button (SHOW_ALWAYS, SHOW_NEVER, SHOW_ACTIVE_ONLY, SHOW_HOVER).
Set the scene tabs in editor to follow the hover policy.

Note : the 4 policies listed above only apply to close button. right_button (used for Script button in scene tabs) are always displayed.

Just to relate : PR #2371 adds the same function as close button, but applied to Mouse Middle Click. If #2371 is to be pulled, please pull #2380 first as there may be a collision between them.

I noticed a minor bug when moving quickly the mouse pointer from the close button to the script button (or the inverse) : both buttons appear hovered at the same time. I'll have a look at this later when I have some time.


docs: Add Swagger Support

What this PR does
This PR adds a Swagger spec for the Kubescape microservice API.
The Spec is a manually written OpenAPI 3.0.1 specification in YAML format.
Reasoning
The two possible approaches to supporting Swagger documentation is writing the spec manually or automatically generating it from code.
Automatic generation
Automatic generation would allow generating a Swagger spec from structs actually used in the project. Tools achieve that by inspecting the Request and Response structs’ annotations. However, since the Kubescape microservice uses structs defined in another package, automatically generating the Swagger Spec would require large changes, either:

moving to a different HTTP framework that both defines the request and response structs and generates a Swagger spec from them (e.g. swaggest/rest).
annotating structs defined in dependencies (e.g utilsv1meta) with tags used by tools that generate Swagger specs.
manually writing Go documentation files that define the request and response structs, annotating them with Swagger tags and then generating an OpenAPI v2 spec using go-swagger, preferably, in CI/CD.

Moving to a different framework requires too much effort and does not look like a priority right now. Annotating structs in a dependency would force us to introduce changes upstream and constantly maintain them, which seems counterintuitive and tedious down the road.
Manually writing Go documentation files that describe the requests and response and autogenerating a Swagger schema is an option that comes with significant drawbacks:

it supports only the OpenAPI 2.0 spec, which lacks features like Type Composition. The latest version is 3.0, which does not have this problem.
it has flaky support for Enums. For example, go-swagger generates redundant descriptions for Enum values.
it boils down to manually writing a Swagger spec, but with an extra step of generating it from the documentation files, since Kubescape would not use the documented structs for handling HTTP request, while also limiting the scope of OpenAPI features that can be used.
alternatively, actually using the documented structs for handling HTTP requests would not only require a refactoring, but also keep the limitation of OpenAPI features we can use to document the API.

Manual writing
Forgoing automatic generation — which is not strictly automatic, as shown — leaves us with manually writing the spec.
While somewhat inconvenient and prone to errors, it allows us to use any OpenAPI version and feature we want to. This ensures we can document our API however we want, not being limited by an autogeneration tool and the features it supports. Compared to keeping separate Go documentation files, it also eliminates an extra step of generating the Spec from docfiles, which would have to be written manually and separately from the actual structs in use anyway.
Therefore, I believe that a manual OpenAPI 3.0 spec is the way to go. For the current version of the API, that is :wink:

Here are the resultsing UIs served directly from the Kubescape Microservice.
SwaggerUI

RapiDoc
NB: The screenshot is truncated since the main contents have a scrollable viewport.

Redoc
NB: Redoc does not provide the ability to try out the documented APIs in its open-source product.

@vladklokun Do you need to update kubescape with the merged opa-utils?

Yup, I do in the long run.
But in the meantime, I vendored the modules and generated the spec. So the included spec is up-to-date.

Message ID: @.***>


Fixes #839: Adding disabled tests to catkin_test_results summary.

I set it to read from both disabled (gtest) and skipattributes. I myself haven't seen skip but it appeared in the ament commit: https://github.com/ament/ament_tools/commit/0c6003b58ff0e5442034c7e643dab9035cdaeff8.

Closing and reopening, deleted two debug print statements that I added and want to rerun tests.

@rsinnet Thank you for the contribution. I have created a new PR #848 against the kinetic-devel branch. Since all these Python function are potentially being used by other code changing their return values to include the skipped tests would break existing code. Therefore I have created new functions (with the suffix 2) which provide the skipped tests while maintaining the existing API. It would be great if you could test the referenced PR and confirm if it works for you.

Closing this since it has been merged via PR #848.


[stable/prometheus-operator] bumping Grafana chart version to fix CVE-2019-15043

What this PR does / why we need it:

updates Grafana to 6.3.4
fixes Grafana CVE-2019-15043

More details can be found here
Signed-off-by: Marius Voila<EMAIL_ADDRESS>
Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/ok-to-test
/lgtm


updated timeline, added animate.css and wow.js

@jtamiace can you review my code

Files Changed (4,910)

Only our design lead can take on such a herculean task.

much libraries

wait, @whitzhu, neither js libraries nor timeline changes are in here (or maybe it is but the PR is too big to show). looks like your entire virtual environment folder has been included in this PR (hence the size). can you add ENV/* to the list in your .gitignore file, commit that change, and then push again to your repo?
@aronasorman would she need to make another PR or will this one update accordingly?

@jtamiace it probably isn't loading.
@whitzhu I'd rather have this PR closed, and then start from a complete fresh branch. Continuing this PR would mean that these 4,910 objects will be recorded in the git history and would thus bloat this repo. You can try either just copying the files you edited over or just start from scratch, if they're easy to reproduce.

TIL


Modify root Presentation & accessing other properties

Hi Thomas,
Thank you for the previous evolutions. It works well !
Could you please add (and illustrate) the following features:

Modify elements in root presentation / root presentation master
Modify additional graph properties like activating (or not) Legend, title, X axis, ...
Modify cell format (e.g. specify format such as @,%, 990,00, ....) or to access to other properties (e.g. set the background color of serie X) ?

I think it would be great to be able to change elements in the root Presentation, for example, for specifying dynamically  a Title, Date of report, ... (*)
Thank you Thomas for the work
Have a nice day
Luc
(*) Actually, the only possibility to do that consists in unzipping the pptx, replacing some elements and then zipping again...

Thanks a lot Luc for sharing your ideas! They seem very useful to me, too.
I will seperate your thoughts into single issues:

Modify elements in root presentation: #6
Modify presentation master: #7
Modify additional graph properties: #8

Regarding this:

* Modify cell format (e.g. specify format such as @,%, 990,00, ....)

Will this be possible by linking data label format to source format in datasheet? automizer should keep the original number format from chart template. I'm not sure on this.

Hi Thomas,
Just for your information but you know that, in VBA, you can call formatNumber() to specify the format and you can specify some formats with data source, formats differents from source template (e.g. 0.35% when the template is configured with ´General' for example).
I agree with you when you say that automizer should keep the original format from chart template.
When you link data with some formats (e.g. 0.35%), the new formats overide the default (in the template) but sometimes we want specify programmatically format such as '999.99'. If it's not correctly define in the template (e.g. forget to set correctly for one cell of a serie in a chart the good format), it doen't matter.
The main aspect for me is when you expand a chart (e.g. adding series), I'm not sure that, the format will be propagate correctly (does it inherit from previous serie ?) ...I don't think so.
Thank you Thomas and have a very nice weekend
Luc

automizer should keep the original number format from chart template

Le 10 avr. 2021 à 22:18, Thomas Singer @.***> a écrit :

Regarding this:

Modify cell format (e.g. specify format such as @,%, 990,00, ....)
Will this be possible by linking data label format to source format in datasheet? automizer should keep the original number format from chart template. I'm not sure on this.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Hi Thomas,
I've found a bug. I send you this by email but if you prefer, I can open a bug on GitHub.
To reproduce it ;
Create a template with 1 categorie and 2 series (Cell are number format)
And then, send these data to the existing chart :
Final Dump : {
series: [
{ label: 'Participants Soc1' },
{ label: 'Participants Soc2' },
[length]: 2
],
categories: [
{ label: 'Feb 2021', values: [ 3, 5, [length]: 2 ] },
{ label: 'Dec2020', values: [ 1, 2, [length]: 2 ] },
{ label: 'Jan2021', values: [ 3, 5, [length]: 2 ] },
[length]: 3
]
}
When I try to write the file :
{
modify: [Function (anonymous)],
children: { v: { modify: [Function (anonymous)] } }
}
Error: Index 0 not found at "c"
at ModifyXmlHelper.assertNode (/Users/luc/gentdb/node_modules/pptx-automizer/dist/helper/modify-xml-helper.js:39:23)
at ModifyXmlHelper.modify (/Users/luc/gentdb/node_modules/pptx-automizer/dist/helper/modify-xml-helper.js:24:18)
at ModifyXmlHelper.modify (/Users/luc/gentdb/node_modules/pptx-automizer/dist/helper/modify-xml-helper.js:31:22)
at /Users/luc/gentdb/node_modules/pptx-automizer/dist/modify/modify-chart.js:143:27
at Array.forEach ()
at ModifyChart.setWorkbook (/Users/luc/gentdb/node_modules/pptx-automizer/dist/modify/modify-chart.js:141:30)
at ModifyChart.modify (/Users/luc/gentdb/node_modules/pptx-automizer/dist/modify/modify-chart.js:69:14)
at /Users/luc/gentdb/node_modules/pptx-automizer/dist/helper/modify-helper.js:49:66
at /Users/luc/gentdb/node_modules/pptx-automizer/dist/classes/shape.js:88:41
at Array.forEach ()
I hope this trace will help you Thomas.
Best regards
Luc

On 11 Apr 2021, at 08:38, Luc zaclys @.***> wrote:
Hi Thomas,
Just for your information but you know that, in VBA, you can call formatNumber() to specify the format and you can specify some formats with data source, formats differents from source template (e.g. 0.35% when the template is configured with ´General' for example).
I agree with you when you say that automizer should keep the original format from chart template.
When you link data with some formats (e.g. 0.35%), the new formats overide the default (in the template) but sometimes we want specify programmatically format such as '999.99'. If it's not correctly define in the template (e.g. forget to set correctly for one cell of a serie in a chart the good format), it doen't matter.
The main aspect for me is when you expand a chart (e.g. adding series), I'm not sure that, the format will be propagate correctly (does it inherit from previous serie ?) ...I don't think so.
Thank you Thomas and have a very nice weekend
Luc

automizer should keep the original number format from chart template

Le 10 avr. 2021 à 22:18, Thomas Singer @.***> a écrit :

Regarding this:

Modify cell format (e.g. specify format such as @,%, 990,00, ....)
Will this be possible by linking data label format to source format in datasheet? automizer should keep the original number format from chart template. I'm not sure on this.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/singerla/pptx-automizer/issues/5#issuecomment-817197083, or unsubscribe https://github.com/notifications/unsubscribe-auth/ATO5MUACXFECEUT5GMBGULTTICXDDANCNFSM42WK6DJQ.


[Bug report] Webpack not working

Description
Related to https://github.com/vuepress/vuepress-next/commit/acbdc7f5c03030fb7fdc32beef898a879fbf8e48
Reproduction
https://github.com/vuepress/webpack-bug
Used Package Manager
pnpm
System Info
Not needed

It should have been fixed by 2ac78bda6299639b8bd15935decc884335d53e02. May need a new release


lightslider breaks when refresh occurs while instance isn't visible (example: device rotates, lightslider in a hidden tab)

This may not affect all uses of lightslider, but the elements get really messed up and many disappear when the browser is resized while lightslider is not visible.  So if a lightslider instance is in a container that has display none (such as in tabbed content) after the lightslider is rendered and the user rotates their tablet or resizes their browser window, when lightslider returns to visibility it is messed up (perhaps because the .clone.left and .clone.right elements are deleted and do not return).  Calling refresh() on the lightslider instance does not fix the issue.  I was able to reproduce the issue even on the lightslider project examples page by toggling a the parent of a lightslider's container element to display: none, resizing the browser window, and returning the container to display: block.
My tests have been with a traditional slider showing one image at a time (gallery:true, item:1, loop:true).
A workaround that seems to resolve the issue is to add an intial line of code in the refresh.init() function to prevent it from running if the lightslider instance isn't visible.  As a result when the lightslider instance returns to visibility it seems to be fine.  There may be better solutions than the one below as far as performance is concerned.
Prevent corrupting refresh when lightslider is not visible:
refresh.init = function () {
    if (!$el.is(':visible')) return;

Perhaps there is a better solution that can be committed back to the project?

I'm having the same issue. My slider is inside a modal. The steps I follow to generate the bug are:

Load the page
Open the modal. When the modal is shown, nothing can be seen.
Resize the browser with the modal shown. Slider loads.
Close the modal
Resize the browser
Open the modal again, nothing can be seen.
Resize the browser again, with the modal open. Slider loads.

This happens all the time that the modal is opened and browser is resized

@andyf101 Could you tell us more in detail how you solved that? Where did you change that code?

Find the declaration of the refresh.init() function in the source / unminified version of the lightslider.js file
The line number will vary by version of the JS file
Add the "if not visible, return/exit init" line of code I provided as the first line in the refresh.init() function, leaving all existing code in place
Use the modified JS file and minify as your project dictates

@andyf101 i found a couple of matches on "refresh.init()", which ones should i replace with your version?

@xd0 The only one that is a function expression and that I identified in my initial post

@andyf101 Thank you sooooo much! Finally got this issue sorted now thanks to you. I've been spending days troubleshooting this.

@andyf101 Thank you for the solution! Works like a charm...


Remove perceptilabs tile and resource

Description
Remove Perceptilabs tile and resources from dashboard

How Has This Been Tested?
Tested changes using local dashboard since change is minor
livebuild: quay.io/modh/rhods-operator-live-bundle:1.15.0-rhods-4183

Merge criteria:

[x] The commits are squashed in a cohesive manner and have meaningful messages.
[x] JIRA link(s): https://issues.redhat.com/browse/RHODS-4183
[x] The Jira story is acked.
[x] The developer has manually tested the changes and verified that the changes work.
[x] QE contact acknowledges that this has been tested and is approved for merge. (Original approved pr is #227 )

This PR was approved by @bdattoma on https://github.com/red-hat-data-services/odh-dashboard/pull/227 . However, that PR was created against the main branch. In 1.16, we are using a separate release branch. This PR updated to point to the release branch.


Allow TLS verification to be skipped

Hello,
In production we're using valid TLS certificates only for securing the connection, not validating the client. However in test I'm using self-signed certificates for this. There doesn't appear to be a way to disable the NATS client from attempting to verify the TLS certificate of the server. It would be nice to have this option.
Thanks

Are you signing the certs using a internal CA for test purpose? This is quite easy and then you just tell it the path to the CA?

No, I didn't want to go through the hassle of setting up a CA.

It's the right thing to do though and its really easy.
That said, server does support disabling verify so seems reasonable CLI should too, its a easy PR if you want to else it might take some time.

This looks like a good first issue to tackle. I will look into it.

Bumping on a slightly different request - would it be possible to report that the server's certificate is not trusted? Currently the NATS CLI just fails with an "i/o timeout" -- but it would be great to report that the x509 certificate was signed by an unknown authority - or whatever the trust issue is.

$ nats --tlsca wrong.pem rtt
nats: error: tls: failed to verify certificate: x509: certificate signed by unknown authority

This is already happening

If I don't supply a --tlsca, then the operation times out. This happens if you're connecting to a context prefixed with tls://, not supplying a CA, and the certificate returned is not trusted.

Struggling to reproduce that, not giving a CA is same as using system CA - but its possible I fixed this in main already so I am not seeing it.
Can you do a build of main branch and compare @thorntonmc, if you don't know Go I can give you some other options to try it

@ripienaar same issue here. Could this be an issue with the version of NATS server? We're running 2.9.15.
mthornton@mbp:~/github.com/nats-io/natscli (main)
$ git branch
* main
$ go run nats/main.go stream ls --tlsca=./mycrt.crt
main: error: setup failed: dial tcp <IP_ADDRESS>:4222: i/o timeout
exit status 1

@thorntonmc weird, this sounds like a bug in nats.go to be honest, or maybe lack of some feedback via callbacks.  do you know go? Can you maybe write a small bit of go to try and reproduce using the client?


Can we add support of ES6 import / export modules?

request-promise-native won't be implemented using ES6 modules because it still needs to support node v0.12. However, you can think of it as:
export default request

Would you like to import anything other than the main request function into your code?
Or is your question a more basic one? Then the answer would be: Yes, you can use babel for that.

I just want to be consistent in my codebase.
I can easily do:
const request = require('request-promise-native');

but
import * as request from 'request-promise-native';

doesn't work :(

Unfortunately, I don't have a test set up for that so please give me more input:

Do you use a transpiler? Babel?
Does import request from 'request-promise-native' work?
What error does import * as request from 'request-promise-native' produce?
To what code is import * as request from 'request-promise-native' transpiled into?

I use Typescript 2.0
It throws an error: error TS2307: Cannot find module 'request-promise-native'.
The same as 2)
It doesn't transpiles because of error in 2) and 3)

Alright, thanks.
The issue is not urgent since you can still use require, right? Since I am really busy right now I hope I still can find some time to look into it rather sooner than later.

Just a heads up I am using babel for transpiling ES6 and
import request from 'request-promise-native';
works without a problem.

Thanks @ericmwalsh , good to know!

Maybe now that Node v0.12 has reach EOL this can be done?

Just tried import * as request from "request-promise-native"; both with typescript (with @types/request-promise-native installed) and ES6 (via babel) and both work without a hitch.
IOW request-promise-native already supports it & @analog-nico, I think you can close this issue.

Try using node with --experimental-modules
echo "import * as request from 'request-promise-native';" > index.mjs
node --experimental-modules index.mjs

Currently getting,
(node:3105) ExperimentalWarning: The ESM module loader is experimental.
TypeError: Cannot read property 'onReady' of undefined
    at Module.load (internal/modules/cjs/loader.js:631:22)
    at tryModuleLoad (internal/modules/cjs/loader.js:560:12)
    at Function.Module._load (internal/modules/cjs/loader.js:552:3)
    at Module.require (internal/modules/cjs/loader.js:657:17)
    at require (internal/modules/cjs/helpers.js:22:18)
    at Object.<anonymous> (/home/ecarroll/code/toponym-esque/node_modules/psl/index.js:14:19)
    at Module._compile (internal/modules/cjs/loader.js:721:30)
    at Object.Module._extensions..js (internal/modules/cjs/loader.js:732:10)
    at Module.load (internal/modules/cjs/loader.js:620:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:560:12)

Really though ti seems like this whole approach is wrong. Moving forward at some point for posterity I would expect request.pm to work with async, await, and to adopt ES6 module syntax.

If anyone is still struggling with this and isn't using a transpiler that handles this, here is an example of how babel manages to get both es6 imports and require to work:
https://babeljs.io/repl/#?babili=false&browsers=&build=&builtIns=false&spec=false&loose=false&code_lz=JYWwDg9gTgLgBAG2gUxHAZlCaBEwwDOAriDgNxA&debug=false&forceAllTransforms=false&shippedProposals=false&circleciRepo=&evaluate=false&fileSize=false&timeTravel=false&sourceType=module&lineWrap=true&presets=es2015%2Creact%2Cstage-2%2Cflow%2Cenv&prettier=false&targets=Node-9.9&version=7.4.4&externalPlugins=

// tsconfig.json
{
  "compilerOptions": {
    "module": "commonjs",
    "noImplicitReturns": true,
    "noUnusedLocals": true,
    "outDir": "lib",
    "sourceMap": true,
    "strict": true,
    "strictNullChecks": false,
    "target": "es2015"
  },
  "compileOnSave": true,
  "include": [
    "src"
  ]
}

// package.json
{
    "scripts": {
        "lint": "tslint --project tsconfig.json",
        "build": "tsc",
        ...
    },
    "dependencies": {
        "request-promise-native": "^1.0.7",
        ...
    },
    "devDependencies": {
        "@types/node": "^12.0.2",
        "@types/request-promise-native": "^1.0.16",
        "tslint": "^5.12.0",
        "typescript": "^3.5.1",
        ...
    }
}

import * as request from 'request-promise-native';

const uri = 'https://api.example.com';
const options = {
      headers: {
          'User-Agent': 'CustomAPI'
      },
      json: true
};

request(uri, options)
      .then(response => {
          console.log('response', response);
          return response;
      })
      .catch(error => {
          return error;
      });

// Global version of typescript is same as imported dependency in package.json
$ tsc -v
Version 3.5.1

Works for me on Firebase Cloud Functions, and I'm not using babel.

I am having the same issue when I am not using any compiler (babel, etc)


how do I use SendKeys? I am using pywinauto 0.6.5 and Python 3.5.1

I am trying to send the following
####to a terminal window###
from pywinauto import keyboard
keyboard.SendKeys.send('jsmith')
########################
but when I try to run the above, I receive the invalid syntax
File "demo.py", line 35
keyboard.SendKeys('jsmith')
^
SyntaxError: invalid syntax

SendKeys is a function, not an object/module. How it works in my Python console:
>>> from pywinauto import keyboard
>>> keyboard.SendKeys.send('jsmith')
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
AttributeError: 'function' object has no attribute 'send'
>>> keyboard.SendKeys('jsmith')
>>> jsmith

So the error message is correct for Python 3.6.5 32-bit. It should be the same on all Pythons. Maybe you have something more with indentation or something on previous lines.
Anyway there is nothing to do with pywinauto.


Updated development roadmap

Hey! I'm enjoying learning to use the benchmark! However, I'm wondering about the plan to support multi-environment training. It says it's on the agenda, but would you happen to know what timeline we're looking at?
Alternatively, what are the guidelines for contributing, or are there specific things you'd like outside support on? I'm not necessarily the person to provide this, but it could be helpful to anyone interested in using the benchmark.
Best, Lars

Closing this as I realized that it's only the feature sim env that does not support multiple environments.


Unique line file

Hi
CREPS_Rimouron963.zip
If the file has one unique line, the program fail.
Problem is here:
with open(gpx_file) as file:
for line in file:

@darrepac thanks for the report, did you get this GPX file from Strava? The parser in this script is very limited, and was only tested with GPX files from Strava.

No, indeed this is a generic file not coming from Strava


Add http header support

Add http-header Support, fixes #33

Would love to see this go in !

👍 good work. This will go in.
Both will be merged & released (this & #31) along with #38 in a couple of days.
If you want to skip the wait, you can fork & build yourself.

What's the ETA on getting this merged and released?  I'd really like to use this functionality and am having issues trying to figure out how to build the project.

sorry for the delay.
The work will appear in the next version of this project.


documentation files missing from EXTRA_DIST in docs/Makefile.am

These files need to be listed in EXTRA_DIST in docs/Makefile.am, or the tarball distribution will not be able to build the docs.
attribute_conventions.md file_format_specifications.md internal.md

I believe this got fixed so I will close this issue.


Spanish section is empty

Needs to be populated and links should work.
Ariel might be able to help with this.

blocked on translator


Add api to remove param's from nodes

Currently we can add parameters, but not remove them. It would be nice to be able to do both, to be able to do things like have math nodes that take a variable number of inputs, or to be able to grow/shrink the number of ouputs so that there's always another free one (for... reasons).
I've implemented this for output param's in 96dc5b6f672a5f8096d5f85b71ed2234ed835f27 - which depends on #30.

Indeed, that would be quite useful. It's good that more people are using the library so we can test it against a variety of use cases :slightly_smiling_face: I'm happy to accept a PR for this once #30 gets merged


Improvements to SQLite metadata reader

Fix bug in primary key ordering.
SQLite reverse typemapping only adds ColumnType annotation if it has a custom type, instead of adding for every type.
Note:
Once approved, will merge into dev after #3367 is merged.
Fixes #3404

/cc @lajones

:shipit:

Fixed typos and formatting. :watch: on #3367


Any support for WPF in the future?

Since the Composition api deals with the DWM and Win2D etc, will you ever support this on WPF ??

Hello ???  Anyone from the Composition team active in these forums??

Could you elaborate more on your needs with Composition APIs in WPF? We'd love to know more specifics of what you want to achieve.
This thread on User Voice is where we are track developer votes for the feature: https://aka.ms/windowsui/inwin32.
These are the WPF related items from our product backlog: https://developer.microsoft.com/en-us/windows/platform/features/?q=wpf

Thanks for the links!  It's exactly that, to have the Composition API available to WPF to keep up with the latest changes to UWP. WPF still has a large user base and such an API would be a great benefit. I will be voting.

Thanks Max. We are well aware of the large user base that WPF has, so we are exploring options to figure out the right way to get Modern UI to WPF and other Win32 Desktop applications. Your votes would definitely help us justify and if you could get a lil more specific on the parts you are trying to modernize or use Composition APIs, that ll help us prioritize and build the right solution.

In my personal opinion, just the basics would be a great start for WPF: translate, scale, rotate, opacity as well as blur and shadow effects. The performance alone from the Composition API is beneficial enough.

That is the exact kind of information I was looking for. Thanks.


Wrap and name LoadBinaryModule arguments

Contributes to #14171.

Check failure on line 1 in Verify Markdown Links.Verify links in /home/vsts/work/1/s/docs/building/internals.md.https://cmake.org/download/ should work

@azure-pipelinesazure-pipelines
/ PowerShell-CI-static-analysis
Verify Markdown Links.Verify links in /home/vsts/work/1/s/docs/building/internals.md.https://cmake.org/download/ should work
Connection timed out


Fix spacing

Other than the one note on the image, this looks good to go. :+1:

Just to confirm:

You run the spec file and it saves all the screengrabs into doc/ux/ with harvest_styleguide.rb
You grab that rendered output and concatenate it together into app/views/styleguide_template.hbs
That same file gets put into the ember app

As far as these changes go, I think they're fine.  Do we need to check in the generated files? I'd prefer not to if possible.  At least not the one in app/views that gets copied over into the client.  Also, could you update the comment in populate_styleguide.rb so that it points to the correct spec file (line 5) it currently says "spec/features/populate_styleguide" when the file is actually named "harvest_styleguide"

thanks @Bestra, I understand why checking the styleguide.hbs Template is not ideal, as it adds to the size of the initial payload.
What is a good alternative here? More specifically, how could we -not- deliver that to every initial request, and only deliver the styleguide if a user hits the /styleguide route directly?

In light of our conversation I'd say :+1:


Use "typos" tool to bulk detect/correct typos in RN core code

Summary:
Changelog:
[Internal] -
While merging a Contributor PR in D43816768, I got curious whether such things as detecting/fixing typos could be automated.
There is a tool called typos, which, as it turns out, works very well on source code files.
The amount of false positives was minimal, most of the suggestions were valid ones.
And the total amount of typos found and fixed was more than 300, which is... quite a bit :)
Metodology

Run typos separately on ReactCommon, Libraries, React, ReactAndroid and scripts folders inside packages/react-native, e.g.:

$ typos -w --config ~/tmp/_typos.toml ~/fbsource/xplat/js/react-native-github/packages/react-native/ReactAndroid

Contents of the _typos.toml config file:
[default.extend-words]
collapsable = "collapsable"
NDK = "NDK"
inout = "inout"

(yeah, it really didn't like the "collapsable" word, for some reason ;))

Inspect all the changes manually and revert false positives (as mentioned, their amount was minimal)

Note that most of the changes are inside things like comments and error messages, however there are a few among identifier name fixes (typos is actually quite smart at understanding naming conventions in code) - I kept the suggestions in cases that are local and not part of a public API. There are also misspelled file name changes in a couple of cases.
Differential Revision: D44172494

@rshest This is awesome! Btw, will it be possible to add typos as dev dep and keep its config file in the root, and a typos script in package.json?
That way the authors can run the script (only for their changes) by passing cli args to the dir/sub-dir to check the typos before pushing the diff to the remote/upstream. (Obvio the script would require some minor changes to handle args from cli.)
Another thing we would like to keep in mind is that not making it mandatory like those pre-commit hooks 🙂 (thank god, we don't have them here). Instead, we can make it mandatory in CI just like linter if we want to...
Lmk WDYT; I'll be happy to submit a PR for this :)

Closing as this landed in b9c64f3

@cortinico My bad, sorry, I must have forgotten to export the latest change there. Thanks for taking care of this!

@rshest This is awesome! ~Btw, will it be possible to add typos as dev dep and keep its config file in the root, and a typos script in package.json?~
Edit: It seems they don't have npm pkg, so what we can do is we can keep the config file in the root as mentioned. If authors want they can download the platform-specific bin. Also, we can try the CI setup btw...
That way the authors can run the script (only for their changes) by passing cli args to the dir/sub-dir to check the typos before pushing the diff to the remote/upstream.
Another thing we would like to keep in mind is that not making it mandatory like those pre-commit hooks 🙂 (thank god, we don't have them here). Instead, we can make it mandatory in CI just like linter if we want to...
Lmk WDYT; I'll be happy to submit a PR for this :)

Hey Pranav! Thanks for the input, and I think that's a sound idea in general!
We should consider, however, whether the benefit from adding the extra moving part would outweigh the cost of maintenance, running and dealing with potential breakages and people consequently losing time, caused by this very moving part.
Would be happy to hear opinions here :)

Yeah. It may create extra chaos.

whether the benefit from adding the extra moving part would outweigh the cost of maintenance

From my initial observations the tool doesn't even take 1sec to scan whole codebase (provided the codebase is already scanned once, and has fixed number of typos).

running and dealing with potential breakages and people consequently losing time, caused by this very moving part.

True. FYI, I'll be experimenting with the typos gh-actions on none other than React.dev repo ci :)
So, hopefully we'll be able to use those insights and finding and we can move forward this discussion afterwards.

True. FYI, I'll be experimenting with the typos gh-actions on none other than React.dev repo ci :) So, hopefully we'll be able to use those insights and finding and we can move forward this discussion afterwards.

Oh, that's great! Let's test drive this on the React repo, then, and if that works well - by all means, please feel free to make a PR for integrating it in RN :)


Can Lux Subscribers support re-subscribing if postal is nuked.

Is this still something we want to look into adding? It seems like our dispose and createSubscribers methods on dispatcher might have removed the pain behind this request. Thoughts @rniemeyer or @ifandelse?

I don't see this as being a big problem any longer. I think that it originated from some pain in testing, but doing proper cleanup of the lux objects was the better choice than resetting postal.

I'm good with closing this if the original pain points have been resolved elsewhere


Dependency upgrades

Update some dependencies to latest version
Remove usage of lexical-core, use str::parse instead

oh hi @Dandandan :D

build failed, so yeah, you probably don't want to touch actix-web because their tokio 1.x support is still beta and going through big refactoring all the time. I have plan to rip it out and switch to warp instead. My experience with actix-web so far has been really unpleasant...

Hi 😁
Let me see if I can revert that.
Maybe you'll also want to have a look at https://github.com/tokio-rs/axum this seems to getting some traction lately

Oh yeah, I was literally trying to find axum because I saw its announcement on reddit the recently but couldn't remember the exact name. Thanks for the pointer!

filed https://github.com/roapi/roapi/issues/76.

@houqp should be fixed now. I minimized the actix / Cargo.lock changes now.

There was a clippy error


Teacher dashboard fixes

This adds an "unarchive this class" button to a class's page if the page is archived. It also refactors out .style-flat which used to be included redundantly.

Need to fix the client test, and handle merge conflicts. Other than that, 👍


Firefox mobile compatibility

As a Firefox user on desktop and mobile, I would really like to be able to install the extension on Firefox mobile.
I do not intend to use the DuckDuckGo browser on mobile. I'm already using Firefox Focus most of the time, and Firefox Mobile the rest of the time (using sync features).

@MariagraziaAlastra @dharb Ping...

Hey, Thanks for the submission!
We don't have this on the roadmap at the moment as the submission process for these addons would require some work in changing our features, however we do agree that it would be great to support this.

@jonathanKingston thanks for leaving this open as a "nice to have", even if it's not on the roadmap.

Could we get a list of the problems in this ticket, for external contributors to be helpful?
Now that even Firefox Beta allows custom addon collections it's possible to install the ddg addon without the addons store. It's possible to document and fix  functional issues even without the listing. Any help there is welcome too...


Add coverageDepth to coverage statements

Adds coverageDepth property to TIPPCoverageStatements
Further changes:

Adjust package TSV export functions

Create a line for each coverage statement
Add book columns

Adjust TIPP upsert
Adjust cleanup function
Adjust tipp template
Adjust Package & TitleInstance OAI serialization

Addresses #46 and #22

Agreed, something we should come back to.


Organize params in ra25 -- minimal base with all defaults..

@rcoreilly will do

this is in latest.  ra25x is experimental version with misc params explored.


This does not appear to work

Followed the steps in 'Set container permissions in the Azure portal' but still get the 'The value for one of the HTTP headers is not in the correct format' error message when trying to access blobs in the browser.

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

ID: a299cd29-143b-c263-7996-3467035b8937
Version Independent ID: a8a1d0d0-411e-44eb-51da-c1f8ab90befa
Content: Enable public read access for containers and blobs in Azure Blob storage
Content Source: articles/storage/blobs/storage-manage-access-to-resources.md
Service: storage
GitHub Login: @tamram
Microsoft Alias: tamram

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

Actually it did work, sorry! I had a link to file.core.windows.net, when I switched to blob.jadajada it did work.
Sorry forgot I made this remark!

@gjvdkamp Thanks for sharing this information. We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.

The document from MS. Docs does not tell how to enable or configure Azure storage account for public access.


Flutter doctor command not working

here is the report
flutter_01.log
flutter_02.log

How did you install?

Seems like the flutter directory is not a git repository? Did you clone the Flutter using Git?

From the website

Extract the zip-file and place the contained flutter in the desired installation location

No i just follow the instructions given on website

Doesn't provide any information about where you installed Flutter.
Without information there is no way to help you.
What OS are you using?

I just extract flutter in c program files
On 10-Jun-2018 2:11 PM, "Günter Zöchbauer"<EMAIL_ADDRESS>wrote:

From the website
Extract the zip-file and place the contained flutter in the desired
installation location
No i just follow the instructions given on website
Doesn't provide any information about where you installed Flutter.
Without information there is no way to help you.
What OS are you using?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/18271#issuecomment-396031907,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Abbk5Ofa7SerJLyzXinxt1F6iM52OkhIks5t7NuggaJpZM4Uep0g
.

Try a directory instead that doesn't require admin privilegs for writing

Ok i will try that
On 10-Jun-2018 4:28 PM, "Günter Zöchbauer"<EMAIL_ADDRESS>wrote:

Try a directory instead that doesn't require admin privilegs for writing
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/flutter/flutter/issues/18271#issuecomment-396039673,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Abbk5P2HcWUYsKqDpJajUx0r4bZq6b2Eks5t7PvAgaJpZM4Uep0g
.

Closing as dup of #16898
Please add a comment if you don't think this is a duplicate.

I disagree that this is a dup.  #16898 deals with read-only directories.  I followed the directions for flutter installation exactly, even copying the flutter directory to c:\src\ and I have the same problem as the original user.  #16898 was not helpful to me.

This issue is closed since quite some time.
Please create a new issue with fully filled issue template.


Improve library import speed

From my tests it will take around 1s:
import jsontableschema

why?

I don't know) I'll profile. There are a few chunks of code which runs outside of functions and classes. But also may be it's normal because of dependencies. Juts discovered it because GT time to create a report for simple dataset could be around 100ms but overall around 2s and most of them on jsontableschema import.

The tabulator import is instant after the fix. But the jsontableschema is still slow to import... ;( I guess it was not only affected by tabulator, but has issue itself. Can you please have a look?

FIXED
Latest tests just show not it's around normal (around 200ms). It still needs to load profiles.


Notification support for recieved files

Is your feature request related to a problem? Please describe.
Add notifications on when you receive a file with a accept or decline button similar to what  Warpinator has so you are notified when a device is sending you a file or folder.
Describe the solution you'd like
This is a example on Manjaro Gnome

Describe alternatives you've considered
No response
Additional context
No response

I would like to see this feature as well on Android and MacOS

And on Windows too!

This package might help, flutter_local_notifications.
But why does it have no windows support?

Would love that too.


chore: fix some type errors around forms

by typing a couple central components we get rid of a lot of type errors.

:tada: This PR is included in version 2.153.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.153.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


feat(ui): add a popup option

This still needs work but I think it's nearly there 😄
You might be able to handle some of the quirks better, since you're more familiar with the state handler.

Here's a test with a small terminal space from the beginning

I still need to figure out where those weird lines at the top are coming from.

I still need to figure out where those weird lines at the top are coming from.

This is probably the indent blankspace plugin. You can disable it for certain buftypes and filetypes, like so https://github.com/williamboman/nvim-config/blob/main/lua/wb/plugins.lua#L180-L181. That reminds we should probably set a dedicated filetype for the UI buffer..

The title was wrong because I wanted to base it on the logger branch for easier debugging, so you can also ignore the logger commits.

This is probably the indent blankspace plugin. You can disable it for certain buftypes and filetypes, like so https://github.com/williamboman/nvim-config/blob/main/lua/wb/plugins.lua#L180-L181. ~That reminds we should probably set a dedicated filetype for the UI buffer..~ this is already being set apparently

I made the options into a list just so that I can quickly toggle them on and off and I still couldn't figure out what's up with it. I need to see if it's something with the redraw hook.

Let's make this default behavior and remove the old window method entirely.

That will actually simplify some of the code and probably protect against some potent bugs, since I didn't really heavily test using a split afterwards.

With ephemeral floating windows we can safely couple the buffer with a single window, where we can also safely assume that if the buffer is valid

If you're set on that then maybe we can also make a meta-table for the popup instead of relying on global state. See this minimal implementation:  https://github.com/LunarVim/LunarVim/blob/9b76a13512b407f23eb9fc16d0210daccccf4802/lua/interface/popup.lua
or something with more bells and whistles: https://github.com/nvim-lua/popup.nvim/blob/f91d80973f80025d4ed00380f2e06c669dfda49d/lua/popup/init.lua

Could you also remove the log code from this PR 🙏?

As I said it's for debugging purposes, but sure, I can use them without committing the files 👍

If you're set on that then maybe we can also make a meta-table for the popup instead of relying on global state. See this minimal implementation: https://github.com/LunarVim/LunarVim/blob/9b76a13512b407f23eb9fc16d0210daccccf4802/lua/interface/popup.lua

Yeah I think this could be nice. However since there's only one UI window currently, maybe let's consider it in the future if there should be a need for more variants of UI windows? I avoided metatable initially because they're so awkward to write from memory. I'm not sure it'll particularly help with state management though? The current state is kept in a closure rather than in say a metatable instance, which to me is just semantics.

That will actually simplify some of the code and probably protect against some potent bugs, since I didn't really heavily test using a split afterwards.

Yep exactly! Let's do it :+1:

I thought I could do a quick clean-up just to see if there's anything broken. We still have a few things to fix/test.

👍

On 25 Sep 2021, at 19:40, kylo252 @.***> wrote:

@kylo252 commented on this pull request.
In lua/nvim-lsp-installer/ui/display.lua:

@@ -99,42 +99,72 @@ end

function M.new_view_only_win(name)
local namespace = vim.api.nvim_create_namespace(("lsp_installer_%s"):format(name))

local buf, renderer, mutate_state, get_state, unsubscribe

local buf_handle, renderer, mutate_state, get_state, unsubscribe, win_id
If we're going by the general usage then it's bufnr and win_id, at least from what I've seen in core and Telescope.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

@kylo252 FYI I created the following PRs to fix some bugs I ran into:

https://github.com/kylo252/nvim-lsp-installer/pull/3
https://github.com/kylo252/nvim-lsp-installer/pull/2

So I had to stop at 832e37c because anything after this commit is causing a segfault when pressing <c-w>

Hmm I think I might've forgot to push some things actually (probably won't fix segfault, but still).. What system are you on?

From my end I think this is ready to go (apart from the segfault issue of course)

I just tried it on two different machines and both of them are crashing if you attempt to press <C-w>. That's bad because it's probably the first button most users reach for when dealing with splits/windows 😢

Haha yes not very good. What OSes and nvim --version did you try? These work fine for me:
- Windows 10, nvim v0.6.0-dev+112
- macOS Monterey Beta, nvim v0.6.0-dev+344

Does increasing verbose level help pinpoint the issue maybe?


All docs need to be sorted to indicate what content they link to

Current state
A list of links which point to various places
Expected:

Video links

Lorem ipsum
Lorem ipsum
Lorem ipsum

Blog links

Lorem ipsum
Lorem ipsum
Lorem ipsum

Not needed if there is only one link
Alternative solution
Links could look like this:
Lorem ipsum - YouTube: Author name
Lorem ipsum - Auth0: Author name

Alternatively add emoji to a link pointing to a video 🎥

I think icons would be a better idea.
Ie.: youtube video - [youtube icon] Author - Lorem Ipsum
blog/article - 📘Author - Some title
etc.


On version 5.2.5 & iOS 13 whole survey screen is pink, and no content is shown.

On version 5.2.5 & iOS 13 whole survey screen is pink, and no content is shown.

It works fine on  version 5.2.5 &  iOS 12
or on version 5.2.2 & iOS 13

Thank you for the report. I will take a look at this right away.

We've identified the cause of this issue. It occurs when Apptentive iOS SDK version 5.2.5 is built on Xcode 10 and run on a device running iOS 13.
If you are unable to wait for a release that fixes this bug, please use version 5.2.4 in the meantime, or build your app using the latest Xcode 11 GM seed.
Again, thank you so much for reporting this issue! We will release a fix as soon as possible.

Thank you!


Patch/oauth default challenge handler

Hi @Rachael-E & @mbcoder,
I've changed the OAuth sample to use the Default Challenge Handler. This simplifies the sample a fair bit, since there's no need any more for the custom ChallengeHandler class.
As I understand, this would be the standard workflow for OAuth on Java? Please let me know if the comments can be improved in any way. Thanks!
The readme is also updated to reflect the use of the Default Challenge Handler, using the current state of the refactored readme from the common samples repo.
I've also added a few lines in the Additional Information section as mentioned in a different issue. This will also be added to the refactored design readme.
@tschie : I've added Mark as a reviewer as we discussed these changes in person here in Edi, and he's worked with the OAuth API recently and advised me plenty on the workflow.

Hi all, just to highlight that I've renamed the sample and package:
OAuth -> AuthenticateWithOAuth
would be good to get an OK on this. Discussion about renaming samples to better reflect workflows rather than just naming ht API has come up a few times recently. Perhaps we could discuss what everyone thinks about this, and whether it should be dealt with in a more focused effort. Regardnig this particular one, happy to change it back to just OAuth if that's prefered.

Renaming of samples should be done as a samples team effort to ensure parity across sample names across SDKs. The time to do this would be during the README updates for individual SDKs. I think the best approach would be to suggest the title change at its design level and let it filter back down that way. Personally, I'd like how OAuthAuthentication to tie in with the TokenAuthentication naming.

Thanks @Rachael-E, I've reverted it to the original naming for now. Sorry about the back and forth - making it a concentrated effort would be best.


gpu support ?

Thanks for the interesting research.
I wanna try this gnn-cv by myself. Did you calculated the gnn-cv with gpu?  If only cpu is used, any comment on calculation cost of simulation examples tested in preprint would be helpful.
Thanks!

Hi, the GPU evaluation of CVs is supported. E.g.: https://github.com/jintuzhang/gnncv/blob/3d0a788a2a215576a5a65b417dd9b82fb574e5a1/nacl/run_biased_gnn/6A_2layer_1c/1/plumed.inp#L29

Thanks.  Now I'm struggling with the torch dependency issues. Could you tell me the torch versions that are tested, especially for this gnncv & mlcolvar modified for graph ?

Tesed PyTorch versions including 2.0.1 and 2.2.2, and tested LibTorch versions including 2.0.1, 2.1.2, and 2.2.2. However, for LibTorch, versions 2.1.2 and 2.2.2 are suggested, as earlier versions may contain memory issues (within PyG).
First, try setting up a simulation using the uploaded files (e.g., the alanine dipeptide) to see if this was an issue with your PLUMED installation. If the provided example runs, then there should be a bug in your training script.
As a side note, when fitting CVs for new systems, you may want to use SchNet instead of GVP from the beginning, which is smoother and easier to train, generally.

Thank you for the comments. I'm having same problem in SchNet or GVP model. Also the example NaCl model.ptc has same errors.

I wonder if using independent libtorch causes problem. I mean, when installing pytorch via conda or pip,  there would be libtorch which is linked to pytorch.   And when compiling plumed, independent libtorch ,which is downloaded by wget, is used.   Is it correct to them in this way?

Could you share the the versions of lightning, torch_geometric, torch_scatter ?  Also, I wonder if all of the pyg related packages are necessary rather than only torch_geometric and torch_scatter (pyg, torch_sparse, torch_cluster ... )

Thanks.
One quick question:  Is your libtorch ABI compatible or not?  I've been using abi compatible version of libtorch 2.1.2 for plumed, but I just noticed that pytorch is usually built without ABI.

Both versions of LibTorch (abi and no-abi) have been tested. It would be fine as long as you could compile PLUMED.

I'm not sure,  but for now I'm suspecting the namespace mangling when saving to torchscript may be causing the error. Still working on it..

Finally I found the part directly causing the error.
In PytorchGNN.cpp ,
#else
  // do it normally
  model = torch::jit::freeze(model);
#endif

  // optimize model for inference
#if (TORCH_VERSION_MAJOR == 2 || TORCH_VERSION_MAJOR == 1 && TORCH_VERSION_MINOR >= 10)
  model = torch::jit::optimize_for_inference(model);
#endif

Those freeze and optimize are both somehow modifying the model.ptc structure.
They add name mangle on  "torch.mlcolvar.graph.core.nn.models.GVPModel" to be "torch.mlcolvar.graph.core.nn.models.___torch_mangle_1.GVPModel "
I still don't understand why it happens in my case, while not in yours.
Also, the error doesn't occur only when I disable both.

This is weird, but thanks for the investigation. We may need to figure out a stable solution.


inview does not work well with elements in

I tried using inview with g elements in svg. I am not sure but, the "enter" and "entered" seem to be triggered at the same time, or sometime only one of them. Same for "exit" and exited". In short, handlers are triggered all over the place, not in their supposed order.
Edit 1
So I tested again:

"entered" is fired at the moment part of element enter viewport.
"exited" is fired at the moment part of element leave viewport.

Edit 2
This seems to be related to scroll direction.

@megafirzen Do you have an example I could look at?

I believe that I have lost the file I found this problem on, so I create this new example. Hope this helps.

@megafirzen It looks like this stems from the use element, perhaps in this situation or in every situation, reporting clientHeight as 0. So jQuery and my noframework adapter both think the height is 0. I'll mark this as a bug since getBoundingClientRect appears to give us the right height and further investigation may reveal that I can switch to using it.

Any changes?


sec43: update expat to 2 6 3

Issue: https://devtopia.esri.com/runtime/SoftwareSecurity/issues/43

https://github.com/libexpat/libexpat/issues/887

This PR adds null checks to address security concerns in the current version of libexpat (2.6.2) used in Runtimecore 200.6.0 by updating to libexpat 2.6.3.
(Following instructions in https://devtopia.esri.com/runtime/milano/blob/main/References/upgrading_3rd_party/libexpat.md)

[x] rich0003/sec43_update_libexpat_to_2_6_3
[x] library build: https://runtime-rtc.esri.com/view/vTest/job/vtest/job/3rdparty-interface/1136/downstreambuildview/
[x] rtc vtest: https://runtime-rtc.esri.com/view/vTest/job/vtest/job/runtimecore-interface/26539/downstreambuildview/

test failures look unrelated

@rshepherd549 you forgot to update the conanfile_rtc.py and update the package version number


powershell or any other automation tooling to create new PIM assignments

Are there any automation tools using PowerShell or c# or any other language to automate the creation of new PIM eligibility records? It is not acceptable that we only can use the Azure portal to create a new configuration.
For example, create a new eligible role assignment of a contributor for a resource group.

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

ID: e20f5831-eb6b-93da-5ac8-4f5e7ae13984
Version Independent ID: 0c49066b-8c38-78b9-2829-506dfdeee61b
Content: Manage access to Azure resources with Azure AD Privileged Identity Management (PIM)
Content Source: articles/role-based-access-control/pim-azure-resource.md
Service: role-based-access-control
GitHub Login: @rolyon
Microsoft Alias: rolyon

@borys86
Thanks for your feedback! We will investigate and update as appropriate.

You can use Microsoft.Azure.ActiveDirectory.PIM.PSModule PowerShell Module to enable or disable the role assignments. The PowerShell Module is available here. Also, Please refer to the link for steps to achieve the same.
Check out the Graph API as well: https://docs.microsoft.com/en-us/graph/api/resources/privilegedidentitymanagement-root?view=graph-rest-beta
For actual new eligibility assignments, automation is being planned but is not currently available. You can upvote the feature request here: https://feedback.azure.com/forums/169401-azure-active-directory/suggestions/34195696-powershell-enable-pim-role-assignment

We will now proceed to close this thread. If you have further questions feel free to tag me in the comments or open a new post on MSDN.

@MarileeTurscak-MSFT
why do you close the incident without reading what I am asking?
I am not asking on how to automate elevation of permissions. I am asking on how to create new eligible assignments.


Hint is missing for Subject Keywords

After configuring the DDC for dc.subject.ddc, the hint is not displayed in the UI. I have attached a picture from my test system.
I noticed the same behaviour for Subject Keywords in the DSpace demo site https://demo7.dspace.org/.
The hint is configured in the [dspace]/config/submission-forms.xml, where the dc.subject.ddc is defined.

I would like to claim this issue, I will bundle it with the fix for #1770

@alexandrevryghem : Sounds good!  I'll assign this to you as well too. Thanks!


fix: address bar changes do not effect request, fix #1189

We made the address bar editable in #1226. The address is actually two values (url + path), so I decided to not update the request state. Which means the updated address isn’t used for the actual request.
This PR fixes that bug, and updates the request state.
We can’t easily update two values with just one value. So I decided to reset the path then and just put everything into the url property.
This will bite us if we ever want to persist the api client state and switch between envs (different base urls), but it’s a pragmatic workaround for now. :)

Quick turnaround! Nice fix Hans :)