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

bdew heat profile shlp -type #126

Closed
simnh opened this Issue Mar 30, 2016 · 6 comments

Comments

Projects
None yet
2 participants
@simnh
Copy link
Member

simnh commented Mar 30, 2016

I tried to run the heatprofile calculation with the bdew equations. Using 'shlp_type'=MFH causes an error.
Can someone of you check @uvchik @birgits ?

Can you reproduce the error?

Traceback (most recent call last):
  File "<stdin>", line 8, in <module>
  File "/home/simon/znes/projects/oemof/oemof_base/oemof/demandlib/bdew_heatprofile.py", line 229, in create_bdew_profile
    KW = annual_heat_demand / (sum(h * F) / 24)
  File "/usr/local/lib/python3.4/dist-packages/pandas/core/ops.py", line 618, in wrapper
    return left._constructor(wrap_results(na_op(lvalues, rvalues)),
  File "/usr/local/lib/python3.4/dist-packages/pandas/core/ops.py", line 553, in na_op
    raise_on_error=True, **eval_kwargs)
  File "/usr/local/lib/python3.4/dist-packages/pandas/computation/expressions.py", line 218, in evaluate
    **eval_kwargs)
  File "/usr/local/lib/python3.4/dist-packages/pandas/computation/expressions.py", line 129, in _evaluate_numexpr
    result = _evaluate_standard(op, op_str, a, b, raise_on_error)
  File "/usr/local/lib/python3.4/dist-packages/pandas/computation/expressions.py", line 71, in _evaluate_standard
    return op(a, b)
ValueError: operands could not be broadcast together with shapes (8760,) (8767,) 

@simnh simnh added the bug label Mar 30, 2016

@simnh simnh added this to the March 2016 release milestone Mar 30, 2016

@simnh

This comment has been minimized.

Copy link
Member Author

simnh commented Mar 30, 2016

Maybe it's located inside the provided csv-files?

@uvchik

This comment has been minimized.

Copy link
Member

uvchik commented Mar 31, 2016

I need the call to reproduce the error.

@simnh

This comment has been minimized.

Copy link
Member Author

simnh commented Mar 31, 2016

can you just try to call the heat_demand_example.py in the development examples

replace 'EFH' with 'MFH' will cause an error....

efh.heat_load = efh.hourly_heat_demand(fun=bdew_heat.create_bdew_profile,
                                       datapath="../../oemof/demandlib/bdew_data",
                                       year=2010, holidays=holidays,
                                       temperature=temperature,
                                       shlp_type='EFH',
                                       building_class=1,
                                       wind_class=1,
                                       annual_heat_demand=25000)
@uvchik

This comment has been minimized.

Copy link
Member

uvchik commented Apr 1, 2016

Fixed with commit b4e8680 and merged into dev. I think it was fast enough to be part of the March release.

@uvchik uvchik closed this Apr 1, 2016

@uvchik uvchik reopened this Apr 1, 2016

@uvchik uvchik assigned simnh and unassigned uvchik Apr 1, 2016

@uvchik

This comment has been minimized.

Copy link
Member

uvchik commented Apr 1, 2016

Sorry, I think the workflow is, that I have to reassign it to you (@simnh) and you can close the issue if everything works for you.

@simnh

This comment has been minimized.

Copy link
Member Author

simnh commented Apr 1, 2016

ok works for me! (i don't get an error)..
Thx!!!

@simnh simnh closed this Apr 1, 2016

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