Turbo87 / aerofiles

waypoint, task, tracklog readers and writers for aviation
http://aerofiles.readthedocs.org/
MIT License
45 stars 28 forks source link

unknown record type AF #283

Closed janzahradka closed 1 month ago

janzahradka commented 1 month ago

I tried to validaty my airspace file by OpenairReader

I got ValueError: unknown record type AF. This tag is used for storing frequencies of the airspace and it's an addition to the original, as written in http://www.winpilot.com/UsersGuide/UserAirspace.asp

validated airspace definition:

AC Q
AN DROPZONE Breclav 119.655
AF 119.655
AG Breclav RADIO
AH FL 95
AL 0 AGL
V X=48:47:27 N 016:53:33 E
DC 2 *NM

is anyone able to extend the reader with these additional tags?

bubeck commented 1 month ago

You are using the OpenAir Extended File format. Currently aerofiles is unable to handle this. However, I am willing to extend it so that it can deal with it. Will need probably 1-2 weeks.

bubeck commented 1 month ago

There is now a first implementation ready for test. Could you please try?

janzahradka commented 1 month ago

Thank you for such a quick response. I tested it, it can handle AF record now but it raises another error: ValueError: could not convert string to float: '2 *NM' in handle_DC_record(self, value). Perhaps it doesn't ignore the '*NM' comment (I use it each time in DC parameter for preventing units mismatch)

bubeck commented 1 month ago

Yes, I will take care. However, early on monday

janzahradka commented 1 month ago

CZ_all_24-08-07.txt I can provide my file which i am trying to validate. It's quite complex one.

bubeck commented 1 month ago

Parser improved for "*" inside lines, openair/reader can now validate files given on command line. Above example sucessfully validated.

bubeck commented 1 month ago

small problem still.

Error in file /home/bubeck/Downloads/CZ_all_24-08-07.txt invalid coordinate form at: 50:16:40 N 013:14:06 E

wait

bubeck commented 1 month ago
Screenshot 2024-08-12 102227

Should be OK now to parse your example file. Please let me know. Looks good, see attached screenshot.

Hint: I think, you should include "FT" into your heights...

janzahradka commented 3 weeks ago

Thank you very much, I was on holiday so I got back to it right now. It works! Also thanks for the hint.