Refactor interpolate_from_file API

[alperaltuntas]

Currently, creating a topo object by interpolating from a file looks like:

topo = Topo(
    grid = grid,
    min_depth = 9.5,
)

topo.interpolate_from_file(
    file_path = "...",
    longitude_coordinate_name="lonh",
    latitude_coordinate_name="lath",
    vertical_coordinate_name="depth"
)

I think we should instead have something like:

topo = Topo.interpolate_from_file(
    grid = grid,
    min_depth = 9.5,
    file_path = "...",
    longitude_coordinate_name="lonh",
    latitude_coordinate_name="lath",
    vertical_coordinate_name="depth"
)

i.e., interpolate_from_file should be a class method instead, similar to the from_topo_file method.

@manishvenu what do you think?

[alperaltuntas]

Just realized interpolate_from_file is implemented in CrocoDash, and not mom6_bathy, because there is a regional_mom6 dependency. With the incroduction of the mapping module, that dependency may not be required moving forward.

[manishvenu]

Yeah, there is that regional mom6 dependency. if we move it over to mom6_bathy it'd also allow us to mess around with the interpolate a bit more. Aidan was planning on working on the interpolate_from_file to reorganize it a bit in the next couple of weeks, so it might be worth touching base on a plan for this