Helpful declarative `__setattr__` and trait validation

[dcamron]

As it was ripped out of #1903 for now, it might be helpful to users to

• Expand validation and type-checking where applicable for declarative traits
• Craft a non-intrusive MetPyHasTraits.__setattr__ to provide errors for users attempting to set traits that don't exist (e.g. misspellings)
• Document/exemplify these and additions from Region modifiers with '+' or '-' suffix #1907, Clean up declarative dir and tab-completion #1903

[sgdecker]

Does this issue address the problem described in this notebook?

https://gist.github.com/sgdecker/8c90f2300da03de296e1d934b47e1cd1

[dcamron]

Does this issue address the problem described in this notebook?

https://gist.github.com/sgdecker/8c90f2300da03de296e1d934b47e1cd1

Accomplishing these tasks should ideally a.) tell you that 'COUNTRIES' is invalid, and b.) tell you so when you attempt to set the trait as opposed to when the figure is drawn. So, "partial yes".

I don't know that this will address the plot parent issue you're having in that notebook. In the mean time it does look like re-creating the PlotObs in the cell above (re-run cells 4-5 instead of just 5) can fix this for now, a full restart/run all not necessary. I'll see if I can boil down where this is happening and open a new issue for it, unless you already have a small reproducible code snippet and want to do that yourself!

[sgdecker]

That notebook is the smallest snippet I have, so by all means, go ahead and open a new issue. Thanks!

[dopplershift]

Here's a minimal reproducer:

from datetime import datetime
from metpy.cbook import get_test_data
from metpy.io import parse_metar_file
from metpy.plots.declarative import PlotObs, MapPanel, PanelContainer

sfc_obs = parse_metar_file(get_test_data('2020010600_sao.wmo', False))

obs = PlotObs()
obs.data = sfc_obs
obs.time = datetime.utcnow()

panel = MapPanel()
panel.projection = 'lcc'
panel.layers = ['countries','states']
panel.plots = [obs]

try:
    pc = PanelContainer()
    pc.panels = [panel]
    pc.show()
except Exception:
    pass

panel = MapPanel()
panel.projection = 'lcc'
panel.plots = [obs]

The call to show() is critical to getting the failure. I'm sure it's some kind of imperfect state management going on here...

[sgdecker]

Let me know if this is worth a new issue, and I realize the existing inconsistencies are just holdovers from the underlying Cartopy feature names, but a list like
panel.layers = ['coastline', 'borders', 'states']
is maddeningly inconsistent.

Why is coastline singular? borders is vague; why isn't it countries?

My request would be to add aliases ('coastlines' = 'coastline', 'countries' = 'borders', and perhaps others).

[dopplershift]

@sgdecker Can you open that as a new feature request? It's reasonable to do for the declarative interface.