I think directly doing dictionary lookups in PDS3Image would be better than using get_nested_dict. The get_nested_dict was "necessary" to inform the parent class, PlanetaryImage where it could find certain things. I think its just a needless distraction/abstraction at this level.
Also, "necessary" is in quotes above because I suspect there's a better way to accomplish what get_nested_dict is doing. For instance properties that return the appropriate values.
I think directly doing dictionary lookups in
PDS3Image
would be better than usingget_nested_dict
. Theget_nested_dict
was "necessary" to inform the parent class,PlanetaryImage
where it could find certain things. I think its just a needless distraction/abstraction at this level.Also, "necessary" is in quotes above because I suspect there's a better way to accomplish what
get_nested_dict
is doing. For instance properties that return the appropriate values.