Skip to content

Commit

Permalink
Response to review
Browse files Browse the repository at this point in the history
- Variable name suggestions.
- Test speedup.
  • Loading branch information
wxtim committed Aug 28, 2024
1 parent 3102062 commit a87be0a
Show file tree
Hide file tree
Showing 4 changed files with 8 additions and 5 deletions.
2 changes: 1 addition & 1 deletion cylc/flow/config.py
Original file line number Diff line number Diff line change
Expand Up @@ -690,7 +690,7 @@ def process_initial_cycle_point(self) -> None:
Sets:
self.initial_point
self.cfg['scheduling']['initial cycle point']
self.raw_icp
self.evaluated_icp
Raises:
WorkflowConfigError - if it fails to validate
"""
Expand Down
2 changes: 1 addition & 1 deletion cylc/flow/workflow_db_mgr.py
Original file line number Diff line number Diff line change
Expand Up @@ -329,7 +329,7 @@ def put_workflow_params(self, schd: 'Scheduler') -> None:
])

# Retrieve raw initial cycle point stored on the config.
value = getattr(schd.config, 'raw_icp', None)
value = getattr(schd.config, 'evaluated_icp', None)
value = None if value == 'reload' else value
self.put_workflow_params_1(self.KEY_INITIAL_CYCLE_POINT, value)

Expand Down
3 changes: 2 additions & 1 deletion tests/functional/cylc-combination-scripts/09-vr-icp-now.t
Original file line number Diff line number Diff line change
Expand Up @@ -35,4 +35,5 @@ run_ok "${TEST_NAME_BASE}-vip" \
echo "# Some Comment" >> flow.cylc

run_ok "${TEST_NAME_BASE}-vr" \
cylc vr "${WORKFLOW_ID}"
cylc vr "${WORKFLOW_ID}" \
--stop-cycle-point 2020-01-01T00:02Z
Original file line number Diff line number Diff line change
@@ -1,7 +1,9 @@
[scheduling]
initial cycle point = now
initial cycle point = 2020
stop after cycle point = 2020-01-01T00:01Z
[[graph]]
PT1M = foo
[runtime]
[[foo]]
script = cylc stop "${CYLC_WORKFLOW_ID}"
[[[simulation]]]
default run length = PT0S

0 comments on commit a87be0a

Please sign in to comment.