Skip to content
This repository has been archived by the owner on Jul 9, 2024. It is now read-only.

IoT Analytics Workshop update #4

Merged
merged 42 commits into from
Jul 24, 2019
Merged

IoT Analytics Workshop update #4

merged 42 commits into from
Jul 24, 2019

Conversation

dhwalters423
Copy link
Contributor

@dhwalters423 dhwalters423 commented Jul 12, 2019

Description of changes:

  • Significant reformatting
  • Align MQTT topic space to best practices
  • Address issues raised in Few observations #2
  • Added new feature: Customer managed S3 buckets
  • Explanations of several steps and added recap sections to summarise the workshop at certain points.

README.md Outdated

#### Create Container Data Set

A container data set allows you to automatically run your analysis tools and generate results. It brings together a SQL data set as input, a Docker container with your analysis tools and needed library files, input and output variables, and an optional schedule trigger. The input and output variables tell the executable image where to get the data and store the results.

Navigate to AWS IOT Analytics console, in the left navigation pane, choose Analyze.
Navigate to AWS IoT Analytics console, in the left navigation pane, choose Analyze.


1. Click on Analyze -> Data Sets → Create
Copy link
Contributor

@neelmitra neelmitra Jul 15, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We donot have the Analyze options anymore on the home page for IoT Analytics , can you please update it to ->
Click on Data sets -> Create

The same changes are required for Datastore, pipeline, channel etc.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi Neel, this should already be done - can you look at commit 6f7dfc8 which is the latest.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok looks good then, feel free to merge with the main branch. Thanks for your help.

Copy link
Contributor

@neelmitra neelmitra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks , i have reviewed looks good, other than 1 comment, where all the Analyze -> needs to be removed and replaced by respective components. Since the dashboard looks different now. Thanks.

@jamesiri jamesiri merged commit d9026b8 into aws-samples:master Jul 24, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants