Closed JiwooChloeLee closed 3 days ago
The updates encompass several areas: adding the "tabulate" module to the configuration for documentation, updating the packaging to include a new data directory and refining the dependency for numpy
, refactoring functions and color mappings in the dopant prediction logic, modifying tests to align with the updated structure, and fixing errors and adjusting visuals in a Jupyter notebook example. These changes enhance functionality, maintainability, and clarity across the project.
Files | Change Summary |
---|---|
docs/conf.py |
Added the "tabulate" module to the list of imported modules. |
setup.py |
Included the species_rep directory in data files and updated numpy dependency to "numpy<2". |
smact/dopant_prediction/doper.py |
Refactored get_dopants function to return a modified dictionary with top results and adjusted the color mapping in plot_dopants . Removed code for selecting top elements from dopants_lists . |
smact/tests/test_doper.py |
Modified test_dopant_prediction method to align with new get_dopants function. Adjusted imports accordingly. |
examples/Dopant_Prediction/doper_example.ipynb |
Updated color scheme in heatmap plot, fixed NameError for quaternary , and changed Python version information from 3.9.16 to 3.9.7 . |
🐰 In the land of code so bright,
There's a tale of change taking flight. 🍃
tabulate
joins the doc's display,While
numpy
now has a stricter say. 📏Dopants dancing in colour new,
Tests align in a merry queue.
Cheers to progress, we hop with might! 🌟
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 74.49%. Comparing base (
a8047e6
) to head (1cb2db1
). Report is 1 commits behind head on develop.:exclamation: Current head 1cb2db1 differs from pull request most recent head f3421a0
Please upload reports for the commit f3421a0 to get more accurate results.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Pull Request Template
Description
Now "Doper" class stores all possible dopants, and returns top n candidates as a return of "get_dopants" method. This is to use all candidates in plotting the periodic table heatmap with "plot_dopants" method.
Fixes # (issue)
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Test Configuration:
Reviewers
@mention individuals who you specifically want to involve in the discussion for this pull request and mention why they are needed in the discussion/why they are needed to review the pull request.
Checklist:
Summary by CodeRabbit
Documentation
New Features
species_rep
directory in the package.Bug Fixes
doper_example.ipynb
.Refactor
get_dopants
function to return a modified dictionary with top results.plot_dopants
function.Tests
DopantPredictionTest
.Chores
numpy
to require a version less than 2.