Sprint Planning Feb 2023

[KoalaGeo]

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

Priority

Upgrades

• [x] Update Python library to v0.4.1
  • [x] Address - https://github.com/BritishGeologicalSurvey/pyagsapi/issues/55
• [x] https://github.com/BritishGeologicalSurvey/pyagsapi/issues/60 (for .ags to .xlsx only)
• [x] Allow validation of .AGS files
• [x] HTML popup default
  • [x] https://github.com/BritishGeologicalSurvey/pyagsapi/issues/56 should flag file size before attempted upload

Bugs

• [x] https://github.com/BritishGeologicalSurvey/pyagsapi/issues/57
  • test files https://github.com/BritishGeologicalSurvey/pyagsapi/wiki/File-Summaries
  • East West railways is huge and takes a long time to parse
• [x] https://github.com/BritishGeologicalSurvey/pyagsapi/issues/62
• [x] https://github.com/BritishGeologicalSurvey/pyagsapi/issues/54
  • Test File - https://github.com/BritishGeologicalSurvey/pyagsapi/commit/5eebc8f2db45350fda9952402df89ddc546f607a
  • SUCT group needs the same rule applied
• [ ] https://github.com/BritishGeologicalSurvey/pyagsapi/issues/58
• [ ] https://github.com/BritishGeologicalSurvey/pyagsapi/issues/53 Run validator before .xlsx to .ags
• [ ] https://github.com/BritishGeologicalSurvey/pyagsapi/issues/71

Outstanding

From - https://github.com/BritishGeologicalSurvey/pyagsapi/issues/30

• [x] Fix UnboundLocalError bug in upstream repo (https://gitlab.com/ags-data-format-wg/ags-python-library/-/issues/66)
• [x] 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
• [ ] https://github.com/BritishGeologicalSurvey/pyagsapi/issues/49
• [ ] https://github.com/BritishGeologicalSurvey/pyagsapi/issues/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