case.qstatus will not work.

Open Beginner friendly
#4,926 3 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
68/100
Issue type
Bug
Clarity
Clearly specified
Activity status
Quiet
Tech stack
python
Domain
devops

Research direction

Start at CIME/XML/env_batch.py lines 1392-1409 and inspect how self.get_optional_child searches the batch_system entries in config_batch.xml and the machine directory. Verify the case.qstatus path finds a non-None batch_query, then exercise the affected configuration to confirm it works.

Written by the indexing model from the issue text.

Description

Low Priority

https://github.com/ESMCI/cime/blob/66dd00b0af206bc93a3ab83df739cddd33ad1e3a/CIME/XML/env_batch.py#L1392-L1409

Since the root= is not specified in the self.get_optional_child call and "batch_query" is not a child of self in most cases but it's grandchild, it will return None. Since we have at list two children (one in config_batch.xml in ccs_config/machines and another one in the machine directory). The trival fix would be too loop through "batch_system" and take the last "batch_query" that is not None. Should I fire a PR?

Dominant language
Python
Stars
174
Forks
225
Avg merge
2d 30m
Merged PRs (30d)
15

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from ESMCI/cime

All issues in ESMCI/cime

Similar issues

More Python issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.