Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

library build #18

Merged
merged 1 commit into from
Feb 10, 2023
Merged

library build #18

merged 1 commit into from
Feb 10, 2023

Conversation

AAndersn
Copy link
Collaborator

Requires modification to import path into docker by @tonya1 :

volumes:
       - ~/topology-processor:/gridappsd/services/gridappsd-topology-processor
       - ~/topology-processor/topology_processor/gridappsd-topology-service.config:/gridappsd/services/gridappsd-topology-service.config
       - ~/topology-processor/topology_processor/gridappsd-topology-daemon.config:/gridappds/services/gridappsd-topology-daemon.config

can be installed as library using poetry build and pip install

can be imported into other projects using from topology_processor import DistributedTopology

from . distributedtopology import DistributedTopology
from . networkmodel import NetworkModel
from . topologydictionary import TopologyDictionary


Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why commented out remove if we won't be using them.

Also based upon this the path should be from topology_processor.distributedtopology import DistributedTopology

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

# add all structure
__all__: List[str] = ['classname']

to this as well

@AAndersn AAndersn merged commit 1388f16 into develop Feb 10, 2023
@AAndersn AAndersn linked an issue Feb 11, 2023 that may be closed by this pull request
@AAndersn AAndersn deleted the library_0.2.0 branch January 25, 2024 21:50
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

EnergyConsumer objects missing from SwitchArea
2 participants