add boolean parsing variations - #2234

Merged
danmoseley merged 4 commits into
dotnet:mainfrom
danmoseley:bool.parse
Feb 6, 2022
Merged

add boolean parsing variations#2234
danmoseley merged 4 commits into
dotnet:mainfrom
danmoseley:bool.parse

Conversation

@danmoseley

@danmoseleydanmoseley commented Feb 4, 2022

Copy link
Copy Markdown
Contributor

What I get on my machine, without dotnet/runtime#64782

MethodvalueMeanErrorStdDevMedianMinMaxAllocated
TryParsetrue9.877 ns0.3212 ns0.3700 ns9.807 ns9.161 ns10.60 ns-
TryParsefalse11.404 ns0.4224 ns0.4865 ns11.344 ns10.722 ns12.22 ns-
TryParsefAlSe11.179 ns0.2641 ns0.2936 ns11.167 ns10.733 ns11.84 ns-
TryParseTrUe12.215 ns1.7123 ns1.9032 ns11.467 ns10.127 ns15.41 ns-
TryParsetru23.106 ns0.6177 ns0.6866 ns22.909 ns22.116 ns24.61 ns-
TryParsetRuu25.520 ns0.5347 ns0.5722 ns25.542 ns24.739 ns26.64 ns-
TryParsefalss26.871 ns0.5399 ns0.5302 ns27.117 ns25.552 ns27.44 ns-
TryParsebogus34.17 ns6.021 ns6.934 ns30.54 ns27.177 ns48.57 ns-
Parsetrue16.222 ns0.7374 ns0.8492 ns16.208 ns14.751 ns18.18 ns-
Parsefalse17.495 ns1.1390 ns1.3117 ns16.919 ns16.059 ns20.40 ns-

@danmoseley

Copy link
Copy Markdown
ContributorAuthor

@DrewScoggins@LoopedBard3Helix API does not contain an entry for Windows.10.Amd64.ClientRS4.DevEx.15.8.Open

@LoopedBard3

Copy link
Copy Markdown
Member

Looks like the ClientRS4 queue is EOL and the likely upgrade is to ClientRS5. I will do a test run to see if using the new queue fixes the issue.

@LoopedBard3

Copy link
Copy Markdown
Member

Testing in PR #2237, will update once the run completes.

@LoopedBard3

Copy link
Copy Markdown
Member

Closing for rebase since #2237 has been merged.

@LoopedBard3

Copy link
Copy Markdown
Member

It appears reopening did not catch the latest queue update.

@danmoseleydanmoseley reopened this Feb 5, 2022
@danmoseley

Copy link
Copy Markdown
ContributorAuthor

performance-ci (windows RS4 x64 micro_net461 Open LTS) results can be ignored. it's a dead queue

@danmoseley

Copy link
Copy Markdown
ContributorAuthor

@LoopedBard3 perhaps you'd like to review so I can merge?

Comment threadsrc/benchmarks/micro/libraries/System.Runtime/Perf.Boolean.cs Outdated
Comment threadsrc/benchmarks/micro/libraries/System.Runtime/Perf.Boolean.cs Outdated
@danmoseleydanmoseley reopened this Feb 5, 2022
@danmoseley

Copy link
Copy Markdown
ContributorAuthor

Now

MethodvalueMeanErrorStdDevMedianMinMaxAllocated
TryParsetrue10.252 ns0.2067 ns0.1832 ns10.211 ns10.011 ns10.583 ns-
TryParsefalse11.393 ns0.5187 ns0.5974 ns11.515 ns10.246 ns12.289 ns-
TryParseTRUE13.440 ns2.3827 ns2.7440 ns12.408 ns10.431 ns19.580 ns-
TryParseFalse13.902 ns2.3719 ns2.6363 ns12.672 ns11.074 ns20.427 ns-
TryParse019.966 ns0.7141 ns0.8223 ns20.130 ns18.458 ns21.506 ns-
TryParseBogus24.709 ns2.0741 ns2.3053 ns23.893 ns22.295 ns29.574 ns-
Parsetrue16.790 ns1.1961 ns1.3774 ns16.326 ns15.212 ns19.532 ns-
ParseTRUE18.818 ns1.4715 ns1.5745 ns18.369 ns16.956 ns22.533 ns-
Parsefalse19.133 ns0.7491 ns0.8016 ns19.191 ns17.416 ns21.096 ns-
ParseFalse19.211 ns1.8161 ns2.0915 ns19.938 ns15.663 ns22.658 ns-
ToStringTrue2.225 ns0.2246 ns0.2206 ns2.210 ns1.849 ns2.786 ns-
ToStringFalse3.220 ns0.2709 ns0.3119 ns3.219 ns2.742 ns3.886 ns-

@danmoseleydanmoseley reopened this Feb 5, 2022
Comment threadsrc/benchmarks/micro/libraries/System.Runtime/Perf.Boolean.cs Outdated
@danmoseley
danmoseley merged commit c40d303 into dotnet:mainFeb 6, 2022
@danmoseley
danmoseley deleted the bool.parse branch February 6, 2022 03:32
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@danmoseley@LoopedBard3@stephentoub
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
Skip to content

add boolean parsing variations - #2234

Merged
danmoseley merged 4 commits into
dotnet:mainfrom
danmoseley:bool.parse
Feb 6, 2022
Merged

add boolean parsing variations#2234
danmoseley merged 4 commits into
dotnet:mainfrom
danmoseley:bool.parse

Conversation

@danmoseley

@danmoseleydanmoseley commented Feb 4, 2022

Copy link
Copy Markdown
Contributor

What I get on my machine, without dotnet/runtime#64782

MethodvalueMeanErrorStdDevMedianMinMaxAllocated
TryParsetrue9.877 ns0.3212 ns0.3700 ns9.807 ns9.161 ns10.60 ns-
TryParsefalse11.404 ns0.4224 ns0.4865 ns11.344 ns10.722 ns12.22 ns-
TryParsefAlSe11.179 ns0.2641 ns0.2936 ns11.167 ns10.733 ns11.84 ns-
TryParseTrUe12.215 ns1.7123 ns1.9032 ns11.467 ns10.127 ns15.41 ns-
TryParsetru23.106 ns0.6177 ns0.6866 ns22.909 ns22.116 ns24.61 ns-
TryParsetRuu25.520 ns0.5347 ns0.5722 ns25.542 ns24.739 ns26.64 ns-
TryParsefalss26.871 ns0.5399 ns0.5302 ns27.117 ns25.552 ns27.44 ns-
TryParsebogus34.17 ns6.021 ns6.934 ns30.54 ns27.177 ns48.57 ns-
Parsetrue16.222 ns0.7374 ns0.8492 ns16.208 ns14.751 ns18.18 ns-
Parsefalse17.495 ns1.1390 ns1.3117 ns16.919 ns16.059 ns20.40 ns-

@danmoseley

Copy link
Copy Markdown
ContributorAuthor

@DrewScoggins@LoopedBard3Helix API does not contain an entry for Windows.10.Amd64.ClientRS4.DevEx.15.8.Open

@LoopedBard3

Copy link
Copy Markdown
Member

Looks like the ClientRS4 queue is EOL and the likely upgrade is to ClientRS5. I will do a test run to see if using the new queue fixes the issue.

@LoopedBard3

Copy link
Copy Markdown
Member

Testing in PR #2237, will update once the run completes.

@LoopedBard3

Copy link
Copy Markdown
Member

Closing for rebase since #2237 has been merged.

@LoopedBard3

Copy link
Copy Markdown
Member

It appears reopening did not catch the latest queue update.

@danmoseleydanmoseley reopened this Feb 5, 2022
@danmoseley

Copy link
Copy Markdown
ContributorAuthor

performance-ci (windows RS4 x64 micro_net461 Open LTS) results can be ignored. it's a dead queue

@danmoseley

Copy link
Copy Markdown
ContributorAuthor

@LoopedBard3 perhaps you'd like to review so I can merge?

Comment threadsrc/benchmarks/micro/libraries/System.Runtime/Perf.Boolean.cs Outdated
Comment threadsrc/benchmarks/micro/libraries/System.Runtime/Perf.Boolean.cs Outdated
@danmoseleydanmoseley reopened this Feb 5, 2022
@danmoseley

Copy link
Copy Markdown
ContributorAuthor

Now

MethodvalueMeanErrorStdDevMedianMinMaxAllocated
TryParsetrue10.252 ns0.2067 ns0.1832 ns10.211 ns10.011 ns10.583 ns-
TryParsefalse11.393 ns0.5187 ns0.5974 ns11.515 ns10.246 ns12.289 ns-
TryParseTRUE13.440 ns2.3827 ns2.7440 ns12.408 ns10.431 ns19.580 ns-
TryParseFalse13.902 ns2.3719 ns2.6363 ns12.672 ns11.074 ns20.427 ns-
TryParse019.966 ns0.7141 ns0.8223 ns20.130 ns18.458 ns21.506 ns-
TryParseBogus24.709 ns2.0741 ns2.3053 ns23.893 ns22.295 ns29.574 ns-
Parsetrue16.790 ns1.1961 ns1.3774 ns16.326 ns15.212 ns19.532 ns-
ParseTRUE18.818 ns1.4715 ns1.5745 ns18.369 ns16.956 ns22.533 ns-
Parsefalse19.133 ns0.7491 ns0.8016 ns19.191 ns17.416 ns21.096 ns-
ParseFalse19.211 ns1.8161 ns2.0915 ns19.938 ns15.663 ns22.658 ns-
ToStringTrue2.225 ns0.2246 ns0.2206 ns2.210 ns1.849 ns2.786 ns-
ToStringFalse3.220 ns0.2709 ns0.3119 ns3.219 ns2.742 ns3.886 ns-

@danmoseleydanmoseley reopened this Feb 5, 2022
Comment threadsrc/benchmarks/micro/libraries/System.Runtime/Perf.Boolean.cs Outdated
@danmoseley
danmoseley merged commit c40d303 into dotnet:mainFeb 6, 2022
@danmoseley
danmoseley deleted the bool.parse branch February 6, 2022 03:32
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@danmoseley@LoopedBard3@stephentoub
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix"); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

