Closed ryanrdoherty closed 2 years ago
@Foxcapades and @dmgaldi Please focus on the following files in your review:
Class encapsulating DB schema: https://github.com/VEuPathDB/EdaSubsettingService/pull/63/files#diff-5dec80f580c67a45167859124aa61102a5b8285601ae362703976f805246d568
Class that loads the new collections metadata: https://github.com/VEuPathDB/EdaSubsettingService/pull/63/files#diff-b0e9d20e2e9fea0b70e76c85aa71482127d83ab280c4b1b8368000489dad6084
Parent collection class (performs some important checks and finishes populating fields based on member vars) https://github.com/VEuPathDB/EdaSubsettingService/pull/63/files#diff-2b79a9618e6d76fe6384c9ffb9e0cb2a914a7f89db3b185a1d77f56a70c1a4ab
Kinda massive PR covering collections API, two other new metadata vars (entity many-to-one and impute_zero), factoring of distribution config into separate objects shared by vars and collections, and reworking DB constants into a hierarchy.