Allow multiple lines with @description decorator

Is your feature request related to a problem? Please describe.
I could not see any current issue or feature request already discussing this.
Describe the solution you'd like
The description decorator's primary purpose (at least to me) is for intellisense from VS Code when the template or module is being consumed. It would seem to be cleaner method of providing those details if we could use multi-line descriptions for when they are displayed by the extension in VS Code.
Having this view:

Would seem cleaner to me and easier to read by the user consuming it if it would allow for some formatting options:
An array of diskInfo for data disks with:
- sizeGB: [int]
- storageAccountType: [string]

@wsmelton - you can actually do this today using markdown-formatted text in a Bicep multi-line string:
@description('''
An array of diskInfo for data disks with:
- sizeGB: [int]
- storageAccountType: [string]
''')

Cool!
Can this be updated to provide that detail in the parameter doc here? That would help for awareness.

Great suggestion! @tfitzmac FYI

@mumian -- can you take care of this one?

@alex-frankel - I am working on this.

I have updated the parameters article - https://github.com/MicrosoftDocs/azure-docs-pr/pull/187387.

I don't see that pull request going to the repo of the source I linked https://github.com/MicrosoftDocs/azure-docs
Is there another location for documentation on Bicep that we should be referencing?

@wsmelton - I believe this is the change: https://github.com/MicrosoftDocs/azure-docs/commit/afb6237d27b13b233ff8fa160c07cf819a3dc982
azure-docs-pr AFAIK is just a private upstream of azure-docs, so changes merged there make it into azure-docs.

Ah, okay didn't scroll enough on the doc page to see it.


Best way to implement a callback

Hi. Do you've a better way to implement a callback to click listener on itemView?

Hi! I don't know a better way to implement the onClick, but if you know one, I will be glad to hear it!


[BUG] Increasing DefaultAzureCredentialOptions MaxRetries dramatically slows down startup of app

Library name and version
Azure.Identity 1.7.0
Describe the bug
We've seen our azure app service crash twice now (about 30 days apart) and the only error message (pasted below) says that ManagedIdentityCredential authentication failed: retry failed after 3 tries.
I updated the code in our DefaultAzureCredential (see below) updating the MaxRetries to 6 (from 3) and the time our app took to get to the OnModelCreating surged from 38 to 54 seconds.  With 9  MaxRetries the startup is 103 seconds.  These numbers are pretty consistent.  There were no key vault errors and this happens consistently every time since I updated the parameters.  If I decrease the parameters the time goes back down.
But since my KeyVault IS accessible why would changing the MaxRetries make any difference?
        var isDeployed = !string.IsNullOrEmpty(Environment.GetEnvironmentVariable("WEBSITE_SITE_NAME"));
        return new DefaultAzureCredential(
            new DefaultAzureCredentialOptions
            {
                // Prevent deployed instances from trying things that don't work and generally take too long
                ExcludeInteractiveBrowserCredential = isDeployed,
                ExcludeVisualStudioCodeCredential = isDeployed,
                ExcludeVisualStudioCredential = isDeployed,
                ExcludeSharedTokenCacheCredential = isDeployed,
                ExcludeAzureCliCredential = isDeployed,
                ExcludeManagedIdentityCredential = false,
                Retry =
                {
                    **MaxRetries = 6,   //was 3
                    NetworkTimeout = TimeSpan.FromSeconds(10),
                    MaxDelay = TimeSpan.FromSeconds(10)**
                },

                // this helps devs use the right tenant
                InteractiveBrowserTenantId = defaultTenantId,
                SharedTokenCacheTenantId = defaultTenantId,
                VisualStudioCodeTenantId = defaultTenantId,
                VisualStudioTenantId = defaultTenantId
            }
        );

