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 initialize_build to SETO System Costing #60

Open
adam-a-a opened this issue Jul 15, 2023 · 3 comments
Open

Add initialize_build to SETO System Costing #60

adam-a-a opened this issue Jul 15, 2023 · 3 comments
Labels
bug Something isn't working enhancement New feature or request

Comments

@adam-a-a
Copy link
Contributor

adam-a-a commented Jul 15, 2023

Running initialize on the system costing block will result in an error (first one I saw was related to absence of aggregate_capital_cost when I tried to add in initialize_build quickly.)

We should revisit this when time permits.

@adam-a-a adam-a-a added bug Something isn't working enhancement New feature or request labels Jul 15, 2023
@kurbansitterley
Copy link
Contributor

wondering if this was de facto addressed after updating to watertap 0.11 since there is now an initialization routine for costing?

@kurbansitterley
Copy link
Contributor

@adam-a-a if we are using watertap 1.0 and inheriting costing packages, can this be closed? #115

@kurbansitterley
Copy link
Contributor

Running initialize on the system costing block will result in an error (first one I saw was related to absence of aggregate_capital_cost when I tried to add in initialize_build quickly.)

We should revisit this when time permits.

Ok just actually tried this and was able to duplicate.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants