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

Update EUKulele module #316

Closed
danilodileo opened this issue Jan 28, 2025 · 3 comments
Closed

Update EUKulele module #316

danilodileo opened this issue Jan 28, 2025 · 3 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@danilodileo
Copy link
Collaborator

Description of feature

There should be another container version of EUKulele. check it out

@danilodileo danilodileo added the enhancement New feature or request label Jan 28, 2025
@danilodileo danilodileo self-assigned this Jan 28, 2025
@danilodileo danilodileo added this to the Upgrade v1.1 milestone Jan 28, 2025
@danilodileo
Copy link
Collaborator Author

testing EUKulele 2.0.9 container

4 - 02 - 2025

  • Test failed with gtdb
  • Test failed with gtdb-r220
  • Test failed with phylodb
    Error message in log/tax_vis.out
    "One of the files, proteins.faa, in the sample directory did not complete successfully."
    Error message in log/tax_est_proteins.err
    "Traceback (most recent call last):
    File "/usr/local/lib/python3.9/site-packages/EUKulele/manage_steps.py", line 589, in manageTaxEstimation
    curr_out = place_taxonomy(tax_tab, cutoff_file, consensus_cutoff, consensus_proportion,
    File "/usr/local/lib/python3.9/site-packages/EUKulele/tax_placement.py", line 297, in place_taxonomy
    tax_table, classes = read_in_taxonomy(tax_file)
    File "/usr/local/lib/python3.9/site-packages/EUKulele/tax_placement.py", line 64, in read_in_taxonomy
    (np.issubdtype(tax_out[str(c)].dtypes, np.float_)):
    File "/usr/local/lib/python3.9/site-packages/numpy/init.py", line 397, in getattr
    raise AttributeError(
    AttributeError: np.float_ was removed in the NumPy 2.0 release. Use np.float64 instead."
    Error message in tax_est_proteins.out
    "Taxonomic estimation did not complete successfully. Check log file for details.
    AttributeError('np.float_ was removed in the NumPy 2.0 release. Use np.float64 instead.')"

I tried also containers eukulele:2.0.7--pyh78b79e7_0 and I got the same error.
It looks like something it's broken and EUKulele can't reach the end of its workflow.
I am trying the docker container on my local machine to replicate the same error.

@erikrikarddaniel
Copy link
Member

The messages you get above are perfect for the Eukulele developers. It seems it's a function call in a library that got updated. I don't think you need to spend much more time on that here, but instead report Eukulele issues to their repo and close them here, I think.

@danilodileo
Copy link
Collaborator Author

I am closing this issue as the new version in containers are currently broken, so we keep v2.0.5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants