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

UI/UX: Clarify better the path to the trained_model for metrics evaluation in metrics_config.yaml #134

Open
jeipollack opened this issue Apr 18, 2024 · 0 comments
Labels
enhancement New feature or request

Comments

@jeipollack
Copy link
Contributor

jeipollack commented Apr 18, 2024

End users might not fully understand what is required as an entry for the parameter trained_model_path in the metrics_config.yaml file.

  # Fill model_params if computing metrics_only on a pre-trained model else leave blank if training model first
  # Specify path to Trained Model - Optional if model is pre-trained
  trained_model_path: 

They might think they need to provide the full path to the psf_model checkpoint files, rather than just the path to the parent directory (i.e. wf-outputs-YEARMONTHDAYHHMMSS for a previously trained model.  Either rename the parameter or improve the instructions.

@jeipollack jeipollack added the enhancement New feature or request label Apr 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant