fix(editor): Fix exception in BufferLine.getPositionAndWidth with negative index

Im curious, how did you come across this? I think I experienced it once with a Hover crash but I couldn't replicate it ever again.

Im curious, how did you come across this? I think I experienced it once with a Hover crash but I couldn't replicate it ever again.

Hmm, ya, I was experiencing some intermittent crashes:
[ERROR]    +7ms Oni2.Exception : Exception (Invalid_argument "index out of bounds"):
Raised by primitive operation at file "src/Core/BufferLine.re", line 198, characters 11-30
Called from file "src/Feature/Editor/Editor.re", line 84, characters 6-108
Called from file "src/Feature/Hover/Feature_Hover.re", line 371, characters 10-211
Called from file "src/UI/EditorView.re", line 94, characters 10-420
Called from file "src/Feature/Editor/EditorSurface.re", line 314, characters 4-34
Called from file "lib/Brisk_reconciler_internal.re", line 831, characters 12-205
Called from file "lib/Brisk_reconciler_internal.re", line 735, characters 12-235
Called from file "lib/Brisk_reconciler_internal.re", line 1346, characters 14-401
Called from file "lib/Brisk_reconciler_internal.re", line 1090, characters 16-497
Called from file "lib/ListTR.re", line 32, characters 22-40
Called from file "lib/Brisk_reconciler_internal.re", line 1067

I figured it might've been related to my change here: https://github.com/onivim/oni2/blame/master/src/Feature/Hover/Feature_Hover.re - the only case I think of that would hit the callstack would be somehow getting a negative character index.

Makes sense! It appeared for me around the same time as the completion being off-positioned.

Makes sense! It appeared for me around the same time as the completion being off-positioned.

Yep, same change was responsible for that 😅


fix(runtime-core): should not take unmount children fast path if children contain cached nodes

close #5154

The direction is right, but checking only the direct children of the current vnode does not account for deeply nested v-once cached nodes.
In https://github.com/vuejs/core/commit/d343a0dc01663f91db42b4ddb693e6fffcb45873 I use setBlockTracking to mark the entire block to avoid this problem.


seed::update(..) + example update_from_js

https://github.com/David-OConnor/seed/issues/85
Hey guys, I experimented a bit with custom events. I want to know your opinions.
Start with examples/websocket/src/client.rs - see seed::update(..) in handlers.

I won't have a chance to dive in until this weekend, but at a glance, solving this worth the requirement for the message to implement [De]Serialize as needed. Clever approach.

@David-OConnor
If [de]serialize for Msg will be problem in practice or for more flexibility we can implement something similar, but for specific Msg variants (inspired by Elm ports):
E.g.
// lib.rs

#[derive(Clone, Serialize, Deserialize)]
enum PortMsg {
    Greet(String)
    Add(i32),
}

#[derive(Clone)]
enum Port{
    Port(PortMsg)
}

fn window_events(_: &Model) -> Vec<seed::dom_types::Listener<Msg>> {
    vec![
        port_handler(PortMsg::Greet, Msg::Port)
        port_with_id_handler("example_port", PortMsg::Add, Msg::Port)
    ]
}
...
    .window_events(window_events)

// something.rs

seed::Port::send("Hello!")  // or `seed::Port::send(PortMsg ::Greet("Hello!"))` for more safety
seed::Port::send_with_id("example_port", 5)

But for now, I think that Msg decorated with [de]serialize is good trade-off.

Excellent addition and example.


Use new progress api from 3.3 in desktop viewer

Closing this pr in favor of, https://github.com/iTwin/itwinjs-core/pull/3735
Will look to use the new progress api in the desktop viewer
Originally posted by @aruniverse in https://github.com/iTwin/itwinjs-core/issues/3069#issuecomment-1198146967

At the moment, progress API for changesets download is not exposed in the frontend and is missing implementation from imodels-clients side.
This should be revisited after following items are completed:
https://github.com/iTwin/itwinjs-core/issues/4419
https://github.com/iTwin/imodels-clients/pull/136


[Figma File Search] ...

Extension
https://www.raycast.com/michaelschultz/figma-files-raycast-extension
Description

Error:

Steps To Reproduce
Reason: Code: 2 (Unhandled Extension Rejection)
Domain: commands
Time: 01:11:44.055
Current Behaviour
No response
Expected Behaviour
No response

@nevertoday can you provide more context on what triggered the error?

Seems like this issue was mistakenly posted. Either way there isn't enough information. This should be closed.

@raycastbot close this issue


merge package from Kronos-Tools/npm-package-template-minimal

package.json

chore(package): update<EMAIL_ADDRESS>chore(package): add module from template

Coverage remained the same at 14.543% when pulling 46ff25df7289489470b97746efa72c4059adb162 on template-sync-1 into 89a5129538d1113e591fd00f389f045bee298e8b on master.


Adding linked objects should be more obvious in "Insert" mode in the data editor

It took me a while to realize that "edit links" is the way to add objects to a linked set when inserting new objects in the data editor.  Perhaps the button can be moved or styled in a more obvious way.

Fixed by #28


create lobby controller and service methods to create a lobby

closed https://github.com/sopra-fs24-group-12/PeekSeek-server/commit/28fe2b268f4fcebc9fde93f145d1975669b21941


chore: Update generated account management API

This PR updates the generated account management API with the aim of making as few changes to the generated output as possible:

It updates the generation script ./gen/gen.sh to use a check-in version of the spec to force tracking what has changed. It is expected that the lasted Open API spec is downloaded and stored as spec.json and then formatted and fixed; this script now operates on ./specs/account_management/spec_formatted_fixed.json. Furthermore we use the option disallowAdditionalPropertiesIfNotPresent=false, as described here.

Add the most recent versions of the spec, as well as a formatted (and "fixed" version). Adds the generated output.

Further, the PR carries out some steps that hopefully won't be needed in the future:

It removes some duplicate functions, reportError(...) and newStrictDecoder(...)

Patch the implementation of parameterAddToHeaderOrQuery(...). Use the implementation here: https://github.com/OpenAPITools/openapi-generator/blob/654f62ce3c3c32257d8b0e9929fd3fd47f2de15d/samples/openapi3/client/petstore/go/go-petstore/client.go#L164-L251

Patch the spec to take care of some inconsistencies: some fields in the API responses are listed as required but they are not included, breaking validation, so these are manually adjusted.

What's the difference between spec.json and spec_formated.json?
In case the formatting is the only one, there is no point to save them both.
And, for future proof, I would add the version (or date if the version is not enough)


Extract Aqua scan into separate step

Closes #660.
Tasks:

[x] Updated design documents in docs/design directory or not applicable
[x] Updated user-facing documentation in docs directory or not applicable
[x] Ran tests (e.g. make test) or not applicable
[x] Updated changelog or not applicable

I'm going ahead and merge this because I want to test this in a cluster with private cert, and having it in master will make my life easier. Feel free to comment later and I will address all feedback.


feat(Notification): add icon prop for notification

The following issues are addressed in this PR:

Notification icons are not customizable in any way. Now it's possible to pass an icon as an elementType prop, which will overwrite the default icon that's based on the notification kind. (if the 'icon' prop is unset, the functionality and look of the component remains the same)
The 'iconDescription' prop was mistakenly supplied both to the close icon and the decorative left-side icon. Now these are separated into iconDescription for the left side decorative icon and closeIconDescription for the close icon.

The ci/circleci check seems to fail because the linter does not recognise PropTypes.elementType. Could this be because it's using an older version of eslint-plugin-react? Locally the linter runs without errors.

Hi @webadam1! 👋 Thanks so much for taking the time to make this PR, unfortunately this codebase is in maintenance mode and is only accepting bugfixes as a result. We can definitely look into adding the fix for iconDescription, though 👍

Allrigth, clear as day, thanks for the clarification @joshblack!
Should I close this PR and open another one for the iconDescription fix, or should I change the code this one to only include the fix?

Feel free to update! Definitely don't want to make things harder on your end 😄

BTW given this PR seems to be against master - Please be advised that the master equivalent (latest 10.x) is maintained in https://github.com/carbon-design-system/carbon repo instead of this repo.


How do I apply 'vcpkg integrate' to e.g. v140 toolchain with v142 installed?

Hello,
I have installed VS 2019 (v142) along with the 2015 and 2017 toolchains (v140 and v141) which are available in the VS 2019 installer.
(The reason is I am developing a free software which aims for high compatibility with many toolchains, I am testing dozen of compilers down to VS 2010 etc., so even though I am using the VS 2019 IDE I need to use older compilers on an daily basis.)
When I use vcpkg integrate install the include/lib integrate works if my project toolchain is set to v142 but not when set to v140, although supposedly vcpkg supports VS 2015 v140 toolchain.
How I can setup VS integration to work with the v140 toolchain? All the FAQ/documents I've checked were surprisingly evasive on how vcpkg support multiple toolchains installed on the computer.
Without support for v140 I need to always retarget my projects to v142. Dealing with many repo and many branches this is becoming very inconvenient.
Note: I do NOT have the full VS 2015 installed, only the VS 2015 toolchain available as part of VS 2019. Perhaps installing the full VS 2015 would work better?
Thank you!
EDIT may be similar to #707

@ocornut

".\vcpkg.exe integrate install" is system wide and user wide for the PC/Laptop
".\vcpkg.exe integrate project" is the route I took, having a vcpkg instance setup for each version of visual studio and forcing each version to use the native toolset, compiler, link, etc for that version.
The initial setup required much thought, attention to detail, to make it versatile yet easy to work with.
The end result has a series of scripts with shortcut setting various environment variables for both Visual Studio and vcpkg in each instance.

It would be a nice enhancement if there was a way to integrate vcpkg into each version of VS separately targeting the toolset of your choice.

Thanks for posting this issue. Please open a new issue if this is still a problem for you.

Hello @LilyWangL,
I don't understand why the issue was closed. @timautry answer is pretty much not answering the issue, albeit ignoring it. integrate install is NOT system wide as it does NOT work with alternate toolsets available in the installer, unless the toolset gets the IDE installed., I don't understand why it couldn't be fixed at it is clearly an assymetry.
e.g. Scenario 1
VS 2019 IDE w/ Toolchain 2019 installed
VS 2017 IDE w/ Toolchain 2017 installed
integrate install works for both toolchains.
e.g. Scenario 2
VS 2019 IDE w/ Toolchain 2019 + Toolchain 2017 installed
integrate install only works for toolchain 2019.

See this documentation - it may help to create an overlay custom triplet file to do what is needed:
https://github.com/microsoft/vcpkg/blob/master/docs/users/triplets.md
In depth details:
https://vcpkg.readthedocs.io/en/latest/


chore: test

ci-pilot deploy to staging

hello

ci-pilot release to staging

ci-pilot release to staging

ci-pilot release to staging

ci-pilot release to staging

ci-pilot release to staging

ci-pilot release to staging

what


Fix dp race

What this PR does / why we need it:
Workaround/Fix for https://github.com/kubevirt/kubevirt/issues/1196
By submitting an empty list of devices on startup, Kubernetes appears to forget existing devices. An true upstream fix was provided in Kubernetes 1.10.3.
Special notes for your reviewer:
NONE

