Skip to content
This repository has been archived by the owner on Dec 1, 2023. It is now read-only.

feat: support state logic #39

Merged
merged 31 commits into from
Aug 25, 2023
Merged

feat: support state logic #39

merged 31 commits into from
Aug 25, 2023

Conversation

hollandjg
Copy link
Member

@hollandjg hollandjg commented Aug 25, 2023 •
edited
Loading

Update the workflow tools to work with the State logic.

hollandjg added 26 commits August 23, 2023 17:38
@hollandjg
chore: specify autora-core versions < 4
57063c2
@hollandjg
deps: update dependency on autora-core
716f982
@hollandjg
chore: remove old testing code
c21c298
@hollandjg
docs: add new minimum working example of using functional CLI with cylc
1c049c1
@hollandjg
docs: add new minimum working example of using functional CLI alone
3d7d67c
@hollandjg
chore: update typer arguments
8574693
@hollandjg
chore: update logging messages
6fa6563
@hollandjg
feat: accept empty arguments for input and output paths
febaa64
@hollandjg
refactor: shorten argument names
1328112
@hollandjg
docs: update examples
10263f0
@hollandjg
chore: remove outdated conda example
daa175a
@hollandjg
chore: update arguments in flow.cylc
39d0e6d
@hollandjg
chore: simplify requirements
53b2647
@hollandjg
fix: add first example of arithmetic on cycle number
6eb54ae
@hollandjg
fix: make cycle only depend on initial state for first cycle, and on …
984ea7d 
…theorist for following cycles
@hollandjg
feat: re-introduce the real function call
2a30e5d
@hollandjg
chore: remove excess prints
d0b6f6d
@hollandjg
fix: make directories if missing in script
2b68db6
@hollandjg
chore: remove lib unignore
a939870
@hollandjg
docs: remove old info from README
b7a69cd
@hollandjg
chore: remove unused no_op function
465f7b5
@hollandjg
chore: reintroduce test dependencies
99875d4
@hollandjg
docs: improve docs for cylc-pip
44d207d
@hollandjg
feat: add setup for cylc and slurm
08d3418
@hollandjg
feat: add directives
eae6748
@hollandjg
slurm: replace --time directives with execution time limit option
be298ca
musslick
musslick approved these changes Aug 25, 2023
View reviewed changes
Copy link
Contributor

@musslick musslick left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks awesome!

@hollandjg hollandjg marked this pull request as ready for review August 25, 2023 17:39
@hollandjg hollandjg added this pull request to the merge queue Aug 25, 2023
Merged via the queue into main with commit 184021d Aug 25, 2023
@hollandjg hollandjg deleted the feat/support-state-logic branch August 25, 2023 17:45
@hollandjg hollandjg mentioned this pull request Aug 25, 2023
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@musslick musslick musslick approved these changes

@benwandrew benwandrew Awaiting requested review from benwandrew

@younesStrittmatter younesStrittmatter Awaiting requested review from younesStrittmatter

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@hollandjg @musslick