Added/updated EndpointDlp

Also massive consistency updated and added new param visible after adding myself to required Azure AD roles

Docs Build status updates of commit c73ff52:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/New-DlpCompliancePolicy.md
:x:Error

Details

exchange/exchange-ps/exchange/New-DlpComplianceRule.md
:x:Error

Details

exchange/exchange-ps/exchange/Add-MailboxFolderPermission.yml
:warning:Warning

Details

exchange/exchange-ps/exchange/index.yml
:warning:Warning

Details

exchange/exchange-ps/exchange/Get-MailboxFolderPermission.yml
:warning:Warning

Details

exchange/exchange-ps/exchange/Get-EXOMailboxFolderPermission.yml
:warning:Warning

Details

exchange/exchange-ps/exchange/New-DlpCompliancePolicy.md

Line 0, Column 0: [Error: MAML2Yaml_LoadFileFailed]

Failed to load file: C:/LocalRun/W/pse1-s/exchange/exchange-ps/exchange/New-DlpCompliancePolicy.md:638:(1335) '###

'
 Unexpected header name #, please try to use PlatyPS validate the content

exchange/exchange-ps/exchange/New-DlpComplianceRule.md

Line 0, Column 0: [Error: MAML2Yaml_LoadFileFailed]

Failed to load file: C:/LocalRun/W/pse1-s/exchange/exchange-ps/exchange/New-DlpComplianceRule.md:3021:(1177) '###

'
 Unexpected header name #, please try to use PlatyPS validate the content

exchange/exchange-ps/exchange/Add-MailboxFolderPermission.yml

Line 22, Column 9: [Warning: file-not-found - See documentation] Invalid file link: 'Set-MailboxFolderPermission.yml'.
Line 24, Column 9: [Warning: file-not-found - See documentation] Invalid file link: 'Remove-MailboxFolderPermission.yml'.

exchange/exchange-ps/exchange/index.yml

Line 16, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Reset-ProvisioningCache'.
Line 17, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-ADServerSettings'.
Line 18, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-ADSite'.
Line 19, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-AdSiteLink'.
Line 76, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.New-HostedContentFilterRule'.
Line 77, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.New-HostedOutboundSpamFilterPolicy'.
Line 78, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.New-HostedOutboundSpamFilterRule'.
Line 79, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.New-MalwareFilterPolicy'.
Line 87, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Preview-QuarantineMessage'.
Line 89, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Remove-AttachmentFilterEntry'.
Line 92, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Remove-HostedContentFilterPolicy'.
Line 93, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Remove-HostedContentFilterRule'.
Line 94, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Remove-HostedOutboundSpamFilterPolicy'.
Line 95, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Remove-HostedOutboundSpamFilterRule'.
Line 96, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Remove-IPAllowListEntry'.
Line 97, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Remove-IPAllowListProvider'.
Line 98, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Remove-IPBlockListEntry'.
Line 99, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Remove-IPBlockListProvider'.
Line 100, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Remove-MalwareFilterPolicy'.
Line 101, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Remove-MalwareFilterRule'.
Line 102, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Remove-PhishSimOverridePolicy'.

This comment lists only the first 25 errors (including error/warning/suggestion) in the pull request.
For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit de47f19:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/New-DlpCompliancePolicy.md
:x:Error

Details

exchange/exchange-ps/exchange/New-DlpComplianceRule.md
:x:Error

Details

exchange/exchange-ps/exchange/Get-MailboxFolderPermission.yml
:warning:Warning

Details

exchange/exchange-ps/exchange/Get-EXOMailboxFolderPermission.yml
:warning:Warning

Details

exchange/exchange-ps/exchange/index.yml
:warning:Warning

Details

exchange/exchange-ps/exchange/Add-MailboxFolderPermission.yml
:warning:Warning

Details

exchange/exchange-ps/exchange/Remove-MailboxFolderPermission.yml
:warning:Warning

Details

exchange/exchange-ps/exchange/New-DlpCompliancePolicy.md

Line 0, Column 0: [Error: MAML2Yaml_LoadFileFailed]

Failed to load file: C:/LocalRun/W/ozmg-s/exchange/exchange-ps/exchange/New-DlpCompliancePolicy.md:638:(1335) '###

'
 Unexpected header name #, please try to use PlatyPS validate the content

exchange/exchange-ps/exchange/New-DlpComplianceRule.md

Line 0, Column 0: [Error: MAML2Yaml_LoadFileFailed]

Failed to load file: C:/LocalRun/W/ozmg-s/exchange/exchange-ps/exchange/New-DlpComplianceRule.md:3021:(1177) '###

'
 Unexpected header name #, please try to use PlatyPS validate the content

exchange/exchange-ps/exchange/Get-MailboxFolderPermission.yml

Line 24, Column 9: [Warning: file-not-found - See documentation] Invalid file link: 'Set-MailboxFolderPermission.yml'.

exchange/exchange-ps/exchange/Get-EXOMailboxFolderPermission.yml

Line 20, Column 9: [Warning: file-not-found - See documentation] Invalid file link: 'Set-MailboxFolderPermission.yml'.

exchange/exchange-ps/exchange/index.yml

Line 17, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-ADServerSettings'.
Line 18, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-ADSite'.
Line 19, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-AdSiteLink'.
Line 87, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Preview-QuarantineMessage'.
Line 97, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Remove-IPAllowListProvider'.
Line 98, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Remove-IPBlockListEntry'.
Line 102, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Remove-PhishSimOverridePolicy'.
Line 104, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Remove-QuarantinePolicy'.
Line 106, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Remove-SecOpsOverrideRule'.
Line 109, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-AttachmentFilterListConfig'.
Line 110, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-ContentFilterConfig'.
Line 111, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-DkimSigningConfig'.
Line 112, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-HostedConnectionFilterPolicy'.
Line 113, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-HostedContentFilterPolicy'.
Line 114, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-HostedContentFilterRule'.
Line 115, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-HostedOutboundSpamFilterPolicy'.
Line 116, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-HostedOutboundSpamFilterRule'.
Line 117, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-IPAllowListConfig'.
Line 118, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-IPAllowListProvider'.
Line 119, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-IPAllowListProvidersConfig'.
Line 120, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-IPBlockListConfig'.

This comment lists only the first 25 errors (including error/warning/suggestion) in the pull request.
For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit da9389f:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/New-DlpCompliancePolicy.md
:x:Error

Details

exchange/exchange-ps/exchange/index.yml
:warning:Warning

Details

exchange/exchange-ps/exchange/Get-EXOMailboxFolderPermission.yml
:warning:Warning

Details

exchange/exchange-ps/exchange/Add-MailboxFolderPermission.yml
:warning:Warning

Details

exchange/exchange-ps/exchange/Get-MailboxFolderPermission.yml
:warning:Warning

Details

exchange/exchange-ps/exchange/New-DlpCompliancePolicy.md

Line 0, Column 0: [Error: MAML2Yaml_LoadFileFailed]

Failed to load file: C:/LocalRun/W/kfpp-s/exchange/exchange-ps/exchange/New-DlpCompliancePolicy.md:638:(1335) '###

'
 Unexpected header name #, please try to use PlatyPS validate the content

exchange/exchange-ps/exchange/index.yml

Line 13, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Get-Trust'.
Line 15, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Remove-ADPermission'.
Line 16, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Reset-ProvisioningCache'.
Line 17, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-ADServerSettings'.
Line 18, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-ADSite'.
Line 19, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Set-AdSiteLink'.
Line 76, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.New-HostedContentFilterRule'.
Line 77, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.New-HostedOutboundSpamFilterPolicy'.
Line 78, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.New-HostedOutboundSpamFilterRule'.
Line 79, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.New-MalwareFilterPolicy'.
Line 83, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.New-QuarantinePolicy'.
Line 84, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.New-SecOpsOverridePolicy'.
Line 86, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.New-TenantAllowBlockListItems'.
Line 88, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Release-QuarantineMessage'.
Line 89, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Remove-AttachmentFilterEntry'.
Line 90, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Remove-BlockedSenderAddress'.
Line 91, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Remove-ContentFilterPhrase'.
Line 92, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Remove-HostedContentFilterPolicy'.
Line 93, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Remove-HostedContentFilterRule'.
Line 94, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Remove-HostedOutboundSpamFilterPolicy'.
Line 95, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Remove-HostedOutboundSpamFilterRule'.
Line 96, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Remove-IPAllowListEntry'.
Line 97, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Remove-IPAllowListProvider'.
Line 98, Column 5: [Warning: xref-not-found] Cross reference not found: 'ExchangePowerShell.Remove-IPBlockListEntry'.

This comment lists only the first 25 errors (including error/warning/suggestion) in the pull request.
For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit fb6658a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/Get-RegulatoryComplianceUI.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-DlpCompliancePolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/New-DlpComplianceRule.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-DlpCompliancePolicy.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-DlpComplianceRule.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/Set-RegulatoryComplianceUI.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


New TFS is very buggy

The new updates now, when copy in grid, does not copy text correctly, causing text to overlap. Can no longer copy and paste text to next test lines when inside a test case. Test cases now open in Execute mode, which causes confusion if you are trying to make changes to the test scripts. These new tabs are very inconvenient.

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

ID: 3c520dee-1218-777e-7405-551623817c03
Version Independent ID: 82b5d172-ae1a-2b4c-82ac-595cd7609d3c
Content: New test plans page - Azure Test Plans
Content Source: docs/test/new-test-plans-page.md
Product: devops
Technology: devops-test
GitHub Login: @ravishan-msft
Microsoft Alias: ravishan-msft

@Nikababy01, thank you for your feedback.
It looks like you've encountered an issue with the product itself, rather than an issue with the documentation. To make sure it gets in front of the right people, please submit your bug here.


update to give each agent individual keys

Closes #46

Coverage decreased (-0.3%) to 58.481% when pulling 533fce0af19068ae53b9fd765233ef7281e0f5cd on individual-keys into 8099cfe7e6f238eadf402ac7e9895c12aba462e4 on master.