This PR restores stability to clusters based on OpenShift 3.10.0. The "Allocatable=0" state does exist briefly (usually around 5 seconds) once virt-handler is restarted, but this is normal/always true on all clusters.

@cynepco3hahue Please see call to tests.EnsureKVMPresent() on line 67. Added by https://github.com/kubevirt/kubevirt/pull/1204

Yee, tests look great


Screen OCR

Description of the new feature / enhancement
OCR feature in snipping Tool or A dedicated tool to copy text by OCR of selected area on screen.
Scenario when this would be used?
Browsing Internet, Pictures, inside word document or anything visible on screen.
Supporting information
No response

Built in :)
https://learn.microsoft.com/en-us/windows/powertoys/text-extractor


Incorrect parser error when calling Function constructor

The testcase:
var func = new Function("return 2");

reports the following error:
        ^
ERROR: Ln 1, Col 9: Expected either ';' or newline token

However this should not cause any errors.

On current master the test cases finishes successfully.

Fixed by #373


Module iconButton firing extra events when receiving keypress

Platform: desktop web
Steps to reproduce:

On a Module.Expandable,  performa a tab focus over an iconButton from a Module and hit space
Additionally to the button's action, the module will also collapse/expand.

Expected Result:
Actual Result: No module collapsing/expanding should happen, only the button's action.
Please add either a screenshot or a video to help us quickly identify the issue.
https://user-images.githubusercontent.com/764487/135550437-d494cd88-7b7e-46de-a784-cfb90cd6c862.mov

@themindfuldev #1722 fixed this issue, correct? If not, feel free to re-open 🙂


WMS ID: 11467 Creating better DV with OA

Add here your WMS ID
General requirements

Information in the workshop is adequate and updated
Code is correct and working
Links are correct
Make sure you enter the help email link in your manifest.json
Please make sure WMS URLs are updated as needed after your PR is approved

Checklist - Refer to the QA document for the complete list
Please confirm that the following is completed before submitting your PR

[ ] All filenames are lower case (including folders, images, files, etc.)
[ ] Filenames are descriptive
[ ] Your workshop folder structure should be similar to the one used in the sample workshop (https://github.com/oracle-livelabs/common/tree/main/sample-livelabs-templates/sample-workshop)
[ ] Are you using multiple versions (desktop/, sandbox/, tenancy/)? Make sure that each of them contains a manifest.json and an index.html
[ ] Image references in markdown contain an alternate text

@NagwangGyamtso - This workshop is in development status. Please update the workshop status to Self QA Complete to review this PR


Will consider support GraphQL ?

Will consider supporting GraphQL in the future? thanks

Will not support. you can add GraphQL to the service code created by sponge.

thanks


changed the order of the target properties for ssh.

This PR changes the order of the target properties for connecting via SSH. The code was originally using public IP first if found. It is more efficient to use public DNS because by design ec2 will return the public IP if the DNS request comes from outside and it will return the private ip if the request comes from inside ec2. This helps organization that a very lock down access to instances and only allow access either via a bastion or vpn device. Although some of the subnets are private and will only have private ip, other subnets are public (with internet access) but are still locked down to prevent public ssh access.
With this change the code will check first for the public dns and if found use that. If not it will go to the public ip and then private

Looks like I'll need to update the documentation changed by #364 when this gets merged


Render TypeContextDescriptor into a proper hierarchy; NFC

The purpose here is to make it easier to add type-specific fields to the descriptor.

@swift-ci Please smoke test and merge.


Clarify usage of FileExplorerContextMenus

Please provide a sample on how we are supposed to use FileExplorerContextMenus with

ExeServer
SurrogateServer

Please extend which interfaces are available for consumption here.

Is IShellExtInit supported?
Is IContextMenu supported?
If not, when will this functionality be available?

How are we supposed to register COM Objects consumable by Windows Explorer Shell to extend the file explorer context menu?
I tried several variants of DllMain, Main(), registering my COM objects with C++, C++/CX, C# on UWP and Packaging Project without any success.
My latest failing attempt looks like this:
<Applications>
	<Application Id="App"
		Executable="$targetnametoken$.exe"
		EntryPoint="$targetentrypoint$">
		<uap:VisualElements
			DisplayName="ShellApp.Package"
			Description="ShellApp.Package"
			BackgroundColor="transparent"
			Square150x150Logo="Images\Square150x150Logo.png"
			Square44x44Logo="Images\Square44x44Logo.png">
			<uap:DefaultTile Wide310x150Logo="Images\Wide310x150Logo.png" />
		</uap:VisualElements>
		<Extensions>
			<desktop4:Extension Category="windows.fileExplorerContextMenus">
				<desktop4:FileExplorerContextMenus>
					<desktop5:ItemType Type="*">
						<desktop5:Verb Id="ShellApp" Clsid="840C5236-4B3F-4320-8CD6-79B8CA95FCC0" />
					</desktop5:ItemType>
					<desktop5:ItemType Type="Directory">
						<desktop5:Verb Id="ShellApp" Clsid="840C5236-4B3F-4320-8CD6-79B8CA95FCC0" />
					</desktop5:ItemType>
					<desktop5:ItemType Type="Directory\Background">
						<desktop5:Verb Id="ShellApp" Clsid="840C5236-4B3F-4320-8CD6-79B8CA95FCC0" />
					</desktop5:ItemType>
				</desktop4:FileExplorerContextMenus>
			</desktop4:Extension>

			<com:Extension Category="windows.comServer">
				<com:ComServer>
					<!--<com:SurrogateServer>
						<com:Class Id="840C5236-4B3F-4320-8CD6-79B8CA95FCC0" Path="ShellApp\ShellApp.exe" ThreadingModel="Neutral" />
					</com:SurrogateServer>-->
					<com:ExeServer Executable="ShellApp\ShellApp.exe">
						<com:Class Id="840C5236-4B3F-4320-8CD6-79B8CA95FCC0" />
					</com:ExeServer>
				</com:ComServer>
			</com:Extension>
		</Extensions>
	</Application>
</Applications>

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

ID: 2b210b6b-7d08-6ac7-ab8c-07920a3eee39
Version Independent ID: 100c17d9-debc-f7c5-048f-68770b2da884
Content: desktop4:FileExplorerContextMenus - Windows UWP applications
Content Source: winrt-related-src/schemas/appxpackage/uapmanifestschema/element-desktop4-fileexplorercontextmenus.md
Product: uwp
Technology: winrt-reference
GitHub Login: @laurenhughes
Microsoft Alias: lahugh

wdkbot, have You found solution ? If  You've found, tell me please. I've got the same prolbem. I just want to add a reference to winodws context menu. But using similar solution as You also have no success.

Hello - We recently added some related packaging examples to this article that demonstrate how to use this packaging extension to register a context menu implemented via IExplorerCommand and/or IExplorerCommandState. I updated the remarks to this article to point to those examples. I hope this helps!

Thanks for those examples. Just a quick question here: Is it possible to somehow get an example in a managed-language instead of relying on C++ (C++/CX)?

@AliveDevil where did you find the <desktop5:ItemType Type="Directory"> documentation ? My understanding is that desktop4:ItemType is limited only to the files, not folders. But the MS documentation about desktop5:ItemType doesn't mention it. (opened #142)

See https://github.com/MicrosoftDocs/winrt-related/issues/142#issuecomment-534898692.

I have included that same DLL in my appx exactly according to the article with guid matching. But it doesn't work. Also, externally using regsvr32 on that DLL given in the sample also does not work, saying DllRegisterServer entry point not found.


Capitalize

The last test case does not pass because capitalize(undefined) outputs the string "Undefined" and I assume it shouldn't be this way to avoid getting mixed up with actual strings like "unDEfiNeD" which would produce the same output when capitalized

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report.
To ensure accuracy in future PRs, please see these guidelines.
A quick fix for this PR: rebase it; your next report should be accurate.

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+2.0%) to 49.057%

Totals

Change from base Build<PHONE_NUMBER>:
2.0%

Covered Lines:
24

Relevant Lines:
43

💛  - Coveralls


[BUG] Cannot scrape DDOS metrics

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Hi,
I am trying to scrape the metric for ddos protection (DDoSDetected) for my elastic ips. I have the DDOS protection configured in AWS side and i can see the metrics in cloud watch in the needed region.
I also set the "Name" tag in all elastic ips.
I use the following config:

  - type: AWS/DDoSProtection
    regions:
    - eu-central-1
    length: {{ index .Values "extraArgs" "length" }}
    searchTags:
      - key: Name
        value: .*
    metrics:
    - name: DDoSDetected
      dimensions:
        - Name: ResourceArn
          Value: "*"
      statistics:
      - Sum
      period: {{ index .Values "extraArgs" "period" }}
      nilToZero: true

But in the logs i see:
{"account":"xxxxxx","arn":"","error":"resource tag list is empty","job_type":"AWS/DDoSProtection","level":"error","msg":"Account contained no tagged resource. Tags must be defined for resources to be discovered.","region":"eu-central-1","time":"2023-02-07T13:41:03Z"}
{"account":"xxxxxx","arn":"","job_type":"AWS/DDoSProtection","level":"info","msg":"No tagged resources made it through filtering","region":"eu-central-1","time":"2023-02-07T13:41:03Z"}
Expected Behavior
I expected to be able to export the metric for all the elastic Ips with the tag: Name.(All values)
Steps To Reproduce

Create static ip in AWS and add it to ddos protection plan.
Inspect that cloudwatch creates the metrics in its side, in the needed region.
Try to use the config above and you should get the same log as i get. (Hopefully not, and it is something with my configuration...)