add boolean parsing variations - #2234

Merged
danmoseley merged 4 commits into
dotnet:mainfrom
danmoseley:bool.parse
Feb 6, 2022
Merged

add boolean parsing variations#2234
danmoseley merged 4 commits into
dotnet:mainfrom
danmoseley:bool.parse

Conversation

@danmoseley

@danmoseleydanmoseley commented Feb 4, 2022

Copy link
Copy Markdown
Contributor

What I get on my machine, without dotnet/runtime#64782

MethodvalueMeanErrorStdDevMedianMinMaxAllocated
TryParsetrue9.877 ns0.3212 ns0.3700 ns9.807 ns9.161 ns10.60 ns-
TryParsefalse11.404 ns0.4224 ns0.4865 ns11.344 ns10.722 ns12.22 ns-
TryParsefAlSe11.179 ns0.2641 ns0.2936 ns11.167 ns10.733 ns11.84 ns-
TryParseTrUe12.215 ns1.7123 ns1.9032 ns11.467 ns10.127 ns15.41 ns-
TryParsetru23.106 ns0.6177 ns0.6866 ns22.909 ns22.116 ns24.61 ns-
TryParsetRuu25.520 ns0.5347 ns0.5722 ns25.542 ns24.739 ns26.64 ns-
TryParsefalss26.871 ns0.5399 ns0.5302 ns27.117 ns25.552 ns27.44 ns-
TryParsebogus34.17 ns6.021 ns6.934 ns30.54 ns27.177 ns48.57 ns-
Parsetrue16.222 ns0.7374 ns0.8492 ns16.208 ns14.751 ns18.18 ns-
Parsefalse17.495 ns1.1390 ns1.3117 ns16.919 ns16.059 ns20.40 ns-

@danmoseley

Copy link
Copy Markdown
ContributorAuthor

@DrewScoggins@LoopedBard3Helix API does not contain an entry for Windows.10.Amd64.ClientRS4.DevEx.15.8.Open

@LoopedBard3

Copy link
Copy Markdown
Member

Looks like the ClientRS4 queue is EOL and the likely upgrade is to ClientRS5. I will do a test run to see if using the new queue fixes the issue.

@LoopedBard3

Copy link
Copy Markdown
Member

Testing in PR #2237, will update once the run completes.

@LoopedBard3

Copy link
Copy Markdown
Member

Closing for rebase since #2237 has been merged.

@LoopedBard3

Copy link
Copy Markdown
Member

It appears reopening did not catch the latest queue update.

@danmoseleydanmoseley reopened this Feb 5, 2022
@danmoseley

Copy link
Copy Markdown
ContributorAuthor

performance-ci (windows RS4 x64 micro_net461 Open LTS) results can be ignored. it's a dead queue

@danmoseley

Copy link
Copy Markdown
ContributorAuthor

@LoopedBard3 perhaps you'd like to review so I can merge?

Comment threadsrc/benchmarks/micro/libraries/System.Runtime/Perf.Boolean.cs Outdated
Comment threadsrc/benchmarks/micro/libraries/System.Runtime/Perf.Boolean.cs Outdated
@danmoseleydanmoseley reopened this Feb 5, 2022
@danmoseley

Copy link
Copy Markdown
ContributorAuthor

Now

MethodvalueMeanErrorStdDevMedianMinMaxAllocated
TryParsetrue10.252 ns0.2067 ns0.1832 ns10.211 ns10.011 ns10.583 ns-
TryParsefalse11.393 ns0.5187 ns0.5974 ns11.515 ns10.246 ns12.289 ns-
TryParseTRUE13.440 ns2.3827 ns2.7440 ns12.408 ns10.431 ns19.580 ns-
TryParseFalse13.902 ns2.3719 ns2.6363 ns12.672 ns11.074 ns20.427 ns-
TryParse019.966 ns0.7141 ns0.8223 ns20.130 ns18.458 ns21.506 ns-
TryParseBogus24.709 ns2.0741 ns2.3053 ns23.893 ns22.295 ns29.574 ns-
Parsetrue16.790 ns1.1961 ns1.3774 ns16.326 ns15.212 ns19.532 ns-
ParseTRUE18.818 ns1.4715 ns1.5745 ns18.369 ns16.956 ns22.533 ns-
Parsefalse19.133 ns0.7491 ns0.8016 ns19.191 ns17.416 ns21.096 ns-
ParseFalse19.211 ns1.8161 ns2.0915 ns19.938 ns15.663 ns22.658 ns-
ToStringTrue2.225 ns0.2246 ns0.2206 ns2.210 ns1.849 ns2.786 ns-
ToStringFalse3.220 ns0.2709 ns0.3119 ns3.219 ns2.742 ns3.886 ns-

@danmoseleydanmoseley reopened this Feb 5, 2022
Comment threadsrc/benchmarks/micro/libraries/System.Runtime/Perf.Boolean.cs Outdated
@danmoseley
danmoseley merged commit c40d303 into dotnet:mainFeb 6, 2022
@danmoseley
danmoseley deleted the bool.parse branch February 6, 2022 03:32
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@danmoseley@LoopedBard3@stephentoub
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms"); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

add boolean parsing variations - #2234