Coverage decreased (-0.3%) to 58.481% when pulling 533fce0af19068ae53b9fd765233ef7281e0f5cd on individual-keys into 8099cfe7e6f238eadf402ac7e9895c12aba462e4 on master.


Fix concurrent access to zedagent metrics

Zedagent metrics can be accessed concurrently and cause zedbox panic.
This is because the recommended usage for MetricsMap was not followed in one particular instance.
Signed-off-by: Milan Lenco<EMAIL_ADDRESS>

But we do all cipher.Append few lines before, seems it is not the root cause.

We do but it is the original zedagentMetrics that is being published here, not cms with the copied content.


chore(cargo): Update dependency

What this PR does 📖

Add limits on names for files and directories (limit is set to 256)
Adds Constellation::max_size and Constellation_current_size to give the amount of data stored and the max allowed
Switch from using Constellation::put_stream to Constellation::put
Minor Optimizations

Which issue(s) this PR fixes 🔨

N/A

Special notes for reviewers 🗒
Should not have any major changes that require testing
Additional comments 🎤

The limit for the names of the files or directories are a hard cap (keeping in compatibility with most major OS max name limits). Uplink still has it capped off at 64 characters, which is fine. If there are any interactions with Directory or File when setting or renaming and the name hits the limit, it will only use the first 256 characters.
Sending/receiving friend request with this PR to/from a prior commit will not work, however this does not require a new account.

linter is failing 🔨


Fix test and adjust code

This PR has several small tweaks and specially fixes the test code which was using the old breakpoints variables.

Update comments on the code that were using the old variables names
Adjust line spacing on some parts of the code
Update .editorconfig
Update packages and add licence to packge.json
Update bower version
Remove trailing spaces

@felics

@renatocarvalho thank you so much for this!

@felics You're welcome! Don't you need to do another release?

@renatocarvalho yup done. needed some beauty-sleep :lipstick:

@felics :smile:


helix-gpt: 0.31.0 -> 0.34.0

simple update
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[X] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 354767

x86_64-linux

:white_check_mark: 1 package built:

helix-gpt

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 354767

x86_64-darwin

:white_check_mark: 1 package built:

helix-gpt


#1388 against rhel-8.4.0 branch

This pull request includes:

[ ] adequate testing for the new functionality or fixed issue
[ ] adequate documentation informing people about the change such as

[ ] create a file in news/unreleased directory if this change should be mentioned in the release news
[ ] submit a PR for the guides repository if this PR changed any behavior described there: https://www.osbuild.org/guides/

FTR:
https://osbuildci.cloud.paas.psi.redhat.com/blue/organizations/jenkins/osbuild%2Fosbuild-composer/detail/PR-1390/2/pipeline/88 - regular execution against the first 3 commits of this PR, with osbuild-composer-tests built from source.


Github 1390: CmdBeginRenderpass issues

Validation was printing incorrect handle values and one of the error messages was incorrect.  Fixes #1390.

Fix errors Chris found, took Dave's suggestion and pushed to Master.


Create views for vasoactive infusion durations similar to MIMIC-III

Creates materialized views for vasoactive infusion durations (similar to MIMIC-II).
Closes #111.
I think it would be reasonable to move concepts into concepts/postgres and create an empty concepts/bigquery for future views.

Just checking back in about this -- I was planning on using these views in a manuscript but would prefer to feed them back into the upstream.

Thanks @ajb5d, and sorry for the delay. We appreciate your work in making the pull request and get onto this as soon as we can.

Is there interest in additional contributed views? I have another one for ventilator durations -- I can create another PR if there is interest. Should it be in big query or Postgres?


Implement train() using PyTorch-Ignite

We do not have complicated logic in the train loop, so it can be simplified with Ignite

Hey, I want to contribute.
How can I help?


feat: Update panels from field change events

resolves #1924

Closing as we'll probably need a different approach and this has been de-prioritized since vscode extension now uses embed widget.


DataFactory Linked to GIT

When performing the Get-AzDataFactoryV2Pipeline or Get-AzDataFactoryV2Trigger on a DataFactory that is connected to GIT, it doesn't return any Pipelines or Triggers, it returns the initial configuration prior to the attachment to GIT.
I have pressed the publish button and it has no issues with the configuration that is stored in GIT, yet still the same issue faced.
This is not helpful as you will generally attach GIT to your development environment and run a script on it to determine to the deployment order in subsequent environments.

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

ID: 1e22e4d6-366f-fbf8-e141-8905ca7ab7ae
Version Independent ID: 5be2acf2-66e7-cadf-6666-5a85a65b9771
Content: Continuous integration and delivery in Azure Data Factory - Azure Data Factory
Content Source: articles/data-factory/continuous-integration-deployment.md
Service: data-factory
GitHub Login: @djpmsft
Microsoft Alias: daperlov

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

Hey @Gitdevnttuser ,
Rest API calls, Powershell scripts, and SDK code all queries against the live factory (aka the 'published' version of your git repo.
For promoting to subsequent environments (test/UAT, PROD, etc), it is recommended to use the ARM template generated in your publish branch and do a Resource Manager deployment (this can be done natively in Azure DevOps)
If you wish to use PowerShell, you will still need to publish from your collaboration branch, prior to promotion.
Thanks,
Daniel

We will now proceed to close this thread as we have not heard back from you. If there are further questions regarding this matter, please tag me in your reply and reopen the issue. We will gladly continue the discussion.


khr::RayTracing::cmd_trace_rays

khr::RayTracing::cmd_trace_rays should accept references to vk::StridedBufferRegionKHR, and not to slices.
See https://www.khronos.org/registry/vulkan/specs/1.2-extensions/man/html/vkCmdTraceRaysKHR.html
Particularly:
• pRaygenShaderBindingTable must be a valid pointer to a valid VkStridedBufferRegionKHR structure
• pMissShaderBindingTable must be a valid pointer to a valid VkStridedBufferRegionKHR structure
• pHitShaderBindingTable must be a valid pointer to a valid VkStridedBufferRegionKHR structure
• pCallableShaderBindingTable must be a valid pointer to a valid VkStridedBufferRegionKHR structure

It is not a block as one can use std::slice::from_ref to convert reference to a slice.

This was fixed in 84624fddd809ef13af4061e154ed130d950326bf.


Added the fetch api

Fixes #4948
I didn't know how to write unit tests for it since its actually a library change.

Hi @GamesMaxed, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, MSBOT;

I have got no idea why the linting failed, I dont know what file even. Can someone please tell me?

I've got no idea why it did not lint correctly, can someone help me out here

I used this fore a basic request and blogged about it here:
http://blog.ctaggart.com/2016/12/fetch-api-via-typescript-async.html
Looking forward to seeing this worked on and merged. Thanks for putting this together.

Has this PR been abandoned?  I'd love to see this get merged in after the concerns are addressed...

Yes sorry, I was busy in real life with exams and some private stuff. What needs to be done now? And I'm not familiar with how the changes work? Do I need to pull them into my repo?

@GamesMaxed just update your fork based on the comments and push to your fork again. The new commit will be built by CI. If the build has no errors and all the changes have been done they will merge your PR.

The fetch API should be in the standard library after https://github.com/Microsoft/TypeScript/pull/13856


fix: Unreferenced Microsoft.ApiManagement files

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-java

Automation for azure-sdk-for-python

Automation for azure-sdk-for-go

Automation for azure-sdk-for-js

Can one of the admins verify this patch?

@solankisamir PTAL

Beore removing the examples, codeowners need to investigate if they still need these example files.


[ALL] Check script

This is my solution for https://github.com/infinitered/ignite/issues/37
Now when you modify the templates/structure you can modify the tester in the same PR, instead of modifying semaphore.
Additionally, I've riddled it with comments so people can see WHY we're checking things in the current stage of the project life.   Besides failing, the script will output friendly error messages to help identify why the checks did not pass.
I've also take the liberty of enforcing standard compliance on the CLI and the Generator code bases, and even caught one linting error in the process.

Failing

Passing

Not the friendliest output, but functional for semaphore to run.


⚠ Turnos 24x7 has degraded performance

In 1b1c3ae, Turnos 24x7 (https://reservas-dev.alternativasinteligentes.com/) experienced degraded performance:

HTTP code: 200
Response time: 2395 ms

Resolved: Turnos 24x7 performance has improved in e01f1a3.


Fix projection matrix computation when using an orthographic projection

Partial correction for https://github.com/JuliaPlots/Makie.jl/issues/1142.

examples3d_20_OrthographicCamera failed, I'm on it.

@SimonDanisch, running the test locally produces:
new

ref

which shows imho a difference in colors, and not a regression.
Where can I submit a PR to change the ref image ?

This is probably a lighting thing. By default the light source is at scene.camera.eyeposition, so you should get maximally bright shading. If that's not the case anymore camera.eyeposition is either out of sync with Camera3D's eyeposition or both are out of sync with where the camera actually is.

@SimonDanisch the CI seems broken, and unrelated to this PR:
 text_252_latexstrings.png: Test Failed at /home/runner/work/Makie.jl/Makie.jl/ReferenceTests/src/runtests.jl:106
  Expression: score < difference
   Evaluated: 0.037494160234979264 < 0.03

I fixed the default eyeposition for Orthopragphic projection:
using GLMakie
using Makie

main(proj) = begin
  xs = LinRange(0, 10, 100)
  ys = LinRange(0, 15, 100)
  zs = [cos(x)*sin(y) for x in xs, y in ys]

  fig, ax, p = fap = surface(xs, ys, zs)
  cam3d!(ax.scene, projectiontype=proj)

  save("sf-$proj.png", fap)
  display(fap)
  return
end

main(Makie.Orthographic)
main(Makie.Perspective)

Orthographic

Perspective

This is probably a lighting thing. By default the light source is at scene.camera.eyeposition, so you should get maximally bright shading. If that's not the case anymore camera.eyeposition is either out of sync with Camera3D's eyeposition or both are out of sync with where the camera actually is.

I thought a bit more about this - with orthographic projection there is no clear position for eyeposition because depth has no effect. There is no difference between eyeposition = lookat - normalize(viewdir) and eyeposition = lookat - 1000 * normalize(viewdir) when it comes to the camera. Well I suppose there would be after this pr, but it's artificial. (Which doesn't mean we shouldn't have it. If it makes things easier to use I'm all for it.)

This is probably a lighting thing. By default the light source is at scene.camera.eyeposition, so you should get maximally bright shading. If that's not the case anymore camera.eyeposition is either out of sync with Camera3D's eyeposition or both are out of sync with where the camera actually is.

I thought a bit more about this - with orthographic projection there is no clear position for eyeposition because depth has no effect. There is no difference between eyeposition = lookat - normalize(viewdir) and eyeposition = lookat - 1000 * normalize(viewdir) when it comes to the camera. Well I suppose there would be after this pr, but it's artificial. (Which doesn't mean we shouldn't have it. If it makes things easier to use I'm all for it.)

