search

uberduck-ai / uberduck-ml-dev

ML models for Uberduck
Apache License 2.0
377 stars 61 forks source link

Nbdev migration #90

Closed sjkoelle closed 2 years ago

sjkoelle commented 2 years ago

Remove nbdev from repo because its annoying.

williamluer commented 2 years ago

Do we need all 16 LJ files?

williamluer commented 2 years ago

Nice job moving everything over to pytest. Just left some comments around cleanup and making sure tests have the asserts

sjkoelle commented 2 years ago

Yeah I think we should just keep the LJ files - useful to have a real batch if we ever wanted to say drive loss to 0.

sjkoelle commented 2 years ago

K think everything is responded to - some minor changes to the asserts and clean up