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

RP dev #109

Merged
merged 7 commits into from
Apr 21, 2021
Merged

RP dev #109

merged 7 commits into from
Apr 21, 2021

Conversation

pernak18
Copy link
Collaborator

quick commit to address #103

@RobertPincus
Copy link
Member

@pernak18 Can you please change the target of this PR from master to develop?

@pernak18 pernak18 changed the base branch from master to develop April 12, 2021 15:28
@pernak18
Copy link
Collaborator Author

@RobertPincus done. sorry -- i misunderstood the original directions.

@RobertPincus
Copy link
Member

Hi @pernak18 So the CI is failing because the encoding of the variables differs between LW and SW. In the LW they are char but in the SW they are strings which I guess is how xarray writes them.

[Mini-Me-2:~/Codes/rte-rrtmgp] robert% ncdump -h rrtmgp/data/rrtmgp-data-lw-g256-2018-12-04.nc | grep string_len
	string_len = 32 ;
	char gas_minor(minor_absorber, string_len) ;
	char gas_names(absorber, string_len) ;
	char identifier_minor(minor_absorber, string_len) ;
	char minor_gases_lower(minor_absorber_intervals_lower, string_len) ;
	char minor_gases_upper(minor_absorber_intervals_upper, string_len) ;
	char scaling_gas_lower(minor_absorber_intervals_lower, string_len) ;
	char scaling_gas_upper(minor_absorber_intervals_upper, string_len) ;
[Mini-Me-2:~/Codes/rte-rrtmgp] robert% ncdump -h rrtmgp/data/rrtmgp-data-sw-g224-2018-12-04.nc | grep string_len
	string_len = 32 ;
	string gas_minor(minor_absorber, string_len) ;
	string gas_names(absorber, string_len) ;
	string identifier_minor(minor_absorber, string_len) ;
	string minor_gases_lower(minor_absorber_intervals_lower, string_len) ;
	string minor_gases_upper(minor_absorber_intervals_upper, string_len) ;
	string scaling_gas_lower(minor_absorber_intervals_lower, string_len) ;
	string scaling_gas_upper(minor_absorber_intervals_upper, string_len) ;

@RobertPincus
Copy link
Member

I think we should provide files that are in every way except data identical in the LW and the SW

@pernak18
Copy link
Collaborator Author

damnit. yes, my intent was identical LW and SW. the link i provided in the issue does warn against this behavior, and i didn't double check the array type. i was so focused on string_len and this phantom string1 that xarray.Dataset.to_netcdf kept putting in for character arrays. i'll look into it more when i have some time

@RobertPincus
Copy link
Member

@pernak18 Thanks for making any progress at all, I'm grateful. I used a lower level (netCDF4) interface some years ago - maybe there's something helpful here? https://github.com/RobertPincus/RFMIP-IRF-Scripts/blob/master/generate-output-file-templates.py

RobertPincus and others added 4 commits April 14, 2021 23:22
…nges, updates to continuous integration (#110)

OpenMP GPU offload, tested with  CCE 11.0.0. Longwave Jacobians aren't computed if the optional arguments aren't provided. Finalization methods for optical properties arrays are introduced. Make is streamlined to use environment variables and omit config files. Small changes for efficiency and robustness. Continuous integration is evolved; includes gfortran 9 and 10 on Github actions; current Nvidia and Intel HPC compilers in containers; Cray CPU and OpenMP GPU and PGI 20 OpenACC. 

Co-authored-by: Nichols Romero <naromero@anl.gov> 
Co-authored-by: Dustin Swales <Dustin.Swales@noaa.gov> 
Co-authored-by: Peter Ukkonen <peterukk@gmail.com>
Co-authored-by: Chiel van Heerwaarden <chiel.vanheerwaarden@wur.nl>
…tring_len` dimension. attributes are also saved
@pernak18
Copy link
Collaborator Author

@RobertPincus how about now?

@RobertPincus
Copy link
Member

Thanks kindly, @pernak18. This looks great. Can you address the file format issue in #103 in this same PR?

@pernak18
Copy link
Collaborator Author

@RobertPincus how about now

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.

None yet

2 participants