# Conditional Concatenation of a Pandas DataFrame

I am concatenating columns of a Python Pandas Dataframe and want to improve the speed of my code.

My data has the following structure:

    Apple    Pear    Cherry
1        2       3
4        5       NaN
7        8       9


I only want to concatenate the contents of the Cherry column if there is actually value in the respective row. If my code works correctly, the result of the example above should be:

    Result
1 :: 2 :: 3
4 :: 5
7 :: 8 :: 9


My code so far is this:

a_dataframe[result] = a_dataframe.apply(lambda r:
str(r.loc['apple']) + ' :: ' + str(r.loc['pear'])+' :: '+str(r.loc['cherry'])
if pd.notnull(r.loc['cherry']) & (r.loc['cherry'] != "")
# if cherry value is empty, do not add cherry into result
else str(r.loc['apple']) + ' :: ' + str(r.loc['pear']),
axis=1)


Any thoughts on how I can improve the speed of my code? Can I run this without an apply statement using only Pandas column operations?

Thanks up front for the help.

• Welcome to codereview. Does your code works exactly as you posted it ? :) – Grajdeanu Alex Feb 6 '17 at 7:55
• Code works as i posted it. Regarding single quote: I changed variable names for simplicity when posting, so I probably lost it in the process :-). – MRR Feb 6 '17 at 8:46

There's no need to create a lambda for this.

Let's suppose we have the following dataframe:

my_df = pd.DataFrame({
'Apple':  ['1', '4', '7'],
'Pear':   ['2', '5', '8'],
'Cherry': ['3', np.nan, '9']})


Which is:

Apple Cherry Pear
1      3    2
4    NaN    5
7      9    8


An easier way to achieve what you want without the apply() function is:

You'll get this:

l = []
for _, row in my_df.iterrows():
l.append(pd.Series(row).str.cat(sep='::'))

empty_df = pd.DataFrame(l, columns=['Result'])


Doing this, NaN will automatically be taken out, and will lead us to the desired result:

Result
1::3::2
4::5
7::9::8


The entire program may look like:

import pandas as pd
import numpy as np

def merge_columns(my_df):
l = []
for _, row in my_df.iterrows():
l.append(pd.Series(row).str.cat(sep='::'))
empty_df = pd.DataFrame(l, columns=['Result'])

return empty_df.to_string(index=False)

if __name__ == '__main__':
my_df = pd.DataFrame({
'Apple': ['1', '4', '7'],
'Pear': ['2', '5', '8'],
'Cherry': ['3', np.nan, '9']})
print(merge_columns(my_df))


As @MathiasEttinger suggested, you can also modify the above function to use list comprehension to get a slightly better performance:

def merge_columns_1(my_df):
l = [pd.Series(row).str.cat(sep='::') for _, row in my_df.iterrows()]

return pd.DataFrame(l, columns=['Result']).to_string(index=False)


I'll let the order of the columns as an exercise for OP.

• I like this a lot (definitely looks cleaner, and this code could easily be scaled for additional columns), but I just timed my code and don't really see a significant difference to the original code. Since we're still looping through every row (before: using apply, now: using iterrows) this seems to make sense. Any thoughts? – MRR Feb 6 '17 at 9:56
• I don't think you can get any better than this in terms of performance – Grajdeanu Alex Feb 6 '17 at 10:37
• Why don't you use a list-comprehension instead of for + append ? – 301_Moved_Permanently Feb 6 '17 at 10:49
• @MathiasEttinger good call. I added that too. Thanks :) – Grajdeanu Alex Feb 6 '17 at 10:56
• Guess I'll just leave it here then. Thanks for the help!! – MRR Feb 6 '17 at 11:12