Update User's guide “convert-a-c++-program”

[behoppe]

Automatically generated by Netlify CMS

[netlify]

✅ Deploy Preview for sage-licorice-6da44d ready!

Name	Link
🔨 Latest commit	8d91f2c
🔍 Latest deploy log	https://app.netlify.com/sites/sage-licorice-6da44d/deploys/632a1811c6f5bd0008498827
😎 Deploy Preview	https://deploy-preview-102--sage-licorice-6da44d.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site settings.

[behoppe]

Hi @timkaler I have implemented a way to display attribution to Intel and Cilk Arts. If you think this article should credit them, then could you please find the new switch in the CMS editor ("provide attribution to Intel and Cilk Arts") or manually add attribution: true to the front matter. If you set the field to true, then the attribution still won't actually display until this PR is merged with the main branch. (A little more info is at #62.) Thank you.

[behoppe]

Hi @timkaler would it be OK with you if I edited this article? Based on feedback we heard from Charles (e.g., use cilk_scope and make it more conversational) I could go through the whole thing, and then maybe we could ask Alex or Charles to give it a final review before publishing.

There is a lot of good content in here, so I hope we can publish it soon.

[behoppe]

Thanks @timkaler for this excellent draft. I reviewed and made small stylistic changes along a few lines listed below.

I think this is good to go after we make sure that our qsort.cpp example is consistent across this article and the Cilkscale user's guide. This includes inline code, related discussion points, and the Cilkscale plots. @ailiop would you like to review these two with that in mind?

My sylistic edits:

• Less formal (more "conversational")
• More consistent with the Cilkscale user's guide
• Adding a "Discussion" section, which holds the part about improving performance of partition

[behoppe]

@ailiop and @timkaler could one of you please review this to make sure it's ready for publication? Thank you. (I edited it many months ago and it fell through the cracks.)
https://deploy-preview-102--sage-licorice-6da44d.netlify.app/doc/users-guide/convert-a-c-program/

[behoppe]

@timkaler what do you recommend we do with this PR (which came up during last week's meeting)? The PR has a lot of good content that is not on the published website. There are some outstanding issues mentioned in the previous comments (from April 2023 and Sept 2022) about aligning this article with the Cilkscale user's guide, because they both rely heavily on quicksort, and probably it would be best for them to share the same quicksort. Would you like to look into that, or recommend someone, or shall we publish this content as-is? Thanks for helping with this.