-
Notifications
You must be signed in to change notification settings - Fork 36
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
Grammar edits to documentation; small additions to make testing more … #208
Conversation
doc/experiment.rst
Outdated
determine the allocation settings for the entire batch, and the ``RunSettings`` | ||
will determine how each individual ``Model`` instance is executed within | ||
that batch. | ||
|
||
The same example as above but tailored towards a running as a batch job | ||
on a slurm system | ||
This is same example as above, but tailored towards a running as a batch job |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be "This is the
"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will do!
doc/overview.rst
Outdated
@@ -58,19 +58,19 @@ deploying HPC workloads alongside an in-memory database: Redis. | |||
|
|||
The key features of the IL are: | |||
|
|||
- An API to start, monitor, and stop HPC jobs from Python or Jupyter notebook. | |||
- An API to start, monitor, and stop HPC jobs from a Python or Jupyter notebook. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
change to "from Python or from a Jupyter notebook"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will do!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice update, thank you! Two minor tweaks to update before merge.
Codecov Report
@@ Coverage Diff @@
## develop #208 +/- ##
===========================================
+ Coverage 84.18% 84.24% +0.06%
===========================================
Files 58 59 +1
Lines 3244 3301 +57
===========================================
+ Hits 2731 2781 +50
- Misses 513 520 +7
|
…clear