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

augur.io.metadata not packaged #1096

Closed
ammaraziz opened this issue Nov 10, 2022 · 3 comments
Closed

augur.io.metadata not packaged #1096

ammaraziz opened this issue Nov 10, 2022 · 3 comments
Labels
bug Something isn't working

Comments

@ammaraziz
Copy link

Current Behavior

I am trying to import augur.io.metadata to access functions such as read_metadata_with_sequences. However these appear to not be packaged:

from augur.io.metadata import  read_metadata_with_sequences

ImportError: cannot import name 'read_metadata_with_sequences' from 'augur.io.metadata' (~/lib/python3.8/site-packages/augur/io/metadata.py)

This could be intentional but I assume if the files/functions are in the main branch it would be packaged/accessible when installed (but please correct me).

https://github.com/nextstrain/augur/blob/master/augur/io/metadata.py

Your environment: if running Nextstrain locally

  • Operating system: Linux Mint 20
  • Augur version: 18.1.2 installed via conda
@ammaraziz ammaraziz added the bug Something isn't working label Nov 10, 2022
@ammaraziz
Copy link
Author

It appears some of the other augur.io files are also not packaged:

@joverlee521
Copy link
Contributor

Hi @ammaraziz,

These are available on the main branch, but they have not been officially released yet.
You can see a diff of all of the commits that are on the main branch but not in the latest release at 18.1.2...master.

I'm planning on doing a new release this week, so these will be available in the next version of Augur.
However, I do want to warn you that these functions are not officially part of our public Python Development API, so they are potentially subject to breaking changes in the future. (See #1011 for related discussion on our API).

@ammaraziz
Copy link
Author

Hi Jover,

That makes a lot of sense. I will treat the augur.io as experimental.

Thanks again,

Ammar

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
No open projects
Development

No branches or pull requests

2 participants