Closed stefanwebb closed 2 years ago
@stefanwebb has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
@stefanwebb has updated the pull request. You must reimport the pull request before landing.
@stefanwebb has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
@stefanwebb has updated the pull request. You must reimport the pull request before landing.
@stefanwebb has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
@stefanwebb has updated the pull request. You must reimport the pull request before landing.
@stefanwebb has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
@stefanwebb has updated the pull request. You must reimport the pull request before landing.
@stefanwebb has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
@stefanwebb merged this pull request in facebookincubator/flowtorch@b46b1a4bbbecaf7e307b72abde280d72c5dfe5d4.
Motivation
It is tiresome to have to add new components to init.py for bijectors, distributions and parameters. We should be able to automatically generate it!
Changes proposed
Autogen for distributions was completed in a previous PR. This one completes it for parameters and bijectors.
I also uncovered and fixed a bug in how
utils.list_bijectors()
,utils.list_distributions()
, andutils.list_parameters
were working