Merged
danmoseley merged 4 commits into
dotnet:mainfrom
danmoseley:bool.parse
Feb 6, 2022
Merged

add boolean parsing variations#2234
danmoseley merged 4 commits into
dotnet:mainfrom
danmoseley:bool.parse

Conversation

@danmoseley

@danmoseleydanmoseley commented Feb 4, 2022

Copy link
Copy Markdown
Contributor

What I get on my machine, without dotnet/runtime#64782

MethodvalueMeanErrorStdDevMedianMinMaxAllocated
TryParsetrue9.877 ns0.3212 ns0.3700 ns9.807 ns9.161 ns10.60 ns-
TryParsefalse11.404 ns0.4224 ns0.4865 ns11.344 ns10.722 ns12.22 ns-
TryParsefAlSe11.179 ns0.2641 ns0.2936 ns11.167 ns10.733 ns11.84 ns-
TryParseTrUe12.215 ns1.7123 ns1.9032 ns11.467 ns10.127 ns15.41 ns-
TryParsetru23.106 ns0.6177 ns0.6866 ns22.909 ns22.116 ns24.61 ns-
TryParsetRuu25.520 ns0.5347 ns0.5722 ns25.542 ns24.739 ns26.64 ns-
TryParsefalss26.871 ns0.5399 ns0.5302 ns27.117 ns25.552 ns27.44 ns-
TryParsebogus34.17 ns6.021 ns6.934 ns30.54 ns27.177 ns48.57 ns-
Parsetrue16.222 ns0.7374 ns0.8492 ns16.208 ns14.751 ns18.18 ns-
Parsefalse17.495 ns1.1390 ns1.3117 ns16.919 ns16.059 ns20.40 ns-

@danmoseley

Copy link
Copy Markdown
ContributorAuthor

@DrewScoggins@LoopedBard3Helix API does not contain an entry for Windows.10.Amd64.ClientRS4.DevEx.15.8.Open

@LoopedBard3

Copy link
Copy Markdown
Member

Looks like the ClientRS4 queue is EOL and the likely upgrade is to ClientRS5. I will do a test run to see if using the new queue fixes the issue.

@LoopedBard3

Copy link
Copy Markdown
Member

Testing in PR #2237, will update once the run completes.

@LoopedBard3

Copy link
Copy Markdown
Member

Closing for rebase since #2237 has been merged.

@LoopedBard3

Copy link
Copy Markdown
Member

It appears reopening did not catch the latest queue update.

@danmoseleydanmoseley reopened this Feb 5, 2022
@danmoseley

Copy link
Copy Markdown
ContributorAuthor

performance-ci (windows RS4 x64 micro_net461 Open LTS) results can be ignored. it's a dead queue

@danmoseley

Copy link
Copy Markdown
ContributorAuthor

@LoopedBard3 perhaps you'd like to review so I can merge?

Comment threadsrc/benchmarks/micro/libraries/System.Runtime/Perf.Boolean.cs Outdated
Comment threadsrc/benchmarks/micro/libraries/System.Runtime/Perf.Boolean.cs Outdated
@danmoseleydanmoseley reopened this Feb 5, 2022
@danmoseley

Copy link
Copy Markdown
ContributorAuthor

Now

MethodvalueMeanErrorStdDevMedianMinMaxAllocated
TryParsetrue10.252 ns0.2067 ns0.1832 ns10.211 ns10.011 ns10.583 ns-
TryParsefalse11.393 ns0.5187 ns0.5974 ns11.515 ns10.246 ns12.289 ns-
TryParseTRUE13.440 ns2.3827 ns2.7440 ns12.408 ns10.431 ns19.580 ns-
TryParseFalse13.902 ns2.3719 ns2.6363 ns12.672 ns11.074 ns20.427 ns-
TryParse019.966 ns0.7141 ns0.8223 ns20.130 ns18.458 ns21.506 ns-
TryParseBogus24.709 ns2.0741 ns2.3053 ns23.893 ns22.295 ns29.574 ns-
Parsetrue16.790 ns1.1961 ns1.3774 ns16.326 ns15.212 ns19.532 ns-
ParseTRUE18.818 ns1.4715 ns1.5745 ns18.369 ns16.956 ns22.533 ns-
Parsefalse19.133 ns0.7491 ns0.8016 ns19.191 ns17.416 ns21.096 ns-
ParseFalse19.211 ns1.8161 ns2.0915 ns19.938 ns15.663 ns22.658 ns-
ToStringTrue2.225 ns0.2246 ns0.2206 ns2.210 ns1.849 ns2.786 ns-
ToStringFalse3.220 ns0.2709 ns0.3119 ns3.219 ns2.742 ns3.886 ns-

@danmoseleydanmoseley reopened this Feb 5, 2022
Comment threadsrc/benchmarks/micro/libraries/System.Runtime/Perf.Boolean.cs Outdated
@danmoseley
danmoseley merged commit c40d303 into dotnet:mainFeb 6, 2022
@danmoseley
danmoseley deleted the bool.parse branch February 6, 2022 03:32
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@danmoseley@LoopedBard3@stephentoub
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links"); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + '
Skip to content

