Let me tell you why - software development does not really need clean code, and neither does it need agile principles. One can craft perfectly working software without either. But, without either maintaining, modifying and scaling the software becomes incredibly cumbersome.
In most if not all instances, data scientists work in the space of applied sciences, and their experiments have to be reproducible and verifiable. This means a data scientist like yourself needs to understand someone else's problem and the respective team and business leads need to understand your solution.
Let’s quickly look at the other team members who you will be frequently collaborating with -
- Data scientists or data analysts collaborate with you on algorithms, models, interpreting results, feature engineering and more
- You collaborate with data engineers because one does not want to end up processing all the data by themselves. Data engineers will optimize queries to hand you, your consolidated results in record time. But, they need to begin by understanding what data it is that you need.
- Business analysts and domain experts help you clarify results in business terms.
Meaningful namesTeam members shouldn’t have to ask for the meaning of the variable used. Clear and intentional names are most suitable. e.g. instead of alpha = 0.3 try, learning_rate = 0.3.
Long names are perfectly acceptable. And, to make this manageable, better editors that don’t require repeated typing the entire variable name instead of shortening the name (making it cryptic) are recommended. Also, let’s avoid magic numbers.
for i in range (1,3000): ... # Use: number_of_iterations = 3000 for i in range(1,number_of_iterations): ...
Avoid mental mappingWhen there are code parameters that need to be changed to obtain desired results, you might want to save team members from reading your mind for those changes by creating a config file/ object. This will contain all the parameters and hyperparameters that have to be changed in your codebase. You could also try keeping relevant parameters together. For e.g. when building a neural network model that runs on the dataset of various different countries, your config file could resemble this:
# Neural Network hyperparameters learning_rate = 0.1 epoc = 3000 lambd = 0 # Other constants needed for your codebase country = "India"
Uniform naming conventionsAvoid multiple names for the same instance/occurrence either in mail, code or anywhere else in the project. For instance, During internal discussions personA might refer to the ‘Earth’ while personB might refer to the ‘Globe,’ both, being the same thing. Stick to one name during discussions and when coding.
Use business domain namesFeature engineering, analyzing, creating models are great places to exercise your creativity. Inventing names might not be the best use of your talents. Let’s say, business calls ‘it’ a sphere, then, the advice is to call ‘it’ a sphere instead of a ‘circle_in_3d’ in your code.
CommentsComments are discouraged because they quickly become obsolete. However, one could add comments that advise on using or not using certain models and features. For instance, a comment that reads, “This ADF (Augment Dickey Fuller) test that identifies stationary signals will take more than 5 hours on the laptop, because we cannot run combinations in parallel,” will let your team know not to run it at the start of their day.
# Neural Network hyperparameters # learning_rate < 0.05 is going to take more than 5 hours to train learning_rate = 0.1 epoc = 3000 lambd = 0 # Other constants needed for your codebase country = "India"
Modularize your codeBig blobs of code are opaque. So, instead of requiring team members to go through every line of the code to understand its purpose, you could create functions that do what their name states. Functions are a good level of abstraction. The reader can choose to go inside a function or a class only if they want to understand the internal working.
Don’t repeat yourselfExtract code you need, out into a function or component to reuse it. This ensures code consistency and the reader only needs to understand that component once.
Unit testsUnit tests are relevant to a data scientist's code because data science code is still code - especially if we are testing functions (like algorithms that recommend product prices) based on which business decisions are taken.
FormattingFormatting is a team decision. This includes how members use spaces vs. tabs for indentation, how the directory is structured, how file naming conventions are decided, the format for output results and more. Use tools like YAPF, autopep8 , prettier to automatically format the code when saving a file.
In conclusion, I vote for MVP approaches and trying out various approaches and models that might sometimes be quick and dirty but, give results. This approach is effective when figuring out how much time one should invest and in what direction. Once you do finalize your approach and incorporate it into your working code, do clean it up.
All of the concepts I have discussed in this article are for the benefit of the unified team. Interestingly, I find it useful when working solo, too. It serves as quick reminders of why I have chosen to do things a certain way and helps me explain results, and quickly cross-examine unexpected results as well.
Some of what I have talked about in this article can be found in the book, Clean Code by Robert C. Martin, an American software engineer, and instructor. He is best known for being one of the authors of the Agile Manifesto and for developing several software design principles. If you’d like a more in-depth view of the subject, I would urge you to read up. And, please remember, while every data scientist might have their own set of practices and rules when working in a team, it’s always good to abide by the commonly accepted set of do’s and don’ts.