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

Meaning of JobManagerInterface defaults #981

Open
dkfellows opened this issue Dec 13, 2016 · 1 comment
Open

Meaning of JobManagerInterface defaults #981

dkfellows opened this issue Dec 13, 2016 · 1 comment
Assignees
Labels
nmpi service The NMPI service back-end that interfaces to SpiNNaker. Former RemoteSpiNNaker question Further information is requested

Comments

@dkfellows
Copy link
Member

All operations in the JobManagerInterface that have default values should specify what happens with those defaults. For example, with getJobMachine (mapped a GET on the path /job/{id}/machine), at most one of the nCores, nChips and nBoards query parameters should be set, and all can be omitted. However, for the runTime query parameter there's a default but it doesn't make sense.

What should happen if the expected(?) running time is unspecified? What does that mean?

@dkfellows dkfellows added the question Further information is requested label Dec 13, 2016
@rowleya
Copy link
Member

rowleya commented Dec 14, 2016

Yes, some comments here would have been useful! It actually looks like this can't be -1, and so this is an error.

The idea of "-1" (or unspecified) run time was that it means "run forever". However, that has the implication that the quota to be used is infinite, which will almost certainly fail! That said, it is possible to not give a run time to a script and then stop the job later in the script with some criteria (e.g. receive live spikes and stop when you have enough spikes or are to have deemed to have learned some pattern).

Maybe this has to be "unsupported" on the portal at present though, as otherwise this becomes complicated; my suggestion is to remove the ability for unspecified run time for now and we can address the issue if a user requests this.

@Christian-B Christian-B added this to the 6.0.0 milestone Jul 8, 2020
@dkfellows dkfellows removed this from the 6.0.0 milestone Apr 13, 2021
@dkfellows dkfellows transferred this issue from SpiNNakerManchester/RemoteSpiNNaker Aug 5, 2023
@dkfellows dkfellows added the nmpi service The NMPI service back-end that interfaces to SpiNNaker. Former RemoteSpiNNaker label Aug 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
nmpi service The NMPI service back-end that interfaces to SpiNNaker. Former RemoteSpiNNaker question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants