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

Add scale factor feature to the REDE technical potential module #76

Closed
EvanRosenliebNREL opened this issue Jan 13, 2021 · 2 comments
Closed
Labels
feature New feature or request

Comments

@EvanRosenliebNREL
Copy link

Why this feature is necessary:
Many of the REDE rasters for the current tool are scaled for performance reasons. Without a scale factor option in the both the h5 creation tool and the technical potential tool, all rasters will need to be rescaled before porting which will increase time to transfer and additionally they will be more costly to store over the long run.

A possible solution is:
Add a feature to the code where we can add a scale factor attribute for a dataset, where it will then be scaled during the analysis portion -- like in other reV/reVX modules

I have considered the following alternatives:
Unscaling the rasters before transfering -- not ideal for reasons listed above.

Charge code
WFED.11084.02.04.07

Urgency / Timeframe
Unfortunately, sort of urgent! We are in cruch time for transferring the REDE tool.

@EvanRosenliebNREL EvanRosenliebNREL added the feature New feature or request label Jan 13, 2021
@EvanRosenliebNREL
Copy link
Author

@MRossol As requested :-)

@MRossol
Copy link
Collaborator

MRossol commented Mar 2, 2021

implemented in #77

@MRossol MRossol closed this as completed Mar 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants