Closed zjgemi closed 1 month ago
dpgen2/op/prep_caly_model_devi.py
1-1: `json` imported but unused --- 2-2: `logging` imported but unused --- 3-3: `pickle` imported but unused --- 4-4: `shutil` imported but unused --- 10-10: `typing.Tuple` imported but unused --- 20-20: `dflow.python.TransientError` imported but unused --- 24-24: `dpgen2.constants.calypso_check_opt_file` imported but unused --- 25-25: `dpgen2.constants.calypso_opt_dir_name` imported but unused --- 26-26: `dpgen2.constants.calypso_run_opt_file` imported but unused --- 27-27: `dpgen2.constants.model_name_pattern` imported but unused --- 30-30: `dpgen2.exploration.task.ExplorationTaskGroup` imported but unused --- 33-33: `dpgen2.utils.BinaryFileInput` imported but unused --- 37-37: `dpgen2.utils.run_command.run_command` imported but unused
dpgen2/op/prep_caly_model_devi.py (1)
`90-94`: The modification to filter out `None` values before resolving paths is correctly implemented and enhances the robustness of the code.
Attention: Patch coverage is 12.50000%
with 7 lines
in your changes missing coverage. Please review.
Project coverage is 84.31%. Comparing base (
60c8f94
) to head (698843e
). Report is 1 commits behind head on master.
Files | Patch % | Lines |
---|---|---|
dpgen2/entrypoint/submit.py | 0.00% | 7 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Summary by CodeRabbit
New Features
Bug Fixes
None
values intraj_results
caused errors by filtering them out before path resolution.