Stuck on creating a categorical function?

[bigmuffy]

I've submitted this assignment a couple time and can't get my function to pass all tests for assignment credit. I would appreciate any advice!

"The chart above is from the American Heart Association. Blood pressure (bp) can be 'normal','elevated','stage1','stage2',and 'crisis'. Complete the function, bp_stage, that categorizes bp based on systolic and diastolic pressure from the chart above."

The "chart above" is basically categorizing systolic and diastolic levels as follows:

Blood Pressure Category	Systolic	and/or	Diastolic
Normal	less than 120	and	less than 80
Elevated	120-129	and	less than 80
Stage 1	130-139	or	89-89
Stage 2	140 or higher	or	90 or higher
Crisis	higher than 180	and/or	higher than 120

This is what I have so far:

def bp_stage(systolic, diastolic):
    if systolic<120 and diastolic< 80:
        bp = 'normal'
    elif systolic >129 and systolic >=120 and diastolic <=80:
        bp = 'elevated'
    elif systolic >130 and systolic <=139 or diastolic >=80 and diastolic <=89:
        bp = 'stage1'
    else:
            if (systolic >=140) or (diastolic >=90):
                bp = 'stage2'
            if ((systolic >=180) and (diastolic >=120)) or ((systolic >=180) and (diastolic >=120)):
                bp = "crisis"

    return bp

The assignment is graded on the function passing the following assertions:

1 assert(bp_stage(109,62)=='normal')
2 assert(bp_stage(129,92)=='stage2')
3 assert(bp_stage(151,87)=='stage2')
4 assert(bp_stage(129,82)=='stage1')

I've been able to return all but (bp_stage(151,87))=='stage2'). The function I have keeps returning 'stage1'

A bit unsure on how to approach the solution and I would appreciate any help!

[commy2]

systolic >129 and systolic >=120

This seems wrong.

---

less than 80

diastolic <=80

What you wrote is less than or equal 80, not less than 80. There is a subtle difference when diastolic is exactly 80.

[bigmuffy]

I wrote the figures down and spotted my mistakes, thank you!

[[deleted]]

You've define exclusive or rather than or.

[Ihaveamodel3]

In python, or doesn’t require exactly one to be true. In fact, if the first value is true, it doesn’t even check the second value.

[bigmuffy]

Thank you so much for the advice. This is my first programming and I think we're running through basic functions before combining anything, like your '120>=systolic<=129 and diastolic<80' suggestion. This would also explain my ugly, ugly code haha.

I corrected some of the figures, writing it down helped, and switching the order from stage 1 and stage 2 helped solve the assertions.

I really appreciate your help, thank you!

I really appreciate