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

CLI single-threaded mode #11

Closed
arctic-hen7 opened this issue Sep 15, 2021 · 1 comment
Closed

CLI single-threaded mode #11

arctic-hen7 opened this issue Sep 15, 2021 · 1 comment
Assignees
Labels
C-enhancement Category: enhancement
Milestone

Comments

@arctic-hen7
Copy link
Member

arctic-hen7 commented Sep 15, 2021

Having set up the CLI to be perfectly asynchronous and parallelized, it occurs to me that there are most definitely systems that will have difficulty with this, or scenarios in which sequential execution may be preferred. I'm thinking particularly of systems with older Intel processors.

In light of this, I think it's perfectly reasonable to create a single-threaded mode for the CLI to make execution less straining on these systems (though it will be considerably slower). This won't affect performance of the multi-threaded mode in any way, which will remain the default. I suggest specifying single threaded mode with --sequential.

@arctic-hen7 arctic-hen7 added the C-enhancement Category: enhancement label Sep 15, 2021
@arctic-hen7 arctic-hen7 self-assigned this Sep 15, 2021
@arctic-hen7 arctic-hen7 added this to the v0.2.0 milestone Sep 17, 2021
@arctic-hen7
Copy link
Member Author

This is done, though using the PERSEUS_CLI_SEQUENTIAL environment variable instead of a flag.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Category: enhancement
Projects
None yet
Development

No branches or pull requests

1 participant