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

Irradiance, irradiation, power density and energy density #597

Closed
5 tasks
l-emele opened this issue Nov 20, 2020 · 4 comments · Fixed by #615
Closed
5 tasks

Irradiance, irradiation, power density and energy density #597

l-emele opened this issue Nov 20, 2020 · 4 comments · Fixed by #615
Assignees
Labels
[A] new term Including new term(s) in the ontology oeo-physical changes the oeo-physical module

Comments

@l-emele
Copy link
Contributor

l-emele commented Nov 20, 2020

Description of the issue

The discussion in #362 has shown that irradiance and irradiation are useful concepts for our ontology

Ideas of solution

From discussion #362 :

  • Irradiance is an instantaneous value and describes a power (Leistung) of solar rays on a defined area. The unit is usualy W/m².
  • Irradiation is an enery, so power integrated over time. The unit are Wh/m². Irradiation defenitely needs a time step (see timestep, timehorizon, timeseries need new place and def #267) over which it has been recorded.

Originally posted by @carstenhoyerklick in #362 (comment)

Irradiance seams to be a specific kind of power density and irradiation an specific kind of energy density. So we should define the concepts of power density and energy density first and then use these definitions to define the concepts of irradiance and irradiation

Workflow checklist

  • I discussed the issue with someone else than me before working on a solution
  • I already read the latest version of the workflow for this repository
  • The goal of this ontology is clear to me

I am aware that

  • every entry in the ontology should have a definition
  • classes should arise from concepts rather than from words
@l-emele l-emele added [A] new term Including new term(s) in the ontology oeo-physical changes the oeo-physical module To do Issues that haven't got discussed yet labels Nov 20, 2020
@stap-m stap-m added this to the oeo-release-1.4.0 milestone Nov 23, 2020
@l-emele
Copy link
Contributor Author

l-emele commented Nov 25, 2020

Some proposals:

Upper classes, I intentionally left the labels open:

  • A is a quantity value that measures the power per area.
    has unit C
  • D is a quantity value that measures the energy per area.
    has unit Joule per square metre

Those upper classes need

  • C is unit that measures the power per surface area.
  • `D is a unit that measures the energy per surface area.

Using the upper classes we can define:

  • E is an A that measures the arriving solar power per area.
  • F is an B that measures the arriving solar energy per area. F is usually measured or calculated for a time step.

I am unsure about the labels, but I have some suggestions:

  • A and B could be labelled power density and energy density. However, these could be mixed up with their volumetric counterparts. So a better approach might be to label these areal power density resp. areal energy density and add power density and energy density only as alternative terms. With that second approach we are open to define e.g. a volumetric energy density later. Then both the areal energy density and the volumetric energy density would have the same alternative terms energy density showing that this term is ambiguous.
  • C and D: following my preferences for A and B I would label these as areal power density unit and areal energy density unit.
  • E and F: One option would be to use irradiance and irradiation. A second option would be to use something more systematic like areal solar power density and areal solar energy density. A combination is also possible by using one set of terms as labels and the second set as alternative terms. If so, I would prefer using (as they are more systematic) areal solar xyz density for the labels and using irradiance and irradiation for the the alternative terms. But I do not have strong feelings against doing it the other way round.

@stap-m and @carstenhoyerklick : What do you think?

@carstenhoyerklick
Copy link
Contributor

carstenhoyerklick commented Nov 26, 2020 via email

@l-emele
Copy link
Contributor Author

l-emele commented Nov 27, 2020

[CHK] This probably should be B?
Yes, sorry for that typo.

I think we have an agreement, as @stap-m gave some thumbs up.

@carstenhoyerklick : Do you want to try to implement this issue? Else I can do that.

@github-actions github-actions bot removed the To do Issues that haven't got discussed yet label Nov 27, 2020
@l-emele
Copy link
Contributor Author

l-emele commented Dec 2, 2020

No reaction, so I'll implement now myself.

l-emele added a commit that referenced this issue Dec 2, 2020
l-emele added a commit that referenced this issue Dec 4, 2020
l-emele added a commit that referenced this issue Dec 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[A] new term Including new term(s) in the ontology oeo-physical changes the oeo-physical module
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants