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

Specify that height = AMSL #118

Closed
mrtnrey opened this issue Feb 11, 2021 · 4 comments
Closed

Specify that height = AMSL #118

mrtnrey opened this issue Feb 11, 2021 · 4 comments

Comments

@mrtnrey
Copy link
Collaborator

mrtnrey commented Feb 11, 2021

We should specify somewhere that the height in the density graphs is expressed above mean sea level. For "high" radars (like Wideumont) it will raise questions if we do not add this.

Either

  • in the y-axis label (see figure)
  • in the text just above the density graph.

I prefer the 2nd option: adding AMSL in the axis-label makes it a bit too heavy. And secondly: how do you translate AMSL?

So text to be added:
Eng: The height in the graph refers to the altitude above mean sea level.
Nl: De hoogte in de figuur refereert naar de hoogte boven zeeniveau.
Fr: Nico?

As an option, we could altitude of the radar to the coordinates above the small map
Wideumont (590m) is located at: 49.9135, 5.5044

CROW_AMSL

@mrtnrey mrtnrey changed the title Specifiy that height = AMSL Specify that height = AMSL Feb 11, 2021
@peterdesmet
Copy link
Member

In the text we already say:

This chart shows bird density (colour) as a function of time (x-axis) and height above the ground (y-axis). The BirdTAM colour scale is tailored to aviation.

Rather than a new sentence, we could say:

This chart shows bird density (colour) as a function of time (x-axis) and height above mean sea level (y-axis). The BirdTAM colour scale is tailored to aviation.

It is indeed hoogte boven zeeniveau in NL.

@mrtnrey
Copy link
Collaborator Author

mrtnrey commented Feb 11, 2021

Okay, then this can be changed easily (didn't notice "above the ground", sorry)

@peterdesmet
Copy link
Member

Fixed in c57ad15 (not yet live).

@mrtnrey
Copy link
Collaborator Author

mrtnrey commented Feb 11, 2021

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants