lewismc
commented
2 years ago Nice this is a lot of work well done Tom.
On Sat, Jan 22, 2022 at 09:43 Tom Barber @.***> wrote:
Merged #245 https://github.com/USCDataScience/sparkler/pull/245 into main.
— Reply to this email directly, view it on GitHub https://github.com/USCDataScience/sparkler/pull/245#event-5937200052, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAI4TF2NVHTGLOLH7YEALQ3UXLUC5ANCNFSM5MSHPHXA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message ID: @.***>
--
Lewis Dr. Lewis J. McGibbney Ph.D, B.Sc Skype: lewis.john.mcgibbney
What changes were proposed in this pull request?
(Please fill in changes proposed in this fix)
Is this related to an already existing issue on sparkler?
If so, mention that issue by referencing its number here.
Will it close an existing issue?
Say 'Closes #IssueNum' here.
How was this patch tested?
We are particularly interested in unit tests, integration tests, manual tests you did to ensure that the patch works as expected, so briefly describe them.
Please review https://github.com/USCDataScience/sparkler/blob/master/.github/CONTRIBUTING.md before opening a pull request.