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

README.md: add 'About the Lesson' section #556

Merged
merged 5 commits into from
Oct 2, 2018

Conversation

Josh-Woodcock
Copy link
Contributor

#495 Improve README.md

Based swcarpentry/r-novice-inflammation README

  • put maintainers in a list, easier to read
  • add a short introduction
  • add links to other language versions of lesson (R and MATLAB)

Copy link
Contributor

@maxim-belkin maxim-belkin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, @Josh-Woodcock! Please see my comments.

README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated
perform a data analysis. Python is used to teach these skills because it is a
commonly used programming language in many scientific disciplines. However, the
emphasis is not on teaching every aspect of Python, but instead on
language agnostic principles like automation with loops and encapsulation with
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

language-agnostic

README.md Outdated
emphasis is not on teaching every aspect of Python, but instead on
language agnostic principles like automation with loops and encapsulation with
functions (see [Best Practices for Scientific Computing][best-practices] to
learn more). This lesson is also available in [R][R]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This lesson... deserves a separate paragraph

README.md Outdated
and [MATLAB][MATLAB].

The example used in this lesson analyzes a set of 12 data files with
inflammation data collected from a trial for a new treatment for arthritis (the data was simulated). Learners are shown how it is better to create a function and apply it to each of the 12 files using a loop instead of using copy-paste to analyze the 12 files individually.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like the idea of having such a paragraph, but not the current wording of it.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also, it might be useful to add a small title like About the lesson or similar

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like the idea @Josh-Woodcock had to give an overview of what the lesson is about (like for the R lesson in the README file).

And @maxim-belkin your suggestion to add "About the lesson" will help to highlight this paragraph.

README.md Show resolved Hide resolved
@maxim-belkin maxim-belkin added the type:enhancement Propose enhancement to the lesson label Jun 22, 2018
Copy link

@annefou annefou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I prefer to keep titles such as Contributing, Maintainers, Authors, Citation, etc. But as suggested by @maxim-belkin , adding "About the lesson" is a very good idea.

@maxim-belkin maxim-belkin added the status:changes requested Waiting for Contributor to update PR label Jun 23, 2018
@Josh-Woodcock
Copy link
Contributor Author

I impleneted the suggestion of adding an "About the Lesson" section, and amended the wording and formatting. The rest of the README.md has been left as is.

…more fully formed.

- put maintainers in a list, easier to read
- add a short introduction
= add links to other language versions of lesson (R and MATLAB)
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated
This lesson teaches novice programmers to write modular code to perform data analysis using Python. However, the emphasis is on teaching language-agnostic principles such as automation with loops and encapsulation with functions (see [Good enough practices in scientific computing][good-enough-practices] to
learn more).

The example used in this lesson analyses a set of 12 data files with inflammation data collected from a trial for a new treatment for arthritis (the data was simulated). Learners are shown how it is better to create a function and apply it to each of the 12 files using a loop instead of using copy-paste to analyse the 12 files individually.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the data was simulated ~> simulated data

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the last sentence could be improved. Don't mention a loop. Use phrases like Python enables ... and automation when talking about Python, and , error-prone (or similar) when talking about analysis done "by hand".

Josh-Woodcock and others added 2 commits October 1, 2018 15:50
- Clarified wording
- Point to Best Practices
- Move the one-line summary outside the `About the Lesson` section
- Remove parentheses
- Minor fixes (commas, articles, etc)
Copy link
Contributor

@maxim-belkin maxim-belkin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me @Josh-Woodcock ! Thank you! I added my changes directly to your branch. Feel free to comment on them if you think they're not appropriate.

@maxim-belkin
Copy link
Contributor

@annefou, please have a look at this PR when you get a chance.

@maxim-belkin maxim-belkin changed the title Changed layout of README to be consistent with the R version that is … README.md: add About the Lesson section Oct 1, 2018
@maxim-belkin maxim-belkin changed the title README.md: add About the Lesson section README.md: add 'About the Lesson' section Oct 1, 2018
@maxim-belkin maxim-belkin added high priority Need to be addressed ASAP and removed status:changes requested Waiting for Contributor to update PR labels Oct 1, 2018
Copy link

@annefou annefou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It sounds very good. Thanks for the updated version.

@maxim-belkin maxim-belkin merged commit e22ff7d into swcarpentry:gh-pages Oct 2, 2018
@maxim-belkin
Copy link
Contributor

Thank you very much, @Josh-Woodcock!

zkamvar pushed a commit that referenced this pull request Apr 21, 2023
* Added "About the Lesson" section.
* Cleaned up formatting.

Pull Request: #556
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
high priority Need to be addressed ASAP type:enhancement Propose enhancement to the lesson
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants