search

noaa-oar-arl / canopy-app

Stand-alone/column canopy codes and parameterizations
MIT License
6 stars 6 forks source link

Feature/nl format #96

Closed drnimbusrain closed 8 months ago

drnimbusrain commented 8 months ago

@zmoon @angehung5 Since we have over 40 NL options and growing, need to organize better. I also added some python/README changes for running global data and an example slurm script to help. Please let me know ASAP what you think so we can update develop going forward.

angehung5 commented 8 months ago

I changed the required cpu time and mem in the slurm script and readme based on my slurm settings on Hopper. Time might be a little off since I set ntask=12, but mem should be accurate (could be less depending on the case but it would be safer to put 60G).

drnimbusrain commented 8 months ago

1:16

@angehung5 OK, thank you. Is this for running both the python global process script and canopy-app for those times?

angehung5 commented 8 months ago

1:16

@angehung5 OK, thank you. Is this for running both the python global process script and canopy-app for those times?

global process script only.

drnimbusrain commented 8 months ago

1:16

@angehung5 OK, thank you. Is this for running both the python global process script and canopy-app for those times?

global process script only.

@angehung5 Right, then the memory suggestion is not correct, and would need to be largely increased for canopy-app too. We need to test run both at same time/job.

drnimbusrain commented 8 months ago

Closing this PR due to updated new PR from @angehung5 built on latest commit.