-
Notifications
You must be signed in to change notification settings - Fork 8
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
added maximum TR length parameter for trf #22
Conversation
Dear atotickov, Thank you so much for your generous suggestion! I noticed the issue that Thank you again for your help! |
Dear Echoring, Thank you for adding the parameter! Example setup.py file:
I would greatly appreciate it if you could add this file. Thank you! Best regards, |
Dear atotickov, Thanks for your advise. However, I haven't get the point of using Meanwhile, I tried to create a
I would greatly appreciate it if you could point out the error here. Thank you! Best regards, |
Dear @Echoring, Since quarTeT is currently unavailable in conda, utilizing it in various pipelines seems challenging from my perspective. As an alternative, installing quarTeT into the conda environment using a .yaml file is feasible, but it cannot be done without setup.py in the GitHub repository. I haven't encountered this error before, so I can't offer any specific advice. However, I'm also getting the "DeprecationWarning: pkg_resources is deprecated as an API." warning, but it doesn't seem to affect the program's functionality. As a temporary measure, I added a setup.py file specifying different packages as dependencies to the forked quarTeT branch. This helped me use the program within the conda environment. Best regards, |
Dear aaranyue,
Thank you for creating the wonderful quarTeT program! I've been testing it for the past few days and getting excellent results. However, among the currently available parameters for trf, I couldn't find the -l parameter, which is crucial for mammals whose genome assemblies contain a high amount of repetitive sequences. Without this trf parameter, it gets stuck and after several weeks of work, it might not finish. I took the liberty of adding this parameter myself. If you don't mind, please consider accepting the merge request.
With respect and gratitude,
atotickov.