nmfs-ost / nmfs-ost/FIT_web_templating

change how including models in config

Open
#20 0 comments 0 reactions 1 assignee View on GitHub

@k-doering-NOAA is already working on this.

Since Oct 13, 2022.

build process
Dominant language
HTML
Stars
1
Forks
1
Avg merge
1h 6m
Merged PRs (30d)
7

Description

Rather than having to edit the config file (example config), It might be easier to have an off/on switch to include in the toolbox within the individual model jsons, something like: include_in_toolbox: "true" or "false". Then there is just a single file to edit when adding new models and it is easier to see explicitly whether it is included or not.

Note that would need to add some code to the python script to deal with this, though.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.