The new mapper.py file has a few areas where it can improve:

• There is a method called get_new_columns which returns an OrderedDict. This method should read from a configuration file which maps the new fields to the old fields.
• The printing should be using the util conditional print functionality.
• There is a part where we bastardize the data. This should get moved to another place.

Improve these things and then submit a pull request for this.

Context

The mapper maps an old Facebook Power Editor template into a new one. Therefore, the new template can be de-dupped using Excel's de-duping abilities.

Cause

There is a new format.

Solution

Add the following filters:

B, L, CV, GP to the docs/filters.txt and label it.

Currently, we are assuming there is a folder called archived in csv_to_xslx.py. Update this logic to confirm to safely create a folder.

Ideally, create a utility function which creates a relative directory if one does not exist.

Context

It's a general practice to have a file in the main directory of a python project called setup.py. This file is in charge of moving everything into the right place, and getting the user to a good starting point to use/develop the project.

Cause

The setup.py is recommended, but there is no need for it here. The main logic lives within image_machine/image_machine.py.

Solution

Remove setup.py.

Context

There are a lot of unused methods which should be updated or removed.

Cause

There was a lot of changes in the original ask, so there might be some methods that are no longer necessary.

Solution

Identify the candidates to be removed in the pull request. Remove the methods that are able to be removed.