Skip to content
This repository has been archived by the owner on Jun 22, 2022. It is now read-only.

refactor adapter.py #63

Closed
kamil-kaczmarek opened this issue May 24, 2018 · 2 comments
Closed

refactor adapter.py #63

kamil-kaczmarek opened this issue May 24, 2018 · 2 comments

Comments

@kamil-kaczmarek
Copy link
Member

Problem: Currently User must from steppy.adapter import Adapter, E in order to use adapters.

Refactor so that:

  • Use does not have to import E
  • add Example to docstrings

Refactor is comprehensive, so that:

  • correct the code
  • correct tests
  • correct docstrings
@kamil-kaczmarek
Copy link
Member Author

@mromaniukcdl here is the task about adapter.py refactor.
Branch is ready for you: https://github.com/minerva-ml/steppy/tree/dev-adapter

@kamil-kaczmarek
Copy link
Member Author

@mromaniukcdl closing this.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants