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

Remove prince config #541

Merged
merged 6 commits into from
Dec 29, 2023
Merged

Remove prince config #541

merged 6 commits into from
Dec 29, 2023

Conversation

genericdata
Copy link
Contributor


name: Remove Prince config
about: Removal of a config of a retired cluster

The NYU Prince cluster was decommissioned on February 1st, 2021.
https://sites.google.com/nyu.edu/nyu-hpc/hpc-systems/retired-clusters#h.h5dd1cn9akn2

  • Remove your custom config file to the conf/ directory
  • Remove your documentation file to the docs/ directory
  • Remove your custom profile to the nfcore_custom.config file in the top-level directory
  • Remove your custom profile to the README.md file in the top-level directory
  • Remove your profile name to the profile: scope in .github/workflows/main.yml

@tobsecret
Copy link
Contributor

Before we remove this, let me check whether the NYU Greene cluster can use the basically same configurations as prince did and we can recycle this config. I no longer work at NYU but I do know people are still using nextflow and maybe also nf-core.

@tobsecret
Copy link
Contributor

Tagging @genericdata to verify whether these nf-core config files that were originally used for NYU Prince can be used with NYU Greene.
nf-core is a repository of ready-to-run nextflow workflows for biological data analysis and processing. It is useful for many menial biological processing tasks that were done frequently at CGSB during my time there - hence why I set up the config in the first place.

@genericdata
Copy link
Contributor Author

I submitted another pull request as nyu_hpc as a more general listing. I guess I could have renamed everything and updated the original.

@tobsecret
Copy link
Contributor

tobsecret commented Dec 25, 2023

@genericdata Oh I am a dummy - totally didn't realize this was your PR, sorry it's my vacation brain.
I looked up your github nick via finding who still works at CGSB and didn't realize you were also the author of this PR.
no this is perfectly fine.

@maxulysse
Copy link
Member

cf #607

@maxulysse maxulysse merged commit a515373 into nf-core:master Dec 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants