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

Updating installation documentation #81

Merged

Conversation

genevievestarke
Copy link
Collaborator

Updating the documentation page to reflect most current installation instructions

Copy link
Collaborator

@jfrederik-nrel jfrederik-nrel left a comment

Choose a reason for hiding this comment

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

Just the one typo

@@ -2,8 +2,8 @@

Still a work in progress but currently there are 3 sets of installation instructions

[Installing on kestrel](install_on_kestrel)
[Local installation instructions](intall_local)
Copy link
Collaborator

Choose a reason for hiding this comment

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

*install

```
which will tell you all of the processes currently working. You can kill processes that you do not want by using the kill command, paired with the process number (filled into the blank in the command below).
```
kill ___
Copy link
Collaborator

Choose a reason for hiding this comment

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

Change to kill <PID>?

@misi9170 misi9170 self-requested a review February 26, 2024 23:48
Copy link
Collaborator

@misi9170 misi9170 left a comment

Choose a reason for hiding this comment

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

Approving now; feel free to merge when you're ready

@genevievestarke genevievestarke merged commit 2720b10 into NREL:develop Feb 27, 2024
3 checks passed
@misi9170 misi9170 mentioned this pull request Sep 27, 2024
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.

3 participants