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

default value is out of synch between the nextflow.config and nextflow_schema.json #1954

Closed
jenmuell opened this issue Oct 15, 2022 · 2 comments
Assignees
Labels

Comments

@jenmuell
Copy link
Contributor

jenmuell commented Oct 15, 2022

Description of the bug

During the hackathon I was working on an issue for the parameter schema and was checking which parameters I can change in the graphical interface. Thereby I changed the default value of a parameter and recognized that the corresponding parameter in the nextflow.config was not updated.
Connected to Issue and PR

A possible solution would be to highlight this behaviour with a tooltip, to inform the user that he has to update the nextflow.config file or to disable a change of the default value.

@jenmuell jenmuell added the bug Something isn't working label Oct 15, 2022
@jenmuell jenmuell changed the title Out of synch default value for the nextflow.config and nextflow_schema.json default value is out of synch between the nextflow.config and nextflow_schema.json Oct 17, 2022
@ewels ewels added this to the 2.8 milestone Nov 29, 2022
@ewels
Copy link
Member

ewels commented Nov 29, 2022

Would be good to lint for this, but allow for the .nf-core.yml file to ignore specific parameters where this may be desired (eg. complex deferred param defaults in the config and a simple string in JSON).

@mirpedrol
Copy link
Member

Added the linting test with #2684

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: No status
Development

No branches or pull requests

3 participants