Issue #657 - Refactor Jobs interface

CHANGELOG.rst

@@ -33,6 +33,7 @@ Changed
     - update the ``Qobj`` JSON schema. (#668, #677, #703, #709)
     - update the local simulators for accepting ``Qobj`` as input. (#667)
 - Use ``get_status_job()`` for checking IBMQJob status. (#641)
+- ``Jobs API`` refactoring for simplifying its usage and some minor bug fixes. (#686)
 
 Removed
 -------

qiskit/backends/__init__.py

@@ -10,3 +10,5 @@
 from .baseprovider import BaseProvider
 from .basejob import BaseJob
 from .jobstatus import JobStatus
+from .joberror import JobError
+from .jobtimeouterror import JobTimeoutError

qiskit/backends/basejob.py

@@ -32,28 +32,8 @@ def cancel(self):
         """Attempt to cancel job."""
         pass
 
-    # Property attributes
-    #####################
-    @property
     @abstractmethod
     def status(self):
         """Get backend status dictionary"""
         pass
 
-    @property
-    @abstractmethod
-    def running(self):
-        """True if job is currently running."""
-        pass
-
-    @property
-    @abstractmethod
-    def done(self):
-        """True if call was successfully finished."""
-        pass
-
-    @property
-    @abstractmethod
-    def cancelled(self):
-        """True if call was successfully cancelled"""
-        pass

qiskit/backends/ibmq/ibmqbackend.py

@@ -53,7 +53,9 @@ def run(self, qobj):
         Returns:
             IBMQJob: an instance derived from BaseJob
         """
-        return IBMQJob(qobj, self._api, not self.configuration['simulator'])
+        job = IBMQJob(self._api, not self.configuration['simulator'], qobj=qobj)
+        job.submit()
+        return job
 
     @property
     def calibration(self):
@@ -216,7 +218,10 @@ def jobs(self, limit=50, skip=0, status=None, db_filter=None):
         job_list = []
         for job_info in job_info_list:
             is_device = not bool(self._configuration.get('simulator'))
-            job = IBMQJob.from_api(job_info, self._api, is_device)
+            job = IBMQJob(self._api, is_device,
+                          job_id=job_info.get('id'),
+                          backend_name=job_info.get('backend').get('name'),
+                          creation_date=job_info.get('creationDate'))
             job_list.append(job)
         return job_list
 
@@ -236,7 +241,10 @@ def retrieve_job(self, job_id):
         if 'error' in job_info:
             raise IBMQBackendError('failed to get job id "{}"'.format(job_id))
         is_device = not bool(self._configuration.get('simulator'))
-        job = IBMQJob.from_api(job_info, self._api, is_device)
+        job = IBMQJob(self._api, is_device,
+                      job_id=job_info.get('id'),
+                      backend_name=job_info.get('backend').get('name'),
+                      creation_date=job_info.get('creationDate'))
         return job