search

opencdms-dev / pyopencdms-old

⭐🐍 pyopencdms aims to build a common Python API on top of multiple Climate Data Management Systems (CDMS) that use different underlying database engines
MIT License
4 stars 6 forks source link

Changes default string to empty string #100

Closed chinedu117 closed 1 year ago

chinedu117 commented 1 year ago

Closes #86

chinedu117 commented 1 year ago

Most of the IDs for the data elements (rather than code tables) will need to be UUIDs within the global climate data system. As such these need to be strings and not null / optional.

Thanks for the observation. Would it be safe to use UUID column type instead ? Then instead of str type we could use uuid type.

Also most of the foreignkeys would still be represented as optional so as to allow us to instantiate the object before attaching the necessary ids or objects. If we make the foreignkeys mandatory then we would not be able to create an instance of the class except the related object exists.