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

Create Million_KMs-Cycled-Per-Year.rmd #94

Closed
wants to merge 5 commits into from

Conversation

natesheehan
Copy link
Contributor

- RE issue itsleeds#59 
- Rough first draft discussing how to calculate, compare and use the PCT models
@Robinlovelace
Copy link
Member

Thanks for this @natesheehan. I think there's an issue with vignette titles:

https://github.com/ITSLeeds/pct/runs/1578485229#step:9:151

Copy link
Member

@Robinlovelace Robinlovelace 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, needs a different title for the tests to pass. Also could you rename the file, e.g. from

Million_KMs-Cycled-Per-Year.rmd 

to

km-cycled.Rmd

Short and sweet!

title: "Estimating million km's cycled per zone in England and Wales regions"
output: rmarkdown::html_vignette
vignette: >
%\VignetteIndexEntry{Reproducing cycling potential estimates in UK cities}
Copy link
Member

Choose a reason for hiding this comment

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

This should be updated to: Estimating million km's cycled per zone in England and Wales regions

Copy link
Contributor Author

@natesheehan natesheehan left a comment

Choose a reason for hiding this comment

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

Ah yep sounds good

@Robinlovelace
Copy link
Member

Many thanks @natesheehan, looking good. It's still complaining about duplicate title, not 100% sure what it's going on about tbh or how to fix it... 2 options: I merge now and fix up on my branch or you give it another bash. You call!

@natesheehan
Copy link
Contributor Author

Hm let me try deleting this PR and creating a new one // if not have a go on your branch.

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.

2 participants