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

Mikado prepare freezes if one of the input files has the wrong extension #381

Closed
lucventurini opened this issue Feb 26, 2021 · 0 comments · Fixed by #383
Closed

Mikado prepare freezes if one of the input files has the wrong extension #381

lucventurini opened this issue Feb 26, 2021 · 0 comments · Fixed by #383
Assignees
Labels
Milestone

Comments

@lucventurini
Copy link
Collaborator

To reproduce:

#. Go into sample_data
#. Change trinity.gff3 to trinity.gtf in the list.txt file
#. launch snakemake daijin.

Mikado prepare will emit an error (showing that one of the child processes died due to not being able to parse the input properly) and then hang indefinitely.

@lucventurini lucventurini added this to the 2.1.2 milestone Feb 26, 2021
lucventurini added a commit that referenced this issue Feb 26, 2021
…s impossible without digging really deeply into how Python deals with pipes and streams, so we will shelve it for now.
lucventurini added a commit that referenced this issue Mar 15, 2021
# Version 2.2.0
Removed Cython from the requirements.txt file. This allows to perform the tests correctly in a Conda environment (as Conda disallows installing Cython as part of a distributed package).
As a result of this change, the preferred installation procedure from source has to be slightly amended:
- either install using `pip wheel -w dist . && pip install dist/Mikado*whl`
- or install with `python setup.py bdist_wheel` **after** having forcibly installed Cython, with `pip install Cython` or the like.

Other changes:
- Fix [#381](#381): now Mikado will be able to guess correctly 
  the input file format, instead of relying on the file name extension or user's settings. Sniffing for files 
  provided as a stream is *disabled* though.
- Fix [#382](#382): now Mikado can accept generic BED12 files 
  as input junctions, not just Portcullis junctions. This allows e.g. a user to provide a ***set of gene models*** 
  in BED12 format as sources of valid junctions.
- Fix [#384](#384): now Mikado convert deals properly with 
  unsorted GTFs/GFFs. 
- Fix [#386](#386): dealing better with unsorted GFFs/GTFs for 
  the stats utility.
- Fix [#387](#387): now Mikado will always use a static seed, 
  rather than generating a new one per call unless specifically instructed to do so. The old behaviour can still be 
  replicated by either setting the `seed` parameter to `null` (ie `None`) in the configuration file, or by 
  specifying `--random-seed` during the command invocation.
- General increase in code unit-test coverage; in particular:  
  - Slightly increased the unit-test coverage for the locus classes, e.g. properly covering the `as_dict` and `load_dict`
    methods. Minor bugfixes related to the introduction of these unit-tests.
- `Mikado.parsers.to_gff` has been renamed to `Mikado.parsers.parser_factory`.
- The code related to the transcript padding has been moved to the submodule `Mikado.transcripts.pad`, rather than 
  being part of the `Mikado.loci.locus` submodule.
- Mikado will error informatively if the scoring configuration file is malformed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants