-
Notifications
You must be signed in to change notification settings - Fork 27
[Breaking] Refactor ptable heatmap plotter #157
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
Merged
Merged
Changes from 1 commit
Commits
Show all changes
185 commits
Select commit
Hold shift + click to select a range
9fe3650
comment out heatmap plotter for refactor
DanielYang59 991f611
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 2875539
group args for refactor
DanielYang59 1bede8f
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] e3c976a
remove None as default
DanielYang59 ab15a46
Merge branch 'main' into refactor-ptable-heatmap
DanielYang59 8b1282d
Merge branch 'main' into refactor-ptable-heatmap
DanielYang59 54cd90d
Merge branch 'main' into refactor-ptable-heatmap
DanielYang59 d845c2c
Merge branch 'main' into refactor-ptable-heatmap
DanielYang59 19e3d3e
remove commented out legacy code
DanielYang59 a15d6ea
fix unsupported default None value for elem_colors
DanielYang59 3006cbb
add missing start_angle arg
DanielYang59 97c2fcf
add heatmap child plotter
DanielYang59 098c8b2
remove Callable type from plot_kwargs
DanielYang59 012b931
set hide_f_block default to "AUTO"
DanielYang59 e973e11
docstring tweak
DanielYang59 8b715ca
add values display functionality
DanielYang59 d9c0708
add docstring for heatmap plotter
DanielYang59 109025e
add fences for blocks in docstring
DanielYang59 7566a73
add log scale preprocessor
DanielYang59 fd9de14
warning upon illegal log value
DanielYang59 c7f9fe0
tweak docstring
DanielYang59 9052cf0
skip ptable heatmap tests
DanielYang59 d563372
add more comments
DanielYang59 82f438e
add data sum normalizer
DanielYang59 b24385d
merge heat_mode and show_values to show_values_mode
DanielYang59 8950167
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 0e4ab42
change default symbol font to white and bold
DanielYang59 8d9495f
Merge branch 'refactor-ptable-heatmap' of github.com:DanielYang59/pym…
DanielYang59 70352fe
make default cbar title font 16 and bold
DanielYang59 d5ee637
fix missing % sign in percent mode
DanielYang59 5575b73
rename show_values_mode to values_show_mode
DanielYang59 8db243f
add tile_colors to store and overwrite tile colors
DanielYang59 7f3b73d
add values f-string formatter
DanielYang59 d8e1123
add setting cbar range, need double check overwrite norm
DanielYang59 017c550
temp save: add tile_size docstring
DanielYang59 fb63e00
change border color to grey
DanielYang59 4067ebd
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] a427d02
fix value in percent mode
DanielYang59 19f16c7
remove percentage mode from normalizer
DanielYang59 6bd8e7c
merge main
DanielYang59 f066499
fix merge main
DanielYang59 941c96d
remove TODO tag
DanielYang59 8a774e5
enable cbar label format in percent mode
DanielYang59 5bdaebd
WIP: separate label fmt generate as method
DanielYang59 f78d332
WIP: add sci_notation arg
DanielYang59 2c076ef
add cbar label sci notation formatter
DanielYang59 b841af7
fix sci notation in fraction mode
DanielYang59 eaa0b87
add TODO tag for tests to migrate
DanielYang59 c32e66f
move get_cbar_label_formatter to utils
DanielYang59 2d95542
turn off useMathText for consistency with values in tile
DanielYang59 808f8bc
remove percentage arg
DanielYang59 312ea5f
add tests for get_cbar_label_formatter
DanielYang59 ad98cf5
track examples, would remove before merge
DanielYang59 c1a3287
overwrite add_child_plots method
DanielYang59 b78d2c9
access colors from tile_colors property
DanielYang59 a34562a
tiny docstring correction
DanielYang59 356c84f
Merge branch 'main' into refactor-ptable-heatmap
DanielYang59 d696f8e
remove accidentally committed test file
DanielYang59 0fb90bc
simplify sci notation of e+0
DanielYang59 acfe4ab
SUBOPTIMAL: pass tile_size to subplots figsize
DanielYang59 95cfc76
WIP, not working: add f-block vertical offset
DanielYang59 ec336a9
fix wrong arg passing
DanielYang59 211941b
separate HMapPTableProjector
DanielYang59 71d9e62
preprocess_ptable_data and downstream funcs now record anomaly
DanielYang59 1c7eada
remove TODO tag
DanielYang59 ab6c011
overload preprocess_ptable_data type
DanielYang59 beb085d
add overwrite_colors generator and unit test
DanielYang59 2d7d1fe
fix log_scale apply, and return type
DanielYang59 3856d6b
docstring tweaks
DanielYang59 ea365f9
In process: migration some of the heatmap tests
DanielYang59 dca1243
remove TODO tag
DanielYang59 70a1212
fix mypy errors
DanielYang59 f80c108
migrate most unit tests
DanielYang59 7b96203
migrate most unit tests
DanielYang59 111458a
add overwrite_colors arg
DanielYang59 80a77e2
add PTableData and unit test, not fully finished
DanielYang59 db2c25b
add drop_elements method
DanielYang59 090567d
add unit test for apply method
DanielYang59 ef02947
move PTableData to ptable and relocate unit tests
DanielYang59 8524478
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 120c219
add placeholder wrap for from_formulas method
DanielYang59 107d12e
fix nan replacement when nan and inf coexist
DanielYang59 744ca9f
fix log scale apply
DanielYang59 13712df
fix type errors
DanielYang59 9a172a5
simplify missing_strategy arg name to strategy
DanielYang59 fc9ac17
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 9ab85f3
Merge branch 'main' into refactor-ptable-heatmap
DanielYang59 d342634
mark tile_size and f-block offset as WIP
DanielYang59 2567e24
add text_colors to unify value/symbol colors and fix inconsistency
DanielYang59 7c0a6ed
fix typo in test and pick_bw_for_contrast allow color as str
DanielYang59 4fa7eb4
NEED CONFIRM: remove test for color consistency
DanielYang59 fd5742c
rename text_color to text_colors for consistency
DanielYang59 b028251
migrate asset maker for ptable-heatmap-atomic-mass
DanielYang59 bf5ec61
migrate ptable-heatmap-percent
DanielYang59 61127d1
fix colormap in percent mode
DanielYang59 f3429f2
fix log scale
DanielYang59 65188a4
uncommit example figures
DanielYang59 6877f1b
add back exclude_elem
DanielYang59 2a32719
minor clean up
DanielYang59 afaa653
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] f386048
add tests to exclude_elems
DanielYang59 a28a29f
Merge branch 'refactor-ptable-heatmap' of github.com:DanielYang59/pym…
DanielYang59 1b092ed
add overwrite_anomalies to handle nan/inf and tests
DanielYang59 371f31a
remove finished TODO
DanielYang59 9f5bdda
make exclude_elements in cline
DanielYang59 b6a042f
fix tile color and value when on_emtpy is show
DanielYang59 975e577
increate title font size
DanielYang59 8d85ab6
remove cbar range to be consistent with previous setting
DanielYang59 51c94dc
Merge branch 'main' into refactor-ptable-heatmap
DanielYang59 3b59fb2
WIP: migrate ptable_heatmap_ratio
DanielYang59 773d4a3
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 9db9a31
fix arg in other examples
DanielYang59 9b79591
Merge branch 'refactor-ptable-heatmap' of github.com:DanielYang59/pym…
DanielYang59 e340482
make overwrite_anomalies in line
DanielYang59 b93d52b
fix color mapping in log mode
DanielYang59 4ce830f
update ratios plot
DanielYang59 89c4f05
optimize norm setter
DanielYang59 08bd9ee
remove excluded elem to avoid metadata pollution and fix text color
DanielYang59 01a6d83
fix color mapping in percent/fraction mode
DanielYang59 a43912e
break into submodules
DanielYang59 4ea1843
remove debug script
DanielYang59 2da872b
Revert "break into submodules"
DanielYang59 94a5c17
WIP: attempt to merge main
DanielYang59 969fa24
fix merge main
DanielYang59 9c5f750
split ptable projector
DanielYang59 d9b87f7
add ratio plot legends
DanielYang59 980f5c3
fix NaN display in ratio plotter
DanielYang59 f98fbcb
migrate unit test for ratio plotter
DanielYang59 f705c63
Merge branch 'main' into refactor-ptable-heatmap
DanielYang59 bedbc01
Merge branch 'main' into refactor-ptable-heatmap
DanielYang59 2904d32
condense tile value, color and text_color generation
DanielYang59 c0999c6
WIP: merge tile symbol, color and value plot into add_heatmap_tiles
DanielYang59 7bde805
first working version after tile refactor
DanielYang59 4269511
fix colorbar format
DanielYang59 6dbabb4
rename args values_xx to value_xx
DanielYang59 ff2317a
add missing docstrings
DanielYang59 52bb866
regenerate asset with new implementation
DanielYang59 9d55705
finish ratio plotter migration
DanielYang59 f6bb56d
fix wrong import of ptableprojector in unit test
DanielYang59 fd3a47d
remove HMapPTableProjector into _projectors module
DanielYang59 f4aecc8
add unit tests for generate_tile_value_colors
DanielYang59 6bc1d59
fix mypy errors
DanielYang59 db2b5c9
set NaN default color as lightgrey
DanielYang59 edf5d30
Merge branch 'main' into refactor-ptable-heatmap
DanielYang59 f9ce145
add deprecated warning for args
DanielYang59 b7ffa40
remove inf_color to infty_color for backwards compatibility
DanielYang59 aa48c3a
add more deprecated args
DanielYang59 f082ed2
WIP: migrate matbench_dielectric_eda.ipynb
DanielYang59 6c53d4c
WIP: migrate mprester_ptable.ipynb
DanielYang59 dd8498a
set default on_empty to show and reset normalizer for values not present
DanielYang59 34ee706
fix matbench_dielectric_eda ipynb
DanielYang59 43b0ac4
ptable_heatmap now filter near zero values internally with a warning
DanielYang59 286ee14
fix value type
DanielYang59 39e79de
migrate mprester_ptable.ipynb
DanielYang59 165db24
migrate example boltztrap_mp
DanielYang59 cc134a0
migrate camd_2022
DanielYang59 09bd698
set default value_fmt to .0f in log mode to be large-number friendly
DanielYang59 abed0a0
fix and migrate ricci_carrier_transport
DanielYang59 92613bd
add debug tag
DanielYang59 821b3b6
migrate dielectric
DanielYang59 8c2b7db
migrate expt_gap
DanielYang59 650adc3
migrate other matbench
DanielYang59 272b90e
simplify count_elements import
DanielYang59 4075a48
fix unit test
DanielYang59 ead5a8c
remove TODO tag
DanielYang59 5799751
Merge branch 'main' into refactor-ptable-heatmap
DanielYang59 4acc671
drop not available structure
DanielYang59 0f9bdc7
add return_type for smoother migration to return figure
DanielYang59 06bf5a4
reapply value filter for percent mode
DanielYang59 3995f33
Merge branch 'main' into refactor-ptable-heatmap
DanielYang59 d43cad7
revert accidental revert during conflict merge
DanielYang59 37a2470
fix return type in test and add test for return_type
DanielYang59 0cee7cc
add debug tag
DanielYang59 6e56c9a
make copy of df
DanielYang59 c121d11
include ax example in asset
DanielYang59 3d4d65d
remove ptabledata from_formula method
DanielYang59 1ff7775
change ptable_matplotlib.py doc str section format # -> ---
janosh d784a82
rename Literal "AUTO"->"auto" for automatic text color and text forma…
janosh 0684072
df_ptable.atomic_mass->df_ptable[Key.atomic_mass]
janosh 1117617
rename HMapPTableProjector->HeatMapPTableProjector
janosh 34713e8
simplify assert isclose(a, b) -> assert a == b
janosh a4ae109
decrease filter_near_zero(tol=1e-4->1e-6)
janosh 426e16e
Merge branch 'main' into pr/DanielYang59/157
janosh 9a3940d
add isinstance(self.anomalies, dict) to remove need for typing.cast
janosh File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.