Skip to content
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

Updated exporting and archiving user guide #3367

Merged
merged 2 commits into from Jan 14, 2019

Conversation

jlstevens
Copy link
Contributor

Updates this user guide to get rid of the %%output and %%opts magics and gets rid of some outdated material.

One more major edit I would do is to cut down on the text talking about reproducibility - it is probably better to just get to the point and show what you can do instead of just describing it in text.

@jbednar
Copy link
Member

jbednar commented Jan 4, 2019

Sounds good! Yes, that section is wordy. Maybe instead of going into that depth, it could link to one of our papers about this topic. Or maybe some of those points could be made more simply at the end of the document, once the examples have been introduced.

@jlstevens
Copy link
Contributor Author

Or maybe some of those points could be made more simply at the end of the document, once the examples have been introduced.

I think that would be reasonable: people can easily skip over extra material that appears at the end after all the code concepts are introduced.

@jlstevens
Copy link
Contributor Author

I've move that large block of text to the bottom where people can read it if they are interested. Ready to review/merge.

@jbednar
Copy link
Member

jbednar commented Jan 9, 2019

Sounds good.

@jlstevens
Copy link
Contributor Author

I'm done with this PR now. Happy to merge?

@philippjfr philippjfr merged commit 5526459 into master Jan 14, 2019
@philippjfr philippjfr deleted the exporting_user_guide_update branch April 29, 2019 11:26
Sign up for free to 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.

None yet

3 participants