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

WIP: Compact interval fixes #28

Open
wants to merge 82 commits into
base: main
Choose a base branch
from
Open

WIP: Compact interval fixes #28

wants to merge 82 commits into from

Conversation

Olllom
Copy link
Contributor

@Olllom Olllom commented Oct 6, 2021

@jonkhler Which of these do we still need?

distribution.py

  • class _SloppyUniform
  • class SloppyUniform

modulify.py

  • def distribution_module

dense.py

  • class SirenDenseNet

approx_inverse.py

  • # class DifferentiableApproximateInverse
  • # class WrapFlowWithInverse
  • # class TransformerToFlowAdapter
  • # class WrapTransformerWithInverse
  • # # class TransformerApproximateInverse
  • class WrapCDFTransformerWithInverse
  • class GridInversion

bisection.py

  • def bisection_inverse
  • def filter_grid
  • def find_interval
  • class BisectionRootFinder

newton.py

  • def stable_newton_rhapson

compact.py

  • def polynomial_ramp
  • def smooth_ramp_pow2
  • def smooth_ramp_pow1
  • def smooth_ramp_pow_beta
  • def generalized_sigmoid_transform
  • def affine_sigmoid_transform
  • def mixture_cdf_transform
  • class ConditionalRamp
  • class PowerRamp
  • class SmoothRamp
  • class SmoothRampWithTrainableExponent
  • class AffineSigmoidComponents
  • class AffineSigmoidComponentInitGrid
  • class MixtureCDFTransformer
  • class ConstrainedSigmoidComponents
  • class ConstantConstrainedSigmoidComponents
  • class UniformIntervalConstrainedSigmoidComponents
  • class ConstrainedBoundaryCDFTransformer

moebius.py

  • def moebius_transform
  • def uniform_to_complex
  • def complex_to_uniform
  • def complex_multiply
  • def complex_conjugate
  • def full_moebius
  • class MoebiusComponents

sigmoid.py

  • def non_compact_affine_sigmoid_transform
  • def rescaled_non_compact_affine_sigmoid_transform
  • class NonCompactAffineSigmoidComponents

spline.py

  • changes in ConditionalSplineTransformer (elementwise dlogp)

@lgtm-com
Copy link
Contributor

lgtm-com bot commented Oct 6, 2021

This pull request introduces 8 alerts and fixes 1 when merging 191465c into 3f23452 - view on LGTM.com

new alerts:

  • 3 for Unused import
  • 2 for Unused local variable
  • 2 for 'import *' may pollute namespace
  • 1 for Variable defined multiple times

fixed alerts:

  • 1 for Unused import

@Olllom Olllom mentioned this pull request Jan 20, 2022
@lgtm-com
Copy link
Contributor

lgtm-com bot commented Jan 20, 2022

This pull request introduces 8 alerts and fixes 1 when merging 8be78aa into ee620a3 - view on LGTM.com

new alerts:

  • 3 for Unused import
  • 2 for Unused local variable
  • 2 for 'import *' may pollute namespace
  • 1 for Variable defined multiple times

fixed alerts:

  • 1 for Unused import

@Olllom
Copy link
Contributor Author

Olllom commented Jan 20, 2022

@jonkhler's jax2torch implementation (#40) of smooth flows is much more efficient.
Once #40 is merged, we'll close the present PR and only maintain the jax version of smooth flows.

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.

2 participants