Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Surface forcing file paths contain are setup for payu users #93

Open
ashjbarnes opened this issue Jan 24, 2024 · 3 comments
Open

Surface forcing file paths contain are setup for payu users #93

ashjbarnes opened this issue Jan 24, 2024 · 3 comments

Comments

@ashjbarnes
Copy link
Collaborator

In the ERA5 example, the paths to each folder begin with INPUT as though the user were using payu. Better to just incorporate the inputdir that users provide when setting up the experiment.

Originally posted by @ashjbarnes in #79 (comment)

@navidcy
Copy link
Contributor

navidcy commented Apr 8, 2024

yeap, let's do that or have payu as dependency of the package and explain what payu is briefly in the docs?

@navidcy navidcy mentioned this issue Apr 8, 2024
@navidcy
Copy link
Contributor

navidcy commented Apr 22, 2024

@ashjbarnes, is this still the case?
I noticed that in the ERA5 example, use_payu is set to False (by default)

@ashjbarnes
Copy link
Collaborator Author

Yeah the use_payu is only turned on for the ACCESS example. To do this we'll just need a bit of logic to leave things as INPUT/ when payu is on, and inputdir when it's off

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

No branches or pull requests

2 participants