F/remove sklarn

[wooohoooo]

• removed dependency on Sklearn etc, in setup.py and all the places where I could find it
• removed run() and other pieces of the API that aren't supposed to be used any more
• made sure the tests cover the use cases
• added notebook that showcases the use of the package (can be used as secondary testing)

[ayoub-assis]

Everything looks fine to me. Albeit I couldn't run the notebook and test files since I don't have an account with a device. But maybe that's something that is irrelevant to this PR. @robinvanemden

[MKaptein]

You should definitely setup your own account and create a device so you can test!

On 10 May 2022, at 09:56, ayoub-assis @.***> wrote:

Everything looks fine to me. Albeit I couldn't run the notebook and test files since I don't have an account with a device. But maybe that's something that is irrelevant to this PR. @robinvanemden https://github.com/robinvanemden — Reply to this email directly, view it on GitHub https://github.com/scailable/sclblpy/pull/49#issuecomment-1122054431, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABGHUD44BM42NMX2YSK3TT3VJIJDTANCNFSM5VIG7IVA. You are receiving this because you are subscribed to this thread.

[ayoub-assis]

• The package documentation should be regenerated: main.m.html
• This file is irrelevant now supported_models.json
• docstring of `test_run_inputs'

[ayoub-assis]

Manual Testing

• test_utils: passed
• test_bundle: passed
• test_main: passed
• Notebook works fine
  . . .
• test_jwt: FAILED saying my username and/or password don't exist

[ayoub-assis]

• test_jwt: Passed after specifying the right usermanager server was correctly specified

[robinvanemden]

Works for me locally. All issues that were found were also resolved by @ayoub-assis . Merging.