creating template from snapshot does not work from snapshot list

If you go to STORAGE (main section menu on the left), then select VOLUMES from the dropdown list, select a particular volume, than see snapshots, then select a snaphost - then clicking on button WILL work fine and popup window is show as follows:

But if you go to STORAGE (main section menu on the left), then select SNAPSHOTS from the dropdown list and find one and click the button - then an exception happens in the javascript code:

ISSUE TYPE

Bug Report

COMPONENT NAME

UI

CLOUDSTACK VERSION

4.13

CONFIGURATION

OS / ENVIRONMENT

SUMMARY

STEPS TO REPRODUCE

EXPECTED RESULTS

ACTUAL RESULTS

during the latest update of jquery the createTemplate functions for volumes and snapshots were unified into a shared function without fully abstracting out the entities 'volumes' and 'snapshots'
We can duplicate the function as a quick-fix or put effort into the abstraction. I am choosing the first as the current UI is to be phased out.
@andrijapanicsb @rhtyd @rafaelweingartner

sounds good @DaanHoogland
I've also seen (probably NOT related, but to mention...) that when you register the template from URL, previously there was a spinning circle just until the template was added to the DB - then you can actually click on it and see it being downloaded etc. In current master when you register the template, the circle is spinning all the time, until the template is actually completely downloaded, which is insane. - not sure if that's also a quick fix...

actually @andrijapanicsb half an hour extra search gave me a location to add a conditional, testing is probably going to be more effort.
What you are saying seems a different location in the code. i'll keep it in mind. Did you create an issue for that already?

No @DaanHoogland, I was lazy...


Morning session code review Tue 21 Dec

Keep it up :100:
The home page looks nice and clean :+1:

Good luck with the upcoming milestones :1st_place_medal:

Oh okay, bro. Thanks


Properly handling image references with both tag and digest

We fixed #1481, and Jib no longer fails when both a tag and a digest is given. However, it only enabled parsing it, and I believe it just ends up ignoring the tag part for a base image. For example, if you have jib.from.image=openjdk:latest@sha256:..., it essentially just becomes<EMAIL_ADDRESS>We should guard accidental misuse and confusion. At least we should show a warning at a stop-gap, but ideally, we should validate the meaning of references having both a tag and a digest. Even though we fixed #1481, it doesn't provide any new value than previous versions.
We also need to think about correct behavior and messages about the base image reference. I think correct error messages are important, as authentication causes the most friction.

I'm curious how Jib will work and respond if there's an error for a base image, e.g.,

tag exists but digest doesn't match
digest exists but tag doesn't

Will they work? If they fail, will the error message make sense and be helpful?

Found these comments.
https://github.com/google/go-containerregistry/issues/540#issuecomment-534219086
https://github.com/google/go-containerregistry/issues/351#issuecomment-459434472
For fetching, if the comment about Docker CLI is right, it does seem like a tag is usually ignored and serves as a comment when a digest is present. Haven't tested though.

This is a nice and reasonable idea, but as in https://github.com/GoogleContainerTools/jib/issues/2398#issuecomment-613111738, most likely we don't need validation.


For flash messages, use the name of the thing that is being affected with a link to the object

@lnchambers this was the other card


Relu activation in PrimaryCap?

the tf.contrib.layers.conv2d applies a relu activation,the PrimaryCap convolution does not included a relu activation before grouping neurons into capsules and then squashed, or did I miss something from the paper
https://github.com/naturomics/CapsNet-Tensorflow/blob/894c79cd8434d7de7784006c1646ff3107bb2a4f/capsLayer.py#L59

In fact, I have the same doubts. I've read the paper again and again, but I can't find out any words in the paper whether PrimaryCaps has a ReLU activation function or not. So I just let it go. Maybe I should write some comments in the code to note that. I'll try it out which is better, and use it as the default.
Thank you for your comment

@oargueta3 I've done a experiment for this doubts. The results show that using ReLU has a higher test accuracy(batch size 48, 50 epoch,~99.4) than no ReLU(batch size 48, 30 epoch, the best result is 99.1 at step 8600, but most time it maintains at 99.0)

@naturomics , thanks for your work, and it is great and really quick!
It seems that there's no ReLU between capsule and squashing. As mentioned in the third paragraph in Sec. 4 of the original paper, both v1 and v2:

One can see PrimaryCapsules as a Convolutional layer with Eq. 1 as its block non-linearity.

So, I think we may confirm that no ReLU is placed there.
Any suggestions?


Why is Docker needed?

Hi there. What does Docker have to do with Channels?
From what I understand, Docker is like VMWare or VirtualBox.
This seems like bloatware.
There are already Pluto TV EPG and M3U8 generators out there. What makes this project so unique?
I don't meant to sound rude or anything. I'm genuinely curious.

Hello, thanks for your great work on this repo, I have it up and running on an old Dell Notebook.
Would you mind if I forked your javascript code, converted it to PHP and hosted on a public website, so people would be able to access the enhanced Channels tags you've created for this? I would make sure the unique deviceID and sid UUID parameters are generated every time someone pulls the m3u file down, so there are no conflicts like some of the existing static web-hosted m3u files for PlutoTV.
I think this would allow people to gain access to these channels in the way you intended in Channels, and not be subject to the dropouts and disconnects due to the static nature of the other hosted m3u sites.  And most of all, people wouldn't have to wrestle with installing and running Docker on different platforms (especially ones that can't run Docker).  I envision this as the "best of both worlds."  Do you foresee any issues or problems with this plan?
Of course, I would put the PHP code back up on github with proper credit.
Thank you.

https://github.com/maddox/pluto-for-channels/blob/main/LICENSE.md

Great, thank you.
Do you foresee any issues with implementing this?

OK, I've created a repo that can replace the need to have a local Docker instance for PlutoTV.
It's really quite simple. It runs a slightly modified version of the pluto-for-channels index.js script on my server, then my PHP program replaces the deviceID and sid URL parameters with uniquely generated UUIDs upon each page load. So every use/access of the playlist.m3u file is unique.  This should prevent any conflicts, dropouts, and disconnects that other online/public m3u files cause that do not have unique parameters in the stream URLs.
https://github.com/HankLloydRight/pluto-for-channels-php
You can access the resulting files here and put these links into the Channels Admin page. They should work exactly the same as the files created by the local Docker instance.  The source files are updated on the backend every three hours automatically.
Playlist:  https://sqrl.ws/pluto/playlist.m3u
and
EPG:  https://sqrl.ws/pluto/epg.xml
Can I share this on the Channels Community site?

Thanks and you're welcome.
I'm looking into the Stirr version now, which is quite a bit harder since I don't know GO.  I may try the same thing to just run the GO programs on the server and process the output, or possibly rewrite it entirely in PHP.
But I'm not sure even that is needed for Stirr, since there are no unique parameters in the Stirr stream URL links.  They're all static.  So I'd guess using the static m3u versions at https://i.mjh.nz/Stirr/ wouldn't cause the same dropouts and disconnects that were happening on PlutoTV.


More complex outputs

Hi:
Can your pkg handle more complex outputs? Like
W"Association"(W"Rule"("baseSalary", W"Quantity"(220000, "USDollars")), W"Rule"("position", "F"))
Thanks,
Gani --

Do you have definitions of Rule, Quantity and Association in Julia?  This library just translates function names


Publish a Marketplace item section missing steps and wrong Powershell Command

