dirac-institute / trailblazer

An open data repository for astronomical data products affected by satellites
MIT License
8 stars 2 forks source link

Use Sequence to avoid issues with typing.List vs list. #20

Closed DinoBektesevic closed 3 years ago

DinoBektesevic commented 3 years ago

In Python 3.9 dataclasses can seemingly define their fields using just a list whereas in Python 3.8 that's an error. Reading online about it I found out that typingSequence should preferentially be used for type annotation of arguments and typing.List for return values.

Moved the list to Sequence which will also, hopefully, avoid the pitfall of the 3.8 vs 3.9 errors. Fixes issue https://github.com/dirac-institute/trailblazer/issues/19