search

NREL / rex

REsource eXtraction Tool (rex)
https://nrel.github.io/rex
BSD 3-Clause "New" or "Revised" License
19 stars 10 forks source link

Bnb/outputs #131

Closed bnb32 closed 2 years ago

bnb32 commented 2 years ago

Moving reV base output handler to rex. We're using it in the sup3r work and it makes more sense for this to be in rex given its general use case. Also would be nice to avoid a reV dependency.

ppinchuk commented 2 years ago

Also, we can just subclass the outputs handler in the downstream repo's to maintain the same import path and to update the handler version attrs.

Makes sense... I assume we will want to do this in reV then? Should be fairly simple but we will probably have to wait until this PR is merged and the new rex version is released.

bnb32 commented 2 years ago

Yeah that's what I was thinking as well. Just import Outputs from rex and leave all the subclasses in reV alone.