Package program for easy use in other projects

SINTEF-Power-system-asset-management / flexible_load_analysis

Flexible Load Analysis (FLA)

GNU General Public License v3.0

3 stars 2 forks source link

Package program for easy use in other projects #3

Closed danielbjerk closed 1 year ago

danielbjerk commented 1 year ago

[x] Make all imports relative
[x] Move old main to scripts
[x] __init__.py-files where relevant
[ ] __all__.py for explicit definition of package interface
[ ] Private/Public functions
[x] Test use in other projects
[ ] Atomize functions where possible
[x] Refactor preprocessing and modelling
[ ] Make all objects interface complete as to dissuade direct access of datatype
- [x] Network
- [ ] Load-points
- [ ] Net

danielbjerk commented 1 year ago

Før evt. ferdigstilling av denne grenen, spes. før inn i main, så bør det vurderes om hvorfor objects/load_points avhenger av preprocessing. Hvorfor ligger ikke prepare_all_load_points i preprocessing som "perform_all_preprocessing"? Weird!!

danielbjerk commented 1 year ago

Det trengs innstramming av bruk av objektene. Akk nu er det flere ganger i koden at objektenes underliggende datastruktur aksesseres direkte, istedenfor via deres hhv moduler. Dette peker mot at modulene ikke er komplette, i den forstand at de støtter alle operasjoner man ønsker. Dette er et problem da man kan bli avhengig av direkteaksessering og dermed svekkes dersom internt format endres

danielbjerk commented 1 year ago

For kompletthets skyld: Har ikke tid til å fullføre modulisering av load points og net, da det krever gjennomgående refactoring. Så kan heller fikse på slike ting når jeg trenger dette i andre prosjekter.