Skip to content

Fix regional platform#154

Open
ocefpaf wants to merge 5 commits intoioos:mainfrom
ocefpaf:fix_regional_platform
Open

Fix regional platform#154
ocefpaf wants to merge 5 commits intoioos:mainfrom
ocefpaf:fix_regional_platform

Conversation

@ocefpaf
Copy link
Copy Markdown
Member

@ocefpaf ocefpaf commented Feb 27, 2026

We don't have snapshots to update this for previous years, right?

xref.: #153 (comment)

@MathewBiddle
Copy link
Copy Markdown
Contributor

We do! Source data comes from https://github.com/ioos/ioos-asset-inventory which I serve on ERDDAP at

https://erddap.ioos.us/erddap/tabledap/processed_asset_inventory.html
and
https://erddap.ioos.us/erddap/tabledap/raw_asset_inventory.html

@ocefpaf ocefpaf force-pushed the fix_regional_platform branch 2 times, most recently from 5924fba to 388b807 Compare February 27, 2026 15:41
@ocefpaf
Copy link
Copy Markdown
Member Author

ocefpaf commented Feb 27, 2026

@MathewBiddle this columns is missing from the mbon_stats:

- missing
+ literature_discovered

@MathewBiddle
Copy link
Copy Markdown
Contributor

Well, that's a bummer. I wonder if the OBIS API changed? We will have to do some sleuthing.

Comment thread ioos_metrics/ioos_metrics.py
@ocefpaf
Copy link
Copy Markdown
Member Author

ocefpaf commented Mar 13, 2026

What should we do for #154 (comment) ? Drop that column? Is it useful in the context here?

@MathewBiddle
Copy link
Copy Markdown
Contributor

ohh, I don't have time to dig into that issue. literature_discovered is a really useful metric. Maybe we merge this in and raise an issue that it's broken?

@ocefpaf
Copy link
Copy Markdown
Member Author

ocefpaf commented Mar 13, 2026

We are not using that column in any metrics. In fact, the mbon_stats function returns a DataFrame and that is it. I guess that, if we merge this with the failing, and open an issue to track this, we can address it later.

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.

2 participants