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

RVD conversion type calculations very inefficient #19

Open
mhallerud opened this issue Feb 27, 2020 · 3 comments
Open

RVD conversion type calculations very inefficient #19

mhallerud opened this issue Feb 27, 2020 · 3 comments
Labels
enhancement A feature request or improvement to model.

Comments

@mhallerud
Copy link
Contributor

The calculations for RVD's conv_code field seems very inefficient, primarily because it creates a bunch of rasters and calculates zonal statistics on those rasters individually. In the future, restructuring this process (perhaps by using "select" to create temporary layers with each value instead of reclassifying a bunch of rasters?) would be an easy way to make the code much more efficient. Currently the outputs are correct, but the process to create this single field seems very clunky and indirect.

@mhallerud mhallerud added the enhancement A feature request or improvement to model. label Feb 27, 2020
@wally-mac
Copy link

@mhallerud thank you for this assessment. Can you please provide a rough time estimate on what you think it would take to restructure this process using selections rather than reclassifications?

@mhallerud
Copy link
Contributor Author

@wally-mac I think this would take 10-15 hours. Another needed improvement is the XMLs. My guess is this would take closer to a full week to get these updated and running well. Currently I have every tool returning the following message before starting the XML write function:

""Writing XML file. NOTE: This is the final step and non-critical to the outputs"

@wally-mac
Copy link

@tyler1218hatch and @DylanBaum we need to check and see if Maggie dealt with this, I doubt that she did so we'll need to make these performance improvements.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement A feature request or improvement to model.
Projects
None yet
Development

No branches or pull requests

2 participants