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 basic vignettes #51

Merged
merged 16 commits into from
Jan 4, 2024
Merged

Add basic vignettes #51

merged 16 commits into from
Jan 4, 2024

Conversation

richfitz
Copy link
Member

@richfitz richfitz commented Jan 3, 2024

No description provided.

Copy link
Contributor

@weshinsley weshinsley left a comment

Choose a reason for hiding this comment

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

Tada.... It reads pretty well to me! Some suggestions to whizz through

R/task.R Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
vignettes/details.Rmd Outdated Show resolved Hide resolved
usethis::edit_r_profile()
```

and then adding a call to `options()` anywhere in that file. If you are comfortable with `vi` on the command line you may prefer using `vi ~/.Rprofile`.
Copy link
Contributor

Choose a reason for hiding this comment

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

You could include how to exit from vi.
One can never hear those instructions often enough.

Copy link
Member Author

Choose a reason for hiding this comment

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

I have included a joke now

vignettes_src/troubleshooting.Rmd Outdated Show resolved Hide resolved
* A misconfigured node (sometimes they are missing particular software)
* A networking issue
* Gremlins
* Network path mapping error
Copy link
Contributor

Choose a reason for hiding this comment

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

  • Running out of disk space at some point after submitting the job

Copy link
Member Author

Choose a reason for hiding this comment

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

added

vignettes_src/troubleshooting.Rmd Outdated Show resolved Hide resolved
vignettes_src/troubleshooting.Rmd Outdated Show resolved Hide resolved
vignettes_src/troubleshooting.Rmd Outdated Show resolved Hide resolved
Copy link
Contributor

@weshinsley weshinsley left a comment

Choose a reason for hiding this comment

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

One more reference to "context"

vignettes_src/troubleshooting.Rmd Outdated Show resolved Hide resolved
richfitz and others added 5 commits January 4, 2024 15:06
@richfitz richfitz requested a review from weshinsley January 4, 2024 15:21
Copy link
Contributor

@weshinsley weshinsley left a comment

Choose a reason for hiding this comment

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

Great!

@weshinsley weshinsley merged commit 3228f1b into main Jan 4, 2024
13 checks passed
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