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

Update helloworld examples to be simple #351

Merged
merged 5 commits into from
Jul 2, 2019
Merged

Conversation

erica-chiu
Copy link
Contributor

Related issues
N/A

Describe the proposed solution
Add simple versions of the Iris and Boston examples.

Describe alternatives you've considered
N/A

Additional context

@codecov
Copy link

codecov bot commented Jul 1, 2019

Codecov Report

Merging #351 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           master     #351    +/-   ##
========================================
  Coverage   86.66%   86.66%            
========================================
  Files         335      335            
  Lines       10781    10781            
  Branches      347      563   +216     
========================================
  Hits         9343     9343            
  Misses       1438     1438

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ecc7f22...28fc56b. Read the comment docs.

```

We then define the set of raw features that we would like to extract from the data. The raw features are defined using [FeatureBuilders](/Developer-Guide#featurebuilders), and are strongly typed. TransmogrifAI supports the following basic feature types: Text, Numeric, Vector, List , Set, Map. In addition it supports many specific feature types which extend these base types: Email extends Text; Integral, Real and Binary extend Numeric; Currency and Percentage extend Real. For a complete view of the types supported see the [Type Hierarchy and Automatic Feature Engineering](/Developer-Guide#type-hierarchy-and-automatic-feature-engineering) section in the Documentation.
We then define the set of raw features that we would like to extract from the data. The raw features are defined using [FeatureBuilders](../developer-guide#featurebuilders), and are strongly typed. TransmogrifAI supports the following basic feature types: Text, Numeric, Vector, List , Set, Map. In addition it supports many specific feature types which extend these base types: Email extends Text; Integral, Real and Binary extend Numeric; Currency and Percentage extend Real. For a complete view of the types supported see the [Type Hierarchy and Automatic Feature Engineering](../developer-guide#type-hierarchy-and-automatic-feature-engineering) section in the Documentation.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I dont think ../ prefixes would work once we deploy to https://docs.transmogrif.ai
were the links broken? how did you test them?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The links are broken on the website, but only for that paragraph. I just imitated the style of the links that work in the other paragraphs. How do I link a page that will work when deployed?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Try running the docs server locally as described here - https://github.com/salesforce/TransmogrifAI/tree/master/docs#docs

Copy link
Collaborator

Choose a reason for hiding this comment

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

Oh, you're right. We actually have them as ../ elsewhere.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

They work.

@tovbinm tovbinm merged commit e6cc43f into master Jul 2, 2019
@tovbinm tovbinm deleted the ec/newhelloworld2 branch July 2, 2019 22:24
This was referenced Jul 10, 2019
@salesforce-cla
Copy link

salesforce-cla bot commented Jul 4, 2020

Thanks for the contribution! Unfortunately we can't verify the commit author(s): Erica Chiu <e***@s***.com>. One possible solution is to add that email to your GitHub account. Alternatively you can change your commits to another email and force push the change. After getting your commits associated with your GitHub account, refresh the status of this Pull Request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants