Skip to content

Review tutorial RStudio #6002

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Review tutorial RStudio #6002

wants to merge 2 commits into from

Conversation

Sch-Da
Copy link
Collaborator

@Sch-Da Sch-Da commented Apr 26, 2025

I went through the tutorial and learned a lot, thank you for creating this training.

I found some typos and corrected those.
I changed a broken link

Todo:

  • Please check if this link still covers the content you intended to cover.
  • I did not get any search results for Student-t test in line 329 but found the other mentioned results. Maybe check if it is just me or consider changing it?

There were two part that I had problems understanding as someone unfamiliar with R until your tutorial:

-In line 177 Hands on: comment in a script
I ran:

Gives the current date

date()

I got:

Gives the current date

date()
[1] "Sat Apr 26 14:03:51 2025"

in the console.
you write below the box: "The comment line is displayed in the console but not executed."
As I saw the date, I thought the command was executed and ruminated over this quite a bit, wondering if I had done something wrong.
Could you maybe add a box showing the expected code outcome, as you did in "Display of function call in the tutorial" or add a further sentence here?

When I want to save my work in Galaxy, do I need to add '' to cover the filename? Using gx_put(filename) as gx_put(Test.R) as indicated in "Stopping RStudio" did unfortunately not work for me.

Thanks in advance for your help and thanks again for the tutorial.

@Sch-Da
Copy link
Collaborator Author

Sch-Da commented Apr 28, 2025

@teresa-m The current link https://swcarpentry.github.io/r-novice-inflammation/05-cmdline/ is broken.
I just wanted to change that and add the best I could find. Yours is working, so I am happy to use that one :)

@teresa-m
Copy link
Collaborator

@Sch-Da Thanks a lot for updating it.

@shiltemann shiltemann added the GTA label Apr 28, 2025
@Sch-Da
Copy link
Collaborator Author

Sch-Da commented Apr 29, 2025

thanks for having this on the radar @shiltemann. This one is independent from the GTA - so less priority as there is no deadline compared to the GTA ones.

@shiltemann shiltemann removed the GTA label Apr 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants