adding year to asset inventory webpage #86
Conversation
|
@ocefpaf mind taking a look at this? |
| @@ -192,6 +193,7 @@ def main(org_config): | |||
| configs = { | |||
| "table": gdf.to_html(table_id="table", index=False, columns=columns), | |||
| "figure": fig, | |||
| "year": gdf['Year'].unique()[0] | |||
There was a problem hiding this comment.
You are getting the first occurrence of Year, so you probably don't need the .unique() there, right? I don't know what that gdf is, I need to run this script first, but if the Year value is the same everywhere and/or what you need is just the first occurrence, this is good to go.
There was a problem hiding this comment.
ah, yes. I was just pulling the string out of the list so it was easier to show on the website.
There was a problem hiding this comment.
BTW, I don't have enough rights to merge this but it is good IMO.
Also, I sent #87 to help with reviews on the page generation in the future.
|
@MathewBiddle is https://erddap.ioos.us/erddap down? I'm trying to test the script locally but I cannot reach it. |
|
should be back up now. Don't know what went wrong. https://erddap.ioos.us/erddap/index.html |
Shouldn't you be on holiday today? All these issues can wait. (And sorry for the pings today.) |
The asset inventory webpage is only showing the inventory for the most recent year. This PR adds the calendar year the webpage is presenting to the descriptive text of the page.