ISSUE-75: Update build script to enable docker container contain all environments for all examples.

[CharityKithaka]

[Closes #75 : Modify the build script to facilitate the building of all environments based on the extensions on the requirements files on all modules.]

Changes

How I tested this

Notes

Checklist

• PR has an informative and human-readable title (this will be pulled into the release notes)
• Changes are limited to a single goal (no scope creep)
• Code passed the pre-commit check & code is left cleaner/nicer than when first encountered.
• Any change in functionality is tested
• New functions are documented (with a description, list of inputs, and expected output)
• Placeholder code is flagged / future TODOs are captured in comments
• Project documentation has been updated if adding/changing functionality.

[CharityKithaka]

Hey, @skrawcz kindly find the above for your review.

[skrawcz]

Thanks for the PR!

I think I might have not explained the desired end state correctly. The ideal end state is that, for each example directory, there's a virtual environment corresponding to each requirements*.txt file.

From what I can tell, this does install all requirements*.txt files, however it installs all of them into the same virtual environment for an example.

Here's some pseudo code that is close to your code now, but adjust things slightly.

find all folders with requirements files
For each folder
   for each requirements file 
       determine whether file has suffix or not (e.g. ray, dask, or none)
       remove & create new environment with suffix
       activate the environment
       install that requirements file
       deactivate environment

Shouldn't be much to change what you have to reflect the above. Let me know if you have questions or need clarification.

[CharityKithaka]

Hey, @skrawcz wanted to confirm if this issue was closed or if its something that's still pending.🙂

Charity.

[skrawcz]

Hey, @skrawcz wanted to confirm if this issue was closed or if its something that's still pending.🙂

Charity.

@CharityKithaka no this is still open!