Conversation
This comment was marked as outdated.
This comment was marked as outdated.
Sorry, something went wrong.
| raise OSError('decimals should be an integer!') | ||
|
|
||
| hours_num = angle_in_degrees*24/180 | ||
| hours_num = angle_in_degrees*24/360 |
There was a problem hiding this comment.
Would spaces between operators help with readability here? What are the style rules for this repo?
tomato-gits
left a comment
tomato-gits
left a comment
There was a problem hiding this comment.
got a test failure
________________________________________ test_angle_to_sexigesimal_360 _________________________________________
def test_angle_to_sexigesimal_360():
"""Simplest test for one crete case"""
assert angle_to_sexigesimal(360.0) == '24:0:0.000'
E AssertionError: assert '48:0:0.000' == '24:0:0.000'
E
E - 24:0:0.000
E ? -
E + 48:0:0.000
E ? +
pr_tutorial/tests/test_buggy_function.py:6: AssertionError
________________________________________ test_angle_to_sexigesimal_360 _________________________________________
def test_angle_to_sexigesimal_360():
"""Simplest test for one crete case"""
assert angle_to_sexigesimal(360.0) == '24:0:0.000'
E AssertionError: assert '48:0:0.000' == '24:0:0.000'
E
E - 24:0:0.000
E ? -
E + 48:0:0.000
E ? +
pr_tutorial/tests/test_buggy_function.py:6: AssertionError
Did you re "pip install ." at the package root? |
3 participants
i fixed the thing