diff --git a/coverage.xml b/coverage.xml index 09e336fc..80fa6a1b 100644 --- a/coverage.xml +++ b/coverage.xml @@ -1,6 +1,6 @@ - - + + /Users/will/Library/CloudStorage/Dropbox/research/macro/pyscope @@ -19,13 +19,12 @@ - - + @@ -33,7 +32,7 @@ - + @@ -66,65 +65,37 @@ - - - - + - + - - - - - - - - - - - - - - - - - - - - - + + + - - - - - - - @@ -140,9 +111,7 @@ - - - + @@ -181,14 +150,12 @@ - - - + @@ -227,7 +194,6 @@ - @@ -241,15 +207,13 @@ - + - - - + @@ -260,278 +224,157 @@ - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - @@ -546,49 +389,34 @@ - - - - - - - - - - - - - - + - - @@ -598,41 +426,23 @@ - - - - - - - - - - - - - - - - - - @@ -646,92 +456,53 @@ - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - @@ -746,19 +517,12 @@ - - - - - - - @@ -773,54 +537,33 @@ - - - - - - - - - - - - - - - - - - - - - - + @@ -831,74 +574,40 @@ - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - @@ -912,52 +621,32 @@ - - - - - - - - - - - - - - - - - - - - @@ -972,11 +661,9 @@ - - - + @@ -987,42 +674,24 @@ - - - - - - - - - - - - - - - - - - - + @@ -1033,293 +702,171 @@ - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + @@ -1329,11 +876,9 @@ - - @@ -1341,14 +886,10 @@ - - - + - - @@ -1434,7 +975,7 @@ - + @@ -1457,78 +998,51 @@ - - + - - + - - - + - - - - - - - + + - - - - + - - - - - + + - - - - - - - + + - - - - - + + - - + - - - + - @@ -1610,7 +1124,7 @@ - + @@ -1620,7 +1134,6 @@ - @@ -1680,7 +1193,6 @@ - @@ -1698,113 +1210,49 @@ - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + @@ -1814,29 +1262,23 @@ - - - - - - - + @@ -1852,35 +1294,19 @@ - - - - - - - - - - - - - - - - @@ -1890,28 +1316,18 @@ - - - - + - - - - - - - - + @@ -1925,7 +1341,6 @@ - @@ -1942,370 +1357,227 @@ - - - - - - - - - - - - - - - - + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + - + @@ -2333,9 +1605,6 @@ - - - @@ -2410,15 +1679,11 @@ - - - - - + @@ -2431,9 +1696,8 @@ - + - @@ -2446,8 +1710,7 @@ - - + @@ -2456,11 +1719,8 @@ - - - - + @@ -2469,8 +1729,6 @@ - - @@ -2480,11 +1738,9 @@ - - - - + + @@ -2496,15 +1752,9 @@ - - - - - - @@ -2614,9 +1864,7 @@ - - @@ -2630,237 +1878,126 @@ - - - - - - + - - - + - - - + - - - + - - - + - - - + - - - + - - - + - - - + - - - - + - - - - + - - - + - - - + - - - + - - - + - - - + - - - + - - - + - - - + - - - - + - - - + - - - - - + - - - - - + - - - - - + - - - - - - - - - + - - - - - - - - - + - - - - - + - - - - - + - - - - - - - - - - - - - + - - - - - + - - - - - - - - - - - @@ -2868,36 +2005,28 @@ - - - - - - + - - + - + - - @@ -2905,17 +2034,11 @@ - + - - - - - - - - + + @@ -2956,93 +2079,59 @@ - - + - - - - - - + + - - + - - - - - + + - - - - - - + - - + - - - + + - + - - - - + - - - - - - - - + - - + - - - + - - - + - - - @@ -3054,124 +2143,81 @@ - - + - - - - - + - - - + - - - - + - - - + - - - + - - - - - - - + - - - - + - - + - - - - - - - + - - - - + - - + - - - - - - - - + - - @@ -3183,184 +2229,119 @@ - - - + + - - - - + + - - - + - - - - - - + + - - - - + - + - + - + - - - + - - - - - - + - + - - - - - - - + + - - + - + - - - - - + - + - - - - - - + + - - - - + - - - - - - - - - - - - - + - - - - + - - - - - - - - - - + + - - - + - - - + - - @@ -3370,16 +2351,12 @@ - + - - - + - - - + @@ -3387,106 +2364,63 @@ - - + - - + - - + - - - - - + - - - + - - - - - + + - - - + - - - - - - - - + - - - + + - - - - - - - - - - - - - - - - - - - - - @@ -3512,32 +2446,23 @@ - - - - - - - - - @@ -3551,31 +2476,17 @@ - - - - - - - - - - - - - - @@ -3585,213 +2496,142 @@ - - - - - + - - - - - - - + - - - - - - - - - - - - + - - - - - - + - - - - - - - - - - - - + - - - - - - - - - - - - - - - - - - - - + - - - - - - - + - - - - - - - - + + - - + - - @@ -3801,32 +2641,21 @@ - - - - - - + + - - - - - - - @@ -3838,179 +2667,114 @@ - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - @@ -4018,87 +2782,53 @@ - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + @@ -4107,13 +2837,9 @@ - - - - - + @@ -4124,199 +2850,107 @@ - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + - - - - - - - - - - - - - - - - + - - + - @@ -4349,7 +2983,7 @@ - + @@ -4360,13 +2994,11 @@ - - @@ -4386,15 +3018,14 @@ - + - - + @@ -4404,7 +3035,6 @@ - @@ -4421,7 +3051,6 @@ - @@ -4432,18 +3061,13 @@ - - - + - - - - + @@ -4611,7 +3235,7 @@ - + @@ -4622,7 +3246,7 @@ - + @@ -4637,34 +3261,23 @@ - - - - + - - - - + - - + - - - - - + @@ -4691,78 +3304,50 @@ - - + - + - - - - + - - + - - - - + - - - - - - - - - - - + - - - - - - - - + + - - + - - - + - - + @@ -4785,126 +3370,65 @@ - - - + - - + - - - - - - - - - - - - - - - - + + - - - - - - - - - - - - - - - - - + - - - - - - - - - - - - - - + - - - - - - - - - - - - - - - @@ -4912,47 +3436,24 @@ - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + - - - - + + @@ -4963,29 +3464,22 @@ - - + - - + - - + - - - - + + - + - - - + + - @@ -4994,44 +3488,31 @@ - - - - - - + - - + - - - - + + - - - - + - - - + @@ -5053,108 +3534,45 @@ - - - + - - - + - - - - - - + - - - - - - - - + - - - - - - - - + - - - - - - + - - - - - - - - + - - - - - - - - - - - - - - + - - - - - - - - - - - - - - - @@ -5165,28 +3583,15 @@ - - - - - - - - - - - - - @@ -5210,27 +3615,24 @@ - - + - - + - @@ -5246,7 +3648,7 @@ - + @@ -5271,9 +3673,6 @@ - - - @@ -5282,31 +3681,23 @@ - - - + - - - - - - + - @@ -5316,11 +3707,10 @@ - - + @@ -5334,62 +3724,36 @@ - - - - + - - - - - - - - - + - - - - - - - - - - - - - - - @@ -5397,7 +3761,7 @@ - + @@ -5424,16 +3788,11 @@ - - - - - + - @@ -5449,7 +3808,6 @@ - @@ -5458,22 +3816,14 @@ - - - - - - - - - - - - + + + + - + @@ -5495,11 +3845,7 @@ - - - - - + @@ -5554,7 +3900,7 @@ - + @@ -5564,16 +3910,14 @@ - - - + - + @@ -5600,11 +3944,7 @@ - - - - - + @@ -5619,19 +3959,13 @@ - - - - - - + - @@ -5641,28 +3975,9 @@ - - - - - - - - - - - - - - - - - - - @@ -5670,23 +3985,20 @@ - - - + - + - - + @@ -5706,8 +4018,7 @@ - - + @@ -5730,11 +4041,9 @@ - - + - - + @@ -5746,14 +4055,10 @@ - - + - - - + - @@ -5762,15 +4067,12 @@ - - + - - + - + - @@ -5778,11 +4080,9 @@ - - + - - + @@ -5790,15 +4090,12 @@ - - + - - + - + - @@ -5806,15 +4103,12 @@ - - + - - + - + - @@ -5822,62 +4116,42 @@ - - + - - + - + - - - + - - - + - - + - - - + - - - + - + - - - + - - - - - - - - + @@ -5886,7 +4160,7 @@ - + @@ -5902,58 +4176,32 @@ - + - - - - - + + - - - - - - - - - - - - - - - - - - - - - - + + + + - + - - - - - @@ -5968,16 +4216,14 @@ - + - - + - @@ -5985,23 +4231,20 @@ - + - - + - - + - @@ -6016,18 +4259,13 @@ - - - + - - - - - + + @@ -6036,9 +4274,8 @@ - + - @@ -6062,15 +4299,10 @@ - - - - - - + + - - + @@ -6094,26 +4326,20 @@ - - - - - + - - + - - + @@ -6138,90 +4364,54 @@ - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + @@ -6287,17 +4477,13 @@ - - - - + - - + @@ -6309,155 +4495,107 @@ - + - - - - - - - - + - - + - - + - - + - - + - - + - - + - - + - - + - - - - - - - - + + + - - + - - - + - - - - - + + - - - + - - - + - - - - - + - - + - - - - + - - - - + - - + - - + - - - - + - - - + @@ -6515,32 +4653,10 @@ - - - - - - - - - - - - - - - - - - + - - - - - @@ -6554,86 +4670,55 @@ - + - - - - - + - - - - - - - + - - - + - - - - - - - + - - - + - + - - - - - + - - - - + + - - - - - - - - + + + @@ -6649,35 +4734,22 @@ - - - - - - - - + + + - - - - + - - - - + - - - + @@ -6686,8 +4758,7 @@ - - + @@ -6695,19 +4766,6 @@ - - - - - - - - - - - - - @@ -6715,21 +4773,13 @@ - - - - - - - - + + + - - - - + @@ -6742,7 +4792,6 @@ - @@ -6751,46 +4800,24 @@ - - - - - - - - - - - - - - + + - - - - + - - - + - - - + - - - @@ -6847,7 +4874,6 @@ - @@ -6866,7 +4892,6 @@ - @@ -6895,7 +4920,7 @@ - + @@ -6919,11 +4944,8 @@ - - - - + @@ -6941,13 +4963,11 @@ - + - - @@ -6955,30 +4975,21 @@ - - - - - + - - - - - @@ -6986,11 +4997,9 @@ - - @@ -7007,72 +5016,51 @@ - - - - - - - - - - - - - + + + - - - + - - - - - + + + - - - - - - + + + - - - - - - + + + - @@ -7083,8 +5071,8 @@ - - + + @@ -7094,7 +5082,7 @@ - + @@ -7189,22 +5177,16 @@ - - + - - + - - - + - - @@ -7245,12 +5227,10 @@ - - + - - + @@ -7258,8 +5238,7 @@ - - + @@ -7305,35 +5284,26 @@ - + - - + - - + - - + - - - - + - - + - - @@ -7358,165 +5328,102 @@ - - - + - - - - - + + - - + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - + + + + + + + + + - - - - - - - + - - + - - - + - - - + - - - - - + - + - - - - - - + + - + - - - - - - - + - - - + - - - + - - - - - + + @@ -7532,20 +5439,14 @@ - - - - - - - + @@ -7555,131 +5456,96 @@ - - - - - - + - - - - - - - + - + - - - + - - - - + + - + - - - + - + - - - - + - - + - + - - - + - + - - - + - + - - - + - + - - - + - - + - + - - - + - + - - - - - - + + + @@ -7687,18 +5553,14 @@ - - + - - - @@ -7709,56 +5571,32 @@ - - - - + - - - + - - - - + - - + - - - - - - - - - - - - - - - - - - + + + @@ -7771,8 +5609,7 @@ - - + @@ -7784,8 +5621,6 @@ - - @@ -7803,11 +5638,8 @@ - - - - - + + @@ -7817,8 +5649,7 @@ - - + @@ -7829,75 +5660,50 @@ - - + - - + - - - + - - + - - - + - - - - + - - + - - - - - - - + - - - - + - - - - - + + - - @@ -7913,31 +5719,24 @@ - - - + - - + - - - - + + - - @@ -7945,186 +5744,142 @@ - + - - - - + - - + - - - - - - + + - - + - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - - @@ -8132,56 +5887,48 @@ - - - - - - - - @@ -8190,14 +5937,12 @@ - - @@ -8205,70 +5950,60 @@ - - - - - - - - - - @@ -8276,7 +6011,6 @@ - @@ -8318,7 +6052,6 @@ - @@ -8601,7 +6334,6 @@ - @@ -8618,7 +6350,7 @@ - + @@ -8628,25 +6360,21 @@ - - + - - - + - - + @@ -8656,19 +6384,13 @@ - + - - - + - - - - @@ -8692,22 +6414,18 @@ - - + - - - - + + - - + @@ -8715,16 +6433,10 @@ - - - - - + + + - - - - @@ -8733,13 +6445,11 @@ - - @@ -8753,9 +6463,9 @@ - + - + @@ -8777,7 +6487,6 @@ - @@ -8786,12 +6495,9 @@ - - + - - @@ -8837,7 +6543,7 @@ - + @@ -9002,11 +6708,10 @@ - + - @@ -9679,8 +7384,6 @@ - - @@ -9844,17 +7547,14 @@ - - - + - diff --git a/pyproject.toml b/pyproject.toml index 97fbd0a0..419b957e 100644 --- a/pyproject.toml +++ b/pyproject.toml @@ -36,4 +36,16 @@ line-length = 88 # Don't complain about abstract methods, they aren't run: "@(abc\\.)?abstractmethod", + + # Don't complain about excepts: + "except", + + # Don't complaina about break/continue/return: + "break", + "continue", + "return", + + # Don't complain about logger statements that aren't info: + "logger\\.(debug||warning|error|critical|exception)", + ] diff --git a/pyscope/observatory/ascom_device.py b/pyscope/observatory/ascom_device.py index addb8e8b..94a2d830 100644 --- a/pyscope/observatory/ascom_device.py +++ b/pyscope/observatory/ascom_device.py @@ -29,7 +29,7 @@ def __init__(self, identifier, alpaca=False, device_type="Device", **kwargs): else: raise ObservatoryException("If you are not on Windows, you must use Alpaca") - def Action(self, ActionName, *ActionParameters): + def Action(self, ActionName, *ActionParameters): # pragma: no cover logger.debug(f"ASCOMDevice.Action({ActionName}, {ActionParameters})") return self._device.Action(ActionName, *ActionParameters) diff --git a/pyscope/observatory/html_observing_conditions.py b/pyscope/observatory/html_observing_conditions.py index 733bcbd2..a5a1c349 100755 --- a/pyscope/observatory/html_observing_conditions.py +++ b/pyscope/observatory/html_observing_conditions.py @@ -251,33 +251,33 @@ def Refresh(self): self._last_updated_numeric, ) - if cloud_cover is not None: + if cloud_cover is not None: # pragma: no cover self._cloud_cover = cloud_cover - if dew_point is not None: + if dew_point is not None: # pragma: no cover self._dew_point = dew_point - if humidity is not None: + if humidity is not None: # pragma: no cover self._humidity = humidity - if pressure is not None: + if pressure is not None: # pragma: no cover self._pressure = pressure - if rain_rate is not None: + if rain_rate is not None: # pragma: no cover self._rain_rate = rain_rate - if sky_brightness is not None: + if sky_brightness is not None: # pragma: no cover self._sky_brightness = sky_brightness - if sky_quality is not None: + if sky_quality is not None: # pragma: no cover self._sky_quality = sky_quality - if sky_temperature is not None: + if sky_temperature is not None: # pragma: no cover self._sky_temperature = sky_temperature - if star_fwhm is not None: + if star_fwhm is not None: # pragma: no cover self._star_fwhm = star_fwhm - if temperature is not None: + if temperature is not None: # pragma: no cover self._temperature = temperature - if wind_direction is not None: + if wind_direction is not None: # pragma: no cover self._wind_direction = wind_direction - if wind_gust is not None: + if wind_gust is not None: # pragma: no cover self._wind_gust = wind_gust - if wind_speed is not None: + if wind_speed is not None: # pragma: no cover self._wind_speed = wind_speed - if last_updated is not None: + if last_updated is not None: # pragma: no cover self._last_updated = last_updated def SensorDescription(self, PropertyName): diff --git a/pyscope/utils/_args_kwargs_config.py b/pyscope/utils/_args_kwargs_config.py index d3572c8c..272b6136 100644 --- a/pyscope/utils/_args_kwargs_config.py +++ b/pyscope/utils/_args_kwargs_config.py @@ -1,5 +1,5 @@ def _kwargs_to_config(kwargs): - if kwargs is None or len(kwargs) == 0: + if kwargs is None or len(kwargs) == 0: # pragma: no cover return "" string = "" for key, value in kwargs.items(): diff --git a/pyscope/utils/_function_synchronicity.py b/pyscope/utils/_function_synchronicity.py index cf0b9771..8b201a40 100644 --- a/pyscope/utils/_function_synchronicity.py +++ b/pyscope/utils/_function_synchronicity.py @@ -28,8 +28,8 @@ def _force_sync(fn): @functools.wraps(fn) def wrapper(*args, **kwargs): res = fn(*args, **kwargs) - if asyncio.iscoroutine(res): + if asyncio.iscoroutine(res): # pragma: no cover return asyncio.get_event_loop().run_until_complete(res) - return res + return res # pragma: no cover return wrapper