Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AMAZONIA-1 metadata extractor #36

Merged
merged 2 commits into from
Oct 26, 2023
Merged

AMAZONIA-1 metadata extractor #36

merged 2 commits into from
Oct 26, 2023

Conversation

floeschau
Copy link
Collaborator

This change adds the metadata extraction functionality for INPE AMAZONIA-1 images.

IMPORTANT: The class Terradue.Stars.Data.Model.Metadata.Cbers.CbersMetadataExtractor was renamed to Terradue.Stars.Data.Model.Metadata.Inpe.InpeMetadataExtractor and covers both CBERS-4/4A and AMAZONIA-1 satellites.

This is updated in stars-data.json and needs to be updated in stars-data.json of deployed instances of Stars if the default file is not used.

@codecov-commenter
Copy link

Codecov Report

Merging #36 (8fc9ded) into develop (1910404) will decrease coverage by 36.31%.
The diff coverage is 96.80%.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##           develop      #36       +/-   ##
============================================
- Coverage    56.09%   19.78%   -36.31%     
============================================
  Files          237      237               
  Lines        34313    34373       +60     
  Branches      2903     2905        +2     
============================================
- Hits         19248     6801    -12447     
- Misses       15064    27572    +12508     
+ Partials         1        0        -1     
Flag Coverage Δ
unittests 19.78% <96.80%> (-36.31%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
...Stars.Data/Model/Metadata/Inpe/Schemas/Metadata.cs 43.86% <ø> (ø)
...rs.Data/Model/Metadata/Inpe/Schemas/MetadataCsv.cs 90.57% <100.00%> (ø)
....Data/Model/Metadata/Inpe/InpeMetadataExtractor.cs 91.56% <96.72%> (ø)

... and 98 files with indirect coverage changes

Copy link
Contributor

@emmanuelmathot emmanuelmathot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great thx @floeschau

@emmanuelmathot emmanuelmathot merged commit a488f8f into develop Oct 26, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants