Skip to content

Commit

Permalink
fixes to validate bands under mlm:input and their corresponding refer…
Browse files Browse the repository at this point in the history
…ences by other extensions
  • Loading branch information
fmigneault committed Apr 24, 2024
1 parent 91f7796 commit 893b4ff
Show file tree
Hide file tree
Showing 8 changed files with 628 additions and 157 deletions.
18 changes: 12 additions & 6 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,24 +5,30 @@ All notable changes to this project will be documented in this file.
The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).

## [Unreleased](https://github.com/crim-ca/dlm-extension/tree/main)
## [Unreleased](https://github.com/crim-ca/mlm-extension/tree/main)

### Added
- n/a
- Add pattern for `mlm:framework`, needing at least one alphanumeric character,
without leading or trailing non-alphanumeric characters.
- Add [`examples/item_eo_and_raster_bands.json`](examples/item_eo_and_raster_bands.json) demonstrating the original
use case represented by the previous [`examples/item_eo_bands.json`](examples/item_eo_bands.json) contents.

### Changed
- n/a
- Adjust `scikit-learn` and `Hugging Face` framework names to match the format employed by the official documentation.

### Deprecated
- n/a

### Removed
- n/a
- Removed combination of `mlm:input` with `bands: null` that could never occur due to pre-requirement of `type: array`.

### Fixed
- n/a
- Fix `AnyBands` definition and use in the JSON schema to better consider possible use cases with `eo` extension.
- Fix [`examples/item_eo_bands.json`](examples/item_eo_bands.json) that was incorrectly also using `raster` extension.
This is not fundamentally wrong, but it did not allow to validate the `eo` extension use case properly, since
the `raster:bands` reference caused a bypass for the `mlm:input[*].bands` to succeed validation.

## [0.1.1.alpha4](https://github.com/crim-ca/dlm-extension/tree/0.1.1.alpha4)
## [v1.0.0](https://github.com/crim-ca/mlm-extension/tree/v1.0.0)

### Added
- more [Task Enum](README.md#task-enum) tasks
Expand Down
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -169,8 +169,8 @@ to use common names when applicable. Below are a few notable entries.

- `PyTorch`
- `TensorFlow`
- `Scikit-learn`
- `Huggingface`
- `scikit-learn`
- `Hugging Face`
- `Keras`
- `ONNX`
- `rgee`
Expand Down
Loading

0 comments on commit 893b4ff

Please sign in to comment.