Add a meta tag to indicate custom elements manifest location on webpage

Documented here: https://github.com/webcomponents/custom-elements-manifest/pull/70
As in some situations, the tooling might not have access to a package.json, it would be beneficial to be able to indicate the location of the custom elements manifest in the html page. A meta tag should be enough for this.
This came up in a discussion with @claviska (https://github.com/claviska), when discussing CEM and DevTools support with the shoelace documentation site.
This would allow documentation sites to indicate the location of the CEM through a meta tag, allowing the devtools to find the manifest for an enhanced experience on the docs page.

I'm not quite following the use case here. Is the idea that the devtools extension will know how to enhance the docs pages somehow? It seems like it'd be better to include whatever functionality is needed right in the page so that viewers don't have to install an extension or open devtools. We can build really nice documentation components to cover these cases, yeah? What kind of enhancements are we talking about?

The devtools case maybe wasn't the best example here. But the use case that raised this need was that you would be able to inspect and fiddle with elements in a storybook/custom similiar environment (shoelace.style in this case).
But thinking about this more, I think it'd be beneficial to allow sites without a package.json or other configuration to allow tooling to discover their custom-elements-manifest more reliably. As the adopting of the CEM grows, more tooling could utilize the meta tag to reliably find the manifest, instead of the developers having to declare it explicitly to every tool.
Cases that come to mind would be a cataloging crawler, something like the custom elements locator. As those tools might not have access to any other reliable source to confirm the location of a custom-elements.json, They'd have to either guess the location, or fetch it from the meta tag.

Nudging this.
shoelace.style has already adopted this pattern and it is a fairly small thing for the developer to do to add support for more easily discoverable CEM's that I don't see a downside on adding this to the README.
The PR here https://github.com/webcomponents/custom-elements-manifest/pull/70 also has a few of the web component people thumb upping it, so I'm guessing this is a feature that would be wanted

If metadata as described here is needed, wouldn't a link relation be more suitable?
https://html.spec.whatwg.org/multipage/semantics.html#the-meta-element:

The meta element represents various kinds of metadata that cannot be expressed using the title, base, link, style, and script elements.

https://html.spec.whatwg.org/multipage/semantics.html#other-metadata-names:

a new metadata name should not be created in any of the following cases:

If either the name is a URL, or the value of its accompanying content attribute is a URL; in those cases, registering it as an extension to the predefined set of link types is encouraged (rather than creating a new metadata name).
[...]

I'm still very confused by the use case here. Where are the manifests going to be stored? Is there a canonical location like unpkg.com? Or should libraries host their manifests somewhere? Should the be versioned? How are the <link> tags going to make it into the page? If a page uses elements from many libraries, should they have many link tags?
Then, what's the use case for a public page to have its manifest(s) published? Is there any precedent with JS libraries, like a <link> tag for saying which npm packages a page uses?


new CSS trigonometry functions

New functions proposed, not implemented.
https://www.zdnet.com/article/css-to-get-support-for-trigonometry-functions/

First implementation in Safari Tech Preview 133.

Gecko implemented trigonometry functions in Firefox 103 behind layout.css.trig.enabled flag Source.


Error on conversation request. Check the log for more details.

VS Code Version:   1.83.0
OS Version:   release version v0.8.0 (preview)
Logs:

2023-10-10T20:48:26.433Z [INFO] [chat] copilot token chat_enabled: true
2023-10-10T20:48:26.437Z [INFO] [auth] Got Copilot token for sh-admin-AW
2023-10-10T20:53:26.090Z [ERROR] [asyncCompute] Failed to compute async parser based result: (TypeError) t is not iterable
2023-10-10T20:53:26.093Z [ERROR] [asyncCompute] Failed to compute async parser based result: (TypeError) t is not iterable
2023-10-10T20:53:26.094Z [ERROR] [asyncCompute] Failed to compute async parser based result: (TypeError) t is not iterable
2023-10-10T20:53:26.144Z [ERROR] [asyncCompute] Failed to compute async parser based result: (TypeError) t is not iterable
2023-10-10T20:53:26.145Z [ERROR] [asyncCompute] Failed to compute async parser based result: (TypeError) t is not iterable
Steps to Reproduce:

open GitHub Copilot Chat through the right click context menu and selecting 'explain this'  or
using GitHub Copilot Chat in the left pane in VS code

reverting to a pre-release version does not resolve.

Can you try insiders with the latest pre-release. I cannot reproduce

it appears it is working again after a reload today (presumably a new version released) I am using the v0.8.0 Preview (not pre-release) .
More Info
Published               2023-4-27, 00:13:54
Last released          2023-10-11, 09:40:50
Last updated          2023-10-5, 10:45:44
Identifier                 github.copilot-chat

same error.
output:
2023-10-12T01:24:33.792Z [INFO] [auth] Logged in as tourbut
2023-10-12T01:24:34.638Z [INFO] [chat] copilot token chat_enabled: true
2023-10-12T01:24:34.642Z [INFO] [auth] Got Copilot token for tourbut
2023-10-12T01:26:17.983Z [INFO] [chat fetch] engine https://api.githubcopilot.com/chat
2023-10-12T01:26:17.983Z [INFO] [chat fetch] modelMaxTokenWindow 4096
2023-10-12T01:26:17.983Z [INFO] [chat fetch] chat model copilot-chat
2023-10-12T01:26:18.821Z [INFO] [chat fetch] engine https://api.githubcopilot.com/chat
2023-10-12T01:26:18.821Z [INFO] [chat fetch] modelMaxTokenWindow 4096
2023-10-12T01:26:18.821Z [INFO] [chat fetch] chat model copilot-chat

I had the same problem

I was mistaken.  the issue continues.  I had to revert to version 0.7.1 which seems to work.

is there any news? still not working

Please try https://code.visualstudio.com/insiders/ and the latest pre-release of the extension it should ve v0.9

Also not working for me using insiders as described above, and hasn't been for the past few days.
Windows 10.0.19045
VS Code v1-84-0-insider
GitHub Copilot v1.126.490 (Pre-Release)
GitHub Copilot Chat v0.9.2023101202 (Pre-Release/Preview)
Unlike the original issue, I don't get any errors in the logs, but the chat extensions fails with the same message.
GitHub Copilot Chat Insiders VS Code & Preview Extension output:
2023-10-13T08:48:27.598Z [INFO] [auth] Logged in as ******
2023-10-13T08:48:28.064Z [INFO] [chat] copilot token chat_enabled: true
2023-10-13T08:48:28.069Z [INFO] [auth] Got Copilot token for ********
2023-10-13T08:48:54.287Z [INFO] [asyncCompute] Computing async parser based result took longer than 55ms
2023-10-13T08:48:54.370Z [INFO] [chat fetch] engine https://api.githubcopilot.com/chat
2023-10-13T08:48:54.370Z [INFO] [chat fetch] modelMaxTokenWindow 8192
2023-10-13T08:48:54.370Z [INFO] [chat fetch] chat model gpt-3.5
2023-10-13T08:49:10.903Z [INFO] [chat fetch] engine https://api.githubcopilot.com/chat
2023-10-13T08:49:10.903Z [INFO] [chat fetch] modelMaxTokenWindow 4096
2023-10-13T08:49:10.903Z [INFO] [chat fetch] chat model gpt-4

GitHub Copilot Chat Standard VS Code/Extension release output:
2023-10-13T08:55:53.926Z [INFO] [auth] Logged in as ***
2023-10-13T08:55:54.839Z [INFO] [chat] copilot token chat_enabled: true
2023-10-13T08:55:54.843Z [INFO] [auth] Got Copilot token for ***
2023-10-13T08:56:00.822Z [INFO] [asyncCompute] Computing async parser based result took longer than 200ms
2023-10-13T08:56:01.430Z [INFO] [chat fetch] engine https://api.githubcopilot.com/chat
2023-10-13T08:56:01.430Z [INFO] [chat fetch] modelMaxTokenWindow 4096
2023-10-13T08:56:01.430Z [INFO] [chat fetch] chat model copilot-chat

Should I raise a separate issue?

@laurenceedge Does inline chat work for you, via Ctrl + i inside a code file?

@lramos15 Nope

@lramos15 Nope

But you receive the normal ghost text completions from Copilot. Just the chat features are not working?

Even I am facing same error. Tried pre release version as well. Only the chat feature is not working. Able to see ghost text completions logs in output. Waiting for the solution.

@lramos15 Nope

But you receive the normal ghost text completions from Copilot. Just the chat features are not working?

I have the same issue.  ghosttext completion works fine.  only the chat extension was having this problem.  still currently running on the older version 0.7.1 until a solution is put in place

JavaScript, TypeScript, TypeScript React, bash script, etc. Doesn't seem to be file-dependent.
Dropping down to 0.7.1 of the Chat extension appears to work for me as well.

same error here. the issue seems to be specifically on the Github Copilot Chat extension as Copilot and Copilot Labs are still working fine. Neither the chat window nor the inline chat are working.
The chat window give Error on conversation request. Check the log for more details.
The inline chat give Failed to make request
i have the same output log as tourbut
The issue really does not appear to be file dependant. i can't use chat even in small files (76 lines), JS files, C# files or Rust files

Is anyone here behind a firewall?

I am using a corporate VPN with Zscaler software.  this did not seem to matter, as I had used both copilot and chat and upgraded through several versions before reaching the version with the problem.

I am using a corporate VPN with Zscaler software. this did not seem to matter, as I had used both copilot and chat and upgraded through several versions before reaching the version with the problem.

Is there a way you could test with your corporate VPN disabled? The URL we're sending requests to has changed and *.githubcopilot.com should be allow listed in your firewall rules

thanks.  I'll check with our security team.  Unfortunately, no, its configured in such a way that there is no internet traffic without establishing a VPN first.

Next pre-release tomorrow will contain better network error logging for this case. My hypothesis is that firewall blocking is causing this error.
Please try the latest insiders and pre-release tomorrow and report back.

Enabling the following setting in VSCode worked for me; In Settings > Application > Proxy enable Experimental: System Certificates V2

Closing this one as a duplicate of https://github.com/microsoft/vscode-copilot-release/issues/439 as they have the same underlying problems which have to do with fancier network setups.

I rolled back the version of Copilot Chat to 0.7.1, and it work

I had the same issue as @sh-admin-AW and tried both release and pre-release versions. After reading @lramos15's comment above, I turned off my corporate VPN and it was able to connect. Trying to get my IT folks to whitelist *.githubcopilot.com now - but thought I'd confirm my experience.


Invalid brew tap

Homebrew 3.4.11
Error: Invalid formula: /usr/local/Homebrew/Library/Taps/openslo/homebrew-openslo/Formula/oslo.rb
oslo: Calling bottle :unneeded is disabled! There is no replacement.
Please report this issue to the openslo/openslo tap (not Homebrew/brew or Homebrew/core):
/usr/local/Homebrew/Library/Taps/openslo/homebrew-openslo/Formula/oslo.rb:10
Error: Cannot tap openslo/openslo: invalid syntax in tap!

Also w/ Homebrew 3.5.1
$ brew tap openslo/openslo
==> Tapping openslo/openslo
Cloning into '/opt/homebrew/Library/Taps/openslo/homebrew-openslo'...
remote: Enumerating objects: 16, done.
remote: Counting objects: 100% (16/16), done.
remote: Compressing objects: 100% (13/13), done.
remote: Total 16 (delta 2), reused 0 (delta 0), pack-reused 0
Receiving objects: 100% (16/16), 6.51 KiB | 6.51 MiB/s, done.
Resolving deltas: 100% (2/2), done.
Error: Invalid formula: /opt/homebrew/Library/Taps/openslo/homebrew-openslo/Formula/oslo.rb
oslo: wrong number of arguments (given 1, expected 0)
Error: Cannot tap openslo/openslo: invalid syntax in tap!

Taking a look at this

Updated the formula, and validated that it works on my end.  @fahadbinrahat @mattupstate can you both confirm on your ends that its working for you now? Thanks!

@ian-bartholomew yep! all good here now


Rename to femr-patientsearch

great idea - will confuse others in the future less, as this is the "parent repo" for fEMR, as it's used in HelloWorld environments, at least the way I understand it.

@ivan-c i was under the impression you wanted this one - true?


feat: add Nushell package

Build on top of https://github.com/brioche-dev/brioche-packages/pull/36, https://github.com/brioche-dev/brioche-packages/pull/35, https://github.com/brioche-dev/brioche-packages/pull/33

Awesome! Thanks for tracking down all the work required to get this working too!

 It helps me to understand a little bit more how the things were interacting below.


TextSpan's recognizer not working for TextSpan's children

Clicking on TextSpan's children doesn't do anything when parent TextSpan has recognizer:
TextSpan(
    text: "baz ",
    children: [
        TextSpan(text: "foo"),
        TextSpan(text: " bar"),
    ],
    recognizer: TapGestureRecognizer()
        ..onTap = () => debugPrint("Clicked!"),

Steps to Reproduce

Run project from https://dartpad.dev/210bbe7377152a225499d67f9b342cab (reproducible in Dartpad and Android emulator)
Try clicking on baz (TextSpan's text) - text color changes
Try clicking on foo (TextSpan's child) - text color doesn't change

Expected results:
Clicking on children should trigger recognizer's onTap method. It should probably respect child's own recognizer if it exists. In my example, clicking on baz, foo and bar should change text color.
Actual results:
Clicking on children doesn't trigger recognizer's onTap method. In my example, clicking on foo or bar doesn't change text color.

Logs

>flutter analyze
Analyzing myproject...
No issues found! (ran in 6.1s)

>flutter doctor -v
[✓] Flutter (Channel stable, 1.22.5, on macOS 11.0.1 20B29 darwin-x64, locale en-FI)
    • Flutter version 1.22.5 at /Users/alex/Downloads/flutter
    • Framework revision<PHONE_NUMBER> (6 days ago), 2020-12-10 11:54:40 -0800
    • Engine revision ae90085a84
    • Dart version 2.10.4

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
    • Android SDK at /Users/alex/Library/Android/sdk
    • Platform android-30, build-tools 30.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.2, Build version 12B45b
    • CocoaPods version 1.9.3

[!] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] IntelliJ IDEA Community Edition (version 2019.2.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 40.2.4
    • Dart plugin version 192.7402

[✓] VS Code (version 1.52.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.17.0

[✓] Connected device (1 available)
    • AOSP on IA Emulator (mobile) • emulator-5554 • android-x86 • Android 9 (API 28) (emulator)

! Doctor found issues in 1 category.

Reproducible on latest master and stable channel.
flutter doctor -v
[✓] Flutter (Channel master, 1.26.0-2.0.pre.64, on Mac OS X 10.15.7 19H2 darwin-x64, locale en)
    • Flutter version 1.26.0-2.0.pre.64 at /Users/pedromassango/dev/SDKs/flutter_master
    • Framework revision 9ff4326e1f (2 hours ago), 2020-12-17 06:19:05 +0000
    • Engine revision 6edb402ee4
    • Dart version 2.12.0 (build 2.12.0-157.0.dev)

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/pedromassango/Library/Android/sdk
    • Platform android-30, build-tools 30.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 12.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 12.1, Build version 12A7403
    • CocoaPods version 1.9.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 4.1)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b3-6915495)

[✓] IntelliJ IDEA Community Edition (version 2020.3)
    • IntelliJ at /Applications/IntelliJ IDEA CE.app
    • Flutter plugin version 52.0.5
    • Dart plugin version 203.5981.152

[✓] VS Code (version 1.51.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.16.0

[✓] Connected device (3 available)
    • iPhone 12 Pro (mobile) • 1EEFA5B2-DA95-49CC-AA0B-968BB248B8F3 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-14-1 (simulator)
    • macOS (desktop)        • macos                                • darwin-x64     • Mac OS X 10.15.7 19H2 darwin-x64
    • Chrome (web)           • chrome                               • web-javascript • Google Chrome 87.0.4280.88

• No issues found!
Process finished with exit code 0

@pedromassangocode Hi,
This is work as design, https://github.com/flutter/flutter/blob/32741c0ec822599c17abd8d80ebeb53c143cc818/packages/flutter/lib/src/painting/text_span.dart#L108-L109

Thanks @xu-baolin.
Closing as working as expected.

This makes it impossible to add a recognizer for a TextSpan containing already-created children, as the recognizer of the children also can't be accessed after they've been created.


V3: node_type considered harmful

I believe that putting the node type (group or array) into the zarr.info object is not a good idea.
In Zarr v2, the node type was encoded in the object name: .zgroup or .zarray. Hence it is possible
to identify all the groups and all the arrays without having to read any object.
With Zarr V3, you can only identify the groups and arrays by reading each occurrence of the zarr.info object.
This is potentially a significant performance hit.
I would propose a change to something like "zgroup.json" and "zarray.json".

Dennis, it's great to finally have your feedback. This issue was discussed extensively nearly a year ago, see https://github.com/zarr-developers/zarr-specs/issues/177
There was a strong rationale for doing it this way (avoiding a race condition with two concurrent writers trying to create a group and array of the same name). But there were also many (including myself), who favored your position.
You're very active today with many great suggestions. Unfortunately, the window for major changes to the V3 spec has passed. On May 8, @WardF voted yes on ZEP-1 on behalf of Unidata: https://github.com/zarr-developers/zarr-specs/issues/227#issuecomment-1536541942.
Now that you are actually trying to implement Zarr V3, you are finding many problems with the spec. I think this is very natural and to be expected. I feel we got the sequencing wrong by first writing this massive spec without really trying to implement it. But if we stick to the process we have defined for ourselves, the comment period is over.
I'd welcome thoughts on how to move forward in a productive way.

A key point that came up in the discussions was that typically when listing the contents of a zarr hierarchy, you will most likely want more information than just whether it is a group or an array, such as the shapes and data types of any arrays, and would need to read the metadata anyway.

"When listing the contents of a zarr hierachy..." Agreed,
But one use case we have is lazy loading of arrays. In that case,
we need to locate where the arrays are located and then read
the zarr.info only when the array is accessed.

I am in the process of implementing V3, and unfortunately, the only way I have of really understanding
the spec and its consequences is through an implementation. Just the way I work.
As for being too late, surely there is room for changes. I see a number of PRs (even referenced in the V3 spec)
proposing significant changes: e.g. moving attributes to a separate object.

"When listing the contents of a zarr hierachy..." Agreed, But one use case we have is lazy loading of arrays. In that case, we need to locate where the arrays are located and then read the zarr.info only when the array is accessed.

Can you say a bit more what "lazy loading" means?  Is the issue that your API provides a list of all of the arrays within a group (but no other information)?

Yes. Also with groups. We also do lazy attribute loading which is why I support storing attributes as a separate object.

Could you add a new API function that returns the list of all members, sub-groups and arrays, and recommend that users call that instead for greater efficiency when using zarr v3?

Not sure I follow. There seems -- to the client -- not a lot of difference in calling
two functions, one for groups, one for variables versus calling a single function.
Where does your proposed efficiency come from with respect to V3?

If there is one API function that lists both arrays and groups, and just returns a flat list of names without indicating which are arrays and which are groups, then you don't need to read the zarr.json files to handle calls to that API function.

I see. Our API does differentiate. And this is important because it is possible to walk the group tree
without having to read any objects (at least in NCZarr).

As for being too late, surely there is room for changes.

Yes, I am very optimistic that we can incorporate your suggestions and input. FWIW, I think it would be better to evolve the spec incrementally towards a "final" version as we work on implementing it, rather than deciding on the full spec up front and approving it through a big vote. The way STAC has developed their spec seems to be more effective.

And this is important because it is possible to walk the group tree without having to read any objects

I see this as a very important goal. The algorithm for walking a hierarchy is considerably more complex if you can't identify whether a node is a group or array based on the node name. As some point, the decision was made that the cost of reading the metadata files was not significant. However, I'd like to see some quantitative analysis to confirm this for realistic datasets.


Appstudio update test-component-pac-koag

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI/test-component-pac-koag-on-pull-request has successfully validated your commit.

StatusDurationName

✅ Succeeded
5 seconds
init

✅ Succeeded
23 seconds
clone-repository

✅ Succeeded
38 seconds
sast-snyk-check

✅ Succeeded
57 seconds
build-container

✅ Succeeded
9 seconds
inspect-image

✅ Succeeded
12 seconds
deprecated-base-image-check

✅ Succeeded
3 minutes
clair-scan

✅ Succeeded
48 seconds
clamav-scan

✅ Succeeded
8 seconds
sbom-json-check

✅ Succeeded
6 seconds
label-check

✅ Succeeded
14 seconds
show-summary

✅ Succeeded
17 seconds
show-sbom

Pipelines as Code CI has failed.There was an error creating the PipelineRun: test-component-pac-koag-on-pull-request-
creating pipelinerun test-component-pac-koag-on-pull-request- in namespace build-e2e-nnqq-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI/test-component-pac-koag-on-pull-request has successfully validated your commit.


Update HttpStreamServerImpl.java

24bits is better tha 16 bits of audio and would offer vinyl less loss

@aschober Any chance you can merge this + release a new version here and an updated Android build on the Google Play Store?
https://play.google.com/store/apps/details?id=tech.schober.vinylcast.playstore
@spiketail34 If not and this project is abandoned then any chance you could release an updated fork under a different name?
Myself and others would love an updated version released on the Google Play Store, and preferably as an improved variant! 😉


Eng workflows sync and branch cleanup additions

Changes so we can sync .github/workflows the same way we're doing with eng/common.

eng/pipelines/eng-workflows-sync.yml - similar to eng/pipelines/eng-common-sync.yml except this does the .github/workflows directory. The common code was already factored out into a template, eng/pipelines/templates/stages/archetype-sdk-tool-repo-sync.yml as part of this PR. Once this has been merged a pipeline will need to be created from this.
eng/pipelines/branch-cleanup.yml - updated to add Azure/azure-sdk to the list of repositories and a second task was added to clean up "sync-.github/workflows" branches.
doc/workflows/engsys_workflows.md - similar to doc/common/common_engsys.md, which describes the purpose and workflow of updating the .github/workflows directory.
eng/common/pipelines/templates/steps/eng-common-workflow-enforcer.yml - It wasn't clear if we wanted a second enforcer or another if statement in the current one. I went with the if statement which shouldn't require anything else in the target repositories.
eng/common/scripts/Delete-RemoteBranches.ps1 - The only actual change here is an additional comment on line 13 to add the regex for sync-.github/workflows work.

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow


"Stellar network error" for any attempt to send XLM

Multiple attempts to send XLM to an external wallet from the Windows Keybase client are failing with the generic message "Stellar network error".
Reddit thread showing at least some other Keybase users are experiencing this issue: https://www.reddit.com/r/Stellar/comments/q6c806/is_keybase_broke/

Verified this also happens in the most recent Android client. Stellar.org's network status doesn't show any indication of significant or consistent errors. Possibly this due to Stellar's recent update to Protocol 18.

I am also experiencing this issue. I was able to send a small amount and confirmed receipt. When I open that transaction on stellar expert, I can see it was successful, but keybase thinks it failed due to stellar network error and my balance remains unchanged on keybase. After that, any transaction I have attempted failed in the same way where keybase has "stellar network error". When I open the transaction on stellar expert, the error message is "Error: Transaction not found on Stellar Network."

@katbow described exactly the scenario which I've also experienced.

Perhaps this has to do with the Protocol 18 changes. That article notes the upgrade should have been completed by Nov 3, which is the date I started seeing issues.
I don't see any commits indicating the Stellar SDK was updated.

Managed to save my XLM by using https://stellarport.io to transfer it.

@wgroenewold Can you elaborate on how you did that?

Loaded my Keybase Secret Key in StellarX. Then openend my StellarX wallet in StellarPort.

Thank you so much @wgroenewold I managed to transfer succesfully!

Can confirm I'm having the same issue. For me I transferred XLM to my wallet which shows on https://stellar.expert but does not show in the balance on my keybase.

Can confirm I'm having the same issue. For me I transferred XLM to my wallet which shows on https://stellar.expert but does not show in the balance on my keybase.

That's because the Stellar network upgraded to v18 but the Keybase client didn't. Better grab your XLM while you can.

Donations can be made at:
Address: GDQP2KPQGKIHYJGXNUIYOMHARUARCA7DJT5FO2FFOOKY3B2WSQHG4W37
Memo: 750209424
<3

Hey @JonMontgo it's the same for me: my Keybase app still shows my assets, but it's because it doesn't retreive updated information from the blockchain, since it has been updated to Protocol 18, as mentioned above

Hey @wgroenewold do you recommend selling all lumens also or just grabbing them out of keybase?

@wgroenewold thank you! I'll send a few tokens your way, my friend. Would you mind elaborating more on stellarx and stellarport and why you used those as middleman services? I've never heard of them before this but they both look interesting. I was able to transfer mine directly from stellarx to coinbase, which was my original target. Thanks again!

Hey @wgroenewold do you recommend selling all lumens also or just grabbing them out of keybase?

I don't do financial advice, but still... Stellar network looks stable and in a lift, so keeping XLM is a good idea. Keybase is abandoned after Zoom aquisition so better get your coins when you still can and transfer them to another exchange.

Loaded my Keybase Secret Key in StellarX. Then openend my StellarX wallet in StellarPort. Then tranfered everything to Bitvavo.

Why the extra step with StellarX? Can't you just login to stellarport with your Keybase Secret Key and transfer from there directly (spoiler you can do that, because I just did it).

Just as a note, I experienced the same with the GUI in terms of sending XLM and actually receiving it but the GUI does not update. I'm trying to use the Keybase CLI keybase wallet list and getting  WARNING Could not load balance for "<address>" and then ERROR stellar network error after that.

I don't know about others here, but I'm presuming things have been fixed at keybase.io's end, as I've been able to deposit XLM into my Binance account from keybase now. I've not updated the client on my Linux laptop or my Android phone. Maybe this ticket can be closed @DeepMac ?

This issue has been resolved.

So I just realized this issue after not using Keybase for a while. I had over $20,000 USD in my keybase.io account. I really really hate crypto at this point because it's a constant bait and switch and scam to invalidate funds.
Anyone know the latest way to recover this?  I have my steller address and private key, where do I go in stellerport to import the private key and everything?


Missing step

There is a step missing in the installation phase. Azure currently asks you to set a service principle and to chose a Cloud Agent Model (ACI or VM).

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

ID: 6c50161e-7d27-2c8d-8be9-7e48aec40a06
Version Independent ID: b6443dd4-1018-e9d1-8c9c-e3d32615690a
Content: Create a Jenkins server on Azure | Microsoft Docs
Content Source: articles/jenkins/install-jenkins-solution-template.md
Service: multiple
GitHub Login: @mlearned
Microsoft Alias: mlearned

@TomArcher can you look at this one?

@joeshepherdmba This was just assigned to me. Give me a day or two to look at this. Thanks much for your contribution!

@mike-urnun-msft Could you assign this to me.

@joeshepherdmba I believe this is now addressed in that article in Step #9. Please let me know if you need anything further. Thanks for the feedback!

#please-closes


COMP: Add template to TransformPhysicalPointToContinuousIndex in tests

Added missing template keywords to TransformPhysicalPointToContinuousIndex calls in Nonunit/Review tests.
Addressed compile errors from Ubuntu-22.04-gcc11.2-TBB-Debug saying:

error: expected primary-expression before '>' token

These compile errors were introduced by pull request https://github.com/InsightSoftwareConsortium/ITK/pull/4000 commit 7cda5badd0ae7802103421b6625e46bc956f6cb9
"COMP: Fix TransformPhysicalPointToContinuousIndex nodiscard warnings"

Follow-up to pull request #4004

@dzenanz Thanks for your approval. I think this PR is ready 😃


update Kubernetes solution flist

Which package/s are you suggesting this feature for?
Kubernetes
Is your feature request related to a problem? Please describe

according to this issue https://github.com/threefoldtech/tf-images/issues/292 a new image for k3s was requested with the new tag not using latest tag.

Describe the solution you'd like

update your used Kubernetes flist to be using this one : https://hub.grid.tf/tf-official-apps/threefolddev-k3s-v1.31.0.flist.md

Requires: #3320

update the flist link in k8s in gridclient

verified on devnet
Version fccc6af
for both worker and master

I faced this but can not produce it again


feat: implemented confirm modal for creating a new property

Proposed Changes
resolve #93

Implementation

Implemented confirmation modal which is displayed when the create a new property button is pressed

@aggre
Could you add a commit changing formal message to this PR?

@k3nt0w Sorry for check late! Yes, I'll commit that messages to this PR.


Pressing "z" results in blank view

Isolate v2
When the nav bar is closed and I press z I get a blank screen and am unable to exit or navigate away. Quitting and reopening fixes the problem.

Fixed in 1f5e3d7, long term this should be addressed by moving the key handler to a reducer, so the state when a key is pressed can be checked.

Cut a new release https://github.com/seenaburns/isolate/releases/tag/v2.0.1 hopefully this should fix it, but let me know if not and I can pursue the more correct solution.
Thanks for the incredibly fast, clear and reproducible bug report 🙏 it doesn't get any better than that!

Fixed! Thanks 👍


documentation: missing configuration of vue when using unlazy-img which leads built docs with 404 of images

I have read the documentation and configured @nolebase/vitepress-plugin-thumbnail-hash and @nolebase/markdown-it-unlazy-img.
After running the docs:dev command, everything works fine in the development server—both thumbnail-hash and unlazy-img functions as expected. 🐞However, after building with docs:preview, the images on the generated site continuously display as blurred thumbnails and fail to load the original images. The developer tools indicate a 404 error when trying to access the images.

I have uploaded the code to the repository branch dubug-thumbnail-unlazy
Any insights or suggestions to resolve this issue would be greatly appreciated. Thank you for your help and support in advance!😊

This part of config is missing:
https://github.com/nolebase/integrations/blob/588a44364596fa1db68246afda92793e605722cd/docs/.vitepress/config.ts#L298-L309
I think the documentation should correctly state this configuration and explain everything out. Since you are here already to help Nolebase community, you're welcome to open up a dedicated PR to add this part of the configuration into the documentation.

Close as docs has been updated.


Drawing in ACS1996 mode throws ValueError: Bad Conformer Id if molecule has no coords.

Describe the bug
Attempting to draw with ACS1996 mode a molecule that doesn't have coordinates throws a ValueError exception.  Since prepareMolBeforeDrawing=True is the default behaviour, and that includes creating 2D coords, this is undesirable.
To Reproduce
from rdkit import Chem
from rdkit.Chem.Draw import rdMolDraw2D, rdDepictor
from rdkit.Chem.Draw import IPythonConsole
from IPython.display import SVG
from rdkit import rdBase
print(rdBase.rdkitVersion)
mol = Chem.MolFromSmiles('[*:1]N[*:2]')
drawer = rdMolDraw2D.MolDraw2DSVG(-1,-1)
rdMolDraw2D.DrawMoleculeACS1996(drawer, mol)
drawer.FinishDrawing()
SVG(drawer.GetDrawingText())

Expected behavior
Coords should be created if necessary, so long as drawOptions.prepareMolBeforeDrawing is true.
Screenshots
If applicable, add screenshots to help explain your problem.
Configuration (please complete the following information):

RDKit version: 2022.09.4
OS: [e.g. Ubuntu 20.04] macOS Ventura 13.2.1 Apple Silicon
Python version (if relevant): 3.10
Are you using conda? mamba
If you are using conda, which channel did you install the rdkit from? conda-forge
If you are not using conda: how did you install the RDKit?

Additional context
I think that the cause of the problem is that ACS1996 needs the mean bond length in the molecule to calculate the pixel scale, and this occurs before the drawing code generates the coords if it needs them.
Working on a fix.

Fixed by #6113


[RHDM-1512] Update EAP version from 7.3.4 on PAM/DM 7.10.x

See https://issues.redhat.com/browse/RHDM-1512
Signed-off-by: desmax74<EMAIL_ADDRESS>Thanks for submitting your Pull Request!
Please make sure your PR meets the following requirements:

[ ] Pull Request title is properly formatted: [RHDM-XYZ] Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket
[ ] Attached commits represent units of work and are properly formatted
[ ] You have read and agreed to the Developer Certificate of Origin (DCO) (see CONTRIBUTING.md)
[ ] Every commit contains Signed-off-by: Your Name<EMAIL_ADDRESS>- use git commit -s

@spolti


Diff is not showing where the changes where

When running a diff on a large directory, the diff output does not give any hints on where to look for the new issue.
For example the below:
    error level: +0 -0 no changes
    warning level: +1 +0
      New issue "deadcode.DeadStores Value stored to 'error' is never read" (1 occurence)
    note level: +0 -0 no changes
    all levels: +1 +0

To figure out where it happened I did the following:
sarif csv -o sarif_org ~/ovs/tests/clang-analyzer-results/2023-06-27-165114-1160621-1
sarif csv -o sarif_new ~/ovs/tests/clang-analyzer-results/2023-06-27-165604-1179114-1

sort sarif_org/static_analysis_output.csv > sarif_org.txt
sort sarif_new/static_analysis_output.csv > sarif_new.txt
diff sarif_org.txt sarif_new.txt

But this is also showing line changes, etc. etc.

Thanks for fixing this!! It works like a charm ;)
error level: +0 -0 no changes
warning level: +0 +0
  Number of occurrences 40 -> 41 (+1) for issue "unix.Malloc Use of memory after it is freed"
    file:///home/echaudron/Documents/review/ovs_quick/lib/netdev.c:243
note level: +0 -0 no changes
all levels: +0 +0


Can not display the Lottie animation

I failed to display my Lottie animation who's Lottie version is 5.5.2 with the Lottie controller. I am wondering whether there exist the maximum Lottie version that is supported by the Lottie controller.

I've just tested it with a Lottie version 5.5.7 animation and it worked fine. Make sure it's a ".json" file because the ".lottie" files don't seem to work with this control. If it's ".lottie", you can open it as a ZIP file and extract the ".json" file from within the "animations" directory. Otherwise, feel free to share the animation and I can take a look.


Cancelled promises do not reject until original promise resolves

Description
When you call cancellablePromise.cancel(), the promise does not reject immediately. Instead, we wait for the original promise that was wrapped to resolve/reject and only then do we reject the promise. If calling cancel, we shouldn't care about the original promise any more, so we should just discard it and reject our cancellable promise
Also, you cannot do PromiseUtils.makeCancelable(something).then().catch() and still call cancel. You have to create the promise in 1 statement and add then/catch in another. Might be a nice thing to cleanup as well
Steps to reproduce
import { PromiseUtils } from '@deephaven/utils';

const a = new Promise((resolve, reject) => {
  setTimeout(() => resolve(), 120 * 1000); // 2 minutes until promise resolves
});

/* This syntax doesn't actually work, but it should. Cancel gets removed
const b = PromiseUtils.makeCancelable(a)
  .then(() => console.log('resolved'))
  .catch(() => console.log('rejected'));

b.cancel();
*/

const b = PromiseUtils.makeCancelable(a);
b.then(() => console.log('resolved')).catch(() => console.log('rejected'));

b.cancel();

Expected results
Logs rejected immediately
Actual results
Logs rejected after 2 minutes
Versions

Web UI Version: 0.22.2

This might be a good one to go through with Emily.

About the promise chaining, should the .then block and .catch blocks return a cancellablePromise? For now, I only addressed the immediate cancellation.

If you're talking about const a = PromiseUtils.makeCancelable(something).then(), then yes it should return a cancelable promise I think. So we would need to add a then and catch, and finally (for good measure) to the wrapped promise.
@mofojed what are your thoughts? The TS around it is correct at least so if you try to do the above chain and then call a.cancel(), it isn't allowed since a is a regular Promise.

Keep this PR for immediate cancelation.
For cancelation chaining, I thought there may be complications in the .catch place, but I may not have thought it all the way through.

How would the cleanup function work for a chained cancelable promise?
I think it's best to just require calling makeCancelable again if you want the chained result to also be cancelable...


Add useful testing information for stream encoding and packaging teams

Description
This PR documents tools available for inspecting and testing streams. This should help stream encoding and packaging teams be more autonomous in testing streams before they deliver them to our Pillarbox players.
Changes made

Enhance stream packaging documentation (extend it to encoding as well).

Checklist

[x] APIs have been properly documented (if relevant).
[x] The documentation has been updated (if relevant).
[x] New unit tests have been written (if relevant).
[x] The demo has been updated (if relevant).

@amtins I added you as a reviewer since you might have some additional insights to provide. Documentation is in the DocC format but can still be conveniently read online, though some formatting will not be applied.
Thanks in advance for your help.

@defagos we can add:

TSDuck which is a toolkit,
the DASH specification
the Supported Media for Google Cast which provides useful information for the encoding and packaging teams.

@amtins Thanks, I added TSDuck. The other tools are not really at home in the documentation of PillarboxPlayer since it only supports HLS.
The documentation is also targeting Apple platform needs specifically since PillarboxPlayer is a generic player for Apple platforms. I guess that once we have similar information for Android and the web we could gather all information as a shared consolidated page that is more specific to our company (and on which we could also mention internal tools like MMF).

@defagos my bad, I thought I was in pillarbox-documentation, misread 😅


Role based Access - New Teacher/Student Login & Reg - STILL needs fixing/testing

Created a new system for RBAC. Essentially, individuals can now use a toggle at the top of the registration page to log in as either a teacher or student, which assigns the role of either "User" or "Teacher." Upon login, the system now uses an email query to verify whether the user is a teacher or student, and proceeds accordingly.
Changes Made:

Added radio switch at the top of register.html page: can be on either the student or teacher position, but not both at once.
Added relevant css styling for the role-switch in register.css.
Updated relevant register.js code to add event listeners for the new role switch.
Updated relevant register.js code to be able to retain position of switch upon failed attempt/redirect.
Updated register.js code so that when in 'teacher' position, the grade field disappears and input for it is no longer needed for form submission.
Updated register route in auth.py so the system now requests the role attribute, and based on "teacher" or "student" assigns a new User or Teacher model.
Updated login route in auth.py so the system now uses the inputted email to determine whether the user is a teacher or student, and redirects accordingly.
Added temporary teacher redirect page (we will decide what teachers should be able to do later).

Notes/Issues:

IMPORTANT: Under models.py, the Teacher class had yrs_specialization set to Nullable=False. This was causing issues as the registration for teachers does not ask for this field, so it should be nullable in reality. I am not sure of how to migrate the database, so I would appreciate if someone could migrate the database with my new changes or show me how.
IMPORTANT: For some reason, line 68 in auth.py (role = request.form.get("role")) is not functioning as intended. I am completely unsure why this issue exists, but it needs to be fixed for the RBAC system to work and be tested.

Overall, there are two general things I wanted to ask you regarding the implementation of your RBAC model here: the mutual exclusivity of consumers being either Users or Teachers, and the design similarities of the Users & Teachers database tables.
For the former, how are you setting up the distinction between consumers within these roles? Can a consumer be both a teacher and a student, or is it an "either-or", one-or-the-other situation? Looking at your auth.py login code at lines 26 to 31, it appears as if you're going for the latter option. You attempt to query one entry in both User and Teacher with the given login email and expect one of them to be None:
# Check both User and Teacher tables
user = User.query.filter_by(email=email).first()
teacher = Teacher.query.filter_by(email=email).first()

# Determine if the email belongs to a Teacher or a User
account = teacher if teacher else user

However, if we take a look at lines 64 to 84:
role = request.form.get("role")
...
if role == "teacher":
    user_email = Teacher.query.filter_by(email=email).first()
    user_name = Teacher.query.filter_by(username=username).first()
else:
    user_email = User.query.filter_by(email=email).first()
    user_name = User.query.filter_by(username=username).first()
...
if user_name:
    flash("This username already exists.", "register_error")
    return redirect(url_for("auth.register"))
...
if user_email:
    flash("This email already exists.", "register_error")
    return redirect(url_for("auth.register"))

This code shows that when a consumer signs up under some arbitrary username A and email B, the auth system checks for duplicate usernames/emails among only other teachers when signing up as a teacher, or only other students when signing up as a student. Thus, an actor/two separate actors could accidentally sign up with the same exact email A and password B under two accounts—one student, and one teacher. Problem is, given the way the if statement at line 31 is structured, the auth system will always choose only one account role over the other, and it always has a bias towards choosing teachers. As such, when logging in with email A and password B, the system will never log into the student's account and always access the teacher's.
If you're going for a "can be both" approach and want the same email to have a teacher and student role, then we should probably add a similar radio role select button/use separate pages for login. Otherwise, you should probably check for duplicate emails/usernames within both the User and Teacher tables in lines 64  to 84, no matter the role.

For the second topic, this initially came to mind due to bugs with logging in as a Teacher; Flask-Login requires any entity inheriting UserMixin to specifically have an id attribute as its primary key (not tid), and Teacher was also missing an extra failed_signin_attempts field from User that was needed in auth.py. These fixes are pretty simple to fix by altering the Teacher table, but the User and Teacher tables have so many similarities between each other at this point that it feels inefficient and a waste of space to have them as separate tables. Wouldn't it just be better to have a main User table with shared attributes (name, hashed_password, email, etc.), then create ISA specializations for each role that inherit from the parent User table (Student having grade, points, etc; Teacher having yrs_experience, specialization, etc.)? This way, not only is our code more concise, but we wouldn't have to constantly create and manage duplicate columns between both tables taking up valuable storage.
I do understand, however, that changing this DB structure this late into the project would most likely take a bit more time in fixing any resulting bugs with queries dependent on the old User model (with Student-specific attributes), so I understand if you aren't too keen on the idea.

I have implemented all of the relevant changes recommended by Alex:

Changes to register.html, register.css, and register.js have been made exactly as recommended, with a few minor modifications.
Changes to models.py have been implemented so that teacher.tid and all of its occurrrences are replaced by teacher.id to better fit the Flask model.
In terms of auth.py, I have chosen to go with the approach of only allowing a user to either be a teacher or student. I modified registration logic to check for email and username in both Teacher and User tables to ensure no individuals have registered with these credentials in the past. I also modified the login logic to ensure there is no bias when checking for account type based on email queries.

Here are some other important things to note:

I definitely agree that creating an ISA specialization to simplify these roles would simply the logic, however I have been told that RBAC is not yet a major feature of our application, and thus it's probably best to continue as is for now.
IMPORTANT: If it is deemed that these new changes have been correctly implemented and this PR is ready for merge, please ensure to migrate the database after doing so. At the moment, changing the teacher tid to id is causing some issues with the login/registration logic. I am not well aware of the alembic system being used for migrations, so this will likely be better handled by someone else on the team.

Added changes to properly account for an error when email associated with both teacher & user.


Automatic Vagrant Plugin Installation

I was wondering if we could make the installation process for users who don't have any experience using vagrant a bit easier (reduce the number of commands needed) by automatically installing the needed Vagrant plugins?
According to this StackOverflow thread this isn't supported by default. But the thread points out a possible solution:
required_plugins = %w( vagrant-hostsupdater vagrant-someotherplugin )
required_plugins.each do |plugin|
  system "vagrant plugin install #{plugin}" unless Vagrant.has_plugin? plugin
end

@mike182uk: Since you added the plugins, what do you think about this approach?

Hey @sebgie,
Yeh i think this would be a good approach. Its probably something i should of done when i first introduced the plugins.
As for the code snippet, i don't think this is going to work as it is - by the time the code has been executed Vagrant has already bootstrapped all its plugin stuff, so despite the fact you have installed the plugins, Vagrant will not recognise them. This will mean you would have to run vagrant up a second time for Vagrant to run with the additional plugins.
Something like:
missing_plugins = false
required_plugins = %w(vagrant-cachier vagrant-hostsupdater vagrant-librarian-puppet)

required_plugins.each do |plugin|
  if !Vagrant.has_plugin? plugin
    missing_plugins = true
    system "vagrant plugin install #{plugin}"
  end
end

if missing_plugins
  exec "vagrant #{ARGV.join(" ")}"
end

should work without having to run vagrant up a second time. This will install the required plugins using system, then if it did needed to install new plugins vagrant up is re-executed using exec. exec will replace the current process (vagrant up without the additional plugins) with vagrant up with the additional plugins.

That sounds perfect. Would you like to send a PR?

Sure - https://github.com/TryGhost/Ghost-Vagrant/pull/74


boards: infineon: cyw920829m2: Fix unit and first address mismatch

This fixes the following warning:

unit address and first address in 'reg' (0x60000) don't match for
/qspi_flash@40890000/flash@60000000/partitions/storage_partition@70000

@sreeramIfx , could you take a look?

@sreeramIfx , could you take a look?

This change looks good to me..


Test automatic port forwarding for ports in URL query

Refs: https://github.com/microsoft/vscode/issues/203869

[ ] dev container @anthonykim1
[x] dev container @benibenj

Complexity: 4
Create Issue

General Setup

Connect to your remote
Set "remote.autoForwardPorts": false
Remove all forwarded ports that you already have for simpler testing
Start a server on your remote, then echo a URL with a localhost redirect to the terminal. ex: http://<IP_ADDRESS>:3005?redirect_uri=http%3A%2F%2Flocalhost%3A3000. See detailed setup for an easy way to do this.

Detailed setup

You can use the vscode-remote-try-node sample to get started quicky (command: Dev Containers: Try a Dev Container Sample...)
Update server.js to contain the following:

const express = require('express');

// Constants
const PORT = 3000;
const HOST = '<IP_ADDRESS>';

function createServer(port) {
    const app = express();
    app.get('/', (req, res) => {
        res.send('Hello remote world!\n');
    });

    app.listen(port, HOST);
}

createServer(PORT);
const port2 = PORT + 5;
createServer(port2);
console.log(`Running on http://${HOST}:${port2}?redirect_uri=http%3A%2F%2Flocalhost%3A${PORT}`);

npm run start
Click on the URL that gets logged to the terminal

Verify

That you see both ports get forwarded in the Ports view

Looks correct to me, but let me know if I'm missing anything! Nice work.

Looks correct, thankn you for testing!


to support python3

In Python 2 byte format  is treated as string
but In python version 3 .. they are separate
and while running redisgraph .. data elements are in bytes format in the code and
python is thowing error with commit command  as
TypeError: a bytes-like object is required, not 'str'
so i think we  need to use
result_set = [res.decode().split(',') for res in data]
to make it compatible with python version 3 instead of
result_set = [res.split(',') for res in data]

Thank you


how to implements using Hashtag and Bold text patterns together?

i added this pattern to handle hashtags: // #HASHTAG
{
  pattern: /#[^\s!@#$%^&*()=+./,\[{\]};:'"?><،؛؟-]+/gi,
  style: [_linkStyle, linkStyle, boldFont],
  onPress: handleHashtagPress,
}

and added this pattern to handle bold text: // **TEXT TO BOLD**
{
  pattern: /(?:\*\*)([^*]*)(?:\*\*)/,
  style: boldStyle,
  renderText: (matchingString, matches) => matches[1],
},

example: **test bold and #hashtag in text**
when using these together, bold text pattern not works. when change priority of patterns, hashtags not works, how can i use both patterns to support hashtag and bold text?

can you please write an example? #81 not works for me!

can you please write an example base on my question options? #81 not works for me!

The point of #81 is: It doesn't work. I'm waiting for a fix too.


AEAD encrypt ca.key at rest with a user supplied passhprase

Would be nice if PKCS#11 was also supported

Agreed! We want to support it for host keys as well, for things like yubikey+laptop.

💯 yubikey is exactly what I had in mind.


Error deleting subnet serviceAssociationLinks AppServiceLink

When trying to delete a subnet in Azure, I get the following message: Failed to save subnet 'devops-subnet'. Error: 'Subnet GumSite-rg-devops/providers/Microsoft.Network/virtualNetworks/Devopsvnet/subnets/devops-subnet'>Devopsvnet/devops-subnet requires any of the following delegation(s) [Microsoft.Web/serverFarms] to reference service association link /subscriptions/**************/resourceGroups/GumSite-rg-devops/providers/Microsoft.Network/virtualNetworks/Devopsvnet/subnets/devops-subnet/serviceAssociationLinks/AppServiceLink.'

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

 In order to best address your question, could you provide us with the URL of the MS Doc that you were following, if any?

Here a Microsoft article "Unable to delete VNet due to serviceAssociationLinks/AppServiceLink" that resembles my situation:
https://social.msdn.microsoft.com/Forums/en-US/f3fa0fb2-d930-484c-90a5-6860e360d87f/unable-to-delete-vnet-due-to-serviceassociationlinksappservicelink?forum=WAVirtualMachinesVirtualNetwork

Here's an article that resembles my situation :Unable to delete VNet due to serviceAssociationLinks/AppServiceLink : https://social.msdn.microsoft.com/Forums/en-US/f3fa0fb2-d930-484c-90a5-6860e360d87f/unable-to-delete-vnet-due-to-serviceassociationlinksappservicelink?forum=WAVirtualMachinesVirtualNetwork

Here's a link to an article that resembles my situation:
https://social.msdn.microsoft.com/Forums/en-US/f3fa0fb2-d930-484c-90a5-6860e360d87f/unable-to-delete-vnet-due-to-serviceassociationlinksappservicelink?forum=WAVirtualMachinesVirtualNetwork
Unable to delete VNet due to serviceAssociationLinks/AppServiceLink

@yvesgermain Thanks for the response. I've routed this to the appropriate Team to investigate and update.

@yvesgermain This is a known issue, and the only way you will be able to get this deleted is to contact Azure Support. If you do not have a support plan, please Email me at<EMAIL_ADDRESS>with your subscription ID and a link to this post, and I will enable a one-time free support request for you to get the subnet deleted.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

Seem to have a related issue, i.e., when I try to delete a subnet, I get an error:
Failed to delete subnet 'xxx'. Error: Subnet xxx is in use by my-resource-group/providers/Microsoft.Network/virtualNetworks/my-vnet/subnets/xxx/serviceAssociationLinks/AppServiceLink'>my-vnet/xxx/AppServiceLink and cannot be deleted. In order to delete the subnet, delete all the resources within the subnet. See aka.ms/deletesubnet.
Is this also something that can only be fixed by making a support call?

@sergevm If you have deleted the App Service and the link, yet the subnet is not able to be deleted, you will need to create a support request.
If you do not have a support plan, please Email me at<EMAIL_ADDRESS>with your subscription ID and a link to this post, and I will enable a one-time free support request for you to get the subnet deleted.

Any idea when this bug is going to be fixed?

I had the same issue. I had the App Service connected to the subnet. I removed the App Service Plan together with App Service which broke my VNet and did not allow to remove the associated subnet. In order to fix the solution:

Create App Service Plan with the same name as the deleted one
Create App Service with the same name as the deleted one
Link App Service with the VNET subnet
Disconnect VNet from App Service -> Networking -> VNet integration -> Disconnect
Delete subnet

@blendsdk this might probably help you

@sergevm If you have deleted the App Service and the link, yet the subnet is not able to be deleted, you will need to create a support request.
If you do not have a support plan, please Email me at<EMAIL_ADDRESS>with your subscription ID and a link to this post, and I will enable a one-time free support request for you to get the subnet deleted.

I sent a message to<EMAIL_ADDRESS>but no-one seems to respond.

@frehnejc I created a support request in the Azure portal, and this was fixed by support soon after. Did not have to send a mail.

@frehnejc Thanks for following up! I was able to find your email, and have enabled your subscription for a free support request.

@frehnejc Thanks for following up! I was able to find your email, and have enabled your subscription for a free support request.

Thanks also have issues with Group IP’s unable to delete when there was an azure firewall can I log a ticket on that too?

@frehnejc You can, but it will be easier to ask the engineer to take care of that with the same support request. If the engineer asks you to make a second ticket and you are unable, reply to my Email and I will enable another support request if needed.

i ran into this with container instances and here is what i did to get this deleted when my container instance was removed and i couldn't remove the delegation.
using the az cli with a logged in user to obtain a jwt token
az account get-access-token --resource https://management.azure.com/

then using postman to issue the following calls:
add to the headers: "Authorization: Bearer |accesstoken from cli|"
DELETE  Service Association
 https://management.azure.com/subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Network/virtualNetworks/{virtualNetworkName}/subnets/{subnetName}/providers/Microsoft.ContainerInstance/serviceAssociationLinks/default?api-version=2018-10-01

DELETE Network Profle
 https://management.azure.com/subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Network/networkProfiles/{networkProfileName}?api-version=2020-05-01

after this i could remove the delegation using the portal, cli or powershell

In case of an app service, you need to clean up the association under App Service Plan > Networking which will list all the links your net has to any of the apps including slots.
Once you disconnect the vnet from app service and slot you should be then able to set the delegation back to None and unselect the service endpoint and finally be able to delete the subnet respectively.

@dgcaron this worked! Thanks a lot.

@dgcaron,
With your help I was able to remove the VNet using PowerShell.
Thank you very much!
[string]$ResourceGroupName = 'MyResourceGroupName'
[string]$VirtualNetworkName = 'VNetName'
[string]$SubnetName = 'SubnetName'

$CurrentAzureContext = Get-AzContext
$AzureRmProfile = [Microsoft.Azure.Commands.Common.Authentication.Abstractions.AzureRmProfileProvider]::Instance.Profile
$ProfileClient = New-Object Microsoft.Azure.Commands.ResourceManager.Common.RMProfileClient($AzureRmProfile)
$Token = $ProfileClient.AcquireAccessToken($CurrentAzureContext.Tenant.TenantId)
[string]$AccessToken = $Token.AccessToken
[string]$ManagementEp = $CurrentAzureContext.Environment.ResourceManagerUrl

[string]$NetworkProfileName = (Get-AzNetworkProfile -ResourceGroupName $ResourceGroupName).Name

[string]$Uri1 = "$($ManagementEp)subscriptions/$($CurrentAzureContext.Subscription.Id)/resourceGroups/$ResourceGroupName/providers/Microsoft.Network/virtualNetworks/$VirtualNetworkName/subnets/$SubnetName/providers/Microsoft.ContainerInstance/serviceAssociationLinks/default?api-version=2018-10-01"
[string]$Uri2 = "$($ManagementEp)subscriptions/$($CurrentAzureContext.Subscription.Id)/resourceGroups/$ResourceGroupName/providers/Microsoft.Network/networkProfiles/$($NetworkProfileName)?api-version=2020-05-01"

Invoke-RestMethod -Method Delete -Uri $Uri1 -Headers @{"Authorization" = ("Bearer " + $AccessToken) }

Invoke-RestMethod -Method Delete -Uri $Uri2 -Headers @{"Authorization" = ("Bearer " + $AccessToken) }

I had the same issue. I had the App Service connected to the subnet. I removed the App Service Plan together with App Service which broke my VNet and did not allow to remove the associated subnet. In order to fix the solution:

Create App Service Plan with the same name as the deleted one
Create App Service with the same name as the deleted one
Link App Service with the VNET subnet
Disconnect VNet from App Service -> Networking -> VNet integration -> Disconnect
Delete subnet

@blendsdk this might probably help you

@ybutkevych this worked for me! Thanks a lot.

I had the same issue. I had the App Service connected to the subnet. I removed the App Service Plan together with App Service which broke my VNet and did not allow to remove the associated subnet. In order to fix the solution:

Create App Service Plan with the same name as the deleted one
Create App Service with the same name as the deleted one
Link App Service with the VNET subnet
Disconnect VNet from App Service -> Networking -> VNet integration -> Disconnect
Delete subnet

@blendsdk this might probably help you

Whoever is experiencing the same issue in the future, follow these steps, I just tried them out and it fixed my problem.

@dgcaron, thanks for doing the API docs digging :) Here's a bash-script for the cleanup:
#!/bin/bash
# Set variables with your identifiers...
subscriptionId=""
resourceGroupName=""
virtualNetworkName=""
subnetName=""

# To get the profile name, try subnet delete with CLI or portal and check the error message
# Should be something like this:
networkProfileName="aci-network-profile-vnet-foo-01-subnet-bar-01"

token=$(az account get-access-token --resource https://management.azure.com/ --query accessToken -o tsv)
http DELETE https://management.azure.com/subscriptions/${subscriptionId}/resourceGroups/${resourceGroupName}/providers/Microsoft.Network/virtualNetworks/${virtualNetworkName}/subnets/${subnetName}/providers/Microsoft.ContainerInstance/serviceAssociationLinks/default?api-version=2018-10-01 \
"Authorization: Bearer ${token}"

http DELETE https://management.azure.com/subscriptions/${subscriptionId}/resourceGroups/${resourceGroupName}/providers/Microsoft.Network/networkProfiles/${networkProfileName}?api-version=2020-05-01 \
"Authorization: Bearer ${token}"

@dgcaron Thank you very much! Due to your help I was able to remove the service delegation.
@jannekeskitalo  @boonwj
I think the simplest solution is to just use az rest like so:
az rest --method delete --uri https://management.azure.com/subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Network/virtualNetworks/{virtualNetworkName}/subnets/{subnetName}/providers/Microsoft.ContainerInstance/serviceAssociationLinks/default?api-version=2018-10-01

i ran into this with container instances and here is what i did to get this deleted when my container instance was removed and i couldn't remove the delegation.
using the az cli with a logged in user to obtain a jwt token
az account get-access-token --resource https://management.azure.com/

then using postman to issue the following calls:
add to the headers: "Authorization: Bearer |accesstoken from cli|"
DELETE  Service Association
 https://management.azure.com/subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Network/virtualNetworks/{virtualNetworkName}/subnets/{subnetName}/providers/Microsoft.ContainerInstance/serviceAssociationLinks/default?api-version=2018-10-01

DELETE Network Profle
 https://management.azure.com/subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Network/networkProfiles/{networkProfileName}?api-version=2020-05-01

after this i could remove the delegation using the portal, cli or powershell

This worked, thanks!

@TravisCragg-MSFT would it be possible to get this fixed properly instead of forcing all these people to contact support?

The az cli command doesn't seem to work for AppServiceLinks. It runs successfully, but the link does not delete.
Can we get an actual fix for this? If you don't provide a way to disconnect the vnet in the case where an app service and the ASP are deleted, then you are likely going to be generating quite a few support tickets for us in non-production environments where deleting entire resource groups and deploying again is more common.

Why is this closed? This is an existing bug.

Still broken!

Still broken!

That helped me.

@yvesgermain This is a known issue, and the only way you will be able to get this deleted is to contact Azure Support. If you do not have a support plan, please Email me at<EMAIL_ADDRESS>with your subscription ID and a link to this post, and I will enable a one-time free support request for you to get the subnet deleted.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

@TravisCragg-MSFT Can I get some help with the same issue?

I hit this issue testing an ARM template deployment against my personal subscription. After deleting the resource group I was left only with the VNet, Subnet and AppServiceLinks, which I couldn't see - They really should be exposed better.
Following @ybutkevych 's approach worked, but this needed doing per connected resource, which is pretty droll.
As I'd already deleted the resource group, none of the connected services existed anymore, apart from the AppServiceLinks. To workaround this, I re-deployed the ARM template against the resource, disconnected them from the subnet, and then I was able to delete the subnet and subsequently the vnet.
I did try contacting the support email above, but as yet have had no response - this isn't a good model for resolving this issue.

I had the same issue. I had the App Service connected to the subnet. I removed the App Service Plan together with App Service which broke my VNet and did not allow to remove the associated subnet. In order to fix the solution:

Create App Service Plan with the same name as the deleted one
Create App Service with the same name as the deleted one
Link App Service with the VNET subnet
Disconnect VNet from App Service -> Networking -> VNet integration -> Disconnect
Delete subnet

@blendsdk this might probably help you

i ran into this with container instances and here is what i did to get this deleted when my container instance was removed and i couldn't remove the delegation.
using the az cli with a logged in user to obtain a jwt token
az account get-access-token --resource https://management.azure.com/

then using postman to issue the following calls:
add to the headers: "Authorization: Bearer |accesstoken from cli|"
DELETE  Service Association
 https://management.azure.com/subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Network/virtualNetworks/{virtualNetworkName}/subnets/{subnetName}/providers/Microsoft.ContainerInstance/serviceAssociationLinks/default?api-version=2018-10-01

DELETE Network Profle
 https://management.azure.com/subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Network/networkProfiles/{networkProfileName}?api-version=2020-05-01

after this i could remove the delegation using the portal, cli or powershell

I've tried this but get the error "UnauthorizedClientApplication", seems like they are blocking any clients other than the portal app for this API?

default?api-version=2018-10-01

This did not work for me.

Is there anyway to delete it without opening a ticket to Microsoft?

The only way I could solve this is by opening a ticket to Microsoft. They had to remove it on their end.

https://github.com/Azure/azure-cli/issues/21637 should be re-opened to fix this in AZ-CLI

i ran into this with container instances and here is what i did to get this deleted when my container instance was removed and i couldn't remove the delegation.
using the az cli with a logged in user to obtain a jwt token
az account get-access-token --resource https://management.azure.com/

then using postman to issue the following calls:
add to the headers: "Authorization: Bearer |accesstoken from cli|"
DELETE  Service Association
 https://management.azure.com/subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Network/virtualNetworks/{virtualNetworkName}/subnets/{subnetName}/providers/Microsoft.ContainerInstance/serviceAssociationLinks/default?api-version=2018-10-01

DELETE Network Profle
 https://management.azure.com/subscriptions/{subscriptionId}/resourceGroups/{resourceGroupName}/providers/Microsoft.Network/networkProfiles/{networkProfileName}?api-version=2020-05-01

after this i could remove the delegation using the portal, cli or powershell

I've tried this but get the error "UnauthorizedClientApplication", seems like they are blocking any clients other than the portal app for this API?

when run it:
$token=$(az account get-access-token --resource https://management.azure.com/ --query accessToken -o tsv)

Invoke-RestMethod -Method Delete -Uri https://management.azure.com/subscriptions/${subscriptionId}/resourceGroups/${resourceGroupName}/providers/Microsoft.Network/virtualNetworks/${virtualNetworkName}/subnets/${subnetName}/providers/Microsoft.ContainerInstance/serviceAssociationLinks/default?api-version=2018-10-01 -Headers @{"Authorization" = ("Bearer " + $token) }

return the error:
Invoke-RestMethod : {"error":{"code":"InvalidAuthenticationToken","message":"The received access token is not valid: at least one of the claims 'puid' or 'altsecid' or 'oid' should be
present. If you are accessing as application please make sure service principal is properly created in the tenant."}}
At line:1 char:1
+ Invoke-RestMethod -Method Delete -Uri https://management.azure.com/su ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : InvalidOperation: (System.Net.HttpWebRequest:HttpWebRequest) [Invoke-RestMethod], WebException
    + FullyQualifiedErrorId : WebCmdletWebResponseException,Microsoft.PowerShell.Commands.InvokeRestMethodCommand

in the case of App Services make sure that any slots you have also have been disconnected

Same issue.
 {
    "error": {
        "code": "InUseSubnetCannotBeDeleted",
        "message": "Subnet snet-xxx is in use by /subscriptions/xxx/resourceGroups/rg-xxx/providers/Microsoft.Network/virtualNetworks/vnet-xxx/subnets/snet-xxx/serviceAssociationLinks/AppServiceLink and cannot be deleted. In order to delete the subnet, delete all the resources within the subnet. See aka.ms/deletesubnet.",
        "details": []
    }
}

I cannot delete the subnet / vNet as there I have an orphaned serviceAssociationLink. I checked the rest API documentation and the only operation available is list.
The only way to resolve this seems to be to open a support ticket which I can't do as I have a basic support plan.
I can't believe this has been open with no fix since early 2020!

I ran into exactly this issue as well; this issue is definitely not fixed and took almost a week to get resolved. While Microsoft support was responsive and attempted to be helpful, it took the involvement of three different groups to get the AppServiceLink deleted. I'm in the process of doing devops development for AKS using Pulumi, and so am creating and tearing down infrastructure multiple times per day. It took a while before I triggered this bug (maybe a race condition?) but it makes development and testing with any infrastructure-as-code solution unworkable, so I will probably need to be looking at other platforms for my application. This really needs to get fixed.

This bug is still present

I'm getting this error as well.  I had an Web app + Database and didn't realize that I had to disconnect the subnet from the vnet before deleting the resource group.  Not I can't delete the subnet or vnet.  I've tried recreating but it either fails because the resource already exists or unable to assign the subnet due to it being assigned already.  Any chance of getting a one-time support ticket?

This is happening for me as well tried the last couple of days using cli powershell and resources.azure com and no joy. I have a basic plan support and looks like more people will have the issue of not able to purge vnet resources in their tenant or subscription. I also agree  Any chance of getting a one-time support ticket? this is a bug that needs to be fixed on the backend

@teedeeuk17 @tigertiburon
please try below methods, it worked for me.
Deleting the App Service plan or web app before disconnecting the VNet integration
If you deleted the web app or the App Service plan without disconnecting the VNet integration first, you will not be able to do any update/delete operations on the virtual network or subnet that was used for the integration with the deleted resource. A subnet delegation 'Microsoft.Web/serverFarms' will remain assigned to your subnet and will prevent the update/delete operations.
In order to update/delete the subnet or virtual network again you need to re-create the VNet integration and then disconnect it:
Re-create the App Service plan and web app (it is mandatory to use the exact same web app name as before).
Navigate to the 'Networking' blade on the web app and configure the VNet integration.
After the VNet integration is configured, select the 'Disconnect' button.
Delete the App Service plan or web app.
Update/Delete the subnet or virtual network.

I had the same issue. I had the App Service connected to the subnet. I removed the App Service Plan together with App Service which broke my VNet and did not allow to remove the associated subnet. In order to fix the solution:

Create App Service Plan with the same name as the deleted one
Create App Service with the same name as the deleted one
Link App Service with the VNET subnet
Disconnect VNet from App Service -> Networking -> VNet integration -> Disconnect
Delete subnet

@blendsdk this might probably help you

Hi what will happen if i do not know exact name of App Service Plan, App services etc. In this case how can I delete subnet?

in the case of App Services make sure that any slots you have also have been disconnected

Thanks, this fixed the problem for me.

This needs to be re-opened until Microsoft can properly fix the process on the backend.

This bug has been around for how many years now?
Agree with above -- should stay open until the issue is resolved.

This is still an issue

Still an issue with Microsoft.ContainerInstance delegation. Workaround via REST does not work (anymore):
> az rest --method delete --uri https://management.azure.com/subscriptions/*****/resourceGroups/*****/providers/Microsoft.Network/virtualNetworks/*****/subnets/*****/providers/Microsoft.ContainerInstance/serviceAssociationLinks/default?api-version=2018-10-01
Internal Server Error({"error":{"code":"InternalServerError","message":"Encountered an internal server error. The tracking activity id is '*****', correlation id is '*****'."}})

This is still an issue with "Microsoft.PowerPlatform/vnetaccesslinks"
For Fabric Vnet Data Gateways
not sure why this is all closed down

Yup, bug is still there. Can't delete the subnet because of the broken serviceAssociationLinks/PowerPlatformSAL.


Change gRPC client service and method descriptor inner Config class name

The gRPC client module's ClientServiceDescriptor and ClientMethodDescriptor classes both have an inner interface called Config. This inner interface's name needs to be changed to Rules.
ClientServiceDescriptor.Config ---> ClientServiceDescriptor.Rules
ClientMethodDescriptor.Config ---> ClientMethodDescriptor.Rules
NOTE: For completeness (and tidiness and elegance) make sure that you change all of the relevant field and parameter names where these classes are used from config to rules.
This work should be done on a branch and merged into the grpc-client branch.

Created Pull request: https://github.com/aseovic/helidon/pull/86

Closed by #86


feat: Lambda VPC Endpoint

Description
AWS recently announced PrivateLink support for Lambda. This PR serves to add the functionality to this module.
Motivation and Context
So the new endpoint can be used by projects already integrated with this module.
Breaking Changes
None.
How Has This Been Tested?
Locally.

Thanks @aiell0 !
v2.63.0 has been just released.


configuration/http_conn_man/http_conn_man.md

原文：https://github.com/servicemesher/envoy/tree/master/configuration/http_conn_man/http_conn_man.md


Fix Python SDK, broken with 0.3.0 package release

Client is now called ComputerVisionClient (was ComputerVisionAPI).
Client takes an endpoint, not a region.
RecognizeText sample was tweaked to work out of the box.
Cleaned up a few cases of trailing whitespace.

@cthrash : Thanks for your contribution! The author, @PatrickFarley, has been notified to review your proposed change.

@PRMerger12 I've approved, but the "Change sent to author" label is still present

Author in this case means author of document?  Or author of PR?

#sign-off
We'll see if that does anything

Approved


Add link to the GitHub Release Planning project

Action item from the face to face

LGTM

LGTM

LGTM

LGTM


Failed to compile a object which have a *mut Option

Hi, I tried to compile a generated file that import windows::win32::intl::*.
However, I found the following errors.
error[E0599]: no method named `map` found for raw pointer `*mut Option<extern "system" fn(*mut UCharIterator, UCharIteratorOrigin) -> i32>` in the current scope
   --> E:\issue\windows-rs-issue\target\debug\build\windows-rs-issue-ea675fe212ad20f8\out/windows.rs:359:43
    |
359 |                         && self.get_index.map(|f| f as usize) == other.get_index.map(|f| f as usize)
    |                                           ^^^ method not found in `*mut Option<extern "system" fn(*mut UCharIterator, UCharIteratorOrigin) -> i32>`
    |
    = note: try using `<*const T>::as_ref()` to get a reference to the type behind the pointer: https://doc.rust-lang.org/std/primitive.pointer.html#method.as_ref
    = note: using `<*const T>::as_ref()` on a pointer which is unaligned or points to invalid or uninitialized memory is undefined behavior
    = note: the method `map` exists but the following trait bounds were not satisfied:
            `*mut Option<extern "system" fn(*mut UCharIterator, UCharIteratorOrigin) -> i32>: Iterator`
            which is required by `&mut *mut Option<extern "system" fn(*mut UCharIterator, UCharIteratorOrigin) -> i32>: Iterator`

20 similar errors in bindings::windows::win32::intl::UCharIterator.
Also I found similar errors in bindings::windows::win32::intl::UTextFuncs and bindings::windows::win32::rpc::NDR_CS_ROUTINES.
The minimal code with UCharIterator:

Rust version: 1.49.0

build.rs

fn main() {
    windows::build!(
        windows::win32::intl::{
            UCharIterator,
        }
    );
}

main.rs

mod bindings {
    ::windows::include_bindings!();
}

fn main() {
}

This is a known issue with the metadata we're consuming. https://github.com/microsoft/win32metadata/issues/132 cc @sotteson1

This is a known issue with the metadata we're consuming. https://github.com/microsoft/win32metadata/issues/132 cc @sotteson1


grpc: close stream with data frame

Description: the gRPC protocol requires closing the client stream in a DATA frame. This PR fixes the GRPCStreamEmitter for iOS.
Risk Level: med - protocol level changes.
Testing: local.
Signed-off-by: Jose Nino<EMAIL_ADDRESS>

@buildbreaker @rebello95 I updated with @goaway's suggestion. lmk what you think

Let's make the appropriate changes on Android as well as part of this PR


Update renewed keys and certificates

Fix https://github.com/wso2/api-manager/issues/1492
Related https://github.com/wso2/carbon-kernel/pull/3523

integration tests passed locally with update kernel 4.8.0 and respective carbon mediation and carbon-apimgt latest versions.


[LLVMGPU] Generalize VectorContractOpInfo based on indexing maps

This patch generalizes VectorContractOpInfo to work on any kind of vector.contract. The "kind" field was not being used by any pass anyway and they were relying on m, n, k dims.

Comment for documentation, this should fix: https://github.com/iree-org/iree/issues/17585 :)


[TT-1252] Create dispatch and wait for workflow job actions

nice! were you able to test these @tateexon ?

@momentmaker

nice! were you able to test these @tateexon ?

Currently using in migration PRs for chainlink-integrations: https://github.com/smartcontractkit/chainlink-integrations/pull/3


Add openshift apiserver pod disruption budget

This commit adds a pod disruption budget to the openshift apiserver
so that a minimum of 1 replica is maintained when the control plane
is configured to be highly available.

/retest


Cursor position

Now, if traveling around blocks, cursor always stops on empty line.
It would be great if cursor stops at beginning / ending of the block.
Like this: https://atom.io/packages/space-block-jumper
Thank you!

@Michal-Mikolas sounds like a decent idea to have that be available as a configuration option for the package. Are you up to the task of implementing that and submitting a PR?


Error on reapp new -> reapp run

Error after creating a new app and running it based on the docs. I've gone to http://localhost:3010/ in the browser, too, and it just is blank with the error-
Failed to load resource: the server responded with a status of 404 (Not Found)
in the browser's console.
$ reapp run                                                                                [2/4]
Your app is running on http://localhost:3010

Building with Webpack...
Hash: e779570c713250e7b167
Version: webpack 1.12.9
Time: 406ms
  Asset     Size  Chunks       Chunk Names
main.js  21.5 kB       0       main
chunk    {0} main.js (main) 52 bytes [rendered]
    [0] multi main 52 bytes {0} [built] [3 errors]

ERROR in multi main
Module not found: Error: Cannot resolve module 'webpack-dev-server/client' in /Users/tejas/Repos/tejasmanohar/weathr
 @ multi main

ERROR in multi main
Module not found: Error: Cannot resolve module 'webpack/hot/only-dev-server' in /Users/tejas/Repos/tejasmanohar/weathr
 @ multi main

ERROR in multi main
Module not found: Error: Cannot resolve module 'babel-loader' in /Users/tejas/Repos/tejasmanohar/weathr
 @ multi main
webpack: bundle is now VALID.

Having the same issue on Windows 10. I'll try with Ubuntu and get back to you.

@Unforgiven-wanda This is BSD / OS X

working / stuck on this, OSX.

Is this working with new version of reapp?
Do:
npm uninstall -g reapp
npm install -g reapp
reapp new test-reapp
cd test-reapp
reapp run

Came here to report this
OSX (El Capitan 10.11.4), trying out reapp for the first time (no previous install).
Tried uninstalling and re-installing
reapp run
Your app is running on http://localhost:3010

Building with Webpack...
Hash: e779570c713250e7b167
Version: webpack 1.13.0
Time: 557ms
  Asset     Size  Chunks       Chunk Names
main.js  22.7 kB       0       main
chunk    {0} main.js (main) 52 bytes [rendered]
    [0] multi main 52 bytes {0} [built] [3 errors]

ERROR in multi main
Module not found: Error: Cannot resolve module 'webpack-dev-server/client' in /Users/jmathews/tmp/test-reapp
 @ multi main

ERROR in multi main
Module not found: Error: Cannot resolve module 'webpack/hot/only-dev-server' in /Users/jmathews/tmp/test-reapp
 @ multi main

ERROR in multi main
Module not found: Error: Cannot resolve module 'babel-loader' in /Users/jmathews/tmp/test-reapp
 @ multi main
webpack: bundle is now VALID.

Build took 0.557 seconds

Get a 404 in the browser
GET http://localhost:3011/main.js

As per #97 tried
npm i reapp-pack --save
But no joy (node lts 4.4.4, npm version 2.15.1)
Updating to latest stable node (using n, node 6.0.0, npm version 3.8.6), and running the npm command again resolved the immediate issue, but leaves warnings.
WARNING in ./app/app.js
Critical dependencies:
7:22-29 require function is used in a way in which dependencies cannot be statically extracted
 @ ./app/app.js 7:22-29
webpack: bundle is now VALID.

Note: you have a require warning on your routes, this is ok!

Dropping back to lts 4.4.4 release and still works as per above (with warnings)


move to new Heroku SSL domain

This is necessary to actually use the admin.

Coverage remained the same at 92.43% when pulling 80a540adbb29961f2076de029d7e09eb088291df on fix-admin-redirect into 8eb3bbb6c28b9edb78c00cb3be555feaedd29b98 on master.


Update: valid_palindrome_ii

Codecov Report

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

@@          Coverage Diff          @@
##           master   #127   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          55     55
  Lines         638    637    -1
=====================================
- Hits          638    637    -1

Impacted Files
Coverage Δ

...roblems/valid-palindrome-ii/valid_palindrome_ii.go
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Make it mobile responsive

@Sukhendu2002 hi there, please assign this to me
thanks

@Sky-De , assigned

@Sukhendu2002 May I ask for your assistance in running this app on my local machine by providing the necessary identification .env variables to contribute to the project? Thank you.

@Sky-De , please refer to the Readme
for the client env, you need to create the following accounts and put the tokens in the env
https://clerk.com/
https://www.pinata.cloud/
and for the server
https://www.mongodb.com/
if you need any assistance feel free to comment


using --ratio in macs2 callpeak to do spike-in normalization?

Hi,
I noticed there is a similar question on how to incorporate spike-in normalization into macs2 peak calling in #356 which recommends doing subcommands.
However, I saw that macs2 callpeak has --ratio option and am wondering whether this is a new feature that enable us to specify spike-in normalization via --ratio instead of doing the subcommands?
Thanks in advance!

@olechnwin NO! Don't use that option. I checked the changelog and it was mentioned almost 6 years ago. In fact, this option is not effective in any of the current versions of MACS2. I will make sure it is removed in the next release!

@taoliu
Thank you so much for the reply!! Really appreciate it!

--ratio has been removed in the MACS2 code. The change will be included in the next release v2.2.7.


Upgrade kustomize version to 5.1.1 from 3.10.0

Why are these changes needed?
Before this change the version of Kustomize used in Kuberay was 3.10.0 This version has bugs while installing with go install. This PR upgrades kustomize version to 5.1.1 which does not have the issue.

Related issue number

Closes #1368
Checks

[ ] I've made sure the tests are passing.
Testing Strategy

[ ] Unit tests
[x] Manual tests
[ ] This PR is not tested :(

@kevin85421 This is the PR for 1368. I don't seem to have access to assignees section to add you as the reviewer. Thanks!

Could you remove kustomize from helm in the Makefile as I mentioned in https://github.com/ray-project/kuberay/issues/1368#issuecomment-1696777888? Additionally, would you test all the kustomize-related targets listed in the Makefile? In addition, are there any breaking changes between 3.10.0 and 5.1.1?

@kevin85421 Apologies for missing the helm part. I have updated the code changes.
Yes sure, I will test all the targets locally.
Also, let me investigate if there are any breaking changes from 3.10.0 to 5.1.1
I will comment on this PR once I am done with the testing and the investigation. Thanks for the review.

@kevin85421 I tried the targets using Kustomize and they are building fine on my machine.
However, there are breaking changes that are introduced in<EMAIL_ADDRESS>as seen here.
Therefore, I will try to investigate what of the breaking changes made are relevant to kuberay.
Also, the kustomize version being used in apiserver is 3.8.7, should we upgrade that as well?

Closing this as the issue was fixed as part of https://github.com/ray-project/kuberay/commit/e7fbf7d73576d4f21671975709426645e5d38764

Closing this as the issue was fixed as part of https://github.com/ray-project/kuberay/commit/e7fbf7d73576d4f21671975709426645e5d38764

Also, the kustomize version being used in apiserver is 3.8.7, should we upgrade that as well?

^ cc @blublinsky

Also, the kustomize version being used in apiserver is 3.8.7, should we upgrade that as well?

^ cc @blublinsky

If needed, I can create a seperate PR for that. Since this one was open, I removed the unnecessary kustomize in make helm

Yes, I think we should upgrade kustomize throughout the project to the same version

Yes, I think we should upgrade kustomize throughout the project to the same version

Thanks! I will create an issue and raise a CR to make kustomize version consistent across the project.


AvoidUserNameAndPasswordParam rule is too noisy

AvoidUserNameAndPasswordParam is firing even when user is supplying the password as a secure string.

Fixed with https://github.com/PowerShell/PSScriptAnalyzer/pull/442


The publish command doesn't work

Summary
The sf community publish command doesn't work.
Steps To Reproduce:

sf plugins install community
sf community publish -n

Expected result
Community successfully published
Actual result
I'm getting TypeError: this.config.getPluginsList is not a function error
Additional information
Also I have the same issue in the GitHub Actions.
I'm using salesforce/cli:latest-full image to run the command.
You can find screenshots below.

VS Code Version:  1.81.1
SFDX CLI Version: @salesforce/cli/2.5.7
OS and version: macOS 13.5.1 (22G90)

Issue resolved with the last SF CLI update.


Investigate scripts as global tools in .Net Core 2.1

It would be nice if we could package a script as a global tool.
An example of such a package
https://www.nuget.org/packages/dotnetsay/
<?xml version="1.0" encoding="utf-8"?>
<DotNetCliTool>
  <Commands>
    <Command Name="dotnetsay" EntryPoint="dotnetsay.dll" Runner="dotnet" />
  </Commands>
</DotNetCliTool>

Might be as simple as
<?xml version="1.0" encoding="utf-8"?>
<DotNetCliTool>
  <Commands>
    <Command Name="dotnetsay" EntryPoint="main.csx" Runner="dotnet script" />
  </Commands>
</DotNetCliTool>

If not we can wrap the script execution inside a console app which seems to be recommended way of creating global tools.
https://blogs.msdn.microsoft.com/dotnet/2018/02/27/announcing-net-core-2-1-preview-1/

done


[Feature] Add Contact Us / Request Feature Page

Description
Implement a new "Contact Us" or "Request Feature" page to allow users to get in touch with the company for support, feedback, or feature requests. This page should provide a form for users to fill out their contact information and message.
To Reproduce
Steps to reproduce the absence of the feature:

Navigate to the website.
Look for a "Contact Us" or "Request Feature" link in the navigation bar or footer.
Notice that there is no dedicated page for contacting the company or requesting features.

Expected behavior
A "Contact Us" or "Request Feature" page with a form where users can enter their name, email, and message. The form should be user-friendly and easy to submit. Additionally, include contact information such as the company's email, phone number, and address if applicable.
Screenshots
If applicable, add screenshots to help explain your request.
Additional context
I am SSOC'24 Contributor.

Request feature page is not needed as we have GitHub and people can create issue here by describing the feature.Someone is already working on contact us form.


Solution to media files browser caching

#2113 Could be a solution to this.
Is your feature request related to a problem? Please describe.
If using Gatsby as a static site generator I choose static directory as a CMS media folder then I'm risking some browser caching problems.
Scenario:

Admin uploads xyz.png file
This file is then referenced in a website as /images/xyz.png.
Users visit the website and browser caches the resource
Admin then removes xyz.png file and uploads updated image with the same name.
User visits the website again and the browser is using a cached image instead of a new one.

Gatsby docs state this downside too:

Result filenames won’t include content hashes so you’ll need to add query arguments or rename them every time they change.

Describe the solution you'd like

Adding unique hash to the image name - something like Gatsby does for media imported in javascript files.
Add a date or store them in directories split by date - #2113

I am facing a related issue I believe. I have recently updated my src/cms/cms.js file, but my browser appears to still use the cached version of the generated admin/index.html file. I tried redeploying from Netlify with clear cache and I get the proper version when using incognito mode, but the regular browser is still stuck at the old version. Any pointers as to how we can resolve this? Thanks!

Sounds more like a Gatsby issue, no? I'd expect an image changed in static to be picked up by Gatsby and cache busting to be handled.

@erquhart nope. Images in a static directory are not processed by Gatsby/Webpack https://www.gatsbyjs.org/docs/static-folder/

Gotcha. I propose we allow media file names to be configurable by template, just like file names of entries. The default would match what you're proposing: {slug}-{uuid}.

I would like to work on that. Do you have any further explanation or recommendation to start?

Thanks @KancerEzeroglu, I'm not sure if this is still an issue. Our Gatsby's guide recommends using relative media folders:
https://www.netlifycms.org/docs/gatsby/#configuration
which in that case, the media files should be processed by Gatsby.
Regardless, this issue can be done as a part of of https://github.com/netlify/netlify-cms/issues/857


Adds extension methods for Task wrapped Result and Option type assertions

This PR adds extension methods that provide assertions for both Task<Option<T>> and Task<Result<TSuccess, TFailure>>.
Example Task<Option<T>> assertion:
Task.FromResult(Option.Some(VALUE))
    .Should()
    .HaveValue()
    .AndValue(v => v.Should().Be(VALUE)))

Example Task<Result<TSuccess, TFailure>> assertion:
Task.FromResult(Result.Failure<int, Exception>(new Exception(VALUE)))
    .Should()
    .BeFaulted()
    .AndFailureValue(e => e.Message.Should().Be("Exception")))

Documentation has been added to the ReadMe for Task<Option<T>> and Task<Result<TSuccess, TFailure>>

Documentation looks good!  Thanks for writing that up.

Please bump the minor version number and add release notes in the Functional.Primitives.FluentAssertions project's properties @Justin-Cooney .  I am OK with merging this in before the Union stuff as there is likely some more work to be done there.
Once the version number is bumped, this PR is good to go.  I can then cut a release later today.