Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update-Set-MpPreference-4519186 #2158

Merged
merged 6 commits into from
Oct 12, 2020
Merged

Conversation

Lovina-Saldanha
Copy link
Contributor

Updated information for the parameter "DisableCpuThrottleOnIdleScans"

Updated information for the parameter "DisableCpuThrottleOnIdleScans"
@opbld32
Copy link

opbld32 commented Oct 1, 2020

Docs Build status updates of commit 6142c5e:

✅ Validation status: passed

File Status Preview URL Details
docset/windows/defender/Set-MpPreference.md ✅Succeeded

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:

Minor sentence formation changes.
@opbld32
Copy link

opbld32 commented Oct 1, 2020

Docs Build status updates of commit 32e5150:

✅ Validation status: passed

File Status Preview URL Details
docset/windows/defender/Set-MpPreference.md ✅Succeeded

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:

Fixed comments from Asha
@opbld30
Copy link

opbld30 commented Oct 1, 2020

Docs Build status updates of commit ed7341a:

✅ Validation status: passed

File Status Preview URL Details
docset/windows/defender/Set-MpPreference.md ✅Succeeded

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:

@o0nj
Copy link
Contributor

o0nj commented Oct 3, 2020

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

@o0nj
Copy link
Contributor

o0nj commented Oct 3, 2020

@dariomws @get-itips @tseward

Peer review needed for this PR. Thanks.

@@ -245,6 +245,18 @@ Accept pipeline input: False
Accept wildcard characters: False
```

### -DisableCpuThrottleOnIdleScans
Indicates whether the CPU will be throttled for scheduled scans while the device is idle. This parameter is enabled by default, thus ensuring that the CPU will not throttle for scheduled scans performed when the device is idle, regardless of what **ScanAvgCPULoadFactor** is set to. For all other scheduled scans, this flag does not have any impact and normal throttling will occur.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Indicates whether the CPU will be throttled for scheduled scans while the device is idle. This parameter is enabled by default, thus ensuring that the CPU will not throttle for scheduled scans performed when the device is idle, regardless of what **ScanAvgCPULoadFactor** is set to. For all other scheduled scans, this flag does not have any impact and normal throttling will occur.
Indicates whether the CPU will be throttled for scheduled scans while the device is idle. This parameter is enabled by default, thus ensuring that the CPU will not be throttled for scheduled scans performed when the device is idle, regardless of what **ScanAvgCPULoadFactor** is set to. For all other scheduled scans, this flag does not have any impact and normal throttling will occur.

Copy link
Contributor

@JohanFreelancer9 JohanFreelancer9 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy review.

Added "Boolean" per comment from Gabriel Bercea  ...   [- DisableCpuThrottleOnIdleScans <Boolean>]
@opbld31
Copy link

opbld31 commented Oct 5, 2020

Docs Build status updates of commit 209cd25:

✅ Validation status: passed

File Status Preview URL Details
docset/windows/defender/Set-MpPreference.md ✅Succeeded

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:

@opbld31
Copy link

opbld31 commented Oct 5, 2020

Docs Build status updates of commit de70739:

✅ Validation status: passed

File Status Preview URL Details
docset/windows/defender/Set-MpPreference.md ✅Succeeded

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:

@opbld33
Copy link

opbld33 commented Oct 5, 2020

Docs Build status updates of commit c29abbf:

✅ Validation status: passed

File Status Preview URL Details
docset/windows/defender/Set-MpPreference.md ✅Succeeded

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:

@o0nj o0nj added the Sign off The pull request is ready to be reviewed and merged by PubOps label Oct 12, 2020
@tiburd tiburd self-assigned this Oct 12, 2020
@tiburd tiburd added the In review PubOps is reviewing the pull request label Oct 12, 2020
@tiburd tiburd merged commit 6441906 into MicrosoftDocs:master Oct 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
In review PubOps is reviewing the pull request Microsoft submitter Sign off The pull request is ready to be reviewed and merged by PubOps
Projects
None yet
Development

Successfully merging this pull request may close these issues.