Skip to content

Commit fa45696

Browse files
Fix/fractional plot bundle (mllam#222)
Co-authored-by: Simon Adamov <kode@mailbox.org>
1 parent 4826d66 commit fa45696

2 files changed

Lines changed: 5 additions & 2 deletions

File tree

CHANGELOG.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
1313

1414
- Changed the hardcoded True to a conditional check "persistent_workers=self.num_workers > 0" [\#235](https://github.com/mllam/neural-lam/pull/235) @santhil-cyber
1515

16+
- `fractional_plot_bundle` now correctly multiplies by fraction instead of dividing [
17+
\#222](https://github.com/mllam/neural-lam/pull/222) @santhil-cyber
18+
1619
### Maintenance
1720

1821
- Update PR template to clarify milestone/roadmap requirement and maintenance changes [\#186](https://github.com/mllam/neural-lam/pull/186) @joeloskarsson

neural_lam/utils.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -299,8 +299,8 @@ def fractional_plot_bundle(fraction):
299299
bundle.update(figsizes.neurips2023())
300300
original_figsize = bundle["figure.figsize"]
301301
bundle["figure.figsize"] = (
302-
original_figsize[0] / fraction,
303-
original_figsize[1],
302+
original_figsize[0] * fraction,
303+
original_figsize[1] * fraction,
304304
)
305305
return bundle
306306

0 commit comments

Comments
 (0)