aws / amazon-sagemaker-clarify

Fairness Aware Machine Learning. Bias detection and mitigation for datasets and models.
Apache License 2.0
66 stars 38 forks source link

Use package_data to include commit_hash instead of data_files #94

Closed Satish615 closed 3 years ago

Satish615 commented 3 years ago

Issue #, if available: Using data_files causes the COMMIT_HASH to be installed in default sys prefix for data /usr/local.

Description of changes:

Use package_data to include the data file for commit_hash instead of data_files. https://setuptools.readthedocs.io/en/latest/userguide/datafiles.html

$ ls -l 3.7/venv/lib/python3.7/site-packages/smclarify
total 8
-rw-r--r--  1 sgollapr  staff   41 Dec  1 11:09 COMMIT_HASH
-rw-r--r--  1 sgollapr  staff    0 Dec  1 11:09 __init__.py
drwxr-xr-x  3 sgollapr  staff   96 Dec  1 11:09 __pycache__
drwxr-xr-x  6 sgollapr  staff  192 Dec  1 11:09 bias
drwxr-xr-x  5 sgollapr  staff  160 Dec  1 11:09 util

Inside the containers

bash-4.2# find /  -name "COMMIT_HASH"
/usr/local/lib/python3.7/site-packages/smclarify/COMMIT_HASH

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.