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

Group entities in the EnergySystem class for easier access #60

Closed
uvchik opened this Issue Jan 22, 2016 · 4 comments

Comments

Projects
None yet
3 participants
@uvchik
Copy link
Member

uvchik commented Jan 22, 2016

No description provided.

@cswh

This comment has been minimized.

Copy link
Contributor

cswh commented Mar 3, 2016

is this still valid? Or will we deal with it at the great refactoring? If so, please change the milestone accordingly.

@cswh cswh modified the milestones: March 2016 Release, February 2016 release Mar 3, 2016

gnn added a commit that referenced this issue Mar 31, 2016

Merge branch 'features/#60' into dev
At least partially addresses issue #60.
@gnn

This comment has been minimized.

Copy link
Member

gnn commented Apr 4, 2016

I partially addressed this in the March release, but there is still some work to be done as the implementation is incomplete. Should be completed before the April release though. In the meantime, feel free to post any issues you run into with the current implementation here.

@gnn

This comment has been minimized.

Copy link
Member

gnn commented Apr 28, 2016

So much for completion before the April release. Going to move that to the next release. Most of the functionality is there, but @simnh pointed me at the fact that crucial documentation is missing.

@gnn gnn modified the milestones: May 2016: v0.1.0, April 2016 release Apr 28, 2016

gnn added a commit that referenced this issue May 19, 2016

Merge branch 'dev' into features/#60
This updates the grouping feature branch with upstream changes.

gnn added a commit that referenced this issue Jun 13, 2016

Merge '5e9adf5' (featurs/#60's current HEAD)
... into first-refactoring/solph/use-improved-groupings.
@gnn

This comment has been minimized.

Copy link
Member

gnn commented Nov 1, 2016

Ok. So now this is actually implemented and should be usable. I'm closing this. If anybody complains about missing functionality or lack of documentation, he or she can feel free to open another, more specific issue.

@gnn gnn closed this Nov 1, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment