Skip to content

Conversation

@lukebaker
Copy link

No description provided.

Choose a reason for hiding this comment

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

good summary

Choose a reason for hiding this comment

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

Ditto! Interesting paper.

Copy link

Choose a reason for hiding this comment

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

Very interesting paper. Also interesting that "switching accounts for none of mainline decline" . Maybe add some information of the method/ model used in the summary.

Choose a reason for hiding this comment

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

Nice. Also, if you want to focus on the difference in fertility rates between the two groups, when not plot that directly for each year and then do a loess on that difference?

In general, it is always good to plot the quantity of interest and not ask the reader to do the transformation you want.

Choose a reason for hiding this comment

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

Cool! A nice project and interesting extension finding. My main question is whether you can automate some of the recoding that you do - perhaps create functions for the different recodes that you can plug into dplyr. I think this might save work by cutting down on repetition and also minimize the chance of mistakes.

Copy link

Choose a reason for hiding this comment

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

I agree with Leah

Copy link

Choose a reason for hiding this comment

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

Nice work. Your commenting is excellent throughout the project.

lukebaker pushed a commit that referenced this pull request May 13, 2015
Merge pull request #1 from JBirdsall/master
lukebaker pushed a commit that referenced this pull request May 13, 2015
Merge pull request #1 from gracetien/master
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.

8 participants