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.

Sign up for GitHub

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

Add pyhmmsearch #51419

Closed
wants to merge 12 commits into from
Closed

Add pyhmmsearch #51419

Show file tree
Hide file tree
Changes from 8 commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
410aef6
Add pyhmmsearch
apcamargo Oct 16, 2024
d50a150
Add noarch
apcamargo Oct 16, 2024
5a69316
Change entrypoints
apcamargo Oct 16, 2024
4498d70
Change entrypoints
apcamargo Oct 16, 2024
306ba88
edit run_exports
mencian Oct 16, 2024
d1d836d
Change entrypoints
apcamargo Oct 16, 2024
63e7ca7
try just import tests
mencian Oct 16, 2024
88ac77a
Merge branch 'master' into pyhmmsearch
mencian Oct 17, 2024
f49a656
Merge branch 'master' into pyhmmsearch
mencian Oct 17, 2024
be3b62d
Update version
apcamargo Oct 18, 2024
5d13e07
Merge branch 'bioconda:master' into pyhmmsearch
apcamargo Oct 18, 2024
2ec368e
Add version constraints
apcamargo Oct 18, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
48 changes: 48 additions & 0 deletions recipes/pyhmmsearch/meta.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,48 @@
{% set name = "pyhmmsearch" %}
{% set version = "2024.7.29" %}

package:
name: {{ name|lower }}
version: {{ version }}

source:
url: https://pypi.org/packages/source/{{ name[0] }}/{{ name }}/pyhmmsearch-{{ version }}.tar.gz
sha256: 5134759a9053a9c3f52ab6c3d672094915292d431f734074d1d33021138c75ec

build:
script: {{ PYTHON }} -m pip install . -vvv --no-deps --no-build-isolation --no-cache-dir
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue

Reconsider the use of --no-deps --no-build-isolation flags.

The build script uses --no-deps --no-build-isolation flags, which may prevent necessary dependencies from being installed during the build process. This could lead to build failures or runtime issues if required packages are not present in the environment.

Consider removing these flags unless there's a specific reason for their inclusion. If they are necessary, please provide a justification in a comment.

-  script: {{ PYTHON }} -m pip install . -vvv --no-deps --no-build-isolation --no-cache-dir
+  script: {{ PYTHON }} -m pip install . -vvv
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
script: {{ PYTHON }} -m pip install . -vvv --no-deps --no-build-isolation --no-cache-dir
script: {{ PYTHON }} -m pip install . -vvv

number: 0
noarch: python
#entry_points:
#- pyhmmsearch.py=pyhmmsearch:main
#- reformat_pyhmmsearch.py=reformat_pyhmmsearch:main
#- serialize_hmm_models.py=serialize_hmm_models:main
run_exports:
- {{ pin_subpackage("pyhmmsearch", max_pin=None) }}

requirements:
host:
- python
- pip
run:
- python
- pyhmmer >=0.10.12
- pandas
- tqdm
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🛠️ Refactor suggestion

Consider adding version constraints for pandas and tqdm.

While you've specified a version constraint for pyhmmer, the pandas and tqdm dependencies don't have any version specifications. To ensure compatibility and reproducibility, it's generally a good practice to specify version constraints for all dependencies.

Consider adding version constraints like this:

  run:
    - python
    - pyhmmer >=0.10.12
-   - pandas
-   - tqdm
+   - pandas >=1.0.0
+   - tqdm >=4.0.0

Please adjust the version numbers according to the minimum versions required by your package.

📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
requirements:
host:
- python
- pip
run:
- python
- pyhmmer >=0.10.12
- pandas
- tqdm
requirements:
host:
- python
- pip
run:
- python
- pyhmmer >=0.10.12
- pandas >=1.0.0
- tqdm >=4.0.0


test:
imports:
- pyhmmsearch
commands:
#- pyhmmsearch.py -h
#- reformat_pyhmmsearch.py -h
#- serialize_hmm_models.py -h
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🛠️ Refactor suggestion

Enhance test coverage with more comprehensive tests.

The current test configuration only checks if the package can be imported. While this is a good start, more comprehensive tests could provide better assurance of package functionality. Consider adding tests that:

  1. Execute the scripts with sample inputs
  2. Verify expected outputs
  3. Check for specific functionalities

For example, you could add:

  commands:
    - pyhmmsearch.py -h
    - reformat_pyhmmsearch.py -h
    - serialize_hmm_models.py -h
    - python -c "import pyhmmsearch; assert pyhmmsearch.__version__ == '{{ version }}'"
    # Add more specific functional tests here

This addition checks if the scripts can be executed and if the package version is correct. Add more specific tests based on the package's functionality.


about:
home: "https://github.com/new-atlantis-labs/pyhmmsearch-stable"
summary: "Fast implementation of HMMSEARCH optimized for high-memory systems using PyHmmer."
license: MIT
license_family: MIT
license_file: LICENSE
dev_url: "https://github.com/new-atlantis-labs/pyhmmsearch-stable"
doc_url: "https://github.com/new-atlantis-labs/pyhmmsearch-stable/blob/main/README.md"
10 changes: 4 additions & 6 deletions sopa/meta.yaml → recipes/sopa/meta.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ build:
entry_points:
- sopa = sopa.main:app
noarch: python
script: {{ PYTHON }} -m pip install . -vv --no-deps --no-build-isolation
script: {{ PYTHON }} -m pip install . -vvv --no-deps --no-build-isolation --no-cache-dir
number: 0

requirements:
Expand All @@ -37,7 +37,7 @@ requirements:
- toml >=0.10.2
- loompy >=3.0.7
- tangram-sc >=1.0.4
- snakemake >=7.32.4,<8.0.0,<8.1.3
- snakemake-minimal >=7.32.4,<8.0.0,<8.1.3
- pulp >=2.3.1,<3.0.0,<2.8
- tiffslide >=2.3.1,<3.0.0
- black >=22.8.0,<23.0.0
Expand All @@ -54,16 +54,14 @@ test:
imports:
- sopa
commands:
- pip check
- sopa --help
requires:
- pip

about:
home: https://gustaveroussy.github.io/sopa
summary: Spatial-omics pipeline and analysis
summary: "Spatial-omics pipeline and analysis."
license: BSD-3-Clause
license_file: LICENSE
dev_url: https://gustaveroussy.github.io/sopa

extra:
recipe-maintainers:
Expand Down