Comments from Tania

Author: crandmck

docs/tasks/includes/_python-build.md

@@ -5,30 +5,30 @@ Use a `Builder` object to add a manifest to an asset.
 
 ```python
 try:
-  # Define a function to sign the claim bytes
-  # In this case we are using a pre-defined sign_ps256 method, passing in our private cert
-  # Normally this cert would be kept safe in some other location
+  # Define a function to sign the claim bytes.
+  # In this case we are using a pre-defined sign_ps256 method, passing in our private cert.
+  # Normally this cert would be kept safe in some other location.
   def private_sign(data: bytes) -> bytes:
     return sign_ps256(data, "tests/fixtures/ps256.pem")
 
-  # read our public certs into memory
+  # Read our public certs into memory.
   certs = open(data_dir + "ps256.pub", "rb").read()
 
-  # Create a signer from the private signer, certs and a time stamp service url
+  # Create a signer from the private signer, certs and a time stamp service URL.
   signer = create_signer(private_sign, SigningAlg.PS256, certs, "http://timestamp.digicert.com")
 
   # Create a builder add a thumbnail resource and an ingredient file.
   builder = Builder(manifest_json)
 
-  # Add the resource from a stream
+  # Add the resource from a stream.
   a_thumbnail_jpg_stream = open("tests/fixtures/A_thumbnail.jpg", "rb")
   builder.add_resource("image/jpeg", a_thumbnail_jpg_stream)
 
-  # Add the resource from a file
+  # Add the resource from a file.
   # The URI provided here, "thumbnail", must match an identifier in the manifest definition.
   builder.add_resource_file("thumbnail", "tests/fixtures/A_thumbnail.jpg")
 
-  # Define an ingredient, in this case a parent ingredient named A.jpg, with a thumbnail
+  # Define an ingredient, in this case a parent ingredient named A.jpg, with a thumbnail.
   ingredient_json = {
     "title": "A.jpg",
     "relationship": "parentOf", # "parentOf", "componentOf" or "inputTo"
@@ -38,19 +38,19 @@ try:
     }
   }
 
-  # Add the ingredient from a stream
+  # Add the ingredient from a stream.
   a_jpg_stream = open("tests/fixtures/A.jpg", "rb")
   builder.add_ingredient("image/jpeg", a_jpg_stream)
 
-  # At this point we could archive or unarchive our Builder to continue later.
-  # In this example we use a bytearray for the archive stream.
-  # all ingredients and resources will be saved in the archive
+  # At this point archive or unarchive Builder to continue later.
+  # This example uses a bytearray for the archive stream.
+  # All ingredients and resources are saved in the archive.
   archive = io.BytesIO(bytearray())
   builder.to_archive(archive)
   archive.seek()
   builder = builder.from_archive(archive)
 
-  # Sign the builder with a stream and output it to a stream
+  # Sign the builder with a stream and output it to a stream.
   # This returns the binary manifest data that could be uploaded to cloud storage.
   input_stream = open("tests/fixtures/A.jpg", "rb")
   output_stream = open("target/out.jpg", "wb")
@@ -63,8 +63,8 @@ except Exception as err:
 FROM SIGNING:
 
 ```python
-request_data = ... # This is the asset being signed
-content_type = ... # MIME type of the asset
+request_data = ... # This is the asset being signed.
+content_type = ... # MIME type of the asset.
 
 manifest = json.dumps({
     "title": "image.jpg",

docs/tasks/includes/_python-get-resources.md

@@ -5,19 +5,19 @@ Retrieve binary resources such as thumbnails from the manifest data, use the `re
 
 _NOTE: Need to add example of using `reader.resource_to_stream()`._
 
-```python
+```py
 try:
-# Create a reader from a file path
-reader = c2pa.Reader.from_file("path/to/media_file.jpg")
+  # Create a reader from a file path
+  reader = c2pa.Reader.from_file("path/to/media_file.jpg")
 
-# Get the active manifest.
-manifest = reader.get_active_manifest()
-if manifest != None:
+  # Get the active manifest.
+  manifest = reader.get_active_manifest()
+  if manifest != None:
 
-  # get the uri to the manifest's thumbnail and write it to a file
-  uri = manifest["thumbnail"]["identifier"]
-  reader.resource_to_file(uri, "thumbnail_v2.jpg")
+    # get the uri to the manifest's thumbnail and write it to a file
+    uri = manifest["thumbnail"]["identifier"]
+    reader.resource_to_file(uri, "thumbnail_v2.jpg") 
 
 except Exception as err:
-  print(err)
+    print(err)
 ```

docs/tasks/includes/_python-read.md

@@ -1,5 +1,7 @@
 
-Use the `Reader` object to read manifest data from a file or stream and perform validation on the manifest store. Use the `json()` method to return a JSON manifest report; If there are validation errors, the report includes a `validation_status` field.
+Use the `Reader` object to read manifest data from a file or stream and perform validation on the manifest store. 
+
+Use the `json()` method to return a JSON manifest report; If there are validation errors, the report includes a `validation_status` field.
 
 An asset file may contain many manifests in a manifest store. The most recent manifest is identified by the value of the `active_manifest` field in the manifests map.