aws-samples / aws-iot-analytics-workshop

AWS IOT Analytics Workshop
MIT No Attribution
44 stars 31 forks source link

Few observations #2

Open asimkumarsasmal opened 5 years ago

asimkumarsasmal commented 5 years ago
  1. As documented after pre-requisites, “You DONOT have more than 3 VPC's in that region”, can we also include one more point for # of Elastic IPs in the account? The workshop creates 2 EIPs and by default AWS account has a limit of 5 EIPs, so to avoid an error - “The maximum number of addresses has been reached. (Service: AmazonEC2; Status Code: 400; Error Code: AddressLimitExceeded; xxxx”, it will be good to call out for readers. I encountered this error as my account had 4 EIPs prior to the workshop.
  2. Under VPC call out - “You DONOT have more than 3 VPC's in that region”, I would also include that the workshop creates 2 VPCs just to make it more explicit.
  3. For 1a,
    1. For widget creation, “Navigate to Modules → Widgets → Add Widget → Select smart-home → Number of Devices 1 → Submit", correct the typo for “Number of Devices” to “Number of widgets”.
  4. For 1b,
    1. Change “Click Create" to "Click Create a channel" for Channel creation
    2. Change “Click Create" to "Click Create a data store" for Data store creation
    3. Change “Click Create" to "Click Create a pipeline" for Pipeline creation
    4. Can we add a comment for the autogenerated “id” attribute under create pipeline #d just mentioning what it is?
    5. Change “Click update preview” to “Click Update preview” under create pipeline #f to match with the exact option to click
  5. For 1c,
    1. Change “Click Create → SQL Data sets → Create SQL” to “Click Create a data set → SQL Data sets → Create SQL”
    2. Can we add a line for the reason of adding an offset of “-5 seconds” and similarly the reason for converting to ISO timestamp function?
    3. Change “Timestamp → from_iso8601_timestamp(timestamp)” to “Timestamp expression → from_iso8601_timestamp(timestamp)” to match the label
  6. For QuickSight,
    1. While trying to create a dataset for IoT Analytics without choosing IoT Analytics permission during QuickSight account setup, you get a message that something went wrong and nothing else which is obviously a QuickSight trouble shooting and less to do with IoT Analytics as such. Can we call out the IoT Analytics selection during account setup in QuickSight explicit to avoid this error?
    2. Can we mention that choose the Line chart for the walkthrough? QuickSight autoselect didn't choose Line chart for me and I was wondering for a while where is the X axis :)
  7. For 2c,
    1. Can we show few sample records for inputdata.csv? An additional column global_intensity shows up for preview which is blank and just showing few results in the readme doc will help.
    2. Also, can we add any clarification for iotstore data store after batch data set is pushed to iotstore? Sounds like it overwrote the earlier data for streaming unless the preview result set is the UNION of streaming and batch?
  8. Under Clean Up,
    1. Correct typo for incurr
    2. Can we also include a step to clean up S3 buckets which the CFN stack doesn't seem to delete?
    3. Similarly, a step to clean up the channels, data store, pipelines, and data sets which were created manually
dhwalters423 commented 4 years ago

Hello @asimkumarsasmal ,

I believe we have addressed the issues you raised in the newest version of the workshop. Could you please take a look?