Skip to content

Project Solution: read the ReadMe file for more information #43

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
afmarquezm11 wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Project Solution: read the ReadMe file for more information #43

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
16 changes: 15 additions & 1 deletion README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,4 +16,18 @@ Follow industry standards for each data type when decided on the final format fo
* For dimensions without units, assume inches. Convert anything which isn't in inches to inches.
* For weights without units, assume pounds. Convert anything which isn't in pounds to pounds.
* UPC / Gtin / EAN should be handled as strings
* Floating point and decimal numbers should preserve as much precision as possible
* Floating point and decimal numbers should preserve as much precision as possible


## Solution

For this project, there's a main function that calls the Transform function, each line corrrespond to one "transformation". Different steps are considered since multiple updates are required.
At a high level, the steps are the following:
1. Read the csv files
2. Rename columns according to a mapping (please check utility file where helpers are located)
3. Countries are transformed into alpha3 (requirements.txt includes the library to do so)
4. Datatype transformation, where mostly datatypes are corrected, missing values are filled and more.
5. Added new columns to match the expected format for the output file
6. EAN13 column, needed a specific formatting so I added this here, it can be improved to be included within the datatype transformation method.

As a final step, it generates a csv in the same working directory as the input files called "formatted.csv" with the output data.
Loading