Closed fmigneault closed 6 months ago
@rbavery FYI
@rbavery Ideally, people will migrate to STAC 1.1 over time, and then bands
would be preferable. However, I don't want this to become a limiting factor in using MLM if for some reason a catalog is stuck on 1.0. For sure, we can increase the emphasis on STAC 1.1 when it's ready.
Description
Properly validate the
bands
reference undermlm:input
to refer to correspondingeo
/raster
/ STAC 1.1 "bands" definition.Added
mlm:framework
, needing at least one alphanumeric character, without leading or trailing non-alphanumeric characters.examples/item_eo_and_raster_bands.json
demonstrating the original use case represented by the previousexamples/item_eo_bands.json
contents.description
field formlm:input
andmlm:output
definitions.Changed
scikit-learn
andHugging Face
framework names to match the format employed by the official documentation.Deprecated
Removed
mlm:input
withbands: null
that could never occur due to pre-requirement oftype: array
.Fixed
AnyBands
definition and use in the JSON schema to better consider possible use cases witheo
extension.examples/item_eo_bands.json
that was incorrectly also usingraster
extension. This is not fundamentally wrong, but it did not allow to validate theeo
extension use case properly, since theraster:bands
reference caused a bypass for themlm:input[*].bands
to succeed validation.Related Issue
n/a
Type of Change
Checklist
CONTRIBUTING.md
guide;make codestyle
;Google
format for all the methods and classes that I used.