nav[aria-label="Primary Navigation"] { padding: 0; & ul { list-style: none; width: 100%; display: flex; flex-direction: row; justify-content: start; align-items: start; gap: 30px; padding: 0; & li { margin: 0; } & ul li { list-style: none; } } }

ModelBuilder Calculate Field

WadeGivens

I'm using ModelBuilder to create a new field ([cat]) that will contain category values based on range criteria of another field ([yield]). I would like to use the Python parser. HEre is what I have so far in the Pre-Logic Script Code:

def Reclass(yield) :
  if (yield <=140) :
    return "Low"
  elif (yield > 140 and yield <= 180) :
    return "Average"
  elif (yield > 180) :

In the expression block I have cat = Reclass(!cat!)

I get a red "X" in the calculate Field dialog stating there is a parse error on line 1. Any ideas as to what I'm doing wrong?

Thanks,

Vind meer berichten getagd met

Accepted answers

TerrySilveus

you can't use yield it is a reserved word

also what you posted doesn't see complete
try this

def Reclass(x):
  if (x <=140):
    return "Low"
  elif (x > 140 and x <= 180):
    return "Average"
  elif (x > 180):
    return "High"

All comments

curtvprice

The expression for the Calculate Field tool should simply be (if "cat" is your field name). CAT is also then entered in the Field argument to Calculate Field.

Ted is correct that you'd want to account for non-covered fields. Seems to me it would be safer to create a new field and calculate to that in case something goes wrong.

Reclass<SPAN class="punctuation token">(</SPAN>!CAT!<SPAN class="punctuation token">)</SPAN>‍<SPAN class="line-numbers-rows"><SPAN>‍</SPAN></SPAN>

TedKowal

</CODE></PRE><P>or perhaps &nbsp;</P><PRE class="lia-code-sample line-numbers language-none"><CODE>cat <SPAN class="operator token">=</SPAN> Reclass<SPAN class="punctuation token">(</SPAN>int<SPAN class="punctuation token">(</SPAN>!cat!<SPAN class="punctuation token">)</SPAN><SPAN class="punctuation token">)</SPAN>&nbsp;&nbsp; <SPAN class="comment token"># --- If you need to use the cat value and overwrite the string number with text</SPAN>
&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; <SPAN class="comment token"># --- Then you would also have to account for possible non numbers text in the Reclass def?‍‍‍‍‍‍‍‍</SPAN><SPAN class="line-numbers-rows"><SPAN>‍</SPAN><SPAN>‍</SPAN></SPAN>

TerrySilveus

Your expression block should be:
cat = Reclass(!yield!)
Otherwise you'd be trying to read an integer from a string field.

"Yield" within the function still needs to be changed as well, but you are right you can't pass a string when the function is looking for a number. Therefore cat = Reclass(!cat!) should be changed to cat = Reclass(!nameOfFieldwithNumbericValue!) unless cat is the field with the numeric value; in that case it should be nameOfFieldWithStringValue = Reclass(!Cat!)

BruceNielsen

Your expression block should be:

cat = Reclass(!yield!)

Otherwise you'd be trying to read an integer from a string field.

TerrySilveus

you can't use yield it is a reserved word

also what you posted doesn't see complete
try this

def Reclass(x):
  if (x <=140):
    return "Low"
  elif (x > 140 and x <= 180):
    return "Average"
  elif (x > 180):
    return "High"

Doe mee aan het gesprek

Aangemelde leden kunnen berichten plaatsen, updates volgen en meer. Nieuw hier? Registreer een gratis account.

Snelle Links

Community Help
Find useful guides, FAQs, and documents to help you navigate and make the most of Esri Community.