Sprint Planning Feb 2023

[KoalaGeo]

Before an official launch there are some updates and bug fixes to be complete with pyagsapi

Priority

Upgrades

• Update Python library to v0.4.1
  • Address - BGS Identified Bugs - DataTypes - dates & strings #55
• Implement excel sheet order parameter #60 (for .ags to .xlsx only)
• Allow validation of .AGS files
• HTML popup default
  • 413 Request Entity Too Large #56 should flag file size before attempted upload

Bugs

• internal server error response when doing BGS checks & f=JSON #57
  • test files https://github.com/BritishGeologicalSurvey/pyagsapi/wiki/File-Summaries
  • East West railways is huge and takes a long time to parse
• Dictionary response "" if not specified in file #62
• SAMP_ID validation error TRIT & SUCT groups #54
  • Test File - 5eebc8f
  • SUCT group needs the same rule applied
• Can't select both AGS & BGS checks via Swagger UI #58
• Allows creation of invalid .ags from .xlsx files  #53 Run validator before .xlsx to .ags
• Handle all problem Points #71

Outstanding

From - #30

• Fix UnboundLocalError bug in upstream repo (https://gitlab.com/ags-data-format-wg/ags-python-library/-/issues/66)
• Make logging messages from bgs.py and ags.py print to console
• Add endpoint to automatically clean files - dataclean

Stretch

• Accept URL's like http://uk-ac-bgs-ags-exporter-1006-develop.kube-idev.bgslcdevops.test/?loca_ids=580315 (BGS AGS Exporter API) - NOT currently possible as the API only accepts POST. GET to be added in future
• Add upload .zip function #49
• Map in HTML data validation response #32
• OGCAPI - Processes alignment?

[volcan01010]

Notes:

The underlying library has changed the default dictionary. One set of test-fixes was to update the value that we expected. Also, the library returns the dictionary that was used, either if the default was taken or it read from the TRANS_AGS field in the file. Our fixtures were updated to account for this.

Another set of changes from the underlying library was that messages are prefixed with AGS Format Rule XX.

[volcan01010]

The underlying library now takes .AGS as well as .ags