radical-cybertools / radical.saga

A Light-Weight Access Layer for Distributed Computing Infrastructure and Reference Implementation of the SAGA Python Language Bindings.
http://radical-cybertools.github.io/saga-python/
Other
83 stars 34 forks source link

Refactor/lsf #753

Closed andre-merzky closed 4 years ago

andre-merzky commented 4 years ago

This simplifies the LSF adaptor and gets it in sync with other adaptors wrt. state monitoring etc.

andre-merzky commented 4 years ago

@lee212 : Will you find time for this review? I'd like to close it (it's open since 5 months). Let me know if not, and I'll try to find another reviewer...

lee212 commented 4 years ago

Yes, I will look into this.

andre-merzky commented 4 years ago

Thanks for the comments, @lee212 ! It seems the PR deteriorated quite a bit due to merges, I'll clean it up.

mtitov commented 4 years ago

@andre-merzky should this commit 0f26d61 be added here? or it will be merged later?

andre-merzky commented 4 years ago

@andre-merzky should this commit 0f26d61 be added here? or it will be merged later?

Thanks for the note! I just merged that commit into devel. Sorry for dropping the ball on this :-/ Please open a PR for anything you want to see merged, otherwise it's unfortunately likely to get delayed attention...