Giter VIP home page Giter VIP logo

Comments (4)

vladsavelyev avatar vladsavelyev commented on June 10, 2024

Can you help to reproduce this?

I re-enabled unit tests #2570 and the symlink search test is passing: https://github.com/MultiQC/test-data/blob/c3952131ec5ae7fc169577027a3c65d5326380ea/unit_tests/test_get_filelist.py#L51-L61

from multiqc.

rorynolan avatar rorynolan commented on June 10, 2024

FWIW I can confirm that multiqc can no longer find files for me after the update. This is probably it because I use nextflow which uses a lot of symlinks.

from multiqc.

rhpvorderman avatar rhpvorderman commented on June 10, 2024

I did a check of the file content searching work but most of it is just copy pastes from the old codebase. I can't find (by naked eye) any changes that would trigger this.

I tested on a symlinked directory: no issues.

I tested on a symlinked directory inside of a normal directory. No issues.

I would gladly fix any issues in the code I contributed, but I need to be able to reproduce the error in order to do so.

from multiqc.

vladsavelyev avatar vladsavelyev commented on June 10, 2024

This is probably it because I use nextflow

Right, turns out the original issue Phil had also had to do with running inside nextflow, Phil shared the workflow to reproduce: https://github.com/ewels/nf-test-multiqc

So it's actually not an issue with symlinks, but rather with the nextflow work folder being ignored in the config defaults: https://github.com/MultiQC/MultiQC/blob/main/multiqc/config_defaults.yaml#L124

fn_ignore_paths:
  - "*/work/??/??????????????????????????????" # Nextflow work directories - always same hash lengths

I'm not really sure why it worked in v1.21 - will check.

UPD:

The reason is the following. Nextflow changes into /work/d3/65b98a3d9c7f67b7f3a2f2b48b5a59 and just runs multiqc as follows:

multiqc .

v1.21 didn't expand the ., so the directory didn't match the exclusion pattern. v1.22 expands . and thus the directory is excluded.

UPD2:

Restored the v1.21 behavior of keeping config.analysis_dir relative, meaning that MultiQC will work as a Nextflow job again. Pushed a "bugfix" release, update accordingly:

pip install multiqc==1.22.1

from multiqc.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.