Skip to content

🔧 update image and bioconda container for VueGen to latest version #9201

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

Open
wants to merge 19 commits into
base: master
Choose a base branch
from
+58 −35
Open

🔧 update image and bioconda container for VueGen to latest version #9201

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
fd604a1
🔧 update image and bioconda container to latest version
enryH Oct 16, 2025
a1d39a8
✅ update test snapshots
enryH Oct 16, 2025
3b4ebd1
Merge branch 'master' into update_vuegen
famosab Oct 17, 2025
5eb02c4
:bug: fix display of version of vuegen
enryH Oct 20, 2025
8de97ed
Merge branch 'master' into update_vuegen
famosab Oct 20, 2025
9b17dc0
:art: display versions.yml content in snapshots
enryH Oct 20, 2025
0956008
Merge branch 'master' into update_vuegen
enryH Oct 22, 2025
b8e8add
🔧 add Dockerfile to install lastet PyPI vuegen version
enryH Oct 22, 2025
6bf04c9
🚧 add wave containers
enryH Oct 22, 2025
9fe09f6
:fire: remove README again
enryH Oct 24, 2025
d9f4c67
Merge branch 'master' into update_vuegen
enryH Oct 24, 2025
3493280
:fire: remove Dockerfile again
enryH Oct 24, 2025
e9b110f
Merge branch 'master' into update_vuegen
enryH Oct 28, 2025
f25c416
🚧 try to follow Mahesh's advice
enryH Oct 28, 2025
15c1836
Merge branch 'master' into update_vuegen
enryH Oct 28, 2025
f71f78c
:bug: add explicit cache directory
enryH Oct 29, 2025
68f79c7
Merge branch 'master' into update_vuegen
enryH Oct 29, 2025
2791af4
🔧 bump to Python 3.12 and remove channel prefix
enryH Oct 29, 2025
58ef775
:wrench: specify singularity image with https
enryH Oct 29, 2025
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
2 changes: 1 addition & 1 deletion modules/nf-core/vuegen/environment.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,5 +4,5 @@ channels:
- bioconda
- conda-forge
dependencies:
- bioconda::vuegen=0.3.2
- bioconda::vuegen=0.5.1
- python=3.11
2 changes: 1 addition & 1 deletion modules/nf-core/vuegen/main.nf
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
process VUEGEN {
label 'process_single'
conda "${moduleDir}/environment.yml"
container "dtu_biosustain_dsp/vuegen:v0.3.2-nextflow"
container "dtu_biosustain_dsp/vuegen:v0.5.1-nextflow"
Copy link
Contributor

@famosab famosab Oct 20, 2025

Choose a reason for hiding this comment

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

Can we get this container moved to the nf-core quay.io please?

Copy link
Author

@enryH enryH Oct 20, 2025

Choose a reason for hiding this comment

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

Do I need special permission? are there instructions for moving it?

Copy link
Contributor

@mashehu mashehu Oct 20, 2025

Choose a reason for hiding this comment

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

a core member (like me) needs to do it. could you prepare it according to step 2 here please https://nf-co.re/docs/checklists/community_governance/core_team#making-custom-docker-containers-for-modules

Copy link
Author

@enryH enryH Oct 22, 2025

Choose a reason for hiding this comment

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

ok. We have a Dockerfile and I will add it to the module here, see: Multiomics-Analytics-Group/nf-vuegen#1

Copy link
Author

@enryH enryH Oct 22, 2025

Choose a reason for hiding this comment

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

I see that most core modules 'only' have a conda environment file. Would you then auto-build the docker container (I like wave, but so far i think one has to either use wave or docker, and cannot 'just' use wave for modules which do not have their own docker images)

Copy link
Contributor

@mashehu mashehu Oct 22, 2025

Choose a reason for hiding this comment

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

if you have everything in a conda env file, you can just use seqera containers to build docker and singularity images: https://nf-co.re/docs/tutorials/nf-core_components/using_seqera_containers

Copy link
Author

@enryH enryH Oct 24, 2025

Choose a reason for hiding this comment

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

done.


input:
val input_type
Expand Down
14 changes: 7 additions & 7 deletions modules/nf-core/vuegen/tests/main.nf.test.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,11 +59,11 @@
{
"output_folder": true,
"output_files": {
"Homepage.py": "Homepage.py:md5,483a41696c225663772d68ff2d664493",
"report_manager.py": "report_manager.py:md5,ed15eefa722b7a2f08cca016c0c8e28d",
"All_Formats.py": "e2a56e761eb537da8f9809cbc0b63a9d",
"Interactive_Plots.py": "3198216240ef9b86d5b1ed722bd9b31c",
"Static_Plots.py": "931fd04e2e49cb14d169747b6d5bbd1e"
"Homepage.py": "Homepage.py:md5,a46395bbdfb127463d51d8f601f2ad83",
"report_manager.py": "report_manager.py:md5,a683071ba8bc6e371c8617ef9c3fb84e",
"All_Formats.py": "1f61acc3ca34f6ff941c109d93570815",
"Interactive_Plots.py": "4857dc511178812e93c2037c8bbe04f4",
"Static_Plots.py": "ae9c2e46ae9a9e0e5e98fcce87cd7f80"
},
"versions": [
"versions.yml:md5,c3e7ebc433dcacf3d15e1113fd44f5ec"
Expand All @@ -72,8 +72,8 @@
],
"meta": {
"nf-test": "0.9.2",
"nextflow": "24.10.5"
"nextflow": "24.10.6"
},
"timestamp": "2025-04-22T09:03:48.375488"
"timestamp": "2025-10-16T15:32:46.503388"
}
}
Loading