Skip to content

Add draft of theano introduction#2387

Merged
junpenglao merged 3 commits intomasterfrom
theano-intro
Jul 18, 2017
Merged

Add draft of theano introduction#2387
junpenglao merged 3 commits intomasterfrom
theano-intro

Conversation

@aseyboldt
Copy link
Member

No description provided.

@junpenglao
Copy link
Member

Nice! Some links to the theano docs maybe?

@@ -0,0 +1,157 @@
# pymc3 and theano
Copy link
Member

Choose a reason for hiding this comment

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

nitpicking. pymc3 should be stylized as PyMC3. Similar for other packages (Theano, NumPy and SciPy)

Copy link
Member Author

Choose a reason for hiding this comment

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

I thought it looked strange in the text, but somehow I didn't think about that :-)


Now we can tell theano to build a function that does this computation.
With a typical configuration theano generates C code, compiles it,
and creates a python function which wrapps the C function::
Copy link
Member

Choose a reason for hiding this comment

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

typo. wraps

Copy link
Member Author

Choose a reason for hiding this comment

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

done

@aseyboldt
Copy link
Member Author

Any idea where this should go in the doc hierarchy?

@aloctavodia
Copy link
Member

Maybe inside Getting started, between Getting started with PyMC3 and API quickstart

in `theano.sparse`. For a detailed overview of available operations,
see `the theano api docs <http://deeplearning.net/software/theano/library/tensor/index.html>`_.

A notable exception where theano variables *don't* behave like
Copy link
Member

Choose a reason for hiding this comment

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

"do not"

@junpenglao
Copy link
Member

Any idea where this should go in the doc hierarchy?

After VI api quick start maybe

@aseyboldt
Copy link
Member Author

An example for custom Ops is still missing, but I think we can do that later.
Otherwise I think this is ready.

@junpenglao junpenglao merged commit 99e3cd3 into master Jul 18, 2017
@junpenglao
Copy link
Member

Thanks @aseyboldt.

@junpenglao junpenglao deleted the theano-intro branch July 18, 2017 21:21
@springcoil
Copy link
Contributor

springcoil commented Jul 18, 2017 via email

@fonnesbeck
Copy link
Member

Yes, very nice. scan is another one that could use some love.

@aseyboldt
Copy link
Member Author

@fonnesbeck Definitely. But the problem with that one is that I'd like to read a good intro for it before I can write one...

@fonnesbeck
Copy link
Member

@aseyboldt Good point. In fact, I would love to attend a Theano workshop, but I've never seen one.

@springcoil
Copy link
Contributor

springcoil commented Jul 18, 2017 via email

.. math::

\text{logp}\colon \mathbb{R} \to \mathbb{R}\\
μ \mapsto \log N(μ|0, 1) + \log N(y|0, 1),

Choose a reason for hiding this comment

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

This might be a typo, it should be

μ \mapsto \log N(μ|0, 1) + \log N(y|μ, 1),

right?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, you are right, thank you.
Fix is coming up.

Choose a reason for hiding this comment

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

Glad I can help. Great introduction by the way:)

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks :-)
Fixed in #2497.

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.

7 participants