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

Cli #51

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Cli #51

wants to merge 8 commits into from

Conversation

taoning
Copy link
Collaborator

@taoning taoning commented Dec 5, 2023

restructure cli and eplus related functionality to be submodule.
This reduce the over import time and cli start up time since the bulk of import time is from eplus related stuff.
This would introduce slight changes in api when access eplus functionality, i.e., all eplus related api can no longer be
accessed through the frads api but have to access by from frads import eplus
for example
before:

import frads
frads.load_energyplus_model

after

from frads import eplus
eplus.load_energyplus_model

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant