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.

Sign up for GitHub

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

Jump to bottom

Remove "PBS -V" from dev driver scripts #464

Closed
malloryprow opened this issue Apr 3, 2024 · 1 comment · Fixed by #525
Closed

Remove "PBS -V" from dev driver scripts #464

malloryprow opened this issue Apr 3, 2024 · 1 comment · Fixed by #525

Comments

@malloryprow
Copy link
Contributor

I was having issues running on Dogwood so I emailed the helpdesk. In their debugging of what was doing on they noted this

We do not recommend using the "-V" flag with qsub since it consumes a large amount of memory for the job data (slowing down server requests when many jobs do this) and can inadvertently impact the job execution, which is what appears to have happened here.

We should remove the "#PBS -V" lines from all the dev driver scripts that have it.
@AliciaBentley-NOAA
Copy link
Contributor

AliciaBentley-NOAA commented Apr 4, 2024 •
edited
Loading

@malloryprow That sounds good to me. We recently (in Perry's PR) removed "#PBS -V" line from NCO's .ecf scripts. If EVS runs fine without "#PBS -V", I think we can safely remove it from all dev drivers.

@PerryShafran-NOAA @ShelleyMelchior-NOAA Thoughts?

malloryprow added a commit to malloryprow/EVS that referenced this issue Apr 29, 2024
@malloryprow
Remove PBS -V from driver scripts
4b79070 
Refs: NOAA-EMC#464
@malloryprow malloryprow mentioned this issue Jul 8, 2024
Merged
18 tasks
PerryShafran-NOAA pushed a commit that referenced this issue Jul 10, 2024
@malloryprow
feature/global_det_v1.0plus (#495)
ec43e0d 
* Add "last" in plots driver script names

* Add "last" in plots ecf script names

* Remove echo of config file

* Change pastXXdays to lastXXdays in wave image names

* Remove LOOP_ORDER from METplus configs

* Remove subprocess import when not used

* Use subprocess.run

* Switch gfs/ecmwf order for headline plot

* Make grid2grid stats job formatting similar to plots

* Make grid2obs stats job formatting similar to plots

* Clean up plots logging messages

Refs: #1

* Mask out negative values under square root

Refs: #1

* Rename stats dev drivers

* Fix cmc_regional atmos order

* Add set -x to missing file mail script

* Define NET prior to HOMEevs

* Remove PBS -V from driver scripts

Refs: #464

* Add 0.25 inch for precip ETS headline plots

* Update default MET version

* Add osisaf COMOUT directory creation

* Add scale specifications for ETS

* Use PB2NC_SKIP_VALID_TIMES

* Remove cat'ing of METplus log files

* Update long-term time series mean to reflect values on plot

* Remove weird space that was causing weird message in log

* Bugfix

* Remove ltrop, utrop, and trop for ozone vertical profile plots

* Format environment variables correctly

* Change warning messages to note/debugs

* Update drivers to use environment VDATE_END
@malloryprow malloryprow mentioned this issue Aug 22, 2024
Merged
9 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feature/rm_PBS_V malloryprow/EVS
2 participants
@malloryprow @AliciaBentley-NOAA