More comments in PHT_on_FHIR and remove unused imports

Author: lukaszimmermann

examples/PHT_on_FHIR/Dockerfile

@@ -1,6 +1,9 @@
+# We use V1 RC4 for implementing the train.
+# Also, we use the -fhir image such that the fhirclient and SPAQRLWrapper modules are available
 FROM personalhealthtrain/train-api-python:1.0rc4-fhir
 LABEL maintainer="luk.zim91@gmail.com"
 
+# Copy all required source code files for the train inside the image
 COPY entrypoint.py /opt/entrypoint.py
 COPY sparql.py /opt/sparql.py
 COPY fhir.py /opt/fhir.py

examples/PHT_on_FHIR/entrypoint.py

@@ -3,6 +3,7 @@
 import sparql
 import os
 
+# The PHT API dependencies
 from pht.train import SimpleTrain
 from pht.internal import StationRuntimeInfo, ConjunctionBuilder
 from pht.train.response import RunResponse
@@ -26,6 +27,10 @@ def __init__(self):
         self.output_file = '/opt/train/output.txt'
 
     def requirements(self) -> ConjunctionBuilder:
+        # Declare the requirements of this train. The station needs to provide:
+        #   * The endpoint type (either 'FHIR' or 'SPARQL'
+        #   * The endpoint url
+        #   * The endpoint token
         return Require(self.endpoint_type) & Require(self.endpoint_url) & Require(self.endpoint_token)
 
     def model_summary(self) -> str:

examples/PHT_on_FHIR/fhir.py

@@ -1,9 +1,6 @@
 from fhirclient import client
-from fhirclient.models.patient import Patient
-from fhirclient.models import fhirsearch as S
 from datetime import datetime
 from datetime import date
-from fhirclient.models import fhirsearch
 
 
 def perform_in(srch_str, server,apiBase):