Assigning iter_row value to variable

[ankey]

I'm trying to load and read through an excel file and based on the value of a cell write the row to another file.

Problem I'm having now is that row[1].value is a string and I need to convert it to a number so that I can apply a great than > condition

The code I have so far which works based on the == is this:

# Import `load_workbook` module from `openpyxl`
from openpyxl import load_workbook

# Load in the workbook
wb = load_workbook('myFile.xlsx')

#activate a sheet into array
sheet = wb['Sheet1']
	
# select all populated rows and iterate through		
for row in sheet.iter_rows(min_col=1, min_row=1, max_col=2, max_row=sheet.max_row):
	if row[1].value == 1:
		pass
	else: 
		print(row[0].value)

If I change the condition in within the loop to a "greater than" like this:

# Import `load_workbook` module from `openpyxl`
from openpyxl import load_workbook

# Load in the workbook
wb = load_workbook('myFile.xlsx')

#activate a sheet into array
sheet = wb['Sheet1']
	
# select all populated rows and iterate through		
for row in sheet.iter_rows(min_col=1, min_row=1, max_col=2, max_row=sheet.max_row):
	if row[1].value == 1:
		pass
	else: 
		print(row[0].value)

Then I get the error:
TypeError: '>' not supported between instances of 'str' and 'int'

Any ideas?

[Larz60+]

if int(row[1].value) == 1:

[ankey]

if int(row[1].value) == 1:

Thanks for the input Larz60+ but when I try the suggested change I get the following error:

ValueError: invalid literal for int() with base 10: 'occurances'

[woooee]

You have to post the complete error message to get some help. And post the code again if you have changed it, as there is no "occurances" in any of the code you posted.

[volcano63]

You have to post the complete error message to get some help. And post the code again if you have changed it, as there is no "occurances" in any of the code you posted.

occurances - I would hazard a guess - is the cell content

if int(row[1].value) == 1:

This will be safe

if str(row[1].value) == '1':

[ankey]

Thanks for the help. Here's the full code:

# Import `load_workbook` module from `openpyxl`
from openpyxl import load_workbook

# Load in the workbook
wb = load_workbook('myFile.xlsx')

#activate a sheet into array
sheet = wb['Sheet1']

# select all populated rows and iterate through		
for row in sheet.iter_rows(min_col=1, min_row=1, max_col=2, max_row=sheet.max_row):
	# occur = int(row[1].value) 
	if int(row[1].value) == 1:
		pass
	else: 
		print(row[0].value)

And error:

Output:
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
C:\python\openpyxl_test.py in <module>()
     11 for row in sheet.iter_rows(min_col=1, min_row=1, max_col=2, max_row=sheet.max_row):
     12         # occur = int(row[1].value)
---> 13         if int(row[1].value) == 1:
     14                 pass
     15         else:

ValueError: invalid literal for int() with base 10: 'occurances'

[volcano63]

Output:
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
C:\python\openpyxl_test.py in <module>()
     11 for row in sheet.iter_rows(min_col=1, min_row=1, max_col=2, max_row=sheet.max_row):
     12         # occur = int(row[1].value)
---> 13         if int(row[1].value) == 1:
     14                 pass
     15         else:

ValueError: invalid literal for int() with base 10: 'occurances'

That was pretty clear - using my advice solves this issue. (Still, posting of traceback should be done before you are asked for it - helping others to help yourself.)

One thing - pass (in most cases) is redundant - if you have to choose a negative test condition, just use the negative test without else

if str(row[1].value) != '1':
    print(row[0].value)

[ankey]

Output:
 --------------------------------------------------------------------------- ValueError Traceback (most recent call last) C:\python\openpyxl_test.py in <module>() 11 for row in sheet.iter_rows(min_col=1, min_row=1, max_col=2, max_row=sheet.max_row): 12 # occur = int(row[1].value) ---> 13 if int(row[1].value) == 1: 14 pass 15 else: ValueError: invalid literal for int() with base 10: 'occurances' 

That was pretty clear - using my advice solves this issue. (Still, posting of traceback should be done before you are asked for it - helping others to help yourself.) One thing - pass (in most cases) is redundant - if you have to choose a negative test condition, just use the negative test without else

if str(row[1].value) != '1': print(row[0].value)

Thank you for the help. Using string does not work as I'm trying to carry out a condition that is a number comparison so I'm still stuck trying to convert the value to a integer.

Here is the code:

# Import `load_workbook` module from `openpyxl`
from openpyxl import load_workbook

# Load in the workbook
wb = load_workbook('myFile.xlsx')

#activate a sheet into array
sheet = wb['Sheet1']

# select all populated rows and iterate through		
for row in sheet.iter_rows(min_col=1, min_row=1, max_col=2, max_row=sheet.max_row):
	# occur = int(row[1].value) 
	if str(row[1].value) > 1:
		pass
	else: 
		print(row[0].value)

and here is the error:

Error:
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
C:\python\openpyxl_test.py in <module>()
     11 for row in sheet.iter_rows(min_col=1, min_row=1, max_col=2, max_row=sheet.max_row):
     12         # occur = int(row[1].value)
---> 13         if str(row[1].value) > 1:
     14                 pass
     15         else:

TypeError: '>' not supported between instances of 'str' and 'int'

[ankey]

Although I feel really dumb I just wanted to share the problem in case anyone finds the thread in the future....

The problem was that I left the column headers in the sheet one of which was 'occurrences'. So when I used the str it fell over on the first row.

Apologies for dumb mistake but thanks to those who helped me. I consider myself schooled and I'm sure I'll not get caught out by this again.