-
Notifications
You must be signed in to change notification settings - Fork 37
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
template for multi-panel VCS maps #408
Comments
@lee1043 branch created with sample script for producing mean climate makes 408_pgjw_vcs ... JL will likely need PG to change some permissions... |
@gleckler1 want to tag a milestone on this one? |
@durack1 no need ... its being worked on right now |
@doutriaux1 @lee1043 VCS grinding to a halt.... CD your hints yesterday helped a little but our plotting is still blowing up... any hints?? (pmpn) bash-4.1$ python -i mk_clim_dif_maps-driver.py pr ACCESS1-0 djf plotting time is 2.39182496071 |
That line is horrible!!!!!
use imports. I’ll clean this up! There’s so many things worng with this… (hard coded path, not using argparse etc….) C. From: gleckler1 <notifications@github.commailto:notifications@github.com> @doutriaux1https://github.com/doutriaux1 @lee1043https://github.com/lee1043 VCS grinding to a halt.... CD your hints yesterday helped a little but our plotting is still blowing up... any hints?? (pmpn) bash-4.1$ python -i mk_clim_dif_maps-driver.py pr ACCESS1-0 djf plotting time is 2.39182496071 — |
@gleckler1 @doutriaux1 @lee1043 @chaosphere2112 this issue is likely related to UV-CDAT/uvcdat#1424 which last time I checked was still a big problem. Alongside your time getting longer and longer, the memory usage is also like blowing up too |
@durack1 while this might be true, with some clean up for this PR I was able to get a constant speed on pngs. A LOT of unnecessary objects were created. |
@doutriaux1 @zshaheen trying to test CD improvements but can get everything recognized. Starting with a new PMP env, "from pcmdi_metrics.mean_climate_maps import plot_4panel" is not recognized which is fixed by running setup.py with --enable-devel. But then from "pcmdi_metrics.pcmdi import pmp_parser" does not work so I follow ZS instructions ... https://github.com/PCMDI/pcmdi_metrics/wiki/PMPParser and then of course loose mean_climate_maps. I've just blown an hour trying to figure this out while simultaneously Jerome has too trying to get the json class back. Please help us get these installations conflicts resolved. |
@gleckler1 I told you in email you need to add the cdp package:
|
@doutriaux1 can this be added as a conda dependency to avoid problems again in the future? |
yes it is already added. But all conda packages have been built before this. Remember we want to push a new version soon. |
@doutriaux1 yes I think tagging a new release ASAP would be a good idea to close out this installation issue and get things back to a smoother install process |
Until we have a cron job for nightly releases, we might think about adding in a section to the wiki for getting the latest version of pmp. Even if that just entails running |
@lee1043 JW and PG to work on template for producing 4 panel maps with VCS module. Driven by arguments using argparse or just as a function. The top row (two) panels will be for the mod and obs for a given variable, with a color scale centered between and just below the two plots. The bottom row (2 plots) will be for two difference plots (MOD-OBS and MMM-OBS), with a color scale centered between and just below these (lower row) plots. The following argument s will need to be inputs with argparse or as a function:
The text was updated successfully, but these errors were encountered: