Skip to content

Added Training Text for Housing Problem#183

Open
mdvsh wants to merge 3 commits into
FluxML:masterfrom
mdvsh:master
Open

Added Training Text for Housing Problem#183
mdvsh wants to merge 3 commits into
FluxML:masterfrom
mdvsh:master

Conversation

@mdvsh

@mdvsh mdvsh commented Dec 19, 2019

Copy link
Copy Markdown

As part of my task to document training text for Machine Learning Models in Flux/model-zoo.

The added text is amenable by Literate.jl and would be helpful to people who want to learn Flux via doing examples. It would also be visible in the notebook generated by Literate.jl

@prakharcode | @dhairyagandhi96
Please review.

Regards,
PseudoCodeNerd

in consideration with Literate.jl
Comment thread other/housing/housing.jl Outdated

@logankilpatrick logankilpatrick left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hey @PseudoCodeNerd did you reference any sources when making this? Is all of this content in your own original words?

@mdvsh

mdvsh commented Dec 19, 2019

Copy link
Copy Markdown
Author

I did consult some medium articles and a research paper. I'll make a References section to address them.

@mdvsh

mdvsh commented Dec 20, 2019

Copy link
Copy Markdown
Author

@logankilpatrick Can you please review it now ?
Thanks.

Comment thread other/housing/housing.jl
Comment thread other/housing/housing.jl Outdated
Comment thread other/housing/housing.jl Outdated
and used backticks for code in markdown.
@mdvsh mdvsh requested a review from DhairyaLGandhi December 20, 2019 05:00

@logankilpatrick logankilpatrick left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice work! I think that more explanation like this helps people just getting acquainted with Flux and ML.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants