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

Add normalizing flows - first draft #228

Closed
wants to merge 36 commits into from
Closed

Add normalizing flows - first draft #228

wants to merge 36 commits into from

Conversation

alicjapolanska
Copy link
Collaborator

Draft implementation of normalizing flow based models. NormalizingFlow class will be deleted and RQSpline will be refactored to follow the same structure as RealNVP.

@codecov-commenter
Copy link

codecov-commenter commented May 12, 2023

Codecov Report

Patch coverage: 1.53% and project coverage change: -18.72 ⚠️

Comparison is base (0244139) 93.38% compared to head (2dea7a8) 74.66%.

❗ Current head 2dea7a8 differs from pull request most recent head 6f12d46. Consider uploading reports for the commit 6f12d46 to get more accurate results

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #228       +/-   ##
===========================================
- Coverage   93.38%   74.66%   -18.72%     
===========================================
  Files           6        8        +2     
  Lines        1028     1287      +259     
===========================================
+ Hits          960      961        +1     
- Misses         68      326      +258     
Impacted Files Coverage Δ
harmonic/flows.py 0.00% <0.00%> (ø)
harmonic/model_nf.py 0.00% <0.00%> (ø)
harmonic/evidence.pyx 94.52% <66.66%> (-0.83%) ⬇️

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@alicjapolanska alicjapolanska deleted the RQSpline branch July 21, 2023 15:15
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.

None yet

2 participants