aiidalab / aiidalab/aiidalab-qe
Consider excluding `options` from `CodeModel.get_model_state()`
- Ngôn ngữ chính
- Python
- Star
- 33
- Fork
- 25
- Merge trung bình
- 6 giờ 7 phút
- Pull request đã merge (30 ngày)
- 8
Mô tả
In the `CodeModel` class:
```python
def get_model_state(self) -> dict:
return {
"options": self.options,
"code": self.selected,
"nodes": self.num_nodes,
"cpus": self.num_cpus,
"ntasks_per_node": self.ntasks_per_node,
"cpus_per_task": self.cpus_per_task,
"max_wallclock_seconds": self.max_wallclock_seconds,
}
```
I added the `options` field (in #946) surely for some reason, but it doesn't make much sense to me now. This issue is a reminder to revisit the code and see if the field is actually needed, and if possible (strongly suggested), remove it.
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
Đánh giá
Issue này chưa được đánh giá.