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 printed dict at looper finish #511

Closed
donaldcampbelljr opened this issue Aug 19, 2024 · 1 comment
Closed

Remove printed dict at looper finish #511

donaldcampbelljr opened this issue Aug 19, 2024 · 1 comment
Milestone

Comments

@donaldcampbelljr
Copy link
Contributor

Currently looper prints a statement in dict format for the commands/jobs submitted:

Looper finished
Samples valid for job generation: 2 of 2
{'Pipestat compatible': False, 'Commands submitted': '2 of 2', 'Jobs submitted': 2}

I believe this was originally implemented such that looper would return this dict so that our pytesting could analyze it for accuracy certain tests. However, we would like to remove seeing this in the CLI when running looper.

@donaldcampbelljr
Copy link
Contributor Author

This is now fixed by removing sys.exit(main()) and simply running main(). This does not hinder pytest results.

@donaldcampbelljr donaldcampbelljr added this to the v2.0.0 milestone Aug 20, 2024
donaldcampbelljr added a commit that referenced this issue Aug 27, 2024
donaldcampbelljr added a commit that referenced this issue Aug 27, 2024
This reverts commit 4c9549b.
@github-project-automation github-project-automation bot moved this to Done in PEP Jan 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant