Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add Watson discovery notebook #42

Merged
merged 1 commit into from
Feb 3, 2021
Merged

Conversation

MichaelClifford
Copy link
Member

Related Issues and Dependencies

#40

This introduces a breaking change

  • Yes
  • No

This Pull Request implements

Adding the incomplete demo notebook from the Watson NLU workshop that we will modify to recreate out keyword analysis work.

Description

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@oindrillac
Copy link
Member

@MichaelClifford thanks. Just a small nit - can we add a url to the source of the notebook/workshop just for code referencing purposes? 🙂

@MichaelClifford
Copy link
Member Author

@oindrillac added the following markdown to the top of the notebook:'

image

Copy link
Member

@oindrillac oindrillac left a comment

Choose a reason for hiding this comment

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

🚢

@@ -0,0 +1,1355 @@
{
Copy link
Member

Choose a reason for hiding this comment

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

Can we load these values from environment if they are not defined or it's executed from automation?


Reply via ReviewNB

@@ -0,0 +1,1355 @@
{
Copy link
Member

Choose a reason for hiding this comment

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

I think the prints here deserve a function. Something like:

from pprint import pprint

def describe_result(heading, obj):
print(f'"{heading}"\n')
pprint(obj, indent=3)

Or you can even consider printing it as a table from a pandas data frame... The data is fortunately consistent, so pandas should be able to auto-format it.



Reply via ReviewNB

@@ -0,0 +1,1355 @@
{
Copy link
Member

Choose a reason for hiding this comment

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

This reminds me that we should update our license...

https://github.com/aicoe-aiops/mailing-list-analysis-toolkit/blob/master/LICENSE


Reply via ReviewNB

@tumido
Copy link
Member

tumido commented Feb 3, 2021

if my comments are missing the point and you're just simply trying to copy a notebook over in here, feel free to dismiss my review 🙂

@MichaelClifford
Copy link
Member Author

Hey @tumido, all good points 😄 , and we should incorporate them into upcoming PR's on this notebook. But as you pointed out, this specific PR was just to include an existing notebook that @oindrillac and I are going to modify to meet our needs for testing out some IBM Watson NLU/Discovery features.

The ultimate goal for this notebook though should be to add another "analyses" notebook to our workflow (see #40)

@oindrillac oindrillac added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Feb 3, 2021
@sesheta sesheta merged commit 9cd1e4d into aicoe-aiops:master Feb 3, 2021
@oindrillac oindrillac mentioned this pull request Feb 9, 2021
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants