search

saga-project / bliss

ATTENTION: bliss is now saga-python. Please check out the new project website for the latest version: http://saga-project.github.com/saga-python/
http://saga-project.github.com/saga-python/
Other
8 stars 4 forks source link

Bliss saga.job.Description incompatible with SAGA C++ saga.job.description #53

Closed drelu closed 12 years ago

drelu commented 12 years ago

SAGA C++:

jd.number_of_processes = "1"

Bliss requires:

jd.number_of_processes = 1

BigJob will not properly work.

oweidner commented 12 years ago

I believe Pradeep has fixed that in his implementation?

On Jun 7, 2012, at 2:15 PM, Andre Luckow wrote:

SAGA C++:

jd.number_of_processes = "1"

Bliss requires:

jd.number_of_processes = 1

BigJob will not properly work.

Reply to this email directly or view it on GitHub: https://github.com/saga-project/bliss/issues/53

pradeepmantha commented 12 years ago

I didn't fix it as I am not sure which one to follow either SAGA/ Bliss and at Pilot-API level we have total_cpu_count which is again integer.

oweidner commented 12 years ago

I think using the correct datatypes is beneficial and I've been trying to implement that in Bliss wherever possible. Would it be difficult to move towards that on the BigJob / Piplot-API level as well?

On Jun 7, 2012, at 2:44 PM, pradeepmantha wrote:

I didn't fix it as I am not sure which one to follow either SAGA/ Bliss and at Pilot-API level we have total_cpu_count which is again integer.

Reply to this email directly or view it on GitHub: https://github.com/saga-project/bliss/issues/53#issuecomment-6185999

pradeepmantha commented 12 years ago

Nope this cannot be handled on bigjob or pilot-api side.

oweidner commented 12 years ago

That's somewhat hard to believe. What exactly is the problem?!?

pradeepmantha commented 12 years ago

Similarly!

SAGA C++:

jd.environment = ['env1=val1']

Bliss requires:

jd.environment = {'env1':'val1'}

BigJob will not properly work.

oweidner commented 12 years ago

Hi Pradeep,

fixed in https://github.com/saga-project/bliss/commit/f4593746a43174172027f7a708c5d69eafbf888d.

Please test and let me know if it works!

Thanks, O.

On Jun 8, 2012, at 10:50 AM, pradeepmantha wrote:

Similarly!

SAGA C++:

jd.environment = ['env1=val1']

Bliss requires:

jd.environment = {'env1':'val1'}

BigJob will not properly work.

Reply to this email directly or view it on GitHub: https://github.com/saga-project/bliss/issues/53#issuecomment-6206221

oweidner commented 12 years ago

All,

what are the plans with regard to consolidating the Pilot API?

Currently, we have two independent (and un-synced) versions floating around in BigJob and in TROY. The one in TROY has doc-strings, the one in BigJob doesn't. There is no up-to-date HTML PIlot-API documentation that we could point users to. The one that's on the BigJob website doesn't have doc-strings and seems somewhat outdated.

We need to consolidate this as soon as possible, otherwise we won't be able to keep things in sync. I would like to add this as an agenda item for the next TROY meeting!

Thanks, Ole

The University of Edinburgh is a charitable body, registered in Scotland, with registration number SC005336.

oweidner commented 12 years ago

This has been fixed in 0.2.2