add boolean parsing variations - #2234

Merged
danmoseley merged 4 commits into
dotnet:mainfrom
danmoseley:bool.parse
Feb 6, 2022
Merged

add boolean parsing variations#2234
danmoseley merged 4 commits into
dotnet:mainfrom
danmoseley:bool.parse

Conversation

@danmoseley

@danmoseleydanmoseley commented Feb 4, 2022

Copy link
Copy Markdown
Contributor

What I get on my machine, without dotnet/runtime#64782

MethodvalueMeanErrorStdDevMedianMinMaxAllocated
TryParsetrue9.877 ns0.3212 ns0.3700 ns9.807 ns9.161 ns10.60 ns-
TryParsefalse11.404 ns0.4224 ns0.4865 ns11.344 ns10.722 ns12.22 ns-
TryParsefAlSe11.179 ns0.2641 ns0.2936 ns11.167 ns10.733 ns11.84 ns-
TryParseTrUe12.215 ns1.7123 ns1.9032 ns11.467 ns10.127 ns15.41 ns-
TryParsetru23.106 ns0.6177 ns0.6866 ns22.909 ns22.116 ns24.61 ns-
TryParsetRuu25.520 ns0.5347 ns0.5722 ns25.542 ns24.739 ns26.64 ns-
TryParsefalss26.871 ns0.5399 ns0.5302 ns27.117 ns25.552 ns27.44 ns-
TryParsebogus34.17 ns6.021 ns6.934 ns30.54 ns27.177 ns48.57 ns-
Parsetrue16.222 ns0.7374 ns0.8492 ns16.208 ns14.751 ns18.18 ns-
Parsefalse17.495 ns1.1390 ns1.3117 ns16.919 ns16.059 ns20.40 ns-

@danmoseley

Copy link
Copy Markdown
ContributorAuthor

@DrewScoggins@LoopedBard3Helix API does not contain an entry for Windows.10.Amd64.ClientRS4.DevEx.15.8.Open

@LoopedBard3

Copy link
Copy Markdown
Member

Looks like the ClientRS4 queue is EOL and the likely upgrade is to ClientRS5. I will do a test run to see if using the new queue fixes the issue.

@LoopedBard3

Copy link
Copy Markdown
Member

Testing in PR #2237, will update once the run completes.

@LoopedBard3

Copy link
Copy Markdown
Member

Closing for rebase since #2237 has been merged.

@LoopedBard3

Copy link
Copy Markdown
Member

It appears reopening did not catch the latest queue update.

@danmoseleydanmoseley reopened this Feb 5, 2022
@danmoseley

Copy link
Copy Markdown
ContributorAuthor

performance-ci (windows RS4 x64 micro_net461 Open LTS) results can be ignored. it's a dead queue

@danmoseley

Copy link
Copy Markdown
ContributorAuthor

@LoopedBard3 perhaps you'd like to review so I can merge?

Comment threadsrc/benchmarks/micro/libraries/System.Runtime/Perf.Boolean.cs Outdated
Comment threadsrc/benchmarks/micro/libraries/System.Runtime/Perf.Boolean.cs Outdated
@danmoseleydanmoseley reopened this Feb 5, 2022
@danmoseley

Copy link
Copy Markdown
ContributorAuthor

Now

MethodvalueMeanErrorStdDevMedianMinMaxAllocated
TryParsetrue10.252 ns0.2067 ns0.1832 ns10.211 ns10.011 ns10.583 ns-
TryParsefalse11.393 ns0.5187 ns0.5974 ns11.515 ns10.246 ns12.289 ns-
TryParseTRUE13.440 ns2.3827 ns2.7440 ns12.408 ns10.431 ns19.580 ns-
TryParseFalse13.902 ns2.3719 ns2.6363 ns12.672 ns11.074 ns20.427 ns-
TryParse019.966 ns0.7141 ns0.8223 ns20.130 ns18.458 ns21.506 ns-
TryParseBogus24.709 ns2.0741 ns2.3053 ns23.893 ns22.295 ns29.574 ns-
Parsetrue16.790 ns1.1961 ns1.3774 ns16.326 ns15.212 ns19.532 ns-
ParseTRUE18.818 ns1.4715 ns1.5745 ns18.369 ns16.956 ns22.533 ns-
Parsefalse19.133 ns0.7491 ns0.8016 ns19.191 ns17.416 ns21.096 ns-
ParseFalse19.211 ns1.8161 ns2.0915 ns19.938 ns15.663 ns22.658 ns-
ToStringTrue2.225 ns0.2246 ns0.2206 ns2.210 ns1.849 ns2.786 ns-
ToStringFalse3.220 ns0.2709 ns0.3119 ns3.219 ns2.742 ns3.886 ns-

@danmoseleydanmoseley reopened this Feb 5, 2022
Comment threadsrc/benchmarks/micro/libraries/System.Runtime/Perf.Boolean.cs Outdated
@danmoseley
danmoseley merged commit c40d303 into dotnet:mainFeb 6, 2022
@danmoseley
danmoseley deleted the bool.parse branch February 6, 2022 03:32
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@danmoseley@LoopedBard3@stephentoub
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default"); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

add boolean parsing variations - #2234

Merged
danmoseley merged 4 commits into
dotnet:mainfrom
danmoseley:bool.parse
Feb 6, 2022
Merged

add boolean parsing variations#2234
danmoseley merged 4 commits into
dotnet:mainfrom
danmoseley:bool.parse

Conversation

@danmoseley

@danmoseleydanmoseley commented Feb 4, 2022

Copy link
Copy Markdown
Contributor

What I get on my machine, without dotnet/runtime#64782

MethodvalueMeanErrorStdDevMedianMinMaxAllocated
TryParsetrue9.877 ns0.3212 ns0.3700 ns9.807 ns9.161 ns10.60 ns-
TryParsefalse11.404 ns0.4224 ns0.4865 ns11.344 ns10.722 ns12.22 ns-
TryParsefAlSe11.179 ns0.2641 ns0.2936 ns11.167 ns10.733 ns11.84 ns-
TryParseTrUe12.215 ns1.7123 ns1.9032 ns11.467 ns10.127 ns15.41 ns-
TryParsetru23.106 ns0.6177 ns0.6866 ns22.909 ns22.116 ns24.61 ns-
TryParsetRuu25.520 ns0.5347 ns0.5722 ns25.542 ns24.739 ns26.64 ns-
TryParsefalss26.871 ns0.5399 ns0.5302 ns27.117 ns25.552 ns27.44 ns-
TryParsebogus34.17 ns6.021 ns6.934 ns30.54 ns27.177 ns48.57 ns-
Parsetrue16.222 ns0.7374 ns0.8492 ns16.208 ns14.751 ns18.18 ns-
Parsefalse17.495 ns1.1390 ns1.3117 ns16.919 ns16.059 ns20.40 ns-

@danmoseley

Copy link
Copy Markdown
ContributorAuthor

@DrewScoggins@LoopedBard3Helix API does not contain an entry for Windows.10.Amd64.ClientRS4.DevEx.15.8.Open

@LoopedBard3

Copy link
Copy Markdown
Member

Looks like the ClientRS4 queue is EOL and the likely upgrade is to ClientRS5. I will do a test run to see if using the new queue fixes the issue.

@LoopedBard3

Copy link
Copy Markdown
Member

Testing in PR #2237, will update once the run completes.

@LoopedBard3

Copy link
Copy Markdown
Member

Closing for rebase since #2237 has been merged.

@LoopedBard3

Copy link
Copy Markdown
Member

It appears reopening did not catch the latest queue update.

@danmoseleydanmoseley reopened this Feb 5, 2022
@danmoseley

Copy link
Copy Markdown
ContributorAuthor

performance-ci (windows RS4 x64 micro_net461 Open LTS) results can be ignored. it's a dead queue

@danmoseley

Copy link
Copy Markdown
ContributorAuthor

@LoopedBard3 perhaps you'd like to review so I can merge?

Comment threadsrc/benchmarks/micro/libraries/System.Runtime/Perf.Boolean.cs Outdated
Comment threadsrc/benchmarks/micro/libraries/System.Runtime/Perf.Boolean.cs Outdated
@danmoseleydanmoseley reopened this Feb 5, 2022
@danmoseley

Copy link
Copy Markdown
ContributorAuthor

Now

MethodvalueMeanErrorStdDevMedianMinMaxAllocated
TryParsetrue10.252 ns0.2067 ns0.1832 ns10.211 ns10.011 ns10.583 ns-
TryParsefalse11.393 ns0.5187 ns0.5974 ns11.515 ns10.246 ns12.289 ns-
TryParseTRUE13.440 ns2.3827 ns2.7440 ns12.408 ns10.431 ns19.580 ns-
TryParseFalse13.902 ns2.3719 ns2.6363 ns12.672 ns11.074 ns20.427 ns-
TryParse019.966 ns0.7141 ns0.8223 ns20.130 ns18.458 ns21.506 ns-
TryParseBogus24.709 ns2.0741 ns2.3053 ns23.893 ns22.295 ns29.574 ns-
Parsetrue16.790 ns1.1961 ns1.3774 ns16.326 ns15.212 ns19.532 ns-
ParseTRUE18.818 ns1.4715 ns1.5745 ns18.369 ns16.956 ns22.533 ns-
Parsefalse19.133 ns0.7491 ns0.8016 ns19.191 ns17.416 ns21.096 ns-
ParseFalse19.211 ns1.8161 ns2.0915 ns19.938 ns15.663 ns22.658 ns-
ToStringTrue2.225 ns0.2246 ns0.2206 ns2.210 ns1.849 ns2.786 ns-
ToStringFalse3.220 ns0.2709 ns0.3119 ns3.219 ns2.742 ns3.886 ns-

@danmoseleydanmoseley reopened this Feb 5, 2022
Comment threadsrc/benchmarks/micro/libraries/System.Runtime/Perf.Boolean.cs Outdated
@danmoseley
danmoseley merged commit c40d303 into dotnet:mainFeb 6, 2022
@danmoseley
danmoseley deleted the bool.parse branch February 6, 2022 03:32
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@danmoseley@LoopedBard3@stephentoub
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers"); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + '
Skip to content

add boolean parsing variations - #2234

Merged
danmoseley merged 4 commits into
dotnet:mainfrom
danmoseley:bool.parse
Feb 6, 2022
Merged

add boolean parsing variations#2234
danmoseley merged 4 commits into
dotnet:mainfrom
danmoseley:bool.parse

Conversation

@danmoseley

@danmoseleydanmoseley commented Feb 4, 2022

Copy link
Copy Markdown
Contributor

What I get on my machine, without dotnet/runtime#64782

MethodvalueMeanErrorStdDevMedianMinMaxAllocated
TryParsetrue9.877 ns0.3212 ns0.3700 ns9.807 ns9.161 ns10.60 ns-
TryParsefalse11.404 ns0.4224 ns0.4865 ns11.344 ns10.722 ns12.22 ns-
TryParsefAlSe11.179 ns0.2641 ns0.2936 ns11.167 ns10.733 ns11.84 ns-
TryParseTrUe12.215 ns1.7123 ns1.9032 ns11.467 ns10.127 ns15.41 ns-
TryParsetru23.106 ns0.6177 ns0.6866 ns22.909 ns22.116 ns24.61 ns-
TryParsetRuu25.520 ns0.5347 ns0.5722 ns25.542 ns24.739 ns26.64 ns-
TryParsefalss26.871 ns0.5399 ns0.5302 ns27.117 ns25.552 ns27.44 ns-
TryParsebogus34.17 ns6.021 ns6.934 ns30.54 ns27.177 ns48.57 ns-
Parsetrue16.222 ns0.7374 ns0.8492 ns16.208 ns14.751 ns18.18 ns-
Parsefalse17.495 ns1.1390 ns1.3117 ns16.919 ns16.059 ns20.40 ns-

@danmoseley

Copy link
Copy Markdown
ContributorAuthor

@DrewScoggins@LoopedBard3Helix API does not contain an entry for Windows.10.Amd64.ClientRS4.DevEx.15.8.Open

@LoopedBard3

Copy link
Copy Markdown
Member

Looks like the ClientRS4 queue is EOL and the likely upgrade is to ClientRS5. I will do a test run to see if using the new queue fixes the issue.

@LoopedBard3

Copy link
Copy Markdown
Member

Testing in PR #2237, will update once the run completes.

@LoopedBard3

Copy link
Copy Markdown
Member

Closing for rebase since #2237 has been merged.

@LoopedBard3

Copy link
Copy Markdown
Member

It appears reopening did not catch the latest queue update.

@danmoseleydanmoseley reopened this Feb 5, 2022
@danmoseley

Copy link
Copy Markdown
ContributorAuthor

performance-ci (windows RS4 x64 micro_net461 Open LTS) results can be ignored. it's a dead queue

@danmoseley

Copy link
Copy Markdown
ContributorAuthor

@LoopedBard3 perhaps you'd like to review so I can merge?

Comment threadsrc/benchmarks/micro/libraries/System.Runtime/Perf.Boolean.cs Outdated
Comment threadsrc/benchmarks/micro/libraries/System.Runtime/Perf.Boolean.cs Outdated
@danmoseleydanmoseley reopened this Feb 5, 2022
@danmoseley

Copy link
Copy Markdown
ContributorAuthor

Now

MethodvalueMeanErrorStdDevMedianMinMaxAllocated
TryParsetrue10.252 ns0.2067 ns0.1832 ns10.211 ns10.011 ns10.583 ns-
TryParsefalse11.393 ns0.5187 ns0.5974 ns11.515 ns10.246 ns12.289 ns-
TryParseTRUE13.440 ns2.3827 ns2.7440 ns12.408 ns10.431 ns19.580 ns-
TryParseFalse13.902 ns2.3719 ns2.6363 ns12.672 ns11.074 ns20.427 ns-
TryParse019.966 ns0.7141 ns0.8223 ns20.130 ns18.458 ns21.506 ns-
TryParseBogus24.709 ns2.0741 ns2.3053 ns23.893 ns22.295 ns29.574 ns-
Parsetrue16.790 ns1.1961 ns1.3774 ns16.326 ns15.212 ns19.532 ns-
ParseTRUE18.818 ns1.4715 ns1.5745 ns18.369 ns16.956 ns22.533 ns-
Parsefalse19.133 ns0.7491 ns0.8016 ns19.191 ns17.416 ns21.096 ns-
ParseFalse19.211 ns1.8161 ns2.0915 ns19.938 ns15.663 ns22.658 ns-
ToStringTrue2.225 ns0.2246 ns0.2206 ns2.210 ns1.849 ns2.786 ns-
ToStringFalse3.220 ns0.2709 ns0.3119 ns3.219 ns2.742 ns3.886 ns-

@danmoseleydanmoseley reopened this Feb 5, 2022
Comment threadsrc/benchmarks/micro/libraries/System.Runtime/Perf.Boolean.cs Outdated
@danmoseley
danmoseley merged commit c40d303 into dotnet:mainFeb 6, 2022
@danmoseley
danmoseley deleted the bool.parse branch February 6, 2022 03:32
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@danmoseley@LoopedBard3@stephentoub
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode"); } } catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); } })(); })();
Skip to content

add boolean parsing variations - #2234

Merged
danmoseley merged 4 commits into
dotnet:mainfrom
danmoseley:bool.parse
Feb 6, 2022
Merged

add boolean parsing variations#2234
danmoseley merged 4 commits into
dotnet:mainfrom
danmoseley:bool.parse

Conversation

@danmoseley

@danmoseleydanmoseley commented Feb 4, 2022

Copy link
Copy Markdown
Contributor

What I get on my machine, without dotnet/runtime#64782

MethodvalueMeanErrorStdDevMedianMinMaxAllocated
TryParsetrue9.877 ns0.3212 ns0.3700 ns9.807 ns9.161 ns10.60 ns-
TryParsefalse11.404 ns0.4224 ns0.4865 ns11.344 ns10.722 ns12.22 ns-
TryParsefAlSe11.179 ns0.2641 ns0.2936 ns11.167 ns10.733 ns11.84 ns-
TryParseTrUe12.215 ns1.7123 ns1.9032 ns11.467 ns10.127 ns15.41 ns-
TryParsetru23.106 ns0.6177 ns0.6866 ns22.909 ns22.116 ns24.61 ns-
TryParsetRuu25.520 ns0.5347 ns0.5722 ns25.542 ns24.739 ns26.64 ns-
TryParsefalss26.871 ns0.5399 ns0.5302 ns27.117 ns25.552 ns27.44 ns-
TryParsebogus34.17 ns6.021 ns6.934 ns30.54 ns27.177 ns48.57 ns-
Parsetrue16.222 ns0.7374 ns0.8492 ns16.208 ns14.751 ns18.18 ns-
Parsefalse17.495 ns1.1390 ns1.3117 ns16.919 ns16.059 ns20.40 ns-

@danmoseley

Copy link
Copy Markdown
ContributorAuthor

@DrewScoggins@LoopedBard3Helix API does not contain an entry for Windows.10.Amd64.ClientRS4.DevEx.15.8.Open

@LoopedBard3

Copy link
Copy Markdown
Member

Looks like the ClientRS4 queue is EOL and the likely upgrade is to ClientRS5. I will do a test run to see if using the new queue fixes the issue.

@LoopedBard3

Copy link
Copy Markdown
Member

Testing in PR #2237, will update once the run completes.

@LoopedBard3

Copy link
Copy Markdown
Member

Closing for rebase since #2237 has been merged.

@LoopedBard3

Copy link
Copy Markdown
Member

It appears reopening did not catch the latest queue update.

@danmoseleydanmoseley reopened this Feb 5, 2022
@danmoseley

Copy link
Copy Markdown
ContributorAuthor

performance-ci (windows RS4 x64 micro_net461 Open LTS) results can be ignored. it's a dead queue

@danmoseley

Copy link
Copy Markdown
ContributorAuthor

@LoopedBard3 perhaps you'd like to review so I can merge?

Comment threadsrc/benchmarks/micro/libraries/System.Runtime/Perf.Boolean.cs Outdated
Comment threadsrc/benchmarks/micro/libraries/System.Runtime/Perf.Boolean.cs Outdated
@danmoseleydanmoseley reopened this Feb 5, 2022
@danmoseley

Copy link
Copy Markdown
ContributorAuthor

Now

MethodvalueMeanErrorStdDevMedianMinMaxAllocated
TryParsetrue10.252 ns0.2067 ns0.1832 ns10.211 ns10.011 ns10.583 ns-
TryParsefalse11.393 ns0.5187 ns0.5974 ns11.515 ns10.246 ns12.289 ns-
TryParseTRUE13.440 ns2.3827 ns2.7440 ns12.408 ns10.431 ns19.580 ns-
TryParseFalse13.902 ns2.3719 ns2.6363 ns12.672 ns11.074 ns20.427 ns-
TryParse019.966 ns0.7141 ns0.8223 ns20.130 ns18.458 ns21.506 ns-
TryParseBogus24.709 ns2.0741 ns2.3053 ns23.893 ns22.295 ns29.574 ns-
Parsetrue16.790 ns1.1961 ns1.3774 ns16.326 ns15.212 ns19.532 ns-
ParseTRUE18.818 ns1.4715 ns1.5745 ns18.369 ns16.956 ns22.533 ns-
Parsefalse19.133 ns0.7491 ns0.8016 ns19.191 ns17.416 ns21.096 ns-
ParseFalse19.211 ns1.8161 ns2.0915 ns19.938 ns15.663 ns22.658 ns-
ToStringTrue2.225 ns0.2246 ns0.2206 ns2.210 ns1.849 ns2.786 ns-
ToStringFalse3.220 ns0.2709 ns0.3119 ns3.219 ns2.742 ns3.886 ns-

@danmoseleydanmoseley reopened this Feb 5, 2022
Comment threadsrc/benchmarks/micro/libraries/System.Runtime/Perf.Boolean.cs Outdated
@danmoseley
danmoseley merged commit c40d303 into dotnet:mainFeb 6, 2022
@danmoseley
danmoseley deleted the bool.parse branch February 6, 2022 03:32
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@danmoseley@LoopedBard3@stephentoub