aiidalab / aiidalab/aiidalab-qe

Consider excluding `options` from `CodeModel.get_model_state()`

Đang mở
#1,103 0 bình luận 0 reaction 1 người được giao Được @edan-bainglass nhận Xem trên GitHub
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á.

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.