Closed wangzyphysics closed 2 months ago
dpgen2/entrypoint/submit.py (34)
2-2: `glob` imported but unused --- 6-6: `pickle` imported but unused --- 16-16: `typing.Type` imported but unused --- 17-17: `typing.Union` imported but unused --- 20-20: `dpdata` imported but unused --- 23-23: `dflow.InputArtifact` imported but unused --- 24-24: `dflow.InputParameter` imported but unused --- 25-25: `dflow.Inputs` imported but unused --- 26-26: `dflow.OutputArtifact` imported but unused --- 27-27: `dflow.OutputParameter` imported but unused --- 28-28: `dflow.Outputs` imported but unused --- 31-31: `dflow.Steps` imported but unused --- 33-33: `dflow.argo_range` imported but unused --- 34-34: `dflow.download_artifact` imported but unused --- 38-38: `dflow.python.OP` imported but unused --- 39-39: `dflow.python.OPIO` imported but unused --- 40-40: `dflow.python.Artifact` imported but unused --- 41-41: `dflow.python.FatalError` imported but unused --- 42-42: `dflow.python.OPIOSign` imported but unused --- 43-43: `dflow.python.PythonOPTemplate` imported but unused --- 44-44: `dflow.python.TransientError` imported but unused --- 45-45: `dflow.python.upload_packages` imported but unused --- 52-52: `dpgen2.constants.default_host` imported but unused --- 65-65: `dpgen2.exploration.report.ExplorationReportTrustLevelsRandom` imported but unused --- 76-76: `dpgen2.exploration.task.CustomizedLmpTemplateTaskGroup` imported but unused --- 78-78: `dpgen2.exploration.task.ExplorationTask` imported but unused --- 79-79: `dpgen2.exploration.task.LmpTemplateTaskGroup` imported but unused --- 80-80: `dpgen2.exploration.task.NPTTaskGroup` imported but unused --- 116-116: `dpgen2.utils.bohrium_config_from_dict` imported but unused --- 117-117: `dpgen2.utils.dump_object_to_file` imported but unused --- 119-119: `dpgen2.utils.load_object_from_file` imported but unused --- 123-123: `dpgen2.utils.workflow_config_from_dict` imported but unused --- 435-435: Local variable `default_config` is assigned to but never used --- 533-533: f-string without any placeholdersdpgen2/exploration/task/caly_task_group.py (1)
126-126: f-string without any placeholdersdpgen2/superop/prep_run_calypso.py (13)
6-6: `pathlib.Path` imported but unused --- 22-22: `dflow.OutputParameter` imported but unused --- 26-26: `dflow.Workflow` imported but unused --- 27-27: `dflow.argo_len` imported but unused --- 28-28: `dflow.argo_range` imported but unused --- 29-29: `dflow.argo_sequence` imported but unused --- 30-30: `dflow.download_artifact` imported but unused --- 31-31: `dflow.upload_artifact` imported but unused --- 35-35: `dflow.python.OPIO` imported but unused --- 36-36: `dflow.python.Artifact` imported but unused --- 37-37: `dflow.python.OPIOSign` imported but unused --- 43-43: `dpgen2.constants.calypso_index_pattern` imported but unused --- 51-51: `.caly_evo_step.CalyEvoStep` imported but unusedtests/op/test_run_caly_model_devi.py (16)
1-1: `os` imported but unused --- 5-5: `ast.Slice` imported but unused --- 11-11: `unittest.mock.Mock` imported but unused --- 12-12: `unittest.mock.call` imported but unused --- 16-16: `numpy` imported but unused --- 24-24: `dflow.Step` imported but unused --- 27-27: `dflow.python.OP` imported but unused --- 29-29: `dflow.python.Artifact` imported but unused --- 30-30: `dflow.python.OPIOSign` imported but unused --- 31-31: `dflow.python.PythonOPTemplate` imported but unused --- 32-32: `dflow.python.TransientError` imported but unused --- 36-36: `dpgen2.constants.calypso_input_file` imported but unused --- 37-37: `dpgen2.constants.calypso_log_name` imported but unused --- 46-46: `dpgen2.utils.BinaryFileInput` imported but unused --- 55-55: `deepmd` imported but unused --- 58-58: Do not use bare `except`
tests/op/test_run_caly_model_devi.py (5)
`1-1`: Remove unused import `os`. --- `5-5`: Remove unused import `Slice`. --- `24-24`: Remove unused import `Step`. --- `31-31`: Remove unused import `PythonOPTemplate`. --- `32-32`: Remove unused import `TransientError`.dpgen2/exploration/task/caly_task_group.py (1)
`126-126`: The error message in the exception uses an f-string without placeholders, which is redundant. Consider simplifying the string or correctly utilizing the f-string if dynamic data is intended to be included.
Attention: Patch coverage is 94.50549%
with 5 lines
in your changes are missing coverage. Please review.
Project coverage is 84.98%. Comparing base (
8bdea14
) to head (6e447e9
).
Files | Patch % | Lines |
---|---|---|
dpgen2/entrypoint/submit.py | 0.00% | 3 Missing :warning: |
dpgen2/exploration/task/caly_task_group.py | 94.73% | 2 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
run_caly_model_devi
distanceofion
in dict format, e.g. distanceofion = {"Mg": 0.8, "Al": 0.9}, the value denotes the radius of each element, and the mindistance of different element is equal to the sum of value of element Mg and Al and multipy 0.7.name_of_element
, e.g. name_of_element = [["Mg", "Al", "Cu", "Li"], ["La", "Ce", "Ca", "Li"], ["H"]], code will randomly choose one of element in each list to set up a input file of calypso.Summary by CodeRabbit
New Features
PrepCalyModelDevi
for organizing and preparing model deviations.Bug Fixes
Tests
PrepCalyModelDevi
class to ensure its functionality.Refactor