-
Notifications
You must be signed in to change notification settings - Fork 22
add blog post about mystification / overall blog structure #542
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
Conversation
👋 Thanks for opening this PR! The Cookbook will be automatically built with GitHub Actions. To see the status of your deployment, click below. |
Preview link to most relevant file: https://projectpythia.org/_preview/542/posts/mystification ![]() ![]() |
Not something that needs to be solved now, but I'm curious if there's a good way to add author metadata for folks who's info isn't already propagating from the upstream config. I suppose we could add that in the MyST config for this repo, but not sure if there's an article/post level option. |
There is, but we should take this opportunity to add people upstream since that will simply all future posts. I'll open a PR for that. |
The answer to your question is that the The shortcut here is that we're just giving the |
Upstream PR: ProjectPythia/pythia-config#16 once that is merged, we can regenerate the preview here and make sure everyone is showing up correctly. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed some errant links here. There is also some formatting that needs fixing in the first section.
Hi from SciPy! The post is looking very nice. Note that in the list of authors, Julia needs to come after Angus to follow last-name alphabetical sorting. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a couple of minor things to take or leave. Otherwise, looks good to me (with Brian's edits).
Thanks for putting this together!
Co-authored-by: Brian Rose <[email protected]>
Co-authored-by: Brian Rose <[email protected]>
Co-authored-by: Brian Rose <[email protected]>
Co-authored-by: Katelyn FitzGerald <[email protected]>
Co-authored-by: Katelyn FitzGerald <[email protected]>
It turns out that MyST doesn't know how to interpret the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suggested some edits here that should fix the formatting in the "Motivation" section.
Closing and reopening one more time to fix the authors list. |
I tried a collapsible-by-year navbar and it isn't working
I'm not pressed on it though. What are your thoughts on the new directory structure @brian-rose ? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it all looks good. We can work on improved blog navigation in a separate PR.
If you are looking for inspiration/examples, Chris Holdgraf's site uses myst and has a nice structure around it! Here is a link to his blog post about building it |
Thanks for this link @mgrover1 I'll look into implementing this after this PR is merged! I was thinking of using the same type of gallery extension we use elsewhere, will look at what his method is this afternoon. |
In light of our link checker being too strict - it is stalled on a call to localhost. There are many incomplete/template links in the contribution guides and elsewhere that aren't meant to be clickable. |
Thanks for editing the document @brian-rose @dcamron and @kafitzgerald