Luc Peterson

Results 16 comments of Luc Peterson

I think this seems like a good idea, for both merlin and the merlin-spellbook. I think we should welcome pull requests that add type annotations to our functions.

- [ ] I'd like to change the spec shortcut name (and add a few others). What about these names? - $(MERLIN_SPEC_RAW) --> *.orig.yaml - $(MERLIN_SPEC_RUN) --> *.partial.yaml - $(MERLIN_SPEC_EXPANDED)...

Please change PR to go into develop, not main —— J. Luc Peterson, PhD (he/him) LLNL ***@***.*** On Jun 14, 2022, at 4:52 PM, Jim Gaffney ***@***.***> wrote:  ________________________________...

Is there an integration test we can add to make sure it’s working?

@ymubarka can you take a look at this?

What about 🚦for priority?

Docs preview: https://merlin--378.org.readthedocs.build/en/378/

I added a PR to stick onto this branch that includes a few changes to get the docs to build without warning

The user block allows other variables / yaml syntax in the workflow file to be propagated through to the workflow (including in the .parital.yaml and .expanded.yaml). This gives the user...