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

Water Observations tables #381

Merged
merged 30 commits into from
Jan 19, 2025
Merged

Water Observations tables #381

merged 30 commits into from
Jan 19, 2025

Conversation

benji-glitsos-ga
Copy link
Collaborator

@benji-glitsos-ga benji-glitsos-ga commented Jan 15, 2025

  • Improving the bit set table
    • Renamed 'Flagging' column to 'Classification'
    • Used title case for these capitalisations to match DEA Maps.
    • Added plain language descriptions based on Bex's notes.
    • Rearranged columns of table so that Classification is the first column and Bit and Decimal value are next to each other.
    • Converted to list-table code format for ease of editing.
  • Added cross-over table based on Bex's spreadsheet.
    • Added documentation based on Bex's notes.

Preview: https://pr-381-preview.khpreview.dea.ga.gov.au/data/product/dea-water-observations-landsat/?tab=description#technical-information

Copy link

github-actions bot commented Jan 15, 2025

Preview is ready!

View the preview: 🌐 PR Preview 381

You can also view the deploy log or read the documentation.

@benji-glitsos-ga benji-glitsos-ga changed the title Water Observations bitmasking table Water Observations tables Jan 16, 2025
@benji-glitsos-ga
Copy link
Collaborator Author

Caitlin has confirmed that all the values in the 'Combination decimal values' table are correct.

@benji-glitsos-ga benji-glitsos-ga marked this pull request as ready for review January 16, 2025 22:54
@BexDunn BexDunn self-assigned this Jan 19, 2025
@BexDunn
Copy link
Contributor

BexDunn commented Jan 19, 2025

Looks good to me Benji

Copy link
Contributor

@BexDunn BexDunn left a comment

Choose a reason for hiding this comment

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

Looks good team

@benji-glitsos-ga benji-glitsos-ga merged commit aafe0d2 into main Jan 19, 2025
1 check passed
@benji-glitsos-ga benji-glitsos-ga deleted the feature/wo-wofs-table branch January 19, 2025 23:35
@robbibt
Copy link
Member

robbibt commented Jan 19, 2025

@BexDunn Thoughts on whether we might want to simplify the table to remove these bespoke double-flag options? My concern is that they make things more complicated than need be - they kind of infer the presence of a "Cloud + water" flag where none exists. It might also add confusion as to why certain other useful combinations (e.g. cloud + shadow) aren't included in the table.

Alternatively, we could just include the main 8 bits (cloud, shadow etc) on the table, and then discuss even more complicated options in the paragraph above it.

image

@BexDunn
Copy link
Contributor

BexDunn commented Jan 19, 2025

Hey @robbibt

  • The absence of Cloud+Shadow is either due to me missing that combination or I may have thought that the two were mutually exclusive bits for some reason. Are any other combinations missing?
  • The original inspiration was the colour map that is used on DEA Maps, which maybe we should also add into this information somehow. The colour map has more options than the original 8 bits in the data where decimal values correspond to multiple bitflags active:

image
image

  • The original purpose of the table was to work out all the mathematically possible values (not necessarily all the values actually physically possible) to help diagnose an issue that a stakeholder had where they had values of 84 present in the data. I started with the colour map, but as the combination bitflag values weren't explained or quantified anywhere, I needed to write the table to understand what was happening. I think the table could be useful, but we could split it into two if it isn't helpful, or line it up against the colour map somehow to make the explanation clearer.

@benji-glitsos-ga
Copy link
Collaborator Author

benji-glitsos-ga commented Jan 20, 2025

Now that I look at the colour map, I realise I have more work to do in making this easy to read:

  • Terms like 'Cloudy Water' really mean 'Cloud + Water', so I should represent that somehow.
  • I should use the colour coding on the table to make it easier to use

I'll open a new PR soon to improve it.

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