NREL / ComStock

National scale modeling of the U.S. commercial building stock supported by U.S. DOE, LADWP, and others and maintained by NREL
Other
29 stars 7 forks source link

Fix the profiler script for #165 #181

Closed wenyikuang closed 3 months ago

wenyikuang commented 3 months ago

Pull request overview

Pull Request Author

This pull request makes changes to (select all the apply):

Author pull request checklist:

Review Checklist

This will not be exhaustively relevant to every PR.

ComStock Licensing Language - Add to Beginning of Each Code File

# ComStock™, Copyright (c) 2023 Alliance for Sustainable Energy, LLC. All rights reserved.
# See top level LICENSE.txt file for license terms.

Since the structure of the openstido.log has been changed majorly. Updated the profiler script to align with the change inorder to summarize the status of eache measures and running status of the steps.

Tested locally: with this initial entry point:

image

and this yml:

image

And this file structure:

image

Result:

image

The whole proint of the PR is used to fix the

https://github.com/NREL/ComStock/issues/165

Once Eric run the updated script and it works,

TODO:

wenyikuang commented 3 months ago
cspp.utils.hpc.parse_and_generate_profiling(yml_path, selecting_upgrade_ids=["up01"])

Example to filter out the upgrade id you want to analyze.