Skip to content

added wflowjl to environment#180

Merged
MarkMelotto merged 1 commit intomainfrom
add-wflowjl
Jul 14, 2025
Merged

added wflowjl to environment#180
MarkMelotto merged 1 commit intomainfrom
add-wflowjl

Conversation

@MarkMelotto
Copy link
Contributor

I think this is all that is needed to add wflowjl to ewatercycle.
Please let me know if there is something else!

@BSchilperoort
Copy link
Member

Did you also add the .sif file to dcache? This is so that ewatercycle users on SRC can access the container.

See the section "Create Apptainer Container" on https://ewatercycle.readthedocs.io/en/latest/infrastructures.html

@BSchilperoort
Copy link
Member

BSchilperoort commented Jul 14, 2025

Note that the instructions there are for snellius, you should be able to do sudo apptainer ... on SRC

@MarkMelotto
Copy link
Contributor Author

@BSchilperoort I am guessing I have to make the ewatercycle-wflowjl-grpc4bmi.sif still.
Since there is a wflowjl-remotebmi:0.2.0.sif already?

@BSchilperoort
Copy link
Member

Since there is a wflowjl-remotebmi:0.2.0.sif already?

Ah I guess I already added the container image. Them you don't need to do anything.

ewatercycle-wflowjl-grpc4bmi.sif still.

No, the docs are from before there was remotebmi as an alternative for grpc4bmi. grp4bmi doesn't work with Julia.

Copy link
Member

@BSchilperoort BSchilperoort left a comment

Choose a reason for hiding this comment

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

lgtm 👍

@MarkMelotto MarkMelotto merged commit b107d7d into main Jul 14, 2025
1 check passed
@MarkMelotto MarkMelotto deleted the add-wflowjl branch July 14, 2025 11:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants