Skip to content

Conversation

@razvanvasile
Copy link
Contributor

No description provided.

@coveralls
Copy link

coveralls commented Jun 1, 2017

Coverage Status

Coverage remained the same at 93.189% when pulling 4001a9d on razvanvasile:improve_docs into bd0855e on willrogers:master.

@@ -1,3 +1,5 @@
""" Class to load the elements of the machine from a csv file."""

Copy link
Collaborator

Choose a reason for hiding this comment

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

Do you know what the difference between a 'module' and a 'class' is?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The module can contain multiple routines or different classes. My bad for calling it a class instead of a module.

@coveralls
Copy link

coveralls commented Jun 1, 2017

Coverage Status

Coverage remained the same at 93.189% when pulling 2037b8f on razvanvasile:improve_docs into bd0855e on willrogers:master.

@@ -1,3 +1,5 @@
""" Module to load the elements of the machine from a csv file."""

Copy link
Collaborator

Choose a reason for hiding this comment

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

It might be worth saying that you load machine data from multiple CSV files in a single directory.

@coveralls
Copy link

coveralls commented Jun 1, 2017

Coverage Status

Coverage remained the same at 93.189% when pulling f86ba7e on razvanvasile:improve_docs into bd0855e on willrogers:master.

@willrogers willrogers merged commit f413597 into DiamondLightSource:master Jun 1, 2017
@razvanvasile razvanvasile deleted the improve_docs branch June 1, 2017 13:52
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