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

The definition of the turbine subclasses do not fit to the definition of turbine. #299

Closed
5 of 7 tasks
l-emele opened this issue Mar 31, 2020 · 4 comments · Fixed by #300
Closed
5 of 7 tasks

The definition of the turbine subclasses do not fit to the definition of turbine. #299

l-emele opened this issue Mar 31, 2020 · 4 comments · Fixed by #300
Assignees
Labels
[C] definition update Update an ontology definition SzenarienDB

Comments

@l-emele
Copy link
Contributor

l-emele commented Mar 31, 2020

Description of the issue

We defined Turbine:
A turbine is an energy converting device that converts energy from a moving fluid flow into rotational energy.

But the WindTurbine is currently defined as:
A wind turbine is a device that converts the wind's kinetic energy into electrical energy.

This definition of WindTurbine currently describes more, what a WindEnergyConvertingUnit.

Current definition of GasTurbine:
A gas turbine, also called a combustion turbine, is a type of continuous combustion, internal combustion engine.

HydrogenTurbine:
A hydrogen turbine is a turbine fueled with liquid hydrogen.

SteamTurbine:
A steam turbine is a device that extracts thermal energy from pressurized steam and uses it to do mechanical work on a rotating output shaft.

WaterTurbine:
A water turbine is a rotary machine that converts kinetic energy and potential energy of water into mechanical work.

Ideas of solution

Update all subclass definition. Example for a new WindTurbine definition:
A wind turbine is a device that converts the wind's kinetic energy into rotational energy.

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
  • I added this issue to the Project 'Issues'. If suitable, I add it to further Projects.
  • The goal of this ontology is clear to me

I am aware that

  • every entry in the ontology should have an annotation
  • classes should arise from concepts rather than from words
  • class or property names should follow the UpperCamelCase
@l-emele l-emele added SzenarienDB [C] definition update Update an ontology definition labels Mar 31, 2020
@stap-m
Copy link
Contributor

stap-m commented Apr 1, 2020

Absolutely. The new def sounds good.
We could also think about renaming the class to e.g. wind rotor to avoid confusion with the very ambigous term wind tubine.

@stap-m
Copy link
Contributor

stap-m commented Apr 1, 2020

The defs of the other turbine subclasses could be refined aswell...

@l-emele l-emele changed the title The definition of the wind turbine does not fit to the definition of turbine. The definition of the turbine subclasses do not fit to the definition of turbine. Apr 1, 2020
@l-emele
Copy link
Contributor Author

l-emele commented Apr 1, 2020

The defs of the other turbine subclasses could be refined aswell...

You are right, I adapted the issue that it covers all.

@l-emele
Copy link
Contributor Author

l-emele commented Apr 1, 2020

Proposals:

  • GasTurbine: A gas turbine is a turbine that converts chemical energy into rotational energy using a continous internal combustion process. Hence it is also called combustion turbine.
  • HydrogenTurbine: A hydrogen turbine is a gas turbine fueled with hydrogen.
  • SteamTurbine: A steam turbine is a turbine that converts heat from pressurized steam into rotational energy.
  • WaterTurbine: A water turbine is a turbine converting kinetic energy and potential energy of water into rotational energy.
  • WindRotor: A wind rotor (or wind turbine) is a turbine that converts the wind's kinetic energy into rotational energy.

In theory we do not need the part into rotational energy for all of the turbine subclasses as already the parent class has it. But for clarity I would include it in all these subclass definitions.

We could also think about renaming the class to e.g. wind rotor to avoid confusion with the very ambigous term wind tubine.

Your idea is actually very good. If we use WindRotor as primary label we could use WindTurbine as secondary label for both WindRotor and WindEnergyConverter. This then shows that the term WindTurbine is ambiguous.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[C] definition update Update an ontology definition SzenarienDB
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants