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

Add mapping in r tips #62

Merged
merged 6 commits into from
Jul 23, 2024
Merged

Add mapping in r tips #62

merged 6 commits into from
Jul 23, 2024

Conversation

cjrace
Copy link
Contributor

@cjrace cjrace commented Jul 18, 2024

Overview of changes

Adding some notes on making maps in R that I started when editing the map app.

Why are these changes being made?

Might be helpful for analysts to shortcut the initial learning process and have some handy immediate references to start from.

Detailed description of changes

Added a variety of notes on mapping in R, an example interactive map in Shiny and tips for improving the load times of interactive maps.

There's definitely more information that could be added, though for now I figured I'd add the notes I had off the top of my head and we can always expand on demand.

I also renamed the project file as it's been bugging me I forgot to do it when we first moved over from the statisticians guide repo.

Issue ticket number/s and link

No related issues.

Checklist before requesting a review

  • I have checked the contributing guidelines
  • I have checked for and linked any relevant issues that this may resolve
  • I have checked that these changes build locally (checking this now)
  • I understand that if merged into main, these changes will be publicly available

@cjrace cjrace marked this pull request as ready for review July 18, 2024 09:14
@cjrace
Copy link
Contributor Author

cjrace commented Jul 18, 2024

Copy link
Contributor

@jen-machin jen-machin left a comment

Choose a reason for hiding this comment

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

Probably nitpicky comments and one entirely pointless one

learning-development/r.qmd Outdated Show resolved Hide resolved
learning-development/r.qmd Show resolved Hide resolved
learning-development/r.qmd Show resolved Hide resolved
@jen-machin jen-machin self-requested a review July 23, 2024 08:46
@cjrace cjrace merged commit c4377ab into main Jul 23, 2024
@cjrace cjrace deleted the add-mapping-in-r-tips branch July 23, 2024 08:47
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