You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
after the changes for input_args.json in #8883 there is a a large amount of confusion and now-obsolete code in DagmanCreator
Take this chance to restructure and cleanup.
Main goals
stop using info dictionary everywhere to mean different things
instead of JOB_SUBMIT string and format where one needs to look in 3 or fours places to find out what goes in 'Job.submit' file, use an htcondor.Submit() object and classad.quote() and avoid as much as possible ad-hoc, unexplained, string manipulations
simplify and clarify what information is really needed to submit the DAG bootstrap job and rework the informaiton transfer from DagmanCreator to DagmanSubmitter. Ideally pass an htcondor.Submit() object with the classAds common to boths submissions, which DagmanSubmitter will extend as needed
address small "historical" differences and oddities in other parts of the code which rely/require specific ads with non-standard syntax/values
The text was updated successfully, but these errors were encountered:
so far I have a new DagmanSubmitter which reproduces current Job.submit file in a simple test task.
Next step is to work on communication with DagmanSubmitter
after the changes for
input_args.json
in #8883 there is a a large amount of confusion and now-obsolete code in DagmanCreatorTake this chance to restructure and cleanup.
Main goals
info
dictionary everywhere to mean different thingsJOB_SUBMIT
string and format where one needs to look in 3 or fours places to find out what goes in 'Job.submit' file, use anhtcondor.Submit()
object andclassad.quote()
and avoid as much as possible ad-hoc, unexplained, string manipulationshtcondor.Submit()
object with the classAds common to boths submissions, which DagmanSubmitter will extend as neededThe text was updated successfully, but these errors were encountered: