-
Notifications
You must be signed in to change notification settings - Fork 5
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
Create Blog “2022-09-07-my-experience-teaching-performance-engineering” #165
base: main
Are you sure you want to change the base?
Create Blog “2022-09-07-my-experience-teaching-performance-engineering” #165
Conversation
✅ Deploy Preview for sage-licorice-6da44d ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
I believe @cleiserson wanted to actually discuss the content here before we published. I think he was enthusiastic about the overall content but had a few knobs he wanted me to tweak. |
Hi @jowens. I have copied your draft article into a pull request on opencilk.org. This provides a place where you/we can make any edits before we publish. If you view this PR on github, you will also see a "Deploy Preview" link where there is an instance of the complete website that includes this post, as if it were published. (See pic below.) I am going to encourage a couple other team members (@DorothyCurtis @neboat @ailiop) to read your draft so that we can provide feedback and advance this to publication. My high-level feedback:
|
@jowens you are right that @cleiserson suggested some editorial review. If it works for you, let's please use this PR for making any edits, by pushing to cms/blog/2022-09-07-my-experience-teaching-performance-engineering on this repo. You can do this with your favorite GitHub tools, or you can use Netlify CMS as documented in our guidelines. Please let me know if you prefer a different way to edit. Thank you. |
Mainly to focus more on the "how-to" aspect and refer to the Reference manual when needed.
- Remove TODO notes - Use publicly accessible URLs for Cilkscale papers - Small edits for clarity
- Minor edits in initial sections for clarity - Remove some distracting comments - Rephrase discussion on Cilkscale Python script to avoid ambiguity on what "benchmarking" means - Regenerate final Cilkscale table and plots, and update "insights" accordingly
- Off-white axes foreground in plots - Dotted grid lines in plots - Correction in CLI-formatted CSV table output - Consistent plot sizes
trim teach performance page and add modern algorithms workshop; add test join-us page with new form
survey iframe in join-us page
link to join-us from the community and teach-performance pages
Add new page for OpenCilk-related software, to hold some content originally included in /doc/users-guide/install.md.
Fix typo, and add reference to the page on contributing to OpenCilk.
Adjust level of headings.
Adjust level of final heading.
and add 11tydata.js files for community and contribute folders
Starting a list of open projects for prospective OpenCilk contributors.
Re-org of contribute.md which features teaching at the top and points to a new standalone page for open projects.
Revised description of DPRNG and pedigree support.
I took key take-aways from "The Future" (concluding section) and put them at the top in a new introductory paragraph. I also added a headshot (based on JDO's GitHub profile).
…engineering' of github.com:opencilk/www.opencilk.org into cms/blog/2022-09-07-my-experience-teaching-performance-engineering
Hi @jowens. I discussed this post with @cleiserson and he suggested I implement the edits we sketched before (back in Sept). Along the way I rebased this branch on main. The biggest changes I made are at the very top -- with a headshot and a new opening paragraph. Can you please take a look? After the top, the rest of my edits are minor. The other edits I hope to make on this post involve another page: Based on other discussions we've had about sharing EEC 289Q material, I plan to give that course a place in the Teach Performance Engineering page, so that MIT 6.106 has a heading, and so does UC Davis EEC 289Q. Thanks, John, for allowing us to share PDFs of your lectures publicly. I expect that page would hold the list of lectures, so that I could remove some/most of that detail from the blog post. |
PS: the blog post preview is here https://deploy-preview-165--sage-licorice-6da44d.netlify.app/posts/2022-09-07-my-experience-teaching-performance-engineering/ (you can also find it from the deploy-preview that is included at the top of this PR). |
Splendid! May I edit that first paragraph so it sounds just a little more like me? I like the changes you made! I am, however, not sure on what I need to do to edit text at this point (what to do in git), need a hand there. |
Yes -- please make it sound like you :-) The post resides in branch cms/blog/2022-09-07-my-experience-teaching-performance-engineering. You should be able to check out that branch and update /src/posts/2022-09-07-my-experience-teaching-performance-engineering.md. If anything seems amiss (perhaps because I rebased with main yesterday) please let me know. When you commit your changes to that same branch, it triggers a new deploy-preview automatically. |
ok, i think i figured out how to do this! stay tuned. |
Thanks @jowens. Looks great! Charles asked that we change the title with "software performance engineering," which I did. I also made one tiny edit, so the sentence now reads:
Charles agrees with us that this is one post -- not something to break up. So we are holding this as "ready to publish" while we get our pipeline primed with more content and a better sense of our publication schedule. Thanks again for this great story. |
I am in support of all that! |
Automatically generated by Netlify CMS