Skip to content
This repository has been archived by the owner on May 8, 2023. It is now read-only.

Can this be generalized? #1

Closed
vsoch opened this issue May 19, 2022 · 4 comments
Closed

Can this be generalized? #1

vsoch opened this issue May 19, 2022 · 4 comments

Comments

@vsoch
Copy link

vsoch commented May 19, 2022

Heyo! I found yamp and it had an install error from pypi, so then I found your project here and tried it out - but it looks like it's hard coded for river?

inspect.getmembers(importlib.import_module("river"), inspect.ismodule)

So my question is - can this be generalized for any module since we provide the name?

@vsoch
Copy link
Author

vsoch commented May 19, 2022

Looking a bit into the code - it does look like it's hard coded for river. I'll try on my own to refactor to see if I can generalize. Thanks, sorry for the noise!

@vsoch vsoch closed this as completed May 19, 2022
@vsoch
Copy link
Author

vsoch commented May 19, 2022

oh btw I tried using pytest to run tests directly from my docstrings - life changing!!! 😆

@MaxHalford
Copy link
Owner

In general this library should be generic! But yeah I kinda copy/pasted the code directly from River, so changes have to be made.

oh btw I tried using pytest to run tests directly from my docstrings - life changing!!! 😆

I know right!

@MaxHalford
Copy link
Owner

Just removed the remaining mentions of river.

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

No branches or pull requests

2 participants