Thanks (:
Anything else?
No response

Hi @nikitakogan1, could you maybe run yace with -debug and attach the scrubbed output?

@cristiangreco Thanks for the answer. Here is the additional output:

{"account":"****","arn":"","level":"debug","msg":"GetMetricStatistics","output":"{\n  Label: \"DDoSDetected\"\n}","region":"eu-central-1","static_job_name":"DDoSDetected","time":"2023-02-07T21:01:55Z"}
{"account":"****","arn":"","level":"debug","msg":"CLI helper - aws cloudwatch get-metric-statistics --metric-name DDoSDetected --dimensions Name=ResourceArn,Value=*  --namespace AWS/DDoSProtection --statistics Maximum --period 60 --start-time 2023-02-07T20:57:53Z --end-time 2023-02-07T21:02:53Z","region":"eu-central-1","static_job_name":"DDoSDetected","time":"2023-02-07T21:02:53Z"}
{"account":"****","arn":"","level":"debug","msg":"createGetMetricStatisticsInput","output":"{\n  Dimensions: [{\n      Name: \"ResourceArn\",\n      Value: \"*\"\n    }],\n  EndTime: 2023-02-07 21:02:53.936021202 +0000 UTC m=+182.993883022,\n  MetricName: \"DDoSDetected\",\n  Namespace: \"AWS/DDoSProtection\",\n  Period: 60,\n  StartTime: 2023-02-07 20:57:53.936021542 +0000 UTC m=-117.006116726,\n  Statistics: [\"Maximum\"]\n}","region":"eu-central-1","static_job_name":"DDoSDetected","time":"2023-02-07T21:02:53Z"}
{"account":"****","arn":"","input":"{\n  Dimensions: [{\n      Name: \"ResourceArn\",\n      Value: \"*\"\n    }],\n  EndTime: 2023-02-07 21:02:53.936021202 +0000 UTC m=+182.993883022,\n  MetricName: \"DDoSDetected\",\n  Namespace: \"AWS/DDoSProtection\",\n  Period: 60,\n  StartTime: 2023-02-07 20:57:53.936021542 +0000 UTC m=-117.006116726,\n  Statistics: [\"Maximum\"]\n}","level":"debug","msg":"GetMetricStatistics","region":"eu-central-1","static_job_name":"DDoSDetected","time":"2023-02-07T21:02:53Z"}
Action=GetMetricStatistics&Dimensions.member.1.Name=ResourceArn&Dimensions.member.1.Value=%2A&EndTime=2023-02-07T21%3A02%3A53.936Z&MetricName=DDoSDetected&Namespace=AWS%2FDDoSProtection&Period=60&StartTime=2023-02-07T20%3A57%3A53.936Z&Statistics.member.1=Maximum&Version=2010-08-01
{"account":"****","arn":"","error":"resource tag list is empty","job_type":"AWS/DDoSProtection","level":"error","msg":"Account contained no tagged resource. Tags must be defined for resources to be discovered.","region":"eu-central-1","time":"2023-02-07T21:02:54Z"}
{"account":"****","arn":"","job_type":"AWS/DDoSProtection","level":"info","msg":"No tagged resources made it through filtering","region":"eu-central-1","time":"2023-02-07T21:02:54Z"}

@nikitakogan1 what version of Yace are you running? Could you please paste the full config file?

Hi,
I am running version v0.42.0.
My full config.yml file:
apiVersion: v1alpha1
sts-region: eu-central-1
discovery:
  jobs:
  - type: AWS/DDoSProtection
    regions:
    - eu-central-1
    length: 60
    searchTags:
    - key: Name
      value: .*
    metrics:
    - name: DDoSDetected
      statistics:
      - Maximum
      period: 60
      nilToZero: true
    - name: DDoSAttackPacketsPerSecond
      statistics:
      - Maximum
      period: 60
      nilToZero: true
    - name: DDoSAttackBitsPerSecond
      statistics:
      - Maximum
      period: 60
      nilToZero: true

In AWS side, i have the elastic ips with the needed tags:

Thanks

Update: I was able to make it work for us-esat-1 region. For some reason, the scraping is not working for eu-central-1. (Both regions has the resources with the needed tags).
I was able to use this configuration to scrape the metrics:
apiVersion: v1alpha1
sts-region: eu-central-1
discovery:
  jobs:
  - type: AWS/DDoSProtection
    regions:
    - eu-central-1
    - us-east-1
    length: 60
    searchTags:
    - key: Name
      value: .*
    metrics:
    - name: DDoSDetected
      statistics:
      - Maximum
      period: 60
      nilToZero: true
    - name: DDoSAttackPacketsPerSecond
      statistics:
      - Maximum
      period: 60
      nilToZero: true
    - name: DDoSAttackBitsPerSecond
      statistics:
      - Maximum
      period: 60
      nilToZero: true

I still see the same error in the logs for eu-central-1 region. But i see all the metrics for us-east-1.

Any idea why it is happening only in eu-central-1? I still cannot resolve this.


TIMEOUT: build_artifact.protoc_macos_x86

https://sponge.corp.google.com/target?id=bf5c2dcf-a79b-4e1a-ba49-650951f5407d&target=grpc/macos/pull_request/grpc_build_artifacts&searchFor=&show=ALL&sortBy=STATUS

seen again in https://sponge.corp.google.com/target?id=6aef4fe3-a684-47e3-bece-e2d54ea65aa9&target=grpc/macos/pull_request/grpc_build_artifacts&searchFor=&show=ALL&sortBy=STATUS for both x86 and x64 builds.

cc @jtattermusch. This has happened more than once in the last week.


[ICP] ergonomy button and tab

Following this issue: https://github.com/Inria-Asclepios/medInria-public/issues/1

Change QCheckBox to QComboBox and add a description
Remove the tab which is useless
Change the title to "Iterative Closest Point", clearer name for user

:m:

ok

Actually, the transform we are using allows three modes: similarity (default), rigid and affine. (http://www.vtk.org/doc/nightly/html/classvtkLandmarkTransform.html#a8edf745c82c2c3f55fe227172cea8fd0)
And from my understanding of our code, what we offer is rigid and similarity (not affine).
Extract from vtkICPFilter.cxx
//if you don't do this, it defaults to similarity transform (translation, rotation, and isotropic scaling)
    if(this->bRigidBody)
        this->ICPTransform->GetLandmarkTransform()->SetModeToRigidBody();

So I would add the third option in the comboBox and modify vtkICPFilter.cpp accordingly.
However, I think @msermesant is the best-suited to guide us here.

Updated. I added Similarity transformation, and change bRigidBody to bTransformation. Thx for the review :)

@LoicCadour it's ok, you can look at this PR, there was an error ^^

Have you tested it? Are you sure the indexes {0;1;2} will be understood by SetMode(int)?
From my understanding, the right inputs are {6;7;12} : http://www.vtk.org/doc/nightly/html/vtkLandmarkTransform_8h_source.html

0, 1 and 2 were saved (setMode do not crash, and you can get back 0, 1 and 2 through getMode) but you're right i'm not sure about the results. So, i used VTK_LANDMARK_RIGIDBODY etc. :)

OK


Add MKVToolNIx as a post-rip option

Is your feature request related to a problem? Please describe.
A thought occurred to me. In my current setup, I use a Raspberry Pi 4 as a Plex Server. My clients (Roku, Chromecast) can't display PGS subs (subtitles files from BD discs), so that forces Plex to transcode, which the Pi can't do. I'm currently shuttling them to my main PC (which has a Ryzen 5600x vs. my ARM's current Intel 4590), so it's much faster at transcodes and anything else. But if I could just remux the files to remove the subtitle files (which MKVToolNix can do), that the ARM server can handle. Remuxing is far less intense (I think?) than transcoding, and storage isn't the issue in my setup at the moment.
Describe the solution you'd like
Build in support for MKVToolNix CLI remuxing options, similar to current Handbrake CLI options present in ARM.
Describe alternatives you've considered
Handbrake can do this as it's transcoding, which would be hard for the current ARM server I have. But, I suppose for most, if their machine is powerful enough, Handbrake can kill two birds with one stone.
Are you able to help code/implement this feature ?
Possibly. I'm no Python coder but I've dabbled with MKVToolNix on my Pi before and it's not awfully difficult.
Additional context
This would be moot if someone has a powerful machine for transcoding and remuxing - again, two birds - but if someone just wanted to remux to remove languages/subtitles from rips (and not deal with MakeMKV's rather....involved arguments), this could be a good option.
Perhaps I just need to really look into MakeMKV's documentation and try it. Or, perhaps this is an opportunity for a more fleshed out MakeMKV docs section here? With examples - that's the part I'm struggling with when reading MakeMKV's docs.

I'll look into this if I get some free time this week, as for the MakeMKV dev docs they aren't the greatest and rather lacking. Their forums are a great place to look for more information though.

Awesome, let me know if I could help with pseudocode or HTML work. Like I said, I might not be much help with the Python. :)
I've tried looking through their forums, I think it's moreso the args that are confusing, more than the docs themselves sometimes! lol

Sorry, this was way on the back of things to do. But I have looked into this, wouldn't it be easier to use a MakeMKV profile and deselect the subs ?
/home/arm/.MakeMKV/settings.conf

app_DefaultSelectionString = "-sel:all,+sel:audio&(eng),-sel:(havemulti),-sel:mvcvideo,+sel:subtitle&(eng),-sel:special,=100:all,-10:eng"

This is what it does split by the , in the string

deselect everything (-sel:all)
select English audio (+sel:audio&(eng))
deselect different angles (-sel:(havemulti))
deselect 3d (-sel:mvcvideo)
select English subtitles (+sel:subtitle&(eng))
deselect special features (-sel:special)
prioritize English (=100:all,-10:eng)

You could do something like
app_DefaultSelectionString = "-sel:all,+sel:audio&(eng),-sel:(havemulti),-sel:mvcvideo,-sel:special,=100:all,-10:eng"
Inside /home/arm/.MakeMKV/settings.conf and that should remove all subtitles.
I'm still happy to look into adding MKVToolNix it's just that it might take a while. I have a bunch of bugs still to fix, so it's unlikely to be anytime soon.

It definitely would!! This is really helpful. MakeMKV's documentation is a
bear to get through, I never fully understood it. Again, this ROCKS.
I'll try all this and see how it goes! Thanks for taking the time to write
this up.
On Mon, Apr 18, 2022, 23:15 1337-server @.***> wrote:

Sorry, this was way on the back of things to do. But I have looked into
this, wouldn't it be easier to use a MakeMKV profile and deselect the subs ?
/home/arm/.MakeMKV/settings.conf
app_DefaultSelectionString = "-sel:all,+sel:audio&(eng),-sel:(havemulti),-sel:mvcvideo,+sel:subtitle&(eng),-sel:special,=100:all,-10:eng"
This is what it does split by the , in the string

deselect everything (-sel:all)
select English audio (+sel:audio&(eng))
deselect different angles (-sel:(havemulti))
deselect 3d (-sel:mvcvideo)
select English subtitles (+sel:subtitle&(eng))
deselect special features (-sel:special)
prioritize English (=100:all,-10:eng)

You could do something like
app_DefaultSelectionString =
"-sel:all,+sel:audio&(eng),-sel:(havemulti),-sel:mvcvideo,-sel:special,=100:all,-10:eng"
Inside /home/arm/.MakeMKV/settings.conf and that should remove all
subtitles.
I'm still happy to look into adding MKVToolNix
https://mkvtoolnix.download/ it's just that it might take a while. I
have a bunch of bugs still to fix, so it's unlikely to be anytime soon.
—
Reply to this email directly, view it on GitHub
https://github.com/1337-server/automatic-ripping-machine/issues/95#issuecomment-1101975952,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFJ3TTPKIY4F4GXP3BTTHKTVFYXOPANCNFSM5C6TP7OA
.
You are receiving this because you authored the thread.Message ID:
@.***>


FileNotFound when attempting to build

After cloning this repo, I tried checking out the files for duckdb master branch at the time for the last update of this repo and build with those file in a subdir, using:
if [ ! -d "duckdb" ]; then git clone https://github.com/duckdb/duckdb.git; fi
cp build.zig duck_files.zig duckdb
cd duckdb && git checkout 'master@{2021-01-07 00:00:00}' && zig build

But there were some complaints about missing files, for example duckdb/src/execution/merge_join/merge_join_complex.cpp etc.
I tried also using git clone https://github.com/kimmolinna/duckdb.git in case those files were more in sync.
It would be nice with for example this gha for building, but I couldn't try that out since it looked like the duck_files.zig referred to files not present in the duckdb repo?
Any hints?

I updated the version for a while ago but I forget to update it thus files weren't in sync with DuckDB. Now it should work.


Demo link broken

AS title said, the demo link is broken is there any other one?

