Feature/cele 46

[dvcorreia]

• validate data to ingest
• return meaningful message of what is the problem encountered or the data that are missing

[aranega]

All looks kind of nice :). There is just some quick modifications that should be addressed. Also, the code for the ingestion should be placed in a dedicated folder at the root of the repository, it's a tool that will be used from the user machine, it's not part of the backend per say.

[dvcorreia]

@aranega, I just moved the code to the project root. Is it ok to define the package like this or do you prefer that I remove the pyproject.toml?

[aranega]

@dvcorreia Thanks for the modifications! For the project at the root as you did is just perfect!

If you have time, perhaps add a README in this folder explaining how to install the project, what's and how to use it (or at least install it while it's a WIP), and an entry in the README at the root of the project explaining that the data ingestion is done using this project and that it needs to be installed/managed manually.

[dvcorreia]

Is it ok for this to be done in https://metacell.atlassian.net/browse/CELE-78? I already have a README ready for it.

@dvcorreia Thanks for the modifications! For the project at the root as you did is just perfect!

If you have time, perhaps add a README in this folder explaining how to install the project, what's and how to use it (or at least install it while it's a WIP), and an entry in the README at the root of the project explaining that the data ingestion is done using this project and that it needs to be installed/managed manually.

[aranega]

Sure, no problem if it's in CELE-78 :)

[dvcorreia]

Latest changes:

• now I uses a colors package (colorama, don't if is the "best", but was close to what I had previously and I think it fixes some ANSI issues on windows)
• not there's a wrapper for the data validation error
• added a few tests for the find and load data functions

[aranega]

Nice work :) thanks for addressing the previous concerns. There is few modification I would suggest, nothing too huge, it's more polishing for this version.

[aranega]

All looks good to me :) thanks a lot @dvcorreia !