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; } } }

Python expression failing to execute

MalcolmLittle

Hello,

I'm trying to process a Python If statement using Field Calculator. What I want is to calculate the value of a string field by evaluating the values recorded in a double field. Essentially, calculating travel mode based on speed ranges (0 = stationary, 1-6 = walk, 7-19 = cycling, 20+ = driving).

However, my expression fails to process in ArcPro. Below is the expression in full, along with the output I received:

Vind meer berichten getagd met

Accepted answers

LanceCole

Malcolm Little

‌

First is you are getting a lock acquisition error, do you have multiple copies open of the table you are attempting to update?

Second use something like TravelMode = speedType (!Speed!) . "Type" is a function in Python and should not be used as a def name. It will work, but is poor practice.

Third for the Code Block, the way you have it coded "sp" is not assigned a value you have assigned the speed to "modey" when you called your function. "sp" is only defined as a global and never set. Your logic also does not handle NULL values or negative values that could be in your data.

<SPAN class="keyword token">def</SPAN> <SPAN class="token function">speedType</SPAN><SPAN class="punctuation token">(</SPAN>sp<SPAN class="punctuation token">)</SPAN><SPAN class="punctuation token">:</SPAN>
  <SPAN class="keyword token">if</SPAN> sp <SPAN class="operator token">==</SPAN> <SPAN class="number token">0</SPAN><SPAN class="punctuation token">:</SPAN>
    <SPAN class="keyword token">return</SPAN> <SPAN class="string token">"stationary"</SPAN>
  <SPAN class="keyword token">elif</SPAN> <SPAN class="number token">1</SPAN> <SPAN class="operator token"><=</SPAN> sp <SPAN class="operator token"><=</SPAN><SPAN class="number token">6</SPAN><SPAN class="punctuation token">:</SPAN>
    <SPAN class="keyword token">return</SPAN> <SPAN class="string token">"walk"</SPAN>
  <SPAN class="keyword token">elif</SPAN> <SPAN class="number token">7</SPAN> <SPAN class="operator token"><=</SPAN> sp <SPAN class="operator token"><=</SPAN> <SPAN class="number token">19</SPAN><SPAN class="punctuation token">:</SPAN>
    <SPAN class="keyword token">return</SPAN> <SPAN class="string token">"cycle"</SPAN>
  <SPAN class="keyword token">else</SPAN><SPAN class="punctuation token">:</SPAN>
    <SPAN class="keyword token">return</SPAN> <SPAN class="string token">"drive"</SPAN>
‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍<SPAN class="line-numbers-rows"><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN></SPAN>

All comments

LanceCole

Dan,

Thanks for that catch. I was looking at the code and not the logic in my responce. Adjusted my reply as well...

DanPatterson_Retired

check your conditions … if sp = 5 it should be in the range 1-6

sp <SPAN class="operator token">=</SPAN> <SPAN class="number token">5</SPAN><SPAN class="punctuation token">.</SPAN>
<SPAN class="number token">1</SPAN> <SPAN class="operator token"><</SPAN> sp <SPAN class="operator token"><=</SPAN> <SPAN class="number token">6</SPAN>  <SPAN class="comment token"># ---- good</SPAN>
<SPAN class="token boolean">True</SPAN>

<SPAN class="number token">1</SPAN> <SPAN class="operator token"><</SPAN> sp <SPAN class="operator token">>=</SPAN> <SPAN class="number token">6</SPAN>  <SPAN class="comment token"># ---- not correct</SPAN>
<SPAN class="token boolean">False</SPAN><SPAN class="line-numbers-rows"><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN></SPAN>

LanceCole

Malcolm Little

‌

First is you are getting a lock acquisition error, do you have multiple copies open of the table you are attempting to update?

Second use something like TravelMode = speedType (!Speed!) . "Type" is a function in Python and should not be used as a def name. It will work, but is poor practice.

<SPAN class="keyword token">def</SPAN> <SPAN class="token function">speedType</SPAN><SPAN class="punctuation token">(</SPAN>sp<SPAN class="punctuation token">)</SPAN><SPAN class="punctuation token">:</SPAN>
  <SPAN class="keyword token">if</SPAN> sp <SPAN class="operator token">==</SPAN> <SPAN class="number token">0</SPAN><SPAN class="punctuation token">:</SPAN>
    <SPAN class="keyword token">return</SPAN> <SPAN class="string token">"stationary"</SPAN>
  <SPAN class="keyword token">elif</SPAN> <SPAN class="number token">1</SPAN> <SPAN class="operator token"><=</SPAN> sp <SPAN class="operator token"><=</SPAN><SPAN class="number token">6</SPAN><SPAN class="punctuation token">:</SPAN>
    <SPAN class="keyword token">return</SPAN> <SPAN class="string token">"walk"</SPAN>
  <SPAN class="keyword token">elif</SPAN> <SPAN class="number token">7</SPAN> <SPAN class="operator token"><=</SPAN> sp <SPAN class="operator token"><=</SPAN> <SPAN class="number token">19</SPAN><SPAN class="punctuation token">:</SPAN>
    <SPAN class="keyword token">return</SPAN> <SPAN class="string token">"cycle"</SPAN>
  <SPAN class="keyword token">else</SPAN><SPAN class="punctuation token">:</SPAN>
    <SPAN class="keyword token">return</SPAN> <SPAN class="string token">"drive"</SPAN>
‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍‍<SPAN class="line-numbers-rows"><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN><SPAN>‍</SPAN></SPAN>

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.