http://aspdatatables.azurewebsites.net/ ? Seems to work fine for me in Chrome


[StreetCode Page] [UI ][Main StreetCode block] The "Оживити картинку" button is displayed under the "Teaser" at the right bottom of the screen on the tablet version.

Environment: iPadOS 16.5.1, Safari
Reproducible: always
Build found: ?
Preconditions:

Go to the site using iPad: https://streetcode.com.ua/

Steps to reproduce:

Enter "Грушевський" in the "Search" field.
Open the "Грушевський" streetcode.
Scroll down under the text of the "Teaser" (about the person).

Actual result:
The "Оживити картинку" button is displayed and clickable under the "Teaser" at the right bottom of the screen.

Expected result:
The "Оживити картинку" button is not displayed under the "Teaser" on the tablet version (according to the Mockup and User story).

Epic: #19
User story: #20
Test case: https://github.com/ita-social-projects/StreetCode/issues/64

The 'Оживити картинку' button is no longer supported.

The 'Оживити картинку' button is removed (according to the Customer`s requirement).


sightings bug

Related to PR #202

Codecov Report

Merging #203 into master will increase coverage by 0.37%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##           master     #203      +/-   ##
==========================================
+ Coverage   69.72%   70.10%   +0.37%
==========================================
  Files          18       18
  Lines        5338     5338
==========================================
+ Hits         3722     3742      +20
+ Misses       1616     1596      -20

Impacted Files
Coverage Δ

stix2elevator/convert_stix.py
76.01% <75.00%> (+1.45%)
: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 c2c25b8...54820a9. Read the comment docs.


🐛 BUG:  Uncaught TypeError: Cannot read properties of undefined (reading 'memory')

What version of Wrangler are you using?
2.0.15
What operating system are you using?
Linux
Describe the Bug
We're using wrangler2 in combination with rust. Our stack is:
worker-rs: 0.0.10
worker-build: 0.10.3

When we try to wrangler publish we get the error:
✘ [ERROR]A request to the Cloudflare API (/accounts/xxxx/workers/scripts/xx-worker-name-xx) failed.
  Uncaught TypeError: Cannot read properties of undefined (reading 'memory')
    at line 0
   [code: 10021]
  If you think this is a bug, please open an issue at: https://github.com/cloudflare/wrangler2/issues/new/choose

The same worker worked with worker-rs: 0.0.9
I'm happy to provide more information, but I'm not sure what you need to get a better grip of this error.

I believe the error appears to be from inside undici (the library providing fetch): https://github.com/nodejs/undici/blob/c8e4f3c670b368442e5ab040ab00af50a5319eb5/lib/client.js#L501
which would indicate that the llhttp object is undefined.
That object is created by instantiating a Wasm module: https://github.com/nodejs/undici/blob/c8e4f3c670b368442e5ab040ab00af50a5319eb5/lib/client.js#L324-L388

I was able to create a minimal repo. At least I hope that you now will have the same problem as I have.
Reproduce

Check out https://github.com/mettke/worker-0.10-test
Check that worker-build 0.0.6 is installed cargo install worker-build --version 0.0.6
Check wasm-pack version

wasm-pack --version
wasm-pack 0.10.3`

Check wrangler version

wrangler2 --version
2.0.15

Run worker-build --release
Run wrangler2 dev
Turn on local mode by using l
Result is [ERROR] Miniflare process exited with code 1

Repo creation
To create the repo I did the following two things:

npm init cloudflare test worker-rust
Update worker-rs from 0.0.9 to 0.0.10

Awesome reproduction @mettke! Thanks. I can confirm that I see the error.
It appears that there is a problem in the shim.mjs.  These lines:
cachedInt32Memory0 = new Int32Array(wasm.memory.buffer);
cachedUint8Memory0 = new Uint8Array(wasm.memory.buffer);

execute before
wasm = new WebAssembly.Instance(_wasm, importsObject).exports;

causing the error.
I think that those lines above are being hoisted too high for some reason.
We need to dig into worker-build I think.

@caass any ideas what changes in workers-rs might have caused this?

Seems related to https://github.com/cloudflare/workers-rs/issues/204.


call passthroughEvent from runner rather than qemitter/utils in geminiFacade

/cc @j0tunn

Coverage decreased (-0.007%) to 83.863% when pulling 0878e8e10ed969af6d381a80a6f05a78a84f452c on fix/call-passthroughEvent-from-runner into 80cccd5812f8f0a6aa69548086bd38af8a34c4ef on master.

Coverage decreased (-0.007%) to 83.863% when pulling 0878e8e10ed969af6d381a80a6f05a78a84f452c on fix/call-passthroughEvent-from-runner into 80cccd5812f8f0a6aa69548086bd38af8a34c4ef on master.

/ok


chore: migrate to GitHub Actions, consider mergify for dep maintenance

This repo is not actively maintained, so it would be helpful to lean on automation as much as possible.

Drop Travis (which appears to no longer work) in favour of GitHub Actions for CI
Use mergify to automatically merge dependabot PRs when tests pass

cc: @cjbarth , in case I never get round to it

Resolved by 91dd3d5


⚠ SceneTime has degraded performance

In 03e5cf7, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 4021 ms

Resolved: SceneTime performance has improved in b348f80 after 6 minutes.


[WIP] Support 'TIMESTAMP' type in read_gbq

Issue link: https://github.com/apache/beam/issues/22909
This PR adds support for the 'TIMESTAMP' type in mapping BQ Schemas to 'BEAM_ROW'

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Python PostCommit 3.8

Run Python 3.8 PostCommit

Run Python 3.8 PostCommit


CLI documentation can be improved

Describe what needs to be documented.
Working on #3324 revealed a few areas where the documentation can be improved. This ticket contains a record of those findings. Some documentation is missing altogether, some just needs to be improved. The link for the CLI function documentation on RTD is here. This can be broken down into a ticket for each issue, but I wanted to record it all here to start with.
Missing documentation
These functions appear in dir(visit) but have no documentation and do not appear in RTD:

[ ] AddColorTable
[ ] AnimationSetNFrames
[ ] Argv
[ ] ClientMethod
[ ] GetColorTable
[ ] GetDatabaseCorrelation
[ ] GetDatabaseCorrelationNames
[ ] GetExportOptions
[ ] GetProcessAttributes
[ ] InitializeViewerProxy
[ ] OpenCLI
[ ] OpenClient
[ ] OpenGUI
[ ] RemoveColorTable
[ ] SetAnimationAttributes
[ ] SetAutoUpdate
[ ] SetCloneWindowOnFirstRef
[ ] SetColorTable
[ ] SetDefaultQueryOverTimeAttributes
[ ] UpdateMouseActions
[ ] UpdateSeedMeStatus
[ ] UserActionFinished
[ ] Axes2D
[ ] Axes3D
[ ] AxisLabels
[ ] AxisTickMarks
[ ] AxisTitles
[ ] ColorAttribute
[ ] ColorAttributeList
[ ] ColorControlPoint
[ ] ColorControlPointList
[ ] EngineProperties
[ ] ExpressionList
[ ] Expression
[ ] GaussianControlPoint
[ ] GaussianControlPointList
[ ] LaunchProfile
[ ] MachineProfile
[ ] SILRestriction
[ ] SelectionProperties
[ ] VisItException
[ ] VisItInterrupt
[ ] WindowInformation

These attribute objects are missing...

[ ] ColorAttribute
[ ] ColorAttributeList
[ ] ColorTableAttributes
[ ] ConnCompReduceAttributes
[ ] ContextAttributes
[ ] CracksClipperAttributes
[ ] DBOptionsAttributes
[ ] DBOptionsAttributes_Helpers
[ ] DecimateAttributes
[ ] DelaunayAttributes
[ ] ExtractPointFunction2DAttributes
[ ] FiveFoldTetSubdivisionAttributes
[ ] GlobalLineoutAttributes
[ ] InteractorAttributes
[ ] LegendAttributesObject
[ ] LineSamplerAttributes
[ ] MergeOperatorAttributes
[ ] MetricThresholdAttributes
[ ] NameschemeAttributes
[ ] PDFAttributes
[ ] PickAttributes
[ ] QueryAttributes
[ ] QueryOverTimeAttributes
[ ] RemoveCellsAttributes
[ ] SaveSubWindowAttributes
[ ] SaveSubWindowsAttributes
[ ] SurfCompPrepAttributes
[ ] TessellateAttributes
[ ] ThresholdOpAttributes
[ ] TopologyAttributes
[ ] ViewerClientAttributes
[ ] WellBoreAttributes
[ ] ZoneDumpAttributes

In addition, these functions seem to pull their documentation in from the .C file, but also do not appear in RTD:

[ ] IsPySideViewerEnabled
[ ] LaunchPyQt
[ ] LaunchPySide
[ ] LaunchWithProxy
[ ] ModelFitAtts
[ ] evalfuncs
[ ] frontend
[ ] visit
[ ] writescript

Documentation that need improvement

[ ] WriteScript - documentation needs to be completed
[ ] ToggleLockTools - Tools are something that can be locked, and this function toggles the lock on those tools. Documentation for this function contains information on the other toggles, but not on lock tools. Replace this documentation with description of how ToggleLockTools works.
[ ] GetEngineProperties - determine the type of input arguments for engine and sim. For example, is engine an EngineObject?
[ ] LoadAttribute - Determine the type for the object argument.
[ ] OverlayDatabase - Determine the type for the state argument.
[ ] SaveAttribute - Determine the type for the object argument.
[ ] SetPlotSILRestriction - Determine the type for the all argument.
[ ] SendSimulationCommand - documentation says that it supports 4 inputs, but visit give an error when trying to use it with 4. Error says that function only supports 3 inputs. Discover the truth and update the documentation accordingly.
[ ] SetDefaultMeshManagementAttributes - This function is the "setter" but the documentation is for the "getter". Update the documentation for the setter.
[ ] SetMeshManagementAttributes - This function is the "setter" but the documentation is for the "getter". Update the documentation for the setter.
[ ] GetQueryOutputObject - Example shows GetQueryOutputString but not GetQueryOutputObject. Also, the return description seems to be wrong. This function probably does not return an xml string but rather some kind of object.
[ ] GetQueryParameters - Add documentation for the argument name
[ ] ListPlots - Add documentation for option argument stringOnly
[ ] OpenComputeEngine - Add documentation for simulation and MachineProfile arguments.
[ ] SetAnimationTimeout - Add documentation for milliseconds argument
[ ] SetPipelineCachingMode - Add documentation for mode argument.
[ ] SetPrecisionType - Documentation for the argument typeAsInt is not clear.

I attached low-hanging fruit to this issue not because I think the whole of the issue is low-hanging fruit but because each line item in it is and given the check-box design of the issue, we can decide to chip-away at this over time taking care not to declare it resolved in any PR until everything is checked off. The alternative, submitting a slew of individual issues, doesn't seem very attractive for each line item here, doesn't seem very productive.

I found 33 additional missing attribute objects and included in the initial comment.

Requesting re-review


Python: CombineGlobally() needs to be able to produce expected output when processing multiple windows simultaneously

What would you like to happen?
CombineGlobally() is supposed to reduce a PCollection to a single value. When given multiple windows to process, it would previously fail due to an assert statement that limited the max length of its output to one. I replaced this assert statement with a more descriptive error statement here (#24435), but a comprehensive solution for situations when the transform handles multiple windows, and thus multiple PCollections, at once, is still needed, since this situation is expected to arise from fairly run-of-the-mill use of the transform in streaming pipelines.
Issue Priority
Priority: 2 (default / most feature requests should be filed as P2)
Issue Components

[X] Component: Python SDK
[ ] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[ ] Component: IO connector
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

.take-issue

We would need to define what it means to combine globally with defaults for non-trivial windowing. (Presumably that'd be a empty value for every empty window, so we'd need some way to enumerate windows.) If one wants to see combined values only when there is data, without_defaults should be advised and used.

fixed by #26922


Add hook method for exception handling to StatelessChecker

I used StatelessChecker from wicket-devutils, It is verifying whether it is StatelessPage or not.
Currently, when it is not StatelessPage, StatelessChecker throw exceptions always.
I logged an error and thought there was a use case I would like to check later.
I implemented a hook method based on #205  feedback.

Do you really need all those #failXYZ() methods ?
One isn't enough ?

In my use case, one fail method is enough.
However, in the current implementation, for components with stateful behavior, IllegalStateException is thrown.
I did not change this part.
Therefore, I created a fail method for each component.
To make one fail method,
Can I change IllegalStateException to IllegalArgmentException?

Neither IllegalStateException nor IllegalArgumentException seems right to me. Probably it's better to just throw WicketRuntimeException.

IllegalStateException looks good to me. The developer expects that the page is in stateless state but it is not!
What about introducing a custom exception and pass it to the #fail() method ? By default it will be thrown but the application can override the method and log it or do whatever it finds suitable.
if (!condition) { fail(new CustomException("A good message")) }
and
protected void fail(CustomException cx) {
  throw cx;
}
```
or
````java
protected void fail(CustomException cx) {
  LOG.warn("The page is not really stateful: {}", cx.getMessage(), cx);
}
```

Thanks for the suggestion.
I think you can create a StatelessCheckFailureException that inherits from WicketRunTimeException
and let StateFullChecker throw a StatelessCheckFailureException.
I will try implementing it again.
This pull request is closed.
Thanks.


No 'version' key in the manifest file for custom integration 'apsystems_ecur'.

2021-03-12 10:10:02 WARNING (MainThread) [homeassistant.loader] No 'version' key in the manifest file for custom integration 'apsystems_ecur'. This will not be allowed in a future version of Home Assistant. Please report this to the maintainer of 'apsystems_ecur'

Temporarily add the following to manifest.json:
"version": "1.0",

Now added to the manifest and made a pull request

PR approved and merged


Delete totalfinder-beta.rb

v1.7.10 is already stable and development appears to have ceased since
the introduction of SIP in El Capitan.

Thank you.


Improve CI (matrix, --workspace)

Surprise! We haven't been running more tests or lints because nobody added --workspace :/
Fixes #228
Removing comment about macOS 11 since it's gonna be the default for macos-latest soon: https://github.com/actions/virtual-environments/issues/4060
So now this should find a bunch of new issues, might wanna merge this only after fixing those. Also after you approve/merge this, i'll add something similar to rusty-editor since i haven't touched its CI in a while.
EDIT: For some reason it doesn't seem to run any CI jobs. Will look into why later.

Wrong repo


This page does not mention  how to send device id as paramter in speech sdk .

[Enter feedback here]
This page does not mention  how to send device id as parameter in speech sdk .
what if multiple azure precepts are connected to same server ? how to identify them here or in Flow?

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

ID: 520d3960-5045-17aa-d473-1ada5e73a741
Version Independent ID: 06f847f0-830d-aea3-2489-f8550078c466
Content: Configure a voice assistant application within Azure Percept Studio
Content Source: articles/azure-percept/how-to-manage-voice-assistant.md
Service: azure-percept
GitHub Login: @mimcco
Microsoft Alias: mimcco

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

Hello @mimcco Could you please help in this matter?

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.


Bit length of the AES encryption when used

[Enter feedback here]
There is no indication of the bit-length used when AES is chosen for the ENCRYPTION option for an Endpoint.
Can this be added so that auditors can know what it is?
Thanks.

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

ID: 9b15a4cb-5e12-427d-146e-fd717a094d37
Version Independent ID: e1996c22-0fa7-3ecf-d04a-fe6cbc8609db
Content: Create a database mirroring endpoint (Transact-SQL) - SQL Server Database Mirroring
Content Source: docs/database-engine/database-mirroring/create-a-database-mirroring-endpoint-for-windows-authentication-transact-sql.md
Product: sql
Technology: database-mirroring
GitHub Login: @MikeRayMSFT
Microsoft Alias: mikeray

@dbaduck -- Ben, thank you for your question.  Please consider these resources:

SQL Server on Microsoft Q&A
DBA Stack Exchange
Stack Overflow

@MikeRayMSFT -- Mike, please look into this issue.

@dbaduck  - thank you for posting this. Reviewing this request. Apologies for the delayed response. Mike


Optimized code

Added "you put" localization key
Optimized the code of "you put".
(can be copied)

i use the code and i fix a few issues with the "zh_ch" the "// TEXT" only work with java and the lang files use JSON [Java Script] and the annotations with "// text" don't exist in JSON

and the pat feature don't work with both heads when is a "non Special or Patable entity"

but i fix all

What about localization keys?

What about localization keys?

Already was in the plans But you help me with that alop :P

Btw is key.changed_addon not key.changedaddon


Fix incorrect frame code offset in coredump

Currently, coredump include incorrect code offsets, the generation relies on func_offset() which returns the instruction's offset relative to the function.
This change uses module_offset() to get the absolute code offset and, using a new method get_relative_codeoffset, converts the offset to be relative to the code section.
Also, the name property has been removed because it's misleading when displayed and not actually useful.

Sorry seems like I misremembered https://github.com/WebAssembly/tool-conventions/blob/main/Coredump.md#threads-and-stack-frames... Code offsets are relative to the function start. Closing this since it's already correct.


Update Catch2 to 2.9.1

I've followed this guide
step by step carefully. Yes

I've tested this package remotely and have excluded all broken builds.
Here is the links to the Travis/AppVeyor with status "All passed":

https://ci.appveyor.com/project/kaihowl/hunter/builds/25341023
https://travis-ci.com/kaihowl/hunter/builds/115851369

This is a replacement for #1895.
The additional question from #1895 still stands:

There is now an additional library integrated into Catch2, called nonius. This is disabled by default and enabled with the flag CATCH_CONFIG_ENABLE_BENCHMARKING. Would it make sense to add an additional compilation unit to the examples that exercises this compilation flag? Or do we keep the testing to a minimum as to not run afoul of testing the entire library again as part of integrating it into Hunter?

Testing:

https://ci.appveyor.com/project/ingenue/hunter/builds/25382731
https://travis-ci.org/ingenue/hunter/builds/547605575?utm_source=github_status&utm_medium=notification

Would it make sense to add an additional compilation unit to the examples that exercises this compilation flag?

You can add extra example with custom configuration, e.g. like this:

https://github.com/ruslo/hunter/tree/v0.23.190/examples/Boost-custom-args
https://github.com/ingenue/hunter/blob/020a7f49f3ac037658929323019ffc7198db8ccd/.travis.yml#L603-L606

Released: https://github.com/ruslo/hunter/releases/tag/v0.23.191


Can't generate Visual Studio project from SCons

I'm trying to generate the Visual Studio project with:
scons vsproj=yes platform=windows

using the VS command prompt as found here: http://stackoverflow.com/questions/21476588/where-is-developer-command-prompt-for-vs2013, but it fails with:
C:\godot>scons vsproj=yes platform=windows
scons: Reading SConscript files ...
internal error
Traceback (most recent call last):
  File "C:\Python27\Scripts\..\Lib\site-packages\scons-2.3.4\SCons\Script\Main.p
y", line 1372, in main
    _exec_main(parser, values)
  File "C:\Python27\Scripts\..\Lib\site-packages\scons-2.3.4\SCons\Script\Main.p
y", line 1335, in _exec_main
    _main(parser)
  File "C:\Python27\Scripts\..\Lib\site-packages\scons-2.3.4\SCons\Script\Main.p
y", line 1002, in _main
    SCons.Script._SConscript._SConscript(fs, script)
  File "C:\Python27\Scripts\..\Lib\site-packages\scons-2.3.4\SCons\Script\SConsc
ript.py", line 260, in _SConscript
    exec _file_ in call_stack[-1].globals
  File "C:\godot\SConstruct", line 399, in <module>
    variant = variants)
  File "C:\Python27\Scripts\..\Lib\site-packages\scons-2.3.4\SCons\Environment.p
y", line 260, in __call__
    return MethodWrapper.__call__(self, target, source, *args, **kw)
  File "C:\Python27\Scripts\..\Lib\site-packages\scons-2.3.4\SCons\Environment.p
y", line 224, in __call__
    return self.method(*nargs, **kwargs)
  File "C:\Python27\Scripts\..\Lib\site-packages\scons-2.3.4\SCons\Builder.py",
line 633, in __call__
    return self._execute(env, target, source, OverrideWarner(kw), ekw)
  File "C:\Python27\Scripts\..\Lib\site-packages\scons-2.3.4\SCons\Builder.py",
line 554, in _execute
    tlist, slist = self._create_nodes(env, target, source)
  File "C:\Python27\Scripts\..\Lib\site-packages\scons-2.3.4\SCons\Builder.py",
line 518, in _create_nodes
    target, source = self.emitter(target=tlist, source=slist, env=env)
  File "C:\Python27\Scripts\..\Lib\site-packages\scons-2.3.4\SCons\Tool\msvs.py"
, line 1642, in projectEmitter
    raise SCons.Errors.InternalError(s + " must be a string or a list of strings
")
InternalError: srcs must be a string or a list of strings

I can build fine.

I don't have the slightest clue, I haven't made msvc support, so I'm not going to fix it :P

Volunteers appreciated!

Working here, just make sure you are running VS[~year~] x64 x86 Cross Tools command prompt when running:
scons vsproj=yes platform=windows
Win32 build fine, but x64 fails about a redefinition of:
#ifdef _WIN64
    typedef unsigned __int64 size_t;
    typedef __int64          ptrdiff_t;
    typedef __int64          intptr_t;
#else

Complaining about ptrdiff_t being redefined in vcruntime.h

But one thing that bothers me is that header files of Godot are scattered everywhere, which means
VS can't find them but Scons will build fine. For example main.cpp is inside Godot/main/
Inside main.cpp where it tries to #include "core/register_core_types.h" for example, VS IDE fails
to detect the file if it exists with red squiggles underlines, causing debugging to go nuts. The reason
for this is that core/register_core_types.h is located one directory above of main. So the include
directive must have been #include "../core/register_core_types.h" instead.
My suggestion would be, to put all header files in their own directories, separated from source files.
Which will make it easier for VS IDE to find them all in one place or a set of directories at least.
I am not sure if I should open an issue for this or not, since it will require a lot of modifications to
SConstruct I believe.
XCode doesn't have problems with searching directories recursively, and I think linux too but I am not
sure. It only affects VS. But Godot is not designed to be code-able in Windows platform. I don't think
the devs will change this, since they are adamant about SCons.

Well moving the headers would be a hacky workaround. Any buildsystem should be able to locate headers provided you pass it the good paths to the folders that contain those headers.
You probably just need to find a way to tell VS that "." is an include directory. With gcc it would be -I..

Indeed, just make sure you have the right include paths.
On Mon, Jan 11, 2016 at 8:18 AM, Rémi Verschelde<EMAIL_ADDRESS>wrote:

Well moving the headers would be a hacky workaround. Any buildsystem
should be able to locate headers provided you pass it the good paths to the
folders that contain those headers.
You probably just need to find a way to tell VS that "." is an include
directory. With gcc it would be -I..
—
Reply to this email directly or view it on GitHub
https://github.com/godotengine/godot/issues/3117#issuecomment-170511903.

@akien-mga No such option in VS I am aware of, I already tried to include the the obvious $(ProjectDir) in VS additional include directories, but VS needs all directories specified individually.
That is, VS is trying to look for header files locations relative to current location of the file being parsed by the prerprocessor, not the base directory every time.
So unfortunately, your suggestion didn't work for me.

For the interested, I created manually a VS project is in the pull: https://github.com/godotengine/godot/pull/3247:

Hey @ex since you already have VS project and a PR for that, you might want to close this issue.
For me to be honest, I just gave up on SCons, the incremental builds are ridiculous! GOD bless the devs for living with that. It is so slow, using SCons with IDE such as VS is no better than using notepad with command prompt to build.
Sorry for my rants.


Minor corrections

Weird spaces were introduced at various points. Removed and corrected.

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@thechriskent Hi! Could you please change the UIS in the Important Note to Bold? Thanks!


Obsidian platform does not generate fast enough

It seems as if the obsidian platform in The End does not generate quickly in order to accommodate the player, which can lead to the player falling out of the world (depending on the seed).
Steps to reproduce:

Make a new world with seed<PHONE_NUMBER>22069433 (any seed without land under the obsidian platform should work) and verify that the end portal is set to point to the obsidian platform
Go to The End, make sure the obsidian platform is generated
Teleport back to the stronghold and break the obsidian
Jump in, the obsidian should generate right after you fall past it, leading for you to fall into the void

Died to this bug in my hardcore world today, rip o7

Alrighty, great to hear! Thank you!


Organize the payment buttons alphabetically

The payment buttons on the website are not organized alphabetically. This should be fixed in website/src/data/buttons.json

I can work on this

i can work on this


Packages.json "files" key must be removed to allow complete install

The package.json contains a key "files": ["dist"] line 6.
This results into missing directories for xcode and android.
After an install, only the dist directory will exist. The rest will be removed.
Could you fix the package.json please ?

Thanks


Remove comma

In order to have a valid json for deploying on Heroku

@Goldenson This was already taken care of. Thanks!


#13260 create destination dir

Task name: CopyFilesOverSSH
Description: Attempt to make the destination directory prior to the copy, fail the task if that does not work.
Documentation changes required: N
Added unit tests: N
(This task doesn't have tests currently, and implementing tests for this task is a significant challenge.)
Attached related issue: https://github.com/microsoft/azure-pipelines-tasks/issues/13260
Checklist:

[x] Task version was bumped - please check instruction how to do it
[x] Checked that applied changes work as expected

Someone else ended up closing this out in a different PR.


Update Play Midnight to Latest Play Music UI changes

After update, I have troubles with "Play queue" that doesn't work properly. Also Google in "Play Queue" added new white gradient, that with "Play Midnight" looks ugly.
The troubles with "Play Queue":

Open it
Try to click outside to close this menu
Click was triggered in wrong place (menu is reopened)

Thanks for the update. I haven't received these changes from Google yet, so it's hard for me to look into them. As soon as I get their changes I'll work on it.
As for the click triggering, etc. I'm not really sure what would be causing that. I don't think anything I'm changing should be causing that, but maybe the extended queue is.
Can you try disabled the "Extended Queue" option and see if the click issue still happens?
Thanks!

As I see, Google rollback this changes(checked by structure of Queue Menu). So everything is works again.

I'm actually not sure that Google did rollback the changes. While I'd never seen them before, today I opened a playlist and saw this:

To be clear, I'm only seeing the very obvious gradient issues. Not any of the other issues that Arilas mentioned.

They just pushed out a new change. I'll look into it tonight. I actually see the changes this time so it's an easy fix.

Awesome, thanks for the prompt reply!

Yup, no worries! Thanks for letting me know!

All fixed up in v2.2.2! Should reach you guys soon.


Getting error saying can not import evaluation metrics.

Traceback (most recent call last):
File "examples/main.py", line 17, in
from reid import datasets
File "./reid/init.py", line 4, in
from . import evaluation_metrics
ImportError: cannot import name 'evaluation_metrics'

Resolved it by copying the modules from open-reid ..


Fastfile finding is case sensitive

On the MacOS (APFS file system) the extension didn't worked because of the case sensitive search of the "Fastfile".  In this case renaming from the "fastfile" to the "Fastfile" fixed the issue. It would be nice if there is any log of reason why the extension is not working.

Fixed in the release 0.1.0 release right now


Revise all submit-transaction methods in the TxHelpers module

The following methods are used for submitting transactions:

submit_transaction
submit_private_transaction
cancel_private_transaction

These methods are used inconsistently across the code, sometimes receiving a signed transaction as input, sometimes receiving an unsigned transaction as input, and sometimes receiving a raw transaction as input.
In addition to that, not every attempt to submit a transaction is complemented with an attempt to cancel it upon timeout.
This requires a higher-level decision on which transactions should be cancelled when needed, and which should not be.
Technically, submitting a transaction requires only the signed transaction as input, but cancelling it requires the original (unsigned) transaction which was used in order to create it, as input.
So designing a solution for this problem should also take into account how every submitted transaction which may need to be cancelled, can be easily cancelled.
An addition bug in the cancellation method, is that it resets the transaction's data attribute but not its value attribute.
As a result, cancelling a transaction which includes ETH, might end up with that transaction being only "partially cancelled" (namely, the ETH-transfer part in that transaction will effectively take place despite the cancellation).
The aforementioned methods are used only in one place outside of this module, in method generate_shutdown_tx.
So the implemented solution should also include this specific usage.

Naming and coding conventions:

A signed transaction can be referred to as signed_tx; no need to use signed_arb_tx in the method itself, as it is designated for handling any type of transaction and not just arb transactions; same goes for unsigned_tx and raw_tx; of course, in the calling method they should be named based on what they truly represent (e.g., approve_tx)
Methods submit_transaction is designated for "tenderly OR non-ethereum", while method submit_private_transaction is designated for all other cases, i.e., "non-tenderly AND ethereum"; These two methods should be named accordingly

An additional issue:

Methods submit_transaction returns tx_receipt
Methods submit_private_transaction returns tx_receipt["transactionHash"]

Where tx_receipt is the dictionary returned from web3.eth.wait_for_transaction_receipt in BOTH cases.
This could potentially lead to unpredictable and undesirable behavior.

Handled in PR #355


Adjust the outputs to reflect states without state income tax

As special cases are growing and more states requires to skip variable calls, they needs to be optimized and generalized. As discussing here

Is this geared towards states without state income taxes - if so we should name the issue that way

Right, this to more to cover the special cases for the states does not have that as well as if any other scenarios related to the state related calculation

Right, this to more to cover the special cases for the states does not have that as well as if any other scenarios related to the state related calculation

Lets keep this issue just to states without state income tax for simplicity

as if any other scenarios related to the state related calculation

such as?

For now they are TX, FL, NV and also the n/a cases in excel list which are not implemented right now.


Image Sorting

I am trying to get the images to sort with newest images on bottom and oldest images on top(similar to the iOS Photos app). Per Issue 71, I tried adding this block of code to the TLPhotosPickerConfigure instance of my picker:
  let options = PHFetchOptions()
  let sortOrder = [NSSortDescriptor(key: "creationDate", ascending: true)]
  options.sortDescriptors = sortOrder
  configure.fetchOption = options

The ordering does not seem to change, even if I change ascending to false. I was looking into the TLPhotoLibrary and it looks like the sortDescriptors property is being modified in the getOption function of TLPhotoLibrary.swift to have a value of false, regardless of the value that is passed in to the TLPhotosPickerConfigure object.

@bport95
It should work release 2.0.1 But i can't published Because cocoapods bugs.


Add requests package  as dependency

The package depends on the requests package, which is not explicitly mentioned in the setup file. This PR adds this as requirement to the setup file.

Hey @stijnvanhoey Thanks for this PR - the checks aren't passing, but that's because the OpenAQ API is being temperamental - should be fine and merged.


Separate PDB symbols from installer bundle

By putting the PDB symbols in a separate package (ie. a zip file) we can reduce the size of the installer bundle from 7.66MB to about 2.5MB (for x64).

Do you want to remove the symbols from the portable zip packages as well?

Yes, and have the symbol package named PicoTorrent-x.y.z-x[64|86].symbols.zip, or something like that :smile: We should also set CompressionLevel="high" on the <Media /> element in PicoTorrent.wxs as a bonus fix for this.


Update stats-rank library.

This fixes an installation issue on *nix, and probably MacOS:
https://github.com/stdlib-js/stdlib/issues/2122
# npm install jandas
[...]
npm error gyp: binding.gyp not found (cwd: ***/node_modules/@stdlib/math-base-assert-is-integer) while trying to load binding.gyp

And a build issue:
Jandas# npm install
[...]
> tsc

node_modules/@stdlib/stats-ranks/docs/types/index.d.ts:21:23 - error TS2688: Cannot find type definition file for '@stdlib/types'.

Thanks for helping fix issues. I have updated the package on NPM.


Mega Menu: Remove h2 heading wrapper around trigger button

Seems kind of weird that we wrap the hamburger button in an h2 and reset the default styles for h2. I think this was probably originally added in anticipation of more content being in this part of the menu that didn't show up.
I also wondered if there's an accessibility reason to have heading content in a nav, but it seems to work the same with or without with voiceover. Anyone else know of any reason the heading wrapper may be needed?
Removals

Remove h2 o-mega-menu_heading heading wrapper around mobile trigger button.

Testing

Pull branch and build.
Visit the mobile version of the site and expand/collapse the hamburger menu. It should work just like production.

@contolini

The heading was added in #4228. I'd take a look at the description of that PR and maybe discuss with @cfarm, but it seems to me like the logic for including it was sound at the time. Possibly some of the structural changes in the new MM change the situation though?

Per W3C I'm okay removing it. We need to add role="menu" to the wrapper though.

Are you aware of any resources saying we should include the menu in the headings list in the rotor? I visited a few sites with a hamburger menu (washingtonpost, nytimes, whitehouse, usps) and none of them include the menu in the headings list. It's clearest on the news sites where the headings list is just a list of news article headings.

Per W3C I'm okay removing it. We need to add role="menu" to the wrapper though.

Are you aware of any resources saying we should include the menu in the headings list in the rotor? I visited a few sites with a hamburger menu (washingtonpost, nytimes, whitehouse, usps) and none of them include the menu in the headings list. It's clearest on the news sites where the headings list is just a list of news article headings.

From a quick survey of some search results for "accessible mega menu", it seems like some folks recommend using them (without saying they are necessary), while others don't. Seems like a judgement call, and one that we've already made. I'd like Cat to weigh in before reversing that.
[Sidenote: I checked out the homepage's heading structure in the WAVE toolbar and some of the recent changes have introduced some issues that we should probably address.]

I didn't see @anselmbradford address this, which is from my PR that introduced the h2

gives a parent heading element to the new  headings that I added for overview text, and to the existing  elements (see details below)

that's the main reason for the change and other decisions were to make sure that the heading hierarchy is sensible which makes it accessible for screen reader users. having heading elements gives screen readers natural landmarks to read aloud to users.
if there are still h3s in this menu then I would not approve this change. that's why the h2 was added, because those were showing up in the headings list on screen readers but the hierarchy made no sense without knowing the context and making the menu navigable from the heading list too.

gives a parent heading element to the new h3 headings that I added for overview text, and to the existing h4 elements

This makes sense, but the h2 is not a parent of the h3s and h4s. It's enclosing the hamburger menu item (aka the "menu trigger"), which is a sibling of the main menu content. It seems like the h2 should be surrounding the top-level overview link, not the menu trigger. So "Consumer Tools" in this image:


Implemented 573

QPushButtonWithDoubleClick is required to be a public class. This is an ugly workaround but it is required by Qt.

This QPushButtonWithDoubleClick seems to be something we can reuse at a later time. Could we put it in its own file?


About zoom/bounds change

Hello,
I'm testing the heatmap and is a great work. Actually I'm using with gmaps and there's a "trouble" when you change the zoom/bounds because the canvas stay relative of parent div of gmaps.
I was reading about -webkit/-moz transform and thinking about use this relative to remake the canvas and stay it "out" of this scale. What u think about? Do u have some idea about?
Thanks.

Hello,
I have the same problem with the zoom.  How can I fix it please?
Thanks,
Philippe


Updated Logic Apps throughput doc to better reflect throttling logic

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@swylezol : Thanks for your contribution to the Azure documentation! The author, @jeffhollan, has been notified to review your proposed change.

#sign-off


[FEATURE REQ] Azure HTTP client based on Vert.x

Is your feature request related to a problem? Please describe.
It'd be useful to expand the set of existing Azure HTTP client implementations with one backed by Vert.x and its HTTP client. I.e for folks already using Vert.x in their application, you can reuse its event loops and not waste resources having to create additional thread pools etc with the other clients.
Describe the solution you'd like
A dedicated azure-core-http-vertx library where the HTTP client is provided by the Vert.x Web Client.
Additional context
I already implemented this as a Quarkus extension here. It should be simple enough to remove the Quarkus specific parts and craft a PR (which I'm happy to do).

Thanks for filing this suggestion @jamesnetherton, this is much appreciated.  @alzimmermsft could you please incorporate this into the Java SDKs Azure Core backlog?

@jamesnetherton, thanks for filing this issue. I've added it to our backlog and have assigned @mssfang for now. Let us know if you intended to own the work and @mssfang can be your contact if you run into issues during implementation or with the PR itself.

Thanks @alzimmermsft! I will try to get a PR together some time in the next 1 - 2 weeks.

Thanks @alzimmermsft! I will try to get a PR together some time in the next 1 - 2 weeks.

Awesome! I've co-assigned you along with @mssfang.

Was the client released? I can't find it in the BOM.

@cescoffier  The client isn't released. We are running into out of memory performance issue when running it in our performance pipeline with larger input data. I will be working on it.

@mssfang Thanks! It would be great to have a release as it's required to use the SDK in a Quarkus application.

@mssfang any info ? something we can help with ?

@mssfang any info ? something we can help with ?

@maxandersen  Working on it. Trying to get it done/released in September/October

@mssfang any updates? we are currently having to manually backport/align deps as we await this release. Would be great to have it out so we can use one version of the client.

@maxandersen Good news. We have it released couple days ago. Please check: https://mvnrepository.com/artifact/com.azure/azure-core-http-vertx


WIP - Do not merge - Implement rubocop

Added gems, setup a rubocop instruction file and then implemented all suggested changes

Do not merge yet.  More changes will be needed after DWP checker is merged in.

Apart from the two comments above, which aren't show stoppers, it seems fine to me.


Support replies in KafkaChannel

Fixes #

Proposed Changes

Support replies in KafkaChannel
Create new ResponseHandler that does it
Rename KafkaResponseHandler to ResponseToKafkaTopicHandler because the original name meant "handle the response from Kafka" to me
Enable e2e tests that use sink replies

Release Note

Docs

/test pull-knative-sandbox-eventing-kafka-broker-reconciler-tests
/test pull-knative-sandbox-eventing-kafka-broker-upgrade-tests
/test pull-knative-sandbox-eventing-kafka-broker-unit-tests

Have you considered using composition over inheritance?

I thought about it. I prefer leaving this as is for now.

/test pull-knative-sandbox-eventing-kafka-broker-integration-tests

/lgtm

/test pull-knative-sandbox-eventing-kafka-broker-upgrade-tests


Feature/multiplexer Multiplex addresses (WIP)

Turns a single address role (like owner) into a multi-address one without modifying or redeploying the original contract.
It's labeled WIP because the tests are not finished yet. yarn test fails to run any tests on master ATM.

Thank you for your contribution @Recmo! I was confused by the owner / manager terminology at first. May I suggest we use the term operator as in ERC777? We could then have authorizeOperator(address) and revokeOperator(address) functions.
Other than that, it would be great to do full return value proxying like it's done here but maybe there is a reason why it's not yet included in Zeppelin core.

Closing due to staleness. Note that this sort of behaviour can be easily implemented using our Roles contracts.


Not compatible since macOS Ventura 13.3

Only a blank / black screen is displayed

Its working fine you needs to set the correct date and save it.

@Zeeshanahmad4 I can confirm that it started working again in the later 13.x versions and is functioning correctly in my current 13.6.7 (22G720).
@chenrui333, after some successful checks on macOS 14.x, you might consider adding the screensaver back to Homebrew, maybe with a caveat hint for at least version 13.3.

can you guys create a PR to restore the cask installation? Thanks!


Style: Ban default export

Enable this rule for eslint
https://github.com/benmosher/eslint-plugin-import/blob/master/docs/rules/no-default-export.md
because when rename / refactoring, WebStorm really not play well with it

closed by #706

closed by #706


Adjust PlacementRule patching logic

Previously, the common function ony replaced the matchExpression part
of the clusterSelector, in order to select the target cluster. If the
selector had a matchLabels section, then the resulting PlacementRule
might not select the target cluster, which was likely not the intent.
Signed-off-by: Justin Kulikauskas<EMAIL_ADDRESS>

/retest

/retest


added myself to contributors.md file

I have added myself to contributors list in contributor's.md file.

This file has been removed in earlier PRs and you can add your name here https://github.com/HacktoberfestinAba/HacktoberfestinAba.github.io/blob/master/CONTRIBUTING.md
You can git pull the repo to sync the updates and send in a new PR/ Re-Open this PR


[Driver] Default SDKROOT to / on FreeBSD.

As suggested by Jordan Rose on the mailing lists.
Thanks!
--
Davide

Since this is a driver change, it should be possible to write a direct test... but leaving it to @jrose-apple.

No, indeed, please add a test case along the lines of test/Driver/sdk.swift. You should be able to use env to make sure SDKROOT is unset.
I'd say we should do this for all non-Darwin platforms. I don't think anyone else commonly has multiple copies of headers on their system, and with Xcode you always use 'xcrun' anyway (which sets SDKROOT). If someone is cross-compiling they'll notice something wrong pretty quick.

No problem, I'll try to write a test right away. Thanks for the quick review!
BTW, what's the recommended way to pass a freebsd triple to the driver?

Looks like Clang uses -target x86_64-unknown-freebsd.

OK, thanks. there's too much noise currently running the test suite on FreeBSD, so to make sure, I'll spawn a Linux VM and try to run there to see if the test passes.
Thanks,
--
Davide

Can one of the admins verify this patch?

@swift-ci please test

@dcci were you able to write a test case?

Haven't yet, but will take a look at this in the next day or two. Sorry for the delay on this :|

@swift-ci Please test

Sorry for slacking off on this one. I plan to fix it today or tomorrow.

This one is superseded by https://github.com/apple/swift/pull/1500
Closing this one.


[logs] updating carbonblack:feed.query.hit.process schema

to @mime-frame
cc @airbnb/streamalert-maintainers
size: small
Changes

After seeing an incoming feed.query.hit.process log that has a interface_ip key within docs entries, I've updated the carbonblack:feed.query.hit.process schema to have interface_ip as an optional top level key

Coverage remained the same at 95.295% when pulling b2ebfab8a969e7fea9a06131f7dbcaa5c7ba2439 on ryandeivert-cb-feed-query-hit-process-fix into d3ac7f3d67d9f4284ede55dbbe0460908e3c3f22 on master.


Decision Transformer for trading

Have tried training a Decision Transformer using trading data. FinRL packages and decision transformer related tutorial from Hugging Face were of great help in the process.
Please check and let me know for any suggestions.
https://github.com/ra9hur/Decision-Transformers-For-Trading

Thanks for sharing, I will check.

Can you share the link for decision transformer related tutorial from Hugging Face? I will also check using google. Also how was your experience with FinRL? I am just starting to use it and checking their tutorials.

Thanks for sharing, I will check. The Decision Transformers Vs Mean Variance Optimization Vs Dow Jones Index chart that you shared, is that on training or testing data?

Sure.
This is using test data.

Can you share the link for decision transformer related tutorial from Hugging Face? I will also check using google. Also how was your experience with FinRL? I am just starting to use it and checking their tutorials.

https://huggingface.co/blog/train-decision-transformers


[Main] Update Rancher Client Configurations and RKE Client to Have Different SSH Configuration Options

Updates Rancher client configuration to have JSON tags.
Add two different SSH options to RKECLI client and its configuration:

SSHKey: mainly for corral and CI usage
SSHPath: mainly for legacy tests and local usage

Update release upgrade-related glue codes with local options and simplify the field names.

Todos:

[x] Rebase and create backports after https://github.com/rancher/shepherd/pull/11

Backports:

2.8: https://github.com/rancher/shepherd/pull/72
2.7: https://github.com/rancher/shepherd/pull/71


[JsonConverter] Fixed hydrate null value

What was changed
JsonConverter
Why?
Issue #385
Checklist

Closes

How was this tested:

Any docs updates needed?

@roxblnfk cc

@roxblnfk plasm failed...

@roxblnfk plasm failed...

Yes


Add Terraform configuration for managing infrastructure

Adding this will make it easier to transfer from my AWS account to @daviddeejjames's AWS account!

Merged in https://github.com/tdmalone/coffee-shop-message/commit/f7426111872ee887a00498623a63f341e98cfc31

@daviddeejjames This can now be set up in your AWS account! I can run you through this during the week; we'll be following these instructions.


DOC-1911 v22.1.0-rc.1 release notes

Addresses: DOC-1911

Release notes for v22.1.0-rc.1, to be released May 9th, 2022.

This change is


[ pwm ] Initial RTL for PWM module

Contains basic functionality.
No support for blink modes yet.

Update:  This PR has been modified in the last couple days to alter the core CDC.
A closer examination of the assertions in the reqack_data CDC primitive made me realize that, though it would be possible to use that scheme, it would take some more care in managing the automatic handshake pulses. I don't want to make the programming model for this IP any more complicated than it needs to be, so I wanted the programming model to be "set and forget", and leave all the handshaking to hardware.   There is a solution I like for using reqack this scheme, but it has some quirks that might make verification harder than it needs to be.
That said I've scrapped the Req-Ack scheme, for something dead-simple:

Any changes to control signals reset the internal counters, and reset the PWM or blink patterns to the beginning.

This allows "live" changes to parameters, even when the channel is enabled.
The PWM no longer requires updates of CDC-synced registers whenever the enable signals are pulsed low.

All signals go through 2 flop synchronizers.

For multi-bit signals this is usually not advised for anything except gray-coded counters, as multiple bits may propagate through the CDC at different rates (at most 1
However given that any change to the post-CDC value now clears the internal counters, this means that the PWM channels are held at zero until all bits make it through the CDC.

Since there haven't been any reviews yet, I'm not sure it's worth splitting out the development history. Would you mind squashing this down into the patch sequence that you'd expect to merge?

Since there haven't been any reviews yet, I'm not sure it's worth splitting out the development history. Would you mind squashing this down into the patch sequence that you'd expect to merge?

Hi Rupert.  It has been squashed. Thanks for taking an interest.

Status update: Thanks for the comments everyone.  I'm trying to get spi_host in PR today. Once that's done I'll loop around and fix @rswarbrick's suggestions, then we can merge this.

Looks good to me, with a couple of stylistic nits that you might want to address.

Thanks Rupert!


error running on older rubies

If user accidentally installed gem and ran inside an older ruby, the attached error will occur.  The workaround is of course to use puppet's supplied modern version of ruby:
/opt/puppetlabs/puppet/bin/gem install ncio
/opt/puppetlabs/puppet/bin/ncio backup

/usr/share/ruby/syslog/logger.rb:177:in `initialize': wrong number of arguments (2 for 0..1) (ArgumentError)
	from /usr/local/share/gems/gems/ncio-2.0.0/lib/ncio/support.rb:28:in `new'
	from /usr/local/share/gems/gems/ncio-2.0.0/lib/ncio/support.rb:28:in `syslog_logger'
	from /usr/local/share/gems/gems/ncio-2.0.0/lib/ncio/support.rb:20:in `reset_logging!'
	from /usr/local/share/gems/gems/ncio-2.0.0/lib/ncio/support.rb:80:in `reset_logging!'
	from /usr/local/share/gems/gems/ncio-2.0.0/lib/ncio/app.rb:41:in `reset!'
	from /usr/local/share/gems/gems/ncio-2.0.0/lib/ncio/app.rb:34:in `initialize'
	from /usr/local/share/gems/gems/ncio-2.0.0/exe/ncio:4:in `new'
	from /usr/local/share/gems/gems/ncio-2.0.0/exe/ncio:4:in `<top (required)>'
	from /usr/local/bin/ncio:23:in `load'
	from /usr/local/bin/ncio:23:in `<main>'

Noted, ncio doesn't support oder rubies.  Good to have this issue for the search indexes though in case someone else runs into this issue.


Remove invalid characters in basename sourced from username

Fixes https://github.com/Azure/azure-sdk-tools/issues/1189
This fixes a bug where a username may contain spaces or other invalid characters, e.g.
(py39_new) PS C:\github.com\azure-sdk-for-python> .\eng\common\TestResources\New-TestResources.ps1 'schemaregistry'
New-TestResources.ps1: The variable cannot be validated because the value Sean Kaneschemaregistry is not a valid value for the BaseName variable.

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


ctr: type *specs.Spec: not found

when I use ctr to run a container by type "ctr run docker.io/library/busybox:latest con " in terminal, I got the error:
"ctr: type *specs.Spec: not found"
so how to run a container by using ctr?

You are probably on a old version. It works perfectly fine on master,
please try to update
On Wed, Aug 9, 2017 at 3:26 AM jongwu<EMAIL_ADDRESS>wrote:

when I use ctr to run a container by type "ctr run
docker.io/library/busybox:latest con " in terminal, I got the error:
"ctr: type *specs.Spec: not found"
so how to run a container by using ctr?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/containerd/containerd/issues/1313, or mute the thread
https://github.com/notifications/unsubscribe-auth/APUifk-dxeny9ySsO5i9WCtan1HUjS42ks5sWYk_gaJpZM4Ox4sv
.

--
Sent from my phone.

I update containerd by building latest source code and run "./ctr run docker.io/library/busybox:latest con
" but I get error:
ctr: failed to create active: parent is not committed snapshot: invalid argument
containerd version now is Version: v1.0.0-alpha3-23-ge8625bc.

You need to download the image first: ctr pull docker.io/library/busybox:latest
On Wed, Aug 9, 2017 at 8:24 PM jongwu<EMAIL_ADDRESS>wrote:

I update containerd by building latest source code and run "./ctr run
docker.io/library/busybox:latest con
" but I get error:
ctr: failed to create active: parent is not committed snapshot: invalid
argument
containerd version now is Version: v1.0.0-alpha3-23-ge8625bc.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/containerd/containerd/issues/1313#issuecomment-321440315,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APUifnMN9ZaRwl9MuBreDk3TIBffq3xDks5sWnfwgaJpZM4Ox4sv
.

--
Sent from my phone.

I pull image using ctr, I get error:
ctr: failed to commit snapshot : incompatible value: unknown, so I can't run this images.

You need to delete your /var/lib/containerd the format changed and is now
incompatible
On Thu, Aug 10, 2017, 6:38 PM jongwu<EMAIL_ADDRESS>wrote:

I pull image using ctr, I get error:
ctr: failed to commit snapshot : incompatible value: unknown, so I can't
run this images.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/containerd/containerd/issues/1313#issuecomment-321715139,
or mute the thread
https://github.com/notifications/unsubscribe-auth/APUifolXhe-r38QR24adFle2XdDkBpLXks5sW7CegaJpZM4Ox4sv
.

--
Sent from my phone.

please give an example of how to use ctr to run a container , thanks.

@jongwu like I said, you need to delete the content of /var/lib/containerd and /run/containerd

@jongwu we also have a getting started guide here: https://github.com/containerd/containerd/blob/master/docs/getting-started.md
It does not use ctr but will help you get setup.

ctr is a development tool, to use containerd you should write your own client. Please refer to the starting guide
for ctr (and the syntax may change without notice):
$ ctr pull docker.io/library/redis:alpine
$ ctr run docker.io/library/redis:alpine redis


Load Asset Once

This is not an actual issue. It is a "would be nice" feature.
It would be wonderful if there could be a flag to indicate that the asset should be loaded once. In this way, we can prevent multiple injection of assets into the page(s) when such assets are already there.
One way could be implementing a solution like in C++ with the diamond problem. In C++, we can use the pre-processor to indicate that we only wish to include a class once. If an specific variable doesn't exist, create such variable and include the content, else do not include the content because its already there.

It already does this unless you are incorrectly using the asset tag to add
page js and css instead of the directives and manifest
On Aug 29, 2016 3:56 PM, "Alejandro G. Carlstein Ramos Mejia" <
<EMAIL_ADDRESS>wrote:

This is not an actual issue. It is a "would be nice" feature.
It would be wonderful if there could be a flag to indicate that the asset
should be loaded once. In this way, we can prevent multiple injection of
assets into the page(s) when such assets are already there.
One way could be implementing a solution like in C++ with the diamond
problem. In C++, we can use the pre-processor to indicate that we only wish
to include a class once. If an specific variable doesn't exist, create such
variable and include the content, else do not include the content because
its already there.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/bertramdev/grails-asset-pipeline/issues/370, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AABaEqCVR6WgnKEG6DBq_2d8O5vAEXaxks5qkzlRgaJpZM4Jv1bc
.

I agree this does not appear to do this currently, and I think this might be a bug because it seems to fundamentally break how plugins using asset pipeline should work.  Because of this issue, they cannot contain assets that could have overlapping dependencies with the applications using them (e.g. JQuery)
Here is an example:  A plugin provides an asset manifest myplugin.js that is required internally in one of the plugins GSPs.
myplugin.js
//= require jquery
//= require myLocalFile

myplugin.gsp
<head>
  <asset:javascript src=" myplugin.js" />
</head>

Then in the Application using the plugin (and JQuery)
application.js
//= require jquery
//= require myApplicationFile

main.gsp (application Sitemesh Template wrapping the myplugin.gsp page)
<head>
  <asset:javascript src=" application.js" />
</head>

If the application includes the plugins GSP then the result is including JQuery twice on the page (which is not correct).

not a bug this is by design and how the assets are digested and bundled and are performant without having to rebundle per gsp file

do not include the same file twice in a gsp. if you use the root application.js or application.css to build your final package your good to go

if you need to package shared assets for other views as a plugin you can place them in a assets subfolder by plugin name like assets/javascripts/plugina/test.js then instruct your users to add this line to application.js
//= require_full_tree plugina


Support for custom alert titles

Thanks for this library Chris!
It would be great if we allowed custom titles like Obsidian does.
> [!INFO] Why **CDK**?
> AWS CDK (Cloud Development Kit) is the easiest and faster way to get your AWS Lambda code up and running

Right now it gets rendered as a quote only instead of the alert:

[!INFO] Why CDK?
AWS CDK (Cloud Development Kit) is the easiest and faster way to get your AWS Lambda code up and running

This is how it looks in obsidian:

[!NOTE] my custom title
github test of a note alert with a custom title

I understand Chris.
Thanks for the reply.
The workaround looks good, I never thought of it.


KOGITO-6348: Remove Filterset from Dashbuilder

This PR fixes the issue  from KOGITO-6348 and also fixes a long term issue with components width and height.

It was a simple unplanned glitch fix. Could you please merge it, @tiagobento ?
Thanks!


Add line highlighting

Users can now click on the line numbers to highlight a line. They can share the link of the page and visitors will be scrolled to the line. The way this works is by storing the file number and line number in the hash of the URL.
Hash: #1L6
Will highlight the first file's 6th line.
https://user-images.githubusercontent.com/32761424/103143947-96237900-46ee-11eb-939c-39adeba24b87.mp4
Resolves #39

Hey this looks awesome, could you retarget the PR to the development-v1.3.0 branch and include a CHANGELOG line perhaps? 👍

Alright I did as you asked. I also had time to think about the code and I added some improvements. My reference for this feature was GitHub's line highlighting. Every time the URL's hash gets updated, you will be scrolled to that line except if you click a line number.

Sounds pretty good I'll take a look at it somewhere this weekend and merge it!

Alright, I'll probably end up making some changes before then.

Perfect, thank you. A 1.3 release will be upcoming somewhere in the next weeks with this included.

Perfect, thank you. A 1.3 release will be upcoming somewhere in the next weeks with this included.