Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Preprocessing utils require os to be imported #6

Closed
jadolfbr opened this issue Jan 26, 2022 · 3 comments
Closed

Preprocessing utils require os to be imported #6

jadolfbr opened this issue Jan 26, 2022 · 3 comments
Labels
bug Something isn't working

Comments

@jadolfbr
Copy link

Installed from Pip.

~/Library/Python/3.8/lib/python/site-packages/evolocity/preprocessing/featurize_seqs.py in populate_embedding(model, seqs, namespace, use_cache, batch_size, verbose)
     69 
     70     if use_cache:
---> 71         mkdir_p('target/{}/embedding'.format(namespace))
     72         embed_prefix = ('target/{}/embedding/{}_512'
     73                         .format(namespace, model.name_,))

~/Library/Python/3.8/lib/python/site-packages/evolocity/preprocessing/utils.py in mkdir_p(path)
     66 def mkdir_p(path):
     67     try:
---> 68         os.makedirs(path)
     69     except OSError as exc:  # Python >2.5
     70         if exc.errno == errno.EEXIST and os.path.isdir(path):

NameError: name 'os' is not defined
@brianhie brianhie added the bug Something isn't working label Jan 26, 2022
@brianhie
Copy link
Owner

Weird, thanks for bringing to my attention along with the other import errors

@brianhie
Copy link
Owner

Should be fixed in https://github.com/brianhie/evolocity/releases/tag/v1.0, feel free to reopen if not.

@jadolfbr
Copy link
Author

Awesome! Thanks for taking a look at them and clearing them out!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants