weglide / bugtracker

WeGlide platform bug tracker.
https://www.weglide.org
11 stars 0 forks source link

'File invalid' ? #212

Closed deltafox44 closed 11 months ago

deltafox44 commented 1 year ago

Describe the bug For some flights it shows "File invalid" red badge without any apparent reason; there is a the link on the badge to Approved loggers but without any "error code".

To Reproduce Steps to reproduce the behavior:

  1. Go to 13 august flight for instance image
  2. Click on 'File invalid' No indication of what is invalid. The logger is the same on all my flights in the year, and there is a G record in the file. Is the G-record invalid ? [edit : most probably it was a logger failure - a few flights later I got an error message saying that private key was not valid] See my other flights : There are 5 invalid flights (one for claiming too late, but 4 for 'file invalid'). Invalid flights are scored normally, only the green badge in first column (#) is missing. image

Expected behavior

  1. 'file invalid' or 'IGC invalid' badge if the file does not pass IGC validity check2.

If the file is invalid :

  1. there should be a link on 'file invalid' badge to the error code that explains why it is invalid and help correct for future flights. As there is when clicking on "Flight valid/invalid" - see first screenshot above.

  2. There should be a 'invalid' red badge in the list of flights, first column (#) (see above), instead of just no green badge, with a link or help ballon to explain that the flight is counted in the pilot's totals but not for his rankings since it is invalid

  3. There should be a box with 'Invalid' / 'invalid file' as in this screenshot from this 25 july flight: image

Also, when the flight is valid, but contains 'information' or 'warning' codes, like in this other flight:

  1. there should be an orange 'flight valid' badge, not a green one ; because you cannot know there have been warnings if you don't click on the badge - as in this screenshot: image
flyingjoe commented 1 year ago

Hi @deltafox44,

thanks for bringing that up.

Your file

I manually checked your invalid files, and they really do seem to be invalid (it's mostly a black box for us aswell):

image_2023-01-26_09-48-24

You may want to reach out to your logger manufacturer about this.

Docs

I think you are correct, there were no clear explanations on what "file invalid" may mean. I added explanations to the doc see (what can I do? and checking myself).

I don't really think it makes sense to expand the "File invalid" badge, since we don't know why it is invalid. I think the improved docs should help with this.

Better indication

As it currently stands, there are three validations to pass to be scored for a contest:

  1. "Flight valid": Formal requirements to flight trace need to be passed (Scoring Error Codes
  2. "IGC valid": G-Record validation of IGC-File needs to pass
  3. Formal requirements of certain contest need to be passed (e.g. uploaded to late -> can vary between contests for a single flight)

We are working on improving the error messages, but it's not that easy as every abstraction can bring in more complexity. E.g.: Let's assume we were to do a single "Flight valid" badge, combining all of them:

  1. Green -> Okay great
  2. Orange -> Click on it -> expand to flight valid and igc-valid -> flight valid is orange -> Click on it -> some contests are valid and some are not, due to different upload date requirements

About the list indicator: We actually experimented with the red indicators in the list before releasing the "new" list but decided against it, as it became quite irritating in some cases.

deltafox44 commented 1 year ago

Hi @flyingjoe

Thank you for your answer. Clicking on 'file invalid' badge now redirects to the explanations you added and it's ok. So 'file invalid' means that integrity check (G-Record) failed. I will see with LXNavigation about that.

About the list indicator: It is not obvious at first glance of the list of flights that a green indicator means 'valid flight', thus a flight without green indicator is invalid. It would be clearer if, for instance, there was 'Valid' instead of '#' as first column header.

Still, some indication that there has been 'information' or 'warning' codes would be useful not to miss the warnings (cf my item # 5 above)

moldhouse commented 1 year ago

I agree regarding item 5. Currently, you miss the warnings and they can be useful (e.g. if you have many invalid fixes you know that you should reposition your logger. Maybe show it in yellow? However, a potential downside is that it indicates that something is wrong although your flight is valid. @samuel-git do you have an opinion?

samuel-git commented 1 year ago

In my opinion we should use green if no action is required from the user. I would leave it as is, but maybe indicate if there are warnings with an icon or similar.

samuel-git commented 1 year ago
grafik

Like this (also for valid flights)?

deltafox44 commented 1 year ago

As it currently stands, there are three validations to pass to be scored for a contest:

  1. "Flight valid": Formal requirements to flight trace need to be passed (Scoring Error Codes
  2. "IGC valid": G-Record validation of IGC-File needs to pass
  3. Formal requirements of certain contest need to be passed (e.g. uploaded to late -> can vary between contests for a single flight)

image The second button is quite straightforward since it is only related to file integrity - Green "IGC valid" or red "IGC invalid" with a link to help -> nothing to change here (except perhaps "IGC invalid" instead of "File invalid")

But in my opinion the first button should provide a better indication:

In some cases, at present, there is a green button "Flight Valid", and a score is indicated, although the flight is NOT scored in the contest rankings nor counted in the list totals. This is very confusing. Only the absence of a green number button in the list of flights indicates the flight is not really scored (as shown in my first post above). There should NOT be a green button but a red (or yellow).

See for instance :

deltafox44 commented 11 months ago

Update : it seems OK now 👍

There have been a significant improvement, it's now much clearer since Status are indicated by red/yellow/green dots and clicking on them give explanations - see for instance the same two flight as above : flight 64063 : image

and flight 74569 : image