While I trust you on the maths of an orthographic projection, I disagree, there is an effect on the camera.
It I restore back to:
# restore previous behavior
w = 0.5f0 * (1f0 + aspect) * zoom_mult
h = 0.5f0 * (1f0 + 1f0 / aspect) * zoom_mult

[...]

# only update eyeposition on Perspective projection
if camera.attributes[:projectiontype][] == Perspective
    camera.eyeposition[] = middle .+ (1.2 * norm(width) .* old_dir)
end

I obtain this:

Please provide a reproducible example.

fig, ax, p = scatter(rand(Point3f0, 100))
cam = Camera3D(ax.scene, projectiontype = Makie.Orthographic)

And then try various factors in
update_cam!(ax.scene, cam, cam.lookat[] - 10 * normalize(cam.lookat[] - cam.eyeposition[]), cam.lookat[], cam.upvector[])

This shouldn't change the plot at all as long as it's between near and far clip. Clipping happens around 1 and 100 for me, between that nothing happens.
Conceptually the view matrix should just position the camera. Move it to eyeposition and rotate it so that upvector is upwards and lookat is what the camera is pointed at. The projection matrix does the scaling + translation to set up the corrdinate space that will be rendered. The orthographic projection scales x, y, z independently, so it doesn't matter for x and y what z is - there is no depth scaling. If the matrix always scales x by 1 it doesn't matter if an object is 1 unit or 100 units away, assuming they're not clipped. Perspective projection on the other hand includes scaling of x and y based on z, so an object 1 unit away might be scaled by a factor of 10 and an object 100 units away by 1.
But I think this is getting a bit off-topic. This was meant to fix translation speed, right? I think the goal should be that if you start a translation with a point under your cursor, it should still be under the cursor when you finish, regardless of zoom etc for an orthographic camera. As far as I can tell we're not there yet?

fig, ax, p = scatter(rand(Point3f0, 100))
cam = Camera3D(ax.scene, projectiontype = Makie.Orthographic)

And then try various factors in
update_cam!(ax.scene, cam, cam.lookat[] - 10 * normalize(cam.lookat[] - cam.eyeposition[]), cam.lookat[], cam.upvector[])

This shouldn't change the plot at all as long as it's between near and far clip. Clipping happens around 1 and 100 for me, between that nothing happens.

I will try that now.

Conceptually the view matrix should just position the camera. Move it to eyeposition and rotate it so that upvector is upwards and lookat is what the camera is pointed at. The projection matrix does the scaling + translation to set up the corrdinate space that will be rendered. The orthographic projection scales x, y, z independently, so it doesn't matter for x and y what z is - there is no depth scaling. If the matrix always scales x by 1 it doesn't matter if an object is 1 unit or 100 units away, assuming they're not clipped. Perspective projection on the other hand includes scaling of x and y based on z, so an object 1 unit away might be scaled by a factor of 10 and an object 100 units away by 1.

Thanks for the detailed explanation.

But I think this is getting a bit off-topic. This was meant to fix translation speed, right?

This PR was originally aimed at fixing the second issue mentioned in https://github.com/JuliaPlots/Makie.jl/issues/1142. I understand that I can use old_cam3d! to restore previous behavior, but I prefer to understand why the new cam3d! or Camera3D that you wrote in https://github.com/JuliaPlots/Makie.jl/pull/1062 is failing this particular mwe.

But I think this is getting a bit off-topic. This was meant to fix translation speed, right?

This PR was originally aimed at fixing the second issue mentioned in #1142. I understand that I can use old_cam3d! to restore previous behavior, but I prefer to understand why the new cam3d! or Camera3D that you wrote in #1062 is failing this particular mwe.

Oh so this is about translations going wild on zoom? That's the shift_lookat branch in zoom!. Seems like I tunneled on getting perspective projection to work and forgot about orthographic entirely... If you want to fix that you should try splitting that into an Orthographic branch and a Perspective branch. You can also just turn off those translations with cam3d!(..., zoom_shift_lookat = false).
I am trying to get orthographic drag-translations to be exact atm, I can look into getting the zoom-shift to work too.

Ok so reverting this PR and using
using Random; Random.seed!(1234)
using LinearAlgebra
using GLMakie

main() = begin
  fig, ax, p = scatter(rand(Point3f0, 100))
  cam = Camera3D(ax.scene, projectiontype=Makie.Orthographic)

  for fact ∈ (2, 10, 50, 100, 200)
    eyeposition = cam.lookat[] - fact * normalize(cam.lookat[] - cam.eyeposition[])
    @show eyeposition
    update_cam!(ax.scene, cam, eyeposition, cam.lookat[], cam.upvector[])

    # important to set update=false, else `update_cam!(scene::Scene, camera::Camera3D, area3d::Rect)` resets the camera
    save("cam-$fact.png", fig; update=false)
  end

  display(fig; update=false)
  return
end

if @__FILE__() ∈ ("string", abspath(PROGRAM_FILE))
  main()
end

does not modify the output figures, apart from the clipping that you mentioned.

Oh so this is about translations going wild on zoom? That's the shift_lookat branch in zoom!. Seems like I tunneled on getting perspective projection to work and forgot about orthographic entirely... If you want to fix that you should try splitting that into an Orthographic branch and a Perspective branch. You can also just turn off those translations with cam3d!(..., zoom_shift_lookat = false).

Thanks, I will try that.

If you want to fix that you should try splitting that into an Orthographic branch and a Perspective branch.
Ok, will modify the PR it that works :wink:

Oh so this is about translations going wild on zoom? That's the shift_lookat branch in zoom!. Seems like I tunneled on getting perspective projection to work and forgot about orthographic entirely... If you want to fix that you should try splitting that into an Orthographic branch and a Perspective branch. You can also just turn off those translations with cam3d!(..., zoom_shift_lookat = false).

Ok, so zoom_shift_lookat=false works on getting data visible.

I am trying to get orthographic drag-translations to be exact atm, I can look into getting the zoom-shift to work too.

Yes, drag under orthographic proj is a bit too fast ...

Superseded by https://github.com/JuliaPlots/Makie.jl/pull/1159.


Fix for typo in informational message

Hey folks, I found a minor typo in an informational message covering the inability to retrieve an IP address from an existing libvirt domain. This small patch fixes it.

Thanks for the patch!

Happy to help!


Rerender qt6

@conda-forge-admin please rerender

Generally you should never push to productive branches directly, and if so, only if you're absolutely sure (and the change is minimal - e.g. a forgotten build number bump).
In this case, with two direct commits (0d02b5ee1ff026b54e0d95b178b24dfbc4fc00ad, 5c9eca503b7a031085ed12b430f2838f595967ae)  that did not achieve the intended result, after this PR we'll now have three builds (_1, _2, _3) for one change (#234), two of which are 100% useless, but will hang around eating storage ~forever.
Please be more careful next time, and remember that things involving compiler versions always require a rerender.

sorry


New-PnPSite can't create new team site

Reporting an Issue or Missing Feature
Reporting and issue with New-PnPSite, the condole hangs after I run the script. They O365 group is created but the site isn't.
Expected behavior
Team site and O365 group assigned get created.
Actual behavior
O365 group is created but team site isn't.
Steps to reproduce behavior
Running PowerShell 7.1 and version 1.3.0 or pnp.powershell.
#1
#.csv with general access sites details
$sites = Import-Csv C:\Code\SPFileSystem\OpsGeneralAccess\OpsGeneralSite.csv
$Creds = Get-Credential
#2
#SharePointPnP Create sites
#Connect SharePointPnP
Register-PnPManagementShellAccess
Connect-PnPOnline -url "https://xxxxx.sharepoint.com" -Credentials $Creds
foreach ($site in $sites){
New-PnPSite -Type $site.type -Title $site.title -Alias $site.alias
}
What is the version of the Cmdlet module you are running?
Running PowerShell 7.1 and version 1.3.0 or pnp.powershell.
Which operating system/environment are you running PnP PowerShell on?

[x ] Windows
[ ] Linux
[ ] MacOS
[ ] Azure Cloud Shell
[ ] Azure Functions
[ ] Other : please specify

Hi @tina-palomino - I can't seem to be able to reproduce your issue, working fine on 1.3.0 and PS7.
Could you try to create only 1 site and not use your .csv file?

@veronicageek me too, encountered the same issue where the site can't be created. The error message also didn't provide a clue of what's going on.
In my case, it was due to the dash or special characters in the URL. Once removed, the site can be created. Can please at least give a clearer error message?
The same issue exists in both v1.3.0 and v1.4.0. I'm using PS 5.1 by the way.

Thank you for your comment @radityoardi
@tina-palomino - I'll close this for now, but do re-open it if you're still facing the issue.


Question: Is there a reason that Gemmini with int8 configuration does not support GEMM operation?

As the title said, I am just curious if there a specific reason that int8 gemmini does not support GEMM operation compared to fp32 configuration. If there is no specific reason, I am trying to implement this operation in int8 configuration.

We do offload int8 GEMMs to Gemmini in fact. We use 8-bit inputs, and accumulate the results into 32-bit partial sums, before scaling the final results back down to 8-bits.

However, it seems that inside systolic_execution_provider.cc code, only QLinearMatMul, QLinearConv, QLinearConvTranspose, QLinearConv_nhwc is supported. Also, we I run googlenet on spike gemm operation is operated on the CPU. Am I missing something?

https://github.com/ucb-bar/onnxruntime-riscv/blob/2021-12-23/onnxruntime/core/providers/systolic/systolic_execution_provider.cc#:~:text=of%20op%20kernels-,%23ifdef%20SYSTOLIC_INT8,-class%20ONNX_OPERATOR_TYPED_KERNEL_CLASS_NAME(kSystolicExecutionProvider

![image](https://user-images.githubusercontent.com/20737367/160756303-0af2c7b3-d325-48aa-80c7-e3209c470a48.png)

Oh, well, we only run quantized layers on Gemmini. It's possible that the Googlenet implementation keeps the final fully-connected layer in floating point, to maximize model accuracy.


Computed attributes are not shown

Describe the bug
Computed attributes are not shown.
Steps to reproduce

Import walrus file nexcloud-mysql from Walrus File Hub.
Edit the nextcloud-db resource, and clone it to another env, and edit the clone resource.

Result
Computed attributes are not shown.

Environment

Walrus version: v0.5.0

Viewing the schema_default_value of the built-in definition mysql matching rule, it's empty.

Viewing the matching rule of manually created resource definitions, default values will be calculated.

Environment

Walrus version: main 5d939ccf90935cc12b8b7cd7d28e5bd255c22df5
UI version: dev-a712ac5

Test Result: pass


skip ubuntu dhcp setting step

Hi,
When installing ubuntu operation system, it always shows the dhcp setting screen, if so , the automatic installation process will be interrupted.  how to skip it?
From DaoCloud

@WangWinson: you may have the answer.

Hi @James-Dao,
The ubuntu installing we support currently is only net installer mode. So the install process will always ensure the net work setting works at first.
So if my guess correct. You may try manually update the dhcp config on the rackhd server, just add the last line to /etc/dhcp/dhcp.conf as below:

...
subnet <IP_ADDRESS> netmask <IP_ADDRESS> {
range <IP_ADDRESS> <IP_ADDRESS>;
Use this option to signal to the PXE client that we are doing proxy DHCP
option vendor-class-identifier "PXEClient";
option routers <IP_ADDRESS>;
Then reboot dhcp service or the rackhd server , then restart the install os workflow to see if this solve the issue.
Note: above example suppose your rackhd control network NIC is set to <IP_ADDRESS>
Thanks

got it. thanks. I will try it.

@WangWinson @yyscamper @leoyjchang  a story for this https://www.pivotaltracker.com/story/show/120504923 was created previously

Close this ticket as question has been answer.


Update README.md

This file is updated as per the description. Now contribution section is at the top and it helps users to see who is contributing.

@caiton1 I have made changes in the read.me file. Please check the changes and if that is looking good you can approve my pull request and close this. Thanks.


[Feature] Resolve resources dependencies in one component

Is your feature request related to a problem? Please describe.

Now a component corresponds to a group of resources, so there may be dependencies between resources
Describe the solution you'd like

Dependencies can be expressed by resource annotations.
for example as follow
outputs: service: {
   metadata: annotations: "oam.dev/dependsOn": "[my-deployment]"
  ...
}

outputs: "my-deployment": {...}

Describe alternatives you've considered

Additional context

Note that this is only a simple startup dependency. Try to avoid data passing demands

Hi @leejanee, I want to help implementing this, could you please provide more info about this?

@LiuYuHui You can contact me on DingTalk(ID: sqleejan)

is this issue still valid? if yes, provide me some resources to get started.

@Sajiyah-Salat Thank you! Seems we need wait for more evidence for this issue.


[AD&D 2E Revised] Repeating section cleanup and bugfixes

Changes / Comments
All changes are described on the forum: https://app.roll20.net/forum/permalink/9683363/
Fields has been removed, and the data has been moved to repeating rows by sheet worker. All the moving has been tested and validated.
Roll20 Requests
Comments are very helpful for reviewing the code changes. Please answer the relevant questions below in your comment.

[X] Does the pull request title have the sheet name(s)? Include each sheet name.
[X] Is this a bug fix?
[ ] Does this add functional enhancements (new features or extending existing features) ?
[ ] Does this add or change functional aesthetics (such as layout or color scheme) ?
[X] If changing or removing attributes, what steps have you taken, if any, to preserve player data ?
[ ] If this is a new sheet, did you follow Building Character Sheets standards ?

If you do not know English. Please leave a comment in your native language.

Character Sheet Info Roll20 Internal Use only.

Character Sheet Info Roll20 Internal Use only.


Removes unused careers CSS

careers.less was not being included in main.less so does not appear to be used at all.
Removals

careers.less
t-careers_social CSS class, which was unused.

Testing

An individual job posting should not experience a change in layout after this change, as  t-careers_social was adding a float:right, but does not appear to be used.

Notes

I'm not sure how to load mock jobs locally to test, and the live site doesn't have any job postings currently so I can't test remove  t-careers_social there, so we may want to wait till a job is posted unless you'll can test.

Looks good, but I'd wait for @sebworks to answer you on the class. I don't remember what it was from either.

@jimmynotjim, I'm writing some tests for this PR.

@anselmbradford, @jimmynotjim please check out the margin top I added to the social sharing.

Works for me! 👍  Let @jimmynotjim sign off before merging. I was trying to find a way to not need the margin util, but didn't find a way.

Sounds good. 💯


smartmontools: fix missing sed

Description of changes
Fixes #185095.
Added gnused to smartmontools.
/etc/smartd_warning.sh requires sed to be in the path.
Previously failing with:
Test of /nix/store/134ckj3riid6ss5iy91b6b14bqyi7qf3-smartd-notify.sh to <nomailer> produced unexpected output (118 bytes) to STDOUT/STDERR:
/nix/store/hhfr3ih96hzgxhhmjxbns726qx6mfff1-smartmontools-7.3/etc/smartd_warning.sh: line 125: sed: command not found
Test of /nix/store/134ckj3riid6ss5iy91b6b14bqyi7qf3-smartd-notify.sh to <nomailer>: failed (32-bit/8-bit exit status: 32512/127)

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Would be nice to have the full context (so what you put in the PR message now as a cover letter) in the commit message, but diff LGTM otherwise

@Mindavi should be in there now.


Add a Heap data structure

[ ] Description
A clear and concise description of request/enhancement.
[ ] Use cases

Save the world.
Stop the imposter.
....

[ ] Proposed design
A clear and concise description of the design planned for this feature/enhancement.
Attach design diagrams if applicable.
[ ] Alternative design approaches
A list of other approaches that could've been taken to design this feature/enhancement,
but hasn't been because of a specified reason.
[ ] Workaround
If any workaround exists to achieve the desired goal without this feature/enhancement.
[ ] Additional information
Add any other context about the problem here.

Priority Queues can be used as binary heaps. But, not able to come across a popular library with D-ary heap implementation for java.
https://stackoverflow.com/a/57833871


[add-resource] New Plugin

For your consideration: 'Perlkonig/grav-plugin-classifier'
Allows you to add class declarations to rendered HTML tags.

Nice one

Added!


Windows: Fix builder\dispatchers.go

Signed-off-by: John Howard<EMAIL_ADDRESS>@swernli
This PR is part of the proposal described in issue 10662 to port the docker daemon to Windows. This fixes the invocation of commands on Windows to use cmd /S /C instead of sh -c on Linux, and blocks Dockerfile commands which are not supported by Windows containers - ENV; USER; VOLUME

Does it make sense to add some testscases to ensure we don't break these new features going forward?

Yes, I'm aware I'm desperately lacking on test cases right now. I do have it on my list to add test cases for changes across the board.

Once the janky issues are resolved, LGTM

LGTM


Integration test 'Should return $true when Test-DscConfiguration is run' does not fail

I recently saw that the integration tests 'Should return $true when Test-DscConfiguration is run' that is part of the integration tests template does not fail as expected.
https://github.com/PowerShell/DscResource.Template/blob/f91a42580bb86642ac389abce5dc04d3f43f080e/Tests/Integration/integration_test_template.ps1#L108-L110
The reason is that  Test-DscConfiguration returns a [System.String] and not a [System.Boolean], so this test always passes regardless of outcome.
Ii can be seen here on this line, it is reported False, but the test passes.
https://ci.appveyor.com/project/johlju/sqlserverdsc/builds/23132521/job/vkxi4xq8jqke7ahf#L1198
Verified it here too
PS > $result = Test-DscConfiguration -Verbose
VERBOSE: Perform operation 'Invoke CimMethod' with following parameters, ''methodName' = TestConfiguration,'className'
= MSFT_DSCLocalConfigurationManager,'namespaceName' = root/Microsoft/Windows/DesiredStateConfiguration'.
VERBOSE: An LCM method call arrived from computer SQLTEST with user sid S-1-5-21-2246212248-426023572-517575333-1441.
VERBOSE: [SQLTEST]: LCM:  [ Start  Test     ]
VERBOSE: [SQLTEST]: LCM:  [ Start  Resource ]  [[PSModule]InstallModule]
VERBOSE: [SQLTEST]: LCM:  [ Start  Test     ]  [[PSModule]InstallModule]
VERBOSE: [SQLTEST]:                            [[PSModule]InstallModule] Determining if the module 'posh-git' is in the
 desired state.
VERBOSE: [SQLTEST]:                            [[PSModule]InstallModule] Getting the current state of the module
'posh-git'.
VERBOSE: [SQLTEST]:                            [[PSModule]InstallModule] Begin invoking Get-Module 'posh-git'.
VERBOSE: [SQLTEST]:                            [[PSModule]InstallModule] Populating RepositorySourceLocation property
for module posh-git.
VERBOSE: [SQLTEST]:                            [[PSModule]InstallModule] Loading module from path 'C:\Program
Files\WindowsPowerShell\Modules\posh-git\0.7.3\posh-git.psm1'.
VERBOSE: [SQLTEST]:                            [[PSModule]InstallModule] Module 'posh-git 0.7.3' is found on the node.
VERBOSE: [SQLTEST]:                            [[PSModule]InstallModule] Module 'posh-git' is found on the node.
VERBOSE: [SQLTEST]:                            [[PSModule]InstallModule] Found the module path: 'C:\Program
Files\WindowsPowerShell\Modules\posh-git\0.7.3\PSGetModuleInfo.xml'.
VERBOSE: [SQLTEST]:                            [[PSModule]InstallModule] Resource 'posh-git' is in the desired state.
VERBOSE: [SQLTEST]: LCM:  [ End    Test     ]  [[PSModule]InstallModule] True in 2.6090 seconds.
VERBOSE: [SQLTEST]: LCM:  [ End    Resource ]  [[PSModule]InstallModule]
VERBOSE: [SQLTEST]: LCM:  [ End    Test     ]     Completed processing test operation. The operation returned True.
VERBOSE: [SQLTEST]: LCM:  [ End    Test     ]    in  3.2500 seconds.
VERBOSE: Operation 'Invoke CimMethod' complete.
VERBOSE: Time taken for configuration job to complete is 3.348 seconds
PS > $result
True
PS > $result.Gettype()

IsPublic IsSerial Name                                     BaseType
-------- -------- ----                                     --------

So we should either change the line
https://github.com/PowerShell/DscResource.Template/blob/f91a42580bb86642ac389abce5dc04d3f43f080e/Tests/Integration/integration_test_template.ps1#L109
to either
[System.Boolean]::Parse($result) | Should -BeTrue

or
$result | Should -Be 'True'

I prefer the former solution $result | Should -Be 'True'. I will sen in a PR with that change, and if there is a better solution I will change to that.


Restore the option to put the token in the payload if needed

Description of the change
Background
In the specific issue report, the access token is changed dynamically in the config in order to send reports to multiple Rollbar projects. An async/delayed sender is used, which doesn't have the dynamically updated config. Therefore all reports are sent using the access token in the delayed sender config.
The above scenario used to work because the token was added to the payload before queuing. Recent versions of rollbar-gem don't add the token to the payload and only add it as an http request header. (See https://github.com/rollbar/rollbar-gem/pull/966) The header is added by the delayed sender process, which uses the access token in its config.
The current interface for async senders only passes the payload and no other arguments, and the sender process doesn't have access to any other context. Even if this could be augmented in a back compatible way for the built-in senders, custom senders would still be affected by this.
Solution
An option (config.use_payload_access_token = true) is added to use the payload instead of the http header for the access token. The http header is specifically omitted in this case, to ensure the Rollbar API uses the intended token when they differ. When the config option is set, the token is sent in the payload regardless of whether the report is async or not.
This can be deprecated and removed on a future major release where an improved interface for async jobs is available.
Type of change

[x] Bug fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Related issues
Fixes ch74796
Development

[x] Lint rules pass locally
[x] The code changed/added as part of this pull request has been covered with tests
[x] All tests related to the changed code pass in development

Code review

[x]  This pull request has a descriptive title and information useful to a reviewer. There may be a screenshot or screencast attached
[x] "Ready for review" label attached to the PR and reviewers mentioned in a comment
[ ] Changes have been reviewed by at least one other engineer
[x] Issue from task tracker has a link to this pull request

@grosser Can you elaborate on how you're sending that causes you to hit this issue? It will help ensure that case is covered in the doc.

Rollbar.configure do |config|
  config.environment = ENV.fetch("RAILS_ENV")
  config.access_token = ENV.fetch("ROLLBAR_ACCESS_TOKEN")
  config.endpoint = ENV.fetch('ROLLBAR_URL') + '/api/1/item/'
  config.web_base = ENV.fetch('ROLLBAR_URL')
  config.use_thread
  config.code_version = ENV.fetch('TAG')
end


Missing period from the copy on the preference page after pinned items at top

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open Preference page on NewDot.
Check the copy under the Priority Mode.

Expected Result:
There should be a period after the full sentence.
Actual Result:
No Period.
Workaround:
Non.
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number:
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:
View all open jobs on GitHub

Triggered auto assignment to @arielgreen (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

A slack discussion where we met an agreement of adding this https://expensify.slack.com/archives/C01GTK53T8Q/p1629496662087100

Triggered auto assignment to @alex-mechler (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

Thanks for reporting this! Spun up a quick pr that adds it!

Done!

Thank you, guys.

Paid and cancelled contract. Thanks Rajat!

🚀 Deployed to production by @roryabraham in version: 1.0.88-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


Update UI

Request for new PR after failing the previous one (#132) due to merge conflicts and time restrictions.
Permission granted for time extension.
Closes #106.

Codecov Report
Base: 62.95% // Head: 62.63% // Decreases project coverage by -0.32% :warning:

Coverage data is based on head (5150ae5) compared to base (4ef783f).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #136      +/-   ##
============================================
- Coverage     62.95%   62.63%   -0.33%
  Complexity      666      666
============================================
  Files           119      119
  Lines          2346     2358      +12
  Branches        295      295
============================================
  Hits           1477     1477
- Misses          759      771      +12
  Partials        110      110

Impacted Files
Coverage Δ

src/main/java/seedu/address/MainApp.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/ui/ContactCard.java
0.00% <0.00%> (ø)

...rc/main/java/seedu/address/ui/PersonViewPanel.java
0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


feat: limit of partial sums of a unconditionally summable ℕ-indexed series

that was indeed easy (but welcome nonetheless)
maintainer merge

Thanks!
bors merge


Add not recommend using '-loader' under resolveLoader.moduleExtensions part of the document

Do you want to request a feature or report a bug?
It's a enhancement on document.
What is the current behavior?
Currently https://webpack.js.org/configuration/resolve/#resolveloader-moduleextensions has no preference on adding '-loader'.
What is the expected behavior?
Updating the https://webpack.js.org/configuration/resolve/#resolveloader-moduleextensions document to not recommend using '-loader'.
If this is a feature request, what is motivation or use case for changing the behavior?
It's not recommended to add '-loader' in https://webpack.js.org/guides/migrating/#automatic-loader-module-name-extension-removed. It'll be better to make the document consistent.

@webpack-bot move to webpack/webpack.js.org


新增設備api login登入時出現 RuntimeError

This error originated from a custom integration.
Logger: custom_components.jcihitachi_tw
Source: custom_components/jcihitachi_tw/init.py:168
Integration: JciHitachi TW (documentation, issues)
First occurred: September 16, 2022, 6:46:05 PM (2 occurrences)
Last logged: 2:34:59 AM
Failed to login API: An error occurred when refreshing support code.
HA版本如下
Home Assistant 2022.9.4
Supervisor 2022.08.6
Operating System 9.0
Frontend 20220907.2 - latest

Hi, I'm not going to make attempts for fixing this as this kind of works in general ineluctably falls prey to incomplete understanding of the architecture's full picture so it would likely be a waste of time in addressing this. And, notably, the cause of a runtime error is usually from the user's environment or API outages, and unlikely from the integration per se. Feel free to reopen if the problem persists every time and try to provide information as comprehensive as possible if you really want to have the issue addressed.


Revert #10 after Isis 1.13.1 is released with fix for ISIS-1497

Revert #10 after Isis 1.13.1 is released with fix for ISIS-1497

moved to https://github.com/incodehq/incode-platform/issues/31


Fix Image & Add Links to UG

Codecov Report

Merging #191 (4658fb2) into master (0ad4c45) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #191   +/-   ##
=========================================
  Coverage     75.06%   75.06%
  Complexity      671      671
=========================================
  Files           100      100
  Lines          2049     2049
  Branches        243      243
=========================================
  Hits           1538     1538
  Misses          410      410
  Partials        101      101

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


56 Make Responsive & Mobile Friendly!

Purpose
This gets us to the "Good Enough" stage for mobile responsiveness while we wait for any potential updates to Figma.
Things to Note
- Our Material-UI theming has been reorganized (I would like to add this to the README)
client/Theme/MainTheme.tsx contains the current information for our theme, and another theme object for Dark Mode is found in DarkMode.tsx
It uses the experimental_extendTheme API which extends the default theme of MUI, which I found to avoid repetitive code like declaring the font family in both Dark Mode and Light Mode. We can easily update things like font family and breakpoints in the MainTheme object and they'll still apply to Dark Mode without having to also define them within the DarkMode object. This was a better dev experience when considering responsive style.
More details:
MUI Theme
extendTheme
- The little boxes on the gameboard that hold our phrases are now called "Square" because I had originally named them "box" which is really confusing when also using MUI Box components for layout
To Verify
Run npm run dev to spin up the game. You can view the bingo game on your mobile device by finding the IP address of the computer and adding the port number that the app is running on (Should be 8080). It should look something like this: ipaddress:8080

@LaneEcho, before we go any further, can you rebase your branch off of dev to pull in the most recent changes?

This looks great!
Like how you setup the themes.
I think there's a couple of files that are getting a little big (App.tsx is an example), we'll want to keep an eye on it and maybe refactor when we're in there again.


feat(cli): passing additional parameters when calling the runner

Follow the new convention of protocol in vrn-deco/boilerplate#109
Affected packages

@vrn-deco/cli-command-boilerplate

Implements passing additional parameters when calling the runner

resolve #11

Codecov Report

Merging #12 (123b931) into main (097ea3f) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #12      +/-   ##
==========================================
+ Coverage   98.03%   98.04%   +0.01%
==========================================
  Files          32       32
  Lines        1985     1996      +11
  Branches      164      164
==========================================
+ Hits         1946     1957      +11
  Misses         37       37
  Partials        2        2

Impacted Files
Coverage Δ

...mmand-boilerplate/src/create/http-create.action.ts
96.12% <ø> (-0.03%)
:arrow_down:

...es/command-boilerplate/src/create/create.action.ts
100.00% <100.00%> (ø)

...s/command-boilerplate/src/create/create.command.ts
100.00% <100.00%> (ø)

...ommand-boilerplate/src/create/git-create.action.ts
100.00% <100.00%> (ø)

...nd-boilerplate/src/create/package-create.action.ts
95.61% <100.00%> (+0.46%)
: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 097ea3f...123b931. Read the comment docs.


remove panel location options in favor of new side panel behavior

This PR removes the old concept of moving the panel part in favor of a simpler model of moving view/containers into the new side panel. It "deprecates old commands" by redirecting them to the new commands to ease the transition though we know usage is already quite low of this deprecated feature.

In case anyone else stumbles upon this like I did today, here are the steps to move the terminal to the side using the new system:

open the terminal
run the command View: Focus into Side Panel
drag the "Terminal" tab into the side panel

@samestep thanks
alternatively, if you want all the traditional panels (terminal, output, debug, problems) on the side panel, you can right click the panel and choose Move Views to Side Panel

@sbatten Ahh that's what I was looking for; thanks!

Is there a way to have that side panel on the right?

@lukecotter the behavior of the side panel is that it remains opposite of the side bar. Toggling the side bar position will move the side panel as well. F1 > Toggle Side Bar Position. It is not possible to stack the side panel and side bar horizontally anymore.

That's a shame. I used to like having the panel and side bar next to each other with the code pushed all the way left.

feedback noted. if your primary use case of the panel is terminal, perhaps terminal editors are a valid solution for you

A small issue with"Move Views to Side Panel" is the ability toggle the terminal.
When the terminal is in its default location underneath the editor you can press ctrl + ` to toggle it, if you move this to the side panel that keyboard shortcut no longer works. I have noticed there is a command to "Toggle Side Panel" which I can rebind and will achieve the same result just a shame the Toggle terminal command does not work the same in the different locations.

That's a shame. I used to like having the panel and side bar next to each other with the code pushed all the way left.

So @sbatten let me get this right - you have "improved" the code by removing functionality that a lot of users (including me) were happy with? For ... reasons? Thanks 🙄
Sometimes I wish MS could just leave the thing alone, I am happy with the way it works, why constantly "enhance" it and confuse your users... we got work to do. I guess I'll now go and spend a couple of hours of my time reading up on what the shiny new "Side Panel" behaviour and "Terminal Editors" are... 😡

Also, with Terminal Editors, the workbench.action.terminal.focus which was burned into my muscle memory cannot be used anymore. Is there an equivalent for Terminal Editors?

@dpooleffdc I have just made a change to fix this behaviour. It will be available in tomorrow's insiders.

What is the purpose of the side panel ? I can't see any documentation about it. I understand one drags views into it. But I don't know what are views in this context.

@dpooleffdc I have just made a change to fix this behaviour. It will be available in tomorrow's insiders.

Thank you @sbatten that is very much appreciated!

@cognominal the side panel is a new location to drop views analogous to the side bar and panel that already exist today. It lacks documentation since it is yet to be released into stable. however that should change soon. views are the things inside of the side bar and panel, e.g. timeline view, outline view, terminal, etc.

@sbatten All this easy shuffling around of views is very nice, once one get the hang of it.
Is this going eventually in the direction of supporting different layouts for different usages or different modes within the same workspace, including possibly having closed editor groups, invisible but still there ?
Speaking of mode, I am thinking of screen casting (as opposed to live sharing) which means choosing setting screen in low resolution mode and adopting in conséquence different layouts.

@cognominal thanks for the feedback! I believe this issue would be a good place to share your thoughts and express your desire for the feature https://github.com/microsoft/vscode/issues/138263


Profile Page

This PR implements the profile page.
Signed-off-by: JohnyTheCarrot<EMAIL_ADDRESS>

Looks good to me -- but we do need a way of standardizing the style and component structure. I'll think about that for the upcoming PR. Looks like this PR has one conflict that we can resolve here in the editor -- James' last PR was merged after you forked, but that should be easy enough to solve. @krook and @andygrillo -- if you're okay with it, I can resolve that here and merge.

I took a look at the merge conflict too. I'm good with you resolving it @rdkelley. Thank you.
Once we merge all outstanding changes, we should run the prettier NPM script to get us to a single consistent format, then set our own style guidelines from there as needed.

Yes go for it, tx
On Fri, Dec 4, 2020, 4:54 PM Ryan Kelley<EMAIL_ADDRESS>wrote:

Looks good to me -- but we do need a way of standardizing the style and
component structure. I'll think about that for the upcoming PR. Looks like
this PR has one conflict that we can resolve here in the editor -- James'
last PR was merged after you forked, but that should be easy enough to
solve. @krook https://github.com/krook and @andygrillo
https://github.com/andygrillo -- if you're okay with it, I can resolve
that here and merge.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/openeew/openeew-dashboard/pull/19#issuecomment-739065498,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABP5GHLK6WIXXBNJA7GAOP3STFSBZANCNFSM4UOAGP4A
.


agent: disable update cancellation

Signed-off-by: Stephen J Day<EMAIL_ADDRESS>cc @aaronlehmann

LGTM
For future reference in case we forget: the problem we were seeing with cancellation was cancelled pulls due to a task update being received while the pull was taking place. This led to rejected tasks and funny rolling update behavior.
cc @cpuguy83

Do we actually need to cancel in the case that DesiredState changes to SHUTDOWN?

@aaronlehmann Can you explain the problem further? I don't understand.

Oh I see the linked issue... let me test.


Default permission for Minecraft.command.help

Hi, I am new to permissions so I have some questions regarding default permissions (if any) of LuckPerms.
As an example - /help command.
Btw, I also have EssentialsX installed so I call vanilla help as /minecraft:help.
And here is the question: when I have Minecraft.command.help permission undefined for default group (no other groups on the server) then members of default group are still able to execute /minecraft:help. And verbose mode also shows that other permissions have been also set for the player executing /minecraft:help command.
Like that:
[01:52:24 INFO]: [LP] VB > Steve - minecraft.command.help - true
[01:52:24 INFO]: [LP] VB > Steve - minecraft.command.advancement - undefined
[01:52:24 INFO]: [LP] VB > Steve - minecraft.command.attribute - undefined
[01:52:24 INFO]: [LP] VB > Steve - minecraft.command.ban - undefined
[01:52:24 INFO]: [LP] VB > Steve - minecraft.command.ban-ip - undefined
[01:52:24 INFO]: [LP] VB > Steve - minecraft.command.banlist - undefined
[01:52:24 INFO]: [LP] VB > Steve - minecraft.command.bossbar - undefined
[01:52:24 INFO]: [LP] VB > Steve - minecraft.command.defaultgamemode - false

As Steve is a member of default group I do some checking:
> lp group default permission check minecraft.command.help
[02:18:45 INFO]: [LP] default has permission minecraft.command.help set to undefined in context global.

> lp group default permission check minecraft.command.defaultgamemode
[02:19:20 INFO]: [LP] default has permission minecraft.command.defaultgamemode set to undefined in context global.

So why some permissions are shown as true or false in verbose mode (and those with true are available to group members of course, like minecraft.command.help) even if they aren't set (undefined) for the group.
Thanks.

It is possible that the minecraft.command.help is not directly set in default group. Perhaps you set it in another group default group inherits from or directly to the user. The best way to check is to run /lp user USER permission checkinherits minecraft.command.help

@benwoo1110 hi! it is fresh installation of paper/luckperms. So there is no other group than default one. That's why I am sure LuckPerms has some default permissions for builtin Minecraft commands.

LuckPerms does not have some default permissions for builtin Minecraft commands, your server software has those.
If you are using bukkit/spigot/paper, here are the command and permission node for these servers: https://bukkit.gamepedia.com/CraftBukkit_Commands#Bukkit_Commands_and_Permissions
From the bukkit website, it seems that the permission node is minecraft.command.help is given to everyone by default, perhaps set that permission node to false if you want to deny user access.

Thank you guys :) I've got it.


[GUI Editor] Add isVisible property on the common control properties …

…menu
Related forum issue: https://forum.babylonjs.com/t/gui-editor-text-not-displaying/41395/7
Adds a isVisible property on the common control property menu, which has the same function as the one on the tree view:

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Lets @PatrickRyanMS to try out once deployed before merging :-)

Snapshot stored with reference name:
refs/pull/13952/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13952/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/13952/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/13952/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/13952/merge
https://gui.babylonjs.com/?snapshot=refs/pull/13952/merge
https://nme.babylonjs.com/?snapshot=refs/pull/13952/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/13952/merge#BCU1XR#0


jsfiddle demo

Hi! I took the liberty of throwing together a quick jsfiddle from your code so I could see it in action. Feel free to fork it and/or let me know if you'd rather I take it down. https://jsfiddle.net/appel/15pn4b0j/

Hi,
No issue, just acknowledge this repo as the original source.
Thank you for taking the time, I will link to your fiddle in the readme and desc.


Vulnerability roundup 85: st-0.8.2: 1 advisory [6.1]

search, files

[ ] CVE-2017-16224 CVSSv3=6.1 (nixos-20.03)

Scanned versions: nixos-20.03: a84b797b28e. May contain false positives.
Cc @andsild

20.03 is Frozen and EOL.
As such, issue is closed.


quick setup clos02

Adding quick-setup script for deploying underlay connectivity for 5-stage CLOS (clos02).

Thanks for this great contribution @siva19susi


I cannot click the slider to slide the page

@getnamo
When I click on the slider of the slider page, it doesn't work. What's wrong with this

Might need a click and hold command. Can you select text in the page with the same method?

I can't achieve click and hold you said to select text, what should I set，Or what should I do?

Try https://github.com/getnamo/BLUI/blob/master/Source/Blu/Public/BluEye.h#L108 and then later https://github.com/getnamo/BLUI/blob/master/Source/Blu/Public/BluEye.h#L116

I tried TiggerLeftMouseDown，TriggerLeftMouseUp，TriggerLeftClick，TriggerMouseWheel and it didn't work when I held down the slider and it didn't work when I selected text

Hi Jan!
Thank you for the great work you did with this plugin! It helped me a lot to display pdf files and to access to protected files from APIs. It seems I have the same problem, I can't use the right slider to view all the pages of the pdf displayed in the browser. I thought Set InputModeUI would solve that like the Webbrowser frim UE, but no. Any idea?
Keep up the great work and have a nice day!

I found it by using the triggers!


Modify converter to allow setting values for object and array

What this PR does / why we need it:

This PR is associated with https://github.com/yorkie-team/yorkie-js-sdk/pull/691. Since modifications have been made to allow setting values in set and add operations, it is necessary to update the converter accordingly. This involves enabling the setting of values for objects and arrays in the converter.

PR#678 triggers the regression error, so I made corrections and re-submitted. I've added handling for cases where values are not set when converting to PbJSONElementSimple. In the bytesToObject function, It was previously creating a new object with initialTicket when the bytes were nil. However, this logic became inappropriate after separating it from bytesToSnapshot, so I removed it.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

Additional documentation:

Checklist:

[x] Added relevant tests or not required
[x] Didn't break anything

CC) @humdrum, @7hong13


Fix issue with coolingSetpoint showing heat

The standard tile shouldn't be showing anything other than an arrow, but I'll merge the change for the sake of clarity.


Create a tutorial for HerbSearch in documentation

Notebook can be found here https://github.com/Herb-AI/Herb.jl/blob/documentation/docs/src/tutorials/advanced_search.ipynb
The tutorial is live here https://herb-ai.github.io/Herb.jl/dev/tutorials/advanced_search/

PR #53

Was pushed to documentation branch on Herb.jl: https://github.com/Herb-AI/Herb.jl/commit/62aa153eb8fab120d612452217d0760959652535


Actix scenarios not running

Some scenarios have stopped running:

Scenario
Environment
Last Run

FortunesPostgreSql-Actix
Linux, Physical, http, KestrelSockets
9/16/19 1:24:04 AM +00:00

Json-Actix
Linux, Physical, http, KestrelSockets
9/16/19 1:23:13 AM +00:00

Plaintext-Actix
Linux, Physical, http, KestrelSockets
9/16/19 1:22:22 AM +00:00

Logs

Duplicate of https://github.com/aspnet/AspNetCore/issues/14495


GH-1382: Republish Recoverer Improvements

Resolves https://github.com/spring-projects/spring-amqp/issues/1382
Add expressions; make private method protected.
cherry-pick to 2.4.x

... and cherry-picked to 2.4.x


How to trigger "finished" event

My implementation of gif creator is controlled by an other process loop.
I have declared gif this way:
    var gif = new GIF({ workers: 2, quality: 10, debug: true, width: canvas.width, height: canvas.height});
      gif.on('finished', function(blob) {
        window.open(URL.createObjectURL(blob));
      });
    gif.render();

I'm processing all frames (around 300) and add them to gif:
gif.addFrame(context, {copy: true});
Then when I'm finished, I nee to have my Gif animation opened in an other window, but don't know how trigger the "finished" event.
Any help please

You should only call gif.render once you're done adding frames. Then gif.render should trigger the "finished" event.

window.open probably needs to be called during a user action such as a click though, so it isn't blocked.

Thanks!
I didn't catch that this render was to be executed only when all frames were added, since I'm working with three.js which render's must be called on every frame... ;-)
Thanks again


Attacher-Component breaks ember-metrics

Sorry but twiddle seems not to work.
What?
https://github.com/poteto/ember-metrics cannot load config if any attach-component is present.
How?

Install latest version of https://github.com/poteto/ember-metrics and ember-attacher
Configure GoogleAnalytics in environment
Add an attach-tooltip on page

Uncaught TypeError: Cannot read property 'id' of null
    at Class.init (google-analytics.js:23)
    at new Class (core_object.js:113)
    at Function._ClassMixinProps.create [as _super] (core_object.js:282)
    at Function.create (validation-decorator.js:41)
    at Function.superWrapper [as create] (ember-utils.js:437)
    at Class._activateAdapter (metrics.js:247)
    at metrics.js:174
    at Array.forEach (<anonymous>)
    at Class.activateAdapters (metrics.js:171)
    at Class.init (metrics.js:119)

This is an issue with ember-metrics. I recommend filing an issue there, but I'm happy to help/answer questions.


Unify the configuration files

Given that we now have the schema definition as a $schema attribute at the top of the configuration file, I think we can restructure this project to just have one file: platformatic.json (or yaml, toml, etc..).

@ivan-tymoshenko @cjihrig wdyt?

Sounds good to me. It would make my life simpler.

It will require updating the onestep.

This was done in #1348


New incompatibility: case class companions don’t extend Function anymore

In Scala 2, a case class companion was extending Function.
case class Foo(x: Int, b: Boolean)

// The Scala 2 compiler synthesizes the following companion
object Foo extends Function2[Int, Boolean, Foo] {
  def apply(x: Int, b: Boolean) = new Foo(x, b)
}

This is not the case anymore, which means that companions don’t inherit function methods such as apply, tupled, andThen, and compose.
One work-around to write code that cross-compiles is to manually write a function that calls the constructor:
val fooConstructor: (Int, Boolean) => Foo = (x, b) => Foo(x, b)
// Then, `fooConstructor` has all the methods of Function

As a work-around, we can also use the apply method because it can be eta-expanded to a Function2
Foo.apply.curried(1)(true) // compiles in Scala 2.13 and Scala 3.0.0-M3

What do you think @julienrf ?

Actually it doesn't compile in Scala 2.13:
[error] Unapplied methods are only converted to functions when a function type is expected.
[error] You can make this conversion explicit by writing `apply _` or `apply(_,_)` instead of `apply`.
[error]   val foo1 = Foo.apply.curried(1)(true)

But this one cross-compiles:
(Foo.apply _).curried(1)(true)


ARC-1414 Standardise ESLint rules across the app

Updates in this PR:

Standardise ESLint rules across app
Add @forge/bridge to project
Update all spaces to tabs
Remove dangling commas

nit: We should have a Jira task for formating these, mentioning which changes we're making to our lint rules. The key you're linking here has nothing to do with standardising linting rules.

[ ] Tested in stagin


⚠ Faculté génie website (prod) has degraded performance

In 42b4d74, Faculté génie website (prod) (https://www.usherbrooke.ca/genie/) experienced degraded performance:

HTTP code: 200
Response time: 1086 ms

Resolved: Faculté génie website (prod) performance has improved in 2240b9e.


Debounce decorator implemented

Implementation of the debounce decorator.

Didn't notice you removed must dependecy, so now build fails.

@dfsq updated to no longer use `must. See https://github.com/jayphelps/core-decorators.js/blob/master/src/deprecate.spec.js for reference.
Mostly looks like you just need your expects like this:
// expect(editor.counter).to.equal(0);
editor.counter.should.equal(0);

Yea, I fixed tests with chai expect.

Thanks @dfsq!


Removed WhatIf and Confirm parameter

Type

[ ] Bug Fix
[ ] New Feature
[ ] Sample
[x] Parameter description

Related Issues?
N/A
What is in this Pull Request ?
Removed parameter description as they are not available in the cmdlet. Added cmdlet description

Thank you @PowershellScripts !!


ENH: vectorize distance loops with gcc

add compiler flags necessary to enable auto-vectorization of distance
loops with gcc.
Works quite fine with recent gcc's without adding any more C code that is so frowned upon.

simpler version of gh-4379, would be interesting to test if it vectorizes with the old compilers one finds on redhat linux.

Thanks Julian. At least for me this is much more readable.

I really don't one to be that guy, and promise to not keep flogging a dead horse after this, if it is a conscious, well thought decision. But merging this PR instead of Julian's other will leave Windows users out of a 2x performance improvement. I find that troubling.

I am sure MSVC and Intel C++ have compiler pragmas we can use as well.
It also leaves out most users of Mac OSX  and FreeBSD because clang is the common C compiler. But I am not complaining.
On the bright side: It does not make it worse.

With Intel C++ (__INTEL_COMPILER is defined) we can use #pragma simd right above the loop, followed by the same syntax used by OpenMP for controlling reduction and private variables in a parallel loop (e.g. reduction(+:acc) for an accumulator and private(i) if i is the loop indexer).
https://software.intel.com/sites/products/documentation/doclib/iss/2013/compiler/cpp-lin/GUID-1EA04294-988E-4152-B584-B028FD6FAC48.htm#GUID-1EA04294-988E-4152-B584-B028FD6FAC48

Something like this?
static NPY_INLINE double
dot_product(const double *u, const double *v, npy_intp n)
{
    double s = 0.0;
    npy_intp i;

    #ifdef __INTEL_COMPILER && (__INTEL_COMPILER >= 1300)
    #pragma simd private(i) reduction(+:s)
    #endif

    for (i = 0; i < n; i++) {
        s += u[i] * v[i];
    }
    return s;
}

@jaimefrio no worries, the horse's corpse is still warm. And this kind of tradeoff is important to discuss well.
I'll just re-summarize my thoughts here: Code maintainability is very important for Scipy. Performance is important as well, but we're still more limited by maintainer time than by anything else for growth and quality improvement of the project (see the unmerged PR list and the lack of maintenance modules like ndimage and fftpack get). Hence a factor 1.5x-2x speedup that comes with a 10x increase in effort for almost all maintainers to touch a piece of code in the future is not worth it. Similar tradeoffs apply to moving code from Python to Cython (2x is usually not worth it), and it's why the order of preference for languages new code is written in is Python/Cython/C/C++/Fortran. These intrinsics are at best at a similar preference level as Fortran I'd say.

The pragma for ICC works, but it is annoying to have that above each though.
For clang I found fhat compiling with -Ofast will work, without any pragmas. That would take some woodoo in setup.py.

As for Fortran the loop will be vectorized with gfortran -O3 -ffast-math -mavx. I forgot -ffast-math in my last test, it makes all the difference in the world. But using Fortran comes with some annoyances. NumPy distutils does not recognize .f03 and .f08, so it is impossible to make a C callable library directly, without resorting to an ABI wrapper.

@rgommers The order of preference for languages new code is written in should probably be documented somewhere. It is useful for contributors to know.
My favorite language (apart from Python) is to the far right. I hope that does not make me a right-wing extremist :smile:

You needn't worry, Sturla, you are just a Real Programmer surrounded by Quiche Eaters... ;-).

I got to get me OS/370 then :smile:

I've been thinking about #pragma solutions too, but the problem is that you need different ones for different compilers (or compiler settings; think #pragma omp simd), and it's not possible to macro them out once and for all.

#pragma simd behaves like #pragma omp, so it cannot be macroed out because it depends on the contents of the loop.

Sure, but what I meant was that I would accept having to write
SIMD_PRAGMA
for (i = 0; i < n; i++) {
    /* work work work */
}

above every loop, but not
#if defined(__INTEL_COMPILER) || __GNUC__ >= 5 and HAVE_CILK_PLUS
# pragma simd
#elif defined _OPENMP
# pragma omp simd
#elif defined MSC_VER
# pragma whatever
#endif
for (i = 0; i < n; i++) {
    /* what were we doing again? */
}

Anyway, we now have a PR that passes all the tests except for a timeout in the coverage tests, and that looks like a net improvement for one of the main platforms. Is this the way we want to go?
@jaimefrio @ewmoore I appreciate your concerns, but I'd like to see timings and/or assembler output from MSVC. If it manages to vectorize this code without compiler magic, Windows users don't lose anything.

@larsmans why not use a pragma inside a macro? That can be done via the approach here:
http://stackoverflow.com/questions/3030099/pragma-in-define-macro/3030312#3030312
There is a comment that mentions a similar method for msvc.

Meh. That requires switching on the compiler to use either _Pragma or __pragma. It seems to work in GCC and Clang even in C89 mode, though.

@insertinterestingnamehere At least for Intel's pragma simd it cannot just be a globally defined macro, but takes arguments depending on the loop (in the same way as OpenMP pragmas).

Test failure is only a timeout on of the builds. I want to merge this because it's a net improvement that doesn't take too much code. Any objections?

Test failure is only a timeout on of the builds. I want to merge this because it's a net improvement that doesn't take too much code. Any objections?

The tests should be run again.  The timeouts are intermittent, and might not occur again on another run.

Closing and reopening to rerun tests.

Merging this PR sets a precedent that is likely to be followed in other C code in scipy.  Before merging, it would be great to have some definitive "thumbs up" recommendations from the experienced C developers out there.
I'm neutral on the change.  I'm hesitant about the use of the unsafe-math-optimizations flag, but that's because I don't know how "unsafe" it really is.  I suspect the differences that might occur are far below the tolerances of our unit tests.
Is it possible to override those pragmas when building scipy, without modifying the C code?

Basically they are equivalent to -ffast-math -funroll-loops as compile flags, which can be passed in  setup.py with the extra_compile_flags keyword argument to  add_extension. Having it in the C code means it cannot be changed without modifying the C code, and it will only work on GCC (not Clang and Intel C). I suggest we use  add_library to compile this file with aggressive optimizations, and then link it to the extension we build with add_extension.

The test suite should assess if the optimizations are safe. I am note sure if we need new tests, but that is where it belongs.

@sturlamolden My GCC manual states that "unsafe" math is a prerequisite for "fast" math, not the other way around. gcc -c -Q -O2 -funsafe-math-optimizations shows that unsafe math also doesn't enable loop unrolling. Unsafe math is in fact equivalent to:
-fno-signed-zeros
-fno-trapping-math
-fassociative-math
-freciprocal-math

I believe associativity of floating point operations is the main barrier to autovectorization, so we should check if that alone turns on the vectorizer (@juliantaylor?)

Ok, that might be correct, but then we can compile with those flags.

you typically need all but reciprocal flags for vectorization
can this still go into 0.16?

In cKDTree (#4374) I got a nice speedup from manually unrolling this loop. It seems to work on most compilers :)
npy_float64
sqeuclidean_distance_double(const npy_float64 *u, const npy_float64 *v, npy_intp n)
{
    npy_float64 s;
    npy_intp i;
    npy_float64 acc[4] = {0, 0, 0, 0};
    for (i = 0; i < n/4; i += 4) {
        npy_float64 _u[4] = {u[i], u[i + 1], u[i + 2], u[i + 3]};
        npy_float64 _v[4] = {v[i], v[i + 1], v[i + 2], v[i + 3]};
        npy_float64 diff[4] = {_u[0] - _v[0],
                               _u[1] - _v[1],
                               _u[2] - _v[2],
                               _u[3] - _v[3]};
        acc[0] += diff[0] * diff[0];
        acc[1] += diff[1] * diff[1];
        acc[2] += diff[2] * diff[2];
        acc[3] += diff[3] * diff[3];
    }
    s = acc[0] + acc[1] + acc[2] + acc[3];
    if (i < n) {
        for(; i<n; ++i) {
            npy_float64 d = u[i] - v[i];
            s += d * d;
        }
    }
    return s;
}

Should be OK to merge for 0.16.0 I think. Does anyone think this needs more discussion?

Okay, there's still an open question about what MSVC actually does, but this is a net win that seems maintainable and it follows a similar change in gh-4653. So merging.

Thanks @juliantaylor, all.


[Discussion] Replace Markdown parser

The markdown parser we use is now deprecated.
Ideally, we should replace it now, before having more features relying on it.
Alternatives

Marked -> No JsonML support
Showdown -> No JsonML support
Markdown-it -> No JsonML support & no plans for it

Other approaches

Use any of the previous three (Marked is the most popular), but handle the jsonML conversion ourselves. Use the JsonML format for our needs, and then convert it back to markdown, in order to feed it to any of the solutions above.

Drop jsonML. Popular static site generators like Next or Gatsby who handle markdown files, use Remark.
Has also a lot of plugins that can help us. For example link validation or auto-linking to github commits/PRS/issues

Edit: Marked uses Unified which converts markdown to Concrete (CST) or Abstract (AST) syntax trees.
Input
<a href="https://github.com/unifiedjs/unified" class="highlight">
  Fork on GitHub
</a>

Output

{
  "type": "element",
  "tagName": "a",
  "properties": {
    "href": "https://github.com/unifiedjs/unified",
    "className": ["highlight"]
  },
  "children": [
    {"type": "text", "value": "\n  Fork on GitHub\n"}
  ]
}

Github repos of

unified
remark

???

The first doesn't sound optimal but should do the trick, although the second appears to be a much better solution to me. JsonML is great as an idea but doesn't appear to have any popularity.

After some internal discussion, we can

Use any to parse markdown to HTML
Transform the HTML to JsonML
Use the JsonML representations for any of our needs

Useful libraries to check

https://www.npmjs.com/package/mark-twain
https://www.npmjs.com/package/md2jsonml
https://www.npmjs.com/package/jsonml-stringify
https://www.npmjs.com/package/html-to-jsonml

Updates

Stand-alone jsonml libraries are either unmaintained, in a poor state, or simply not following the spec.
The current unmaintained library (markdown-js) doesn't support tables. There's no way around.

Unified & Remark sound like the best options forward.

https://github.com/balena-io/landr/pull/264


afterCreate() in Sub-Object Creates Weird Error

We have a pretty denormalized schema, so we need to make a LOT of association calls to create a single object... When I get to the 3rd association, I get this error by simply defining afterCreate(): TypeError: Cannot read property 'get' of undefined in my 3rd-level factory.
Here's my example:
// STEP 1: create a customer record
factory.define('customer', models.Customer, {
    company_id: factory.assoc('company', 'id'), // NOTE: this code is skipped because it works fine
  }, {
    async afterCreate (model, attrs, buildOptions) {
      // attempting to create a user record for the customer
      const user = await factory.create('user', { customer_id: model.id })
      console.info('user:', user)
    },
  },
)

// STEP 2: create a user record
factory.define('user', models.User, {
  contact_id: factory.assoc('contact', 'id'), // create a contact
  username: factory.sequence('User.username', (n) => `Username${n}`),
})

// STEP 3: create a contact record
factory.define('contact', models.Contact, {
    title: 'Sir',
    first_name: factory.sequence('Contact.first_name', (n) => `FirstName{n}`),
    last_name: factory.sequence('Contact.last_name', (n) => `LastName{n}`),
    notes: 'No thanks',
    avatar: 'www.avatar.com/something.jpeg',
  }, {
    // defining this function causes the error
    async afterCreate (model, attrs, buildOptions) {
      // TODO: define even more sub-models...
      return Promise.resolve()
    },
  },
)

We're using Sequelize as our ORM as well. I tried Googling for answers, but couldn't find any solutions.

for some reasons in case when factory-girl build schema doesn't match DB-schema,  FG doesn't throw error. It just returns undefined. So guys who still uses this project : make sure your build schema fits to DB schema.


📓 Updated HUN

With most used and referred phrases

@KristofKekesi Please update the XLF files, you can check out the wiki for information on how to do this https://github.com/files-community/Files/wiki/Translating-Files-into-your-language.

 Sorry for that. I will update, thanks for your reply.

I have made the changes  at .resw as well.


Fix extra -dev in conda binstar package name

Hm, conda is calling this file pymbar-dev-dev:
vagrant@precise64:~$ conda install -c https://conda.binstar.org/omnia pymbar-dev
Fetching package metadata: .....
Solving package specifications: .
Package plan for installation in environment /home/vagrant/anaconda:

The following packages will be downloaded:

    package                    |            build
    ---------------------------|-----------------
    conda-3.5.5                |           py27_0         134 KB
    cython-0.20.2              |           py27_0         4.8 MB
    openssl-1.0.1h             |                0         2.1 MB
    pymbar-dev-dev             |       np18py27_0         116 KB
    python-2.7.8               |                0        11.3 MB
    ------------------------------------------------------------
                                           Total:        18.5 MB

Maybe we should rename the package on binstar to pymbar, and just have the release version be called dev?

AFAIK this will be fixed when #163 gets merged into master.