it should be
Add-AzsGalleryItem -GalleryItemUri `
https://sample.blob.core.windows.net/gallerypackages/Microsoft.SimpleTemplate.1.0.0.azpkg –Verbose
this is to be done after adding and connecting to environment
Install-Module -Name AzureRm.BootStrapper -Force
Use-AzureRmProfile -Profile 2017-03-09-profile
Install-Module -Name AzureStack -RequiredVersion 1.3.0
#imports mmodules for azure stack
cd C:\AzureStack-Tools-master\Connect
Import-Module .\AzureStack.Connect.psm1

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

ID: 83d0ba3d-9766-30da-70f0-5f4c3ec9a47e
Version Independent ID: 446b6ecc-7b39-df7f-69f5-f3f0ea9b0a94
Content: Create and publish a Marketplace item in Azure Stack
Content Source: articles/azure-stack/azure-stack-create-and-publish-marketplace-item.md
Service: azure-stack
GitHub Login: @Brenduns
Microsoft Alias: brenduns

#in-progress

@Jquintero190   Thank you for the feedback.  I've checked with our engineering team and because this will be more nuanced than a fast content fix can manage, I've created a work item in our documentation issue tracking system so we can take a deeper look at this.
The cmdlet you note seems correct, but before making changes we'll need to account not only for the PowerShell 1.3.0 and the latest Azure Stack versions, but introduce revisions that can address older versions that remain in support.  We also need to ensure the story of where the modules for azure stack are obtained from is stitched into this content.  Thus a deeper review before we begin acting.
Because we are now actively tracking your feedback, I'll need to close this Github issue.

#please-close

@Jquintero190 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.


Render a Resource

Should automatically acquire on mount and release on unmount.

I was thinking about having a render instance for resources, but the type signature does not actually work like I thought and in the end and you normally get an effect like IO that you can render anyhow. For other things, we have managed. So I am closing this for now.


Impl Write for Cursor<&mut alloc::vec::Vec> when alloc feature is enabled

@core2-rs could you review this addition? Not sure exactly who is the maintainer of this crate.


[JsonPatch]Change ConvertToActualType method to use JsonContract custom converters

@kirthik can you investigate the design for this?

merged 8b5223518f3fb1d4b90744370ef4a08d9a2a31c9


Add shebang to bin/build.sh

Without the shebang, the buildscript might fail on environments which do
not use bash or a compatible shell as default.
This PR fixes the issue and makes the script more robust to the
environment.

Good catch. Thanks!


i'm getting error while running you sample 'Figure' object has no attribute '_remove_ax' using matplotlib==3.3.0

[Enter feedback here]

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

ID: 0300ff9d-ace6-93b9-64b7-c570a5015d5e
Version Independent ID: ad14f936-2cc5-b4b2-2727-63d1ee82f6f9
Content: Python tutorial: Explore and visualize data - SQL Server Machine Learning Services
Content Source: docs/machine-learning/tutorials/python-taxi-classification-explore-data.md
Product: sql
Technology: machine-learning
GitHub Login: @dphansen
Microsoft Alias: davidph

@mkobilansky - Thanks for taking time to send this feedback. Can you provide more context: Which sample code from the text were you running when this occurred? Thanks.

The error coming from the
https://docs.microsoft.com/en-us/sql/machine-learning/tutorials/python-taxi-classification-introduction?view=sql-server-ver15
NYC
taxi example binary plot object generation, following line :  fig =
pickle.loads(tables[i][0])
Quite possible bug is in matplitlib 3.2.0 version
On Wed, Aug 12, 2020 at 5:26 PM Gary Ericson<EMAIL_ADDRESS>wrote:

@mkobilansky https://github.com/mkobilansky - Thanks for taking time to
send this feedback. Can you provide more context: Which sample code from
the text were you running when this occurred? Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/sql-docs/issues/5321#issuecomment-673119389,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABCKUMOYCHO67JMJACOE5ETSAMCJRANCNFSM4P3S4SBA
.

@mkobilansky - Thanks. I'm working on reproducing this.

SQL server. SQL part is working OK.  The error coming from the python code
running in either AzureData Studio, jupyter notebook or vscode
On Thu, Aug 13, 2020 at 4:25 PM Gary Ericson<EMAIL_ADDRESS>wrote:

@mkobilansky https://github.com/mkobilansky - Thanks. I'm working on
reproducing this.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/sql-docs/issues/5321#issuecomment-673692746,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABCKUMLSN6TRJIUERMAQCGTSARDZZANCNFSM4P3S4SBA
.

Sorry, I thought I responded to this already.
I've been trying to reproduce this. I have seen this specific error once, and a couple other similar errors, when I had different versions of Python or Python packages between SQL and the client computer. After I reinstalled locally (as necessary) to get the local versions to match the SQL versions, it runs for me without an error.
If matching versions doesn't work for you, the next step would for you to raise the question in SQL Server on Q&A, or contact Microsoft Support to open a ticket for this issue. The process is described in SQL Server help and feedback.
I've been using this code to check the local Python and package versions:
# List the versions of Python packages used in this tutorial
import matplotlib
import pandas
import pickle
import pyodbc
import sys
import pkg_resources

print("Python = " + str(sys.version_info))

pkg_name = "matplotlib";  print(pkg_name + " = " + pkg_resources.get_distribution(pkg_name).version)
pkg_name = "pandas";      print(pkg_name + " = " + pkg_resources.get_distribution(pkg_name).version)
pkg_name = "pickleshare"; print(pkg_name + " = " + pkg_resources.get_distribution(pkg_name).version)
pkg_name = "pyodbc";      print(pkg_name + " = " + pkg_resources.get_distribution(pkg_name).version)

And this to check the same thing in SQL:
-- List the versions of Python packages used in this tutorial
EXECUTE sp_execute_external_script
  @language = N'Python',
  @script = N'
import matplotlib
import pandas
import pickle
import pyodbc
import sys
import pkg_resources

print("Python = " + str(sys.version_info))

pkg_name = "matplotlib";  print(pkg_name + " = " + pkg_resources.get_distribution(pkg_name).version)
pkg_name = "pandas";      print(pkg_name + " = " + pkg_resources.get_distribution(pkg_name).version)
pkg_name = "pickleshare"; print(pkg_name + " = " + pkg_resources.get_distribution(pkg_name).version)
pkg_name = "pyodbc";      print(pkg_name + " = " + pkg_resources.get_distribution(pkg_name).version)
'

#please-close - I've done all I can. The next step is to use the Q&A Forum or Microsoft Support.


Fix missing action icons in action map editor

Fixes #53708
Issue
Unlike in 3.x branch, in action map editor of master branch, icon action are missing in action tree.
Fix proposal
Add an icon depending of the action event type.
Before

After

Note that this doesn't completely fix #53708, since device names still need to be relisted.

Thanks!

This Pr break the build under linux

Yeah I just realised after rebasing for #59514. Got the same error. This is an old PR and probably should have been rebased. I will submit hotfix.


swift 2.3 support

[x] I have verified there are no duplicate active or recent bugs, questions, or requests
[x] I have verified that I am using the latest version of CVCalendar.
[x] I have given my issue a non-generic title.

Expected Behavior

Current Behavior

Possible Solution

Steps to Reproduce (for bugs)

Context

Are you planning to release a version for the swift 2.3 compiler?
Your Environment

Version used (Cocoapods spec):
Device used i.e. iPhone 6S:
Physical device (y/n):
Link to your project (if available):

You obviously didn't check recent issues as you assured, since #349 is dealing with the very same question.

I'm sorry, I didn't check on closed issues


Document interfaces a plugin can implement

I had to search the sources to find these, so I figured it would be useful to have them all on the plugin API docs along with links to the respective pages about writing the plugins.
Signed-off-by: Linus Heckemann<EMAIL_ADDRESS>

LGTM
/cc @cpuguy83

re-LGTM thanks!

LGTM 🐮


Dependency: ACPI: Define _CCA behavior for RISC-V

Recommend this as ASWG change to mandate _CCA on any RISC-V platform or force a specific meaning when _CCA is not present.

Should we document _CCA to follow X64/AARCH64 semantics? Defaulting to cache coherence when absent?

Probably good for a default in ACPI-land. A particular RISC-V Platform may have stronger expectations for all devices in a system to be cache coherent.

Recommend this as ASWG change to mandate _CCA on any RISC-V platform or force a specific meaning when _CCA is not present.

Agreed with either of these two approaches

Filed https://mantis.uefi.org/mantis/view.php?id=2407. On RISCV, if the _CCA object is not supplied, the OSPM will assume the devices are hardware cache coherent.


Package registry login

I'd like to add some more info to the CONTRIBUTING.md file regarding how to login to the github npm registry, where the @action-rs packages are located. I've tried a few different solutions

adding GITHUB_TOKEN to .npmrc (but not committing it)
using npm-cli-login
using npm login

None of which feels fantastic... What is your preferred local workflow?

I'm very far from an expert here, so I just used npm login but if there's a better way, I'm open to it.


Memory issue when sharing CancelToken between posts for IO implementation

Package
dio
Version
5.4.0
Operating-System
Android
Output of flutter doctor -v
[✓] Flutter (Channel stable, 3.13.6, on macOS 14.2.1 23C71 darwin-arm64, locale en-US)
    • Flutter version 3.13.6 on channel stable at /Users/stevenyi/fvm/versions/3.13.6
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ead455963c (5 months ago), 2023-09-26 18:28:17 -0700
    • Engine revision a794cf2681
    • Dart version 3.1.3
    • DevTools version 2.25.0

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
    • Android SDK at /Users/stevenyi/Library/Android/sdk
    • Platform android-33, build-tools 33.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.7+0-17.0.7b1000.6-10550314)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS (Xcode 15.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15C500b
    ✗ Unable to get list of installed Simulator runtimes.
    • CocoaPods version 1.15.2

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

[✓] Android Studio (version 2023.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 17.0.7+0-17.0.7b1000.6-10550314)

[✓] IntelliJ IDEA Ultimate Edition (version 2023.2)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    • 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

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

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-arm64   • macOS 14.2.1 23C71 darwin-arm64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 121.0.6167.184

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

Dart Version
3.1.3
Steps to Reproduce

Create a CancelToken
Make multiiple requests passing in the same CancelToken

Expected Result
No out of memory issues on Android.
Actual Result
Out of memory issues occur.
Dio's use of cancelFuture in dio/lib/src/adapters/io_adapter.dart differs from how it is used in browser_adapter.dart and exhibits different behavior. In IO, it captures on request in the callback to whenComplete in _fetch, which does not happen in the browser implementation.
This took us some time to discover as a memory leak issue. It would be great if this could be fixed, otherwise would love to see this documented so that future users will be aware of this issue.

Could you attach the minimal reproducible example and some analysis graphs or details?

We have notice an Out of Memory error as well, on iOS since 5.4.x.  I haven't had a chance yet to debug it in detail.

If yes, we might add a global map to cache the necessary instances for the corresponding cancel token.

I tried to enable the new leak tracking but didn't get very far yet...

@AlexV525 I haven't had time to followup, but yes, as mentioned in my first message, we found that the .whenComplete callback captures over the Request:
      if (cancelFuture != null) {
        cancelFuture.whenComplete(() => request.abort());
      }

and whenComplete() also adds additional handlers each time it is called. So if a CancelToken is passed in to multiple fetches, it will keep accruing whenComplete handlers that all capture over individual requests.

@kunstmusik Can you verify if #2252 fixes the issue?

Not release data in "multipart/form-data" request


Self-Signed Certificate Question

Refer to the document warning, the automatically generated certificate will expire 60 days after installation.  Does we generate the self-signed certificate which can use more than 60 days?

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

ID: fe84fb4b-3420-bc1e-6a1d-b64ecc56fc5e
Version Independent ID: f089767e-4613-d595-a0ee-0dab012724a1
Content: Install Windows Admin Center
Content Source: WindowsServerDocs/manage/windows-admin-center/deploy/install.md
Product: windows-server-threshold
GitHub Login: @nwashburn-ms
Microsoft Alias: niwashbu

Please note: The self-signed certificate is for testing purposes only and not recommended for production use.
To generate a new certificate, re-run the installer. The previously generated certificate thumbprint will be automatically populated, but toggling back to the generate option will create new self-signed certificate with a new expiration date.
Another option would be to create and install your own self-signed certificate using the New-SelfSignedCertificate PowerShell Cmdlet.


0.3.0-alpha.01 should be rejected

The version string 0.3.0-alpha.01 is currently accepted by semver 0.11.0.
Playground Link
The issue is that 01 is not a valid pre-release identifier. According to Semantic Versioning 2.0.0 reference, a pre-release identifier can be either a numeric identifier or an alphanumeric identifier. However, the identifier 01 is neither of them. Numeric identifiers cannot start with 0 (EDIT: unless it is a single digit 0), and alphanumeric identifiers must have at least one non-digit character.
This bug led to the publication of a crate with an invalid version on crates.io:
https://crates.io/crates/hxgm30-client

Another case found on crates.io: https://crates.io/crates/bluetooth_client/0.0.1-001

Yep, this is a bug, thanks. We will probably have to add in a compat shim to https://github.com/steveklabnik/semver/blob/master/src/version_req.rs#L323-L332

This is fixed in 1.0.0.
fn main() {
    println!("{}", semver::Version::parse("0.3.0-alpha.01").unwrap_err());
}

invalid leading zero in pre-release identifier


version 0.9.4, nulls excluded from LiquidValues

Current coverage is 90.53%

Merging #41 into master will not affect coverage as of d0c0194

@@            master     #41   diff @@
======================================
  Files          134     134
  Stmts         4299    4299
  Branches       445     445
  Methods          0       0
======================================
  Hit           3892    3892
  Partial        155     155
  Missed         252     252

Review entire Coverage Diff as of d0c0194

Powered by Codecov. Updated on successful CI builds.


Converted hilbert.py into a class

New class contains the dimensions and iterations to use so they don't need to be passed every time
I'm no python wiz, so this may not be the best practice for making it a class, but it works.

thanks for the PR.  this was definitely on the road map and then I got sidetracked.  I'm working on some new examples with very large integers and I'll incorporate a class structure into my next update.


[grpc] tools no uwp

Fix https://github.com/microsoft/vcpkg/issues/33622

The libprotoc.lib built by upstream protobuf does not support uwp, so disable uwp to until the protobuf fix this.
https://github.com/microsoft/vcpkg/issues/33622#issuecomment-1711384821

[x] Changes comply with the maintainer guide
[ ] SHA512s are updated for each updated download
[x] The "supports" clause reflects platforms that may be fixed by this new version
[ ] Any fixed CI baseline entries are removed from that file.
[ ] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

Thanks!

I think protobuf uwp support was fixed in https://github.com/microsoft/vcpkg/pull/35229 does that mean that uwp for grpc could be enabled again?

AFAICS only grpc feature codegen is unsupported on uwp. This feature builds grpc plugin tools to run on uwp. Is this expected to work?

Thanks, I had not spotted it was just for the codegen component.


TODO

[ ] Implement/test dropping on target (drop on drop target, endDrag(true) on drag source)

[ ] Implement/test dropping somewhere else (endDrag(false) on drag source)

[ ] Think: should we let drop return a value and pass it to endDrag instead of a boolean?

[ ] Implement/test subscription to state (let DragDropContext expose addChangeListener/removeChangeListener that proxies to DragOperationStore?), we'll use that in React for letting components update their state

[ ] Think: consumer may want to “recalculate” drag state at some point. Imagine canDrop (we haven't talked about it yet, but it should be implemented at some point) that depends on props, and props changed. Can React part tell dnd-core that it needs to “reconsider” some stuff it may have calculated concerning a drop zone or drag source?

[ ] MY FAVORITE PART (closely related to previous point). You have a list of React components in an array but without keys. The items reorder. How to make sure their drag states still match? This might be canJoinDrag or some other API that makes more sense. TEST IT.

[ ] Figure out how we represent “entered” drop targets (gotta store a list in manager to support nesting, DragDropContext will ask the manager to implement isOver(descriptor), isInside(descriptor)). Implement and test.

[ ] Finally, the boring part: tracking drag coordinates.

Just thinking about event bubbling, and how to emulate/simulate/facilitate that

Only drag event left!

Closing, we'll track dragOver implementation in #6.
We don't know enough to implement it yet.


wireguard: setup some initial devices in a triangle

The fuzzer will wind up undoing some of this, which is fine, but at
least it now has the chance of hitting some other paths it wasn't
before.
Closes: #1599

3 failures in tests:
      In file included from ../../executor/common.h:393:0,
                         from ../../executor/executor.cc:133:
        ../../executor/common_linux.h: In function ‘void netlink_wireguard_setup()’:
        ../../executor/common_linux.h:848:40: sorry, unimplemented: non-trivial designated initializers not supported
              .sin6_addr = IN6ADDR_LOOPBACK_INIT};
                                                ^
        ../../executor/common_linux.h:852:40: sorry, unimplemented: non-trivial designated initializers not supported
              .sin6_addr = IN6ADDR_LOOPBACK_INIT};
                                                ^
        ../../executor/common_linux.h:856:40: sorry, unimplemented: non-trivial designated initializers not supported
              .sin6_addr = IN6ADDR_LOOPBACK_INIT};
                                                ^
        ../../executor/common_linux.h:833:27: error: unused variable ‘endpoint_a_v4’ [-Werror=unused-variable]
          const struct sockaddr_in endpoint_a_v4 = {
                                   ^
        ../../executor/common_linux.h:849:28: error: unused variable ‘endpoint_b_v6’ [-Werror=unused-variable]
          const struct sockaddr_in6 endpoint_b_v6 = {
                                    ^
        cc1plus: all warnings being treated as errors

       --- FAIL: TestExecutorMistakes (0.00s)
    csource_test.go:230: pattern "\\)\n\\t*(debug|debug_dump_data)\\(" matches executor source:

                if (err)
                        debug("netlink: failed to setup wireguard instance: %s\n", strerror(err));
    csource_test.go:230: pattern "\\)\n\\t*(debug|debug_dump_data)\\(" matches executor source:

                if (err)
                        debug("netlink: failed to setup wireguard instance: %s\n", strerror(err));
    csource_test.go:230: pattern "\\)\n\\t*(debug|debug_dump_data)\\(" matches executor source:

                if (err)
                        debug("netlink: failed to setup wireguard instance: %s\n", strerror(err));

                <stdin>: In function ‘netlink_wireguard_setup’:
                <stdin>:642:28: error: unused variable ‘endpoint_b_v6’ [-Werror=unused-variable]
                <stdin>:626:27: error: unused variable ‘endpoint_a_v4’ [-Werror=unused-variable]

Codecov Report

Merging #1600 into master will increase coverage by 0.04%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1600      +/-   ##
==========================================
+ Coverage   56.74%   56.79%   +0.04%
==========================================
  Files         144      144
  Lines       27029    27029
==========================================
+ Hits        15337    15350      +13
+ Misses      10913    10900      -13
  Partials      779      779

Impacted Files
Coverage Δ

prog/prog.go
79.89% <0%> (-1.01%)
:arrow_down:

prog/any.go
88.4% <0%> (-0.73%)
:arrow_down:

prog/rand.go
87.65% <0%> (-0.36%)
:arrow_down:

prog/rotation.go
95.41% <0%> (+1.52%)
:arrow_up:

prog/hints.go
91.12% <0%> (+1.61%)
:arrow_up:

prog/mutation.go
94.02% <0%> (+2.71%)
: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 84f4fc8...84a29d9. Read the comment docs.


feat: Headless Benchmark

The benchmark can be run in editor using the "editorArgs" or it can be run in a standalone using normal command line args. Example command line arg usage:
.\MirrorNG.exe -transport tcp -server -monster 100 -client 10

In its current design at the time of this comment the specified players and monsters will constantly translate and rotate to provide network load. There is also a SyncVar float that will constantly update its value with a random number.
In future changes I would like to make the frequency of the objects moving adjustable and the number of syncvar/rpcs also adjustable to better simulate real gameplay.

merge master to fix kcp loading issue


Update user-guide.md

"-" instead of  "_" to fix the button at the end of the page

Hi @Jerzeek, thanks for the patch!
It fixes the issue partially, as the yaml text on top of the MD file has a wrong path with _article/_article/developer... I'll accept your PR and fix the other part here. This way we can still have your contribution in, and fix the issue.


miio protocol udp proxy

I have a zhimi.airpurifier.v6 which works nicely. However, I had the briliant idea to segment out all my IOT device onto a separate iot-only network. Turns out, the purifier only talks locally on the same network segment, which is a bummer since it does a great job of reporting the temperature to my home assistant setup. Any idea if a udp proxy will do the job, or is there something magical in the packets?

It will only respond to requests on the same subnet.  You need to set up a 1:1 NAT so that the device running your code can have a local address on the same subnet as the purifier.  This worked fine for me.

Thanks @ekrekeler  that is a much more limited rule.  The way I implemented the 1:1 NAT is by creating an IP for my raspberry pi that sends commands to the Mi Devices on the sub-net with the devices.  I also limited traffic to only the Mi devices by using an alias that included all the devices.  That way I could limit any comms from the subnet to the pi.  I'm going to change my setup to your suggestion.


add missing test cases for matrix params array indexing validation

Changes
This commit adds missing test cases for matrix params array indexing validation. The array indexing for matrix params is supported in previous PR but test cases are missing.
Closes #6278
/kind misc
Signed-off-by: Yongxuan Zhang<EMAIL_ADDRESS>
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

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

Release Notes
NONE

/assign
/lgtm


Fix #2609 - Handle out-of-bounds scalar replacements.

When SROA tries to do a replacement for an OpAccessChain that is exactly
one element out of bounds, the code was trying to access its internal
array of replacements and segfaulting.
This protects the code from doing this, and it additionally fixes the
way SROA works by not returning failure when it refuses to do a
replacement.  Instead of failing the optimization pass, SROA will now
simply refuse to do the replacement and keep going.

@alan-baker thanks for spotting the bad test I had pasted.  I've done the constant propagation manually and now the test does the proper indexing that I was getting with -O.  PTAL.


Why type/interface/class types does not extend explicitly Object class.

TypeScript Version:  3.9.7

Search Terms: types in typescript does not extend Object
Code
const o = {};
//no method completion from Object class in tsplayground, vscode, intellij
o.toString();
o.hasOwnProperty();

// always true
if(o instanceof Object) {
//method completion from Object class in tsplayground, vscode, intellij
o.toString();
o.hasOwnProperty();
}

class MyClass {

}
//no code completion
new MyClass().toString()

class MyClass2 extends Object {

}
//code completion present
new MyClass2().toString();

Expected behavior:  Typescript suggest methods from Object class. Almost all object in javascript are sub-type of Object class (with exception to Object.create(null) and those with replaced prototype during runtime).  I would expect that for almost every object the methods from Object class will be suggested for me.
Actual behavior: Typescript is prompting me about methods from Object class only if a will use explicit type annotation or extend Object class (in the interface or class or type).
Playground Link: Playground Link
Related Issues:

Duplicate of #14291.


Add shfmt for formatting shell scripts

shfmt can be added as a make target as well as GH workflow to auto-format the shell scripts so that changes can be checked locally and on CI

Fixed https://github.com/korrel8r/korrel8r/commit/0a4bf66ede7bba8a2c51213a4fba8fcbb214702f


Only use cpuType=host for OS with accel available

This should be matching the qemu getAccel() values
Specifically, no acceleration on FreeBSD right now
Issue #892

I will share some information because I hit that on NetBSD too but feel free to ignore such notes (I think that for NetBSD -cpu host can be added easily when NVMM accelerator is used):
I think that's a bit more complicated then that, unfortunately.
I.e. if I'm not missing something (quite probable!) only certain accelerators supports -cpu host for x86 I think only KVM and HVF:
https://git.qemu.org/?p=qemu.git;a=blob;f=target/i386/cpu.c;h=6a57ef13af8695f210b02f4adb1aeedb7aff869a;hb=HEAD#l1487
So I guess that's Linux and macOS are probably good in that case but at least for NetBSD not and I have no idea regarding Windows.
The other possible problem is that also trying to possibly list that via qemu -cpu help can points out that there is a -cpu host but that could work only if a certain accelerator is selected (but maybe that's just a theoretical issue that is not triggered by lima :)).

Currently it just assumed that all accelerators support -cpu host, if that is not true it should probably be fine tuned...
It also assumes that it is always available and enabled, for instance enabled in BIOS etc and kernel module loaded.

It sounds like a good idea to have the flags return -cpu max on platforms where -cpu host does not work...
Care to make a PR ?

Just checked, and there is no -cpu host on Windows either.

Thank you @afbjorklund!


NLog ApplicationInsightsTarget Flush with async delay

But only when something to flush

Thank you for another contribution. We appreciate it.
For the housekeeping purposes - can you please create an issue with the problem description and then include changelog update in PR that would refer this issue.

@SergeyKanzhelev Updated changelog for you

I'll wait for other people to chime in and will merge tomorrow

It's a disappointment we need to work around this here, not make it a feature of base SDK

Yes it is a little weird not to have proper flush-logic in a logging framework.


Enable syslog logging driver for windows

Syslog was disabled in https://github.com/docker/docker/commit/655a58e27bfcfd2494bb5d46f95cb49d655ad17a by @jhowardmsft because that time it was using "log/syslog" package (not supported on windows).
Now syslog.go is using "github.com/RackSec/srslog" so it's possbile to use it on windows. (I was able to compile and run docker on windows with syslog enabled).
Would you consider to enable this driver for windows?

No objection at all. Do you want to put a PR together?

Ok.
I'll do some additional testing and then will create a PR.


Develop vote

Develop Vote Feature
I added the step to cast votes on proposals hosted on the IXO Network.

@RafeeqWeideman-YDS I notice that there are conflicts that need to be resolved; are you able to do this?


[dashboards] Add tests

[x] I added a CHANGELOG entry for this change.
[ ] I adjusted the corresponding documentation for this change.
[ ] I adjusted the values.yaml file and the documentation for all values.

Codecov Report

Merging #266 (af9ab6a) into main (7feac26) will increase coverage by 1.58%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #266      +/-   ##
==========================================
+ Coverage   31.50%   33.08%   +1.58%
==========================================
  Files          90       90
  Lines        7612     7613       +1
==========================================
+ Hits         2398     2519     +121
+ Misses       5182     5058     -124
- Partials       32       36       +4

Impacted Files
Coverage Δ

plugins/dashboards/dashboards.go
92.18% <100.00%> (+77.22%)
:arrow_up:

plugins/applications/applications.go
100.00% <0.00%> (+6.76%)
:arrow_up:

...lugins/dashboards/pkg/placeholders/placeholders.go
40.00% <0.00%> (+40.00%)
: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 7feac26...af9ab6a. Read the comment docs.


array in conditional type issue

Hi this is the code to reproduce the issue
class Item{
    name:string
}
abstract class ItemContainer{
    items: Item[]
}

class Drawer extends ItemContainer{
    size:number
}

type ArrayInnerType< A> =  A extends (infer inferred)[]?inferred:A;

class Manager<T extends ItemContainer>{
    manage(object:T){
        type rightType = ArrayInnerType<ItemContainer["items"]>;
        type wrongType =ArrayInnerType<T["items"]>;
        //   ^?
    }
}

I expect the type of wrongType to be Item but it is (inferred & inferred[]) | T["items"]
can you help me understand why?
Thanks

To expedite the triage process, we need everyone to follow the issue template and instructions.
When you clicked "Create New Issue", the issue form was pre-populated with a template and some instructions. We need you to read those instructions completely, follow them, and then fill in all the fields in that template.
We are not able to assist with issues that don't follow the template instructions as they represent a significantly larger amount of work compared to issues which are correctly specified. Thank you for understanding.


optimize use of ^/\A

At the moment, the VM always scans the entire input unless one of the two following things happens:

The regex has a literal prefix and that prefix cannot be found anywhere in the input. (This always scans the entire input.)
A match is found. This may not scan the entire input, but it requires a match.

Ideally, given a regex ^{something}..., it could terminate the search early if {something} doesn't match at the start of the input.

@alexcrichton Is this the right place to report issues for regex? There are some issues on the rust repo...
Also, if changes get made to this repo, do they get pulled into rust repo? Or is it the other way around?

Sure! I think any existing issues should be migrated here as well. Feel free to ping nrc on IRC for any lingering issues, he's got a script which will auto-transfer issues between repos.


Fixed static Color redefinition error

This should fix #31. Declared all Color::Color inline outside of class definition.

Ah, nice workaround... Let me know if we need a ifdef implementation and we can add it.


RunTimeException: Unable to create application caused by TrayContract.getAuthority

tray Version (E.g. 0.12.0)
0.12.0
Device(s) (E.g. Samsung Galaxy S8)
unknown, likely many different device manufactures and types
How have you setup tray (E.g. Initialized in Application.onCreate, in an Activity, BroadcastReceiver, IntentService, MainThread)
In my case, I use Tray in a library project. The library splits into client and service. The client part of the library is used in application main process. The service part is a remote service running in a separate process. Both client and service initialize tray AppPreferences object and do read/write operations.
Both client and server side share use Tray in the same way.
initialize it this way
private AppPreferences appPreferences = new AppPreferences(context);

and call put and get
Android Version (E.g. Marshmallow or better API 23)
Around version 4.2 to 4.4
Stacktrace

Stacktrace
Fatal Exception: java.lang.RuntimeException: Unable to create application xxxxxxxx: xxxxx: Internal tray error. Could not find the provider authority. Please fill an issue at https://github.com/grandcentrix/tray/issues
     at android.app.ActivityThread.handleBindApplication(ActivityThread.java:6431)
     at android.app.ActivityThread.access$1800(ActivityThread.java:229)
     at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1887)
     at android.os.Handler.dispatchMessage(Handler.java:102)
     at android.os.Looper.loop(Looper.java:148)
     at android.app.ActivityThread.main(ActivityThread.java:7331)
     at java.lang.reflect.Method.invoke(Method.java)
     at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1230)
     at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1120)
Caused by net.grandcentrix.tray.a.h: Internal tray error. Could not find the provider authority. Please fill an issue at https://github.com/grandcentrix/tray/issues
     at net.grandcentrix.tray.provider.TrayContract.getAuthority(TrayContract.java:138)
     at net.grandcentrix.tray.provider.TrayContract.generateContentUri(TrayContract.java:109)
     at net.grandcentrix.tray.provider.TrayContract.generateContentUri(TrayContract.java:80)
     at net.grandcentrix.tray.provider.TrayUri.(TrayUri.java:75)
     at net.grandcentrix.tray.provider.ContentProviderStorage.(ContentProviderStorage.java:146)
     at net.grandcentrix.tray.TrayPreferences.(TrayPreferences.java:43)
     at net.grandcentrix.tray.TrayPreferences.(TrayPreferences.java:48)
     at net.grandcentrix.tray.AppPreferences.(AppPreferences.java:41)

Description (Just a place with additional information, more == better)

@passsy Thanks for the reply. My project is a library project. Inside this library, I access Tray in the library constructor, but I don't really know how the library users construct my library, I would imagine there is a high chance that a user constructs my library in his/her Apllication#onCreate(), when this happens, we are accessing Tray directly in Apllication#onCreate().

@passsy Is there something I can do on my end to fix the issue?

@OUCHUNYU Could you please update the library to fix this issue?


LCA still gives root in some cases

If I select all Clostridium genomes, the LCA of the selected genomes is still indicated as "root".
Original issue by @pdawyndt on Sun Jun 15 2014 at 12:42.
Closed by @bmesuere on Sun May 03 2015 at 11:28.

This is because one of the uniprot entries is labeled as Clostridium phage phiSM101 which is classified as a virus: http://www.ncbi.nlm.nih.gov/Taxonomy/Browser/wwwtax.cgi?id=396359
Original comment by @bmesuere on Sun Jun 15 2014 at 13:00.

The guilty genome is Clostridium perfringens SM101
Original comment by @bmesuere on Sun Jun 15 2014 at 13:02.

@bmesuere If you check the bioproject page of Clostridium perfringens SM101, you'll see that this project has four sequence records in GenBank/RefSeq, corresponding to

one chromosome (taxon ID: 289380 = Clostridium perfringens SM101)
two plasmids
one bacteriophage = virus infecting a bacterial cell (taxon ID: 396359 = Clostridium phage phiSM101)

As you'll see from the bioproject page of Clostridium perfringens SM101, there's also a taxon ID associated to the bioproject itself: taxon ID 289380 = Clostridium perfringens SM101. So, we should take over the taxon ID from the bioproject, and not from one of the composing parts of the bioproject.
Original comment by @pdawyndt on Sun Jun 15 2014 at 17:32.

This isn't in the database yet. I will add it when the current uniprot update is done.
Original comment by @bmesuere on Sun Jun 15 2014 at 18:00.

Delayed till 2.4 because the format of the source file changed and there's no explanation about the new/changed fields
Original comment by @bmesuere on Wed Jul 16 2014 at 19:06.

This was fixed by switching to assemblies.
Original comment by @bmesuere on Sun May 03 2015 at 11:28.


new: Improved error level style in built-in themes

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 95.65%. Comparing base (c12b938) to head (abbaeca).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #28   +/-   ##
=======================================
  Coverage   95.65%   95.65%
=======================================
  Files          21       21
  Lines        1703     1703
=======================================
  Hits         1629     1629
  Misses         61       61
  Partials       13       13

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


python 3.9 pip install fuzzylogic

Things are getting messy so I thought I would be put python 3.9 on another issue.
I ran into some sort of numpy issue and had to dumb down numpy to 1.19.2 with pip install numpy==1.19.2 to even install numpy on vanilla python 3.9
Anyway pip installed combinators too.
Seems like I get the same error as python 3.8 on the other git issue using anaconda.
>>>
= RESTART: C:\Users\bbartling\OneDrive - Slipstream\Desktop\fuzzyStuff\test_new.py
Traceback (most recent call last):
  File "C:\Users\bbartling\OneDrive - Slipstream\Desktop\fuzzyStuff\test_new.py", line 1, in <module>
    from fuzzylogic.classes import Domain, Set, Rule
  File "C:\Python39\lib\site-packages\fuzzylogic\classes.py", line 13, in <module>
    from combinators import MAX, MIN, bounded_sum, product, simple_disjoint_sum
ImportError: cannot import name 'MAX' from 'combinators' (C:\Python39\lib\site-packages\combinators\__init__.py)
>>>

This is the script:
from fuzzylogic.classes import Domain, Set, Rule
from fuzzylogic.hedges import very

temp = classes.Domain("Temperature", -80, 80)
hum = classes.Domain("Humidity", 0, 100)
motor = classes.Domain("Speed", 0, 2000)

temp.cold = functions.S(0,20)
temp.hot = functions.R(15,30)

hum.dry = functions.S(20,50)
hum.wet = functions.R(40,70)

motor.fast = functions.R(1000,1500)
motor.slow = ~motor.fast

R1 = classes.Rule({(temp.hot, hum.dry): motor.fast})
R2 = classes.Rule({(temp.cold, hum.dry): very(motor.slow)})
R3 = classes.Rule({(temp.hot, hum.wet): very(motor.fast)})
R4 = classes.Rule({(temp.cold, hum.wet): motor.slow})

Rules = classes.Rule({(temp.hot, hum.dry): motor.fast,
                      (temp.cold, hum.dry): very(motor.slow),
                      (temp.hot, hum.wet): very(motor.fast),
                      (temp.cold, hum.wet): motor.slow,
                      })

Rules == R1 | R2 | R3 | R4 == sum([R1, R2, R3, R4])

print(Rules(hum(30), temp(9)))

Thanks Anselm no rush!

I made all imports relative, which should fix this issue. I released a new version on pypi, you can try to pip install it (it works for me like this on a fresh Lubuntu VM with python 3.9).

Hi Anselm
So I upgraded to 1.0.1, could this be an issue running on Windows?
Ultimately hopefully near term future I will be trying to use this on Linux on a real world type test bench.
When I run the code I get an error in py39 on Windows:
Traceback (most recent call last):
  File "C:\Users\bbartling\OneDrive - Slipstream\Desktop\fuzzyStuff\test_new.py", line 4, in <module>
    temp = classes.Domain("Temperature", -80, 80)
NameError: name 'classes' is not defined

Hey Ben,
what's the code you were running?


Adds UI permissions guard for wrangler movethread

Summary
Adds a canWrangler selector, which returns true or false based on server's system settings for Wrangler.
When canWrangler returns false the menu item to move a thread is hidden.
Ticket Link
https://community.mattermost.com/boards/team/rcgiyftm7jyrxnma1osd8zswby/b9phiux686tyiby9eiokyj9n8oh/vjkrq3q44p7887qat7e1pd4ocrr/csnrpoeefc3gturu51pu3nbu5by
Release Note
NONE

/update-branch

/update-branch


Add Prettier

While working on this awesome tool I was really missing auto formatting. In this PR I've added Prettier and I added a pre-commit hook so that all commits will be consistently formatted. I also removed standard as it clashes with the formatting of Prettier, if desired I can add back standard by utilizing eslint-config-prettier which will disable all rules that clash with Prettier.
Let me know if you'd like me to change anything to suit your needs best :)
If you intend to merge this PR you can run yarn prettier afterwards to format all existing files.

Thanks @TimonVS! Happy to switch to Prettier. I've just applied the yarn prettier changes and merged 🙌🏼


feat: Adding IAM PassRole for ECS tasks as it is required for Fargate

Description
If one runs ECS tasks on Fargate, the task execution role mast be passed on. Otherwise one gets a ECS.AccessDeniedException error (more here and here).
Motivation and Context
This PR adds the possibility to reference the task execution role as any other resource in ECS tasks making it simpler for the user no to create erroneous aws_iam_policy_document documents.
Breaking Changes
No.
How Has This Been Tested?

[ ] I have tested and validated these changes using one or more of the provided examples/* projects
Deployed in my own project filling out the

ecs_Sync = {
      ecs = [
        data.aws_ecs_task_definition.xxxx.id
      ]
      ecs_Wildcard = [
        data.aws_ecs_task_definition.xxxx.id
      ]
      iam_PassRole = [
        data.aws_iam_role.ecs_task_execution_role.arn
      ]
    }

which would add the statement
                  + {
                      + Action    = "iam:PassRole"
                      + Condition = {
                          + StringEquals = {
                              + iam:PassedToService = [
                                  + "ecs-tasks.amazonaws.com",
                                ]
                            }
                        }
                      + Effect    = "Allow"
                      + Resource  = "arn:aws:iam::xxxxxx:role/xxxxxxx"
                      + Sid       = "ecsSyncIamPassRole"
                    },

as well as without which results in adding statement to the policy (in that case it would not add or remove)
                  - {
                      - Action    = "iam:PassRole"
                      - Condition = {
                          - StringEquals = {
                              - iam:PassedToService = [
                                  - "ecs-tasks.amazonaws.com",
                                ]
                            }
                        }
                      - Effect    = "Allow"
                      - Resource  = "arn:aws:iam::xxxxxx:role/xxxxxxx"
                      - Sid       = "ecsSyncIamPassRole"
                    },

The results for ecs_WaitForTaskToken is identical.

Thanks for this addition, @kalosisz !
v2.5.0 has been just released.


Allow extension authors to set step, totalSteps of a QuickPick/InputBox in Options

Similar to #119144 @TylerLeonhardt
Allows showQuickPick and showInputBox to be used with step, totalSteps so that extension authors don't need to use the more complex createQuickPick and createInputBox.
Also allow valueSelection to be set with createQuickPick and createInputBox.
Test with this extension- https://github.com/r3m0t/vscode-extension-samples/tree/quickinput/quickinput-sample

I'll bring this to the API review but I won't get hopes too high. We intentionally haven't done this in the past is because the pick API is suppose to be for one-offs picks while the createQuickPick is for longer form quick pick use cases (multi-step).
With this proposal, you'll have to keep track of totalSteps outside of the quickpick and pass that into pick every single time.
I do acknowledge that the pick API is simpler than createQuickPick and this PR enables:

const a = await window.showInputBox({
		title: "Example",
		prompt: "Please select step 1's value",
		step: 1,
		totalSteps: 3,
	}, ['a', 'b', 'c']);

const b = await window.showInputBox({
		title: "Example",
		prompt: "Please select step 2's value",
		step: 2,
		totalSteps: 3,
	}, ['a', 'b', 'c']);

but that's actually 2 pieces data that are duplicated: title and totalSteps... which isn't ideal and would be avoided using createQuickPick.

I'm more interested in valueSelection actually,  which I implemented in this PR. I am using the multi step input and want to set valueSelection on the second step. I just added step to QuickPickOptions for symmetry.

Yes that's right, I remember your issue. I'd rather have valueSelection?: [number, number] on InputBox then totalSteps on pick.

Hi @r3m0t we discussed this in our API review because I wanted to get you an answer for taking this on.
We all feel similar to what I outlined above with the steps, but would 💯 take a PR with the valueSelection. If you update this PR with that, I'd happily get that in!

@TylerLeonhardt there you go, I kept the other branch in r3m0t/quick-pick-steps-c

@TylerLeonhardt all sorted


Material design

Is there any plan to port layout to use the Material Design library angular-material, instead of bootstrap? or, by other means, would it be viable to port on my own fork?

hi @gerardolima! I don't think that bootstrap should be moved to material as it is still a widely used framework. Instead, I think you could support both by configuring it in the provider and creating the proper material templates. What do you think?

on the "adf.js" file is injecting 'ui.bootstrap'.
Having both could lead to layout problems.
It's possible to override the templates, but this dependency force to use bootstrap, only because internal default templates, right?

I think it would be good to let the views (client-code) choose their preferred layout library, @raulgomis. As @jcvpacheco said, the current implementation requires bootstrap and mixing other library for layout will surely cause incompatibilities errors.

I think we need a way for supporting multiple ui frameworks, because this is not the first question for another ui framework #145. But supporting multiple ui frameworks sounds like a hard business, but i think it is possible.
We need something like an abstract module for the ui components, with this we could remove the direct dependency to ui-boostrap from adf. Then we could create a module for each ui framework which implements the abstract module. The adf user must include the adf module and one ui-implementation module.
What do you think?

It makes sense to me to have the views inject the layout library they prefer. Unfortunately, though, I don't know the codebase well enough yet to assert the complexity of these changes.

Is a bit trivial but there are some kind of layouts like this http://bootswatch.com/paper/ that can help to mimic this behaviour with bootstrap


[BUG] Users are unable to create RKE2 vSphere Node Driver clusters with more than 1 Windows worker node

Rancher Server Setup

Rancher version: v2.6.5 and v2.6.8
Installation option (Docker install/Helm Chart): Docker

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): N/A

Proxy/Cert Details: N/A

Information about the Cluster

Kubernetes version: v1.24.6+rke2r1
Cluster Type (Local/Downstream): Downstream

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider): Node Driver (Infrastructure Provider)

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom): Standard User with Cluster Owner Role

If custom, define the set of permissions: N/A

Describe the bug
When attempting to create a vSphere Node Driver cluster with more than one Windows worker node or when attempting to scale up a second Windows worker node on an existing cluster, only the first Windows worker node will provision successfully.
To Reproduce
Scenario 1:

On a vSphere Hosted instance of Rancher, attempt to create a vSphere Node Driver cluster with 3 Linux ETCD nodes, 2 Linux Control Plane Nodes, 2 Linux Worker Nodes, and 2 Windows worker nodes

Scenarios 2:

On a vSphere Hosted instance of Rancher, create a vSphere Node Driver cluster with 3 Linux ETCD nodes, 2 Linux Control Plane Nodes, 2 Linux Worker Nodes, and 1 Windows worker node
Once the cluster becomes active, attempt to Scale up a Second Windows worker node

Scenario 3:

On a vSphere Hosted instance of Rancher, attempt to create a vSphere Node Driver cluster with 3 Linux ETCD nodes, 2 Linux Control Plane Nodes, 2 Linux Worker Nodes, 1 Windows worker node, and 1 Windows worker node (separate node pool) with the same OS Template as the first.

Scenario 4:

On a vSphere Hosted instance of Rancher, attempt to create a vSphere Node Driver cluster with 3 Linux ETCD nodes, 2 Linux Control Plane Nodes, 2 Linux Worker Nodes, 1 Windows worker node, and 1 Windows worker node (separate node pool) with a different Windows OS Template/Version from the first.

Result
The first 3 scenarios result in the first Windows worker node being Active, but the second (and any subsequent ones after that) being stuck in a Reconciling state. However, in the 4th scenario, all Windows nodes will become active.
Expected Result
Users are able to provision vSphere Node Driver clusters with multiple Windows worker nodes without issue.
Screenshots
Windows 2022 Template:

Windows 2019 Template:

Additional context
This issue will reproduce for all supported versions of the Windows Operating System.
It was observed during testing that the Node Name, Hostname, and DNS of the Windows worker nodes are the same for every node every time dependent upon the OS Template being used (refer to the Screenshots). We were unable to confirm during testing, but it is our understanding that all of the Windows Templates in our setup have been properly Sysprepped/Generalized during their creation (though we were not able to confirm if this is true about the User that initially reported this issue).
SURE-5319

Moved back into working to investigate an issue where windows vms are rebooted.

The issue mentioned above is now tracked separately as https://github.com/rancher/rancher/issues/39658, moving this one back to test.

During testing on v2.6.5, I observed the set_hostname error in the Cloudbase-Init logs for the Windows Nodes. However, during testing on v2.7-head (Commit ID: ae44c39), I no longer observed the error in the logs, and I was able to successfully provision vSphere RKE2 clusters with multiple Windows nodes following each of the previously failing Scenarios in the Description's Steps to Reproduce. Therefore I am closing this ticket.
v2.6.5:
CloudBase-Init Logs v265.txt
v2.7-head (Commit ID: ae44c39):
CloudBase-Init Logs v27head.txt

/backport 2023-Q1-v2.6x


EOF error when working with confluent_kafka_topic resource

I can't get a minimal example creating a new topic working due to an unclear EOF error even though I spent quite some time researching it. My main.tf file (simplified to a minimal example) looks as follows:
terraform {
  required_providers {
    confluent = {
      source  = "confluentinc/confluent"
      version = "1.25.0"
    }
  }
}

provider "confluent" {
  kafka_id            = var.kafka_id
  kafka_rest_endpoint = var.kafka_rest_endpoint
  kafka_api_key       = var.kafka_api_key
  kafka_api_secret    = var.kafka_api_secret
}

resource "confluent_kafka_topic" "my_terraform_topic" {
    topic_name       = "my_terraform_topic"
    partitions_count = 1
}

where
kafka_id = XXX-XXXXX, and
kafka_rest_endpoint = https://YYY-YYYYY.germanywestcentral.azure.confluent.cloud:9092.
kafka_api_key and kafka_api_secret are 16 and 64 character long strings, respectively.
Upon terraform apply, I get the following error:

Error: error creating Kafka Topic: Post "https://YYY-YYYYY.germanywestcentral.azure.confluent.cloud:9092/kafka/v3/clusters/XXX-XXXXX/topics": POST https://YYY-YYYYY.germanywestcentral.azure.confluent.cloud:9092/kafka/v3/clusters/XXX-XXXXX/topics giving up after 5 attempt(s): Post "https://YYY-YYYYY.germanywestcentral.azure.confluent.cloud:9092/kafka/v3/clusters/XXX-XXXXX/topics": EOF
│
│   with confluent_kafka_topic.my_terraform_topic,
│   on main.tf line 17, in resource "confluent_kafka_topic" "my_terraform_topic":
│   17: resource "confluent_kafka_topic" "my_terraform_topic" {

What is going on here?
Some more information:
I also tried to set
  cloud_api_key    = var.kafka_api_key
  cloud_api_secret = var.kafka_api_secret

(which should only be necessary for confluent_kafka_acl resources according to the docs, but I am not sure where to get the cloud api keys if the are different from the kafka api keys), but to no avail.
On the other hand, setting credentials for the schema registry and deploying schemas is working (showing that the general setup is working?), but I left it out here as it seem irrelevant.
Terraform version:
terraform --version
Terraform v1.3.7
on darwin_arm64
+ provider registry.terraform.io/confluentinc/confluent v1.25.0

If I try to import an existing topic, I get the same error from the Get request.

@antoncyrol thanks for creating this issue!
It's pretty rare but we've seen this issues a few times already.
Could you try to use
provider "confluent" {
   max_retries = 10 # defaults to 4; or any other number other than 10
}

also could you please share your cluster ID (lkc-xxxxx) to<EMAIL_ADDRESS>so we could investigate on our side (backend / proxy setup) it as well?

Let me know if it helps!

The retries didn't help and I don't have any proxy, VPN, etc. in place.
However, your hint with the ports directly yields the solution:
The kafka_rest_endpoint must not include the port, i.e., using
https://YYY-YYYYY.germanywestcentral.azure.confluent.cloud instead of
https://YYY-YYYYY.germanywestcentral.azure.confluent.cloud:9092
works without any retries.
When you create an API key via the browser, the bootstrap server in the downloaded file includes the port, which is probably the reason some people like me get it wrong. Libraries like kstreams or the Confluent Python Kafka client accept (need?) the bootstrap server with the port.
Thanks for the quick and useful answer!

@antoncyrol that's a great catch!
That's smart fix! Was :9092 displayed in the UI or something? I wonder where :9092 come from in the first place.

You must not overlook the difference between the bootstrap server and the REST endpoint (and, somewhat misleading, you are only provided the bootstrap server in the .txt file that you may download when you create an API key for use with TF):

I think it could be worth validating the kafka_rest_endpoint in the Terraform provider (check that it starts with https:// and ends with no or the correct port) if it is easily possible.


No service discovered with nRF Connect Android App

I setup bleno on an Ubuntu 16.04 system and trying to discover the examples/battery-service with the nRF Connect App under Android. The device scan shows the 'Battery' device but no service can be discovered. This is the log of the nRF Connect App:
nRF Connect, 2017-12-12
Battery (XX:XX:XX:XX:XX:XX)
V	10:37:54.807	Connecting to XX:XX:XX:XX:XX:XX...
D	10:37:54.807	gatt = device.connectGatt(autoConnect = false, TRANSPORT_LE)
D	10:37:54.869	[Broadcast] Action received: android.bluetooth.device.action.ACL_CONNECTED
D	10:37:54.872	[Callback] Connection state changed with status: 0 and new state: CONNECTED (2)
I	10:37:54.872	Connected to XX:XX:XX:XX:XX:XX
V	10:37:54.900	Discovering services...
D	10:37:54.900	gatt.discoverServices()
D	10:38:25.277	[Broadcast] Action received: android.bluetooth.device.action.ACL_DISCONNECTED
D	10:38:25.282	[Callback] Connection state changed with status: 22 and new state: DISCONNECTED (0)
I	10:38:25.284	Disconnected
E	10:38:25.292	Error 22 (0x16): GATT CONN TERMINATE LOCAL HOST

I see to be having a similar issue.   When I run nRFConnect I also see no services.  Here is the nRFConnect trace:
V	15:52:49.675	Connecting to 00:1A:7D:DA:71:0E...
D	15:52:49.675	gatt = device.connectGatt(autoConnect = false, TRANSPORT_LE)
D	15:52:50.266	[Server callback] Connection state changed with status: 0 and new state: CONNECTED (2)
I	15:52:50.266	[Server] Device with address 00:1A:7D:DA:71:0E connected
V	15:52:50.266	[Server] Creating server connection...
D	15:52:50.266	server.connect(device, autoConnect = false)
I	15:52:50.278	[Server] MTU changed to 517
D	15:52:50.366	[Broadcast] Action received: android.bluetooth.device.action.ACL_CONNECTED
D	15:52:50.370	[Callback] Connection state changed with status: 0 and new state: CONNECTED (2)
I	15:52:50.370	Connected to 00:1A:7D:DA:71:0E
V	15:52:50.384	Discovering services...
D	15:52:50.384	gatt.discoverServices()
D	15:52:50.415	[Callback] Services discovered with status: 0
I	15:52:50.415	Services discovered
V	15:52:50.415	No services found

and the application I am running:
var bleno = require("bleno");
var name = "BLENO";
bleno.on("stateChange", (state) => {
	console.log("onStateChange: " + state);
	if (state === "poweredOn") {
		bleno.startAdvertising(name, ["91bad492b9504226aa2b4ede9fa42f59"], (error) => {
			console.log("Advertising cb: " + error);
		});
		console.log("Started advertising");
	}
});
bleno.on("advertisingStart", (error) => {
	console.log("Adverising started!: " + error);
	bleno.setServices([ new bleno.PrimaryService({
		uuid: "91bad492b9504226aa2b4ede9fa42f59",
		characteristics: [ new bleno.Characteristic({
			uuid: "4fafc2011fb5459e8fccc5c9c331914b",
			properties: ["read", "write", "writeWithoutResponse", "notify", "indicate"],
			value: null
		})]

	})], (error) => {
		console.log("setServices: " + error);
	});
});

Hi all,
I still have the same issue. Please see below for my btmon log. I cannot reopen this issue, too (can not find an option on this web page to re-open the issue)
$ uname -a
Linux vbox 4.10.0-42-generic #46~16.04.1-Ubuntu SMP Mon Dec 4 15:57:59 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
$ btmon --write ~/hcitrace.snoop
Bluetooth monitor ver 5.37
= Note: Linux version 4.10.0-42-generic (x86_64)                                                                                0.427422
= Note: Bluetooth subsystem version 2.22                                                                                0.427425
= New Index: 5C:F3:70:61:B7:73 (BR/EDR,USB,hci0)                                                                                [hci0] 0.427426
= Open Index: 5C:F3:70:61:B7:73                                                                                [hci0] 0.427427
= Index Info: 5C:F3:70:61:B7:73 (Broadcom Corporation)                                                                                [hci0] 0.427427

Unknown packet (code 14 len 30)                                                                                0.427428
01 00 00 00 02 00 01 0e 00 01 00 00 00 10 62 6c  ..............bl
75 65 74 6f 6f 74 68 64 00 00 00 00 00 00        uetoothd......
Unknown packet (code 14 len 30)                                                                                0.427438
02 00 00 00 02 00 01 0e 00 01 00 00 00 10 62 74  ..............bt
6d 6f 6e 00 00 00 00 00 00 00 00 00 00 00        mon...........
Unknown packet (code 14 len 30)                                                                                16.007704
03 00 00 00 02 00 01 0e 00 01 00 00 00 10 6e 6f  ..............no
64 65 00 00 00 00 00 00 00 00 00 00 00 00        de............
Unknown packet (code 14 len 30)                                                                                16.019121
04 00 00 00 00 00 02 16 00 01 00 00 00 10 6e 6f  ..............no
64 65 00 00 00 00 00 00 00 00 00 00 00 00        de............
Unknown packet (code 15 len 4)                                                                                16.019410
04 00 00 00                                      ....
Unknown packet (code 14 len 30)                                                                                [hci0] 16.019412
04 00 00 00 00 00 02 16 00 01 00 00 00 10 6e 6f  ..............no
64 65 00 00 00 00 00 00 00 00 00 00 00 00        de............
< HCI Command: Set Event Mask (0x03|0x0001) plen 8                                                                                [hci0] 16.021167
Mask: 0x3dbff807fffbffff
Inquiry Complete
Inquiry Result
Connection Complete
Connection Request
Disconnection Complete
Authentication Complete
Remote Name Request Complete
Encryption Change
Change Connection Link Key Complete
Master Link Key Complete
Read Remote Supported Features Complete
Read Remote Version Information Complete
QoS Setup Complete
Command Complete
Command Status
Hardware Error
Flush Occurred
Role Change
Mode Change
Return Link Keys
PIN Code Request
Link Key Request
Link Key Notification
Loopback Command
Data Buffer Overflow
Max Slots Change
Read Clock Offset Complete
Connection Packet Type Changed
QoS Violation
Page Scan Mode Change
Page Scan Repetition Mode Change
Flow Specification Complete
Inquiry Result with RSSI
Read Remote Extended Features Complete
Synchronous Connection Complete
Synchronous Connection Changed
Sniff Subrating
Extended Inquiry Result
Encryption Key Refresh Complete
IO Capability Request
IO Capability Request Reply
User Confirmation Request
User Passkey Request
Remote OOB Data Request
Simple Pairing Complete
Link Supervision Timeout Changed
Enhanced Flush Complete
User Passkey Notification
Keypress Notification
Remote Host Supported Features Notification
LE Meta

HCI Event: Command Complete (0x0e) plen 4                                                                                [hci0] 16.032475
Set Event Mask (0x03|0x0001) ncmd 1
Status: Success (0x00)
< HCI Command: LE Set Event Mask (0x08|0x0001) plen 8                                                                                [hci0] 16.032520
Mask: 0x000000000000001f
LE Connection Complete
LE Advertising Report
LE Connection Update Complete
LE Read Remote Used Features Complete
LE Long Term Key Request
HCI Event: Command Complete (0x0e) plen 4                                                                                [hci0] 16.034262
LE Set Event Mask (0x08|0x0001) ncmd 1
Status: Success (0x00)
< HCI Command: Read Local Version Information (0x04|0x0001) plen 0                                                                                [hci0] 16.034300
HCI Event: Command Complete (0x0e) plen 12                                                                                [hci0] 16.036260
Read Local Version Information (0x04|0x0001) ncmd 1
Status: Success (0x00)
HCI version: Bluetooth 4.0 (0x06) - Revision 4096 (0x1000)
LMP version: Bluetooth 4.0 (0x06) - Subversion 8718 (0x220e)
Manufacturer: Broadcom Corporation (15)
Firmware: 001.002.014 (BCM20702A1)
Build: 0000
< HCI Command: Write LE Host Supported (0x03|0x006d) plen 2                                                                                [hci0] 16.036299
Supported: 0x01
Simultaneous: 0x00
HCI Event: Command Complete (0x0e) plen 4                                                                                [hci0] 16.038487
Write LE Host Supported (0x03|0x006d) ncmd 1
Status: Success (0x00)
< HCI Command: Read LE Host Supported (0x03|0x006c) plen 0                                                                                [hci0] 16.038551
HCI Event: Command Complete (0x0e) plen 6                                                                                [hci0] 16.040297
Read LE Host Supported (0x03|0x006c) ncmd 1
Status: Success (0x00)
Supported: 0x01
Simultaneous: 0x00
< HCI Command: Read BD ADDR (0x04|0x0009) plen 0                                                                                [hci0] 16.040325
HCI Event: Command Complete (0x0e) plen 10                                                                                [hci0] 16.043283
Read BD ADDR (0x04|0x0009) ncmd 1
Status: Success (0x00)
Address: 5C:F3:70:61:B7:73 (CC&C Technologies, Inc)
< HCI Command: LE Set Advertise Enable (0x08|0x000a) plen 1                                                                                [hci0] 16.043424
Advertising: Disabled (0x00)
HCI Event: Command Complete (0x0e) plen 4                                                                                [hci0] 16.047440
LE Set Advertise Enable (0x08|0x000a) ncmd 1
Status: Command Disallowed (0x0c)
< HCI Command: LE Set Advertising Parameters (0x08|0x0006) plen 15                                                                                [hci0] 16.047594
Min advertising interval: 100.000 msec (0x00a0)
Max advertising interval: 100.000 msec (0x00a0)
Type: Connectable undirected - ADV_IND (0x00)
Own address type: Public (0x00)
Direct address type: Public (0x00)
Direct address: 00:00:00:00:00:00 (OUI 00-00-00)
Channel map: 37, 38, 39 (0x07)
Filter policy: Allow Scan Request from Any, Allow Connect Request from Any (0x00)
HCI Event: Command Complete (0x0e) plen 4                                                                                [hci0] 16.050912
LE Set Advertising Parameters (0x08|0x0006) ncmd 1
Status: Success (0x00)
< HCI Command: LE Set Scan Response Data (0x08|0x0009) plen 32                                                                                [hci0] 16.062085
Length: 9
Name (short): Battery
HCI Event: Command Complete (0x0e) plen 4                                                                                [hci0] 16.065722
LE Set Scan Response Data (0x08|0x0009) ncmd 1
Status: Success (0x00)
< HCI Command: LE Set Advertising Data (0x08|0x0008) plen 32                                                                                [hci0] 16.065809
Length: 7
Flags: 0x06
LE General Discoverable Mode
BR/EDR Not Supported
16-bit Service UUIDs (complete): 1 entry
Battery Service (0x180f)
HCI Event: Command Complete (0x0e) plen 4                                                                                [hci0] 16.071281
LE Set Advertising Data (0x08|0x0008) ncmd 1
Status: Success (0x00)
< HCI Command: LE Set Advertise Enable (0x08|0x000a) plen 1                                                                                [hci0] 16.071330
Advertising: Enabled (0x01)
HCI Event: Command Complete (0x0e) plen 4                                                                                [hci0] 16.075426
LE Set Advertise Enable (0x08|0x000a) ncmd 1
Status: Success (0x00)
< HCI Command: LE Set Scan Response Data (0x08|0x0009) plen 32                                                                                [hci0] 16.075553
Length: 9
Name (short): Battery
HCI Event: Command Complete (0x0e) plen 4                                                                                [hci0] 16.078462
LE Set Scan Response Data (0x08|0x0009) ncmd 1
Status: Success (0x00)
< HCI Command: LE Set Advertising Data (0x08|0x0008) plen 32                                                                                [hci0] 16.078491
Length: 7
Flags: 0x06
LE General Discoverable Mode
BR/EDR Not Supported
16-bit Service UUIDs (complete): 1 entry
Battery Service (0x180f)
HCI Event: Command Complete (0x0e) plen 4                                                                                [hci0] 16.081244
LE Set Advertising Data (0x08|0x0008) ncmd 1
Status: Success (0x00)

@rena2019 it still looks like bluetoothd is running, unless your bleno app sets the battery service.
Please share your bleno app if you have stopped bluetoothd


I’d like the openlyrics component interface to be displayed separately on the top level

Openlyrics is a good component for foobar。
Is it possible to separately keep the openlyrics component interface （Openlyrics Panel） displayed at the top level when the main foobar interface is hidden?
This feature is important for people like us who like to listen to songs in an immersive way。
Thanks！

Oh， I know how to do this, there is an Open external window option.


Add an upgrade section in the README

We are missing a simple upgrade section in the README.

Implemented


fix(tabs-extended): tabs extended not rendering when in 1056 breakpoint

Related Ticket(s)
Closes #
ADCMS-4241
Description
Tab extended component is not rendering the tabs section when breakpoint is exactly 1056px, if author sets the browser simulator at 1057 and other then tab extended does render properly the same is true for a breaking point below like 1055 and below then it the component will render properly.
Changelog
Changed

Issue was fixed by changing the way of checking the breakpoint to render the mobile or the desktop version

Tabs Extended with Media also fixed! Ready for review


Component: Slider

Figma
https://www.figma.com/file/xZ0IW0MJO0vnFicmrHiKaY/Components-Post?type=design&node-id=16545%3A53351&mode=design&t=UhNqX9sEMKDoYiIW-1
old
https://www.figma.com/file/KDAqx5qYIhMt5YnB7HJVSE/post_library_atoms?type=design&node-id=0%3A971&mode=design&t=3AC7rWArgkfhvKj8-1
Code
https://next.design-system.post.ch/?path=/docs/components-range--docs
https://design-system.post.ch/#/bootstrap-samples/form-range
Doc
https://www.experience-hub.ch/document/2803#/atoms-basis-elemente/controls/slider

Review is done, I think we need to discuss with the dev team about double value sliders because the single variant is easy to implement with html + css, the double valued needs to be a web component.

Discuss how we want to release or document double valued range sliders in figma
Adress comments in figma

created the following tickets:

https://github.com/swisspost/design-system/issues/2242
https://github.com/swisspost/design-system/issues/2243

I updated the component tracker board (https://github.com/orgs/swisspost/projects/4/views/1?pane=issue&itemId=14098527) and created a dev task https://github.com/swisspost/design-system/issues/2297


Bug in example direction cosine

Can you please rebase this topic onto the current master.

Pull request has been rebased.

LGTM 👍


MariaDB apt_repository failed with UK repo

When adding the MariaDB Ubuntu repo for my nearest mirror the apt_repository module failed:
W: Failed to fetch http://mirrors.coreix.net/mariadb/repo/10.0/ubuntu/dists/trusty/main/i18n/Translation-en_US  Unable to connect to mirrors.coreix.net:http: [IP: <IP_ADDRESS> 80]
W: Failed to fetch http://mirrors.coreix.net/mariadb/repo/10.0/ubuntu/dists/trusty/main/i18n/Translation-en  Unable to connect to mirrors.coreix.net:http: [IP: <IP_ADDRESS> 80]

The MariaDB download site says that the repos should be:
deb http://mirrors.coreix.net/mariadb/repo/10.0/ubuntu trusty main
deb-src http://mirrors.coreix.net/mariadb/repo/10.0/ubuntu trusty main

So it looks like dists/trusty/main/i18n/Translation-en_US is coming from the mariadb_version variable which is setup in roles/mariadb/defaults/main.yml. I couldn't track down how the playbook was selecting the closest mirror for the MariaDB Ubuntu repos or how it was overriding the mariadb_version variable.

Those repos failing aren't the main ones. We are adding the correct ones here
mariadb_dist  = trusty set here.
Those failures you get are "sub" repos that are fetched after the main ones are added. I've actually had those errors before as well once or twice but they went away on their own. Have you tried again since then? Or maybe another mirror?


Add run-wasm and --include-all-builtins

This pr adds the command run-wasm and the option --include-all-builtins:

run-wasm: Runs a compiled wasm binary. Warns if the binary doesn't have debug information and provide error ids instead of messages
--include-all-builtins: Forces all builtins to be included into the wasm. Use porf --include-all-builtins -e ';' -b to get the size of all builtins together in one wasm.
Also fixes #187.

Also adds --size-log and --size-log-count:

--size-log: Show the functions with the highest amount of instructions
--size-log-count: How many instructions to show for --size-log


Remove size property from sections

The size property is unused in section objects. Requiring it to be specified does not pose any inconvenience for reading, but it would make it very difficult to construct sections for writing.

I was thinking about this today after your prev PR.
If it doesn't hurt we can keep the additional argument with the hope that this library will, one day, become a useful foundation for reading and writing wasm Modules on the JVM.
That said, if you have a good argument like perf or similar I'm open to learning more.

We need to remove it if we want to be able to support writing. The reason is that you will not know the size until all elements have been added, and their indices are known... i.e. it's a late computation which can change if and when we support adding elements to sections.

I leave the last word here to @bhelx just in case he left the sizes around for a reason.

Yes, this makes sense. We need to know the size mostly for skipping sections, but it doesn't need to be stored on the Section. Also, I'd love to start on a wasm encoder. It's gonna be key to building other tooling and I just put it off for the time being.


Availability Zones Messages and events are or are not replicated?

Hi, can you please describe more detail about EventHub and Availability Zones? Is it same functionality like Geo-disaster (Messages and events are NOT replicated), or does it replicate events between zones? Thank you.

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

ID: 863a845f-4b06-126e-3cd3-88875fadf8e4
Version Independent ID: 2ab93b2c-5796-1623-3396-f294d9c469b5
Content: Geo-disaster recovery - Azure Event Hubs
Content Source: articles/event-hubs/event-hubs-geo-dr.md
Service: event-hubs
GitHub Login: @ShubhaVijayasarathy
Microsoft Alias: shvija

@eerik47 Thanks for the feedback. We are looking into your query and will update you shortly.

@eerik47 Availability in Event hubs :
Azure Event Hubs uses a partitioning model to improve availability and parallelization within a single event hub. For example, if an event hub has four partitions, and one of those partitions is moved from one server to another in a load balancing operation, you can still send and receive from three other partitions. Additionally, having more partitions enables you to have more concurrent readers processing your data, improving your aggregate throughput. Understanding the implications of partitioning and ordering in a distributed system is a critical aspect of solution design.
For more information on Disaster Recovery, please go through the following article :
https://docs.microsoft.com/en-us/azure/architecture/framework/resiliency/backup-and-recovery#disaster-recovery-plan
Hope this helps.

@eerik47 Since I haven't heard back from you in quite some time, I am assuming that your query was resolved.
I will proceed to close this thread. Feel free to still comment and tag me if you have any questions on the same.

@eerik47 Availability in Event hubs :
Azure Event Hubs uses a partitioning model to improve availability and parallelization within a single event hub. For example, if an event hub has four partitions, and one of those partitions is moved from one server to another in a load balancing operation, you can still send and receive from three other partitions. Additionally, having more partitions enables you to have more concurrent readers processing your data, improving your aggregate throughput. Understanding the implications of partitioning and ordering in a distributed system is a critical aspect of solution design.
For more information on Disaster Recovery, please go through the following article :
https://docs.microsoft.com/en-us/azure/architecture/framework/resiliency/backup-and-recovery#disaster-recovery-plan
Hope this helps.

This answer is not at all what the OP was asking. The OP is asking for more specifics on how Microsoft has implemented Availability Zone support for Event Hub.
I'm chiming in because I'm looking for the same answer.

I am looking at the Service Bus documentation and it says both metadata and data are replicated when the Availability Zone support is enabled.
https://docs.microsoft.com/en-us/azure/service-bus-messaging/service-bus-outages-disasters#availability-zones
"Service Bus manages three copies of messaging store (1 primary and 2 secondary). Service Bus keeps all the three copies in sync for data and management operations."

Just confirmed with the product team that
"We replicate both metadata and data (events) across zones when Availability Zone support for an event hub is enabled"


[WIP] Add support for dep, the official package management experiment

TODO

[ ] Write up a better summary and explanation
[ ] Parse metadata from Gopkg.toml instead of my adhoc heroku.json
[ ] Support vendorless repositories by calling dep ensure
[ ] Update the documentation to explain the metadata options
[ ] Update the test suite to include dep examples

Thanks for creating this ticket! How can I help?

FWIW: I started a branch at gophercon using https://github.com/heroku/tq for querying toml. I just pushed that branch (it's not done yet). I think there are parts from both branches that are useful, but I have to spend more time looking into yours.

@freeformz @stran12 I need help with a few things. The buildpack downloads binaries from an official Heroku S3 bucket. Can we work on getting tq and dep uploaded to that bucket? Once that happens, testing this will be much easier.

@freeformz looking over your branch, I think we should just go with yours. The only thing I noticed is your use of the metadata field in Gopkg.toml. According to the dep docs, metadata should take the following form:
[metadata]
key1 = "value that convey data to other systems"
system1-data = "value that is used by a system"
system2-data = "value that is used by another system"

So I think we should use the following values
[metadata]
heroku-root-package = "github.com/heroku/fixture"

@kyleconroy @freeformz Unfortunately, I do not have access to that Heroku S3 bucket neither.

@kyleconroy That is not how it works currently because dep complained. I'll have to update my dep and try again. I hope to take another pass at this later this week

Let me know if I can help in some way on this one!
dep is suggesting that although it's still not an official tool, it's an official experiment and safe for production ready. I was going through all my Go projects this morning and updating their dependency management from govendor to dep before running into this (it was remarkably stable and problem-free otherwise).

Is this PR still going forward?

Sadly I'm still stuck on this PR. For us to add dep support, we need two binaries added to the S3 bucket. @brandur Do you know who has access to do that? I can updated files.json with the appropriate entries, but I need someone to run the sync script

Maybe @freeformz can help with this point?

@kyleconroy I added the current versions of dep and tq (a small tomlq like tool that I/we wrote) to the s3 bucket and updated files.json. Can you rebase and change this PR to work off those files/tools?

PS: I'm now better positioned and committed to helping you get this merged.
I'm extremely sorry for the delays.

Edward,
Thank you so much. Testing this should be much easier now.
On Thu, Sep 21, 2017 at 12:07 PM Edward Muller<EMAIL_ADDRESS>wrote:

PS: I'm now better positioned and committed to helping you get this merged.
I'm extremely sorry for the delays.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/heroku/heroku-buildpack-go/pull/196#issuecomment-331252296,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AACITVXpouwI8CG2Qd2L91AEZoOLHudnks5skrP2gaJpZM4OZJXY
.

@kyleconroy My own branch is coming along: https://github.com/heroku/heroku-buildpack-go/tree/dep

@kyleconroy What do you think about closing this now that #210 is merged?
If some functionality is missing in master now I'll also happily consider a PR.
Thanks!

Thanks for getting your branch over the finish line. I ended up not having the time.


unable to find definition in some cases

I find some cases it cannot find definition when dealing with this antique code. I will try if I can isolate the problem. Does this remind you of anything?

If I try multiple times, it might or might not go to definition. If I select the name, it can always go to the definition.

The definition could be found and shown in the pop out. But if I try again, it disappeared.

Hmmm, unfortunately I have not seen this type of behavior. It looks like you are using Atom, however the Atom IDE support that the language server uses is currently in limbo. Can you try VSCode or another editor and see if you get the same issues?

@hansec I'm using vscode. I would try to isolate the problem so you can have a try.

Gotcha, that was the only immediate idea I had. Sometimes I have noticed a lag in definitions, hover and other requesting being available after an edit, but never just appearing and disappearing without file changes.
One thing to look into is using the debug log option to see if the definition request is getting a response or if the "missing" result is due to the response taking too long. Make sure you turn off debug_log when you are done, as it can generate rather large files if you are editing for an extended period of time with it enabled.

@lonelykid Have you been able to get any additional information on this issue? Are you still seeing it?

This is what happened when I try going to definition directly
DEBUG:fortls.jsonrpc:RECV {"jsonrpc":"2.0","id":5,"method":"textDocument/definition","params":{"textDocument":{"uri":"file:///c%3A/Users/xxx/xxx/Main.f90"},"position":{"line":303,"character":10}}}
DEBUG:fortls.langserver:REQUEST 5 textDocument/definition
DEBUG:fortls.jsonrpc:SEND {"jsonrpc":"2.0","id":5,"result":null}
This is when I selected the name and tried going to defintion
DEBUG:fortls.jsonrpc:RECV {"jsonrpc":"2.0","id":8,"method":"textDocument/definition","params":{"textDocument":{"uri":"file:///c%3A/Users/xxx/xxx/Main.f90"},"position":{"line":303,"character":15}}}
DEBUG:fortls.langserver:REQUEST 8 textDocument/definition
DEBUG:fortls.jsonrpc:SEND {"jsonrpc":"2.0","id":8,"result":{"uri":"file:///c%3A/Users/xxx/xxx/Marquardt.f90","range":{"start":{"line":14,"character":11},"end":{"line":14,"character":17}}}}
Then I find out this depends on where I move my cursor to.

Hmmm, is there any chance you could reproduce this in a smaller test case where you could send me the code?

I spent some time to figure out what's wrong. It turns out this is due to indentation. Converting indentation to spaces makes it work.
demo1.zip

Great, this should now be fixed in master and will be included in the next patch. Let me know if you continue to observe this issue. Thanks for the report and debugging!

Thank you! I can confirm it is fixed.


Bump dependencies

Bump openpyxl
Bump black pre-commit hook

Codecov Report

Merging #167 (61f979e) into main (3773acd) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main      #167   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines          184       184
=========================================
  Hits           184       184

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 3773acd...61f979e. Read the comment docs.


[release-0.20] Remove flux helm chart

Description of changes:
This is a manual cherry-pick of this commit in #3825.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/cherry-pick release-0.19

/approve


ARROW-15658: [C++] Parquet pushdown filtering fails if the filter expression uses numeric field references

This PR contains an initial step towards a fix for ARROW-15658

[x] Added a fix to enable numeric field references
[x] Adopted a test case from the JIRA description by @westonpace

I think you accidentally pushed some submodule changes.

@lidavidm I updated the test cases and added a check for the nested field paths. To add an error checking test, I think some of the util functions need to be changed to grab a returning NotImplemented status. The SetFilter function call triggers the error. Is it okay to return a Status from that method. I didn't do the change but would like to know if it is okay to do. I can proceed with it if it is okay. But is there a better alternative here?

Closing because it has been untouched for a while, in case it's still relevant feel free to reopen and move it forward 👍


Contact Page UI Changed successfully

I have sucessfully changed UI of contact page,
Closes Issue #70
Earlier page was looking as :-

Now the contact page look as :-

Kindly merge my PR

Hey @hkaur008 could you please check what's exactly the issue is with this PR, if possible could you please resolve it and merge my PR


Fix deprecated warnings in some specs

I hope to be useful.
Regards

There was a recent discussion about this. The idea is to test the deprecated methods to see that they still work because people might still be using them. We should only remove them once we remove the associated methods. So I'm not sure we'll merge this PR or similar PRs.

All of this to say, deprecation warnings in specs are not a big deal.

I see, very interesting discussion.
@asterite feel free to close the pull request

The motivation is to not have untested code in the std-lib since it might lead to code that does not actually compile.
Sometimes specs needs to be updated to get rid of the deprecation warning. That is a good change. This is the case when the spec uses the method but is not the tested method.
In other cases, the spec should be already present for the replacement alternative and probably the spec can be removed once the deprecation method is effectively removed.