<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">Hi Pavol,&nbsp;<div class=""><br class=""></div><div class=""><blockquote type="cite" class=""><div dir="ltr" class=""><div class="gmail_extra"><div class="gmail_quote"><div class="">Could you tell me (point me to source code)&nbsp;how does the Swift-CI integration on GitHub piece together the benchmark report? I'd like to fix the subheadings "Unoptimized (Onone)" and "Hardware Overview" that are not being displayed correctly. I suggest we use ### Subheading instead of trying to **bold** the text.</div></div></div></div></blockquote><br class=""></div><div class="">This is part of Jenkins CI config, I will update the config to use ###.&nbsp;</div><div class=""><br class=""></div><div class="">
<div><br class=""><blockquote type="cite" class=""><div class="">On May 25, 2017, at 2:14 PM, Pavol Vaskovic &lt;<a href="mailto:pali@pali.sk" class="">pali@pali.sk</a>&gt; wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class=""><div class="gmail_extra"><div class="gmail_quote">Hello Mishal, hello swift-dev<br class=""><div class=""><br class=""></div><div class="">Could you tell me (point me to source code)&nbsp;how does the Swift-CI integration on GitHub piece together the benchmark report? I'd like to fix the subheadings "Unoptimized (Onone)" and "Hardware Overview" that are not being displayed correctly. I suggest we use ### Subheading instead of trying to **bold** the text.</div><div class=""><br class=""></div><div class="">I'm also toying with a fix for overflowing results table by inserting breaks in our ever longer test names in the comp_perf_test.py. The situation is particularly egregious in GitHub's mobile view:</div><div class=""><span id="cid:ii_j34wydvj0_15c41731d672d697">&lt;File 25-05-2017, 23 07 35.png&gt;</span><br class="">​<br class=""></div><div class=""><br class=""></div><div class="">I could use soft hypens or zero-width spaces:</div><div class=""><br class=""></div><div class=""><h3 style="box-sizing:border-box;margin-bottom:16px;font-size:1.25em;line-height:1.25;color:rgb(36,41,46);font-family:-apple-system,system-ui,&quot;Segoe UI&quot;,Helvetica,Arial,sans-serif,&quot;Apple Color Emoji&quot;,&quot;Segoe UI Emoji&quot;,&quot;Segoe UI Symbol&quot;;margin-top:0px" class="">Soft Hyphenated</h3><table style="box-sizing:border-box;border-collapse:collapse;margin-top:0px;margin-bottom:16px;display:block;width:672px;overflow:auto;color:rgb(36,41,46);font-family:-apple-system,system-ui,&quot;Segoe UI&quot;,Helvetica,Arial,sans-serif,&quot;Apple Color Emoji&quot;,&quot;Segoe UI Emoji&quot;,&quot;Segoe UI Symbol&quot;;font-size:14px" class=""><thead style="box-sizing:border-box" class=""><tr style="box-sizing:border-box;border-top:1px solid rgb(198,203,209)" class=""><th style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">TEST</th><th style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">MIN</th><th style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">MAX</th><th style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">MEAN</th><th style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">MAX_RSS</th></tr></thead><tbody style="box-sizing:border-box" class=""><tr style="box-sizing:border-box;border-top:1px solid rgb(198,203,209)" class=""><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">Existential­Test­Pass­Existential­One­Method­Call_­Class­Value­Buffer1</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">664663</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">664928</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">664810</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">9409877</td></tr><tr style="box-sizing:border-box;background-color:rgb(246,248,250);border-top:1px solid rgb(198,203,209)" class=""><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">Existential­Test­Pass­Existential­One­Method­Call_­Class­Value­Buffer2</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">943681</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">943979</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">943869</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">9418069</td></tr><tr style="box-sizing:border-box;border-top:1px solid rgb(198,203,209)" class=""><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">Existential­Test­Pass­Existential­One­Method­Call_­Class­Value­Buffer3</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">1252577</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">1254327</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">1253654</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">9412608</td></tr><tr style="box-sizing:border-box;background-color:rgb(246,248,250);border-top:1px solid rgb(198,203,209)" class=""><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">Existential­Test­Pass­Existential­One­Method­Call_­Class­Value­Buffer4</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">933119</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">933913</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">933470</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">9415339</td></tr></tbody></table><h3 style="box-sizing:border-box;margin-top:24px;margin-bottom:16px;font-size:1.25em;line-height:1.25;color:rgb(36,41,46);font-family:-apple-system,system-ui,&quot;Segoe UI&quot;,Helvetica,Arial,sans-serif,&quot;Apple Color Emoji&quot;,&quot;Segoe UI Emoji&quot;,&quot;Segoe UI Symbol&quot;" class="">Zero-width Spaced</h3><table style="box-sizing:border-box;border-collapse:collapse;margin-top:0px;display:block;width:672px;overflow:auto;color:rgb(36,41,46);font-family:-apple-system,system-ui,&quot;Segoe UI&quot;,Helvetica,Arial,sans-serif,&quot;Apple Color Emoji&quot;,&quot;Segoe UI Emoji&quot;,&quot;Segoe UI Symbol&quot;;font-size:14px;margin-bottom:0px" class=""><thead style="box-sizing:border-box" class=""><tr style="box-sizing:border-box;border-top:1px solid rgb(198,203,209)" class=""><th style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">TEST</th><th style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">MIN</th><th style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">MAX</th><th style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">MEAN</th><th style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">MAX_RSS</th></tr></thead><tbody style="box-sizing:border-box" class=""><tr style="box-sizing:border-box;border-top:1px solid rgb(198,203,209)" class=""><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">Existential​Test​Pass​Existential​One​Method​Call_​Class​Value​Buffer1</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">664663</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">664928</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">664810</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">9409877</td></tr><tr style="box-sizing:border-box;background-color:rgb(246,248,250);border-top:1px solid rgb(198,203,209)" class=""><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">Existential​Test​Pass​Existential​One​Method​Call_​Class​Value​Buffer2</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">943681</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">943979</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">943869</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">9418069</td></tr><tr style="box-sizing:border-box;border-top:1px solid rgb(198,203,209)" class=""><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">Existential​Test​Pass​Existential​One​Method​Call_​Class​Value​Buffer3</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">1252577</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">1254327</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">1253654</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">9412608</td></tr><tr style="box-sizing:border-box;background-color:rgb(246,248,250);border-top:1px solid rgb(198,203,209)" class=""><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">Existential​Test​Pass​Existential​One​Method​Call_​Class​Value​Buffer4</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">933119</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">933913</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">933470</td><td style="box-sizing:border-box;padding:6px 13px;border:1px solid rgb(223,226,229)" class="">9415339</td></tr></tbody></table></div><div class=""><br class=""></div><div class="">IMO this would considerably improve the situation. The only downside is that if you copied the test names, you would not be able to find them in the project, because of the invisible characters inserted between the came case word boundaries. What do you think?</div></div></div></div></div></blockquote><div><br class=""></div>I think zero-with spaced might be better, because if test contains “-“ it might cause confusion.&nbsp;<br class=""><blockquote type="cite" class=""><div class=""><div dir="ltr" class=""><div class="gmail_extra"><div class="gmail_quote"><div class=""><br class=""></div><div class="">--Pavol</div></div></div></div>
</div></blockquote></div><br class=""></div></body></html>