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

Jump to bottom

Initial add telosearchlr-v1.0.0 #51438

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
+47 −0
Open

Initial add telosearchlr-v1.0.0 #51438

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
48fdb59
Initial add telosearchlr-v1.0.0
jayramr Oct 16, 2024
cd99797
build custom telosearchlr-v1.0.0
jayramr Oct 17, 2024
fb5d7af
build custom telosearchlr-v1.0.0
jayramr Oct 17, 2024
f059587
build custom telosearchlr-v1.0.0
jayramr Oct 17, 2024
c168153
build custom telosearchlr-v1.0.0
jayramr Oct 17, 2024
6964d90
build custom telosearchlr-v1.0.0
jayramr Oct 17, 2024
19bd23e
build custom telosearchlr-v1.0.1
jayramr Oct 18, 2024
69b7c15
build custom telosearchlr-v1.0.1
jayramr Oct 18, 2024
9c1473d
Merge branch 'master' into telosearchlr-v1.0.0
jayramr 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
36 changes: 36 additions & 0 deletions recipes/telosearchlr/meta.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,36 @@
{% set name = "telosearchlr" %}
{% set version = "1.0.0" %}

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

source:
url: "https://github.com/gchchung/TeloSearchLR/archive/refs/tags/v1.0.0.tar.gz"
sha256: "37426e76cae03254a7746becd6b3174f9e0476ef13656998b93f036ec9468101"

build:
noarch: python
number: 0
script: "{{ PYTHON }} -m pip install . -vv"

requirements:
host:
- pip
- python >=3.9
run:
- python >=3.9
- Bio
- pillow
- svgutils
- plotly
- tidehunter

test:
commands:
- TeloSearchLR.py -h
Comment on lines +29 to +31
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 more comprehensive tests

While the current test command ensures that the main script is accessible and can be executed, it doesn't verify the actual functionality of the package. Consider adding more comprehensive tests to ensure the package works as expected.

For example, you could add a simple test case that processes a small sample dataset, if available:

test:
  commands:
    - TeloSearchLR.py -h
    - TeloSearchLR.py --version
    - python -c "import telosearchlr; assert telosearchlr.__version__ == '{{ version }}'"
  # If a sample dataset is available:
  # - TeloSearchLR.py -i sample_data.fastq -o test_output

This would provide more confidence in the package's functionality.

about:
home: "https://github.com/gchchung/TeloSearchLR"
license: "Academic and Non-Commercial Research Use"
license_file: LICENSE
summary: "TeloSearchLR (telomere search using long sequencing reads) is a Python script for aiding the identificaiton of telomeric repeat motifs."
Comment on lines +32 to +36
Copy link
Contributor

Choose a reason for hiding this comment

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

💡 Codebase verification

⚠️ Potential issue

License Specification Mismatch

  • The meta.yaml file specifies the license as "Academic and Non-Commercial Research Use", but the LICENSE file contains the MIT License. Please ensure that the license in meta.yaml accurately reflects the project's actual license.
🔗 Analysis chain

Please clarify the license and fix the typo in the summary

  1. The license "Academic and Non-Commercial Research Use" is somewhat unusual. Please confirm if this is the official license name. If possible, consider using a standard SPDX license identifier or provide a link to the full license text for clarity.

  2. There's a typo in the summary: "identificaiton" should be "identification".

Please correct the typo in the summary:

summary: "TeloSearchLR (telomere search using long sequencing reads) is a Python script for aiding the identification of telomeric repeat motifs."

To verify the license, please run the following script:

This will help ensure that the license in the meta.yaml file accurately reflects the project's actual license.

🏁 Scripts executed

The following scripts were executed for the analysis:

Script:

#!/bin/bash
# Description: Check the content of the LICENSE file
cat LICENSE

Length of output: 1103

Loading