========== Message from Azure App Service when our app crashed today (this is why I'm trying to up the retries)
Application: w3wp.exe
CoreCLR Version: 6.0.1122.52304
.NET Version: 6.0.11
Description: The process was terminated due to an unhandled exception.
Exception Info: Azure.Identity.AuthenticationFailedException: ManagedIdentityCredential authentication failed: Retry failed after 3 tries. Retry settings can be adjusted in ClientOptions.Retry. (The operation was cancelled because it exceeded the configured timeout of 0:00:05. Network timeout can be adjusted in ClientOptions.Retry.NetworkTimeout.) (The operation was cancelled because it exceeded the configured timeout of 0:00:05. Network timeout can be adjusted in ClientOptions.Retry.NetworkTimeout.) (The operation was cancelled because it exceeded the configured timeout of 0:00:05. Network timeout can be adjusted in ClientOptions.Retry.NetworkTimeout.)
See the troubleshooting guide for more information. https://aka.ms/azsdk/net/identity/managedidentitycredential/troubleshoot
---> System.AggregateException: Retry failed after 3 tries. Retry settings can be adjusted in ClientOptions.Retry. (The operation was cancelled because it exceeded the configured timeout of 0:00:05. Network timeout can be adjusted in ClientOptions.Retry.NetworkTimeout.) (The operation was cancelled because it exceeded the configured timeout of 0:00:05. Network timeout can be adjusted in ClientOptions.Retry.NetworkTimeout.) (The operation was cancelled because it exceeded the configured timeout of 0:00:05. Network timeout can be adjusted in ClientOptions.Retry.NetworkTimeout.)
---> System.Threading.Tasks.TaskCanceledException: The operation was cancelled because it exceeded the configured timeout of 0:00:05. Network timeout can be adjusted in ClientOptions.Retry.NetworkTimeout.
---> System.Threading.Tasks.TaskCanceledException: The operation was canceled.
---> System.IO.IOException: Unable to read data from the transport connection: The I/O operation has been aborted because of either a thread exit or an application request..
---> System.Net.Sockets.SocketException (995): The I/O operation has been aborted because of either a thread exit or an application request.
--- End of inner exception stack trace ---
Expected behavior
Updating MaxRetries from 3 to 6 in the code below shouldn't cause any delay UNLESS the key vault is unreachable.
return new DefaultAzureCredential(
new DefaultAzureCredentialOptions
{
// Prevent deployed instances from trying things that don't work and generally take too long
ExcludeInteractiveBrowserCredential = isDeployed,
ExcludeVisualStudioCodeCredential = isDeployed,
ExcludeVisualStudioCredential = isDeployed,
ExcludeSharedTokenCacheCredential = isDeployed,
ExcludeAzureCliCredential = isDeployed,
ExcludeManagedIdentityCredential = false,
Retry =
{
MaxRetries = 6,
NetworkTimeout = TimeSpan.FromSeconds(10),
MaxDelay = TimeSpan.FromSeconds(10)
},
...
Actual behavior
Our app startup time went from 36 to 54 seconds increasing MaxRetries from 3 to 6.   For 9 retries startup time was > 100 seconds.
Reproduction Steps
Adjusted the MaxRetries as described above and observed startup timing.
Environment
Currently running locally on Win 10 laptop with 32GB memory, VS 2022 (64bit) v 17.2.2
.NET SDK (reflecting any global.json):
Version:   6.0.300
Commit:    8473146e7d
Runtime Environment:
OS Name:     Windows
OS Version:  10.0.19044
OS Platform: Windows
RID:         win10-x64
Base Path:   C:\Program Files\dotnet\sdk\6.0.300\
global.json file:
Not found
Host:
Version:      6.0.9
Architecture: x64
Commit:       163a63591c
.NET SDKs installed:
2.2.110 [C:\Program Files\dotnet\sdk]
3.1.301 [C:\Program Files\dotnet\sdk]
5.0.407 [C:\Program Files\dotnet\sdk]
6.0.300 [C:\Program Files\dotnet\sdk]
.NET runtimes installed:
Microsoft.AspNetCore.All 2.1.30 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.All 2.2.8 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
Microsoft.AspNetCore.App 2.1.30 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 2.2.8 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.1.24 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 3.1.25 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 5.0.16 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 5.0.17 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.AspNetCore.App 6.0.5 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
Microsoft.NETCore.App 2.1.30 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 2.2.8 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 3.1.24 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 3.1.25 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 5.0.16 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 5.0.17 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 6.0.5 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.NETCore.App 6.0.9 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
Microsoft.WindowsDesktop.App 3.1.24 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 3.1.25 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 5.0.16 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 5.0.17 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]
Microsoft.WindowsDesktop.App 6.0.5 [C:\Program Files\dotnet\shared\Microsoft.WindowsDesktop.App]

Thank you for your feedback.  Tagging and routing to the team member best able to assist.

Because of the US holiday season, please expect delayed responses.  There are two important things that I notice about your snippet and the error message:

The timeout that you're seeing is from DefaultAzureCredential attempting to talk to the managed identity endpoint.

Azure.Identity.AuthenticationFailedException: ManagedIdentityCredential authentication failed

The DefaultAzureCredential is a chained token source, meaning that it will flow through multiple credential types.  If you are not intending to use managed identity for authorization, then I'd suggest disabling it by setting the ExcludeManagedIdentityCredential  option.
If you are to use managed credential authorization, I'd suggest referencing the troubleshooting guide provided in the error message to help investigate.
-Your snippet overrides the default 100 second network timeout and allows only 10 seconds (10% of the recommended value).    Your stack trace consistently indicates that your requests to the mangaged identity endpoint are timing out and being cancelled as a result:
 The operation was cancelled because it exceeded the configured timeout of 0:00:05. Network timeout can be adjusted in ClientOptions.Retry.NetworkTimeout.

If you are attempting to use managed identity authorization, this would seem to indicate that you are not giving the credential enough time to authenticate with your host's MI endpoint.

Hi @jsquire,  We do require the ManagedIdentityCredential so that's not an option, but I guess my main concern is WHY would the retry count/timeout modification make so much difference when running locally (and managed identity is perfectly accessible).  It's not failing, but it seems the sheer act of modifying the MaxRetries make such a large difference during startup?   I did remove our modifications - will let them default for now and see if the prevents the (very) intermittent server crashes, but would love to get an explanation on how increasing MaxRetries makes such an impact if possible.

Can you help me understand your statement that "managed identity is perfectly accessible?"   The error message and stack trace clearly indicate that requests to the managed identity endpoint are timing out:
 The operation was cancelled because it exceeded the configured timeout of 0:00:05. Network timeout can be adjusted in ClientOptions.Retry.NetworkTimeout.

As mentioned, you're overriding the default and allowing about 10% of the recommended value.   It appears that the MI endpoint is intermittently not responding quickly enough to meet your timeout value.   Because this is considered a transient exception, it will be retried.   Because this is retried, adding more retries will cause more requests and, thus, take more time.    However, because these requests still do not have sufficient time to interact with your MI endpoint, they'll continue to fail.
I'd suggest removing your override to the network timeout and testing with the default value of 100 seconds to see if that alleviates the issue.


user docs: Create separate "Code blocks" user documentation page.

Code block ocumentation from "Format your messages" page and "Add a custom playground" page
is combined under "Code blocks".
Note that this commit changes how we refer to "```python" type syntax for code blocks. Instead of being called a "syntax highlighting label", this is now referred to as a "language tag", since it serves both syntax highlighting and playgrounds.

#16432
Testing plan:
Tested links on the "Code blocks" page, from docs left side menu and in Organization settings>Code playgrounds menu. Other links were spot-checked, but not systematically tested.
GIFs or screenshots:

This is great @alya!  I made the following tweaks to the main commit:
diff --git a/templates/zerver/for/open-source.md b/templates/zerver/for/open-source.md
index 5e714f2f7a..f895191f2f 100644
--- a/templates/zerver/for/open-source.md
+++ b/templates/zerver/for/open-source.md
@@ -138,7 +138,7 @@ it to an [external code playground](/help/code-blocks#code-playgrounds) to
 interactively run and debug the code.

 If your community primarily uses a single programming language,
-consider setting a default language for syntax highlighting.
+consider setting a [default code block language](/help/code-blocks#default-code-block-language).

 ### Permalink to conversations.

diff --git a/templates/zerver/help/code-blocks.md b/templates/zerver/help/code-blocks.md
index 66cae75534..1edf8c1f35 100644
--- a/templates/zerver/help/code-blocks.md
+++ b/templates/zerver/help/code-blocks.md
@@ -1,6 +1,7 @@
 # Code blocks

-You can use markdown syntax to format code blocks in your Zulip messages.
+Zulip supports the standard [Markdown syntax][markdown] for code
+blocks.

 ~~~
 Inline: `let x = 5`
@@ -19,22 +20,26 @@ def fib(n):

+
![Markdown code](/static/images/help/markdown-code.png)

You can also use `~~~` to start codeblocks, or just indent the code 4 or more
spaces.

+A widget in the upper-right corner of code blocks allows you to easily
+copy the code to your clipboard.

## Language tagging

-Tagging a code block with a language endables syntax highlighting and (if set
-up) the code playgrounds feature. Zulip supports syntax highlighting for
-hundreds of languages.
+Tagging a code block with a language enables syntax highlighting and
+(if configured) [code playgrounds](#code-playgrounds). Zulip supports syntax highlighting
+for hundreds of languages.

-A code block can be tagged by typing the language name after the ` ``` ` that
-begins a code block.  A typeahead will pop up when you start typing after the `
-``` `. If you can't find your language, search for it [here][pygments-lexers]
-and try the **short names** listed for the lexers for your language.
+A code block can be tagged by typing the language name after the fence
+(` ``` `) that begins a code block, as shown here.  Typeahead will
+help you enter the name for the language.  The **Short names** values
+from the [Pygments lexer documentation][pygments-lexers] are the
+complete set of values that support syntax highlighting.

@@ -42,7 +47,7 @@ print("Hello world!")

-### Default language
+### Default code block language

Organization administrators can also configure a default language for code
blocks, which will be used whenever the code block has no tag.
@@ -59,7 +64,7 @@ blocks, which will be used whenever the code block has no tag.

When a default language is configured, one can use ````text` to display code
-blocks without any syntax highlighting.
+blocks without any syntax highlighting (E.g. to paste an error message).

## Code playgrounds

@@ -119,6 +124,15 @@ If you have any trouble setting in setting up a code playground, please [contact
us](/help/contact-support) with details on what you're trying to do, and we'll
be happy to help you out.

+## Related articles
+
+[Math blocks][math-block], [spoiler blocks][spoiler-block], and [quote
+blocks][quote-block] use similar fenced block syntax.
+
[pygments-lexers]: https://pygments.org/docs/lexers/
[get_lexer_by_name]: https://pygments-doc.readthedocs.io/en/latest/lexers/lexers.html#pygments.lexers.get_lexer_by_name

+[markdown]: /help/format-your-message-using-markdown
+[math-block]: /help/format-your-message-using-markdown#latex
+[quote-block]: /help/format-your-message-using-markdown#quotes
+[spoiler-block]: /help/format-your-message-using-markdown#spoilers
diff --git a/templates/zerver/help/format-your-message-using-markdown.md b/templates/zerver/help/format-your-message-using-markdown.md
index 57e9a0c027..9cc8db89aa 100644
--- a/templates/zerver/help/format-your-message-using-markdown.md
+++ b/templates/zerver/help/format-your-message-using-markdown.md
@@ -109,7 +109,9 @@ def fib(n):

You can also use `~~~` to start codeblocks, or just indent the code 4 or more spaces.

-Check out the [Code blocks](/help/code-blocks) help page to learn more about code blocks, including [syntax highlighting](/help/code-blocks#language-tagging) and [code playgrounds](/help/code-blocks#code-playgrounds).
+See the main [code blocks article](/help/code-blocks) for details on
+[syntax highlighting](/help/code-blocks#language-tagging), [code
+playgrounds](/help/code-blocks#code-playgrounds), and other features.

## LaTeX

(The changes are essentially all adding links or removing historical complexity -- e.g. )

I also hand-deployed the new page to zulip.com so that CI will be happy.  (One gets CI failures whenever adding a new link from `docs/` to a `/help/` not yet in production).

Finally, while reviewing this I noticed yesterday we added a second duplicate "CODE BLOCKS" sections on /features; I fixed that via https://github.com/zulip/zulip/pull/18367/commits/f3dc9a554895c3de89ab5351925b29698b724484; the result is in this screenshot:

![image](https://user-images.githubusercontent.com/2746074/117221482-359ff900-adbe-11eb-84a7-bd93368b2dea.png)

Thanks @alya!


feat(origin-ui): using in-browser private key

Add option to import private key and store it in browser encrypted with a password
Provide option to unlock previously saved private key and use it
Allow browsing site without MetaMask by fallback to default web3 RPC
Start using hooks (which simplify whole component structure, for example - no need to declaring types as much as it was before)
Add settings icon next to user indicator to navigate to Account settings
Make demand creation test end-to-end, so it tests also blockchain layer
Use WebSocket RPC ws://localhost:8545 in tests, because http one fails for some reason
Update JSdom in tests from v11 to v15
Update material UI, closes https://github.com/energywebfoundation/origin/pull/122

Closing in favor of https://github.com/energywebfoundation/origin/pull/136


Client sketch scene

[x] Camera stuff, Making sure pan syncs up with engine camera, sketch needs to be moveable
[x] More camera stuff, hide engine scene when "camera" is moving when in sketch mode (since they need a little time to sync up)
[x] get draft straight segments working
[x] Get selections working again

[x] highlight code on segment hover
[x] move cursor on segment click
[x] highlight segment on cursor move

[x] tangential arcs

[x] Draws the arcs
[x] animates the arcs when sketch handles are dragged
[x] Draws dashed arc for adding new tangential arcs

[x] Polish on 2d lines, have consistent screen space size (don't disappear when zooming out)
[x] close profile logic needs to be added back in
[x] pixel grace (getting you mouse close to a segment highlights it etc without needing to have your mouse in the exact right place)
[x] Add sketch selectable X-Y axis for adding constraints
[x] clean up
[ ] (Optional before merge) sketch on face

closes https://github.com/KittyCAD/modeling-app/issues/569

Carge tests fail locally too, but they are intermittent, the tests that fail change each time I run it. Need to look into what's happening.

Going to merge, we can follow up with whatever we need. and while it's no hard blocking, is soft-blocking other work like https://github.com/KittyCAD/modeling-app/pull/1362 since there a big possibility of conflicts since this is such a large PR.

follow up: https://github.com/KittyCAD/modeling-app/pull/1398


Add ability to reuse same component multiple times in the same view

Solution for #25

All should be fixed


add explicit checks in OnChanUpgradeTry callback of ICA host

Description

ref: https://github.com/cosmos/ibc-go/issues/5462#issuecomment-1866851254
closes: #XXXX
Commit Message / Changelog Entry
type: commit message

see the guidelines for commit messages. (view raw markdown for examples)

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against the correct branch (see CONTRIBUTING.md).
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Code follows the module structure standards and Go style guide.
[ ] Wrote unit and integration tests.
[ ] Updated relevant documentation (docs/) or specification (x/<module>/spec/).
[ ] Added relevant godoc comments.
[ ] Provide a commit message to be used for the changelog entry in the PR description for review.
[ ] Re-reviewed Files changed in the Github PR explorer.
[ ] Review Codecov Report in the comment section below once CI passes.

Thank you very much for the reviews everyone, and 💚 🙇 🙏 to @charleenfei for addressing the review comments!


Failed to load elf.

I'm trying to use this lib to load a hello world of ebpf, but I got the following errors:
panic: invalid relocation: insn code=0x85, symbol name=bpf_trace_printk
symbol section: Name=, Type=SHT_NULL, Flags=0x0

goroutine 1 [running]:
main.main()
        /home/ubuntu/go/src/github.com/k82cn/ebpf-learn/hello/main.go:15 +0x285

The source code is: https://github.com/k82cn/ebpf-learn/tree/main/hello
If any suggestion/comments, please let me know :)

Hi, you may miss out this compile warning:
hello.c:9:5: warning: implicit declaration of function 'bpf_trace_printk' is invalid in C99 [-Wimplicit-function-declaration]
    bpf_trace_printk(msg, sizeof(msg));
    ^
1 warning generated.

You can obtain a copy of bpf_helpers.h from here, include this file and your hello sample works.

Hi, you may miss out this compile warning:
hello.c:9:5: warning: implicit declaration of function 'bpf_trace_printk' is invalid in C99 [-Wimplicit-function-declaration]
    bpf_trace_printk(msg, sizeof(msg));
    ^
1 warning generated.

You can obtain a copy of bpf_helpers.h from here, include this file and your hello sample works.


Only deploy HSMs in weekly tests

Description
Using the logic from archetype-sdk-tests.yml to apply behavior to weekly test pipelines only: https://github.com/Azure/azure-sdk-for-python/blob/aec6d92587c63e76468527246013f0728b6a5f32/eng/pipelines/templates/stages/archetype-sdk-tests.yml#L149
This PR changes Key Vault's live test pipelines to no longer deploy Managed HSMs for nightly tests; instead, they're only used in weekly test pipelines (and only for Public cloud, since the resource is unavailable in USGov and China). This is to save on resource costs and reduce the number of false flags from failed HSM deployments in nightly runs.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

Validating pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=2704024&view=results

Validating pipeline run for nightly tests: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=2707390&view=results

@benbp what do you think about doing it this way instead of what you suggested with the matrix?
Also, @mccoyp please hold off on merging this until we're given the go-ahead to reduce testing.

@heaths this is way better than what I suggested, let's do this.


Link to Chart.java

I maintain a Java library that produces valid json to use with Chart.js.
As an hommage to Chart.js, it's named Chart.java: https://github.com/mdewilde/chart
Do you have a policy regarding which projects you link to from the integrations section of your documentation?
In short, it'd be great to see Chart.java linked from there.

@mdewilde looks great, can you submit a PR that adds a Java with a link to Chart.java repository (or website)?

Done in #4527


headerView was not rendering when refreshable is false

can you include that in 0.0.7

Hello Farid, can you also bump the version

as I need to merge my branch to master, I have already used 0.0.7:
https://github.com/iSimar/HackerNews-React-Native/commit/ef773595cc1f8adcfb9ac7a88b7458134b1e7cca

Hello Simar, sure

I released 0.0.7

Thank you

FYI, I plan to add support for PullToRefreshViewAndroid in the next version

oh okay, I was actually going to implement that with react-native-gifted-listview, because I need it for the hacker news android app.
https://github.com/iSimar/HackerNews-React-Native

I will have some availability to work on it tomorrow

okay, looking forward to it. Thank you again, great work.

Hello Simar, I published a new version with PullToRefreshViewAndroid support


[#16] Rename Command contract to Action

Closes #16

LGTM :+1:

@wilderrodrigues how about merging?


Healthchecks and port mappings for docker-compose

For https://github.com/lbryio/lbry-docker/issues/59

The proposed start script shouldn't change any behavior.  Variables have a
default value that is overridden by environment variables defined in docker.
On Wed, Oct 9, 2019, 10:16 PM Leopere<EMAIL_ADDRESS>wrote:

@Leopere commented on this pull request.
This looks as though it will lose a lot of the features, however your
comment in #59 https://github.com/lbryio/lbry-docker/issues/59 is right
it will improve readability significantly.
The idea is that the configuration uses one replacement mechanism to find
the source configuration string and replaces it but this said I have not
kept up to date on the default state of the Chainquery configuration in a
while so this may be the best method to take at this point.
The goal with my previous start.sh script configuration was to allow
people to lean on the docker container via environment variables as that
seems to be the most multi container platform compatible solution.
All this said upon my review of my own goals here I'm seeing a few flaws
in what I decided on using here as well so as long as we don't lose too
much in this PR I will gladly merge it.
Thank you for your contributions!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/lbryio/lbry-docker/pull/87?email_source=notifications&email_token=AAALCOHHVI5BVOYRRRPG57LQN2F7HA5CNFSM4I7GX2L2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCHPGN6Q#pullrequestreview-299788026,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAALCOEHUIGIOXL55AVCLW3QN2F7HANCNFSM4I7GX2LQ
.

@dangarthwaite thanks for the PR!
Can we show you some appreciation for the contribution?
Also, we're giving away some Hacktoberfest bonuses and goodies for this month. We'll get you the form required once you send us an email.


Remove preview warning from spo hubsite commands

Currently quite some spo hubsite and spo site hubsite commands have a warning that the API is in preview.
However, when consulting the MS docs, there is no warning anymore so we can assume that the API is stable right now.
Example: https://learn.microsoft.com/en-us/sharepoint/dev/features/hub-site/rest-joinhubsite-method
To do
Remove the This command is based on a SharePoint API that is currently in preview and is subject to change once the API reached general availability.  warning from the docs.
Command docs:

[ ] spo site hubsite connect
[ ] spo site hubsite disconnect
[ ] spo site hubsite theme sync
[ ] spo hubsite data get
[ ] spo hubsite rights grant
[ ] spo hubsite rights revoke
[ ] spo hubsite set

Apart from that, some command docs have an important admonition in their docs saying To use this command you have to have permissions to access the tenant admin site.
This description is quite vague in my opinion. Let's change this to To use this command you must be a Global or SharePoint administrator.
Let's:

Rename the message in the important admonition so the exact roles are displayed. (To use this command you must be a Global or SharePoint administrator.)
Make sure that this important admonition is under the Remarks section and not the Options section.

Command docs to update:

[ ] spo hubsite rights grant
[ ] spo hubsite rights revoke
[ ] spo hubsite set

@milanholemans I can work on this one.

Awesome, thanks @Abderahman88, all yours!


new upload button rendering

I noticed 2 issues:

the upload button doesn't change at all on mouse-hover to provide the "clickable" affordance
on desktop, the window always thinks it's slightly too small:

#51 should address parts of this, as the hover effects are back, but not full ripple effects just yet. The screenshot also shows that (a) the right fonts aren't being loaded, and (b) there's spacing around the top bar as well. That's suggesting to me that the stylesheet is not being properly loaded.
I cannot reproduce at least (b) when navigating to https://llaminator-dev.netlify.app/ directly.

The button is also not sticky. it scroll with the page when the image is large

I cannot reproduce at least (b) when navigating to llaminator-dev.netlify.app directly.

I can't anymore either - might have been fixed from one of your earlier UI PRs.


FilesCommunity.Files version 2.4.33

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Dependency support is not yet fully implemented.
@denelon - Not sure how this passed in #94170

[Policy] Blocking-Issue

[Policy] Dependencies

/AzurePipelines run

Hey @afermar, we're now doing validation with dependencies if you want to add them to the manifest :)


Misspelled socket_io_transports in Dockerfile

Line 13 in Dockerfile is ENV        CRONICLE_socker_io_transports '["polling", "websocket"]' where as it should be ENV        CRONICLE_socket_io_transports '["polling", "websocket"]'. socket is spelled wrong.

Thanks! Good catch. has been resolved


JdbcRegisteredClientRepository should not write JDK classes to the database

Expected Behavior

JdbcRegisteredClientRepository should not write class names into the database. There should be an out of the box way to save the values without including the class names.
It is not ideal to leak the domain model into the database unless necessary (eg you do not know the types that are being persisted and need to support generic serialization/deserialization). This is the equivalent of using blobs for objects and only slightly better than using ObjectMapper to write RegisteredClient into a single column.
Current Behavior

Currently JdbcRegisteredClientRepository writeMap uses ObjectMapper which serializes the map as a JSON string and includes  the JDK classes that are used. For example, token settings looks like:
{"@class":"java.util.Collections$UnmodifiableMap","settings.token.reuse-refresh-tokens":true,"settings.token.id-token-signature-algorithm":["org.springframework.security.oauth2.jose.jws.SignatureAlgorithm","RS256"],"settings.token.access-token-time-to-live":["java.time.Duration",300.000000000],"settings.token.access-token-format":{"@class":"org.springframework.security.oauth2.server.authorization.settings.OAuth2TokenFormat","value":"self-contained"},"settings.token.refresh-token-time-to-live":["java.time.Duration",3600.000000000],"settings.token.authorization-code-time-to-live":["java.time.Duration",300.000000000],"settings.token.device-code-time-to-live":["java.time.Duration",300.000000000]}

Context

There is a lot of extra data written to the database and it leaks the object model into the persistence layer.

@rwinch TokenSettings and ClientSettings don't have associated Mixin's, hence the reason for the class names being serialized.
The intent behind JdbcRegisteredClientRepository is to provide a simplified JDBC implementation that can be customized however users want. The registeredClientParametersMapper and registeredClientRowMapper provide the hook for customization. Furthermore, the following test demonstrates how users can alter the db schema if they prefer not to go with the flattened approach:
https://github.com/spring-projects/spring-authorization-server/blob/6eeca4942849681d763facbcaa673bcc1796790b/oauth2-authorization-server/src/test/java/org/springframework/security/oauth2/server/authorization/client/JdbcRegisteredClientRepositoryTests.java#L262
Again, this is meant to be a simplified JDBC implementation that can be customized however users wish. However, there is a high likelihood that production applications will provide their own implementation of RegisteredClientRepository that persists using another approach other than JDBC and/or relational store.
I'm going to close this as the current implementation works and can be customized if users choose to do so.


Configure Infrastructure logger

What kind of change does this PR introduce?
Would be possible to configure infrastructure logger. For example overrides colors for some specific plugin.
Related to #11572

Did you add tests for your changes?
Yes.

Does this PR introduce a breaking change?

Yes.  If someone overrides the webpack console impl via options.infrastructureLogging, it changed from console to getConsole getter.
What needs to be documented once your changes are merged?

https://webpack.js.org/api/logging/

Let's rethink this approach, closing for now.


Couldn't get any scraped content for this URL

Hi Guys,
I wanted to use AngleSharp to scrape the page text content of some pages.
It worked for most of the pages.
But I found these 2 URLs, those not return any text content.
https://crawlq.ai
https://crawlq.ai/pricing
Please have a look..
Let me know if found the reason..
Thanks...

Sorry, we cannot handle such individual responses as the time is limited. Potentially, these pages are SPAs and use JS to render their content.
Hope that helps!


🐛 [Firebase Remote Config] Support for Desktop

Why is remote config not supported for desktop yet?
And is it going to be supported in near future ever?
Are there any plans for it?

The desktop support seems to be in beta per this table : https://firebase.google.com/docs/flutter/setup?platform=ios , so hopefully it'll reach stable in future releases. I suggest you to keep an eye on the table for reference.
Also, moving this issue to discussion.


Page indicator doesn't show on iOS

Hi @alexrainman we have again same issue with page indicator on iOS as was #39 . After update to version 3.0.0 page indicator doesn't show.
Page containing the carousel:
<AbsoluteLayout BackgroundColor="Yellow"
	                HorizontalOptions="Fill"
	                VerticalOptions="Fill">
		<StackLayout AbsoluteLayout.LayoutBounds="0,0,1,1"
		             AbsoluteLayout.LayoutFlags="All"
		             BackgroundColor="Blue"
		             HorizontalOptions="FillAndExpand"
		             VerticalOptions="FillAndExpand">
			<controls:CarouselViewControl x:Name="IntroCarousel"
			                              CurrentPageIndicatorTintColor="#808080"
			                              HorizontalOptions="FillAndExpand"
			                              InterPageSpacing="0"
			                              InterPageSpacingColor="{StaticResource WindowBackground}"
			                              ItemTemplate="{helpers:IntroTemplateSelector}"
			                              ItemsSource="{Binding Pages}"
			                              PageIndicatorTintColor="#C0C0C0"
			                              Position="0"
			                              ShowIndicators="True"
			                              VerticalOptions="FillAndExpand" />
		</StackLayout>
		<StackLayout AbsoluteLayout.LayoutBounds="1,1,0.9,1"
		             AbsoluteLayout.LayoutFlags="All"
		             BackgroundColor="Red"
		             Orientation="Horizontal"
		             VerticalOptions="End">
			<Button x:Name="prevBtn"
			        IsVisible="false"
			        Style="{StaticResource IntroPreviousBtnStyle}"
			        Text="Previous" />
			<Button x:Name="nextBtn"
			        Style="{StaticResource IntroNextBtnStyle}"
			        Text="Next" />
		</StackLayout>

The issue is not the Carousel control, it is your XAML.
Here's you have it with some fixes:
    <AbsoluteLayout HorizontalOptions="Fill"
                    VerticalOptions="Fill"
                    BackgroundColor="Yellow">
        <controls:CarouselViewControl x:Name="IntroCarousel"
                                      HorizontalOptions="FillAndExpand"
                                      VerticalOptions="FillAndExpand"
                                      AbsoluteLayout.LayoutBounds="0,0,1,0.9"
                                      AbsoluteLayout.LayoutFlags="All"
                                      CurrentPageIndicatorTintColor="#808080"
                                      InterPageSpacing="0"
                                      InterPageSpacingColor="{StaticResource WindowBackground}"
                                      ItemTemplate="{helpers:IntroTemplateSelector}"
                                      ItemsSource="{Binding Pages}"
                                      PageIndicatorTintColor="#C0C0C0"
                                      Position="0"
                                      ShowIndicators="True" />
        <StackLayout VerticalOptions="End"
                     AbsoluteLayout.LayoutBounds="1,1,1,0.1"
                     AbsoluteLayout.LayoutFlags="All"
                     BackgroundColor="Red"
                     Orientation="Horizontal">
            <Button x:Name="prevBtn"
                    IsVisible="false"
                    Style="{StaticResource IntroPreviousBtnStyle}"
                    Text="Previous" />
            <Button x:Name="nextBtn"
                    Style="{StaticResource IntroNextBtnStyle}"
                    Text="Next" />
        </StackLayout>
    </AbsoluteLayout>

If my work is helping you, please help me back: https://xamarinhq.wufoo.com/forms/nominate-a-xamarin-mvp/
This is what i have done that is community visible:

Alex has been pretty active sharing his knowledge and answering questions in StackOverflow, GitHub and Xamarin Forums.
Alex has been open sourcing Xamarin components and plugin with the community, with more of 10 repositories being maintained on GitHub.
One of Alex's Xamarin.Forms plugin, the CarouselView, is currently being pulled into the Xamarin.Forms core (https://github.com/xamarin/Xamarin.Forms/pull/853). As an stable widget, this control has become the standard for Xamarin.Forms carousel screens with more that 16K downloads in NuGet and more than 100 solved and not open issues.
Another Alex's plugin, the SegmentedControl (https://github.com/alexrainman/SegmentedControl), is a big candidate to be also pulled into Xamarin.Forms core.
Plus Xamarin.Forms UI related plugins, Alex shares plugins to set app icon Badge, Google Analytics and KeyChain access.
Alex shares his knowledge about cutting edge technologies like Couchbase NoSQL and MobileIron integration with Xamarin Apps.

You'll get my vote for sure.


Pleas add a dark theme option

I suppose that would be an easy task if you know how the CSS classes are organized.
But I have tried to change the background to a bg-near-black color and it is not so easy.
The background changes but the letter colors are quite dark to be read in many places.
I have tried to change the forground colors like this:
[params]
background_color_class = "bg-black"
body_classes = "bg-near-black"
post_content_classes = "moon-gray"
It works for post pages but there are many other places where letters remain to black.
So having an option to select different skins or color combinations (a dark theme in particular) would be great.
If that is too complicated to be implemented, meanwhile it would be great if you add in the readme a [params] section to get it to work changing the appropiate css classes.

It would be great. I don't use images and the white is glaring for me.

Closing for #258


added xhr.withCredentials=true to allow proper handling of credentials…

… if cors is used

@geraintluff can you take a look and merge this?
By the way jsonary.com is still not up and running....


make formatting functions public

If we can promote functions in formatter.go to be public, we can allow users to manage their own connectivity.

done.


[BUG] {{ SmartSearch }} - {{ Cannot enter search in a range }}

Try to fill item input search with filters of a range, greater than some value but smaller than another:
For item search:

createdAt < 05/12/2023 AND createdAt > 01/12/2023
metadata.system.size < 10 AND metadata.system.size > 1

For Annotation search:

confidence > 0.1 AND confidence < 0.9

The component will sanitize the input and wont allow the search

Closing as #755 is merged


Use window/document relative to DOM element

Use the ownerDocument and window of the DOM element being manipulated
rather than the global window and document objects. This makes
react-draggable work with a tool like
https://github.com/ryanseddon/react-frame-component
This replaces #177 which still needs some tests.

Beautiful! This is exciting.


Fix for issue #7 : add blank line after last HEADER in MSearch reques…

…t and enclose MAN header value in double quotes

Should not do pull requests from my master branch, sorry


Add creator field to excel export

Enhancement
Add creator field to excel export in backoffice, similar to UGent author / UGent editor.

[ ] Full name
[ ] UGent ID(s?) tbd if multiple IDs possible/needed
[ ] Department

Context
Needed for business analysis in Biblio team.

After Mini Biblio


merge commits prior to metadatcaching

After this PR is merged, we will need to do cherry-pick into master on commits from dev branch that are merged after revert-metadataCaching.

Codecov Report

Merging #369 into master will decrease coverage by 2.63%.
The diff coverage is 0%.

@@             Coverage Diff             @@
##             master    #369      +/-   ##
===========================================
- Coverage     40.13%   37.5%   -2.64%
+ Complexity     1883    1707     -176
===========================================
  Files           107     103       -4
  Lines         24486   23673     -813
  Branches       4039    3882     -157
===========================================
- Hits           9827    8878     -949
- Misses        12809   13241     +432
+ Partials       1850    1554     -296

Flag
Coverage Δ
Complexity Δ

#JDBC41
37.35% <0%> (-2.66%)
1700 <0> (-179)

#JDBC42
37.41% <0%> (-2.59%)
1705 <0> (-175)

Impacted Files
Coverage Δ
Complexity Δ

...com/microsoft/sqlserver/jdbc/dns/DNSUtilities.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...t/sqlserver/jdbc/SQLServerEncryptionAlgorithm.java
0% <0%> (-100%)
0% <0%> (-1%)

...dbc/SQLServerColumnEncryptionKeyStoreProvider.java
0% <0%> (-100%)
0% <0%> (-1%)

...rver/jdbc/SQLServerEncryptionAlgorithmFactory.java
0% <0%> (-100%)
0% <0%> (-1%)

src/main/java/com/microsoft/sqlserver/jdbc/AE.java
0% <0%> (-71.57%)
0% <0%> (ø)

...bc/SQLServerAeadAes256CbcHmac256EncryptionKey.java
0% <0%> (-71.43%)
0% <0%> (-4%)

...ver/jdbc/SQLServerAeadAes256CbcHmac256Factory.java
0% <0%> (-70%)
0% <0%> (-4%)

...r/jdbc/SQLServerAeadAes256CbcHmac256Algorithm.java
0% <0%> (-68.51%)
0% <0%> (-10%)

...rosoft/sqlserver/jdbc/SQLServerEncryptionType.java
0% <0%> (-66.67%)
0% <0%> (-5%)

...crosoft/sqlserver/jdbc/KeyStoreProviderCommon.java
0% <0%> (-63.02%)
0% <0%> (-9%)

... and 47 more

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 0b64232...f4b87c3. Read the comment docs.


Adding support for JDK17 and removing JDK8

Description
Making necessary adjustments to make build compatible with JDK17:

add add-exports arguments due to java.lang.IllegalAccessError when spotlessJavaCheck uses classes from com.sun.*
remove bad javadoc formatting as this generates warnings and build is running with -Werror option

Changing runtime java version from 8 to 11.
Issues Resolved
Remove support for Java 8, add support for Java 17
Check List

[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Gradle check is failing testWithKeywordLongAndMissingBucket test which seems flaky. Its worth retyring Gradle check again.
REPRODUCE WITH: ./gradlew ':server:test' --tests "org.opensearch.search.aggregations.bucket.composite.CompositeAggregatorTests.testWithKeywordLongAndMissingBucket" -Dtests.seed=4E1342EC6B964D83 -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1 -XX:ReservedCodeCacheSize=64m" -Dtests.locale=zh-Hans-CN -Dtests.timezone=Europe/Isle_of_Man -Druntime.java=17

org.opensearch.search.aggregations.bucket.composite.CompositeAggregatorTests > testWithKeywordLongAndMissingBucket FAILED
    org.junit.ComparisonFailure: expected:<{keyword=null, long=[null]}> but was:<{keyword=null, long=[100]}>
        at __randomizedtesting.SeedInfo.seed([4E1342EC6B964D83:AF2CA28B0C4823CC]:0)
        at org.junit.Assert.assertEquals(Assert.java:117)
        at org.junit.Assert.assertEquals(Assert.java:146)
        at org.opensearch.search.aggregations.bucket.composite.CompositeAggregatorTests.lambda$testWithKeywordLongAndMissingBucket$85(CompositeAggregatorTests.java:1057)
        at org.opensearch.search.aggregations.bucket.composite.CompositeAggregatorTests.executeTestCase(CompositeAggregatorTests.java:2712)
        at org.opensearch.search.aggregations.bucket.composite.CompositeAggregatorTests.testSearchCase(CompositeAggregatorTests.java:2660)
        at org.opensearch.search.aggregations.bucket.composite.CompositeAggregatorTests.testWithKeywordLongAndMissingBucket(CompositeAggregatorTests.java:1045)

Gradle check is failing testWithKeywordLongAndMissingBucket test which seems flaky. Its worth retyring Gradle check again.
REPRODUCE WITH: ./gradlew ':server:test' --tests "org.opensearch.search.aggregations.bucket.composite.CompositeAggregatorTests.testWithKeywordLongAndMissingBucket" -Dtests.seed=4E1342EC6B964D83 -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1 -XX:ReservedCodeCacheSize=64m" -Dtests.locale=zh-Hans-CN -Dtests.timezone=Europe/Isle_of_Man -Druntime.java=17

org.opensearch.search.aggregations.bucket.composite.CompositeAggregatorTests > testWithKeywordLongAndMissingBucket FAILED
    org.junit.ComparisonFailure: expected:<{keyword=null, long=[null]}> but was:<{keyword=null, long=[100]}>
        at __randomizedtesting.SeedInfo.seed([4E1342EC6B964D83:AF2CA28B0C4823CC]:0)
        at org.junit.Assert.assertEquals(Assert.java:117)
        at org.junit.Assert.assertEquals(Assert.java:146)
        at org.opensearch.search.aggregations.bucket.composite.CompositeAggregatorTests.lambda$testWithKeywordLongAndMissingBucket$85(CompositeAggregatorTests.java:1057)
        at org.opensearch.search.aggregations.bucket.composite.CompositeAggregatorTests.executeTestCase(CompositeAggregatorTests.java:2712)
        at org.opensearch.search.aggregations.bucket.composite.CompositeAggregatorTests.testSearchCase(CompositeAggregatorTests.java:2660)
        at org.opensearch.search.aggregations.bucket.composite.CompositeAggregatorTests.testWithKeywordLongAndMissingBucket(CompositeAggregatorTests.java:1045)

hey Suraj,
How can I restart the check (Gradle check that you've mentioned)? Checks that are listed in PR's check tab are all green.
^ @dreamer-89

start gradle check

x   Gradle Check failure 988b457 Log 2150
Reports 2150

Hehe ... @owaiskazi19 seems like without our https://github.com/opensearch-project/OpenSearch/pull/2024 PR merged, not much hope for builds to succeed :(
* What went wrong:
Execution failed for task ':distribution:docker:buildDockerImage'.
> A failure occurred while executing org.opensearch.gradle.docker.DockerBuildTask$DockerBuildAction
   > Process 'command 'docker'' finished with non-zero exit value 1

Yeah. Looks like all the build failed for the same reason.

x   Gradle Check failure 988b457 Log 2150
Reports 2150

Hehe ... @owaiskazi19 seems like without our #2024 PR merged, not much hope for builds to succeed :(
* What went wrong:
Execution failed for task ':distribution:docker:buildDockerImage'.
> A failure occurred while executing org.opensearch.gradle.docker.DockerBuildTask$DockerBuildAction
   > Process 'command 'docker'' finished with non-zero exit value 1

Yeah. Looks like all the build failed for the same reason.

start gradle check

Tests with failures:
 - org.opensearch.cluster.routing.MovePrimaryFirstTests.testClusterGreenAfterPartialRelocation

https://github.com/opensearch-project/OpenSearch/issues/1957

This overlaps with https://github.com/opensearch-project/OpenSearch/pull/1977.
@martin-gaievski want to merge these two and update docs?

This overlaps with #1977.
@martin-gaievski want to merge these two and update docs?

sure, added changes from #1977 so now this PR consolidates all changes
^ @dblock

The developer guide needs to be updated as well.

Ack, updated in next commit


Allow configuring subpixel rendering (per monitor)

Description
I admit I don't 100% understand what this requires and if this is actually fully (separately for both displays) implementable using current Wayland; but Wayland seems to have information about the subpixel format of displays, as displayed by, for example, wayland-info. For my two monitors in Hyprland, it says under "interface: 'wl_output'":
subpixel_orientation: unknown, output_transform: normal (or 270° for the other).
This results in suboptimal grayscale antialiasing for fonts.
Use case 1 (simpler)
I'd want to set the same subpixel rendering for all displays. I believe this is at least something supported by Wayland (or maybe I'm wrong and it's a fontconfig setting, but then I don't understand why Wayland knows about subpixel order?)
Use case 2
Two different monitors with different pixel orders; perhaps most commonly one horizontal and one vertical monitor.
I actually tried to hack Hyprland to call wlr_output_state_set_subpixel(state.wrl(), WL_OUTPUT_SUBPIXEL_HORIZONTAL_RGB) in CMonitor::onConnect() to see if that would have an effect (I'm speculating that in Wayland the compositor somehow sends this information to applications). The effect that I do see is wayland-info showing that subpixel orientation; however, all text in all applications still seems to be grayscale antialiased. Documentation seems to indicate that some other compositors (like sway) have settings for this; perhaps I should test what kinds of results I could get with those.

makes sense, maybe we can also query the properties via the edid in aq

I am not familiar with what client apps actually make use of this, but in Sway it's configurable in the settings. Sway doesn't autodetect it though so you'd have to manually set it.
It doesn't seem like this is required to have subpixel rendered text; it works for me even on sway where swaymsg -t get_outputs shows unknown for the subpixel orientation. It looks to me like proper subpixel rendering has to be configured in fontconfig based on ArchWiki?

Yeah, this is a very interesting question (if I even understand it correctly). In fontconfig I think you cannot configure it per display (and this makes sense to me because I don't think fontconfig ought to know about displays). In sway you can. I never questioned whether that actually works in the sense that it gives us different subpixel rendering on different displays. Or, in other words, while it seems wayland can somehow provide this information, is it actually used by anything at all.


Add force_destroy option for S3 bucket

What
Add force_destroy option for S3 bucket
Why
After running destroy the S3 bucket for load balancer logs is not destroyed. This pull request will enable the destroy through force_destroy option.

Could you give feedback please?

Done. Any other changes needed?


How to navigate between pages in Sveltkit programmatically like /home /contactus if any condition is true in home want to navigate to contact page , Any example will help

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. For example: I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
How important is this feature to you?
Note: the more honest and specific you are here the more we will take you seriously.
Additional context
Add any other context or screenshots about the feature request here.

GitHub issues are reserved for feature request or bug reports. This does read like a usage question (which requires more details, btw), therefore I'm going to close this.


Make the toast content grow to fill remaining space

At the moment, the content div only takes up as much space as is needed for the title/description of the toast.
This caused a problem for me when I rendered a progress bar in the toast description. By default, the progress bar was only as wide as the title, so it looked a bit funny. This PR fixes that by making the content grow to fill available space, so that variable-width items like progress bars take up the full content area by default.

I wonder if this will resolve the issue I'm facing here


How to assert a table data was not changed

Our flow is:

before each test delete all data
than insert users in users table
call some methods, test logic
assert data was not changed

DB db = ... ;
Table t = table("USERS")
         .columns("ID", "LOGIN", "NAME", "PASSWORD", "CREATED")
         .build(db);
DataSet users
   = data(t)
    .row(0,   "root",  "Root User",  "god",       null)
    .row(101, "john",  "John Doe",   "justDoeIt", Date.valueOf("2014-07-12"));

It is not clear how to assert a table data was not changed.
Can you show an example?

As I understand we should use
JDBDT.assertUnchanged(accounts.getSource())
If this is correct can you please update documentation?
http://jdbdt.org/DBAssertions.html
Please add/modify
// [1] Populate a table with some data.
Table t = ...;
DataSet data = data(t). ...;
populate(data); // --> 'data' becomes the reference snapshot
// add next line in docs
data.getSource(); // to get the snapshot

I don't understand your doubt.  You only need assertUnchanged(t) in your example, you don't need to get the snapshot back (?).
You may check the JDBDT tutorial for other examples - http://jdbdt.org/Tutorial.html and the corresponding code here on GitHub.

For instance in the JDBDT tutorial code you have the following test:
@Test
public void testNonExistingUserDelete() throws SQLException {
    User u = nonExistingUser();
    boolean deleted = theDAO.deleteUser(u);
    assertUnchanged("No DB changes", theTable);
    assertFalse("return value", deleted);
}


Race condition in sshnpd if starting multiple sshnp clients simultaneously

Describe the bug
_privateKey is an instance variable of SSHNPDImpl. It is set when sshnpd receives a privatekey notification from sshnp, and is later used when sshnpd receives a sshd notiication from sshnp.
If there are two sshnp clients connecting to the same sshnpd at approximately the same time, then it is possible for the following to happen

client 1 sends privatekey
client 2 sends privatekey
client1 sends sshd --> sshnpd will use privatekey sent by client2 when creating the ssh socket

The race condition is rarely likely to occur in real world usage, but should be fixed anyway. However - see "Additional context" section below for why we may not wish to fix this problem. Assigning Type of 'Discussion' so we can discuss on an architecture call.
Steps to reproduce
Haven't yet built a test harness to reproduce, found while doing a code walkthrough
Expected behavior
This race condition should be prevented.
The sshnp client sends its sessionId when it sends the sshd notification. One way of solving the problem would be

also send the client's session ID as part of the privatekey notification in either the key or the value of the notification.
sshnpd could then store the privatekeys it receives in a map keyed by the client's sessionId.
However this would break backwards compatibility, since sshnpd v3.3.0 and earlier expect the value of the notification to be just the private key, and expect the key of the notification to be privatekey.$device.sshnp

Fix proposal

Once we have the ability for sshnp to find out the version of the sshnpd it is connecting to, then we can implement the behaviour above when sshnpd is at a version which supports it. I suggest the best approach is to make the value of the notifications be serialized JSON objects rather than simple Strings - e.g. {"privateKey":"$privateKey","sessionId":"$sessionId"}
sshnpd can then check the value of the privatekey notification and if it is JSON, it can deserialize it and then get the privateKey and the sessionId, or if it is still just a String (from an older sshnp client) then it can just get the privateKey

Additional context
Note that this race condition will not exist when we support direct ssh via rvd rather than a reverse ssh, since sshnpd will only be adding to its authorized_keys file the ephemeral publickey it receives from sshnp

Is this something that was fixed when you wrote fwd ssh @gkc ?

Is this something that was fixed when you wrote fwd ssh @gkc ?

Yes @XavierChanth this is no longer an issue in v4+


Fatal error: Class 'TYPO3\SwiftMailer\Message' not found

This line in the EmailFinisher class does not ensure that the required Swiftmailer class is actually available (and causes a fatal error if you don't have Swiftmailer installed):
$mail = new \TYPO3\SwiftMailer\Message();
I suggest to either include Swiftmailer as dependency in composer.json or to check for installed Swiftmailer and throw an informative exception instead.

Thanks, good point! I guess we should at least suggest the switftmailer package if we don't want to create a hard dependency.

Fixed with #23


implemented 'scoped' flash messages

pull request for functionality within Issue #32

Thank you for your PR. I hope I have more time to check this soon, but before that there's a few things that should be considered also:

Documentation needs to be updated.
Please use ES2015/ES6 functionality as it is much more readable. For example replace the function () {} with () => {} and use const and let instead of var.

Merged @javoire PR in, and seems to be about the same. If not, please open a new PR with the issues mentioned fixed.


Deleting mat-tab-link from links array removes selected tab's focus

Reproduction
Use StackBlitz to reproduce your issue: https://stackblitz.com/edit/angular-cjny6b
Steps to reproduce:

Third tab "TestActiveTab" is currently active.
Click "Delete Index(1)" to delete second tab.
Tab "TestActiveTab" is no longer displayed as active.

Expected Behavior
What behavior were you expecting to see?
When the tab before "TestActiveTab" was deleted, I expected "TestActiveTab" to remain active since activeLink is still set to "TestActiveTab".
Actual Behavior
What behavior did you actually see?
Looking at tab-nav-bar.ts
 @Input()
  get active(): boolean { return this._isActive; }
  set active(value: boolean) {
    if (value !== this._isActive) {
      this._isActive = value;
      this._tabNavBar.updateActiveLink(this.elementRef);
    }
  }

The tabnavbar is only prompted to update when this._isActive !== value. So in our case, we have "TestActiveTab" tab set to active at index(2). Then we delete the second tab. "TestActiveTab" now has an index of 1 but the selected index is still set to 2. Since TestActiveTab's value never changed, the tabNavBar never changes it's selected index and "TestActiveTab" does not appear active, even though it's active property is still set to true.
Environment

Angular: 8.2.4
CDK/Material: 8.1.4
Browser(s): Chrome
Operating System (e.g. Windows, macOS, Ubuntu): Windows

My workaround was to introduce a deleting property in the active boolean.
<nav mat-tab-nav-bar [backgroundColor]="background">
  <a mat-tab-link *ngFor="let link of links"
     (click)="activeLink = link"
     [active]="activeLink == link && !deleting"> {{link}} </a>
</nav>

The deleting property is toggled within a setTimeout, causing "TestActiveTab"'s active to toggle off and on as well. It works but feels wrong.

My workaround as of right now was to view the MatTabNav
@ViewChild(MatTabNav, { static: false }) _tabNavBar: MatTabNav;
and update the tabNavBar manually, with a timeout. But is there a cleaner solution to this issue?
deleteTab(navIndex: number) {
    this.links.splice(navIndex, 1);
    console.log(this.activeLink);
    setTimeout(() => {
      this._tabNavBar.updateActiveLink();
    });
  }

I couldn't reproduce it against master. I believe that https://github.com/angular/components/pull/16624 should've fixed it which should be in the next release. I'm closing the issue, but it can be reopened if the issue is still there after the next release.

This line from your commit seems to fix it. I'll look forward to the next release, thank you!
this._items.changes.pipe(startWith(null), takeUntil(this._destroyed)).subscribe(() => {
      this.updateActiveLink();
    });


Error: DOM representation of control 'Element sap.m.Input#__input2-__clone54-__clone83' has no focus DOM reference

When changing a cell value of a table using iChangeRow() there is an error message when test script is running headless in pipeline. When running locally in browser there is no error.
This is the same Error as in auto-closed issue #545 where you find more details and screenshots:
Issue 545
I was trying both options of iChangeRow() with and without optional parameter bInputNotFinalized
When.onTheDetailPage.onTable({property: "orderLines", qualifier: "Positionen"}).iChangeRow({product_productNr: "1000801", quantity: "3"}, {quantity: "6"}, true)
When.onTheDetailPage.onTable({property: "orderLines", qualifier: "Positionen"}).iChangeRow({product_productNr: "1000801", quantity: "3"}, {quantity: "6"})
Error Message:
"Changing row values of table '{id: fe::table::orderLines::LineItem::Positionen}' with old values='{product_productNr: 1000801, quantity: 3}' to new values='{quantity: 6}' - FAILURE Failure in Opa success function Exception thrown by the testcode:'Error: DOM representation of control 'Element sap.m.Input#__input2-__clone54-__clone83' has no focus DOM reference Error: DOM representation of control 'Element sap.m.Input#__input2-__clone54-__clone83' has no focus DOM reference     at constructor.$ ([https://ui5.sap.com/1.120.1/resources/sap/ui/test/actions/Action.js:6:1158) ](https://ui5.sap.com/1.120.1/resources/sap/ui/test/actions/Action.js:6:1158)/n)    at constructor.executeOn ([https://ui5.sap.com/1.120.1/resources/sap/ui/test/actions/EnterText.js:6:719) ](https://ui5.sap.com/1.120.1/resources/sap/ui/test/actions/EnterText.js:6:719)/n)    at Object.executeOn ([https://ui5.sap.com/1.120.1/resources/sap/fe/test/builder/MdcTableBuilder.js:1202:20) ](https://ui5.sap.com/1.120.1/resources/sap/fe/test/builder/MdcTableBuilder.js:1202:20)/n)    at [https://ui5.sap.com/1.120.1/resources/sap/ui/test/pipelines/ActionPipeline.js:6:353 ](https://ui5.sap.com/1.120.1/resources/sap/ui/test/pipelines/ActionPipeline.js:6:353/n)    at Array.forEach (<anonymous>)     at [https://ui5.sap.com/1.120.1/resources/sap/ui/test/pipelines/ActionPipeline.js:6:331 ](https://ui5.sap.com/1.120.1/resources/sap/ui/test/pipelines/ActionPipeline.js:6:331/n)    at Array.forEach (<anonymous>)     at c.process ([https://ui5.sap.com/1.120.1/resources/sap/ui/test/pipelines/ActionPipeline.js:6:309) ](https://ui5.sap.com/1.120.1/resources/sap/ui/test/pipelines/ActionPipeline.js:6:309)/n)    at O ([https://ui5.sap.com/1.120.1/resources/sap/ui/test/OpaBuilder.js:6:2151) ](https://ui5.sap.com/1.120.1/resources/sap/ui/test/OpaBuilder.js:6:2151)/n)    at Object.executeOn ([https://ui5.sap.com/1.120.1/resources/sap/ui/test/OpaBuilder.js:6:10873)' This](https://ui5.sap.com/1.120.1/resources/sap/ui/test/OpaBuilder.js:6:10873)'/nThis) is what Opa logged: Found 0 blocking out of 1106 tracked timeouts -  sap.ui.test.autowaiter._timeoutWaiter#hasPending AutoWaiter syncpoint -  sap.ui.test.autowaiter._autoWaiter Found view with ID 'orders::OrdersObjectPage' and viewName 'undefined' -  sap.ui.test.Opa5 Found control with ID 'fe::table::orderLines::LineItem::Positionen' and controlType 'sap.ui.mdc.Table' in view 'sap.fe.templates.ObjectPage.ObjectPage' -  sap.ui.test.Opa5 1 out of 1 controls met the matchers pipeline requirements

Hi,
when it is only failing in the headless mode, could you please check the window-size? If you don't define something specifically the "invisible" browser starts with a viewport of only 800x600px which is to small for most of the responsive applications.
Regards
Simon

Hi @Siolto thank you for your response. In the meantime I was checking different ways and what I found out that iChangeRow() is under no circumstances working on the ubuntu image.
When changing to the windows image or running locally on my windows machine it is working.
The window size is set to 1920, 1080. Also when it is set to lower resolution it is working on a windows machine.
Must be something that is different on ubuntu..


TypeError in PolynomialMutation and SBXCrossover

I came across a small bug in the implementation of PolynomialMutation and SBXCrossover.
Both use pow(...) multiple times. Sometimes the exponent is between 0 and 1 by default. Therefore, complex numbers are introduced if the base is negative and a TypeError occurs.
My solution proposal is to add abs(...) around the base of the deltaq computation in PolynomialMutation (line 74 and 78) and around the base of the betaq computation in SBXCrossover (line 173, 175, 182 and 184).

Note: I just noticed the line numbers are off by a few lines in the original post.

Hi.
I have never found such situations when using polynomial mutation and SBX crossover. What distribution index values are you using?
Antonio

Hi,
for example let's look at the following code-snipet:
betaq = pow(rand*alpha, (1.0/(self.distribution_index + 1.0)))
For any distribution_index -2 > x > 0 there is no problem, since the exponent will not be a fraction value between -1 and 1.
But anything else will result in a value between -1 and 1. And any value to the power of such a value, will be interpreted as root. And the root of a negative number is a complex number.

I have used distribution index values within the range (5.0, 400.0).

I've applied those operators in many studies in the last 15 years and I have never had an error related to complex numbers in pow(), so this issue is intrigues me.

I see your point. But mathematically it makes sense (in my opinion) to add abs(). I don't always get a negative base, but sometimes.

The issue is to decide that, if you get a negative base, that should be considered as an error or as a valid value. If it is the former, using abs() would hide the error, which could be very difficult to find later if an algorithms yields unexpected results.


Fix linkcheck error

Linkcheck was reporting a valid anchor as a broken link. Added line instructing linkcheck to ignore anchor.
QA Steps: ran a clean linkcheck, no issues found.
No related task, quick fix.
PR Template and Checklist
Please complete as much as possible to speed up the reviewing process.
Readiness and adding reviewers as appropriate is required.
All PRs should be reviewed by a technical writer/documentation team and a peer.
If effecting customers—which is a majority of content changes—a member of Customer Success must also review.
Readiness

[x] Merge (pending reviews)
[ ] Merge after date or event
[ ] Draft

Overview
Why merge this PR? What does it solve?
Checklist

[x] Avoid changing any header associated with a link/reference.
[x] Run make linkcheck.
[x] follow best practices for commits.

[x] Descriptive title written in the imperative.
[x] Include brief overview of QA steps taken.
[x] Mention any related issues numbers.
[x] End message with sign off/DCO line (-s, --signoff).
[x] Sign commit with your gpg key (-S, --gpg-sign).
[x] Squash commits if needed.

Docs for 42ddc197f998a4274ac2947fb5a71b0784184d94 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2719/docs/artifacts/html/index.html

Docs for c24148fcc87f7b26e8f8027cda2f9a780b26e0a9 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2720/docs/artifacts/html/index.html

Docs for aa4e3736ef3d5980f2c077caa9e39331e64bfa39 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2723/docs/artifacts/html/index.html

Docs for 764b40b0f7be7199be5f83858dbae18d771b6ceb are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2724/docs/artifacts/html/index.html

Docs for e669ccd0f2120c89d1f5a38bbe4f56d614b576c7 are browsable at: https://ci.foundries.io/projects/fio-docs/builds/2725/docs/artifacts/html/index.html


doesn't appear to be working

not sure what the issue is. says clipping coupons in the console but it's not doing anything

i can confirm 'she's dead jim'

tried activating after already being logged in - tried fresh browser session clicked plugin which did take me to the bjs site as per instructions - logged in and it doesn't seem to do anything - can't think of any other way to possibly jump start the clicking process

Confirmed, it stopped working. I'll take a look at it and see if I can get it working again. They probably changed the coupon API.

Okay, I think I have it patched up and added support for a 2nd site. Check out the new release!
Clip-It!

working great thanks - ublock origin seemed to be interfering with it but once disabled it grabbed all the coupons


Use requirements files for dependencies in ci.

Status
READY
Description
Use requirements files for dependency installation in circle.

Pull Request Test Coverage Report for Build 213

0 of 0   changed or added relevant lines in 0 files are covered.
26 unchanged lines in 11 files lost coverage.
Overall coverage increased (+13.4%) to 64.691%

Files with Coverage Reduction
New Missed Lines
%

icetea_lib/wireshark.py
1
67.48%

icetea_lib/GitTool.py
1
74.7%

icetea_lib/ResultList.py
1
77.78%

icetea_lib/TestSuite/TestSuite.py
1
79.54%

icetea_lib/TestSuite/TestcaseFilter.py
1
82.86%

icetea_lib/Searcher.py
2
93.62%

icetea_lib/AllocationContext.py
2
55.56%

icetea_lib/ResourceProvider/ResourceProvider.py
3
86.44%

icetea_lib/TestSuite/TestcaseContainer.py
4
82.08%

icetea_lib/LogManager.py
4
76.36%

Totals

Change from base Build 201:
13.4%

Covered Lines:
4223

Relevant Lines:
6528

💛  - Coveralls


Allaman in the Lazyman Neovim Configuration Manager

I'm working on including this configuration in Lazyman, a Neovim configuration manager.
In order to get this config to initialize cleanly with nvim --headless "+Lazy! sync" +qa it was necessary to patch mason.lua slightly to refresh the registry. Not sure why. Below is the patch I am using.
If you have any concerns, questions, comments, or suggestions as to how best to incorporate this configuration into Lazyman please feel free to fire away. It was one of the Neovim configs that motivated and inspired me to write a configuration manager. Your install.sh script acted as a template for the Lazyman install_neovim.sh although it has since diverged quite a bit. I kept your copyright notice intact, let me know if you want me to remove it.
I also based a lot of my own configuration off of some of what you have done, especially your settings.lua so thanks all around!
After patching (during the initial installation prior to initialization) the 'Allaman' Neovim configuration works well and I am planning on including it in the next release (v2.1.4r4).
mason.lua patch:
--- lua/core/plugins/mason.lua.orig	2023-06-05 18:56:59.153237963 -0700
+++ lua/core/plugins/mason.lua	2023-06-05 18:59:38.646269035 -0700
@@ -11,12 +11,19 @@

     -- ensure tools (except LSPs) are installed
     local mr = require("mason-registry")
-    for _, tool in ipairs(settings.tools) do
-      local p = mr.get_package(tool)
-      if not p:is_installed() then
-        p:install()
+    local function install_ensured()
+      for _, tool in ipairs(settings.tools) do
+        local p = mr.get_package(tool)
+        if not p:is_installed() then
+          p:install()
+        end
       end
     end
+    if mr.refresh then
+      mr.refresh(install_ensured)
+    else
+      install_ensured()
+    end

     -- install LSPs
     require("mason-lspconfig").setup({ ensure_installed = settings.lsp_servers })

Hello @doctorfree,
Thank you for your warm words. I am glad that you like my config and that it gives you inspiration.
I will definitely take a look at Lazyman! It's crazy what drive the Neovim community develops after the introduction of Lua as a first class citizen. :)
I have no concerns nor any questions regarding your integration of my config. Please don't hesitate to integrate it as you like.
Furthermore, thanks for the patch!

Implemented https://lazyman.dev/info/Allaman.html :)
Thanks!


Feat: added improvement for searchbar and buttons ui

In this PR have fixed the issue related to the UI design of searchbar and buttons for larger screen sizes
Fixes - #275
Have improved the design to this and have shared the screenshot of the improved UI below

Can you please check build error what causing to fail?

Please create new PR i have resolved build error, I recommend forking the latest version of this repository and add your changes. I've recently pushed some updates as well

ok will do that
Thanks!


add pipelinetask metadata validation

Changes
Add parameter validation of metatdata in pipelinetask taskSpec.
Avoid invalid metadata causing PR to run stuck.
Fixes:  #6453
/kind bug
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Has Docs if any changes are user facing, including updates to minimum requirements e.g. Kubernetes version bumps
[x] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[ ] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
Add parameter validation of metatdata in pipelinetask taskSpec.

@cugykw doesn't this also need to be ported to the v1/pipeline_types.go?

/assign

doesn't this also need to be ported to the v1/pipeline_types.go?
@dibyom i have updated.

I think it would be better to let k8s handle validation of metadata. From #6453 it sounds like when the controller failed to create a taskrun, the pipelinerun got stuck and was still marked as running. It sounds to me like we should address that issue directly. It doesn't fail as fast as the solution in this PR, but I'm worried we could still run into the same problem if taskrun creation fails for other reasons.

@lbernick @vdemeester I'm so sorry that I've noticed lately that I haven't been replying to everyone. I very much agree with this opinion. When our users had this error at the beginning, I thought it was a pending situation again. In fact, this error will not lead to a pending situation, and the displayed error message is relatively clear.

/close


Reduce typo count

This reduces the number of typos within the codebase's comments and text.

I went and submitted fixes for other typos to the actual wasi-* repositories ... but no idea when or how that'll get fixed and propagated.


Yeelight cannot be assigned to an area

The problem
I have configured a Yeelight lamp and it does not appear under entities or devices and thus cannot be assigned to an area.
Environment

Home Assistant release with the issue: 0.105.5
Last working Home Assistant release (if known): n/a
Operating environment (Hass.io/Docker/Windows/etc.): hass.io
Integration causing this issue: yeelight
Link to integration documentation on our website: https://www.home-assistant.io/integrations/yeelight/

Problem-relevant configuration.yaml
yeelight:
  devices:
    <IP_ADDRESS>:
      name: office
      model: ceiling3

Traceback/Error logs

Additional information

This is not a bug but a feature that is not implemented by the Yeelight integration.
We use GitHub for tracking issues, not for providing support or tracking feature requests.
If you want to suggest a feature, you should try our Community Forum: Feature Requests.
If you have additional questions, feel free to join our Discord chat server.
Thanks! 👍


XCTSpeziAccount

Problem
Creating UI tests for the Spezi Account package currently requires a significant level of UI testing code to achieve the login or sign-up functionality.
Solution
XCTSpeziAccount should be a new target in the SpeziAccount module that automates the UI testing for SpeziAccount-related UI elements. By knowing insights into the Account Module and being used in the UI tests of the Account Module, we can simplify the UI testing flow for applications using the module.
An alternative or additional use case of the XCTSpeziAccount module could facilitate internal functions in SpeziAccount that allow one to skip the onboarding UI completely using a predefined set of command line arguments passed to the application under test.
Additional context
A good first issue to get familiar with the framework and its testing infrastructure.
Code of Conduct

[X] I agree to follow this project's Code of Conduct and Contributing Guidelines

We started this target with #76


Changing FancyZones layouts results in windows of recent Outlook items being created, all of which have blank content

Microsoft PowerToys version
0.61.1
Running as admin

[ ] Yes

Area(s) with issue?
FancyZones
Steps to reproduce
Press Windows key ` to bring up the Zones window, and choose a new layout.
✔ Expected Behavior
New layout happens and existing apps are moved to them
❌ Actual Behavior
In addition to the expected behavior, new windows appear for all recent Outlook items I've worked on (eg as if they emails have been opened up into new windows), but the content of them is all blank
Other Software
No response

/dup #19440


improving Contributing guide

I saw some good description related to the issues and pull request is missing . I will try to improve this .

Thank you, @dans77777.
Can you please give some more details as to the improvements to be made?

@c-rindi  This is the screenshot of contribute.md file. here under contributing guide point I wanted to add one line description for developers about gh-pages branch and link to the same and also link for understanding the workflow of the project. bcz I know when I was new to this project during MLH journey , there was no way for me to know about gh-pages branch to work with website . also I wanted to change some sentence structure like they have given "send us a pull request" , raising a pull request is most appropriate according to the standard. these minor things surely makes high quality docs and really useful .
I hope this is clear to you.
Thank you

Hey team! Please add your planning poker estimate with ZenHub @dejanzele @jayofdoom @kannon92 @richscott

Hi @dans77777 Please have you worked on this? and if no I want to work on it what do I need to know.

Hi @dans77777 Please have you worked on this? and if no I want to work on it what do I need to know.

@kindyluv heyy, I already have a PR on this. can you please start with #1144 ? Thank you.


Return err instead of nil while add config files in flare

What does this PR do?
The code swallows the error for adding config file to flare. This suppressed warning messages.  @DataDog/burrito
Motivation
bug fix.
Additional Notes
Anything else we should know when reviewing?

👍


ZEVA - Credit transactions pages idir, Edit comments

Title: The comment feature between the analyst/director does not have the edit function.
Description:
The Edit function should be available to both the Analyst and Director to edit their own comments only as available on MY Reports on the following pages:

Credit Transfer
Initiative Agreement
Purchase Agreement
Administrative Credit Allocation
Administrative Credit Reduction
Automatic Administrative Penalty

Wireframe:
Private Zenhub Image
Private Zenhub Image
Purpose and benefit to user: Users can correct problems with their comments. The expectation to edit comments has been set by the MY Report commenting feature.
Acceptance Criteria:

[ ] Given I am a idir\analyst or director, When I am processing a credit transaction of any type, then when I add a comment I can edit that comment up until the transaction is in its final state when it will no longer be editable

Development Checklist:

[ ] A
[ ] B
[ ] C

Notes:

Hey team! Please add your planning poker estimate with Zenhub @tim738745 @JulianForeman


ionic build does not reflect cordova's exit code

If cordova fails to build when I run e.g. ionic build android, the log will say that cordova exited with a certain exit code but ionic will exit with code 0.

+1 This is really silly when you want to integrare ionic build android on your CI system and you rely on the status code to stop the script

@jthoms1 you closed it because it has been fixed or because it won't be fixed?

Please reopen, this is obviously a bug.
The same happens for all the ionic commands.


Footer Section

I want to enhance the footer now it is simple but i can make it good please assign this to me under SSOC

Can i take this

I would like to work on this assign it to me ssoc

Hey! @TusharKesarwani, I would like to work on this issue as I am competing in SSOC'23 and I will enhance the UI/UX design of footer section.

hey can you assign this issue under ssoc 2023

Hi @TusharKesarwani,
As a participant of the SSOC 2023 program kindly consider me for this issue because I have experience in web design, frontend development, and user interface enhancement.

Hey I can handle this issue easily I have experience in UX and web dev

Hey @TusharKesarwani
Will you assign these task to me ??

Create the fresh issue after 1st June


Added file

@JessedeB Please provide us with:

the school official website URL, if different from the domain you are submitting
an URL of a page on the official website where an IT-related long-term (>1 year) course is offered by the school
an URL of a page or another proof (.pdf or a screenshot are OK) showing that the school recognizes the domain which you are submitting as an official email domain for the students.

Hello,
Here the details of my school :
School website:
Https://rocva.nl
URL course software development :
https://www.rocva.nl/MBO-onderwijs/Sectoren/ICT/Software_Developer
Image of proof @talnet.nl belongs to our school :
[cid:B0CB0D06-C9BF-4568-89DE-04D439207F97-L0-001]
Op 9 mrt. 2020 om 17:58 heeft Philip Torchinsky<EMAIL_ADDRESS>het volgende geschreven:

@JessedeBhttps://github.com/JessedeB Please provide us with:

the school official website URL, if different from the domain you are submitting
an URL of a page on the official website where an IT-related long-term (>1 year) course is offered by the school
an URL of a page or another proof (.pdf or a screenshot are OK) showing that the school recognizes the domain which you are submitting as an official email domain for the students.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/JetBrains/swot/pull/7879?email_source=notifications&email_token=AEBESBPNK5L2UKN5WQGVPXLRGUNZ5A5CNFSM4LEM5SS2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOIC4AA#issuecomment-596651520, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEBESBK2HUOCHWRTSF433O3RGUNZ5ANCNFSM4LEM5SSQ.

@JessedeB I am sorry, GitHub strips out any attachments if you send it by email. Could you please attach the image with proof via web interface? Thank you.

Hello,
I have created a new pull request with my school email<EMAIL_ADDRESS>), I also provided my school details and image url in the talnet.txt
Op 9 mrt. 2020 om 20:44 heeft Philip Torchinsky<EMAIL_ADDRESS>het volgende geschreven:

@JessedeBhttps://github.com/JessedeB I am sorry, GitHub strips out any attachments if you send it by email. Could you please attach the image with proof via web interface? Thank you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/JetBrains/swot/pull/7879?email_source=notifications&email_token=AEBESBLFU4ZASLJFF7V64G3RGVBKTA5CNFSM4LEM5SS2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEOIZOZI#issuecomment-596744037, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEBESBJFZEEVM4OT5H4AIG3RGVBKTANCNFSM4LEM5SSQ.

@JessedeB Pull request merged. I did not find your other pull request as for now, but I found the evidence of the domain ownership: https://www.rocva.nl/getmedia/55717d2d-9216-417a-a11d-f49fafcbad9f/Studentenwijzer-2019-2020.pdf.aspx
Thank you!


fix(recipes): Allow undefined for variants

Closes #1442

Hi @askoufis, is there anything I can do to get this reviewed and merged any quicker?

@askoufis thank you, i fixed up the changes

@askoufis thank you, i fixed up the changes

Thanks. Your branch is also out-of-date relative to master. Ordinarily I'd be able to update it, but maybe you have some stricter permissions on your fork?

@askoufis ah sorry, i think it's because i had removed the Allow edits by maintainers checkmark. should be rebased now


error report when evaluate  Open-Vocabulary Segmentation for cityscapes

Hello,
when i try to evaluate Open-Vocabulary Segmentation for cityscapes, there are some following errors
[02/07 15:28:29 detectron2]: Start inference on 500 batches
0it [00:14, ?it/s]
Traceback (most recent call last):
File "/home/user/workspace/Osprey/osprey/eval/eval_open_vocab_seg_detectron2.py", line 634, in
evaluator.process(inputs, outputs)
File "/home/user/workspace/Osprey/detectron2/detectron2/evaluation/evaluator.py", line 88, in process
evaluator.process(inputs, outputs)
File "/home/user/workspace/Osprey/detectron2/detectron2/evaluation/cityscapes_evaluation.py", line 75, in process
class_id = name2label[classes].id
KeyError: 'car,cars'
I finished setup as https://github.com/CircleRadon/Osprey/blob/main/osprey/eval/README.md#:~:text=1.-,Open-Vocabulary Segmentation,-Download SentenceBERT model
and data preparation as well done.
https://github.com/CircleRadon/Osprey/blob/main/osprey/eval/datasets/README.md
thanks.

Hi, @koda-11. You can add a line classes = classes.split(‘,’)[0] at line 74, where the error is raised,
in detectron2/detectron2/evaluation/cityscapes_evaluation.py and it works.


Zero division error with new spanish formulas

If I use:
>>> textstat.szigriszt_pazos("")
>>> textstat.gutierrez_polini("")
>>> textstat.crawford("")

It will all raise an error because total_words is 0. I can make a PR to solve it.
I think that the ideal solution would be catching the error, raising a warning and returning a 0. Letting the division error raise, can be problematic.

@GuillemGSubies thank you for raising this. Yes, I think you are right. There are other examples where ZeroDivision errors can occur, and in those cases we return 0. If you can raise a PR to address this that would be great!

Here it is #144


Evaluate the performance of generative models (realism and diversity)

Fix https://github.com/Project-MONAI/GenerativeModels/issues/109

Hi @marksgraham, I saw that you are reviewing this, but I left a few comments too. I hope it is okay


fix(cmd): fix pod name parsing for newer k8s

resolves https://github.com/teamhephy/workflow-cli/issues/45

second - this sort of works, but it breaks again if the app-name doesn't contain a -, fixing...

Very nice work on the test. Is this ready for review and merge?

@Cryptophobia yeah, it's ready for review 👍


The user, group or application does not have secrets set permission on key vault??

I'm trying to set and get the secret keys on Azure key vault and I'm following the below document to implement the same
https://docs.microsoft.com/en-us/azure/key-vault/secrets/quick-create-java#sample-code
I have followed the complete process mentioned on the above doc but getting the below error while i run the sample code using java to set the secret key on Azure Key vault
Servlet.service() for servlet [dispatcherServlet] in context with path [] threw exception [Request processing failed; nested exception is com.azure.core.exception.HttpResponseException: Status code 403, "{"error":{"code":"Forbidden","message":"The user, group or application 'appid=;oid=;iss=https://sts.windows.net/tenantid/' does not have secrets set permission on key vault 'kv;location=uksouth'. For help resolving this issue, please see https://go.microsoft.com/fwlink/?linkid=123455","innererror":{"code":"AccessDenied"}}}"] with root cause
Can anyone help me out with this.

@vcolin7 could you please assist?

Hi @ajaysethi8789, I ran the code in a test environment I have and encountered no issues. Based on the error message you got, I believe the problem might rely on the configuration for the service principal you are using for this example. Make sure the --secret-permissions section of the following command included the set permission, as laid out here:
az keyvault set-policy -n <your-unique-keyvault-name> --spn <clientId-of-your-service-principal> --secret-permissions delete get list set --key-permissions create decrypt delete encrypt get list unwrapKey wrapKey
You can also verify the service principal's permissions on the Azure Portal under {your key-vault-name} > Access Policies.

Hi @ajaysethi8789, can you kindly make sure the service principal you are trying to use has the correct permissions? Please refer to my previous reply :)


fetchnsearch

This issues will be solved when data from html is received by flask and subsequently python makes queries for the same.
To checkout to this branch on your local use
git fetch origin
git checkout 2-fetchnsearch

Connected HTML -> Python using form tag.
Connected Python -> MYSQL


Petter okta

What this PR does / why we need it: allows okta to be used as an oidc provider to dex
Which issue this PR fixes (use the format fixes #<issue number>(, fixes #<issue_number>, ...) to automatically close the issue when PR gets merged): fixes #201
Public facing documentation PR (if applicable)

Special notes for reviewer:oktaClientID and oktaClientSecret need to be added under dex in secrets.yaml.  We have an okta account - ask myself or Petter for access
Checklist:

[ ] Added relevant notes to WIP-CHANGELOG.md
[ ] Proper commit message prefix on all commits
[ ] Updated the public facing documentation
Is this changeset backwards compatible for existing clusters? Applying:

[ ] is completely transparent, will not impact the workload in any way.
[ ] requires running a migration script.
[ ] will create noticeable cluster degradation.
E.g. logs or metrics are not being collected or Kubernetes API server
will not be responding while upgrading.
[ ] requires draining and/or replacing nodes.
[ ] will change any APIs.
E.g. removes or changes any CK8S config options or Kubernetes APIs.
[ ] will break the cluster.
I.e. full cluster migration is required.

If you address Olles comments and fill out the PR template this LGTM


fix: removed connection tab on user profile page

fix: #3141
Description
This PR removes the "Connections" Tab from the user's profile page (Was unsure if this is considered a refactor or fix)

Related Tickets & Documents

Mobile & Desktop Screenshots/Recordings

I do not have a pro account with OpenSauced, so I was unable to view the "Connections" Tab on my profile page. See below:

I followed the directions of the file to edit and the file to remove. Once the changes were made, I ensured the profile page was still populating and here is the screenshot:

Steps to QA

From OpenSauced homepage -> click on profile picture on top right corner -> from the dropdown menu, select {username}
Once on Profile Page, check the tabs on the right. They should read "Highlights", "Contributions", "Recommendations"
"Connections" tab should be gone on the Pro Accounts. Free tiers will not see this tab populated. (I found this in the docs, but now am unable to find where I read this. I can keep searching if needed. Especially if documentation needs to be edited)
I did notice in the repo that lines 114-117 were showing an error. I did not touch these lines since it was not in the instructions provided. But this error does not break the page. (https://github.com/open-sauced/app/blob/18d3b15f12a993d5bc8ce0061cd4a5520b72e846/components/organisms/ContributorProfileTab/contributor-profile-tab.tsx)

Tier (staff will fill in)

[ ] Tier 1
[ ] Tier 2
[ ] Tier 3
[x] Tier 4

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

Hoping I followed the directions correctly

@beckyrich, looks like there is one more spot that needsto be udpated as well.
10:33:37 AM: ./components/organisms/ContributorProfileTab/contributor-profile-tab.tsx:114:38
10:33:37 AM: Type error: This comparison appears to be unintentional because the types 'TabKey' and '"connections"' have no overlap.
10:33:37 AM:   112 |         "font-bold text-transparent bg-clip-text bg-gradient-to-r from-[#EA4600] to-[#EB9B00]",
10:33:37 AM:   113 |       user && user_name !== login && tab === "recommendations" && "hidden",
10:33:37 AM: > 114 |       user && user_name !== login && tab === "connections" && "hidden",
10:33:37 AM:       |                                      ^
10:33:37 AM:   115 |       !user && tab === "recommendations" && "hidden",
10:33:37 AM:   116 |       !user && tab === "connections" && "hidden",
10:33:37 AM:   117 |       !receive_collaboration && tab === "connections" && "hidden"

You just need to remove these lines
  const getTabTriggerClassName = (tab: TabKey): string => {
    return clsx(
      "data-[state=active]:border-sauced-orange shrink-0 data-[state=active]:border-b-2 text-lg",
      tab === "recommendations" &&
        "font-bold text-transparent bg-clip-text bg-gradient-to-r from-[#EA4600] to-[#EB9B00]",
      user && user_name !== login && tab === "recommendations" && "hidden",
-      user && user_name !== login && tab === "connections" && "hidden",
      !user && tab === "recommendations" && "hidden",
-      !user && tab === "connections" && "hidden",
-      !receive_collaboration && tab === "connections" && "hidden"
    );
  };

You can check once you've made those additional changes by running npm run build. It should build without any errors.

Thank you @nickytonline. I just made the changes locally. Do I need to use npm run push again for the commit message before I push up the requested changes?

Thank you @nickytonline. I just made the changes locally. Do I need to use npm run push again for the commit message before I push up the requested changes?

Yeah, you need to do git push once you've committed those changes, but I see you've got it pushed. 😎


Guava and caffeine cache refactoring

Cache refactoring and
Please answer these questions before submitting pull request

Why submit this pull request?

[ ] Bug fix

[ ] New feature provided

[ x ] Improve performance

Related issues
https://github.com/apache/incubator-skywalking/pull/1069

@nikitap492 This pr still includes Object util refactor and cache refactor both. Could you separate them?

@nikitap492 Please make the cache refactor only included in the pull request. Then use another branch or pr to do the object util refactor. OK?


MWPW-126552 - Fix georouting stylesheet

Load modal styles for georouting

Resolves: MWPW-126552
Test URLs:

Before: https://main--milo--adobecom.hlx.page/drafts/methomas/georouting?martech=off
After: https://methomas-locale-modal-bug--milo--adobecom.hlx.page/drafts/methomas/georouting?martech=off
After: https://main--bacom--adobecom.hlx.page/resources/videos/marketo-product-tour?milolibs=methomas-locale-modal-bug&martech=off

Bug:
To reproduce, join the hamburg vpn and open the link in an incognito tab. (You can verify your country with the "akamai" session storage item)

Fix:

Victor already has a PR for this #489


Supply label config in external yaml/json file

I'm submitting a ...
[ ] bug report
[x] feature request
[ ] question about the decisions made in the repository
[ ] question about how to use this project

Summary

It would be great if the configuration for which labels to add could be supplied as an external yaml/json/whatever file (rather than having to edit the source).

@klieret thank you for your request.
I will work on this and the other you posted.

Awesome :)   I would love to use this tool like github-label-manger add-labels -o my-organization -r my-repo -c label-config.json.
But it's already really helpful as-is.
I'm surprised this doesn't have more stars, I'm sure there's plenty of github organizations/users for whom that would be useful...