Improve errors and remove checks on ncviewjs:-prefixed attributes

[katamartin]

This PR makes several changes to make rendered errors more applicable to issues that different datasets make encounter. In the process, it also removes checks on ncviewjs:-prefixed attributes, which means that this change is not compatible with what is currently on main in https://github.com/carbonplan/leap-data-catalog/.

• Improve 2D variable detection (0e37d89)
• Consolidate general variable checks with cfAxes checks (2a78b17)
  • In order to do this, it was necessary to remove checks against ncviewjs:-prefixed attributes (c206859)
• Add tests for metadata utility functions

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
data-viewer	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 30, 2024 6:01pm

[katamartin]

Documenting Slack discussion: addressing changes to functions that were moved in this PR in separate issue (#40).

[andersy005]

In the process, it also removes checks on ncviewjs:-prefixed attributes, which means that this change is not compatible with what is currently on main in carbonplan/leap-data-catalog.

• i believe merging Merge staging into main leap-data-catalog#39 is enough for ncview-js and leap-data-catalog to be in sync, right?

[katamartin]

@andersy005 yeah! I was thinking we might want to iron out all of the remaining issues in carbonplan/leap-data-catalog#40 before merging these two PRs, but I could be convinced otherwise.